Mailing-List: contact issues-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
Date: Thu, 23 Jan 2014 13:09:38 +0000 (UTC)
From: "Wei Zhou (JIRA)" <jira@apache.org>
To: cloudstack-issues@incubator.apache.org
Message-ID: <JIRA.12687559.1389053769103.688.1390482578621@arcas>
In-Reply-To: <JIRA.12687559.1389053769103@arcas>
References: <JIRA.12687559.1389053769103@arcas>
Subject: [jira] [Commented] (CLOUDSTACK-5810) addIpToNic: the owner of the
 secondary ip should be derived from vmInstance object, not from the caller
 account
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Wei Zhou commented on CLOUDSTACK-5810:
--------------------------------------

Alena, Jayapal,

Could you change these patches for 4.3.0 ? Thanks!

-Wei

> 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.1.5#6160)