#819 frontend: pagure_flag() to not spam log.info()
Merged 4 years ago by praiskup. Opened 4 years ago by praiskup.

@@ -1009,9 +1009,9 @@

              'uid': str(build.id),

          }

  

-         log.info('Sending data to Pagure API: %s', pprint.pformat(data))

+         log.debug('Sending data to Pagure API: %s', pprint.pformat(data))

          response = requests.post(api_url, data=data, headers=headers)

-         log.info('Pagure API response: %s', response.text)

+         log.debug('Pagure API response: %s', response.text)

  

      @classmethod

      def cancel_build(cls, user, build):

That pagure request/response are pretty large dictionaries and (by
default) it was just unnecessarily bloat the log size. The info
might still be useful, but on request (debug enabled).

rebased onto a47186e

4 years ago

(some rant, ... of course we shouldn't even use frontend for this communication...)

Pull-Request has been merged by praiskup

4 years ago