-
Notifications
You must be signed in to change notification settings - Fork 118
Generated code for LROs uses incorrect service #3125
Copy link
Copy link
Closed
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Milestone
Description
When generating mixins for LROs the gRPC service name and RPC name should be the names of the original service, and not the name of the target service.
For example:
let extensions = {
let mut e = tonic::Extensions::new();
e.insert(tonic::GrpcMethod::new(
- "google.storage.control.v2.StorageControl",
+ "google.longrunning.Operations",
"GetOperation",
));
e
};
let path = http::uri::PathAndQuery::from_static(
- "/google.storage.control.v2.StorageControl/GetOperation",
+ "/google.longrunning.Operations/GetOperation",
);Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.