Skip to content

[Update] NodeBalancer Config with ProxyProtocol #317

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.

Sign up for GitHub

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

Merged
merged 5 commits into from
Sep 8, 2020
Merged

[Update] NodeBalancer Config with ProxyProtocol #317

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
13e2fac
updated nb config to include proxyprotocol
Aug 31, 2020
996dd5f
updated enum
Sep 1, 2020
d35b342
update description
Sep 1, 2020
b593acb
updated description
Sep 1, 2020
e572184
formatting the list
Sep 1, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 27 additions & 0 deletions openapi.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9437,6 +9437,7 @@ paths:
"check_path": "/test",
"check_body": "it works",
"check_passive": true,
"proxy_protocol": "v1",
"cipher_suite": "recommended"
}' \
https://api.linode.com/v4/nodebalancers/12345/configs
Expand All @@ -9454,6 +9455,7 @@ paths:
--check_path "/test" \
--check_body "it works" \
--check_passive true \
--proxy_protocol "v1" \
--cipher_suite recommended
/nodebalancers/{nodeBalancerId}/configs/{configId}:
parameters:
Expand Down Expand Up @@ -9548,6 +9550,7 @@ paths:
"check_path": "/test",
"check_body": "it works",
"check_passive": true,
"proxy_protocol": "v1",
"cipher_suite": "recommended"
}' \
https://api.linode.com/v4/nodebalancers/12345/configs/4567
Expand All @@ -9566,6 +9569,7 @@ paths:
--check_path "/test" \
--check_body "it works" \
--check_passive true \
--proxy_protocol "v1" \
--cipher_suite recommended
delete:
x-linode-grant: read_write
Expand Down Expand Up @@ -9697,6 +9701,7 @@ paths:
"check_path": "/test",
"check_body": "it works",
"check_passive": true,
"proxy_protocol": "v1",
"cipher_suite": "recommended",
"nodes": [
{
Expand Down Expand Up @@ -9730,6 +9735,7 @@ paths:
--check_path "/test" \
--check_body "it works" \
--check_passive true \
--proxy_protocol "v1" \
--cipher_suite recommended
/nodebalancers/{nodeBalancerId}/configs/{configId}/nodes:
parameters:
Expand Down Expand Up @@ -17139,6 +17145,27 @@ components:
enough for it to be considered unhealthy and taken out of rotation.
example: true
x-linode-cli-display: 6
proxy_protocol:
Copy link

Choose a reason for hiding this comment

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

@hzoppetti it accepts "v1", "v2", and "none", so it is not nullable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

updated enum

description: >
ProxyProtocol is a TCP extension that sends initial TCP connection
information such as source/destination IPs and ports to backend devices.
This information would be lost otherwise. Backend devices must be
configured to work with ProxyProtocol if enabled.


* If ommited, or set to `none`, the NodeBalancer doesn't send any auxilary
data over TCP connections. This is the default.

* If set to `v1`, the human-readable header format (Version 1) is used.

* If set to `v2`, the binary header format (Version 2) is used.
type: string
enum:
Copy link

Choose a reason for hiding this comment

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

none should come first (compare with other items, e.g., stickiness)

- none
- v1
- v2
example: none
default: none
cipher_suite:
type: string
enum:
Expand Down