/juju

Wire up CAAS unit provisioner #8174

Merged
merged 4 commits into from Dec 5, 2017

Conversation

Reviewers

@wallyworld wallyworld

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

axw commented Dec 5, 2017

Description of change

Wire the CAAS unit provisioner into the manifolds. Update the CAAS broker interface with a method for creating unit pods, and implement it for k8s; call it in the provisioner.

QA steps

None - cannot yet create CAAS units.

Documentation changes

None.

Bug reference

None.

axw added some commits Dec 5, 2017

@axw
cmd/jujud/agent/model: add CAAS unit provisioner
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
4e3b621
@axw
caas: add Broker.EnsureUnit method 
This method accepts a unit name and
provider-specific "spec", and creates
or updates resources according to that
spec for the unit. The resources created
must be identifiable given the unit name,
so that we can later delete them.

Also, change the EnsureOperator signature
to just accept a *caas.OperatorConfig,
rather than a function to get one. It's
always going to be called, so the
indirection doesn't help.
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
1473ae6
@axw
worker/caasunitprovisioner: call EnsureUnit 
Call the ContainerBroker.EnsureUnit method
when the container spec for a unit changes.
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
d53d54f
@axw
worker/caasoperator: fix container-spec-set 
Entity name first, then spec. If no unit
is specified, application is assumed.
This commit was signed with a verified signature.
@axw axw Andrew Wilkins
GPG key ID: 55314F7628952B05 Learn about signing commits
92003f1
@axw
Member

axw commented Dec 5, 2017

CAAS: extend CAAS broker for units, wire up unit provisioner
@wallyworld
Owner

wallyworld commented Dec 5, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Dec 5, 2017

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

@jujubot jujubot merged commit b786dff into juju:develop Dec 5, 2017
1 check passed

1 check passed

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