- Engine-commits - oVirt List Archives

Change in ovirt-live[master]: drop post chroot install of yad
by sbonazzo＠redhat.com 12 Sep '14

12 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: drop post chroot install of yad ...................................................................... drop post chroot install of yad yad is already installed by the kickstart using 0.27.0. No reason to try to update to a legacy version. Change-Id: I4fc06e99b86778f1829882b195282d53916e44db Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/kickstart/ovirt-live-ovirt-custom.ks.in D centos/oVirtLiveFiles/rpms/repodata/283c3ad68f6590ab7be9b55d6cbcec4f8801c7fc9d6a3a029aef282a15cca8bc-filelists.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/5f443c702d353ece1e70fb6fdcd957f8b51ca48d377285bd7e5deaf4fcb8e527-filelists.xml.gz D centos/oVirtLiveFiles/rpms/repodata/61e4ecd65729eea3ea701911223debda48c620c0a31958e89ca1c3fd7b6910ef-other.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/80a0c9d20118ed501861cc4dfd635020d432e212ade5611d14f7f461e2e911ce-other.xml.gz D centos/oVirtLiveFiles/rpms/repodata/89b0b69ff746917b3c41818919953e694b15aff5a5c8bb963d275ffd3c49c03c-primary.xml.gz D centos/oVirtLiveFiles/rpms/repodata/dea2d8919301bd2ac8b65441596d06f69c551f380b5c99ba42fac792896438c8-primary.sqlite.bz2 D centos/oVirtLiveFiles/rpms/repodata/repomd.xml D centos/oVirtLiveFiles/rpms/yad-0.14.2-1.fc14.x86_64.rpm 9 files changed, 0 insertions(+), 56 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32869 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4fc06e99b86778f1829882b195282d53916e44db Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-live[master]: network: updated network configuration
by sbonazzo＠redhat.com 12 Sep '14

12 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: network: updated network configuration ...................................................................... network: updated network configuration Updated network configuration for matching new requirements for vdsm and ovirt-engine. Fixes permissions on network scripts files. Change-Id: I17f6df9a8fa5c74b8110d9db593601aad1b7af32 Bug-Url: https://bugzilla.redhat.com/1139687 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos/kickstart/ovirt-live-ovirt-custom.ks.in M centos/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-dummy0 M centos/oVirtLiveFiles/root/etc/sysconfig/network-scripts/ifcfg-ovirtmgmt A centos/oVirtLiveFiles/root/etc/sysconfig/network-scripts/route-ovirtmgmt A centos/oVirtLiveFiles/root/etc/sysconfig/network-scripts/rule-ovirtmgmt 5 files changed, 17 insertions(+), 7 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32868 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I17f6df9a8fa5c74b8110d9db593601aad1b7af32 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-live[ovirt-live-3.5]: packaging: setup: drop legacy 3.2 patch
by sbonazzo＠redhat.com 12 Sep '14

12 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: drop legacy 3.2 patch ...................................................................... packaging: setup: drop legacy 3.2 patch Change-Id: Ibeff28ec86c5efafeb67c9a20070275f17748d7f Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit 3883e3603f541f537cf18612ed21b6f3fb6e5d53) --- D centos/oVirtLiveFiles/patches/0001-oVirtLive-patches.patch 1 file changed, 0 insertions(+), 91 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32865 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibeff28ec86c5efafeb67c9a20070275f17748d7f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.5 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: placeholders of child commands aren't cleared when f...
by oourfali＠redhat.com 12 Sep '14

12 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: engine: placeholders of child commands aren't cleared when failing during the CDA phase ...................................................................... engine: placeholders of child commands aren't cleared when failing during the CDA phase AsyncTask placeholders for child commands are not cleared when canDoAction returns false. Change-Id: Ie77a00b0071c92d074d837768af012c80374169f Bug-Url: https://bugzilla.redhat.com/1139678 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Ravi Nori: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32790 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie77a00b0071c92d074d837768af012c80374169f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: changing error severity of illegal search
by oourfali＠redhat.com 12 Sep '14

12 Sep '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: changing error severity of illegal search ...................................................................... core: changing error severity of illegal search WARN message logged versus DEBUG or INFO message with rhevm-shell provided bogus search This patch changes the severity of illegal searches done by the user from WARN to INFO Change-Id: Ic6abcdc24e1538937cc317c2fc13a913cb542786 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1133553 Signed-off-by: Eli Mesika <emesika(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/SearchQuery.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Eli Mesika: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32839 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic6abcdc24e1538937cc317c2fc13a913cb542786 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi: Add HttpOnly flag to session cookie
by piotr.kliczewski＠gmail.com 12 Sep '14

12 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Add HttpOnly flag to session cookie ...................................................................... restapi: Add HttpOnly flag to session cookie The CSRF protection mechanism has been implemented in a way that doesn't require reading the value of this cookie, so the flag can be enabled now. Change-Id: Id0a315fda675ec2c606589f9028d7284d68496d5 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32840 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0a315fda675ec2c606589f9028d7284d68496d5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com>

1 0

Change in ovirt-engine[master]: restapi: Add HttpOnly flag to session cookie
by juan.hernandez＠redhat.com 12 Sep '14

12 Sep '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add HttpOnly flag to session cookie ...................................................................... restapi: Add HttpOnly flag to session cookie The CSRF protection mechanism has been implemented in a way that doesn't require reading the value of this cookie, so the flag can be enabled now. Change-Id: Id0a315fda675ec2c606589f9028d7284d68496d5 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified Alexander Wels: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32833 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0a315fda675ec2c606589f9028d7284d68496d5 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: aaa: use Authn.Capabilities.AUTHENTICATE_CREDENTIALS correctly
by piotr.kliczewski＠gmail.com 12 Sep '14

12 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: aaa: use Authn.Capabilities.AUTHENTICATE_CREDENTIALS correctly ...................................................................... aaa: use Authn.Capabilities.AUTHENTICATE_CREDENTIALS correctly Topic: AAA Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120720 Change-Id: I8ed1d4523ddf2bcefb3f1b1be868e4caf6c04ad7 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/internal/InternalAuthn.java M backend/manager/modules/builtin-extensions/src/main/java/org/ovirt/engine/extensions/aaa/builtin/kerberosldap/KerberosLdapAuthn.java 3 files changed, 30 insertions(+), 11 deletions(-) Approvals: Alon Bar-Lev: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/32804 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8ed1d4523ddf2bcefb3f1b1be868e4caf6c04ad7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: frontend: Fix validation of UnitVmModel
by piotr.kliczewski＠gmail.com 12 Sep '14

12 Sep '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: frontend: Fix validation of UnitVmModel ...................................................................... frontend: Fix validation of UnitVmModel Individual entity models were validated but the validate() method always returned true since the validity of individual tabs was not taken into account. This resulted in submitting invalid edits and subsequent backend validation errors. Change-Id: Iabba7f93a0541292a1a4c82b675ab0faf141b9b2 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128444 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/UnitVmModel.java 1 file changed, 18 insertions(+), 13 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Martin Betak: Verified -- To view, visit http://gerrit.ovirt.org/32694 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iabba7f93a0541292a1a4c82b675ab0faf141b9b2 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.2]: packaging: updated dependencies
by sbonazzo＠redhat.com 12 Sep '14

12 Sep '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: updated dependencies ...................................................................... packaging: updated dependencies updated dependencies with packages released in 3.5.0 RC2. Change-Id: I10b888f6a307256a4227cb013610064037d732f8 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 11 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/32842 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I10b888f6a307256a4227cb013610064037d732f8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0