IGNITE-17617 Fixed NPE in the snapshot restore status command on client nodes and non-persistent cluster. (#10234)

(cherry picked from commit c511252)

Author: NSAmelchev

modules/control-utility/src/test/java/org/apache/ignite/util/GridCommandHandlerTest.java

@@ -3475,6 +3475,16 @@ public void testSnapshotRestoreCancelAndStatus() throws Exception {
         assertNull(ig.cache(DEFAULT_CACHE_NAME));
     }
 
+    /** @throws Exception If fails. */
+    @Test
+    public void testSnapshotStatusInMemory() throws Exception {
+        persistenceEnable(false);
+
+        startGrid();
+
+        checkSnapshotStatus(false, false, null);
+    }
+
     /** @throws Exception If fails. */
     @Test
     public void testSnapshotStatus() throws Exception {
@@ -3483,6 +3493,8 @@ public void testSnapshotStatus() throws Exception {
 
         IgniteEx srv = startGrids(3);
 
+        startClientGrid("client");
+
         srv.cluster().state(ACTIVE);
 
         createCacheAndPreload(srv, keysCnt);
@@ -3528,7 +3540,9 @@ public void testSnapshotStatus() throws Exception {
      * @param expName Expected snapshot name.
      */
     private void checkSnapshotStatus(boolean isCreating, boolean isRestoring, String expName) throws Exception {
-        assertTrue(waitForCondition(() -> G.allGrids().stream().allMatch(
+        Collection<Ignite> srvs = F.view(G.allGrids(), n -> !n.cluster().localNode().isLocal());
+
+        assertTrue(waitForCondition(() -> srvs.stream().allMatch(
                 ignite -> {
                     IgniteSnapshotManager mgr = ((IgniteEx)ignite).context().cache().context().snapshotMgr();
 
@@ -3557,7 +3571,7 @@ private void checkSnapshotStatus(boolean isCreating, boolean isRestoring, String
 
         assertContains(log, out, "Snapshot name: " + expName);
 
-        G.allGrids().forEach(srv -> assertContains(log, out, srv.cluster().localNode().id().toString()));
+        srvs.forEach(srv -> assertContains(log, out, srv.cluster().localNode().id().toString()));
     }
 
     /** @throws Exception If failed. */

modules/core/src/main/java/org/apache/ignite/internal/visor/snapshot/VisorSnapshotStatusTask.java

@@ -32,6 +32,7 @@
 import org.apache.ignite.internal.processors.task.GridInternal;
 import org.apache.ignite.internal.util.typedef.F;
 import org.apache.ignite.internal.util.typedef.T2;
+import org.apache.ignite.internal.util.typedef.internal.CU;
 import org.apache.ignite.internal.visor.VisorJob;
 import org.apache.ignite.internal.visor.VisorMultiNodeTask;
 import org.apache.ignite.internal.visor.VisorTaskArgument;
@@ -58,7 +59,7 @@ public class VisorSnapshotStatusTask extends VisorMultiNodeTask<Void, VisorSnaps
 
     /** {@inheritDoc} */
     @Override protected Collection<UUID> jobNodes(VisorTaskArgument<Void> arg) {
-        return F.nodeIds(ignite.cluster().nodes());
+        return F.nodeIds(ignite.cluster().forServers().nodes());
     }
 
     /** {@inheritDoc} */
@@ -106,6 +107,9 @@ protected VisorSnapshotStatusJob(@Nullable Void arg, boolean debug) {
 
         /** {@inheritDoc} */
         @Override protected SnapshotStatus run(@Nullable Void arg) throws IgniteException {
+            if (!CU.isPersistenceEnabled(ignite.context().config()))
+                return null;
+
             IgniteSnapshotManager snpMgr = ignite.context().cache().context().snapshotMgr();
 
             SnapshotOperationRequest req = snpMgr.currentCreateRequest();