[Starlingx-discuss] TLs to sign off on initial core reviewer groups - ACTION NEEDED

Ildiko Vancsa ildiko.vancsa at gmail.com
Wed Nov 7 16:16:58 UTC 2018


Hi Scott,

On the technical side we had one starlingx-core group setup for the community in Gerrit. This group was assigned to every initial stx-* repo which gave everyone on that group +2 rights in all those repositories.

As we are moving forward and formalizing the sub-project teams and overall project governance we got to the steps of defining the core reviewer groups for each sub-project and the repositories they manage which is the change you are experiencing now.

It is not about demoting anybody but rather identifying those individuals for each sub-project and relevant repositories who have the duties of looking after the mission and scope of the sub-project as well as architecture and code quality by performing code reviews and being able to approve patches in Gerrit.

I believe StarlingX is already big and complex enough that we don’t expect anyone to be on top of what is happening in every single sub-project and repo on code-level but rather focus on one or a few sub-projects.

In case you feel that you are actively participating in some projects and should retain your core reviewer rights please reach out the the Technical Lead of the sub-project to discuss and fix this.

Please let me know if you have further questions.

Thanks and Best Regards,
Ildikó


> On 2018. Nov 7., at 16:58, Scott Little <scott.little at windriver.com> wrote:
> 
> Actually I had pretty broad +2/+1 power to most repos, at least initially, to facilitate repo creation and restructuring.  Roughly on par with Dean.  I wasn't that I was a core reviewer in the normal sense.  Just a backup to Dean that can do some of the structural work.
> 
> If I'm being demoted, fine, as long as that was the intent.
> 
> Scott
> 
> 
> On 18-11-05 05:46 PM, Ildiko Vancsa wrote:
>> Hi Scott,
>> 
>> I got the following initial list for stx-upstream:
>> 
>> stx-upstream - Upstream packages
>>   Bob Church (Robert.Church at windriver.com)
>>   Al Bailey (Al.Bailey at windriver.com)
>>   Chris Friesen (Chris.Friesen at windriver.com)
>> 
>> And so far it got confirmed here: http://lists.starlingx.io/pipermail/starlingx-discuss/2018-November/001742.html
>> 
>> I’m working with the TLs to double check the lists, however I don’t seem to be able to figure out all the TLs for all the repos at this point, will start a separate thread about that.
>> 
>> Thanks,
>> Ildikó
>> 
>> 
>>> On 2018. Nov 5., at 23:01, Scott Little <scott.little at windriver.com> wrote:
>>> 
>>> Strange,  I used to have +2 powers over stx-upstream a few days ago.  Now it seems to be gone.
>>> 
>>> Please restore.
>>> 
>>> Thanks
>>> 
>>> Scott
>>> 
>>> 
>>> 
>>> On 18-11-05 03:31 PM, McKenna, Jason wrote:
>>>> Hi Cindy,
>>>> 
>>>> I think I should be added to stx-integ as well.
>>>> 
>>>> Thanks in advance,
>>>> -Jason
>>>> 
>>>>> -----Original Message-----
>>>>> From: Saul Wold <sgw at linux.intel.com>
>>>>> Sent: November 5, 2018 3:07 PM
>>>>> To: Xie, Cindy <cindy.xie at intel.com>; Ildiko Vancsa
>>>>> <ildiko.vancsa at gmail.com>; starlingx-discuss at lists.starlingx.io
>>>>> Subject: Re: [Starlingx-discuss] TLs to sign off on initial core reviewer groups -
>>>>> ACTION NEEDED
>>>>> 
>>>>> 
>>>>> I agree with this, I also think I should be added to stx-root, stx-tools, stx-
>>>>> manifest and stx-tis-repo as part of the build.
>>>>> 
>>>>> Thanks
>>>>> 
>>>>> Sau!
>>>>> 
>>>>> 
>>>>> On 11/4/18 10:24 PM, Xie, Cindy wrote:
>>>>>> For stx-integ, this repo was used for non-openstack distro sub-project a lot.
>>>>> I am hoping that both Saul and Brent can be the core reviewers for it.
>>>>>> stx-integ - Integration and packaging
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Bob Church (Robert.Church at windriver.com)
>>>>>>     Don Penney (Don.Penney at windriver.com )
>>>>>>     Wold, Saul <saul.wold at intel.com>
>>>>>>     Rowsell, Brent <Brent.Rowsell at windriver.com>
>>>>>> 
>>>>>> Thx. - cindy
>>>>>> 
>>>>>> -----Original Message-----
>>>>>> From: Smith, Tyler [mailto:tyler.smith at windriver.com]
>>>>>> Sent: Friday, November 2, 2018 11:20 AM
>>>>>> To: Ildiko Vancsa <ildiko.vancsa at gmail.com>;
>>>>>> starlingx-discuss at lists.starlingx.io
>>>>>> Subject: Re: [Starlingx-discuss] TLs to sign off on initial core
>>>>>> reviewer groups - ACTION NEEDED
>>>>>> 
>>>>>> OK for stx-gui
>>>>>> 
>>>>>> Thanks,
>>>>>> Tyler
>>>>>> 
>>>>>> -----Original Message-----
>>>>>> From: Ildiko Vancsa [mailto:ildiko.vancsa at gmail.com]
>>>>>> Sent: Friday, November 2, 2018 1:37 PM
>>>>>> To: starlingx-discuss at lists.starlingx.io
>>>>>> Subject: [Starlingx-discuss] TLs to sign off on initial core reviewer
>>>>>> groups - ACTION NEEDED
>>>>>> 
>>>>>> Hi Technical Leads,
>>>>>> 
>>>>>> I started to work on creating the core reviewer groups in Gerrit for each of
>>>>> the sub-projects in StarlingX and I need your input to complete this task.
>>>>>> I have an initial list below containing the sub-project repos and the names
>>>>> of the core reviewers belonging to the groups. Please check your sub-project
>>>>> and respond with corrections or with an ‘OK’ in case the core team for your
>>>>> project is listed correctly.
>>>>>> Please let me know if you have any questions.
>>>>>> 
>>>>>> Thanks and Best Regards,
>>>>>> Ildikó
>>>>>> 
>>>>>> 
>>>>>> stx-clients - client libraries
>>>>>>     Jerry Sun (Jerry.Sun at windriver.com)
>>>>>>     Greg Waines (Greg.Waines at windriver.com)
>>>>>> 
>>>>>> stx-config - Configuration Management
>>>>>>     Bart Wensley (Barton.Wensley at windriver.com)
>>>>>>     John Kung (John.Kung at windriver.com)
>>>>>>     Don Penney (Don.Penney at windriver.com)
>>>>>>     Bob Church (Robert.Church at windriver.com)
>>>>>>     Chris Friesen (Chris.Friesen at windriver.com)
>>>>>> 
>>>>>> stx-fault - Fault Management
>>>>>>     Tao Liu (Tao.Liu at windriver.com)
>>>>>>     Al Bailey (Al.Bailey at windriver.com)
>>>>>> 
>>>>>> stx-integ - Integration and packaging
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Bob Church (Robert.Church at windriver.com)
>>>>>>     Don Penney (Don.Penney at windriver.com )
>>>>>> 
>>>>>> stx-gui - Horizon plugins for new services
>>>>>>     Tyler Smith (Tyler.Smith at windriver.com)
>>>>>>     Angie Wang (Angie.Wang at windriver.com)
>>>>>>     Kristine Bujold (Kristine.Bujold at windriver.com)
>>>>>> 
>>>>>> stx-ha - High Availability/Service Management
>>>>>>     Bin Qian (Bin.Qian at windriver.com)
>>>>>>     Eric Macdonald (Eric.MacDonald at windriver.com)
>>>>>>     John Kung (John.Kung at windriver.com)TLs to sign off on initial core reviewer groups
>>>>>> 
>>>>>> stx-metal - Bare Metal Management
>>>>>>     Eric Macdonald (Eric.MacDonald at windriver.com)
>>>>>>     John Kung (John.Kung at windriver.com)
>>>>>>     Tao Liu (Tao.Liu at windriver.com)
>>>>>> 
>>>>>> stx-nfv - NFVI Orchestration
>>>>>>     Bart Wensley (Barton.Wensley at windriver.com)
>>>>>>     Al Bailey (Al.Bailey at windriver.com)
>>>>>>     Jack Ding (Jack.Ding at windriver.com)
>>>>>> 
>>>>>> stx-update - Installation/Update/Patching
>>>>>>     Bart Wensley (Barton.Wensley at windriver.com)
>>>>>>     Don Penney (Don.Penney at windriver.com)
>>>>>>     David Sullivan (David.Sullivan at windriver.com)
>>>>>> 
>>>>>> stx-upstream - Upstream packages
>>>>>>     Bob Church (Robert.Church at windriver.com)
>>>>>>     Al Bailey (Al.Bailey at windriver.com)
>>>>>>     Chris Friesen (Chris.Friesen at windriver.com)
>>>>>> 
>>>>>> stx-distcloud – Distributed Cloud
>>>>>>     Bart Wensley (Barton.Wensley at windriver.com)
>>>>>>     Brent Rowsell brent.rowsell at windriver.com
>>>>>>     Chris Friesen (Chris.Friesen at windriver.com)
>>>>>>     Dean Troyer dtroyer at gmail.com
>>>>>>     John Kung john.kung at windriver.com
>>>>>>     Saul Wold sgw at linux.intel.com
>>>>>> 
>>>>>> stx-distcloud-client  – Distributed Cloud
>>>>>>     Bart Wensley (Barton.Wensley at windriver.com)
>>>>>>     Chris Friesen (Chris.Friesen at windriver.com)
>>>>>>     John Kung (John.Kung at windriver.com )
>>>>>> 
>>>>>> stx-manifest - source manifest to manage the repositories
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Jason McKenna (Jason.McKenna at windriver.com)
>>>>>> 
>>>>>> stx-root - build source tree root
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Jason McKenna (Jason.McKenna at windriver.com)
>>>>>> 
>>>>>> stx-tis-repo - build support
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Jason McKenna (Jason.McKenna at windriver.com)
>>>>>> 
>>>>>> stx-tools - build tools
>>>>>>     Scott Little (scott.little at windriver.com)
>>>>>>     Jason McKenna (Jason.McKenna at windriver.com)
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> _______________________________________________
>>>>>> Starlingx-discuss mailing list
>>>>>> Starlingx-discuss at lists.starlingx.io
>>>>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
>>>>>> _______________________________________________
>>>>>> Starlingx-discuss mailing list
>>>>>> Starlingx-discuss at lists.starlingx.io
>>>>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
>>>>>> _______________________________________________
>>>>>> Starlingx-discuss mailing list
>>>>>> Starlingx-discuss at lists.starlingx.io
>>>>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
>>>>>> 
>>>>> _______________________________________________
>>>>> Starlingx-discuss mailing list
>>>>> Starlingx-discuss at lists.starlingx.io
>>>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
>>>> _______________________________________________
>>>> Starlingx-discuss mailing list
>>>> Starlingx-discuss at lists.starlingx.io
>>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
>>> 
>>> 
>>> _______________________________________________
>>> Starlingx-discuss mailing list
>>> Starlingx-discuss at lists.starlingx.io
>>> http://lists.starlingx.io/cgi-bin/mailman/listinfo/starlingx-discuss
> 
> 




More information about the Starlingx-discuss mailing list