8341310: Test TestJcmdWithSideCar.java should skip ACCESS_TMP_VIA_PROC_ROOT (after JDK-8327114)

Reviewed-by: kevinw
This commit is contained in:
Severin Gehwolf 2024-10-03 12:55:54 +00:00
parent 7d524d7e37
commit 21f8ccf4a9

View File

@ -42,8 +42,8 @@ import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.nio.file.Paths;
import java.util.Arrays;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.EnumSet;
import java.util.List;
@ -52,7 +52,9 @@ import java.util.concurrent.TimeUnit;
import java.util.function.Consumer;
import java.util.regex.Pattern;
import java.util.stream.Collectors;
import jdk.test.lib.Container;
import jdk.test.lib.Platform;
import jdk.test.lib.Utils;
import jdk.test.lib.containers.docker.Common;
import jdk.test.lib.containers.docker.DockerRunOptions;
@ -108,6 +110,11 @@ public class TestJcmdWithSideCar {
mainContainer.waitForMainMethodStart(TIME_TO_WAIT_FOR_MAIN_METHOD_START);
for (AttachStrategy attachStrategy : EnumSet.allOf(AttachStrategy.class)) {
if (attachStrategy == AttachStrategy.ACCESS_TMP_VIA_PROC_ROOT &&
elevated && !Platform.isRoot()) {
// Elevated attach via proc/root not yet supported.
continue;
}
long mainProcPid = testCase01(attachStrategy, elevated);
// Excluding the test case below until JDK-8228850 is fixed