Skip to content

[Backport 2.x] Fix create or update alias API doesn't throw exception for unsupported parameters#14756

Merged
reta merged 2 commits into2.xfrom
backport/backport-14719-to-2.x
Jul 15, 2024
Merged

[Backport 2.x] Fix create or update alias API doesn't throw exception for unsupported parameters#14756
reta merged 2 commits into2.xfrom
backport/backport-14719-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport 29a3e2c from #14719.

…d parameters (#14719)

* Fix create or update alias API doesn't throw exception for unsupported parameters

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

* Update version check in yml test

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

* modify change log

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

---------

Signed-off-by: Gao Binlong <gbinlong@amazon.com>
(cherry picked from commit 29a3e2c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@github-actions
Copy link
Copy Markdown
Contributor

❌ Gradle check result for 79336bb: null

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
Copy Markdown
Contributor

❌ Gradle check result for 7b834f7: null

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
Copy Markdown
Contributor

✅ Gradle check result for 79336bb: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Jul 15, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 71.55%. Comparing base (fd3d162) to head (79336bb).

Files Patch % Lines
.../action/admin/indices/RestIndexPutAliasAction.java 14.28% 6 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #14756      +/-   ##
============================================
- Coverage     71.59%   71.55%   -0.05%     
+ Complexity    62535    62464      -71     
============================================
  Files          5106     5106              
  Lines        293547   293554       +7     
  Branches      42770    42772       +2     
============================================
- Hits         210174   210042     -132     
- Misses        65738    65902     +164     
+ Partials      17635    17610      -25     

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

kkewwei pushed a commit to kkewwei/OpenSearch that referenced this pull request Jul 24, 2024
… for unsupported parameters (opensearch-project#14756)

* Fix create or update alias API doesn't throw exception for unsupported parameters (opensearch-project#14719)

* Fix create or update alias API doesn't throw exception for unsupported parameters

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

* Update version check in yml test

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

* modify change log

Signed-off-by: Gao Binlong <gbinlong@amazon.com>

---------

Signed-off-by: Gao Binlong <gbinlong@amazon.com>
(cherry picked from commit 29a3e2c)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Update 10_basic.yml

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>

---------

Signed-off-by: Gao Binlong <gbinlong@amazon.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Andriy Redko <andriy.redko@aiven.io>
Signed-off-by: kkewwei <kkewwei@163.com>
@dbwiddis dbwiddis deleted the backport/backport-14719-to-2.x branch July 24, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant