Tuesday, 2016-11-22

*** jdurgin1 has joined #openstack-nova00:00
*** mhickey has quit IRC00:02
*** WillUK2010 has quit IRC00:02
*** armax has quit IRC00:08
*** tbachman has joined #openstack-nova00:09
*** armax has joined #openstack-nova00:22
*** jgriffith has quit IRC00:25
*** siva_krish has quit IRC00:25
*** siva_krish has joined #openstack-nova00:26
*** jgriffith has joined #openstack-nova00:26
*** gmann has quit IRC00:28
*** gmann has joined #openstack-nova00:28
openstackgerritijw-ubuntu proposed openstack/nova-specs: VIF plugging negotiation between Nova and Neutron  https://review.openstack.org/39051200:30
*** armax has quit IRC00:30
*** jgriffith has quit IRC00:30
*** jgriffith has joined #openstack-nova00:33
*** armax has joined #openstack-nova00:36
*** mtan_____ has joined #openstack-nova00:36
*** jgriffith has quit IRC00:36
*** Dave has quit IRC00:37
*** mtanino has quit IRC00:37
*** dimtruck is now known as zz_dimtruck00:39
*** siva_krish has quit IRC00:42
*** hongbin has quit IRC00:43
*** victorhe has joined #openstack-nova00:43
oomichialex_xu: johnthetubaguy: can you take a look at https://review.openstack.org/#/c/400413/ and https://review.openstack.org/#/c/329704/ ?00:44
*** mriedem has joined #openstack-nova00:45
*** mdrabe has quit IRC00:45
*** victorhe has quit IRC00:47
*** tbachman has quit IRC00:54
*** thorst has joined #openstack-nova00:54
eandersson_alaski: Could you take a look at https://review.openstack.org/#/c/390984/ (or if you know anyone else that could take a look)00:59
*** thorst has quit IRC00:59
*** tbachman has joined #openstack-nova00:59
*** Dave_____ has joined #openstack-nova01:00
*** rama_y has quit IRC01:00
*** Dave_____ is now known as Dave01:05
*** diogogmt has quit IRC01:05
*** armax has quit IRC01:08
alaskieandersson_: johnthetubaguy and bauzas would be good folks to have look at that01:09
*** armax has joined #openstack-nova01:09
*** huanxie has joined #openstack-nova01:09
eandersson_thanks alaski01:09
eandersson_johnthetubaguy, bauzas whenever you have time, could you take a look at https://review.openstack.org/#/c/39098401:10
*** Swami has quit IRC01:12
openstackgerritMatt Riedemann proposed openstack/nova-specs: Fix bullet formatting in RP custom-resource-classes spec  https://review.openstack.org/40045901:13
*** tuanluong has joined #openstack-nova01:15
*** nic has quit IRC01:18
*** iceyao has joined #openstack-nova01:19
*** penick has quit IRC01:20
*** rfolco has joined #openstack-nova01:21
*** Sukhdev has quit IRC01:23
*** moshele has joined #openstack-nova01:25
*** zhangjl has joined #openstack-nova01:26
*** yuntongjin has joined #openstack-nova01:29
*** dave-mccowan has joined #openstack-nova01:30
*** gcb has joined #openstack-nova01:35
*** takashin has joined #openstack-nova01:36
*** penick has joined #openstack-nova01:37
openstackgerritMatt Riedemann proposed openstack/nova: Add index on instances table across project_id and updated_at  https://review.openstack.org/39967201:40
*** Apoorva has quit IRC01:41
*** zz_dimtruck is now known as dimtruck01:41
*** alexpilotti has joined #openstack-nova01:44
*** victorhe has joined #openstack-nova01:45
*** victorhe_ has joined #openstack-nova01:48
*** diogogmt has joined #openstack-nova01:49
*** alexpilotti has quit IRC01:49
*** ljxiash has joined #openstack-nova01:49
*** jgriffith has joined #openstack-nova01:49
*** victorhe has quit IRC01:52
*** victorh__ has joined #openstack-nova01:52
alex_xuoomichi: will check that after a meeting01:55
*** victorhe_ has quit IRC01:55
*** mtanino has joined #openstack-nova01:56
*** mtan_____ has quit IRC01:57
*** alexpilotti has joined #openstack-nova01:58
*** victorh__ has quit IRC02:01
*** victorhe_ has joined #openstack-nova02:02
*** alexpilotti has quit IRC02:03
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec  https://review.openstack.org/39719302:04
openstackgerritTakashi NATSUME proposed openstack/nova-specs: Amend cold-migration-with-target-ocata spec  https://review.openstack.org/39719302:04
*** mtanino has quit IRC02:04
*** victorhe_ has quit IRC02:07
*** iceyao has quit IRC02:07
*** dave-mccowan has quit IRC02:07
*** victorhe has joined #openstack-nova02:07
*** iceyao has joined #openstack-nova02:07
*** jogo has quit IRC02:07
*** gongysh has joined #openstack-nova02:08
*** zhangjl has quit IRC02:08
*** namnh has joined #openstack-nova02:09
*** gcb has quit IRC02:09
*** victorhe has quit IRC02:09
*** zhangjl has joined #openstack-nova02:09
*** victorhe has joined #openstack-nova02:09
*** yufei has joined #openstack-nova02:11
*** victorhe has quit IRC02:11
*** victorhe has joined #openstack-nova02:11
*** jgriffith has quit IRC02:12
mriedemdansmith: found the cause of that cinder vol snapshot gate failure02:13
mriedem\o/02:13
mriedemwinning02:13
*** iceyao has quit IRC02:14
dansmithmriedem: score02:14
*** armax has quit IRC02:14
dansmiththat has been the suck for a while02:14
*** gcb has joined #openstack-nova02:15
mriedemfudge yeah it has02:15
mriedemhttps://review.openstack.org/#/c/400465/02:15
dansmithnice02:15
*** iceyao has joined #openstack-nova02:16
*** penick has quit IRC02:18
*** gcb has quit IRC02:19
*** armax has joined #openstack-nova02:20
*** armax has quit IRC02:21
*** gcb has joined #openstack-nova02:22
mriedemsdague has been pung02:22
openstackgerritSTEW TY proposed openstack/nova: Complete verification of servers-action-fixed-ip.inc  https://review.openstack.org/33860602:25
openstackgerritChangBo Guo(gcb) proposed openstack/nova: conf: Remove deprecated service manager opts  https://review.openstack.org/31953202:29
*** siva_krish has joined #openstack-nova02:30
*** liuyulong has joined #openstack-nova02:30
*** siva_krish has quit IRC02:34
*** iceyao has quit IRC02:36
*** Alagar has joined #openstack-nova02:37
*** tbachman has quit IRC02:38
*** moshele has quit IRC02:46
*** kaisers_ has joined #openstack-nova02:48
*** tjones has quit IRC02:49
*** kaisers__ has quit IRC02:51
*** Jeffrey4l has joined #openstack-nova02:53
*** moshele has joined #openstack-nova02:54
*** sudipto_ has joined #openstack-nova03:00
*** Guest56972 has joined #openstack-nova03:00
*** kaisers has quit IRC03:01
*** gjayavelu has joined #openstack-nova03:03
*** rama_y has joined #openstack-nova03:07
*** gjayavelu has quit IRC03:07
*** rfolco has quit IRC03:08
*** bkopilov has quit IRC03:10
*** dikonoor has joined #openstack-nova03:10
*** unicell1 has quit IRC03:11
AlagarIam trying to integrate openstack with xen03:13
Alagari have installed openstack using devstack script top of xen hypervisor, in this open stack as a virtual machine.03:13
Alagarwhen i create instance in openstack,   the instance should create in xen hypervisor.  but its not happening.03:13
AlagarSome one could you please guide me please03:13
*** kaisers has joined #openstack-nova03:14
*** jogo has joined #openstack-nova03:15
*** jogo has quit IRC03:15
*** jogo has joined #openstack-nova03:15
*** ekuris has joined #openstack-nova03:19
*** ekuris has quit IRC03:19
*** jogo has quit IRC03:20
*** liuyulong has quit IRC03:22
*** tbachman has joined #openstack-nova03:29
*** yuntongjin has quit IRC03:31
*** yamahata has quit IRC03:32
*** huanxie has quit IRC03:33
*** jgriffith has joined #openstack-nova03:35
*** jgriffith has quit IRC03:36
*** gjayavelu has joined #openstack-nova03:36
*** fragatin_ has joined #openstack-nova03:36
*** fragatin_ has quit IRC03:38
*** fragati__ has joined #openstack-nova03:38
*** armax has joined #openstack-nova03:39
*** fragatina has quit IRC03:39
*** gjayavelu has quit IRC03:41
*** fragati__ has quit IRC03:41
*** mriedem has quit IRC03:41
openstackgerritZhenyu Zheng proposed openstack/nova: Transform instance.resize_prep notification  https://review.openstack.org/39698803:44
*** giri has joined #openstack-nova03:44
*** prashkre_ has joined #openstack-nova03:46
*** psachin has joined #openstack-nova03:47
*** dikonoor has quit IRC03:47
*** ljxiash has quit IRC03:48
*** ljxiash has joined #openstack-nova03:49
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Don't use 'updated_at' to check service's status  https://review.openstack.org/29708503:51
*** tjones has joined #openstack-nova03:52
*** ljxiash has quit IRC03:53
*** gongysh has quit IRC03:54
*** Guest56972 has quit IRC03:55
*** sudipto_ has quit IRC03:55
*** fragatina has joined #openstack-nova03:57
*** unicell has joined #openstack-nova03:59
*** jwcroppe has quit IRC04:04
*** jwcroppe has joined #openstack-nova04:04
*** fragatina has quit IRC04:06
*** thorst has joined #openstack-nova04:06
*** fragatina has joined #openstack-nova04:07
*** links has joined #openstack-nova04:08
*** dikonoor has joined #openstack-nova04:08
*** victorhe has quit IRC04:10
*** victorhe has joined #openstack-nova04:11
*** fragatina has quit IRC04:11
*** alexpilotti has joined #openstack-nova04:14
*** diogogmt has quit IRC04:15
*** victorhe has quit IRC04:15
*** moshele has quit IRC04:15
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port test_keypairs to Python 3  https://review.openstack.org/39890904:17
*** alexpilotti has quit IRC04:18
*** victorhe has joined #openstack-nova04:19
*** ijw has quit IRC04:20
*** mdnadeem has joined #openstack-nova04:23
*** gongysh has joined #openstack-nova04:24
*** fragatina has joined #openstack-nova04:29
*** fragatina has quit IRC04:29
*** fragatina has joined #openstack-nova04:29
*** bkopilov has joined #openstack-nova04:29
*** GB21 has joined #openstack-nova04:33
*** Alagar has quit IRC04:33
*** Alagar has joined #openstack-nova04:33
*** victorhe has quit IRC04:35
*** victorhe has joined #openstack-nova04:35
*** victorhe has quit IRC04:40
*** tlian has quit IRC04:40
*** prashkre_ has quit IRC04:41
*** ayogi has joined #openstack-nova04:42
*** yuntongjin has joined #openstack-nova04:43
*** nandal has joined #openstack-nova04:47
*** dikonoor has quit IRC04:52
*** kylek3h_ has joined #openstack-nova04:53
*** nandal_in has joined #openstack-nova04:54
*** kylek3h has quit IRC04:56
*** nandal has quit IRC04:56
*** Sukhdev has joined #openstack-nova04:57
*** huanxie has joined #openstack-nova05:02
*** nandal_in has quit IRC05:03
*** Alagar has quit IRC05:04
*** Alagar has joined #openstack-nova05:06
*** mdnadeem has quit IRC05:08
*** thorst has quit IRC05:08
BobBallAlagar: libvirt+xen or XenAPI?05:11
*** nandal has joined #openstack-nova05:17
*** ratailor has joined #openstack-nova05:19
*** ljxiash has joined #openstack-nova05:19
*** I has joined #openstack-nova05:21
*** I is now known as Guest8303605:22
*** sudipto_ has joined #openstack-nova05:22
*** prateek has joined #openstack-nova05:22
*** sandanar has joined #openstack-nova05:24
*** GB21 has quit IRC05:28
*** ratailor_ has joined #openstack-nova05:31
*** sridharg has joined #openstack-nova05:40
openstackgerritHuan Xie proposed openstack/nova: xenapi: support the hotplug of a neutron port  https://review.openstack.org/36550505:42
*** andybotting has quit IRC05:44
openstackgerritAbhishek Kekane proposed openstack/nova: Use is_valid_cidr from oslo_utils  https://review.openstack.org/32037505:45
*** unicell has quit IRC05:45
*** unicell has joined #openstack-nova05:45
*** GB21 has joined #openstack-nova05:46
*** mdnadeem has joined #openstack-nova05:56
*** yufei has quit IRC06:00
*** thorst has joined #openstack-nova06:00
*** lpetrut has joined #openstack-nova06:01
*** claudiub has joined #openstack-nova06:03
*** rama_y has quit IRC06:03
*** yufei has joined #openstack-nova06:04
*** yuntongjin has quit IRC06:06
*** thorst has quit IRC06:08
*** gjayavel_ has joined #openstack-nova06:08
*** GB21 has quit IRC06:10
*** giri has quit IRC06:11
*** giri has joined #openstack-nova06:15
*** gjayavel_ has quit IRC06:15
*** jaosorior has joined #openstack-nova06:17
openstackgerritBéla Vancsics proposed openstack/nova: Transform instance.trigger_crash_dump notification  https://review.openstack.org/39622506:21
*** GB21 has joined #openstack-nova06:22
*** yuntongjin has joined #openstack-nova06:23
openstackgerritTakashi NATSUME proposed openstack/nova: Fix a typo in nova/api/openstack/compute/volumes.py  https://review.openstack.org/40053306:26
*** prashkre has joined #openstack-nova06:27
openstackgerritZhenyu Zheng proposed openstack/nova: Transform instance.resize_prep notification  https://review.openstack.org/39698806:29
*** ratailor has quit IRC06:33
*** prashkre has quit IRC06:33
*** prashkre has joined #openstack-nova06:34
*** yamahata has joined #openstack-nova06:38
*** mjura has joined #openstack-nova06:46
*** lpetrut has quit IRC06:46
*** jaosorior has quit IRC06:49
*** jaosorior has joined #openstack-nova06:49
*** yufei has quit IRC06:50
*** mjura has quit IRC06:50
*** oanson has joined #openstack-nova06:51
*** mjura has joined #openstack-nova06:51
*** moshele has joined #openstack-nova06:52
*** josecastroleon has joined #openstack-nova06:56
*** thorst has joined #openstack-nova06:56
*** lpetrut has joined #openstack-nova06:56
*** fandi has joined #openstack-nova07:00
*** nandal_in has joined #openstack-nova07:03
*** rfolco has joined #openstack-nova07:04
*** thorst has quit IRC07:04
*** rfolco has quit IRC07:04
*** nandal has quit IRC07:04
*** nandal has joined #openstack-nova07:05
openstackgerritJohn Hua proposed openstack/nova: WIP: fix memory usage miscalculation  https://review.openstack.org/39485707:05
*** nherciu has joined #openstack-nova07:06
*** nandal_in has quit IRC07:07
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_floating_ip_pools.py  https://review.openstack.org/32262407:09
*** tjones has quit IRC07:10
*** lpetrut has quit IRC07:11
openstackgerritDao Cong Tien proposed openstack/nova: Remove mox from api/openstack/compute/test_fping.py  https://review.openstack.org/32262907:18
*** yufei has joined #openstack-nova07:19
*** gcb has quit IRC07:22
*** belmoreira has joined #openstack-nova07:23
*** belmoreira has quit IRC07:24
openstackgerritRajesh Tailor proposed openstack/nova: Allow force-delete irrespective of VM task_state  https://review.openstack.org/39754207:29
*** yufei has quit IRC07:30
*** yufei has joined #openstack-nova07:30
*** tovin07 has quit IRC07:33
*** gcb has joined #openstack-nova07:35
*** tovin07_ has joined #openstack-nova07:36
AlagarBobBall: Thank you, i take apology for the delay reply,   could you please explain  ?  iam new to open stack07:36
*** ekuris has joined #openstack-nova07:37
*** huanxie has quit IRC07:38
*** huanxie has joined #openstack-nova07:48
*** daemontool has joined #openstack-nova07:48
*** ljxiash has quit IRC07:50
*** fandi__ has joined #openstack-nova07:51
*** thorst has joined #openstack-nova07:51
*** ralonsoh has joined #openstack-nova07:51
*** armax has quit IRC07:52
*** ljxiash has joined #openstack-nova07:52
*** belmoreira has joined #openstack-nova07:53
*** fandi has quit IRC07:55
*** pcaruana has joined #openstack-nova07:56
*** matrohon has joined #openstack-nova07:57
*** thorst has quit IRC07:58
*** tbachman has quit IRC07:59
*** nkrinner_afk is now known as nkrinner08:04
*** Sukhdev has quit IRC08:06
BobBallAlagar: Which guide have you followed for installing?08:09
*** gouthamr has joined #openstack-nova08:12
*** sahid has joined #openstack-nova08:12
openstackgerritZhenyu Zheng proposed openstack/nova: Transform instance.resize_prep notification  https://review.openstack.org/39698808:14
*** rcernin_ has joined #openstack-nova08:14
*** alexpilotti has joined #openstack-nova08:15
*** takashin has quit IRC08:18
*** alexpilotti has quit IRC08:19
*** takashin has joined #openstack-nova08:20
*** jdurgin1 has quit IRC08:26
*** victorhe has joined #openstack-nova08:27
*** markus_z has joined #openstack-nova08:28
*** victorhe has quit IRC08:32
*** daemontool has quit IRC08:37
*** zenoway has joined #openstack-nova08:37
*** rwmjones_holiday is now known as rwmjones08:40
*** amoralej|off is now known as amoralej08:40
openstackgerritJohn Hua proposed openstack/nova: WIP: fix memory usage miscalculation  https://review.openstack.org/39485708:40
*** thorst has joined #openstack-nova08:40
*** hogepodge has quit IRC08:45
*** alexpilotti has joined #openstack-nova08:45
*** thorst has quit IRC08:48
*** alexpilotti has quit IRC08:50
*** giri has quit IRC08:50
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Remove unused code  https://review.openstack.org/40060908:52
*** alexpilo_ has joined #openstack-nova08:53
sahidmarkus_z: can you take care of the serie about serial console ?08:54
sahidi'm demotivated to work on it, really08:54
markus_zsahid: Yep, will do that. Thanks for you work there.08:54
*** zzzeek has quit IRC09:00
*** zzzeek has joined #openstack-nova09:01
*** lpetrut has joined #openstack-nova09:03
*** gongysh has quit IRC09:05
*** sridharg has quit IRC09:08
*** lpetrut has quit IRC09:11
*** sridharg has joined #openstack-nova09:12
*** yuntongjin has quit IRC09:12
*** lpetrut has joined #openstack-nova09:17
*** lpetrut1 has joined #openstack-nova09:20
*** Cristina__ is now known as CristinaPauna09:21
*** lpetrut has quit IRC09:21
*** lpetrut1 is now known as lpetrut09:21
*** mhickey has joined #openstack-nova09:21
*** giri has joined #openstack-nova09:22
*** GB21 has quit IRC09:26
*** victorhe has joined #openstack-nova09:27
*** gjayavel_ has joined #openstack-nova09:29
*** bks has joined #openstack-nova09:29
*** giri has quit IRC09:30
*** lucas-afk is now known as lucasagomes09:31
*** victorhe has quit IRC09:31
*** derekh has joined #openstack-nova09:36
*** GB21 has joined #openstack-nova09:40
*** jwcroppe has quit IRC09:40
johnthetubaguyjcook: JayF: would be great if you could take a peak at this one, with ironic glasses on: https://review.openstack.org/#/c/39098409:40
*** jwcroppe has joined #openstack-nova09:41
*** paul-carlton has joined #openstack-nova09:43
*** tovin07_ has quit IRC09:48
*** yassine has joined #openstack-nova09:49
*** thorst has joined #openstack-nova09:50
*** yassine is now known as Guest631209:50
*** ihrachys has joined #openstack-nova09:50
*** tuanluong has quit IRC09:56
*** thorst has quit IRC09:58
*** namnh has quit IRC09:59
*** baoli has quit IRC10:02
*** sambetts|afk is now known as sambetts10:05
*** FL1SK has quit IRC10:07
*** gjayavel_ has quit IRC10:08
*** GB21 has quit IRC10:13
*** brault has joined #openstack-nova10:13
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Combine injection info in InstanceConfig  https://review.openstack.org/33779010:13
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Never copy a swap disk during cold migration  https://review.openstack.org/33111510:13
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Don't re-resize disks in finish_migration()  https://review.openstack.org/33111810:13
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Don't check config disk during cold migration  https://review.openstack.org/33715810:13
mdboothWhoops10:14
* mdbooth thought he'd only touched the top 2 :/10:14
lyarwoodsenior moment? :D10:16
* lyarwood runs10:16
mdbooth:P10:16
*** gcb_ has joined #openstack-nova10:16
*** gcb_ has quit IRC10:16
*** gcb_ has joined #openstack-nova10:17
*** gcb_ has quit IRC10:17
*** lxsli has joined #openstack-nova10:19
*** lxsli has left #openstack-nova10:20
*** gcb has quit IRC10:20
*** hshiina has joined #openstack-nova10:21
*** gcb has joined #openstack-nova10:22
*** victorhe has joined #openstack-nova10:24
*** yufei has quit IRC10:24
*** fandi__ has quit IRC10:26
openstackgerritStephen Finucane proposed openstack/nova: Use is_valid_cidr from oslo_utils  https://review.openstack.org/32037510:28
*** victorhe has quit IRC10:28
*** gjayavelu has joined #openstack-nova10:29
*** zhangjl has left #openstack-nova10:31
*** TommyLikeHu is now known as tommylikehu10:31
*** gjayavelu has quit IRC10:33
*** GB21 has joined #openstack-nova10:35
*** gjayavelu has joined #openstack-nova10:45
*** huanxie has quit IRC10:45
*** brault has quit IRC10:46
*** thorst has joined #openstack-nova10:46
*** gouthamr has quit IRC10:49
*** gjayavelu has quit IRC10:49
*** GB21 has quit IRC10:52
*** thorst has quit IRC10:53
*** hoonetorg has quit IRC10:53
*** brault has joined #openstack-nova10:53
*** brault has quit IRC10:58
*** josecastroleon has quit IRC10:59
*** tbachman has joined #openstack-nova10:59
*** nkrinner has quit IRC11:00
*** brault has joined #openstack-nova11:00
*** nkrinner has joined #openstack-nova11:00
*** yufei has joined #openstack-nova11:01
paul-carltonlyarwood, looking at https://review.openstack.org/#/c/400384, looks good if it is that easy!#11:04
openstackgerritGábor Antal proposed openstack/nova: Remove mox in unit/cells/test_cells_manager.py  https://review.openstack.org/39903611:04
*** GB21 has joined #openstack-nova11:05
*** gjayavelu has joined #openstack-nova11:05
paul-carltonlyarwood, firing up devstack to test it, if it works I'll +1 and then apply my live migration fix (https://review.openstack.org/#/c/389608/) on top of it and verify that11:06
*** Guest83036 has quit IRC11:06
*** sudipto_ has quit IRC11:06
*** gjayavelu has quit IRC11:09
lyarwoodpaul-carlton: cool, thanks. WORKSFORME so let me know if you have issues.11:11
openstackgerritTakashi NATSUME proposed openstack/python-novaclient: Remove unused test code  https://review.openstack.org/40068411:13
*** yamahata has quit IRC11:14
*** foexle has joined #openstack-nova11:14
*** mvk has quit IRC11:15
*** victorhe has joined #openstack-nova11:15
paul-carltonlyarwood, don't doubt it, but this has been broken for so long I will enjoy seeing it working!11:16
*** victorhe has quit IRC11:20
*** hshiina has quit IRC11:23
*** brault has quit IRC11:24
*** tbachman has quit IRC11:25
*** gjayavelu has joined #openstack-nova11:26
*** gjayavelu has quit IRC11:30
*** bks has quit IRC11:34
*** rfolco has joined #openstack-nova11:35
openstackgerritDanil Akhmetov proposed openstack/nova: Use proper user and tenant in the owner section of libvirt.xml.  https://review.openstack.org/39967911:36
*** tbachman has joined #openstack-nova11:36
*** tbachman_ has joined #openstack-nova11:38
*** gszasz has joined #openstack-nova11:38
openstackgerritChangBo Guo(gcb) proposed openstack/nova: Port libvirt test_driver to Python 3  https://review.openstack.org/40069711:39
*** tbachman_ has quit IRC11:40
*** tbachman has quit IRC11:41
*** mvk has joined #openstack-nova11:43
*** Guest83036 has joined #openstack-nova11:43
*** sudipto_ has joined #openstack-nova11:43
*** thorst has joined #openstack-nova11:43
*** rtheis has joined #openstack-nova11:45
*** gjayavelu has joined #openstack-nova11:46
*** thorst has quit IRC11:48
*** al_indig_ has quit IRC11:48
*** nicolasbock has joined #openstack-nova11:49
*** gjayavelu has quit IRC11:50
*** prateek has quit IRC12:00
*** FL1SK has joined #openstack-nova12:04
*** gjayavelu has joined #openstack-nova12:07
*** Guest83036 has quit IRC12:09
*** sudipto_ has quit IRC12:09
*** gjayavelu has quit IRC12:11
*** ljxiash has quit IRC12:14
*** ljxiash has joined #openstack-nova12:15
*** victorhe has joined #openstack-nova12:15
*** bvanhav_ has joined #openstack-nova12:15
*** WillUK2010 has joined #openstack-nova12:17
*** aloga has quit IRC12:18
*** aloga has joined #openstack-nova12:18
*** tesseract has joined #openstack-nova12:19
*** ljxiash has quit IRC12:19
*** tesseract is now known as Guest378712:19
*** bkopilov has quit IRC12:19
*** victorhe has quit IRC12:20
*** ljxiash has joined #openstack-nova12:24
*** gjayavelu has joined #openstack-nova12:27
*** ljxiash has quit IRC12:29
*** gouthamr has joined #openstack-nova12:30
*** gjayavelu has quit IRC12:31
*** rmart04 has joined #openstack-nova12:33
*** ratailor_ has quit IRC12:35
openstackgerritSergey Nikitin proposed openstack/nova: Modify fields of 'Diagnostics' object  https://review.openstack.org/39448012:36
openstackgerritSergey Nikitin proposed openstack/nova: [WIP] Implement instance diagnostics for Xen  https://review.openstack.org/39961312:36
*** thorst has joined #openstack-nova12:38
*** thorst has quit IRC12:45
*** GB21 has quit IRC12:45
*** takashin has left #openstack-nova12:47
*** kaisers__ has joined #openstack-nova12:50
*** kaisers_ has quit IRC12:51
openstackgerritjichenjc proposed openstack/nova: Refactored query user_id code for quota-sets  https://review.openstack.org/28016512:52
*** slaweq_ has quit IRC12:54
*** claudiub|2 has joined #openstack-nova12:58
*** lucasagomes is now known as lucas-hungry12:59
*** jaypipes has joined #openstack-nova13:00
*** victorhe has joined #openstack-nova13:00
*** gcb has quit IRC13:00
*** claudiub has quit IRC13:01
*** moshele has quit IRC13:01
*** mjura has quit IRC13:01
sfinucanlbeliveau, wznoinsk: SR-IOV meeting?13:03
*** goutham has joined #openstack-nova13:03
sfinucani.e. are we having one?13:04
*** tbachman has joined #openstack-nova13:04
lbeliveau sfinucan: moshele can't, we'll skip it this week13:04
*** victorhe has quit IRC13:04
*** dave-mccowan has joined #openstack-nova13:05
sfinucanlbeliveau: Cool cool :)13:05
*** mdnadeem has quit IRC13:05
*** foexle has quit IRC13:05
wznoinskterrible joke warning: this PF (moshele) should bond with someone on chairing this13:06
*** amoralej is now known as amoralej|lunch13:06
*** slaweq_ has joined #openstack-nova13:07
*** tbachman has quit IRC13:07
wznoinskI didn't have anymuch for this meeting, I guess tho if there's any urgency anyone wanted to use the meeting to bring attention to (i.e.: missing people and deadlines coming) we should ping ppl directly then13:07
openstackgerritjichenjc proposed openstack/nova: Add context param to confirm_migration virt call  https://review.openstack.org/38369913:07
openstackgerritFeodor Tersin proposed openstack/nova-specs: Add ScaleIO ephemeral storage backend  https://review.openstack.org/30471513:08
*** gjayavelu has joined #openstack-nova13:08
*** GB21 has joined #openstack-nova13:10
*** goutham has quit IRC13:11
*** gjayavelu has quit IRC13:13
openstackgerritjichenjc proposed openstack/nova: make purge_option to be default False  https://review.openstack.org/33277713:15
*** ayogi has quit IRC13:18
*** josecastroleon has joined #openstack-nova13:19
*** jwcroppe has quit IRC13:21
*** jwcroppe has joined #openstack-nova13:22
*** GB21 has quit IRC13:22
openstackgerritjichenjc proposed openstack/nova: api-ref: Body verification for os-simple-tenant-usage.inc  https://review.openstack.org/38368313:26
*** jwcroppe has quit IRC13:26
*** lamt has joined #openstack-nova13:28
*** tlian has joined #openstack-nova13:28
*** slaweq_ has quit IRC13:28
*** ociuhandu has quit IRC13:30
*** zul has quit IRC13:30
*** tojuvone has joined #openstack-nova13:31
*** Guest83036 has joined #openstack-nova13:32
*** sudipto_ has joined #openstack-nova13:32
*** zul has joined #openstack-nova13:33
*** mdrabe has joined #openstack-nova13:35
openstackgerritjichenjc proposed openstack/nova: Catch InstanceNotFound exception  https://review.openstack.org/40076313:35
*** thorst has joined #openstack-nova13:36
*** prateek has joined #openstack-nova13:40
jcookjohnthetubaguy pretty sure I've reviewed it up to the most recent changes. I think a core working on cells needs to take a look at it. I believe from an Ironic standpoint we're good in the current form, not so sure for a cells standpoint. I'm on holiday for the rest of the week. If there are any updates, I can take a looksee Monday.13:40
johnthetubaguyjcook: cool, have a good break13:41
jcookjohnthetubaguy o/13:41
johnthetubaguyjcook: I got more context reading the bug report13:41
*** tbachman has joined #openstack-nova13:41
*** thorst has quit IRC13:42
johnthetubaguyjcook: its has zero effect on cells v2, for sure, cells v1 wise I would need a closer look13:42
*** slaweq_ has joined #openstack-nova13:42
jcookjohnthetubaguy yeah tl;dr: Ironic must control the host resources in such a way to prevent sending builds to broken nodes. Overcommitments were broken because negative weights were stopped. This patch makes overcommits work without negative numbers.13:43
*** jwcroppe has joined #openstack-nova13:43
jcookjohnthetubaguy from a cells standpoint, the number of ram slots are calculated and passed up to the top level cells. However, there are some changes to not use ram slots but calculate free ram to allow overcommits. I'm not certain this won't affect cells v1. I'd have to really dig in.13:44
jcookmaybe it's more correct to say, some of the code is changed to calculate free based on used instead of pass it around and since cells v1 is still passing around free while the scheduler is calculating it, there's a possibility that something in the patch might unintentionally break cells v1. That said since the changes were focused on the scheduler, it might be fine.13:48
jcooksigning off o/13:49
openstackgerritDanil Akhmetov proposed openstack/nova: Use proper user and tenant in the owner section of libvirt.xml.  https://review.openstack.org/39967913:50
*** jgriffith has joined #openstack-nova13:52
*** slaweq_ has quit IRC13:53
*** yufei has quit IRC13:54
*** slaweq_ has joined #openstack-nova13:55
*** thorst has joined #openstack-nova13:55
openstackgerritjichenjc proposed openstack/nova: api-ref: Body verification for os-simple-tenant-usage.inc  https://review.openstack.org/38368313:55
*** lucas-hungry is now known as lucasagomes13:56
*** tbachman_ has joined #openstack-nova13:58
tdurakovPaulMurray paul-carlton2 eliqiao shaohe_feng pkoniszewski yuntongjin mdbooth kashyap kmartin claudiub tobasco tdurakov andreas_s abhishekk mrhillsman raj_singh wznoinsk siva_krishnan live migration subteam meeting in 3 mintues on #openstack-meeting-313:58
*** gongysh has joined #openstack-nova13:58
*** brault has joined #openstack-nova13:59
*** mlavalle has joined #openstack-nova14:00
*** ljxiash has joined #openstack-nova14:01
*** tbachman has quit IRC14:01
*** tbachman_ is now known as tbachman14:01
*** bkopilov has joined #openstack-nova14:01
*** sridharg has quit IRC14:02
*** sree has joined #openstack-nova14:03
openstackgerritPaul Carlton proposed openstack/nova: WIP: Live migration of instances with encrypted volumes  https://review.openstack.org/38960814:04
pkholkinnova-specs cores, please review https://review.openstack.org/#/c/38191214:04
*** gjayavelu has joined #openstack-nova14:06
*** siva_krish has joined #openstack-nova14:06
*** brault has quit IRC14:07
*** READ10 has joined #openstack-nova14:08
openstackgerritwang jiedong proposed openstack/nova: I think i didn't describe it clearly, the virt_type is kvm, and a parameter usbredir_channels is added in nova.conf after spice section, my total nova.conf like this:  https://review.openstack.org/40077914:08
*** moshele has joined #openstack-nova14:09
*** thorst has quit IRC14:09
*** bvanhav__ has joined #openstack-nova14:10
*** bvanhav__ is now known as bvanhav14:10
*** zul has quit IRC14:10
*** gjayavelu has quit IRC14:10
*** bvanhav_ has quit IRC14:10
*** thorst has joined #openstack-nova14:10
*** baoli has joined #openstack-nova14:12
*** daemontool has joined #openstack-nova14:13
*** sree has quit IRC14:13
*** zul has joined #openstack-nova14:14
*** alex_xu has quit IRC14:14
openstackgerritMoshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex  https://review.openstack.org/19948814:16
*** mriedem has joined #openstack-nova14:18
*** alex_xu has joined #openstack-nova14:18
*** amoralej|lunch is now known as amoralej14:18
openstackgerritSergey Nikitin proposed openstack/nova: Modify fields of 'Diagnostics' object  https://review.openstack.org/39448014:19
openstackgerritSergey Nikitin proposed openstack/nova: [WIP] Implement instance diagnostics for Xen  https://review.openstack.org/39961314:19
*** thorst has quit IRC14:19
openstackgerritCorentin Ardeois proposed openstack/nova: Add CORS filter to versions pipeline  https://review.openstack.org/40078514:19
*** cardeois has joined #openstack-nova14:22
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Reconnect volumes and encryptors during a hard reboot  https://review.openstack.org/40038414:23
*** sree has joined #openstack-nova14:25
mriedemyikes libvirt crash in a gate job http://logs.openstack.org/44/386844/17/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/91befad/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-11-22_00_27_46_57114:25
*** thorst has joined #openstack-nova14:26
*** nandal has quit IRC14:27
*** thorst has quit IRC14:27
mdboothmriedem: Don't suppose zuul keeps core dumps?14:28
mriedemidk, we have the libvirtd logs though14:28
mriedemand syslog14:28
*** thorst has joined #openstack-nova14:28
*** daemontool has quit IRC14:28
*** brault has joined #openstack-nova14:29
*** gouthamr has quit IRC14:30
*** zenoway has quit IRC14:30
mriedemmdbooth: there is a dump in syslog14:31
mriedemhttp://logs.openstack.org/44/386844/17/check/gate-tempest-dsvm-neutron-full-ubuntu-xenial/91befad/logs/syslog.txt.gz#_Nov_22_00_27_4614:31
*** zenoway has joined #openstack-nova14:31
mriedemNov 22 00:27:46 ubuntu-xenial-rax-ord-5717228 virtlogd[16875]: End of file while reading data: Input/output error14:31
mriedemNov 22 00:27:46 ubuntu-xenial-rax-ord-5717228 libvirtd[16847]: *** Error in `/usr/sbin/libvirtd': malloc(): memory corruption: 0x0000558ff1c7c800 ***14:31
*** gouthamr has joined #openstack-nova14:32
*** kaisers__ has quit IRC14:32
mdboothJust before that:14:33
mdbooth2016-11-22 00:27:36.668+0000: 16847: error : qemuMonitorIO:689 : internal error: End of file from monitor14:33
mdbooth2016-11-22 00:27:36.668+0000: 16847: debug : qemuMonitorIO:738 : Error on monitor internal error: End of file from monitor14:33
mdbooth2016-11-22 00:27:36.668+0000: 16847: debug : qemuMonitorIO:759 : Triggering EOF callback14:33
mdbooth2016-11-22 00:27:36.668+0000: 16847: debug : qemuProcessHandleMonitorEOF:285 : Received EOF on 0x7f4f300090e0 'instance-00000016'14:33
*** thorst has quit IRC14:33
mdboothI wonder if the qemu process died, triggering a bug in libvirtd14:33
*** brault has quit IRC14:33
*** zenoway has quit IRC14:37
*** zenoway has joined #openstack-nova14:37
mriedemonly 3 hits in 7 days in voting jobs14:38
mriedemso it's pretty rare14:38
mriedem12 hits in 7 days overall14:38
*** mjura has joined #openstack-nova14:39
kashyapHmm, it's the dreaded "main-loop: WARNING: I/O thread spun for 1000 iterations" error14:39
kashyapThere's some effort upstream QEMU to make debugging it easier14:40
mnestratovHi nova14:40
*** daemontool has joined #openstack-nova14:41
mnestratovmriedem: Could you take a look at https://review.openstack.org/#/c/282398/ when you have time?14:41
*** siva_krish has quit IRC14:42
*** gjayavelu has joined #openstack-nova14:42
mriedemmnestratov: you should probably tag that with a DocImpact also so that you can add a note to this docs page about requiring at least libvirt 1.2.12 and virtuozzo 7 http://docs.openstack.org/newton/config-reference/compute/hypervisor-virtuozzo.html14:44
*** siva_krish has joined #openstack-nova14:45
*** jamesdenton has joined #openstack-nova14:45
*** mjura has quit IRC14:47
mriedemmnestratov: comments inline14:47
*** gjayavelu has quit IRC14:47
*** prashkre has quit IRC14:48
*** burt has joined #openstack-nova14:48
mnestratovmriedem: ok makes sense, thanks14:48
*** jaosorior has quit IRC14:49
*** jaosorior has joined #openstack-nova14:50
*** rama_y has joined #openstack-nova14:51
*** jaosorior has quit IRC14:51
*** jaosorior has joined #openstack-nova14:51
*** victorhe has joined #openstack-nova14:52
*** brault has joined #openstack-nova14:52
mriedemmnestratov: here is that docs page too btw http://git.openstack.org/cgit/openstack/openstack-manuals/tree/doc/config-reference/source/compute/hypervisor-virtuozzo.rst14:55
kashyapmriedem: Was checking w/ a libvirt dev on OFTC network, and wondering if it's possible to run libvirtd under valgrind14:55
kashyapChecking w/ clark on -infra14:55
*** victorhe_ has joined #openstack-nova14:56
*** victorhe_ has quit IRC14:56
*** victorhe_ has joined #openstack-nova14:56
*** victorhe has quit IRC14:56
*** victorhe_ has quit IRC14:56
*** victorhe has joined #openstack-nova14:57
*** ekuris has quit IRC14:58
mnestratovmriedem: yeah, I know. I already had a chance to commit there, anyway thanks14:58
*** rama_y has quit IRC14:58
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: do not return serial address if disabled on destination  https://review.openstack.org/39727614:59
*** mjura has joined #openstack-nova15:00
*** anush has joined #openstack-nova15:00
*** brault has quit IRC15:01
*** Bofu2U has joined #openstack-nova15:02
*** kaisers_ has joined #openstack-nova15:03
*** karthiks has quit IRC15:04
*** claudiub has joined #openstack-nova15:05
*** nic has joined #openstack-nova15:06
*** nic has quit IRC15:06
*** claudiub|2 has quit IRC15:07
*** ljxiash_ has joined #openstack-nova15:10
*** ljxiash has quit IRC15:13
*** slaweq_ has quit IRC15:13
toabctlmriedem, hi. could you have another look at https://review.openstack.org/#/c/398308/ please?15:15
openstackgerritint32bit proposed openstack/nova: Trivial fix of indentation with 8 spaces in libvirt conf  https://review.openstack.org/39686315:16
*** thorst has joined #openstack-nova15:17
*** anush has quit IRC15:17
*** gjayavelu has joined #openstack-nova15:18
mriedemjaypipes: bauzas: johnthetubaguy: dansmith: i've got some questions in https://review.openstack.org/#/c/300178/ before we approve that spec15:19
mriedemedleafe-: ^15:19
*** slaweq_ has joined #openstack-nova15:19
*** karthiks has joined #openstack-nova15:20
*** tjones has joined #openstack-nova15:21
*** thorst has quit IRC15:21
mriedemtoabctl: yeah after my morning meetings15:21
toabctlmriedem, thanks!15:22
*** thorst has joined #openstack-nova15:22
*** sree has quit IRC15:22
*** yamahata has joined #openstack-nova15:22
*** sree has joined #openstack-nova15:22
*** gjayavelu has quit IRC15:23
*** sree has quit IRC15:27
*** thorst has quit IRC15:27
openstackgerritjichenjc proposed openstack/nova: no instance info cache update if instance deleted  https://review.openstack.org/36358515:28
jaypipesmriedem: k, will answer on the spec.15:28
*** hoonetorg has joined #openstack-nova15:37
*** mfeoktistov has joined #openstack-nova15:38
openstackgerritint32bit proposed openstack/nova: Trivial fix of indentation with 8 spaces in libvirt conf  https://review.openstack.org/39686315:39
*** unicell1 has joined #openstack-nova15:42
*** unicell has quit IRC15:42
*** xyang1 has joined #openstack-nova15:43
dansmithcarl_baldwin: if you could be less well-liked it would really help my self-esteem.. kthx.15:44
*** Alagar has quit IRC15:44
*** Alagar has joined #openstack-nova15:45
*** ayoung has quit IRC15:45
*** daemontool has quit IRC15:46
jaypipesbauzas: you around today?15:47
*** gongysh has quit IRC15:47
openstackgerritChris Dent proposed openstack/nova: [placement] increase gabbi coverage of handlers.allocation  https://review.openstack.org/39636315:48
openstackgerritChris Dent proposed openstack/nova: [placement] increase gabbi coverage of handlers.inventory  https://review.openstack.org/39636415:48
openstackgerritChris Dent proposed openstack/nova: [placement] increase gabbi coverage of handlers.resource_provider  https://review.openstack.org/39636515:48
*** ntpttr is now known as ntpttr-vacation15:52
*** liangy_ has joined #openstack-nova15:54
*** gjayavelu has joined #openstack-nova15:54
*** diga has joined #openstack-nova15:54
*** jaosorior has quit IRC15:55
openstackgerritMikhail Feoktistov proposed openstack/nova: Remove sata bus for virtuozzo hypervisor  https://review.openstack.org/28239815:55
*** dave-mccowan has quit IRC15:55
carl_baldwindansmith: lol15:55
dansmithcarl_baldwin:  :)15:56
*** gjayavelu has quit IRC15:58
jaypipesmriedem: responded to your review comments on https://review.openstack.org/#/c/391918. Thanks much for the review!16:00
*** ihrachys has quit IRC16:01
*** ihrachys has joined #openstack-nova16:01
*** anush has joined #openstack-nova16:01
*** ociuhandu has joined #openstack-nova16:02
*** owalsh is now known as owalsh-afk16:03
*** links has quit IRC16:04
alaskioh man, I didn't think to send a "so long and thanks for all the fish email" and now I'm afraid to. What if fewer people respond to mine...16:05
*** rcernin_ has quit IRC16:06
*** mfeoktistov has quit IRC16:06
jaypipesalaski: you should send that email. :)16:06
*** ayogi has joined #openstack-nova16:07
*** jschlueter has joined #openstack-nova16:08
dansmithalaski: super risky at this point based on carl_baldwin's count :/16:09
alaskidansmith: I know!16:09
mriedemalaski: you can just link to this https://www.youtube.com/watch?v=qIqeXSYc8nE16:09
*** nic has joined #openstack-nova16:09
alaskihah16:10
openstackgerritint32bit proposed openstack/python-novaclient: Show non-owner project servers  https://review.openstack.org/39781516:10
*** dave-mccowan has joined #openstack-nova16:10
*** zenoway has quit IRC16:11
*** zenoway has joined #openstack-nova16:12
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: Acquire TCP ports for console during live migration  https://review.openstack.org/27580116:13
openstackgerritMarkus Zoeller (markus_z) proposed openstack/nova: libvirt: prepare domain XML update for serial ports  https://review.openstack.org/40084316:13
*** psachin has quit IRC16:13
markus_zpkoniszewski: tdurakov: ^ Let me know your concerns here. In parallel I'm double-checking them in a real setup.16:14
*** Guest3787 has quit IRC16:16
*** zenoway has quit IRC16:16
*** ociuhandu has quit IRC16:17
*** oanson has quit IRC16:17
*** belmoreira has quit IRC16:18
*** mjura has quit IRC16:19
*** ociuhandu has joined #openstack-nova16:19
*** armax has joined #openstack-nova16:19
*** ayogi has quit IRC16:21
*** stewie925 has joined #openstack-nova16:22
openstackgerritint32bit proposed openstack/python-novaclient: Show other project servers using --all-tenants option  https://review.openstack.org/39781516:23
stewie925gmann: am requesting for your review of https://review.openstack.org/#/c/338606/?    thank you16:24
stewie925https://review.openstack.org/#/c/338606/16:24
*** rdopiera has left #openstack-nova16:25
openstackgerritMichal Pryc proposed openstack/python-novaclient: Respect upper constraints file while installing dependencies.  https://review.openstack.org/40084816:25
*** diogogmt has joined #openstack-nova16:25
*** sandanar has quit IRC16:26
*** ayoung has joined #openstack-nova16:27
*** claudiub|2 has joined #openstack-nova16:28
*** ociuhandu has quit IRC16:29
openstackgerritDiana Clarke proposed openstack/nova: api-ref: add missing os-server-groups parameters  https://review.openstack.org/40085116:29
*** ociuhandu has joined #openstack-nova16:29
openstackgerritMichal Pryc proposed openstack/python-novaclient: Respect upper constraints file while installing dependencies.  https://review.openstack.org/40084816:29
*** hongbin has joined #openstack-nova16:30
*** gjayavelu has joined #openstack-nova16:30
*** Alagar has quit IRC16:30
*** Alagar has joined #openstack-nova16:31
*** claudiub has quit IRC16:31
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Use the mirror ready element to detect when to pivot  https://review.openstack.org/39973516:31
openstackgerritMoshe Levi proposed openstack/nova: Enhance PCI passthrough whitelist to support regex  https://review.openstack.org/19948816:32
*** moshele has quit IRC16:34
openstackgerritRoman Podoliaka proposed openstack/nova: Integrate OSProfiler and Nova  https://review.openstack.org/25470316:34
*** gjayavelu has quit IRC16:35
*** claudiub|2 has quit IRC16:37
*** brault has joined #openstack-nova16:38
openstackgerritPaul Carlton proposed openstack/nova: WIP: Live migration of instances with encrypted volumes  https://review.openstack.org/38960816:40
openstackgerritPaul Carlton proposed openstack/nova: WIP libvirt: Reconnect volumes and encryptors during a hard reboot  https://review.openstack.org/40038416:40
lyarwoodpaul-carlton: ^ was that just a rebase? I'll sort the commit message out now16:42
*** brault has quit IRC16:42
*** paul-carlton has quit IRC16:43
*** abalutoiu has joined #openstack-nova16:45
*** paul-carlton has joined #openstack-nova16:45
paul-carltonlyarwood, yep16:46
*** rmart04_ has joined #openstack-nova16:46
mriedemtoabctl: comments inline, the test needs some work16:46
mriedemtoabctl: basically either use the db or don't, but not both16:46
paul-carltonlyarwood, I rebased my live mig fix ontop of your reboot fix, then removed some code in my fix and pushed it16:46
paul-carltonlyarwood, it seems to work now, will work on tests and tidy up code tomorrow16:47
lyarwoodpaul-carlton: cool, I'll sort my commit out now so just rebase onto it in the morning16:47
lyarwoodpaul-carlton: thanks again for testing this16:47
*** mvk has quit IRC16:48
paul-carltonmriedem, can you take a look at https://review.openstack.org/#/q/status:open+topic:bp/detach-boot-volume when you get time pls16:48
toabctlmriedem, is there a method to create a fake instance?16:48
paul-carltonlyarwood, will do16:48
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Mock imagebackend template funcs in ImageBackendFixture  https://review.openstack.org/40087116:48
mriedemtoabctl: nova.tests.fake_instance.fake_instance_obj?16:48
mriedem*nova.tests.unit16:48
*** rmart04 has quit IRC16:49
*** rmart04_ is now known as rmart0416:49
*** matrohon has quit IRC16:49
*** paul-carlton has quit IRC16:50
*** rama_y has joined #openstack-nova16:50
*** abalutoiu has quit IRC16:50
markus_zhttps://review.openstack.org/#/c/385367/ needs a 2nd +2 (that's a prep step for the virtlogd feature). danpb already approved.16:52
*** siva_krish has quit IRC16:53
*** mhickey has quit IRC16:53
*** siva_krish has joined #openstack-nova16:54
openstackgerritLee Yarwood proposed openstack/nova: libvirt: Reconnect volumes and encryptors during a hard reboot  https://review.openstack.org/40038416:55
*** nherciu_ has joined #openstack-nova16:57
*** bvanhav has quit IRC16:57
*** nherciu has quit IRC16:57
*** kaisers_ has quit IRC16:58
openstackgerritMatthew Booth proposed openstack/nova: libvirt: Add create_from_image and create_from_func to Backend  https://review.openstack.org/33324416:59
*** siva_krish has quit IRC17:01
*** browne has joined #openstack-nova17:01
openstackgerritsahid proposed openstack/nova: objects: add new field cpu_emulator_threads_policy  https://review.openstack.org/38502217:01
openstackgerritsahid proposed openstack/nova: numa: add numa constraints for emulator threads policy  https://review.openstack.org/38502317:01
openstackgerritsahid proposed openstack/nova: libvirt: return a CPU overhead if isolate emulator threads requested  https://review.openstack.org/38536417:01
openstackgerritsahid proposed openstack/nova: virt: update overhead to take into account vCPUs  https://review.openstack.org/38502417:02
openstackgerritsahid proposed openstack/nova: objects: add new field cpus_overhead in NUMACell  https://review.openstack.org/40087417:02
openstackgerritsahid proposed openstack/nova: numa: fit instance toplogy with cpu overhead onto cores  https://review.openstack.org/40087517:02
openstackgerritsahid proposed openstack/nova: numa: take into account cpus overhead  https://review.openstack.org/40087617:02
openstackgerritsahid proposed openstack/nova: numa: update numa usage incording the cpu overhead  https://review.openstack.org/40087717:02
openstackgerritsahid proposed openstack/nova: libvirt: make emulator threads to run on the reserved pCPU  https://review.openstack.org/40087817:02
mdboothKABOOM!17:02
macszwoah, what a chain :D17:02
*** Alagar has quit IRC17:02
mdboothsahid: FYI there used to be a race in jenkins/zuul which meant that after the first couple in the chain all the remaining get a false merge conflict17:03
mriedemmdbooth: i pushed up a large chain the other night and didn't hit that issue17:03
mriedemmdbooth: so seems to be fixe17:04
mriedem*fixed17:04
mdboothMy solution was to git rebase -i ... -x "git -y review; sleep 20"17:04
mdboothmriedem: \o/17:04
*** anush has quit IRC17:04
sahidmdbooth: oh interesting17:05
*** slaweq_ has quit IRC17:05
mdboothsahid: mriedem reports it's fixed. Be interested to know if you get that.17:05
* mdbooth used to get it reliably every time.17:05
*** fragatina has quit IRC17:06
*** gjayavelu has joined #openstack-nova17:06
openstackgerritint32bit proposed openstack/nova-specs: Add lock action record  https://review.openstack.org/39703417:06
sahidmdbooth: ok so let see what will happen17:06
dansmithmdbooth: does that work? I thought git review won't submit if you're mid-rebase17:06
mdboothdansmith: WFM.17:07
mdboothI do it all the time.17:07
dansmithhrm17:07
mdboothI mean manually, without the sleep thing.17:07
dansmithyeah, I would like to, but ..17:07
mdboothi.e. when I want to go back and push only the first couple in my series17:07
mdboothI rebase -i back to where I want to submit up to, and do git review there.17:07
*** tbachman has quit IRC17:07
mdboothThen continue17:08
dansmithmdbooth: http://pastebin.com/Zs9uvhUn17:08
dansmiththat's what I get when I try17:08
dansmithah, but if I use -R it'll do it it looks like17:08
mdboothdansmith: Ah, I don't auto-rebase17:08
dansmithyour config must be .. yeah17:08
dansmithwell, that's a nifty trick17:08
mdbooth[gitreview]17:08
mdbooth    rebase = false17:08
dansmithyeah17:09
dansmithI know you can, just didn't really think about that being the thing that would let me do that17:09
dansmiththat's super handy17:09
*** gjayavelu has quit IRC17:10
*** WillUK2010 has quit IRC17:11
gibimriedem, johnthetubaguy: hi! there is quite a lot of notification transformation patch waiting for core review, list on the etherpad https://etherpad.openstack.org/p/ocata-nova-priorities-tracking is up to date. Could you spend some time on them?17:11
*** slaweq_ has joined #openstack-nova17:14
*** bvanhav has joined #openstack-nova17:14
openstackgerritScott Moser proposed openstack/nova: Limit exposure of network device types to the guest.  https://review.openstack.org/40088317:16
*** sahid has quit IRC17:19
*** giri has joined #openstack-nova17:19
*** paul-carlton has joined #openstack-nova17:19
openstackgerritScott Moser proposed openstack/nova: Limit exposure of network device types to the guest.  https://review.openstack.org/40088317:19
openstackgerritint32bit proposed openstack/nova-specs: Add lock action record  https://review.openstack.org/39703417:19
*** weshay is now known as weshay_brb17:20
*** markus_z has quit IRC17:20
*** nkrinner is now known as nkrinner_afk17:22
*** lucasagomes is now known as lucas-afk17:25
openstackgerritLudovic Beliveau proposed openstack/nova: PCI: Release PCI devices on drop_move_claim()  https://review.openstack.org/37037417:27
mriedemgibi: ok17:27
*** josecastroleon has quit IRC17:28
*** siva_krish has joined #openstack-nova17:28
*** weshay_brb is now known as weshay17:28
*** penick has joined #openstack-nova17:28
*** owalsh-afk is now known as owalsh17:29
*** moshele has joined #openstack-nova17:29
*** rmart04 has quit IRC17:30
gibimriedem: thanks17:31
*** jvgrant_ has quit IRC17:32
*** whenry has quit IRC17:32
*** Bofu2U has quit IRC17:33
*** unicell1 has quit IRC17:35
* dansmith sniffs17:36
*** prashkre has joined #openstack-nova17:36
snetijohnthetubaguy, can you please review these simple bug fixes: https://review.openstack.org/#/c/384724/ https://review.openstack.org/#/c/392305/17:37
*** jvgrant has joined #openstack-nova17:37
*** Alagar has joined #openstack-nova17:39
*** mvk has joined #openstack-nova17:41
*** nherciu_ has quit IRC17:41
*** Guest83036 has quit IRC17:44
*** sudipto_ has quit IRC17:44
cardeoisHello there ! If anybody have some time to review this https://review.openstack.org/#/c/400785/ that would really help to unblock me17:44
cardeoisAnd I'll be glad to answer questions if any17:45
johnthetubaguysneti: will try put those in my queue, but the queue is looking a bit crazy right now I am afraid17:49
*** anush has joined #openstack-nova17:49
*** tbachman has joined #openstack-nova17:49
openstackgerritint32bit proposed openstack/python-novaclient: Show other project servers using --all-tenants option  https://review.openstack.org/39781517:49
*** giri has quit IRC17:51
*** moshele has quit IRC17:51
*** lpetrut has quit IRC17:53
*** nherciu has joined #openstack-nova17:54
*** diogogmt has quit IRC17:56
*** raildo has quit IRC17:57
*** weshay is now known as weshay1-117:59
*** kaisers_ has joined #openstack-nova17:59
*** fragatina has joined #openstack-nova17:59
*** diogogmt has joined #openstack-nova17:59
*** raildo has joined #openstack-nova17:59
*** gjayavelu has joined #openstack-nova18:01
*** derekh has quit IRC18:01
*** penick has quit IRC18:02
*** baoli has quit IRC18:02
*** ralonsoh has quit IRC18:03
*** baoli has joined #openstack-nova18:03
*** gszasz has quit IRC18:04
*** penick has joined #openstack-nova18:04
*** gjayavelu has quit IRC18:06
*** unicell has joined #openstack-nova18:10
*** tbachman has quit IRC18:11
snetijohnthetubaguy, np..please do when you have time.18:11
*** Alagar has quit IRC18:11
*** rama_y has quit IRC18:14
*** Alagar has joined #openstack-nova18:14
*** prateek has quit IRC18:16
*** rmart04 has joined #openstack-nova18:19
*** ihrachys has quit IRC18:19
*** weshay1-1 is now known as weshay18:20
*** rama_y has joined #openstack-nova18:21
*** gjayavelu has joined #openstack-nova18:22
*** yamahata has quit IRC18:24
*** unicell has quit IRC18:26
*** gjayavelu has quit IRC18:26
*** anush has quit IRC18:27
pkholkincores, please review the spec https://review.openstack.org/#/c/38191218:27
openstackgerritjichenjc proposed openstack/nova: no instance info cache update if instance deleted  https://review.openstack.org/36358518:28
*** oanson has joined #openstack-nova18:28
*** tbachman has joined #openstack-nova18:29
*** penick has quit IRC18:29
*** rmart04_ has joined #openstack-nova18:34
*** penick has joined #openstack-nova18:34
*** thorst has joined #openstack-nova18:34
*** rmart04 has quit IRC18:36
*** rmart04_ is now known as rmart0418:36
*** thorst has quit IRC18:39
*** thorst has joined #openstack-nova18:40
*** paul-carlton has quit IRC18:40
*** diogogmt has quit IRC18:43
*** Guest6312 has quit IRC18:45
*** diogogmt has joined #openstack-nova18:45
*** thorst has quit IRC18:45
*** lpetrut has joined #openstack-nova18:45
*** Alagar has quit IRC18:46
*** penick has quit IRC18:47
*** oanson has quit IRC18:47
*** brault has joined #openstack-nova18:48
*** slaweq_ has quit IRC18:48
*** Sukhdev has joined #openstack-nova18:48
*** slaweq_ has joined #openstack-nova18:48
*** Alagar has joined #openstack-nova18:49
*** anush has joined #openstack-nova18:49
*** penick has joined #openstack-nova18:50
*** jgriffith has quit IRC18:51
*** READ10 has quit IRC18:51
*** brault has quit IRC18:52
*** unicell has joined #openstack-nova18:54
*** diogogmt has quit IRC18:55
*** ijw has joined #openstack-nova18:56
openstackgerritMaciej Szankin proposed openstack/nova: Optimize creating security_group  https://review.openstack.org/31531118:59
*** diogogmt has joined #openstack-nova19:00
*** prashkre has quit IRC19:00
*** AAR has joined #openstack-nova19:02
*** ijw has quit IRC19:03
AARwhat are the networking requirements to have successful pin between VMs on different compute nodes?19:03
*** yamahata has joined #openstack-nova19:04
*** slaweq_ has quit IRC19:06
*** liangy_ has quit IRC19:08
artomAAR, they'd need to be on the same (vitual) network and have security group (essentially firewall) rules that allow it19:10
artomAAR, in the future, #openstack is a better channel for this kind of question, this channel is more about development of Nova19:10
AARok. great,. thanks alot.19:10
*** AAR has left #openstack-nova19:12
*** ijw has joined #openstack-nova19:12
*** AAR has joined #openstack-nova19:13
*** AAR has left #openstack-nova19:13
*** bvanhav has quit IRC19:20
*** ducttape_ has joined #openstack-nova19:21
*** Alagar has quit IRC19:22
*** brault has joined #openstack-nova19:23
*** matrohon has joined #openstack-nova19:28
*** thorst has joined #openstack-nova19:29
*** bvanhav has joined #openstack-nova19:29
mriedemtoabctl: have you seen this? https://review.openstack.org/#/c/363585/19:30
mriedembasically the same fix for your bug, but showed up in august19:31
*** thorst has quit IRC19:33
*** moshele has joined #openstack-nova19:34
*** moshele has quit IRC19:41
*** nherciu has quit IRC19:44
*** WillUK2010 has joined #openstack-nova19:45
mriedemgibi: i'd like to see a small change here https://review.openstack.org/#/c/384621/19:46
*** amoralej is now known as amoralej|off19:46
mriedemgibi: if someone can't make it today yet i can make the change quick and then +219:46
*** WillUK2010 has quit IRC19:47
*** odyssey4me has quit IRC19:56
*** oanson has joined #openstack-nova19:57
*** diogogmt has quit IRC20:00
*** baoli has quit IRC20:01
openstackgerritJay Pipes proposed openstack/nova-specs: resource-providers: Scheduler filters in DB  https://review.openstack.org/30017820:01
jaypipesmriedem: ^20:01
*** odyssey4me has joined #openstack-nova20:02
mriedemack20:03
openstackgerritMatt Riedemann proposed openstack/nova: Transform instance.create.error notification  https://review.openstack.org/39771520:04
openstackgerritMatt Riedemann proposed openstack/nova: Transform instance.create notification  https://review.openstack.org/38462120:04
*** rfolco has quit IRC20:04
mriedem+2 on https://review.openstack.org/#/c/384621/ btw for someone else to grab20:04
*** sambetts is now known as sambetts|afk20:05
*** baoli has joined #openstack-nova20:05
*** swamireddy has quit IRC20:05
mriedemdansmith: would like you to weigh in on this https://review.openstack.org/#/c/300178/6/specs/ocata/approved/resource-providers-scheduler-db-filters.rst@6420:07
*** dave-mccowan has quit IRC20:10
*** catinthe_ has joined #openstack-nova20:11
mriedemjaypipes: repied a bit in ^20:12
mriedem*replied20:12
mriedemjaypipes: i can sort of see the self-healing aspect, basically by starving out a small subset of computes that are registered with placement but are so full they can't satisfy any requests,20:12
mriedemwhich makes us fall back to doing the old db get_all() lookup20:12
*** catintheroof has quit IRC20:13
* dansmith looks20:14
jaypipesmriedem: right, for Ocata, and then in Pike, remove that.20:14
mriedemand in pike we drop ram/cpu/disk filters from the default list of scheduler filters?20:15
jaypipesmriedem: yes.20:15
*** catintheroof has joined #openstack-nova20:16
*** kaisers_ has quit IRC20:16
mriedemok. in multi-node grenade that would also be tested b/c we'd have an n-1 newton compute node that's not registered with the placement service,20:16
mriedemand an ocata node that is20:16
jaypipescorrect.20:17
mriedemok unless dansmith can poke holes in that idea then i'm on board20:17
dansmithmriedem: I guess I'm more in favor of executing a more immediate takeover, but either way is fine20:19
dansmithwe're just kicking the can further down the street if we fail too gracefully20:19
*** ltomasbo has quit IRC20:19
*** catinthe_ has quit IRC20:20
*** jschwarz has quit IRC20:20
mriedemok, i just wanted to avoid NoValidHost out of the box on upgrade to ocata20:21
dansmithyeah20:21
mriedemi mean, failure as a service is fun20:21
dansmiths'what I would have done, but I know the desire is to fail gracefully20:21
dansmithwe should splatter red all over the logs if we're falling back for sure20:21
dansmithlike real hurtful, hateful shit to make sure it gets seen20:22
*** markmc has quit IRC20:22
mriedem'these filters are deprecated and redundant, just like that novel you've been working on for 10 years, dave.'20:22
dansmithhaha20:22
mriedemjaypipes: want to throw the high level fallback plan in the spec so it's not lost as bauzas isn't here?20:23
*** ajo has quit IRC20:25
*** ltomasbo has joined #openstack-nova20:25
*** ajo has joined #openstack-nova20:25
*** markmc has joined #openstack-nova20:25
jaypipesmriedem: you got it. gimme five.20:26
*** jschwarz has joined #openstack-nova20:26
*** kaisers_ has joined #openstack-nova20:32
*** bvanhav has quit IRC20:34
*** mhickey has joined #openstack-nova20:34
*** pradk has quit IRC20:35
*** diogogmt has joined #openstack-nova20:37
*** raildo has quit IRC20:39
*** lpetrut has quit IRC20:42
openstackgerritJay Pipes proposed openstack/nova-specs: resource-providers: Scheduler filters in DB  https://review.openstack.org/30017820:43
jaypipesmriedem, dansmith: added into the Deployer Impact section ^^20:43
mriedemthakns20:43
mriedem*thanks even20:43
*** kaisers_ has quit IRC20:44
*** kaisers_ has joined #openstack-nova20:45
jaypipesmriedem: hmm, nice catch on the ovo 1.17 thing. thought those fields were added ages ago... guess I was wrong. :(20:45
mriedem\o/20:45
openstackgerritMatt Riedemann proposed openstack/nova: Remove redundant VersionedObject Fields  https://review.openstack.org/38984020:48
mriedemjaypipes: you can +2 ^ added the depends-on to the g-r bump20:48
jaypipesmriedem: done.20:48
jaypipesgibi: we're plowing through versioned notification patches today :)20:49
mriedemhttps://www.youtube.com/watch?v=uYXEt7xOh1M ?20:50
*** penick has quit IRC20:50
*** penick has joined #openstack-nova20:51
jaypipesheh20:51
*** pradk has joined #openstack-nova20:52
*** oanson has quit IRC20:53
mriedemdansmith: want to +W this? https://review.openstack.org/#/c/300178/20:54
*** oanson has joined #openstack-nova20:54
*** mhickey has quit IRC20:55
*** ayoung has quit IRC20:55
*** ayoung has joined #openstack-nova20:56
mriedemgibi: didn't you ask me something about sending the new flavor in the resize notifications?20:57
*** rtheis has quit IRC20:57
mriedemoh that was for resize.prep.end i think20:58
mriedemb/c extra_usage_info contains the new flavor we're resizing to20:58
*** ayoung has quit IRC21:01
*** baoli has quit IRC21:08
-openstackstatus- NOTICE: Gerrit is offline until 21:30 UTC for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107379.html21:09
*** ChanServ changes topic to "Gerrit is offline until 21:30 UTC for scheduled maintenance: http://lists.openstack.org/pipermail/openstack-dev/2016-November/107379.html"21:09
*** slaweq_ has joined #openstack-nova21:10
*** jamesdenton has quit IRC21:11
*** jgriffith has joined #openstack-nova21:13
*** Apoorva has joined #openstack-nova21:14
dansmithoh ffs21:14
dansmitha week to get that one patch merged and now I can't push up the updated set21:15
openstackgerritDan Smith proposed openstack/nova: Return build_requests instead of instances  https://review.openstack.org/37426721:16
openstackgerritDan Smith proposed openstack/nova: Refactor two nearly useless secgroup tests  https://review.openstack.org/39343821:16
openstackgerritDan Smith proposed openstack/nova: Make RPCFixture support multiple connections  https://review.openstack.org/39641721:16
dansmithoh good it let me21:16
macszsomehow it worked :D21:16
*** rmart04 has quit IRC21:16
dansmithmriedem: melwitt: jaypipes would like to get these refactors in ^21:16
openstackgerritMatt Riedemann proposed openstack/nova: Transform instance.finish_resize notifications  https://review.openstack.org/39095021:17
*** jgriffith has quit IRC21:17
*** jgriffith has joined #openstack-nova21:20
mriedemi've been meaning to get to https://review.openstack.org/#/c/374267/21:22
mriedemso, you know, i have good intentions21:22
melwittI'm looking at it now also21:23
*** oanson has quit IRC21:24
*** jgriffith has quit IRC21:29
*** xinliang has quit IRC21:32
*** Jeffrey4l has quit IRC21:34
*** Jeffrey4l has joined #openstack-nova21:35
*** ChanServ changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Ocata_Release_Schedule"21:40
*** jwcroppe has quit IRC21:41
*** jwcroppe has joined #openstack-nova21:41
*** jgriffith has joined #openstack-nova21:44
*** jwcroppe has quit IRC21:45
*** ayoung has joined #openstack-nova21:46
*** xinliang has joined #openstack-nova21:46
*** gouthamr has quit IRC21:49
*** baoli has joined #openstack-nova21:50
*** whenry has joined #openstack-nova21:52
*** tblakes has joined #openstack-nova21:54
*** matrohon has quit IRC21:56
*** gjayavelu has joined #openstack-nova22:00
*** kaisers_ has quit IRC22:01
mriedemdansmith: i think that change has been sufficiently hammered into the ground22:06
*** jgriffith has quit IRC22:06
dansmithmriedem: yep, thanks22:06
*** jgriffith_ has joined #openstack-nova22:07
dansmithwould have liked that last week, but I'll take it whenever :)22:07
mriedemspecs were last week, it's impossible to do both22:07
dansmithyeah just saying I'm going to work up some of these changes and then go away for a while22:11
*** jgriffith_ is now known as jgriffith22:13
mriedemi'm happily +2 on https://review.openstack.org/#/c/393438/22:16
*** tblakes has quit IRC22:18
*** nicolasbock has quit IRC22:18
*** lamt has quit IRC22:20
*** anush has quit IRC22:21
dansmithyeah, I meant to put that at the bottom, and the rpc thing too22:21
*** gjayavelu has quit IRC22:23
*** sree has joined #openstack-nova22:23
mriedemdansmith: something confusing me in here https://review.openstack.org/#/c/396417/22:26
mriedemdoesn't look like we actually even use RPCFixture anyway in our existing tests22:27
*** pradk has quit IRC22:28
mriedemoh weird nevermind22:28
*** sree has quit IRC22:28
*** Apoorva has quit IRC22:28
*** Apoorva has joined #openstack-nova22:28
*** Apoorva_ has joined #openstack-nova22:29
*** kaisers_ has joined #openstack-nova22:32
*** rama_y has quit IRC22:33
*** Apoorva has quit IRC22:33
dansmithmelwitt: mriedem: so, the double call on the bdm validation thing22:33
dansmiththere is something really obscure, probably because of the clones and copies,22:33
dansmithwhich is causing the BDM objects to lose their _context between setting them on the reqspec and doing the create down where you noticed the dup22:33
dansmithsince it's not a lot of work can we just leave that since it all becomes swizzled in the later patch to move create out/22:34
dansmithI just don't want to debug that whole thing to avoid calling that validate thing twice right now22:34
mriedems/reqspec/build_request/?22:34
*** Apoorva has joined #openstack-nova22:35
dansmithyes22:35
mriedemhmm, so _bdm_validate_set_size_and_instance calls instance_block_device_mapping = block_device_mapping.obj_clone()22:36
mriedemsets the size and instance_uuid on the bdm clone22:36
mriedemand returns that, which is then set in the build_request22:36
dansmithyep, which should be available to us right afterwards22:36
*** Apoorva has quit IRC22:36
*** Apoorva__ has joined #openstack-nova22:36
dansmithif we do that, we get:22:37
dansmithoslo_versionedobjects.exception.OrphanedObjectError: Cannot call update_or_create on orphaned BlockDeviceMapping object22:37
*** Apoorva__ has quit IRC22:37
dansmithon a couple hundred tests22:37
mriedemthe obj_clone() drops the context?22:37
*** Apoorva_ has quit IRC22:37
dansmithno22:37
*** Apoorva has joined #openstack-nova22:37
dansmithbut in _create_block_device_mapping we also copy it,22:38
dansmithbut it's None there already22:38
dansmithso I guess the answer is no22:38
*** WillUK2010 has joined #openstack-nova22:38
dansmithI'll keep going22:38
*** WillUK2010 has quit IRC22:38
mriedemwell, in _provision_instances we set the BlockDeviceMappingList on the BuildRequest object, then eventually call BuildRequest.create() which serializes the BlockDeviceMappingList22:39
*** Apoorva has quit IRC22:39
*** Apoorva has joined #openstack-nova22:40
mriedemthen repopulates the BuildRequest from what's in the db, does the bdm context get lost there?22:40
dansmithprobably22:40
mriedemyeah22:40
mriedemself.block_device_mappings = (22:40
mriedem            objects.BlockDeviceMappingList.obj_from_primitive(22:40
mriedem                jsonutils.loads(db_bdms)))22:40
mriedemso there you go22:40
mriedemwe could put the context back on those hydrated internal object fields but not sure that's wise22:41
dansmithit's just that it goes away here: https://review.openstack.org/#/c/319379/33/nova/compute/api.py22:42
*** Apoorva has quit IRC22:42
dansmithbut it's fine22:42
mriedemyou could return the cloned block_device_mapping list from _provision_instances along with the instances_to_build mega tuple o doom22:42
*** Apoorva_ has joined #openstack-nova22:42
mriedemi'd vote to either pass the cloned block_device_mapping list back out of _provision_instances along with the instances_to_build tuple, or if we're going to just call _bdm_validate_set_size_and_instance again later, add a TODO to remove it when we move to conductor22:43
*** anush has joined #openstack-nova22:44
mriedemTODO(danms): temporary and transitional, this goes away when we move this to conductor22:44
*** Apoorva has joined #openstack-nova22:44
mriedemsomething like that22:44
mriedemeither / or22:44
*** Apoorva__ has joined #openstack-nova22:45
mriedemdansmith: this conductor method doesn't actually exist yet right? https://review.openstack.org/#/c/319379/33/nova/compute/api.py@117822:46
mriedemdon't see it in that change or before it in the series22:46
dansmithmriedem: there's a patch up for it yeah22:46
mriedemoh https://review.openstack.org/#/c/367557/22:46
*** Apoorva_ has quit IRC22:47
mriedemew gross so yeah it copies _bdm_validate_set_size_and_instance to https://review.openstack.org/#/c/367557/20/nova/conductor/manager.py22:48
*** Apoorva has quit IRC22:49
*** Apoorva__ has quit IRC22:49
mriedemwhich means we lose the bdm validation that happens in the API in _bdm_validate_set_size_and_instance22:49
mriedembecause now it's going to happen in conductor after we've cast off from the scheduler22:49
mriedemnot sure i like that22:49
mriedemno i guess we still validate in api https://review.openstack.org/#/c/319379/33/nova/compute/api.py@95022:50
mriedemwhen we set the bdms on the build request22:50
dansmithmriedem: no we still call it in api22:50
mriedemthen we cast off to conductor22:50
mriedemwhich then redoes that same stuff all over again... :/22:50
melwittcan we just not do it again if we know it was fine in the API? or could something have changed22:52
dansmithmelwitt: there's something else going on between those two calls that I'm not seeing22:52
dansmithjust moving it, after fixing the context, ands up with still a hundred test fails in the db layer22:53
melwittI mean leave the API one and take out the one that would move to conductor22:53
dansmiththat's what I'm doing22:54
melwittoh. hmm22:54
*** victorhe_ has joined #openstack-nova22:54
*** burt has quit IRC22:55
melwittI guess it does say "validate and set" so it sets something that isn't getting preserved wherever bdm is being taken out the second time22:55
mriedem_create_block_device_mapping copies the bdmlist again22:56
mriedembut should be fine if the context was set before22:57
dansmithmriedem: you already figured out the context thing22:57
dansmithnow there's something up with the changes list not being set correctly22:57
mriedemwouldn't just simply returning that cloned block_device_mapping_list from _provision_instances make this all go away?22:57
dansmithwhich is causing a failure to actually create them later22:57
mriedemthat's the list we want to work with22:57
dansmithmriedem: then we have two lists that look like they're the same, but are actually separate22:58
dansmithwhich is pretty gross22:58
*** victorhe has quit IRC22:58
mriedempsh22:58
dansmithI mean, it's what we have now and what you're opposed to22:58
mriedemwe already have that for security_groups too :)22:58
dansmithjust changes where those come from22:58
mriedeminstances_to_build, updated_block_device_mappings = self._provision_instances(...)22:58
mriedemftw22:58
mriedem*validated_block_device_mappings22:59
*** gjayavelu has joined #openstack-nova22:59
mriedemor maybe bdms_to_create22:59
mriedemto match instances_to_build22:59
*** victorhe has joined #openstack-nova22:59
mriedemanyway, i think you get the gist. i think i have to head home.23:00
*** xyang1 has quit IRC23:00
*** jklare has quit IRC23:01
*** victorhe_ has quit IRC23:01
mriedemdansmith: are you around tomorrow?23:01
dansmithI dunno, I might ragequit between now and then23:01
mriedemthat's fair23:02
*** mriedem has quit IRC23:02
*** jklare has joined #openstack-nova23:03
*** victorhe has quit IRC23:03
openstackgerritLudovic Beliveau proposed openstack/nova: PCI: Race condition between audit and migrate/resize revert  https://review.openstack.org/40099523:04
*** ducttape_ has quit IRC23:04
*** tbachman has quit IRC23:10
openstackgerritsean mooney proposed openstack/os-vif: remove use of contextlib and with nested  https://review.openstack.org/38272523:10
*** baoli has quit IRC23:18
*** jamielennox is now known as jamielennox|away23:24
*** jamielennox|away is now known as jamielennox23:24
*** victorhe has joined #openstack-nova23:29
*** pradk has joined #openstack-nova23:33
*** victorhe has quit IRC23:34
*** browne has quit IRC23:36
*** baoli has joined #openstack-nova23:40
*** baoli has quit IRC23:40
*** tbachman has joined #openstack-nova23:42
*** catintheroof has quit IRC23:46
*** catintheroof has joined #openstack-nova23:47
*** catintheroof has quit IRC23:51
*** hongbin has quit IRC23:51
*** penick has quit IRC23:53
dansmithmelwitt: the test for that clone-like method explains why the original author did it23:53
dansmithto dirty the fields.. the same problem as above with BDMs, effectively23:53
melwittdansmith: oh. I wish they had included a code comment23:54
*** victorhe has joined #openstack-nova23:55
melwittdansmith: so it's done to make a save() work?23:58
dansmithcreate, but yeah23:58
eandersson_What is the cleanest way of deleting old hypervisors in Mitaka?23:59
eandersson_Do you still need to ninja the database?23:59

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