March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.3]: build: ovirt-hosted-engine-ha-1.3.5
by msivak＠redhat.com 15 Mar '16

15 Mar '16

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.3.5 ...................................................................... build: ovirt-hosted-engine-ha-1.3.5 Change-Id: I9e5cfc6433f380fac928d5b1fc06af0cde354fa3 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 12 insertions(+), 9 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/54748 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9e5cfc6433f380fac928d5b1fc06af0cde354fa3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(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[master]: core: call refreshCapabilities upon "|net|host_conn|no_id" e...
by alkaplan＠redhat.com 15 Mar '16

15 Mar '16

Alona Kaplan has submitted this change and it was merged. Change subject: core: call refreshCapabilities upon "|net|host_conn|no_id" event ...................................................................... core: call refreshCapabilities upon "|net|host_conn|no_id" event • added method org.ovirt.engine.core.vdsbroker.VdsManager#refreshHost() so that calling code does not have to have VDS to refresh VDS. Change-Id: Ide27a805b412cb3ae005a0ad2ba66cb25b60ecb7 Signed-off-by: Martin Mucha <mmucha(a)redhat.com> --- A backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HostConnectionRefresher.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java 2 files changed, 76 insertions(+), 0 deletions(-) Approvals: Martin Mucha: Verified Alona Kaplan: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54250 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ide27a805b412cb3ae005a0ad2ba66cb25b60ecb7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: engine: Remove deadcode
by masayag＠redhat.com 15 Mar '16

15 Mar '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: Remove deadcode ...................................................................... engine: Remove deadcode This patch removes unreachable and addresses Coverity CID 1068808. Change-Id: Id0ab34e3a4f8f28a67ac36d3ef07d30882f5f4ec Signed-off-by: Ala Hino <ahino(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/RpmVersionUtils.java 1 file changed, 1 insertion(+), 4 deletions(-) Approvals: Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve Ala Hino: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54707 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0ab34e3a4f8f28a67ac36d3ef07d30882f5f4ec Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: post ovirt-hosted-engine-setup-1.3.4.0
by stirabos＠redhat.com 15 Mar '16

15 Mar '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: post ovirt-hosted-engine-setup-1.3.4.0 ...................................................................... build: post ovirt-hosted-engine-setup-1.3.4.0 Change-Id: Ice6b231b96d51e0c0fca84693e3a744854120ace Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 8 insertions(+), 5 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 Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54747 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ice6b231b96d51e0c0fca84693e3a744854120ace Gerrit-PatchSet: 4 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: build: ovirt-hosted-engine-setup-1.3.4.0
by stirabos＠redhat.com 15 Mar '16

15 Mar '16

Simone Tiraboschi has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-setup-1.3.4.0 ...................................................................... build: ovirt-hosted-engine-setup-1.3.4.0 Change-Id: I82abfb4db156d99436359eaf6b67f50d74d7007c Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-setup.spec.in 2 files changed, 13 insertions(+), 10 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests Rafael Martins: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54745 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I82abfb4db156d99436359eaf6b67f50d74d7007c Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 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[master]: core: open Cockpit port in firewall
by emesika＠redhat.com 15 Mar '16

15 Mar '16

Eli Mesika has submitted this change and it was merged. Change subject: core: open Cockpit port in firewall ...................................................................... core: open Cockpit port in firewall Cockpit will be the default administration interface in Node 4.0. The cockpit port (9090/tcp) should be added to the list of default ports to be opened, when the user decides to let engine manage the firewall fo the host. Change-Id: Ieef7fa196ae3459fb5d0697687d46b4747ce7860 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1314781 Signed-off-by: emesika <emesika(a)redhat.com> --- M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Fabian Deutsch: Looks good to me, but someone else must approve Martin Peřina: Looks good to me, but someone else must approve Eli Mesika: Verified; Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54586 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieef7fa196ae3459fb5d0697687d46b4747ce7860 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in otopi[master]: filetransaction: Convert sequences toStr
by didi＠redhat.com 15 Mar '16

15 Mar '16

Yedidyah Bar David has submitted this change and it was merged. Change subject: filetransaction: Convert sequences toStr ...................................................................... filetransaction: Convert sequences toStr Bug-Url: https://bugzilla.redhat.com/1295887 Bug-Url: https://bugzilla.redhat.com/1316908 Change-Id: I66946b833a69ad503097f7c91a101280145dda62 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M src/otopi/filetransaction.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Yedidyah Bar David: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54731 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I66946b833a69ad503097f7c91a101280145dda62 Gerrit-PatchSet: 2 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(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[master]: storage: reloading other volumes uuid from disk
by sbonazzo＠redhat.com 15 Mar '16

15 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: reloading other volumes uuid from disk ...................................................................... storage: reloading other volumes uuid from disk On system upgrades from past releases the answerfile could miss some values about ancillary volumes. Realoading the ancillary volumes uuid from disk to overcame the lack of that values in the answerfile. Change-Id: I284e10689ea4a4909dfb9ae88f2464de90b89739 Bug-Url: https://bugzilla.redhat.com/1306825 Depends-on: https://gerrit.ovirt.org/#/c/54557/ Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 4 files changed, 109 insertions(+), 74 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I284e10689ea4a4909dfb9ae88f2464de90b89739 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(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-engine[master]: core: Rename JsonExtDeserializer to JsonExtKeyDeserializer
by mperina＠redhat.com 15 Mar '16

15 Mar '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Rename JsonExtDeserializer to JsonExtKeyDeserializer ...................................................................... core: Rename JsonExtDeserializer to JsonExtKeyDeserializer Renames JsonExtDeserializer to JsonExtKeyDeserializer Change-Id: Iba7fb889326ec93372374efe54ce6804f4ebf988 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- R backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/JsonExtKeyDeserializer.java M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/JsonExtMapMixIn.java R backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonExtKeyDeserializer.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/serialization/json/JsonExtMapMixIn.java 4 files changed, 4 insertions(+), 4 deletions(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Ravi Nori: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54670 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iba7fb889326ec93372374efe54ce6804f4ebf988 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-hosted-engine-setup[ovirt-hosted-engine-setup-1.3]: storage: reloading other volumes uuid from disk
by sbonazzo＠redhat.com 15 Mar '16

15 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: storage: reloading other volumes uuid from disk ...................................................................... storage: reloading other volumes uuid from disk On system upgrades from past releases the answerfile could miss some values about ancillary volumes. Realoading the ancillary volumes uuid from disk to overcame the lack of that values in the answerfile. Change-Id: I284e10689ea4a4909dfb9ae88f2464de90b89739 Bug-Url: https://bugzilla.redhat.com/1306825 Depends-on: https://gerrit.ovirt.org/#/c/54557/ Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M src/ovirt_hosted_engine_setup/constants.py M src/plugins/ovirt-hosted-engine-setup/core/remote_answerfile.py M src/plugins/ovirt-hosted-engine-setup/sanlock/lockspace.py M src/plugins/ovirt-hosted-engine-setup/storage/storage.py 4 files changed, 109 insertions(+), 74 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/54737 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I284e10689ea4a4909dfb9ae88f2464de90b89739 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-setup Gerrit-Branch: ovirt-hosted-engine-setup-1.3 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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