/juju

CAAS workers use new application config #8214

Merged
merged 2 commits into from Dec 14, 2017

Conversation

Reviewers

@axw axw

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@wallyworld @jujubot @axw
@wallyworld
Owner

wallyworld commented Dec 13, 2017

Description of change

The CAAS workers use the new application config infrastructure to create service and ingress resources.

QA steps

deploy kubernetes-core bundle and make a model
note the IP address of the worker machine
deploy my gitlab charm using the above IP as a config attribute:
juju deploy ./gitlab --config juju-external-hostname=10.152.144.191.xip.io
expose the app
juju expose gitlab

check that the app can be reached via a browser
@axw

axw approved these changes Dec 13, 2017
View changes

@wallyworld
Owner

wallyworld commented Dec 14, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Dec 14, 2017

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

@jujubot jujubot merged commit b7c94c1 into juju:develop Dec 14, 2017
1 check failed

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