fedora-ci / monitor-gating

#24 Ensure the API returns a valid code before proceeding and add missing import
Closed by asaleh. Opened by pingou.
fedora-ci/ pingou/monitor-gating staging  into  master

Download 24.patch
no initial comment

The first commit needs to be removed before this PR can be merged, only the other commits are of interest here.

Metadata Update from @nphilipp:
- Request assigned

Looks good to me, but as discussed let's leave this open until it got some more testing.

1 new commit added

  • When notifying on errors, don't send the raw error object, send its text version

1 new commit added

  • Flush some prints

1 new commit added

  • Add some more debugging

2 new commits added

  • Fix typo in the rpmdev-bumpspec name
  • Fix appending an element to a list rather than trying to call the list

1 new commit added

  • Be more specific about the error

9 new commits added

  • Be more specific about the error
  • Fix typo in the rpmdev-bumpspec name
  • Fix appending an element to a list rather than trying to call the list
  • Add some more debugging
  • Flush some prints
  • When notifying on errors, don't send the raw error object, send its text version
  • Add missing import to the runner.py
  • Check that the request return a 2xx code before proceeding
  • Add more debugging

1 new commit added

  • Fix field name when opening an issue on pagure

1 new commit added

  • Fix returning the MonitorUtils object

rebased onto ad0f320d8481659c24a2b22bfbba12c1eec6d3df

Pull-Request has been closed by asaleh
Metadata
nphilipp
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.