Revert "presubmit: Skip owners checks if code-owners plugin is enabled."
This reverts commit 2cf835a9bad9ef5bfe477719d3376eccab3cac68.
Reason for revert: doesn't work with depot_tools
error 404 Not found: tools (with applied patch from 2729404)
Bug: 1183975
Original change's description:
> presubmit: Skip owners checks if code-owners plugin is enabled.
>
> If code-owners plugin is enable for the repo, skip owners check on
> commit, and skip checking owners format, as that will be done by
> the plugin.
>
> Change-Id: I1663baef4f0f27b00423071343fe740f6da50ce7
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2727131
> Auto-Submit: Edward Lesmes <ehmaldonado@chromium.org>
> Reviewed-by: Gavin Mak <gavinmak@google.com>
> Commit-Queue: Edward Lesmes <ehmaldonado@chromium.org>
Change-Id: Id4d560701e4b0144e0aafc5263e09ed6927f6222
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/2729409
Auto-Submit: Josip Sokcevic <sokcevic@google.com>
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
diff --git a/gerrit_util.py b/gerrit_util.py
index 1b6bca6..7704e23 100644
--- a/gerrit_util.py
+++ b/gerrit_util.py
@@ -771,20 +771,13 @@
'in change %s' % change)
-def IsCodeOwnersEnabledOnHost(host):
+def IsCodeOwnersEnabled(host):
"""Check if the code-owners plugin is enabled for the host."""
path = 'config/server/capabilities'
capabilities = ReadHttpJsonResponse(CreateHttpConn(host, path))
return 'code-owners-checkCodeOwner' in capabilities
-def IsCodeOwnersEnabledOnRepo(host, repo):
- """Check if the code-owners plugin is enabled for the repo."""
- path = '/projects/%s/code_owners.project_config' % repo
- config = ReadHttpJsonResponse(CreateHttpConn(host, path))
- return config['status'].get('disabled', False)
-
-
def GetOwnersForFile(host, project, branch, path, limit=100,
resolve_all_users=True, o_params=('DETAILS',)):
"""Gets information about owners attached to a file."""