specify _all as target in ClearCacheModal (#1020)

When none of the indices are selected, ClearCacheModal makes call to `//_cache/clear` path. Here, the intention is to clear the cache for all indices, let's make it explicit by passing `_all` as target. New call: `/_all/_cache/clear` Signed-off-by: Rohit Ashiwal <[email protected]> (cherry picked from commit 134f23f) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Apr 9, 2024 · cfa35d8 · cfa35d8
1 parent a268539
commit cfa35d8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/public/containers/ClearCacheModal/ClearCacheModal.tsx b/public/containers/ClearCacheModal/ClearCacheModal.tsx
@@ -113,7 +113,7 @@ export default function ClearCacheModal(props: ClearCacheModalProps) {
         const result = await services.commonService.apiCaller({
           endpoint: "indices.clearCache",
           data: {
-            index: unBlockedItems.join(","),
+            index: unBlockedItems.join(",") || "_all",
           },
         });
         if (result && result.ok) {

diff --git a/public/pages/Indices/containers/IndicesActions/IndicesActions.test.tsx b/public/pages/Indices/containers/IndicesActions/IndicesActions.test.tsx
@@ -185,7 +185,7 @@ describe("<IndicesActions /> spec", () => {
       expect(browserServicesMock.commonService.apiCaller).toHaveBeenCalledWith({
         endpoint: "indices.clearCache",
         data: {
-          index: "",
+          index: "_all",
         },
       });
       expect(coreServicesMock.notifications.toasts.addSuccess).toHaveBeenCalledTimes(1);
@@ -238,7 +238,7 @@ describe("<IndicesActions /> spec", () => {
       expect(browserServicesMock.commonService.apiCaller).toHaveBeenCalledWith({
         endpoint: "indices.clearCache",
         data: {
-          index: "",
+          index: "_all",
         },
       });
       expect(coreServicesMock.notifications.toasts.addError).toHaveBeenCalledTimes(1);