Skip to content

Add Diffusion to third-party software list.#1450

Merged
brian-brazil merged 2 commits intoprometheus:masterfrom
diffusiondata:master
Oct 5, 2019
Merged

Add Diffusion to third-party software list.#1450
brian-brazil merged 2 commits intoprometheus:masterfrom
diffusiondata:master

Conversation

@philipa
Copy link
Contributor

@philipa philipa commented Oct 5, 2019

@brian-brazil

Diffusion is a commercial software product from Push Technology.

Signed-off-by: Philip Aston <philipa@mail.com>
@brian-brazil
Copy link
Contributor

That URL isn't right, and I think https://developers.pushtechnology.com/hc/en-us/articles/360034215494-How-To-Use-Metric-Collectors would be more useful.

You don't appear to actually document how to get the metrics. I presume "gateway" doesn't mean the pushgateway, I can imagine that confusing some of your users so would suggest clarifying.

Signed-off-by: Philip Aston <philipa@mail.com>
@philipa
Copy link
Contributor Author

philipa commented Oct 5, 2019

Thanks Brian; yes I agree. The blog you found is about general configuration of Diffusion's metric collection and not specific to Prometheus.

I've updated the URL to point to the relevant page in our manual.

@brian-brazil brian-brazil merged commit dbfecea into prometheus:master Oct 5, 2019
@brian-brazil
Copy link
Contributor

Thanks!

diffusion_sessions_peak sounds like a gauge rather than a counter. Counters should also generally end in _total (except if part of a Summary/Histogram), and _count should be for that series in a Summary/Histogram.

@philipa
Copy link
Contributor Author

philipa commented Oct 5, 2019

diffusion_sessions_peak can only go up. It is the highest number of sessions the server has ever hosted (it is the maximum value of diffusion_sessions_open). There is a separate diffusion_sessions_total metric

@brian-brazil
Copy link
Contributor

That's still a Gauge. You care about the absolute value, not how fast it is going up.

@philipa
Copy link
Contributor Author

philipa commented Oct 5, 2019

Since _peak is monotonically increasing, isn't a counter appropriate? What benefit would using a gauge bring?

@brian-brazil
Copy link
Contributor

Counters always go up, but that doesn't mean that everything that always goes up is a counter. Counters are not meant to be usable without using rate() first, whereas diffusion_sessions_peak is usable as-is.

@SuperQ
Copy link
Member

SuperQ commented Oct 5, 2019

@philipa Even tho it's monotonically increasing, it's not counting anything.

@philipa
Copy link
Contributor Author

philipa commented Oct 5, 2019

OK, schooled! I'll get this fixed. Thanks for your patience.

eightnoneone pushed a commit to eightnoneone/docs that referenced this pull request Apr 20, 2020
* Add Diffusion to third-party software list.

Signed-off-by: Philip Aston <philipa@mail.com>
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