Github user wilderrodrigues commented on the pull request:
https://github.com/apache/cloudstack/pull/904#issuecomment-144700051
Hi @rags22489664 and @wido,
I think this fix will not work properly. For example:
1. Create 2 VMs on an isolated networkl
2. Acquire a new Public IP
3. Add a firewall rule to port 22
4. Add a firewall rule to port 80
5. Add a Load Balancing rule to one of the VMs for port 22
6. Add a Load Balancing rule to one of the VMs for port 80
See what I mean? I might want to see both VMs on the list in order to
decide which one should have the rule assigned to.
Will your change remove the first VM (assigned to port 22) form the list?
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.
---
