mozilla / code-review

Automated static analysis & linting bot for Mozilla repositories
Mozilla Public License 2.0
55 stars 42 forks source link

try-comm-central code-review results not reported #2042

Open jfx2006 opened 8 months ago

jfx2006 commented 8 months ago

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.

marco-c commented 8 months ago

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?

jfx2006 commented 8 months ago

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.

marco-c commented 8 months ago

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

marco-c commented 8 months ago

@jfx2006 is it working now?

jfx2006 commented 8 months ago

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/