Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 pull request adds support for handling "sentinel default" variables in Terraform modules—variables that have a placeholder default value (like "inject-at-runtime") to indicate their true value should be supplied externally (e.g., by a CI/CD pipeline). The new functionality allows these variables to be auto-filled, tracked, and rendered as commented-out entries in emitted
.tfvarsfiles, making it clear to users which variables are expected to be provided externally. The documentation, API surface, and tests have all been updated to support and describe this feature.Sentinel default variable support:
FillSentinelmethod toModuleCallBuilderthat auto-fills variables whose default matches a specified sentinel value, tracks them, and ensures they are rendered as commented-out entries in emitted.tfvarsfiles. [1] [2] [3] [4] [5] [6] [7] [8]HasSentinelDefaulttoTerraformVariableandGetSentinelVariablestoTerraformModuleto identify and retrieve variables using a sentinel default. [1] [2] [3] [4]Documentation updates:
docs/terraformdotnet.mdto document the sentinel default variable feature, including usage examples, API additions, and.tfvarsoutput format. [1] [2] [3]Testing:
FillSentinelinModuleCallBuilderTests, covering argument population, skipping of already set variables, exclusion from optional comments, and error handling.Public API changes:
PublicAPI.Shipped.txtto include the new methods and properties for sentinel variable support. [1] [2] [3]