[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-5810?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14226433#comment-14226433
 ] 

Rohit Yadav commented on CLOUDSTACK-5810:
-----------------------------------------

Can any of [~alena1108] or [~jayapal] help backport the fixes to 4.3 branch, 
thanks?

> addIpToNic: the owner of the secondary ip should be derived from vmInstance 
> object, not from the caller account
> ---------------------------------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-5810
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-5810
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Management Server
>    Affects Versions: 4.3.0
>            Reporter: Alena Prokharchyk
>            Assignee: Jayapal Reddy
>            Priority: Critical
>             Fix For: 4.4.0
>
>
> Steps to reproduce:
> ====================
> 1) Deploy a vm as a regular user account.
> 2) Login as admin, add secondary ip to the user's vm's nic.
> Bug: the secondary ip account owner is set to Admin account. This is wrong, 
> as in CS we never let link objects belonging to diff accounts, unless its a 
> public resource (template, network). In cases like this one, the owner info 
> should be derived from the vm instance object.
> Jayapal, I will fix the API, and you have to fix the DB upgrade part. The fix 
> should be: compare the sec ips accounts with the account of corresponding 
> vms, and update nic_secondary_ips if account info is different. Should be 
> done as a part of 43-44 upgrade.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to