Pass whether service is primary or subordinate via relation

Bug #1568161 reported by Haw Loeung
48
This bug affects 10 people
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Unassigned
juju-core
Won't Fix
Medium
Unassigned

Bug Description

Hi,

Is it possible to include whether a service/charm is primary or subordinate via the relation (which currently passes unit name, public, and private IP, and service name).

An example of where this is useful would be in the NRPE monitoring charms where it generates hostnames using servicename and unit number. Unfortunately, this can fail because it could be using the wrong relation to work out service name and unit number.

http://bazaar.launchpad.net/~charmers/charms/precise/nrpe-external-master/trunk/view/head:/hooks/config-changed#L90

Curtis Hovey (sinzui)
tags: added: charm subordinate
Revision history for this message
Cheryl Jennings (cherylj) wrote :

Seems like it wouldn't be too difficult to add this information. I'll ask for some input on how to best convey the info.

Changed in juju-core:
status: New → Triaged
importance: Undecided → Medium
Revision history for this message
Haw Loeung (hloeung) wrote :

Any updates on this?

Another example is container-log-archive where it tries to get the correct servicename to use as the swift bucket to upload/archive logs to.

Haw Loeung (hloeung)
tags: added: canonical-is
Changed in juju-core:
status: Triaged → Won't Fix
Changed in juju:
status: New → Triaged
importance: Undecided → High
milestone: none → 2.1.0
Revision history for this message
Anastasia (anastasia-macmood) wrote :

Removing 2.1 milestone as we will not be addressing this issue in 2.1.

Changed in juju:
milestone: 2.1.0 → 2.2.0
Revision history for this message
Tim Kuhlman (timkuhlman) wrote :

I have submitted a code fix to https://github.com/juju/juju/pull/7142

Curtis Hovey (sinzui)
Changed in juju:
milestone: 2.2-beta1 → 2.2-beta2
Revision history for this message
Joel Sing (jsing) wrote :

This has been implemented in in https://github.com/juju/juju/pull/7143, which has been merged.

Changed in juju:
status: Triaged → Fix Committed
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.