April 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Avoid C90 warnings
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Avoid C90 warnings ...................................................................... Avoid C90 warnings Compiling the C extension with the C90 standard generates some warnings due to the use of declaration and assignment in the same statement. This patch changes the code to avoid them. Change-Id: Ifce6ed0a58e36c0e71836b0d675889e841e99a56 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/ext/ovirtsdk4c/ov_xml_reader.c M sdk/ext/ovirtsdk4c/ov_xml_writer.c 2 files changed, 72 insertions(+), 41 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55864 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifce6ed0a58e36c0e71836b0d675889e841e99a56 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Use httpclient version 4.5
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Use httpclient version 4.5 ...................................................................... Use httpclient version 4.5 This patch add support for httpclient version 4.5. Also clean some parts of unused code Change-Id: Ie6695191621edf25535a7ca027e3ac1650af7f58 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/pom.xml M sdk/pom.xml M sdk/src/main/java/org/ovirt/engine/sdk/ConnectionBuilder.java M sdk/src/main/java/org/ovirt/engine/sdk/internal/HttpConnection.java 4 files changed, 17 insertions(+), 74 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55851 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie6695191621edf25535a7ca027e3ac1650af7f58 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk-java 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-sdk-ruby[master]: Add rake-compiler to the .gemspec file
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add rake-compiler to the .gemspec file ...................................................................... Add rake-compiler to the .gemspec file This patch adds the requirement for the "rake-compiler" gem to the .gemspec file, as that gem is required to build the C extension. Change-Id: Ic44432183d008c3e7283e05da3e015e7e8d0ba03 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/ovirt-engine-sdk.gemspec 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55862 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic44432183d008c3e7283e05da3e015e7e8d0ba03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Prepare the POM of SDK for releasing
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Prepare the POM of SDK for releasing ...................................................................... Prepare the POM of SDK for releasing This patch prepares the POM of the SDK to have it ready for releasing to Maven Central. Change-Id: I9f3904c6ed676a85b92d63f19a87d80dcd3df5ce Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M sdk/README.adoc M sdk/pom.xml 2 files changed, 192 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55838 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9f3904c6ed676a85b92d63f19a87d80dcd3df5ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(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-sdk-java[master]: Add support for follow link
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add support for follow link ...................................................................... Add support for follow link This patch is adding missing support for follow link feature for StructTypes. Support for ListType is missing. Change-Id: I9f9e119d476fcbcc038860a4a6e3d6e2ddedacd6 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/java/Tool.java M sdk/src/main/java/org/ovirt/engine/sdk/internal/HttpConnection.java 2 files changed, 27 insertions(+), 11 deletions(-) Approvals: Ondra Machacek: Verified Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55845 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9f9e119d476fcbcc038860a4a6e3d6e2ddedacd6 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine-sdk-java 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-sdk-java[master]: Add automation and packaging scripts
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: Add automation and packaging scripts ...................................................................... Add automation and packaging scripts This patch adds the automation scripts and the RPM packaging scripts and spec files. In order to be able to package the generated SDK, and in preparation to be able to publish it in Maven Central, the patch also changes the POM files of the project, so that the POM of the SDK itself doesn't use the POM of the project as parent. This also means that the configurations needed to run the generator have to be moved out of the POM of the SDK, and into the POM of the generator. Change-Id: I80d140e9504dc0948dbad1b2c0fa9a1e1b8ac21b Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M .gitignore M README.adoc A automation/README.adoc A automation/build-artifacts.packages A automation/build-artifacts.sh A automation/check-merged.packages A automation/check-merged.sh A automation/check-patch.packages A automation/check-patch.sh A automation/version.py M generator/pom.xml A packaging/README.adoc A packaging/build.sh A packaging/spec.el7.centos.in A packaging/spec.el7.in A packaging/spec.fc23.in M pom.xml M sdk/pom.xml 18 files changed, 443 insertions(+), 122 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/55837 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I80d140e9504dc0948dbad1b2c0fa9a1e1b8ac21b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> 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 jenkins[master]: ovirt-dwh: add 3.6 branch and remove master
by sbonazzo＠redhat.com 08 Apr '16

08 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-dwh: add 3.6 branch and remove master ...................................................................... ovirt-dwh: add 3.6 branch and remove master Change-Id: I3bcec20517bc42b661a9eb73d4da97fa523f2776 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M jobs/confs/projects/ovirt-dwh/ovirt-dwh_create-rpms.yaml 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55822 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3bcec20517bc42b661a9eb73d4da97fa523f2776 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Add links to SSH public keys
by juan.hernandez＠redhat.com 08 Apr '16

08 Apr '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add links to SSH public keys ...................................................................... restapi: Add links to SSH public keys Currently the SSH public keys added to users don't have their links populated. This patch fixes that issue. Change-Id: I6d4cf76efa85e3acab6ab8d61c910de5cfb999fb Bug-Url: https://bugzilla.redhat.com/1324935 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/util/LinkHelper.java 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/55842 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6d4cf76efa85e3acab6ab8d61c910de5cfb999fb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Add jobs to build version 4 of the Java SDK
by sbonazzo＠redhat.com 08 Apr '16

08 Apr '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add jobs to build version 4 of the Java SDK ...................................................................... Add jobs to build version 4 of the Java SDK This patch adds new jobs to build version 4 of the Java SDK, from the master branch of the project. Change-Id: I6287ee6e83af276c5ee3357166687633a8cf136f Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A jobs/confs/projects/ovirt-engine-sdk-java/ovirt-engine-sdk-java_build-artifacts.yaml 1 file changed, 33 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/55849 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6287ee6e83af276c5ee3357166687633a8cf136f Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Don't run Java SDK 3.6 build jobs on master merge
by eedri＠redhat.com 08 Apr '16

08 Apr '16

Eyal Edri has submitted this change and it was merged. Change subject: Don't run Java SDK 3.6 build jobs on master merge ...................................................................... Don't run Java SDK 3.6 build jobs on master merge This patch changes the existing Java SDK build job so that it doesn't build the SDK when changes are merged to the master branch. These builds will be performed by other jobs introduced in the next patch. Eventually, when there is no need to build the 3.6 Java SDK, this job will be removed. Change-Id: I27a81be8af242dd5dac441c4a7128171738cdbe3 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M jobs/confs/projects/ovirt-engine-sdk-java/ovirt-engine-sdk-java_create-rpms.yaml 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Juan Hernandez: Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55848 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27a81be8af242dd5dac441c4a7128171738cdbe3 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0