[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field)
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17682295#comment-17682295 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - The only failure is reported in CASSANDRA-18144 Committed to trunk, thanks To https://github.com/apache/cassandra.git [9ec90d0e69..6c90950489|https://github.com/apache/cassandra/commit/6c90950489a42a38a6ddfa45f79c84a110dffbe7] trunk -> trunk > Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field) > > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field)
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17682286#comment-17682286 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Well, at the end a bunch of commits ended on Friday while I was doing "final" testing Rebase and last pre-commit run only with J8 for sanity check: [https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2201/workflows/12e1be71-d6d1-49be-8c3e-7e471bfac92a] > Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field) > > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field)
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681457#comment-17681457 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - And while discussing some CircleCI problem I realized it is good to run also the [in-jvm upgade tests|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2199/workflows/70f4a964-3427-4110-9403-43aedbf16566] > Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field) > > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field)
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681373#comment-17681373 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Thank you for the review! Last rebase and CircleCI sanity check J11 run [here|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2199/workflows/87115397-1e81-4eaa-9277-500bcb0a1032] (to save resources only unit tests, in-jvm and simulator as they are to be affected by this change) On successful run and if you do not want more tests run I will commit. (the new patches in trunk were doc oriented from what I saw so it is really sanity check run for me) > Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field) > > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field)
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17681336#comment-17681336 ] Michael Semb Wever commented on CASSANDRA-18181: +1 > Fix tests post JDK-8210522 (rewrite reflection of "modifiers" field) > > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680854#comment-17680854 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - The build finished successfully. Submitting the patch for review: ||Patch||CircleCI||Jenkins||CircleCI|| |[trunk|https://github.com/ekaterinadimitrova2/cassandra/commit/a06dbb6b8f90d5b92dfa6ebffd276a218fae57fe]|[simulator tests, still not running n Jenkns|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2185/workflows/5370326f-a9b9-472a-a226-675b24201f3c]|[first try|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2220/], [the aborted built successfully rerun|https://ci-cassandra.apache.org/job/Cassandra-devbranch-artifacts/]|in-jvm and unit tests(J17 test workflow, non-related failures expected) - [J11|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2182/workflows/9ca49487-937a-4c72-bc78-4fbefe23db76], [J17|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2182/workflows/e3dc630b-b7a4-4f5b-8f29-489bf43ad90f]| [~dcapwell], [~benedict] anyone of you up for review, we've already discussed this briefly in CASSANDRA-17178? > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680809#comment-17680809 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Ops, totally misunderstood. I just reloaded it [here|https://ci-cassandra.apache.org/job/Cassandra-devbranch-artifacts/], thanks! > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680807#comment-17680807 ] Michael Semb Wever commented on CASSANDRA-18181: I did not initiate the matrix reloaded rerun. Just explaining how it's done. > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680804#comment-17680804 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Thanks, [~mck] {quote}[https://ci-cassandra.apache.org/job/Cassandra-devbranch-artifacts/2157/] {quote} Is that the right link? I can see the yesterday aborted build, but I am not sure where is the newly submitted one? > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680662#comment-17680662 ] Michael Semb Wever commented on CASSANDRA-18181: bq. Does anyone know how can I rerun only the aborted build in favor of saving time and resources? "Matrix Reloaded" on https://ci-cassandra.apache.org/job/Cassandra-devbranch-artifacts/2157/ (it won't change the result of the parent pipeline job though) > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680643#comment-17680643 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Thanks, I will open a ticket and ping Anthony there. Does anyone know how can I rerun only the aborted build in favor of saving time and resources? > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680618#comment-17680618 ] Brandon Williams commented on CASSANDRA-18181: -- Most of the time in the script is spent in create_adoc, which is serially forking nodetool help for every command to gather it. > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680612#comment-17680612 ] Stefan Miklosovic commented on CASSANDRA-18181: --- Btw while I was wokring on CASSANDRA-18183 I noticed that this script does not work either if build.dir and build.dir.lib are set to something else but default. Probably not worth to fix it here, just saying. > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680610#comment-17680610 ] Michael Semb Wever commented on CASSANDRA-18181: This happens a lot, the {{python3 ./scripts/gen-nodetool-docs.py}} is slw. [~anthonygrasso]? > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680440#comment-17680440 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Weird, one of the builds now got aborted with: {code:java} 15:09:48 gen-asciidoc: 15:09:48 [exec] python3 ./scripts/gen-nodetool-docs.py 15:14:48 Build timed out (after 5 minutes). Marking the build as aborted. 15:14:48 [exec] Terminated 15:14:48 [exec] Makefile:23: recipe for target 'gen-asciidoc' failed 15:14:48 [exec] make: *** [gen-asciidoc] Terminated 15:14:48 [exec] Result: 2 15:14:48 15:14:48 gen-doc: 15:14:48 15:14:48 sources-jar: 15:14:48 Build was aborted 15:14:48 SSH: Current build result is [ABORTED], not going to run. 15:14:48 [PostBuildScript] - [INFO] Executing post build scripts. 15:14:48 [cassandra] $ /bin/sh -xe /home/jenkins/tmp/jenkins16069333297066315187.sh {code} This has nothing to do with this patch... > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17680323#comment-17680323 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - New Jenkins run [here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2220/] It seems that CI was not in a good shape the other day so better to rebase and run it again. On successful completion I will ping people for review > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17679650#comment-17679650 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Interesting, I still see issues related to the Python support tickets in Jenkins, but if I run DTests in CircleCI with the same branches as Jenkins - they finished [fine|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2187/workflows/0b2d1d8b-c2c2-48da-9345-d25c01f39e5a] I will investigate further tomorrow. > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17679489#comment-17679489 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - I just started the simulator tests in CircleCI as they are not added yet to Jenkins (CASSANDRA-18003) - they finished successfully [here|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2185/workflows/5370326f-a9b9-472a-a226-675b24201f3c] > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17679481#comment-17679481 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - Wall of failures in DTests, probably it didn't download the updated DTest trunk branch Just pushed it again [here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2213/] > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17679342#comment-17679342 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - As part of another ticket there were new images pushed and CI was in a broken state for a bit until the second image was also pushed. This broke the Jenkins CI build I had. I just restarted the build [here|https://ci-cassandra.apache.org/view/patches/job/Cassandra-devbranch/2212/] > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
[jira] [Commented] (CASSANDRA-18181) Fix tests post JDK-8210522
[ https://issues.apache.org/jira/browse/CASSANDRA-18181?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17679225#comment-17679225 ] Ekaterina Dimitrova commented on CASSANDRA-18181: - h5. This affects Java distributed tests, Instance.startup and some other tests: * ConnectionTest * ProxyHandlerConnectionsTest * ConnectionBurnTestTest * FramingTest * MessageSerializationPropertyTest * MessageTest * OutboundConnectionsTest * ProxyHandlerConnectionsTest * DriverBurnTests * SimpleClientBurnTest * MessagePayloadTest The patch is posted [here|https://github.com/ekaterinadimitrova2/cassandra/commit/a06dbb6b8f90d5b92dfa6ebffd276a218fae57fe] The solution used is similar to what other projects did, for example HBase. It was already discussed in CASSANDRA-17178 Pushed Jenkins CI run (to cover J8+J11), the job is in the queue as there are already 2 jobs running in Jenkins dev. I will check back and post a link later today to the trunk J8+J11 Jenkins CI run but I am fairly confident the run will be fine as I ran some of the tests locally and I was running this patch also with JDK17 for some time, too. Here is the run with J17 branch [J11|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2182/workflows/9ca49487-937a-4c72-bc78-4fbefe23db76] and [J17|https://app.circleci.com/pipelines/github/ekaterinadimitrova2/cassandra/2182/workflows/e3dc630b-b7a4-4f5b-8f29-489bf43ad90f] there are certain failures but those are not related to what we address here. While this patch is not needed for J8+J11 It is a preparation for when we switch to J11+J17. Trying to push in everything we can until CASSANDRA-17281 and a few other things are disentangled > Fix tests post JDK-8210522 > -- > > Key: CASSANDRA-18181 > URL: https://issues.apache.org/jira/browse/CASSANDRA-18181 > Project: Cassandra > Issue Type: Bug > Components: CI >Reporter: Ekaterina Dimitrova >Assignee: Ekaterina Dimitrova >Priority: Normal > Fix For: 4.x > > > > From JDK-8210522: > {code:java} > Core reflection has a filtering mechanism to hide security and integrity > sensitive fields and methods from Class getXXXField(s) and getXXXMethod(s). > The filtering mechanism has been used for several releases to hide security > sensitive fields such as System.security and Class.classLoader. > This CSR proposes to extend the filters to hide fields from a number of > highly security sensitive classes in java.lang.reflect and java.lang.invoke. > {code} > We are using at a few places in our tests > {code:java} > Field.class.getDeclaredField("modifiers");{code} > This breaks as expected when tests are run with JDK17, example: > > {code:java} > java.lang.RuntimeException: java.lang.NoSuchFieldException: modifiers > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:79) > at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native > Method) > at > java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) > at > java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) > at java.base/java.lang.reflect.Method.invoke(Method.java:568) > at > org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:50) > at > org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12) > at > org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:47) > at > org.junit.internal.runners.statements.RunBefores.evaluate(RunBefores.java:24) > at > org.junit.internal.runners.statements.RunAfters.evaluate(RunAfters.java:27) > at org.junit.runners.ParentRunner.run(ParentRunner.java:363) at > org.junit.runner.JUnitCore.run(JUnitCore.java:137) > at > com.intellij.junit4.JUnit4IdeaTestRunner.startRunnerWithArgs(JUnit4IdeaTestRunner.java:69) > at > com.intellij.rt.junit.IdeaTestRunner$Repeater.startRunnerWithArgs(IdeaTestRunner.java:33) > at > com.intellij.rt.junit.JUnitStarter.prepareStreamsAndStart(JUnitStarter.java:221) > at com.intellij.rt.junit.JUnitStarter.main(JUnitStarter.java:54) > Caused by: java.lang.NoSuchFieldException: modifiers at > java.base/java.lang.Class.getDeclaredField(Class.java:2610) > at > org.apache.cassandra.transport.MessagePayloadTest.makeCqlQueryHandlerAccessible(MessagePayloadTest.java:70) > > ... 15 more{code} > -- This message was sent by Atlassian Jira (v8.20.10#820010) - To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org