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

[ci/cd] Refactor e2e pipeline #459

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Conversation

kang-makes
Copy link
Contributor

@kang-makes kang-makes commented May 24, 2022

This dynamically generates the matrix for the Kubernetes clusters to test while allowing to set the latest version supported instead of using the latest stable.

Fix #438
Fix #438 Minikube still does not support 1.24

@kang-makes kang-makes marked this pull request as draft May 24, 2022 11:31
@kang-makes kang-makes force-pushed the feature/e2e-kubernetes-1-24 branch 24 times, most recently from 1e5f256 to a001ebd Compare May 25, 2022 15:42
@kang-makes kang-makes force-pushed the feature/e2e-kubernetes-1-24 branch from a001ebd to dc97fa2 Compare May 25, 2022 15:43
@kang-makes kang-makes marked this pull request as ready for review May 25, 2022 16:04
@kang-makes kang-makes requested a review from a team May 25, 2022 16:05
@kang-makes
Copy link
Contributor Author

There is a missing metrics that got deprecated in 1.21 that is missing in 1.23 and is breaking these tests:

time="2022-05-25T15:56:01Z" level=fatal msg="after 5 attempts, last errors: [finding Metric: k8s.apiserver.etcd.objectCount_*]"

Fixing integration tests should fix this issue #460

@kang-makes kang-makes marked this pull request as draft May 26, 2022 08:11
Comment on lines +61 to +62
cri:
- containerd
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not happy about stopping to test with docker. It might be a mess to check which of the versions actually support docker, so perhaps for now we could just hardcode it here?

- ${{ needs.generate-kubernetes-versions.outputs.previous_1 }}
- ${{ needs.generate-kubernetes-versions.outputs.previous_2 }}
- ${{ needs.generate-kubernetes-versions.outputs.previous_3 }}
- ${{ needs.generate-kubernetes-versions.outputs.previous_4 }}
Copy link
Contributor

Choose a reason for hiding this comment

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

I'd add, even if hardcoded, the earliest version we support to ensure we do not break anything for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update e2e tests running on CI/CD to support v1.23
2 participants