~zzehring/charm-juju-controller:bugfix/nrpe-relation-functest-not-series-aware

Last commit made on 2021-10-15
Get this branch:
git clone -b bugfix/nrpe-relation-functest-not-series-aware https://git.launchpad.net/~zzehring/charm-juju-controller
Only Zachary Zehring can upload to this branch. If you are Zachary Zehring please log in for upload directions.

Branch merges

Branch information

Name:
bugfix/nrpe-relation-functest-not-series-aware
Repository:
lp:~zzehring/charm-juju-controller

Recent commits

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

e71de94... by Xav Paice

Minor update to README.md format

a49a21f... by Adam Dyess

Add Monitoring for juju-db installed by snap

Reviewed-on: https://code.launchpad.net/~llama-charmers/charm-juju-controller/+git/charm-juju-controller/+merge/406258
Reviewed-by: Xav Paice <email address hidden>
Reviewed-by: Paul Goins <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>