Skip to content

repo-updater: Include owners when syncing external service

Warren Gifford requested to merge rs/include-extsvc-owners into main

Created by: ryanslade

We were sending the owners in the sync request but not including them when creating the external services struct. This meant that it would appear as a site owned external service.

At the very least, this is causing us to log metrics against the wrong external service and it may be causing bugs in other areas of the code base.

Thanks @unknwon for spotting this.

Test plan

Confirm that our metrics bug is fixed after deployment

Merge request reports

Loading