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

We've exceeded the concurrency limit for GitHub Actions #1644

Closed
jessebraham opened this issue May 30, 2024 · 2 comments · Fixed by #1824
Closed

We've exceeded the concurrency limit for GitHub Actions #1644

jessebraham opened this issue May 30, 2024 · 2 comments · Fixed by #1824
Assignees
Labels
CI Continuous integration/deployment

Comments

@jessebraham
Copy link
Member

We are limited to 20 concurrent jobs, meaning that any additional jobs will be queued until a runner is available. This will obviously drag out the total runtime of the workflow, and we should consider restructuring things.

@jessebraham jessebraham added the CI Continuous integration/deployment label May 30, 2024
@jessebraham jessebraham self-assigned this Jun 4, 2024
@jessebraham
Copy link
Member Author

With #1658 merged there are unfortunately now an additional 7 jobs to deal with.

@bjoernQ
Copy link
Contributor

bjoernQ commented Jun 6, 2024

Yes - that's why I initially didn't add CI in #1658

Only thing I can think of would be to handle multiple chips in one job.

e.g. let the xtask take a "chips" argument which populates features and target from that

Downside would be that we don't immediately see for which target the build failed but on the other hand that information is just one click away

@jessebraham jessebraham removed their assignment Jun 24, 2024
@MabezDev MabezDev self-assigned this Jul 19, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in esp-rs Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous integration/deployment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants