November 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-java[sdk_3.6]: sdk: Add spec file for Fedora 23
by juan.hernandez＠redhat.com 10 Nov '15

10 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add spec file for Fedora 23 ...................................................................... sdk: Add spec file for Fedora 23 This patch adds an spec file for Fedora 23. Change-Id: I5aafe1e22806cc5f8d1a9d3e398ca40f465eb2d7 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit b3f589c17bda0fbdee2428444916a3b917001aa3) --- A sdk/spec.fc23.in 1 file changed, 377 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48297 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aafe1e22806cc5f8d1a9d3e398ca40f465eb2d7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.6]: sdk: Remove -Xdoclint in EL6 and EL7
by juan.hernandez＠redhat.com 10 Nov '15

10 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Remove -Xdoclint in EL6 and EL7 ...................................................................... sdk: Remove -Xdoclint in EL6 and EL7 The "-Xdoclint" option isn't available in the version of the Javadoc tool available in the EL6 and EL7 distributions. This patch changes the spec files so that they remove it from the POM before the build. Change-Id: I18f0dd87b91bb5c19e3bcf1114985fa3c19575bc Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/spec.el6.in M sdk/spec.el7.in 2 files changed, 12 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48299 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18f0dd87b91bb5c19e3bcf1114985fa3c19575bc Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk-java[sdk_3.6]: sdk: Add -Xdoclint:none to the Javadoc options
by juan.hernandez＠redhat.com 10 Nov '15

10 Nov '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add -Xdoclint:none to the Javadoc options ...................................................................... sdk: Add -Xdoclint:none to the Javadoc options This patch adds "-Xdoclint:none" to the Javadoc optins, as otherwise recent versions of the Javadoc tool fail the build because they don't accept some of the constructs that we use in the Javadoc comments. Change-Id: Ic9930b26213b8c0411884b413d37aa178befa72e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/pom.xml 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48298 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic9930b26213b8c0411884b413d37aa178befa72e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: sdk_3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: aaa: Fix javadoc to remove compilation errors on Oracle JDK 8
by piotr.kliczewski＠gmail.com 10 Nov '15

10 Nov '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: aaa: Fix javadoc to remove compilation errors on Oracle JDK 8 ...................................................................... aaa: Fix javadoc to remove compilation errors on Oracle JDK 8 JDK 8 enables javadoc linter by default and any javadoc error fails the build. There were couple of errors in the HTML syntax I fixed: <p> element does not allow any block element inside it <br/> is an invalid syntax for html 4 Also some generic parameter <T> descriptions were missing. Change-Id: I09cd2754d2de9f01581a5ef871300bf8bf97d2d9 Signed-off-by: Martin Sivák <msivak(a)redhat.com> --- M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/Base.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/ExtMap.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authn.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Authz.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/aaa/Mapping.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/package-info.java 6 files changed, 31 insertions(+), 26 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Sivák: Looks good to me, but someone else must approve Martin Peřina: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/48288 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I09cd2754d2de9f01581a5ef871300bf8bf97d2d9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Remove remains of JPA/Hibernate configuration
by oourfali＠redhat.com 10 Nov '15

10 Nov '15

Oved Ourfali has submitted this change and it was merged. Change subject: core: Remove remains of JPA/Hibernate configuration ...................................................................... core: Remove remains of JPA/Hibernate configuration Removes remains of JPA/Hibernate configurations from application server configuration file. Change-Id: Ic0c5ac4bc6838c8d58fc711a51cf818e84b3ba3c Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/services/ovirt-engine/ovirt-engine.xml.in 1 file changed, 0 insertions(+), 21 deletions(-) Approvals: Martin Peřina: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48303 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic0c5ac4bc6838c8d58fc711a51cf818e84b3ba3c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master 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: automation(a)ovirt.org

1 0

Change in ovirt-release[master]: Updating deps repos
by sbonazzo＠redhat.com 10 Nov '15

10 Nov '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Updating deps repos ...................................................................... Updating deps repos - Updated Gluster repositories GPG key URL - Added Fedora 23 for 3.6 and master Change-Id: Ia106df6e6069fe34e80dada88f05f32ad9345451 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A ovirt-release-3.5/.gitignore M ovirt-release-3.5/configure.ac M ovirt-release-3.5/ovirt-el6-deps.repo.in M ovirt-release-3.5/ovirt-el7-deps.repo.in M ovirt-release-3.5/ovirt-release35.spec.in M ovirt-release-3.6/Makefile.am M ovirt-release-3.6/configure.ac M ovirt-release-3.6/ovirt-el6-deps.repo.in M ovirt-release-3.6/ovirt-el7-deps.repo.in M ovirt-release-3.6/ovirt-f21-deps.repo.in M ovirt-release-3.6/ovirt-f22-deps.repo.in A ovirt-release-3.6/ovirt-f23-deps.repo.in M ovirt-release-3.6/ovirt-release36.spec.in M ovirt-release-master/Makefile.am M ovirt-release-master/configure.ac M ovirt-release-master/ovirt-el7-deps.repo.in M ovirt-release-master/ovirt-f21-deps.repo.in M ovirt-release-master/ovirt-f22-deps.repo.in A ovirt-release-master/ovirt-f23-deps.repo.in M ovirt-release-master/ovirt-release-master.spec.in 20 files changed, 72 insertions(+), 24 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/48325 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia106df6e6069fe34e80dada88f05f32ad9345451 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(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-engine[master]: aaa: AVAILABLE_NAMESPACES are Collection<String> per api spec
by masayag＠redhat.com 10 Nov '15

10 Nov '15

Moti Asayag has submitted this change and it was merged. Change subject: aaa: AVAILABLE_NAMESPACES are Collection<String> per api spec ...................................................................... aaa: AVAILABLE_NAMESPACES are Collection<String> per api spec for some reason it a List<String> was assumed. Change-Id: I0f5e7923e3dddc0ada939bcb277b03401e9c82c9 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1279589 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/GetDirectoryGroupByIdQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/GetDirectoryUserByIdQuery.java 2 files changed, 4 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Ondra Machacek: Verified Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/48314 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0f5e7923e3dddc0ada939bcb277b03401e9c82c9 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: extapi: extmap: do not cast to default type class
by masayag＠redhat.com 10 Nov '15

10 Nov '15

Moti Asayag has submitted this change and it was merged. Change subject: extapi: extmap: do not cast to default type class ...................................................................... extapi: extmap: do not cast to default type class default type can be different subclass of the spec type and may cause casting exception. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1279589 Change-Id: Id6a07149189a9316acc1985b2cc496a439efccfe Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M backend/manager/extensions-tool/src/main/java/org/ovirt/engine/exttool/aaa/AAAServiceImpl.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAuthzGroupsByUserIdQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetAvailableNamespacesQuery.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/DirectoryUtils.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/GetDirectoryGroupsForUserQuery.java M backend/manager/modules/extensions-api-root/extensions-api/src/main/java/org/ovirt/engine/api/extensions/ExtMap.java 6 files changed, 14 insertions(+), 12 deletions(-) Approvals: Alon Bar-Lev: Verified Ondra Machacek: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48318 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id6a07149189a9316acc1985b2cc496a439efccfe Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[ovirt-3.6]: configure: increase release
by dougsland＠redhat.com 09 Nov '15

09 Nov '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: configure: increase release ...................................................................... configure: increase release Change-Id: I178151a156bcfe1cd53f3421860b104d9258d2d4 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/48320 --- M configure.ac 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/48320 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I178151a156bcfe1cd53f3421860b104d9258d2d4 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: mock_runner: in case of no logs, say so
by dcaro＠redhat.com 09 Nov '15

09 Nov '15

David Caro has submitted this change and it was merged. Change subject: mock_runner: in case of no logs, say so ...................................................................... mock_runner: in case of no logs, say so Until now, it just failed with 'file not found' when there was an error and there were no logs yet written to disk Change-Id: Ie919822134dba057fa529cb6b96d7c42eb80e2ea Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M mock_configs/mock_runner.sh 1 file changed, 10 insertions(+), 6 deletions(-) Approvals: Sandro Bonazzola: Looks good to me; Ready for merge David Caro: Verified; Passed CI tests; Ready for review -- To view, visit https://gerrit.ovirt.org/48205 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie919822134dba057fa529cb6b96d7c42eb80e2ea Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(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: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0