Skip to content

Conversation

@wtrocki
Copy link
Member

@wtrocki wtrocki commented Mar 30, 2025

Overview

This PR updates the curl command parameters used in our Postman calls, based on the observations from our CI/CD pipeline from the last failure.

Background

During our evaluation of last failure based on tracing we added, we discovered that postman in fact was overwhelmed and took entire 2 minutes to get error. I could not reproduce this locally :(

This resulted in only two requests being made due to the previous --retry-max-time limit of 300 seconds.

Changes

To enhance the robustness and reliability of our calls, we have decided to adjust the parameters as follows:

  • Retry Attempts: Changing the --retry option to retry up to 7 times to finetune between instant failures and each request taking max 2 minutes.

  • Updated Maximum Retry Time: increased it to --retry-max-time 1200 to allow more retries within the 20-minute window (1200 seconds).

These changes lead to a potential maximum execution time of 7 attempts * 2 minutes per attempt + 7 * 30 seconds delay, which totals less than 1200 seconds.

Note

IMHO change is only temporary to help us with stability of OpenAPI job, while we are waiting for Postman support.

@wtrocki wtrocki requested a review from a team as a code owner March 30, 2025 08:32
@wtrocki wtrocki requested a review from Copilot March 31, 2025 07:30
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@wtrocki wtrocki requested a review from Copilot March 31, 2025 08:10
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@wtrocki wtrocki merged commit e806c9c into main Apr 1, 2025
7 checks passed
@wtrocki wtrocki deleted the CLOUDP-309577 branch April 1, 2025 09:12
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.

2 participants