summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2020-05-14 12:28:32 +0200
committerHarald Musum <musum@verizonmedia.com>2020-05-14 12:28:32 +0200
commitcd98c493a23f6c1da3da274ea8c82459f764465d (patch)
tree0e1f29d0a4be013369f8d667483c0b5e1c68af59
parent82cb4a30a63900c0aa853874e123baaab8162021 (diff)
Add method for adding application package to file registry
-rw-r--r--config-model-api/src/main/java/com/yahoo/config/application/api/FileRegistry.java2
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java6
2 files changed, 4 insertions, 4 deletions
diff --git a/config-model-api/src/main/java/com/yahoo/config/application/api/FileRegistry.java b/config-model-api/src/main/java/com/yahoo/config/application/api/FileRegistry.java
index 7df0e941731..0d4c0ad76eb 100644
--- a/config-model-api/src/main/java/com/yahoo/config/application/api/FileRegistry.java
+++ b/config-model-api/src/main/java/com/yahoo/config/application/api/FileRegistry.java
@@ -11,8 +11,10 @@ import com.yahoo.config.FileReference;
*/
public interface FileRegistry {
+
FileReference addFile(String relativePath);
FileReference addUri(String uri);
+ default FileReference addApplicationPackage() { return addFile(""); }
/**
* Returns the name of the host which is the source of the files
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java
index 1bd8de7a494..54d4cc8654b 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java
@@ -77,7 +77,6 @@ public class SessionPreparer {
private final ConfigDefinitionRepo configDefinitionRepo;
private final Curator curator;
private final Zone zone;
- private final FlagSource flagSource;
private final SecretStore secretStore;
private final BooleanFlag distributeApplicationPackage;
private final BooleanFlag unsetEndpoints;
@@ -101,7 +100,6 @@ public class SessionPreparer {
this.configDefinitionRepo = configDefinitionRepo;
this.curator = curator;
this.zone = zone;
- this.flagSource = flagSource;
this.secretStore = secretStore;
this.distributeApplicationPackage = Flags.CONFIGSERVER_DISTRIBUTE_APPLICATION_PACKAGE.bindTo(flagSource);
this.unsetEndpoints = Flags.CONFIGSERVER_UNSET_ENDPOINTS.bindTo(flagSource);
@@ -124,7 +122,7 @@ public class SessionPreparer {
.value();
Preparation preparation = new Preparation(context, logger, params, currentActiveApplicationSet, tenantPath,
allowUnsettingEndpoints);
- preparation.distributeApplicationPackage();
+ preparation.distributeApplicationPackage(); // Note: Done before pre-processing, requires that to be done for users of this
preparation.preprocess();
try {
AllocatedHosts allocatedHosts = preparation.buildModels(now);
@@ -244,7 +242,7 @@ public class SessionPreparer {
if ( ! distributeApplicationPackage.value()) return;
FileRegistry fileRegistry = fileDistributionProvider.getFileRegistry();
- FileReference fileReference = fileRegistry.addFile("");
+ FileReference fileReference = fileRegistry.addApplicationPackage();
FileDistribution fileDistribution = fileDistributionProvider.getFileDistribution();
log.log(Level.INFO, "Distribute application package for " + applicationId + " (" + fileReference + ") to other config servers");
properties.configServerSpecs().stream()