Skip to content

fix: template variables and values in .wait actions#4604

Merged
AustinAbro321 merged 11 commits intomainfrom
fix-wait-variable-template
Feb 17, 2026
Merged

fix: template variables and values in .wait actions#4604
AustinAbro321 merged 11 commits intomainfrom
fix-wait-variable-template

Conversation

@AustinAbro321
Copy link
Copy Markdown
Member

@AustinAbro321 AustinAbro321 commented Feb 12, 2026

Description

Fixes #4603 which is a regression from #4549. In #4549, we no longer sent the command as a bash command, meaning environment variables of variables were not applied. Values templating was also no longer applied.

Checklist before merging

Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 12, 2026

Deploy Preview for zarf-docs canceled.

Name Link
🔨 Latest commit f60a02a
🔍 Latest deploy log https://app.netlify.com/projects/zarf-docs/deploys/69947f2bb5dda4000880cc94

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 12, 2026

Codecov Report

❌ Patch coverage is 28.12500% with 46 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pkg/packager/actions/actions.go 28.12% 46 Missing ⚠️
Files with missing lines Coverage Δ
src/api/internal/v1beta1/package.go 73.68% <ø> (ø)
src/api/v1alpha1/package.go 32.55% <ø> (ø)
src/pkg/packager/actions/actions.go 24.12% <28.12%> (+2.07%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@AustinAbro321 AustinAbro321 changed the title fix: template variables in .wait actions fix: template variables and values in .wait actions Feb 12, 2026
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@AustinAbro321 AustinAbro321 marked this pull request as ready for review February 13, 2026 14:24
@AustinAbro321 AustinAbro321 requested review from a team as code owners February 13, 2026 14:24
Copy link
Copy Markdown
Member

@brandtkeller brandtkeller left a comment

Choose a reason for hiding this comment

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

Super minor nit - otherwise I am just running some additional tests.

Comment thread src/test/e2e/30_component_action_cluster_test.go Outdated
@github-project-automation github-project-automation Bot moved this to In progress in Zarf Feb 13, 2026
AustinAbro321 and others added 4 commits February 17, 2026 08:19
Co-authored-by: Brandt Keller <43887158+brandtkeller@users.noreply.github.com>
Signed-off-by: Austin Abro <37223396+AustinAbro321@users.noreply.github.com>
Signed-off-by: Austin Abro <austinabro321@gmail.com>
@AustinAbro321
Copy link
Copy Markdown
Member Author

Fixed some new lint errors in this PR also

Copy link
Copy Markdown
Member

@brandtkeller brandtkeller left a comment

Choose a reason for hiding this comment

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

lgtm

@AustinAbro321 AustinAbro321 added this pull request to the merge queue Feb 17, 2026
Merged via the queue into main with commit bfc0582 Feb 17, 2026
31 checks passed
@AustinAbro321 AustinAbro321 deleted the fix-wait-variable-template branch February 17, 2026 18:23
@github-project-automation github-project-automation Bot moved this from In progress to Done in Zarf Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Zarf v0.71.1 wait.cluster no longer works with variables

2 participants