> + 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