Friday, 2016-12-30

*** ducttape_ has quit IRC00:01
*** ducttape_ has joined #openstack-nova00:03
*** ducttape_ has quit IRC00:08
openstackgerritMatt Riedemann proposed openstack/nova: Make test_server_usage work with CONF.use_neutron=True by default  https://review.openstack.org/41577000:13
*** baoli_ has joined #openstack-nova00:25
*** baoli has quit IRC00:25
*** baoli_ has quit IRC00:28
*** baoli has joined #openstack-nova00:29
*** baoli has quit IRC00:29
*** baoli has joined #openstack-nova00:30
openstackgerritMatt Riedemann proposed openstack/nova: Make test_config_drive pass with CONF.use_neutron=True by default  https://review.openstack.org/41577100:30
*** ssurana1 has joined #openstack-nova00:33
*** ssurana has quit IRC00:36
*** baoli_ has joined #openstack-nova00:36
*** markvoelker has joined #openstack-nova00:36
*** moshele has joined #openstack-nova00:37
*** baoli has quit IRC00:39
*** sacharya has quit IRC00:44
*** stvnoyes has quit IRC00:52
*** yamahata has joined #openstack-nova00:52
*** swamireddy has quit IRC00:53
*** stvnoyes has joined #openstack-nova00:53
*** hoangcx has joined #openstack-nova00:54
*** zz_dimtruck is now known as dimtruck01:02
*** dtp has quit IRC01:03
*** ducttape_ has joined #openstack-nova01:04
*** hfu has joined #openstack-nova01:06
*** ducttape_ has quit IRC01:09
*** mlavalle has quit IRC01:12
*** sacharya has joined #openstack-nova01:15
*** rcernin has quit IRC01:17
*** sacharya has quit IRC01:19
*** phuongnh has joined #openstack-nova01:27
*** harlowja has quit IRC01:39
*** zul has quit IRC01:49
*** zul has joined #openstack-nova01:50
*** zhs__ has quit IRC01:59
*** zhs has joined #openstack-nova02:00
*** hogepodge has joined #openstack-nova02:03
*** hongbin has joined #openstack-nova02:15
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514202:21
openstackgerritMatt Riedemann proposed openstack/nova: Make test_keypairs work with CONF.use_neutron=True by default  https://review.openstack.org/41577502:29
*** baoli_ has quit IRC02:31
*** zhs_ has joined #openstack-nova02:31
*** baoli has joined #openstack-nova02:32
*** baoli has quit IRC02:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_security_groups work with CONF.use_neutron=True by default  https://review.openstack.org/41574702:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_security_group_default_rules work with use_neutron=True by default  https://review.openstack.org/41576702:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_server_usage work with CONF.use_neutron=True by default  https://review.openstack.org/41577002:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_tenant_networks pass with CONF.use_neutron=True by default  https://review.openstack.org/41575502:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_config_drive pass with CONF.use_neutron=True by default  https://review.openstack.org/41577102:32
openstackgerritMatt Riedemann proposed openstack/nova: Make test_keypairs work with CONF.use_neutron=True by default  https://review.openstack.org/41577502:32
*** zhs has quit IRC02:34
*** ducttape_ has joined #openstack-nova02:35
*** gongysh has joined #openstack-nova02:37
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514202:38
*** ducttape_ has quit IRC02:39
*** zhs_ has quit IRC02:44
openstackgerritLi Yingjun proposed openstack/nova: Move FlavorPayload to a seperate file  https://review.openstack.org/41577602:46
openstackgerritLi Yingjun proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/41577702:46
openstackgerritMatt Riedemann proposed openstack/nova: Remove unused init_only kwarg from wsgi app init  https://review.openstack.org/41577902:53
openstackgerritLi Yingjun proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/39817102:53
*** zhs has joined #openstack-nova02:53
*** diga has quit IRC02:54
*** dave-mccowan has joined #openstack-nova02:56
*** wlhc has joined #openstack-nova02:58
*** yufei has joined #openstack-nova03:02
*** mdnadeem has joined #openstack-nova03:11
*** mdnadeem has quit IRC03:13
openstackgerritMatt Riedemann proposed openstack/nova: Make test_scheduler_hints work with CONF.use_neutron=True by default  https://review.openstack.org/41578103:14
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514203:14
*** sam_nowitzki has joined #openstack-nova03:15
*** moshele has quit IRC03:16
*** sacharya has joined #openstack-nova03:16
*** sacharya has quit IRC03:20
*** mdnadeem has joined #openstack-nova03:25
*** gongysh has quit IRC03:26
*** zhs has quit IRC03:31
*** zhs has joined #openstack-nova03:31
*** psachin has joined #openstack-nova03:34
*** kiran-r has joined #openstack-nova03:37
*** zhs_ has joined #openstack-nova03:37
*** zhs has quit IRC03:40
*** wlhc has quit IRC04:05
*** wlhc has joined #openstack-nova04:07
*** links has joined #openstack-nova04:09
*** baoli has joined #openstack-nova04:09
openstackgerritLi Yingjun proposed openstack/nova: Update notification for flavor  https://review.openstack.org/41384604:11
*** wlhc has quit IRC04:12
*** wlhc has joined #openstack-nova04:13
*** baoli has quit IRC04:14
*** wlhc has quit IRC04:16
*** wlhc has joined #openstack-nova04:17
*** david-lyle has joined #openstack-nova04:23
*** nicolasbock has joined #openstack-nova04:26
*** david-lyle has quit IRC04:28
*** sacharya has joined #openstack-nova04:30
*** psachin_ has joined #openstack-nova04:34
*** nicolasbock has quit IRC04:34
*** psachin_ has quit IRC04:42
*** ducttape_ has joined #openstack-nova04:43
*** prateek has joined #openstack-nova04:45
*** nicolasbock has joined #openstack-nova04:47
*** ducttape_ has quit IRC04:48
*** dimtruck is now known as zz_dimtruck04:51
*** baoli has joined #openstack-nova04:56
*** baoli has quit IRC04:56
*** markvoelker has quit IRC04:58
*** salv-orlando has joined #openstack-nova05:03
*** salv-orlando has quit IRC05:08
*** baoli has joined #openstack-nova05:11
*** ayogi has joined #openstack-nova05:13
*** hongbin has quit IRC05:19
*** diga has joined #openstack-nova05:23
*** swamireddy has joined #openstack-nova05:31
*** ecelik has joined #openstack-nova05:32
*** wlhc has quit IRC05:32
*** wlhc has joined #openstack-nova05:32
*** dave-mccowan has quit IRC05:35
*** ecelik has left #openstack-nova05:36
*** satyar has joined #openstack-nova05:38
*** baoli has quit IRC05:54
*** yufei has quit IRC06:04
*** salv-orlando has joined #openstack-nova06:04
*** diga has quit IRC06:04
*** ociuhandu has quit IRC06:05
*** yufei has joined #openstack-nova06:06
*** salv-orlando has quit IRC06:09
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514206:12
*** yufei has quit IRC06:13
*** ducttape_ has joined #openstack-nova06:14
*** baoli has joined #openstack-nova06:16
*** ducttape_ has quit IRC06:19
*** baoli has quit IRC06:21
*** yufei has joined #openstack-nova06:21
*** ociuhandu has joined #openstack-nova06:25
*** links has quit IRC06:32
*** lpetrut has joined #openstack-nova06:34
*** haplo37_ has quit IRC06:39
*** haplo37_ has joined #openstack-nova06:42
*** salv-orlando has joined #openstack-nova06:43
*** ratailor has joined #openstack-nova06:43
*** links has joined #openstack-nova06:44
openstackgerritZhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail  https://review.openstack.org/41514206:53
*** sridharg has joined #openstack-nova06:54
*** wlhc has quit IRC06:55
*** wlhc has joined #openstack-nova06:56
*** markvoelker has joined #openstack-nova06:58
*** markvoelker has quit IRC07:03
*** karthiks has joined #openstack-nova07:04
*** rcernin has joined #openstack-nova07:05
*** wlhc has quit IRC07:06
*** sacharya has quit IRC07:08
*** sacharya has joined #openstack-nova07:09
*** sacharya has quit IRC07:09
*** tesseract has joined #openstack-nova07:11
*** salv-orl_ has joined #openstack-nova07:13
*** salv-orlando has quit IRC07:16
*** baoli has joined #openstack-nova07:17
*** baoli has quit IRC07:21
*** links has quit IRC07:28
*** tojuvone_ has joined #openstack-nova07:33
*** tojuvone has quit IRC07:33
*** yamahata has quit IRC07:35
*** wlhc has joined #openstack-nova07:41
*** lpetrut has quit IRC07:42
*** kiran-r has quit IRC07:43
*** slaweq has joined #openstack-nova07:50
*** links has joined #openstack-nova07:52
*** hoangcx has quit IRC08:00
*** wlhc_ has joined #openstack-nova08:04
*** wlhc has quit IRC08:04
*** yuntongjin has joined #openstack-nova08:05
*** yuntongjin has quit IRC08:05
*** gcb has quit IRC08:18
Kevin_Zhengalex_xu: Hi, I updated patches: https://review.openstack.org/#/c/408571/  https://review.openstack.org/#/c/415142/  according to your comments, please have a look if you have time :)08:19
openstackgerritZhenyu Zheng proposed openstack/nova: [WIP]Add new policy for server list/detail with all_tenants  https://review.openstack.org/41533008:22
*** baoli has joined #openstack-nova08:24
*** baoli has quit IRC08:29
*** salv-orl_ has quit IRC08:30
*** yufei has quit IRC08:30
openstackgerritKevin Benton proposed openstack/nova: Revert "fix for auth during live-migration"  https://review.openstack.org/41582008:30
*** yufei has joined #openstack-nova08:32
*** gcb has joined #openstack-nova08:36
*** lpetrut has joined #openstack-nova08:38
*** baoli has joined #openstack-nova08:39
*** baoli has quit IRC08:44
*** pcaruana has joined #openstack-nova08:48
*** lpetrut has quit IRC08:50
*** baoli has joined #openstack-nova08:54
*** wlhc_ has quit IRC08:57
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.rebuild notification  https://review.openstack.org/38492208:58
*** baoli has quit IRC08:59
*** markvoelker has joined #openstack-nova08:59
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:01
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622509:02
*** baoli has joined #openstack-nova09:04
*** markvoelker has quit IRC09:04
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.rebuild.error notification  https://review.openstack.org/39662109:08
*** baoli has quit IRC09:09
alex_xuKevin_Zheng: thanks, will check them asap09:17
*** kfarr has quit IRC09:18
openstackgerritBalazs Gibizer proposed openstack/nova: Transform instance.resize_confirm notification  https://review.openstack.org/39065409:20
*** david-lyle has joined #openstack-nova09:27
*** david-lyle has quit IRC09:31
Kevin_Zhengalex_xu: Thanks Happy New Year09:41
alex_xuKevin_Zheng: you too :)09:41
*** psachin has quit IRC09:46
*** salv-orlando has joined #openstack-nova09:47
*** gcb has quit IRC09:49
*** satyar has quit IRC09:55
*** tojuvone_ is now known as tojuvone10:04
*** yufei has quit IRC10:17
*** psachin has joined #openstack-nova10:23
*** david-lyle has joined #openstack-nova10:27
*** baoli has joined #openstack-nova10:29
*** david-lyle has quit IRC10:32
*** baoli has quit IRC10:34
*** matrohon has joined #openstack-nova10:34
openstackgerritBalazs Gibizer proposed openstack/nova: add availability_zone field in service.update  https://review.openstack.org/41585710:35
*** matrohon has quit IRC10:38
*** mdnadeem has quit IRC10:39
*** ssurana1 has quit IRC10:57
*** phuongnh has quit IRC10:57
*** pcaruana has quit IRC10:58
*** markvoelker has joined #openstack-nova11:00
*** ssurana has joined #openstack-nova11:00
*** ssurana has quit IRC11:05
*** markvoelker has quit IRC11:05
*** ssurana has joined #openstack-nova11:05
*** tbachman has quit IRC11:09
*** fragatina has joined #openstack-nova11:12
*** xinliang has quit IRC11:14
*** nicolasbock has quit IRC11:17
*** psachin has quit IRC11:17
*** ratailor has quit IRC11:22
*** xinliang has joined #openstack-nova11:26
*** sam_nowitzki has quit IRC11:27
*** salv-orlando has quit IRC11:28
*** baoli has joined #openstack-nova11:30
*** rfolco_ has joined #openstack-nova11:31
*** rfolco_ has quit IRC11:31
*** rfolco has joined #openstack-nova11:32
*** baoli has quit IRC11:35
*** takedakn has joined #openstack-nova11:41
*** takedakn has quit IRC11:41
*** zhs_ has quit IRC11:44
*** baoli has joined #openstack-nova11:44
*** baoli has quit IRC11:48
*** psachin has joined #openstack-nova11:51
openstackgerritVasyl Saienko proposed openstack/nova: Support Ironic interface attach/detach in nova virt  https://review.openstack.org/36441311:52
*** bkopilov has quit IRC11:53
*** bkopilov has joined #openstack-nova11:55
*** hfu has quit IRC11:57
*** sridharg has quit IRC11:58
*** sridharg has joined #openstack-nova12:03
*** moshele has joined #openstack-nova12:07
*** Dinesh_Bhor has joined #openstack-nova12:08
*** ducttape_ has joined #openstack-nova12:19
*** baoli has joined #openstack-nova12:19
*** salv-orlando has joined #openstack-nova12:22
*** psachin has quit IRC12:23
*** baoli has quit IRC12:23
*** ayogi has quit IRC12:24
*** ducttape_ has quit IRC12:24
*** ssurana1 has joined #openstack-nova12:28
*** lpetrut has joined #openstack-nova12:28
*** ssurana has quit IRC12:28
*** ssurana1 has quit IRC12:33
*** sheel has quit IRC12:47
*** tbachman has joined #openstack-nova12:50
*** jaypipes has joined #openstack-nova12:52
*** baoli has joined #openstack-nova12:53
*** baoli has quit IRC12:53
*** mnasiadka has joined #openstack-nova12:53
*** jaypipes is now known as leakypipes12:53
*** prateek has quit IRC12:56
*** markvoelker has joined #openstack-nova13:01
*** tbachman_ has joined #openstack-nova13:01
*** salv-orlando has quit IRC13:03
*** tbachman has quit IRC13:04
*** tbachman_ is now known as tbachman13:04
*** markvoelker has quit IRC13:06
openstackgerritJay Pipes proposed openstack/nova: Give NUMACell a uuid field  https://review.openstack.org/41573813:07
*** bhagyashris has quit IRC13:12
*** psachin has joined #openstack-nova13:13
leakypipespkoniszewski: there's nothing that says happy holidays like coding on NUMA and resource providers...13:18
*** ducttape_ has joined #openstack-nova13:18
*** takedakn has joined #openstack-nova13:23
*** links has quit IRC13:24
*** sridhargaddam has joined #openstack-nova13:29
*** baoli has joined #openstack-nova13:34
*** baoli has quit IRC13:39
*** baoli has joined #openstack-nova13:39
*** ducttape_ has quit IRC13:39
*** lpetrut has quit IRC13:40
openstackgerritWang Qing wu proposed openstack/nova: WIP(2): Introduce base PowerVM driver (disk drv)  https://review.openstack.org/40940113:42
*** baoli has quit IRC13:43
*** Dinesh_Bhor has quit IRC13:49
*** ducttape_ has joined #openstack-nova13:52
*** zz_dimtruck is now known as dimtruck13:53
*** jamesdenton has joined #openstack-nova13:53
*** baoli has joined #openstack-nova13:54
*** jheroux has joined #openstack-nova13:58
*** baoli has quit IRC13:59
*** salv-orlando has joined #openstack-nova14:03
*** catintheroof has joined #openstack-nova14:03
*** tbachman has quit IRC14:04
*** catintheroof has quit IRC14:08
*** salv-orlando has quit IRC14:08
*** catintheroof has joined #openstack-nova14:08
*** sridhargaddam has quit IRC14:14
*** sridharg has quit IRC14:15
*** baoli has joined #openstack-nova14:19
*** satyar has joined #openstack-nova14:19
*** salv-orlando has joined #openstack-nova14:20
*** liverpooler has quit IRC14:23
*** liverpooler has joined #openstack-nova14:23
*** baoli has quit IRC14:23
*** tbachman has joined #openstack-nova14:27
*** david-lyle has joined #openstack-nova14:30
*** takedakn has quit IRC14:30
*** moshele has quit IRC14:31
*** tbachman has quit IRC14:32
*** david-lyle has quit IRC14:35
*** baoli has joined #openstack-nova14:39
*** baoli has quit IRC14:44
*** psachin has quit IRC14:51
*** nklenke has quit IRC14:52
*** salv-orlando has quit IRC14:53
openstackgerritRafael Folco proposed openstack/nova-specs: DELETE all inventories for a resource provider  https://review.openstack.org/41588514:57
*** nklenke has joined #openstack-nova15:00
*** markvoelker has joined #openstack-nova15:02
*** ducttape_ has quit IRC15:02
*** psachin has joined #openstack-nova15:04
*** markvoelker has quit IRC15:06
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Improve get_disk_type_from_path  https://review.openstack.org/41194215:10
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Fix snapshot operation under LXC  https://review.openstack.org/41588715:10
*** mlavalle has joined #openstack-nova15:17
*** psachin has quit IRC15:28
*** satyar has quit IRC15:28
*** catintheroof has quit IRC15:29
*** fragatina has quit IRC15:31
mriedemleakypipes: does this look ok to you? https://review.openstack.org/#/c/397545/15:32
*** fragatina has joined #openstack-nova15:34
openstackgerritTuan Luong-Anh proposed openstack/nova: Fix hypervisor-stats show invalid info of baremetal resource  https://review.openstack.org/39141515:40
*** psachin has joined #openstack-nova15:42
mriedempretty easy review here https://review.openstack.org/#/c/394511/15:48
*** mlavalle has quit IRC15:53
*** salv-orlando has joined #openstack-nova15:54
*** mlavalle has joined #openstack-nova15:56
*** salv-orlando has quit IRC15:58
*** rcernin has quit IRC16:01
*** hongbin has joined #openstack-nova16:02
*** markvoelker has joined #openstack-nova16:03
*** ducttape_ has joined #openstack-nova16:03
*** tesseract has quit IRC16:05
*** mnasiadka has quit IRC16:06
*** markvoelker has quit IRC16:07
*** ducttape_ has quit IRC16:07
*** rfolco has quit IRC16:08
*** cdent has joined #openstack-nova16:12
*** rfolco has joined #openstack-nova16:15
*** rfolco has quit IRC16:15
*** fragatina has quit IRC16:22
*** satyar has joined #openstack-nova16:25
*** vtech has joined #openstack-nova16:28
*** ssurana has joined #openstack-nova16:29
*** jheroux has quit IRC16:30
*** harlowja has joined #openstack-nova16:31
*** david-lyle has joined #openstack-nova16:32
*** ssurana has quit IRC16:34
*** cdent has quit IRC16:35
*** jheroux has joined #openstack-nova16:35
*** david-lyle has quit IRC16:37
*** psachin has quit IRC16:37
*** dave-mccowan has joined #openstack-nova16:38
mriedemgah we weren't actually testing our online flavors migration routine16:42
openstackgerritDiana Clarke proposed openstack/python-novaclient: Blacklist rather than whitelist autodoc modules  https://review.openstack.org/41575616:49
*** efried has quit IRC16:50
*** ducttape_ has joined #openstack-nova16:51
*** dave-mccowan has quit IRC16:54
*** diga has joined #openstack-nova16:54
*** dtp has joined #openstack-nova16:54
*** rama_y has joined #openstack-nova16:54
leakypipesmriedem: :(16:55
*** salv-orlando has joined #openstack-nova16:55
openstackgerritMatt Riedemann proposed openstack/nova: Actually test online flavor migrations  https://review.openstack.org/41589616:56
mriedemleakypipes: ^16:56
mriedemleakypipes: that's not the worst thing i've found out today,16:56
leakypipesmriedem: on it.16:56
mriedemthe worst thing is laura fully dried my favorite shirt,16:56
leakypipesmriedem: :(16:56
mriedemand now it doesn't fit *just right* anymore16:57
mriedem2016 is truly a terrible year16:57
digajaypipes - you there ?16:57
*** tbachman has joined #openstack-nova16:57
openstackgerritFeodor Tersin proposed openstack/nova: libvirt: Skip file operations for not appropriate disk types  https://review.openstack.org/41589716:57
leakypipesdiga: yup!16:59
*** salv-orlando has quit IRC16:59
digaleakypipes: Hello17:00
digaleakypipes: it is regarding placement API work17:00
leakypipesmriedem: I am being stymied by the fact that the NUMATopology/NUMACell objects (of which I need to uuidify) are being recreated each and every time the libvirt driver's get_available_resource() method is called :(17:01
leakypipesmriedem: thus invalidating/recreating the UUID identifiers.17:01
leakypipesguh./17:01
leakypipesdiga: yep. I have some thoughts on that.17:01
digaleakypipes: okay17:01
mriedemleakypipes: so they are an insert rather than an update on each call?17:02
*** efried has joined #openstack-nova17:02
leakypipesmriedem: because, you know, the NUMA nova objects originated as non-nova objects that were JSONified and stuffed into a string field in the compute_nodes table and recreated each and every time a method in hardware.py is called. :(17:02
leakypipesmriedem: there's no update or insert. the compute_nodes.numa_topology field is the JSONified serialization of the NUMATopology object.17:03
*** markvoelker has joined #openstack-nova17:03
leakypipesmriedem: and unfortunately, get_available_resource() returns a brand new NUMATopology object, created fdrom the libvirt XML file (!) each time get_available_resource() is called, and therefore it's making each NUMA cell a new UUID each time...17:04
leakypipesmriedem: which is making my code that relies on looking up NUMA cells as resource providers by UUID a little "problematic" :)17:04
leakypipesdiga: anyway... tell me about your familiarity (if any) with the placement service/API so far.17:05
leakypipesdiga: it's cool if you have none.17:05
leakypipesjust need to know where to start17:05
digaleakypipes: I have done some work on nova schedular but not contributed to community but it was in icehouse release17:06
leakypipesdiga: gotcha.17:06
mriedemleakypipes: so are you saying that we're creating new resources in those tables every 60 seconds?17:06
leakypipesdiga: I'm in need of someone to take ownership of adding support for notification events to the placement service. would you be interested in doing that? basically it would entail adding notification support into the placement service and plumbing up various events for when inventories are created, allocations are made and deleted, etc17:07
leakypipesmriedem: no, we *would* be if this NUMA cell UUID field patch went through.17:08
mriedemor i guess we don't have a separate table for those, but numa_topology in the compute_nodes table is rewritten each time?17:08
mriedemoh17:08
digaleakypipes: Sure, I would be happy to do it17:08
*** markvoelker has quit IRC17:08
leakypipesmriedem: the objects that get recreated by the get_available_resource() method all look the same to the resource tracker right now, but once UUID fields change, the RT thinks the topo has changed and would write an update.17:08
*** sacharya_ has joined #openstack-nova17:08
leakypipesdiga: cool. this would be for Pike, of course, since we're well past Ocata spec freeze.17:09
leakypipesdiga: but would be good to get started on now and begin proof of concept work.17:09
digaleakypipes: Yes, sure17:09
leakypipesdiga: this would be an ideal way for you to get familiar with the new placement service, how it is different from the scheduler and nova-compute resource tracker, and do some learnin'.17:10
digaleakypipes: I will start the work now17:10
digaleakypipes: Yep17:10
leakypipesdiga: awesome. lemme create a blueprint in Nova with some general steps.17:10
leakypipesdiga: what's your launchpad id?17:11
digaleakypipes: Sure17:11
mriedemleakypipes: fwiw i think the notifications were called out in the original generic-resource-pools spec17:11
digaleakypipes: digambarpatil1517:11
mriedemso that could be worked under generic-resource-pools-ocata bp17:11
leakypipescool, cheers17:11
mriedemhttps://specs.openstack.org/openstack/nova-specs/specs/newton/implemented/generic-resource-pools.html#notifications-impact17:12
leakypipesmriedem: ah, ok. I still think it should be pike though :) too much work for ocata already17:12
mriedemhttps://blueprints.launchpad.net/nova/+spec/generic-resource-pools-ocata17:12
mriedemleakypipes: i'm totally fine with that17:12
mriedemi would personally like that bp to be less generic17:12
mriedemas a catch-all17:12
leakypipesmriedem: you cool with me creating a sep blueprint?17:12
mriedemleakypipes: totally17:13
leakypipescoolio.17:13
digaleakypipes: Thank you!17:14
digaleakypipes: Wish you a happy New Year!17:15
*** hongbin has quit IRC17:19
digaHappy New year to Everyone!17:20
*** sacharya_ has quit IRC17:20
leakypipesdiga: you too! :)17:21
leakypipesdiga: https://blueprints.launchpad.net/nova/+spec/placement-notifications17:21
digaleakypipes: :) will go through it now17:21
*** sacharya has joined #openstack-nova17:21
leakypipesdiga: you will want to start reading the placement code (nova/api/openstack/placement* and nova/objects/resource_provider.py)17:22
leakypipesdiga: and you will want to say hi to gibi who is our master of notifications in Nova :)17:22
digaleakypipes: yes, I will17:22
digagibi: Hi17:22
mriedemhmm, if we are migrating flavors from main to api dbs and hit a duplicate, we don't delete the old duplicate flavor from the main db, which means we might never report the online flavor migration as being 'done'17:25
mriedembut at the same time, you can't create new flavors in the api db via the object unless there are no flavors in the main db17:25
mriedemso getting a duplicate in the api db seems unlikely, unless you were populating the api db yourself, which you shouldn't do17:25
*** diga has quit IRC17:32
*** tbachman_ has joined #openstack-nova17:32
openstackgerritMatt Riedemann proposed openstack/nova: Handle unicode when dealing with duplicate flavors during online migrations  https://review.openstack.org/41589817:32
mriedemdansmith: i know how you love the unicode ^ :)17:32
*** tbachman has quit IRC17:33
*** tbachman_ is now known as tbachman17:33
*** diga has joined #openstack-nova17:34
*** mvensky has joined #openstack-nova17:44
*** catintheroof has joined #openstack-nova17:44
*** catinthe_ has joined #openstack-nova17:49
*** dave-mccowan has joined #openstack-nova17:49
*** mvensky has quit IRC17:49
*** mvensky has joined #openstack-nova17:50
*** catintheroof has quit IRC17:50
*** dave-mcc_ has joined #openstack-nova17:51
*** dave-mccowan has quit IRC17:54
*** harlowja has quit IRC17:56
*** salv-orlando has joined #openstack-nova17:56
*** mvensky has quit IRC17:57
*** mvensky has joined #openstack-nova17:57
*** mvensky has quit IRC17:58
openstackgerritMatt Riedemann proposed openstack/nova: Handle unicode when dealing with duplicate aggregate errors during migration  https://review.openstack.org/41589917:58
*** mvensky has joined #openstack-nova17:59
*** salv-orlando has quit IRC18:00
*** mvensky has quit IRC18:00
*** ducttape_ has quit IRC18:00
*** mvensky has joined #openstack-nova18:01
*** markvoelker has joined #openstack-nova18:04
*** mnestratov has quit IRC18:06
*** dave-mcc_ has quit IRC18:06
*** markvoelker has quit IRC18:09
leakypipesmriedem: I'm raging about this fucking NUMACell.uuid thing :(18:13
*** yamahata has joined #openstack-nova18:13
leakypipestrying to figure out a way of doing this without touching a crap-ton of code in the libvirt driver :(18:13
mriedemi'm not really familiar with any of that stuff18:14
leakypipesmriedem: all goes back to originally letting all the NFV shit into Nova using a completely separate object modeling and data storage strategy.18:15
*** catinthe_ has quit IRC18:15
leakypipesmriedem: still suffering from it.18:15
mriedemgibi: question in https://review.openstack.org/#/c/394512/9/nova/tests/unit/compute/test_compute.py18:16
*** mvensky has quit IRC18:16
*** mvensky has joined #openstack-nova18:16
*** mvensky has quit IRC18:17
*** mvensky has joined #openstack-nova18:18
mriedemleakypipes: can you pop the uuid or somehow take it out of the object comparison in the RT so we don't do a db update every 60 seconds?18:22
* mriedem doesn't know where this code is that's causing the problem18:22
mriedemalso if we're persisting the uuid in the db, why would it change every time?18:23
leakypipesmriedem: the issue is I need to uniquely identify the NUMA cell. I am going to have to get rid of the NUMACell.uuid attribute and manually construct a unique name -> uuid lookup hash in the scheduler report client :(18:23
leakypipesmriedem: exactly what I was trying to avoid with this...18:24
leakypipesmriedem: but because of the crap design of all the NUMA object modeling (and data storage) I cannot add a uuid field to the NUMA cell object.18:24
*** mvensky has quit IRC18:25
mriedemhow do these things get an id? i see they aren't persistent objects18:26
*** mvensky has joined #openstack-nova18:26
leakypipesmriedem: because even if we're persisting the uuid in the NUMACell in the compute_nodes.numa_topology DB field, the libvirt driver completely ignores ComputeNode object and the database and reconstructs all the NUMATopology object from the libvirt XML definition each and every time get_available_resource() is called.18:26
leakypipesmriedem: which is why I said above "I can't do this without touching a crap-ton of code in the libvirt driver.18:27
mriedemcan we store the uuid in the libvirt xml?18:27
leakypipesno18:27
*** salv-orlando has joined #openstack-nova18:27
mriedemno literally or no philosophically?18:27
mriedem:)18:27
leakypipeswe don't persist the libvirt XML.18:27
leakypipesit gets recreated on things like live migration.18:28
mriedemno i mean put the uuid in the guest ml18:28
leakypipes:(18:28
mriedem*xml18:28
leakypipesyes, I know, but we recreate it all on migration.18:28
mriedemwe store other nova metadata stuff in the xml, i wasn't sure if the elements for this had some extras18:28
mriedemon migration couldn't we get the uuid from our object that's persisted in the nova db?18:28
leakypipesmriedem: part of the problem with that is that it will be specific to the libvirt driver.18:29
mriedemlike most things...18:29
mriedemso "the libvirt driver completely ignores ComputeNode object and the database" - is there a path forward to changing that?18:29
leakypipesmriedem: we should always be using the DB's persistent storage for this stuff, but unfortunately, libvirt (and the virt driver interface itself) ignores the Nova DB state and re-creates crap all over the place.18:29
mriedemi imagine if there is, it blows up getting this done anytime soon18:30
leakypipesmriedem: not without major surgery, which is what I'm trrying to avoid.18:30
*** sacharya has quit IRC18:30
mriedemyeah18:30
mriedemso i guess you hash the numa cell object?18:30
*** mvensky has quit IRC18:31
mriedemi see it uses an all_things_equal method for __eq__18:31
mriedemguess you'd need to do something similar for hashing a value based on the fields18:31
leakypipesnot hash, but create a unique name based on the compute node UUID and the cell ordinal ID.18:31
mriedemoh18:31
*** diga has quit IRC18:32
*** mvensky has joined #openstack-nova18:32
mriedemdoes anything external to nova need to ever understand what those are?18:33
leakypipesagain, I cannot change the NUMACell object to add a uuid field like I did with the PCIDevice.uuid field because the libvirt driver ignores the DB when it comes to NUMA objects. For PCI devices, the thing that creates the PciDevice objects is the /nova/pci/manager.py not the libvirt driver, and the pci devices are looked up and stored in the pci_devices DB table, again unlike NUMA stuff with the libvirt driver.18:33
*** david-lyle has joined #openstack-nova18:33
*** sacharya has joined #openstack-nova18:34
leakypipesmriedem: yes, the placement API needs a globally unique and consistent way of identifying resource providers. and NUMA cells on a compute node are child providers to the parent resource provider of the compute node.18:34
mriedemyes i get that, but it's just going to be the nova resource tracker dealing with these resource providers right?18:35
leakypipesmriedem: and the problem is I have no consistent UUID identifiers for each NUMA cell because the uuid field keeps getting regenerated by the libvirt driver when get_available_resource() is called.18:35
*** pece has joined #openstack-nova18:35
leakypipesmriedem: right now, just the RT yes. but I can see things like nomad and maybe numad being external things that would need to identify the resource providers uniquely.18:36
mriedemnvm, i'm not really asking the right question. i was just thinking about how we needed to expose the aggregate uuid out of the REST API so that external systems can use it with the placement API18:36
mriedemyeah, so my line of thinking was, it might be ok to hack this internal for now to get it working,18:36
mriedembut external things couldn't rely on it because they don't know how the uuid is generated18:36
leakypipesmriedem: "for now" <-- will be a forever thing...18:37
mriedemor rely on it i guess18:37
mriedemleakypipes: sure, but the alternative then is the major surgery right?18:37
leakypipesmriedem: right.18:37
mriedemfwiw i'm fine with that18:37
leakypipesmriedem: k18:37
*** mvensky has quit IRC18:37
leakypipesthe hack, not the surgery, right?18:37
mriedemeither i think18:37
*** mvensky has joined #openstack-nova18:38
mriedemthis seems lower priority when looking at all of the other things going on, but i can see how it's very high priority for some people that rely on this stuff18:38
*** david-lyle has quit IRC18:38
mriedemdoes the hack block the major surgery later at all? like does it paint us into any corners?18:38
leakypipesmriedem: kinda, yeah. basically the surgery we need is a refactor of get_available_resource() into an interface method that accepts an InventoryList object of what the system thinks the current state of the hypervisor's resources are and returns a new InventoryList object with any changes that may have occurred.18:40
leakypipesmriedem: basically, what this series was trying to do: https://review.openstack.org/#/c/334035/18:42
mriedemok. well, i've got a date with star wars in 15 minutes so i need to head out. i'll be on later though.18:42
leakypipesno worries, catchya laters.18:42
leakypipesmriedem: have fun!18:43
*** mvensky has quit IRC18:45
*** mvensky has joined #openstack-nova18:45
*** rfolco has joined #openstack-nova18:48
*** rfolco has quit IRC18:48
*** rfolco has joined #openstack-nova18:51
*** rfolco has quit IRC18:51
*** sacharya has quit IRC18:52
*** mvensky has quit IRC18:59
*** mvensky has joined #openstack-nova18:59
*** pece has quit IRC18:59
*** mvensky has quit IRC18:59
*** ducttape_ has joined #openstack-nova19:02
*** markvoelker has joined #openstack-nova19:05
*** lpetrut has joined #openstack-nova19:06
*** ducttape_ has quit IRC19:06
*** slaweq has quit IRC19:09
*** markvoelker has quit IRC19:09
*** salv-orl_ has joined #openstack-nova19:13
*** ducttape_ has joined #openstack-nova19:13
*** slaweq has joined #openstack-nova19:15
*** salv-orlando has quit IRC19:16
*** kentb1 has joined #openstack-nova19:17
*** catintheroof has joined #openstack-nova19:21
*** jose-phillips has joined #openstack-nova19:22
*** ducttape_ has quit IRC19:25
*** ducttape_ has joined #openstack-nova19:26
*** ssurana has joined #openstack-nova19:31
*** sacharya has joined #openstack-nova19:32
*** ssurana1 has joined #openstack-nova19:34
*** ssurana has quit IRC19:34
*** vtech has quit IRC19:35
*** salv-orl_ has quit IRC19:52
*** catintheroof has quit IRC19:52
*** ducttape_ has quit IRC19:53
*** jheroux has quit IRC19:59
*** ducttape_ has joined #openstack-nova20:02
*** lpetrut has quit IRC20:04
*** dave-mccowan has joined #openstack-nova20:04
*** markvoelker has joined #openstack-nova20:06
*** lpetrut has joined #openstack-nova20:06
*** markvoelker has quit IRC20:10
*** kentb1 has quit IRC20:12
*** satyar has quit IRC20:22
*** salv-orlando has joined #openstack-nova20:24
*** dave-mccowan has quit IRC20:24
openstackgerritDiana Clarke proposed openstack/nova: Remove Rules.load_json warning  https://review.openstack.org/41590420:28
*** ducttape_ has quit IRC20:37
*** ducttape_ has joined #openstack-nova20:43
*** vtech has joined #openstack-nova20:45
*** nicolasbock has joined #openstack-nova20:58
*** moshele has joined #openstack-nova21:01
*** moshele has quit IRC21:04
*** markvoelker has joined #openstack-nova21:06
*** harlowja has joined #openstack-nova21:07
*** moshele has joined #openstack-nova21:10
*** markvoelker has quit IRC21:11
*** harlowja has quit IRC21:15
*** ducttape_ has quit IRC21:15
*** Jeffrey4l has quit IRC21:34
*** ducttape_ has joined #openstack-nova21:35
*** tlian has joined #openstack-nova21:35
*** david-lyle has joined #openstack-nova21:36
*** Jeffrey4l has joined #openstack-nova21:36
*** david-lyle has quit IRC21:40
*** Jeffrey4l has quit IRC21:40
*** Jeffrey4l has joined #openstack-nova21:41
*** jose-phillips has quit IRC21:43
dtpwhat is the console stuff in nova/console for?  how does it relate to my spice console?21:44
*** ducttape_ has quit IRC21:45
mriedemleakypipes: did you solve your problems or just give up for the year?21:49
mriedemleakypipes: just imagine the numa topology code as the empire and you're a scruffy member of the rebel alliance21:50
*** jose-phillips has joined #openstack-nova21:54
leakypipesmriedem: I'm still working on it...21:56
leakypipesmriedem: a refactoring/surgery route.21:57
*** jose-phillips has quit IRC21:58
*** jose-phillips has joined #openstack-nova22:05
*** moshele has quit IRC22:07
*** markvoelker has joined #openstack-nova22:07
*** markvoelker has quit IRC22:12
mriedemgibi: question here for you https://review.openstack.org/#/c/398171/16/doc/notification_samples/flavor-delete.json22:12
mriedemgibi: we seem to be inconsistent about the payloads on delete notifications,22:13
mriedemsometimes we send the full object in the payload, and sometimes just an id22:13
*** jose-phillips has quit IRC22:17
*** yamahata has quit IRC22:19
*** tlian has quit IRC22:21
mriedemgibi: just reply at your leisure http://lists.openstack.org/pipermail/openstack-dev/2016-December/109501.html22:24
*** nicolasbock has quit IRC22:25
*** ssurana1 has quit IRC22:29
*** jamesdenton has quit IRC22:36
*** jcook has quit IRC22:38
*** lpetrut has quit IRC22:38
*** jcook has joined #openstack-nova22:42
*** ducttape_ has joined #openstack-nova22:45
*** ducttape_ has quit IRC22:50
*** ssurana has joined #openstack-nova22:50
openstackgerritMatt Riedemann proposed openstack/nova: Move FlavorPayload to a seperate file  https://review.openstack.org/41577622:54
openstackgerritMatt Riedemann proposed openstack/nova: Update notification for flavor  https://review.openstack.org/41384622:54
openstackgerritMatt Riedemann proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/39817122:54
*** ssurana has quit IRC22:57
*** catintheroof has joined #openstack-nova23:07
*** markvoelker has joined #openstack-nova23:08
*** haplo37_ has quit IRC23:11
*** markvoelker has quit IRC23:13
dtpis there a max code width in nova?23:17
*** haplo37_ has joined #openstack-nova23:20
*** david-lyle has joined #openstack-nova23:37
*** david-lyle has quit IRC23:42
openstackgerritMatt Riedemann proposed openstack/nova: Update notification for flavor  https://review.openstack.org/41384623:46
openstackgerritMatt Riedemann proposed openstack/nova: Notifications for flavor operations  https://review.openstack.org/39817123:46
*** vtech has quit IRC23:47
*** ducttape_ has joined #openstack-nova23:47
*** salv-orlando has quit IRC23:47
*** ducttape_ has quit IRC23:51
mriedemdtp: 80 chars23:53
mriedempep823:53
mriedemi.e. tox -e pep8 will tell you23:53

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