Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add credential schema for lxd, cleanup get-model-config #5825

Merged
merged 1 commit into from Jul 19, 2016

Conversation

wallyworld
Copy link
Member

@wallyworld wallyworld commented Jul 19, 2016

  1. Add model-config alias for get-model-config
  2. Filter authorized-keys from model-config output
    (these are accessible from juju ssh-keys)
  3. Filter credential attributes from model-config output
    (these will be removed, but we want clean output until then)

The LXD provider supports a new "certificate" auth type. At the moment, the client and server keys are generated when the LXD server is connected to. So we don't use detect credentials. But we can model the credential attributes used.

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

@wallyworld wallyworld force-pushed the cleanup-get-model-output branch 3 times, most recently from 6fac461 to 011f228 Compare July 19, 2016 06:41
@wallyworld
Copy link
Member Author

$$JFDI$$

@jujubot
Copy link
Collaborator

jujubot commented Jul 19, 2016

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

@jujubot
Copy link
Collaborator

jujubot commented Jul 19, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8451

@wallyworld
Copy link
Member Author

$$JFDI$$

@jujubot
Copy link
Collaborator

jujubot commented Jul 19, 2016

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

@jujubot
Copy link
Collaborator

jujubot commented Jul 19, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8453

@wallyworld
Copy link
Member Author

$$JFDI$$

@jujubot
Copy link
Collaborator

jujubot commented Jul 19, 2016

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

@jujubot jujubot merged commit 71d913e into juju:master Jul 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants