cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rohit Yadav (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CLOUDSTACK-5810) addIpToNic: the owner of the secondary ip should be derived from vmInstance object, not from the caller account
Date Wed, 26 Nov 2014 16:49:13 GMT

    [ 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)

Mime
View raw message