Skip to content

Commit

Permalink
Use correct container log path for Trino nodes
Browse files Browse the repository at this point in the history
  • Loading branch information
wendigo authored and losipiuk committed Jan 27, 2021
1 parent 8436c95 commit 4a026ff
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
Expand Up @@ -123,7 +123,7 @@ public static DockerContainer createPrestoContainer(DockerFiles dockerFiles, Fil
{
DockerContainer container = new DockerContainer(dockerImageName, logicalName)
.withNetworkAliases(logicalName + ".docker.cluster")
.withExposedLogPaths("/var/presto/var/log", "/var/log/container-health.log")
.withExposedLogPaths("/var/trino/var/log", "/var/log/container-health.log")
.withCopyFileToContainer(forHostPath(dockerFiles.getDockerFilesHostPath()), "/docker/presto-product-tests")
.withCopyFileToContainer(forHostPath(dockerFiles.getDockerFilesHostPath("conf/presto/etc/jvm.config")), CONTAINER_PRESTO_JVM_CONFIG)
.withCopyFileToContainer(forHostPath(dockerFiles.getDockerFilesHostPath("health-checks/trino-health-check.sh")), CONTAINER_HEALTH_D + "trino-health-check.sh")
Expand Down
Expand Up @@ -10,4 +10,4 @@ fi
PID=$(jps | grep TrinoServer | cut -f1 -d ' ')

# Dump process information for debugging purposes
jstack -l $PID > /var/presto/var/log/stack.log || true
jstack -l $PID > /var/trino/var/log/stack.log || true
Expand Up @@ -24,5 +24,5 @@ export PATH="${JAVA_HOME}/bin:${PATH}"
exec /docker/presto-server/bin/launcher \
-Dnode.id="${HOSTNAME}" \
--etc-dir="/docker/presto-product-tests/conf/presto/etc" \
--data-dir=/var/presto \
--data-dir=/var/trino \
run

0 comments on commit 4a026ff

Please sign in to comment.