Fix LOGS_SERVER to not be required 46/15646/2
authorAndrew Grimberg <agrimberg@linuxfoundation.org>
Mon, 13 May 2019 20:05:10 +0000 (13:05 -0700)
committerAndrew Grimberg <agrimberg@linuxfoundation.org>
Mon, 13 May 2019 20:10:04 +0000 (13:10 -0700)
Change 0251b7fda7266760bd82a6e56a696160822a57a0 added in a regression
that forces LOGS_SERVER to be defined when it is not supposed to be
required for jobs.

Change-Id: I3800371e831616c9e9332d0a43e3721d8580a834
Signed-off-by: Andrew Grimberg <agrimberg@linuxfoundation.org>
releasenotes/notes/fix_logs_server-4391c56780db6c17.yaml [new file with mode: 0644]
shell/logs-deploy.sh

diff --git a/releasenotes/notes/fix_logs_server-4391c56780db6c17.yaml b/releasenotes/notes/fix_logs_server-4391c56780db6c17.yaml
new file mode 100644 (file)
index 0000000..561e327
--- /dev/null
@@ -0,0 +1,7 @@
+---
+fixes:
+  - |
+    Fix log shipping script to not require a LOGS_SERVER. There was a regression
+    that caused the log shipping script to start requiring a LOGS_SERVER which
+    fails in the case of a system that does not have that optional environment
+    variable set.
index 359642a..aee1b81 100644 (file)
@@ -13,7 +13,7 @@ echo "---> logs-deploy.sh"
 # Ensure we fail the job if any steps fail.
 set -eu -o pipefail
 
-if [[ -z $LOGS_SERVER ]]; then
+if [[ -z $"${LOGS_SERVER:-}" ]]; then
     echo "WARNING: Logging server not set"
 else
     nexus_url="${NEXUSPROXY:-$NEXUS_URL}"