code-review icon indicating copy to clipboard operation
code-review copied to clipboard

Fix or silence most common SENTRY issues

Open Archaeopteryx opened this issue 1 year ago • 6 comments

From code-review-bot

Archaeopteryx avatar Jul 16 '24 09:07 Archaeopteryx

Most common issues are currently:

A lot of warnings in the bot could also be silenced by simply lowering the log level to info

I resolved this one which seemed to be a Taskcluster issue over the past week (no new events)

La0 avatar Jul 30 '24 10:07 La0

The first one, missing m-c repo, is only happening on testing. Testing has the following in the list of repositories:

    - checkout: robust
      try_url: ssh://hg.mozilla.org/try
      try_mode: json
      try_name: try
      name: m-c
      ssh_user: reviewbot
      url: https://hg.mozilla.org/conduit-testing/m-c/
      decision_env_revision: GECKO_HEAD_REV
      decision_env_repository: GECKO_HEAD_REPOSITORY
      decision_env_prefix: GECKO

The problem is that this repo is only available on Phabricator dev, and at the moment testing is pointing at Phabricator prod.

marco-c avatar Jul 30 '24 14:07 marco-c

I filed 2 issues for @vrigal on the bot, and removed the MC repo from testing secret, then restarted code-review events testing o, heroku. This sentry issue is now marked as resolved

La0 avatar Aug 27 '24 12:08 La0

I just noticed another config problem (on this task):

2024-08-29 12:17:48.000824 [WARNING ] [warning  ] Backend rejected the payload: b'{"base_repository":["Object with url=https://hg.mozilla.org/comm-unified does not exist."]}'

I do not know if we want to track this repo or not @marco-c @Archaeopteryx

La0 avatar Aug 29 '24 15:08 La0

comm-unified is similar to mozilla-unified would be by assumption - and development with mozilla-unified i prefered compare to a checkout of every stage of the development cycle/ESR branch (central, beta, release, esrXXX).

I am in favor of supporting it.

Archaeopteryx avatar Aug 30 '24 18:08 Archaeopteryx

comm-* used to work, but I think it is not working anymore, see #2042. We should fix that, and it will likely make this error disappear.

marco-c avatar Sep 17 '24 08:09 marco-c