Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: renovate dockerfile handling #4381

Closed
wants to merge 4 commits into from

Conversation

tiagomeireles
Copy link
Contributor

@tiagomeireles tiagomeireles commented Mar 23, 2024

what

The Dockerfile defines default versions of terraform, opentofu and conftest in ARG's. Renovate will now update these.

image

This also corrects the renovate configuration for opentofu.

why

This broke in 6d3aef2 when it switched the defines from ENVs to ARGs.

tests

Manually tested on my fork of Atlantis.

references

@tiagomeireles tiagomeireles requested review from a team as code owners March 23, 2024 22:19
@tiagomeireles tiagomeireles requested review from jamengual, lukemassa and nitrocode and removed request for a team March 23, 2024 22:20
@github-actions github-actions bot added build Relating to how we build Atlantis github-actions labels Mar 23, 2024
@tiagomeireles tiagomeireles changed the title Fix renovate dockerfile handling fix: renovate dockerfile handling Mar 23, 2024
@tiagomeireles tiagomeireles reopened this Mar 24, 2024
@@ -6,7 +6,7 @@ ARG GOLANG_VERSION=1.22.1

# renovate: datasource=github-releases depName=hashicorp/terraform versioning=hashicorp
ARG DEFAULT_TERRAFORM_VERSION=1.7.2
# renovate: datasource=github-releases depName=hashicorp/terraform versioning=hashicorp
# renovate: datasource=github-releases depName=opentofu/opentofu versioning=hashicorp
Copy link
Contributor

Choose a reason for hiding this comment

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

is versioning=hashicorp for opentofu?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, I believe so.

@jamengual jamengual added the waiting-on-response Waiting for a response from the user label Apr 2, 2024
@nitrocode
Copy link
Member

Hi tiago. Thanks for this pr.

Could you resolve the conflicts? Also could you link to your fork's renovate page where it shows the additional dependencies screenshot above?

@erpel
Copy link
Contributor

erpel commented May 13, 2024

I don't want to steal Tiagos contribution, but would like to see this progress.
I don't know if there is another way to help with rebasing, but I did rebase it in my fork. Available in https://github.com/erpel/atlantis/tree/rebased_tiagos_renovate_fix if that's any use.

@chenrui333
Copy link
Member

Thanks @tiagomeireles!

@chenrui333 chenrui333 removed the waiting-on-response Waiting for a response from the user label May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Relating to how we build Atlantis github-actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants