ci: run e2e tests on otel-plugin changes - #631
Conversation
The e2e-tests workflow's paths filter omitted otel-plugin/, so any PR touching only that module (including real code changes, not just dependency bumps) silently skipped e2e coverage. Add 'otel-plugin/**' to the paths filter on both the pull_request and push triggers.
Codex AI reviewNo actionable findings. The symmetric path additions correctly trigger E2E tests for Reviewed commit |
Claude AI reviewThis PR adds The change is correct and well-scoped:
No correctness, API-compatibility, replay/checkpoint, concurrency, logging, or serialization concerns apply to a two-line YAML No actionable findings. Residual test risk: Reviewed commit |
|
Nice, I was going to cover this in #630 |
Issue Link, if available
N/A
Description
The e2e-tests workflow's paths filter omitted otel-plugin/, so any PR touching only that module (including real code changes, not just dependency bumps) silently skipped e2e coverage. Add 'otel-plugin/**' to the paths filter on both the pull_request and push triggers.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Demo/Screenshots
N/A
Checklist
Testing
Unit Tests
Have unit tests been written for these changes? N/A
Integration Tests
Have integration tests been written for these changes? N/A
Examples
Has a new example been added for the change? (if applicable) N/A