~martin-kalcok/juju-verify:ceph-mon

Branch merges

Propose for merging

Merged into ~canonical-solutions-engineering/juju-verify:master at revision e3a7cfa95e66b493ad11a1a165b8d65fbe389ea7

Robert Gildein: Approve on 2021-05-04

🤖 prod-jenkaas-bootstack (community): Approve (continuous-integration) on 2021-05-04

Canonical Solutions Engineering Team: Pending requested 2021-04-28

Canonical Solutions Engineering Team: Pending requested 2021-04-28

Diff: 460 lines (+306/-8)

10 files modified

docs/verifiers/ceph-mon.rst (+33/-0)
juju_verify/verifiers/__init__.py (+2/-1)
juju_verify/verifiers/ceph.py (+81/-1)
setup.cfg (+2/-1)
tests/functional/requirements.txt (+0/-0)
tests/functional/tests/bundles/ceph.yaml (+1/-1)
tests/functional/tests/bundles/hacluster.yaml (+1/-1)
tests/functional/tests/test_ceph.py (+29/-0)
tests/unit/verifiers/test_ceph.py (+156/-2)
tox.ini (+1/-1)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

fbbac86... by Martin Kalcok on 2021-05-04

Quality of code improvements.

ca7393f... by Martin Kalcok on 2021-04-30

Documentation for ceph-mon verifier.

27201e2... by Martin Kalcok on 2021-04-30

Optimize ceph-mon cluster health check

15f5c2d... by Martin Kalcok on 2021-04-29

Prevent warnings about similar imports

547951d... by Martin Kalcok on 2021-04-29

Merge remote-tracking branch 'upstream/master' into ceph-mon

f5f6593... by Robert Gildein on 2021-04-28

removed `BaseVerifier.run_action_on_units`

The function `BaseVerifier.run_action_on_units` was used with restriction
that prevents actions being run on units other than those that are part
of the self.units.

Because this restriction was no longer useful, it has been removed, and
without it, there is no reason for the `run_action_on_units` function
to be part of the `BaseVerifier`. At the same time, the
`run_action_on_unit` function has been moved from `BaseVerifier` to
`utils.unit`.

Fixes: LP#1921505

Reviewed-on: https://code.launchpad.net/~rgildein/juju-verify/+git/juju-verify/+merge/401950
Reviewed-by: Martin Kalcok <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

8b3228b... by Martin Kalcok on 2021-04-28

Improve ceph-mon min. version check and quorum check.

600312f... by Robert Gildein on 2021-04-28

removed `BaseVerifier.run_action_on_units`

The function `BaseVerifier.run_action_on_units` was used with restriction
that prevents actions being run on units other than those that are part
of the self.units.

Because this restriction was no longer useful, it has been removed, and
without it, there is no reason for the `run_action_on_units` function
to be part of the `BaseVerifier`. At the same time, the
`run_action_on_unit` function has been moved from `BaseVerifier` to
`utils.unit`.

8c5e413... by Alvaro Uria on 2021-04-28

ceph-osd verifier rst format fixes

Reviewed-on: https://code.launchpad.net/~aluria/juju-verify/+git/juju-verify/+merge/401884
Reviewed-by: Robert Gildein <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

ad95ac2... by Alvaro Uria on 2021-04-27

[FIX] vscode IDE files: gitignore and skip removal