Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
provider/openstack: update goose #6501
Conversation
hmlanigan
and others
added some commits
Oct 11, 2016
|
QA
|
| @@ -381,11 +381,14 @@ func (s *cinderVolumeSource) attachVolume(arg storage.VolumeAttachmentParams) (* | ||
| return nil, err | ||
| } | ||
| } | ||
| + if novaAttachment.Device == nil { | ||
| + return nil, errors.Errorf("device not assigned to volume attachment") |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
|
Build failed: Tests failed |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit e9ee950
into
juju:develop
Oct 26, 2016
axw
referenced this pull request
Oct 26, 2016
Closed
Update Openstack Provider related to goose endpoint version and tests… #6428
added a commit
to kat-co/juju
that referenced
this pull request
Oct 26, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
axw commentedOct 26, 2016
Use new version of goose, to omit the device
field on volume attachment requests when no
device is specified.
This branch also uses the new OpenStack API
version discovery code in goose.
Fixes https://bugs.launchpad.net/juju/+bug/1636648