Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-09-06 Thread Jim Rollenhagen
On Tue, Sep 6, 2016 at 4:24 PM, Jim Rollenhagen wrote: > On Mon, Aug 29, 2016 at 5:03 AM, Lucas Alvares Gomes > wrote: >> Hi, >> >> I overall agree with the proposed plan. I like the idea of having a >> "supported" flag (or another name as per Kurt's email) that makes it >> easy mark a driver as

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-09-06 Thread Jim Rollenhagen
On Mon, Aug 29, 2016 at 5:03 AM, Lucas Alvares Gomes wrote: > Hi, > > I overall agree with the proposed plan. I like the idea of having a > "supported" flag (or another name as per Kurt's email) that makes it > easy mark a driver as "unsupported" indicating it might be removed > soon. > > About po

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-29 Thread Lucas Alvares Gomes
Hi, I overall agree with the proposed plan. I like the idea of having a "supported" flag (or another name as per Kurt's email) that makes it easy mark a driver as "unsupported" indicating it might be removed soon. About point #3 I'm indifferent, it's a common approach in the project to log a warn

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-23 Thread Vladyslav Drok
On Mon, Aug 22, 2016 at 9:48 PM, Loo, Ruby wrote: > Hi, > > > > I admit, I didn't read the entire thread [0], but did read the summary > [1]. I like this, except that I'm not sure about #3. What's the rationale > of adding a new config option 'enable_unsupported_drivers' that defaults to > False.

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Kurt Taylor
In my opinion, we have to be careful about the "supported" label. Saying that third-party tested drivers are community supported implies a commitment that may have not been intended. Personally, I see no problem with leaving everything just as planned. Drivers that are in tree are either tested up

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Devananda van der Veen
I like this approach more than the current "you're in or you're out". +1 --deva On Mon, Aug 22, 2016 at 7:02 AM Vladyslav Drok wrote: > +1 from me then :) > > On Mon, Aug 22, 2016 at 4:52 PM, Jim Rollenhagen > wrote: > >> On Mon, Aug 22, 2016 at 8:01 AM, Vladyslav Drok >> wrote: >> > On Fri,

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Loo, Ruby
Hi, I admit, I didn't read the entire thread [0], but did read the summary [1]. I like this, except that I'm not sure about #3. What's the rationale of adding a new config option 'enable_unsupported_drivers' that defaults to False. Versus not having it, and "just" logging a warning if they are

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Vladyslav Drok
+1 from me then :) On Mon, Aug 22, 2016 at 4:52 PM, Jim Rollenhagen wrote: > On Mon, Aug 22, 2016 at 8:01 AM, Vladyslav Drok > wrote: > > On Fri, Aug 19, 2016 at 5:15 PM, Jim Rollenhagen > > > wrote: > >> > >> Hi Ironickers, > >> > >> There was a big thread here[0] about Cinder, driver removal

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Jim Rollenhagen
On Mon, Aug 22, 2016 at 8:01 AM, Vladyslav Drok wrote: > On Fri, Aug 19, 2016 at 5:15 PM, Jim Rollenhagen > wrote: >> >> Hi Ironickers, >> >> There was a big thread here[0] about Cinder, driver removal, and standard >> deprecation policy. If you haven't read through it yet, please do before >> co

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Vladyslav Drok
On Fri, Aug 19, 2016 at 5:15 PM, Jim Rollenhagen wrote: > Hi Ironickers, > > There was a big thread here[0] about Cinder, driver removal, and standard > deprecation policy. If you haven't read through it yet, please do before > continuing here. :) > > The outcome of that thread is summarized well

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-22 Thread Sam Betts (sambetts)
;> Subject: [openstack-dev] [ironic] Driver removal policies - should we >>make it >> softer? >> >> Hi Ironickers, >> >> There was a big thread here[0] about Cinder, driver removal, and >>standard >> deprecation policy. If you haven't read thro

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Villalovos, John L
> -Original Message- > From: Jim Rollenhagen [mailto:j...@jimrollenhagen.com] > Sent: Friday, August 19, 2016 7:15 AM > To: OpenStack Development Mailing List (not for usage questions) > > Subject: [openstack-dev] [ironic] Driver removal policies - should we make it

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Doug Hellmann
Excerpts from Jim Rollenhagen's message of 2016-08-19 11:02:58 -0400: > On Fri, Aug 19, 2016 at 10:54 AM, Mathieu Mitchell > wrote: > > > > > > On 2016-08-19 10:42 AM, Doug Hellmann wrote: > >>> > >>> 3) Add a ``enable_unsupported_drivers`` config option that allows > >>> enabling > >>> >drive

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Jim Rollenhagen
On Fri, Aug 19, 2016 at 10:54 AM, Mathieu Mitchell wrote: > > > On 2016-08-19 10:42 AM, Doug Hellmann wrote: >>> >>> 3) Add a ``enable_unsupported_drivers`` config option that allows >>> enabling >>> >drivers marked supported=False. If a driver is in enabled_drivers, >>> > has >> >> Do you mea

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Mathieu Mitchell
On 2016-08-19 10:42 AM, Doug Hellmann wrote: 3) Add a ``enable_unsupported_drivers`` config option that allows enabling >drivers marked supported=False. If a driver is in enabled_drivers, has Do you mean "in disabled_drivers" there? enabled_drivers is the list of all drivers to be loade

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Mathieu Mitchell
On 2016-08-19 10:15 AM, Jim Rollenhagen wrote: 3) Add a ``enable_unsupported_drivers`` config option that allows enabling drivers marked supported=False. If a driver is in enabled_drivers, has supported=False, and enable_unsupported_drivers=False, ironic-conductor will fail to start. Set

Re: [openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Doug Hellmann
Excerpts from Jim Rollenhagen's message of 2016-08-19 10:15:20 -0400: > Hi Ironickers, > > There was a big thread here[0] about Cinder, driver removal, and standard > deprecation policy. If you haven't read through it yet, please do before > continuing here. :) > > The outcome of that thread is s

[openstack-dev] [ironic] Driver removal policies - should we make it softer?

2016-08-19 Thread Jim Rollenhagen
Hi Ironickers, There was a big thread here[0] about Cinder, driver removal, and standard deprecation policy. If you haven't read through it yet, please do before continuing here. :) The outcome of that thread is summarized well here.[1] I know that I previously had a different opinion on this, b