- Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk-ruby[master]: Improve the documentation of action methods
by juan.hernandez＠redhat.com 18 Aug '16

18 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: Improve the documentation of action methods ...................................................................... Improve the documentation of action methods Currently the action methods, like "start", "stop", etc, don't their parameters documented. This patch fixes that issue. Change-Id: I18de9b55b7a5d0ce021951c4155d83dec964f651 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/ServicesGenerator.java 1 file changed, 16 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62553 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I18de9b55b7a5d0ce021951c4155d83dec964f651 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-ruby[master]: Update to model 4.1.10
by juan.hernandez＠redhat.com 18 Aug '16

18 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: Update to model 4.1.10 ...................................................................... Update to model 4.1.10 The more relevant changes since the previoius update of the model are the following: * Add `logicalName` attribute to the disk attachment type. * Fix the name of the parameter to get virtual machine from affinity label, should be `vm` instead of `host`. * Remove the "active" property from Disk. * Add the `clone_permissions` parameter to the operations that add virtual machines and templates. * Add the `clone` parameter to the operation that adds a new virtual machine. * Add the `ad_partner_mac` attribute to the bonding type. * Add the `ad_aggregator_id` attribute to the host NIC type. * Add the `upgradecheck` action to the host service. * Add the `ExternalVmImportsService` and related types. * Add the `active` property back to the disk type. * Add the missing `@Out` annotation to the primary parameter of the service that imports virtual machines from external systems. Change-Id: I15ba12d2cd35af8d7fe8f4942e8435644540a7c6 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M pom.xml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62552 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I15ba12d2cd35af8d7fe8f4942e8435644540a7c6 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[master]: Name element tag by the enitity, which is written
by omachace＠redhat.com 18 Aug '16

18 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Name element tag by the enitity, which is written ...................................................................... Name element tag by the enitity, which is written When writing XML, we always written XML element same as member, which we wanted to write. This patch changes it, so we name the singular element tag same as entity we write, not the member. For example if we want to create bonds, we will send following xml: <action> <modified_bonds> <host_nic> <bonding> <options>...</options> <slaves> <host_nic> <name>eth0</name> </host_nic> <host_nic> <name>eth1</name> </host_nic> </slaves> </bonding> <name>bond0</name> </host_nic> </modified_bonds> </action> Previously we sent <slave> instead of <host_nic> in <slaves> element. Change-Id: I277bc4678ab8d6da3d640a3863c6341b6e7a62ea Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/python/WritersGenerator.java M sdk/tests/test_setupnetworks.py 2 files changed, 6 insertions(+), 5 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/62554 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I277bc4678ab8d6da3d640a3863c6341b6e7a62ea 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 jenkins[master]: Revert "publisher: master: drop ioprocess ppc64le"
by sbonazzo＠redhat.com 18 Aug '16

18 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "publisher: master: drop ioprocess ppc64le" ...................................................................... Revert "publisher: master: drop ioprocess ppc64le" ioprocess build was fixed so python-ioprocess depend on ioprocess of same version, so we can enable the ppc builds now. This reverts commit a76290a8ad706176659c60980b951c16009d9e6d. Change-Id: I663282feffae3e3d17af243ab4b3544e2934cd83 Signed-off-by: Nir Soffer <nsoffer(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 3 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/62551 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I663282feffae3e3d17af243ab4b3544e2934cd83 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Send query parameters with Add/Update actions
by omachace＠redhat.com 18 Aug '16

18 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Send query parameters with Add/Update actions ...................................................................... Send query parameters with Add/Update actions There are use cases where user can sent query parameters, along with creating/updating entity action. For example when creating virtual machine he can also copy permissions from template to new virtual machine. This patch add the support so the SDK sends the query parameters, supported by backend. Change-Id: Icaee85b912f575a911f2e618b910b7f6f52dfd71 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1366546 Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M .gitignore M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java A sdk/src/test/java/org/ovirt/engine/sdk4/SecondaryParametersTest.java M sdk/src/test/java/org/ovirt/engine/sdk4/ServerTest.java A sdk/src/test/java/org/ovirt/engine/sdk4/examples/AddIndependentVm.java 5 files changed, 206 insertions(+), 53 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62546 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icaee85b912f575a911f2e618b910b7f6f52dfd71 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: Ondra Machacek <omachace(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: ovirt-engine_standard: added 4.0.3 branch
by sbonazzo＠redhat.com 18 Aug '16

18 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine_standard: added 4.0.3 branch ...................................................................... ovirt-engine_standard: added 4.0.3 branch Change-Id: I8683a552856fe4315dc5ac3fc295d35cad42f039 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 10 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Ready for merge Martin Peřina: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62548 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8683a552856fe4315dc5ac3fc295d35cad42f039 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: Revert "fedora: moving to ganeric branding"
by sbonazzo＠redhat.com 18 Aug '16

18 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Revert "fedora: moving to ganeric branding" ...................................................................... Revert "fedora: moving to ganeric branding" This reverts commit da0d8d3311441d1fb7a0ed639c34562a079ffb81. Without fedora branding live media creation fails with OSError: nothing matching /var/tmp/lorax.imgutils.x3lnikv1/usr/share/anaconda/boot/syslinux-splash.png in / So until we add oVirt artwork we need to use fedora logos. Change-Id: I9cc9c5e94a2fab6096dcd4b0d09ee1321275f061 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M fedora-24/kickstart/ovirt-live-ovirt-custom.ks.in 1 file changed, 0 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62550 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9cc9c5e94a2fab6096dcd4b0d09ee1321275f061 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-java[master]: Update to model 4.1.10 and metamodel 1.1.2
by omachace＠redhat.com 18 Aug '16

18 Aug '16

Ondra Machacek has submitted this change and it was merged. Change subject: Update to model 4.1.10 and metamodel 1.1.2 ...................................................................... Update to model 4.1.10 and metamodel 1.1.2 The more relevant changes in this new version of the model are the following: * Add the `clone_permissions` parameter to the operations that add virtual machines and templates. * Add the `clone` parameter to the operation that adds a new * virtual machine. * Add the `ad_partner_mac` attribute to the bonding type. * Add the `ad_aggregator_id` attribute to the host NIC type. * Add the `upgradecheck` action to the host service. * Add the `ExternalVmImportsService` and related types. The more relevant changes in this new version of the metamodel are the following: * Name element tag by the enitity, which is written * Improve name parser * Report correctly incorrect operators * Use dash as section id separator Change-Id: I44b2139c22f51ef7482ce5d0204bfe50740e953a Signed-off-by: Ondra Machacek <omachace(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/java/ServicesImplGenerator.java M pom.xml 2 files changed, 4 insertions(+), 2 deletions(-) Approvals: Ondra Machacek: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62534 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44b2139c22f51ef7482ce5d0204bfe50740e953a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-java Gerrit-Branch: master 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[master]: core: update cluster to query only static vm data
by ahadas＠redhat.com 18 Aug '16

18 Aug '16

Arik Hadas has submitted this change and it was merged. Change subject: core: update cluster to query only static vm data ...................................................................... core: update cluster to query only static vm data In order to update VM, we need only its static data so no need to use the costly query of whole VM. Change-Id: If31cd57110eb73574ce2eb6e8e63e232c7c06dc9 Bug-Url: https://bugzilla.redhat.com/1366786 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateClusterCommand.java 1 file changed, 6 insertions(+), 4 deletions(-) Approvals: Jakub Niedermertl: Looks good to me, but someone else must approve Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62522 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If31cd57110eb73574ce2eb6e8e63e232c7c06dc9 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[master]: fedora-24: move to lorax
by sbonazzo＠redhat.com 18 Aug '16

18 Aug '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: fedora-24: move to lorax ...................................................................... fedora-24: move to lorax moving from livecd-creator to livemedia-creator due to bug https://bugzilla.redhat.com/1368108 lorax uses dnf instead of yum so it's not affected. Change-Id: Icfe137d36bbc200a6d6f007ad052d43873279c97 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M fedora-24/Makefile 1 file changed, 7 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62539 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfe137d36bbc200a6d6f007ad052d43873279c97 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live 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: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0