September 2016 - Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: core: set display and management network
by masayag＠redhat.com 21 Sep '16

21 Sep '16

Moti Asayag has submitted this change and it was merged. Change subject: core: set display and management network ...................................................................... core: set display and management network If a display network is not set by any network except the management one , set it to the management network If a migration network is not set by any network except the management one , set it to the management network Change-Id: Ia6c8435f87b48fea8afc0707920a9e100e4bd02b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1372955 Signed-off-by: emesika <emesika(a)redhat.com> Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- A packaging/dbscripts/upgrade/04_01_0320_set_display_migration_cluster_network.sql 1 file changed, 16 insertions(+), 0 deletions(-) Approvals: Moti Asayag: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/63979 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia6c8435f87b48fea8afc0707920a9e100e4bd02b Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: tests: introduce ExtendSANStorageDomainCommandTest
by tnisan＠redhat.com 21 Sep '16

21 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: tests: introduce ExtendSANStorageDomainCommandTest ...................................................................... tests: introduce ExtendSANStorageDomainCommandTest Change-Id: Ic4d053f70a5e64df11c60d6dd0a7c37fdfdae2a7 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/ExtendSANStorageDomainCommand.java A backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/domain/ExtendSANStorageDomainCommandTest.java 2 files changed, 63 insertions(+), 5 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/64115 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4d053f70a5e64df11c60d6dd0a7c37fdfdae2a7 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: Jsonrpc 4.0 branch created
by eedri＠redhat.com 21 Sep '16

21 Sep '16

Eyal Edri has submitted this change and it was merged. Change subject: Jsonrpc 4.0 branch created ...................................................................... Jsonrpc 4.0 branch created Change-Id: Ia26cc6ec9f4672a2f95b09ef07a82bf7684899b2 Signed-off-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> --- M jobs/confs/projects/vdsm-jsonrpc-java/vdsm-jsonrpc-java_standard.yaml 1 file changed, 3 insertions(+), 1 deletion(-) Approvals: Eyal Edri: Verified; Looks good to me; Ready for merge Gil Shinar: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/63301 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia26cc6ec9f4672a2f95b09ef07a82bf7684899b2 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <sharonaftaly(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Recursively scan for service interfaces
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Recursively scan for service interfaces ...................................................................... restapi: Recursively scan for service interfaces The creation of links to collections relies on scanning of the implementation classes, in order to find the implemented service interfaces. But currently the code that does this scanning doesn't take into account that some interfaces may not be implememented directly, but by base classes. In those situations the links to collections aren't created correctly. The 'BackendDataCenterNetworksResource' class is an example of that. It doesn't directly implement any service interface, only the base class does. To avoid that issue this patch changes the scanning code so that it recursively scans the parent classes. Change-Id: Ice711e9e94048ac695c4cd502e8beb563f6f2017 Bug-Url: https://bugzilla.redhat.com/1375668 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendResource.java 1 file changed, 20 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64193 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice711e9e94048ac695c4cd502e8beb563f6f2017 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-sdk-ruby[sdk_4.0]: Copy test log files to the exported artifacts directory
by juan.hernandez＠redhat.com 21 Sep '16

21 Sep '16

Juan Hernandez has submitted this change and it was merged. Change subject: Copy test log files to the exported artifacts directory ...................................................................... Copy test log files to the exported artifacts directory This patch changes the automation scripts so that the log files generated by the tests are copied to the exported artifacts directory. This way, if there are test failures, it will be easier to debug them. Change-Id: I9e849b04836cb9bc324dec79d22fe7cd93ba74d4 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 7cd28f8393e130eb9113040f5c50d1a0b928c7c4) --- M automation/build.py M sdk/spec/spec_helper.rb 2 files changed, 13 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64226 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e849b04836cb9bc324dec79d22fe7cd93ba74d4 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