Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Add calico e2e test to aws clusterclass suite #3031

Conversation

codegold79
Copy link
Contributor

@codegold79 codegold79 commented Jul 28, 2022

Signed-off-by: F. Gold fgold@vmware.com, Leo Li lhaoran@vmware.com

What this PR does / why we need it

  • Add Calico CNI E2E tests to AWS_CC suite

Which issue(s) this PR fixes

Fixes #2460

Describe testing done for PR

  • E2E tests passed locally

Release note

Add Calico CNI E2E tests to AWS_CC suite

Additional information

Special notes for your reviewer

@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220728172718/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #3031 (0745c47) into main (2cafeed) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3031      +/-   ##
==========================================
- Coverage   44.45%   44.44%   -0.01%     
==========================================
  Files         417      417              
  Lines       42378    42378              
==========================================
- Hits        18841    18837       -4     
- Misses      21807    21810       +3     
- Partials     1730     1731       +1     
Impacted Files Coverage Δ
addons/controllers/clusterbootstrap_controller.go 63.90% <0.00%> (-1.06%) ⬇️
...ons/controllers/packageinstallstatus_controller.go 79.15% <0.00%> (+1.15%) ⬆️
pkg/v1/tkg/tkgpackageclient/package_update.go 85.00% <0.00%> (+1.42%) ⬆️
addons/controllers/machine_controller.go 68.68% <0.00%> (+3.03%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220728191026/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@codegold79 codegold79 force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 9efdbb4 to 203e8c2 Compare July 28, 2022 23:30
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220728233905/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@codegold79 codegold79 marked this pull request as ready for review July 30, 2022 18:11
@codegold79 codegold79 requested review from a team as code owners July 30, 2022 18:11
@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 203e8c2 to b96a84a Compare August 3, 2022 17:39
@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220803175005/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from b96a84a to 75279cd Compare August 3, 2022 19:36
@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220803194425/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 75279cd to 56a38c6 Compare August 3, 2022 21:29
@github-actions
Copy link

github-actions bot commented Aug 3, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220803213821/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 56a38c6 to 2fa2a11 Compare August 4, 2022 03:12
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220804032302/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 2fa2a11 to 5c7fbf9 Compare August 4, 2022 05:15
@github-actions
Copy link

github-actions bot commented Aug 4, 2022

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220804052501/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo added the ok-to-merge PRs should be labelled with this before merging label Aug 11, 2022
Signed-off-by: F. Gold <fgold@vmware.com>
@haoranleo haoranleo force-pushed the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch from 5c7fbf9 to 0745c47 Compare August 12, 2022 00:00
@github-actions
Copy link

Cluster Generation A/B Results:
https://storage.googleapis.com/tkg-clustergen/3031/20220812000803/clustergen.diff.txt
Author/reviewers:
Please review to verify that the effects on the generated cluster configurations are exactly what the PR intended, and give a thumbs-up if so.

@haoranleo haoranleo merged commit 6b1f162 into main Aug 12, 2022
@haoranleo haoranleo deleted the 2460-aws-e2e-test-verify-calico-cni-is-created-successfully-on-guest-cluster branch August 12, 2022 01:43
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
area/addons area/testing cla-not-required ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

TC11: AWS Cluster - Ability to create CC guest cluster on AWS with calico CNI and verify it is successful
4 participants