Remove FailedToCommitClusterStateException Check (elastic#135846)

Removes a check on `FailedToCommitClusterStateException`
with a message "node closed" inside 1QueryableBuiltInRolesSynchronizer
.isExpectedFailure1 since this exception was changed to a
1NotMasterException1 inside elastic#135008

Relates to: ES-13061

Author: joshua-adams-1

x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/support/QueryableBuiltInRolesSynchronizer.java

@@ -20,7 +20,6 @@
 import org.elasticsearch.cluster.ClusterStateTaskListener;
 import org.elasticsearch.cluster.NotMasterException;
 import org.elasticsearch.cluster.SimpleBatchedExecutor;
-import org.elasticsearch.cluster.coordination.FailedToCommitClusterStateException;
 import org.elasticsearch.cluster.metadata.IndexMetadata;
 import org.elasticsearch.cluster.metadata.Metadata;
 import org.elasticsearch.cluster.metadata.ProjectMetadata;
@@ -309,8 +308,7 @@ private static boolean isExpectedFailure(final Exception e) {
             || cause instanceof ResourceAlreadyExistsException
             || cause instanceof VersionConflictEngineException
             || cause instanceof DocumentMissingException
-            || cause instanceof FailedToMarkBuiltInRolesAsSyncedException
-            || (e instanceof FailedToCommitClusterStateException && "node closed".equals(cause.getMessage()));
+            || cause instanceof FailedToMarkBuiltInRolesAsSyncedException;
     }
 
     private boolean shouldSyncBuiltInRoles(final ClusterState state) {