[Segment Replication + Remote Store] Removing exception throw when remote store is used with system indices

[Rishikesh1159]

Description

This PR Removes exception throw when remote store is used with system indices, which was introduced in PR #8186. Instead when system index is used with remote store we just set remote store to false and use document replication. This is a temporary workaround until issue #8158 is fixed.

Related Issues

Resolves #8162

Check List

• New functionality includes testing.
  • All tests pass
• New functionality has been documented.
  • New functionality has javadoc added
• Commits are signed per the DCO using --signoff
• Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: FAILURE ❌
• URL: https://build.ci.opensearch.org/job/gradle-check/18241/
• CommitID: 285630d
  Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green.
  Is the failure a flaky test unrelated to your change?

[rishabh6788]

@Rishikesh1159 Would the index setting to disable remote-store on system indices override cluster setting to enable remote-store?

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: SUCCESS ✅
• URL: https://build.ci.opensearch.org/job/gradle-check/18248/
• CommitID: 7401c12

[codecov]

Codecov Report

Merging #8235 (01a6d43) into main (84dffac) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main    #8235      +/-   ##
============================================
+ Coverage     70.83%   70.85%   +0.01%     
- Complexity    56652    56698      +46     
============================================
  Files          4733     4733              
  Lines        268125   268127       +2     
  Branches      39283    39284       +1     
============================================
+ Hits         189932   189974      +42     
- Misses        62154    62168      +14     
+ Partials      16039    15985      -54     

Impacted Files	Coverage Δ
...h/cluster/metadata/MetadataCreateIndexService.java	79.75% <100.00%> (-0.11%)	⬇️

... and 449 files with indirect coverage changes

[Rishikesh1159]

@Rishikesh1159 Would the index setting to disable remote-store on system indices override cluster setting to enable remote-store?

@rishabh6788 yes, index settings will always override cluster level settings

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: SUCCESS ✅
• URL: https://build.ci.opensearch.org/job/gradle-check/18249/
• CommitID: be431e7

[github-actions]

Gradle Check (Jenkins) Run Completed with:

• RESULT: UNSTABLE ❕
• TEST FAILURES:

      1 org.opensearch.remotestore.RemoteStoreRefreshListenerIT.testRemoteRefreshRetryOnFailure
      1 org.opensearch.action.admin.cluster.node.tasks.ResourceAwareTasksTests.testBasicTaskResourceTracking

• URL: https://build.ci.opensearch.org/job/gradle-check/18269/
• CommitID: 01a6d43
  Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.