Introduce a new search node role to hold search only shards #17620
Conversation
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
vinaykpud
requested review from
anasalkouz,
andrross,
ashking94,
bugmakerrrrrr,
Bukhtawar,
CEHENKLE,
cwperks,
dblock,
dbwiddis,
gbbafna,
jainankitk,
kotwanikunal,
linuxpi,
mch2,
msfroh,
nknize,
owaiskazi19,
reta,
Rishikesh1159,
sachinpkale,
saratvemulapalli,
shwetathareja,
sohami and
VachaShah
as code owners
|
❌ Gradle check result for 75838a5: 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/cluster/node/DiscoveryNodeRole.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/opensearch/cluster/node/DiscoveryNodeRole.java
Outdated
Show resolved
Hide resolved
server/src/test/java/org/opensearch/node/NodeRoleSettingsTests.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
|
❌ Gradle check result for 4717b9b: 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? |
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
vinaykpud
force-pushed
the
rw/add-search-role
branch
from
199f5ac to
972d73a
Compare
|
❌ Gradle check result for 972d73a: 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? |
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
|
❌ Gradle check result for add23cf: 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? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17620 +/- ##
============================================
- Coverage 72.48% 72.38% -0.11%
+ Complexity 65771 65688 -83
============================================
Files 5311 5311
Lines 304973 305011 +38
Branches 44229 44232 +3
============================================
- Hits 221045 220767 -278
- Misses 65830 66111 +281
- Partials 18098 18133 +35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
vinaykpud
changed the title
…ch-project#17620) * Introduce a new search node role Signed-off-by: Vinay Krishna Pudyodu <[email protected]> * Add changelog Signed-off-by: Vinay Krishna Pudyodu <[email protected]> * fixed PR comments Signed-off-by: Vinay Krishna Pudyodu <[email protected]> * rename search to warm in FsProbe test Signed-off-by: Vinay Krishna Pudyodu <[email protected]> * fixed ClusterStatsIT tests Signed-off-by: Vinay Krishna Pudyodu <[email protected]> * added ClusterStatsIT test for search node role Signed-off-by: Vinay Krishna Pudyodu <[email protected]> --------- Signed-off-by: Vinay Krishna Pudyodu <[email protected]>
Description
Introducing a new node role "search". This will be used for node to hold search replica
Related Issues
Related #17422
Check List
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.