diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-05-14 12:28:32 +0200 |
---|---|---|
committer | Harald Musum <musum@verizonmedia.com> | 2020-05-14 12:28:32 +0200 |
commit | cd98c493a23f6c1da3da274ea8c82459f764465d (patch) | |
tree | 0e1f29d0a4be013369f8d667483c0b5e1c68af59 | |
parent | 82cb4a30a63900c0aa853874e123baaab8162021 (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.java | 2 | ||||
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionPreparer.java | 6 |
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() |