Skip to content

Revert "Optimize EDS stream allocation (#22419)"#25157

Merged
ggreenway merged 3 commits intoenvoyproxy:mainfrom
ggreenway:backout-22419
Jan 26, 2023
Merged

Revert "Optimize EDS stream allocation (#22419)"#25157
ggreenway merged 3 commits intoenvoyproxy:mainfrom
ggreenway:backout-22419

Conversation

@ggreenway
Copy link
Copy Markdown
Member

This reverts commit 97a7f00.

There was a crash reported by go-control-plane attributed to this change: envoyproxy/go-control-plane#625

Signed-off-by: Greg Greenway [email protected]

Commit Message:
Additional Description:
Risk Level:
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:]

This reverts commit 97a7f00.

There was a crash reported by go-control-plane attributed to this change:
envoyproxy/go-control-plane#625

Signed-off-by: Greg Greenway <[email protected]>
@repokitteh-read-only
Copy link
Copy Markdown

CC @envoyproxy/runtime-guard-changes: FYI only for changes made to (source/common/runtime/runtime_features.cc).

🐱

Caused by: #25157 was opened by ggreenway.

see: more, trace.

mattklein123
mattklein123 previously approved these changes Jan 25, 2023
@ggreenway ggreenway enabled auto-merge (squash) January 25, 2023 18:05
@ggreenway ggreenway merged commit 8affda4 into envoyproxy:main Jan 26, 2023
VishalDamgude pushed a commit to freshworks-oss/envoy that referenced this pull request Feb 2, 2023
…y#25157)

This reverts commit 97a7f00.

There was a crash reported by go-control-plane attributed to this change:
envoyproxy/go-control-plane#625

Signed-off-by: Greg Greenway <[email protected]>
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.

2 participants