Merge ~axino/influxdb-charm/+git/trunk:create_nagios_account_once into influxdb-charm:master

Proposed by Junien F
Status: Merged
Approved by: Paul Gear
Approved revision: 7cf259c1293ee88be4b640e06bd665ceca465e3f
Merged at revision: ee682e278059a461d50413260750389668d7b448
Proposed branch: ~axino/influxdb-charm/+git/trunk:create_nagios_account_once
Merge into: influxdb-charm:master
Diff against target: 12 lines (+2/-0)
1 file modified
reactive/influxdb.py (+2/-0)
Reviewer Review Type Date Requested Status
Paul Gear (community) Approve
Review via email: mp+359097@code.launchpad.net

Commit message

Create the nagios account only once

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
Paul Gear (paulgear) wrote :

LGTM

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

Change successfully merged at revision ee682e278059a461d50413260750389668d7b448

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/reactive/influxdb.py b/reactive/influxdb.py
2index cfbf194..db19fca 100644
3--- a/reactive/influxdb.py
4+++ b/reactive/influxdb.py
5@@ -241,5 +241,7 @@ def update_nrpe_config(svc):
6
7
8 @when('nrpe-external-master.available', 'influxdb.admin.account.created')
9+@when_not('influxdb.nagios.account.created')
10 def create_nagios_influxdb_user(svc):
11 create_influxdb_user('nagios', admin=True, local_user='nagios')
12+ set_flag('influxdb.nagios.account.created')

Subscribers

People subscribed via source and target branches

to all changes: