Revert "Add canonical CEL support: dev.cel.expr (cncf#75)"

This reverts commit 0f5e0d9.

Author: sergiitk

bazel/external_proto_deps.bzl

@@ -29,14 +29,10 @@ EXTERNAL_PROTO_GO_BAZEL_DEP_MAP = {
     #    go_googleapis in https://github.com/bazelbuild/rules_go/blob/master/go/dependencies.rst#overriding-dependencies
     "@com_google_googleapis//google/api/expr/v1alpha1:checked_proto": "@go_googleapis//google/api/expr/v1alpha1:expr_go_proto",
     "@com_google_googleapis//google/api/expr/v1alpha1:syntax_proto": "@go_googleapis//google/api/expr/v1alpha1:expr_go_proto",
-    "@dev_cel//proto/cel/expr:checked_proto": "@dev_cel//proto/cel/expr:checked_go_proto",
-    "@dev_cel//proto/cel/expr:syntax_proto": "@dev_cel//proto/cel/expr:syntax_go_proto",
 }
 
 # This maps from the Bazel proto_library target to the C++ language binding target for external dependencies.
 EXTERNAL_PROTO_CC_BAZEL_DEP_MAP = {
     "@com_google_googleapis//google/api/expr/v1alpha1:checked_proto": "@com_google_googleapis//google/api/expr/v1alpha1:checked_cc_proto",
     "@com_google_googleapis//google/api/expr/v1alpha1:syntax_proto": "@com_google_googleapis//google/api/expr/v1alpha1:syntax_cc_proto",
-    "@dev_cel//proto/cel/expr:checked_proto": "@dev_cel//proto/cel/expr:checked_cc_proto",
-    "@dev_cel//proto/cel/expr:syntax_proto": "@dev_cel//proto/cel/expr:syntax_cc_proto",
 }

bazel/repositories.bzl

@@ -22,10 +22,6 @@ def xds_api_dependencies():
         "com_google_protobuf",
         locations = REPOSITORY_LOCATIONS,
     )
-    xds_http_archive(
-        name = "dev_cel",
-        locations = REPOSITORY_LOCATIONS,
-    )
     xds_http_archive(
         "io_bazel_rules_go",
         locations = REPOSITORY_LOCATIONS,

bazel/repository_locations.bzl

@@ -28,11 +28,6 @@ REPOSITORY_LOCATIONS = dict(
         strip_prefix = "protobuf-3.21.5",
         urls = ["https://github.com/protocolbuffers/protobuf/archive/v3.21.5.zip"],
     ),
-    dev_cel = dict(
-        sha256 = "6d4bf5a0ac04b35c421cb8317cd35bbe78aacffee787bc7aa89daa1e211ff6be",
-        strip_prefix = "cel-spec-0.13.0",
-        urls = ["https://github.com/google/cel-spec/archive/refs/tags/v0.13.0.tar.gz"],
-    ),
     io_bazel_rules_go = dict(
         sha256 = "6dc2da7ab4cf5d7bfc7c949776b1b7c733f05e56edc4bcd9022bb249d2e2a996",
         urls = [

go/xds/type/v3/cel.pb.go

@@ -7,7 +7,5 @@ xds_proto_package(
         "//xds/annotations/v3:pkg",
         "@com_google_googleapis//google/api/expr/v1alpha1:checked_proto",
         "@com_google_googleapis//google/api/expr/v1alpha1:syntax_proto",
-        "@dev_cel//proto/cel/expr:checked_proto",
-        "@dev_cel//proto/cel/expr:syntax_proto",
     ],
 )

go/xds/type/v3/cel.pb.validate.go

@@ -4,8 +4,6 @@ package xds.type.v3;
 
 import "google/api/expr/v1alpha1/checked.proto";
 import "google/api/expr/v1alpha1/syntax.proto";
-import "cel/expr/checked.proto";
-import "cel/expr/syntax.proto";
 import "google/protobuf/wrappers.proto";
 
 import "xds/annotations/v3/status.proto";
@@ -25,28 +23,14 @@ option (xds.annotations.v3.file_status).work_in_progress = true;
 // <https://github.com/google/cel-spec>`_ (CEL) program.
 message CelExpression {
   oneof expr_specifier {
+    option (validate.required) = true;
+
     // Parsed expression in abstract syntax tree (AST) form.
-    //
-    // Deprecated -- use ``cel_expr_parsed`` field instead.
-    // If ``cel_expr_parsed`` or ``cel_expr_checked`` is set, this field is not used.
-    google.api.expr.v1alpha1.ParsedExpr parsed_expr = 1  [deprecated = true];
+    google.api.expr.v1alpha1.ParsedExpr parsed_expr = 1;
 
     // Parsed expression in abstract syntax tree (AST) form that has been successfully type checked.
-    //
-    // Deprecated -- use ``cel_expr_checked`` field instead.
-    // If ``cel_expr_parsed`` or ``cel_expr_checked`` is set, this field is not used.
-    google.api.expr.v1alpha1.CheckedExpr checked_expr = 2  [deprecated = true];
+    google.api.expr.v1alpha1.CheckedExpr checked_expr = 2;
   }
-
-  // Parsed expression in abstract syntax tree (AST) form.
-  //
-  // If ``cel_expr_checked`` is set, this field is not used.
-  cel.expr.ParsedExpr cel_expr_parsed = 3;
-
-  // Parsed expression in abstract syntax tree (AST) form that has been successfully type checked.
-  //
-  // If set, takes precedence over ``cel_expr_parsed``.
-  cel.expr.CheckedExpr cel_expr_checked = 4;
 }
 
 // Extracts a string by evaluating a `Common Expression Language