Skip to content

Conversation

julian-elastic
Copy link
Contributor

@julian-elastic julian-elastic commented Sep 11, 2025

Reverts #134454

Original PR caused a test failure on main. I am not sure why it is not happening on the build machines, but it is definitely happening on multiple people's Macs. To reproduce ./gradlew -p x-pack/plugin/esql/ test
results in 100+ failures.

@julian-elastic julian-elastic requested a review from a team as a code owner September 11, 2025 16:53
@elasticsearchmachine elasticsearchmachine added needs:triage Requires assignment of a team area label v9.2.0 labels Sep 11, 2025
@julian-elastic julian-elastic added :Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests labels Sep 11, 2025
@elasticsearchmachine elasticsearchmachine removed the needs:triage Requires assignment of a team area label label Sep 11, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

@prdoyle
Copy link
Contributor

prdoyle commented Sep 11, 2025

Let me spend a moment trying to fix this. It was backported and I'd rather not have to backport all the revert PRs and then re-backport the eventual fix.

@prdoyle
Copy link
Contributor

prdoyle commented Sep 11, 2025

Stack trace:

 1> org.elasticsearch.entitlement.runtime.api.NotEntitledException: component [(server)], module [java.desktop], class [class java.awt.Toolkit], entitlement [file], operation [read], path [/Users/prdoyle/.accessibility.properties]
  1>    at org.elasticsearch.entitlement.runtime.policy.PolicyCheckerImpl.notEntitled(PolicyCheckerImpl.java:467) ~[elasticsearch-entitlement-9.2.0-SNAPSHOT.jar:9.2.0-SNAPSHOT]
  1>    at org.elasticsearch.entitlement.runtime.policy.PolicyCheckerImpl.checkFileRead(PolicyCheckerImpl.java:245) ~[elasticsearch-entitlement-9.2.0-SNAPSHOT.jar:9.2.0-SNAPSHOT]
  1>    at org.elasticsearch.entitlement.runtime.policy.PolicyCheckerImpl.checkFileRead(PolicyCheckerImpl.java:208) ~[elasticsearch-entitlement-9.2.0-SNAPSHOT.jar:9.2.0-SNAPSHOT]
  1>    at org.elasticsearch.entitlement.runtime.policy.PolicyCheckerImpl.checkFileRead(PolicyCheckerImpl.java:202) ~[elasticsearch-entitlement-9.2.0-SNAPSHOT.jar:9.2.0-SNAPSHOT]
  1>    at org.elasticsearch.entitlement.runtime.policy.ElasticsearchEntitlementChecker.check$java_io_FileInputStream$(ElasticsearchEntitlementChecker.java:1641) ~[elasticsearch-entitlement-9.2.0-SNAPSHOT.jar:9.2.0-SNAPSHOT]
  1>    at java.io.FileInputStream.<init>(FileInputStream.java) ~[?:?]
  1>    at java.awt.Toolkit.initAssistiveTechnologies(Toolkit.java:407) ~[?:?]
  1>    at java.awt.Toolkit.initStatic(Toolkit.java:1319) ~[?:?]
  1>    at java.awt.Toolkit.<clinit>(Toolkit.java:1299) ~[?:?]
  1>    at java.awt.Font.<clinit>(Font.java:285) ~[?:?]
  1>    at net.nextencia.rrdiagram.grammar.rrdiagram.RRDiagramToSVG.<init>(RRDiagramToSVG.java:33) ~[rrdiagram-0.9.4.jar:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.RailRoadDiagram.toSvg(RailRoadDiagram.java:155) ~[test/:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.RailRoadDiagram.functionSignature(RailRoadDiagram.java:49) ~[test/:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.DocsV3Support.buildFunctionSignatureSvg(DocsV3Support.java:1082) ~[test/:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.DocsV3Support$FunctionDocsSupport.renderSignature(DocsV3Support.java:592) ~[test/:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.DocsV3Support.renderDocs(DocsV3Support.java:129) ~[test/:?]
  1>    at org.elasticsearch.xpack.esql.expression.function.AbstractFunctionTestCase.renderDocs(AbstractFunctionTestCase.java:985) ~[test/:?]

@breskeby
Copy link
Contributor

A buildscan showing it passing and one failing would be great to figure out what's going on here

@julian-elastic
Copy link
Contributor Author

Closing this as revert is no longer needed. A proper fix will be checked in with #134578

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

Labels

:Core/Infra/Entitlements Entitlements infrastructure Team:Core/Infra Meta label for core/infra team >test Issues or PRs that are addressing/adding tests v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants