Skip to content
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: Add node restart to cniv2 E2E #2901

Merged
merged 6 commits into from
Aug 17, 2024
Merged

ci: Add node restart to cniv2 E2E #2901

merged 6 commits into from
Aug 17, 2024

Conversation

jpayne3506
Copy link
Contributor

@jpayne3506 jpayne3506 commented Aug 7, 2024

Reason for Change:

Adds test coverage to validate CNI binary install occurs correctly on all scenarios. Additionally, refreshes current E2E to remove duplicate testing, pull out windows nodepool creation, and leverages existing makefile commands for validation testing.

Issue Fixed:

Contributes to CI/CD to cover #2818

Requirements:

Notes:

@jpayne3506 jpayne3506 requested a review from a team as a code owner August 7, 2024 00:17
@jpayne3506 jpayne3506 requested a review from rayaisaiah August 7, 2024 00:17
@jpayne3506 jpayne3506 changed the title ci: add node restart to cniv2 E2E ci: Add node restart to cniv2 E2E Aug 7, 2024
@jpayne3506 jpayne3506 force-pushed the jpayne3506/cns-cicd branch from 5c031b3 to cc07553 Compare August 12, 2024 16:02
@jpayne3506 jpayne3506 force-pushed the jpayne3506/cns-cicd branch from 77d7cfb to 443cde8 Compare August 14, 2024 17:36
@jpayne3506 jpayne3506 added ci Infra or tooling. release/1.4 Change affects v1.4 release train needs-backport Change needs to be backported to previous release trains release/1.5 Change affects v1.5 release train labels Aug 15, 2024
@jpayne3506 jpayne3506 self-assigned this Aug 15, 2024
@jpayne3506
Copy link
Contributor Author

Will address general pipeline testing not utilizing set -e in #2929.

@jpayne3506 jpayne3506 added this pull request to the merge queue Aug 16, 2024
github-merge-queue bot pushed a commit that referenced this pull request Aug 16, 2024
* ci: add node restart to cniv2 E2E

* chore: change directory after scale

* chore: fixup nodepool

* chore: pipeline cleanup

* ci: cleanup test resources

* chore: address comments
Merged via the queue into master with commit 742ca41 Aug 17, 2024
92 checks passed
@jpayne3506 jpayne3506 deleted the jpayne3506/cns-cicd branch August 17, 2024 08:06
jpayne3506 added a commit that referenced this pull request Aug 20, 2024
* ci: add node restart to cniv2 E2E

* chore: change directory after scale

* chore: fixup nodepool

* chore: pipeline cleanup

* ci: cleanup test resources

* chore: address comments
jpayne3506 added a commit that referenced this pull request Aug 20, 2024
* ci: add node restart to cniv2 E2E

* chore: change directory after scale

* chore: fixup nodepool

* chore: pipeline cleanup

* ci: cleanup test resources

* chore: address comments
github-merge-queue bot pushed a commit that referenced this pull request Aug 21, 2024
* ci: Add node restart to cniv2 E2E (#2901)

* ci: add node restart to cniv2 E2E

* chore: change directory after scale

* chore: fixup nodepool

* chore: pipeline cleanup

* ci: cleanup test resources

* chore: address comments

* [FIX] ci: Update AzCLI task to v2 (#2922)

* ci: Update AzCLI task to v2

* ci: add scriptType: "bash"

* ci: Shift E2E cluster creation (#2862)

* ci: shift cluster creation

* chore: change depends on for release test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Infra or tooling. needs-backport Change needs to be backported to previous release trains release/1.4 Change affects v1.4 release train release/1.5 Change affects v1.5 release train
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants