feat!: Update LRO metadata type to google.cloud.common.OperationMetadata

Google APIs · copybara-github · commit 4931d1b387a3 · 2023-03-29T12:09:09.000-07:00
BREAKING CHANGE

PiperOrigin-RevId: 520405218
diff --git a/google/cloud/video/stitcher/v1/BUILD.bazel b/google/cloud/video/stitcher/v1/BUILD.bazel
@@ -36,12 +36,12 @@ proto_library(
         "//google/api:client_proto",
         "//google/api:field_behavior_proto",
         "//google/api:resource_proto",
+        "//google/cloud/common:common_proto",
         "//google/longrunning:operations_proto",
         "@com_google_protobuf//:duration_proto",
         "@com_google_protobuf//:empty_proto",
         "@com_google_protobuf//:field_mask_proto",
         "@com_google_protobuf//:struct_proto",
-        "@com_google_protobuf//:timestamp_proto",
     ],
 )
 
@@ -90,6 +90,7 @@ java_gapic_library(
     deps = [
         ":stitcher_java_proto",
         "//google/api:api_java_proto",
+        "//google/cloud/common:common_java_proto",
     ],
 )
 
@@ -132,6 +133,7 @@ go_proto_library(
     protos = [":stitcher_proto"],
     deps = [
         "//google/api:annotations_go_proto",
+        "//google/cloud/common:common_go_proto",
         "//google/longrunning:longrunning_go_proto",
     ],
 )
@@ -148,6 +150,7 @@ go_gapic_library(
     transport = "grpc",
     deps = [
         ":stitcher_go_proto",
+        "//google/cloud/common:common_go_proto",
         "//google/longrunning:longrunning_go_proto",
         "@com_google_cloud_go_longrunning//:go_default_library",
         "@com_google_cloud_go_longrunning//autogen:go_default_library",
@@ -192,6 +195,7 @@ py_gapic_library(
     service_yaml = "videostitcher_v1.yaml",
     transport = "grpc",
     deps = [
+        "//google/cloud/common:common_py_proto",
     ],
 )
 
@@ -285,6 +289,7 @@ nodejs_gapic_assembly_pkg(
     deps = [
         ":stitcher_nodejs_gapic",
         ":stitcher_proto",
+        "//google/cloud/common:common_proto",
     ],
 )
 
diff --git a/google/cloud/video/stitcher/v1/video_stitcher_service.proto b/google/cloud/video/stitcher/v1/video_stitcher_service.proto
@@ -20,6 +20,7 @@ import "google/api/annotations.proto";
 import "google/api/client.proto";
 import "google/api/field_behavior.proto";
 import "google/api/resource.proto";
+import "google/cloud/common/operation_metadata.proto";
 import "google/cloud/video/stitcher/v1/ad_tag_details.proto";
 import "google/cloud/video/stitcher/v1/cdn_keys.proto";
 import "google/cloud/video/stitcher/v1/live_configs.proto";
@@ -29,7 +30,6 @@ import "google/cloud/video/stitcher/v1/stitch_details.proto";
 import "google/longrunning/operations.proto";
 import "google/protobuf/empty.proto";
 import "google/protobuf/field_mask.proto";
-import "google/protobuf/timestamp.proto";
 
 option go_package = "cloud.google.com/go/video/stitcher/apiv1/stitcherpb;stitcherpb";
 option java_multiple_files = true;
@@ -55,7 +55,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "parent,cdn_key,cdn_key_id";
     option (google.longrunning.operation_info) = {
       response_type: "CdnKey"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -83,7 +83,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "name";
     option (google.longrunning.operation_info) = {
       response_type: "google.protobuf.Empty"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -97,7 +97,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "cdn_key,update_mask";
     option (google.longrunning.operation_info) = {
       response_type: "CdnKey"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -181,7 +181,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "parent,slate,slate_id";
     option (google.longrunning.operation_info) = {
       response_type: "Slate"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -210,7 +210,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "slate,update_mask";
     option (google.longrunning.operation_info) = {
       response_type: "Slate"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -222,7 +222,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "name";
     option (google.longrunning.operation_info) = {
       response_type: "google.protobuf.Empty"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -254,7 +254,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "parent,live_config,live_config_id";
     option (google.longrunning.operation_info) = {
       response_type: "LiveConfig"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 
@@ -286,7 +286,7 @@ service VideoStitcherService {
     option (google.api.method_signature) = "name";
     option (google.longrunning.operation_info) = {
       response_type: "google.protobuf.Empty"
-      metadata_type: "OperationMetadata"
+      metadata_type: "google.cloud.common.OperationMetadata"
     };
   }
 }
@@ -763,18 +763,3 @@ message DeleteLiveConfigRequest {
     }
   ];
 }
-
-// Represents the metadata of the long-running operation.
-message OperationMetadata {
-  // The time the operation was created.
-  google.protobuf.Timestamp create_time = 1;
-
-  // The time the operation finished running.
-  google.protobuf.Timestamp end_time = 2;
-
-  // Server-defined resource path for the target of the operation.
-  string target = 3;
-
-  // Name of the verb executed by the operation.
-  string verb = 4;
-}
diff --git a/google/cloud/video/stitcher/v1/videostitcher_v1.yaml b/google/cloud/video/stitcher/v1/videostitcher_v1.yaml
@@ -7,13 +7,6 @@ apis:
 - name: google.cloud.video.stitcher.v1.VideoStitcherService
 - name: google.longrunning.Operations
 
-documentation:
-  rules:
-  - selector: google.longrunning.Operations.ListOperations
-    description: |-
-      Lists operations that match the specified filter in the request. If
-      the server doesn't support this method, it returns `UNIMPLEMENTED`.
-
 backend:
   rules:
   - selector: 'google.cloud.video.stitcher.v1.VideoStitcherService.*'
