Friday, 2018-08-10

*** esberglu has quit IRC00:54
*** esberglu has joined #openstack-powervm00:55
*** esberglu has quit IRC01:00
*** esberglu has joined #openstack-powervm03:38
*** esberglu has quit IRC03:43
*** esberglu has joined #openstack-powervm04:48
*** esberglu has quit IRC04:53
*** esberglu has joined #openstack-powervm06:38
*** esberglu has quit IRC06:43
*** esberglu has joined #openstack-powervm07:18
*** esberglu has quit IRC07:22
*** esberglu has joined #openstack-powervm08:38
*** esberglu has quit IRC08:43
*** esberglu has joined #openstack-powervm09:18
*** esberglu has quit IRC09:23
*** esberglu has joined #openstack-powervm09:58
*** esberglu has quit IRC10:03
*** esberglu has joined #openstack-powervm11:59
*** esberglu has quit IRC12:03
*** efried is now known as fried_rice13:35
openstackgerritEric Fried proposed openstack/nova-powervm master: WIP: update_provider_tree with device exposure  https://review.openstack.org/58966816:01
*** fried_rice is now known as fried_rolls16:08
*** fried_rolls is now known as fried_rice18:47
fried_riceedmondsw: Woot, ^ passes powervm CI. That's a fairly big deal, really.20:21
fried_riceIt completely changes the code path taken by the nova resource tracker.20:23
fried_ricemeans I've successfully duplicated the logic in that code path via update_provider_tree (and that our device inventorying successfully no-ops with no inventory.yaml file present :)20:23
edmondswfried_rice cool. I'll look20:54
fried_riceedmondsw: Note, still needs unit tests, but this is good for initial confidence.20:55
edmondswack20:55
edmondswfried_rice pep8 :)21:00
fried_riceyeah, got that fix locally.21:00
fried_ricedidn't bother pushing a 1-liner for that since I need to respin for tests anyway.21:00
edmondswfried_rice so we're just adding here... nothing goes away?21:01
fried_riceedmondsw: Yes, I believe that's correct.21:02
fried_riceI'm actually thinking we should land just the compat update_provider_tree as a separate patch, both in tree and out. Then add the dev stuff in this series.21:03
edmondswmakes sense21:04
edmondswfried_rice posted a few comments21:25
fried_ricethx21:26
edmondswdidn't look at the inv stuff much, since I'm not caught up on the rest of that series21:26
fried_riceight21:26

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!