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

Frequent failures in Github Actions #214

Closed
bugadani opened this issue Mar 23, 2023 · 7 comments
Closed

Frequent failures in Github Actions #214

bugadani opened this issue Mar 23, 2023 · 7 comments
Labels
bug Something isn't working

Comments

@bugadani
Copy link
Contributor

Sometimes my GHA runs fail with the following error message:
thread 'main' panicked at 'called Result::unwrap()on anErr value: Error { error: GithubQuery, total_delay: 500ms, tries: 6 }', src/toolchain/mod.rs:165:6

I imagine that if something fails twice in a row in 100ms, it won't likely succeed in the next .3s either. What would you think of

  • increasing the retry count a bit and
  • moving to exponential backoff?
@bugadani bugadani added the bug Something isn't working label Mar 23, 2023
@SergioGasquez
Copy link
Member

Hi!
A few questions:

As you mention, maybe the retry strategy is not the best, I'll have a deeper look at your suggestions.

@bugadani
Copy link
Contributor Author

bugadani commented Mar 23, 2023

No, I am not using xtensa-toolchain. I have the espup binary cached and I espup install during each of my CI runs. Though, maybe I could cache whatever espup installs, too, but that's a bit of a tangent here.

Sorry, the repo is private. "Frequent" might be an overstatement on my part, but its still a noticeable failure rate. Thanks for the GITHUB_TOKEN suggestion, I think I might indeed want to set that. It might not be a bad idea to include that in this repo's README, too, I think.

@SergioGasquez
Copy link
Member

Is there any reason why you don't want to add GITHUB_TOKEN and/or use xtensa-toolchain?

Regarding updating the Readme, I think it is a good idea to add a section explaining how to use espup in CI, thanks for the suggestion!

@bugadani
Copy link
Contributor Author

Is there any reason why you don't want to add GITHUB_TOKEN

Oh my, that was a bad slip of the finger on my part. I wanted to say that I want to do it, and I have done it since then.

@SergioGasquez
Copy link
Member

Oh my, that was a bad slip of the finger on my part. I wanted to say that I want to do it, and I have done it since then.

Great, I'll close this issue since the error should now disappear, but feel free to reopen this issue if it reappears. Here is some more information about the issue esp-rs/xtensa-toolchain#15.

@bugadani
Copy link
Contributor Author

Thanks!

@SergioGasquez
Copy link
Member

Just added a small note to clarify this: ef483ed

@bugadani bugadani changed the title Frequest failures in Github Actions Frequent failures in Github Actions Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants