Skip to content

PREQ-2947 retry on version not found and gracefully exit after retries#116

Merged
tomverin merged 1 commit intomasterfrom
feat/tom/PREQ-2947
Nov 27, 2025
Merged

PREQ-2947 retry on version not found and gracefully exit after retries#116
tomverin merged 1 commit intomasterfrom
feat/tom/PREQ-2947

Conversation

@tomverin
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings November 27, 2025 09:54
@tomverin tomverin requested a review from a team as a code owner November 27, 2025 09:54
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR implements retry logic for version retrieval and improves error handling when versions are not found. The changes ensure the workflow gracefully handles cases where version information is temporarily unavailable or missing entirely.

Key Changes:

  • Added retry mechanism with configurable attempts and delays when fetching version information
  • Enhanced version validation with better error messages and input sanitization
  • Modified workflow to continue gracefully when version is not found instead of failing

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/utils/version_helper.py Enhanced version validation with type checking, whitespace stripping, and improved error messages
releasability-status/find_version.sh Added retry loop for version fetching and changed exit code from 0 to 2 when version not found
releasability-status/action.yml Added conditional checks to skip releasability steps when version is not found

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@sonarqubecloud
Copy link
Copy Markdown

SonarQube reviewer guide

Summary: Add graceful handling for missing promoted versions in releasability status checks.

Review Focus: The retry logic in find_version.sh (12 attempts with 5s delays = 60s total wait time) and the error handling flow using continue-on-error with conditional step execution.

Start review at: releasability-status/find_version.sh. This script now includes retry logic and changes exit behavior (exit 2 vs exit 0), which drives the conditional execution of all downstream steps in the workflow.

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@tomverin tomverin merged commit d629774 into master Nov 27, 2025
7 checks passed
@tomverin tomverin deleted the feat/tom/PREQ-2947 branch November 27, 2025 10:23
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.

3 participants