Change in ovirt-engine-cli[cli_3.6]: cli: Prepare for next release
by juan.hernandez@redhat.com
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>
8 years, 3 months
Change in ovirt-engine-cli[cli_3.6]: cli: Release 3.6.9.0
by juan.hernandez@redhat.com
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>
8 years, 3 months
Change in ovirt-engine-cli[cli_3.6]: cli: Ignore "id" and "name" options for the "show" command
by juan.hernandez@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: restapi: Adding ad_aggregator_id to host/nic
by juan.hernandez@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: restapi: Adding ad_partner_mac to Bonding
by juan.hernandez@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: core: cleanup managing vm
by ahadas@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: engine: remove redundant dependencies from UpdateNetworkVali...
by masayag@redhat.com
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>
8 years, 3 months
Change in gerrit-admin[master]: Added missing links and files to the hooks directory.
by eedri@redhat.com
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>
8 years, 3 months
Change in ovirt-engine[master]: core: cleanup in destroy vm
by ahadas@redhat.com
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>
8 years, 3 months
Change in jenkins[master]: Add label restriction to master repos-check-closure job
by eedri@redhat.com
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>
8 years, 3 months