Add remote path settings to RemoteStoreSettings class

[ashking94]

Description

This is a follow up PR from #13155 (comment). In this PR, we have moved 2 settings - cluster.remote_store.index.path.type and cluster.remote_store.index.path.hash_algorithm to the RemoteStoreSettings class.

Check List

• [ ] New functionality includes testing.
  • All tests pass
• [ ] New functionality has been documented.
  • [ ] New functionality has javadoc added
• Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
• Commits are signed per the DCO using --signoff
• [ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)
• [ ] Public documentation issue/PR created

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 result for ed21977: SUCCESS

[sachinpkale]

Changes look good. Branch has multiple conflicts with main, most probably due to recent revert of: #13155

[ashking94]

@sachinpkale adding the reverted changes back as part of #13251. Once that is merged back, we should be good with the conflict part.

[ashking94]

Detect breaking changes is failing due to refactoring of some methods that are present in only main branch as of now and yet to be released.

***! MODIFIED CLASS: PUBLIC org.opensearch.indices.IndicesService  (not serializable)
	===  CLASS FILE FORMAT VERSION: 55.0 <- 55.0
	---! REMOVED FIELD: PUBLIC(-) STATIC(-) FINAL(-) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm> CLUSTER_REMOTE_STORE_PATH_HASH_ALGORITHM_SETTING
	---! REMOVED FIELD: PUBLIC(-) STATIC(-) FINAL(-) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathType> CLUSTER_REMOTE_STORE_PATH_TYPE_SETTING
***  MODIFIED CLASS: PUBLIC org.opensearch.indices.RemoteStoreSettings  (not serializable)
	===  CLASS FILE FORMAT VERSION: 55.0 <- 55.0
	+++  NEW FIELD: PUBLIC(+) STATIC(+) FINAL(+) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm> CLUSTER_REMOTE_STORE_PATH_HASH_ALGORITHM_SETTING
	+++  NEW FIELD: PUBLIC(+) STATIC(+) FINAL(+) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathType> CLUSTER_REMOTE_STORE_PATH_TYPE_SETTING
	+++  NEW METHOD: PUBLIC(+) org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm getPathHashAlgorithm()
	+++  NEW METHOD: PUBLIC(+) org.opensearch.index.remote.RemoteStoreEnums$PathType getPathType()

We should ignore the "Detect breaking failure" changes here. cc @sachinpkale

[github-actions]

❕ Gradle check result for 61a0532: UNSTABLE

• TEST FAILURES:

      1 org.opensearch.remotestore.RemoteStoreStatsIT.testNonZeroPrimaryStatsOnNewlyCreatedIndexWithZeroDocs
      1 org.opensearch.remotestore.RemoteStoreStatsIT.testDownloadStatsCorrectnessSinglePrimaryMultipleReplicaShards

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.45%. Comparing base (b15cb0c) to head (61a0532).
Report is 186 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff              @@
##               main   #13225      +/-   ##
============================================
+ Coverage     71.42%   71.45%   +0.03%     
- Complexity    59978    60679     +701     
============================================
  Files          4985     5040      +55     
  Lines        282275   285456    +3181     
  Branches      40946    41338     +392     
============================================
+ Hits         201603   203975    +2372     
- Misses        63999    64662     +663     
- Partials      16673    16819     +146     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ashking94]

❕ Gradle check result for 61a0532: UNSTABLE

• TEST FAILURES:

      1 org.opensearch.remotestore.RemoteStoreStatsIT.testNonZeroPrimaryStatsOnNewlyCreatedIndexWithZeroDocs
      1 org.opensearch.remotestore.RemoteStoreStatsIT.testDownloadStatsCorrectnessSinglePrimaryMultipleReplicaShards

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Flaky tests - #10983 and #10152

[sachinpkale]

Detect breaking changes is failing due to refactoring of some methods that are present in only main branch as of now and yet to be released.

***! MODIFIED CLASS: PUBLIC org.opensearch.indices.IndicesService  (not serializable)
	===  CLASS FILE FORMAT VERSION: 55.0 <- 55.0
	---! REMOVED FIELD: PUBLIC(-) STATIC(-) FINAL(-) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm> CLUSTER_REMOTE_STORE_PATH_HASH_ALGORITHM_SETTING
	---! REMOVED FIELD: PUBLIC(-) STATIC(-) FINAL(-) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathType> CLUSTER_REMOTE_STORE_PATH_TYPE_SETTING
***  MODIFIED CLASS: PUBLIC org.opensearch.indices.RemoteStoreSettings  (not serializable)
	===  CLASS FILE FORMAT VERSION: 55.0 <- 55.0
	+++  NEW FIELD: PUBLIC(+) STATIC(+) FINAL(+) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm> CLUSTER_REMOTE_STORE_PATH_HASH_ALGORITHM_SETTING
	+++  NEW FIELD: PUBLIC(+) STATIC(+) FINAL(+) org.opensearch.common.settings.Setting<org.opensearch.index.remote.RemoteStoreEnums$PathType> CLUSTER_REMOTE_STORE_PATH_TYPE_SETTING
	+++  NEW METHOD: PUBLIC(+) org.opensearch.index.remote.RemoteStoreEnums$PathHashAlgorithm getPathHashAlgorithm()
	+++  NEW METHOD: PUBLIC(+) org.opensearch.index.remote.RemoteStoreEnums$PathType getPathType()

We should ignore the "Detect breaking failure" changes here. cc @sachinpkale

Thanks for adding these details.