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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 30, 2025
Merged

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

merged 2 commits into from
May 30, 2025

Conversation

@prdoyle
Copy link
Contributor

@prdoyle prdoyle commented May 29, 2025 •
edited
Loading

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 prdoyle mentioned this pull request May 29, 2025
Merged
@prdoyle prdoyle force-pushed the backport-128004 branch from 7cd78c2 to e41fd46 Compare May 29, 2025 21:41
prdoyle and others added 2 commits May 29, 2025 20:21
@prdoyle
Split PolicyChecker from PolicyManager (elastic#128004)
0bcd3d7 
* Split PolicyChecker from PolicyManager

* Restore EntitlementCheckerUtils

* [CI] Auto commit changes from spotless

---------

Co-authored-by: elasticsearchmachine <[email protected]>
@prdoyle
PolicyChecker change in JavaXX entitlement checkers
99f24a3
@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

Reviewers

No reviews

Assignees

@prdoyle prdoyle

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@prdoyle @elasticsearchmachine