Skip to content

docs: weighted cluster route manipulation#531

Merged
zuercher merged 2 commits intoenvoyproxy:masterfrom
turbinelabs:stephan/weighted-cluster-header-manip
Mar 13, 2018
Merged

docs: weighted cluster route manipulation#531
zuercher merged 2 commits intoenvoyproxy:masterfrom
turbinelabs:stephan/weighted-cluster-header-manip

Conversation

@zuercher
Copy link
Copy Markdown
Member

@zuercher zuercher commented Mar 8, 2018

Unhide the request_headers_to_add, response_headers_to_add, and response_headers_to_remove fields in ClusterWeight. Update HTTP conn manager docs related to same.

Doc update for envoyproxy/envoy#2765.

Signed-off-by: Stephan Zuercher stephan@turbinelabs.io

Signed-off-by: Stephan Zuercher <stephan@turbinelabs.io>
htuch
htuch previously approved these changes Mar 8, 2018
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@htuch
Copy link
Copy Markdown
Member

htuch commented Mar 8, 2018

@zuercher docs build fails.

Signed-off-by: Stephan Zuercher <stephan@turbinelabs.io>
Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ready to merge when Envoy-side PR merges.

zuercher added a commit to envoyproxy/envoy that referenced this pull request Mar 13, 2018
…clusters (#2765)

Implements the request_headers_to_add, response_headers_to_add, and response_headers_to_remove fields added to weighted clusters by envoyproxy/data-plane-api#441.

Risk Level: Low - no change in behavior without configuration changes

Testing: unit and integration tests

Docs Changes: envoyproxy/data-plane-api#531

Release Notes: updated

Fixes: #2455

Signed-off-by: Stephan Zuercher stephan@turbinelabs.io
@zuercher zuercher merged commit 0be1aa4 into envoyproxy:master Mar 13, 2018
@zuercher zuercher deleted the stephan/weighted-cluster-header-manip branch March 13, 2018 20:36
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