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

Add additional granularity to HTTP Request Grafana #360

Merged

Conversation

martinbrose
Copy link
Contributor

@martinbrose martinbrose commented Mar 21, 2022

Closes #358:

Adding additional 5 granularity overviews of the HTTP Request Duration to enable the user to troubleshoot further, showing HTTP Request Duration:

  • Connect
  • Processing
  • Resolve
  • TLS
  • Transfer

Example screenshots:
Screenshot 2022-04-02 at 22 50 29
Screenshot 2022-04-02 at 22 50 46

@martinbrose
Copy link
Contributor Author

martinbrose commented Apr 28, 2022

Somehow my softmin and softmax are not working. Will need to look into that.

@stale
Copy link

stale bot commented Jul 30, 2022

This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark pull requests as stale.

@stale stale bot added the stale label Jul 30, 2022
@stale
Copy link

stale bot commented Sep 19, 2022

This pull request has been closed due to inactivity. If you feel this is in error, please reopen the pull request or file a new PR with the relevant details.

@stale stale bot closed this Sep 19, 2022
@geerlingguy geerlingguy reopened this Jan 16, 2023
@stale
Copy link

stale bot commented Jan 21, 2023

This pull request is no longer marked for closure.

@martinbrose
Copy link
Contributor Author

martinbrose commented Jan 21, 2023

Updated my branch:

  • brought it up to the current master
  • updated to lastest time series template in Grafana
  • fixed my soft min and soft max issue
  • added jitter in line with discussion Jitter #469
  • added units to the graphs

Tested on two Raspberry Pis with Debian and Vagrant Debian & Ubuntu instances. Works well thus far and didn't see any issues in the Grafana docker log or in the Grafana UI.

Example screenshot:
Grafana-additionalDetails-screenshot

@stale stale bot removed the stale label Jan 21, 2023
@stale
Copy link

stale bot commented Jan 21, 2023

This issue is no longer marked for closure.

@geerlingguy geerlingguy merged commit 16926bf into geerlingguy:master Jan 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Help requested - latency spikes
2 participants