Conjure-up uses double the deployed hardware when deploying openstack-base and kubernetes-core #553

Closed
canonical-cgregan opened this Issue Jan 11, 2017 · 6 comments

Comments

Projects
None yet
4 participants

cs:kubernetes-core uses 2 physical machines to deploy the services for kubernetes. Conjure-up bootstraps one node for juju, then 4 more for the services. Not sure why this is the case

Contributor

battlemidget commented Jan 11, 2017

Thanks will look into this

@battlemidget battlemidget self-assigned this Jan 11, 2017

@battlemidget battlemidget added the bug label Jan 11, 2017

Contributor

simonklb commented Jan 12, 2017

I think this could be the issue I'm facing as well, but with OpenStack base for MAAS. Rather than deploying all of the charms onto LXD machines, conjure up spins up one machine per charm. Ended up with ~20 servers running instead of 4... :)

Contributor

battlemidget commented Jan 12, 2017

@simonklb thanks we'll get this sorted

@battlemidget battlemidget changed the title from Conjure-up uses double the deployed hardware when deploying kuberbetes-core than the bundle directly deployed to Conjure-up uses double the deployed hardware when deploying openstack-base and kubernetes-core Jan 12, 2017

@battlemidget battlemidget added this to the 2.1.0 milestone Jan 12, 2017

Contributor

mikemccracken commented Jan 12, 2017

@canonical-cgregan are you seeing this on a MAAS controller or something else? Curious if it's limited to MAAS - the add-machine handling path is different for MAAS vs. the rest.

Contributor

mikemccracken commented Jan 12, 2017

@canonical-cgregan thanks. I have a fix in progress, it should be committed today and will be available in our daily-git ppa for testing.

mikemccracken pushed a commit that referenced this issue Jan 12, 2017

sync controller's assignments with bundle at start
Fixes #553.

When bundle has machines, slurp in all assignments.
If no machines, warn if assignments are found.

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>

@battlemidget battlemidget closed this in #564 Jan 13, 2017

battlemidget added a commit that referenced this issue Jan 13, 2017

Ensure that bundle placement directives are always used and shown in …
…UI (#564)

* sync controller's assignments with bundle at start

Fixes #553.

When bundle has machines, slurp in all assignments.
If no machines, warn if assignments are found.

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>

* Add warning to log wrapper

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>

* fix lint

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>

* always sync placements

placement syncing is important for ensure_machines to work, so do it
every time even if the user doesn't invoke the architecture view

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>

* fix tests

add a mock to avoid test failure

Signed-off-by: Michael McCracken <mike.mccracken@canonical.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment