-
Notifications
You must be signed in to change notification settings - Fork 368
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
Run Multiple Kind CI Jobs on a single testbed #6010
Conversation
5b66784
to
5672557
Compare
1.Estimated size required for a single job run is ~4gb, after each job run success/failure, custom image must be cleaned to allow other jobs to start. ( It is standard practice ) |
5672557
to
b671b5a
Compare
We can't say that the estimated size for the job is 4gb, it is actually double the size, because we will have multiple images getting downloaded for test, then we will have kubernetes cluster, and different containers running as well. When I mentioned size 4GB, it was the size that was occupied on my local docker desktop, by the antrea images. |
b671b5a
to
33bc506
Compare
For 2 & 3, need to have actual space identified for a single job to decide on no of kind test beds. |
but if we will have more testbed, then what will be the purpose of multiple kind jobs on single testbed, we can either leverage one testbed with more resources. |
It will depend on max allowed space we can reserve for a kind vm, even if run ci jobs in parallel , we might not be allowed to reserve a powerful vm which can run all ci jobs in parallel , however benefit is that we would reduce no of vms needed for running kind ci tests. |
33bc506
to
b9dcae3
Compare
b9dcae3
to
6e73ae9
Compare
6e73ae9
to
e0e30e5
Compare
LGTM, I will wait for @KMAnju-2021 's confirmation regarding the jenkins kind verification before approving. Please ensure that all related Jenkins jobs can pass. |
80581a5
to
e077b17
Compare
/test-rancher-networkpolicy |
/test-rancher-conformance |
e077b17
to
fbd9286
Compare
fbd9286
to
abf7bef
Compare
Signed-off-by: Pulkit Jain <[email protected]>
abf7bef
to
0a6246a
Compare
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.
LGTM
@XinShuYang can you take one last look please
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.
LGTM, for kind CI changes, @KMAnju-2021 do you have more comments?
Hi @XinShuYang, I don't have any comments, thanks ! |
/test-kind-conformance |
I don't see any status for |
@KMAnju-2021 can you check the trigger comments. |
/test-kind-networkpolicy |
Hi @antoninbas, fixed |
Run Multiple kind Jobs on same testbed.
Fixes #5734