Error message to mention UpstreamCodec filter #36367
+64
−13
Open
CI (Envoy) / Mobile/ASAN
skipped
Sep 27, 2024 in 0s
Check was skipped
This check was not triggered in this CI run
Details
Request (pr/36367/main@7b12a1c)
7b12a1c 
#36367 merge main@39851cd
Error message to mention UpstreamCodec filter
Commit Message: Error message to mention UpstreamCodec filter
Additional Description: The requirement described in #26138 doesn't seem to be mentioned in documentation, and even if it was I most likely wouldn't have seen it and probably neither would the person who filed the issue, so documenting the requirement in the failed validation error message seems a more practical solution.
Risk Level: Only changes an error message, so ~none.
Testing: Added unit test coverage (previously coverage was partial and from integration tests).
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a
Environment
Request variables
| Key | Value |
|---|---|
| ref | fac6a9e |
| sha | 7b12a1c |
| pr | 36367 |
| base-sha | 39851cd |
| actor | @ravenblackx |
| message | Error message to mention UpstreamCodec filter... |
| started | 1727465817.20699 |
| target-branch | main |
| trusted | false |
Build image
Container image/s (as used in this CI run)
| Key | Value |
|---|---|
| default | envoyproxy/envoy-build-ubuntu:f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
| mobile | envoyproxy/envoy-build-ubuntu:mobile-f94a38f62220a2b017878b790b6ea98a0f6c5f9c |
Version
Envoy version (as used in this CI run)
| Key | Value |
|---|---|
| major | 1 |
| minor | 32 |
| patch | 0 |
| dev | true |
Loading