Merge lp://staging/~bigdata-dev/charms/trusty/hdp-tez/trunk into lp://staging/charms/trusty/hdp-tez

Proposed by Charles Butler
Status: Merged
Merged at revision: 20
Proposed branch: lp://staging/~bigdata-dev/charms/trusty/hdp-tez/trunk
Merge into: lp://staging/charms/trusty/hdp-tez
Diff against target: 363 lines (+130/-45)
10 files modified
README.md (+31/-31)
hooks/hadoop-nodes-relation-broken (+31/-0)
hooks/hadoop-nodes-relation-changed (+33/-0)
hooks/hdputils.py (+2/-0)
hooks/install (+11/-3)
hooks/namenode-relation-changed (+8/-3)
hooks/resourcemanager-relation-changed (+4/-2)
metadata.yaml (+2/-1)
tests/10-deploy-tez (+3/-2)
tests/hadoop-tez-cluster.yaml (+5/-3)
To merge this branch: bzr merge lp://staging/~bigdata-dev/charms/trusty/hdp-tez/trunk
Reviewer Review Type Date Requested Status
Matt Bruzek (community) Approve
Review via email: mp+243856@code.staging.launchpad.net

Description of the change

Includes Amir's openstack networking fixes, and corrects the test case that was failing

To post a comment you must log in.
18. By Charles Butler

Remove IPDB leftover

Revision history for this message
Matt Bruzek (mbruzek) wrote :

Hey lazyPower I found some ipdb bits left over, but you already removed them.

19. By Charles Butler

Adjust timeout down from 2 weeks+

Revision history for this message
Matt Bruzek (mbruzek) wrote :

The latest code LGTM. Thanks for fixing those minor things!

review: Approve
Revision history for this message
Matt Bruzek (mbruzek) wrote :

I typed too soon. Fix the Upstream Project name in the README.md and the Upstream website that link does not render from Markdown to HTML correctly.

20. By Charles Butler

Adjustments to readme for linking, and typos

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