August 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: setup: aaa: Fix message about creating aaa-jdbc db
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: setup: aaa: Fix message about creating aaa-jdbc db ...................................................................... setup: aaa: Fix message about creating aaa-jdbc db Fixes message that indicates creation of AAA-JDBC database for internal domain as it was too broad. Change-Id: I3a14ea7c4dccbefa12d1cb6f73ebc25fc226c163 Bug-Url: https://bugzilla.redhat.com/1258355 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/aaajdbc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Looks good to me, approved Martin Peřina: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45538 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a14ea7c4dccbefa12d1cb6f73ebc25fc226c163 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: EngineEncryptionUtils: grammar and spelling
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: core: EngineEncryptionUtils: grammar and spelling ...................................................................... core: EngineEncryptionUtils: grammar and spelling Fixed the javadoc strings "Please notice that empty strings are not [decrypted|encrypted] and returend as-is." to the more correct sentences "Pleasese note that empty strings are not decrypted and are returned as-is." Change-Id: Ic21f8941c3285d9a33260fb0adb434b59c2c288f Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/crypt/EngineEncryptionUtils.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified -- To view, visit https://gerrit.ovirt.org/45550 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic21f8941c3285d9a33260fb0adb434b59c2c288f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: packaging: setup: docker: move from latest to kilo
by piotr.kliczewski＠gmail.com 31 Aug '15

31 Aug '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: setup: docker: move from latest to kilo ...................................................................... packaging: setup: docker: move from latest to kilo latest tag is not available anymore, moving to kilo. The tag name could be override. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1251946 Change-Id: I8a5a698ea1c46a4ab39bcbc6e3f6f9e2a1739213 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dockerc/constants.py M packaging/setup/plugins/ovirt-engine-common/dockerc/core.py M packaging/setup/plugins/ovirt-engine-setup/dockerc/config.py 3 files changed, 31 insertions(+), 14 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45450 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a5a698ea1c46a4ab39bcbc6e3f6f9e2a1739213 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: docker: move from latest to kilo
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: docker: move from latest to kilo ...................................................................... packaging: setup: docker: move from latest to kilo latest tag is not available anymore, moving to kilo. The tag name could be override. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1251946 Change-Id: I8a5a698ea1c46a4ab39bcbc6e3f6f9e2a1739213 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dockerc/constants.py M packaging/setup/plugins/ovirt-engine-common/dockerc/core.py M packaging/setup/plugins/ovirt-engine-setup/dockerc/config.py 3 files changed, 31 insertions(+), 14 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45186 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a5a698ea1c46a4ab39bcbc6e3f6f9e2a1739213 Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Dan Kenigsberg <danken(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: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: build: ovirt-hosted-engine-ha-1.2.6.1
by msivak＠redhat.com 31 Aug '15

31 Aug '15

Martin Sivák has submitted this change and it was merged. Change subject: build: ovirt-hosted-engine-ha-1.2.6.1 ...................................................................... build: ovirt-hosted-engine-ha-1.2.6.1 Change-Id: I8cb9e1524f9daac18befc858d263b0b85385f39e Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-hosted-engine-ha.spec.in 2 files changed, 5 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified Martin Sivák: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45543 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8cb9e1524f9daac18befc858d263b0b85385f39e Gerrit-PatchSet: 1 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-hosted-engine-ha[ovirt-hosted-engine-ha-1.2]: Support for unconfigured startup handling and API access
by msivak＠redhat.com 31 Aug '15

31 Aug '15

Martin Sivák has submitted this change and it was merged. Change subject: Support for unconfigured startup handling and API access ...................................................................... Support for unconfigured startup handling and API access Hosted engine correctly shuts down when no configuration can be found. Missing configuration files generate error message or debug message depending on the importance of the file. API call to get the host id returns None in the unconfigured case and also handles missing config files properly. Configuration file access errors are reported to a separate logger (either ovirt_hosted_engine_ha.env.config or the passed logger with .config added to the end) Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1252801 Change-Id: I0b6a60dda3afc97a91f542c2449174fc09131a4b Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M ovirt_hosted_engine_ha/agent/agent.py M ovirt_hosted_engine_ha/agent/hosted_engine.py M ovirt_hosted_engine_ha/client/client.py M ovirt_hosted_engine_ha/env/config.py 4 files changed, 91 insertions(+), 23 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, but someone else must approve Martin Sivák: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/45306 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0b6a60dda3afc97a91f542c2449174fc09131a4b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-hosted-engine-ha Gerrit-Branch: ovirt-hosted-engine-ha-1.2 Gerrit-Owner: Martin Sivák <msivak(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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: sercon: setup: ovirt-vmconsole-proxy-sshd startup
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: sercon: setup: ovirt-vmconsole-proxy-sshd startup ...................................................................... sercon: setup: ovirt-vmconsole-proxy-sshd startup If engine-setup configures the ovirt-vmconsole-proxy integration, it should make sure the service is up, like we do for the other services Engine depens on. Change-Id: I656e1b85fcffc4dd94ea6f19ed3f3df3df91a2d6 Bug-Url: https://bugzilla.redhat.com/1252901 Signed-off-by: Francesco Romani <fromani(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/vmconsole_proxy_helper/constants.py M packaging/setup/plugins/ovirt-engine-setup/vmconsole_proxy_helper/config.py 2 files changed, 23 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Francesco Romani: Verified -- To view, visit https://gerrit.ovirt.org/45069 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I656e1b85fcffc4dd94ea6f19ed3f3df3df91a2d6 Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: setup: aaa: Fix message about creating aaa-jdbc db
by sbonazzo＠redhat.com 31 Aug '15

31 Aug '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: setup: aaa: Fix message about creating aaa-jdbc db ...................................................................... setup: aaa: Fix message about creating aaa-jdbc db Fixes message that indicates creation of AAA-JDBC database for internal domain as it was too broad. Change-Id: I3a14ea7c4dccbefa12d1cb6f73ebc25fc226c163 Bug-Url: https://bugzilla.redhat.com/1258355 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/aaajdbc.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Looks good to me, approved; Passed CI tests Martin Peřina: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/45513 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3a14ea7c4dccbefa12d1cb6f73ebc25fc226c163 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine: validate MTU against any non-VLAN network in the leg...
by tnisan＠redhat.com 31 Aug '15

31 Aug '15

Tal Nisan has submitted this change and it was merged. Change subject: engine: validate MTU against any non-VLAN network in the legacy flow ...................................................................... engine: validate MTU against any non-VLAN network in the legacy flow Validate MTU against any non-VLAN network in the legacy flow. Change-Id: Ie0ca2b3b9bb813a6ed769c0a52ccc81e1f182bcb Bug-Url: https://bugzilla.redhat.com/1242227 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java 1 file changed, 18 insertions(+), 5 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/45536 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0ca2b3b9bb813a6ed769c0a52ccc81e1f182bcb Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine: validate MTU against any non-VLAN network in the leg...
by alkaplan＠redhat.com 31 Aug '15

31 Aug '15

Alona Kaplan has submitted this change and it was merged. Change subject: engine: validate MTU against any non-VLAN network in the legacy flow ...................................................................... engine: validate MTU against any non-VLAN network in the legacy flow Validate MTU against any non-VLAN network in the legacy flow. Change-Id: Ie0ca2b3b9bb813a6ed769c0a52ccc81e1f182bcb Bug-Url: https://bugzilla.redhat.com/1242227 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/host/SetupNetworksHelper.java 1 file changed, 18 insertions(+), 5 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved Jenkins CI: Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/45537 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie0ca2b3b9bb813a6ed769c0a52ccc81e1f182bcb Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0