Skip to content

fix(workflow): default updater_ref to v0 and validate ref before checkout - #56

Merged
d-lazenby merged 1 commit into
mainfrom
55-reusable-workflow-fails-when-default-updater_refv1-does-not-exist
Aug 11, 2026
Merged

fix(workflow): default updater_ref to v0 and validate ref before checkout#56
d-lazenby merged 1 commit into
mainfrom
55-reusable-workflow-fails-when-default-updater_refv1-does-not-exist

Conversation

@d-lazenby

@d-lazenby d-lazenby commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary
This PR fixes a reusable workflow failure caused by an invalid default updater ref and aligns documentation with current release behaviour.

Closes #55

What Changed

  • Updated workflow default updater_ref from v1 to v0 for both workflow_call and workflow_dispatch in .github/workflows/update-tf-modules.yml.
  • Added an early validation step before source checkout to verify updater_ref exists as a branch or tag (or is SHA-like), with a clear actionable error message on failure in .github/workflows/update-tf-modules.yml.
  • Updated docs/examples to use the current major channel v0 and corrected updater_ref default references.

Why

  • The project is currently on 0.x releases, so v1 may not exist and causes low-signal checkout failures.
  • Preflight validation provides clearer diagnostics and faster troubleshooting for callers.

Impact

  • Existing callers that do not pass updater_ref now default to a valid major channel.
  • Invalid refs now fail early with a clear message instead of a generic git fetch error.

@d-lazenby
d-lazenby requested a review from a team as a code owner August 10, 2026 14:57
@d-lazenby d-lazenby linked an issue Aug 10, 2026 that may be closed by this pull request
@d-lazenby d-lazenby added the bug Something isn't working label Aug 10, 2026
@d-lazenby d-lazenby self-assigned this Aug 10, 2026

@small-e small-e left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Looks good to me

@d-lazenby
d-lazenby merged commit 48d5306 into main Aug 11, 2026
5 checks passed
@d-lazenby
d-lazenby deleted the 55-reusable-workflow-fails-when-default-updater_refv1-does-not-exist branch August 11, 2026 08:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Reusable workflow fails when default updater_ref=v1 does not exist

2 participants