Multi-project for TransportExlainLifecycleAction

This converts this class to support multi-project.

Author: PeteGillinElastic

x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportExplainLifecycleAction.java

@@ -18,7 +18,7 @@
 import org.elasticsearch.cluster.metadata.IndexMetadata;
 import org.elasticsearch.cluster.metadata.IndexNameExpressionResolver;
 import org.elasticsearch.cluster.metadata.LifecycleExecutionState;
-import org.elasticsearch.cluster.metadata.Metadata;
+import org.elasticsearch.cluster.metadata.ProjectMetadata;
 import org.elasticsearch.cluster.project.ProjectResolver;
 import org.elasticsearch.cluster.service.ClusterService;
 import org.elasticsearch.common.Strings;
@@ -56,6 +56,7 @@ public class TransportExplainLifecycleAction extends TransportLocalClusterStateA
 
     private final NamedXContentRegistry xContentRegistry;
     private final IndexNameExpressionResolver indexNameExpressionResolver;
+    private final ProjectResolver projectResolver;
 
     /**
      * NB prior to 9.0 this was a TransportMasterNodeReadAction so for BwC it must be registered with the TransportService until
@@ -82,6 +83,7 @@ public TransportExplainLifecycleAction(
         );
         this.xContentRegistry = xContentRegistry;
         this.indexNameExpressionResolver = indexNameExpressionResolver;
+        this.projectResolver = projectResolver;
 
         transportService.registerRequestHandler(
             actionName,
@@ -97,7 +99,11 @@ public TransportExplainLifecycleAction(
     @Override
     protected ClusterBlockException checkBlock(ExplainLifecycleRequest request, ClusterState state) {
         return state.blocks()
-            .indicesBlockedException(ClusterBlockLevel.METADATA_READ, indexNameExpressionResolver.concreteIndexNames(state, request));
+            .indicesBlockedException(
+                projectResolver.getProjectId(),
+                ClusterBlockLevel.METADATA_READ,
+                indexNameExpressionResolver.concreteIndexNames(state, request)
+            );
     }
 
     @Override
@@ -117,7 +123,7 @@ protected void localClusterStateOperation(
             try {
                 indexResponse = getIndexLifecycleExplainResponse(
                     index,
-                    state.metadata(),
+                    projectResolver.getProjectMetadata(state.metadata()),
                     request.onlyErrors(),
                     request.onlyManaged(),
                     xContentRegistry,
@@ -140,13 +146,12 @@ protected void localClusterStateOperation(
     @Nullable
     static IndexLifecycleExplainResponse getIndexLifecycleExplainResponse(
         String indexName,
-        Metadata metadata,
+        ProjectMetadata project,
         boolean onlyErrors,
         boolean onlyManaged,
         NamedXContentRegistry xContentRegistry,
         boolean rolloverOnlyIfHasDocuments
     ) throws IOException {
-        final var project = metadata.getProject();
         IndexMetadata indexMetadata = project.index(indexName);
         Settings idxSettings = indexMetadata.getSettings();
         LifecycleExecutionState lifecycleState = indexMetadata.getLifecycleExecutionState();

x-pack/plugin/ilm/src/test/java/org/elasticsearch/xpack/ilm/action/TransportExplainLifecycleActionTests.java

@@ -9,7 +9,7 @@
 
 import org.elasticsearch.cluster.metadata.IndexMetadata;
 import org.elasticsearch.cluster.metadata.LifecycleExecutionState;
-import org.elasticsearch.cluster.metadata.Metadata;
+import org.elasticsearch.cluster.metadata.ProjectMetadata;
 import org.elasticsearch.index.IndexVersion;
 import org.elasticsearch.test.ESTestCase;
 import org.elasticsearch.xcontent.NamedXContentRegistry;
@@ -78,14 +78,14 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
                 .numberOfReplicas(randomIntBetween(0, 5))
                 .putCustom(ILM_CUSTOM_METADATA_KEY, errorStepState.build().asMap())
                 .build();
-            Metadata metadata = Metadata.builder()
+            ProjectMetadata project = ProjectMetadata.builder(randomProjectIdOrDefault())
                 .put(indexMetadata, true)
                 .putCustom(IndexLifecycleMetadata.TYPE, createIndexLifecycleMetadata())
                 .build();
 
             IndexLifecycleExplainResponse onlyErrorsResponse = getIndexLifecycleExplainResponse(
                 indexInErrorStep,
-                metadata,
+                project,
                 true,
                 true,
                 REGISTRY,
@@ -111,14 +111,14 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
                 .numberOfReplicas(randomIntBetween(0, 5))
                 .putCustom(ILM_CUSTOM_METADATA_KEY, checkRolloverReadyStepState.build().asMap())
                 .build();
-            Metadata metadata = Metadata.builder()
+            ProjectMetadata project = ProjectMetadata.builder(randomProjectIdOrDefault())
                 .put(indexMetadata, true)
                 .putCustom(IndexLifecycleMetadata.TYPE, createIndexLifecycleMetadata())
                 .build();
 
             IndexLifecycleExplainResponse onlyErrorsResponse = getIndexLifecycleExplainResponse(
                 indexInCheckRolloverStep,
-                metadata,
+                project,
                 true,
                 true,
                 REGISTRY,
@@ -128,7 +128,7 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
 
             IndexLifecycleExplainResponse allManagedResponse = getIndexLifecycleExplainResponse(
                 indexInCheckRolloverStep,
-                metadata,
+                project,
                 false,
                 true,
                 REGISTRY,
@@ -150,14 +150,14 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
                 .numberOfShards(randomIntBetween(1, 5))
                 .numberOfReplicas(randomIntBetween(0, 5))
                 .build();
-            Metadata metadata = Metadata.builder()
+            ProjectMetadata project = ProjectMetadata.builder(randomProjectIdOrDefault())
                 .put(indexMetadata, true)
                 .putCustom(IndexLifecycleMetadata.TYPE, createIndexLifecycleMetadata())
                 .build();
 
             IndexLifecycleExplainResponse onlyErrorsResponse = getIndexLifecycleExplainResponse(
                 indexWithMissingPolicy,
-                metadata,
+                project,
                 true,
                 true,
                 REGISTRY,
@@ -175,14 +175,14 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
                 .numberOfShards(randomIntBetween(1, 5))
                 .numberOfReplicas(randomIntBetween(0, 5))
                 .build();
-            Metadata metadata = Metadata.builder()
+            ProjectMetadata project = ProjectMetadata.builder(randomProjectIdOrDefault())
                 .put(indexMetadata, true)
                 .putCustom(IndexLifecycleMetadata.TYPE, createIndexLifecycleMetadata())
                 .build();
 
             IndexLifecycleExplainResponse onlyManaged = getIndexLifecycleExplainResponse(
                 "index",
-                metadata,
+                project,
                 false,
                 true,
                 REGISTRY,
@@ -205,14 +205,14 @@ public void testGetIndexLifecycleExplainResponse() throws IOException {
                 .numberOfReplicas(randomIntBetween(0, 5))
                 .putCustom(ILM_CUSTOM_METADATA_KEY, checkRolloverReadyStepState.build().asMap())
                 .build();
-            Metadata metadata = Metadata.builder()
+            ProjectMetadata project = ProjectMetadata.builder(randomProjectIdOrDefault())
                 .put(indexMetadata, true)
                 .putCustom(IndexLifecycleMetadata.TYPE, createIndexLifecycleMetadata())
                 .build();
 
             IndexLifecycleExplainResponse response = getIndexLifecycleExplainResponse(
                 indexInCheckRolloverStep,
-                metadata,
+                project,
                 false,
                 true,
                 REGISTRY,