- Engine-commits - oVirt List Archives

Change in jenkins[master]: Added libvirt restart to the cleanup
by dcaro＠redhat.com 15 Jan '16

15 Jan '16

David Caro has submitted this change and it was merged. Change subject: Added libvirt restart to the cleanup ...................................................................... Added libvirt restart to the cleanup Also refactored all the lago/libvirt related cleanups into a function Change-Id: I20d4edbb0a7a1c624bf2b4e1f39364bc888314a9 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/shell-scripts/cleanup_slave.sh 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for review Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/51863 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I20d4edbb0a7a1c624bf2b4e1f39364bc888314a9 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(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: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Added slave cleanup to system tests
by sbonazzo＠redhat.com 15 Jan '16

15 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Added slave cleanup to system tests ...................................................................... Added slave cleanup to system tests Change-Id: Ia2239ec060de76d4629990820ef8b75a77742fb1 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/templates/system_tests_templates.yaml 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge Jenkins CI: Passed CI tests David Caro: Verified; Ready for review -- To view, visit https://gerrit.ovirt.org/51862 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia2239ec060de76d4629990820ef8b75a77742fb1 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(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: Sagi Shnaidman <sshnaidm(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: vdsm-jsonrpc-java: added 3.6 jobs
by sbonazzo＠redhat.com 15 Jan '16

15 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: vdsm-jsonrpc-java: added 3.6 jobs ...................................................................... vdsm-jsonrpc-java: added 3.6 jobs Change-Id: I01870ea697b5e687ebb16e72a28463bdfcec332d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-3.6.yaml M jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_create-rpms.yaml 2 files changed, 25 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51876 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01870ea697b5e687ebb16e72a28463bdfcec332d Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: AUTHORS: Added dougsland
by sbonazzo＠redhat.com 14 Jan '16

14 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: AUTHORS: Added dougsland ...................................................................... AUTHORS: Added dougsland I have contributed with project, adding dougsland entry. Change-Id: Ib95bd029caac8c14483242a80bcc2cb20c391103 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> --- M AUTHORS 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Douglas Schilling Landgraf: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/51873 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib95bd029caac8c14483242a80bcc2cb20c391103 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(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 ovirt-hosted-engine-setup[master]: packaging: setup: fix fetch of the answer file
by sbonazzo＠redhat.com 14 Jan '16

14 Jan '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: fix fetch of the answer file ...................................................................... packaging: setup: fix fetch of the answer file deployment of the additional host, must fetch answer file from the first host, also when root password of the first host specified under answer file Bug-Url: https://bugzilla.redhat.com/1298592 Change-Id: Ic0a2a9c443a0392b5f8ab3125222844594060b7c Signed-off-by: Lukianov Artyom <alukiano(a)redhat.com> --- M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py 1 file changed, 12 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Artyom Lukianov: Verified -- To view, visit https://gerrit.ovirt.org/51852 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0a2a9c443a0392b5f8ab3125222844594060b7c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Artyom Lukianov <alukiano(a)redhat.com> Gerrit-Reviewer: Artyom Lukianov <alukiano(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

Change in ovirt-guest-agent[ovirt-3.6]: win: Try to revert to previous value on TranslateName failure
by vfeenstr＠redhat.com 14 Jan '16

14 Jan '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: win: Try to revert to previous value on TranslateName failure ...................................................................... win: Try to revert to previous value on TranslateName failure In case TranslateName fails, we're now reverting back to the previous value of the username only - Most of the time the translation fails on the hostname which cannot be resolved. Which would cause random data to be returned. This has been resolved with this patch. Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Bug-Url: https://bugzilla.redhat.com/1117504 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentWin32.py 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51857 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[ovirt-3.5]: win: Try to revert to previous value on TranslateName failure
by vfeenstr＠redhat.com 14 Jan '16

14 Jan '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: win: Try to revert to previous value on TranslateName failure ...................................................................... win: Try to revert to previous value on TranslateName failure In case TranslateName fails, we're now reverting back to the previous value of the username only - Most of the time the translation fails on the hostname which cannot be resolved. Which would cause random data to be returned. This has been resolved with this patch. Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Bug-Url: https://bugzilla.redhat.com/1117504 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentWin32.py 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Vinzenz Feenstra: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/51858 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: ovirt-3.5 Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-guest-agent[master]: win: Try to revert to previous value on TranslateName failure
by vfeenstr＠redhat.com 14 Jan '16

14 Jan '16

Vinzenz Feenstra has submitted this change and it was merged. Change subject: win: Try to revert to previous value on TranslateName failure ...................................................................... win: Try to revert to previous value on TranslateName failure In case TranslateName fails, we're now reverting back to the previous value of the username only - Most of the time the translation fails on the hostname which cannot be resolved. Which would cause random data to be returned. This has been resolved with this patch. Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Bug-Url: https://bugzilla.redhat.com/1117504 Signed-off-by: Vinzenz Feenstra <vfeenstr(a)redhat.com> --- M ovirt-guest-agent/GuestAgentWin32.py 1 file changed, 5 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Vinzenz Feenstra: Verified; Looks good to me, approved Milan Zamazal: Looks good to me, but someone else must approve Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/51849 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4c5cf6db6d906625cdd01415e2c1153ae3fa0c08 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-guest-agent Gerrit-Branch: master Gerrit-Owner: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Milan Zamazal <mzamazal(a)redhat.com> Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Add check to validator if host has a NUMA architectur
by tnisan＠redhat.com 14 Jan '16

14 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add check to validator if host has a NUMA architectur ...................................................................... core: Add check to validator if host has a NUMA architectur Check in the backend if someone tries to add numa nodes to a host without a numa architecture. Change-Id: I0935761d206ca9c7b9e708cb771c43bdde06b552 Bug-Url: https://bugzilla.redhat.com/1279178 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/numa/vm/NumaValidator.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/numa/vm/AddVmNumaNodesCommandTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/numa/vm/NumaValidatorTest.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/numa/vm/UpdateVmNumaNodesCommandTest.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/webadmin/src/main/resources/org/ovirt/engine/ui/frontend/AppErrors.properties 8 files changed, 20 insertions(+), 3 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Roman Mohr: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51641 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0935761d206ca9c7b9e708cb771c43bdde06b552 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roman Mohr <rmohr(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: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Fix check if selected dedicated host supports NUMA
by tnisan＠redhat.com 14 Jan '16

14 Jan '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix check if selected dedicated host supports NUMA ...................................................................... webadmin: Fix check if selected dedicated host supports NUMA Since multiple dedicated hosts are supported we have to use the first selected host of the dedicated host list to detect if numa is supported and listen to the right multiselect event to detect selection changes. Additional checks make sure that only one host is selected. Change-Id: I69a06d01b70ba38bd1a8a7a636172616218207b2 Bug-Url: https://bugzilla.redhat.com/1279178 Bug-Url: https://bugzilla.redhat.com/1289967 Signed-off-by: Roman Mohr <rmohr(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 3 files changed, 38 insertions(+), 12 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Roman Mohr: Verified Jenkins CI: Passed CI tests Vojtech Szocs: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/51640 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I69a06d01b70ba38bd1a8a7a636172616218207b2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Roman Mohr <rmohr(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0