Skip to content

feat: nginx 采集插件添加对 tengine request_time 采集支持#1408

Merged
kongfei605 merged 2 commits intoflashcatcloud:mainfrom
0x0034:feat/nginxInputSupportTengine
Feb 26, 2026
Merged

feat: nginx 采集插件添加对 tengine request_time 采集支持#1408
kongfei605 merged 2 commits intoflashcatcloud:mainfrom
0x0034:feat/nginxInputSupportTengine

Conversation

@0x0034
Copy link
Contributor

@0x0034 0x0034 commented Feb 26, 2026

Active connections: 13495
server accepts handled requests request_time
 250674001 250674001 243630421 99268364027350
Reading: 0 Writing: 13487 Waiting: 7
  1. 添加对request_time 指标采集支持
  2. 更新 sonic 版本

Signed-off-by: ruochen <wanxialianwei@gmail.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request adds support for collecting the request_time metric from Tengine's nginx status endpoint and updates the sonic JSON library to version 1.15.0.

Changes:

  • Added parsing logic for the optional 4th field (request_time) in the nginx stub_status response, which is specific to Tengine
  • Updated github.com/bytedance/sonic from v1.9.1 to v1.15.0 and added related dependencies
  • Moved github.com/elastic/go-sysinfo from indirect to direct dependency (likely from running go mod tidy)

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.

File Description
inputs/nginx/nginx.go Added conditional parsing for the optional request_time field when it exists in the status response
go.mod Updated sonic library to v1.15.0, added new dependencies (gopkg, sonic/loader, cloudwego/base64x), moved elastic/go-sysinfo to direct dependency, and updated klauspost/cpuid/v2
go.sum Updated checksums for all modified and new dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@kongfei605
Copy link
Collaborator

Tahnk you @0x0034

@kongfei605 kongfei605 merged commit d3d5b8a into flashcatcloud:main Feb 26, 2026
2 of 3 checks passed
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.

3 participants