cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Daan Hoogland <daan.hoogl...@gmail.com>
Subject Re: Review Request 14320: add boolean option httpModeEnabled to the service offering for use in haproxy conf
Date Sat, 28 Sep 2013 11:18:58 GMT
Chiradeep,

the network offerings are created by the cloud operator aren't they? The
netscaler  en f5 modules will have to implement it's own behavior on
httpClose. in case of haproxy it means no mode http and option httpclose
(and some other things)

If you define it zone wide every tenant has the same setting whilst you
want this to tune setting (like with maxConnections) for a tenant.

regards,
Daan


On Thu, Sep 26, 2013 at 10:57 PM, Chiradeep Vittal <chiradeepv@gmail.com>wrote:

>    This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/14320/
>
> Not sure if this should be in the API since it is a HAProxy-specific configuration. This
wouldn't apply to Netscaler or F5.
> After all the end user has no idea if he is using HAProxy of Netscaler or F5.
>
> Likely this flag is of interest to the cloud operator only, so why not put it in zone-wide
config instead of the network offering.
> Do you really see someone creating 2 offerings: one with HttpClose and one without HttpClose?
>
>
> - Chiradeep Vittal
>
> On September 26th, 2013, 7:01 p.m. UTC, daan Hoogland wrote:
>   Review request for cloudstack and Wei Zhou.
> By daan Hoogland.
>
> *Updated Sept. 26, 2013, 7:01 p.m.*
>  *Bugs: * CLOUDSTACK-4328
>  *Repository: * cloudstack-git
> Description
>
> add boolean option httpModeEnabled to the service offering for use in haproxy conf
>
>   Testing
>
> created unit test.
> instantiated a network with some loadbalancer rule based on a netoffer with the option
to true/false and maxconnections to a non default value -> checked haproxy.cfg on the router
>
>   Diffs
>
>    - api/src/com/cloud/offering/NetworkOffering.java (6c5573e)
>    - api/src/org/apache/cloudstack/api/ApiConstants.java (f85784b)
>    - api/src/org/apache/cloudstack/api/command/admin/network/CreateNetworkOfferingCmd.java
>    (bdad904)
>    - api/src/org/apache/cloudstack/api/command/admin/network/UpdateNetworkOfferingCmd.java
>    (c9c4c8a)
>    - core/src/com/cloud/agent/api/routing/LoadBalancerConfigCommand.java
>    (ee29290)
>    - core/src/com/cloud/network/HAProxyConfigurator.java (2309125)
>    - core/test/com/cloud/network/HAProxyConfiguratorTest.java
>    (PRE-CREATION)
>    - engine/components-api/src/com/cloud/configuration/ConfigurationManager.java
>    (5e1b9b5)
>    - engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
>    (53f64fd)
>    - engine/schema/src/com/cloud/offerings/NetworkOfferingVO.java
>    (eefdc94)
>    - plugins/network-elements/elastic-loadbalancer/src/com/cloud/network/lb/ElasticLoadBalancerManagerImpl.java
>    (ecd6006)
>    - plugins/network-elements/internal-loadbalancer/src/org/apache/cloudstack/network/lb/InternalLoadBalancerVMManagerImpl.java
>    (587ae99)
>    - server/src/com/cloud/configuration/ConfigurationManagerImpl.java
>    (8a0f7a6)
>    - server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java
>    (7c026a4)
>    - server/test/com/cloud/vpc/MockConfigurationManagerImpl.java (c9a0480)
>    - server/test/org/apache/cloudstack/networkoffering/CreateNetworkOfferingTest.java
>    (1f1fb75)
>    - setup/db/db/schema-420to430.sql (44a884d)
>
> View Diff <https://reviews.apache.org/r/14320/diff/>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message