> + this.groupNamingConvention = namingConvention;
> +// this.serverLatestJobCompleted = retry(new
> ServerLatestJobCompleted(client.getJobServices()),
> +// timeouts.nodeRunning * 9l / 10l);
> +// this.serverLatestJobCompletedShort = retry(new
> ServerLatestJobCompleted(client.getJobServices()),
> +// timeouts.nodeRunning * 1l / 10l);
> + }
> +
> +
> + @Override
> + public NodeAndInitialCredentials<VirtualMachine>
> createNodeWithGroupEncodedIntoName(String group, String name,
> +
> Template template) {
> + logger.info("createNodeWithGroupEncodedIntoName");
> + checkNotNull(template, "template was null");
> + checkNotNull(template.getOptions(), "template options was null");
> +
> checkArgument(template.getOptions().getClass().isAssignableFrom(AzureTemplateOptions.class),
> + "options class %s should have been assignable from
> AzureTemplateOptions", template.getOptions().getClass());
These three precondition checks are redundant. Will always be OK, as the
injector and the compute service module configuration will enforce this.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/132/files#r24748900
