August 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine-cli[cli_3.6]: cli: Prepare for next release
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Prepare for next release ...................................................................... cli: Prepare for next release In preparation for the next release this patch increases the version number and sets the RPM release number to 0.1. Change-Id: Ib903397a9c71975a7283d0bd78fda838a515ebc8 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M setup.py 2 files changed, 3 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62678 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib903397a9c71975a7283d0bd78fda838a515ebc8 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-cli[cli_3.6]: cli: Release 3.6.9.0
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Release 3.6.9.0 ...................................................................... cli: Release 3.6.9.0 Change-Id: I24c0d1773402a375701d08d2d70c8e08ce0de30a Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-cli.spec.in M setup.py 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62677 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I24c0d1773402a375701d08d2d70c8e08ce0de30a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine-cli[cli_3.6]: cli: Ignore "id" and "name" options for the "show" command
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Ignore "id" and "name" options for the "show" command ...................................................................... cli: Ignore "id" and "name" options for the "show" command Currently auto-completion for the "show" command includes the "id" and "name" options. This is wrong, as these options aren't used to specify the object to be displayed. The reason they are included in auto-completion is that the underlying Python SDK method is "get", and that method usually has both the "id" and "name" parameters. We simply and blindly extract the parameters from the method and assume that they are valid options. This patch changes the CLI so that it ignores these parameters, thus they won't appear in the auto-completion list. Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13 Bug-Url: https://bugzilla.redhat.com/1358227 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/ovirtcli/shell/showcmdshell.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/62676 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I068686a6c600d0aac582adc239439c5f7dba0f13 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: cli_3.6 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Adding ad_aggregator_id to host/nic
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Adding ad_aggregator_id to host/nic ...................................................................... restapi: Adding ad_aggregator_id to host/nic Change-Id: I0c9f7eebe9c1131f57429935be224779eeedb83e Bug-Url: https://bugzilla.redhat.com/1281666 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java 1 file changed, 3 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved; Passed CI tests Marcin Mirecki: Verified -- To view, visit https://gerrit.ovirt.org/62135 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0c9f7eebe9c1131f57429935be224779eeedb83e Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Adding ad_partner_mac to Bonding
by juan.hernandez＠redhat.com 22 Aug '16

22 Aug '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Adding ad_partner_mac to Bonding ...................................................................... restapi: Adding ad_partner_mac to Bonding Change-Id: Ie4263db073d66d21a629fc8a3292746a9aa9c930 Bug-Url: https://bugzilla.redhat.com/1281666 Signed-off-by: mirecki <mmirecki(a)redhat.com> --- M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Marcin Mirecki: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61910 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4263db073d66d21a629fc8a3292746a9aa9c930 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cleanup managing vm
by ahadas＠redhat.com 22 Aug '16

22 Aug '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup managing vm ...................................................................... core: cleanup managing vm 1. Use the member resourceManager rather than the deprecated method ResourceManager#getInstance 2. Inline the method 'updateVmDataChangedTime' Change-Id: I4834e0e6ce876850ec74edf22247a84adf4a60e5 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ManagingVmCommand.java 1 file changed, 3 insertions(+), 10 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62310 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4834e0e6ce876850ec74edf22247a84adf4a60e5 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(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[master]: engine: remove redundant dependencies from UpdateNetworkVali...
by masayag＠redhat.com 22 Aug '16

22 Aug '16

Moti Asayag has submitted this change and it was merged. Change subject: engine: remove redundant dependencies from UpdateNetworkValidator ...................................................................... engine: remove redundant dependencies from UpdateNetworkValidator Change-Id: I6218c36f372a7015812257bdc7ac63a9ba72a72c Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkCommand.java M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkValidatorTest.java 2 files changed, 2 insertions(+), 17 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/62233 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6218c36f372a7015812257bdc7ac63a9ba72a72c Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(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 gerrit-admin[master]: Added missing links and files to the hooks directory.
by eedri＠redhat.com 22 Aug '16

22 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: Added missing links and files to the hooks directory. ...................................................................... Added missing links and files to the hooks directory. - Added the following links: change-abandoned ==> hook-dispatcher change-merged ==> hook-dispatcher comment-added ==> hook-dispatcher patchset-created ==> hook-dispatcher config ==> /home/gerrit2/config - Added missing config file that holds all the required params (i.e GERRIT_SRV, BZ_USER, BZ_PASS etc) to gerrit2 home directory. Change-Id: I1ffb0fa8f3a364b96f4cd192a86ec6b4d32165ca Signed-off-by: Shlomi Ben-David <sbendavi(a)redhat.com> --- A hooks/change-abandoned A hooks/change-merged A hooks/comment-added A hooks/config R hooks/config_files/README R hooks/config_files/ovirt-engine.config R hooks/config_files/vdsm.config R hooks/custom_hooks/comment-added.propagate_review_values.py A hooks/custom_hooks/patchset-created.bz.1.bug_url A hooks/custom_hooks/patchset-created.bz.1.bug_url.sh M hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch R hooks/custom_hooks/patchset-created.warn_if_not_merged_to_previous_branch.py M hooks/custom_hooks/update_tracker A hooks/patchset-created 14 files changed, 179 insertions(+), 24 deletions(-) Approvals: Shlomo Ben David: Verified Eyal Edri: Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62640 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1ffb0fa8f3a364b96f4cd192a86ec6b4d32165ca Gerrit-PatchSet: 4 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Shlomo Ben David <sbendavi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: cleanup in destroy vm
by ahadas＠redhat.com 22 Aug '16

22 Aug '16

Arik Hadas has submitted this change and it was merged. Change subject: core: cleanup in destroy vm ...................................................................... core: cleanup in destroy vm 1. query vm dynamic data only if the vm is actually destroyed 2. rename curVm to vm (no other VM is defined) 3. remove unused argument from #changeStatus Change-Id: I4a168284a366b19b05ab2df1df3869f589eec78b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java 1 file changed, 6 insertions(+), 11 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/62309 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a168284a366b19b05ab2df1df3869f589eec78b Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(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 jenkins[master]: Add label restriction to master repos-check-closure job
by eedri＠redhat.com 22 Aug '16

22 Aug '16

Eyal Edri has submitted this change and it was merged. Change subject: Add label restriction to master repos-check-closure job ...................................................................... Add label restriction to master repos-check-closure job Restrict master repos-check-closure jobs to run on fc24 nodes. This is a temporary solution until the job will run in mock. Change-Id: I7bde43984ed83cea06ca7285eb588805dc0c26aa Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- M jobs/confs/projects/repos-check-closure/repos-check-closure.yaml M jobs/confs/yaml/templates/repos-check-closure.yaml 2 files changed, 2 insertions(+), 0 deletions(-) Approvals: Sharon Naftaly: Verified; Ready for review Eyal Edri: Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/62647 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7bde43984ed83cea06ca7285eb588805dc0c26aa Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0