cmasso@: could you fix the auditing rules asap for Chrome release branches? If not, we'd better disable the auditing on release branches by end of day.

This is spamming Monorail.
There were 1700+ bugs filed, and I suspected that many of them are false positives like the ones reported here

https://bugs.chromium.org/p/chromium/issues/list?can=1&q=reporter%3Acr-audit-commits%40appspot.gserviceaccount.com+label%3AMerge-Without-Approval&colspec=ID+Pri+M+Stars+ReleaseBlock+Component+Status+Owner+Summary+OS+Modified&x=m&y=releaseblock&cells=ids

We should be getting fewer and few bugs as of late. Unfortunately, we can only test things were they are in production. 

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/5bb0353799c5255f2707fe75e35064733dde4152

commit 5bb0353799c5255f2707fe75e35064733dde4152
Author: Claude <cmasso@google.com>
Date: Wed Aug 08 21:27:03 2018

Replace ListIssues with GetIssue RPC

Bug:872358
Change-Id: If72c70f8b60723dfe4faa975af14d7e0893689d6
Reviewed-on: https://chromium-review.googlesource.com/1167963
Commit-Queue: Estelle Yomba <cmasso@chromium.org>
Reviewed-by: Nodir Turakulov <nodir@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/5bb0353799c5255f2707fe75e35064733dde4152/go/src/infra/appengine/cr-audit-commits/app/utils.go
[modify] https://crrev.com/5bb0353799c5255f2707fe75e35064733dde4152/go/src/infra/appengine/cr-audit-commits/app/merge_approval_rules.go
[modify] https://crrev.com/5bb0353799c5255f2707fe75e35064733dde4152/go/src/infra/appengine/cr-audit-commits/app/merge_approval_rules_test.go

The issue I am hitting now is the audit app not able to view security bugs

"Found an invalid Monorail bug %!s(int=863069) on relevant commit d8474edf8dc7e6f479937bb67fd191366493315b :: {"error":"unexpected status \"403 Forbidden\". Response: {\"error\": {\"message\": \"User is not allowed to view this  issue chromium:863069 \"}}"}" 

I've talked to Estelle, we can make the rule that is failing set the status to ruleSkipped while we get the audit app permissions to see these bugs.

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/5502b27883a180bca19e544f8d8e16efe40c2ab8

commit 5502b27883a180bca19e544f8d8e16efe40c2ab8
Author: Claude <cmasso@google.com>
Date: Thu Aug 09 17:52:41 2018

Skip auditing commits with security crbugs

Bug:872358
Change-Id: I52ef4e6430fecd54a31f3211ec6879eacd4fecb4
Reviewed-on: https://chromium-review.googlesource.com/1168462
Commit-Queue: Roberto Carrillo <robertocn@chromium.org>
Reviewed-by: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/5502b27883a180bca19e544f8d8e16efe40c2ab8/go/src/infra/appengine/cr-audit-commits/app/merge_approval_rules.go