looks good Wei, I have minor questions about the implementation but please go ahead.
I see you introduce a scope and also fields for network, vpc *and* loadbalancer UUIDs. Is this not redundant? scope and UUID would seem enough as the scope already defines the object type of the UUID, not? You create a 'created' field but not a 'removed' or 'updated'. For audit purposes these might be good to have as well, don't you think? In no way critique of your design, that ^^. Once a gain, looks good. On Fri, May 1, 2020 at 8:02 AM Wei ZHOU <ustcweiz...@gmail.com> wrote: > Our improvements to cloudstack load balancer (implemented by HAproxy in the > VRs) allow cloudstack users to manage certain restricted configuration > settings. With this feature, users can > > * Change basic configuration of HAproxy (e.g. set the amount of allowed > connections), > > * choose if load balancer is transparent, > > * enable or disable the support for SSL offloading in isolated networks. > > * choose if load balancer supports HTTP/2. > > * more settings. > > > > To make this possible to the user, we provide two forms on cloudstack GUI > (old GUI, will add changes on Primate) from which the settings can be > managed and applied in virtual routers. > > > > FS can be found at > > https://cwiki.apache.org/confluence/display/CLOUDSTACK/VR+haproxy+customization+in+CloudStack > > > Any suggestions ? > > > > Kind regards, > > *Wei Zhou* > > Principal Cloud Engineer > > Leaseweb Global B.V. > -- Daan