Skip to content

Commit

Permalink
Merge pull request kubernetes#58685 from tsmetana/fix-cinder-resize
Browse files Browse the repository at this point in the history
Automatic merge from submit-queue (batch tested with PRs 58539, 58685). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>.

Openstack: Fill size attribute for the V3 API volumes

The getVolume method in OpenStack provider is not filling the Size for the V3 API type volumes. This
breaks the PV resizing of Cinder volumes which compares the existing volume size with the new request. This leads to redundant volume resize calls to the cloud provider that end with errors.

cc: @gnufied

```release-note
NONE
```
  • Loading branch information
Kubernetes Submit Queue authored Jan 23, 2018
2 parents c2ad536 + 11081ac commit ee14d2e
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions pkg/cloudprovider/providers/openstack/openstack_volumes.go
Original file line number Diff line number Diff line change
Expand Up @@ -222,6 +222,7 @@ func (volumes *VolumesV3) getVolume(volumeID string) (Volume, error) {
ID: volumeV3.ID,
Name: volumeV3.Name,
Status: volumeV3.Status,
Size: volumeV3.Size,
}

if len(volumeV3.Attachments) > 0 {
Expand Down

0 comments on commit ee14d2e

Please sign in to comment.