Skip to content

[xds] Relax the CHECK on factories for typed per-filter config #15770

@stevenzzzz

Description

@stevenzzzz

Title: Do not reject the typed per-filter-config if a factory can't be found, remove binaries version coupling.

Description:
Right now for the typedConfig in route we demand the factory class to be linked in at Envoy, otherwise it rejects the whole DiscoveryResponse.
This adds coupling between management server and Envoy, which introduces complexity to rollout/backs of these binaries.

Let's be more tolerant to such config change. (with a runtime feature flag guard), similar to Protobuf upgrade.

Instead of throwing an exception, let's do logging and adding stats for such missing factories.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions