Skip to content

fix(release-automation): retry regression bot comment reads#346

Merged
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/ra-regression-bot-reply-parsing
Jun 17, 2026
Merged

fix(release-automation): retry regression bot comment reads#346
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/ra-regression-bot-reply-parsing

Conversation

@hdamker

@hdamker hdamker commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

bug
tests

What this PR does / why we need it:

The release automation regression runner currently treats a failed comments API read as a final failure after the caller workflow has already succeeded. One canary run hit a transient/non-JSON gh api response while fetching Release Issue comments, even though the expected snapshot-created bot reply was present.

This PR retries issue-comment reads before reporting an infrastructure failure. It keeps the existing bot reply title checks unchanged.

Which issue(s) this PR fixes:

N/A - follow-up to #336.

Special notes for reviewers:

Validation:

  • python3 -m pytest release_automation/tests/test_regression_runner.py -q
  • python3 -m pytest release_automation/tests -q
  • Full ReleaseTest round trip from this branch: PASS, 5 of 5 phases. The runner created snapshot PR #209, read the expected bot replies, discarded the snapshot, and restored the Release Issue to planned.

Changelog input

release-note
None

Additional documentation

This section can be blank.

docs
N/A

@hdamker hdamker requested review from Kevsy and rartych as code owners June 17, 2026 16:55
@hdamker

hdamker commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

Stability change for regression runner only, no change to RA or Validation. Merging before #345 to see it in action after the merge of #345.

@hdamker hdamker merged commit c6a8b84 into camaraproject:main Jun 17, 2026
8 checks passed
@hdamker hdamker mentioned this pull request Jun 19, 2026
12 tasks
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