Skip to content

Harden workflow artifact selection - #1803

Open
wiegandm wants to merge 1 commit into
mainfrom
harden-workflow-artifact-selection
Open

Harden workflow artifact selection#1803
wiegandm wants to merge 1 commit into
mainfrom
harden-workflow-artifact-selection

Conversation

@wiegandm

@wiegandm wiegandm commented Sep 4, 2026

Copy link
Copy Markdown
Member

What

  • Preserve the configured workflow status string so GitHub receives status=success rather than status=completed.
  • Detect duplicate workflow-run IDs and non-successful runs in a success query.
  • Retry invalid workflow-run responses and responses with no usable artifact before failing.

Why

GitHub API responses have intermittently omitted recent workflow runs and included duplicate old runs, causing downstream jobs to select expired artifacts or find no usable artifact.

References

DOS-836

Checklist

  • Tests

Generated with the help of pi coding agent harness (model: openai-codex/gpt-5.6-terra).

Generated with the help of `pi` coding agent harness (model: openai-codex/gpt-5.6-terra).
@wiegandm
wiegandm requested a review from a team as a code owner September 4, 2026 09:10
@greenbonebot
greenbonebot enabled auto-merge (rebase) September 4, 2026 09:10
@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA eb3209c.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown

Conventional Commits Report

😢 No conventional commits found.

👉 Learn more about the conventional commits usage at Greenbone.

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