fix: default variable assignment and tar
command syntax
#1878
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
TITLE
Description
I’ve corrected two issues in the script:
The syntax for setting the default value for the
NODE_EXPORTER_DIR
variable was wrong. The corrected version now properly uses:-
for default assignment in Bash:NODE_EXPORTER_DIR="${NODE_EXPORTER_DIR:-"$BASE_DIR/.node_exporter"}"
This ensures that if
NODE_EXPORTER_DIR
isn’t set, it defaults to$BASE_DIR/.node_exporter
.Fixed the incorrect order of flags in the
tar
command. The-z
flag for gzip compression should follow-x
and precede-v
. The corrected command is:Both changes ensure the script runs without errors in these areas.
Type of change
Please delete options that are not relevant.
Checklist
testnet
, everything else tostaging