Skip to content

Create 50_remove_and_add_same_alias.yaml#14611

Open
theahura wants to merge 1 commit intoopensearch-project:mainfrom
theahura:patch-1
Open

Create 50_remove_and_add_same_alias.yaml#14611
theahura wants to merge 1 commit intoopensearch-project:mainfrom
theahura:patch-1

Conversation

@theahura
Copy link
Copy Markdown

@theahura theahura commented Jul 1, 2024

Description

Adds a test to ensure that POST _aliases with [remove, add] actions on the same preexisting alias no-ops

Related Issues

Related to (but maybe does not resolve) #14503

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.

Signed-off-by: Amol Kapoor <theahura@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 1, 2024

❌ Gradle check result for ade8260: 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?

@gaobinlong
Copy link
Copy Markdown
Contributor

❌ Gradle check result for ade8260: 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?

From the test failure we can see that the file suffix yaml is not supported, so please change it to yml, thanks!

Copy link
Copy Markdown
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@gaobinlong thanks for catching this, this file .yml extension should be consistent with others

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

This PR is stalled because it has been open for 30 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip-changelog stalled Issues that have stalled

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants