Skip to content

Commit 8954fc2

Browse files
author
Sean Smith
committed
Simplify Deployment args
Signed-off-by: Sean Smith <[email protected]>
1 parent 09b9052 commit 8954fc2

File tree

2 files changed

+8
-6
lines changed

2 files changed

+8
-6
lines changed

parallelcluster-setup/install-monitoring.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ usermod -a -G docker $cfn_cluster_user
1818
curl -L "https://github.com/docker/compose/releases/download/1.27.4/docker-compose-$(uname -s)-$(uname -m)" -o /usr/local/bin/docker-compose
1919
chmod +x /usr/local/bin/docker-compose
2020

21-
monitoring_dir_name=${cfn_postinstall_args[1]}
21+
monitoring_dir_name=aws-parallelcluster-monitoring
2222
monitoring_home="/home/${cfn_cluster_user}/${monitoring_dir_name}"
2323

2424
echo "$> variable monitoring_dir_name -> ${monitoring_dir_name}"

post-install.sh

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,16 +4,18 @@
44
# Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
55
# SPDX-License-Identifier: MIT-0
66
#
7-
#
7+
# Usage: ./post-install [version]
88

99
#Load AWS Parallelcluster environment variables
1010
. /etc/parallelcluster/cfnconfig
1111

12-
#get GitHub repo to clone and the installation script
13-
monitoring_url=${cfn_postinstall_args[0]}
14-
monitoring_dir_name=${cfn_postinstall_args[1]}
12+
version=${1:-v0.9}
13+
monitoring_dir_name=aws-parallelcluster-monitoring
1514
monitoring_tarball="${monitoring_dir_name}.tar.gz"
16-
setup_command=${cfn_postinstall_args[2]}
15+
16+
#get GitHub repo to clone and the installation script
17+
monitoring_url=https://github.com/aws-samples/aws-parallelcluster-monitoring/archive/refs/tags/${version}.tar.gz
18+
setup_command=install-monitoring.sh
1719
monitoring_home="/home/${cfn_cluster_user}/${monitoring_dir_name}"
1820

1921
case ${cfn_node_type} in

0 commit comments

Comments
 (0)