[openstack-dev] [tripleo] Removing old baremetal commands from python-tripleoclient

2017-12-14 Thread Tony Breeds
Hi All, In review I01837a9daf6f119292b5a2ffc361506925423f11 I updated ValidateInstackEnv to handle the case when then instackenv.json file needs to represent a node that deosn't require a pm_user for IMPI to work. It turns out that I foudn that code path with grep rather than the result of a d

Re: [openstack-dev] [tripleo] Removing old baremetal commands from python-tripleoclient

2017-12-15 Thread Dmitry Tantsur
On 12/15/2017 04:49 AM, Tony Breeds wrote: Hi All, In review I01837a9daf6f119292b5a2ffc361506925423f11 I updated ValidateInstackEnv to handle the case when then instackenv.json file needs to represent a node that deosn't require a pm_user for IMPI to work. It turns out that I foudn that cod

Re: [openstack-dev] [tripleo] Removing old baremetal commands from python-tripleoclient

2017-12-15 Thread Dmitry Tantsur
On 12/15/2017 01:04 PM, Dmitry Tantsur wrote: On 12/15/2017 04:49 AM, Tony Breeds wrote: Hi All, In review I01837a9daf6f119292b5a2ffc361506925423f11 I updated ValidateInstackEnv to handle the case when then instackenv.json file needs to represent a node that deosn't require a pm_user for IM

Re: [openstack-dev] [tripleo] Removing old baremetal commands from python-tripleoclient

2017-12-17 Thread Tony Breeds
On Fri, Dec 15, 2017 at 01:04:52PM +0100, Dmitry Tantsur wrote: > On 12/15/2017 04:49 AM, Tony Breeds wrote: > > Hi All, > > In review I01837a9daf6f119292b5a2ffc361506925423f11 I updated > > ValidateInstackEnv to handle the case when then instackenv.json file > > needs to represent a node that

Re: [openstack-dev] [tripleo] Removing old baremetal commands from python-tripleoclient

2017-12-18 Thread Ben Nemec
On 12/17/2017 05:55 PM, Tony Breeds wrote: On Fri, Dec 15, 2017 at 01:04:52PM +0100, Dmitry Tantsur wrote: On 12/15/2017 04:49 AM, Tony Breeds wrote: Hi All, In review I01837a9daf6f119292b5a2ffc361506925423f11 I updated ValidateInstackEnv to handle the case when then instackenv.json fil