- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: aaa: Kerberos negotiation for RESTAPI should be enabled by d...
by tnisan＠redhat.com 18 Aug '16

18 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: aaa: Kerberos negotiation for RESTAPI should be enabled by default to maintain SDK v3 compatibility ...................................................................... aaa: Kerberos negotiation for RESTAPI should be enabled by default to maintain SDK v3 compatibility The restapi negotiate should be enabled by default, ENGINE_RESTAPI_NEGO=true. This maintains compatibilty with v3 restapi clients. Change-Id: I2da5a99a9dc58c4ca896e8bcf9bee7c1bb80ad10 Bug-Url: https://bugzilla.redhat.com/1367513 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Ravi Nori: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62474 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2da5a99a9dc58c4ca896e8bcf9bee7c1bb80ad10 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(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[master]: aaa: Kerberos negotiation for RESTAPI should be enabled by d...
by mperina＠redhat.com 18 Aug '16

18 Aug '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Kerberos negotiation for RESTAPI should be enabled by default to maintain SDK v3 compatibility ...................................................................... aaa: Kerberos negotiation for RESTAPI should be enabled by default to maintain SDK v3 compatibility The restapi negotiate should be enabled by default, ENGINE_RESTAPI_NEGO=true. This maintains compatibilty with v3 restapi clients. Change-Id: I2da5a99a9dc58c4ca896e8bcf9bee7c1bb80ad10 Bug-Url: https://bugzilla.redhat.com/1367513 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.conf.in 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62469 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2da5a99a9dc58c4ca896e8bcf9bee7c1bb80ad10 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add experimental repo to repo closure script and jobs
by eedri＠redhat.com 17 Aug '16

17 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: Add experimental repo to repo closure script and jobs ...................................................................... Add experimental repo to repo closure script and jobs Change-Id: Id719efb9fa229b4f51944655c6c795fd3a90cd34 Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- M jobs/confs/shell-scripts/repos-check-closure_run-check.sh M jobs/confs/yaml/templates/repos-check-closure.yaml M jobs/packaging/repo_closure_check.sh 3 files changed, 26 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for review Sharon Naftaly: Verified Eyal Edri: Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62384 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id719efb9fa229b4f51944655c6c795fd3a90cd34 Gerrit-PatchSet: 5 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Yamlized repo closure jobs
by eedri＠redhat.com 17 Aug '16

17 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: Yamlized repo closure jobs ...................................................................... Yamlized repo closure jobs Change-Id: Ib16c44e56b8bfe507203a729e33ab9023a3f6f8a Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- A jobs/confs/groovy-scripts/repos-check-closure_post-build.groovy A jobs/confs/projects/repos-check-closure/repos-check-closure.yaml A jobs/confs/shell-scripts/repos-check-closure_apply-cherry-pick.sh A jobs/confs/shell-scripts/repos-check-closure_run-check.sh A jobs/confs/yaml/templates/repos-check-closure.yaml 5 files changed, 114 insertions(+), 0 deletions(-) Approvals: Sharon Naftaly: Verified; Ready for review Eyal Edri: Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62327 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib16c44e56b8bfe507203a729e33ab9023a3f6f8a Gerrit-PatchSet: 7 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <david(a)dcaro.es> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Allow setting "Never" SPM priority
by amureini＠redhat.com 17 Aug '16

17 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: webadmin: Allow setting "Never" SPM priority ...................................................................... webadmin: Allow setting "Never" SPM priority Bug-Url: https://bugzilla.redhat.com/1367399 Change-Id: Ibba84e54743b0dfc14e28a411176adc3549f0d4d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostModel.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/62439 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibba84e54743b0dfc14e28a411176adc3549f0d4d Gerrit-PatchSet: 1 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: Daniel Erez <derez(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: Extract HostSpmPriorityComparator
by amureini＠redhat.com 17 Aug '16

17 Aug '16

Allon Mureinik has submitted this change and it was merged. Change subject: engine: Extract HostSpmPriorityComparator ...................................................................... engine: Extract HostSpmPriorityComparator Extract HostSpmPriorityComparator to a top class in the org.ovirt.core.common.businessentities.comparators package instead of having two identical implementations in VdsDaoImpl and Linq. Change-Id: Ic7b75c24ee74cc6ae2e94e8d19549b39f4b10a93 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/comparators/HostSpmPriorityComparator.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsDaoImpl.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 5 files changed, 19 insertions(+), 19 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/62438 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic7b75c24ee74cc6ae2e94e8d19549b39f4b10a93 Gerrit-PatchSet: 1 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: Daniel Erez <derez(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[ovirt-engine-4.0]: core: Add audit log messages for affinity labels
by tnisan＠redhat.com 17 Aug '16

17 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Add audit log messages for affinity labels ...................................................................... core: Add audit log messages for affinity labels This patch adds messages to the AuditLogMessages.properties file that correspond to the AuditLogType values used for affinity label commands. This ensures messages are displayed in the UI and avoids translation errors in the engine log. Change-Id: Ia7120f4288b825c2a511b4507b1f17021a137cad Bug-Url: https://bugzilla.redhat.com/1351519 Signed-off-by: Phillip Bailey <phbailey(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddLabelCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LabelCommandBase.java M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 3 files changed, 20 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Phillip Bailey: Verified Andrej Krejcir: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/62125 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7120f4288b825c2a511b4507b1f17021a137cad Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Phillip Bailey <phbailey(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Phillip Bailey <phbailey(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: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Make new VM disk mark as bootable by default
by tnisan＠redhat.com 17 Aug '16

17 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Make new VM disk mark as bootable by default ...................................................................... webadmin: Make new VM disk mark as bootable by default Change-Id: If8e36584d8cfd3c8f3b8c469926f612317641e37 Bug-Url: https://bugzilla.redhat.com/1365087 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImageLineModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewDiskModel.java 2 files changed, 7 insertions(+), 0 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62405 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If8e36584d8cfd3c8f3b8c469926f612317641e37 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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[ovirt-engine-4.0]: core: Fix hosted engine SD initial status warning
by tnisan＠redhat.com 17 Aug '16

17 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: core: Fix hosted engine SD initial status warning ...................................................................... core: Fix hosted engine SD initial status warning Changed the warning message displayed in a situation where the hosted engine has been imported, but has no storage domain, to be clearer for users. Change-Id: Ia7ab035cee62600398a59d824adc87900d54174e Bug-Url: https://bugzilla.redhat.com/1358313 Signed-off-by: yquinn <yquinn(a)redhat.com> (cherry picked from commit f31ff2d) Signed-off-by: yquinn <yquinn(a)redhat.com> --- M backend/manager/modules/dal/src/main/resources/bundles/AuditLogMessages.properties 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved Yanir Quinn: Verified -- To view, visit https://gerrit.ovirt.org/62420 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia7ab035cee62600398a59d824adc87900d54174e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.2]: engine: Set default switchType if none is received from vdsm
by tnisan＠redhat.com 17 Aug '16

17 Aug '16

Tal Nisan has submitted this change and it was merged. Change subject: engine: Set default switchType if none is received from vdsm ...................................................................... engine: Set default switchType if none is received from vdsm 3.6 vdsms do not report a switch type. This must be set to legacy, otherwise the network will be reported as out of sync. Change-Id: Ib09c1c826919c8e5084148dd9599612f20f11938 Bug-Url: https://bugzilla.redhat.com/1367483 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Looks good to me, approved Marcin Mirecki: Verified Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/62387 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib09c1c826919c8e5084148dd9599612f20f11938 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0