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

try-comm-central code-review results not reported

Open jfx2006 opened this issue 2 years ago • 11 comments

It looks like the bot needs to know about the new comm-unified repository.

https://firefox-ci-tc.services.mozilla.com/tasks/ChccDnINRpWxOCdRPstHdg/runs/0/logs/public/logs/live.log

2024-01-16 19:04:47.000265 [WARNING ] Backend rejected the payload: b'{"base_repository":["Object with url=https://hg.mozilla.org/comm-unified does not exist."]}'
2024-01-16 19:04:47.000275 [ERROR   ] Static analysis failure | extra={'revision': PHID-DIFF-jonkpeqnid5d46vgdawy, 'error': AssertionError('Stopping early because revision could not be created nor retrieved from the backend')

COMM_BASE_REPOSITORY is now set to use https://hg.mozilla.org/comm-unified.

jfx2006 avatar Jan 16 '24 19:01 jfx2006

Here's the current config:

    - checkout: robust
      try_url: ssh://hg.mozilla.org/try-comm-central
      try_mode: json
      try_name: try-comm-central
      name: comm-central
      ssh_user: reviewbot
      url: https://hg.mozilla.org/comm-central
      decision_env_revision: COMM_HEAD_REV
      decision_env_repository: COMM_HEAD_REPOSITORY
      decision_env_prefix: COMM

Should we only change https://hg.mozilla.org/comm-central to https://hg.mozilla.org/comm-unified?

marco-c avatar Jan 18 '24 14:01 marco-c

I am not sure. comm-unified is built the same way as mozilla-unified so it has all of the revisions from comm-central, c-beta, c-release, and c-esr*. With the exception of comm-esr115, all of the CI jobs are now using comm-unified as COMM_BASE_REPOSITORY instead of comm-central.

I'm guessing what needs to happen based on the error message in the log.

jfx2006 avatar Jan 18 '24 15:01 jfx2006

Done, we should see it working after the next restart of the service.

marco-c avatar Jan 18 '24 19:01 marco-c

@jfx2006 is it working now?

marco-c avatar Jan 22 '24 10:01 marco-c

As of earlier today it's not working.

If I query the backend I don't see comm-unified. https://api.code-review.moz.tools/v1/repository/

jfx2006 avatar Jan 23 '24 21:01 jfx2006

Hey @marco-c Could you take a look at this again? It seems it didn't get resolved.

coreycb avatar Oct 29 '24 13:10 coreycb

@coreycb we're working on a few improvements for the bot, we will investigate this again soon.

marco-c avatar Oct 30 '24 09:10 marco-c

I see the following in the logs:

Oct 29 22:26:24 [34.19.26.8](https://my.papertrailapp.com/systems/14229533590/events?focus=1788046735835279360&selected=1788046735835279360) [code-review/production/bot](https://my.papertrailapp.com/events?q=program%3Acode-review%2Fproduction%2Fbot&focus=1788046735835279360&selected=1788046735835279360) 2024-10-29 22:26:24 backend.py: [warning  ] Backend rejected the payload: b'{"base_repository":["Object with url=https://hg.mozilla.org/comm-unified does not exist."]}'

marco-c avatar Oct 31 '24 10:10 marco-c

We likely need to add the repo to the DB, just like #985 (and we should finally document this better, see #988).

marco-c avatar Oct 31 '24 11:10 marco-c