/juju

provider/azure: discover AD/auth URL #6247

Merged
merged 1 commit into from Sep 15, 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 Sep 14, 2016

This diff introduces several changes that will
be used for a new auth method for Azure.

  • remove tenant-id, and instead we discover it along
    with the AD endpoint, using the subscription ID
  • introduce a means for providers to transform
    credentials. Azure will be updated so that it
    supports device-code auth, which it will use to
    generate service principal / client-secret credentials
  • change the identity-url for azure from the AD endpoint
    to the graph endpoint. We need the graph endpoint
    for making changes to AD.
@axw
provider/azure: discover AD/auth URL 
This diff introduces several changes that will
be used for a new auth method for Azure.

- remove tenant-id, and instead we discover it along
  with the AD endpoint, using the subscription ID
- introduce a means for providers to transform
  credentials. Azure will be updated so that it
  supports device-code auth, which it will use to
  generate service principal / client-secret credentials
- change the identity-url for azure from the AD endpoint
  to the graph endpoint. We need the graph endpoint
  for making changes to AD.
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
f70a9f0
@axw
Member

axw commented Sep 15, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Sep 15, 2016

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

jujubot commented Sep 15, 2016

Build failed: Generating tarball failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/9231

@jujubot jujubot merged commit 6e69cfd into juju:master Sep 15, 2016

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