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

Begin process of removing singleton use by StringMatcher #32829

Merged
merged 15 commits into from
Mar 14, 2024

reduce coverage on config validation

8aac756
Select commit
Loading
Failed to load commit list.
Merged

Begin process of removing singleton use by StringMatcher #32829

reduce coverage on config validation
8aac756
Select commit
Loading
Failed to load commit list.
CI (Envoy) / Mobile/Android tests succeeded Mar 13, 2024 in 4m 8s

Mobile/Android tests (success)

Check has finished

Details

Check run finished (success ✔️)

The check run can be viewed here:

Mobile/Android tests (pr/32829/main@8aac756)

Check started by

Request (pr/32829/main@8aac756)

ggreenway @ggreenway 8aac756 #32829 merge main@3c84a24

Begin process of removing singleton use by StringMatcher

Commit Message: Begin process of removing singleton use by StringMatcher

This PR temporarily adds two versions of StringMatcherImpl: one that uses the existing singletons, and one that takes a factory context.

Subsequent PRs will gradually convert all uses of StringMatcherImpl to use the factory context. Then the version using singletons will be deleted, along with the code that sets the singletons.
Additional Description: This work is being broken into multiple PRs because otherwise it would be very large and difficult to review.

Part of #32792
Risk Level: low
Testing:
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 23a0f9df084f64b795d45ae21bcf247f40a8c643
sha 8aac756
pr 32829
base-sha 3c84a24
actor ggreenway @ggreenway
message Begin process of removing singleton use by StringMatcher...
started 1710358351.341222
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 30
patch 0
dev true