[MediaWiki-commits] [Gerrit] operations/puppet[production]: auto-reimage: improved output

2016-09-14 Thread Volans (Code Review) 
Volans has submitted this change and it was merged.

Change subject: auto-reimage: improved output
..


auto-reimage: improved output

Bug: T143536
Change-Id: I6b75e55e7da73e92da78411279f169f00d17bb9f
---
M modules/salt/files/wmf_auto_reimage.py
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Volans: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/salt/files/wmf_auto_reimage.py 
b/modules/salt/files/wmf_auto_reimage.py
index d27d985..b0e1749 100644
--- a/modules/salt/files/wmf_auto_reimage.py
+++ b/modules/salt/files/wmf_auto_reimage.py
@@ -648,6 +648,8 @@
 hosts_commands = {}
 audit_commands = {}
 
+print("Running wmf-reimage on hosts: {hosts}".format(hosts=hosts))
+
 for host in hosts:
 mgmt_host = get_mgmt(host)
 if mgmt_host is None:
@@ -662,6 +664,8 @@
 password=ipmi_password, host=host, mgmt=mgmt_host)]
 audit_commands[host] = [command.format(
 password='**', host=host, mgmt=mgmt_host)]
+print("wmf-reimage log is on {puppetmaster}:/root/{host}.log".format(
+puppetmaster=puppetmaster_host, host=host))
 
 for host, result in proxy_command(
 'reimage_hosts', puppetmaster_host, hosts_commands,
@@ -671,8 +675,7 @@
 success_hosts.append(host)
 
 # See TODO in the docstring
-print("Run wmf-reimage on hosts: {hosts}".format(
-hosts=hosts))
+print("Run wmf-reimage on hosts: {hosts}".format(hosts=hosts))
 return hosts
 
 
@@ -938,6 +941,7 @@
 user -- the user that launched the script, for auditing purposes
 """
 print('START')
+print('To monitor the full log:\ntail -F {log}'.format(log=LOG_PATH))
 
 # Get additional informations
 ipmi_password = get_ipmi_password()

-- 
To view, visit https://gerrit.wikimedia.org/r/310680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I6b75e55e7da73e92da78411279f169f00d17bb9f
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Volans 
Gerrit-Reviewer: Volans 
Gerrit-Reviewer: jenkins-bot <>

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

[MediaWiki-commits] [Gerrit] operations/puppet[production]: auto-reimage: improved output

2016-09-14 Thread Volans (Code Review) 
Volans has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/310680

Change subject: auto-reimage: improved output
..

auto-reimage: improved output

Bug: T143536
Change-Id: I6b75e55e7da73e92da78411279f169f00d17bb9f
---
M modules/salt/files/wmf_auto_reimage.py
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/80/310680/1

diff --git a/modules/salt/files/wmf_auto_reimage.py 
b/modules/salt/files/wmf_auto_reimage.py
index d27d985..b0e1749 100644
--- a/modules/salt/files/wmf_auto_reimage.py
+++ b/modules/salt/files/wmf_auto_reimage.py
@@ -648,6 +648,8 @@
 hosts_commands = {}
 audit_commands = {}
 
+print("Running wmf-reimage on hosts: {hosts}".format(hosts=hosts))
+
 for host in hosts:
 mgmt_host = get_mgmt(host)
 if mgmt_host is None:
@@ -662,6 +664,8 @@
 password=ipmi_password, host=host, mgmt=mgmt_host)]
 audit_commands[host] = [command.format(
 password='**', host=host, mgmt=mgmt_host)]
+print("wmf-reimage log is on {puppetmaster}:/root/{host}.log".format(
+puppetmaster=puppetmaster_host, host=host))
 
 for host, result in proxy_command(
 'reimage_hosts', puppetmaster_host, hosts_commands,
@@ -671,8 +675,7 @@
 success_hosts.append(host)
 
 # See TODO in the docstring
-print("Run wmf-reimage on hosts: {hosts}".format(
-hosts=hosts))
+print("Run wmf-reimage on hosts: {hosts}".format(hosts=hosts))
 return hosts
 
 
@@ -938,6 +941,7 @@
 user -- the user that launched the script, for auditing purposes
 """
 print('START')
+print('To monitor the full log:\ntail -F {log}'.format(log=LOG_PATH))
 
 # Get additional informations
 ipmi_password = get_ipmi_password()

-- 
To view, visit https://gerrit.wikimedia.org/r/310680
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I6b75e55e7da73e92da78411279f169f00d17bb9f
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Volans 

___
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits