Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Add SetVolumeMetadata to cinder.Client #39
Conversation
| + urlPath := url.URL{Path: fmt.Sprintf("volumes/%s/metadata", volumeId)} | ||
| + url := client.endpoint.ResolveReference(&urlPath).String() | ||
| + // Contrary to what the documentation says, using POST here means | ||
| + // that we can updates of a single key. Using PUT will always |
| + | ||
| +// SetVolumeMetadata sets metadata on a server. Replaces metadata | ||
| +// items that match keys - doesn't modify items that aren't in the | ||
| +// request. |
|
$$merge$$ |
howbazaar
commented
Feb 6, 2017
|
$$merge$$ |
|
No dice, huh? |
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-goose |
|
Ok, now my membership is public. $$merge$$ |
jujubot
merged commit 51ccf4f
into
go-goose:v1
Feb 6, 2017
babbageclunk
deleted the
babbageclunk:set-volume-metadata
branch
Feb 6, 2017
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
babbageclunk commentedFeb 3, 2017
This allows updating the tags for a volume.
Will be used by Juju after a successful migration to update the owning controller for the volume.
Part of fixing https://bugs.launchpad.net/juju/+bug/1648063
Includes a drive-by to make some debug logging not be a warning.