Merge lp:~bigdata-dev/charms/trusty/hdp-hadoop/jpshookfixes into lp:~bigdata-dev/charms/trusty/hdp-hadoop/trunk
Status: | Needs review |
---|---|
Proposed branch: | lp:~bigdata-dev/charms/trusty/hdp-hadoop/jpshookfixes |
Merge into: | lp:~bigdata-dev/charms/trusty/hdp-hadoop/trunk |
Diff against target: |
104 lines (+23/-6) 3 files modified
README.md (+1/-1) hooks/bdutils.py (+10/-1) hooks/hdp-hadoop-common.py (+12/-4) |
To merge this branch: | bzr merge lp:~bigdata-dev/charms/trusty/hdp-hadoop/jpshookfixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju Big Data Development | Pending | ||
Review via email: mp+248978@code.launchpad.net |
Description of the change
We test if services are running with 'is_jvm_
root@juju-
27783 Jps
22062 -- process information unavailable
23542 -- process information unavailable
That's less than helpful. So much so, that we get relation failures because the charms try to fire up services that are already running. With this MP, we run jps as the appropriate user for a given service (usually either hdfs or yarn).
This yields goodness:
ubuntu@
22062 NameNode
27825 Jps
ubuntu@
23542 ResourceManager
27839 Jps
I'm not a big fan of having a dict with hard coded strings, but the alternative is to pass a username in with every call to is_jvm_
Unmerged revisions
- 41. By Kevin W Monroe
-
update readme with sane deployment. need to run 'jps' as the correct user in is_jvm_
service_ active. stop services if they're already running during certain hooks.
Oh, I also snuck in a readme change like a boss. It made the suggested deployment undumb.