Sandro Bonazzola has submitted this change and it was merged.
Change subject: packaging: setup: Add dwhIsUp function
......................................................................
packaging: setup: Add dwhIsUp function
The function was added to the dwh_history_timekeeping module.
This function replaces the _remote_dwh_is_up function from the dwh
upgrade plugin, with a more generic function.
Change-Id: I30212df28cddb835ea65448376994a8be8601d73
Signed-off-by: Lev Veyde <lveyde(a)redhat.com>
---
M packaging/setup/ovirt_engine_setup/engine_common/dwh_history_timekeeping.py
M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/upgrade/dwh.py
2 files changed, 12 insertions(+), 9 deletions(-)
Approvals:
Lev Veyde: Verified
Sandro Bonazzola: Looks good to me, approved
Simone Tiraboschi: Looks good to me, but someone else must approve
oVirt Jenkins CI Server: Looks good to me, but someone else must approve
Yedidyah Bar David: Looks good to me, but someone else must approve
Objections:
oVirt Jenkins CI Server: Fails
--
To view, visit
https://gerrit.ovirt.org/38373
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I30212df28cddb835ea65448376994a8be8601d73
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server