Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add automated functional test for tracing #1911

Closed
Tracked by #622
sjberman opened this issue May 1, 2024 · 0 comments · Fixed by #2004
Closed
Tracked by #622

Add automated functional test for tracing #1911

sjberman opened this issue May 1, 2024 · 0 comments · Fixed by #2004
Assignees
@sjberman
Labels
refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week tests Pull requests that update tests
Milestone
v1.3.0

Comments

@sjberman
Copy link
Contributor

sjberman commented May 1, 2024 •
edited

Acceptance:

  • Test that traces are sampled for a specific route or routes when configured. Test statuses on the policy.
    • Cases:
      • one policy and one route
      • one policy and multiple routes
@sjberman sjberman mentioned this issue May 1, 2024
Open
@sjberman sjberman self-assigned this May 1, 2024
@mpstefan mpstefan added this to the v1.3.0 milestone May 1, 2024
@mpstefan mpstefan added the tests Pull requests that update tests label May 1, 2024
@sjberman sjberman added refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week labels May 6, 2024
@sjberman sjberman mentioned this issue May 21, 2024
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sjberman sjberman

Labels
refined Requirements are refined and the issue is ready to be implemented. size/medium Estimated to be completed within a week tests Pull requests that update tests
Projects
NGINX Gateway Fabric
Status: ✅ Done
Milestone
v1.3.0
Development

Successfully merging a pull request may close this issue.

Support tracing via the ObservabilityPolicy sjberman/nginx-gateway-fabric
2 participants
@mpstefan @sjberman