Skip to content

Commit

Permalink
[FIX] Use locally-scoped IFS w/in display_notices function (#714)
Browse files Browse the repository at this point in the history
  • Loading branch information
gsmith-sas authored Feb 14, 2025
1 parent aba0afd commit 359a590
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
# SAS Viya Monitoring for Kubernetes

## unreleased

* **Metrics**
* [CHANGE] Removed temporary fix (added w/1.2.27) replacing a small number of Grafana dashboards
inherited from the Kube-Prometheus Stack Helm chart that did not work with Grafana 11.x due to Angular
Expand All @@ -25,6 +24,7 @@ deployment of the (generally unused) `Node Exporter/MacOS` dashboard within Graf
some special characters.
* [CHANGE] A minor tweak to an internal function, `populateValuesYAML`, was made to sort the list of
files in the USER_DIR directory captured as part of deploying the V4M Helm Chart.
* [FIX] Use locally-scoped IFS variable within the `display_notices` function


## Version 1.2.33 (14JAN2025)
Expand Down
2 changes: 1 addition & 1 deletion bin/log-include.sh
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ function add_notice {

function display_notices {
if [ -f "$TMP_DIR/notices.txt" ]; then
IFS=''
local IFS=''
cat $TMP_DIR/notices.txt | while read line || [[ -n "$line" ]];
do
log_notice "$line"
Expand Down

0 comments on commit 359a590

Please sign in to comment.