- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: core: Remove BackwardCompatibilityTaskCreationTest
by tnisan＠redhat.com 20 Sep '15

20 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Remove BackwardCompatibilityTaskCreationTest ...................................................................... core: Remove BackwardCompatibilityTaskCreationTest This test was added in order to future-proof the code against accidentally changing the behavior of concreteCreateTask() as part of the development of the SEAT framework. As this framework is now deprecated in favor of Command Coordination, and this method does nothing but delegate to CommandCoordinationUtil, there really is no reason to keep this test. Especially in light of the occasional failures it displayed in the CI environment. Change-Id: I56b1f6378d3b9e6df9cfaaefc1d2bad260bffaf9 Bug-Url: https://bugzilla.redhat.com/1227310 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- D backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/BackwardCompatibilityTaskCreationTest.java 1 file changed, 0 insertions(+), 204 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/46423 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I56b1f6378d3b9e6df9cfaaefc1d2bad260bffaf9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: column checkboxes
by gshereme＠redhat.com 20 Sep '15

20 Sep '15

Greg Sheremeta has submitted this change and it was merged. Change subject: webadmin: column checkboxes ...................................................................... webadmin: column checkboxes - Column check boxes were not working as they would never update the underlying model due to not handling the 'change' event. This patch fixes the issue. Change-Id: I4b38428b8fe7cda349f0c2f420ff35e243930523 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1264041 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/table/cell/CheckboxCell.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Greg Sheremeta: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46385 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b38428b8fe7cda349f0c2f420ff35e243930523 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Filter out external VMs for OVF_SOTRE disk.
by mlipchuk＠redhat.com 20 Sep '15

20 Sep '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Filter out external VMs for OVF_SOTRE disk. ...................................................................... core: Filter out external VMs for OVF_SOTRE disk. Filter out external VMs when we store VMs to the OVF_STORE disk in the Storage Domain. External VMs represent processes which run on the Host and does not reflect entities which the user will want to preserve for disaster, therefore there is no need to preserve also external VMs in the OVF_STORE disk. Change-Id: I253bfb974f78d0e7441da020d4bfe46c8aa5ac10 Bug-Url: https://bugzilla.redhat.com/1259467 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M packaging/dbscripts/vms_sp.sql 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/45878 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I253bfb974f78d0e7441da020d4bfe46c8aa5ac10 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest DAO mocks
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest DAO mocks ...................................................................... core: GetLunsByVgIdQueryTest DAO mocks Standardized the way DAOs are mocked and used so the class has a single pattern (which complies to the patterns of the other query tests) instead of three different ways to handle three different DAOs. Change-Id: I52ac07b7420de9b0d978bc909ec303667d21777b Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 14 insertions(+), 24 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46409 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52ac07b7420de9b0d978bc909ec303667d21777b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest array initializers
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest array initializers ...................................................................... core: GetLunsByVgIdQueryTest array initializers Removed redundant array initializers to improve code readability. Change-Id: Id386fd0939965f57d00b9962e88a2e531f902bbf Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46408 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id386fd0939965f57d00b9962e88a2e531f902bbf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdTest->GetLunsByVgIdQueryTest
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdTest->GetLunsByVgIdQueryTest ...................................................................... core: GetLunsByVgIdTest->GetLunsByVgIdQueryTest Renamed test class to represent the class it is testing, as per JUnit's best practices. Change-Id: I7b2208195ba173b4778307600919db0786aea81c Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- R backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46407 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b2208195ba173b4778307600919db0786aea81c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest pointless assertion
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest pointless assertion ...................................................................... core: GetLunsByVgIdQueryTest pointless assertion Remove a pointless assertion on the query's return value type. Anything that can be validated in compile time needn't be asserted in runtime. Change-Id: If46db1d0b4bfd6d72a7e553a615495e6119b689e Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46414 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If46db1d0b4bfd6d72a7e553a615495e6119b689e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest#GUIDS
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest#GUIDS ...................................................................... core: GetLunsByVgIdQueryTest#GUIDS Changed GUIDs to be a String, as it's always referenced by its toString() anyway. Change-Id: I1ebd7d1481008ad23861a55a22d3f6a286b34428 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 12 insertions(+), 12 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46415 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ebd7d1481008ad23861a55a22d3f6a286b34428 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest#checkReturnValue()
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest#checkReturnValue() ...................................................................... core: GetLunsByVgIdQueryTest#checkReturnValue() Stop passing the query as an argument, as it's available by calling a simple getter - getQuery(). Change-Id: Ifd0e4512b642526c9ac78a7bac43103c2a3f2355 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46413 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifd0e4512b642526c9ac78a7bac43103c2a3f2355 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: GetLunsByVgIdQueryTest generics
by amureini＠redhat.com 20 Sep '15

20 Sep '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: GetLunsByVgIdQueryTest generics ...................................................................... core: GetLunsByVgIdQueryTest generics Updated the use of generics to Java 7's diamond syntax to clear up the code. Change-Id: I52e3265252c393a432dffe49e9b45289bcdfde53 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/GetLunsByVgIdQueryTest.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Freddy Rolland: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/46412 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I52e3265252c393a432dffe49e9b45289bcdfde53 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0