Hi Paul,
On 30 October 2016 at 11:47, Paul Sherwood <paul.sherwood(a)codethink.com> wrote:
On 2016-10-28 17:45, Javier Jardón wrote:
>>
>> [1]
>>
https://gitlab.com/baserock/definitions/commit/e2255bd601a12c9f6924874f2d...
>
>
> To clarify this a little more, the only change I'm requesting here is
> to stop mirroring the g.b.o definitions repo into
>
gitlab.com/baserock/definitions
> So to make a change in definitions, the process would as simple as:
>
> - Create branch in
gitlab.com/baserock/definition -> test branch in
> the gitlab ci -> Create Merge request -> review -> patch is applied to
> master
>
> (currently the patches have to be send to gerrit and then wait for the
> change to be mirrored in gitlab)
>
> I'd like to do this change on a week counting from today (so next
> Friday 4th of October)
> Let me know if you have any objection for this to not happen
I'm happy with your proposed change, but I think there are a few more
general implications:
1) as discussed on irc, gitlab services will require maintenance/security
updates etc
- review gitlab setup with infra team
- establish who will maintain, and how
No maintenance is required if we store the projects in
gitlab.com.
Some people seems to have interest on have our own gitlab instance but
Its not what Im suggesting here (although for sure It can be done in
the future if there is enough intesrest)
If we want to use our own runners, then the only maintenance required
is for the "manager" runner that will spin up the other runners on
demand.
So, the maintenance would be keep that machine updated + gitlab-runner
software up-to-date + toml config file (which should not change much)
(Ive attached it so you can have a look to it)
I already volunteer to do this work if nobody is available at the moment
2) new canonical upstream location for definitions
- abandon any in-progress patches/reviews in gerrit, or bring them across to
gitlab
Yeah, this should be easy; there are actually only a few patches being
actively on review at the moment
- update the wiki for contributors, and all references to definitions
Yes, we can do this over the weekend
- add a lorry file for definitions itself, to land at its new mirror
location, which i expect would be
https://git.baserock.org/delta/definitions
Sure, we can do this or use gitlab directly: you can push gitlab repos
to any other repo if you give gitlab the required credentials
3) other projects that have up to now been reviewed via the gerrit
-
https://gerrit.baserock.org/#/admin/projects/
- i assume the logical implication is to migrate those too?
Yeah, some project are already being mirrored [1] . Feel free to add
more if some active one is missing
- and then deprecate the gerrit, take down the service, decommission
whatever's hosting it?
Sure, this can be done when the migration is complete
4) turn
git.baserock.org into a strict mirror? or establish a new
mirror-only service?
- historically we've pushed branches to g.b.o (eg when we've had to patch
upstream for baserock builds)
- in discussion with users, some have expressed a preference that troves
would act as pure mirrors
- so any work we or they would need to do, would happen elsewhere (say on a
gitlab server), and thereafter be mirrored into the trove
Sure, we can discuss about this point later
Clearly your proposed transition could happen without resolving all
of the
above, but we should aim to minimise confusion and tidy things up as we go
if possible.
Agree, let me know if you have any other questions / concerns
Cheers,
Javier
[1]
https://gitlab.com/baserock/