> +            InitializeRunScriptOnNodeOrPlaceInBadMap.Factory 
> initScriptRunnerFactory,
> +            RunScriptOnNode.Factory runScriptOnNodeFactory, InitAdminAccess 
> initAdminAccess,
> +            PersistNodeCredentials persistNodeCredentials, Timeouts timeouts,
> +            @Named(Constants.PROPERTY_USER_THREADS) ListeningExecutorService 
> userExecutor, CloudStackClient client,
> +            LoadingCache<ZoneAndName, SecurityGroup> securityGroupMap,
> +            LoadingCache<String, SshKeyPair> keyPairCache,
> +            Function<Set<? extends NodeMetadata>, Multimap<String, String>> 
> orphanedGroupsByZoneId,
> +            GroupNamingConvention.Factory namingConvention,
> +            Supplier<LoadingCache<String, Zone>> zoneIdToZone,
> +            Optional<ImageExtension> imageExtension) {
> +      super(context, credentialStore, images, sizes, locations, 
> listNodesStrategy, getImageStrategy,
> +               getNodeMetadataStrategy, runNodesAndAddToSetStrategy, 
> rebootNodeStrategy, destroyNodeStrategy,
> +               startNodeStrategy, stopNodeStrategy, templateBuilderProvider, 
> templateOptionsProvider, nodeRunning,
> +               nodeTerminated, nodeSuspended, initScriptRunnerFactory, 
> initAdminAccess, runScriptOnNodeFactory,
> +               persistNodeCredentials, timeouts, userExecutor, 
> imageExtension);
> +      this.zoneIdToZone = zoneIdToZone;

Is this nullable? If it's not nullable, should we `checkNotNull`? If yes, 
should we add `@Nullable`?

---
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/45/files#r4842125

Reply via email to