feat(terraform/lock): parse provider hostname onto lock download client #1870
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This enables support for opentofu for lock files.
The provider already supported providing full provider names ex:
registry.opentofu.org/hashicorp/hcp
, but the hostname wasn't parsed onto the lock client (Which defaults toregistry.terraform.io
). By passing this on it adds support for opentofu with very little changes.Part of: #1861
Test
To test this pull request, you can run the following commands:
go test ./pkg/plugins/resources/terraform/lock/
Also tested with existing terraform project.
Then adding the hostname to the provider:
Produces the following output:
Additional Information
Tradeoff
Potential improvement
Further investigation into differences between opentofu and terraform as they deveop.
Add documentation for opentofu support.
Provider support for opentofu still needs to be added.