~vultaire/charm-nagios:LP1939068-duplicates

Branch merges

Propose for merging

Merged into charm-nagios:master at revision 67fe2db9f283fb4fdf584578597e8520d566b92d

🤖 prod-jenkaas-bootstack: Approve (continuous-integration) on 2021-08-17

BootStack Reviewers: Pending requested 2021-08-17

BootStack Reviewers: Pending requested 2021-08-17

Diff: 28 lines (+5/-5)

1 file modified

hooks/monitors_relation_changed.py (+5/-5)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related charm recipes

2a4325f... by Paul Goins on 2021-08-18

Changed suffix to prefix for the sake of automatic hostgroups

The automatic hostgroup code operates by splitting a hostname on the
final hyphen; if we tack a suffix on after that (assuming no hyphen in
the suffix), the suffix will have zero impact on the hostgroup and thus
"duplicate" hostnames will unintentionally end up in the same
hostgroup. Using a prefix instead avoids this problem.

da4849a... by Paul Goins on 2021-08-18

Oops; need to pull the deduped target ID

a0b4ea4... by Paul Goins on 2021-08-18

Replace unique IDs based upon relation identifiers

4aa7177... by Paul Goins on 2021-08-18

minor fix

58091c2... by Paul Goins on 2021-08-18

New dedupe logic. WIP.

59fa0d8... by Paul Goins on 2021-08-17

Added warning log message in case of duplicate hostname

137f7a6... by Connor Chamberlain on 2021-08-13

Added comment to explain duplicate hostname modification

8c6d53f... by Connor Chamberlain on 2021-08-11

Removed redundant Nones in dict.get

f857177... by Connor Chamberlain on 2021-08-11

Merged in parent branch

88eb9b0... by Connor Chamberlain on 2021-08-10

Address review comments