Filippo Giunchedi has submitted this change and it was merged.
Change subject: wmf-reimage: handle unset IPMI_PASSWORD
......................................................................
wmf-reimage: handle unset IPMI_PASSWORD
Change-Id: Ia2a891240dc2ffd3016811e23cada0bd7192bb00
---
M modules/puppetmaster/files/wmf-reimage
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Filippo Giunchedi: Verified; Looks good to me, approved
diff --git a/modules/puppetmaster/files/wmf-reimage
b/modules/puppetmaster/files/wmf-reimage
index 7305fab..a26b4b3 100755
--- a/modules/puppetmaster/files/wmf-reimage
+++ b/modules/puppetmaster/files/wmf-reimage
@@ -101,7 +101,7 @@
function set_pxe_and_reboot {
mgmtname=${1}
- if [ -z "$IPMI_PASSWORD" ]; then
+ if [ -z "${IPMI_PASSWORD:-}" ]; then
echo "WARNING: IPMI_PASSWORD not found."
if ! tty -s; then
echo "Assuming bash, do: "
--
To view, visit https://gerrit.wikimedia.org/r/205835
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia2a891240dc2ffd3016811e23cada0bd7192bb00
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Filippo Giunchedi <[email protected]>
Gerrit-Reviewer: Filippo Giunchedi <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
