Skip to content

[8.19] Split PolicyChecker from PolicyManager (#128004) #128641

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

Merged
merged 2 commits into from
May 30, 2025

Conversation

prdoyle
Copy link
Contributor

@prdoyle prdoyle commented May 29, 2025

Backport of #128004.

  • Split PolicyChecker from PolicyManager

  • Restore EntitlementCheckerUtils

  • [CI] Auto commit changes from spotless

@prdoyle prdoyle self-assigned this May 29, 2025
@prdoyle prdoyle added >refactoring backport auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) v8.19.0 :Core/Infra/Entitlements Entitlements infrastructure labels May 29, 2025
@prdoyle prdoyle marked this pull request as draft May 29, 2025 20:24
@prdoyle
Copy link
Contributor Author

prdoyle commented May 29, 2025

Putting into draft mode. I don't think this builds. There's some sort of semantic conflict here.

@prdoyle
Copy link
Contributor Author

prdoyle commented May 29, 2025

For some reason, PolicyManagerTests.java is different in 8.19 from 9.1.

@prdoyle prdoyle force-pushed the backport-128004 branch from b08520c to 7cd78c2 Compare May 29, 2025 21:19
@prdoyle
Copy link
Contributor Author

prdoyle commented May 29, 2025

Blocked on #128645.

prdoyle and others added 2 commits May 29, 2025 20:21
* Split PolicyChecker from PolicyManager

* Restore EntitlementCheckerUtils

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
@prdoyle prdoyle force-pushed the backport-128004 branch from e41fd46 to 99f24a3 Compare May 30, 2025 00:22
@prdoyle prdoyle marked this pull request as ready for review May 30, 2025 00:22
@elasticsearchmachine elasticsearchmachine merged commit 0a994c0 into elastic:8.19 May 30, 2025
15 checks passed
@prdoyle prdoyle deleted the backport-128004 branch May 30, 2025 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport :Core/Infra/Entitlements Entitlements infrastructure >refactoring v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants