Skip to content

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
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