-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Move SnapshotsRecoveryPlannerService to its own x-pack plugin #79637
Conversation
ee373e2
to
a0bb0e1
Compare
@henningandersen I'm leaving this as a draft since I think there's a problem in the task wiring for the qa tests, but it would be nice to get this reviewed. The changes are really similar to #78928 the change-set is large since automatic formatting is applied to new projects and therefore it has changed everything :D. I copied the relevant tests from |
a0bb0e1
to
a2e9138
Compare
@elasticsearchmachine test this please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, provided CI is happy, also when running the s3/gc/azure tests.
...ork/src/main/java/org/elasticsearch/indices/recovery/AbstractIndexRecoveryIntegTestCase.java
Show resolved
Hide resolved
throw new IllegalStateException("A single RecoveryPlannerPlugin was expected but got: " + recoveryPlannerPlugins); | ||
} | ||
|
||
final ShardSnapshotsService shardSnapshotsService = new ShardSnapshotsService(client, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe we could also move the creation of this service and the service into the x-pack module as a follow-up? No need to do it now.
No description provided.