Skip to content

Conversation

@mikemhenry
Copy link
Contributor

I've seen a few cases in the wild where CI fails because a file failed to download. I think we should try bumping to 5 since with each retry it increases the wait time and I think that ~3s might be too fast so I want to try 5s.

Checklist

  • Added a news entry

Developers certificate of origin

I've seen a few cases in the wild where CI fails because a file failed
to download. I think we should try bumping to 5 since with each retry it
increases the wait time and I think that ~3s might be too fast so I want
to try 5s.
@mikemhenry mikemhenry requested review from IAlibay and atravitz August 22, 2025 17:44
@mikemhenry
Copy link
Contributor Author

mikemhenry commented Aug 22, 2025

For example, this release failed in CI conda-forge/openfe-feedstock#98 with E requests.exceptions.HTTPError: 503 Server Error: Service Unavailable for url: https://files.rcsb.org/download/6CZJ.pdb

@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.37%. Comparing base (d0af3f5) to head (3188b21).
⚠️ Report is 97 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1487      +/-   ##
==========================================
- Coverage   95.23%   92.37%   -2.87%     
==========================================
  Files         165      165              
  Lines       12640    12640              
==========================================
- Hits        12038    11676     -362     
- Misses        602      964     +362     
Flag Coverage Δ
fast-tests 92.37% <ø> (?)
slow-tests ?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@atravitz
Copy link
Contributor

worth a shot!

@atravitz atravitz merged commit a0154ef into main Aug 26, 2025
2 of 3 checks passed
@github-actions
Copy link

No API break detected ✅

@atravitz atravitz deleted the bump-num-retry branch August 26, 2025 14:38
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.

4 participants