> + Optional<ImageExtension> imageExtension,
> Optional<SecurityGroupExtension> securityGroupExtension) {
> + super(context, credentialStore, images, hardwareProfiles, locations,
> listNodesStrategy, getImageStrategy,
> + getNodeMetadataStrategy, runNodesAndAddToSetStrategy,
> rebootNodeStrategy, destroyNodeStrategy,
> + resumeNodeStrategy, suspendNodeStrategy,
> templateBuilderProvider, templateOptionsProvider, nodeRunning,
> + nodeTerminated, nodeSuspended, initScriptRunnerFactory,
> initAdminAccess, runScriptOnNodeFactory,
> + persistNodeCredentials, timeouts, userExecutor,
> imageExtension, securityGroupExtension);
> + }
> +
> + /**
> + * Returns template options, except of type {@link
> org.jclouds.azurecompute.compute.options.AzureTemplateOptions}.
> + */
> + @Override
> + public AzureTemplateOptions templateOptions() {
> + return AzureTemplateOptions.class.cast(super.templateOptions());
> + }
> +}
There is no need to subclass the BaseComputeService just to override this. No
one will directly use this concrete implementation, and the injector will
already provide the right instance of the options class, as a binding for it
should be configured in the ComputeserviceContextModule.
This class could be removed.
---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds-labs/pull/132/files#r24748268
