caas/kubernetes/clientconfig: read cert/key files #8143

Merged
merged 1 commit into from Nov 28, 2017

Conversation

Projects
None yet
3 participants
Member

axw commented Nov 28, 2017

Description of change

When parsing a kubernetes client config file,
read the contents of certificate and key files
referenced within.

(Without this, Juju cannot deal with the kube config
generated by minikube, for example.)

QA steps

  1. minikube start
  2. juju bootstrap localhost
  3. juju add-caas kubernetes k8s
  4. juju add-model k8s k8s
  5. juju deploy ubuntu
  6. juju debug-log -m controller
    (should not see any errors)

Documentation changes

None.

Bug reference

None.

caas/kubernetes/clientconfig: read cert/key files
When parsing a kubernetes client config file,
read the contents of certificate and key files
referenced within.
Member

axw commented Nov 28, 2017

$$merge$$

Contributor

jujubot commented Nov 28, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju

Contributor

jujubot commented Nov 28, 2017

Build failed: Tests failed
build url: http://ci.jujucharms.com/job/github-merge-juju/599

Owner

wallyworld commented Nov 28, 2017

$$merge$$

Contributor

jujubot commented Nov 28, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju

@jujubot jujubot merged commit c577089 into juju:state-controller-refactor Nov 28, 2017

1 check failed

continuous-integration/jenkins/pr-merge This commit cannot be built
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment