Skip to content
This repository has been archived by the owner on Jan 13, 2024. It is now read-only.

Add support for upstream response duration metric #5

Open
Firehed opened this issue Sep 7, 2018 · 1 comment
Open

Add support for upstream response duration metric #5

Firehed opened this issue Sep 7, 2018 · 1 comment
Labels
blocked Requires an upstream change of some kind new metric Request for a new metric

Comments

@Firehed
Copy link
Contributor

Firehed commented Sep 7, 2018

This is blocked by upsteam support, tracked at influxdata/nginx-influxdb-module#3. It will also likely require a version bump in the ingress controller itself. Since it will be based on entirely new data, conditional logic will be required to handle the data not being present.

@Firehed Firehed added new metric Request for a new metric blocked Requires an upstream change of some kind labels Sep 7, 2018
@Firehed Firehed changed the title Add support for response duration metric Add support for upstream response duration metric Sep 7, 2018
@Firehed
Copy link
Contributor Author

Firehed commented Sep 7, 2018

Updated the title to more accurately reflect what this data would be. I had initially thought that the existing request_time (see #1) was the nginx-exclusive duration, but in fact it's the end-to-end request duration. That certainly makes this issue way less important, though still useful.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocked Requires an upstream change of some kind new metric Request for a new metric
Projects
None yet
Development

No branches or pull requests

1 participant