Open Stack

Simon,
structure of fuel-web repo is much more complex than fuel-qa, ~ 50 active
contributors work with fuel-web.
There is the functionality of the different Fuel domains and each requires
its own expertise, so maintenance is divided by folders.
In case of fuel-qa maintainers are doing review for whole repository,
structure of file[0] is correct.


Nastya.
[0] https://github.com/openstack/fuel-qa/blob/master/MAINTAINERS

On Wed, Oct 21, 2015 at 2:15 AM, Mike Scherbakov <mscherbakov at mirantis.com>
wrote:

> Simon,
> I believe that it's a mistake in fuel-qa. Valid structure is in fuel-web.
> Please fix the one in fuel-qa.
>
> I'm also looking forward for automated adding of people to review requests
> based on this file. Here is the task to track it:
> https://bugs.launchpad.net/fuel/+bug/1497655
>
> On Tue, Oct 20, 2015 at 2:10 AM Simon Pasquier <spasquier at mirantis.com>
> wrote:
>
>> Thanks for the reply, Andrew! I must admit that I haven't read thoroughly
>> the specification on the new team structure [1]. IIUC plugin developers
>> should be added to the MAINTAINERS file of fuel-qa for the directories that
>> concern their plugins. If I take LMA as an example, this would be:
>> fuelweb_test/tests/plugins/plugin_elasticsearch
>> fuelweb_test/tests/plugins/plugin_lma_collector
>> fuelweb_test/tests/plugins/plugin_lma_infra_alerting
>>
>> Is that right?
>>
>> I can submit a change to fuel-qa for adding the LMA team to the
>> MAINTAINERS file but I can't figure out the structure of the YAML data:
>> fuel-web/MAINTAINERS [2] is organized as "{directory1: [maintainer1,
>> maintainer2, ...], directory2: [...], ...}" while for fuel-qa [3] (and
>> other Fuel projects), it's "[maintainer1, maintainer2, ...]".
>>
>> BR,
>> Simon
>>
>> [1]
>> http://specs.fuel-infra.org/fuel-specs-master/policy/team-structure.html
>> [2] https://github.com/openstack/fuel-web/blob/master/MAINTAINERS
>> [3] https://github.com/openstack/fuel-qa/blob/master/MAINTAINERS
>>
>>
>> On Sat, Oct 17, 2015 at 2:21 AM, Andrew Woodward <xarses at gmail.com>
>> wrote:
>>
>>> We have already discussed this to be a result of describing data driven
>>> testing, untill this spec is completed there is little sense to remove all
>>> of these since fuel-qa is 100% required to operate this way. In the interim
>>> we should just specify the appropriate SME with the MAINTAINERS file.
>>>
>>> On Fri, Oct 16, 2015 at 11:34 AM Sergii Golovatiuk <
>>> sgolovatiuk at mirantis.com> wrote:
>>>
>>>> Tests should be in plugin
>>>>
>>>> --
>>>> Best regards,
>>>> Sergii Golovatiuk,
>>>> Skype #golserge
>>>> IRC #holser
>>>>
>>>> On Fri, Oct 16, 2015 at 5:58 PM, Simon Pasquier <spasquier at mirantis.com
>>>> > wrote:
>>>>
>>>>> Hello Alexey,
>>>>>
>>>>> On Fri, Oct 16, 2015 at 5:35 PM, Alexey Elagin <aelagin at mirantis.com>
>>>>> wrote:
>>>>>
>>>>>> Hello Simon!
>>>>>>
>>>>>> We are going to remove plugins' functional tests from fuel-qa because
>>>>>> this tests don't use for our plugins CI process.
>>>>>>
>>>>>
>>>>> And where are the existing tests going to be stored then?
>>>>>
>>>>> Thanks,
>>>>> Simon
>>>>>
>>>>>
>>>>>>
>>>>>>
>>>>>> __________________________________________________________________________
>>>>>> OpenStack Development Mailing List (not for usage questions)
>>>>>> Unsubscribe:
>>>>>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>>>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>>>>
>>>>>
>>>>>
>>>>>
>>>>> __________________________________________________________________________
>>>>> OpenStack Development Mailing List (not for usage questions)
>>>>> Unsubscribe:
>>>>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>>>
>>>>>
>>>>
>>>> __________________________________________________________________________
>>>> OpenStack Development Mailing List (not for usage questions)
>>>> Unsubscribe:
>>>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>>
>>> --
>>>
>>> --
>>>
>>> Andrew Woodward
>>>
>>> Mirantis
>>>
>>> Fuel Community Ambassador
>>>
>>> Ceph Community
>>>
>>>
>>> __________________________________________________________________________
>>> OpenStack Development Mailing List (not for usage questions)
>>> Unsubscribe:
>>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>>
>>>
>> __________________________________________________________________________
>> OpenStack Development Mailing List (not for usage questions)
>> Unsubscribe:
>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>>
> --
> Mike Scherbakov
> #mihgen
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20151021/28009ffb/attachment.html>