Maniphest T9491

When packages are archived, packages shouldn't be listed as auditor
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	revi
	Oct 1 2015, 11:30 AM

Tags

Referenced Files

None

Subscribers

Description

steps to reproduce

Make a Diffusion repo, and an Owners package.
Enable auditing.
Make a few commits.
Now, archive the package.

Expected behaviors

When new commit is pushed, package is not shown as auditor.

what I see

Package is still added as an auditor.

I tried disabling auditing, but it is still listing package as auditor.

Related Objects

Mentioned In: T9946: Not adding archived owner packages as auditors on audits
Mentioned Here: D15907: Don't trigger audits for archived packages

Event Timeline

revi created this task.Oct 1 2015, 11:30 AM

revi updated the task description. (Show Details)

revi added a project: Owners.

revi added a subscriber: revi.

cburroughs added a subscriber: cburroughs.Oct 1 2015, 11:47 AM

joshuaspence added a subscriber: joshuaspence.Oct 2 2015, 8:53 AM

epriestley mentioned this in T9946: Not adding archived owner packages as auditors on audits.Dec 10 2015, 1:30 PM

epriestley merged a task: T9946: Not adding archived owner packages as auditors on audits.

epriestley added subscribers: epriestley, JohnHeroHD.

epriestley merged a task: T10482: audit-disabled & archived packages still show up on commits.Mar 1 2016, 3:50 PM

cburroughs added a project: Restricted Project.Apr 20 2016, 3:32 PM

cburroughs moved this task from Restricted Project Column to Restricted Project Column on the Restricted Project board.Apr 20 2016, 3:57 PM

Archived packages are no longer added as auditors to new commits after D15907 by the "Auditing Enabled" setting.

Existing commits which already had audits triggered will retain their audit status. This is by design, and consistent with other objects (disabling a user or project doesn't remove them as an auditor, reviewer, or subscriber from objects, for example).