Skip to content

Conversation

@jcoelho93
Copy link
Contributor

@jcoelho93 jcoelho93 commented Aug 25, 2025

what

This change adds the image.registry value to the helm-chart, and changes the Atlantis StatefulSet to use this new value when composing the Atlantis image name to pull.

This change is backwards compatible. No breaking change since the registry value defaults to ghcr.io/

why

In cases where you have a multi-region/multi-environment deployment and use container image caches, you want to be able to specify different registries per region/environment. This way you can override only the image registry and keep the image name the same across environments.

Yes, this is syntatic sugar more than anything but it's a pattern that's been used a lot.

tests

  • ran helm template chart/atlantis and checked the resulting manifests
  • apply the helm chart on my local minikube and checked atlantis pull the correct image

references

n/a

@jcoelho93 jcoelho93 requested a review from a team as a code owner August 25, 2025 13:13
Signed-off-by: José Coelho <[email protected]>
Signed-off-by: José Coelho <[email protected]>
Signed-off-by: José Coelho <[email protected]>
Copy link
Member

@GMartinez-Sisti GMartinez-Sisti left a comment

Choose a reason for hiding this comment

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

Hi @jcoelho93, thank you for the change, unfortunately this can be a breaking change for users that are already using another repository for the atlantis image.

E.g. if I'm currently using image.repository: myrepo/atlantis, this change would set the StatefulSet container to use ghcr.io/myrepo/atlantis, unless I set image.registry: "".

For this reason, please document the change in the README file and bump major.

PS: another option is to use a helper and always crop ghcr.io/ from image.repository. This should work for everyone.

@GMartinez-Sisti GMartinez-Sisti added the waiting-on-response Waiting for a response from the user label Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

waiting-on-response Waiting for a response from the user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants