-
Notifications
You must be signed in to change notification settings - Fork 10
Description
When there is an error (eg. 404 returned from GH) there are two error records sent to Sentry.
-
One for the actual problem:
logged here: https://github.com/arquillian/ike-prow-plugins/blob/master/pkg/plugin/test-keeper/event_handler.go#L189
the related issue in the sentry https://errortracking.prod-preview.openshift.io/openshift_io/ike-prow-plugins/issues/5663/ -
And as the error is then returned back, then there is the second record sent to the Sentry from this place: https://github.com/arquillian/ike-prow-plugins/blob/master/pkg/plugin/test-keeper/event_handler.go#L44 (but will be from server.go when this PR is merged). However, this record doesn't provide meaningful information - here is an example of such a sentry issue https://errortracking.prod-preview.openshift.io/openshift_io/ike-prow-plugins/issues/5667/
So, change the logging sent to Sentry so the records/issues provide more meaningful information.