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

[elasticsearch] Get information about pending_tasks #1507

Merged
merged 2 commits into from May 15, 2015

Conversation

bjarocki
Copy link
Contributor

@bjarocki bjarocki commented Apr 3, 2015

Adding new elasticsearch metrics

  • pending_task_total - total number of pending tasks
  • pending_tasks_priority_high - number of high priority pending tasks
  • pending_tasks_priority_urgent - number of urgent priority pending tasks

@yannmh yannmh added this to the 5.4.0 milestone Apr 3, 2015
@yannmh
Copy link
Member

yannmh commented Apr 3, 2015

Great, thanks @bjarocki.

Let's aim it for the 5.4 agent release !

@yannmh yannmh changed the title Get information about Elasticsearch pending_tasks [elastic] Get information about Elasticsearch pending_tasks Apr 3, 2015
@yannmh yannmh changed the title [elastic] Get information about Elasticsearch pending_tasks [elasticsearch] Get information about Elasticsearch pending_tasks Apr 3, 2015
@yannmh yannmh added the checks label Apr 3, 2015
@yannmh yannmh changed the title [elasticsearch] Get information about Elasticsearch pending_tasks [elasticsearch] Get information about pending_tasks Apr 3, 2015
@LeoCavaille LeoCavaille force-pushed the bjarocki/elasticsearch-pending_tasks branch 2 times, most recently from f95b55d to d664196 Compare May 14, 2015 22:41
bjarocki and others added 2 commits May 15, 2015 11:40
Adding new elasticsearch metrics
[X] pending_task_total - total number of pending tasks
[X] pending_tasks_priority_high - number of high priority pending tasks
[X] pending_tasks_priority_urgent - number of urgent priority pending tasks
`NODES_URL` does not exist anymore thus adjust the code to use the new
structures, although this code should not be used anymore as it's an
end-of-life version of elasticsearch.
Also removed a useless case in the elasticsearch testsuite
@LeoCavaille LeoCavaille force-pushed the bjarocki/elasticsearch-pending_tasks branch from d664196 to bc3bc2c Compare May 15, 2015 15:40
@LeoCavaille LeoCavaille merged commit bc3bc2c into master May 15, 2015
@LeoCavaille LeoCavaille deleted the bjarocki/elasticsearch-pending_tasks branch May 15, 2015 15:41
degemer added a commit that referenced this pull request Jun 26, 2015
In order to do this:
- `elastic`: add tests for new metrics from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jun 26, 2015
In order to do this:
- `elastic`: add tests for new metrics from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jun 26, 2015
In order to do this:
- `elastic`: add tests for new metrics from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jun 29, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jun 29, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jul 9, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jul 9, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jul 9, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
degemer added a commit that referenced this pull request Jul 9, 2015
In order to do this:
- `elastic`: add tests for new metrics and sc from
  #1507
- `http_check`: add a missing service check test (`http.can_connect` is also
  collected when testing `http.ssl_cert`)
- `disk`: 100% coverage for the 5.4.2 check (full compatibility with old check)
- `lighttpd`: add two missing metrics for 100% coverage
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.

None yet

3 participants