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

[BUG] cni_compatible installs obsolete cluster (v1.21 and no cgroup v2 support). Part of cert ? #1965

Closed
collivier opened this issue Apr 11, 2024 · 4 comments
Labels
bug Something isn't working v1.2.0 Issue included in v1.2.0 release

Comments

@collivier
Copy link
Collaborator

Describe the bug

cni_compatible installs a very EOL Kubernetes version (3y old)

INFO -- cnf-testsuite: KindManager#create_cluster(online) stderr: Creating cluster "calico-test" ...
 • Ensuring node image (kindest/node:v1.21.1) 🖼   ...
 ✓ Ensuring node image (kindest/node:v1.21.1) 🖼 
 • Preparing nodes 📦   ...
 ✗ Preparing nodes 📦 

More this kind/kubernetes version fails vs cgroup v2 because of lack of support
ERROR: failed to create cluster: could not find a log line that matches "Reached target .*Multi-User System.*|detected cgroup v1"

Please note that the test keeps failing and doesn't stop after the first issue

I, [2024-04-11 12:53:31 +02:00 #34997]  INFO -- cnf-testsuite: Helm.install output:
I, [2024-04-11 12:53:31 +02:00 #34997]  INFO -- cnf-testsuite: Helm.install stderr: Error: INSTALLATION FAILED: Kubernetes cluster unreachable: stat /home/cedric/.cnf-testsuite/tools/kind/calico-test_admin.conf: no such file or directory

I would consider that cert won't install any additional cluster as it is a big issue out of a lab environment.

To Reproduce

./cnf-testsuite setup 
./cnf-testsuite cnf_setup cnf-config=example-cnfs/coredns
./cnf-testsuite cni_compatible -l debug

Expected behavior

cni_compatible removed from cert and updated to an active kubernetes version.

@collivier collivier added the bug Something isn't working label Apr 11, 2024
@collivier collivier changed the title [BUG] cni_compatible installed obsolete cluster (v1.21 and no cgroup v2 support). Part of cert ? [BUG] cni_compatible installs obsolete cluster (v1.21 and no cgroup v2 support). Part of cert ? Apr 11, 2024
@martin-mat
Copy link
Collaborator

Since cni_compatible is not part of the certification criterions (not "essential"), from my point of view it is ok to be removed from "cert" category.

@agentpoyo
Copy link
Collaborator

I see no issues in removing this from the cert option and from the list of tests for the certification prior to the v1.0.0 release we have planned. Once any bugs are fixed it can be added back, despite not being an essential test, I still think it's a valuable test. It could likely be rewritten and improved.

Thoughts @wavell @denverwilliams @HashNuke ?

martin-mat added a commit to martin-mat/cnf-testsuite that referenced this issue Apr 25, 2024
martin-mat added a commit to martin-mat/cnf-testsuite that referenced this issue Apr 25, 2024
@lixuna lixuna moved this to Peer Review in CNTI Test Catalog Apr 25, 2024
@taylor
Copy link
Collaborator

taylor commented Apr 25, 2024

This test when running with the cert command crashes with an error vs returning PASS/FAIL.

Suggestions on cert call for this test longer term

  • Update test to use current stable K8s releases (eg. v1.29 at this time)
  • Don't start up a new cluster

Suggestions for short term:

  • Remove from list of tests turn with cert command

@taylor
Copy link
Collaborator

taylor commented Apr 25, 2024

@lixuna lixuna added the v1.2.0 Issue included in v1.2.0 release label May 8, 2024
@lixuna lixuna closed this as completed May 21, 2024
@github-project-automation github-project-automation bot moved this from Peer Review to Done in CNTI Test Catalog May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working v1.2.0 Issue included in v1.2.0 release
Projects
Status: Done
Development

No branches or pull requests

5 participants