Move jenkins/ dir into ci/jenkins and spread docs around #11927
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
eebab38 to
70dc4fc
Compare
driazati
approved these changes
Jun 28, 2022
| against Windows and MacOS using GitHub Actions. | ||
|
|
||
| This page describes how contributors and committers can use TVM's CI to verify their code. You can | ||
| read more about the design of TVM CI in the |
|
|
||
| ## Jenkins | ||
|
|
||
| Jenkins runs all of the linux-based TVM CI-enabled regression tests. This includes tests against accelerated hardware such as GPUs. It excludes those regression tests that run against hardware not available in the cloud (those tests aren't currently exercised in TVM CI). The tests run by Jenkins represent most of the merge-blocking tests (and passing Jenkins should mostly correlate with passing the remaining Windows/Mac builds). |
There was a problem hiding this comment.
Suggested change
| Jenkins runs all of the linux-based TVM CI-enabled regression tests. This includes tests against accelerated hardware such as GPUs. It excludes those regression tests that run against hardware not available in the cloud (those tests aren't currently exercised in TVM CI). The tests run by Jenkins represent most of the merge-blocking tests (and passing Jenkins should mostly correlate with passing the remaining Windows/Mac builds). | |
| Jenkins runs all of the Linux-based TVM CI-enabled regression tests. This includes tests against accelerated hardware such as GPUs. It excludes those regression tests that run against hardware not available in the cloud (those tests aren't currently exercised in TVM CI). The tests run by Jenkins represent most of the merge-blocking tests (and passing Jenkins should mostly correlate with passing the remaining Windows/Mac builds). |
| Some tests are also flaky and occasionally fail for reasons unrelated to the PR. The [CI monitoring | ||
| rotation](https://github.com/apache/tvm/wiki/CI-Monitoring-Runbook) watches for these failures and | ||
| disables tests as necessary. It is the responsibility of those who wrote the test to ultimately fix | ||
| and re-enable the test. | ||
|
|
||
|
|
||
| ## Dealing with Flakiness |
There was a problem hiding this comment.
This stuff should be either in .rst or in .md, can you delete one or the other?
There was a problem hiding this comment.
oh whoops, this was pretty sloppy. done.
ci/jenkins/README.md
Outdated
| @@ -153,6 +160,7 @@ _venv/bin/python3 jenkins/generate.py | |||
|
|
|||
| # Infrastructure | |||
|
|
|||
| While all TVM tests are contained within the apache/tvm repository, the infrastructure used to run the tests is donated by the TVM Community. To encourage collaboration, | |||
ci/jenkins/README.md
Outdated
| @@ -153,6 +160,7 @@ _venv/bin/python3 jenkins/generate.py | |||
|
|
|||
| # Infrastructure | |||
|
|
|||
| While all TVM tests are contained within the apache/tvm repository, the infrastructure used to run the tests is donated by the TVM Community. To encourage collaboration, | |||
| Jenkins runs in AWS on an EC2 instance fronted by an ELB which makes it available at https://ci.tlcpack.ai. These definitions are declared via Terraform in the [tlc-pack/ci-terraform](https://github.com/tlc-pack/ci-terraform) repository. The Terraform code references custom AMIs built in [tlc-pack/ci-packer](https://github.com/tlc-pack/ci-packer). [tlc-pack/ci](https://github.com/tlc-pack/ci) contains Ansible scripts to deploy the Jenkins head node and set it up to interact with AWS. | |||
|
|
|||
| The Jenkins head node has a number of autoscaling groups with labels that are used to run jobs (e.g. `CPU`, `GPU` or `ARM`) via the [EC2 Fleet](https://plugins.jenkins.io/ec2-fleet/) plugin. | |||
There was a problem hiding this comment.
the tlcpack stuff is merged now, can you delete the diagram from here?
70dc4fc to
72af6cf
Compare
72af6cf to
80b3377
Compare
blackkker
pushed a commit
to blackkker/tvm
that referenced
this pull request
Jul 7, 2022
masahi
pushed a commit
to masahi/tvm
that referenced
this pull request
Jul 15, 2022
mikeseven
pushed a commit
to mikeseven/tvm
that referenced
this pull request
Sep 27, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Reorganizing some of the Jenkins docs to make it easier to present.
jenkins/README.md->ci/README.mdand retain the overview sectiontlc-pack/ciso that they live along with that impl.@driazati @konturn