July 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Add missing audit log severities to OVF_STORE audit log.
by mlipchuk＠redhat.com 14 Jul '15

14 Jul '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Add missing audit log severities to OVF_STORE audit log. ...................................................................... core: Add missing audit log severities to OVF_STORE audit log. Change-Id: Ibe4ab285a26e5efc35999f971269beb62c48473e Bug-Url: https://bugzilla.redhat.com/1242948 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Maor Lipchuk: Verified; Passed CI tests Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43568 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe4ab285a26e5efc35999f971269beb62c48473e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Maor Lipchuk <mlipchuk(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: Block adding Cinder with same URL.
by mlipchuk＠redhat.com 14 Jul '15

14 Jul '15

Maor Lipchuk has submitted this change and it was merged. Change subject: core: Block adding Cinder with same URL. ...................................................................... core: Block adding Cinder with same URL. Adding the same Cinder provider should be blocked. Change-Id: I67c15559d00a992ffc4b2e6b1836339300acc171 Bug-Url: https://bugzilla.redhat.com/1242944 Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/storage/CinderProviderValidator.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineMessage.java M backend/manager/modules/dal/src/main/resources/bundles/AppErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/AppErrors.java M frontend/webadmin/modules/userportal-gwtp/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties M frontend/webadmin/modules/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 6 files changed, 24 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Maor Lipchuk: Verified -- To view, visit https://gerrit.ovirt.org/43567 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67c15559d00a992ffc4b2e6b1836339300acc171 Gerrit-PatchSet: 4 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: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Spy validator directly
by amureini＠redhat.com 14 Jul '15

14 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Spy validator directly ...................................................................... core: Spy validator directly Fixed RemoveStorageDomainCommandTest to spy the StorageDomainToPoolRelationValidator it uses directly, instead of mocking its internal DAO calls. This way, RemoveStorageDomainCommandTest actually tests the logic in RemoveStorageDomainCommand instead of testing the internal implementation of the validatiors it uses. Change-Id: I23276e19bae974075f5438b421a4f5b7f37a3d79 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/RemoveStorageDomainCommandTest.java 1 file changed, 3 insertions(+), 22 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43577 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I23276e19bae974075f5438b421a4f5b7f37a3d79 Gerrit-PatchSet: 3 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: Amit Aviram <aaviram(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: Data member modifiers
by amureini＠redhat.com 14 Jul '15

14 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Data member modifiers ...................................................................... core: Data member modifiers Made the members of StorageHandlingCommandBaseTest private as per Java and JUnit's best practices. Change-Id: Ia91702de4bc8d8ddebb5b2b7239feaeecd07712f Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBaseTest.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43576 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia91702de4bc8d8ddebb5b2b7239feaeecd07712f Gerrit-PatchSet: 3 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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Removed useless data memebers
by amureini＠redhat.com 14 Jul '15

14 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Removed useless data memebers ...................................................................... core: Removed useless data memebers The attachedDomains and isoMap members are constant empty maps. Since they are never changed, there's no point in holding them as data members. Instead, where they are used as mock return values, and inline call to Collections.emptyList() is clearer, and leaves less room for potential mistakes in the future. Change-Id: I03d3b2109ef7939cf7ad7c77f6a7b0b214b3bf67 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBaseTest.java 1 file changed, 3 insertions(+), 7 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43575 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I03d3b2109ef7939cf7ad7c77f6a7b0b214b3bf67 Gerrit-PatchSet: 3 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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Remove useless DbFacade mocking
by amureini＠redhat.com 14 Jul '15

14 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove useless DbFacade mocking ...................................................................... core: Remove useless DbFacade mocking Removed useless mocking of DbFacade from StorageHandlingCommandBaseTest. The DAOs are mocked directly, and mocking the DbFacade does nothing but redundantly expose implementation details to the test. Change-Id: I4dd520dbf7fa5907b22bc6053b1658e316dfc84c Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/StorageHandlingCommandBaseTest.java 1 file changed, 8 insertions(+), 15 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43574 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4dd520dbf7fa5907b22bc6053b1658e316dfc84c Gerrit-PatchSet: 4 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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Remove useless DbFacade mocking
by amureini＠redhat.com 14 Jul '15

14 Jul '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove useless DbFacade mocking ...................................................................... core: Remove useless DbFacade mocking Removed useless mocking of DbFacade from AddExistingBlockStorageDomainCommandTest. The DAOs are mocked directly, and mocking the DbFacade does nothing but redundantly expose implementation details to the test. Change-Id: I47338c264b8ea9392f1ef4dd939727b94cefb158 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/AddExistingBlockStorageDomainCommandTest.java 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/43573 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I47338c264b8ea9392f1ef4dd939727b94cefb158 Gerrit-PatchSet: 4 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: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: invoke freeze and thaw for cinder disks
by derez＠redhat.com 14 Jul '15

14 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: core: invoke freeze and thaw for cinder disks ...................................................................... core: invoke freeze and thaw for cinder disks Invoke freeze/thaw on live snapshot with Cinder disks. Change-Id: I01d3af3cff290c2e1fc6b2ddabc3715fde84713a Bug-Url: https://bugzilla.redhat.com/1240636 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 3 files changed, 82 insertions(+), 8 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43283 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01d3af3cff290c2e1fc6b2ddabc3715fde84713a Gerrit-PatchSet: 12 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: added AuditLog messages for freeze and thaw
by derez＠redhat.com 14 Jul '15

14 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: core: added AuditLog messages for freeze and thaw ...................................................................... core: added AuditLog messages for freeze and thaw Adding AuditLog warnings for failure in freeze/thaw. Change-Id: Ifc4379136c6a5b5c8084370857c2affc7047390b Bug-Url: https://bugzilla.redhat.com/1240636 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 2 files changed, 4 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/43282 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifc4379136c6a5b5c8084370857c2affc7047390b Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: EngineError - add freeze and thaw errors
by derez＠redhat.com 14 Jul '15

14 Jul '15

Daniel Erez has submitted this change and it was merged. Change subject: core: EngineError - add freeze and thaw errors ...................................................................... core: EngineError - add freeze and thaw errors Added vdsm errors relevant to freeze/thaw API. Change-Id: I8c29c3ec18dac9a3b86ab9c538766fe83af7a8a4 Bug-Url: https://bugzilla.redhat.com/1240636 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/errors/EngineError.java M backend/manager/modules/dal/src/main/resources/bundles/VdsmErrors.properties M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/VdsmErrors.java 3 files changed, 15 insertions(+), 0 deletions(-) Approvals: Daniel Erez: Verified; Passed CI tests Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/43281 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c29c3ec18dac9a3b86ab9c538766fe83af7a8a4 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0