- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Change random data generation in SSCEDaoTest
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Change random data generation in SSCEDaoTest ...................................................................... core: Change random data generation in SSCEDaoTest Change random data generation StorageServerConnectionExtensionDaoTest to use only alphanumeric characters Change-Id: I7b76e13ec1d0d84426b8de45857840fb1a2b0dea Bug-Url: https://bugzilla.redhat.com/1239266 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/StorageServerConnectionExtensionDaoTest.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/45579 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7b76e13ec1d0d84426b8de45857840fb1a2b0dea Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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: Implement toString() in StorageServerConnectionExtension
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Implement toString() in StorageServerConnectionExtension ...................................................................... core: Implement toString() in StorageServerConnectionExtension Change-Id: Idc4060042b6847366cc764158a92b0ca79e46ce3 Bug-Url: https://bugzilla.redhat.com/1239266 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/StorageServerConnectionExtension.java 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified; Passed CI tests Allon Mureinik: Looks good to me, approved Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/45578 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idc4060042b6847366cc764158a92b0ca79e46ce3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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-hosted-engine-ha[master]: states, broker: Check for already starting VMs on other hosts
by msivak＠redhat.com 02 Sep '15

02 Sep '15

Martin Sivák has submitted this change and it was merged. Change subject: states, broker: Check for already starting VMs on other hosts ...................................................................... states, broker: Check for already starting VMs on other hosts When two or more hosts try to take over when one host goes down, only one of them can start the hosted engine VM. The others will fail to acquire the storage lock. The health monitor now detects when this is the case, so that the host, which could not acquire the lock, can remove the VM which it could not start and go back to the down state. Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477 Bug-Url: https://bugzilla.redhat.com/1207634 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/states.py M ovirt_hosted_engine_ha/broker/submonitors/engine_health.py M ovirt_hosted_engine_ha/lib/engine.py M ovirt_hosted_engine_ha/lib/util.py 4 files changed, 25 insertions(+), 4 deletions(-) Approvals: Martin Sivák: Looks good to me, approved Roman Mohr: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45592 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5d321039801a9868f2b1232ce0acfdfe756b477 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Adding 3.6 branch jobs for ovirt-node
by tlitovsk＠redhat.com 02 Sep '15

02 Sep '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Adding 3.6 branch jobs for ovirt-node ...................................................................... Adding 3.6 branch jobs for ovirt-node Change-Id: Ie4de3bfa3844fa3eee87d894a6b96b3322962791 Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M jobs/confs/projects/ovirt-node/ovirt-node-job.yaml 1 file changed, 9 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Looks good to me Jenkins CI: Passed CI tests Anatoly Litovsky: Verified; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/45643 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4de3bfa3844fa3eee87d894a6b96b3322962791 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: restapi: Treat template watchdogs as template devices
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: restapi: Treat template watchdogs as template devices ...................................................................... restapi: Treat template watchdogs as template devices When support for WildFly was intruduced a series of changes were needed to move the DELETE methods from collections to resources. The change to the watchdogs resources introduced a regression in the treatment of template watchdogs: they are treated as VM resources instead of template resources. This patch fixes that issue. Change-Id: I8c8dbad721446e71bd6dc135af37ad42fd8518bf Bug-Url: https://bugzilla.redhat.com/1258224 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 550d2a9eb73ab6fa5fd4239ff43a9a6e9eb253cf) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendWatchdogsResource.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45619 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8c8dbad721446e71bd6dc135af37ad42fd8518bf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Allow user to config CertificationValidityCheckTimeInH...
by tnisan＠redhat.com 02 Sep '15

02 Sep '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Allow user to config CertificationValidityCheckTimeInHours ...................................................................... core: Allow user to config CertificationValidityCheckTimeInHours Change-Id: Ifa356d10dc70b86781a451bfdad6dae6dd1ebcc9 Bug-Url: https://bugzilla.redhat.com/1257905 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CertificationValidityChecker.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql M packaging/etc/engine-config/engine-config.properties 4 files changed, 11 insertions(+), 2 deletions(-) Approvals: Eli Mesika: Looks good to me, approved Jenkins CI: Passed CI tests Moti Asayag: Verified -- To view, visit https://gerrit.ovirt.org/45625 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifa356d10dc70b86781a451bfdad6dae6dd1ebcc9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: adding sync all networks button
by alkaplan＠redhat.com 02 Sep '15

02 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: adding sync all networks button ...................................................................... webadmin: adding sync all networks button Adding new button to SubTabHostInterface. The button will allow to sync all host's networks at once by calling SyncAllHostNetworks Vdc command Change-Id: Ic8ade04016cfe0cec9cb8190637dac7983ec262a Bug-Url: https://bugzilla.redhat.com/1194297 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/help/HelpTag.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostInterfaceListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostInterfaceView.java 5 files changed, 83 insertions(+), 9 deletions(-) Approvals: Eliraz Levi: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42024 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8ade04016cfe0cec9cb8190637dac7983ec262a Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Introduce syncAllHostNetwork command
by alkaplan＠redhat.com 02 Sep '15

02 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Introduce syncAllHostNetwork command ...................................................................... engine: Introduce syncAllHostNetwork command the command will sync all host's networks using PersistentHostSetupCommand Change-Id: I200bd53d77d15a11a3279d2aa147b92fc8791b95 Bug-Url: https://bugzilla.redhat.com/1194297 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SyncAllHostNetworksCommand.java A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/predicate/NetworkNotInSyncPredicate.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/predicate/NetworkNotInSyncPredicateTest.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/AuditLogType.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 6 files changed, 126 insertions(+), 1 deletion(-) Approvals: Eliraz Levi: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/41526 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I200bd53d77d15a11a3279d2aa147b92fc8791b95 Gerrit-PatchSet: 19 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: Introduce PersistentHostSetupNetworks command
by alkaplan＠redhat.com 02 Sep '15

02 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Introduce PersistentHostSetupNetworks command ...................................................................... engine: Introduce PersistentHostSetupNetworks command A replacement to the old PersistentSetupNetworkCommand. Change-Id: I659e432be1e2e1e1883811d59badf08558982944 Bug-Url: https://bugzilla.redhat.com/11942971 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/PersistentHostSetupNetworksCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java 2 files changed, 68 insertions(+), 1 deletion(-) Approvals: Eliraz Levi: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/42050 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I659e432be1e2e1e1883811d59badf08558982944 Gerrit-PatchSet: 8 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: adding CTOR for GetNetworkAttachmentsByHostIdQuery
by alkaplan＠redhat.com 02 Sep '15

02 Sep '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: adding CTOR for GetNetworkAttachmentsByHostIdQuery ...................................................................... engine: adding CTOR for GetNetworkAttachmentsByHostIdQuery Change-Id: Ie267270e69964248a5ca8fcb479eaa97a75bfd95 Bug-Url: https://bugzilla.redhat.com/1194297 Signed-off-by: Eliraz Levi <elevi(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/GetNetworkAttachmentsByHostIdQuery.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Eliraz Levi: Verified Alona Kaplan: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44017 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie267270e69964248a5ca8fcb479eaa97a75bfd95 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Eliraz Levi <elevi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0