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

kubeadm: run tests with sysext images #524

Merged
merged 3 commits into from
Apr 16, 2024
Merged

Conversation

tormath1
Copy link
Contributor

@tormath1 tormath1 commented Apr 12, 2024

In this PR, we test Kubernetes using the sysext images from the sysext-bakery. It eases the test writing as we remove the installation of Kubernetes components and we are actually testing sysext images that are being used in other projects (Kubernetes deployments and CAPI deployments).

Changes:

  • use systemd sysext instead of binaries
  • convert CLC to Butane configuration
  • remove useless variables
  • remove 'checksum.sh'
  • enable calico and cilium CNI tests on Brightbox (they were deactivated due to CLC limitation)
  • sync Kubernetes version with available sysext images for 1.27 and 1.28

Testing done

$ cat _kola_temp/brightbox-latest/test.tap
1..3
ok - kubeadm.v1.27.6.calico.cgroupv1.base
ok - kubeadm.v1.28.7.flannel.base
ok - kubeadm.v1.29.2.cilium.base

Notes for reviewers

  • As usual testdata content is generated from templates.go, no need to check the content of this folder, only templates.go is important

@tormath1 tormath1 self-assigned this Apr 12, 2024
@tormath1 tormath1 marked this pull request as ready for review April 12, 2024 15:48
@tormath1 tormath1 requested a review from a team April 12, 2024 15:48
Copy link
Member

@pothos pothos left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!
We could rerun the Kubernetes test on all platforms or let the nightly do it next time.

Do you think we should have a clilium and help sysext image? These two are downloaded as plain binaries now and it might be good to keep them up-to-date, not so important for this test but it might be helpful for users.

* use systemd sysext instead of binaries
* convert CLC to Butane configuration
* remove useless variables
* remove 'checksum.sh'

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
This reverts commit bced13d.

Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
Signed-off-by: Mathieu Tortuyaux <mtortuyaux@microsoft.com>
@tormath1
Copy link
Contributor Author

Force pushed to update the testdata to be more consistent with reality. Will wait for the release to be done before merging this.

@tormath1 tormath1 merged commit 5e0f168 into flatcar-master Apr 16, 2024
3 checks passed
@tormath1 tormath1 deleted the tormath1/kubernetes branch April 16, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants