Skip to content

Commit

Permalink
fix!: Roll back the changes that update of LRO metadata type to googl…
Browse files Browse the repository at this point in the history
…e.cloud.common.OperationMetadata

PiperOrigin-RevId: 520464508
  • Loading branch information
Google APIs authored and Copybara-Service committed Mar 29, 2023
1 parent 273994f commit cf01552
Show file tree
Hide file tree
Showing 3 changed files with 32 additions and 15 deletions.
7 changes: 1 addition & 6 deletions google/cloud/video/stitcher/v1/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -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",
],
)

Expand Down Expand Up @@ -90,7 +90,6 @@ java_gapic_library(
deps = [
":stitcher_java_proto",
"//google/api:api_java_proto",
"//google/cloud/common:common_java_proto",
],
)

Expand Down Expand Up @@ -133,7 +132,6 @@ go_proto_library(
protos = [":stitcher_proto"],
deps = [
"//google/api:annotations_go_proto",
"//google/cloud/common:common_go_proto",
"//google/longrunning:longrunning_go_proto",
],
)
Expand All @@ -150,7 +148,6 @@ 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",
Expand Down Expand Up @@ -195,7 +192,6 @@ py_gapic_library(
service_yaml = "videostitcher_v1.yaml",
transport = "grpc",
deps = [
"//google/cloud/common:common_py_proto",
],
)

Expand Down Expand Up @@ -289,7 +285,6 @@ nodejs_gapic_assembly_pkg(
deps = [
":stitcher_nodejs_gapic",
":stitcher_proto",
"//google/cloud/common:common_proto",
],
)

Expand Down
33 changes: 24 additions & 9 deletions google/cloud/video/stitcher/v1/video_stitcher_service.proto
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ 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";
Expand All @@ -30,6 +29,7 @@ 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;
Expand All @@ -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: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand Down Expand Up @@ -83,7 +83,7 @@ service VideoStitcherService {
option (google.api.method_signature) = "name";
option (google.longrunning.operation_info) = {
response_type: "google.protobuf.Empty"
metadata_type: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand All @@ -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: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand Down Expand Up @@ -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: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand Down Expand Up @@ -210,7 +210,7 @@ service VideoStitcherService {
option (google.api.method_signature) = "slate,update_mask";
option (google.longrunning.operation_info) = {
response_type: "Slate"
metadata_type: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand All @@ -222,7 +222,7 @@ service VideoStitcherService {
option (google.api.method_signature) = "name";
option (google.longrunning.operation_info) = {
response_type: "google.protobuf.Empty"
metadata_type: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand Down Expand Up @@ -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: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}

Expand Down Expand Up @@ -286,7 +286,7 @@ service VideoStitcherService {
option (google.api.method_signature) = "name";
option (google.longrunning.operation_info) = {
response_type: "google.protobuf.Empty"
metadata_type: "google.cloud.common.OperationMetadata"
metadata_type: "OperationMetadata"
};
}
}
Expand Down Expand Up @@ -763,3 +763,18 @@ 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;
}
7 changes: 7 additions & 0 deletions google/cloud/video/stitcher/v1/videostitcher_v1.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,13 @@ 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.*'
Expand Down

0 comments on commit cf01552

Please sign in to comment.