envoy icon indicating copy to clipboard operation
envoy copied to clipboard

deps: bump up cel-cpp to v0.11.0

Open agrawroh opened this issue 6 months ago • 7 comments
trafficstars

Description

This PR bumps up the version of cel-cpp to v0.11.0.

There are some major changes where we now have cel::expr::ParsedExpr instead of google::api::expr::v1alpha1::ParsedExpr. We are doing the conversion in the code to keep it backward compatible. We should remove it in a follow-up PR by migrating everything to use cel::expr::ParsedExpr instead.


Commit Message: deps: bump up cel-cpp to v0.11.0 Additional Description: Bump up the version of cel-cpp to v0.11.0 Risk Level: Low Testing: CI Docs Changes: N/A Release Notes: N/A

agrawroh avatar May 05 '25 17:05 agrawroh

As a reminder, PRs marked as draft will not be automatically assigned reviewers, or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/39334 was opened by agrawroh.

see: more, trace.

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch). envoyproxy/dependency-shepherds assignee is @mattklein123

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/39334 was opened by agrawroh.

see: more, trace.

/retest

agrawroh avatar May 13 '25 19:05 agrawroh

cc @kyessenov Could you PTAL?

agrawroh avatar May 13 '25 19:05 agrawroh

/retest

agrawroh avatar May 13 '25 19:05 agrawroh

/wait

kyessenov avatar May 16 '25 18:05 kyessenov

The change looks good to me. Does it need main merge to fix coverage?

@kyessenov can you take another pass, please?

/wait-any

yanavlasov avatar Jun 18 '25 14:06 yanavlasov

/wait

kyessenov avatar Jun 20 '25 18:06 kyessenov

CC @envoyproxy/coverage-shephards: FYI only for changes made to (test/coverage.yaml). envoyproxy/coverage-shephards assignee is @RyanTheOptimist

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/39334 was synchronize by agrawroh.

see: more, trace.

/retest

agrawroh avatar Jun 27 '25 21:06 agrawroh

/wait

kyessenov avatar Jul 07 '25 18:07 kyessenov

/wait

RyanTheOptimist avatar Jul 14 '25 02:07 RyanTheOptimist

/retest

agrawroh avatar Jul 26 '25 16:07 agrawroh