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

Add overlay metrics. #112

Merged
merged 3 commits into from
Jul 8, 2019
Merged

Add overlay metrics. #112

merged 3 commits into from
Jul 8, 2019

Conversation

ichernetsky
Copy link
Contributor

@ichernetsky ichernetsky commented Jun 25, 2019

High-level description

Add overlay metrics to ease detection of common issues users of the overlay module encounter.

Changelog automation

DCOS_OSS-5321 Add overlay metrics.

@ichernetsky ichernetsky self-assigned this Jun 25, 2019
overlay/master_metrics.hpp Show resolved Hide resolved
overlay/master.cpp Show resolved Hide resolved
@ichernetsky
Copy link
Contributor Author

Team, could you please let me know if these metrics names are fine with you. What do you think of having overlay/master and overlay/slave prefixes? If you think this distinction makes sense, what do you think about renaming overlay/log/ensemble_size and overlay/log/recovered to overlay/master/log/ensemble_size and overlay/master/log/recovered respectively? The Telegraf Mesos plugin doesn't seem to collect them as of now.

@urbanserj
Copy link
Contributor

@ichernetsky I really like your idea! btw how do we convert mesos metric names, i.e. what names will be in prometheus?

@ichernetsky
Copy link
Contributor Author

@urbanserj, metrics names are used as is in the plugin itself, and AFAIU, the Telegraf's Prometheus output plugin replaces invalid (from the POV of Prometheus) characters with underscores. Please refer to https://github.com/dcos/telegraf/blob/1.9.4-dcos/plugins/inputs/mesos/mesos.go for further details.

@ichernetsky
Copy link
Contributor Author

The Telegraf plugin adjustments are here: dcos/telegraf#59
This one should (obviously) be merged before the PR against Telegraf repo.

@ichernetsky
Copy link
Contributor Author

@GoelDeepak, do I have your blessing to merge it?

@ichernetsky ichernetsky merged commit 694479c into master Jul 8, 2019
@ichernetsky ichernetsky deleted the inc/overlay-metrics branch July 8, 2019 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants