New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 831750 link

Starred by 1 user

Issue metadata

Status: Untriaged
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

[cr-audit-commits] Audit refs for up to 24 hours after they have been superseded by a new branch.

Project Member Reported by robert...@chromium.org, Apr 11 2018

Issue description

This comes from stgao's observation in (go/audit-configuration-for-dynamic-refs) :

'There is a potential race condition here:
1. Audit App scan ref 1000.
2. A dev landed a new commit X in ref 1000.
3. Release eng cut a new branch, and ref becomes "old" as defined here.
4. Audit App scan again: if we ignore ref 1000, we would miss commit X there.

One potential solution is: for "old" refs here, still scan them if they are only 1 day old for example.'
 
Components: Infra>Audit

Sign in to add a comment