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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to