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

Separate out upgrade specs into its own job #1869

Closed
shysank opened this issue Nov 16, 2021 · 4 comments
Closed

Separate out upgrade specs into its own job #1869

shysank opened this issue Nov 16, 2021 · 4 comments
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@shysank
Copy link
Contributor

shysank commented Nov 16, 2021

/kind feature

Describe the solution you'd like
With new upgrade tests added as part of #1810 and #1771 capi e2e spec is now taking ~2.5 hrs to complete. This is undesirable. #1816 tried to address this by increasing ginkgo nodes count, but it increased the percentage of flakes, and we are still working on it find the root cause.

In the interim, to reduce the time taken to run capi e2e tests, we want to split the spec into different jobs so that they can be run simultaneously. We want something like this:

  1. Remove kcp upgrades and merge with cluster upgrade tests. v1alpha4 -> v1beta1 clusterctl upgrade test #1810 (comment)
  2. Extract clusterctl upgrade tests (v1alpha3->v1beta1, v1alpha4->v1beta1) into a separate spec
  3. Keep the remaining capi e2e spec as it is.

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]

Environment:

  • cluster-api-provider-azure version:
  • Kubernetes version: (use kubectl version):
  • OS (e.g. from /etc/os-release):
@k8s-ci-robot k8s-ci-robot added the kind/feature Categorizes issue or PR as related to a new feature. label Nov 16, 2021
@CecileRobertMichon
Copy link
Contributor

CecileRobertMichon commented Nov 16, 2021

I took care of 1 in #1767

@shysank
Copy link
Contributor Author

shysank commented Jan 7, 2022

@CecileRobertMichon can we close this one?

@CecileRobertMichon
Copy link
Contributor

Yes!

/close

@k8s-ci-robot
Copy link
Contributor

@CecileRobertMichon: Closing this issue.

In response to this:

Yes!

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

3 participants