Addressed PR comments

Signed-off-by: Bansi Kasundra <kasundra@amazon.com>

Author: kasundra07

server/src/internalClusterTest/java/org/opensearch/snapshots/DeleteSnapshotIT.java

@@ -8,6 +8,7 @@
 
 package org.opensearch.snapshots;
 
+import org.junit.Ignore;
 import org.opensearch.action.ActionFuture;
 import org.opensearch.action.admin.cluster.snapshots.create.CreateSnapshotResponse;
 import org.opensearch.client.Client;
@@ -139,6 +140,7 @@ public void testDeleteMultipleShallowCopySnapshotsCase1() throws Exception {
     // Deleting multiple shallow copy snapshots as part of single delete call with both partial and full copy snapshot present in the repo
     // And then deleting multiple full copy snapshots as part of single delete call with both partial and shallow copy snapshots present in
     // the repo
+    @Ignore("https://github.com/opensearch-project/OpenSearch/issues/8610")
     public void testDeleteMultipleShallowCopySnapshotsCase2() throws Exception {
         disableRepoConsistencyCheck("Remote store repository is being used in the test");
         FeatureFlagSetter.set(FeatureFlags.REMOTE_STORE);
@@ -227,6 +229,7 @@ public void testDeleteMultipleShallowCopySnapshotsCase2() throws Exception {
     }
 
     // Deleting subset of shallow and full copy snapshots as part of single delete call and then deleting all snapshots in the repo.
+    @Ignore("https://github.com/opensearch-project/OpenSearch/issues/8610")
     public void testDeleteMultipleShallowCopySnapshotsCase3() throws Exception {
         disableRepoConsistencyCheck("Remote store repository is being used in the test");
         FeatureFlagSetter.set(FeatureFlags.REMOTE_STORE);

server/src/main/java/org/opensearch/repositories/blobstore/BlobStoreRepository.java

@@ -1538,7 +1538,6 @@ private void executeOneStaleIndexDelete(
                 } catch (Exception e) {
                     assert false : e;
                     logger.warn(new ParameterizedMessage("[{}] Exception during single stale index delete", metadata.name()), e);
-                    listener.onFailure(e);
                 }
 
                 executeOneStaleIndexDelete(staleIndicesToDelete, remoteStoreLockManagerFactory, listener);

server/src/main/java/org/opensearch/snapshots/SnapshotsService.java

@@ -2236,6 +2236,7 @@ private void deleteSnapshotsFromRepository(
             // when the repository currently have the flag disabled and we try to delete the shallow snapshots taken prior to disabling
             // the flag. This can be improved by having the info whether there ever were any shallow snapshot present in this repository
             // or not in RepositoryData.
+            // SEE https://github.com/opensearch-project/OpenSearch/issues/8610
             final boolean cleanupRemoteStoreLockFiles = REMOTE_STORE_INDEX_SHALLOW_COPY.get(repository.getMetadata().settings());
             if (cleanupRemoteStoreLockFiles) {
                 repository.deleteSnapshotsAndReleaseLockFiles(