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
feat(terragrunt): add support for gitlab-tags datasource #27903
feat(terragrunt): add support for gitlab-tags datasource #27903
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please reduce test fixtures, we don't need them all. just one for each new datasource is enough.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why do you change so many fixtures? I think you mostly only need to change a few samples to trigger the new code paths
Just re-arranged them a bit to have a better grouping like having all GitTag and skipReason fixtures together |
please do that in a separate PR |
Alright, I've reverted the extra refactorings. |
next time you should do the test url renaming in a separate PR 🤗 |
🎉 This PR is included in version 37.253.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
with this PR, all of our gitlab MergeRequest was closed by renovate. renovate.json
terragrunt.hcl
logs
|
Seems like the API call url is wrongly constructed now, it should be sth. like: - "url": "https://gitlab.com/api/v4/projects/https%3A%2F%2Fgit.mycompany.de%2Fhycloud%2Faws%2Faws-eks%2Fmodules%2Faws-eks-cluster.git/repository/tags?per_page=100"
+ "url": "https://gitlab.com/api/v4/projects/hycloud%2Faws%2Faws-eks%2Fmodules%2Faws-eks-cluster/repository/tags?per_page=100" |
@rarkins revert? |
I maybe found the issue. It seems like the Currently the manager always sets the whole URL as |
Changes
Hi everyone,
Unfortunately the PRs for Terragrunt dependency updates do not include any changelog, since the terragrunt manager currently only supports git-tags, github-tags and terraform-module datasources, which are not able to fetch changelogs from out GitLab Releases.
Context
Documentation (please check one with an [x])
How I've tested my work (please select one)
I have verified these changes via: