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

ci: Set apt to retry operations on failure #25720

Merged
merged 1 commit into from Apr 19, 2023
Merged

Conversation

QuLogic
Copy link
Member

@QuLogic QuLogic commented Apr 18, 2023

PR Summary

At some point, Azure servers were having a small hiccup, and I noticed this setup in some unrelated repository. Allowing a couple retries may help with minor issues like those.

PR Checklist

Documentation and Tests

  • [n/a] Has pytest style unit tests (and pytest passes)
  • [n/a] Documentation is sphinx and numpydoc compliant (the docs should build without error).
  • [n/a] New plotting related features are documented with examples.

Release Notes

  • [n/a] New features are marked with a .. versionadded:: directive in the docstring and documented in doc/users/next_whats_new/
  • [n/a] API changes are marked with a .. versionchanged:: directive in the docstring and documented in doc/api/next_api_changes/
  • [n/a] Release notes conform with instructions in next_whats_new/README.rst or next_api_changes/README.rst

At some point, Azure servers were having a small hiccup, and I noticed
this setup in some unrelated repository. Allowing a couple retries _may_
help with minor issues like those.
@QuLogic QuLogic added the CI: testing CI configuration and testing label Apr 18, 2023
@QuLogic QuLogic added this to the v3.8.0 milestone Apr 18, 2023
@ksunden
Copy link
Member

ksunden commented Apr 18, 2023

Worth trying, I think, though will be hard to tell if it actually helps, given the stochastic nature of the failures (and I suspect that even with retrying, some of the failures will persist as it is rare for just one job to fail, and I doubt all of the jobs are installing at just the right time)

@oscargus oscargus merged commit 540a328 into matplotlib:main Apr 19, 2023
38 of 40 checks passed
@QuLogic QuLogic deleted the apt-retry branch April 19, 2023 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI: testing CI configuration and testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants