Skip to content

[TT-15243] InfluxDB 2 Pump does not include Latency.Total and Latency.Uptime#881

Closed
deeparavirdc wants to merge 2 commits intoTykTechnologies:masterfrom
deeparavirdc:influx2-latency-metric
Closed

[TT-15243] InfluxDB 2 Pump does not include Latency.Total and Latency.Uptime#881
deeparavirdc wants to merge 2 commits intoTykTechnologies:masterfrom
deeparavirdc:influx2-latency-metric

Conversation

@deeparavirdc
Copy link
Copy Markdown
Contributor

Description

Add latency metrics to influx2 pump

Related Issue

Motivation and Context

This would be crucial in deriving gateway processing time responsible for latency calculation in turn impacting SLA.

How This Has Been Tested

Added tests and tested with

 go test ./pumps -v -run TestInflux2PumpMappingIncludesLatency

# github.com/shirou/gopsutil/disk
iostat_darwin.c:28:2: warning: 'IOMasterPort' is deprecated: first deprecated in macOS 12.0 [-Wdeprecated-declarations]
/Library/Developer/CommandLineTools/SDKs/MacOSX.sdk/System/Library/Frameworks/IOKit.framework/Headers/IOKitLib.h:143:1: note: 'IOMasterPort' has been explicitly marked deprecated here
=== RUN   TestInflux2PumpMappingIncludesLatency
--- PASS: TestInflux2PumpMappingIncludesLatency (0.00s)
PASS
ok      github.com/TykTechnologies/tyk-pump/pumps       2.964s

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side). If pulling from your own
    fork, don't request your master!
  • Make sure you are making a pull request against the master branch (left side). Also, you should start
    your branch off our latest master.
  • My change requires a change to the documentation.
    • If you've changed APIs, describe what needs to be updated in the documentation.
  • I have updated the documentation accordingly.
  • Modules and vendor dependencies have been updated; run go mod tidy && go mod vendor
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Check your code additions will not fail linting checks:
    • go fmt -s
    • go vet

@vverbani vverbani changed the title feat: add latency metrics to influx2 pump [TT-15243] InfluxDB 2 Pump does not include Latency.Total and Latency.Uptime Aug 19, 2025
@andrei-tyk andrei-tyk mentioned this pull request Oct 8, 2025
14 tasks
@andrei-tyk
Copy link
Copy Markdown
Contributor

First and foremost thank you for this contribution!

I have created an internal PR on the original repo to allow all of our CI workflows to run. All your commits are there so no credit will be taken away from you.

You can find it at: #900

@andrei-tyk andrei-tyk closed this Oct 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants