Juju Charms Collection
daisy-retracer package

Merge lp://staging/~barryprice/charms/xenial/daisy-retracer/trunk into lp://staging/~daisy-pluckers/charms/xenial/daisy-retracer/trunk

  1. Xenial Xerus (16.04)
  2. trunk
  3. Merge into trunk
Proposed by Barry Price
Status: Merged
Merged at revision: 213
Proposed branch: lp://staging/~barryprice/charms/xenial/daisy-retracer/trunk
Merge into: lp://staging/~daisy-pluckers/charms/xenial/daisy-retracer/trunk
Diff against target: 14 lines (+4/-0)
1 file modified
hooks/common (+4/-0)
To merge this branch: bzr merge lp://staging/~barryprice/charms/xenial/daisy-retracer/trunk
Reviewer Review Type Date Requested Status
Daisy Pluckers Pending
Review via email: mp+400239@code.staging.launchpad.net

Commit message

Ensure nrpe relation actually has a unit ID before attempting to use it

Seen deploying a couple of new units recently, when RELATION_ID was defined, but a relation-list didn't yet return any units:

+ nrpe_setup
++ relation-ids nrpe-external-master
+ RELATION_ID=nrpe-external-master:193
+ '[' -z nrpe-external-master:193 ']'
++ relation-list -r nrpe-external-master:193
+ relation_unit=
++ relation-get -r nrpe-external-master:193 nagios_host_context
ERROR no unit or application specified
+ nagios_host_context=

To post a comment you must log in.

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

to all changes: