- Engine-commits - oVirt List Archives

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

Change in ovirt-engine[master]: core: SnapshotVmConfigurationHelper injected
by ahadas＠redhat.com 21 Sep '16

21 Sep '16

Arik Hadas has submitted this change and it was merged. Change subject: core: SnapshotVmConfigurationHelper injected ...................................................................... core: SnapshotVmConfigurationHelper injected ... instead of instantiation using `new`. The old approach caused NPE inside SnapshotVmConfigurationHelper since its fields were not injected. Change-Id: Ic104b94ddbfd43bfc7f587fc6d3c3dc536678e8a Bug-Url: https://bugzilla.redhat.com/1377827 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmNextRunConfigurationQuery.java 1 file changed, 5 insertions(+), 5 deletions(-) Approvals: Jakub Niedermertl: Verified Jenkins CI: Passed CI tests Martin Betak: Looks good to me, but someone else must approve Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/64224 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic104b94ddbfd43bfc7f587fc6d3c3dc536678e8a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(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: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

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

20 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> --- 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/64225 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: 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[ovirt-engine-4.0]: core: disable migration compression for default migration po...
by tnisan＠redhat.com 20 Sep '16

20 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: core: disable migration compression for default migration policy ...................................................................... core: disable migration compression for default migration policy It may cause additional load on the host cpu we don't want to risk in default policy. Change-Id: I7776b31b31d41469f61f5357e67e34739d3336f3 Bug-Url: https://bugzilla.redhat.com/1374731 Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Tomas Jelinek: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64188 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7776b31b31d41469f61f5357e67e34739d3336f3 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: engine : Engine should not invoke revoke all on session expi...
by tnisan＠redhat.com 20 Sep '16

20 Sep '16

Tal Nisan has submitted this change and it was merged. Change subject: engine : Engine should not invoke revoke all on session expiration ...................................................................... engine : Engine should not invoke revoke all on session expiration On session expiration SessionDataContainer should not invoke revoke-all for the token. If there are no other sessions for the token SSO will cleanup the session on revoke. Change-Id: I8bffd0551efd7ec4850b8913e4e9c8789c36785d Bug-Url: https://bugzilla.redhat.com/1377422 Signed-off-by: Ravi Nori <rnori(a)redhat.com> (cherry picked from commit fe492153c06ba788c6ba6053275de394acd2fe37) --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Looks good to me, approved Ravi Nori: Verified; Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/64195 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8bffd0551efd7ec4850b8913e4e9c8789c36785d Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-host-deploy[master]: hosted-engine: set local maintenance to false when installin...
by stirabos＠redhat.com 20 Sep '16

20 Sep '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: hosted-engine: set local maintenance to false when installing host ...................................................................... hosted-engine: set local maintenance to false when installing host When installing hosted engine host that was installed and then removed from the engine its local maintenance would remain set to true. This patch sets the local maintenance to false when installing the host. Change-Id: I94edee9245728776af15099a3abd707bac19b11a Bug-Url: https://bugzilla.redhat.com/1353600 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> --- M src/plugins/ovirt-host-common/hosted-engine/configureha.py 1 file changed, 56 insertions(+), 1 deletion(-) Approvals: Simone Tiraboschi: Looks good to me, approved Jenkins CI: Passed CI tests Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/64208 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I94edee9245728776af15099a3abd707bac19b11a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-host-deploy Gerrit-Branch: master Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0