diff --git a/java/com.sap.sailing.landscape.test/src/com/sap/sailing/landscape/test/TestProcedures.java b/java/com.sap.sailing.landscape.test/src/com/sap/sailing/landscape/test/TestProcedures.java index 9c00a3fde58..459e37c4795 100755 --- a/java/com.sap.sailing.landscape.test/src/com/sap/sailing/landscape/test/TestProcedures.java +++ b/java/com.sap.sailing.landscape.test/src/com/sap/sailing/landscape/test/TestProcedures.java @@ -207,9 +207,8 @@ public class TestProcedures { SailingAnalyticsApplicationConfiguration, AppConfigBuilderT> multiServerAppDeployerBuilder = DeployProcessOnMultiServer.>, - SailingAnalyticsApplicationConfiguration, AppConfigBuilderT> builder(multiServerAppConfigBuilder); + SailingAnalyticsApplicationConfiguration, AppConfigBuilderT> builder(multiServerAppConfigBuilder, host); multiServerAppDeployerBuilder - .setHostToDeployTo(host) .setPrivateKeyEncryptionPassphrase(privateKeyEncryptionPassphrase) .setOptionalTimeout(optionalTimeout); multiServerAppConfigBuilder diff --git a/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/impl/LandscapeServiceImpl.java b/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/impl/LandscapeServiceImpl.java index 5ca099149a3..f7db52b7330 100644 --- a/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/impl/LandscapeServiceImpl.java +++ b/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/impl/LandscapeServiceImpl.java @@ -192,9 +192,10 @@ public class LandscapeServiceImpl implements LandscapeService { region, release); final DeployProcessOnMultiServer.Builder, SailingAnalyticsMasterConfiguration, AppConfigBuilderT> multiServerAppDeployerBuilder = DeployProcessOnMultiServer., SailingAnalyticsMasterConfiguration, AppConfigBuilderT> builder( - masterConfigurationBuilder); - multiServerAppDeployerBuilder.setHostToDeployTo(hostToDeployTo) - .setPrivateKeyEncryptionPassphrase(privateKeyEncryptionPassphrase).setOptionalTimeout(LandscapeService.WAIT_FOR_HOST_TIMEOUT); + masterConfigurationBuilder, hostToDeployTo); + multiServerAppDeployerBuilder + .setPrivateKeyEncryptionPassphrase(privateKeyEncryptionPassphrase) + .setOptionalTimeout(LandscapeService.WAIT_FOR_HOST_TIMEOUT); final DeployProcessOnMultiServer, SailingAnalyticsMasterConfiguration, AppConfigBuilderT> deployer = multiServerAppDeployerBuilder.build(); deployer.run(); final SailingAnalyticsProcess master = deployer.getProcess(); diff --git a/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/procedures/DeployProcessOnMultiServer.java b/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/procedures/DeployProcessOnMultiServer.java index 858f6e69505..0a4e17323ea 100755 --- a/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/procedures/DeployProcessOnMultiServer.java +++ b/java/com.sap.sailing.landscape/src/com/sap/sailing/landscape/procedures/DeployProcessOnMultiServer.java @@ -91,7 +91,6 @@ implements Procedure { ApplicationConfigurationT extends SailingAnalyticsApplicationConfiguration, ApplicationConfigurationBuilderT extends SailingAnalyticsApplicationConfiguration.Builder> extends AbstractAwsProcedureImpl.Builder, ShardingKey> { - BuilderT setHostToDeployTo(SailingAnalyticsHost hostToDeployTo); BuilderT setKeyName(String keyName); BuilderT setPrivateKeyEncryptionPassphrase(byte[] privateKeyEncryptionPassphrase); } @@ -102,13 +101,15 @@ implements Procedure { extends AbstractAwsProcedureImpl.BuilderImpl, ShardingKey> implements Builder { private final SailingAnalyticsApplicationConfiguration.BuilderImpl applicationConfigurationBuilder; - private SailingAnalyticsHost hostToDeployTo; + private final SailingAnalyticsHost hostToDeployTo; private Optional optionalKeyName = Optional.empty(); private byte[] privateKeyEncryptionPassphrase; - protected BuilderImpl(SailingAnalyticsApplicationConfiguration.BuilderImpl applicationConfigurationBuilder) { + protected BuilderImpl(SailingAnalyticsApplicationConfiguration.BuilderImpl applicationConfigurationBuilder, + SailingAnalyticsHost hostToDeployTo) { super(); this.applicationConfigurationBuilder = applicationConfigurationBuilder; + this.hostToDeployTo = hostToDeployTo; } @Override @@ -182,12 +183,6 @@ implements Procedure { return (AwsLandscape) super.getLandscape(); } - @Override - public BuilderT setHostToDeployTo(SailingAnalyticsHost hostToDeployTo) { - this.hostToDeployTo = hostToDeployTo; - return self(); - } - protected SailingAnalyticsHost getHostToDeployTo() { return hostToDeployTo; } @@ -228,11 +223,12 @@ implements Procedure { public static , ShardingKey, HostT extends AwsInstance, ApplicationConfigurationT extends SailingAnalyticsApplicationConfiguration, ApplicationConfigurationBuilderT extends SailingAnalyticsApplicationConfiguration.Builder> - Builder builder(ApplicationConfigurationBuilderT applicationConfigurationBuilder) { + Builder builder(ApplicationConfigurationBuilderT applicationConfigurationBuilder, + SailingAnalyticsHost hostToDeployTo) { @SuppressWarnings("unchecked") final SailingAnalyticsApplicationConfiguration.BuilderImpl applicationConfigurationBuilderCast = (SailingAnalyticsApplicationConfiguration.BuilderImpl) applicationConfigurationBuilder; - return new BuilderImpl<>(applicationConfigurationBuilderCast); + return new BuilderImpl<>(applicationConfigurationBuilderCast, hostToDeployTo); } protected DeployProcessOnMultiServer(BuilderImpl builder) throws Exception {