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.

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

dynamic_modules: adds integration tests #38285

Merged

test

84a3296
Select commit
Loading
Failed to load commit list.
Merged

dynamic_modules: adds integration tests #38285

test
84a3296
Select commit
Loading
Failed to load commit list.
CI (Envoy) / Mobile/iOS tests skipped Feb 1, 2025 in 0s

Check was skipped

This check was not triggered in this CI run

Details

Request (pr/38285/main@84a3296)

mathetake @mathetake 84a3296 #38285 merge main@9b0c0ba

dynamic_modules: adds integration tests

Commit Message: dynamic_modules: adds integration tests
Additional Description:

This adds various integration test cases for dynamic modules
that cover all currently implemented callbacks. While running
tests, I discovered a few edge cases around decodeData and
fixed a bug.

Risk Level: low
Testing: added
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

Environment

Request variables

Key Value
ref bcd83ad
sha 84a3296
pr 38285
base-sha 9b0c0ba
actor mathetake @mathetake
message dynamic_modules: adds integration tests...
started 1738396547.891374
target-branch main
trusted false
Build image

Container image/s (as used in this CI run)

Key Value
default envoyproxy/envoy-build-ubuntu:d2be0c198feda0c607fa33209da01bf737ef373f
mobile envoyproxy/envoy-build-ubuntu:mobile-d2be0c198feda0c607fa33209da01bf737ef373f
Version

Envoy version (as used in this CI run)

Key Value
major 1
minor 34
patch 0
dev true