Merge lp://staging/~asanjar/charms/trusty/hdp-tez/hpcloud into lp://staging/charms/trusty/hdp-tez

Proposed by amir sanjar
Status: Merged
Merged at revision: 15
Proposed branch: lp://staging/~asanjar/charms/trusty/hdp-tez/hpcloud
Merge into: lp://staging/charms/trusty/hdp-tez
Diff against target: 40 lines (+11/-6)
1 file modified
hooks/namenode-relation-changed (+11/-6)
To merge this branch: bzr merge lp://staging/~asanjar/charms/trusty/hdp-tez/hpcloud
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
amir sanjar (community) Approve
Antonio Rosales (community) community Approve
Review via email: mp+240532@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Antonio Rosales (arosales) wrote :

I successfully tested this in AWS cloud per the ReadME:
  https://jujucharms.com/trusty/hdp-tez-1/?text=hdp-tez#readme

and got the correct results back.

review: Approve (community)
Revision history for this message
amir sanjar (asanjar) :
review: Approve
Revision history for this message
Charles Butler (lazypower) wrote :

+1 LGTM - thanks for the fix amir, this has verified to fix connectivity with HDFS nodes on AWS and HPCloud.

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

to all changes: