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

Add memcached module with stats metricset #3693

Merged
merged 2 commits into from Mar 23, 2017

Conversation

Projects
None yet
2 participants
@ruflin
Collaborator

ruflin commented Feb 28, 2017

A new module "memcached" was added which gathers metrics from https://memcached.org/. The stats command is used in the stats metricset to gather all infos about the service.

@ruflin

This comment has been minimized.

Collaborator

ruflin commented Mar 3, 2017

jenkins, retest it

1 similar comment
@ruflin

This comment has been minimized.

Collaborator

ruflin commented Mar 3, 2017

jenkins, retest it

@@ -98,6 +98,7 @@ https://github.com/elastic/beats/compare/v5.1.1...master[Check the HEAD diff]
- The Docker, Kafka, and Prometheus modules are now Beta, instead of experimental. {pull}3525[3525]
- The HAProxy module is now GA, instead of experimental. {pull}3525[3525]
- Add the ability to collect the environment variables from system processes. {pull}3337[3337]
- Add memcached module with stats metricset.

This comment has been minimized.

@tsg

tsg Mar 14, 2017

Collaborator

Link to PR please :)

This comment has been minimized.

@ruflin

ruflin Mar 15, 2017

Collaborator

added

[float]
=== memcached.stats.uptime

This comment has been minimized.

@tsg

tsg Mar 14, 2017

Collaborator

Is this seconds? Should we have it as uptime.secs then?

This comment has been minimized.

@ruflin

ruflin Mar 15, 2017

Collaborator

yes, changed it to uptime.sec.

func New(base mb.BaseMetricSet) (mb.MetricSet, error) {
logp.Warn("BETA: The docker container metricset is beta")

This comment has been minimized.

@tsg

tsg Mar 14, 2017

Collaborator

s/docker/memcached

@tsg tsg merged commit 06d6cf1 into elastic:master Mar 23, 2017

4 checks passed

CLA Commit author has signed the CLA
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
default Build finished.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment