/goose

nova: omit "id" and "serverId" from attach request #28

Merged
merged 1 commit into from Nov 3, 2016

Conversation

Reviewers

@wallyworld wallyworld

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@axw @jujubot @wallyworld
@axw
Member

axw commented Nov 3, 2016

When POSTing to os-volume_attachments, we should not
be passing anything for "id" or "serverId". Set these
to omitempty.
@axw
nova: omit "id" and "serverId" from attach request 
When POSTing to os-volume_attachments, we should not
be passing anything for "id" or "serverId". Set these
to omitempty.
f381998
@axw
Member

axw commented Nov 3, 2016

See: http://developer.openstack.org/api-ref/compute/?expanded=attach-a-volume-to-an-instance-detail
@axw
Member

axw commented Nov 3, 2016

$$merge$$
@jujubot
Member

jujubot commented Nov 3, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-goose

@jujubot jujubot merged commit 328e6b2 into go-goose:v1 Nov 3, 2016

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