search

packit / hardly

Project is archived since Packit team agreed on prioritising work with more benefits and bigger user base.
https://packit.dev/source-git/status
MIT License
0 stars 7 forks source link

Add a log message to debug missing merge_request_url #74

Closed jpopelka closed 2 years ago

jpopelka commented 2 years ago

to know whether it was a flake or there's some pattern. packit_service/worker/parser.py#L1193 says that the merge_request can be missing in the original webhook payload if the source is push, but in the https://sentry.io/organizations/red-hat-0p/issues/3427461150 case the source was merge_request_event.

The trigger was https://gitlab.com/redhat/centos-stream/rpms/gnome-settings-daemon/-/pipelines/588479707 from https://gitlab.com/redhat/centos-stream/rpms/gnome-settings-daemon/-/merge_requests/7 which doesn't look anyhow special.

Unfortunately, we don't know what was in the payload, so let's add some debug logging to have some clues next time.

softwarefactory-project-zuul[bot] commented 2 years ago

Build succeeded.

:heavy_check_mark: pre-commit SUCCESS in 2m 19s :heavy_check_mark: hardly-tests SUCCESS in 3m 11s

softwarefactory-project-zuul[bot] commented 2 years ago

Build succeeded (gate pipeline).

:heavy_check_mark: pre-commit SUCCESS in 1m 24s