Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scripts not permitted to use staticMethod org.tap4j.consumer.TapConsumerFactory makeTap13YamlConsumer #16649

Closed
pshipton opened this issue Feb 2, 2023 · 6 comments
Assignees

Comments

@pshipton
Copy link
Member

pshipton commented Feb 2, 2023

https://openj9-jenkins.osuosl.org/job/Test_openjdk17_j9_sanity.functional_x86-64_linux_Nightly/403

org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use staticMethod org.tap4j.consumer.TapConsumerFactory makeTap13YamlConsumer
	at org.jenkinsci.plugins.scriptsecurity.sandbox.whitelists.StaticWhitelist.rejectStaticMethod(StaticWhitelist.java:243)
	at org.jenkinsci.plugins.scriptsecurity.sandbox.groovy.SandboxInterceptor.onStaticCall(SandboxInterceptor.java:189)
	at org.kohsuke.groovy.sandbox.impl.Checker$2.call(Checker.java:192)
	at org.kohsuke.groovy.sandbox.impl.Checker.checkedStaticCall(Checker.java:196)
	at org.kohsuke.groovy.sandbox.impl.Checker.checkedCall(Checker.java:103)
	at com.cloudbees.groovy.cps.sandbox.SandboxInvoker.methodCall(SandboxInvoker.java:17)
	at Script1.addFailedTestsGrinderLink(Script1.groovy:1023)
	at Script1.run_parallel_tests(Script1.groovy:1172)

https://openj9-jenkins.osuosl.org/job/Test_openjdk17_j9_sanity.functional_x86-64_linux_Nightly_testList_1/388/console

02:23:26  Exception: org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use staticMethod org.tap4j.consumer.TapConsumerFactory makeTap13YamlConsumer

@AdamBrousseau seeing multiple failures like above. A child job failed a test, it seems jenkins needs an additional permission to handle it now?
@llxia fyi

@llxia
Copy link
Contributor

llxia commented Feb 2, 2023

makeTap13YamlConsumer is newly added to the test pipeline adoptium/aqa-tests#4284 . I should have tested it on both openj9 and internal Jenkins. I was too excited about the PR.
I have approved the makeTap13YamlConsumer in internal Jenkins. For the permission on openj9 Jenkins, do we need to open an Eclipse ticket?

@AdamBrousseau
Copy link
Contributor

Approved

@AdamBrousseau AdamBrousseau self-assigned this Feb 2, 2023
@pshipton
Copy link
Member Author

pshipton commented Feb 2, 2023

This just exposed the next problem
15:12:28 Exception: org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use method org.tap4j.model.TapElement getDiagnostic

@pshipton pshipton reopened this Feb 2, 2023
@AdamBrousseau
Copy link
Contributor

Should be 1 more

@pshipton
Copy link
Member Author

pshipton commented Feb 2, 2023

17:24:35 Exception: org.jenkinsci.plugins.scriptsecurity.sandbox.RejectedAccessException: Scripts not permitted to use staticMethod org.codehaus.groovy.runtime.DefaultGroovyMethods toUnique java.lang.Object[]

@AdamBrousseau
Copy link
Contributor

Done

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants