Skip to content

Commit b4fe8b0

Browse files
authored
added delay after container stops to try to smooth out the intermittant errors with container communication (#652)
smoketest build passed... hopefully this helps future builds
1 parent e071587 commit b4fe8b0

File tree

1 file changed

+2
-2
lines changed
  • test/smoke/framework/testCore/src/main/java/com/microsoft/applicationinsights/smoketest

1 file changed

+2
-2
lines changed

test/smoke/framework/testCore/src/main/java/com/microsoft/applicationinsights/smoketest/AiSmokeTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,6 @@ protected static void stopContainer(ContainerInfo info) throws Exception {
124124
//endregion
125125

126126
//region: application fields
127-
128-
129127
protected String targetUri;
130128
protected String httpMethod;
131129
protected boolean expectSomeTelemetry = true;
@@ -134,6 +132,7 @@ protected static void stopContainer(ContainerInfo info) throws Exception {
134132
//region: options
135133
public static final int APPLICATION_READY_TIMEOUT_SECONDS = 120;
136134
public static final int TELEMETRY_RECEIVE_TIMEOUT_SECONDS = 10;
135+
public static final int DELAY_AFTER_CONTAINER_STOP_MILLISECONDS = 1500;
137136
//endregion
138137

139138
private static final Properties testProps = new Properties();
@@ -390,6 +389,7 @@ public static void tearDownContainer(final String appServer, final String os, fi
390389
if (!docker.isContainerRunning(currentContainerInfo.getContainerId())) { // for good measure
391390
currentContainerInfo = null;
392391
}
392+
TimeUnit.MILLISECONDS.sleep(DELAY_AFTER_CONTAINER_STOP_MILLISECONDS);
393393
}
394394

395395
//region: test helper methods

0 commit comments

Comments
 (0)