feat: ensure merged lab config is materialized for infra actions#140
Merged
feat: ensure merged lab config is materialized for infra actions#140
Conversation
…ration
Materialize merged overlay configs to the path terragrunt HCL expects
(ad/GOAD/data/{env}-config.json) before running any infra action. This
fixes terragrunt failing with 'no file exists' when only an overlay
file is present.
Extract infraLogPath helper to reduce cyclomatic complexity of
runInfraAction.
Add gitignore pattern for generated merged lab configs.
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
1 participant
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.
Key Changes:
.gitignoreto exclude generated merged lab config filesAdded:
ensuring that terragrunt always receives the correct config, even when overlays
are used
materializeLabConfigfunction to handle merging and placement of lab configfiles as required by terragrunt
infraLogPathhelper function for consistent and centralized log file pathgeneration logic
Changed:
infraLogPathfunction, reducing duplication and improving maintainabilitymaterializeLabConfigbefore executing anyinfra action to prevent missing or outdated config errors
.gitignoreto exclude all generated merged lab config files(
ad/GOAD/data/*-config.json) from version control to prevent accidentalcommits of build artifacts
Removed:
infraLogPathfor better clarity and reuse