Re: [openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2017-01-04 Thread Vladyslav Drok
On Wed, Jan 4, 2017 at 5:45 PM, Vladyslav Drok wrote: > Thanks all for replies! > > On Tue, Jan 3, 2017 at 5:16 PM, Jay Faulkner wrote: > >> Hey Vdrok, some comments inline. >> >> > On Dec 30, 2016, at 8:40 AM, Vladyslav Drok wrote: >> > >>

Re: [openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2017-01-04 Thread Vladyslav Drok
Thanks all for replies! On Tue, Jan 3, 2017 at 5:16 PM, Jay Faulkner wrote: > Hey Vdrok, some comments inline. > > > On Dec 30, 2016, at 8:40 AM, Vladyslav Drok wrote: > > > > Hi all! > > > > There is a long standing problem of resources reporting in ironic

Re: [openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2017-01-03 Thread Jay Faulkner
Hey Vdrok, some comments inline. > On Dec 30, 2016, at 8:40 AM, Vladyslav Drok wrote: > > Hi all! > > There is a long standing problem of resources reporting in ironic virt > driver. It's described in a couple of bugs I've found - [0], [1]. Switching > to placement API

Re: [openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2017-01-03 Thread Pavlo Shchelokovskyy
Hi, a comment about 'report as full' vs 'remove from inventory' On Mon, Jan 2, 2017 at 7:53 PM, Jay Pipes wrote: > Great questions, Vlad. Comments inline. > > On 12/30/2016 11:40 AM, Vladyslav Drok wrote: > >> Hi all! >> >> There is a long standing problem of resources

Re: [openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2017-01-02 Thread Jay Pipes
Great questions, Vlad. Comments inline. On 12/30/2016 11:40 AM, Vladyslav Drok wrote: Hi all! There is a long standing problem of resources reporting in ironic virt driver. That would be an understatement :) > It's described in a couple of bugs I've found - [0], [1]. Switching to placement

[openstack-dev] [ironic] [nova] Ironic virt driver resources reporting

2016-12-30 Thread Vladyslav Drok
Hi all! There is a long standing problem of resources reporting in ironic virt driver. It's described in a couple of bugs I've found - [0], [1]. Switching to placement API will make things better, but still there are some problems there. For example, there are cases when ironic needs to say "this