[sdk][congress][octavia][designate][magnum][senlin][swift] Adding project-specific cores to SDK
Heya, I’d like to try something. I’d like to try adding some project-specific people to the core team so that they can more directly help maintain the support for their service in SDK. In some of these cases the person I’m suggestion has next to no review experience in SDK. I think let’s be fine with that for now - we’re still a 2x +2 in general thing - but I know currently when reviewing neutron or ironic changes I always want to see a +2 from slaweq or dtantsur … so in the spirit of trying new things and trying to move the project forward in a healthy and welcoming way - how about we give this a try? The idea here is that we’re trusting people to use their good judgement and to only use their new +2 powers for good in their project. Over time, if they feel like they’ve gotten a handle on things more widely, there’s nothing stopping them from reviewing other patches - but I think that most of us aren’t looking for additional review work anyway. Specifically this would be: Shogo Saito - congress Adam Harwell - octavia Graham Hayes - designate Bharat Kumar - magnum Erik Olof Gunnar Andersson - senlin Tim Burke - swift I think we should also add a file in the repo that lists “subject matter experts” for each service we’ve got support for, where we have them. My list of current cores who I’d ping for specific service suitability are: Sean McGinnis - cinder Slawek Kaplonski - neutron Dmitry Tantsur - ironic Eric Fried - nova (at least until tomorrow my friend) How does that sound to folks? Monty
Seems like a good way to kind of unblock/get things moving a little faster for some more projects. Also, we did something similar to this in releases where a few new cores were added, but we were instructed to not +W things. Basically we could give the first +2 to move things along, but weren't making the merge decision and I would say its gone rather well. -Kendall (diablo_rojo) On Thu, Mar 5, 2020 at 9:56 AM Monty Taylor <mordred@inaugust.com> wrote:
Heya,
I’d like to try something.
I’d like to try adding some project-specific people to the core team so that they can more directly help maintain the support for their service in SDK. In some of these cases the person I’m suggestion has next to no review experience in SDK. I think let’s be fine with that for now - we’re still a 2x +2 in general thing - but I know currently when reviewing neutron or ironic changes I always want to see a +2 from slaweq or dtantsur … so in the spirit of trying new things and trying to move the project forward in a healthy and welcoming way - how about we give this a try?
The idea here is that we’re trusting people to use their good judgement and to only use their new +2 powers for good in their project. Over time, if they feel like they’ve gotten a handle on things more widely, there’s nothing stopping them from reviewing other patches - but I think that most of us aren’t looking for additional review work anyway.
Specifically this would be:
Shogo Saito - congress Adam Harwell - octavia Graham Hayes - designate Bharat Kumar - magnum Erik Olof Gunnar Andersson - senlin Tim Burke - swift
I think we should also add a file in the repo that lists “subject matter experts” for each service we’ve got support for, where we have them. My list of current cores who I’d ping for specific service suitability are:
Sean McGinnis - cinder Slawek Kaplonski - neutron Dmitry Tantsur - ironic Eric Fried - nova (at least until tomorrow my friend)
How does that sound to folks?
Monty
+1 from me for Erik on Senlin. On Thu, Mar 5, 2020 at 10:02 AM Kendall Nelson <kennelson11@gmail.com> wrote:
Seems like a good way to kind of unblock/get things moving a little faster for some more projects. Also, we did something similar to this in releases where a few new cores were added, but we were instructed to not +W things. Basically we could give the first +2 to move things along, but weren't making the merge decision and I would say its gone rather well.
-Kendall (diablo_rojo)
On Thu, Mar 5, 2020 at 9:56 AM Monty Taylor <mordred@inaugust.com> wrote:
Heya,
I’d like to try something.
I’d like to try adding some project-specific people to the core team so that they can more directly help maintain the support for their service in SDK. In some of these cases the person I’m suggestion has next to no review experience in SDK. I think let’s be fine with that for now - we’re still a 2x +2 in general thing - but I know currently when reviewing neutron or ironic changes I always want to see a +2 from slaweq or dtantsur … so in the spirit of trying new things and trying to move the project forward in a healthy and welcoming way - how about we give this a try?
The idea here is that we’re trusting people to use their good judgement and to only use their new +2 powers for good in their project. Over time, if they feel like they’ve gotten a handle on things more widely, there’s nothing stopping them from reviewing other patches - but I think that most of us aren’t looking for additional review work anyway.
Specifically this would be:
Shogo Saito - congress Adam Harwell - octavia Graham Hayes - designate Bharat Kumar - magnum Erik Olof Gunnar Andersson - senlin Tim Burke - swift
I think we should also add a file in the repo that lists “subject matter experts” for each service we’ve got support for, where we have them. My list of current cores who I’d ping for specific service suitability are:
Sean McGinnis - cinder Slawek Kaplonski - neutron Dmitry Tantsur - ironic Eric Fried - nova (at least until tomorrow my friend)
How does that sound to folks?
Monty
Naming names is a highly effective demotivator for those whose contributions are not recognized. https://www.stackalytics.com/?module=openstacksdk&metric=loc Maybe the project teams should select their own liaison? Michael On Thu, Mar 5, 2020 at 9:58 AM Monty Taylor <mordred@inaugust.com> wrote:
Heya,
I’d like to try something.
I’d like to try adding some project-specific people to the core team so that they can more directly help maintain the support for their service in SDK. In some of these cases the person I’m suggestion has next to no review experience in SDK. I think let’s be fine with that for now - we’re still a 2x +2 in general thing - but I know currently when reviewing neutron or ironic changes I always want to see a +2 from slaweq or dtantsur … so in the spirit of trying new things and trying to move the project forward in a healthy and welcoming way - how about we give this a try?
The idea here is that we’re trusting people to use their good judgement and to only use their new +2 powers for good in their project. Over time, if they feel like they’ve gotten a handle on things more widely, there’s nothing stopping them from reviewing other patches - but I think that most of us aren’t looking for additional review work anyway.
Specifically this would be:
Shogo Saito - congress Adam Harwell - octavia Graham Hayes - designate Bharat Kumar - magnum Erik Olof Gunnar Andersson - senlin Tim Burke - swift
I think we should also add a file in the repo that lists “subject matter experts” for each service we’ve got support for, where we have them. My list of current cores who I’d ping for specific service suitability are:
Sean McGinnis - cinder Slawek Kaplonski - neutron Dmitry Tantsur - ironic Eric Fried - nova (at least until tomorrow my friend)
How does that sound to folks?
Monty
On Mar 5, 2020, at 6:53 PM, Michael Johnson <johnsomor@gmail.com> wrote:
Naming names is a highly effective demotivator for those whose contributions are not recognized.
https://www.stackalytics.com/?module=openstacksdk&metric=loc
Maybe the project teams should select their own liaison?
Great point - and totally! I was mostly keying off of recent interactions but you’re absolutely right about that. Also - apologies for my brain not immediately calling you up there… it’s been a week. I think adding you is a great idea. There’s some other thoughts in the responses here that are interesting that we should try as well, but I think those will take a bit longer to sort out.
On Thu, Mar 5, 2020 at 9:58 AM Monty Taylor <mordred@inaugust.com> wrote:
Heya,
I’d like to try something.
I’d like to try adding some project-specific people to the core team so that they can more directly help maintain the support for their service in SDK. In some of these cases the person I’m suggestion has next to no review experience in SDK. I think let’s be fine with that for now - we’re still a 2x +2 in general thing - but I know currently when reviewing neutron or ironic changes I always want to see a +2 from slaweq or dtantsur … so in the spirit of trying new things and trying to move the project forward in a healthy and welcoming way - how about we give this a try?
The idea here is that we’re trusting people to use their good judgement and to only use their new +2 powers for good in their project. Over time, if they feel like they’ve gotten a handle on things more widely, there’s nothing stopping them from reviewing other patches - but I think that most of us aren’t looking for additional review work anyway.
Specifically this would be:
Shogo Saito - congress Adam Harwell - octavia Graham Hayes - designate Bharat Kumar - magnum Erik Olof Gunnar Andersson - senlin Tim Burke - swift
I think we should also add a file in the repo that lists “subject matter experts” for each service we’ve got support for, where we have them. My list of current cores who I’d ping for specific service suitability are:
Sean McGinnis - cinder Slawek Kaplonski - neutron Dmitry Tantsur - ironic Eric Fried - nova (at least until tomorrow my friend)
How does that sound to folks?
Monty
participants (4)
-
Duc Truong
-
Kendall Nelson
-
Michael Johnson
-
Monty Taylor