pprof-datalog only supports OSE...should also support origin.

[jeremyeder]

Only the service names and /etc/sysconfig locations differ. They should be conditionalized. /cc @ekuric

diff -pruN /opt/pbench-agent/tool-scripts/datalog/pprof-datalog.orig /opt/pbench-agent/tool-scripts/datalog/pprof-datalog

--- /opt/pbench-agent/tool-scripts/datalog/pprof-datalog.orig 2016-02-09 12:29:38.079425999 -0500 +++ /opt/pbench-agent/tool-scripts/datalog/pprof-datalog 2016-02-09 12:30:17.701923330 -0500 @@ -1,7 +1,7 @@

!/usr/bin/env bash

-openshift_master="/etc/sysconfig/atomic-openshift-master" -openshift_node="/etc/sysconfig/atomic-openshift-node" +openshift_master="/etc/sysconfig/origin-master" +openshift_node="/etc/sysconfig/origin-node"

profile="$1" osecomponent="$2" @@ -12,18 +12,18 @@ ose_pprof() { case "$profile" in cpu) if grep -q "^OPENSHIFT_PROFILE=cpu" $openshift_master; then

• systemctl restart atomic-openshift-master.service
• systemctl restart origin-master.service else echo "OPENSHIFT_PROFILE=cpu" >> $openshift_master
• systemctl restart atomic-openshift-master.service
• systemctl restart origin-master.service fi ;; mem) if grep -q "^OPENSHIFT_PROFILE=mem" $openshift_master; then
• systemctl restart atomic-openshift-master.service
• systemctl restart origin-master.service else echo "OPENSHIFT_PROFILE=mem" >> $openshift_master
• systemctl restart atomic-openshift-master.service
• systemctl restart origin-master.service fi ;; esac @@ -32,18 +32,18 @@ ose_pprof() { case "$profile" in cpu) if grep -q "^OPENSHIFT_PROFILE=cpu" $openshift_node; then
• systemctl restart atomic-openshift-node.service
• systemctl restart origin-node.service else echo "OPENSHIFT_PROFILE=cpu" >> $openshift_node
• systemctl restart atomic-openshift-node.service
• systemctl restart origin-node.service fi ;; mem) if grep -q "^OPENSHIFT_PROFILE=mem" $openshift_master; then
• systemctl restart atomic-openshift-node.service
• systemctl restart origin-node.service else echo "OPENSHIFT_PROFILE=mem" >> $openshift_node
• systemctl restart atomic-openshift-node.service
• systemctl restart origin-node.service fi ;; esac

[ekuric]

@jeremyeder I will fix this.

[ashishkamra]

@ekuric still planning to work on this? @sjug is this required since you are reworking pprof?