Those commits are not expected to trigger an audit after D17122 / T10312. The first line of commit messages is now always parsed as a title, even if it begins "Auditors: ...", "Tests: ...", "Reviewers: ...", etc.

A commit like rGITTEST702cd23a, which has "Auditors" elsewhere than the title line, is expected to trigger an audit; it did so.

Does that explain what you're seeing locally?

No, not yet... In those examples, I just happened to have Auditors on the first line. Usually, it's not on the first line for us.

I'm trying to see if I can replicate on secure.phabricator.com. Locally, this is with an SVN repo. I just did this rSVNTEST9, and it worked (despite it taking quite a while to process). I'll try a few more things...

Ahhh... Before both "Auditor: ..." and "Auditors: ..." worked. Now it appears that only "Auditors: ..." works. I must not have been trying it with "Auditors: ..." at all.

Side note: are all of the recognized labels documented in one place?

Thanks

Thanks! I think this is the timeline of events:

• Until recently, Differential and Diffusion had different code to parse "Auditors:".
• I think Differential never accepted "Auditor", while Diffusion did.
• Somewhat recently, in connection with T11114, code for parsing commit messages was made more general.
• Very recently, in D17262, the custom ad-hoc code for parsing "Auditors" was unified, so both Differential and Diffusion use the same code. As a side effect, "Auditor" stopped being recognized.

D17306 should fix this by restoring "Auditor" as an alias for the "Auditors" field.

Side note: are all of the recognized labels documented in one place?

Not currently. Which labels are available depends on configuration and third-party code, so we'd have to write a dynamic page showing labels for the current viewer/install/config, not just a static piece of documentation. This would rarely help solve problems and there isn't a great place to shove it in the UI. T9713 may improve clarity here while making more concrete UI improvements.

Gotcha... Thanks for the explanation. Initially, I wasn't exactly sure what was going on... I sent an email out telling folks to use "Auditor: ..." in their commit messages, upgraded to clean up what I mentioned here T12181: Bulk Deleting Audits does not remove "Audit Required" from Commit, and then the "Auditor: ..." tag stopped working when our team started using it. Appreciate the fix!

This should now be resolved at HEAD of master, and should promote to stable later today.

Note that this isn't retroactive: after upgrading, "Auditor" should work for new commits, but won't apply to existing commits made before the upgrade.

Let us know if this doesn't fix things or you're still seeing issues. Thanks for the report!