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

Enabled k8s conformance test #118

Merged
merged 24 commits into from
Aug 30, 2022
Merged

Conversation

deepakm-ntnx
Copy link
Contributor

@deepakm-ntnx deepakm-ntnx commented Aug 26, 2022

What this PR does / why we need it:

  • This test enable other tests which are already checked in and fail. Hence this is critical to merge
  • Added conformance.yaml and conformance test is passing
  • Added upgrades flavor related yaml
  • Updated configuration parameters required for upgrades to work
  • clusterctl upgrade and cluster upgrade is partially progressing

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

How Has This Been Tested?:

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration and test output

Special notes for your reviewer:
Testing passed for following tests:

~ make test-e2e LABEL_FILTERS=conformance
~ make list-e2e

Currently following tests fail due to version of the provider issues. we dont need this at this time as we only support v1beta1 when we release.

make test-e2e LABEL_FILTERS=clusterctl_upgrade

this one fails with following error

make test-e2e LABEL_FILTERS=md-scale-out-in
  Timed out after 300.001s.
  Timed out waiting for Cluster md-scale-8oeu8m/md-scale-iai4de to provision
  Expected
      : Provisioning
  to equal
      : Provisioned
  In [It] at: /Users/deepak.muley/go/pkg/mod/sigs.k8s.io/cluster-api/test@v1.2.0-beta.0.0.20220823125924-6e30f66cc3df/framework/cluster_helpers.go:144

  Full Stack Trace

cluster-upgrade-conformance fails but so far dont see our provider issue. will continue to debug or later disable

make test-e2e LABEL_FILTERS=cluster-upgrade-conformance
      failed to init node with kubeadm: command "docker exec --privileged test-zvav8o-control-plane kubeadm init --skip-phases=preflight --config=/kind/kubeadm.conf --skip-token-print --v=6" failed with error: exit status 1
  occurred
  In [SynchronizedBeforeSuite] at: /Users/deepak.muley/go/pkg/mod/sigs.k8s.io/cluster-api/test@v1.2.0-beta.0.0.20220823125924-6e30f66cc3df/framework/bootstrap/kind_provider.go:170

  Full Stack Trace
    sigs.k8s.io/cluster-api/test/framework/bootstrap.(*KindClusterProvider).createKindCluster(0xc00070c2d0)
    	/Users/deepak.muley/go/pkg/mod/sigs.k8s.io/cluster-api/test@v1.2.0-beta.0.0.20220823125924-6e30f66cc3df/framework/bootstrap/kind_provider.go:170 +0x866
    sigs.k8s.io/cluster-api/test/framework/bootstrap.(*KindClusterProvider).Create(0xc00070c2d0, {0x321b528?, 0xc000112c40?})
    	/Users/deepak.muley/go/pkg/mod/sigs.k8s.io/cluster-api/test@v1.2.0-beta.0.0.20220823125924-6e30f66cc3df/framework/bootstrap/kind_provider.go:118 +0x1aa
    sigs.k8s.io/cluster-api/test/framework/bootstrap.CreateKindBootstrapClusterAndLoadImages({0x321b528?, 0xc000112c40}, {{0xc0009626b0, 0xb}, {0xc000962620, 0x7}, 0x0, {0xc000125800, 0x4, 0x4}, ...})
    	/Users/deepak.muley/go/pkg/mod/sigs.k8s.io/cluster-api/test@v1.2.0-beta.0.0.20220823125924-6e30f66cc3df/framework/bootstrap/kind_util.go:81 +0x634
    github.com/nutanix-cloud-native/cluster-api-provider-nutanix/test/e2e.setupBootstrapCluster(0xc0001b5950, 0x20?, 0x0?)
    	/Users/deepak.muley/go/src/github.com/deepakm-ntnx/cluster-api-provider-nutanix/test/e2e/e2e_suite_test.go:222 +0x398
    github.com/nutanix-cloud-native/cluster-api-provider-nutanix/test/e2e.glob..func9()
    	/Users/deepak.muley/go/src/github.com/deepakm-ntnx/cluster-api-provider-nutanix/test/e2e/e2e_suite_test.go:139 +0x3bf

we can test it on prow setup

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:


@deepakm-ntnx deepakm-ntnx changed the title [WIP] E2e Enabled k8s conformance test Aug 30, 2022
test/e2e/nutanix_client_test.go Outdated Show resolved Hide resolved
test/e2e/projects_test.go Outdated Show resolved Hide resolved
test/e2e/multi_nic_test.go Outdated Show resolved Hide resolved
test/e2e/ccm_test.go Outdated Show resolved Hide resolved
test/e2e/categories_test.go Outdated Show resolved Hide resolved
yannickstruyf3
yannickstruyf3 previously approved these changes Aug 30, 2022
yannickstruyf3
yannickstruyf3 previously approved these changes Aug 30, 2022
@deepakm-ntnx deepakm-ntnx merged commit 79ed097 into nutanix-cloud-native:main Aug 30, 2022
@deepakm-ntnx deepakm-ntnx deleted the e2e branch August 30, 2022 17:36
@tuxtof tuxtof added the ignore-for-release Internal change to not include in Release Notes label Sep 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ignore-for-release Internal change to not include in Release Notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants