Just a reminder, this still needs to be commited.

Łukasz Mierzwa

2012/8/3 Łukasz Mierzwa <[email protected]>:
> Dnia czwartek, 24 maja 2012 15:00:36 Łukasz Mierzwa pisze:
>> With rev 2228 [1] carbon stats for workers can be skipped, it works fine but
>> using this option also disables total rss size, vsz size and avg response
>> time metrics that are calculated by iterating per worker stats.
>> I've made a quick patch, please take a look at it.
>>
>> [1] -
>> http://projects.unbit.it/uwsgi/changeset/ebde5e6b88cdc590e3b9c53cc2a65709117
>> d8acb
>
>
> It seems that I made a typo, I think that "!" should be removed:
>
> diff -r 662aa21d109f plugins/carbon/carbon.c
> --- a/plugins/carbon/carbon.c   Thu Jul 26 22:12:55 2012 -0700
> +++ b/plugins/carbon/carbon.c   Fri Aug 03 16:11:29 2012 +0200
> @@ -92,7 +92,7 @@
>                                 }
>
>                                 //skip per worker metrics when disabled
> -                               if (!u_carbon.no_workers) continue;
> +                               if (u_carbon.no_workers) continue;
>
>                                 rlen = snprintf(ptr, 4096, "uwsgi.%s.
> %s.worker%d.requests %llu %llu\n", uwsgi.hostname, u_carbon.id, i, (unsigned
> long long ) uwsgi.workers[i].requests, (unsigned long long )
> uwsgi.current_time);
>                                 if (rlen < 1) goto clear;
>



-- 
Łukasz Mierzwa
_______________________________________________
uWSGI mailing list
[email protected]
http://lists.unbit.it/cgi-bin/mailman/listinfo/uwsgi

Reply via email to