~jneo8/charm-infra-node:build/juju-3.1

Last commit made on 2023-10-05
Get this branch:
git clone -b build/juju-3.1 https://git.launchpad.net/~jneo8/charm-infra-node
Only JamesLin can upload to this branch. If you are JamesLin please log in for upload directions.

Branch merges

Branch information

Name:
build/juju-3.1
Repository:
lp:~jneo8/charm-infra-node

Recent commits

6d600c4... by JamesLin

build(juju-3.1): Un-pin juju version

1a0d4cc... by Erhan Sunar

fix build issue

the build issue wax caused by the changes in layer-basic
https://github.com/juju-solutions/layer-basic/issues/215

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

4838bf3... by Erhan Sunar

fix build issue

the build issue wax caused by the changes in layer-basic
https://github.com/juju-solutions/layer-basic/issues/215

19d6eec... by JamesLin

fix: Remove corosync ring check if distrib > eoan

Reviewed-on: https://code.launchpad.net/~jneo8/charm-infra-node/+git/charm-infra-node-public/+merge/439983
Reviewed-by: Martin Kalcok <email address hidden>
Reviewed-by: Andrea Ieri <email address hidden>
Reviewed-by: Robert Gildein <email address hidden>
Reviewed-by: 🤖 prod-jenkaas-bootstack <email address hidden>

8d7a66d... by JamesLin

docs: Update comment wording

efafac7... by JamesLin

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

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

check_crm_attribute: print clearer messages

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

cd9696a... by Mert Kirpici

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

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>