diff options
author | Harald Musum <musum@yahooinc.com> | 2021-12-29 14:52:24 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2021-12-29 14:52:24 +0100 |
commit | 73081f287816ea553c6d213649e7e6a8a9bd7d08 (patch) | |
tree | ced775c9dd85b6da9782bd2ffbe2ed350f59c1cc /config-application-package | |
parent | 7aebfa88991131648fcb2f3858849c09f198211e (diff) |
Add some logging to debug issue sometimes seen when running system tests
Diffstat (limited to 'config-application-package')
-rw-r--r-- | config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java | 32 |
1 files changed, 21 insertions, 11 deletions
diff --git a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java index 1223f438029..22285e027a1 100644 --- a/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java +++ b/config-application-package/src/main/java/com/yahoo/config/model/application/provider/FilesApplicationPackage.java @@ -47,6 +47,7 @@ import java.io.IOException; import java.io.InputStream; import java.io.Reader; import java.io.StringReader; +import java.nio.file.Files; import java.security.MessageDigest; import java.util.ArrayList; import java.util.Arrays; @@ -61,6 +62,7 @@ import java.util.Optional; import java.util.Set; import java.util.logging.Level; import java.util.logging.Logger; +import java.util.stream.Collectors; import static com.yahoo.text.Lowercase.toLowerCase; @@ -600,17 +602,25 @@ public class FilesApplicationPackage implements ApplicationPackage { @Override public ApplicationPackage preprocess(Zone zone, DeployLogger logger) throws IOException { - IOUtils.recursiveDeleteDir(preprocessedDir); - IOUtils.copyDirectory(appDir, preprocessedDir, -1, (dir, name) -> ! name.equals(preprocessed) && - ! name.equals(SERVICES) && - ! name.equals(HOSTS) && - ! name.equals(CONFIG_DEFINITIONS_DIR)); - File servicesFile = validateServicesFile(); - preprocessXML(new File(preprocessedDir, SERVICES), servicesFile, zone); - preprocessXML(new File(preprocessedDir, HOSTS), getHostsFile(), zone); - FilesApplicationPackage preprocessed = fromFile(preprocessedDir, includeSourceFiles); - preprocessed.copyUserDefsIntoApplication(); - return preprocessed; + try { + IOUtils.recursiveDeleteDir(preprocessedDir); + IOUtils.copyDirectory(appDir, preprocessedDir, -1, (dir, name) -> !name.equals(preprocessed) && + !name.equals(SERVICES) && + !name.equals(HOSTS) && + !name.equals(CONFIG_DEFINITIONS_DIR)); + File servicesFile = validateServicesFile(); + preprocessXML(new File(preprocessedDir, SERVICES), servicesFile, zone); + preprocessXML(new File(preprocessedDir, HOSTS), getHostsFile(), zone); + FilesApplicationPackage preprocessed = fromFile(preprocessedDir, includeSourceFiles); + preprocessed.copyUserDefsIntoApplication(); + return preprocessed; + } catch (FileNotFoundException e) { + // TODO: Temporary logging for debugging purposes + log.log(Level.INFO, "File not found. Files in application package dir: " + Files.list(appDir.toPath()) + .map(java.nio.file.Path::toString) + .collect(Collectors.joining(","))); + throw e; + } } private File validateServicesFile() throws IOException { |