Skip to content

Fix provider mirroring in air-gapped environments#157

Merged
dgannon991 merged 2 commits intogetporter:mainfrom
lbergnehr:main
Feb 23, 2026
Merged

Fix provider mirroring in air-gapped environments#157
dgannon991 merged 2 commits intogetporter:mainfrom
lbergnehr:main

Conversation

@lbergnehr
Copy link
Contributor

Replace 'terraform providers mirror' with direct copy from .terraform/providers. The mirror command always contacts the official registry for metadata, which fails in air-gapped environments (HashiCorp issue #33931). Copying providers directly after 'terraform init' works universally since init respects the configured providerHost.

@lbergnehr
Copy link
Contributor Author

@tamirkamara , @kichristensen .

@tamirkamara
Copy link
Contributor

@lbergnehr I can test it in a secured environment tomorrow to see that it still works as expected.

lbergnehr and others added 2 commits January 20, 2026 14:30
Replace 'terraform providers mirror' with direct copy from .terraform/providers.
The mirror command always contacts the official registry for metadata, which
fails in air-gapped environments (HashiCorp issue #33931). Copying providers
directly after 'terraform init' works universally since init respects the
configured providerHost.

Signed-off-by: Leo Bergnéhr <leo@bergnehr.se>
Co-authored-by: Tamir Kamara <26870601+tamirkamara@users.noreply.github.com>
Signed-off-by: Leo Bergnehr <leo.bergnehr@sectra.com>
@dgannon991
Copy link
Member

@tamirkamara - are you OK to confirm it worked in your environment as well if that's OK, please? :)

@lbergnehr
Copy link
Contributor Author

Ping @tamirkamara .

@lbergnehr
Copy link
Contributor Author

@dgannon991 would it be possible to get some progress on this you think?

Copy link
Contributor

@tamirkamara tamirkamara left a comment

Choose a reason for hiding this comment

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

@dgannon991 can we get this merged so that I could test it from canary?

@dgannon991 dgannon991 merged commit 8e18e6c into getporter:main Feb 23, 2026
2 checks passed
@dgannon991
Copy link
Member

@tamirkamara - absolutely. If you're both able to give me a thumbs up that you're happy with the canary release, I can cut a full release. Cheers both!

@lbergnehr
Copy link
Contributor Author

Looking good here!

@tamirkamara
Copy link
Contributor

@dgannon991 tested on my air-gapped environment so I support releasing.

@dgannon991
Copy link
Member

@tamirkamara & @lbergnehr, that's released. Thank you so much for all of your work and time on this one :)

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.

3 participants