Skip to content

Commit 84e5026

Browse files
committed
Increase TracerFlare smoke test time
1 parent 2f8c79e commit 84e5026

File tree

3 files changed

+8
-7
lines changed

3 files changed

+8
-7
lines changed

dd-java-agent/agent-profiling/src/main/java/com/datadog/profiling/agent/ProfilerFlare.java renamed to dd-java-agent/agent-profiling/src/main/java/com/datadog/profiling/agent/ProfilerFlareReporter.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@
1313
import java.util.Map;
1414
import java.util.zip.ZipOutputStream;
1515

16-
public final class ProfilerFlare implements TracerFlare.Reporter {
17-
private static final ProfilerFlare INSTANCE = new ProfilerFlare();
16+
public final class ProfilerFlareReporter implements TracerFlare.Reporter {
17+
private static final ProfilerFlareReporter INSTANCE = new ProfilerFlareReporter();
1818
private static Exception profilerInitializationException;
1919

2020
public static void register() {

dd-java-agent/agent-profiling/src/main/java/com/datadog/profiling/agent/ProfilingAgent.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ public static synchronized boolean run(final boolean earlyStart, Instrumentation
9595

9696
// Register the profiler flare before we start the profiling system, but early during the
9797
// profiler lifecycle
98-
ProfilerFlare.register();
98+
ProfilerFlareReporter.register();
9999

100100
boolean startForceFirst =
101101
Platform.isNativeImage()
@@ -176,11 +176,11 @@ public static synchronized boolean run(final boolean earlyStart, Instrumentation
176176
// a detailed telemetry message has been sent from the attempts to enable the controllers
177177
// -----------------------------------------------------------------------------------------
178178
// but we do want to report this within the profiler flare
179-
ProfilerFlare.reportInitializationException(e);
179+
ProfilerFlareReporter.reportInitializationException(e);
180180
} catch (final ConfigurationException e) {
181181
log.warn("Failed to initialize profiling agent! {}", e.getMessage());
182182
log.debug(SEND_TELEMETRY, "Failed to initialize profiling agent!", e);
183-
ProfilerFlare.reportInitializationException(e);
183+
ProfilerFlareReporter.reportInitializationException(e);
184184
}
185185
}
186186
return false;

dd-smoke-tests/tracer-flare/src/test/groovy/datadog/smoketest/TracerFlareSmokeTest.groovy

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,9 @@ import java.util.zip.ZipInputStream
2323
*/
2424
class TracerFlareSmokeTest extends AbstractSmokeTest {
2525

26-
// Time in seconds after which flare is triggered
27-
private static final int FLARE_TRIGGER_SECONDS = 15
26+
// Time in seconds after which flare is triggered.
27+
// We delay the profiler start on Oracle JDK 8, so increase the wait time to at least 20s.
28+
private static final int FLARE_TRIGGER_SECONDS = 20
2829
// Number of processes to run in parallel for testing
2930
private static final int NUMBER_OF_PROCESSES = 2
3031

0 commit comments

Comments
 (0)