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

Cherry-pick #8551 to 6.x: Better error messaging for Elastic stacks' metricsets #8577

Merged
merged 1 commit into from
Oct 5, 2018

Conversation

ycombinator
Copy link
Contributor

Cherry-pick of PR #8551 to 6.x branch. Original message:

This PR makes error messages better and more consistent across all metricsets for Elastic stack products, including the xpack monitoring code paths:

  • elasticsearch/ccr
  • elasticsearch/cluster_stats
  • elasticsearch/index
  • elasticsearch/index_recovery
  • elasticsearch/index_summary
  • elasticsearch/ml_job
  • elasticsearch/node
  • elasticsearch/node_stats
  • elasticsearch/pending_tasks
  • elasticsearch/shard
  • kibana/status
  • kibana/stats
  • logstash/node
  • logstash/node_stats

* Better error messaging for elasticsearch/ccr metricset

* Better error messaging for elasticsearch/cluster_stats metricset

* Better error messaging for elasticsearch/index metricset

* Better error messaging for elasticsearch/index_recovery metricset

* Better error messaging for elasticsearch/index_summary metricset

* Better error messaging for elasticsearch/ml_job metricset

* Better error messaging for elasticsearch/node metricset

* Better error messaging for elasticsearch/node_stats metricset

* Better error messaging for elasticsearch/pending_tasks metricset

* Better error messaging for elasticsearch/shard metricset

* Better error messaging for kibana/stats metricset

* Better error messaging for kibana/status metricset

* Better error messaging for logstash/node metricset

* Better error messaging for logstash/node_stats metricset

* Cleaning up imports

(cherry picked from commit a164d7b)
@ycombinator
Copy link
Contributor Author

jenkins, test this

@ycombinator ycombinator merged commit 252e6cd into elastic:6.x Oct 5, 2018
@ycombinator ycombinator deleted the backport_8551_6.x branch October 5, 2018 06:11
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.

2 participants