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

envoy: Update envoy version to the latest build #27819

Merged
merged 2 commits into from
Aug 31, 2023

Conversation

jrajahalme
Copy link
Member

@jrajahalme jrajahalme commented Aug 30, 2023

This commit is to perform the below:

  • Update envoy image to latest build
  • Include new resources into pkg/envoy/resource for serialization
    • grpc related resources are for upcoming support with GRPCRoute
  • Change policy id to uint32, related to cilium/proxy@f37daf7

Related build: https://github.com/cilium/proxy/actions/runs/5974676861/job/16210098929

This PR supersedes #27699 to fix merge conflict while @sayboras is on PTO.

This commit is to perform the below:

- Update envoy image to latest build
- Include new resources into pkg/envoy/resource for serialization
  - grpc related resources are for upcoming support with GRPCRoute
- Change policy id to uint32, related to cilium/proxy@f37daf7

Related build: https://github.com/cilium/proxy/actions/runs/5974676861/job/16210098929

Signed-off-by: Tam Mach <tam.mach@cilium.io>
This commit is to perform the below:

- Update envoy image to latest build
- Include new resources into pkg/envoy/resource for serialization
  - grpc related resources are for upcoming support with GRPCRoute
- Change policy id to uint32, related to cilium/proxy@f37daf7

Related build: https://github.com/cilium/proxy/actions/runs/5974676861/job/16210098929

Signed-off-by: Tam Mach <tam.mach@cilium.io>
@jrajahalme jrajahalme added area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. release-note/minor This PR changes functionality that users may find relevant to operating Cilium. feature/k8s-gateway-api labels Aug 30, 2023
@jrajahalme jrajahalme requested review from a team as code owners August 30, 2023 08:28
@jrajahalme
Copy link
Member Author

@kaworu @tklauser @lambdanis @youngnick Please review, this supersedes #27699 to fix merge conflict while @sayboras is on PTO (I do not have write access to his fork to push this fix to the original PR).

Copy link
Member

@tklauser tklauser left a comment

Choose a reason for hiding this comment

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

LGTM for vendor

@jrajahalme
Copy link
Member Author

/test

Copy link
Member

@kaworu kaworu left a comment

Choose a reason for hiding this comment

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

Helm changes LGTM.

install/kubernetes/cilium/values.yaml Show resolved Hide resolved
@jrajahalme
Copy link
Member Author

/ci-ingress

@jrajahalme
Copy link
Member Author

/ci-runtime

@jrajahalme
Copy link
Member Author

/ci-integration

@jrajahalme jrajahalme merged commit af35b1a into cilium:main Aug 31, 2023
64 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. feature/k8s-gateway-api release-note/minor This PR changes functionality that users may find relevant to operating Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants