- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-4.0]: core: Fix type of ConfigValues.Ipv6Supported
by piotr.kliczewski＠gmail.com 10 Jun '16

10 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Fix type of ConfigValues.Ipv6Supported ...................................................................... core: Fix type of ConfigValues.Ipv6Supported Fixes type of ConfigValues.Ipv6Supported from String to Boolean to be able to call Config.<Boolean> get() method. If type is String then of course ClassCastException it thrown. Change-Id: I773af6c508f8249eb5de6c9c1fac8a3777a2d763 Bug-Url: https://bugzilla.redhat.com/1344355 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58967 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I773af6c508f8249eb5de6c9c1fac8a3777a2d763 Gerrit-PatchSet: 1 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: installvds: increase logging of network config issues
by piotr.kliczewski＠gmail.com 10 Jun '16

10 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: installvds: increase logging of network config issues ...................................................................... installvds: increase logging of network config issues Change-Id: Ibf9fef77d9c62ddfe455cb424d4ae11c4dab551a Signed-off-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/InstallVdsInternalCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58966 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf9fef77d9c62ddfe455cb424d4ae11c4dab551a Gerrit-PatchSet: 1 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>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: Fix fallback to XMLRPC during protocol detection
by piotr.kliczewski＠gmail.com 10 Jun '16

10 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: Fix fallback to XMLRPC during protocol detection ...................................................................... core: Fix fallback to XMLRPC during protocol detection Fixes issue when we fell back to XMLRPC after the 1st unsuccessful attempt to connect to the host instead of number of retries defined in ProtocolFallbackRetries engine-config variable. Change-Id: I35a0bc9a0181e03dff143ceaaed647eab15b7de9 Bug-Url: https://bugzilla.redhat.com/1344337 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/transport/ProtocolDetector.java 1 file changed, 10 insertions(+), 8 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58965 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35a0bc9a0181e03dff143ceaaed647eab15b7de9 Gerrit-PatchSet: 1 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: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Fix type of ConfigValues.Ipv6Supported
by mperina＠redhat.com 10 Jun '16

10 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Fix type of ConfigValues.Ipv6Supported ...................................................................... core: Fix type of ConfigValues.Ipv6Supported Fixes type of ConfigValues.Ipv6Supported from String to Boolean to be able to call Config.<Boolean> get() method. If type is String then of course ClassCastException it thrown. Change-Id: I773af6c508f8249eb5de6c9c1fac8a3777a2d763 Bug-Url: https://bugzilla.redhat.com/1344355 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58962 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I773af6c508f8249eb5de6c9c1fac8a3777a2d763 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: installvds: increase logging of network config issues
by mperina＠redhat.com 10 Jun '16

10 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: installvds: increase logging of network config issues ...................................................................... installvds: increase logging of network config issues Change-Id: Ibf9fef77d9c62ddfe455cb424d4ae11c4dab551a Signed-off-by: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/InstallVdsInternalCommand.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Michal Skrivanek: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58933 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibf9fef77d9c62ddfe455cb424d4ae11c4dab551a Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: VM Start action properties list reviewed in RSDL
by juan.hernandez＠redhat.com 10 Jun '16

10 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: VM Start action properties list reviewed in RSDL ...................................................................... restapi: VM Start action properties list reviewed in RSDL The action properties list conforms the 'Initialization to VmInit' mapper (org.ovirt.engine.api.restapi.types.InitializationMapper) Bug-Url: https://bugzilla.redhat.com/1318701 Change-Id: I4b2e32ef4d0af0301df0444c075c445b9cf6acc7 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml 1 file changed, 23 insertions(+), 0 deletions(-) Approvals: Ondra Machacek: Looks good to me, but someone else must approve Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/58920 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4b2e32ef4d0af0301df0444c075c445b9cf6acc7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marek Libra <mlibra(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[master]: core: Fix fallback to XMLRPC during protocol detection
by mperina＠redhat.com 10 Jun '16

10 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: core: Fix fallback to XMLRPC during protocol detection ...................................................................... core: Fix fallback to XMLRPC during protocol detection Fixes issue when we fell back to XMLRPC after the 1st unsuccessful attempt to connect to the host instead of number of retries defined in ProtocolFallbackRetries engine-config variable. Change-Id: I35a0bc9a0181e03dff143ceaaed647eab15b7de9 Bug-Url: https://bugzilla.redhat.com/1344337 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/transport/ProtocolDetector.java 1 file changed, 10 insertions(+), 8 deletions(-) Approvals: Piotr Kliczewski: Looks good to me, but someone else must approve Martin Peřina: Verified Jenkins CI: Passed CI tests Michal Skrivanek: Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/58911 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I35a0bc9a0181e03dff143ceaaed647eab15b7de9 Gerrit-PatchSet: 4 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: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Set ActiveDirectoryOu in InitializationMapper
by mperina＠redhat.com 10 Jun '16

10 Jun '16

Martin Peřina has submitted this change and it was merged. Change subject: restapi: Set ActiveDirectoryOu in InitializationMapper ...................................................................... restapi: Set ActiveDirectoryOu in InitializationMapper The value was not set before this fix. Change-Id: Ic9f121cac0807f1ae3a4fbc47f1910528addb00f Bug-Url: https://bugzilla.redhat.com/1344426 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/InitializationMapper.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/58930 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic9f121cac0807f1ae3a4fbc47f1910528addb00f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: node: Drop placeholder requires
by fabiand＠redhat.com 09 Jun '16

09 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Drop placeholder requires ...................................................................... node: Drop placeholder requires Up to now the placeholder required ovirt-release-host-node, but this requirement is not needed, as the placeholder is pratcially self contained. The dependency was not problem, except that it became an issue when the version of the placeholder is different than ovirt-relase-host-node. Change-Id: Ifb50605549458bd5de2e0c1099a8e8631b0b7d0b Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M ovirt-release40.spec.in 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58924 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb50605549458bd5de2e0c1099a8e8631b0b7d0b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: node: Drop placeholder requires
by fabiand＠redhat.com 09 Jun '16

09 Jun '16

Fabian Deutsch has submitted this change and it was merged. Change subject: node: Drop placeholder requires ...................................................................... node: Drop placeholder requires Up to now the placeholder required ovirt-release-host-node, but this requirement is not needed, as the placeholder is pratcially self contained. The dependency was not problem, except that it became an issue when the version of the placeholder is different than ovirt-relase-host-node. Change-Id: Ifb50605549458bd5de2e0c1099a8e8631b0b7d0b Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M configure.ac M ovirt-release-master.spec.in 2 files changed, 4 insertions(+), 3 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58919 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb50605549458bd5de2e0c1099a8e8631b0b7d0b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0