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

openstack-volume driver unable to find virtio-scsi devices. #4797

Closed
shannonmitchell opened this issue May 20, 2020 · 4 comments
Closed

openstack-volume driver unable to find virtio-scsi devices. #4797

shannonmitchell opened this issue May 20, 2020 · 4 comments
Labels
sig/cloud-provider Categorizes an issue or PR as relevant to SIG Cloud Provider.

Comments

@shannonmitchell
Copy link

shannonmitchell commented May 20, 2020

The issue is with this device matching code in the openstack_volumes.go file: https://gitlab.dev.cncf.ci/kubernetes/kubernetes/blob/6c7afd1222ebf2cb91567364471d74ac056437b7/pkg/cloudprovider/providers/openstack/openstack_volumes.go#L483

It is limiting the volumeid to 20 characters when looking for /dev/disk/by-id/scsi-0QEMU_QEMU_HARDDISK_ devices. We are seeing the full volume id within the OS causing the match to fail against the check with the 20 char limit. The result is the following error:

W0520 16:21:31.842907 17689 openstack_volumes.go:561] Failed to find device for the volumeID: "e4bdcdb3-c858-48d3-86b1-52a92d05ae3b" E0520 16:21:31.842937 17689 attacher.go:246] Error: could not find attached Cinder disk "e4bdcdb3-c858-48d3-86b1-52a92d05ae3b" (path: ""): <nil>

We created a new symlink with the 20 char limit under /dev/disk/by-id/ to verify. As soon as the link was in place the pod saw the device and came online.

Updating the check to remove the ':20' char limit or adding a new one without the limit should fix the issue.

@k8s-ci-robot k8s-ci-robot added the needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. label May 20, 2020
@shannonmitchell
Copy link
Author

/sig cloud-provider

@k8s-ci-robot k8s-ci-robot added sig/cloud-provider Categorizes an issue or PR as relevant to SIG Cloud Provider. and removed needs-sig Indicates an issue or PR lacks a `sig/foo` label and requires one. labels May 20, 2020
@shannonmitchell
Copy link
Author

Looks like there is a related open PR(kubernetes/kubernetes#89726) and a fix in cloud-provider-openstack. (kubernetes/cloud-provider-openstack#853)

@mrbobbytables
Copy link
Member

The community repo is used as the general management point of Kubernetes contributor community. Questions like that are not likely to be answered here. I'd follow up in one of those open issues.

/close

@k8s-ci-robot
Copy link
Contributor

@mrbobbytables: Closing this issue.

In response to this:

The community repo is used as the general management point of Kubernetes contributor community. Questions like that are not likely to be answered here. I'd follow up in one of those open issues.

/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
sig/cloud-provider Categorizes an issue or PR as relevant to SIG Cloud Provider.
Projects
None yet
Development

No branches or pull requests

3 participants