~jneo8/charm-infra-node:fix/remove-corosync-ring-check-distrib-higher-then-eoan

Branch merges

Propose for merging

Merged into charm-infra-node:master at revision 19d6eec0d2023385df0df494a9cd4ddb7fb870c1

Andrea Ieri: Approve on 2023-04-02

🤖 prod-jenkaas-bootstack (community): Approve (continuous-integration) on 2023-03-30

Robert Gildein: Approve on 2023-03-30

Diff: 99 lines (+59/-2)

2 files modified

src/lib/lib_infra_node.py (+16/-2)
src/tests/unit/test_lib.py (+43/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

8d7a66d... by JamesLin on 2023-03-30

docs: Update comment wording

efafac7... by JamesLin on 2023-03-30

fix: Remove corosync ring check if distrib > eoan

According to LP #2011836 and #1902919. The corosync verion 2.99 changed
the ring status output for udp/udpu to hardcode the status always report
'OK'. This result in the check providing no value over what is provided
by corosync_rings_check.

1f4af3a... by Mert Kirpici on 2023-02-23

Close LP #1855707

Reviewed-on: https://code.launchpad.net/~mertkirpici/charm-infra-node/+git/charm-infra-node-public/+merge/437657
Reviewed-by: Erhan Sunar <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>
Reviewed-by: Andrea Ieri <email address hidden>

6562789... by Mert Kirpici on 2023-02-23

check_crm_attribute: print clearer messages

Works-on: 1855707
Signed-off-by: Mert Kırpıcı <email address hidden>

cd9696a... by Mert Kirpici on 2023-02-22

files/sudoers/nagios: allow single command

This is a fix since sudo expects the full command or e glob for the rest
of the arguments which is not safe. Apparently it also supports regex
but required version is too recent for that, so we are using the full
command to restrict access of the nagios user since
/usr/sbin/crm_attribute can potentially update crm values as well.

Works-on: 1855707
Signed-off-by: Mert Kırpıcı <email address hidden>

a8bd372... by Mert Kirpici on 2023-02-22

tests/unit: add unit test for new nrpe check

Also get rid of the mock package dependency, which is already part of
the python stdlib for a long time now.

Works-on: 1855707
Signed-off-by: Mert Kırpıcı <email address hidden>

a2a734a... by Mert Kirpici on 2023-02-21

lib: add nrpe check for pgsql cluster health

Works-on: 1855707
Signed-off-by: Mert Kırpıcı <email address hidden>

7fc77d5... by Mert Kirpici on 2023-02-21

Close LP #2007834

Reviewed-on: https://code.launchpad.net/~mertkirpici/charm-infra-node/+git/charm-infra-node-public/+merge/437583
Reviewed-by: Ramesh Sattaru <email address hidden>
Reviewed-by: JamesLin <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

521907d... by Mert Kirpici on 2023-02-20

tests/functional: pin juju<3

We are running functests against juju controller 2.x for now, new
libjuju versioning is in agreement with controller version.

Works-on: 2007834
Signed-off-by: Mert Kırpıcı <email address hidden>

726bb03... by Mert Kirpici on 2022-11-25

Drop bionic support

Reviewed-on: https://code.launchpad.net/~mertkirpici/charm-infra-node/+git/charm-infra-node-public/+merge/433428
Reviewed-by: Ramesh Sattaru <email address hidden>
Reviewed-by: Eric Chen <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>
Reviewed-by: Erhan Sunar <email address hidden>