2013/1/15 Łukasz Mierzwa <[email protected]>

> 2. Different namespace
>> Currently carbon plugin does not allow me to put my metrics in
>> arbitrary location (besides I found default metric path a bit too long
>> for my needs).
>> Maybe extra option(?) to allow to override base path for metrics ?
>>
>
> We could use extra options for specificity graphs root.
>

https://github.com/unbit/uwsgi/pull/119

I've added --carbon-root option, usage: --carbon-root "test.ns1"


> 4. Busy / idle workers
>> Currently we only have busyness which is not exactly what busy/idle
>> is eg. yesterday one of our instances locked up, so all workers were
>> busy and uwsgi couldn't dispatch any requests to workers (no request
>> was served by that time period). During that time busyness was 0 (most
>> likely due to the fact that no request was served during that period).
>>
>
> If I remeber correctly busyness is calculated by counting time needed for
> requests which were completed during last carbon interval. If there was no
> *complated* requests, than we can't count it, so we get 0.
> In case of carbon stats it is a minor issue, but busyness is also used in
> cheaper_busyness plugin so I will try to improve calculation in such case.
>

This is now fixed in master branch for both carbon and cheaper busyness
plugin, it should detect such condition and will set busyness to 100%.


> 5. Empty metrics
>> Some of the metrics are generated even though their value is always 0
>> (eg. rss/vsz) as those only get updated in certain circumstances
>> (enabled resource limiting base don memory footprint or memory usage
>> reporting). In such circumstances I don't think it makes sense to
>> generate them at all.
>
>
https://github.com/unbit/uwsgi/pull/120



> 6. Calculation for cheaped / active workers
>> I see that in some cases we calculate some total metrics (regading
>> whole instance) for all (even cheaped) workers and some only for active
>> workers (process is running).
>> I would say that eg. harakiri metric should be counted over all
>> rather than active only workers as it's a counter.
>
>
https://github.com/unbit/uwsgi/pull/120

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

Reply via email to