| *** itlinux has joined #openstack-nova | 00:03 | |
| *** hamzy_ has joined #openstack-nova | 00:05 | |
| openstackgerrit | Merged openstack/nova master: api-ref: Replace non UUID string with UUID https://review.openstack.org/608854 | 00:09 |
|---|---|---|
| *** moshele has quit IRC | 00:11 | |
| *** lbragstad has joined #openstack-nova | 00:16 | |
| *** gyee has quit IRC | 00:20 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add compute version 36 to support ``volume_type`` https://review.openstack.org/579360 | 00:22 |
| *** k_mouza has quit IRC | 00:24 | |
| *** tetsuro has joined #openstack-nova | 00:24 | |
| *** cfriesen has quit IRC | 00:27 | |
| openstackgerrit | iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091 | 00:34 |
| *** Dinesh_Bhor has joined #openstack-nova | 00:38 | |
| *** ircuser-1 has joined #openstack-nova | 00:45 | |
| *** Dinesh_Bhor has quit IRC | 00:49 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:52 | |
| *** tetsuro has quit IRC | 01:00 | |
| *** slaweq has joined #openstack-nova | 01:11 | |
| *** hongbin has joined #openstack-nova | 01:14 | |
| *** slaweq has quit IRC | 01:15 | |
| openstackgerrit | Sam Morrison proposed openstack/nova master: Allow ability for non admin users to list all flavors. https://review.openstack.org/608474 | 01:17 |
| *** erlon_ has quit IRC | 01:20 | |
| *** mrsoul has joined #openstack-nova | 01:32 | |
| *** swamireddy has quit IRC | 01:37 | |
| *** openstackgerrit has quit IRC | 01:37 | |
| *** mgoddard has quit IRC | 01:37 | |
| *** Dinesh_Bhor has quit IRC | 01:37 | |
| *** mhen has quit IRC | 01:37 | |
| *** tristanC has quit IRC | 01:37 | |
| *** mikeoschen has quit IRC | 01:37 | |
| *** mugsie has quit IRC | 01:37 | |
| *** jackding has quit IRC | 01:37 | |
| *** jroll has quit IRC | 01:37 | |
| *** logan- has quit IRC | 01:37 | |
| *** ajo has quit IRC | 01:37 | |
| *** odyssey4me has quit IRC | 01:37 | |
| *** jcosmao has quit IRC | 01:37 | |
| *** vdrok has quit IRC | 01:37 | |
| *** mvkr has quit IRC | 01:37 | |
| *** markvoelker has quit IRC | 01:37 | |
| *** jaypipes has quit IRC | 01:37 | |
| *** sean-k-mooney has quit IRC | 01:37 | |
| *** icey has quit IRC | 01:37 | |
| *** jmlowe has quit IRC | 01:37 | |
| *** edleafe has quit IRC | 01:37 | |
| *** burt has quit IRC | 01:37 | |
| *** gibi has quit IRC | 01:37 | |
| *** wznoinsk has quit IRC | 01:37 | |
| *** jhesketh has quit IRC | 01:37 | |
| *** nehaalhat_ has quit IRC | 01:37 | |
| *** Sigyn has quit IRC | 01:37 | |
| *** yonglihe has quit IRC | 01:37 | |
| *** dtroyer has quit IRC | 01:37 | |
| *** McNinja has quit IRC | 01:37 | |
| *** dave-mccowan has quit IRC | 01:37 | |
| *** rpittau has quit IRC | 01:37 | |
| *** jiapei has quit IRC | 01:37 | |
| *** devananda has quit IRC | 01:37 | |
| *** Cardoe has quit IRC | 01:37 | |
| *** spsurya has quit IRC | 01:37 | |
| *** etp has quit IRC | 01:37 | |
| *** sdake has quit IRC | 01:37 | |
| *** xyang has quit IRC | 01:37 | |
| *** masayukig[m] has quit IRC | 01:37 | |
| *** pas-ha has quit IRC | 01:37 | |
| *** johnsom has quit IRC | 01:37 | |
| *** mgagne has quit IRC | 01:37 | |
| *** knikolla has quit IRC | 01:37 | |
| *** dklyle has quit IRC | 01:37 | |
| *** kukacz has quit IRC | 01:37 | |
| *** Kevin_Zheng has quit IRC | 01:37 | |
| *** BlackDex has quit IRC | 01:37 | |
| *** szaher has quit IRC | 01:37 | |
| *** gmann has quit IRC | 01:37 | |
| *** StevenK has quit IRC | 01:37 | |
| *** kashyap has quit IRC | 01:37 | |
| *** hamzy_ has quit IRC | 01:37 | |
| *** efried has quit IRC | 01:37 | |
| *** gouthamr has quit IRC | 01:37 | |
| *** belmoreira has quit IRC | 01:37 | |
| *** wxy-xiyuan has quit IRC | 01:37 | |
| *** MasterofJOKers has quit IRC | 01:37 | |
| *** dosaboy has quit IRC | 01:37 | |
| *** tinwood has quit IRC | 01:37 | |
| *** zzzeek_ has quit IRC | 01:37 | |
| *** manjeets has quit IRC | 01:37 | |
| *** hfjvjffju has quit IRC | 01:37 | |
| *** bandini has quit IRC | 01:37 | |
| *** dr_gogeta86 has quit IRC | 01:37 | |
| *** ebbex has quit IRC | 01:37 | |
| *** nicolasbock has quit IRC | 01:37 | |
| *** beagles has quit IRC | 01:37 | |
| *** sayalilunkad has quit IRC | 01:37 | |
| *** ShilpaSD has quit IRC | 01:37 | |
| *** pooja_jadhav has quit IRC | 01:37 | |
| *** yikun has quit IRC | 01:37 | |
| *** jistr has quit IRC | 01:37 | |
| *** amotoki has quit IRC | 01:37 | |
| *** jpena|off has quit IRC | 01:37 | |
| *** dtantsur|afk has quit IRC | 01:37 | |
| *** stephenfin has quit IRC | 01:37 | |
| *** sorrison has quit IRC | 01:37 | |
| *** whoami-rajat has quit IRC | 01:37 | |
| *** hogepodge has quit IRC | 01:37 | |
| *** jbryce has quit IRC | 01:37 | |
| *** kmalloc has quit IRC | 01:37 | |
| *** lamt has quit IRC | 01:37 | |
| *** coreycb has quit IRC | 01:37 | |
| *** jungleboyj has quit IRC | 01:37 | |
| *** Hazelesque has quit IRC | 01:37 | |
| *** geekinutah has quit IRC | 01:37 | |
| *** hongbin has quit IRC | 01:37 | |
| *** lbragstad has quit IRC | 01:37 | |
| *** panda has quit IRC | 01:37 | |
| *** imacdonn has quit IRC | 01:37 | |
| *** artom has quit IRC | 01:37 | |
| *** penick has quit IRC | 01:37 | |
| *** jiaopengju has quit IRC | 01:37 | |
| *** SpamapS has quit IRC | 01:37 | |
| *** mgariepy has quit IRC | 01:37 | |
| *** d34dh0r53 has quit IRC | 01:37 | |
| *** yankcrime has quit IRC | 01:37 | |
| *** NostawRm has quit IRC | 01:37 | |
| *** tonyb has quit IRC | 01:37 | |
| *** lennyb has quit IRC | 01:37 | |
| *** jbernard has quit IRC | 01:37 | |
| *** bnemec has quit IRC | 01:37 | |
| *** andreykurilin has quit IRC | 01:37 | |
| *** egonzalez has quit IRC | 01:37 | |
| *** fnordahl has quit IRC | 01:37 | |
| *** rm_work has quit IRC | 01:37 | |
| *** fungi has quit IRC | 01:37 | |
| *** ttx has quit IRC | 01:37 | |
| *** samueldmq has quit IRC | 01:37 | |
| *** melwitt has quit IRC | 01:37 | |
| *** s1061123 has quit IRC | 01:37 | |
| *** andymccr has quit IRC | 01:37 | |
| *** rabel has quit IRC | 01:37 | |
| *** jamesdenton has quit IRC | 01:37 | |
| *** nicholas has quit IRC | 01:37 | |
| *** chason has quit IRC | 01:37 | |
| *** bauzas has quit IRC | 01:37 | |
| *** mnaser has quit IRC | 01:37 | |
| *** TheJulia has quit IRC | 01:37 | |
| *** mordred has quit IRC | 01:37 | |
| *** hughsaunders has quit IRC | 01:37 | |
| *** itlinux has quit IRC | 01:37 | |
| *** jaosorior has quit IRC | 01:37 | |
| *** _pewp_ has quit IRC | 01:37 | |
| *** _hemna has quit IRC | 01:37 | |
| *** dims has quit IRC | 01:37 | |
| *** kencjohnston has quit IRC | 01:37 | |
| *** DinaBelova has quit IRC | 01:37 | |
| *** larsks has quit IRC | 01:37 | |
| *** gnuoy has quit IRC | 01:37 | |
| *** mrsoul has quit IRC | 01:37 | |
| *** sambetts_ has quit IRC | 01:37 | |
| *** vabada has quit IRC | 01:37 | |
| *** alex_xu has quit IRC | 01:37 | |
| *** jdillaman has quit IRC | 01:37 | |
| *** purplerbot has quit IRC | 01:37 | |
| *** naichuans has quit IRC | 01:37 | |
| *** jamiec_ has quit IRC | 01:37 | |
| *** mmedvede has quit IRC | 01:37 | |
| *** elod has quit IRC | 01:37 | |
| *** rnoriega has quit IRC | 01:37 | |
| *** obre has quit IRC | 01:37 | |
| *** johnthetubaguy has quit IRC | 01:37 | |
| *** spotz has quit IRC | 01:37 | |
| *** Jeffrey4l has quit IRC | 01:37 | |
| *** raginbajin has quit IRC | 01:37 | |
| *** dansmith has quit IRC | 01:37 | |
| *** antonym has quit IRC | 01:37 | |
| *** liuyulong has quit IRC | 01:37 | |
| *** jangutter has quit IRC | 01:37 | |
| *** owalsh_away has quit IRC | 01:37 | |
| *** zigo has quit IRC | 01:37 | |
| *** eandersson has quit IRC | 01:37 | |
| *** tobias-urdin has quit IRC | 01:37 | |
| *** smcginnis has quit IRC | 01:37 | |
| *** gryf has quit IRC | 01:37 | |
| *** niceplace has quit IRC | 01:37 | |
| *** breton has quit IRC | 01:37 | |
| *** ircuser-1 has quit IRC | 01:37 | |
| *** rcernin has quit IRC | 01:37 | |
| *** kaisers has quit IRC | 01:37 | |
| *** kevinbenton has quit IRC | 01:37 | |
| *** ianw has quit IRC | 01:37 | |
| *** jlvillal has quit IRC | 01:37 | |
| *** edmondsw has quit IRC | 01:37 | |
| *** aloga has quit IRC | 01:37 | |
| *** dulek has quit IRC | 01:37 | |
| *** aarents has quit IRC | 01:37 | |
| *** andreaf has quit IRC | 01:37 | |
| *** alanmeadows has quit IRC | 01:37 | |
| *** zioproto has quit IRC | 01:37 | |
| *** rha has quit IRC | 01:37 | |
| *** frickler has quit IRC | 01:37 | |
| *** hemna has quit IRC | 01:37 | |
| *** markmcclain has quit IRC | 01:37 | |
| *** ericyoung has quit IRC | 01:37 | |
| *** rmk has quit IRC | 01:37 | |
| *** ChanServ has quit IRC | 01:37 | |
| *** andreaf has joined #openstack-nova | 01:43 | |
| *** dulek has joined #openstack-nova | 01:43 | |
| *** aloga has joined #openstack-nova | 01:43 | |
| *** aarents has joined #openstack-nova | 01:43 | |
| *** edmondsw has joined #openstack-nova | 01:43 | |
| *** jlvillal has joined #openstack-nova | 01:43 | |
| *** ianw has joined #openstack-nova | 01:43 | |
| *** kevinbenton has joined #openstack-nova | 01:43 | |
| *** kaisers has joined #openstack-nova | 01:43 | |
| *** rcernin has joined #openstack-nova | 01:43 | |
| *** ircuser-1 has joined #openstack-nova | 01:43 | |
| *** odyssey4me has joined #openstack-nova | 01:43 | |
| *** jcosmao has joined #openstack-nova | 01:43 | |
| *** vdrok has joined #openstack-nova | 01:43 | |
| *** ajo has joined #openstack-nova | 01:43 | |
| *** logan- has joined #openstack-nova | 01:43 | |
| *** jroll has joined #openstack-nova | 01:43 | |
| *** jackding has joined #openstack-nova | 01:43 | |
| *** mikeoschen has joined #openstack-nova | 01:43 | |
| *** mhen has joined #openstack-nova | 01:43 | |
| *** mugsie has joined #openstack-nova | 01:43 | |
| *** Dinesh_Bhor has joined #openstack-nova | 01:43 | |
| *** niceplace has joined #openstack-nova | 01:43 | |
| *** breton has joined #openstack-nova | 01:43 | |
| *** gryf has joined #openstack-nova | 01:43 | |
| *** smcginnis has joined #openstack-nova | 01:43 | |
| *** tobias-urdin has joined #openstack-nova | 01:43 | |
| *** eandersson has joined #openstack-nova | 01:43 | |
| *** zigo has joined #openstack-nova | 01:43 | |
| *** owalsh_away has joined #openstack-nova | 01:43 | |
| *** jangutter has joined #openstack-nova | 01:43 | |
| *** liuyulong has joined #openstack-nova | 01:43 | |
| *** antonym has joined #openstack-nova | 01:43 | |
| *** dansmith has joined #openstack-nova | 01:43 | |
| *** raginbajin has joined #openstack-nova | 01:43 | |
| *** Jeffrey4l has joined #openstack-nova | 01:43 | |
| *** spotz has joined #openstack-nova | 01:43 | |
| *** johnthetubaguy has joined #openstack-nova | 01:43 | |
| *** obre has joined #openstack-nova | 01:43 | |
| *** rnoriega has joined #openstack-nova | 01:43 | |
| *** elod has joined #openstack-nova | 01:43 | |
| *** jamiec_ has joined #openstack-nova | 01:43 | |
| *** mmedvede has joined #openstack-nova | 01:43 | |
| *** naichuans has joined #openstack-nova | 01:43 | |
| *** purplerbot has joined #openstack-nova | 01:43 | |
| *** jdillaman has joined #openstack-nova | 01:43 | |
| *** alex_xu has joined #openstack-nova | 01:43 | |
| *** vabada has joined #openstack-nova | 01:43 | |
| *** sambetts_ has joined #openstack-nova | 01:43 | |
| *** mrsoul has joined #openstack-nova | 01:43 | |
| *** dklyle has joined #openstack-nova | 01:44 | |
| *** kukacz has joined #openstack-nova | 01:44 | |
| *** Kevin_Zheng has joined #openstack-nova | 01:44 | |
| *** BlackDex has joined #openstack-nova | 01:44 | |
| *** szaher has joined #openstack-nova | 01:44 | |
| *** gmann has joined #openstack-nova | 01:44 | |
| *** kashyap has joined #openstack-nova | 01:44 | |
| *** StevenK has joined #openstack-nova | 01:44 | |
| *** Sigyn has joined #openstack-nova | 01:44 | |
| *** nehaalhat_ has joined #openstack-nova | 01:44 | |
| *** bnemec has joined #openstack-nova | 01:44 | |
| *** andreykurilin has joined #openstack-nova | 01:44 | |
| *** egonzalez has joined #openstack-nova | 01:44 | |
| *** fnordahl has joined #openstack-nova | 01:44 | |
| *** rm_work has joined #openstack-nova | 01:44 | |
| *** fungi has joined #openstack-nova | 01:44 | |
| *** ttx has joined #openstack-nova | 01:44 | |
| *** samueldmq has joined #openstack-nova | 01:44 | |
| *** melwitt has joined #openstack-nova | 01:44 | |
| *** hamzy_ has joined #openstack-nova | 01:44 | |
| *** efried has joined #openstack-nova | 01:44 | |
| *** gouthamr has joined #openstack-nova | 01:44 | |
| *** belmoreira has joined #openstack-nova | 01:44 | |
| *** wxy-xiyuan has joined #openstack-nova | 01:44 | |
| *** MasterofJOKers has joined #openstack-nova | 01:44 | |
| *** dosaboy has joined #openstack-nova | 01:44 | |
| *** tinwood has joined #openstack-nova | 01:44 | |
| *** zzzeek_ has joined #openstack-nova | 01:44 | |
| *** manjeets has joined #openstack-nova | 01:44 | |
| *** bandini has joined #openstack-nova | 01:44 | |
| *** dr_gogeta86 has joined #openstack-nova | 01:44 | |
| *** ebbex has joined #openstack-nova | 01:44 | |
| *** nicolasbock has joined #openstack-nova | 01:44 | |
| *** beagles has joined #openstack-nova | 01:44 | |
| *** sayalilunkad has joined #openstack-nova | 01:44 | |
| *** openstackgerrit has joined #openstack-nova | 01:44 | |
| *** mgoddard has joined #openstack-nova | 01:44 | |
| *** zioproto has joined #openstack-nova | 01:45 | |
| *** alanmeadows has joined #openstack-nova | 01:45 | |
| *** rha has joined #openstack-nova | 01:45 | |
| *** frickler has joined #openstack-nova | 01:45 | |
| *** hemna has joined #openstack-nova | 01:45 | |
| *** markmcclain has joined #openstack-nova | 01:45 | |
| *** ericyoung has joined #openstack-nova | 01:45 | |
| *** rmk has joined #openstack-nova | 01:45 | |
| *** ShilpaSD has joined #openstack-nova | 01:45 | |
| *** pooja_jadhav has joined #openstack-nova | 01:45 | |
| *** yikun has joined #openstack-nova | 01:45 | |
| *** jistr has joined #openstack-nova | 01:45 | |
| *** amotoki has joined #openstack-nova | 01:45 | |
| *** jpena|off has joined #openstack-nova | 01:45 | |
| *** dtantsur|afk has joined #openstack-nova | 01:45 | |
| *** lamt has joined #openstack-nova | 01:45 | |
| *** sorrison has joined #openstack-nova | 01:45 | |
| *** stephenfin has joined #openstack-nova | 01:45 | |
| *** coreycb has joined #openstack-nova | 01:45 | |
| *** whoami-rajat has joined #openstack-nova | 01:45 | |
| *** jungleboyj has joined #openstack-nova | 01:45 | |
| *** hogepodge has joined #openstack-nova | 01:45 | |
| *** geekinutah has joined #openstack-nova | 01:45 | |
| *** jbryce has joined #openstack-nova | 01:45 | |
| *** kmalloc has joined #openstack-nova | 01:45 | |
| *** Hazelesque has joined #openstack-nova | 01:45 | |
| *** yonglihe has joined #openstack-nova | 01:45 | |
| *** dtroyer has joined #openstack-nova | 01:45 | |
| *** McNinja has joined #openstack-nova | 01:45 | |
| *** mvkr has joined #openstack-nova | 01:46 | |
| *** markvoelker has joined #openstack-nova | 01:46 | |
| *** edleafe has joined #openstack-nova | 01:46 | |
| *** sean-k-mooney has joined #openstack-nova | 01:46 | |
| *** icey has joined #openstack-nova | 01:46 | |
| *** jmlowe has joined #openstack-nova | 01:46 | |
| *** wznoinsk has joined #openstack-nova | 01:46 | |
| *** burt has joined #openstack-nova | 01:46 | |
| *** gibi has joined #openstack-nova | 01:46 | |
| *** jhesketh has joined #openstack-nova | 01:46 | |
| *** jamesdenton has joined #openstack-nova | 01:46 | |
| *** nicholas has joined #openstack-nova | 01:46 | |
| *** chason has joined #openstack-nova | 01:46 | |
| *** bauzas has joined #openstack-nova | 01:46 | |
| *** mnaser has joined #openstack-nova | 01:46 | |
| *** TheJulia has joined #openstack-nova | 01:46 | |
| *** mordred has joined #openstack-nova | 01:46 | |
| *** s1061123 has joined #openstack-nova | 01:46 | |
| *** rabel has joined #openstack-nova | 01:46 | |
| *** andymccr has joined #openstack-nova | 01:46 | |
| *** hongbin has joined #openstack-nova | 01:46 | |
| *** panda has joined #openstack-nova | 01:46 | |
| *** imacdonn has joined #openstack-nova | 01:46 | |
| *** artom has joined #openstack-nova | 01:46 | |
| *** jiaopengju has joined #openstack-nova | 01:46 | |
| *** penick has joined #openstack-nova | 01:46 | |
| *** SpamapS has joined #openstack-nova | 01:46 | |
| *** mgariepy has joined #openstack-nova | 01:46 | |
| *** d34dh0r53 has joined #openstack-nova | 01:46 | |
| *** yankcrime has joined #openstack-nova | 01:46 | |
| *** NostawRm has joined #openstack-nova | 01:46 | |
| *** tonyb has joined #openstack-nova | 01:46 | |
| *** lennyb has joined #openstack-nova | 01:46 | |
| *** jbernard has joined #openstack-nova | 01:46 | |
| *** jaosorior has joined #openstack-nova | 01:46 | |
| *** _pewp_ has joined #openstack-nova | 01:46 | |
| *** kencjohnston has joined #openstack-nova | 01:46 | |
| *** DinaBelova has joined #openstack-nova | 01:46 | |
| *** larsks has joined #openstack-nova | 01:46 | |
| *** gnuoy has joined #openstack-nova | 01:46 | |
| *** ChanServ has joined #openstack-nova | 01:47 | |
| *** card.freenode.net sets mode: +o ChanServ | 01:47 | |
| *** lbragstad has joined #openstack-nova | 01:48 | |
| *** tristanC has joined #openstack-nova | 01:48 | |
| *** mgagne has joined #openstack-nova | 01:49 | |
| *** Cardoe has joined #openstack-nova | 01:49 | |
| *** Guest10461 has joined #openstack-nova | 01:49 | |
| *** dave-mccowan has joined #openstack-nova | 01:50 | |
| *** hughsaunders has joined #openstack-nova | 01:52 | |
| *** swamireddy has joined #openstack-nova | 01:52 | |
| *** itlinux has joined #openstack-nova | 01:55 | |
| *** mhen has quit IRC | 01:56 | |
| *** mhen has joined #openstack-nova | 01:59 | |
| *** hshiina has joined #openstack-nova | 02:02 | |
| *** tetsuro has joined #openstack-nova | 02:03 | |
| *** sapd1 has joined #openstack-nova | 02:11 | |
| *** takashin has joined #openstack-nova | 02:11 | |
| *** lbragstad has quit IRC | 02:14 | |
| *** trungnv has joined #openstack-nova | 02:16 | |
| *** hoangcx has joined #openstack-nova | 02:16 | |
| openstackgerrit | Jack Ding proposed openstack/nova master: Add I/O Semaphore to limit concurrent disk ops https://review.openstack.org/609180 | 02:30 |
| openstackgerrit | huanhongda proposed openstack/nova master: api-ref: add two tables in the note of DELETE /os-services https://review.openstack.org/609186 | 02:33 |
| *** psachin has joined #openstack-nova | 02:57 | |
| openstackgerrit | Sam Morrison proposed openstack/nova master: Allow ability for non admin users to list all flavors. https://review.openstack.org/608474 | 03:03 |
| *** dave-mccowan has quit IRC | 03:08 | |
| *** slaweq has joined #openstack-nova | 03:11 | |
| *** slaweq has quit IRC | 03:16 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add compute API version for when a ``volume_type`` is requested https://review.openstack.org/605573 | 03:26 |
| *** hongbin has quit IRC | 03:30 | |
| *** whoami-rajat has quit IRC | 03:48 | |
| *** udesale has joined #openstack-nova | 03:52 | |
| *** Dinesh_Bhor has quit IRC | 03:58 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (7) https://review.openstack.org/571992 | 04:04 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (8) https://review.openstack.org/571993 | 04:05 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.openstack.org/566153 | 04:05 |
| openstackgerrit | Sundar Nadathur proposed openstack/nova-specs master: Nova Cyborg interaction specification. https://review.openstack.org/603955 | 04:27 |
| *** janki has joined #openstack-nova | 04:29 | |
| *** ircuser-1 has quit IRC | 04:38 | |
| *** cfriesen has joined #openstack-nova | 04:38 | |
| *** spsurya has joined #openstack-nova | 04:44 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:54 | |
| *** ratailor has joined #openstack-nova | 05:11 | |
| *** slaweq has joined #openstack-nova | 05:11 | |
| *** whoami-rajat has joined #openstack-nova | 05:23 | |
| *** ircuser-1 has joined #openstack-nova | 05:28 | |
| *** janki has quit IRC | 05:53 | |
| *** janki has joined #openstack-nova | 05:54 | |
| gmann | alex_xu hi, will you be there for API office hour ? | 05:55 |
| alex_xu | gmann: yea | 05:56 |
| gmann | cool, | 05:56 |
| gmann | let's start | 06:01 |
| gmann | #startmeeting nova api | 06:02 |
| openstack | Meeting started Wed Oct 10 06:02:03 2018 UTC and is due to finish in 60 minutes. The chair is gmann. Information about MeetBot at http://wiki.debian.org/MeetBot. | 06:02 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 06:02 |
| *** openstack changes topic to " (Meeting topic: nova api)" | 06:02 | |
| openstack | The meeting name has been set to 'nova_api' | 06:02 |
| gmann | #link https://wiki.openstack.org/wiki/Meetings/NovaAPI#Agenda_for_next_Office_hours | 06:02 |
| gmann | agenda ^^ | 06:02 |
| gmann | i have not got the much time to review for couple of weeks. | 06:03 |
| gmann | #topic API Subteam Tracking | 06:03 |
| *** openstack changes topic to "API Subteam Tracking (Meeting topic: nova api)" | 06:03 | |
| gmann | #linkhttps://etherpad.openstack.org/p/stein-nova-subteam-tracking | 06:03 |
| gmann | i added the approved/under review BP related to API on this extherpad | 06:03 |
| gmann | l63 | 06:04 |
| alex_xu | currently we have an api bp in the runway | 06:04 |
| gmann | yeah | 06:04 |
| gmann | #link https://etherpad.openstack.org/p/nova-runways-stein | 06:05 |
| alex_xu | yea, the volume type in boot | 06:06 |
| alex_xu | how much we left for extension merge? | 06:07 |
| gmann | alex_xu: link https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/api-extensions-merge-stein+status:open | 06:07 |
| gmann | 3-4 patch i will say. | 06:07 |
| gmann | i need to rebase them though | 06:07 |
| alex_xu | cool | 06:07 |
| gmann | this is last patch where i am removing the extensions register logic from wsgi.py - https://review.openstack.org/#/c/607092/ | 06:08 |
| gmann | this is WIP, still mode code to remove | 06:08 |
| alex_xu | cool, simpler code | 06:09 |
| gmann | yeah, i will work on those and finish soon | 06:10 |
| alex_xu | thanks | 06:10 |
| gmann | alex_xu: i think we can remove this too - wsgi_action_extensions | 06:12 |
| *** Dinesh_Bhor has quit IRC | 06:12 | |
| gmann | do you remember any of action extensions ? | 06:12 |
| gmann | this one - https://github.com/openstack/nova/blob/master/nova/api/openstack/wsgi.py#L468 | 06:13 |
| alex_xu | https://review.openstack.org/#/c/607092/1/nova/api/openstack/compute/routes.py@252 | 06:13 |
| alex_xu | what benefit we remove it? | 06:13 |
| gmann | humm, i was thinking for same resource side but these are all separate entry point for resource action | 06:14 |
| gmann | agree to keep them separate. | 06:15 |
| alex_xu | still feel a lot of complex thing in https://review.openstack.org/#/c/607092/1/nova/api/openstack/wsgi.py | 06:16 |
| gmann | true, it could be more simpler | 06:16 |
| alex_xu | without extension, does this still useful https://review.openstack.org/#/c/607092/1/nova/api/openstack/wsgi.py@88 | 06:17 |
| alex_xu | gmann: ^ maybe worth to check this in your last patch | 06:17 |
| alex_xu | I remember that is used for cache the response obj for the extension, not sure whether it is used by other place | 06:17 |
| alex_xu | gmann: sorry, I mean those cache interface https://review.openstack.org/#/c/607092/1/nova/api/openstack/wsgi.py@97 | 06:18 |
| gmann | yeah those were mainly used in extensions code but i can check if anywhere we use them | 06:19 |
| gmann | but Request object we need | 06:19 |
| alex_xu | yes, we need Req obj | 06:20 |
| *** adrianc has joined #openstack-nova | 06:20 | |
| gmann | alex_xu: quickly grep them and it is only extensions code. i will remove them thanks. | 06:21 |
| alex_xu | cool \o/ | 06:22 |
| gmann | alex_xu: i will make this patch up by tonight and make it ready for you by tomorrow so that u can check if any more bits we can remove | 06:22 |
| alex_xu | gmann: yea, will do | 06:22 |
| gmann | thanks | 06:23 |
| gmann | moving next | 06:23 |
| gmann | api cleanup things | 06:24 |
| gmann | #link https://etherpad.openstack.org/p/nova-api-cleanup | 06:24 |
| gmann | and spec which need to include the hypervisor API cleanup - https://review.openstack.org/#/c/603969/ | 06:24 |
| gmann | we need to weight on those with what worth to do and what not | 06:25 |
| gmann | and next question on this is - should we do this in stein ? or wait for T to collect more worthy cleanup ? | 06:25 |
| gmann | i can keep it updated for stein and wait till T if any more related API cleanup. And then we do those in single version bump based on agreement | 06:27 |
| gmann | alex_xu: what u say ? | 06:27 |
| alex_xu | pretty sure we won't get rid of all the cleanup in one microversion | 06:27 |
| alex_xu | see the past, how much microversion we spend on deprecate proxy API :) | 06:27 |
| *** skatsaounis has joined #openstack-nova | 06:27 | |
| gmann | :) yeah | 06:27 |
| alex_xu | I will review the spec | 06:28 |
| gmann | all you mean "all listed in etherpad" or all in API (because we do not know them now) | 06:28 |
| *** Dinesh_Bhor has joined #openstack-nova | 06:28 | |
| alex_xu | all in API | 06:28 |
| gmann | true, so that's is reason i want to wait till T so that we can cover max. | 06:28 |
| alex_xu | maybe based on the requirement, if there is nothing urgent to change, then we needn't do it very soon | 06:29 |
| gmann | and conclude that "this is we are not going to do" "this is ok to do". so that these does not comes again in future | 06:29 |
| gmann | yeah | 06:29 |
| gmann | i do not think anything urgent on those but i will also get opinion from mriedem and melwitt | 06:30 |
| gmann | also | 06:31 |
| alex_xu | I'm afriad another option is microversion shouldn't be huge | 06:31 |
| gmann | humm | 06:31 |
| gmann | i feel it should not be huge in term of complexity but should be ok in term of numbers. | 06:32 |
| gmann | anyways that we can discuss on spec with exact list of changes | 06:33 |
| alex_xu | yea | 06:34 |
| gmann | anything else on this or we move next ? | 06:34 |
| alex_xu | no more from me | 06:35 |
| gmann | ok | 06:35 |
| gmann | i do not have anything else to discuss form API subteam tracking section. | 06:35 |
| gmann | alex_xu: do you have ? | 06:36 |
| alex_xu | no | 06:36 |
| gmann | ok | 06:36 |
| gmann | #topic Bug Triage/Discussion | 06:36 |
| *** openstack changes topic to "Bug Triage/Discussion (Meeting topic: nova api)" | 06:36 | |
| gmann | #link https://etherpad.openstack.org/p/nova-api-weekly-bug-report | 06:36 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Spec: Support filtering by forbidden aggregate https://review.openstack.org/603352 | 06:37 |
| *** janki has quit IRC | 06:38 | |
| gmann | there are 4 new bugs | 06:38 |
| *** janki has joined #openstack-nova | 06:38 | |
| gmann | #link https://bugs.launchpad.net/nova/+bug/1789382 | 06:38 |
| openstack | Launchpad bug 1789382 in OpenStack Compute (nova) "openstack server list error" [Undecided,New] | 06:38 |
| gmann | seems timeout from cell ? | 06:39 |
| alex_xu | also see 'TypeError: 'object' object is not iterable' after cell timeout | 06:40 |
| alex_xu | cell timeout is just a warning | 06:41 |
| *** moshele has joined #openstack-nova | 06:41 | |
| *** fanzhang has joined #openstack-nova | 06:42 | |
| alex_xu | sounds like we handle something wrong | 06:42 |
| *** Swami has joined #openstack-nova | 06:42 | |
| gmann | humm object is none or something may be | 06:43 |
| gmann | let me ask more log of api and compute | 06:44 |
| gmann | i did not find any other pointer to debug without those | 06:45 |
| alex_xu | yes | 06:45 |
| *** helenafm has joined #openstack-nova | 06:46 | |
| *** tssurya has joined #openstack-nova | 06:47 | |
| gmann | done | 06:49 |
| gmann | it seems hitting here - https://github.com/openstack/nova/blob/c6218428e9b29a2c52808ec7d27b4b21aadc0299/nova/compute/multi_cell_list.py#L265 | 06:49 |
| *** pcaruana has joined #openstack-nova | 06:50 | |
| gmann | #link https://bugs.launchpad.net/nova/+bug/1796132 | 06:50 |
| openstack | Launchpad bug 1796132 in OpenStack Compute (nova) "SSL Verification Error on Launch Instance (queens)" [Undecided,New] | 06:50 |
| gmann | "Seems forcing everythign to https isn't the wisest choice and requires more nova specific ssl config, I'll be going back to using http, so this bug may no longer be valid to work on." | 06:50 |
| gmann | comments from author ^^ | 06:50 |
| gmann | should we mark it invalid then ? | 06:52 |
| gmann | seems like glance was not on https | 06:52 |
| gmann | SSLError: SSL exception connecting to https://10.5.25.19:9292/v2/images/fd9637fb-57e8-4c43-9 | 06:53 |
| alex_xu | I don't know, is it configure mistake? | 06:53 |
| gmann | seems so | 06:54 |
| alex_xu | i don't familar this part | 06:54 |
| gmann | i will ask on bug is that is solved with all having on https or not | 06:54 |
| *** Luzi has joined #openstack-nova | 06:55 | |
| *** mdbooth has joined #openstack-nova | 06:56 | |
| gmann | done | 06:56 |
| gmann | it is almost time and other bugs we can do next week or in between of next office hour | 06:57 |
| alex_xu | yes | 06:57 |
| gmann | #topic Open Discussion | 06:57 |
| *** openstack changes topic to "Open Discussion (Meeting topic: nova api)" | 06:57 | |
| gmann | anything else alex_xu you want to discuss otherwise we close for today ? | 06:57 |
| *** rcernin has quit IRC | 06:58 | |
| alex_xu | nothing from me | 06:58 |
| *** zhanglong has joined #openstack-nova | 06:58 | |
| gmann | alex_xu: thanks for joining. this really help. | 06:58 |
| alex_xu | np | 06:58 |
| gmann | #endmeeting | 06:58 |
| *** openstack changes topic to "Current runways: use-nested-allocation-candidates -- This channel is for Nova development. For support of Nova deployments, please use #openstack." | 06:58 | |
| openstack | Meeting ended Wed Oct 10 06:58:37 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 06:58 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-10-10-06.02.html | 06:58 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-10-10-06.02.txt | 06:58 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/nova_api/2018/nova_api.2018-10-10-06.02.log.html | 06:58 |
| *** tssurya has quit IRC | 07:01 | |
| *** cfriesen has quit IRC | 07:06 | |
| *** mikeoschen has quit IRC | 07:07 | |
| *** vdrok has quit IRC | 07:07 | |
| *** odyssey4me has quit IRC | 07:07 | |
| *** vdrok has joined #openstack-nova | 07:08 | |
| *** ajo has quit IRC | 07:08 | |
| *** odyssey4me has joined #openstack-nova | 07:08 | |
| *** ajo has joined #openstack-nova | 07:08 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:PropertyCollector for caching instance properties https://review.openstack.org/608278 | 07:08 |
| *** mugsie has quit IRC | 07:08 | |
| *** mgagne has quit IRC | 07:09 | |
| *** logan- has quit IRC | 07:09 | |
| *** mgagne has joined #openstack-nova | 07:10 | |
| *** jroll has quit IRC | 07:10 | |
| *** janki has quit IRC | 07:11 | |
| *** jiapei has joined #openstack-nova | 07:11 | |
| *** logan- has joined #openstack-nova | 07:11 | |
| *** janki has joined #openstack-nova | 07:11 | |
| *** jroll has joined #openstack-nova | 07:12 | |
| *** ttsiouts has joined #openstack-nova | 07:12 | |
| openstackgerrit | Lucian Petrut proposed openstack/nova master: Fix os-simple-tenant-usage result order https://review.openstack.org/608685 | 07:14 |
| *** Swami has quit IRC | 07:22 | |
| *** panda has quit IRC | 07:30 | |
| *** helenafm has quit IRC | 07:32 | |
| *** panda has joined #openstack-nova | 07:32 | |
| *** mgagne has quit IRC | 07:34 | |
| *** ttsiouts has quit IRC | 07:35 | |
| *** mgagne has joined #openstack-nova | 07:36 | |
| *** ralonsoh has joined #openstack-nova | 07:36 | |
| *** zhanglong has quit IRC | 07:39 | |
| *** zhanglong has joined #openstack-nova | 07:40 | |
| *** hoonetorg has joined #openstack-nova | 07:42 | |
| *** alexchadin has joined #openstack-nova | 07:50 | |
| *** jchhatbar has joined #openstack-nova | 07:51 | |
| *** janki has quit IRC | 07:52 | |
| *** jchhatbar has quit IRC | 07:53 | |
| *** janki has joined #openstack-nova | 07:53 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:55 | |
| *** ttsiouts has joined #openstack-nova | 07:59 | |
| *** k_mouza has joined #openstack-nova | 07:59 | |
| *** Dinesh_Bhor has quit IRC | 08:01 | |
| *** janki has quit IRC | 08:01 | |
| *** bhagyashris__ has joined #openstack-nova | 08:01 | |
| *** janki has joined #openstack-nova | 08:01 | |
| *** ttsiouts has quit IRC | 08:05 | |
| *** ttsiouts has joined #openstack-nova | 08:08 | |
| *** janki has quit IRC | 08:10 | |
| *** zhanglong has quit IRC | 08:13 | |
| *** ociuhandu has joined #openstack-nova | 08:13 | |
| *** zhanglong has joined #openstack-nova | 08:14 | |
| *** ttsiouts has quit IRC | 08:14 | |
| *** rpittau has joined #openstack-nova | 08:18 | |
| *** ttsiouts has joined #openstack-nova | 08:21 | |
| *** logan- has quit IRC | 08:23 | |
| *** alexchadin has quit IRC | 08:25 | |
| *** logan- has joined #openstack-nova | 08:27 | |
| *** jangutter has quit IRC | 08:32 | |
| *** jangutter has joined #openstack-nova | 08:33 | |
| *** derekh has joined #openstack-nova | 08:33 | |
| *** jangutter has quit IRC | 08:37 | |
| *** ygk12345 has joined #openstack-nova | 08:37 | |
| *** jangutter has joined #openstack-nova | 08:37 | |
| ygk12345 | hi all | 08:38 |
| ygk12345 | I am unable to unshelve an instance whihc is in shelve offload state | 08:38 |
| bauzas | ygk12345: stack trace ? | 08:39 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add microversion 2.67 to support volume_type https://review.openstack.org/606398 | 08:39 |
| ygk12345 | the instance is still in shelved state and in shutdown state | 08:40 |
| ygk12345 | bauzas: it is mitaka | 08:40 |
| ygk12345 | bauzas: I dont see any hypervisor allocated to it | 08:41 |
| ygk12345 | bauzas: DEBUG (session:277) RESP: [404] Date: Wed, 10 Oct 2018 08:41:47 GMT Content-Length: 52 Content-Type: text/plain; charset=UTF-8 X-Compute-Request-Id: req-f192f22e-9cd9-4b41-9863-c9752acbf95f RESP BODY: 404 Not Found The resource could not be found. | 08:42 |
| ygk12345 | bauzas: is it deleted ? | 08:42 |
| bauzas | ygk12345: do you have an ERROR log when unshelving? | 08:46 |
| *** tetsuro has quit IRC | 08:46 | |
| ygk12345 | no | 08:46 |
| ygk12345 | it is saying "resource could not be found" in the debug output | 08:47 |
| *** hshiina has quit IRC | 08:49 | |
| bauzas | ygk12345: what tells you the os-instance-actions API ? | 08:49 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting migrations https://review.openstack.org/607798 | 08:51 |
| *** paiboinaritesh has joined #openstack-nova | 08:53 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting instance action records https://review.openstack.org/607801 | 08:55 |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Add restrictions on ``updated_at`` when getting migrations https://review.openstack.org/607798 | 08:56 |
| *** belmorei_ has joined #openstack-nova | 08:57 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Remove force flag from live-migrate and evacuate https://review.openstack.org/609330 | 08:58 |
| *** belmoreira has quit IRC | 09:00 | |
| *** k_mouza has quit IRC | 09:01 | |
| *** brinzhang has joined #openstack-nova | 09:03 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge used_limits extension response into limit view builder https://review.openstack.org/606031 | 09:07 |
| *** alexchadin has joined #openstack-nova | 09:07 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge image_size extension response into image view builder https://review.openstack.org/606845 | 09:07 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088 | 09:08 |
| *** sambetts_ is now known as sambetts|afk | 09:10 | |
| *** Dinesh_Bhor has joined #openstack-nova | 09:12 | |
| *** k_mouza has joined #openstack-nova | 09:14 | |
| *** k_mouza has quit IRC | 09:17 | |
| *** k_mouza has joined #openstack-nova | 09:17 | |
| *** k_mouza has quit IRC | 09:20 | |
| *** bhagyashris__ has quit IRC | 09:21 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: [WIP]Remove extensions loading framework from wsgi.py https://review.openstack.org/607092 | 09:21 |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Support initial allocation ratios https://review.openstack.org/552105 | 09:35 |
| *** jarodwl has joined #openstack-nova | 09:39 | |
| *** ygk12345 has quit IRC | 09:41 | |
| *** mikeoschen has joined #openstack-nova | 09:43 | |
| *** adrianc has quit IRC | 09:47 | |
| *** imacdonn has quit IRC | 09:52 | |
| *** adrianc has joined #openstack-nova | 09:52 | |
| *** imacdonn has joined #openstack-nova | 09:52 | |
| *** lpetrut has joined #openstack-nova | 09:53 | |
| *** mvkr has quit IRC | 10:01 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Don't configure '[scheduler] discover_hosts_in_cells_interval' https://review.openstack.org/609345 | 10:10 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate the 'discover_hosts_in_cells_interval' option https://review.openstack.org/609346 | 10:10 |
| gmann | alex_xu: do you remember any case where we extended the response of action class with @wsgi.extends(action ? | 10:16 |
| gmann | alex_xu: this case - https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/api/openstack/wsgi.py#L772 | 10:17 |
| gmann | alex_xu: because that is what populate self.wsgi_action_extensions - https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/api/openstack/wsgi.py#L489 | 10:17 |
| gmann | alex_xu: this list goes as action (self.wsgi_action) - https://github.com/openstack/nova/blob/master/nova/api/openstack/compute/routes.py#L266 | 10:18 |
| gmann | alex_xu: so we are good to remove the self.wsgi_action_extensions too along with self.wsgi_xtensions. | 10:19 |
| *** ttsiouts has quit IRC | 10:20 | |
| *** zhanglong has quit IRC | 10:23 | |
| *** Dinesh_Bhor has quit IRC | 10:25 | |
| *** ttsiouts has joined #openstack-nova | 10:32 | |
| *** dave-mccowan has joined #openstack-nova | 10:32 | |
| *** mvkr has joined #openstack-nova | 10:38 | |
| *** Dinesh_Bhor has joined #openstack-nova | 10:42 | |
| *** priteau has joined #openstack-nova | 10:45 | |
| *** belmorei_ has quit IRC | 10:46 | |
| *** belmoreira has joined #openstack-nova | 10:47 | |
| *** paiboinaritesh has quit IRC | 10:50 | |
| alex_xu | gmann: I thought wsgi_action_extensions is for the whole action? | 10:51 |
| alex_xu | not just the response of the action | 10:51 |
| *** ttsiouts has quit IRC | 10:53 | |
| *** ttsiouts has joined #openstack-nova | 10:53 | |
| alex_xu | gmann: I think you are right | 10:55 |
| alex_xu | gmann: we don't need 'register_extensions' method at all | 10:55 |
| gmann | alex_xu: yea. | 10:57 |
| *** ttsiouts has quit IRC | 10:58 | |
| *** jpena|off has quit IRC | 10:58 | |
| *** takashin has left #openstack-nova | 11:01 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Use oslo_db.sqlalchemy.test_fixtures https://review.openstack.org/609352 | 11:02 |
| *** alex_xu has quit IRC | 11:09 | |
| *** udesale has quit IRC | 11:11 | |
| *** alex_xu has joined #openstack-nova | 11:13 | |
| *** Dinesh_Bhor has quit IRC | 11:14 | |
| *** dtantsur|afk is now known as dtantsur | 11:16 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: [DNM] test linux pytoute impl https://review.openstack.org/609354 | 11:17 |
| *** ttsiouts has joined #openstack-nova | 11:17 | |
| *** aspiers has joined #openstack-nova | 11:18 | |
| *** k_mouza has joined #openstack-nova | 11:20 | |
| *** k_mouza has quit IRC | 11:25 | |
| *** ratailor has quit IRC | 11:33 | |
| *** spatel has joined #openstack-nova | 11:55 | |
| aspiers | melwitt: following on from my colleague JP's discussion with nova folks at the PTG regarding adding AMD SEV support to nova, I've done a bunch of research regarding possible implementations and am writing a spec and bp which I hope to submit very shortly. Thought I'd give you a heads-up just in case there's anything you'd like to discuss. | 11:59 |
| *** spatel has quit IRC | 11:59 | |
| *** alexchadin has quit IRC | 12:00 | |
| *** alexchadin has joined #openstack-nova | 12:01 | |
| *** brinzhang has quit IRC | 12:02 | |
| *** belmoreira has quit IRC | 12:05 | |
| *** belmoreira has joined #openstack-nova | 12:08 | |
| *** k_mouza has joined #openstack-nova | 12:09 | |
| *** k_mouza has quit IRC | 12:13 | |
| *** adrianc has quit IRC | 12:13 | |
| *** tbachman has joined #openstack-nova | 12:38 | |
| *** cfriesen has joined #openstack-nova | 12:39 | |
| *** udesale has joined #openstack-nova | 12:45 | |
| *** jaypipes has joined #openstack-nova | 12:47 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Time how long select_destinations() takes in conductor https://review.openstack.org/608575 | 12:53 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/rocky: Replace usage of get_legacy_facade() with get_engine() https://review.openstack.org/608574 | 12:54 |
| mikeoschen | join #openstack-sdks | 12:54 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the extensions framework from wsgi.py https://review.openstack.org/607092 | 12:57 |
| *** mriedem has joined #openstack-nova | 12:58 | |
| pooja_jadhav | hi team, how I can create a BFV instance in the api-sample-tests for testing. please guide me for the same. | 13:04 |
| *** mvkr has quit IRC | 13:05 | |
| pooja_jadhav | I am referring https://github.com/openstack/nova/blob/85b36cd2f82ccd740057c1bee08fc722209604ab/nova/tests/functional/api_sample_tests/api_samples/servers/v2.42/server-create-req.json.tpl | 13:05 |
| pooja_jadhav | for creating an instance. but compute_utils. is_volume_backed giving False for the instance. | 13:07 |
| *** mdbooth has quit IRC | 13:08 | |
| pooja_jadhav | gmann: ^^ | 13:11 |
| gmann | pooja_jadhav: oh, that did not work ? | 13:13 |
| pooja_jadhav | gmann: unfortunately, no :( | 13:14 |
| *** mdbooth has joined #openstack-nova | 13:14 | |
| mriedem | you'd have to use the cinder fixture for one thing | 13:14 |
| pooja_jadhav | gmann: can u plz tell me.. which existing json.tpl should I use to create a BFV instance?? | 13:15 |
| mriedem | https://github.com/openstack/nova/blob/85b36cd2f82ccd740057c1bee08fc722209604ab/nova/tests/functional/api_sample_tests/api_samples/servers/v2.42/server-create-req.json.tpl isn't going to work since the cinder fixture doesn't create volumes | 13:15 |
| *** k_mouza has joined #openstack-nova | 13:15 | |
| *** jdillaman has quit IRC | 13:15 | |
| gmann | i suggest to write funtional test for that instead of sample tests | 13:15 |
| mriedem | see how https://github.com/openstack/nova/blob/85b36cd2f82ccd740057c1bee08fc722209604ab/nova/tests/fixtures.py#L1583 is used in functional tests | 13:16 |
| mriedem | e.g. https://review.openstack.org/#/c/608771/ | 13:16 |
| *** eharney has joined #openstack-nova | 13:20 | |
| Luzi | hey Nova, we've written the Spec for Image Encryption for Nova and would appreciate reviews :) https://review.openstack.org/#/c/608696/ | 13:21 |
| pooja_jadhav | mriedem, gmann: thanks will look into this | 13:21 |
| gmann | mriedem: yeah, pooja_jadhav tried to add the create volumes stub there | 13:22 |
| jangutter | sean-k-mooney, jaypipes: I'm looking at https://github.com/openstack/os-vif/blob/master/os_vif/objects/host_info.py and I've got some questions. (will post them to the room in general) | 13:22 |
| sean-k-mooney | jangutter: sure go for it | 13:22 |
| jaypipes | I blame sean-k-mooney. | 13:22 |
| stephenfin | sean-k-mooney: I wonder if we should be setting 'model' here for SR-IOV VIFs? https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/virt/libvirt/vif.py#L130-L133 | 13:22 |
| gmann | pooja_jadhav: mriedem i feel normal functional tests make more sense to verify the compute_utils. is_volume_backed instead of api sample tests | 13:23 |
| gmann | like example mriedem mentioned | 13:23 |
| jangutter | for the host_info objects, there's a mechanism to filter vif types by name and version, but not a corresponding mechanism for port_profiles. | 13:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the caching the resource on Request object https://review.openstack.org/609403 | 13:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove more code related to extensions and testing https://review.openstack.org/607088 | 13:23 |
| sean-k-mooney | stephenfin sorry taught that message was from jangutter am in what context | 13:23 |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Remove the extensions framework from wsgi.py https://review.openstack.org/607092 | 13:24 |
| stephenfin | sean-k-mooney: If we do and that hw_vif_model==virtio, we'll undo the hard work of https://github.com/openstack/nova/commit/622ebf2fab0a9bf75ee12437bef28f60e083f849 right? | 13:24 |
| openstackgerrit | Yikun Jiang proposed openstack/nova-specs master: Support initial allocation ratios https://review.openstack.org/552105 | 13:24 |
| sean-k-mooney | jangutter: yes currently nothing uses the filter. it was there for future use but its still in the future | 13:24 |
| sean-k-mooney | jangutter: so we can extend it | 13:24 |
| stephenfin | sean-k-mooney: Heh, yeah, two people talking about related stuff is confusing :) I'll test this myself in a bit. Just sanity checking | 13:24 |
| gmann | alex_xu: mriedem melwitt this complete the API extensions merge work - https://review.openstack.org/#/q/topic:bp/api-extensions-merge-stein+status:open | 13:25 |
| mriedem | pooja_jadhav: gmann: i'm not sure why we need a new functional test for bfv? | 13:25 |
| sean-k-mooney | well if you set hw_vif_model=Anything we shoudl respect that | 13:25 |
| mriedem | we have existing functional tests that cover that flow | 13:25 |
| mriedem | including the patch i just linked to you | 13:25 |
| gmann | i will check the gate if any tests i need to fix otherwise it is good to go | 13:26 |
| sean-k-mooney | stephenfin: but i need to check both patches to see how they interact | 13:26 |
| mriedem | gmann: ack on the api extension merge series - if you haven't, you should queue that up in a runway | 13:26 |
| jangutter | sean-k-mooney, jaypipes: (host_info context) right, so there's two ways of doing a check like this. the 'naive bayes' way, or the really explode the dependencies way. | 13:26 |
| *** jdillaman has joined #openstack-nova | 13:26 | |
| gmann | mriedem: right, those should work untill pooja_jadhav trying anything special or new feature (that is not up for review so not sure) | 13:26 |
| pooja_jadhav | mriedem: I am writting in api-sample-test for (if instance is BFV, then local_gb_used should be 0 in in the simple tenant usage api) | 13:26 |
| sean-k-mooney | stephenfin: if your asking should we remove https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/virt/libvirt/vif.py#L130-L133 the answer is no but we may need to adapt the queue size patch | 13:27 |
| pooja_jadhav | mriedem, gmann : https://bugs.launchpad.net/nova/+bug/1715570 | 13:27 |
| openstack | Launchpad bug 1715570 in OpenStack Compute (nova) "simple tenant usage api calculating disk usages incorrectly" [Medium,In progress] - Assigned to Bhagyashri Shewale (bhagyashri-shewale) | 13:27 |
| jangutter | sean-k-mooney, jaypipes: (host_info context) the difference comes in, are you going to ever worry about a case where VIFobject 1.0 supports PortProfile 1.0 and not 2.0? | 13:27 |
| jaypipes | jangutter: the host_info should be able to communicate to Neutron's agents exactly the kinds of VIFs that it can plug. If that means we need to pass some additional information about port profiles supported by the host, so be it. | 13:27 |
| sean-k-mooney | jaypipes: we should but we dont currently | 13:28 |
| gmann | mriedem: added | 13:28 |
| jaypipes | jangutter: if that makes a difference to what Neutron negotiates with the host, then yes... | 13:28 |
| *** k_mouza has quit IRC | 13:28 | |
| jangutter | jaypipes: (host_info context) it's similar to the question: does the base object need to bump its version if any of the members bump up a version? | 13:29 |
| mriedem | pooja_jadhav: we don't need to use an api sample test for that bug | 13:29 |
| sean-k-mooney | jangutter: unfortunetly no. i think it should be we do not bump for compostion | 13:29 |
| jaypipes | jangutter: no | 13:29 |
| mriedem | pooja_jadhav: a simple functional test which creates a volume-backed server and then queries the simple tenant usage API to assert it's showing disk usage when it shouldn't should suffice | 13:30 |
| jaypipes | jangutter: but see my comment on the datapath offload port profile patch that you shouldn't be modifying the VIFPortProfileBase object like that. | 13:30 |
| pooja_jadhav | mriedem: but existing tests for simple tenant usage api are in api sample tests. | 13:30 |
| mriedem | pooja_jadhav: that doesn't really matter | 13:30 |
| sean-k-mooney | jangutter: if you add a filed to base then you bump the version of all the derived types too | 13:30 |
| pooja_jadhav | mriedem: okies | 13:30 |
| jangutter | jaypipes: yep, that's revving because of inheritance, another story. | 13:30 |
| sean-k-mooney | jangutter: so bump for inheritance changes but not for composition | 13:30 |
| jaypipes | sean-k-mooney: that's not true any more AFAIK. | 13:30 |
| mriedem | pooja_jadhav: something like the setup in https://review.openstack.org/#/c/608771/ should be most of the work, | 13:31 |
| mriedem | then it's just querying the simple tenant usage API and asserting the results | 13:31 |
| sean-k-mooney | jaypipes: it better be or we are screwed | 13:31 |
| *** awaugama has joined #openstack-nova | 13:31 | |
| pooja_jadhav | mriedem: yeah.. thanks | 13:31 |
| mriedem | pooja_jadhav: we could just put the test patch on top of ^ to re-use the same setup | 13:31 |
| mriedem | i can try that quick | 13:31 |
| jaypipes | sean-k-mooney: the version manifest tracks versions for sub-classes separately from the base classes, meaning you don't need to bump the sub-class versions when a base version increases. | 13:32 |
| sean-k-mooney | jaypipes: correct but if we dont then adding a filed to the base mean the derived has a new filed also and no version bump | 13:32 |
| sean-k-mooney | similary if we remove a filed form the base that filed goes away in the derived without a version bump and we are similarly screwed | 13:33 |
| jaypipes | sean-k-mooney: the derived version is only indicating the version of the derived-specific fields. | 13:33 |
| jaypipes | dansmith: you up yet? :) | 13:33 |
| sean-k-mooney | jaypipes: no its indicating the version fo the whole object if not then its not safe to inherit OVOs | 13:34 |
| dansmith | jaypipes: yes | 13:34 |
| *** mchlumsky has joined #openstack-nova | 13:34 | |
| jaypipes | dansmith: we are discussing whether it is required to bump derived class object versions if a base versions is incremented. | 13:34 |
| jangutter | jaypipes, sean-k-mooney: If you don't bump because of composition, then you can flatten out the list of objects, regardless of how which members they belong to. | 13:34 |
| dansmith | jaypipes: if you change something in the base, you've changed it in the sub and the hash should change (thus need a version bump) IIRC | 13:35 |
| sean-k-mooney | dansmith: that is my understanding too. | 13:35 |
| dansmith | jaypipes: _simply_ changing the master version shouldn't require a child bump, but there would be no reason | 13:35 |
| jaypipes | dansmith: oh, am I confusing the composition rules with inheritance rules? | 13:35 |
| stephenfin | sean-k-mooney: Nope. Rather, we just should make that if conditional on whether we're requesting SR-IOV VIFs or not | 13:35 |
| jangutter | jaypipes, sean-k-mooney: I mean then you don't need to specifically associate _which_ VIF and which port-profile object/version combination go together, you just need to check if you understand each object/version combination separately. | 13:35 |
| dansmith | jaypipes: I think you're confusing how including an object in another object used to require lockstep versioning, but that isn't the case any more | 13:35 |
| sean-k-mooney | stephenfin: nope for macvtap i think you can set the model | 13:36 |
| jaypipes | dansmith: right. I was confusing composition rules with inheritance rules. sorry (again) :( | 13:36 |
| stephenfin | *SR-IOV direct | 13:36 |
| sean-k-mooney | stephenfin: im also not sure if that assumtion is safe to make but it might be | 13:36 |
| pooja_jadhav | mriedem: Let me try if u dont mind? | 13:37 |
| stephenfin | moshele: Any thoughts on the above? | 13:37 |
| stephenfin | moshele: tl;dr: I wonder if we should be setting 'model' here for SR-IOV VIFs? https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/virt/libvirt/vif.py#L130-L133 | 13:37 |
| sean-k-mooney | dansmith: right but if the composed object version change how we make object compatible today is actully inccorect | 13:37 |
| stephenfin | moshele: If we do and that hw_vif_model==virtio, we'll undo the hard work of https://github.com/openstack/nova/commit/622ebf2fab0a9bf75ee12437bef28f60e083f849 I think | 13:37 |
| stephenfin | moshele: SR-IOV direct VIFs, that is | 13:38 |
| jangutter | dansmith: if you use composition, would it make sense to check a list of classes and versions for compatibility, or would you need to check the combinations? | 13:38 |
| *** lbragstad has joined #openstack-nova | 13:38 | |
| sean-k-mooney | stephenfin: you can have VF that are virtio just an FYI | 13:38 |
| dansmith | sean-k-mooney: I don't understand what you're saying | 13:38 |
| dansmith | jangutter: heh, I'm also not sure what you mean | 13:39 |
| dansmith | point me at code? | 13:39 |
| jangutter | dansmith: let me set up a quick etherpad? | 13:39 |
| moshele | stephenfin: sorry I don't follow | 13:39 |
| dansmith | yah | 13:40 |
| sean-k-mooney | dansmith: if you look at https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/objects/migrate_data.py#L243-L270 we call support with the target version fo the derived but that may not corralate to a target version of the base | 13:40 |
| jaypipes | dansmith: they are wondering how the versioning system works when you have an ObjectField field... | 13:40 |
| jaypipes | dansmith: and if the version of the ObjectField changes, why the base doesn't need to change. | 13:40 |
| stephenfin | moshele: In https://github.com/openstack/nova/commit/622ebf2fab0a9bf75ee12437bef28f60e083f849 a check was added to ensure we don't see RX/TX queue sizes for non-virtio interfaces | 13:40 |
| jangutter | jaypipes, sean-k-mooney, dansmith: https://etherpad.openstack.org/p/ovo-versioning | 13:41 |
| dansmith | jaypipes: oh, because we send a manifest.. a list of object names and versions.. for rpc | 13:41 |
| sean-k-mooney | jaypipes: yes. and in the past we used to have a verion map thing that track the version fo the object fileds and what version the correspondeed to | 13:41 |
| dansmith | well, when we backport, we do that | 13:41 |
| stephenfin | I'm thinking it's possible we could break the check if a user sets 'hw_vif_type=virtio' in image metadata. If they do that, then this line will be true | 13:41 |
| stephenfin | https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/virt/libvirt/vif.py#L156 | 13:42 |
| sean-k-mooney | dansmith: that is part of my argument as to why we should version for compoltion and inheritenc but since the has does not change for compostion versiosn its not enforece by our tests | 13:43 |
| dansmith | sean-k-mooney: yeah that was a total disaster | 13:43 |
| stephenfin | which wouldn't be the case normally because this line would be false for direct https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/virt/libvirt/vif.py#L137 | 13:43 |
| dansmith | sean-k-mooney: it doesn't need to change | 13:43 |
| sean-k-mooney | dansmith: the lockstep stuff. it was a pain but it worked | 13:43 |
| dansmith | sean-k-mooney: it was unnecessary for us | 13:43 |
| stephenfin | moshele: If that makes sense? I'm just thinking it's a latent bug. I'll test myself but doing so requires me setting up an environment :) | 13:43 |
| dansmith | maybe for you because you don't have the same communication we have between nodes | 13:43 |
| sean-k-mooney | dansmith: it maye if version 1 of my containing clase used version 1 of a composed calss and version 2 of the conatine uses version 5 then downgrading the containing class to 1 results in it having an object filed of 5 | 13:44 |
| moshele | stephenfin: not sure, but you maybe right here | 13:44 |
| dansmith | sean-k-mooney: yep, which is fine, because the objects are forward compatible | 13:45 |
| mriedem | gibi: i'm late to this party but a few comments inline https://review.openstack.org/#/c/609330/ | 13:45 |
| dansmith | sean-k-mooney: and if there's some reason, the containing class can downlevel the version it has | 13:45 |
| mriedem | i haven't read the ML thread yet | 13:45 |
| mriedem | pooja_jadhav: go ahead | 13:46 |
| sean-k-mooney | dansmith: today we dont track that it should down level the object filed to 2 when it downlevels to 1 which is the gap | 13:46 |
| moshele | stephenfin: please update me with your result | 13:46 |
| stephenfin | moshele: Yup, will open a bug if it's an issue | 13:47 |
| dansmith | sean-k-mooney: we don't backlevel it if and only if the requesting party says that they support version 5 of the sub object | 13:47 |
| sean-k-mooney | dansmith: its totally solveable in the make_compatible function just we dont enforce it today and we should when we start passing os-vif objects over the api | 13:47 |
| dansmith | sean-k-mooney: which wouldn't happen, and thus we'd backport both | 13:47 |
| gibi | mriedem: you are not late at all | 13:47 |
| dansmith | sean-k-mooney: sure we don't enforce it | 13:48 |
| moshele | stephenfin: is 'hw_vif_type=virtio' in image metadata per network interface? | 13:48 |
| dansmith | sean-k-mooney: that's why I'm saying if you are making objects where that matters, then you can backlevel it | 13:48 |
| stephenfin | moshele: I don't think so. I don't see how it could be | 13:48 |
| sean-k-mooney | dansmith: this isnt in relation to RPC by they way. this is relation to passing os-vif object betwen nova an neutron via the rest api | 13:48 |
| stephenfin | moshele: Assuming you mean can it be configured per interface. I assume it affects all network interfaces | 13:48 |
| moshele | stephenfin: I see, so we need to ignore it or reject it | 13:49 |
| dansmith | sean-k-mooney: I know that, which is why I said you may have less flexibility because you don't control the communication in the same way | 13:49 |
| stephenfin | For direct SR-IOV, I would imagine | 13:49 |
| sean-k-mooney | dansmith: sure. we have added code in the past to do this and taken it out as part of code review because it did not go over rpc | 13:49 |
| dansmith | sean-k-mooney: fwiw, the relation mapping stuff is all still in o.vo because it's a library, so you can use it if you really think it's necessary | 13:49 |
| moshele | stephenfin: for VNIC_TYPES_DIRECT_PASSTHROUGH | 13:49 |
| stephenfin | moshele: yeah | 13:49 |
| sean-k-mooney | dansmith: ya we can but its also easy to do this in the make compatible fuction without that | 13:50 |
| sean-k-mooney | jangutter: does any of this help? | 13:50 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384 | 13:51 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: clean up ip_command interface https://review.openstack.org/609414 | 13:51 |
| jangutter | sean-k-mooney: I'm still downleveling the conversation to the simple version I can understand :-p | 13:51 |
| gibi | mriedem: good point about the pre 2.29 and pre 2.30 microversion. I have to think that through. | 13:52 |
| mriedem | gibi: i don't see operators on this thread either in the ML...has anyone reached out to the ops community to see if they are cool with dropping support for the force parameter? and if not, why not. | 13:52 |
| gibi | mriedem: bauzas forwarded it to the ops I think | 13:53 |
| mriedem | although as noted you can still use the force flag with the older microversion, or use 2.1 | 13:53 |
| dansmith | jangutter: so you might be assuming there's more magic going on here than there is | 13:53 |
| mriedem | this spec is really just about signaling | 13:53 |
| mriedem | like when we deprecated personality files | 13:53 |
| dansmith | jangutter: you can't define that a field is an object at (or below or above) a particular version | 13:53 |
| bauzas | gibi: mriedem: yup but no answers yet | 13:53 |
| bauzas | gibi: and I asked you about two separate microversions in the change | 13:53 |
| jangutter | dansmith: right, this comes into play when serializing | 13:53 |
| dansmith | jangutter: within a scope, it's assumed that you support a set of object schemas (by name) at a particular version, with compatibility for older versions | 13:53 |
| mriedem | bauzas: no one but you wants 2 microversions for this | 13:53 |
| gibi | bauzas: it seems to me that others more like having a single microversion instead | 13:54 |
| mriedem | 2 microversions for the same thing is excessive | 13:54 |
| mriedem | as efried noted in the spec review, we don't do multiple microversions for something just because it touches multiple APIs | 13:54 |
| dansmith | jaypipes: so when we serialize today, we have the object and a list of versions that the other side supports, and we backlevel everything in the tree to the versions they support, as needed | 13:54 |
| dansmith | er jangutter ^ | 13:54 |
| mriedem | otherwise i would have had multiple microversions for volume multiattach and removing personality files | 13:54 |
| dansmith | jangutter: if you don't have that manifest of supported versions, then you'd have to make inferences based on the version of the parent you're pinned to, which is what we used to do | 13:55 |
| dansmith | jangutter: like this: https://github.com/openstack/nova/blob/kilo-eol/nova/objects/instance.py#L257 | 13:55 |
| dansmith | jangutter: if you set those up on your object then o.vo will still respect them | 13:55 |
| jangutter | dansmith: yep. the trick comes when downlevelling an object that has members containing other objects which are versioned. | 13:55 |
| dansmith | jangutter: right, but that's what this obj_relationships is for | 13:56 |
| sean-k-mooney | dansmith: that is assume that the latest version of each object both side knows about is the desired version correct | 13:56 |
| dansmith | jangutter: note that it specifies which versions the contains-another-object fields refer to | 13:56 |
| bauzas | mriedem: gibi: okay, just the fact that we had 2 different versions previously (2.29 and 2.30) | 13:56 |
| jangutter | dansmith: I've seen that mapping, the question is, do we need/want it in os-vif? | 13:56 |
| dansmith | sean-k-mooney: it means within a scope you don't randomly upgrade one object and not another yeah | 13:57 |
| dansmith | sean-k-mooney: which is what this was designed to do, not for "I loaded a new plugin which loaded one new object into a sea of older ones" | 13:57 |
| sean-k-mooney | dansmith: right... so we might have to deal with the lather in os-vif but perhaps we dont | 13:58 |
| dansmith | yeah I dunno | 13:58 |
| mriedem | bauzas: we had 2 different versions previously because of all the related plumbing those required | 13:58 |
| mriedem | this is a simple schema change | 13:59 |
| sean-k-mooney | dansmith: did any of this impact out of tree virt drivers on the nova side? or are all the OVOs managed above the virt dirver level | 13:59 |
| bauzas | mriedem: okay, fair then | 13:59 |
| dansmith | sean-k-mooney: we don't support out of tree virt drivers | 13:59 |
| *** tbachman has quit IRC | 13:59 | |
| sean-k-mooney | dansmith: :) well thats one answer. but i think we are still fine. plugins are not allowed to provide vif types or profiles they can only consume them so i think we wont have the mixed case | 14:00 |
| dansmith | oh okay I thought that was the point of os-vif | 14:00 |
| dansmith | but cool if not | 14:00 |
| sean-k-mooney | plugins can consume the datamodels but not extend them | 14:01 |
| sean-k-mooney | dansmith: that said kuryr-kubernetes are violating that contract https://github.com/openstack/kuryr-kubernetes/blob/master/kuryr_kubernetes/objects/vif.py | 14:02 |
| mriedem | bauzas: i think you wanted to copy the ops list but didn't | 14:03 |
| mriedem | "adding openstack-operators@ accordingly." | 14:03 |
| mriedem | the ops list isn't on copy | 14:03 |
| sean-k-mooney | dansmith: i told them that this was not be supported a few cycles ago and we will be moving those vif types before nova and neutron start useing os-vif objects to negociate bindings | 14:03 |
| mriedem | oh i see, "Shit, I forgot to add openstack-operators@..." | 14:03 |
| bauzas | mriedem: I had a problem with my other gmail address | 14:03 |
| bauzas | hence the three emails | 14:03 |
| *** tbachman has joined #openstack-nova | 14:03 | |
| * bauzas hides | 14:04 | |
| bauzas | hopefully the single ML address will simplify it | 14:04 |
| dansmith | sean-k-mooney: ack | 14:04 |
| jangutter | dansmith, sean-k-mooney, jaypipes: thanks very much for this discussion, I think it might be a good time to explicitly fix os-vif into the simpler model rather than allowing a possible version explosion. | 14:06 |
| sean-k-mooney | jangutter: im not sure how you would do that | 14:07 |
| jaypipes | jangutter: well, since nothing yet uses the object-over-http I wouldn't mind keeping all the versions at 1.0 at this point. | 14:08 |
| sean-k-mooney | i dont think its partacally complex | 14:08 |
| jaypipes | but I need to run away now for a half hour. back soon. | 14:08 |
| *** munimeha1 has joined #openstack-nova | 14:08 | |
| sean-k-mooney | jaypipes: we proably can make the all 1 but i would be fine to bump them all to 2.0 or something wehn we use them over http | 14:09 |
| jaypipes | sean-k-mooney: ++ | 14:09 |
| jaypipes | sean-k-mooney: and after the version manifest stuff is added in... | 14:09 |
| sean-k-mooney | jaypipes: yes assuming that is needed | 14:11 |
| sean-k-mooney | we will need to document all this in a spec in any case | 14:11 |
| *** dtantsur is now known as dtantsur|brb | 14:13 | |
| mriedem | gibi: jaypipes: bauzas: efried: i'm all caught up on that ML thread, | 14:14 |
| mriedem | replies inline, | 14:14 |
| mriedem | tl;dr i agree with jay on failing hard if the source has nested allocatoins and we're being forced | 14:14 |
| efried | ++ | 14:15 |
| bauzas | yup, +1 | 14:15 |
| mriedem | force predates these exotic topologies | 14:15 |
| bauzas | if they really want to migrate, just don't force | 14:15 |
| mriedem | and we shouldn't attempt to support it | 14:15 |
| bauzas | so, first they force, and if they get a non-accepted migration, they could just not force | 14:15 |
| bauzas | which looks good to me | 14:15 |
| mriedem | right | 14:16 |
| sean-k-mooney | mriedem: taking that one step futher thw will mean eventually force will never work as at some point all compute nodes will be nested | 14:16 |
| gibi | mriedem: so do we all agree that moving from nested to flat is OK to fail even if it would succeed in placement? | 14:16 |
| mriedem | the one case i can see for using force is what bauzas mentioned in the thread, which is you disable a compute so new instances can't go there, meanwhile rebalancing by forcing things there via live migrate | 14:16 |
| *** alexchadin has quit IRC | 14:16 | |
| mriedem | gibi: that would only succeed in the case that the dest host is not upgraded and reshaped yet | 14:17 |
| mriedem | right? | 14:17 |
| gibi | mriedem: right | 14:17 |
| mriedem | so it's a narrow window where we might get lucky and it works | 14:17 |
| jangutter | sean-k-mooney: one way of doing this is to explicitly ignore relationships established by composition - in other words, you assume that a particular plugin will not require distinct port_profile versions for different vifs that it supports. | 14:17 |
| mriedem | so i don't care about supporting that | 14:17 |
| gibi | mriedem: yes | 14:17 |
| gibi | mriedem: OK | 14:17 |
| * gibi changing in flight implementation :) | 14:17 | |
| mriedem | sean-k-mooney: yes i'm ok with that | 14:17 |
| bauzas | mriedem: for the disabled host, they can still call the scheduler | 14:17 |
| bauzas | mriedem: but yeah the computefilter will then return no | 14:18 |
| mriedem | bauzas: the ComputeFilter will reject it | 14:18 |
| mriedem | right | 14:18 |
| sean-k-mooney | mriedem: i know :) so am i force is a pain in the ass. but where i was going with this is shoudl we consider deprecating force | 14:18 |
| mriedem | sean-k-mooney: that's gibi's spec | 14:18 |
| mriedem | https://review.openstack.org/#/c/609330/1 | 14:18 |
| bauzas | mriedem: actually I guess that's probably why force_hosts on boot doesn't run on filters | 14:18 |
| bauzas | mriedem: but we shouldn't really skip all filters, just the compute one | 14:18 |
| sean-k-mooney | mriedem: oh :) | 14:19 |
| sean-k-mooney | jangutter: ya i can see jsut removing the version info in general that said it may be useful to keep jsut so we can say this is the newest version i supprot to nova/neutron | 14:19 |
| bauzas | mriedem: jaypipes: gibi: actually, hold on | 14:19 |
| bauzas | I think the usecase I said for force is still needed for operators :( | 14:20 |
| * gibi is holding on | 14:20 | |
| bauzas | because they probably want to migrate instances to some disabled compute | 14:20 |
| jangutter | sean-k-mooney: you can still keep port_profiles versioned, but you don't require the combination mapping. | 14:20 |
| sean-k-mooney | jangutter: so in teh host info obejct jsut filter by name not by version, but keep version in the object to say i know about at most version x of this object | 14:20 |
| *** k_mouza has joined #openstack-nova | 14:20 | |
| sean-k-mooney | jangutter: yep | 14:20 |
| sean-k-mooney | bauzas: but why do we want to supprot that | 14:21 |
| sean-k-mooney | they may but it instead of using disabled to reserve a host for mainainace would it not be better to have a maintanace availablity zone they could move the host into | 14:22 |
| bauzas | sean-k-mooney: I guess operators want to migrate some instances from some host because for example the RAID situation is bad | 14:22 |
| sean-k-mooney | bauzas: well migrating from a disable host is fine | 14:22 |
| bauzas | sean-k-mooney: but then they want to migrate to a single host which is exactly like the source one for all the instances | 14:23 |
| sean-k-mooney | bauzas: migrating too a disable host i think is strange | 14:23 |
| bauzas | sean-k-mooney: because of capacity I guess | 14:23 |
| bauzas | gibi: either way, I think we need to be super clear in the spec that this usecase won't be possible once we remove the force field in a microversion | 14:24 |
| mriedem | bauzas: force_hosts on boot still goes through the filters | 14:24 |
| bauzas | mriedem: not thru the filters | 14:24 |
| bauzas | mriedem: thru the filter scheduler yep | 14:24 |
| bauzas | but then it says "all good" | 14:24 |
| bauzas | and then returens | 14:24 |
| bauzas | returns | 14:24 |
| mriedem | show me the code | 14:24 |
| *** mlavalle has joined #openstack-nova | 14:24 | |
| *** k_mouza has quit IRC | 14:25 | |
| sean-k-mooney | bauzas: your refing to when you use --availability-zone ZONE:HOST right | 14:25 |
| gibi | bauzas: I can make it clear in the spec that force migration or force evacuating to the disable destination will not work as scheduler's ComputeFilter will reject that host | 14:25 |
| sean-k-mooney | which adds the force flag an just check the availablity zone and host exits then skips all the filters | 14:25 |
| bauzas | mriedem: https://github.com/openstack/nova/blob/master/nova/scheduler/host_manager.py#L589 | 14:25 |
| mriedem | yup just found that | 14:26 |
| mriedem | huh i didn't realize | 14:26 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Deprecate the 'discover_hosts_in_cells_interval' option https://review.openstack.org/609346 | 14:26 |
| *** spatel has joined #openstack-nova | 14:26 | |
| mriedem | bauzas: "I think we need to be super clear in the spec that this usecase won't be possible once we remove the force field in a microversion" isn't accurate | 14:27 |
| mriedem | you can still hit the force code with the older microversions | 14:27 |
| mriedem | and osc's default behavior | 14:27 |
| bauzas | mriedem: okay, I wasn't clear | 14:27 |
| bauzas | I meant "microversions >2.XX won't support this usecase now" | 14:27 |
| gibi | mriedem, bauzas: but that old code won't work for nested allocatons | 14:27 |
| mriedem | gibi: yeah i left a comment on that in the spec just now, and added mnaser and tobias from city network (public cloud SIG chair) | 14:27 |
| mriedem | gibi: agree | 14:28 |
| mriedem | i know mnaser does the rebalance dance, | 14:28 |
| bauzas | mriedem: both are in OSDN AFAIK | 14:28 |
| mriedem | i just don't know if he disables computes before doing so | 14:28 |
| bauzas | at least that's what twitter claims :) | 14:28 |
| gibi | sooo, the operators will use the force case totally as soon as the allocations are nested | 14:28 |
| gibi | regardless of microversion | 14:28 |
| sean-k-mooney | mriedem: have we increased the minium microverion by the way since we started using them. | 14:28 |
| mriedem | no we don't do that | 14:28 |
| gibi | s/use/lose/ | 14:28 |
| sean-k-mooney | mriedem: so we will always effectly have to support force then as you can always use an old microverion ? | 14:29 |
| mriedem | until everything is nested | 14:29 |
| gibi | sean-k-mooney: we could not suppor force for nested allocations | 14:29 |
| mriedem | which is whenever jaypipes' cpu resource tracking stuff happens | 14:30 |
| sean-k-mooney | mriedem: or numa in placement | 14:30 |
| mriedem | unless we start supporting VMs without compute resources... | 14:30 |
| mriedem | you can't live migrate with numa today anyway so meh | 14:30 |
| sean-k-mooney | mriedem: oh you mean "serverless" workloads | 14:30 |
| mriedem | sean-k-mooney: it was a joke | 14:30 |
| *** lyarwood has joined #openstack-nova | 14:31 | |
| *** johnsom has joined #openstack-nova | 14:31 | |
| sean-k-mooney | mriedem: yes but i just wanted to bitch about "serverless" marketing | 14:31 |
| bauzas | MEEEEH | 14:31 |
| mriedem | qinling already handles that | 14:31 |
| mriedem | see https://qinling.readthedocs.io/ | 14:32 |
| mriedem | heh | 14:32 |
| mriedem | https://docs.openstack.org/qinling/latest/ | 14:32 |
| sean-k-mooney | mriedem: either jays cpu think of bauzas's numa stuff "should" land in stein | 14:32 |
| bauzas | there are a couple of proposals about serverless w/ OpenStack that don't necessarly imply a high-level OpenStack service for this, but meh :) | 14:32 |
| sean-k-mooney | mriedem: so does storelets | 14:32 |
| bauzas | anyway, we're running VMs | 14:33 |
| sean-k-mooney | https://github.com/openstack/storlets | 14:33 |
| bauzas | looks like we're diverting | 14:33 |
| pooja_jadhav | mriedem: able to write FT :).. thanks alot | 14:33 |
| sean-k-mooney | bauzas: sorry your right. so are you ok with documenting that force is availabel for old microverions and will not be supported for nested hosts? | 14:34 |
| mriedem | pooja_jadhav: np. | 14:34 |
| mriedem | pooja_jadhav: i'd recommend starting with a patch that adds the functional test to show the bug, as my patch does | 14:35 |
| mriedem | then you can work the fix on top of that separately | 14:35 |
| *** moshele has quit IRC | 14:35 | |
| pooja_jadhav | mriedem: sure | 14:35 |
| *** mvkr has joined #openstack-nova | 14:36 | |
| *** ttsiouts has quit IRC | 14:40 | |
| *** ttsiouts has joined #openstack-nova | 14:46 | |
| *** tbachman has quit IRC | 14:47 | |
| *** tbachman has joined #openstack-nova | 14:48 | |
| *** Luzi has quit IRC | 14:55 | |
| *** maciejjozefczyk has quit IRC | 14:55 | |
| stephenfin | moshele, sean-k-mooney: Yeah, it's a bug https://bugs.launchpad.net/nova/+bug/1797146 | 14:56 |
| openstack | Launchpad bug 1797146 in OpenStack Compute (nova) "failed to boot guest with vnic_type direct when rx_queue_size, tx_queue_size and hw_vif_type are set" [Undecided,New] | 14:56 |
| sean-k-mooney | stephenfin: yes but only for vnic_types direct or direct_physical corect | 14:58 |
| stephenfin | yup | 14:58 |
| sean-k-mooney | stephenfin: if the guest also has ovs ports they should still use the specifid model | 14:58 |
| stephenfin | Yup https://bugs.launchpad.net/nova/+bug/1797146/comments/1 | 14:58 |
| openstack | Launchpad bug 1797146 in OpenStack Compute (nova) "failed to boot guest with vnic_type direct when rx_queue_size, tx_queue_size and hw_vif_type are set" [Undecided,New] | 14:58 |
| sean-k-mooney | we should test macvtap ports too | 14:58 |
| stephenfin | sean-k-mooney: Is there any configuration where we'd have a direct or direct_physical vnic_type and still use virtio? | 14:59 |
| stephenfin | I'm assuming not because those are two opposing things | 15:00 |
| sean-k-mooney | stephenfin: yes but we should ignore the model in that case | 15:00 |
| sean-k-mooney | stephenfin: e.g. if the VF itself is a virtio-net-pci device that is fine | 15:01 |
| sean-k-mooney | but we should not set the model to virtio expcitly in that case as qemu is not emulating virtio | 15:01 |
| sean-k-mooney | virtio is being implemneted in hardware | 15:01 |
| stephenfin | Sweet | 15:02 |
| stephenfin | That makes this nice and easy so | 15:02 |
| sean-k-mooney | stephenfin: just add vnic_type not in VNIC_TYPES_DIRECT_PASSTHROUGH | 15:02 |
| stephenfin | sean-k-mooney: Yup, exactly what I'm doing | 15:02 |
| sean-k-mooney | VNIC_TYPES_DIRECT_PASSTHROUGH is form https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/network/model.py#L116 | 15:02 |
| sean-k-mooney | :) | 15:02 |
| melwitt | . | 15:03 |
| sean-k-mooney | melwitt: i read "." as basically you saying "um" and pausing | 15:03 |
| melwitt | I don't know what that means, but ok :) | 15:04 |
| sean-k-mooney | melwitt: you typed "." on irc so i just assumed you were about to say something and paused to think for a sec | 15:05 |
| *** ociuhandu has quit IRC | 15:05 | |
| *** ociuhandu has joined #openstack-nova | 15:08 | |
| *** lpetrut has quit IRC | 15:11 | |
| sean-k-mooney | stephenfin: your activly fixing https://bugs.launchpad.net/nova/+bug/1797146 right so im jsut going to assign it to you on launchpad | 15:11 |
| openstack | Launchpad bug 1797146 in OpenStack Compute (nova) "failed to boot guest with vnic_type direct when rx_queue_size, tx_queue_size and hw_vif_type are set" [Medium,Confirmed] | 15:11 |
| stephenfin | Go for it | 15:11 |
| *** panda has quit IRC | 15:13 | |
| jaypipes | mriedem: have you ever used the "query" scheduler hint? https://github.com/openstack/nova/blob/0163b9bfb54aaa89b0574c86e7fd36321eebccfe/nova/api/openstack/compute/schemas/servers.py#L122 | 15:14 |
| *** gyee has joined #openstack-nova | 15:14 | |
| *** panda has joined #openstack-nova | 15:14 | |
| sean-k-mooney | jaypipes: as in the json filter "query" schduer hint | 15:14 |
| *** spatel has quit IRC | 15:15 | |
| mriedem | jaypipes: hell no | 15:16 |
| cfriesen | efried: jaypipes: new version of the emulated TPM spec is up. you folks okay with HW_SYSTEM_TPM for the trait, or do you want something like COMPUTE_SECURITY_TPM ? | 15:18 |
| *** spatel has joined #openstack-nova | 15:19 | |
| sean-k-mooney | cfriesen: when i was proposing tpm traits before i was going with something slightly different https://review.openstack.org/#/c/514712/3/os_traits/hw/platform/security.py | 15:19 |
| *** k_mouza has joined #openstack-nova | 15:19 | |
| sean-k-mooney | cfriesen: so it would be HW_PLATFORM_SECURITY_TPM | 15:20 |
| sean-k-mooney | or HW_PLATFORM_SECURITY_TPM_2_0 | 15:21 |
| *** ttsiouts has quit IRC | 15:21 | |
| *** lpetrut has joined #openstack-nova | 15:21 | |
| sean-k-mooney | COMPUTE_SECURITY_TPM would indicate taht the hypervior can emulate a TPM and HW_PLATFORM_SECURITY_TPM would be the host has a tpm | 15:21 |
| *** ttsiouts has joined #openstack-nova | 15:21 | |
| cfriesen | yeah, in the review I did call out whether we want to embed the TPM version in the trait | 15:22 |
| cfriesen | sean-k-mooney: I thought from the hangout that physical TPM would be handled via a resource with inventory? | 15:23 |
| *** spatel has quit IRC | 15:23 | |
| *** k_mouza has quit IRC | 15:23 | |
| sean-k-mooney | in the tpm i guess it could be as it is passthough to the vm and not subdevied | 15:23 |
| sean-k-mooney | in which case COMPUTE_SECURITY_TPM should be the only trait that is needed | 15:24 |
| jaypipes | sean-k-mooney: is TPM2.0 an Intel-specific thing? | 15:24 |
| sean-k-mooney | moduleo adding a version | 15:24 |
| sean-k-mooney | jaypipes: no its an open standard | 15:24 |
| jaypipes | k. | 15:24 |
| jaypipes | sean-k-mooney: I was going to suggest prefixing with X86 if it was Intel-specific. | 15:24 |
| sean-k-mooney | jaypipes: https://www.iso.org/standard/66510.html | 15:25 |
| cfriesen | okay, I can switch to COMPUTE_SECURITY_TPM_1_2 and COMPUTE_SECURITY_TPM_2_0 if jaypipes is cool with that | 15:25 |
| jaypipes | sean-k-mooney: we have used COMPUTE_ to refer to virt-driver specific capabilities. I don't believe this is that? | 15:26 |
| jaypipes | sean-k-mooney: isn't TPM a hardware thing? | 15:26 |
| cfriesen | jaypipes: with the trait we're talking about an emulated TPM | 15:26 |
| cfriesen | the hardware TPM would be a resource since there would be a finite number of them | 15:26 |
| jaypipes | ah, right.. | 15:26 |
| *** ttsiouts has quit IRC | 15:26 | |
| jaypipes | OK, good with me then. | 15:26 |
| sean-k-mooney | yep what cfriesen said :) | 15:27 |
| jaypipes | this is what I was confusing HPET with :) | 15:27 |
| sean-k-mooney | yep | 15:27 |
| cfriesen | okay, I'll respin the TPM spec with that minor change | 15:27 |
| jaypipes | cfriesen: you proposing the os-trait patch? | 15:27 |
| cfriesen | sure | 15:27 |
| * jaypipes still parsing sean-k-mooney's words... "subdevied" is a new one to me. | 15:28 | |
| sean-k-mooney | *subdivided | 15:28 |
| jaypipes | cfriesen: ok, when you do, please be sure to be crystal clear that the trait refers to the virt-driver emulated capability, not a physical resource. | 15:29 |
| jaypipes | sean-k-mooney: I'm kidding with you :) | 15:29 |
| cfriesen | jaypipes: right, makes sense | 15:29 |
| jaypipes | ty sir | 15:29 |
| sean-k-mooney | what i have realised while typeing used to massively reduce the amount of missspelling i made due to reversing or omitting letter now that i touch type entirly without thinking they are reappearing | 15:30 |
| *** macza has joined #openstack-nova | 15:31 | |
| sean-k-mooney | it used to be a huge problem in school when i would hand write things and just stop writing a word half way through and start in the middle fo the next one bacially because my hand could not keep up with my brian when writing | 15:31 |
| jangutter | In the past few years I've noticed that I've started to mistype things really really badly, reading back the sentence once is no longer enough for me to figure out what I messed up. | 15:32 |
| jangutter | Worse, it's happening to my speech too. Pretty hilarious when you realise that your words leaving your mouth and in your brain don't match up. | 15:33 |
| *** mdbooth has quit IRC | 15:34 | |
| jangutter | but really, I try to spread that rumour so I can claim I actually meant "offloads" when I said "screwdriver". | 15:34 |
| *** lpetrut has quit IRC | 15:37 | |
| *** mvkr has quit IRC | 15:40 | |
| *** helenafm has joined #openstack-nova | 15:41 | |
| *** ociuhandu has quit IRC | 15:41 | |
| *** Guest10461 is now known as dims | 15:44 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Ignore hw_vif_type for direct, direct-physical vNIC types https://review.openstack.org/609460 | 15:48 |
| stephenfin | sean-k-mooney: ^ | 15:49 |
| openstackgerrit | Chris Friesen proposed openstack/nova-specs master: Add support for emulated virtual TPM https://review.openstack.org/571111 | 15:57 |
| *** udesale has quit IRC | 15:57 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Replace openSUSE experimental check with newer version https://review.openstack.org/609467 | 16:00 |
| *** mvkr has joined #openstack-nova | 16:02 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Consider nested allocations during allocation cleanup https://review.openstack.org/606050 | 16:03 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Reject forced move with nested source allocation https://review.openstack.org/605785 | 16:03 |
| gibi | jaypipes, mriedem, bauzas, efried: here is the rework of the force migration with nested allocation https://review.openstack.org/#/c/605785 | 16:04 |
| gibi | and I'm leaving for today | 16:05 |
| bauzas | I'm done for the day, but I'll look tomorrow | 16:05 |
| bauzas | hah | 16:05 |
| sean-k-mooney | stephenfin: can you take a look at https://review.openstack.org/#/c/609414/ before you leave | 16:09 |
| stephenfin | sean-k-mooney: Can do | 16:09 |
| sean-k-mooney | master is currently broken for hardware offloaded ovs and some other cases this will fix it and harden up the interface a little | 16:09 |
| *** dtantsur|brb is now known as dtantsur | 16:15 | |
| *** k_mouza has joined #openstack-nova | 16:26 | |
| stephenfin | sean-k-mooney: Yeah, good spot | 16:26 |
| sean-k-mooney | stephenfin: lennyb spotted it and pingged rodlfo and i on the neutron irc this morning | 16:27 |
| *** k_mouza has quit IRC | 16:28 | |
| sean-k-mooney | i was thinking about fixing the name spaceing of those module too but i think there is already enough in one patch | 16:28 |
| sean-k-mooney | i might submit a followup later | 16:28 |
| stephenfin | sean-k-mooney: I'm going to see if we can do the same thing claudiub did here to avoid that happening again https://review.openstack.org/#/c/470775/ | 16:28 |
| *** k_mouza has joined #openstack-nova | 16:28 | |
| sean-k-mooney | stephenfin: im not sure that would have help in this case but i might not under stand awhat that dose enough either | 16:29 |
| stephenfin | sean-k-mooney: Yeah, it wouldn't actually. I thought https://review.openstack.org/#/c/609414/1/os_vif/tests/unit/internal/command/ip/windows/test_impl_netifaces.py was mocking the wrong stuff but it's not | 16:30 |
| *** moshele has joined #openstack-nova | 16:30 | |
| sean-k-mooney | this is the improtant test https://review.openstack.org/#/c/609414/1/os_vif/tests/unit/internal/command/ip/test_api.py | 16:31 |
| sean-k-mooney | before we were returing an instance of the ip lib for windows and the module for linux | 16:31 |
| stephenfin | sean-k-mooney: Just left a nit comment on that, actually | 16:31 |
| *** mrjk has joined #openstack-nova | 16:32 | |
| mriedem | dansmith: on that initial allocation ratios spec, there is one place we still use the compute node fields, and that's in the scheduler https://github.com/openstack/nova/blob/6bf11e1dc14afad78b11d980c2544a3dc41579ff/nova/scheduler/host_manager.py#L256 | 16:32 |
| sean-k-mooney | ya i realsied i coudl have used that too. im going to be adding more unit tests wehn ir change the module layout so ill change that then if your ok with that | 16:32 |
| mriedem | so we set the values on the compute (in the RT) and then read in the scheduler | 16:32 |
| dansmith | mriedem: but aren't those just for the filters? | 16:33 |
| dansmith | what else cares about those? | 16:33 |
| *** k_mouza has quit IRC | 16:33 | |
| mriedem | weighers use it too i guess | 16:33 |
| *** tssurya has joined #openstack-nova | 16:33 | |
| dansmith | to pick things that aren't set for oversubscription? | 16:33 |
| dansmith | that seems weird | 16:33 |
| mriedem | the numa topology filter is also using it | 16:34 |
| dansmith | oh, numa filter uses it | 16:34 |
| dansmith | hah | 16:34 |
| dansmith | christ | 16:34 |
| dansmith | so | 16:34 |
| dansmith | (a) we should try to get someone to fix that | 16:34 |
| stephenfin | sean-k-mooney: Yup, it's just a nit | 16:35 |
| dansmith | (b) we can just update the compute node record with the same policy as we update placement | 16:35 |
| mriedem | which is use config if set | 16:35 |
| mriedem | that's what the spec proposed yeah | 16:35 |
| dansmith | yep | 16:35 |
| dansmith | okay I see the use in the weigher | 16:35 |
| mriedem | the question was about upgrading from compute nodes that have 0.0 values in the db | 16:35 |
| mriedem | and i said we could online data migrate those by reading config if the values are 0.0 | 16:36 |
| dansmith | the 0.0 means that scheduler uses its own config, right? | 16:36 |
| sean-k-mooney | regarding the numa topology filter im not sure it actully need the allocation ratiors | 16:36 |
| mriedem | no | 16:36 |
| dansmith | allocation_ratio of zero makes no sense otherwise right? | 16:36 |
| mriedem | nothing in the scheduler reads config for allocation ratios | 16:36 |
| dansmith | mriedem: it used to though yeah? | 16:36 |
| mriedem | in the long long ago i guess, | 16:37 |
| mriedem | oh well, | 16:37 |
| mriedem | the facade would i guess yeah... | 16:37 |
| dansmith | point being, 0.0 is nonsense | 16:37 |
| mriedem | the ComputeNode._from_db_object would read from config if 0.0 in the db | 16:37 |
| jaypipes | dansmith: ++ | 16:37 |
| dansmith | so we could leave that, and only set it if they override per compute.. same logic as placement | 16:37 |
| dansmith | and anything existing with 0.0 gets the same treatment as usual I guess | 16:38 |
| mriedem | leave the facade in ComputeNode._from_db_object? | 16:38 |
| dansmith | i imagine it won't affect compute's use if it is ignoring what is set on the object, so .. sure? | 16:38 |
| sean-k-mooney | stephenfin: for https://github.com/openstack/nova/blob/0163b9bfb54aaa89b0574c86e7fd36321eebccfe/nova/scheduler/filters/numa_topology_filter.py#L91-L102 can you think of a use case wehre we actully need the allcoation ratiios here | 16:38 |
| sean-k-mooney | stephenfin: we are not allowed to over subsibe against our selves wehn fitting to a host so setting them to 1.0 i think would be valid | 16:39 |
| sean-k-mooney | stephenfin: i would have to go through the code to check but i think we could make the numa topolgy filter work without them. | 16:42 |
| sean-k-mooney | dansmith: although it sound like we dont need to remove them if we go wtih the facade right | 16:42 |
| dansmith | sean-k-mooney: it won't be right if you do | 16:43 |
| dansmith | sean-k-mooney: because if people set the ratio in placement per-compute, which is what we're trying to enable with all of this work, | 16:43 |
| dansmith | the filter will consider a value other than what is in placement | 16:43 |
| dansmith | I'm guessing we don't get back ratios in /a_c, but if we did, we could update our host states before we call the filters | 16:44 |
| sean-k-mooney | dansmith: yes but this is not using the placement value anyway | 16:44 |
| jaypipes | mriedem: why would we keep the facade stuff in ComputeNode._from_db_object()? | 16:44 |
| dansmith | sean-k-mooney: right but right now they have to be the same | 16:44 |
| dansmith | sean-k-mooney: in the future they will not be | 16:44 |
| dansmith | jaypipes: see the discussion just now on the filters that use it, and existing computes in the db with 0.0 set | 16:44 |
| mriedem | jaypipes: we either need to leave that or online data migrate the 0.0 entries from existing records in the db on read | 16:45 |
| sean-k-mooney | dansmith: placement woudl have already filtered out any host that did not pass its allocation right. so we should not have to check twice and sicne we can oversubsibe againat ourselve an allocation ration of 1 i think would still be correct | 16:45 |
| sean-k-mooney | dansmith: anywway its not important right now i guess | 16:45 |
| mriedem | jaypipes: because the compute won't deal with those fields on the object, and if not set, the scheduler reads them from config via the facade | 16:45 |
| dansmith | sean-k-mooney: I didn't look to see what that code in the filter was doing, so maybe? | 16:45 |
| dansmith | sean-k-mooney: if so, we just remove it right? | 16:46 |
| * jaypipes remembers why he stepped away from these specs. | 16:46 | |
| sean-k-mooney | dansmith: ya i think soo but i would have to double check the hardware.numa_fit_instance_to_host fucntion first | 16:46 |
| dansmith | we get back the resource summaries from the providers in /a_c, so including the allocation_ratios in there might be useful for things like this and for the weigher case | 16:47 |
| sean-k-mooney | dansmith: ya. the filters dont currently have access to the allocation candiates today is that correct | 16:48 |
| sean-k-mooney | unless there in the spec_objec? | 16:48 |
| dansmith | sean-k-mooney: that isn't what I'm saying | 16:48 |
| dansmith | I'm saying the scheduler, when it gets back candidates, gets a summary of all covered providers, with inventory information | 16:49 |
| dansmith | if that included the ratios, it could update host_states before calling the filter loop | 16:49 |
| jaypipes | sean-k-mooney: pls see my comment on https://review.openstack.org/#/c/609414/ | 16:49 |
| sean-k-mooney | oh ok that would work too ya i was assuming you were suggsting passing in the candiates but your way we do that update once and dont have to update any code in the filters | 16:49 |
| sean-k-mooney | jaypipes: there isnt a bug number because i just got pingged on irc this morning but i can open one | 16:50 |
| dansmith | mriedem: melwitt tssurya: cells meeting today? | 16:51 |
| mriedem | nack | 16:51 |
| mriedem | we might want to just cancel that meeting | 16:51 |
| dansmith | I'd also be fine with that | 16:51 |
| *** k_mouza has joined #openstack-nova | 16:51 | |
| dansmith | I think I suggested that last year even | 16:51 |
| *** helenafm has quit IRC | 16:52 | |
| jaypipes | sean-k-mooney: yes pls. if this is truly a "currently broken for hardware offloaded ovs" scenario, it definitely should be a bug. | 16:53 |
| *** k_mouza has quit IRC | 16:54 | |
| tssurya | dansmith: no problems in cancelling | 16:54 |
| sean-k-mooney | its broke for all ovs backends that use ip command because api _get_impl on linux retrun a module instead of the insatnce of the pyroute2 class | 16:55 |
| mriedem | dansmith: you want to propose the change to cancel the meeting or want me to? | 16:56 |
| sean-k-mooney | jaypipes: os that is ovs + iptables or ovs + hardware offloads. i have added some extra test to catch this case | 16:56 |
| dansmith | melwitt: what do you think about canceling the cells meeting altogether and making it ad-hoc as needed? | 16:56 |
| dansmith | mriedem: I figure we don't need to make a federal case out of it | 16:56 |
| dansmith | if everyone agrees, we just take it off the schedule | 16:56 |
| mriedem | dansmith: it's literally in a schedule though | 16:56 |
| dansmith | I know | 16:56 |
| mriedem | http://git.openstack.org/cgit/openstack-infra/irc-meetings/tree/meetings/nova-cells-v2-meeting.yaml | 16:56 |
| dansmith | oh, | 16:57 |
| dansmith | that schedule | 16:57 |
| mriedem | yeah, free up the time slot in that channel if we're not going to use it | 16:57 |
| dansmith | I thought it was just on the old wiki list | 16:57 |
| mriedem | no this is very official and federale | 16:57 |
| dansmith | I'll propose.. I need to do something useful today | 16:57 |
| *** k_mouza has joined #openstack-nova | 16:58 | |
| *** macza has quit IRC | 16:59 | |
| *** macza has joined #openstack-nova | 16:59 | |
| melwitt | dansmith: yup sounds ok to me | 17:01 |
| dansmith | https://review.openstack.org/#/c/609496/ | 17:01 |
| *** k_mouza has quit IRC | 17:02 | |
| *** derekh has quit IRC | 17:03 | |
| *** k_mouza has joined #openstack-nova | 17:05 | |
| *** k_mouza has quit IRC | 17:08 | |
| mriedem | hmm live migration failure in the gate, not something i've seen before i don't think, looks like it was aborted but i'm not sure why | 17:09 |
| mriedem | http://logs.openstack.org/31/606031/4/check/nova-live-migration/9d106bb/logs/subnode-2/libvirt/libvirtd.txt.gz#_2018-10-10_15_27_01_313 | 17:09 |
| mriedem | 2018-10-10 15:27:01.313+0000: 18210: error : qemuMigrationFinish:5533 : migration successfully aborted | 17:09 |
| *** erlon has joined #openstack-nova | 17:14 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/os-vif master: API _get_impl() for Linux should return PyRoute2() object https://review.openstack.org/609354 | 17:14 |
| *** k_mouza has joined #openstack-nova | 17:18 | |
| *** k_mouza has quit IRC | 17:23 | |
| *** orange_julius has joined #openstack-nova | 17:24 | |
| orange_julius | I've been looking into ways to use ARM images inside of an Openstack installation and was wondering if anybody had any experience with this. From what I've seen we basically have two options: Purchase an ARM server and set up as a compute node. Configure a server to use qemu instead of KVM and virtualize. Is it possible to tell a compute node to us | 17:26 |
| orange_julius | e both kvm and qemu depending on the image? Is there a better way to accomplish this? | 17:26 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: clean up ip_command interface https://review.openstack.org/609414 | 17:26 |
| openstackgerrit | sean mooney proposed openstack/os-vif master: add support for generic tap device plug https://review.openstack.org/602384 | 17:26 |
| *** k_mouza has joined #openstack-nova | 17:27 | |
| sean-k-mooney | jaypipes: done ^ let me know if there is anything else you would like me to change | 17:27 |
| *** psachin has quit IRC | 17:28 | |
| *** moshele has quit IRC | 17:29 | |
| *** k_mouza has quit IRC | 17:31 | |
| jaypipes | sean-k-mooney: +Wd | 17:32 |
| *** mvkr has quit IRC | 17:33 | |
| sean-k-mooney | jaypipes: thanks i should have created the bug when it was reported to me on irc instead of starting on the patch but i need to base my other patch on top of it anyway | 17:34 |
| *** dtantsur is now known as dtantsur|afk | 17:34 | |
| jaypipes | sean-k-mooney: no worries man | 17:34 |
| sean-k-mooney | ok so time for dinner ill be back online later | 17:36 |
| *** moshele has joined #openstack-nova | 17:39 | |
| *** adrianc has joined #openstack-nova | 17:39 | |
| *** ociuhandu has joined #openstack-nova | 17:39 | |
| *** k_mouza has joined #openstack-nova | 17:40 | |
| *** macza has quit IRC | 17:40 | |
| *** macza has joined #openstack-nova | 17:41 | |
| *** ociuhandu has quit IRC | 17:41 | |
| cfriesen | orange_julius: currently I think you'd need to make it a nova-compute config option. theoretically you could make it depend on the image properties, but I think that'd be a feature to be added. | 17:42 |
| orange_julius | cfriesen: Just so I understand, you are saying that we'd have to change the nova config on a hypervisor to use qemu instead of KVM. Basically dedicating that entire machine to QEMU workloads instead of KVM | 17:43 |
| mriedem | orange_julius: mnaser can probably help here | 17:44 |
| mriedem | i think he's running arm nodes somewhere | 17:45 |
| *** k_mouza has quit IRC | 17:45 | |
| *** tssurya has quit IRC | 17:46 | |
| *** adrianc has quit IRC | 17:47 | |
| *** adrianc has joined #openstack-nova | 17:47 | |
| cfriesen | orange_julius: the libvirt driver in nova looks at caps.host.cpu.arch which comes from libvirt. so you'd need to make that return an arm architecture I think. | 17:52 |
| *** pcaruana has quit IRC | 17:55 | |
| *** k_mouza has joined #openstack-nova | 18:06 | |
| *** k_mouza has quit IRC | 18:10 | |
| *** pcaruana has joined #openstack-nova | 18:12 | |
| *** itlinux has quit IRC | 18:13 | |
| *** k_mouza has joined #openstack-nova | 18:18 | |
| *** panda has quit IRC | 18:20 | |
| *** k_mouza has quit IRC | 18:20 | |
| *** priteau has quit IRC | 18:20 | |
| *** panda has joined #openstack-nova | 18:23 | |
| *** k_mouza has joined #openstack-nova | 18:24 | |
| *** moshele has quit IRC | 18:25 | |
| *** k_mouza has quit IRC | 18:27 | |
| *** mvkr has joined #openstack-nova | 18:28 | |
| larsks | Hey folks. There are comments in the nova history that suggest the quota_usage_refresh manage command was replaced by API functionality. What is that API? | 18:30 |
| *** k_mouza has joined #openstack-nova | 18:34 | |
| larsks | stephenfin: ...if you're around, they were your comments :) ^^^^ | 18:36 |
| *** k_mouza_ has joined #openstack-nova | 18:38 | |
| *** k_mouza has quit IRC | 18:39 | |
| melwitt | larsks: as of Pike, we don't track quota usages separately from resource counts, so there's no notion of refreshing quota. we only use resource counting | 18:39 |
| melwitt | so quota can't get out-of-sync | 18:40 |
| melwitt | are you asking about a version earlier than Pike? | 18:40 |
| *** k_mouza_ has quit IRC | 18:40 | |
| *** corvus has joined #openstack-nova | 18:45 | |
| larsks | melwitt: pike, actually, so that's good to know. I'm an indirect conduit for the issue right now, so I will need to wait until I get my hands on the environment in question I guess before I can better assess what's going on. Thanks! | 18:46 |
| melwitt | larsks: ok, feel free to ping me if you have more questions | 18:47 |
| *** moshele has joined #openstack-nova | 18:48 | |
| mriedem | dansmith: https://github.com/kk7ds/openstack-gerrit-dashboard/pull/29 | 18:51 |
| dansmith | mriedem: yep will look in a sec | 18:51 |
| dansmith | mriedem: was going to ask you this morning if you had fixed that yet :D | 18:51 |
| dansmith | omg it works! | 18:54 |
| *** macza has quit IRC | 18:55 | |
| *** macza has joined #openstack-nova | 18:55 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | 19:03 |
| *** pcaruana has quit IRC | 19:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix NoneType error in _notify_volume_usage_detach https://review.openstack.org/609518 | 19:08 |
| *** adrianc has quit IRC | 19:08 | |
| mriedem | dansmith: i needed something to do besides review code and/or specs | 19:13 |
| * dansmith nods | 19:13 | |
| *** spatel has joined #openstack-nova | 19:17 | |
| *** k_mouza has joined #openstack-nova | 19:23 | |
| openstackgerrit | Merged openstack/nova-specs master: Update blueprint name so spec matches launchpad https://review.openstack.org/607347 | 19:25 |
| *** k_mouza has quit IRC | 19:27 | |
| imacdonn | dansmith: I implemented what I understood from our discussion yesterday at https://review.openstack.org/608091 . It needs a little polish, but sean-k-mooney wants to use new the exit status if there are any exceptions, whether or not any migrations may still be pending. I don't think we can do that, because it can't be automated | 19:27 |
| *** k_mouza has joined #openstack-nova | 19:28 | |
| *** k_mouza has quit IRC | 19:32 | |
| *** dave-mccowan has quit IRC | 19:34 | |
| *** k_mouza has joined #openstack-nova | 19:40 | |
| mriedem | hey gang, two easy +Ws https://review.openstack.org/#/c/608802/ https://review.openstack.org/#/c/609467/ | 19:41 |
| *** k_mouza has quit IRC | 19:43 | |
| *** k_mouza_ has joined #openstack-nova | 19:44 | |
| *** moshele has quit IRC | 19:45 | |
| *** k_mouza has joined #openstack-nova | 19:47 | |
| melwitt | dansmith, mriedem: I linked my nova-consoleauth patch on L52 here that is ready for subteam review https://etherpad.openstack.org/p/stein-nova-subteam-tracking | 19:48 |
| *** macza has quit IRC | 19:48 | |
| *** macza has joined #openstack-nova | 19:48 | |
| *** k_mouza_ has quit IRC | 19:49 | |
| *** macza has quit IRC | 19:49 | |
| *** k_mouza has quit IRC | 19:50 | |
| artom | I thought we had functional live migration tests? | 19:50 |
| *** macza has joined #openstack-nova | 19:50 | |
| artom | Ah, nova/tests/functional/test_servers.py | 19:50 |
| artom | Ignore me | 19:50 |
| dansmith | imacdonn: I don't understand sean's concern or desire | 19:51 |
| imacdonn | dansmith: thanks for commenting. One thing is still a bit fuzzy .. what does "work was done" mean? The way "ran" is currently implemented, it only counts how many rows were migrated *in the last batch*, so of you use the default of 50 at a time, ran will always end up as 0 | 19:53 |
| imacdonn | ... because that's the only way it can break out of the loop | 19:55 |
| dansmith | imacdonn: ran is the sum of all the "done" values from any migration it4eration right? | 19:55 |
| *** pcaruana has joined #openstack-nova | 19:55 | |
| dansmith | ran becoming nonzero is how you break out | 19:55 |
| imacdonn | dansmith: no, only the last iteration | 19:55 |
| imacdonn | dansmith: it gets reset to 0 at https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L718 | 19:56 |
| dansmith | imacdonn: oh I see, but that's a bug I guess | 19:56 |
| dansmith | from when this went from a fixed number to having an --until-done | 19:56 |
| dansmith | or whatever | 19:56 |
| dansmith | or the opposite, but you know what I mean | 19:57 |
| sean-k-mooney | dansmith: we break out 1 of 2 way. ran becomes 0 or we pass --max-count in which case we do not loop as unlimited is false | 19:57 |
| dansmith | so yeah you have to fix that for this to work | 19:57 |
| imacdonn | dansmith: that's what I need to get nailed down ... I tried to fix that, in PS2, but made grenade blow up, becaused grenade needs the command to exit with status 0 | 19:57 |
| dansmith | imacdonn: well, if this is wrong it's possible grenade is wrong | 19:58 |
| jaypipes | mriedem: yesterday, which scheduler filter did you say already looked at instance metadata by querying the BuildRequest? | 19:58 |
| mriedem | jaypipes: the one you're writing | 19:58 |
| dansmith | oh wait, | 19:58 |
| dansmith | maybe I'm remembering this now | 19:58 |
| imacdonn | dansmith: it's possible, yes ... although it seems like if you run the command without --max-count, an outcome of exit status 0 (generally interpreted as success), is what would be expected | 19:58 |
| dansmith | in the unlimited case, you really need the exit code to be zero | 19:59 |
| dansmith | right | 19:59 |
| dansmith | oof, this should be commented in here for sure | 19:59 |
| dansmith | imacdonn: so maybe just keep the full count separate from ran and use that for the gate on exit 2 | 19:59 |
| imacdonn | so that's how I came up with the term "migrations may still be pending" .... when ran is not zero, which can only happen if you use --max-count | 19:59 |
| sean-k-mooney | dansmith: yes if you dont pass --max-count the exit 0 should mean all migration ran sucessfuly | 20:00 |
| dansmith | imacdonn: like I say, you don't know what is pending or not really | 20:00 |
| imacdonn | dansmith: that's why I used "may" :) | 20:00 |
| dansmith | imacdonn: don't. | 20:00 |
| sean-k-mooney | actully if you do pass --max-could exit 0 should still mean the same thing | 20:00 |
| dansmith | imacdonn: so if you keep total_ran and use that then you're good right? | 20:00 |
| dansmith | sean-k-mooney: no it shouldn't | 20:01 |
| dansmith | sean-k-mooney: because you ran --max-count=50 and you get zero, you expect you're done | 20:01 |
| dansmith | sean-k-mooney: if you don't pass --max-count, zero means all of them were completed, no errors so you're done | 20:01 |
| sean-k-mooney | if there are 50 migrations and i pass --max-count=100 it should return 0 if all 50 ran successfully | 20:01 |
| dansmith | sean-k-mooney: no it shouldn't | 20:01 |
| dansmith | sean-k-mooney: read the man page | 20:01 |
| imacdonn | dansmith: are you saying exit 2 if exceptions and total_ran>0, and exit 1 if ran (not total_ran) > 0 ? | 20:01 |
| sean-k-mooney | so if max is greater then total you dont want the same behavior as if there was no max | 20:02 |
| dansmith | imacdonn: I'm saying leave the 0 and 1 the way they are, and exit 2 if total_ran=0 and exceptions | 20:02 |
| dansmith | sean-k-mooney: it's about definition of done-ness | 20:02 |
| imacdonn | dansmith: that's what I had in PS2, and it broke grenade, because it exits with 1 if any migrations were done | 20:02 |
| imacdonn | dansmith: or maybe I didn't interpret your last statement right | 20:03 |
| sean-k-mooney | yes which is why im saying if i set a max larger then the amount to do and they all ran sucsessfully im done hence 0 | 20:03 |
| dansmith | imacdonn: I'm not talking about changing the definition of 1 | 20:03 |
| dansmith | imacdonn: right | 20:03 |
| dansmith | imacdonn: if you don't change the logic for 1 then grenade can't break, unless we're actually hiding exceptions | 20:03 |
| *** k_mouza has joined #openstack-nova | 20:03 | |
| *** spatel has quit IRC | 20:04 | |
| imacdonn | dansmith: OK, I think that might work ... I'll have to try it ... sean-k-mooney, do you still have concerns ? | 20:05 |
| sean-k-mooney | dansmith: we used to hide exceptions but my understand of 1 is there are still more migration to run i would prefer it to be "there are more migration to run and all the ones i just ran did not have error" | 20:05 |
| dansmith | sean-k-mooney: that's not what 1 means | 20:05 |
| dansmith | we can't really know if there are more unless we try and hit zero | 20:05 |
| dansmith | without doing more database stuff to count separately from doing | 20:05 |
| dansmith | which is not worth it, IMHO | 20:05 |
| dansmith | so, while (rc==1) { doit } | 20:06 |
| imacdonn | personally, I kinda wish there was a way to find out how many migrations are needed, before attempting any | 20:06 |
| imacdonn | I might want to do this when planning an upgrade | 20:06 |
| *** pcaruana has quit IRC | 20:06 | |
| sean-k-mooney | ok so 1 just used to mean there was a partial update so we need to loop that i think makes sense | 20:06 |
| sean-k-mooney | and 0 means we are done | 20:07 |
| dansmith | sean-k-mooney: no, 1 means we did things, that's ALL it means :) | 20:07 |
| sean-k-mooney | dansmith: what is the smantics of 2 in your case | 20:07 |
| dansmith | might be partial, might be full | 20:07 |
| mriedem | sorrison: if you wanted to policy something, i'd think having policy on the ability to create multiple servers in a single request would be a good one so you can avoid tenants killing your scheduler with large burst multi-create requests | 20:07 |
| mriedem | especially if those users have higher than normal quota | 20:07 |
| dansmith | sean-k-mooney: hopefully this can be the last time I say this, but 2 means we couldn't make any more progress, and exceptions were raised | 20:07 |
| mriedem | we probably had a policy on the multi-create API extension at some point | 20:07 |
| mriedem | make it 2.5.... | 20:08 |
| mriedem | and i'm sold | 20:08 |
| dansmith | sean-k-mooney: so you can distinguish between "can't make any more progress" and "can't make any more progress, but things seem unhappy" | 20:08 |
| sean-k-mooney | dansmith: ok but we dont stop trying migration on the first exception | 20:08 |
| *** k_mouza has quit IRC | 20:08 | |
| dansmith | sean-k-mooney: exactly | 20:08 |
| *** moshele has joined #openstack-nova | 20:09 | |
| sean-k-mooney | my piont it 2 would only ever be retruned if all the migrations in the first iteration failed | 20:09 |
| dansmith | no | 20:09 |
| mriedem | sorrison: maybe that's just never been a problem b/c of quota restrictions, the multi-create thing i mean | 20:09 |
| dansmith | sean-k-mooney: did you mean the last iteration? my answer is still no, but... | 20:10 |
| imacdonn | no, 2 gets returned if the only remaining possible migrations barfed for some reason | 20:10 |
| dansmith | sean-k-mooney: with the suggestion I just made to imacdonn, we keep a total_run count and use that for 2 instead of ran, which is just "the last iteration" | 20:10 |
| mriedem | melwitt: isn't this more than just the db backend for console auth? https://bugs.launchpad.net/nova/+bug/1795982 - it's been regressed since multi-cell support in pike | 20:10 |
| openstack | Launchpad bug 1795982 in OpenStack Compute (nova) "/os-console-auth-tokens/{console_token} API doesn't handle the database backend" [High,In progress] - Assigned to melanie witt (melwitt) | 20:10 |
| dansmith | imacdonn: right | 20:10 |
| dansmith | imacdonn: maybe just tweak and push that up and we can argue about it after there's something to see? | 20:10 |
| dansmith | I'm getting kinda frustrated with this overly minute detail and really want to get on to other stuff before I end my day | 20:11 |
| imacdonn | dansmith: roger. will do after lunch | 20:11 |
| dansmith | imacdonn: thanks | 20:11 |
| melwitt | mriedem: I don't think so because nova-consoleauth is global, you don't need to know anything about cells to query it | 20:11 |
| sean-k-mooney | dansmith: yes i was referring to "total_run==0 and exceptions" for retrun 2 | 20:11 |
| melwitt | mriedem: that is, it's storage of console token auths is global across all cells | 20:12 |
| efried | dansmith, imacdonn: I don't intend to get super involved in this, but have we considered allowing behavior change(s) (e.g. where exceptions cause failure) based on an env var or CLI switch? (Sorry, this thought has been bouncing around in my head for a week, had to get it out.) | 20:12 |
| dansmith | efried: I don't think we need a behavior change here | 20:12 |
| mriedem | hmm, ok https://docs.openstack.org/nova/queens/user/cellsv2-layout.html#consoleauth-service-and-console-proxies | 20:12 |
| efried | ...so that existing automations aren't affected, but new users can have the benefit of potentially improved UX | 20:12 |
| mriedem | got confused about what's global or not | 20:12 |
| dansmith | efried: if we did, then sure | 20:12 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Handle volume API failure in post_live_migration https://review.openstack.org/609517 | 20:13 |
| *** moshele has quit IRC | 20:13 | |
| sean-k-mooney | dansmith: imacdonn you know what im fine with what ever you implement and ill read teh code after to understand what the exit code actully mean. | 20:13 |
| efried | Okay. When I stopped looking, the patch was suggesting a behavior change that seemed fair and just to me, but was nacked for that reason. | 20:13 |
| imacdonn | efried: we've introduced a new exit status, so the logic for the existing ones doesn't have to change .. I think it'll only "break" when there are exceptions and no more possible migrations, and we *want* it to break in that case | 20:14 |
| dansmith | efried: the only thing that could make this more dreadful would be two behaviors in the same set of code :) | 20:15 |
| dansmith | imacdonn: ++ | 20:15 |
| efried | dansmith: Hm, we should introduce a microversioning system for that. | 20:15 |
| dansmith | efried: we should microversion your butt. | 20:15 |
| * efried almost choked on his steak | 20:16 | |
| dansmith | if we ever have nova-manage cellv2 thingy --cli-version=2.123 | 20:16 |
| dansmith | then just shoot me | 20:16 |
| melwitt | mriedem: yeah, so the /os-console-auth-tokens/{console_token} API calls nova-consoleauth over RPC, and nova-consoleauth was made cell-aware sometime in the past, so all was working fine with multi-cell (unless there's a bug we don't know about). but when we moved to the database backend, that's what made it so that the /os-console-auth-tokens API would need to be able to talk to cell databases directly instead of going through the | 20:16 |
| melwitt | nova-consoleauth service | 20:16 |
| *** erlon has quit IRC | 20:17 | |
| efried | what we really need is nested microversions | 20:17 |
| sean-k-mooney | imacdonn: retruaning any new code will still requrie autoation scirpt to handel the new error case or manual intervention so the fact there is a behavior change or not is slight less important that said in the sucess case the codes should not change | 20:17 |
| mriedem | god i bet my cross-cell resize stuff needs to recreate console auth tokens for the moved instance in the target cell db too... | 20:18 |
| mriedem | like bdms and tags | 20:18 |
| * dansmith gets out his irc ops to kickban people discussing nested microversions | 20:18 | |
| *** ralonsoh has quit IRC | 20:18 | |
| mriedem | and virtual_interfaces... | 20:18 |
| melwitt | you could probably just punt that though, let them have to get a fresh console token after a cross-cell move | 20:19 |
| imacdonn | sean-k-mooney: that's true, and that's why there's a release note for this, and it may not be backportable ... but the most common case will be running the command without --max-count and expecting a 0, or rerunning it until you don't get 1 | 20:19 |
| melwitt | default TTL for console token auth is 10 minutes so they aren't designed to live long. operators can configure longer TTL but I'm not sure they'd expect you to solve for that | 20:19 |
| imacdonn | sean-k-mooney: I guess the possible case where it could break automation is if the command is being rerun infinitely until it gets a zero, which would never happen if it's returning 2 every time | 20:20 |
| *** tbachman has quit IRC | 20:20 | |
| sean-k-mooney | imacdonn: if i was writingin this in ansiable and i called it without --max-count i would have interpereted a non 0 result as an error just fyi | 20:20 |
| imacdonn | sean-k-mooney: right, and we're not going to change that | 20:21 |
| imacdonn | sean-k-mooney: without --max-count, you'll either get 0 (it worked), or 2 (something unexpectedly broke, and you need to figure out why) | 20:21 |
| sean-k-mooney | imacdonn: if you gurarentee that in the code that is fine | 20:22 |
| imacdonn | sean-k-mooney: I believe I can .. I'll post it this afternoon, and we can nit-pick :) | 20:23 |
| *** spatel has joined #openstack-nova | 20:23 | |
| *** k_mouza has joined #openstack-nova | 20:24 | |
| sean-k-mooney | ill be offline by then but enjoy your lunch and ill take a look at it tomorow | 20:24 |
| imacdonn | k, thanks! | 20:24 |
| *** k_mouza has quit IRC | 20:28 | |
| *** erlon has joined #openstack-nova | 20:31 | |
| *** devananda has joined #openstack-nova | 20:33 | |
| sean-k-mooney | have people see a 404 failing to retriva allocationf form resource provierded before | 20:34 |
| sean-k-mooney | http://logs.openstack.org/84/602384/4/check/kuryr-kubernetes-tempest-daemon-octavia/33adb32/controller/logs/screen-n-cpu.txt.gz?#_Oct_10_19_02_09_953771 | 20:34 |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 20:36 |
| openstackgerrit | Jack Ding proposed openstack/nova-specs master: High Precision Event Timer (HPET) on x86 guests https://review.openstack.org/607989 | 20:36 |
| *** erlon has quit IRC | 20:37 | |
| *** k_mouza has joined #openstack-nova | 20:40 | |
| *** awaugama has quit IRC | 20:43 | |
| *** k_mouza has quit IRC | 20:43 | |
| *** tbachman has joined #openstack-nova | 20:44 | |
| *** itlinux has joined #openstack-nova | 20:46 | |
| *** erlon has joined #openstack-nova | 20:48 | |
| *** tbachman has quit IRC | 20:48 | |
| *** spatel has quit IRC | 20:48 | |
| *** spatel has joined #openstack-nova | 20:49 | |
| *** erlon has quit IRC | 20:53 | |
| *** liuyulong has quit IRC | 20:55 | |
| *** k_mouza has joined #openstack-nova | 20:55 | |
| mriedem | melwitt: ok +2 | 20:56 |
| melwitt | thanks | 20:57 |
| *** k_mouza_ has joined #openstack-nova | 20:58 | |
| *** k_mouza has quit IRC | 21:00 | |
| *** k_mouza_ has quit IRC | 21:03 | |
| *** spatel has quit IRC | 21:04 | |
| *** slaweq has quit IRC | 21:04 | |
| *** spatel has joined #openstack-nova | 21:06 | |
| *** k_mouza has joined #openstack-nova | 21:06 | |
| mriedem | sean-k-mooney: yes it's a known bug | 21:10 |
| mriedem | https://bugs.launchpad.net/nova/+bug/1789998 | 21:10 |
| openstack | Launchpad bug 1789998 in OpenStack Compute (nova) "ResourceProviderAllocationRetrievalFailed ERROR log message on fresh n-cpu startup" [Low,Triaged] | 21:10 |
| mriedem | happens on every start of a new compute | 21:10 |
| *** slaweq has joined #openstack-nova | 21:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't log an error if attachment_create fails https://review.openstack.org/609547 | 21:14 |
| *** slaweq has quit IRC | 21:16 | |
| sorrison | mriedem: Missing some context RE: "maybe that's just never been a problem b/c of quota restrictions, the multi-create thing i mean" | 21:23 |
| mriedem | meaning maybe no one has ever felt the need to restrict certain groups of users from being able to make multi-create requests | 21:23 |
| mriedem | because multi-create can be abused, e.g. https://review.openstack.org/#/c/607735/ | 21:24 |
| *** priteau has joined #openstack-nova | 21:24 | |
| mriedem | sorrison: maybe a better question is, what is the highest any of your tenants have for instance quota? | 21:24 |
| *** priteau has quit IRC | 21:26 | |
| sorrison | mriedem: 2048 is the highest just looking in our DB | 21:26 |
| *** macza has quit IRC | 21:26 | |
| mriedem | jesus | 21:26 |
| mriedem | have you ever tried to create 2048 servers in a single create request? | 21:27 |
| *** macza has joined #openstack-nova | 21:27 | |
| sorrison | haha don't be silly :-) | 21:27 |
| mriedem | because the API will let you do that | 21:27 |
| mriedem | there is no rate limiting on multi-create requests | 21:27 |
| mriedem | and that size of request will melt your scheduler | 21:27 |
| mriedem | related: https://review.openstack.org/#/c/510235/ | 21:28 |
| sorrison | is the multi create number set in the request spec? I can have a look in the db to see what our stats are like | 21:28 |
| mriedem | yes, it's the request spec "num_instances" field | 21:28 |
| *** eharney has quit IRC | 21:31 | |
| openstackgerrit | Merged openstack/os-vif master: clean up ip_command interface https://review.openstack.org/609414 | 21:35 |
| sorrison | mriedem: very tricky to get that info out of mysql due to json blob. We're not running a version of mysql that has json support sadly | 21:36 |
| mriedem | damn | 21:36 |
| melwitt | I never knew request spec was a json blob until now O.o | 21:40 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Don't log error in _remove_deleted_instances_allocations if compute is new https://review.openstack.org/609552 | 21:40 |
| sorrison | either way, we're aware of the issue but haven't had any major issues | 21:41 |
| mriedem | maybe i can get some data from some public cloud ops | 21:41 |
| sorrison | Just trying to get my sql foo on to see if I can extract the num_instances | 21:41 |
| *** munimeha1 has quit IRC | 21:44 | |
| sorrison | ok so max num_instances we've had is 49 | 21:51 |
| sorrison | mriedem: stats here http://paste.openstack.org/show/731867/ | 21:52 |
| melwitt | neat | 21:53 |
| mriedem | sorrison: nice, thanks | 21:56 |
| *** tbachman has joined #openstack-nova | 22:00 | |
| sorrison | mriedem: I sorted out https://review.openstack.org/#/c/608474/ still not sure about correct name for policy as it does affect list and show for a flavor | 22:01 |
| mriedem | that's not checked on show is it? | 22:01 |
| mriedem | or you mean, allow non-admins to show private flavors that they don't have access to? | 22:02 |
| mriedem | i.e. support person trying to triage a bug for a server created with a private flavor? | 22:02 |
| sorrison | Yes | 22:07 |
| sorrison | The change to allow policy for a flavor show is https://review.openstack.org/#/c/608474/3/nova/objects/flavor.py | 22:07 |
| sorrison | it's for reporting scripts | 22:08 |
| mriedem | yup took me a second to sort that out | 22:09 |
| sorrison | yeah it goes down a few layers from the api | 22:09 |
| *** burt has quit IRC | 22:11 | |
| *** slaweq has joined #openstack-nova | 22:11 | |
| mriedem | sorrison: ok comments inline; | 22:12 |
| mriedem | i left some suggestions about the rule name, but they aren't awesome | 22:12 |
| mriedem | maybe alex_xu or gmann or dansmith have ideas | 22:12 |
| sorrison | yeah I can't think of something good there | 22:15 |
| *** slaweq has quit IRC | 22:15 | |
| sorrison | with disabled flavors I think they are useful. We are planning on retiring some flavors soon and were planning on disabling them by updating the DB | 22:16 |
| sorrison | unless there is a better way to retire flavors? | 22:16 |
| *** mlavalle has quit IRC | 22:20 | |
| mriedem | there is no way to disable flavors via the API, which is why i'm sort of hesitant to mention them | 22:27 |
| mriedem | but it's not a big deal to leave that in if you're hacking flavors.disabled via the db directly | 22:27 |
| mriedem | it just sucks you have to do that... | 22:27 |
| mriedem | we have a PUT /flavors/{flavor_id} now...seems that would be a place to disable/enable flavors | 22:28 |
| melwitt | AFAIK, people retired flavors by deleting them. and I remember we had a bug back from eons ago where 'nova show' would fail on existing instances with the retired flavor because it was trying to pull a flavor that was deleted from the db | 22:28 |
| melwitt | that got fixed by flavors embedded on instances | 22:28 |
| mriedem | and we expose the flavor details embedded in the instance in the API now | 22:28 |
| sorrison | hmm yeah that makes sense and can prob just retire by deleting now | 22:34 |
| *** spatel has quit IRC | 22:36 | |
| *** erlon has joined #openstack-nova | 22:40 | |
| *** rcernin has joined #openstack-nova | 22:41 | |
| *** spatel has joined #openstack-nova | 22:42 | |
| *** spatel has quit IRC | 22:47 | |
| *** moshele has joined #openstack-nova | 22:56 | |
| mriedem | efried: see the thread on the ML about moving taskflow out of openstack governance? | 22:57 |
| mriedem | powervm might care about that | 22:57 |
| mriedem | oh heh i see you did :) | 22:58 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip _remove_deleted_instances_allocations if compute is new https://review.openstack.org/609552 | 22:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Skip _remove_deleted_instances_allocations if compute is new https://review.openstack.org/609552 | 23:00 |
| *** moshele has quit IRC | 23:00 | |
| *** macza has quit IRC | 23:03 | |
| *** slaweq has joined #openstack-nova | 23:11 | |
| *** aloga has quit IRC | 23:15 | |
| *** slaweq has quit IRC | 23:16 | |
| *** k_mouza has quit IRC | 23:17 | |
| *** mriedem has quit IRC | 23:18 | |
| *** aloga has joined #openstack-nova | 23:27 | |
| *** icey has quit IRC | 23:39 | |
| *** icey has joined #openstack-nova | 23:40 | |
| openstackgerrit | iain MacDonnell proposed openstack/nova master: Handle online_data_migrations exceptions https://review.openstack.org/608091 | 23:44 |
| *** mchlumsky has quit IRC | 23:46 | |
| *** takashin has joined #openstack-nova | 23:56 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!