commit | e932ace4e465dddb32a5c9685ea1175c47981eb0 | [log] [tgz] |
---|---|---|
author | Roberto Carrillo <robertocn@google.com> | Fri Apr 20 18:37:02 2018 |
committer | Commit Bot <commit-bot@chromium.org> | Fri Apr 20 22:29:19 2018 |
tree | eda8043c310be2fd418fffc01292c200245bc0bd | |
parent | a875517db7b8099c7f2bfdb54c5120e7501a27f0 [diff] |
[cr-audit-commits] Use a taskqueue to run audits. Also, use a single cron job to schedule the audit tasks. To be clear, this new approach involves two phases (Scheduler Cron Job, and Auditor Task Queue) Phase I - Scheduler Cron Job A new handler called Scheduler is added to iterate over the available configs and schedule needed audit tasks, creating new RepoState datastore entries as needed. Note that the URL to each ref is passed to the taskqueue task (Auditor handler below), and that the name of the corresponding config has been added as a field to the RepoState entity. Phase II - Auditor Task Queue This change replaces the CommitScanner and CommitAuditor handlers with a single handler called Auditor, intended to be run as a taskqueue task when scheduled by the 'Scheduler' cron job above. This task will scan the git ref, whose url is given as a parameter and using the matching configuration (from the hardcoded RuleMap) it will create RelevantCommit entities when appropriate, followed by an audit of such commits and notifications for those that are found to be in violation of any of the rules. The following design docs contain the discussions about this change: - go/audit-configuration-for-dynamic-refs - go/syncing-audit-app-scheduler-with-audit-task R=stgao,chanli BUG=803314 Change-Id: I1b7f52fa68b41db9cc5575f2a42041afde36443c Reviewed-on: https://chromium-review.googlesource.com/1010976 Commit-Queue: Roberto Carrillo <robertocn@chromium.org> Reviewed-by: Shuotao Gao <stgao@chromium.org>
Welcome to the Chrome Infra repository!
Wondering where to start? Check out General Chrome Infrastructure documentation. In particular, to check out this repo and the rest of the infrastructure code, follow the instructions here. The rest of this page is specific to this repo.
sys.path
modifications.from infra.path_hacks.common import <stg>
is actually getting <stg>
from build/scripts/common.git push origin <updated hash>:deployed
git push
commandIf you've added a new module, integrate your tests with test.py:
Double-check that your tests are getting picked up when you want them to be: ./test.py test <path-to-package>
.
Tests still not getting picked up by test.py? Double-check to make sure you have init.py files in each directory of your module so Python recognizes it as a package.