December 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: Add dependency on ovirt-web-ui
by Code Review 02 Dec '16

02 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: core: Add dependency on ovirt-web-ui ...................................................................... core: Add dependency on ovirt-web-ui Dependency on ovirt-web-ui package is added to ovirt-engine.spec. Package is built in Copr and published in http://resources.ovirt.org/pub/ovirt-master-snapshot-static Change-Id: I153d9f62e73e9ee4e66241c8d8a469469675e674 Project: https://github.com/mareklibra/userportal Trello: https://trello.com/c/blwDpjcJ/24-poc-userportal-replacement Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/67625 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I153d9f62e73e9ee4e66241c8d8a469469675e674 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: aaa: Handle session creation during clean expired user sessions
by Code Review 02 Dec '16

02 Dec '16

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: aaa: Handle session creation during clean expired user sessions ...................................................................... aaa: Handle session creation during clean expired user sessions If a session is created during the execution of cleanExpiredUserSessions after the retrival of session statuses from the server the session should not be handled until the next iteration. Change-Id: I9b62048fb4b4acc67079ebe53715c10bed68f1ed Bug-Url: https://bugzilla.redhat.com/1396833 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java 1 file changed, 7 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67724 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b62048fb4b4acc67079ebe53715c10bed68f1ed Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix potential NPE in SsoPostLoginFilter
by Code Review 02 Dec '16

02 Dec '16

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: core: Fix potential NPE in SsoPostLoginFilter ...................................................................... core: Fix potential NPE in SsoPostLoginFilter The GetUserBySessionId query might return null, this patch provides fix. Change-Id: I42315cd5949a85e6ed4864469f4b6e85dd8ff6fc Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/SsoPostLoginFilter.java 1 file changed, 8 insertions(+), 3 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/67726 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I42315cd5949a85e6ed4864469f4b6e85dd8ff6fc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(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]: Updated Fedora distro list for repoman (-fc23 +fc25)
by Code Review 02 Dec '16

02 Dec '16

>From Anton Marchukov <amarchuk(a)redhat.com>: Anton Marchukov has submitted this change and it was merged. Change subject: Updated Fedora distro list for repoman (-fc23 +fc25) ...................................................................... Updated Fedora distro list for repoman (-fc23 +fc25) Change-Id: Ia34ddee88cf4ccb4307da61bb92663ee0bf0507a Signed-off-by: Anton Marchukov <amarchuk(a)redhat.com> --- M jobs/confs/projects/repoman/repoman_build-artifacts.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Anton Marchukov: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/67448 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia34ddee88cf4ccb4307da61bb92663ee0bf0507a Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-ha[master]: he: Prevent VM from migrating when there is no host for it
by Code Review 01 Dec '16

01 Dec '16

>From Martin Sivák <msivak(a)redhat.com>: Martin Sivák has submitted this change and it was merged. Change subject: he: Prevent VM from migrating when there is no host for it ...................................................................... he: Prevent VM from migrating when there is no host for it With this patch VM migration during transition to LocalMaintenance mode is prevented if all other HE hosts are already in LocalMaintenane mode or not capable to run th VM. Change-Id: Idf5030aeada317a01b68b825f74eba57bc6d2c56 Bug-Url: https://bugzilla.redhat.com/1394570 Signed-off-by: Denis Chaplygin <dchaplyg(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/state_machine.py M ovirt_hosted_engine_ha/agent/states.py 2 files changed, 6 insertions(+), 2 deletions(-) Approvals: Denis Chaplygin: Verified Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67256 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idf5030aeada317a01b68b825f74eba57bc6d2c56 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: master Gerrit-Owner: Denis Chaplygin <dchaplyg(a)redhat.com> Gerrit-Reviewer: Andrej Krejcir <akrejcir(a)redhat.com> Gerrit-Reviewer: Denis Chaplygin <dchaplyg(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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yanir Quinn <yquinn(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: projects: add ovirt-node-ng builders for 4.1
by Code Review 01 Dec '16

01 Dec '16

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: projects: add ovirt-node-ng builders for 4.1 ...................................................................... projects: add ovirt-node-ng builders for 4.1 Change-Id: I3e6bffc320f56b5204b3dc6d7b7a0df0ebd308db Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M jobs/confs/projects/ovirt-node-ng/ovirt-node-ng.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Passed CI tests; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67685 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3e6bffc320f56b5204b3dc6d7b7a0df0ebd308db Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(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 jenkins[master]: projects: add 4.1 builder for imgbased
by Code Review 01 Dec '16

01 Dec '16

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: projects: add 4.1 builder for imgbased ...................................................................... projects: add 4.1 builder for imgbased Also, correct the autobuilds for 4.0 so they're built off of ovirt-4.0 instead of master Change-Id: I0ba9dc24472132b438edfd013ef9869294733443 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M jobs/confs/projects/imgbased/imgbased.yaml 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Eyal Edri: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67684 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0ba9dc24472132b438edfd013ef9869294733443 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(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 jenkins[master]: projects: add appliance builders for 4.1
by Code Review 01 Dec '16

01 Dec '16

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: projects: add appliance builders for 4.1 ...................................................................... projects: add appliance builders for 4.1 Change-Id: I072dc55716aae7ee6d2c76ada6e1ec6120964166 Signed-off-by: Ryan Barry <rbarry(a)redhat.com> --- M jobs/confs/projects/ovirt-appliance/ovirt-appliance.yaml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67702 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I072dc55716aae7ee6d2c76ada6e1ec6120964166 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Ryan Barry <rbarry(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-engine-sdk-ruby[master]: Update cloud-init example with custom script
by Code Review 01 Dec '16

01 Dec '16

>From Juan Hernandez <juan.hernandez(a)redhat.com>: Juan Hernandez has submitted this change and it was merged. Change subject: Update cloud-init example with custom script ...................................................................... Update cloud-init example with custom script This patch updates run vm with cloud-init example, with custom script, which will be executed when VM is deployed. Change-Id: I9de04e90187fe001f63378b22631cfc6f7278d8a Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M sdk/examples/start_vm_with_cloud_init.rb 1 file changed, 13 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/67641 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9de04e90187fe001f63378b22631cfc6f7278d8a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Reduce the severity of check-upgrade log
by Code Review 01 Dec '16

01 Dec '16

>From Martin Peřina <mperina(a)redhat.com>: Martin Peřina has submitted this change and it was merged. Change subject: engine: Reduce the severity of check-upgrade log ...................................................................... engine: Reduce the severity of check-upgrade log When there is an attempt to check for an upgrade, while another process is already running, the reported message is logged with an ERROR severity. This specific message severity can be reduced to warning. In addition, there is an inflation of error messages during the process which can be reduced. Change-Id: Ia325f85527491ce2c1a3a16263d2b68867f63b2a Bug-Url: https://bugzilla.redhat.com/1400500 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/AvailableUpdatesFinder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/HostUpgradeManager.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 3 files changed, 6 insertions(+), 8 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/67671 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia325f85527491ce2c1a3a16263d2b68867f63b2a Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0