March 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: pki: Create ca cert with no text
by Code Review 03 Mar '17

03 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: pki: Create ca cert with no text ...................................................................... packaging: pki: Create ca cert with no text Change-Id: I01ec5d8afd73e0fcbdd31684ee214d2d23af586c Bug-Url: https://bugzilla.redhat.com/1406858 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/bin/pki-create-ca.sh 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/69597 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I01ec5d8afd73e0fcbdd31684ee214d2d23af586c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Rafael Martins <rmartins(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: adding job for master ansible suite
by Code Review 03 Mar '17

03 Mar '17

>From Eyal Edri <eedri(a)redhat.com>: Eyal Edri has submitted this change and it was merged. Change subject: adding job for master ansible suite ...................................................................... adding job for master ansible suite Change-Id: I14fa49e520147fc481950a86d0c0d997dfba0b64 Signed-off-by: Eyal Edri <eedri(a)redhat.com> --- M jobs/confs/projects/ovirt/system-tests.yaml M jobs/confs/yaml/templates/system_tests_templates.yaml 2 files changed, 26 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me Eyal Edri: Verified; Looks good to me, approved; Ready for merge Daniel Belenky: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73456 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14fa49e520147fc481950a86d0c0d997dfba0b64 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Daniel Belenky <dbelenky(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.1]: build: fix el7 repo
by Code Review 03 Mar '17

03 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: build: fix el7 repo ...................................................................... build: fix el7 repo it was checkinf for gpg key files not installed on the system Change-Id: I2d2f1c844640e699b39affab125bd12c564f5a0b --- M configure.ac M ovirt-el7-x86_64-deps.repo.in M ovirt-release41.spec.in 3 files changed, 6 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/73535 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2d2f1c844640e699b39affab125bd12c564f5a0b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: nsis-simple-service-plugin: drop fc24 on master
by Code Review 03 Mar '17

03 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: nsis-simple-service-plugin: drop fc24 on master ...................................................................... nsis-simple-service-plugin: drop fc24 on master fc24 will be EOL when 4.2 will be out Change-Id: I9f24b4602892a6b5c49e89015e15f474fa501527 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/nsis-simple-service-plugin/nsis-simple-service-plugin.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.1.yaml 2 files changed, 18 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73055 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9f24b4602892a6b5c49e89015e15f474fa501527 Gerrit-PatchSet: 4 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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]: pywin32-py2.7: drop fc24 on master
by Code Review 03 Mar '17

03 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: pywin32-py2.7: drop fc24 on master ...................................................................... pywin32-py2.7: drop fc24 on master fc24 will be EOL before 4.2 release Change-Id: Id19aa3174d9ac2907f53b619de27e946b41dd6af Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/pywin32-py2.7/pywin32-py2.7.yaml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73533 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id19aa3174d9ac2907f53b619de27e946b41dd6af Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(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-engine-sdk[sdk_4.1]: [maven-release-plugin] prepare release 4.1.2
by Code Review 03 Mar '17

03 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: [maven-release-plugin] prepare release 4.1.2 ...................................................................... [maven-release-plugin] prepare release 4.1.2 Change-Id: Id69d609c35e4d619777060497fd8ea48f1af71ec Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/pom.xml M pom.xml M sdk/pom.xml 3 files changed, 4 insertions(+), 4 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73529 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id69d609c35e4d619777060497fd8ea48f1af71ec Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.1]: Read correctly attributes with enum values
by Code Review 03 Mar '17

03 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Read correctly attributes with enum values ...................................................................... Read correctly attributes with enum values A recent patch introduced a mechanism to ignore incorrect values of enum types. That patch also introduced a regresion for attributes of enum types that have a name different to their type name. For example, the 'vm.migration.auto_converge' attribute is of type 'InheritableBoolean', not of type 'AutoConverge' as assumed by the code generator. This patch fixes that issue. Change-Id: I7e8df50f4dd59881238346f7b9ce06dd2a653546 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit ecc6928e683feec8ea62b1fa3885c1e31dfc7fd8) --- M generator/src/main/java/org/ovirt/sdk/python/ReadersGenerator.java A sdk/tests/test_vm_reader.py 2 files changed, 107 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73528 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e8df50f4dd59881238346f7b9ce06dd2a653546 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[sdk_4.1]: Don't avoid reserved words in enum values text
by Code Review 03 Mar '17

03 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Don't avoid reserved words in enum values text ...................................................................... Don't avoid reserved words in enum values text The SDK appends an underscore to generated names that conflict with Python reserved words. But it also does so for the text of enum values. This is wrong. This patch fixes it. Change-Id: Ibf6106a7f34cdf5bf4fb71100f9c247300ea7f02 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 1b8f9d4efb11c134297b5e5556282481ac4cef0b) --- M generator/src/main/java/org/ovirt/sdk/python/TypesGenerator.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73527 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf6106a7f34cdf5bf4fb71100f9c247300ea7f02 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_4.1 Gerrit-Owner: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk[master]: Don't avoid reserved words in enum values text
by Code Review 03 Mar '17

03 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Don't avoid reserved words in enum values text ...................................................................... Don't avoid reserved words in enum values text The SDK appends an underscore to generated names that conflict with Python reserved words. But it also does so for the text of enum values. This is wrong. This patch fixes it. Change-Id: Ibf6106a7f34cdf5bf4fb71100f9c247300ea7f02 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/TypesGenerator.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73518 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf6106a7f34cdf5bf4fb71100f9c247300ea7f02 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk 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[master]: Read correctly attributes with enum values
by Code Review 03 Mar '17

03 Mar '17

>From Ondra Machacek <omachace(a)redhat.com>: Ondra Machacek has submitted this change and it was merged. Change subject: Read correctly attributes with enum values ...................................................................... Read correctly attributes with enum values A recent patch introduced a mechanism to ignore incorrect values of enum types. That patch also introduced a regresion for attributes of enum types that have a name different to their type name. For example, the 'vm.migration.auto_converge' attribute is of type 'InheritableBoolean', not of type 'AutoConverge' as assumed by the code generator. This patch fixes that issue. Change-Id: I7e8df50f4dd59881238346f7b9ce06dd2a653546 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/ReadersGenerator.java A sdk/tests/test_vm_reader.py 2 files changed, 107 insertions(+), 1 deletion(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73514 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e8df50f4dd59881238346f7b9ce06dd2a653546 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine-sdk 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