Skip to content

Conversation

@mrproliu
Copy link
Contributor

If the tunnel (ambien mesh) is detected in the eBPF access log receiver, it will also adapt to the mesh metrics.

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

@mrproliu mrproliu added this to the 10.3.0 milestone May 21, 2025
@mrproliu mrproliu requested review from kezhenxu94 and wu-sheng May 21, 2025 06:35
@mrproliu mrproliu added submodule update Change submodule commit ID, including network protocol, query GraphQL and UI enhancement Enhancement on performance or codes labels May 21, 2025
kezhenxu94
kezhenxu94 previously approved these changes May 21, 2025
ALS_ANALYZER: ${{ matrix.analyzer }}
with:
e2e-file: $GITHUB_WORKSPACE/test/e2e-v2/cases/istio/als/e2e.yaml
e2e-file: test/e2e-v2/cases/istio/als/e2e.yaml
Copy link
Member

Choose a reason for hiding this comment

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

What was the $GITHUB_WORKSPACE/ for? @kezhenxu94

Copy link
Member

Choose a reason for hiding this comment

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

What was the $GITHUB_WORKSPACE/ for? @kezhenxu94

Just to make sure the path is absolute file path, if the e2e-file is relative to the workspace folder it’s OK to remove this

kezhenxu94
kezhenxu94 previously approved these changes May 21, 2025
@mrproliu mrproliu marked this pull request as draft May 21, 2025 09:16
@mrproliu mrproliu marked this pull request as ready for review May 22, 2025 05:42
@wu-sheng wu-sheng merged commit 5622cb9 into apache:master May 22, 2025
170 checks passed
@mrproliu mrproliu deleted the ambient-e2e branch May 22, 2025 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement on performance or codes submodule update Change submodule commit ID, including network protocol, query GraphQL and UI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants