Skip to content

Unregister rest and transport action handlers for HotToWarmTiering#18247

Merged
gbbafna merged 1 commit intoopensearch-project:mainfrom
skumawat2025:migration-api
May 9, 2025
Merged

Unregister rest and transport action handlers for HotToWarmTiering#18247
gbbafna merged 1 commit intoopensearch-project:mainfrom
skumawat2025:migration-api

Conversation

@skumawat2025
Copy link
Copy Markdown
Contributor

@skumawat2025 skumawat2025 commented May 9, 2025

Description

Unregister rest and transport action handlers for HotToWarmTIering. We will add later when this feature's complete implementation is finalised.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

@skumawat2025 skumawat2025 changed the title Remove rest and transport action handlers for HotToWarmTIering Unregister rest and transport action handlers for HotToWarmTIering May 9, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 9, 2025

❌ Gradle check result for 704a37e: 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: Sandeep Kumawat <skumwt@amazon.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented May 9, 2025

❕ Gradle check result for 1fd5bad: 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.

@codecov
Copy link
Copy Markdown

codecov bot commented May 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 72.67%. Comparing base (c677397) to head (1fd5bad).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main   #18247   +/-   ##
=========================================
  Coverage     72.66%   72.67%           
- Complexity    67404    67442   +38     
=========================================
  Files          5483     5483           
  Lines        310776   310772    -4     
  Branches      45176    45174    -2     
=========================================
+ Hits         225814   225840   +26     
+ Misses        66614    66575   -39     
- Partials      18348    18357    +9     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@skumawat2025 skumawat2025 marked this pull request as ready for review May 9, 2025 14:51
@skumawat2025 skumawat2025 requested a review from a team as a code owner May 9, 2025 14:51
@gbbafna gbbafna changed the title Unregister rest and transport action handlers for HotToWarmTIering Unregister rest and transport action handlers for HotToWarmTiering May 9, 2025
@gbbafna gbbafna merged commit 2c1b2aa into opensearch-project:main May 9, 2025
34 checks passed
rayshrey pushed a commit to rayshrey/OpenSearch that referenced this pull request Jun 26, 2025
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
…ensearch-project#18247)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>
neuenfeldttj pushed a commit to neuenfeldttj/OpenSearch that referenced this pull request Jun 26, 2025
# This is the 1st commit message:

Unregister rest and transport action handlers for HotToWarmTiering(opensearch-project#18247)

Signed-off-by: Sandeep Kumawat <skumwt@amazon.com>
# This is the commit message opensearch-project#2:

got query and agg working with new abstraction poc

Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>
# This is the commit message opensearch-project#3:

trying to refactor with a composite pb not done yet

Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>
# This is the commit message opensearch-project#4:

need to fix SearchProfileShardResult

Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>

SearchProfileShardResult still needs to be fixed but tests pass

Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>

SearchProfileShardResult still needs to be fixed but tests pass

Signed-off-by: TJ Neuenfeldt <tjneu@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants