launchpad-reviewers team mailing list archive

[Jürgen Gmach <mp+472970@xxxxxxxxxxxxxxxxxx>]

Thanks for the review! I was not aware how pre-requisite MPs work in detail when I created this MP, so I missed to set one for this MP. As you noticed, the last commit has the relevant changes. I spoke with Guruprasad about that, and he mentioned that it is not necessary to recreate the MP, as you cannot set a pre-req branch after it was created (on purpose :-().

Sorry, I should have made that more explicit.

Diff comments:

> diff --git a/database/schema/security.cfg b/database/schema/security.cfg
> index 2c23fd6..a906e9a 100644
> --- a/database/schema/security.cfg
> +++ b/database/schema/security.cfg
> @@ -2917,3 +2917,34 @@ public.teammembership                   = SELECT
>  public.teamparticipation                = SELECT
>  public.webhook                          = SELECT
>  public.webhookjob                       = SELECT, INSERT
> +
> +[rock-build-job]

That code is not part of this MP, see above.

> +type=user
> +groups=script
> +public.account                          = SELECT
> +public.builder                          = SELECT
> +public.buildfarmjob                     = SELECT, INSERT
> +public.buildqueue                       = SELECT, INSERT, UPDATE
> +public.rockfile                         = SELECT
> +public.rockrecipe                       = SELECT, UPDATE
> +public.rockrecipebuild                  = SELECT, INSERT, UPDATE
> +public.rockrecipebuildjob               = SELECT, UPDATE
> +public.rockrecipejob                    = SELECT, UPDATE
> +public.distribution                     = SELECT
> +public.distroarchseries                 = SELECT
> +public.distroseries                     = SELECT
> +public.emailaddress                     = SELECT
> +public.gitref                           = SELECT
> +public.gitrepository                    = SELECT
> +public.job                              = SELECT, INSERT, UPDATE
> +public.libraryfilealias                 = SELECT
> +public.libraryfilecontent               = SELECT
> +public.person                           = SELECT
> +public.personsettings                   = SELECT
> +public.pocketchroot                     = SELECT
> +public.processor                        = SELECT
> +public.product                          = SELECT
> +public.teammembership                   = SELECT
> +public.teamparticipation                = SELECT
> +public.webhook                          = SELECT
> +public.webhookjob                       = SELECT, INSERT


-- 
https://code.launchpad.net/~jugmac00/launchpad/+git/launchpad/+merge/472970
Your team Launchpad code reviewers is requested to review the proposed merge of ~jugmac00/launchpad:delete-rock-recipe-builds-and-jobs-when-deleting-recipes into launchpad:master.