feat: replace container base images with pixi-managed environments#810
Merged
fstagni merged 7 commits intoDIRACGrid:mainfrom Mar 26, 2026
Merged
feat: replace container base images with pixi-managed environments#810fstagni merged 7 commits intoDIRACGrid:mainfrom
fstagni merged 7 commits intoDIRACGrid:mainfrom
Conversation
chrisburr
force-pushed
the
feat/pixi-containers
branch
from
7dbead8 to
e840466
Compare
Documentation build overview
Show files changed (2 files in total): 📝 2 modified | ➕ 0 added | ➖ 0 deleted
|
chrisburr
force-pushed
the
feat/pixi-containers
branch
13 times, most recently
from
a86608d to
c11d95c
Compare
Contributor
|
I think the problem here is that we need to override the entrypoint of the keystore init job as well, but I can't try right now |
chaen
reviewed
Mar 24, 2026
containers/Dockerfile
Outdated
| ARG PIXI_ENV=container-services | ||
| ARG DIRACX_EXTENSIONS=diracx | ||
| ENV PIXI_ENV=${PIXI_ENV} | ||
| ENV DIRACX_EXTENSIONS=${DIRACX_EXTENSIONS} |
Contributor
There was a problem hiding this comment.
You redefine DIRACX_EXTENSIONS from 2 lines above
Member
Author
There was a problem hiding this comment.
This is the env var not the arg?
Member
Author
There was a problem hiding this comment.
In any case, this variable is no longer needed since DIRACGrid/DIRAC#8488
chaen
reviewed
Mar 24, 2026
| with: | ||
| context: extensions/containers/services | ||
| context: . | ||
| file: containers/Dockerfile |
Contributor
There was a problem hiding this comment.
should that be extensions/containers/Dockerfile ?
Member
Author
There was a problem hiding this comment.
No it builds from the top level pixi.toml
Contributor
There was a problem hiding this comment.
OK so the extensions/containers/Dockerfile is just for docs purposes ?
chrisburr
force-pushed
the
feat/pixi-containers
branch
2 times, most recently
from
2710220 to
a8350cf
Compare
…28.0 Since opentelemetry-sdk v1.28.0 (open-telemetry/opentelemetry-python#4166), LoggingHandler._translate() calls self.format(record) when a formatter is set. The existing formatter uses %(otelTraceID)s placeholders but set_logging_format=False meant the LoggingInstrumentor never injected those attributes, causing a KeyError at runtime.
chrisburr
force-pushed
the
feat/pixi-containers
branch
from
a8350cf to
9b8f0a6
Compare
chrisburr
force-pushed
the
feat/pixi-containers
branch
from
9b8f0a6 to
71d4284
Compare
chrisburr
force-pushed
the
feat/pixi-containers
branch
from
1002637 to
1a2ef83
Compare
Member
Author
|
I think there is nothing left blocking this from being merged. |
fstagni
reviewed
Mar 26, 2026
fstagni
approved these changes
Mar 26, 2026
chrisburr
added a commit
to chrisburr/diracx
that referenced
this pull request
Mar 26, 2026
Container images are now built from source via pixi (since DIRACGrid#810), so editable pip installs of mounted source dirs are redundant. Removing them fixes the gubbins init-cs failure caused by git "dubious ownership" errors when the container UID differs from the mounted source owner.
chrisburr
added a commit
to chrisburr/diracx
that referenced
this pull request
Mar 27, 2026
Container images are now built from source via pixi (since DIRACGrid#810), so editable pip installs of mounted source dirs are redundant. Removing them fixes the gubbins init-cs failure caused by git "dubious ownership" errors when the container UID differs from the mounted source owner.
aldbr
pushed a commit
to chrisburr/diracx
that referenced
this pull request
Mar 27, 2026
Container images are now built from source via pixi (since DIRACGrid#810), so editable pip installs of mounted source dirs are redundant. Removing them fixes the gubbins init-cs failure caused by git "dubious ownership" errors when the container UID differs from the mounted source owner.
fstagni
pushed a commit
that referenced
this pull request
Mar 27, 2026
Container images are now built from source via pixi (since #810), so editable pip installs of mounted source dirs are redundant. Removing them fixes the gubbins init-cs failure caused by git "dubious ownership" errors when the container UID differs from the mounted source owner.
chrisburr
added a commit
to chrisburr/diracx
that referenced
this pull request
Mar 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #763
Replace container builds from the multi-repo base image approach
(micromamba + conda
environment.yml+pip install --no-deps) withself-contained, multi-stage Dockerfiles using pixi environments from the
committed lock file. This ensures reproducible builds with exact version
parity between development and production.
Summary
containers/Dockerfilethat builds bothservicesandclientimages via aPIXI_ENVbuild argpixi.lockis now committed (previously gitignored) to pin alldependencies
container-services,container-client) aredefined in
pixi.tomlin the same solve group as dev environmentsentrypoint from the matching diracx version at build time to avoid
maintaining a duplicate
linux-aarch64added to supported platformsgubbins integration tests
containers/client/,containers/services/,extensions/containers/) removedKeyErrorwithsdk >=1.28.0and pinimportlib-metadata <8.8.0to work around a pixi conda/PyPI conflictSee also #852 for the planned split of
extensions/gubbins/into astandalone repository.