-
Notifications
You must be signed in to change notification settings - Fork 0
Slurm prolog #50
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.
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
Slurm prolog #50
Conversation
I need to move the config template file from |
96592db
to
98b5a30
Compare
cat << _EOF_ >> /opt/slurm/etc/scripts/prolog.d/80_cloudwatch_agent_config_prolog.sh | ||
#!/bin/bash | ||
|
||
CWAGENT_CONFIG=/sbo/data/scratch/CWAgent_config_\$SLURM_CLUSTER_NAME.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This assumes that we have Lustre mounted on the cluster, which I believe it will fail if we enable benchmark but disable lustre?
Can we have that file inside /scripts
, for example? Or some other folder we know for sure it will be present?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I had a similar thought but forgot to update this variable!
@WeinaJi does this have to live in the scratch folder, or can it live in /opt/slurm so it gets removed together with the cluster?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The CWAgent_config*.json
is only used by the prolog script. It can live anywhere as long as it can be read by the prolog script.
|
||
if [ ! -f \$CWAGENT_CONFIG ]; then | ||
echo "Create CWAGENT_CONFIG " \$CWAGENT_CONFIG | ||
sed "s/\\$CLUSTER_NAME/\$SLURM_CLUSTER_NAME/g" /opt/slurm/CWAgent_config_tpl.json > \$CWAGENT_CONFIG |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're not going to download it anyway
Failing tests can be ignored on this branch; the fixes will be taken care of in #54. |
No description provided.