Skip to content

Commit

Permalink
Merge pull request #291 from zakkak/2024-10-18-setup-jaeger-for-tracing
Browse files Browse the repository at this point in the history
Start Jaeger to avoid errors related to opentracing/opentelemetry
  • Loading branch information
Karm authored Oct 29, 2024
2 parents a323b04 + 0811c8d commit a2afef3
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -80,6 +80,7 @@
import static org.graalvm.tests.integration.utils.Commands.parsePort;
import static org.graalvm.tests.integration.utils.Commands.parseSerialGCLog;
import static org.graalvm.tests.integration.utils.Commands.processStopper;
import static org.graalvm.tests.integration.utils.Commands.removeContainer;
import static org.graalvm.tests.integration.utils.Commands.runCommand;
import static org.graalvm.tests.integration.utils.Commands.waitForFileToMatch;
import static org.graalvm.tests.integration.utils.Commands.waitForTcpClosed;
Expand Down Expand Up @@ -430,6 +431,7 @@ public void testQuarkusFullMicroProfile(TestInfo testInfo) throws IOException, I
try {
// Cleanup
cleanTarget(app);
removeContainer("quarkus_jaeger");
Files.createDirectories(Paths.get(appDir.getAbsolutePath(), "logs"));

if (patch != null) {
Expand Down Expand Up @@ -524,6 +526,7 @@ public void testQuarkusFullMicroProfile(TestInfo testInfo) throws IOException, I
"target", "quarkus-native-image-source-jar", "quarkus-json.json").toFile());
Logs.archiveLog(cn, mn, processLog);
cleanTarget(app);
removeContainer("quarkus_jaeger");
if (patch != null) {
runCommand(getRunCommand("git", "apply", "-R", patch), appDir);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,7 @@
import static org.graalvm.tests.integration.utils.Commands.getRunCommand;
import static org.graalvm.tests.integration.utils.Commands.parsePort;
import static org.graalvm.tests.integration.utils.Commands.processStopper;
import static org.graalvm.tests.integration.utils.Commands.removeContainer;
import static org.graalvm.tests.integration.utils.Commands.runCommand;
import static org.graalvm.tests.integration.utils.Commands.stopAllRunningContainers;
import static org.graalvm.tests.integration.utils.Commands.stopRunningContainer;
Expand Down Expand Up @@ -86,6 +87,7 @@ public void testRuntime(TestInfo testInfo, Apps app, Map<String, String> switchR
try {
// Cleanup
cleanTarget(app);
removeContainer("quarkus_jaeger");
if (app.runtimeContainer != ContainerNames.NONE) {
// If we are about to be working with containers, we need a clean slate.
stopAllRunningContainers();
Expand Down Expand Up @@ -170,6 +172,7 @@ public void testRuntime(TestInfo testInfo, Apps app, Map<String, String> switchR
// it might be valuable to have the binary and not just the logs?
// Nope: Delete it. One can reproduce it from the journal file we maintain.
cleanTarget(app);
removeContainer("quarkus_jaeger");
}
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -99,6 +99,9 @@ public enum BuildAndRunCmds {
),
QUARKUS_FULL_MICROPROFILE(
new String[][] {
{ CONTAINER_RUNTIME, "run", "--rm", "-d", "-p", "4317:4317", "-e", "COLLECTOR_OTLP_ENABLED=true", "-p", "5775:5775/udp", "-p", "6831:6831/udp", "-p", "6832:6832/udp", "-p", "5778:5778", "-p",
"16686:16686", "-p", "14268:14268", "--name", "quarkus_jaeger", "quay.io/jaegertracing/all-in-one:latest"
},
{ "mvn", "package", "-Pnative", "-Dquarkus.version=" + QUARKUS_VERSION.getVersionString(),
"-Dquarkus.native.additional-build-args=" +
"-H:Log=registerResource:," +
Expand All @@ -119,6 +122,9 @@ public enum BuildAndRunCmds {
),
QUARKUS_FULL_MICROPROFILE_PERF(
new String[][] {
{ CONTAINER_RUNTIME, "run", "--rm", "-d", "-p", "4317:4317", "-e", "COLLECTOR_OTLP_ENABLED=true", "-p", "5775:5775/udp", "-p", "6831:6831/udp", "-p", "6832:6832/udp", "-p", "5778:5778", "-p",
"16686:16686", "-p", "14268:14268", "--name", "quarkus_jaeger", "quay.io/jaegertracing/all-in-one:latest"
},
{ "mvn", "package", "-Pnative", "-Dquarkus.version=" + QUARKUS_VERSION.getVersionString(),
"-Dquarkus.native.additional-build-args=" +
"-H:Log=registerResource:," +
Expand Down

0 comments on commit a2afef3

Please sign in to comment.