Re: Review Request 28049: CLOUDSTACK-7917: Load Balancer Rule is not validated when updating LB

2014-11-25 Thread Daniel Vega Simoes
Thanks Rajani - Daniel --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28049/#review62950 --- On Nov. 24, 2014, 4:23 p.m., Daniel Vega Simoes

Re: Review Request 28049: CLOUDSTACK-7917: Load Balancer Rule is not validated when updating LB

2014-11-24 Thread Daniel Vega Simoes
/UpdateLoadBalancerTest.java - Test to guarantee that validate is being called when updating LB rule - Test to guarantee that it throws exception if LB rule is not validated by provider Ran integration tests (smoke), all seems OK. Thanks, Daniel Vega Simoes

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

2014-11-18 Thread Daniel Vega Simoes
master. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28043/#review61817 --- On Nov. 17, 2014, 6:26 p.m., Daniel Vega Simoes

Re: Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

2014-11-17 Thread Daniel Vega Simoes
with DefaultSharedNetworkWithSourceNat, acquire new IP, list load balancer algorithms (UI) It should work correctly as long as network offering is configured with LB provider/capabilities. Otherwise, list of algorithms is empty in UI. Thanks, Daniel Vega Simoes

Review Request 28043: CLOUDSTACK 7915: Remove hard-coded values for Load Balancer algorithms in UI

2014-11-14 Thread Daniel Vega Simoes
ui/scripts/vpc.js af19d87 Diff: https://reviews.apache.org/r/28043/diff/ Testing --- Thanks, Daniel Vega Simoes

Review Request 28049: CLOUDSTACK-7917: Load Balancer Rule is not validated when updating LB

2014-11-14 Thread Daniel Vega Simoes
--- Validate Load Balancer rule with provider before commiting to DB and applying new rule. Diffs - server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java a28d108 Diff: https://reviews.apache.org/r/28049/diff/ Testing --- Thanks, Daniel Vega Simoes

Re: Review Request 28049: CLOUDSTACK-7917: Load Balancer Rule is not validated when updating LB

2014-11-14 Thread Daniel Vega Simoes
: https://reviews.apache.org/r/28049/diff/ Testing --- Thanks, Daniel Vega Simoes

Re: Review Request 27641: CLOUDSTACK-7849

2014-11-06 Thread Daniel Vega Simoes
., Daniel Vega Simoes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27641/ --- (Updated Nov. 5, 2014, 8:39 p.m.) Review

Re: Review Request 27641: CLOUDSTACK-7849

2014-11-06 Thread Daniel Vega Simoes
/27641/#review60112 --- On Nov. 5, 2014, 8:39 p.m., Daniel Vega Simoes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27641

Review Request 27641: CLOUDSTACK-7849

2014-11-05 Thread Daniel Vega Simoes
Diff: https://reviews.apache.org/r/27641/diff/ Testing --- Thanks, Daniel Vega Simoes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-21 Thread Daniel Vega Simoes
that version/release of ACS. I'll be able to test this tomorrow, meanwhile can you share with us demo/test credentials for testing against the DNS service (or point us to a link to sign up for it if it's free). Daniel Vega Simoes wrote: I'll fix the imports and the APICommand. We're

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-20 Thread Daniel Vega Simoes
. Thanks, Daniel Vega Simoes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-20 Thread Daniel Vega Simoes
that version/release of ACS. I'll be able to test this tomorrow, meanwhile can you share with us demo/test credentials for testing against the DNS service (or point us to a link to sign up for it if it's free). Daniel Vega Simoes wrote: I'll fix the imports and the APICommand. We're

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-19 Thread Daniel Vega Simoes
that version/release of ACS. I'll be able to test this tomorrow, meanwhile can you share with us demo/test credentials for testing against the DNS service (or point us to a link to sign up for it if it's free). Daniel Vega Simoes wrote: I'll fix the imports and the APICommand. We're

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-14 Thread Daniel Vega Simoes
that version/release of ACS. I'll be able to test this tomorrow, meanwhile can you share with us demo/test credentials for testing against the DNS service (or point us to a link to sign up for it if it's free). Daniel Vega Simoes wrote: I'll fix the imports and the APICommand. We're

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
. - Daniel --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24611/#review50437 --- On Aug. 12, 2014, 9:43 p.m., Daniel Vega Simoes wrote

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
of DNS domains when network is destroyed You can check all use cases related to the override flag on the design document. On this first release, tests were performed using Bind server as DNS server. Further improvements could include PowerDNS as well. Thanks, Daniel Vega Simoes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
--- On Aug. 13, 2014, 2:43 p.m., Daniel Vega Simoes wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24611/ --- (Updated Aug. 13, 2014

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-13 Thread Daniel Vega Simoes
all use cases related to the override flag on the design document. On this first release, tests were performed using Bind server as DNS server. Further improvements could include PowerDNS as well. Thanks, Daniel Vega Simoes

Re: Review Request 24611: New DNS provider (GloboDNS)

2014-08-12 Thread Daniel Vega Simoes
performed using Bind server as DNS server. Further improvements could include PowerDNS as well. Thanks, Daniel Vega Simoes