> +         this.diskMode = diskMode;
> +         this.isBootDisk = isBootDisk;
> +         this.diskSizeGb = diskSizeGb;
> +      }
> +
> +      public AutoCreateDiskOptions(final AttachDisk.Type diskType, final 
> AttachDisk.Mode diskMode, final boolean isBootDisk, final int diskSizeGb, 
> final String diskName) {
> +         this(diskType, diskMode, isBootDisk, diskSizeGb);
> +         // TODO: Validate against regex according to GCE spec.
> +         this.diskName = diskName;
> +      }
> +
> +      public String getDiskName(final String nodeName) {
> +         if (null != diskName)
> +            return diskName;
> +         else
> +            return "jclouds-" + UUID.randomUUID().toString();

Do you think it makes sense to include some aspect of the nodeName in the 
diskName? I am still figuring out the jclouds resource naming 
conventions/scheme.

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs-google/pull/135/files#r28214469

Reply via email to