> +      /**
> +       * Provides the administrative state for this HealthMonitor's Builder.
> +       *
> +       * @return the Builder.
> +       * @see HealthMonitor#getAdminStateUp()
> +       */
> +      public ParameterizedBuilderType adminStateUp(Boolean adminStateUp) {
> +         healthMonitor.adminStateUp = adminStateUp;
> +         return self();
> +      }
> +   }
> +
> +   /**
> +    * Create builder (inheriting from Builder).
> +    */
> +   public static class CreateBuilder extends Builder<CreateBuilder> {

Those fields are required only if type is HTTP/HTTPS and to overwrite the 
default values. 
CreateBuilder inherits from Builder<CreateBuilder> that exposes the fields.

      HealthMonitor.CreateBuilder builder = 
HealthMonitor.createBuilder(ProbeType.HTTP, 10, 5, 
2).httpMethod(HttpMethod.POST).urlPath("/path").expectedCodes("201").build();

I could add additional CreateBuilder constructors dedicated to HTTP/HTTPS and 
with those parameters, but I'm not sure this is a good idea:

      HealthMonitor.CreateBuilder httpBuilder = 
HealthMonitor.createHttpBuilder(10, 5, 2, HttpMethod.POST, "/path", 
"201").build();
      HealthMonitor.CreateBuilder httpsBuilder = 
HealthMonitor.createHttpsBuilder(10, 5, 2, HttpMethod.POST, "/path", 
"201").build();

Let me know

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-openstack/pull/146/files#r17957618

Reply via email to