Change to install.sh to allow monitor to be installed on controller node

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Change to install.sh to allow monitor to be installed on controller node

jcalcote
Hi Yaguang,

Here's a simple change to install.sh that allows the controller node to be used as a monitor. (We have limited hardware resources in our bare metal test system.)

--- install.sh.orig     2015-12-02 17:17:19.175940921 -0700
+++ install.sh  2015-12-02 17:00:05.502013747 -0700
@@ -361,7 +361,7 @@
 function kill_diamond() {
     cat <<"EOF" >kill_diamond.sh
 #!/bin/bash
-diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|awk -F " " '{print $2}'`
+diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|grep -v kill_diamond|awk -F " " '{print $2}'`
 for pid in $diamond_pid; do
     sudo -E kill -9 $pid
 done

Essentially, this change modifies the kill_diamond.sh script so it filters out the process running the ssh command if the script is being remotely executed on the node on which the installer is running.

Note - this change was made to install.sh on the wip53 dev branch.

John
Reply | Threaded
Open this post in threaded view
|

RE: Change to install.sh to allow monitor to be installed on controller node

ywang19
Administrator

Thanks john, I will check and merge.

 

From: jcalcote [via vsm-discuss] [mailto:ml-node+[hidden email]]
Sent: Thursday, December 03, 2015 7:38 AM
To: Wang, Yaguang
Subject: Change to install.sh to allow monitor to be installed on controller node

 

Hi Yaguang,

Here's a simple change to install.sh that allows the controller node to be used as a monitor. (We have limited hardware resources in our bare metal test system.)

--- install.sh.orig     2015-12-02 17:17:19.175940921 -0700
+++ install.sh  2015-12-02 17:00:05.502013747 -0700
@@ -361,7 +361,7 @@
 function kill_diamond() {
     cat <<"EOF" >kill_diamond.sh
 #!/bin/bash
-diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|awk -F " " '{print $2}'`
+diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|grep -v kill_diamond|awk -F " " '{print $2}'`
 for pid in $diamond_pid; do
     sudo -E kill -9 $pid
 done

Essentially, this change modifies the kill_diamond.sh script so it filters out the process running the ssh command if the script is being remotely executed on node on which the installer is running.

Note - this change was made to install.sh on the wip53 dev branch.

John


If you reply to this email, your message will be added to the discussion below:

http://vsm-discuss.33411.n7.nabble.com/Change-to-install-sh-to-allow-monitor-to-be-installed-on-controller-node-tp261.html

To start a new topic under vsm-discuss, email [hidden email]
To unsubscribe from vsm-discuss, click here.
NAML

Reply | Threaded
Open this post in threaded view
|

RE: Change to install.sh to allow monitor to be installed on controller node

ywang19
Administrator
In reply to this post by jcalcote

Merged, thanks.

 

 

From: jcalcote [via vsm-discuss] [mailto:ml-node+[hidden email]]
Sent: Thursday, December 03, 2015 7:38 AM
To: Wang, Yaguang
Subject: Change to install.sh to allow monitor to be installed on controller node

 

Hi Yaguang,

Here's a simple change to install.sh that allows the controller node to be used as a monitor. (We have limited hardware resources in our bare metal test system.)

--- install.sh.orig     2015-12-02 17:17:19.175940921 -0700
+++ install.sh  2015-12-02 17:00:05.502013747 -0700
@@ -361,7 +361,7 @@
 function kill_diamond() {
     cat <<"EOF" >kill_diamond.sh
 #!/bin/bash
-diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|awk -F " " '{print $2}'`
+diamond_pid=`ps -ef|grep diamond|grep -v grep|grep -v bash|grep -v kill_diamond|awk -F " " '{print $2}'`
 for pid in $diamond_pid; do
     sudo -E kill -9 $pid
 done

Essentially, this change modifies the kill_diamond.sh script so it filters out the process running the ssh command if the script is being remotely executed on node on which the installer is running.

Note - this change was made to install.sh on the wip53 dev branch.

John


If you reply to this email, your message will be added to the discussion below:

http://vsm-discuss.33411.n7.nabble.com/Change-to-install-sh-to-allow-monitor-to-be-installed-on-controller-node-tp261.html

To start a new topic under vsm-discuss, email [hidden email]
To unsubscribe from vsm-discuss, click here.
NAML