July 2016 - Engine-commits - oVirt List Archives

Change in jenkins[master]: otopi: move from fc22 to fc24
by sbonazzo＠redhat.com 12 Jul '16

12 Jul '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: otopi: move from fc22 to fc24 ...................................................................... otopi: move from fc22 to fc24 - Drop fc22 builds since fc22 gone EOL - Move master to fc24 Change-Id: Ic29b4916af7d0f3c031e951b0b6d40f8b2824ed1 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/otopi/otopi_create-rpms.yaml M jobs/confs/projects/otopi/otopi_standard.yaml 2 files changed, 6 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Eyal Edri: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60545 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic29b4916af7d0f3c031e951b0b6d40f8b2824ed1 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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 jenkins[master]: Replaced "big" if with smarter expression
by eedri＠redhat.com 12 Jul '16

12 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: Replaced "big" if with smarter expression ...................................................................... Replaced "big" if with smarter expression Just some better writing syntax Change-Id: Ic8a71812d18a1b57e8ece6de44415801bb364076 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- M jobs/confs/groovy-scripts/system_report.groovy 1 file changed, 8 insertions(+), 22 deletions(-) Approvals: Eyal Edri: Ready for merge Barak Korren: Looks good to me Gil Shinar: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60461 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic8a71812d18a1b57e8ece6de44415801bb364076 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Gil Shinar <gshinar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[otopi-1.5]: packaging: post otopi-1.5.1
by didi＠redhat.com 12 Jul '16

12 Jul '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: post otopi-1.5.1 ...................................................................... packaging: post otopi-1.5.1 Change-Id: I09264108a549baadedd73eb1bb961fe0b6c955a1 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac 4 files changed, 7 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60543 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09264108a549baadedd73eb1bb961fe0b6c955a1 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.5 Gerrit-Owner: Yedidyah Bar David <didi(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 otopi[otopi-1.5]: packaging: otopi-1.5.1
by didi＠redhat.com 12 Jul '16

12 Jul '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: packaging: otopi-1.5.1 ...................................................................... packaging: otopi-1.5.1 Change-Id: Ia79af236053a9457665e92f88bcc220b86f07f3b Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ChangeLog M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac M otopi.spec.in 5 files changed, 9 insertions(+), 5 deletions(-) Approvals: Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60541 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia79af236053a9457665e92f88bcc220b86f07f3b Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.5 Gerrit-Owner: Yedidyah Bar David <didi(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 otopi[otopi-1.5]: network: firewalld: Fix version check
by didi＠redhat.com 12 Jul '16

12 Jul '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: network: firewalld: Fix version check ...................................................................... network: firewalld: Fix version check A recent change to firewalld stopped exposing the version through 'client', but only through 'config' - which it always did. Bug-Url: https://bugzilla.redhat.com/1354607 Change-Id: Ief793a3d0c2076630e04521e8d442cb47af55ea4 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 9a83b4aa679f13cdf34f528a1a3b813a37425a2c) --- M src/plugins/otopi/network/firewalld.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60540 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief793a3d0c2076630e04521e8d442cb47af55ea4 Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: otopi-1.5 Gerrit-Owner: Yedidyah Bar David <didi(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 otopi[master]: network: firewalld: Fix version check
by didi＠redhat.com 12 Jul '16

12 Jul '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: network: firewalld: Fix version check ...................................................................... network: firewalld: Fix version check A recent change to firewalld stopped exposing the version through 'client', but only through 'config' - which it always did. Bug-Url: https://bugzilla.redhat.com/1354607 Change-Id: Ief793a3d0c2076630e04521e8d442cb47af55ea4 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/plugins/otopi/network/firewalld.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60518 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief793a3d0c2076630e04521e8d442cb47af55ea4 Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(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]: webadmin: Do not fail CI for 'too big to analyze' reason
by juan.hernandez＠redhat.com 12 Jul '16

12 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: webadmin: Do not fail CI for 'too big to analyze' reason ...................................................................... webadmin: Do not fail CI for 'too big to analyze' reason Change-Id: I05f8d6e41e8825155983fdf0a839680d7d4b2374 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- M frontend/webadmin/modules/uicompat/exclude-filters.xml 1 file changed, 6 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Martin Mucha: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60470 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I05f8d6e41e8825155983fdf0a839680d7d4b2374 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: instance type not reported for pools
by tnisan＠redhat.com 12 Jul '16

12 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: restapi: instance type not reported for pools ...................................................................... restapi: instance type not reported for pools Added the same way as the templates have it. Change-Id: Iae0c7b5c4f17ad7961ed41da196dbc4e1217db2a Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1310804 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java 1 file changed, 8 insertions(+), 4 deletions(-) Approvals: Tomas Jelinek: Verified Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60458 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae0c7b5c4f17ad7961ed41da196dbc4e1217db2a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.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]: core, restapi: /graphicsconsoles did not work properly with '...
by tnisan＠redhat.com 12 Jul '16

12 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core,restapi: /graphicsconsoles did not work properly with 'current' ...................................................................... core,restapi: /graphicsconsoles did not work properly with 'current' The ;current did not work properly with /graphicsconsoles. Fixed by adding a new query GetNextRunGraphicsDevicesQuery which returns the graphics consoles from the stateless snapshot and using it on REST resources. Change-Id: I76cc883580c38d8aa3c64863f7de41df7ea9ddbf Bug-Url: https://bugzilla.redhat.com/1128453 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetNextRunGraphicsDevicesQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmByVmIdQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmNextRunConfigurationQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendGraphicsConsoleResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendGraphicsConsolesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/DisplayHelper.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmResourceTest.java 11 files changed, 89 insertions(+), 21 deletions(-) Approvals: Tomas Jelinek: Verified Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60459 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76cc883580c38d8aa3c64863f7de41df7ea9ddbf Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.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]: engine : Missing replacement values for add cluster audit lo...
by tnisan＠redhat.com 12 Jul '16

12 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: engine : Missing replacement values for add cluster audit log message ...................................................................... engine : Missing replacement values for add cluster audit log message Adding a new cluster generates audit log messages where the replacement values are missing and shows up as <UNKNOWN>. User/Group Everyone, Namespace *, Authorization provider: <UNKNOWN> was granted permission for Role CpuProfileOperator on Cpu Profile <UNKNOWN>, by admin@internal-authz. Change-Id: I82b936fabe938296a19857ba873953eca9696e8e Bug-Url: https://bugzilla.redhat.com/1350353 Signed-off-by: Ravi Nori <rnori(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/PermissionsCommandBase.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/profiles/AddCpuProfileCommand.java 2 files changed, 11 insertions(+), 2 deletions(-) Approvals: Martin Peřina: Looks good to me, approved Ravi Nori: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60365 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82b936fabe938296a19857ba873953eca9696e8e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0