- Engine-commits - oVirt List Archives

Change in jenkins[master]: Removing absolete emails
by tlitovsk＠redhat.com 22 Sep '16

22 Sep '16

Anatoly Litovsky has submitted this change and it was merged. Change subject: Removing absolete emails ...................................................................... Removing absolete emails Change-Id: I5d2d7cf9a9649327bd4c2e52845b97f11c39a5da --- M jobs/confs/projects/ovirt/system-tests.yaml M jobs/confs/yaml/templates/ovirt-node-job-templates.yaml 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests Anatoly Litovsky: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/64267 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5d2d7cf9a9649327bd4c2e52845b97f11c39a5da Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[ovirt-host-deploy-1.5]: Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG
by sbonazzo＠redhat.com 22 Sep '16

22 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG ...................................................................... Fix location for OPENSTACK_NEUTRON_OPENVSWITCH_CONFIG With the release of OpenStack Liberty the Open vSwitch agent ini file location has changed. Includes logic to use OLD path if running OpenStack Kilo or older. OLD: /etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini NEW: /etc/neutron/plugins/ml2/openvswitch_agent.ini Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1373968 Change-Id: I6adf1078442693410d3435616e4dafb5600a77fb Signed-off-by: Tony James <tony(a)anthonyjames.org> (cherry picked from commit c0a9fda5be80966560c60c00638912d03f3248e2) --- M src/ovirt_host_deploy/constants.py 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64266 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6adf1078442693410d3435616e4dafb5600a77fb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: ovirt-host-deploy-1.5 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-ruby[sdk_4.0]: Write correctly lists of enum values
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Write correctly lists of enum values ...................................................................... Write correctly lists of enum values Currently the SDK doesn't write correctly lists of enum values, because it tries to end the list calling the "end_element" method of the "XmlWriter" class, which doesn't exist, the correct name is "write_end". This is a bug in the code generator, fixed by this patch. Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Bug-Url: https://bugzilla.redhat.com/1378066 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 1e05f5d2aaee90ef9a39506bbd206c186423b0fe) --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java M sdk/spec/vm_writer_spec.rb 2 files changed, 33 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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]: Write correctly lists of enum values
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Write correctly lists of enum values ...................................................................... Write correctly lists of enum values Currently the SDK doesn't write correctly lists of enum values, because it tries to end the list calling the "end_element" method of the "XmlWriter" class, which doesn't exist, the correct name is "write_end". This is a bug in the code generator, fixed by this patch. Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Bug-Url: https://bugzilla.redhat.com/1378066 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M generator/src/main/java/org/ovirt/sdk/ruby/WritersGenerator.java M sdk/spec/vm_writer_spec.rb 2 files changed, 33 insertions(+), 1 deletion(-) Approvals: Boris Odnopozov: 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/64245 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87577ec88be3dcfe21d879fb4393f677a74511e7 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(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[sdk_4.0]: Added Probing for API version capability
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Added Probing for API version capability ...................................................................... Added Probing for API version capability This patch adds the Probe class that allows to check which API versions the oVirt server supports. Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Signed-off-by: borod108 <bodnopoz(a)redhat.com> (cherry picked from commit 9cf6f163d66e265423e8b2d65e14edbc6aba2c31) --- M sdk/Gemfile.lock M sdk/lib/ovirtsdk4.rb M sdk/lib/ovirtsdk4/http.rb A sdk/lib/ovirtsdk4/probe.rb M sdk/spec/connection_create_spec.rb A sdk/spec/probe_spec.rb M sdk/spec/spec_helper.rb 7 files changed, 318 insertions(+), 43 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64247 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: sdk_4.0 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[master]: spec: Make jboss optional
by sbonazzo＠redhat.com 21 Sep '16

21 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: spec: Make jboss optional ...................................................................... spec: Make jboss optional So that using jboss-eap does not require pulling in also wildfly. Change-Id: I3ccdabb504e086092f61c1e9c50637b32c94ba09 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/64236 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3ccdabb504e086092f61c1e9c50637b32c94ba09 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-2.0]: storage: added deprecation warning to additional host setup
by sbonazzo＠redhat.com 21 Sep '16

21 Sep '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: added deprecation warning to additional host setup ...................................................................... storage: added deprecation warning to additional host setup Bug-Url: https://bugzilla.redhat.com/1378111 Change-Id: Ie06701629264c7c7594b47a8ed1cc64c518474ce Signed-off-by: Rafael Martins <rmartins(a)redhat.com> --- M src/plugins/gr-he-setup/storage/storage.py 1 file changed, 7 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Simone Tiraboschi: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64171 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie06701629264c7c7594b47a8ed1cc64c518474ce Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-2.0 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[master]: Added Probing for API version capability
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Added Probing for API version capability ...................................................................... Added Probing for API version capability This patch adds the Probe class that allows to check which API versions the oVirt server supports. Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Signed-off-by: borod108 <bodnopoz(a)redhat.com> --- M sdk/Gemfile.lock M sdk/lib/ovirtsdk4.rb M sdk/lib/ovirtsdk4/http.rb A sdk/lib/ovirtsdk4/probe.rb M sdk/spec/connection_create_spec.rb A sdk/spec/probe_spec.rb M sdk/spec/spec_helper.rb 7 files changed, 318 insertions(+), 43 deletions(-) Approvals: Boris Odnopozov: Verified Juan Hernandez: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/63897 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icca8a5bd75841f198f1580b811c96a6fe7dda78e Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine-sdk-ruby Gerrit-Branch: master Gerrit-Owner: Boris Odnopozov <bodnopoz(a)redhat.com> Gerrit-Reviewer: Boris Odnopozov <bodnopoz(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[master]: backend: Migrate Monitoring Strategies to CDI
by mbetak＠redhat.com 21 Sep '16

21 Sep '16

Martin Betak has submitted this change and it was merged. Change subject: backend: Migrate Monitoring Strategies to CDI ...................................................................... backend: Migrate Monitoring Strategies to CDI Change-Id: Icaab283c25119b5fe950d108d216be17fc806a19 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/GlusterMonitoringStrategy.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/MonitoringStrategyFactory.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/monitoring/VirtMonitoringStrategy.java 3 files changed, 39 insertions(+), 22 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Martin Betak: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/63491 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icaab283c25119b5fe950d108d216be17fc806a19 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Check patch scripts to find issues with publish jobs
by eedri＠redhat.com 21 Sep '16

21 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Check patch scripts to find issues with publish jobs ...................................................................... Check patch scripts to find issues with publish jobs Includes a python script that'll go over all publish rpms jobs and make sure that all jobs in the publish jobs exists. For eample, if a user deleted a job that the publisher uses and hadn't updated the publisher job, check patch will fail Change-Id: Id0a71caf5b91218e6f17c7b215ebf595386c3575 Signed-off-by: Gil Shinar <gshinar(a)redhat.com> --- A automation/check-patch.packages A automation/check-patch.repos M automation/check-patch.sh A automation/check_publishers_not_deleted.py A automation/jenkins_check_yaml.sh 5 files changed, 99 insertions(+), 1 deletion(-) Approvals: Nadav Goldin: Looks good to me Eyal Edri: Ready for merge Gil Shinar: Verified; Ready for review Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64214 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0a71caf5b91218e6f17c7b215ebf595386c3575 Gerrit-PatchSet: 5 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: Evgheni Dereveanchin <ederevea(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: Nadav Goldin <ngoldin(a)redhat.com> Gerrit-Reviewer: Pavel Zhukov <pavel(a)zhukoff.net> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0