Allow specifying Team requirements

[dobey]

Tarmac should have a per-branch configuration option to specify a Team or Teams which an Author must be a member of in order for their code to land in the Target. For Canonical related projects which require the Contributor Agreement to be signed, this would allow us to have a team which includes all the users who have signed the agreement, which we could check against automatically.

[Paul Hummer (rockstar)]

Launchpad is going to support this soon.

[Paul Hummer (rockstar)]

So, I think that isPersonTrustedReviewer is probably what we want here. Maybe this is now a dupe of bug #426950. Rodney, what do you think?

[dobey (dobey)]

I'm not sure about that. Using isPersonTrustedReviewer probably can solve part of the issue. But the lp:~contributor-agreement-canonical team is a special case still. It's not a member of the teams which have direct access to the branches in question, nor should it be. It should itself not be a member of other teams. But it should be usable as a reference in an automatic manner, for some things.

This is not so much about reviews, but rather who is allowed to contribute code. But it should be easy to write a plug-in to handle this now. I think I'll write one real quick. :)

[dobey (dobey)]

This will depend on bug #397853 being fixed though.