[Tarantool-patches] [PATCH v1] Set full testing for all branches
Alexander Turenko
alexander.turenko at tarantool.org
Wed Jun 3 15:29:32 MSK 2020
I have no more objections.
If everything works as expected and we're ready to such amount of jobs,
the change will allow to catch some problems before a patch will land
into master.
BTW, please add 'gitlab-ci' prefix.
> > > .pack_only_template: &pack_only_definition
> > > only:
> > > - schedules
> > > - external_pull_requests
> > > - merge_requests
> > > - - /^.*-full-ci$/
> > > + - branches
> > > + - tags
> >
> > First, there is not reason to list 'schedules', 'external_pull_requests'
> > and so if you want to just run it everywhere.
> >
>
> Actually I just wanted to make it more visible for the users, but anyway
> if it not really needed I'll change it to "except: master".
>
> > Second, we should not run both 'just packaging' and 'packaging and
> > deployment' jobs on master. I would make ''pack_only_template' be just
> > 'except: master'. Yep, it is not perfect: 'schedules',
> > 'external_pull_requests' will don't run neither 'pack', nor 'deploy'.
> > But this change is simple and downsides are acceptable.
> >
>
> Right, it is the same as previous my comment.
I would mention the known downside somewhere, preferably just in this
config.
More information about the Tarantool-patches
mailing list