Skip to content

ControllerPublishVolume should Verify Disk Attached #43

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.

Sign up for GitHub

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

Jump to bottom
Closed
davidz627 opened this issue Jul 3, 2018 · 0 comments
Closed

ControllerPublishVolume should Verify Disk Attached #43

davidz627 opened this issue Jul 3, 2018 · 0 comments
Assignees
@davidz627
Labels
kind/bug Categorizes issue or PR as related to a bug.
Milestone
Beta

Comments

@davidz627
Copy link
Contributor

After the attach call, and waiting for the attach op to complete. Sometimes the attach will not error but the disk will not be attached.

We should add an additional verification to make sure that the disk has been attached. We can see this on the disk object:

// Users: [Output Only] Links to the users of the disk (attached
// instances) in form: project/zones/zone/instances/instance
Users []string `json:"users,omitempty"`

OR on the instance there is AttachedDisks.
@davidz627 davidz627 added this to the Beta milestone Jul 3, 2018
@davidz627 davidz627 added the kind/bug Categorizes issue or PR as related to a bug. label Jul 3, 2018
@davidz627 davidz627 self-assigned this Jul 3, 2018
@davidz627 davidz627 mentioned this issue Jul 3, 2018
Merged
jsafrane pushed a commit to jsafrane/gcp-compute-persistent-disk-csi-driver that referenced this issue Nov 8, 2023
@openshift-ci
Merge pull request kubernetes-sigs#43 from dobsonj/CVE-2023-44487-origin
269ed09 
OCPBUGS-21593: CVE-2023-44487: bump golang.org/x/net to v0.17.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davidz627 davidz627

Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Milestone
Beta
Development

No branches or pull requests
1 participant
@davidz627