Bug 1328011
Summary: | Engine: internal admin cannot migrate VM (permission issue) | ||||||
---|---|---|---|---|---|---|---|
Product: | [oVirt] ovirt-engine | Reporter: | Yaniv Kaul <ykaul> | ||||
Component: | BLL.Virt | Assignee: | Tomas Jelinek <tjelinek> | ||||
Status: | CLOSED CURRENTRELEASE | QA Contact: | Israel Pinto <ipinto> | ||||
Severity: | high | Docs Contact: | |||||
Priority: | high | ||||||
Version: | 4.0.0 | CC: | bugs, eedri, sbonazzo, tjelinek | ||||
Target Milestone: | ovirt-4.0.0-beta | Keywords: | AutomationBlocker, Regression | ||||
Target Release: | 4.0.0 | Flags: | rule-engine:
ovirt-4.0.0+
rule-engine: blocker+ rule-engine: planning_ack+ michal.skrivanek: devel_ack+ rule-engine: testing_ack+ |
||||
Hardware: | x86_64 | ||||||
OS: | Linux | ||||||
Whiteboard: | |||||||
Fixed In Version: | Doc Type: | Bug Fix | |||||
Doc Text: |
Cause: migrating a VM using REST API without specifying the target cluster ID failed with "not authorized" error
Consequence: to migrate a VM using REST API the target cluster ID had to be specified explicitly even if migrated inside the same cluster.
Fix: Made sure that when migrating inside the same cluster the cluster ID does not need to be specified.
|
Story Points: | --- | ||||
Clone Of: | Environment: | ||||||
Last Closed: | 2016-07-05 07:58:47 UTC | Type: | Bug | ||||
Regression: | --- | Mount Type: | --- | ||||
Documentation: | --- | CRM: | |||||
Verified Versions: | Category: | --- | |||||
oVirt Team: | Virt | RHEL 7.3 requirements from Atomic Host: | |||||
Cloudforms Team: | --- | Target Upstream Version: | |||||
Embargoed: | |||||||
Attachments: |
|
Description
Yaniv Kaul
2016-04-18 08:48:25 UTC
Created attachment 1148132 [details]
engine.log
regression introduced by https://gerrit.ovirt.org/#/c/53092/2 - fix on the way workaround is to send target cluster id explicitly, hence decreasing severity (In reply to Michal Skrivanek from comment #3) > workaround is to send target cluster id explicitly, hence decreasing severity Please fix ovirt-system-tests then, or fix the bug. I'm still unsure how it was not noticed until now. I'm testing if the workaround works right now, will notify here Adding this to the call did not work (not using the patch): migrate_params = params.Action( host=params.Host( name=sorted(host_names)[1] ), #TODO: remove once bz1328011 is solved cluster=params.Cluster( name=TEST_CLUSTER, ), ) Will try now with latest master with the merged patch (In reply to David Caro from comment #6) > Adding this to the call did not work (not using the patch): > > migrate_params = params.Action( > host=params.Host( > name=sorted(host_names)[1] > ), > #TODO: remove once bz1328011 is solved > cluster=params.Cluster( > name=TEST_CLUSTER, because only the cluster ID works, not the cluster name. Patch is following soon > ), > ) > > Will try now with latest master with the merged patch (In reply to Tomas Jelinek from comment #7) > (In reply to David Caro from comment #6) > > Adding this to the call did not work (not using the patch): > > > > migrate_params = params.Action( > > host=params.Host( > > name=sorted(host_names)[1] > > ), > > #TODO: remove once bz1328011 is solved > > cluster=params.Cluster( > > name=TEST_CLUSTER, > > because only the cluster ID works, not the cluster name. Patch is following > soon after this is merged also the name will work: https://gerrit.ovirt.org/#/c/56312/ > > > ), > > ) > > > > Will try now with latest master with the merged patch Verify with: RHEVM Version: 4.0.0.2-0.1.el7ev Hosts: OS Version:RHEL - 7.2 - 9.el7_2.1 Kernel Version:3.10.0 - 327.22.1.el7.x86_64 KVM Version:2.3.0 - 31.el7_2.15 LIBVIRT Version:libvirt-1.2.17-13.el7_2.5 VDSM Version:vdsm-4.18.3-0.el7ev Steps: 1. Create VM 2. Migrate VM in the same cluster via REST API Results: VM migrate successfully oVirt 4.0.0 has been released, closing current release. |