-
Notifications
You must be signed in to change notification settings - Fork 136
fix(ci): ensure correct latest release after backport publishing #1800
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Replace unreliable getLatestRelease() API with semver-based max version detection. After publishing a backport release, explicitly restore the latest flag on the highest semver release to handle cases where GitHub API ignores make_latest: 'false'. Also remove dead code (unused steps) from tags.yaml workflow. Co-Authored-By: Claude <[email protected]> Signed-off-by: Andrei Kvapil <[email protected]>
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
|
Caution Review failedThe pull request is closed. 📝 WalkthroughWalkthroughThis pull request refactors release workflow logic by moving semver comparison and latest-release determination from external API calls into the workflows themselves. The tags workflow is simplified by removing semver comparison, while the pull-requests-release workflow adds internal semver parsing and conditional release adjustment logic. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Poem
📜 Recent review detailsConfiguration used: defaults Review profile: CHILL Plan: Pro 📒 Files selected for processing (2)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Git push to origin failed for release-0.39 with exitcode 1 |
|
Git push to origin failed for release-0.38 with exitcode 1 |
## What this PR does Fixes an issue where backport releases incorrectly became marked as "Latest" despite passing `make_latest: 'false'` to the GitHub API. **Root cause:** The `getLatestRelease()` API returns the release with the "Latest" flag, not the highest semver version. Combined with race conditions during parallel release publishing and GitHub API potentially ignoring `make_latest: 'false'`, backport releases were incorrectly marked as latest. **Solution:** - Replace `getLatestRelease()` with semver-based max version detection across all published releases - After publishing a backport release, explicitly restore the latest flag on the highest semver release - Remove unused dead code from `tags.yaml` workflow ### Release note ```release-note [ci] Fix latest release detection to use semver comparison instead of GitHub's "Latest" flag ```
## What this PR does Fixes an issue where backport releases incorrectly became marked as "Latest" despite passing `make_latest: 'false'` to the GitHub API. **Root cause:** The `getLatestRelease()` API returns the release with the "Latest" flag, not the highest semver version. Combined with race conditions during parallel release publishing and GitHub API potentially ignoring `make_latest: 'false'`, backport releases were incorrectly marked as latest. **Solution:** - Replace `getLatestRelease()` with semver-based max version detection across all published releases - After publishing a backport release, explicitly restore the latest flag on the highest semver release - Remove unused dead code from `tags.yaml` workflow ### Release note ```release-note [ci] Fix latest release detection to use semver comparison instead of GitHub's "Latest" flag ```
## What this PR does Fixes an issue where backport releases incorrectly became marked as "Latest" despite passing `make_latest: 'false'` to the GitHub API. **Root cause:** The `getLatestRelease()` API returns the release with the "Latest" flag, not the highest semver version. Combined with race conditions during parallel release publishing and GitHub API potentially ignoring `make_latest: 'false'`, backport releases were incorrectly marked as latest. **Solution:** - Replace `getLatestRelease()` with semver-based max version detection across all published releases - After publishing a backport release, explicitly restore the latest flag on the highest semver release - Remove unused dead code from `tags.yaml` workflow ### Release note ```release-note [ci] Fix latest release detection to use semver comparison instead of GitHub's "Latest" flag ```
What this PR does
Fixes an issue where backport releases incorrectly became marked as "Latest" despite passing
make_latest: 'false'to the GitHub API.Root cause: The
getLatestRelease()API returns the release with the "Latest" flag, not the highest semver version. Combined with race conditions during parallel release publishing and GitHub API potentially ignoringmake_latest: 'false', backport releases were incorrectly marked as latest.Solution:
getLatestRelease()with semver-based max version detection across all published releasestags.yamlworkflowRelease note
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.