[IMP] runbot_merge, forwardport: update some logging

Downgrade an error and a warning to info, and upgrade two warnings to
error.

Point is to improve logic of log levels & sentry visibility.

Fixes #258
This commit is contained in:
Xavier Morel 2021-01-13 15:48:39 +01:00
parent 95e35dde90
commit ba1a8ee089
3 changed files with 4 additions and 4 deletions

View File

@ -74,7 +74,7 @@ class Project(models.Model):
'Authorization': 'token %s' % project.fp_github_token
})
if not (r0.ok and r1.ok):
_logger.warning("Failed to fetch bot information for project %s: %s", project.name, (r0.text or r0.content) if not r0.ok else (r1.text or r1.content))
_logger.error("Failed to fetch bot information for project %s: %s", project.name, (r0.text or r0.content) if not r0.ok else (r1.text or r1.content))
continue
project.fp_github_name = r0.json()['login']
project.fp_github_email = next((
@ -587,7 +587,7 @@ class PullRequests(models.Model):
notarget = [p.repository.name for p in self if not p.repository.fp_remote_target]
if notarget:
_logger.warning(
_logger.error(
"Can not forward-port %s: repos %s don't have a remote configured",
self, ', '.join(notarget)
)

View File

@ -160,7 +160,7 @@ def handle_pr(env, event):
pr_obj = env['runbot_merge.pull_requests']._get_or_schedule(r, pr['number'])
if not pr_obj:
_logger.warning("webhook %s on unknown PR %s#%s, scheduled fetch", event['action'], repo.name, pr['number'])
_logger.info("webhook %s on unknown PR %s#%s, scheduled fetch", event['action'], repo.name, pr['number'])
return "Unknown PR {}:{}, scheduling fetch".format(repo.name, pr['number'])
if event['action'] == 'synchronize':
if pr_obj.head == pr['head']['sha']:

View File

@ -1691,7 +1691,7 @@ class Stagings(models.Model):
})
def fail(self, message, prs=None):
_logger.error("Staging %s failed: %s", self, message)
_logger.info("Staging %s failed: %s", self, message)
prs = prs or self.batch_ids.prs
prs.write({'state': 'error'})
for pr in prs: