Fix: Skip Terraform tests in publish workflow (#193)

* fix: skip terraform tests in publish workflow

Terraform tests require Terraform CLI to be installed, which is not
available in the publish workflow environment. These tests are already
run in the CI workflow and don't need to be re-run during publishing.

This change adds --ignore=tests/terraform to the pytest command to
skip terraform-specific tests while still running all other package tests.

* fix: revert to -service suffix for backwards compatibility

- Package names: lablink-allocator-service, lablink-client-service
- Update workflow tags to match: lablink-allocator-service_v*, lablink-client-service_v*
- Update Dockerfiles to install correct package names from PyPI
- Remove old directory structure (lablink-allocator, lablink-client-base, terraform)
- Skip terraform tests in publish workflow (--ignore=tests/terraform)
- Maintains backwards compatibility with existing PyPI packages

* refactor: rename module directories to match -service package names

- Rename src/lablink_allocator/ → src/lablink_allocator_service/
- Rename src/lablink_client/ → src/lablink_client_service/
- Update all imports in source files
- Maintains backwards compatibility with PyPI package names

* fix: update pyproject.toml entry points for renamed modules

- Update lablink-allocator entry point to lablink_allocator_service.main
- Update lablink-client scripts to lablink_client_service modules
- Update package data paths to match new directory names
- Maintains package names as lablink-allocator-service and lablink-client-service

* test: update all test imports for renamed modules

- Update monkeypatch setattr strings to lablink_allocator_service
- Update @patch decorators to lablink_client_service
- Update all test file imports to use _service suffix
- All 139 tests passing (117 allocator + 22 client)

* chore: update uv.lock files after module rename

* fix: update CI workflow imports for renamed modules

- Update Docker Build Test to import from lablink_allocator_service
- Update test coverage path to lablink_client_service
- Fixes failing Docker Build Test - Allocator CI check

* fix: update console script entry point imports in CI workflow

- Update entry point imports to lablink_allocator_service
- Fixes remaining Docker Build Test failures

* fix: update all imports in lablink-images workflow

- Update 5 allocator imports to lablink_allocator_service
- Update 6 client imports to lablink_client_service
- Fixes Verify Allocator Image and Verify Client Image failures

Author: eberrigan

.github/workflows/ci.yml

@@ -52,7 +52,7 @@ jobs:
           - package: lablink-client
             python-version: "3.10"
             working-directory: packages/client
-            test-command: "uv run pytest tests --cov=src/lablink_client --cov-report=term-missing:skip-covered --cov-report=xml --cov-report=html"
+            test-command: "uv run pytest tests --cov=src/lablink_client_service --cov-report=term-missing:skip-covered --cov-report=xml --cov-report=html"
           - package: lablink-allocator
             python-version: "3.10"
             working-directory: packages/allocator
@@ -127,11 +127,11 @@ jobs:
           import logging
           logging.basicConfig(level=logging.ERROR)
           print(\"Importing lablink-allocator packages:\")
-          from lablink_allocator.main import main
+          from lablink_allocator_service.main import main
           print(\"  - main.main imported\")
-          from lablink_allocator.database import PostgresqlDatabase
+          from lablink_allocator_service.database import PostgresqlDatabase
           print(\"  - database.PostgresqlDatabase imported\")
-          from lablink_allocator.get_config import get_config
+          from lablink_allocator_service.get_config import get_config
           print(\"  - get_config.get_config imported\")
           print(\"Package imports successful\")
           "'
@@ -145,8 +145,8 @@ jobs:
         run: |
           echo "Testing console script entry points can be imported..."
           docker run --rm lablink-allocator-dev:test bash -c 'source /app/.venv/bin/activate && python -c "
-          from lablink_allocator.main import main as allocator_main
-          from lablink_allocator.generate_init_sql import main as generate_sql_main
+          from lablink_allocator_service.main import main as allocator_main
+          from lablink_allocator_service.generate_init_sql import main as generate_sql_main
           assert callable(allocator_main), \"allocator main() is not callable\"
           assert callable(generate_sql_main), \"generate_init_sql main() is not callable\"
           print(\"✓ All entry points are importable and callable\")

.github/workflows/lablink-images.yml

@@ -224,11 +224,11 @@ jobs:
           import logging
           logging.basicConfig(level=logging.ERROR)
           print(\"Importing allocator packages:\")
-          from lablink_allocator.main import main
+          from lablink_allocator_service.main import main
           print(\"  - main.main imported\")
-          from lablink_allocator.database import PostgresqlDatabase
+          from lablink_allocator_service.database import PostgresqlDatabase
           print(\"  - database.PostgresqlDatabase imported\")
-          from lablink_allocator.get_config import get_config
+          from lablink_allocator_service.get_config import get_config
           print(\"  - get_config.get_config imported\")
           print(\"Allocator imports verified\")
           "'
@@ -254,8 +254,8 @@ jobs:
         run: |
           echo "Testing allocator console script entry points can be imported..."
           docker run --rm ${{ steps.get_tag.outputs.image }} bash -c 'source /app/.venv/bin/activate && python -c "
-          from lablink_allocator.main import main as allocator_main
-          from lablink_allocator.generate_init_sql import main as generate_sql_main
+          from lablink_allocator_service.main import main as allocator_main
+          from lablink_allocator_service.generate_init_sql import main as generate_sql_main
           assert callable(allocator_main), \"allocator main() is not callable\"
           assert callable(generate_sql_main), \"generate_init_sql main() is not callable\"
           print(\"✓ All entry points are importable and callable\")
@@ -305,11 +305,11 @@ jobs:
           echo "Testing client package imports..."
           docker run --rm ${{ steps.get_tag.outputs.image }} bash -c 'source /home/client/.venv/bin/activate && python -c "
           print(\"Importing client packages:\")
-          from lablink_client.subscribe import main
+          from lablink_client_service.subscribe import main
           print(\"  - subscribe.main imported\")
-          from lablink_client.check_gpu import main as check_main
+          from lablink_client_service.check_gpu import main as check_main
           print(\"  - check_gpu.main imported\")
-          from lablink_client.update_inuse_status import main as update_main
+          from lablink_client_service.update_inuse_status import main as update_main
           print(\"  - update_inuse_status.main imported\")
           print(\"Client imports verified\")
           "'
@@ -340,9 +340,9 @@ jobs:
         run: |
           echo "Testing client console script entry points can be imported..."
           docker run --rm ${{ steps.get_tag.outputs.image }} bash -c 'source /home/client/.venv/bin/activate && python -c "
-          from lablink_client.check_gpu import main as check_gpu_main
-          from lablink_client.subscribe import main as subscribe_main
-          from lablink_client.update_inuse_status import main as update_main
+          from lablink_client_service.check_gpu import main as check_gpu_main
+          from lablink_client_service.subscribe import main as subscribe_main
+          from lablink_client_service.update_inuse_status import main as update_main
           assert callable(check_gpu_main), \"check_gpu main() is not callable\"
           assert callable(subscribe_main), \"subscribe main() is not callable\"
           assert callable(update_main), \"update_inuse_status main() is not callable\"

.github/workflows/publish-pip.yml

@@ -5,13 +5,13 @@ on:
     types: [published]
   push:
     tags:
-      - lablink-allocator_v*
-      - lablink-client_v*
+      - lablink-allocator-service_v*
+      - lablink-client-service_v*
   workflow_dispatch:
     inputs:
       package:
         type: choice
-        options: [lablink-allocator, lablink-client]
+        options: [lablink-allocator-service, lablink-client-service]
         required: true
         description: Which package to publish
       dry_run:
@@ -34,13 +34,13 @@ jobs:
       fail-fast: true
       matrix:
         include:
-          - package: lablink-allocator
+          - package: lablink-allocator-service
             dir: packages/allocator
-            tag_prefix: lablink-allocator_v
+            tag_prefix: lablink-allocator-service_v
             path_filter: packages/allocator
-          - package: lablink-client
+          - package: lablink-client-service
             dir: packages/client
-            tag_prefix: lablink-client_v
+            tag_prefix: lablink-client-service_v
             path_filter: packages/client
 
     steps:
@@ -167,6 +167,7 @@ jobs:
           source .venv/bin/activate
           echo "Running test suite..."
           uv run pytest tests \
+            --ignore=tests/terraform \
             --cov=src/${{ matrix.package }} \
             --cov-report=term-missing \
             --cov-fail-under=0