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(manager/terragrunt): Update terragrunt packageName #33810

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

mblum14
Copy link

@mblum14 mblum14 commented Jan 23, 2025

Fix dependency.packageName evaluated by the terragrunt manager when the terraform source url contains double slashes and ssh as the protocol (e.g., git::ssh://git@mygit.com/hashicorp/example.git//subdir/test?ref=1.1.0 )

Changes

Changes behavior when evaluating dependencies with the terragrunt manager. Specifically, this changes the behavior with terraform sources that have double slashes in the URL path and ssh as the protocol

for example:

# terragrunt.hcl
terraform {
  source = "git::ssh://git@mygit.com/hashicorp/example.git//subdir/test?ref=v1.0.4"
}

Context

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests but ran on a real repository, or
  • Both unit tests + ran on a real repository

@mblum14 mblum14 force-pushed the fix_terragrunt_manager branch from 2f9d3b9 to 870e10c Compare January 23, 2025 21:24
Closes renovate/renovate#33370

- Closes renovatebot#33370

Fix dependency.packageName evaluated by the terragrunt manager when the
terraform source url contains double slashes and ssh as the protocol (e.g.,
git::ssh://git@mygit.com/hashicorp/example.git//subdir/test?ref=1.1.0 )
@mblum14 mblum14 force-pushed the fix_terragrunt_manager branch from 8231442 to 4d8061e Compare January 23, 2025 21:29
@viceice viceice changed the title fix(terragrunt-manager): Update terragrunt packageName fix(manager/terragrunt): Update terragrunt packageName Jan 24, 2025
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.

Terragrunt: Failed to properly parse git/ssh dependency with sub directory
3 participants