search

aws-samples / aws-parallelcluster-monitoring

Monitoring Dashboard for AWS ParallelCluster
MIT No Attribution
34 stars 24 forks source link

sed fails due to path containing "/" #1

Closed nonatofabio closed 3 years ago

nonatofabio commented 3 years ago

The log_group_names variable is a full path with "/". sed fails to interpret the substitution without adding backslash \ before the forward slashes on the path.

https://github.com/aws-samples/aws-parallelcluster-monitoring/blob/2797f77905a1682e1c265b2aed2e29403d406404/parallelcluster-setup/install-monitoring.sh#L61

Can be corrected in the log_group_names definition here:

log_group_names="\/aws\/parallelcluster\/$(echo ${stack_name} | cut -d "-" -f2-)"
nicolaven commented 3 years ago

Fixed here, thanks Fabio!