Page MenuHomePhabricator

Herald adding different auditors but only the last one is kept
Closed, ResolvedPublic

Assigned To
Authored By
nicolast
Feb 21 2017, 3:56 PM
Referenced Files
F3120241: 3.png
Feb 21 2017, 3:56 PM
F3120248: 4.png
Feb 21 2017, 3:56 PM
F3120236: 1.png
Feb 21 2017, 3:56 PM
F3120239: 2.png
Feb 21 2017, 3:56 PM
F3120250: 5.png
Feb 21 2017, 3:56 PM

Description

Hi;

This seems to be related to https://secure.phabricator.com/T10978
We update from master weekly.

We have Herald rules in place that add auditors to commits, for example:

  1. [1.png]
    1.png (516×588 px, 25 KB)
  2. [2.png]
    2.png (498×516 px, 20 KB)
  3. [3.png]
    3.png (489×516 px, 20 KB)

Until now, some commits were affected by those 3 rules and therefore 3 different auditors were added.

After the update to latest version last week, the following happens:

[4.png]

4.png (758×1 px, 70 KB)

As you can see, Herald indicates that it's adding the different auditors but only the last one is kept.

Here you can see the Herald transcript:

[5.png]

5.png (596×594 px, 38 KB)

This is happening in every case where there should be more than one auditor.

Let me know if I can provide more details.

Thanks

Revisions and Commits

Restricted Differential Revision

Event Timeline

epriestley added a revision: Restricted Differential Revision.Feb 23 2017, 10:44 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Feb 23 2017, 11:15 PM
epriestley added a commit: Restricted Diffusion Commit.

Thanks for the report! This should be fixed in HEAD of master, and promote to stable tomorrow, a little more than 24 hours from now. Let us know if you run into any issues.

Also, thanks for the clear, concise reproduction instructions: they were instrumental in reproducing and fixing this problem quickly.