[GitHub] cloudstack issue #756: CLOUDSTACK-8781 : Superfluous field during VPC creati...

[prashanthvarma]

Github user prashanthvarma commented on the issue:

    https://github.com/apache/cloudstack/pull/756
  
    @rhtyd Here are the screen-shots without and with this UI bug fix:
    
    
![6a4ded44-adb4-4821-bceb-3eb8c44c79f8](https://cloud.githubusercontent.com/assets/3722369/20486648/d037587a-b000-11e6-9cda-f82e52747626.jpg)
    
    
![rsz_screenshot_from_2016-11-21_15-15-52](https://cloud.githubusercontent.com/assets/3722369/20486654/d65af0ae-b000-11e6-9201-d2df4e8bcd7f.jpg)
    
    As pointed out by @nlivens in the previous comment, "The actual Public Load 
Balancer provider is derived from the VPC offering, and not from this extra 
field "Public Load Balancer Provider". So, this field has no use at all since 
it's completely ignored."
    
    The cleaner implementation for this UI tab would be to only list "Public 
Load Balancer Providers" mentioned in the selected VPC offering. Moreover, 
don't show this tab, if there are no "Public Load Balancer Providers" being 
mentioned in the selected VPC offering. Finally, make use of the selected  
"Public Load Balancer Provider" in the actual implementation of the VPC.


---
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.
---