Change Details

=Version Information= phabricator 2604c5af55f654d36f8db2f080b96486c4572216 (Sat, Jan 28) (branched from 1be3ef02276812296c01e41122f19d6ea8077f81 on origin) arcanist 9503b941cc02be637d967bb50cfb25f852e071e4 (Sat, Jan 7) (branched from ade25facfdf22aed1c1e20fed3e58e60c0be3c2b on origin) phutil 10963f771f118baa338aacd3172aaede695cde62 (Sat, Jan 14) (branched from 9d85dfab0f532d50c2343719e92d574a4827341b on origin) =Reproduction Steps= # Commit to a repository (tested with SVN and Git) and specify an Auditor using //Auditor: <username>// in the commit (I've tested it on commits that triggered audit this way as well as with Herald). # Find the audit ID via `bin/audit delete --dry-run`. # Delete the audit ID using `bin/audit delete --ids <id>'. =Expectect Result= "Audit Required" is completely removed from the commit, and the audit no longer appears when viewing the Audit application. =Actual Result= In the Audit application, Audits still exist under the **Waiting on Auditors** list with //Auditors: None// displayed under each audit that I had deleted. Viewing the actual commit in Diffusion, it still says "Audit Required", but there are no auditors listed. Is this the intended behavior? I can reproduce this every time on our server, which is a typical install.

=Version Information= phabricator 2604c5af55f654d36f8db2f080b96486c4572216 (Sat, Jan 28) (branched from 1be3ef02276812296c01e41122f19d6ea8077f81 on origin) arcanist 9503b941cc02be637d967bb50cfb25f852e071e4 (Sat, Jan 7) (branched from ade25facfdf22aed1c1e20fed3e58e60c0be3c2b on origin) phutil 10963f771f118baa338aacd3172aaede695cde62 (Sat, Jan 14) (branched from 9d85dfab0f532d50c2343719e92d574a4827341b on origin) =Reproduction Steps= # Commit to a repository (tested with SVN and Git) and specify an Auditor using //Auditor: <username>// in the commit (I've tested it on commits that triggered audit this way as well as with Herald). # Find the audit ID via `bin/audit delete --dry-run`. # Delete the audit ID using `bin/audit delete --ids <id>`. =Expectect Result= "Audit Required" is completely removed from the commit, and the audit no longer appears when viewing the Audit application. =Actual Result= In the Audit application, Audits still exist under the **Waiting on Auditors** list with //Auditors: None// displayed under each audit that I had deleted. Viewing the actual commit in Diffusion, it still says "Audit Required", but there are no auditors listed. Is this the intended behavior? I can reproduce this every time on our server, which is a typical install.

=Version Information= phabricator 2604c5af55f654d36f8db2f080b96486c4572216 (Sat, Jan 28) (branched from 1be3ef02276812296c01e41122f19d6ea8077f81 on origin) arcanist 9503b941cc02be637d967bb50cfb25f852e071e4 (Sat, Jan 7) (branched from ade25facfdf22aed1c1e20fed3e58e60c0be3c2b on origin) phutil 10963f771f118baa338aacd3172aaede695cde62 (Sat, Jan 14) (branched from 9d85dfab0f532d50c2343719e92d574a4827341b on origin) =Reproduction Steps= # Commit to a repository (tested with SVN and Git) and specify an Auditor using //Auditor: <username>// in the commit (I've tested it on commits that triggered audit this way as well as with Herald). # Find the audit ID via `bin/audit delete --dry-run`. # Delete the audit ID using `bin/audit delete --ids <id>'id>`. =Expectect Result= "Audit Required" is completely removed from the commit, and the audit no longer appears when viewing the Audit application. =Actual Result= In the Audit application, Audits still exist under the **Waiting on Auditors** list with //Auditors: None// displayed under each audit that I had deleted. Viewing the actual commit in Diffusion, it still says "Audit Required", but there are no auditors listed. Is this the intended behavior? I can reproduce this every time on our server, which is a typical install.