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

[Monitoring] Remove improper usage #37099

Conversation

chrisronline
Copy link
Contributor

Resolves #37098

There isn't an easy way to test this, but I noticed this issue when I didn't see the CCR tab in monitoring and after digging, realized it was because the terminate_after was causing the response to indicate it wasn't available, but after removing that part of the query, it worked fine (as it should have)

@chrisronline chrisronline self-assigned this May 24, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/stack-monitoring

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@chrisronline chrisronline requested review from ycombinator and pickypg and removed request for ycombinator May 24, 2019 17:23
Copy link
Member

@pickypg pickypg left a comment

Choose a reason for hiding this comment

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

LGTM

@chrisronline chrisronline merged commit 1a2ec83 into elastic:master May 24, 2019
@chrisronline chrisronline deleted the monitoring/remove_terminate_after_usage branch May 24, 2019 19:36
chrisronline added a commit to chrisronline/kibana that referenced this pull request May 24, 2019
chrisronline added a commit to chrisronline/kibana that referenced this pull request May 24, 2019
chrisronline added a commit to chrisronline/kibana that referenced this pull request May 24, 2019
chrisronline added a commit that referenced this pull request May 24, 2019
chrisronline added a commit that referenced this pull request May 24, 2019
@chrisronline
Copy link
Contributor Author

chrisronline commented May 24, 2019

Backport:

7.x: 924bcfd
7.2: 6f29f2c
7.1: 892c4b7
6.8: eb7cd29

chrisronline added a commit that referenced this pull request May 28, 2019
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.

[Monitoring] Audit/remove usage of terminate_after
3 participants