~xavpaice/charm-juju-controller:add_charmcraft

Last commit made on 2022-03-17
Get this branch:
git clone -b add_charmcraft https://git.launchpad.net/~xavpaice/charm-juju-controller
Only Xav Paice can upload to this branch. If you are Xav Paice please log in for upload directions.

Branch merges

Branch information

Name:
add_charmcraft
Repository:
lp:~xavpaice/charm-juju-controller

Recent commits

fa44f51... by Xav Paice

add charmcraft

57fe32b... by Zachary Zehring

Fix nrpe relation test thru series awareness.

Reviewed-on: https://code.launchpad.net/~zzehring/charm-juju-controller/+git/charm-juju-controller/+merge/410321
Reviewed-by: James Troup <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

b3991fa... by Zachary Zehring

Fix nrpe relation test thru series awareness.

Testing nrpe relation does not work for focal-deployed controllers. The
test fails as the nrpe check generated is different on focal vs bionic
(focal uses snap.juju-db.daemon and bionic juju-db).

This commit adds checking of the inner controller series which allows
for different check paths and expected content.

Additionally, we add constraints to the outer machine for accommodating
the size of juju snaps/binaries.

5fb9259... by Paul Goins

Multiple fixes to enable functional tests

Added a TEST_REPO_ROOT env var, which will be passed through by zaza,
for exposing the root directory of the repository. This allows us
to cleanly refer to the locally-prepped customized Ubuntu charm
required for functional testing.

Additionally, refactored overlays to reduce duplication, and added
a missing num_units setting needed for proper deployment.

Reviewed-on: https://code.launchpad.net/~vultaire/charm-juju-controller/+git/charm-juju-controller/+merge/406317
Reviewed-by: James Troup <email address hidden>
Reviewed-by: Xav Paice <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

bcb1ef1... by Paul Goins

Minimized code

* Removed unused bits
* replaced model.async_block_until with model.block_until (slight
  simplification)

559f997... by Paul Goins

Removed local charm deployment via zaza

juju-controller is deployed into the nested Juju environment living on
the ubuntu unit; it is not used in the parent environment at all, so it
should be removed.

0c19ad9... by Paul Goins

Fixed and reworked overlays

* Moved the templated local charm clause to
  local-charm-overlay.yaml.j2, reducing duplication between the
  bionic/focal overlays.

* Added missing num_units clause.

80d1cea... by Paul Goins

Fixed pathing to modified ubuntu charm

b2751b9... by Xav Paice

Minor update to README.md format

Reviewed-on: https://code.launchpad.net/~llama-charmers/charm-juju-controller/+git/charm-juju-controller/+merge/406274
Reviewed-by: James Troup <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>
Reviewed-by: Drew Freiberger <email address hidden>

f99ca37... by Drew Freiberger

resolved ambiguity in README and removed deprecated landscape-client:registration example