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.

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

ci: v4overlay conformance test cases #2274

Merged
merged 8 commits into from
Oct 5, 2023
Merged

ci: v4overlay conformance test cases #2274

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
2ef2167
v4overlay conformance test cases
paulyufan2 Oct 3, 2023
9e3f869
add hostPort test case
paulyufan2 Oct 3, 2023
8d576ee
Merge branch 'master' into v4overlayConformanceTest
paulyufan2 Oct 4, 2023
4f0cfbd
[CNI] zap logger telemetry package (#2266)
paulyufan2 Oct 4, 2023
99923c9
ci: Agent pool default set at stage level (#2272)
jpayne3506 Oct 4, 2023
820c861
fix issue
paulyufan2 Oct 4, 2023
85f3875
Merge branch 'master' into v4overlayConformanceTest
paulyufan2 Oct 4, 2023
2d1dba7
Merge branch 'master' into v4overlayConformanceTest
paulyufan2 Oct 5, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .pipelines/pipeline.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -379,6 +379,7 @@ stages:
parameters:
name: "azure_overlay_e2e"
displayName: Azure Overlay
os: linux
testDropgz: ""
clusterType: overlay-byocni-up
clusterName: "azureoverlaye2e"
Expand All @@ -390,6 +391,7 @@ stages:
parameters:
name: "azure_overlay_cni_e2e"
displayName: CNI - Azure Overlay
os: linux
testDropgz: true
clusterType: overlay-byocni-up
clusterName: "azureoverlaycnie2e"
Expand Down
27 changes: 27 additions & 0 deletions .pipelines/singletenancy/azure-cni-overlay/azure-cni-overlay-e2e-job-template.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,14 +60,41 @@ stages:
testDropgz: ${{ parameters.testDropgz }}
clusterName: ${{ parameters.clusterName }}-$(commitID)

- template: ../../cni/k8s-e2e/k8s-e2e-job-template.yaml
parameters:
sub: $(BUILD_VALIDATIONS_SERVICE_CONNECTION)
clusterName: ${{ parameters.clusterName }}-$(commitID)
os: ${{ parameters.os }}
dependsOn: ${{ parameters.name }}
dns: true
portforward: true
hostport: true
service: true

- template: ../../cni/k8s-e2e/k8s-e2e-job-template.yaml
parameters:
sub: $(BUILD_VALIDATIONS_SERVICE_CONNECTION)
clusterName: ${{ parameters.clusterName }}-$(commitID)
os: windows
dependsOn: cni_${{ parameters.os }}
dns: true
portforward: true
paulyufan2 marked this conversation as resolved.
Show resolved Hide resolved
hostport: true
service: true
hybridWin: true
datapath: true

- job: logs
displayName: "Log Failure"
dependsOn:
- ${{ parameters.name }}
- cni_${{ parameters.os }}
- cni_windows
condition: failed()
steps:
- template: ../../templates/log-template.yaml
parameters:
clusterName: ${{ parameters.clusterName }}-$(commitID)
os: ${{ parameters.os }}
cni: cniv2

23 changes: 0 additions & 23 deletions .pipelines/singletenancy/azure-cni-overlay/azure-cni-overlay-e2e-step-template.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,20 +41,6 @@ steps:
name: "installKubectl"
displayName: "Install Kubectl"

- script: |
echo "install kubetest2 and gsutils"
go get github.com/onsi/ginkgo/ginkgo
go get github.com/onsi/gomega/...
go install github.com/onsi/ginkgo/ginkgo@latest
go install sigs.k8s.io/kubetest2@latest
go install sigs.k8s.io/kubetest2/kubetest2-noop@latest
go install sigs.k8s.io/kubetest2/kubetest2-tester-ginkgo@latest
wget https://storage.googleapis.com/pub/gsutil.tar.gz
tar xfz gsutil.tar.gz
sudo mv gsutil /usr/local/bin
name: "installKubetest"
displayName: "Set up Conformance Tests"

- ${{ if eq( parameters['testDropgz'], true) }}:
- script: |
echo "##vso[task.setvariable variable=dropgzVersion]$(make cni-dropgz-test-version)"
Expand Down Expand Up @@ -141,15 +127,6 @@ steps:
pathtoPublish: "$(Build.ArtifactStagingDirectory)/test-output"
condition: always()

- script: |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make sure that those 16 service conformance tests are covered when we are using k8s e2e template. (since we were using the latest image for kubetest2)

echo "Run Service Conformance E2E"
export PATH=${PATH}:/usr/local/bin/gsutil
KUBECONFIG=~/.kube/config kubetest2 noop \
--test ginkgo -- \
--focus-regex "Services.*\[Conformance\].*"
name: "servicesConformance"
displayName: "Run Services Conformance Tests"

- script: |
echo "Run wireserver and metadata connectivity Tests"
bash test/network/wireserver_metadata_test.sh
Expand Down
Loading