Change http_request_<duration> for PlugPipelineInstrumenter

[PhillippOhlandt]

Hey,

I tried switching from :microseconds to :seconds since Prometheus suggests to use second as base unit. The problem is, the config setting only changes the metric name but not the buckets and value. While defining custom buckets is not a big deal, the reported value is still in microseconds.

Or do I miss something here?

[sorliem]

I see the same thing... shouldn't the diff take into account the duration_unit?

Looking at

prometheus-plugs/lib/prometheus/plug_pipeline_instrumenter.ex

Lines 150 to 151 in 72fa654

	stop = :erlang.monotonic_time()
	diff = stop - start

Shouldn't that instead be

diff = :erlang.convert_time_unit(stop - start, :native, duration_unit)