Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
8255787: Tag container tests that use cGroups with cgroups keyword
Backport-of: 4df8abc
  • Loading branch information
GoeLin committed May 10, 2022
1 parent 60588db commit c95c796
Show file tree
Hide file tree
Showing 15 changed files with 20 additions and 5 deletions.
3 changes: 2 additions & 1 deletion test/hotspot/jtreg/TEST.ROOT
Expand Up @@ -28,7 +28,8 @@

# The list of keywords supported in this test suite
# randomness: test uses randomness, test cases differ from run to run
keys=cte_test jcmd nmt regression gc stress metaspace headful intermittent randomness
# cgroups: test uses cgroups
keys=cte_test jcmd nmt regression gc stress metaspace headful intermittent randomness cgroups

groups=TEST.groups TEST.quick-groups

Expand Down
Expand Up @@ -23,6 +23,7 @@

/*
* @test CgroupSubsystemFactory
* @key cgroups
* @requires os.family == "linux"
* @library /testlibrary /test/lib
* @build sun.hotspot.WhiteBox
Expand Down
1 change: 1 addition & 0 deletions test/hotspot/jtreg/containers/cgroup/PlainRead.java
Expand Up @@ -23,6 +23,7 @@

/*
* @test PlainRead
* @key cgroups
* @requires os.family == "linux"
* @library /testlibrary /test/lib
* @build sun.hotspot.WhiteBox
Expand Down
1 change: 1 addition & 0 deletions test/hotspot/jtreg/containers/docker/TestCPUAwareness.java
Expand Up @@ -24,6 +24,7 @@

/*
* @test
* @key cgroups
* @summary Test JVM's CPU resource awareness when running inside docker container
* @requires docker.support
* @library /test/lib
Expand Down
1 change: 1 addition & 0 deletions test/hotspot/jtreg/containers/docker/TestCPUSets.java
Expand Up @@ -24,6 +24,7 @@

/*
* @test
* @key cgroups
* @summary Test JVM's awareness of cpu sets (cpus and mems)
* @requires docker.support
* @requires (os.arch != "s390x")
Expand Down
1 change: 1 addition & 0 deletions test/hotspot/jtreg/containers/docker/TestJFREvents.java
Expand Up @@ -24,6 +24,7 @@

/*
* @test
* @key cgroups
* @summary Ensure that certain JFR events return correct results for resource values
* when run inside Docker container, such as available CPU and memory.
* Also make sure that PIDs are based on value provided by container,
Expand Down
Expand Up @@ -24,6 +24,7 @@

/*
* @test
* @key cgroups
* @summary Test JVM's memory resource awareness when running inside docker container
* @requires docker.support
* @library /test/lib
Expand Down
3 changes: 2 additions & 1 deletion test/jdk/TEST.ROOT
Expand Up @@ -13,8 +13,9 @@
# run. Tests that are not headful are "headless".
# A test flagged with key "printer" requires a printer to succeed, else
# throws a PrinterException or the like.
# A test flagged with cgroups uses cgroups.

keys=2d dnd headful i18n intermittent printer randomness jfr
keys=2d dnd headful i18n intermittent printer randomness jfr cgroups

# Tests that must run in othervm mode
othervm.dirs=java/awt java/beans javax/accessibility javax/imageio javax/sound javax/swing javax/print \
Expand Down
3 changes: 2 additions & 1 deletion test/jdk/jdk/internal/platform/cgroup/TestCgroupMetrics.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2018, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2018, 2020, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,6 +23,7 @@

/*
* @test
* @key cgroups
* @requires os.family == "linux"
* @modules java.base/jdk.internal.platform
* @library /test/lib
Expand Down
Expand Up @@ -43,6 +43,7 @@

/*
* @test
* @key cgroups
* @requires os.family == "linux"
* @modules java.base/jdk.internal.platform
* @library /test/lib
Expand Down
Expand Up @@ -50,6 +50,7 @@

/*
* @test
* @key cgroups
* @requires os.family == "linux"
* @modules java.base/jdk.internal.platform
* java.base/jdk.internal.platform.cgroupv1
Expand Down
Expand Up @@ -32,6 +32,7 @@

/*
* @test
* @key cgroups
* @summary Test JDK Metrics class when running inside docker container
* @requires docker.support
* @library /test/lib
Expand Down
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2018, 2020, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -30,6 +30,7 @@

/*
* @test
* @key cgroups
* @summary Test JDK Metrics class when running inside docker container
* @requires docker.support
* @library /test/lib
Expand Down
Expand Up @@ -23,6 +23,7 @@

/*
* @test
* @key cgroups
* @bug 8242480
* @requires docker.support
* @library /test/lib
Expand Down
3 changes: 2 additions & 1 deletion test/jdk/jdk/internal/platform/docker/TestSystemMetrics.java
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2018, 2019, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2018, 2020, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand All @@ -23,6 +23,7 @@

/*
* @test
* @key cgroups
* @summary Test JDK Metrics class when running inside docker container
* @requires docker.support
* @library /test/lib
Expand Down

1 comment on commit c95c796

@openjdk-notifier
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please sign in to comment.