Merge lp://qastaging/~timkuhlman/charms/trusty/rsyslog-forwarder-ha/deps into lp://qastaging/charms/trusty/rsyslog-forwarder-ha

Proposed by Tim Kuhlman
Status: Merged
Merged at revision: 15
Proposed branch: lp://qastaging/~timkuhlman/charms/trusty/rsyslog-forwarder-ha/deps
Merge into: lp://qastaging/charms/trusty/rsyslog-forwarder-ha
Diff against target: 25 lines (+11/-0)
2 files modified
hooks/hooks.py (+2/-0)
hooks/install (+9/-0)
To merge this branch: bzr merge lp://qastaging/~timkuhlman/charms/trusty/rsyslog-forwarder-ha/deps
Reviewer Review Type Date Requested Status
Review Queue (community) automated testing Needs Fixing
Stuart Bishop (community) Approve
Review via email: mp+295891@code.qastaging.launchpad.net

Description of the change

Explicitly install the dependencies for this package. I found this was needed for xenial as the package this is subordinate to moved to using python3 and so didn't install these anymore for python2.

To post a comment you must log in.
Revision history for this message
Stuart Bishop (stub) wrote :

Looks good

review: Approve
Revision history for this message
Review Queue (review-queue) wrote :

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-lxc/4407/

review: Needs Fixing (automated testing)
Revision history for this message
Review Queue (review-queue) wrote :

This item has failed automated testing! Results available here http://juju-ci.vapour.ws:8080/job/charm-bundle-test-aws/4427/

review: Needs Fixing (automated testing)

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