Repository navigation
DEVOPS-14554: consolidate atmos-runner tooling into github-runner - #12
Merged
Merged
Conversation
aledesma-qventus
approved these changes
May 29, 2026
There was a problem hiding this comment.
Pull request overview
This PR consolidates additional DevOps tooling into the github-runner container image to reduce reliance on external installs during CI runs and support upcoming workflow simplifications.
Changes:
- Adds Kubernetes tooling (Helm, Helmfile, kubectl) plus
helm-diff, and adds Infracost to the image. - Bumps Atmos to
v1.207.0and centralizes tool versions via DockerfileARGs. - Expands the CI workflow to verify the newly preinstalled tools inside the built image.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.
| File | Description |
|---|---|
Dockerfile |
Introduces version ARGs and installs additional IaC/Kubernetes tooling (including helm-diff via plugin install) directly into the runner image. |
.github/workflows/test-docker-image.yaml |
Extends the tool verification step to validate the newly added binaries/plugins within the test image. |
Comments suppressed due to low confidence (1)
Dockerfile:23
- This apt-get layer leaves /var/lib/apt/lists/* behind (and removing it in a later layer won’t reduce image size). Consider cleaning apt lists in the same RUN after installing nodejs to keep the image smaller and reduce CVE surface from stale package indices.
RUN sudo apt-get update && \
sudo apt-get install -y git unzip zip jq openssh-client curl git-lfs perl && \
sudo apt-get clean && \
curl -fsSL https://deb.nodesource.com/setup_20.x | bash - && \
sudo apt-get install -y nodejs
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.
Summary
HELM_DATA_HOME=/usr/local/share/helmso helm-diff plugin is found regardless of$HOMEThis is the first step of the image consolidation: once merged and deployed, we can remove
container:directives from devops-multiaccount CI workflows and drop the dind sidecar from the RunnerSet.Test plan
docker buildsucceededatmos,terraform,tfcmt,terraform-docs,infracost,helm,helmfile,kubectl,helm diff,aws)HELM_DATA_HOMEcorrectly set, plugin foundtest-docker-image.yamlpasses on this PR