Skip to content

Fixed the path for PID_DIR, when the WORKDIR is home #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
MaheshIBM wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixed the path for PID_DIR, when the WORKDIR is home #6

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion pmonitor/pmon
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ while [ "$1" != "" ]; do
shift;
done;

export PID_DIR=${WORKDIR}/pid_monitor
export PID_DIR=${WORKDIR}/perftools-setup/pid_monitor
Copy link
Copy Markdown
Collaborator

@josiahsams josiahsams Jan 25, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please check the path where we extract the pid_monitor in our setup scripts: https://github.com/josiahsams/perftools-setup/blob/master/install.sh#L80

Copy link
Copy Markdown
Author

@MaheshIBM MaheshIBM Jan 30, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the place below, so seems there should not be a need to modify the code, I have destroyed my cluster, will revisit this when we plan to re run the code.

Steps to install pid_monitor.

if [ ! -d ${WORKDIR}/pid_monitor ]; then
git clone https://github.com/jschaub30/pid_monitor

mkdir -p ${WORKDIR}/pid_monitor/rundir

#Set execute permission for others so that apache can read files from this directory.
sudo chmod o+x ${WORKDIR} ${WORKDIR}/pid_monitor ${WORKDIR}/pid_monitor/rundir

fi


export WORKLOAD_NAME=${workname}
export DESCRIPTION=${workname}
Expand Down