chore: Refine provider-integration GitHub workflows#142
Open
X-Guardian wants to merge 4 commits intoopen-constructs:mainfrom
Open
chore: Refine provider-integration GitHub workflows#142X-Guardian wants to merge 4 commits intoopen-constructs:mainfrom
X-Guardian wants to merge 4 commits intoopen-constructs:mainfrom
Conversation
jsteinich
reviewed
May 2, 2026
jsteinich
approved these changes
May 4, 2026
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.
Description
Refine the
provider-integrationGitHub workflows:1. Terraform cache path →
${RUNNER_TEMP}/terraform-cacheWas previously
/usr/local/share/.cache/terraform, hard-coded as a Linux path. On Windows runners under Git Bash this resolves into a Git Bash–virtual path that doesn't actually exist on the Windows filesystem, producing this warning every run:Replaced with
${RUNNER_TEMP}/...which is set by the runner on both Linux and Windows. Bothlinux_providerandwindows_providerupdated for consistency.Side effect: Windows now actually gets a working terraform plugin cache (it never did before — the path was non-existent).
2. Fix windows_provider terraform cache key
Was using
${{ matrix.terraform }}while linux_provider used${{ inputs.terraform_version }}. The matrix doesn't expose aterraformfield, so the windows cache key was silently interpolating to an empty string — every windows matrix entry was sharing a single cache regardless of Terraform version. Aligned to useinputs.terraform_versionlike linux.3. Remove unused steps
Remove 'Install pipenv' and 'Install Go' steps from the
windows_providerworkflow as they not used.Checklist