Skip to content

Conversation

@prdoyle
Copy link
Contributor

@prdoyle prdoyle commented Jan 28, 2025

Adds five entitlements:

  1. create_thread. Added to Thread constructors. (This one is debatable, partly because we're not sure we want to instrument Thread creation at all, and partly because even if we do, the constructors don't create the thread: that's done by Thread.start().)
  2. interrupt_thread
  3. set_thread_context_class_loader. This is the only one of the thread properties that's set by a plugin (namely repository-hdfs).
  4. set_thread_property. This is for all other sensitive thread properties, like name and priority.
  5. (Never entitled) "Disruptive thread action". Used for ThreadGroup.interrupt().

See ES-10358.

@prdoyle prdoyle added >non-issue :Core/Infra/Core Core issues without another label auto-backport Automatically create backport pull requests when merged v9.0.0 v8.18.0 labels Jan 28, 2025
@prdoyle prdoyle self-assigned this Jan 28, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Core/Infra Meta label for core/infra team label Jan 28, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@prdoyle prdoyle force-pushed the entitlements-thread-properties branch from b639b0f to 0134a93 Compare February 6, 2025 14:04
@prdoyle prdoyle marked this pull request as draft February 6, 2025 14:06
SetThreadContextClassLoaderEntitlement.class,
SetThreadPropertyEntitlement.class,
WriteAllSystemPropertiesEntitlement.class,
WriteSystemPropertiesEntitlement.class
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I sorted these.

@prdoyle prdoyle force-pushed the entitlements-thread-properties branch from 9211e23 to db8131c Compare February 6, 2025 14:27
@prdoyle prdoyle force-pushed the entitlements-thread-properties branch from fcfb2d4 to 39fb3ce Compare February 6, 2025 21:55
@prdoyle prdoyle closed this Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged :Core/Infra/Core Core issues without another label >non-issue Team:Core/Infra Meta label for core/infra team v8.18.1 v8.19.0 v9.0.1 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants