Change in vdsm[master]: vdsm-upgrade: stop vdsm when complete upgrade

Alon Bar-Lev alonbl at redhat.com
Tue Sep 18 07:01:31 UTC 2012


Alon Bar-Lev has posted comments on this change.

Change subject: vdsm-upgrade: stop vdsm when complete upgrade
......................................................................


Patch Set 2: (1 inline comment)

....................................................
File vdsm_reg/vdsm-upgrade
Line 154:     else:
Line 155:         msg = "<BSTRAP component='RHEV_INSTALL' status='OK'/>"
Line 156:         logging.debug("<BSTRAP component='RHEV_INSTALL' status='OK'/>")
Line 157:         # Stop vdsm to avoid any communication with Engine before the reboot happens
Line 158:         deployUtil._logExec([constants.EXT_SERVICE, 'vdsmd', 'stop'])
Thinking of it more... I think we should do this before we call the node upgrade script, as:

1. It may take time, and we have only few seconds before reboot at this place.

2. The node update changes the filesystem, so we are in unstable state at this point... so better to stop it when state is stable.

What do you think?
Line 159: 
Line 160:     print (msg)
Line 161: 
Line 162:     sys.stdout.flush()


--
To view, visit http://gerrit.ovirt.org/8029
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ifa889880c7f02c7ecc13daa595873c1ad3e15c67
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>


More information about the vdsm-patches mailing list