Not at all clear how to add labels to the Summary(...) time() decorator pattern

In the README, in one of the samples, we have:

# Create a metric to track time spent and requests made.
REQUEST_TIME = Summary('request_processing_seconds', 'Time spent processing request')

# Decorate function with metric.
@REQUEST_TIME.time()
def process_request(t):

But what if I want to add labels to this? I can’t figure out how to make it work. My current code which doesn’t work is:

TRADE_REQUEST_TIME = Summary(
    'request_processing_seconds',
    'Time spent processing a request',
    labelnames=('method', 'endpoint', 'whitelabel'),
    labelvalues=('get', '/trade/', settings.WHITELABEL_NAME))


@TRADE_REQUEST_TIME.time()
def render_trade_page(request, sport):
    ...

But I get:

    @TRADE_REQUEST_TIME.time()
AttributeError: '_LabelWrapper' object has no attribute 'time'

Any advice?

[kamalmarhubi]

Unfortunately that’s not valid syntax. However, this works:

REQUEST_TIME = Summary(
    'request_processing_seconds',
    'Time spent processing a request',
   ('method', 'endpoint', 'whitelabel')
)

TRADE_REQUEST_TIME = REQUEST_TIME.labels('GET', '/trade/', settings.WHITELABEL_NAME)

@TRADE_REQUEST_TIME.time()
def render_trade_page(request, sport):
  ...

It’s probably worth pushing this down further into your serving code, so you don’t need to have the noise of all the submetric definitions.

[brian-brazil]

Yes, that’s my up. The top is where the calls enter the system.