Merge lp://staging/~fginther/charms/trusty/jenkaas/add-slave-relation into lp://staging/~canonical-ci-engineering/charms/trusty/jenkaas/trunk

Proposed by Francis Ginther
Status: Merged
Approved by: Francis Ginther
Approved revision: 23
Merged at revision: 5
Proposed branch: lp://staging/~fginther/charms/trusty/jenkaas/add-slave-relation
Merge into: lp://staging/~canonical-ci-engineering/charms/trusty/jenkaas/trunk
Diff against target: 81 lines (+31/-3)
3 files modified
hooks/actions.py (+5/-0)
hooks/services.py (+23/-3)
metadata.yaml (+3/-0)
To merge this branch: bzr merge lp://staging/~fginther/charms/trusty/jenkaas/add-slave-relation
Reviewer Review Type Date Requested Status
Joe Talbott (community) Approve
Review via email: mp+262219@code.staging.launchpad.net

Commit message

Add relationship hooks for handling the jenkaas-slave relationship.

Description of the change

Add relationship hooks and boilerplate for handling the jenkaas-slave relationship.

This doesn't do anything useful yet, just accepts the relationship from a jenkaas-slave. install_slaves() will report:

{'jenkaas-slave': {u'jenkaas-slave:0': {'jenkaas/0': {u'private-address': u'10.25.176.81', u'master-address': u'10.25.176.81', u'port': u'48484'}, u'jenkaas-slave/0': {u'private-address': u'10.25.176.82', u'slave-address': u'10.25.176.82'}}}}

To post a comment you must log in.
Revision history for this message
Joe Talbott (joetalbott) :
review: Approve

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