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
Add a retry option on getting builds for a train #9331
Conversation
Generated by 🚫 Danger |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🐑 Nice one!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome, extra 👍 for adding a test
Hey @ohayon 👋 Thank you for your contribution to fastlane and congrats on getting this pull request merged 🎉 Please let us know if this change requires an immediate release by adding a comment here 👍 |
Congratulations! 🎉 This was released as part of fastlane 2.37.0 🚀 |
1 similar comment
Congratulations! 🎉 This was released as part of fastlane 2.37.0 🚀 |
Sometimes intermittent 502s happen while the build watcher is waiting for the build to finish processing and show up in the list of available builds. We shouldnt fail the build watching process for that, we should just try again a couple of times to continue to wait for the build
I tested this locally but can't fudge a 500 in the middle of the waiting process, so assuming no 500's come in, this will continue to work as it always has.