Skip to content

Container maintenance#2351

Draft
GernotMaier wants to merge 6 commits into
mainfrom
container-maintenance
Draft

Container maintenance#2351
GernotMaier wants to merge 6 commits into
mainfrom
container-maintenance

Conversation

@GernotMaier

@GernotMaier GernotMaier commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Maintenance points:

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses issue #2349 by updating the AlmaLinux-based container images to run as a non-root default user (simtools) instead of root, aligning the container runtime posture with common security expectations.

Changes:

  • Add configurable container user/UID/GID build args and switch runtime to USER simtools (with HOME set accordingly).
  • Ensure /workdir and the user home are owned appropriately for non-root execution.
  • Add a changelog fragment documenting the container maintenance change.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
docs/changes/2351.maintenance.md Changelog entry for switching containers away from root.
docker/Dockerfile-simtools-prod Create non-root user and set it as default user for the production simtools container.
docker/Dockerfile-simtools-dev Create non-root user and set it as default user for the development simtools container.
docker/Dockerfile-simtel_array Create non-root user and set it as default user for the sim_telarray runtime container.
docker/Dockerfile-corsika7 Create non-root user and set it as default user for the CORSIKA7 runtime container.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docker/Dockerfile-simtools-prod
Comment thread docker/Dockerfile-simtools-dev Outdated
Comment thread docker/Dockerfile-simtel_array
Comment thread docker/Dockerfile-corsika7
@ctao-sonarqube

Copy link
Copy Markdown

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.

The "almalinux" image runs with "root" as the default user.

2 participants