Skip to content

Commit 5500602

Browse files
committed
Only warn about DockerContainer log paths when configured
1 parent 2303e30 commit 5500602

File tree

1 file changed

+4
-1
lines changed
  • testing/trino-product-tests-launcher/src/main/java/io/trino/tests/product/launcher/env

1 file changed

+4
-1
lines changed

testing/trino-product-tests-launcher/src/main/java/io/trino/tests/product/launcher/env/DockerContainer.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,6 @@ public void copyFileToContainer(Transferable transferable, String containerPath)
157157

158158
public DockerContainer withExposedLogPaths(String... logPaths)
159159
{
160-
requireNonNull(this.logPaths, "log paths are already exposed");
161160
this.logPaths.addAll(Arrays.asList(logPaths));
162161
return this;
163162
}
@@ -292,6 +291,10 @@ public ExecResult execCommandForResult(String... command)
292291

293292
public void copyLogsToHostPath(Path hostPath)
294293
{
294+
if (logPaths.isEmpty()) {
295+
return;
296+
}
297+
295298
if (!isRunning()) {
296299
log.warn("Could not copy files from stopped container %s", logicalName);
297300
return;

0 commit comments

Comments
 (0)