-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Prometheus support #477
Comments
It would be possible to create a sub command that consumes vegeta attack results (i.e. like vegeta report or vegeta encode) that is a prometheus exporter. |
ProposalWhat do you think about creating a new command category called "monitor" aside report, encode, dump? MonitorSo for monitoring with Prometheus one could do something like
After launching the tests, just point Prometheus scraper to http://host:8880/metrics or do something like
Prometheus Metrics
What do you think? |
That’s a good idea.
Sent via Superhuman iOS ( https://sprh.mn/?vip=tsenart@gmail.com )
…On Tue, Jun 30 2020 at 1:46 AM, Flavio Stutz < ***@***.*** > wrote:
--------
Proposal
--------
What do you think about creating a new command category called "monitor"
aside report, encode, dump?
Monitor
-------
So for monitoring with Prometheus one could do something like
GET http://www.google.com" | vegeta attack -duration=5m | vegeta monitor
-type='prometheus[8880,0.0.0.0]'
* "-type" may be omitted. defaults to 'prometheus'
* "8880" is the Prometheus exporter http port (may be omitted and defaults
to 8880)
* "0.0.0.0" is the bind host for the listener (may be omitted and defaults
to 0.0.0.0)
After launching the tests, just point Prometheus scraper to http://host:8880/metrics
or do something like
curl http://host:8880/metrics
Prometheus Metrics
------------------
* request_seconds_count{method, path, status}
* request_seconds_sum{method, path, status}
What do you think?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub (
#477 (comment) ) , or
unsubscribe (
https://github.com/notifications/unsubscribe-auth/AAAQPDZZWGP2IH7P6UECLR3RZERU5ANCNFSM4KVEPIMA
).
|
Alternatively we’d instrument attack directly.
Sent via Superhuman iOS ( https://sprh.mn/?vip=tsenart@gmail.com )
…On Tue, Jun 30 2020 at 8:31 AM, < ***@***.*** > wrote:
That’s a good idea.
Sent via Superhuman iOS ( ***@***.*** )
On Tue, Jun 30 2020 at 1:46 AM, Flavio Stutz < ***@***.*** >
wrote:
>
>
>
>
>
>
> --------
> Proposal
> --------
>
>
>
> What do you think about creating a new command category called "monitor"
> aside report, encode, dump?
>
>
>
> Monitor
> -------
>
>
>
> So for monitoring with Prometheus one could do something like
>
> GET http://www.google.com" | vegeta attack -duration=5m | vegeta monitor
> -type='prometheus[8880,0.0.0.0]'
>
>
> * "-type" may be omitted. defaults to 'prometheus'
> * "8880" is the Prometheus exporter http port (may be omitted and defaults
> to 8880)
> * "0.0.0.0" is the bind host for the listener (may be omitted and defaults
> to 0.0.0.0)
>
>
>
>
> After launching the tests, just point Prometheus scraper to http://host:8880/metrics
> or do something like
>
> curl http://host:8880/metrics
>
>
> Prometheus Metrics
> ------------------
>
>
>
> * request_seconds_count{method, path, status}
> * request_seconds_sum{method, path, status}
>
>
>
>
> What do you think?
>
>
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub (
> #477 (comment) ) , or
> unsubscribe (
> https://github.com/notifications/unsubscribe-auth/AAAQPDZZWGP2IH7P6UECLR3RZERU5ANCNFSM4KVEPIMA
> ).
>
>
>
|
Much better!
Instrumenting “attack” is great because it avoids parsing stdin contents and performance will be better for large scale distributed deployments.
…Sent from my iPhone
On 30 Jun 2020, at 03:32, Tomás Senart ***@***.***> wrote:
Alternatively we’d instrument attack directly.
Sent via Superhuman iOS ( ***@***.*** )
On Tue, Jun 30 2020 at 8:31 AM, < ***@***.*** > wrote:
>
> That’s a good idea.
>
> Sent via Superhuman iOS ( ***@***.*** )
>
>
>
> On Tue, Jun 30 2020 at 1:46 AM, Flavio Stutz < ***@***.*** >
> wrote:
>
>>
>>
>>
>>
>>
>>
>> --------
>> Proposal
>> --------
>>
>>
>>
>> What do you think about creating a new command category called "monitor"
>> aside report, encode, dump?
>>
>>
>>
>> Monitor
>> -------
>>
>>
>>
>> So for monitoring with Prometheus one could do something like
>>
>> GET http://www.google.com" | vegeta attack -duration=5m | vegeta monitor
>> -type='prometheus[8880,0.0.0.0]'
>>
>>
>> * "-type" may be omitted. defaults to 'prometheus'
>> * "8880" is the Prometheus exporter http port (may be omitted and defaults
>> to 8880)
>> * "0.0.0.0" is the bind host for the listener (may be omitted and defaults
>> to 0.0.0.0)
>>
>>
>>
>>
>> After launching the tests, just point Prometheus scraper to http://host:8880/metrics
>> or do something like
>>
>> curl http://host:8880/metrics
>>
>>
>> Prometheus Metrics
>> ------------------
>>
>>
>>
>> * request_seconds_count{method, path, status}
>> * request_seconds_sum{method, path, status}
>>
>>
>>
>>
>> What do you think?
>>
>>
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub (
>> #477 (comment) ) , or
>> unsubscribe (
>> https://github.com/notifications/unsubscribe-auth/AAAQPDZZWGP2IH7P6UECLR3RZERU5ANCNFSM4KVEPIMA
>> ).
>>
>>
>>
>
>
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
|
@flaviostutz: Contributions welcome! :) |
Sure! :)
…On Wed, Jul 1, 2020 at 6:12 AM Tomás Senart ***@***.***> wrote:
@flaviostutz <https://github.com/flaviostutz>: Contributions welcome! :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#477 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AB3N4XFQJ6DQGRNBYCMXLD3RZL4WXANCNFSM4KVEPIMA>
.
|
…viostutz/vegeta into issue/tsenart#477-prometheus
Do you have news about prom metrics? I am really looking forward! |
I implemented it and a PR is being analyzed. Soon we will have it! ;) |
Is there any update on the status of this PR? This feature would be really useful, thanks! |
Is there any update on the status of this PR? This is a very useful feature in our DevOps process. |
…senart#477-promlib Fixing small typos
Signed-off-by: Flávio Stutz <flaviostutz@gmail.com>
Signed-off-by: Flávio Stutz <flaviostutz@gmail.com>
🎉🎉🎉 |
Hi!! Can you please add a command example on the docs? I've been trying to make it work but simply cant. |
Hello, it is great to see metrics in prometheus, is it possible to do this?
jaggr is used to plot realtime graph, so I think it should possible to convert it to prometheus format.
Any thoughts?
The text was updated successfully, but these errors were encountered: