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

Upgrade OpenShift CNV to a newer version to avoid datavolume name limit of 55 chars #57

Open
moadqassem opened this issue Sep 23, 2020 · 3 comments
Assignees
Labels
area/storage Storage related kind/task General task lifecycle/rotten Nobody worked on this for 12 months (final aging stage) platform/kubevirt Container Native Virtualization (CNV) KubeVirt platform/infrastructure

Comments

@moadqassem
Copy link
Contributor

moadqassem commented Sep 23, 2020

The current cnv version in the Openshift cluster is v2.3.0, which has a cdi data volume webhook that validates the value of the data volume name. If the name of the data volume is longer than 55 characters, the creation of creating/cloning will be rejected thus no proper data volume will be created, thus no virtual machine instance. In order to solve this issue, the cnv should be upgraded to include this kubevirt/containerized-data-importer#1200, which increaseс the limit to 253 instead of 55.

@moadqassem moadqassem added area/storage Storage related platform/kubevirt Container Native Virtualization (CNV) KubeVirt platform/infrastructure priority/normal kind/enhancement Enhancement, improvement, extension labels Sep 23, 2020
@stoyanr stoyanr changed the title Upgrade opensift cnv to a newer version than the current one Upgrade OpenShift CNV to a newer version to avoid datavolume name limit of 55 chars Sep 25, 2020
@stoyanr
Copy link
Contributor

stoyanr commented Sep 25, 2020

/assign @rootfs

@stoyanr
Copy link
Contributor

stoyanr commented Sep 25, 2020

In addition to the issue reported above, we also encountered the following issue with the current CNV version. This is what we see after the first machine that clones a pre-allocated data volume is created:

shoot--dev--openshift-worker1-z-556bbbd4f8-l7khq  Succeeded  100.0% 100s
shoot--dev--openshift-worker1-z-58ca0 ImportInProgress 47.01% 103s

So basically, there is already a cloned datavolume (and a VMI, and a pod, etc.), even though the import of the pre-allocated data volume hasn't even finished. This machine doesn't join the cluster properly, no console to it, no ssh, etc.

In the logs of the launcher pod there are errors such as the following:

{"component":"virt-launcher","kind":"","level":"error","msg":"Failed to sync vmi","name":"shoot--dev--openshift-worker1-z-556bbbd4f8-l7khq","namespace":"default","pos":"server.go:161","reason":"neither found block device nor regular file for volume datavolumedisk","timestamp":"2020-09-25T10:18:33.932558Z","uid":"e65e7ec6-771b-4363-875d-4dc4c298a861"}`

If this first machine is deleted and a second one is created once the pre-allocated data volume import has finished, the cloning process take place and the machine joins the cluster, etc.

We strongly suspect this is yet another issue with the CDI version used in the current CNV version.

@stoyanr
Copy link
Contributor

stoyanr commented Sep 28, 2020

The issue reported above is likely caused by kubevirt/containerized-data-importer#1102

@stoyanr stoyanr added kind/task General task and removed kind/enhancement Enhancement, improvement, extension labels Sep 28, 2020
@gardener-robot gardener-robot added the lifecycle/stale Nobody worked on this for 6 months (will further age) label Nov 28, 2020
@gardener-robot gardener-robot added lifecycle/rotten Nobody worked on this for 12 months (final aging stage) and removed lifecycle/stale Nobody worked on this for 6 months (will further age) labels Sep 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/storage Storage related kind/task General task lifecycle/rotten Nobody worked on this for 12 months (final aging stage) platform/kubevirt Container Native Virtualization (CNV) KubeVirt platform/infrastructure
Projects
None yet
Development

No branches or pull requests

4 participants