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

Create 50_remove_and_add_same_alias.yaml #14611

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

theahura
Copy link

@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>
Copy link
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
Collaborator

❌ 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
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
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