-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Add Error Reporting gRPC #2687
Copy link
Copy link
Closed
Labels
api: clouderrorreportingIssues related to the Error Reporting API.Issues related to the Error Reporting API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.
Metadata
Metadata
Assignees
Labels
api: clouderrorreportingIssues related to the Error Reporting API.Issues related to the Error Reporting API.priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.
This is a tracking work item to get the Error Reporting GAPIC into this library:
https://pypi.python.org/pypi/gapic-google-cloud-error-reporting-v1beta1
@steren and I agree that the existing (small)
reportandreport_exceptionis what we want to keep, along with possible more framework integrations in the future. So I think what we primarily want is the same thing as with done with Logging where we add the GAPIC as a dependency, and then fallback to the existing code if gRPC is disabled.One small question is if and to what extent we want to expose the rest of the GAPIC besides events.report via this library. There are a few API calls that will likely only be used by power users, but it might be worth having some sort of pass-through mechanism. Unless I'm mistake, it looks like so far we either wrap all the GAPIC calls or pass-through completely, but we haven't done both, so not sure exactly how this would look.
cc @bjwatson @jgeewax @omaray in case you have comments.