Dima Kuznetsov has posted comments on this change.
Change subject: vdsm-tool: Change upgrade mechanism
......................................................................
Patch Set 8:
Upon further examination, this was missing logs, because logger was ignoring DEBUG level logs, fixed in http://gerrit.ovirt.org/#/c/27481,
Here is a comparison of old logs (lhs) and new logs (rhs):
http://www.diffchecker.com/f65qrxae
Also changed the order of checks in upgrades to match the original code, upgrade function is only reached if there is any sense to run it (previously, it checked inside upgrade func).
--
To view, visit
http://gerrit.ovirt.org/27193
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I6e1d28570dedfeff9fe60624b1db72d8cadf136a
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov
dkuznets@redhat.com
Gerrit-Reviewer: Antoni Segura Puimedon
asegurap@redhat.com
Gerrit-Reviewer: Assaf Muller
amuller@redhat.com
Gerrit-Reviewer: Dan Kenigsberg
danken@redhat.com
Gerrit-Reviewer: Dima Kuznetsov
dkuznets@redhat.com
Gerrit-Reviewer: Douglas Schilling Landgraf
dougsland@redhat.com
Gerrit-Reviewer: Saggi Mizrahi
smizrahi@redhat.com
Gerrit-Reviewer: Yaniv Bronhaim
ybronhei@redhat.com
Gerrit-Reviewer: automation@ovirt.org
Gerrit-Reviewer: mooli tayer
mtayer@redhat.com
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No