[crac] Integrated: 8346178: [CRaC] Failure in com/sun/management/DiagnosticCommandMBean/DcmdMBeanPermissionsTest.java

Timofei Pushkin duke at openjdk.org
Tue Dec 17 11:35:54 UTC 2024


On Fri, 13 Dec 2024 15:08:14 GMT, Timofei Pushkin <duke at openjdk.org> wrote:

> The test takes all dcmd commands and invokes each to check if a command exceeds permissions set up in the test.
> 
> This fix simply excludes `jdk.checkpoint` command from the test and keeps the rest of the test logic untouched. This approach was chosen because (1) checkpointing requires many permissions and it would be better to create a separate test for this if we want to "document" these permissions since the current test does not separate permission sets by commands, (2) the test would need additional adjustments to actually perform a checkpoint due to the CRaC specifics.

This pull request has now been integrated.

Changeset: 7d9f93ca
Author:    Anton Kozlov <akozlov at azul.com>
Committer: Radim Vansa <rvansa at openjdk.org>
URL:       https://git.openjdk.org/crac/commit/7d9f93ca937bfb89d1606e6cbf159a1edffc16c4
Stats:     3 lines in 1 file changed: 3 ins; 0 del; 0 mod

8346178: [CRaC] Failure in com/sun/management/DiagnosticCommandMBean/DcmdMBeanPermissionsTest.java

Reviewed-by: rvansa

-------------

PR: https://git.openjdk.org/crac/pull/161


More information about the crac-dev mailing list