Skip to content

fix(opentofu): surface validation step failures - #37

Merged
xnoto merged 1 commit into
mainfrom
fix/opentofu-visible-validation-failures
Aug 30, 2026
Merged

fix(opentofu): surface validation step failures#37
xnoto merged 1 commit into
mainfrom
fix/opentofu-visible-validation-failures

Conversation

@xnoto

@xnoto xnoto commented Aug 30, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove continue-on-error from OpenTofu test setup and validation steps so GitHub Actions renders the originating step as failed
  • keep same-repository PR reporting with always()
  • continue the full pre-commit run after a docs-key failure when checkout, config fetch, and OpenTofu initialization succeeded
  • remove the redundant terminal Mark validation failed step

Root cause

continue-on-error: true preserved each step's outcome as failure but converted its displayed conclusion to success. The terminal marker then failed the job, obscuring the originating step.

Compatibility and safety

  • no reusable-workflow inputs, secrets, runner labels, plan/apply behavior, or fork protections change
  • failure reporting remains restricted to same-repository PRs
  • no credentials or sensitive values are added

Validation

GitHub Actions CI is the validation authority; no local OpenTofu operation was run.

@xnoto
xnoto requested a review from a team as a code owner August 30, 2026 19:29
@github-actions

github-actions Bot commented Aug 30, 2026

Copy link
Copy Markdown

Shared Workflows CI

Shared workflows CI passed.

View the workflow run.

View run output
[INFO] Initializing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Initializing environment for https://github.com/gitleaks/gitleaks.
[INFO] Initializing environment for https://github.com/rhysd/actionlint.
[INFO] Installing environment for https://github.com/pre-commit/pre-commit-hooks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/gitleaks/gitleaks.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
[INFO] Installing environment for https://github.com/rhysd/actionlint.
[INFO] Once installed this environment will be reused.
[INFO] This may take a few minutes...
check for case conflicts.................................................Passed
check for merge conflicts................................................Passed
check for broken symlinks............................(no files to check)Skipped
check vcs permalinks.....................................................Passed
detect destroyed symlinks................................................Passed
detect private key.......................................................Passed
mixed line ending........................................................Passed
trim trailing whitespace.................................................Passed
Detect hardcoded secrets.................................................Passed
Lint GitHub Actions workflow files.......................................Passed

@xnoto
xnoto merged commit a4ef5f0 into main Aug 30, 2026
2 checks passed
@xnoto
xnoto deleted the fix/opentofu-visible-validation-failures branch August 30, 2026 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant