June 2016 - Engine-commits - oVirt List Archives

Change in jenkins[master]: hosted-engine: drop fc22 build on 4.0
by eedri＠redhat.com 19 Jun '16

19 Jun '16

Eyal Edri has submitted this change and it was merged. Change subject: hosted-engine: drop fc22 build on 4.0 ...................................................................... hosted-engine: drop fc22 build on 4.0 Change-Id: Ic2593d5e01bbea10d1a5d2e806825f00cfffbcfe Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M jobs/confs/projects/ovirt-hosted-engine-setup/ovirt-hosted-engine_stardard.yaml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Eyal Edri: Looks good to me; Ready for merge Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/59427 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2593d5e01bbea10d1a5d2e806825f00cfffbcfe Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: use generic callback in MergeExtend
by laravot＠redhat.com 19 Jun '16

19 Jun '16

Liron Aravot has submitted this change and it was merged. Change subject: engine: use generic callback in MergeExtend ...................................................................... engine: use generic callback in MergeExtend Use ConcurrentChildCommandsExecutionCallback in MergeExtend command instead of MergeExtendCommandCallback, in order to minimize the use of non-generic callbacks. Change-Id: I828c3a3fd0cc006e81a16271ab33fab8e7c67518 Signed-off-by: Fred Rolland <frolland(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommand.java D backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommandCallback.java 2 files changed, 1 insertion(+), 44 deletions(-) Approvals: Jenkins CI: Passed CI tests Freddy Rolland: Verified Ala Hino: Looks good to me, but someone else must approve Liron Aravot: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/53830 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I828c3a3fd0cc006e81a16271ab33fab8e7c67518 Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: aaa: Logout should not be invoked on non-persistent Bearer A...
by oourfali＠redhat.com 18 Jun '16

18 Jun '16

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: Logout should not be invoked on non-persistent Bearer Auth api access ...................................................................... aaa: Logout should not be invoked on non-persistent Bearer Auth api access When the RESTAPI is accessed using Bearer Authentication, the RestApiSessionMgmtFilter should not invoke Logout. Change-Id: I01abfdfbcc8706c14b0f031d8ece2fab7674c6eb Bug-Url: https://bugzilla.redhat.com/1347478 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/RestApiSessionMgmtFilter.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59415 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01abfdfbcc8706c14b0f031d8ece2fab7674c6eb 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: Oved Ourfali <oourfali(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: Logout should not be invoked on non-persistent Bearer A...
by mperina＠redhat.com 18 Jun '16

18 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: aaa: Logout should not be invoked on non-persistent Bearer Auth api access ...................................................................... aaa: Logout should not be invoked on non-persistent Bearer Auth api access When the RESTAPI is accessed using Bearer Authentication, the RestApiSessionMgmtFilter should not invoke Logout. Change-Id: I01abfdfbcc8706c14b0f031d8ece2fab7674c6eb Bug-Url: https://bugzilla.redhat.com/1347478 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/RestApiSessionMgmtFilter.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59414 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01abfdfbcc8706c14b0f031d8ece2fab7674c6eb 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: Oved Ourfali <oourfali(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]: publishers:Remove fingerprinting and job stability check
by dcaro＠redhat.com 17 Jun '16

17 Jun '16

David Caro has submitted this change and it was merged. Change subject: publishers:Remove fingerprinting and job stability check ...................................................................... publishers:Remove fingerprinting and job stability check Change-Id: Ib766b409f5de18d9f1cd7c4f9b2754ddfc1837f9 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M jobs/confs/yaml/builders/copy-create-job-artifact.yaml 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me Jenkins CI: Passed CI tests David Caro: Verified; Ready for merge -- To view, visit https://gerrit.ovirt.org/59400 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib766b409f5de18d9f1cd7c4f9b2754ddfc1837f9 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(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 <dcaro(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(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-release[ovirt-3.6]: repo: using gluster 3.7 repo for f22
by fabiand＠redhat.com 17 Jun '16

17 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: repo: using gluster 3.7 repo for f22 ...................................................................... repo: using gluster 3.7 repo for f22 gluster 3.8 has been released on June 14th so gluster LATEST now points to 3.8 but 3.8 is not going to be built for f22. Change-Id: Ib3784a667f785467c48e23f71717985f0586a214 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M ovirt-f22-deps.repo.in 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/59384 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib3784a667f785467c48e23f71717985f0586a214 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Rafael Martins <rmartins(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]: release: Fix nightly trigger
by dcaro＠redhat.com 17 Jun '16

17 Jun '16

David Caro has submitted this change and it was merged. Change subject: release: Fix nightly trigger ...................................................................... release: Fix nightly trigger Change-Id: I8a714c09f019d2605f3e8ce30ebcddaef4adc7b0 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M jobs/confs/projects/ovirt-release/ovirt-release_standard.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Fabian Deutsch: Verified; Looks good to me; Ready for review Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me; Ready for merge -- To view, visit https://gerrit.ovirt.org/59397 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a714c09f019d2605f3e8ce30ebcddaef4adc7b0 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Fix of NPE while creating Template from Snapshot
by piotr.kliczewski＠gmail.com 17 Jun '16

17 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Fix of NPE while creating Template from Snapshot ...................................................................... webadmin: Fix of NPE while creating Template from Snapshot Problem: Setting entity value of UnitVmModel property `isHighlyAvailable` in VmSnapshotListModel.cloneTemplate method invoked listener method VmModelBehaviorBase.updateMigrationAvailability in which NPE was thrown since `isAutoAssign` was null. This problem appeared e.g. if the snapshoted VM was highly available. Fix: Method updateMigrationAvailability() just returns if UnitVmModel is not yet fully initialized - required properties are `null`. Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Bug-Url: https://bugzilla.redhat.com/1346095 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 1 file changed, 7 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59341 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Fix of NPE while creating Template from Snapshot
by piotr.kliczewski＠gmail.com 17 Jun '16

17 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Fix of NPE while creating Template from Snapshot ...................................................................... webadmin: Fix of NPE while creating Template from Snapshot Problem: Setting entity value of UnitVmModel property `isHighlyAvailable` in VmSnapshotListModel.cloneTemplate method invoked listener method VmModelBehaviorBase.updateMigrationAvailability in which NPE was thrown since `isAutoAssign` was null. This problem appeared e.g. if the snapshoted VM was highly available. Fix: Method updateMigrationAvailability() just returns if UnitVmModel is not yet fully initialized - required properties are `null`. Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Bug-Url: https://bugzilla.redhat.com/1346095 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java 1 file changed, 7 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59342 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bf28adbda195dd4d60344764ac800d9d41d3092 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Initialization of UnitVmModel in VmSnapshotListModel
by tjelinek＠redhat.com 17 Jun '16

17 Jun '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Initialization of UnitVmModel in VmSnapshotListModel ...................................................................... webadmin: Initialization of UnitVmModel in VmSnapshotListModel Properties of UnitVmModel `vmType` and `isHighlyAvailable` are now set after `initialize()` method is called in VmSnapshotListModel#cloneTemplate. Change-Id: Iff99493c519133a235ff9a173ab565707dc4cf06 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmSnapshotListModel.java 1 file changed, 2 insertions(+), 3 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/59380 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iff99493c519133a235ff9a173ab565707dc4cf06 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0