Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Maintenance]Enable indices.breaker.total.use_real_memory setting for integTest cluster #2395

Merged
merged 6 commits into from
Jan 17, 2025

Conversation

Rajrahane
Copy link
Contributor

@Rajrahane Rajrahane commented Jan 15, 2025

Description

Enables indices.breaker.total.use_real_memory setting for integTest Cluster vai build.gradle for catching heap CB in local integration testing.
Enabled the setting in -

  1. testClusters.integTest configuration
  2. task integTest

Testing

Manually tested the setting was enabled using logging statements on local using
logger.lifecycle("Breaker setting: ${getBreakerSetting()}")

Related Issues

Resolves #2114

Check List

  • New functionality includes testing.
  • New functionality has been documented.
  • API changes companion pull request created.
  • Commits are signed per the DCO using --signoff.
  • 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.

0ctopus13prime and others added 4 commits January 14, 2025 16:22
Signed-off-by: Dooyong Kim <[email protected]>
Co-authored-by: Dooyong Kim <[email protected]>
Signed-off-by: Rajvaibhav Rahane <[email protected]>
Signed-off-by: owenhalpert <[email protected]>
Signed-off-by: Rajvaibhav Rahane <[email protected]>
…uster vai build.gradle for catching heap CB in local integration testing.

Signed-off-by: Rajvaibhav Rahane <[email protected]>
Signed-off-by: Rajvaibhav Rahane <[email protected]>
@navneet1v
Copy link
Collaborator

@Rajrahane the CIs are failing please check

@Rajrahane
Copy link
Contributor Author

Update
All 20 workflows passed after rerun.

@jmazanec15 jmazanec15 merged commit eecb45c into opensearch-project:main Jan 17, 2025
44 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jan 17, 2025
… integTest cluster (#2395)

Enable indices.breaker.total.use_real_memory setting for integTest Cluster  vai build.gradle for catching heap CB in local integration testing.

Signed-off-by: Rajvaibhav Rahane <[email protected]>
(cherry picked from commit eecb45c)
naveentatikonda pushed a commit to naveentatikonda/k-NN that referenced this pull request Jan 22, 2025
… integTest cluster (opensearch-project#2395)

Enable indices.breaker.total.use_real_memory setting for integTest Cluster  vai build.gradle for catching heap CB in local integration testing.

Signed-off-by: Rajvaibhav Rahane <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants