filter_state: remove checks for read-only #44343
Open
CI (Envoy) / Mobile/Perf
skipped
Apr 9, 2026 in 0s
Check was skipped
This check was not triggered in this CI run
Details
Request (pr/44343/main@71ccfee)
71ccfee 
#44343 merge main@d4fd8e3
filter_state: remove checks for read-only
The checks for mutable vs read-only have caused many subtle runtime bugs, and have not provided much benefit.
The parameter for read-only vs mutable is now ignored, and will be removed from the codebase in phases.
Commit Message:
Additional Description:
Risk Level: Low
Testing: Updated tests
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]
Environment
Request variables
| Key | Value |
|---|---|
| ref | 62c4d74 |
| sha | 71ccfee |
| pr | 44343 |
| base-sha | d4fd8e3 |
| actor | @ggreenway |
| message | filter_state: remove checks for read-only... |
| started | 1775775657.163252 |
| target-branch | main |
| trusted | false |
Build image
Container image/s (as used in this CI run)
| Key | Value |
|---|---|
| default | docker.io/envoyproxy/envoy-build:86873047235e9b8232df989a5999b9bebf9db69c |
| mobile | docker.io/envoyproxy/envoy-build:mobile-86873047235e9b8232df989a5999b9bebf9db69c |
Version
Envoy version (as used in this CI run)
| Key | Value |
|---|---|
| major | 1 |
| minor | 38 |
| patch | 0 |
| dev | true |
Loading