/juju

Update names dependency to accept cloud credential name containing +. #7135

Merged
merged 2 commits into from Mar 22, 2017

Conversation

Reviewers

@wallyworld wallyworld

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

anastasiamac commented Mar 22, 2017

Description of change

This brings in a new version on juju/names that supports having + as part of cloud credential name.

QA steps

As per https://bugs.launchpad.net/juju/+bug/1671915, add credential that contains +. The call should succeed.

Documentation changes

N/A

Bug reference

https://bugs.launchpad.net/juju/+bug/1671915
@anastasiamac
Member

anastasiamac commented Mar 22, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Mar 22, 2017

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

jujubot commented Mar 22, 2017

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/10517
@anastasiamac
Member

anastasiamac commented Mar 22, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Mar 22, 2017

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

@jujubot jujubot merged commit b43e6e9 into juju:develop Mar 22, 2017

@anastasiamac anastasiamac deleted the anastasiamac:valid-credential-lp1671915 branch Mar 22, 2017

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