Skip to content

Fix: remove the redundant tags from the envoy-ai-gateway.yaml LAL configuration.#13854

Merged
wu-sheng merged 2 commits intoapache:masterfrom
wankai123:envoy-ai-gateway-lal
Apr 30, 2026
Merged

Fix: remove the redundant tags from the envoy-ai-gateway.yaml LAL configuration.#13854
wu-sheng merged 2 commits intoapache:masterfrom
wankai123:envoy-ai-gateway-lal

Conversation

@wankai123
Copy link
Copy Markdown
Member

  • If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
  • Update the CHANGES log.

@wankai123 wankai123 added this to the 10.5.0 milestone Apr 30, 2026
@wankai123 wankai123 requested review from Copilot and wu-sheng April 30, 2026 02:20
@wankai123 wankai123 added the bug Something isn't working and you are sure it's a bug! label Apr 30, 2026
Copy link
Copy Markdown

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

Removes redundant no-op tag assignments from the Envoy AI Gateway LAL rule so the config only adds the intentional ai_route_type tag while relying on existing LogData tags for the rest.

Changes:

  • Remove self-copy tag assignments (e.g., tag 'x': tag("x")) from the LLM and MCP extractor blocks in envoy-ai-gateway.yaml.
  • Add a corresponding entry to the 10.5.0 changelog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
oap-server/server-starter/src/main/resources/lal/envoy-ai-gateway.yaml Simplifies LAL extractor blocks by removing redundant tag copy-through assignments.
docs/en/changes/changes.md Records the configuration cleanup in the release notes.

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

Comment thread docs/en/changes/changes.md Outdated
@wu-sheng wu-sheng merged commit 3064b5f into apache:master Apr 30, 2026
430 of 435 checks passed
@wankai123 wankai123 deleted the envoy-ai-gateway-lal branch April 30, 2026 05:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working and you are sure it's a bug!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants