On Tue, Dec 6, 2011 at 2:07 PM, William Grant <email address hidden> wrote:
> +activereviews was failing to show a merge proposal for
> lp:~michael.nelson/software-center-agent/scaclient-add-distroseries-
> filter into lp:~canonical-isd-hackers/software-center-agent/scaclient.
> ~canonical-isd-hackers is subscribed to the proposed branch, owns the
> target branch, and has a requested review on the merge proposal.
>
> The problem is that noodles' branch is stacked on lp:software-center-
> agent, which used to be lp:~canonical-isd-hackers/software-center-
> agent/trunk, but was recently moved to lp:~canonical-ca-hackers
> /software-center-agent/trunk -- its owner was changed. Access
> transitivity over stacking results in ~canonical-isd-hackers not being
> able to see the source branch, nor the MP.
>
> To fix the immediate +activereviews issue, subscribe ~canoncial-isd-
> hackers to lp:software-center-agent. If ISD shouldn't have access any
> more, the project's branch visibility policies will need tweaking to
> point at ~canonical-ca-hackers instead.
>
> The bug here is that the visibility checks in LP's branch queries don't
> include the transitive stacking check. WITH RECURSIVE ftw?
On Tue, Dec 6, 2011 at 2:07 PM, William Grant <email address hidden> wrote: isd-hackers is subscribed to the proposed branch, owns the center- agent/trunk -- its owner was changed. Access isd-hackers not being ca-hackers instead.
> +activereviews was failing to show a merge proposal for
> lp:~michael.nelson/software-center-agent/scaclient-add-distroseries-
> filter into lp:~canonical-isd-hackers/software-center-agent/scaclient.
> ~canonical-
> target branch, and has a requested review on the merge proposal.
>
> The problem is that noodles' branch is stacked on lp:software-center-
> agent, which used to be lp:~canonical-isd-hackers/software-center-
> agent/trunk, but was recently moved to lp:~canonical-ca-hackers
> /software-
> transitivity over stacking results in ~canonical-
> able to see the source branch, nor the MP.
>
> To fix the immediate +activereviews issue, subscribe ~canoncial-isd-
> hackers to lp:software-center-agent. If ISD shouldn't have access any
> more, the project's branch visibility policies will need tweaking to
> point at ~canonical-
>
> The bug here is that the visibility checks in LP's branch queries don't
> include the transitive stacking check. WITH RECURSIVE ftw?
I thought rvba recently overhauled this?
-Rob