-
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
Improving Antrea Matrix Compatibility CI #6161
Comments
Currently Antrea matrix test wrt k8s versions with CAPA is still not merged #5476 , in case we want to move to kind based matrix testing for k8 versions, we need to streamline current CPAV matrix jobs as well ! |
The current CAPV matrix job tests are based on Kubernetes v1.17.5 and v1.18.2, so I believe they have not been used for a long time. Before merging the CAPA job PR, I will first attempt to restore the necessary CAPV jobs for matrix tasks. Given that some tests can be covered by Kind testing, we can consider removing them and replacing others with CAPA afterward. |
Overall sounds good to me.
I am not sure I agree with this statement, or maybe I am misunderstanding. We should still validate releases by running a "full" test suite, and periodic CI jobs will not change that. However, I imagine that well-monitored periodic CI jobs will reduce the amount of test failures that have to be investigated and resolved "at the last minute" during the release phase. However, historically, we haven't been good at keeping track of failures in periodic CI jobs and addressing the issues as they arise, as far as I know. Some mechanism that automatically opens a Github issue when a periodic CI job starts failing (e.g., with 2 consecutive failures) could help, but I am not sure how easy it is to set up. |
What's the value in trying to restore / recover these old jobs before introducing the new (Kind / CAPA based) ones? |
There are matrix jobs validating Antrea based on CentOS and Photon OS. Do we have other running jobs that can replace them? If so, we indeed don't need to restore the CAPV matrix jobs before the CAPA upgrade. |
Yes, what I mean is periodic jobs can report failures in time. Regarding job failure notifications, the current Jenkins email notification is disabled due to the migration. I will attempt to enable it first. |
Compatibility Matrix test of antrea with k8 versions for ubuntu is taken care ( CAPA PR #5476 - non kind) , For centos and photon Antrea matrix compatibility tests, kind can be leveraged. Periodic job approach seems to save on resources which should be usually available for regular jobs. |
@XinShuYang you have these 2 list items:
I guess I do not understand how they relate to each other. If you want to enable CAPA jobs for these OS's, why do you need the first list item? |
@XinShuYang "3.Introducing Kind to Ubuntu test cases in Antrea matrix compatibility jobs. This job will automatically trigger periodic runs to ensure the ongoing compatibility of Antrea across different Kubernetes versions and configurations. |
@antoninbas The final goal is to use CAPA instead of CAPV in the matrix test. However, since the PR for CAPA #5476 has not been merged yet (and I think it may take longer for review and merge), I prefer to first restore the existing CAPV jobs. |
@rajnkamr Yes, I created two issues to track the progress, thanks for the suggestion. |
This issue is stale because it has been open 90 days with no activity. Remove stale label or comment, or this will be closed in 90 days |
Move this plan to stage 2 of #6698. |
Describe what you are trying to solve
We are aiming to simplify the Antrea matrix compatibility CI jobs by leveraging Kind and CAPA for testing instead of the existing CAPV setup. This initiative involves recovering outdated Jenkins CI jobs and introducing kind and CAPA to Antrea matrix compatibility tests.
Describe the solution you have in mind
Describe how your solution impacts user flows
Additional context
Any feedback or recommendations are appreciated, as these jobs were created a long time ago, and I may have missed some details. cc @tnqn @antoninbas @luolanzone
The text was updated successfully, but these errors were encountered: