-
Notifications
You must be signed in to change notification settings - Fork 242
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
ci: Replace k8s conformance tests within aks-swift CI #2590
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
aggarwal0009
previously approved these changes
Feb 14, 2024
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
777c0d9
to
70bab7f
Compare
aggarwal0009
approved these changes
Feb 14, 2024
estebancams
reviewed
Feb 15, 2024
- job: failedE2ELogs | ||
displayName: "Failure Logs" | ||
dependsOn: | ||
- ${{ parameters.name }} | ||
- cni_linux |
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.
Should this be also parametrized by OS? (since the template as a whole is receiving os parameter now)
jpayne3506
added a commit
that referenced
this pull request
Mar 1, 2024
* ci:[CNI] ACN PR Pipeline speed improvements (#2077) * Move Create Cluster * Lower Windows scaleup * Delete echo-external-node for Cilium validate * Change cluster creation from job to stage * Decrease k8utils retry delay time * Adjust submodule pipeline * Remove test comments * Addressing Comments * Add delete cluster template * Single Stage Delete * ACN merge commitID fix * Addressing Comments * Refactor dualstack from #2098 * ci: Changing Service Connection for PR pipeline (#2153) * ci: change service connection * add: change ACN PR azureSubscription * ci: Replace manual install of kubectl with ADO KubectlInstaller task (#2307) * ci: remove kubectl install * ci: add KubectlInstaller for kubectl * ci: Add k8s conformance tests to Cilium CI/CD (#2348) * ci: add k8s conformance * ci: remove kubetest2 k8se2e * chore: remove test comments * ci: add k8se2e to nightly pipeline * ci: Replace k8s conformance tests within aks-swift CI (#2590) ci: k8se2e for aks-swift * ci: v4overlay conformance test cases (#2274) v4overlay conformance test cases * fix: Match Cilium CLI to Cilium Agent (#2365) * fix: Match Cilium CLI to Cilium Agent * ci: Capture single digit versions * ci: Update makefile cluster creation (#2358) ci: Update aks cluster creation --------- Co-authored-by: Paul Yu <129891899+paulyufan2@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
ci
Infra or tooling.
release/latest
Change affects latest release train
release/1.4
Change affects v1.4 release train
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.
Reason for Change:
Replaces kubetest2 to run k8s conformances tests with k8se2e job templates that use exclusively ginkgo.
Issue Fixed:
Requirements:
Notes:
Needs to be backported to support future CI runs