Skip to content

Conversation

@skumawat2025
Copy link
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
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?

@github-actions
Copy link
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

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
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 <[email protected]>
# This is the commit message opensearch-project#2:

got query and agg working with new abstraction poc

Signed-off-by: TJ Neuenfeldt <[email protected]>
# This is the commit message opensearch-project#3:

trying to refactor with a composite pb not done yet

Signed-off-by: TJ Neuenfeldt <[email protected]>
# This is the commit message opensearch-project#4:

need to fix SearchProfileShardResult

Signed-off-by: TJ Neuenfeldt <[email protected]>

SearchProfileShardResult still needs to be fixed but tests pass

Signed-off-by: TJ Neuenfeldt <[email protected]>

SearchProfileShardResult still needs to be fixed but tests pass

Signed-off-by: TJ Neuenfeldt <[email protected]>
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