NRPE Charm

Merge ~aurelien-lourot/charm-nrpe/+git/nrpe-charm:bug/1827703 into ~nrpe-charmers/charm-nrpe:master

  1. Git
  2. lp:~aurelien-lourot/charm-nrpe/+git/nrpe-charm
  3. bug/1827703
  4. Merge into master
Proposed by Aurelien Lourot
Status: Merged
Approved by: Jeremy Lounder
Approved revision: 4ce5f269f47e8471d2d7b59b9e74b2c8667885aa
Merged at revision: 51c1bff2264a20756619b359b9228ab978d482e3
Proposed branch: ~aurelien-lourot/charm-nrpe/+git/nrpe-charm:bug/1827703
Merge into: ~nrpe-charmers/charm-nrpe:master
Diff against target: 33 lines (+12/-3)
1 file modified
hooks/nrpe_helpers.py (+12/-3)
Reviewer Review Type Date Requested Status
Jeremy Lounder (community) Approve
Giuseppe Petralia Approve
Review via email: mp+378268@code.qastaging.launchpad.net

Commit message

Allow connections from Nagios' egress-subnets instead of from its ingress-address

If egress-subnets isn't available it will fall back to ingress-adress and finally private-address.

Closes-Bug: #1827703

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Giuseppe Petralia (peppepetra) wrote :

Looks good to me. Thanks!

review: Approve
Revision history for this message
Jeremy Lounder (jldev) :
review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 51c1bff2264a20756619b359b9228ab978d482e3

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches