Skip to content
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.

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

v1.9 backports 2021-03-08 #15254

Merged
merged 5 commits into from
Mar 9, 2021
Merged

v1.9 backports 2021-03-08 #15254

merged 5 commits into from
Mar 9, 2021

Conversation

jrajahalme
Copy link
Member

@jrajahalme jrajahalme commented Mar 8, 2021

Only the last two commits of #14462 were backported, and after #15232, as they contained necessary Envoy API v3 fixes not previously backported to v1.9.

Once this PR is merged, you can update the PR labels via:

$ for pr in 14462 15232 15253; do contrib/backporting/set-labels.py $pr done 1.9; done

[ upstream commit 8105ca4 ]

Use new options instead of these deprecated fields to avoid deprecation warnings:

- RouteAction.max_grpc_timeout
- Cluster.protocol_selection
- Cluster.http2_protocol_options

Define runtime option "overload.global_downstream_max_connections" to avoid a warning like:

       "there is no configured limit to the number of allowed active connections.
       Set a limit via the runtime key overload.global_downstream_max_connections"

Fixes: #14919
Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
[ upstream commit 79d0719 ]

Instruct users to pull the Cilium image before disabling 'systemd-resolved.service'.

Mention that the port 2379 needs to be explicitly given in LoadBalancer instructions.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@jrajahalme jrajahalme requested a review from a team as a code owner March 8, 2021 19:56
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.9 kind/backports This PR provides functionality previously merged into master. labels Mar 8, 2021
@jrajahalme jrajahalme added kind/backports This PR provides functionality previously merged into master. and removed kind/backports This PR provides functionality previously merged into master. labels Mar 8, 2021
@jrajahalme
Copy link
Member Author

test-backport-1.9

Update Envoy API to be able to compile without using deprecated fields.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@jrajahalme
Copy link
Member Author

test-backport-1.9

1 similar comment
@jrajahalme
Copy link
Member Author

test-backport-1.9

[ upstream commit 2df25bb ]

protobuf.Message may contain a mutex, which trips the go vet linter in
the CI.

Fix by using .String() instead on debug messages, and using a pointer
to transfer ownership rather than a copy.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
[ upstream commit 57b25c5 ]

The updated protobuf implementation contains cyclical data structures
and unexported fields that can not be compared.

Add a new checker "ExportedEquals" that only compares exported fields
of the given data structures. This ignoring of unexported fields is
not safe for comparing arbitrary data structures which may store
internal state in unexported fields.

Use this new "ExportedEquals" checker to compare protobuf
Messages. This avoids comparing global data structures pointer to by
protobuf implementation specific fields.

Avoid infinite recursion by keeping track which pointers have already
been followed.

Change existing use of checker.Equals to use simple Equals or HasLen
instead when possible.

Signed-off-by: Jarno Rajahalme <jarno@covalent.io>
@jrajahalme
Copy link
Member Author

test-backport-1.9

@jrajahalme
Copy link
Member Author

Unrelated fail #15255 on test-1.12-netnext

@jrajahalme jrajahalme merged commit 452b182 into v1.9 Mar 9, 2021
@jrajahalme jrajahalme deleted the pr/v1.9-backport-2021-03-08 branch March 9, 2021 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant