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

[release/v1.0] Cherrypick commits for v1.0.3 #4174

Closed
wants to merge 9 commits into from

Conversation

sanposhiho and others added 9 commits September 6, 2024 19:02
…her xdsIR (envoyproxy#3771)

* fix: prevent xdsIR updates from overwriting RateLimit configs from other xdsIR

Signed-off-by: Kensei Nakada <[email protected]>

* fix: handle deletion events appropriately

Signed-off-by: Kensei Nakada <[email protected]>

* test: add a unit test for subscribeAndTranslate

Signed-off-by: Kensei Nakada <[email protected]>

* chore: sort import order

Signed-off-by: Kensei Nakada <[email protected]>

---------

Signed-off-by: Kensei Nakada <[email protected]>
Co-authored-by: zirain <[email protected]>
(cherry picked from commit 762eb42)
Signed-off-by: Arko Dasgupta <[email protected]>
* This ensures the readinessProbe kicks in only after the container has
  started
* max startup time is 300s - 30 (failureThreshold) x 10 (periodSeconds).
After this the container is killed and the `restartPolicy` kicks in
https://kubernetes.io/docs/concepts/configuration/liveness-readiness-startup-probes/#startup-probe

Fixes: envoyproxy#3511

Signed-off-by: Arko Dasgupta <[email protected]>
(cherry picked from commit fa65316)
Signed-off-by: Arko Dasgupta <[email protected]>
* bugfix: fix upstream get unwanted /.

Signed-off-by: qicz <[email protected]>

* ut for bugfix

Signed-off-by: qicz <[email protected]>

---------

Signed-off-by: qicz <[email protected]>
Co-authored-by: Xunzhuo <[email protected]>
(cherry picked from commit b77f6a4)
Signed-off-by: Arko Dasgupta <[email protected]>
Signed-off-by: Kobi Levi <[email protected]>
(cherry picked from commit 97830e9)
Signed-off-by: Arko Dasgupta <[email protected]>
* fix: multiple reference grants in same namespace

Signed-off-by: Ardika Bagus <[email protected]>

* test: add e2e test

Signed-off-by: Ardika Bagus <[email protected]>

* chore: wrong service port

Signed-off-by: Ardika Bagus <[email protected]>

---------

Signed-off-by: Ardika Bagus <[email protected]>
(cherry picked from commit b82f4b2)
Signed-off-by: Arko Dasgupta <[email protected]>
…4021)

* Reduces time for the endpoint to be removed from the endpointSlice
from `30s` (3 * 10) to `5s` (1 * 5)

* Since kube-proxy and CNIs rely on this info and so do external LBs
like GKE https://cloud.google.com/kubernetes-engine/docs/concepts/service-load-balancer

Signed-off-by: Arko Dasgupta <[email protected]>
(cherry picked from commit 67575b8)
Signed-off-by: Arko Dasgupta <[email protected]>
fix ratelimit statd not working

Signed-off-by: zirain <[email protected]>
(cherry picked from commit 6ab6482)
Signed-off-by: Arko Dasgupta <[email protected]>
fix IsNotFound check for secret and configmap

Signed-off-by: TasdidurRahman <[email protected]>
(cherry picked from commit c20315f)
Signed-off-by: Arko Dasgupta <[email protected]>
Signed-off-by: qicz <[email protected]>
Co-authored-by: zirain <[email protected]>
(cherry picked from commit b50f5fa)
Signed-off-by: Arko Dasgupta <[email protected]>
@arkodg arkodg requested a review from a team as a code owner September 7, 2024 02:22
@arkodg arkodg requested a review from Xunzhuo September 7, 2024 02:22
@zirain
Copy link
Contributor

zirain commented Sep 7, 2024

we may need #4175?

Copy link

github-actions bot commented Oct 7, 2024

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Please feel free to give a status update now, ping for review, when it's ready. Thank you for your contributions!

@github-actions github-actions bot added the stale label Oct 7, 2024
@arkodg arkodg closed this Nov 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants