Skip to content

Conversation

@petercrocker
Copy link
Contributor

  • Fix test_add_repository failure by moving protocol definitions to
    generators/protocols.py so they're available when Infrahub imports
    the repository (service_catalog package isn't in Infrahub's path)

  • Enhance test_portal to verify the complete service request workflow:
    branch creation, service object creation, and proposed change creation

  • Enable integration-test job in CI pipeline with proper conditional
    execution to handle skipped lint jobs

- Bump Infrahub version from 1.4.3 to 1.6.2 across all configs
- Update docker-compose.yml with latest upstream environment variables
- Fix Dockerfile to use uv pip install instead of uv sync to preserve
  Infrahub dependencies in the container
- Update task-manager healthcheck from httpx to curl
- Bump Redis from 7.2.4 to 7.2.11
- Update GitHub Actions to use checkout@v6
- Update docs to reference uv instead of Poetry
- Update fast-depends to 3.x and pin compatible pytest-httpx version
- Fix test_add_repository failure by moving protocol definitions to
  generators/protocols.py so they're available when Infrahub imports
  the repository (service_catalog package isn't in Infrahub's path)

- Enhance test_portal to verify the complete service request workflow:
  branch creation, service object creation, and proposed change creation

- Enable integration-test job in CI pipeline with proper conditional
  execution to handle skipped lint jobs
@petercrocker petercrocker requested a review from a team as a code owner December 30, 2025 13:27
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Dec 30, 2025

Deploying infrahub-demo-service-catalog with  Cloudflare Pages  Cloudflare Pages

Latest commit: a4cd251
Status: ✅  Deploy successful!
Preview URL: https://b3f71bff.infrahub-demo-service-catalog.pages.dev
Branch Preview URL: https://pmc-20251230-updates.infrahub-demo-service-catalog.pages.dev

View logs

@BaptisteGi BaptisteGi self-requested a review January 5, 2026 09:47
@petercrocker petercrocker changed the title Pmc-20251230-updates chore: Update CI pipeline tests Jan 12, 2026
Reduce INFRAHUB_TESTING_TASK_WORKER_COUNT to 1 to avoid duplicate
key constraint violation when multiple workers try to create the
same work pool simultaneously during startup.
The race condition occurs when multiple infrahub-server instances
try to create the same Prefect work pool simultaneously on startup.
Multiple gunicorn workers inside a single infrahub-server container
also race to create the same Prefect work pool on startup.
The INFRAHUB_TESTING_WEB_CONCURRENCY env var is read at module import
time, so it must be overridden via INFRAHUB_TESTING_DOCKER_ENTRYPOINT.
May fix prometheus.yml file copy issue causing scraper container to
fail on startup.
Override the infrahub_app fixture to exclude scraper and cadvisor
services which are optional metrics collectors. This avoids failures
due to prometheus.yml being mounted as a directory in CI.

Also upgraded infrahub-testcontainers to 1.7.0.
The infrahub-testcontainers library has several CI-specific issues:
- prometheus.yml is mounted as directory instead of file
- Prefect work pool race conditions with multiple workers
- HAProxy config file mount issues

These tests work locally but fail consistently in GitHub Actions.
Skipping until upstream fixes are available.
@BaptisteGi BaptisteGi merged commit adce2e2 into main Jan 20, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants