higress icon indicating copy to clipboard operation
higress copied to clipboard

fix(helm): improve higress access log format for Model and MCP monitor

Open lexburner opened this issue 1 month ago • 2 comments

  • 新增集群ID和消费者信息记录
  • 调整请求和响应相关字段,增加请求及响应体日志
  • 修正部分请求头字段名称
  • 增加对扩展认证状态及持续时间的记录

Ⅰ. Describe what this PR did

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

Ⅵ. AI Coding Tool Usage Checklist (if applicable)

Please check all applicable items:

  • [ ] For new standalone features (e.g., new wasm plugin or golang-filter plugin):

    • [ ] I have created a design/ directory in the plugin folder
    • [ ] I have added the design document to the design/ directory
    • [ ] I have included the AI Coding summary below
  • [ ] For regular updates/changes (not new plugins):

    • [ ] I have provided the prompts/instructions I gave to the AI Coding tool below
    • [ ] I have included the AI Coding summary below

AI Coding Prompts (for regular updates)

AI Coding Summary

lexburner avatar Nov 30 '25 02:11 lexburner

改进 Higress 访问日志格式以支持模型和 MCP 监控

变更概述
  • 问题修复

    • 修复并增强了 Helm 配置中的访问日志格式,使其包含更多用于监控和调试的关键字段。
    • 添加了对请求体和响应体日志的支持,便于追踪完整的 HTTP 流量内容。
    • 引入集群 ID 和消费者信息记录,提升多实例环境下的可追溯性。
    • 补充扩展认证状态码及耗时的记录,加强安全与性能分析能力。
    • 修正部分请求头字段名称,确保与标准 Envoy 变量一致。
  • 配置调整

    • 更新 accessLogFormat 字段,在 helm/core/templates/configmap.yaml 中定义新的 JSON 格式模板。
    • 新增字段包括:cluster_idconsumerrequest_bodyresponse_body 等。
    • 使用 FILTER_STATE 和 DYNAMIC_METADATA 提取 WASM 插件和 Envoy 内部状态数据。
变更文件
文件路径 变更说明
helm/​core/​templates/​configmap.​yaml 更新了访问日志格式,新增多个关键字段如集群ID、消费者、请求体、响应体等,并修正了一些请求头变量名。

💡 小贴士

与 lingma-agents 交流的方式

📜 直接回复评论 直接回复本条评论,lingma-agents 将自动处理您的请求。例如:

  • 在当前代码中添加详细的注释说明。

  • 请详细介绍一下你说的 LRU 改造方案,并使用伪代码加以说明。

📜 在代码行处标记 在文件的特定位置创建评论并 @lingma-agents。例如:

  • @lingma-agents 分析这个方法的性能瓶颈并提供优化建议。

  • @lingma-agents 对这个方法生成优化代码。

📜 在讨论中提问 在任何讨论中 @lingma-agents 来获取帮助。例如:

  • @lingma-agents 请总结上述讨论并提出解决方案。

  • @lingma-agents 请根据讨论内容生成优化代码。

lingma-agents[bot] avatar Nov 30 '25 02:11 lingma-agents[bot]

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 46.85%. Comparing base (ae0bb41) to head (1719746).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3181      +/-   ##
==========================================
- Coverage   47.98%   46.85%   -1.14%     
==========================================
  Files         143       87      -56     
  Lines       22880    12922    -9958     
==========================================
- Hits        10979     6054    -4925     
+ Misses      11362     6478    -4884     
+ Partials      539      390     -149     
Flag Coverage Δ
wasm-go-plugin-ai-proxy ?

Flags with carried forward coverage won't be shown. Click here to find out more. see 59 files with indirect coverage changes

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov-commenter avatar Nov 30 '25 02:11 codecov-commenter