-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Documentation: Update list of Jenkins jobs #14592
Conversation
| `Ginkgo-CI-Tests-k8s1.13-Pipeline <https://jenkins.cilium.io/job/Ginkgo-CI-Tests-k8s1.13-Pipeline/>`_ | test-missed-k8s | No | | ||
+----------------------------------------------------------------------------------------------------------------+-------------------+--------------------+ | ||
| `Cilium-Ginkgo-Test-k8s <https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-k8s/>`_ | test-missed-k8s | Yes | | ||
| `Ginkgo-CI-Tests-k8s1.20-Pipeline <https://jenkins.cilium.io/job/Ginkgo-CI-Tests-k8s1.20-Pipeline/>`_ | test-missed-k8s | Yes | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this doesn't look right... test-missed-k8s
runs k8s versions from 1.14-1.19
@@ -194,29 +194,27 @@ illustrating which subset of tests the job runs. | |||
+----------------------------------------------------------------------------------------------------------------+-------------------+--------------------+ | |||
| Jenkins Job | Trigger Phrases | Required To Merge? | | |||
+================================================================================================================+===================+====================+ | |||
| `K8s-1.20-kernel-4.9 <https://jenkins.cilium.io/job/Cilium-PR-K8s-newest-kernel-4.9/>`_ | test-me-please, | Yes | | |||
| `Cilium-PR-K8s-1.19-kernel-4.9 <https://jenkins.cilium.io/job/Cilium-PR-K8s-1.19-kernel-4.9/>`_ | test-me-please, | Yes | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't look right as well, test-me-please
runs k8s 1.20 + kernel 4.9, k8s 1.19 + kernel 4.19 and k8s 1.13 + kernel net-next
Signed-off-by: Tom Payne <tom@isovalent.com>
d4a641b
to
59f6c09
Compare
This should now be correct with respect to the trigger phrases too. I've removed the pipeline jobs from the list as they are run periodically rather than being triggered by PRs. |
This updates the list of Jenkins jobs that can be triggered with trigger phrases in the documentation to more closely match the jobs currently running in Jenkins.
I'm not sure if the trigger phrases are correct.