Skip to content

[WIP] Graduate ExpandInUsePersistentVolumes feature to beta#10101

Closed
wongma7 wants to merge 1 commit intokubernetes:release-1.12from
wongma7:resize
Closed

[WIP] Graduate ExpandInUsePersistentVolumes feature to beta#10101
wongma7 wants to merge 1 commit intokubernetes:release-1.12from
wongma7:resize

Conversation

@wongma7
Copy link
Contributor

@wongma7 wongma7 commented Aug 27, 2018

docs for online/in-use resize kubernetes/enhancements#531

@k8s-ci-robot k8s-ci-robot added do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels Aug 27, 2018
@k8sio-netlify-preview-bot
Copy link
Collaborator

Deploy preview for kubernetes-io-vnext-staging processing.

Built with commit c0b4ee4

https://app.netlify.com/sites/kubernetes-io-vnext-staging/deploys/5b8445a2dd28ef4a9e907348

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: bradtopol

If they are not already assigned, you can assign the PR to them by writing /assign @bradtopol in a comment when ready.

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Aug 27, 2018
@Bradamant3 Bradamant3 added this to the 1.12 milestone Aug 27, 2018
@zparnold
Copy link
Contributor

@wongma7 Could I please have you take a look at this again for documentation please?

@justaugustus
Copy link
Member

Removing this from the milestone per this comment: kubernetes/enhancements#531 (comment)

/milestone clear

@zparnold
Copy link
Contributor

/close

@k8s-ci-robot
Copy link
Contributor

@zparnold: Closing this PR.

Details

In response to this:

/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.

@zparnold zparnold removed this from the 1.12 milestone Sep 12, 2018
@Wade201801
Copy link

Hi,

I am trying PVC online resizing based on storage cinder on kubernetes1.11.2, but failed.
The log indicates "Warning VolumeResizeFailed 1m (x25 over 1h) volume_expand Error expanding volume "default/myclaim-expend" of plugin kubernetes.io/cinder : volume status is not available".

The function ExpandVolume in file kubernetes / pkg / cloudprovider / providers / openstack / openstack_volumes.go reported above error, actually is line 410~413.
405 func (os *OpenStack) ExpandVolume(volumeID string, oldSize resource.Quantity, newSize resource.Quantity) (resource.Quantity, error) {
406 volume, err := os.getVolume(volumeID)
407 if err != nil {
408 return oldSize, err
409 }
410 if volume.Status != volumeAvailableStatus {
411 // cinder volume can not be expanded if its status is not available
412 return oldSize, fmt.Errorf("volume status is not available")
413 }
414

My confusion is: once running pods bind pvc, the cinder volume's status is in-use, how can online resizing pass the line410 check?

Best wishes,
Wade Li

@shubheksha
Copy link
Contributor

@Wade201801, you can also try asking in one of the support channels listed. :)

@Wade201801
Copy link

@Wade201801, you can also try asking in one of the support channels listed. :)

Thanks for your reminder.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants