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

[skip ci] Increase the number of shards for Cortex-M from 4 to 8. #12334

Merged
merged 1 commit into from
Aug 8, 2022

Conversation

gigiblender
Copy link
Contributor

@gigiblender gigiblender commented Aug 7, 2022

After #11809, the CI time for Cortex-M shards increased to around 2.5 hours. This PR Increases the number of shards for Cortex-M from 4 to 8.

#12333.

@mehrdadh

cc @Mousius @areusch @driazati

@gigiblender gigiblender changed the title [CI] Increase the number of shards for Cortex-M from 4 to 8. [skip ci] Increase the number of shards for Cortex-M from 4 to 8. Aug 8, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Aug 8, 2022

Built docs for commit bb7996a can be found here.

@Mousius
Copy link
Member

Mousius commented Aug 8, 2022

@gigiblender do we know why this failed with the jobs being terminated?

@gigiblender
Copy link
Contributor Author

@gigiblender, do we know why this failed with the jobs being terminated?

Seems to me the CI times out. #11809 adds more tests to the Cortex-M shards.

@Mousius
Copy link
Member

Mousius commented Aug 8, 2022

@gigiblender, do we know why this failed with the jobs being terminated?

Seems to me the CI times out. #11809 adds more tests to the Cortex-M shards.

Do we need to increase the shards further rather than skipping CI here to prevent the timeouts?

@gigiblender
Copy link
Contributor Author

gigiblender commented Aug 8, 2022

@gigiblender, do we know why this failed with the jobs being terminated?

Seems to me the CI times out. #11809 adds more tests to the Cortex-M shards.

Do we need to increase the shards further rather than skipping CI here to prevent the timeouts?

The CI will not run with the updated Jenkinsfile. It still uses 4 shards.

@Mousius
Copy link
Member

Mousius commented Aug 8, 2022

@gigiblender, do we know why this failed with the jobs being terminated?

Seems to me the CI times out. #11809 adds more tests to the Cortex-M shards.

Do we need to increase the shards further rather than skipping CI here to prevent the timeouts?

The CI will not run with the updated Jenkinsfile. It still uses 4 shards.

Haha, my bad, thanks for the reminder 😸

@Mousius Mousius merged commit b4e36c6 into apache:main Aug 8, 2022
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
…12334)

Co-authored-by: Florin-Gabriel Blanaru <fgb@system76-pc.localdomain>
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.

2 participants