- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Enable tooltip on greyed out LUNs
by tnisan＠redhat.com 26 Apr '16

26 Apr '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Enable tooltip on greyed out LUNs ...................................................................... webadmin: Enable tooltip on greyed out LUNs When editing an ISCSI/FC storage domain, if a LUN is already in use it will be greyed out. If the user hovers over the LUN a tooltip appears with the reason why it is disabled. In this patch, the tooltip is added to the whole row, not only the checkbox cell. It will always appear on the left of the row. Bug-Url: https://bugzilla.redhat.com/1330483 Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b Signed-off-by: Fred Rolland <frolland(a)redhat.com> Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/AbstractSanStorageList.java 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Verified -- To view, visit https://gerrit.ovirt.org/56610 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Create default network QoS for ovirtmgmt upon DC cre...
by alkaplan＠redhat.com 26 Apr '16

26 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Create default network QoS for ovirtmgmt upon DC creation ...................................................................... engine: Create default network QoS for ovirtmgmt upon DC creation Change-Id: I3551563903ca5a4548c08f8b774c7bd82ce7874a Bug-Url: https://bugzilla.redhat.com/1255474 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/AddEmptyStoragePoolCommand.java 1 file changed, 39 insertions(+), 1 deletion(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/56429 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3551563903ca5a4548c08f8b774c7bd82ce7874a Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: [ovirt-scheduler] dropping el6, fc22 on master jobs
by sbonazzo＠redhat.com 26 Apr '16

26 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: [ovirt-scheduler] dropping el6, fc22 on master jobs ...................................................................... [ovirt-scheduler] dropping el6, fc22 on master jobs dropping ovirt-scheduler-proxy-master_build-artifacts-{el6, fc22} jobs Change-Id: I89d973666d5505af87eb9018f31ce3e7dc983c55 Signed-off-by: ngoldin <ngoldin(a)redhat.com> --- M jobs/confs/projects/ovirt-scheduler-proxy/ovirt-scheduler-proxy_standard.yaml 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Nadav Goldin: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56624 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I89d973666d5505af87eb9018f31ce3e7dc983c55 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Send IPv6 address to VDSM as a migration destination.
by alkaplan＠redhat.com 26 Apr '16

26 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: Send IPv6 address to VDSM as a migration destination. ...................................................................... engine: Send IPv6 address to VDSM as a migration destination. Send IPV6 address as the destination migration address if IPv4 isn't configured on both sides and IPv6 is. Change-Id: I751d540c0ebd3f0db57d1ca5809f661c0c713fdd Bug-Url: https://bugzilla.redhat.com/1080725 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmCommand.java 1 file changed, 47 insertions(+), 11 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/56540 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I751d540c0ebd3f0db57d1ca5809f661c0c713fdd Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: minimize access to database during vm analysis
by ahadas＠redhat.com 26 Apr '16

26 Apr '16

Arik Hadas has submitted this change and it was merged. Change subject: core: minimize access to database during vm analysis ...................................................................... core: minimize access to database during vm analysis VM fetchers provide 3 types of pairs to VmAnalyzer: 1. pair of reported VM from VDSM and VM from database 2. pair of VM from database and NULL 3. pair of VM from VDSM and NULL 1) means that the VM is 'stable', that is it is reported from the host that it is expected to run on. 2) means that the VM disappeared, that is no longer reported from the host that it is expected to run on. 3) means that the VM is reported from host that the VM was not running on. To resolve these cases the fetchers get the reported VM from VDSM and the ones in the database that are running on the monitored host. In case of #3, VmAnalyzer needs to query the database again. This patch queries the VM from the database, if needed, in the factory of VmAnalyzer. That way, we reduce the access to the database inside VmAnalyzer and replace unclear null-checks with more declarative checks. Change-Id: I5d6b83c4f4a2d8ee6d1bb3f7f69bebeeec0675cb Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmTestPairs.java 5 files changed, 25 insertions(+), 83 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/56566 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d6b83c4f4a2d8ee6d1bb3f7f69bebeeec0675cb Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: allow QoS and non-QoS networks to be attached to a nic
by alkaplan＠redhat.com 26 Apr '16

26 Apr '16

Alona Kaplan has submitted this change and it was merged. Change subject: engine: allow QoS and non-QoS networks to be attached to a nic ...................................................................... engine: allow QoS and non-QoS networks to be attached to a nic Change-Id: Id83fa947c76fbea4cf313ef50487c2505507d3f3 Bug-Url: https://bugzilla.redhat.com/1271094 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/host/HostSetupNetworksValidatorTest.java 2 files changed, 1 insertion(+), 98 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/56470 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id83fa947c76fbea4cf313ef50487c2505507d3f3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: oVirt Live 3.6.6 rc1
by sbonazzo＠redhat.com 26 Apr '16

26 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: oVirt Live 3.6.6 rc1 ...................................................................... oVirt Live 3.6.6 rc1 Change-Id: Ife1eb5064420e037fb52c3eba68c7c6c1bc6e0ce Signed-off-by: Lev Veyde <lveyde(a)redhat.com> --- M centos-7/Makefile M centos-7/kickstart/ovirt-live-ovirt-custom.ks.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56621 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife1eb5064420e037fb52c3eba68c7c6c1bc6e0ce Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: f22 repo on master is non-existant, moving to f23
by eedri＠redhat.com 26 Apr '16

26 Apr '16

Eyal Edri has submitted this change and it was merged. Change subject: f22 repo on master is non-existant, moving to f23 ...................................................................... f22 repo on master is non-existant, moving to f23 Change-Id: Ia487d84ad4b60d468b2eb4715e27a9be27bfbeb9 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/shell-scripts/system_tests.sh 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eyal Edri: Verified; Ready for merge Jenkins CI: Passed CI tests David Caro: Looks good to me -- To view, visit https://gerrit.ovirt.org/56541 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia487d84ad4b60d468b2eb4715e27a9be27bfbeb9 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: isolate analysis of 'unstable' vm
by ahadas＠redhat.com 26 Apr '16

26 Apr '16

Arik Hadas has submitted this change and it was merged. Change subject: core: isolate analysis of 'unstable' vm ...................................................................... core: isolate analysis of 'unstable' vm This patch extracts the handling of VM reported by host that is different than the one set as the host that the VM runs on in the database and this change simplifies the later stages of VM analysis. Change-Id: Ifb52268f7d6d6e1411699f6271f328f5c017c9f3 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzer.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VmsMonitoring.java M backend/manager/modules/vdsbroker/src/test/java/org/ovirt/engine/core/vdsbroker/monitoring/VmAnalyzerTest.java 4 files changed, 127 insertions(+), 143 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/56535 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb52268f7d6d6e1411699f6271f328f5c017c9f3 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: post ovirt-hosted-engine-setup-1.3.6.0
by stirabos＠redhat.com 26 Apr '16

26 Apr '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.3.6.0 ...................................................................... build: post ovirt-hosted-engine-setup-1.3.6.0 Change-Id: I1e476ce80a85311796099ba6efb859e365639cc4 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M automation/build-artifacts.sh M configure.ac M ovirt-hosted-engine-setup.spec.in 3 files changed, 11 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/56606 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1e476ce80a85311796099ba6efb859e365639cc4 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0