/juju

jujuclient: strictly one account per controller #5788

Merged
merged 2 commits into from Jul 14, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@axw @jujubot
@axw
Member

axw commented Jul 13, 2016 •
Edited 1 time

jujuclient has been rejiggered again, this time
to stop storing multiple accounts against
controllers. A controller can now have at most
one account entry. This tidies up the commands
considerably, simplifies the output of show/list-
controllers.

We also fix juju/api.go, so that we can once
again use external authentication. There will
be additional changes coming that

(Review request: http://reviews.vapour.ws/r/5232/)

@axw axw changed the title from juju: handle empty username (WIP) to jujuclient: strictly one account per controller Jul 14, 2016

axw added some commits Jul 14, 2016

@axw
jujuclient: strictly one account per controller 
jujuclient has been rejiggered again, this time
to stop storing multiple accounts against
controllers. A controller can now have at most
one account entry. This tidies up the commands
considerably, simplifies the output of show/list-
controllers.

We also fix juju/api.go, so that we can once
again use external authentication. There will
be additional changes coming that
1abb6f5
@axw
Update dependencies.tsv 
Temporary branch because of import cycle.
f3f427f
@axw
Member

axw commented Jul 14, 2016

$$JFDI$$
@jujubot
Contributor

jujubot commented Jul 14, 2016

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

@jujubot jujubot merged commit ef5ff9f into juju:master Jul 14, 2016

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