Merge lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only into lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/trunk

Proposed by Cory Johns
Status: Merged
Merged at revision: 17
Proposed branch: lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only
Merge into: lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/trunk
Diff against target: 12 lines (+1/-1)
1 file modified
hooks/common.py (+1/-1)
To merge this branch: bzr merge lp:~bigdata-dev/charms/trusty/apache-flume-hdfs/hdfs-only
Reviewer Review Type Date Requested Status
Kevin W Monroe Approve
Review via email: mp+268002@code.launchpad.net

Description of the change

Added support for HDFS-only, for use with the new hadoop-plugin

To post a comment you must log in.
Revision history for this message
Kevin W Monroe (kwmonroe) wrote :

LGTM

review: Approve
17. By Kevin W Monroe

[merge] Flume only really requires HDFS

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'hooks/common.py'
--- hooks/common.py 2015-07-28 03:28:46 +0000
+++ hooks/common.py 2015-08-13 21:29:09 +0000
@@ -62,7 +62,7 @@
62 ],62 ],
63 'requires': [63 'requires': [
64 flume.verify_resources,64 flume.verify_resources,
65 jujubigdata.relations.HadoopPlugin(),65 jujubigdata.relations.HadoopPlugin(hdfs_only=True),
66 callbacks.update_blocked_status, # not really a requirement, but best way to fit into framework66 callbacks.update_blocked_status, # not really a requirement, but best way to fit into framework
67 ],67 ],
68 'callbacks': [68 'callbacks': [

Subscribers

People subscribed via source and target branches

to all changes: