Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 7: Verified+1
Verified after manually applied JSON patch
http://gerrit.ovirt.org/#/c/33424/
--
To view, visit
Dan Kenigsberg has submitted this change and it was merged.
Change subject: vdsm: making PM 'on' and 'off' sync
..
vdsm: making PM 'on' and 'off' sync
Consider the following scenario:
DC1 with H1 (with PM) and H2 on cluster
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 8:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_create-rpms_merged_test_debug/229/ :
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 7:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11686/ : SUCCESS
Yaniv Bronhaim has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 7: Code-Review+1
looks fine to me. although i want to see the re-factor later on
--
To view, visit
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 7: Code-Review+2
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 6: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/32695/6/vdsm/API.py
File vdsm/API.py:
Line 1221:
Yaniv Bronhaim has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 5: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/32695/5/vdsm/API.py
File vdsm/API.py:
Line 1234:
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 5:
(1 comment)
http://gerrit.ovirt.org/#/c/32695/5/vdsm/API.py
File vdsm/API.py:
Line 1234:
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 6: Verified+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 6:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11604/ : SUCCESS
Martin Peřina has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 6: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 4:
(4 comments)
http://gerrit.ovirt.org/#/c/32695/4/vdsm/API.py
File vdsm/API.py:
Line 1227: try:
Line
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 5:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11592/ : SUCCESS
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 5: Verified+1
verified and working as expected on engine 3.3
--
To view, visit http://gerrit.ovirt.org/32695
To
Martin Peřina has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 5: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Yaniv Bronhaim has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 4: Code-Review-1
(4 comments)
http://gerrit.ovirt.org/#/c/32695/4/vdsm/API.py
File vdsm/API.py:
Line 1227:
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3: -Verified
(1 comment)
http://gerrit.ovirt.org/#/c/32695/3/vdsm/API.py
File vdsm/API.py:
Line 1253:
Martin Peřina has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/32695/3/vdsm/API.py
File vdsm/API.py:
Line 1253:
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3:
(1 comment)
http://gerrit.ovirt.org/#/c/32695/3/vdsm/API.py
File vdsm/API.py:
Line 1253:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 4:
Build Failed
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11554/ : FAILURE
Martin Peřina has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 4: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Oved Ourfali has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 4: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Yaniv Bronhaim has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/32695/2/vdsm/API.py
File vdsm/API.py:
Line 1247:
Yaniv Bronhaim has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2:
Dan, afaiu about the verification Eli checked that the fence script never hangs
more than 20 second
--
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2:
(1 comment)
http://gerrit.ovirt.org/#/c/32695/2/vdsm/API.py
File vdsm/API.py:
Line 1247:
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit_tests_gerrit_el/11541/ : SUCCESS
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3: Verified+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3:
Eli, setting v+1 is not enough. Please explain how you have verified this. In
particular, this change
Martin Peřina has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 3: Code-Review-1
(1 comment)
http://gerrit.ovirt.org/#/c/32695/3/vdsm/API.py
File vdsm/API.py:
Line 1253:
Eli Mesika has uploaded a new change for review.
Change subject: vdsm: making PM 'on' and 'off' sync
..
vdsm: making PM 'on' and 'off' sync
Consider the following scenario:
DC1 with H1 (with PM) and H2 on cluster C1
another
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 1: Verified+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 1: Code-Review-1 Verified-1
Build Unstable
Eli Mesika has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2: Verified+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
Oved Ourfali has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2: Code-Review+1
--
To view, visit http://gerrit.ovirt.org/32695
To unsubscribe, visit
oVirt Jenkins CI Server has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2:
Build Successful
http://jenkins.ovirt.org/job/vdsm_master_unit-tests_created/12353/ : SUCCESS
Dan Kenigsberg has posted comments on this change.
Change subject: vdsm: making PM 'on' and 'off' sync
..
Patch Set 2: Code-Review-1
How did you verify this patch? We were forced to use asynchronous call to the
fence script,
37 matches
Mail list logo