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.

Sign up for GitHub

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

Jump to bottom

Swiftv2 e2e tests #2626

Merged

csfmomo merged 50 commits into master from shchenTest0304

May 8, 2024

Contributor

csfmomo commented Mar 5, 2024 •

edited

Loading

Reason for Change:

Add integration test in the pipeline to test against the swiftv2 scenarios.

Issue Fixed:

Requirements:

uses conventional commit messages
includes documentation
adds unit tests
relevant PR labels added

Notes:

csfmomo requested a review from a team as a code owner

March 5, 2024 08:17

csfmomo requested a review from bohuini

March 5, 2024 08:17

csfmomo changed the title ~~[DRAFT] tmp commit to test whether the cluster can be set up.~~ Swiftv2 e2e tests

csfmomo force-pushed the shchenTest0304 branch from 0e41528 to 5ae449c Compare

April 4, 2024 03:32

jaer-tsun reviewed

View reviewed changes

.pipelines/multitenancy/swiftv2-e2e-step-template.yaml Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

.pipelines/templates/create-cluster-swiftv2.yaml Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

.pipelines/templates/create-cluster-swiftv2.yaml Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

hack/aks/Makefile Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/manifests/swiftv2/mtpod1.yaml Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/swiftv2/swiftv2_test.go Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/swiftv2/swiftv2_test.go Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/swiftv2/swiftv2_test.go Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/swiftv2/swiftv2_test.go Outdated Show resolved Hide resolved

jaer-tsun reviewed

View reviewed changes

test/integration/swiftv2/swiftv2_test.go Outdated Show resolved Hide resolved

Member

timraymond commented May 3, 2024

Could you fill out the description for this PR to document the intentions of this change?

csfmomo added 15 commits

May 6, 2024 14:14


          tmp commit to test whether the cluster can be set up.

3a62dc5


          Resolved merge conflicts.

1f6ccc4

Update the route.


          Fix the route.

5d5dd84


          Remove unnecessary dependency.

364b1ee


          Modify the cluster creation.

68fed0a


          Swift v2 e2e test structure.

aedd344


          Add variables.

6f7e24e


          Fix the format and make it a valid yaml.

84af076


          Add datapath tests for swiftv2.

072dae1


          Link the tests with the pipeline steps.

a8450fa


          Update the testing file name.

337b281


          Remove unnecessary comment.

6e03ae1


          Fix the pod name

816a17f


          Sleep before checking the pod.

8ab41ad


          Check the pni after baking time.

f4433d2

csfmomo added 15 commits

May 6, 2024 14:17


          Update the dummy cluster since the previous one has been auto deleted.

1fb9acf


          Update the IP of the pod.

e7611d9


          Update the host IP in the test part too.

c52c5a0


          Separate different test stages.

33588c9


          Use mtpnc to grep IP.

5b1295d


          Update k to kubectl

7be53c1


          Add mtpnc detailed info.

6fd12a8


          Remove the wrong comments.

ff6773b


          Use the script to run the test for now.

4e5ae36


          Remove unnecessary variables.

37e2c14


          Update the cluster creation.

223e713


          Update parameters.

93d4b31


          Remove unnecessary parameters.

07854b0


          Update the testing code part to match the script way.

98b6e42


          Remove unnecessary node part.

05ec1c7

csfmomo force-pushed the shchenTest0304 branch from ef45bd6 to 05ec1c7 Compare

May 6, 2024 21:17

jaer-tsun previously approved these changes

View reviewed changes


          Remove the tests from script part.

89b910f

csfmomo dismissed jaer-tsun’s stale review via

89b910f

May 6, 2024 23:22

jaer-tsun previously approved these changes

View reviewed changes


          Resolve the lint complaint.

e0d94e9

csfmomo dismissed jaer-tsun’s stale review via

e0d94e9

May 7, 2024 04:33


          Resume the "Remove AKS Engine storage account usage"

ef0bfa8

jaer-tsun approved these changes

View reviewed changes


          Merge branch 'master' into shchenTest0304

01a9273

csfmomo enabled auto-merge

May 7, 2024 17:14

csfmomo added this pull request to the merge queue

Merged via the queue into master with commit 893b6e6

91 checks passed

csfmomo deleted the shchenTest0304 branch

May 8, 2024 01:47

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment