[Remote Store] Add RemoteStoreSettings class to handle remote store r… #12962
Conversation
…elated settings (opensearch-project#12838) * Add RemoteStoreSettings class to handle remote store related settings --------- Signed-off-by: Sachin Kale <kalsac@amazon.com> Co-authored-by: Sachin Kale <kalsac@amazon.com>
sachinpkale
requested review from
abbashus,
adnapibar,
anasalkouz,
andrross,
Bukhtawar,
CEHENKLE,
dblock,
dbwiddis,
dreamer-89,
gbbafna,
kotwanikunal,
mch2,
msfroh,
nknize,
owaiskazi19,
reta,
Rishikesh1159,
ryanbogan,
saratvemulapalli,
shwetathareja,
sohami,
tlfeng and
VachaShah
as code owners
Signed-off-by: Sachin Kale <kalsac@amazon.com>
Compatibility status:Checks if related components are compatible with change d710640 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/security.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/performance-analyzer.git] |
|
❌ Gradle check result for b5737fa: FAILURE 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? |
server/src/main/java/org/opensearch/common/settings/ClusterSettings.java
Show resolved
Hide resolved
|
❌ Gradle check result for f2930b0: FAILURE 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? |
|
Needed to create manual backport PR due to conflicts in ClusterSettings class. |
Signed-off-by: Sachin Kale <kalsac@amazon.com>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #12962 +/- ##
============================================
- Coverage 71.28% 71.12% -0.16%
- Complexity 60145 60389 +244
============================================
Files 4957 4987 +30
Lines 282799 284469 +1670
Branches 41409 41595 +186
============================================
+ Hits 201591 202329 +738
- Misses 64189 65070 +881
- Partials 17019 17070 +51 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Sachin Kale <kalsac@amazon.com>
|
❕ Gradle check result for d710640: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Backport 8ad0dc0 from #12838.