March 2017 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: Backend - evaluateCorrelationId() unneeded decleration
by Code Review 02 Mar '17

02 Mar '17

>From Liron Aravot <laravot(a)redhat.com>: Liron Aravot has submitted this change and it was merged. Change subject: Backend - evaluateCorrelationId() unneeded decleration ...................................................................... Backend - evaluateCorrelationId() unneeded decleration Change-Id: If5367a7bf26fada9f97e0518e263e1b640c65f83 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Moti Asayag: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72982 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5367a7bf26fada9f97e0518e263e1b640c65f83 Gerrit-PatchSet: 9 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: MoveImageGroup - remove image correlation id
by Code Review 02 Mar '17

02 Mar '17

>From Liron Aravot <laravot(a)redhat.com>: Liron Aravot has submitted this change and it was merged. Change subject: core: MoveImageGroup - remove image correlation id ...................................................................... core: MoveImageGroup - remove image correlation id This patch changes the MoveImageGroup command to pass the correlation id to the executed RemoveImage to ease tracking the entire flow in the log. Change-Id: I31141ac43903ce723c3549efe4f58aa99f62e2a1 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/MoveImageGroupCommand.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Daniel Erez: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72953 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I31141ac43903ce723c3549efe4f58aa99f62e2a1 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Backend - set the parent correlation id
by Code Review 02 Mar '17

02 Mar '17

>From Liron Aravot <laravot(a)redhat.com>: Liron Aravot has submitted this change and it was merged. Change subject: core: Backend - set the parent correlation id ...................................................................... core: Backend - set the parent correlation id When command is executed as a child command the correlation id should be derived from the parent parameters to ease tracking the flow in the logs. Change-Id: Iaea59a769f6a44051340a9b9247f27fc5fc76be3 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java 1 file changed, 6 insertions(+), 3 deletions(-) Approvals: Moti Asayag: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72952 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iaea59a769f6a44051340a9b9247f27fc5fc76be3 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: CommandCallbacksPoller - set the command correlation id
by Code Review 02 Mar '17

02 Mar '17

>From Liron Aravot <laravot(a)redhat.com>: Liron Aravot has submitted this change and it was merged. Change subject: core: CommandCallbacksPoller - set the command correlation id ...................................................................... core: CommandCallbacksPoller - set the command correlation id This patch updates the CommandCallbacksPoller to set the command correlation id before executing the needed callback method in order to be able to track the flow easily in the logs. Change-Id: Ib934bf4421a7fbf890b5c006263ff559b9d88462 Signed-off-by: Liron Aravot <laravot(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/tasks/CommandCallbacksPoller.java 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Moti Asayag: Looks good to me, approved Liron Aravot: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/72934 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib934bf4421a7fbf890b5c006263ff559b9d88462 Gerrit-PatchSet: 10 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: no need to apply NextRunConfiguration on suspending VM
by Code Review 02 Mar '17

02 Mar '17

>From Shahar Havivi <shavivi(a)redhat.com>: Shahar Havivi has submitted this change and it was merged. Change subject: core: no need to apply NextRunConfiguration on suspending VM ...................................................................... core: no need to apply NextRunConfiguration on suspending VM its cause to wrong behaviour - committing vm variables without restarting the VM. Change-Id: I8234d7f7d163637e8a597ed0ef249b871d1bd3c7 Bug-Url: https://bugzilla.redhat.com/1351215 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ProcessDownVmCommand.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/73361 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8234d7f7d163637e8a597ed0ef249b871d1bd3c7 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.1.1.z]: Remove ovirt-engine-hosts-ansible-inventory
by Code Review 02 Mar '17

02 Mar '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: Remove ovirt-engine-hosts-ansible-inventory ...................................................................... Remove ovirt-engine-hosts-ansible-inventory And require instead ovirt-engine-metrics, which got its functionality Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Bug-Url: https://bugzilla.redhat.com/1422505 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 3350190eb74dc413b88570c41a625f5a5d1bc27a) --- M ovirt-engine.spec.in D packaging/setup/ansible/ansible.cfg D packaging/setup/ansible/config.yml.example D packaging/setup/ansible/configure_ovirt_hosts_for_metrics.sh D packaging/setup/ansible/group_vars/all.yml D packaging/setup/ansible/inventory/dynamic_ovirt_hosts D packaging/setup/ansible/inventory/hosts D packaging/setup/ansible/playbook.yml D packaging/setup/ansible/roles/collectd/defaults/main.yml D packaging/setup/ansible/roles/collectd/handlers/main.yml D packaging/setup/ansible/roles/collectd/tasks/main.yml D packaging/setup/ansible/roles/firewall/README.md D packaging/setup/ansible/roles/firewall/commit/README.md D packaging/setup/ansible/roles/firewall/commit/meta/main.yml D packaging/setup/ansible/roles/firewall/commit/tasks/main.yml D packaging/setup/ansible/roles/firewall/defaults/main.yml D packaging/setup/ansible/roles/firewall/handlers/main.yml D packaging/setup/ansible/roles/firewall/tasks/main.yml D packaging/setup/ansible/roles/fluentd/defaults/main.yml D packaging/setup/ansible/roles/fluentd/handlers/main.yml D packaging/setup/ansible/roles/fluentd/tasks/main.yml D packaging/setup/ansible/roles/fluentd/templates/fluentd.conf D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/templates/builtins-conf-for-ovirt.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/templates/read-statsd.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/write_http/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/templates/write-http.conf.j2 D packaging/setup/ansible/roles/ovirt_fluentd/client/README.md D packaging/setup/ansible/roles/ovirt_fluentd/client/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_no_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/http_input/README.md D packaging/setup/ansible/roles/ovirt_fluentd/http_input/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/templates/http-input.conf D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/README.md D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/templates/ovirt-processing.conf D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/README.md D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/tasks/main.yml D packaging/setup/ansible/tools/validate-playbooks D packaging/setup/bin/ovirt-engine-hosts-ansible-inventory M packaging/setup/ovirt_engine_setup/constants.py 54 files changed, 1 insertion(+), 1,046 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73417 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1.1.z Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-4.1]: Remove ovirt-engine-hosts-ansible-inventory
by Code Review 02 Mar '17

02 Mar '17

>From Tal Nisan <tnisan(a)redhat.com>: Tal Nisan has submitted this change and it was merged. Change subject: Remove ovirt-engine-hosts-ansible-inventory ...................................................................... Remove ovirt-engine-hosts-ansible-inventory And require instead ovirt-engine-metrics, which got its functionality Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Bug-Url: https://bugzilla.redhat.com/1422505 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> (cherry picked from commit 3350190eb74dc413b88570c41a625f5a5d1bc27a) --- M ovirt-engine.spec.in D packaging/setup/ansible/ansible.cfg D packaging/setup/ansible/config.yml.example D packaging/setup/ansible/configure_ovirt_hosts_for_metrics.sh D packaging/setup/ansible/group_vars/all.yml D packaging/setup/ansible/inventory/dynamic_ovirt_hosts D packaging/setup/ansible/inventory/hosts D packaging/setup/ansible/playbook.yml D packaging/setup/ansible/roles/collectd/defaults/main.yml D packaging/setup/ansible/roles/collectd/handlers/main.yml D packaging/setup/ansible/roles/collectd/tasks/main.yml D packaging/setup/ansible/roles/firewall/README.md D packaging/setup/ansible/roles/firewall/commit/README.md D packaging/setup/ansible/roles/firewall/commit/meta/main.yml D packaging/setup/ansible/roles/firewall/commit/tasks/main.yml D packaging/setup/ansible/roles/firewall/defaults/main.yml D packaging/setup/ansible/roles/firewall/handlers/main.yml D packaging/setup/ansible/roles/firewall/tasks/main.yml D packaging/setup/ansible/roles/fluentd/defaults/main.yml D packaging/setup/ansible/roles/fluentd/handlers/main.yml D packaging/setup/ansible/roles/fluentd/tasks/main.yml D packaging/setup/ansible/roles/fluentd/templates/fluentd.conf D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/templates/builtins-conf-for-ovirt.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/templates/read-statsd.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/write_http/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/templates/write-http.conf.j2 D packaging/setup/ansible/roles/ovirt_fluentd/client/README.md D packaging/setup/ansible/roles/ovirt_fluentd/client/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_no_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/http_input/README.md D packaging/setup/ansible/roles/ovirt_fluentd/http_input/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/templates/http-input.conf D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/README.md D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/templates/ovirt-processing.conf D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/README.md D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/tasks/main.yml D packaging/setup/ansible/tools/validate-playbooks D packaging/setup/bin/ovirt-engine-hosts-ansible-inventory M packaging/setup/ovirt_engine_setup/constants.py 54 files changed, 1 insertion(+), 1,046 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73416 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.1 Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: Remove ovirt-engine-hosts-ansible-inventory
by Code Review 02 Mar '17

02 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: Remove ovirt-engine-hosts-ansible-inventory ...................................................................... Remove ovirt-engine-hosts-ansible-inventory And require instead ovirt-engine-metrics, which got its functionality Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Bug-Url: https://bugzilla.redhat.com/1422505 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M ovirt-engine.spec.in D packaging/setup/ansible/ansible.cfg D packaging/setup/ansible/config.yml.example D packaging/setup/ansible/configure_ovirt_hosts_for_metrics.sh D packaging/setup/ansible/group_vars/all.yml D packaging/setup/ansible/inventory/dynamic_ovirt_hosts D packaging/setup/ansible/inventory/hosts D packaging/setup/ansible/playbook.yml D packaging/setup/ansible/roles/collectd/defaults/main.yml D packaging/setup/ansible/roles/collectd/handlers/main.yml D packaging/setup/ansible/roles/collectd/tasks/main.yml D packaging/setup/ansible/roles/firewall/README.md D packaging/setup/ansible/roles/firewall/commit/README.md D packaging/setup/ansible/roles/firewall/commit/meta/main.yml D packaging/setup/ansible/roles/firewall/commit/tasks/main.yml D packaging/setup/ansible/roles/firewall/defaults/main.yml D packaging/setup/ansible/roles/firewall/handlers/main.yml D packaging/setup/ansible/roles/firewall/tasks/main.yml D packaging/setup/ansible/roles/fluentd/defaults/main.yml D packaging/setup/ansible/roles/fluentd/handlers/main.yml D packaging/setup/ansible/roles/fluentd/tasks/main.yml D packaging/setup/ansible/roles/fluentd/templates/fluentd.conf D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/ovirt_host/templates/builtins-conf-for-ovirt.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/read_statsd/templates/read-statsd.conf.j2 D packaging/setup/ansible/roles/ovirt_collectd/write_http/defaults/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/meta/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/tasks/main.yml D packaging/setup/ansible/roles/ovirt_collectd/write_http/templates/write-http.conf.j2 D packaging/setup/ansible/roles/ovirt_fluentd/client/README.md D packaging/setup/ansible/roles/ovirt_fluentd/client/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_no_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/client/templates/forward_ssl.conf D packaging/setup/ansible/roles/ovirt_fluentd/http_input/README.md D packaging/setup/ansible/roles/ovirt_fluentd/http_input/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/http_input/templates/http-input.conf D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/README.md D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/meta/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/tasks/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/ovirt_host/templates/ovirt-processing.conf D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/README.md D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/defaults/main.yml D packaging/setup/ansible/roles/ovirt_fluentd/read_local_ca_cert/tasks/main.yml D packaging/setup/ansible/tools/validate-playbooks D packaging/setup/bin/ovirt-engine-hosts-ansible-inventory M packaging/setup/ovirt_engine_setup/constants.py 54 files changed, 1 insertion(+), 1,046 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified -- To view, visit https://gerrit.ovirt.org/73363 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9b95eba077b73cbcc8f09fa22d25f1846762ae51 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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 jenkins[master]: publish ovirt-engine-metrics
by Code Review 02 Mar '17

02 Mar '17

>From Sandro Bonazzola <sbonazzo(a)redhat.com>: Sandro Bonazzola has submitted this change and it was merged. Change subject: publish ovirt-engine-metrics ...................................................................... publish ovirt-engine-metrics Change-Id: I63cbeeda06375ab018e998c69c5fdbcc09fe86f0 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M jobs/confs/projects/ovirt/publish-rpms-nightly-4.1.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-master.yaml 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/73378 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I63cbeeda06375ab018e998c69c5fdbcc09fe86f0 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: support for virt-v2v new feature '--vdsm-compat'
by Code Review 02 Mar '17

02 Mar '17

>From Shahar Havivi <shavivi(a)redhat.com>: Shahar Havivi has submitted this change and it was merged. Change subject: core: support for virt-v2v new feature '--vdsm-compat' ...................................................................... core: support for virt-v2v new feature '--vdsm-compat' Change-Id: Ia1761324cdcf30cd1756e4fbf9609edc9e11b315 Bug-Url: https://bugzilla.redhat.com/1417823 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ConvertVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmFromExternalProviderCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/ConvertVmParameters.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/ConvertVmVDSParameters.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/ConvertVmVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsProperties.java 6 files changed, 37 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Verified Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/71581 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia1761324cdcf30cd1756e4fbf9609edc9e11b315 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0