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

Use any index specified by .watches for Watcher (#39541) #39708

Merged

Conversation

gwbrown
Copy link
Contributor

@gwbrown gwbrown commented Mar 5, 2019

Backport of #39541


Previously, Watcher only attached its listener to indices that started
with the prefix .watches, which causes Watcher to silently fail to
schedule newly created Watches if the .watches alias is redirected to
an index that does not start with .watches.

Watcher now attaches the listener to all indices, so that Watcher can
respond to changes in which index has the .watches alias.

Also adjusts the tests to randomly use non-prefixed concrete indices for .watches and .triggered_watches.

Previously, Watcher only attached its listener to indices that started
with the prefix `.watches`, which causes Watcher to silently fail to
schedule newly created Watches if the `.watches` alias is redirected to
an index that does not start with `.watches`.

Watcher now attaches the listener to all indices, so that Watcher can
respond to changes in which index has the `.watches` alias.

Also adjusts the tests to randomly use non-prefixed concrete indices for .watches and .triggered_watches.
@gwbrown gwbrown merged commit eb288a6 into elastic:7.x Mar 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant