Skip to content

RdsConfigProvider dedup#8268

Closed
lambdai wants to merge 3 commits intoenvoyproxy:masterfrom
lambdai:provider
Closed

RdsConfigProvider dedup#8268
lambdai wants to merge 3 commits intoenvoyproxy:masterfrom
lambdai:provider

Conversation

@lambdai
Copy link
Copy Markdown
Contributor

@lambdai lambdai commented Sep 17, 2019

WIP still fixing tests

Description:
If N http conn managers refers the same rds config name, currently envoy

  • create only one subscription (good!)
  • create N rds config provider at main thread (hmm...)
  • thundering herd when rds is update, since all N rds configs need updating

Risk Level:
Testing:
Docs Changes:
Release Notes:
[Optional Fixes #Issue]
[Optional Deprecated:]

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants