Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Measure DNS lookup times for HTTP requests #1011

Open
Tracked by #2461
jonboydell-massive opened this issue Apr 26, 2019 · 2 comments
Open
Tracked by #2461

Measure DNS lookup times for HTTP requests #1011

jonboydell-massive opened this issue Apr 26, 2019 · 2 comments
Labels
enhancement feature new-http issues that would require (or benefit from) a new HTTP API

Comments

@jonboydell-massive
Copy link

I don't see the http_req_looking_up metric in the std output and if I try to create a Trend recording myTrend.add(res.timings.looking_up); I get a series of 0s in the output of that Trend.

I'm using v0.24.0

@na--
Copy link
Member

na-- commented Apr 30, 2019

Thanks for reporting this and sorry for the confusion - the documentation was wrong, it seems like it hadn't been updated after the tracking of DNS lookup times was removed in k6 v0.13 by this commit.

I don't know the full details about what prompted the removal of the metric, though #41 points to the possibility that it was also part of the http_req_duration meta-metric and skewed it too much. In any case, we plan to bring it back, and improve DNS support in general. I'll leave this issue open as a tracking point for the DNS metric measurements specifically, and here are the other DNS-related issues that we want to fix:

@na-- na-- changed the title http_req_looking_up metric missing from output Measure DNS lookup times for HTTP requests Apr 30, 2019
@na-- na-- added this to the v1.0.0 milestone Aug 27, 2019
@MattDodsonEnglish
Copy link
Contributor

This response metric was still documented in the API. It said "currently unavailable", but nevertheless at least one user tried to use it anyway (that user was me).

I've removed the documentation. If this issue is ever fixed, we can redocument it.

grafana/k6-docs@26e5109

@na-- na-- modified the milestones: v1.0.0, TBD Nov 9, 2022
@codebien codebien removed this from the TBD milestone Sep 27, 2023
@mstoykov mstoykov added the new-http issues that would require (or benefit from) a new HTTP API label Nov 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement feature new-http issues that would require (or benefit from) a new HTTP API
Projects
None yet
Development

No branches or pull requests

5 participants