Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cloudstack.apache.org
From: wilderrodrigues <git@git.apache.org>
To: dev@cloudstack.apache.org
Reply-To: dev@cloudstack.apache.org
References: <git-pr-904-cloudstack@git.apache.org>
In-Reply-To: <git-pr-904-cloudstack@git.apache.org>
Subject: [GitHub] cloudstack pull request: CLOUDSTACK-8929
Content-Type: text/plain
Message-Id: <20151001114441.00DE3E08CE@git1-us-west.apache.org>
Date: Thu,  1 Oct 2015 11:44:41 +0000 (UTC)

Github user wilderrodrigues commented on the pull request:

    https://github.com/apache/cloudstack/pull/904#issuecomment-144704442
  
    Hi @runseb 
    
    If one tries to add the same rule for the same VM, ACS will say it cannot be done because the rule for that given VM already exists.
    
    If one doesn't want to see the 2 VMs on the list, instead of getting the error message after clicking "ok", the fix would be something like this:
    
    * Read the public/private ports entered and the VM info of those ports. Based on that, show the list with.
      * If the ports are different, all VMs in the network should be displayed. 
    
    If it's his intention to have the feature as described above, then he can amend it. But that would be an improvement, not a bug fix.
    
    Cheers,
    Wilder


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---