Skip to content
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

Fix Helm Chart check retries #338

Merged
merged 2 commits into from
Aug 1, 2024
Merged

Fix Helm Chart check retries #338

merged 2 commits into from
Aug 1, 2024

Conversation

josvazg
Copy link
Collaborator

@josvazg josvazg commented Aug 1, 2024

I tried to reset the repo, to enforce update. I also doubled the pause in every retry.

The retry is now isolated to its own function and that together with SKIP_EXECUTION on the script allows for easy local testing.

cc @nammn

All Submissions:

  • Have you signed our CLA?

@josvazg
Copy link
Collaborator Author

josvazg commented Aug 1, 2024

cc @nammn

Test preparation, loading the bash functions in the current shell:

SKIP_EXECUTION=true source .github/actions/releaser/cr.sh      

Successful test:

% check_chart_version_released mongodb-atlas-operator 2.4.1 && echo OK 
OK

Test failing to fail non existing version:

% check_chart_version_released mongodb-atlas-operator 2.4.2 && echo OK
2024-08-01T09:07:34+00:00: Retrying in 1 seconds... (1/5)
2024-08-01T09:07:35+00:00: Retrying...
"mongodb" has been removed from your repositories
"mongodb" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "mongodb" chart repository
Update Complete. ⎈Happy Helming!⎈
2024-08-01T09:07:35+00:00: Retrying in 2 seconds... (2/5)
2024-08-01T09:07:38+00:00: Retrying...
"mongodb" has been removed from your repositories
"mongodb" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "mongodb" chart repository
Update Complete. ⎈Happy Helming!⎈
2024-08-01T09:07:38+00:00: Retrying in 4 seconds... (3/5)
2024-08-01T09:07:42+00:00: Retrying...
"mongodb" has been removed from your repositories
"mongodb" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "mongodb" chart repository
Update Complete. ⎈Happy Helming!⎈
2024-08-01T09:07:42+00:00: Retrying in 8 seconds... (4/5)
2024-08-01T09:07:50+00:00: Retrying...
"mongodb" has been removed from your repositories
"mongodb" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "mongodb" chart repository
Update Complete. ⎈Happy Helming!⎈
2024-08-01T09:07:51+00:00: Retrying in 16 seconds... (5/5)
2024-08-01T09:08:07+00:00: Retrying...
"mongodb" has been removed from your repositories
"mongodb" has been added to your repositories
Hang tight while we grab the latest from your chart repositories...
...Successfully got an update from the "mongodb" chart repository
Update Complete. ⎈Happy Helming!

@josvazg josvazg merged commit 478332a into main Aug 1, 2024
1 check passed
@josvazg josvazg deleted the fix-retries branch August 1, 2024 12:10
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