| *** archit has quit IRC | 00:00 | |
| openstackgerrit | melissaml proposed openstack/nova-specs master: fix typos in documentation https://review.openstack.org/545528 | 00:18 |
|---|---|---|
| *** gyee has quit IRC | 00:21 | |
| *** masber has quit IRC | 00:22 | |
| openstackgerrit | Merged openstack/nova master: Add disabled field to CellMapping object https://review.openstack.org/550090 | 00:24 |
| openstackgerrit | Merged openstack/nova master: Add CellMappingList.get_by_disabled() query method https://review.openstack.org/550188 | 00:24 |
| *** weshay is now known as weshay_PTO | 00:25 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:28 | |
| *** hamzy has joined #openstack-nova | 00:30 | |
| *** danpawlik has joined #openstack-nova | 00:31 | |
| *** Dinesh_Bhor has quit IRC | 00:31 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:33 | |
| *** danpawlik has quit IRC | 00:36 | |
| *** yingjun has joined #openstack-nova | 00:36 | |
| *** yamahata has quit IRC | 00:38 | |
| *** odyssey4me has quit IRC | 00:38 | |
| *** odyssey4me has joined #openstack-nova | 00:39 | |
| *** yingjun has quit IRC | 00:40 | |
| *** _ix has joined #openstack-nova | 00:42 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Parse placement forbidden traits query string https://review.openstack.org/554665 | 00:42 |
| *** yingjun has joined #openstack-nova | 00:45 | |
| *** edmondsw has quit IRC | 00:46 | |
| *** hshiina has joined #openstack-nova | 00:46 | |
| *** _ix has quit IRC | 00:47 | |
| *** idlemind has quit IRC | 00:47 | |
| *** jichen has joined #openstack-nova | 00:48 | |
| *** oomichi has quit IRC | 00:50 | |
| *** sdeath has quit IRC | 00:57 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:57 | |
| *** yingjun has quit IRC | 01:00 | |
| *** gjayavelu has quit IRC | 01:01 | |
| *** esberglu has joined #openstack-nova | 01:03 | |
| *** wolverineav has quit IRC | 01:04 | |
| *** tiendc has joined #openstack-nova | 01:06 | |
| openstackgerrit | Merged openstack/nova stable/queens: Make archive_deleted_rows handle a missing CONF.api_database.connection https://review.openstack.org/553742 | 01:07 |
| *** esberglu has quit IRC | 01:08 | |
| *** danpawlik has joined #openstack-nova | 01:08 | |
| *** suresh12 has joined #openstack-nova | 01:09 | |
| *** mriedem has quit IRC | 01:10 | |
| *** masber has joined #openstack-nova | 01:12 | |
| *** harlowja_ has quit IRC | 01:13 | |
| *** yamahata has joined #openstack-nova | 01:13 | |
| *** danpawlik has quit IRC | 01:14 | |
| *** hongbin has joined #openstack-nova | 01:15 | |
| *** zhaochao has joined #openstack-nova | 01:17 | |
| *** licanwei has joined #openstack-nova | 01:23 | |
| *** Tom-Tom has quit IRC | 01:23 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/555576 | 01:25 |
| *** bhagyashris has quit IRC | 01:26 | |
| *** germs has left #openstack-nova | 01:26 | |
| *** bhagyashris has joined #openstack-nova | 01:27 | |
| *** Zames has joined #openstack-nova | 01:27 | |
| *** licanwei_ has joined #openstack-nova | 01:28 | |
| *** licanwei has quit IRC | 01:30 | |
| *** sapd has quit IRC | 01:34 | |
| *** Zames has quit IRC | 01:35 | |
| *** wolverineav has joined #openstack-nova | 01:36 | |
| *** elmaciej has quit IRC | 01:41 | |
| *** Tom-Tom has joined #openstack-nova | 01:41 | |
| *** Tom-Tom has quit IRC | 01:42 | |
| *** Tom-Tom has joined #openstack-nova | 01:42 | |
| *** danpawlik has joined #openstack-nova | 01:47 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/555590 | 01:50 |
| *** danpawlik has quit IRC | 01:51 | |
| *** zhurong has joined #openstack-nova | 01:52 | |
| *** sapd has joined #openstack-nova | 02:01 | |
| *** sdague has quit IRC | 02:04 | |
| openstackgerrit | Merged openstack/nova stable/ocata: Don't try to delete build request during a reschedule https://review.openstack.org/529703 | 02:11 |
| *** Dinesh_Bhor has quit IRC | 02:12 | |
| openstackgerrit | Patricia Domingues proposed openstack/nova master: load up the volume drivers by checking architecture https://review.openstack.org/541393 | 02:21 |
| *** Dinesh_Bhor has joined #openstack-nova | 02:22 | |
| *** danpawlik has joined #openstack-nova | 02:26 | |
| *** germs has joined #openstack-nova | 02:28 | |
| *** danpawlik has quit IRC | 02:31 | |
| *** edmondsw has joined #openstack-nova | 02:34 | |
| *** edmondsw has quit IRC | 02:39 | |
| *** masber has quit IRC | 02:41 | |
| *** salv-orl_ has joined #openstack-nova | 02:47 | |
| *** andreas_s has joined #openstack-nova | 02:48 | |
| *** salv-orlando has quit IRC | 02:50 | |
| *** psachin has joined #openstack-nova | 02:51 | |
| *** andreas_s has quit IRC | 02:53 | |
| *** wolverineav has quit IRC | 02:54 | |
| *** wolverineav has joined #openstack-nova | 02:55 | |
| *** wolverineav has quit IRC | 02:59 | |
| *** danpawlik has joined #openstack-nova | 03:00 | |
| *** danpawlik has quit IRC | 03:05 | |
| *** tuanla____ has joined #openstack-nova | 03:11 | |
| *** yingjun has joined #openstack-nova | 03:15 | |
| *** AlexeyAbashkin has joined #openstack-nova | 03:15 | |
| *** masber has joined #openstack-nova | 03:19 | |
| *** AlexeyAbashkin has quit IRC | 03:20 | |
| *** fragatin_ has joined #openstack-nova | 03:25 | |
| *** yingjun has quit IRC | 03:25 | |
| *** fragatin_ has quit IRC | 03:27 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove a comment about ec2 https://review.openstack.org/555615 | 03:28 |
| *** fragatin_ has joined #openstack-nova | 03:28 | |
| *** fragatina has quit IRC | 03:29 | |
| openstackgerrit | Merged openstack/nova master: Migrate tempest-dsvm-cells job to an in-tree job definition https://review.openstack.org/549780 | 03:29 |
| *** fragatin_ has quit IRC | 03:33 | |
| *** danpawlik has joined #openstack-nova | 03:36 | |
| *** hshiina has quit IRC | 03:41 | |
| *** danpawlik has quit IRC | 03:41 | |
| *** hshiina has joined #openstack-nova | 03:42 | |
| openstackgerrit | licanwei proposed openstack/nova master: Make nova-manage capable of syncing all cell databases https://review.openstack.org/519275 | 03:42 |
| *** fragatina has joined #openstack-nova | 03:45 | |
| *** vladikr has quit IRC | 03:46 | |
| *** vladikr has joined #openstack-nova | 03:46 | |
| *** liusheng has quit IRC | 03:47 | |
| *** liusheng has joined #openstack-nova | 03:47 | |
| *** hongbin has quit IRC | 03:47 | |
| *** hshiina has quit IRC | 03:48 | |
| *** fragatina has quit IRC | 03:49 | |
| *** fragatina has joined #openstack-nova | 03:54 | |
| *** itlinux has quit IRC | 03:59 | |
| *** hiro-kobayashi has quit IRC | 04:00 | |
| *** germs has quit IRC | 04:01 | |
| *** germs has joined #openstack-nova | 04:02 | |
| *** zhurong has quit IRC | 04:04 | |
| *** hshiina has joined #openstack-nova | 04:05 | |
| *** danpawlik has joined #openstack-nova | 04:11 | |
| *** hiro-kobayashi has joined #openstack-nova | 04:15 | |
| *** inara has quit IRC | 04:16 | |
| *** danpawlik has quit IRC | 04:17 | |
| *** harlowja has joined #openstack-nova | 04:19 | |
| *** inara has joined #openstack-nova | 04:19 | |
| *** fragatina has quit IRC | 04:19 | |
| *** fragatina has joined #openstack-nova | 04:20 | |
| *** weshay_PTO is now known as weshay | 04:21 | |
| *** edmondsw has joined #openstack-nova | 04:23 | |
| *** links has joined #openstack-nova | 04:26 | |
| *** edmondsw has quit IRC | 04:27 | |
| *** moshele has joined #openstack-nova | 04:29 | |
| *** gus has quit IRC | 04:36 | |
| *** gus has joined #openstack-nova | 04:37 | |
| *** abhishekk has joined #openstack-nova | 04:40 | |
| *** udesale has joined #openstack-nova | 04:41 | |
| *** neha_alhat_ has joined #openstack-nova | 04:44 | |
| neha_alhat_ | Hi All, I just want to know which ubuntu version supports libvirt version >= 3.10? | 04:44 |
| *** danpawlik has joined #openstack-nova | 04:45 | |
| openstackgerrit | Merged openstack/nova master: Make nova-cells-v1 run with neutron https://review.openstack.org/549789 | 04:47 |
| *** danpawlik has quit IRC | 04:50 | |
| *** sapd_ has joined #openstack-nova | 04:51 | |
| *** sapd has quit IRC | 04:51 | |
| *** ratailor has joined #openstack-nova | 04:59 | |
| *** lajoskatona has joined #openstack-nova | 04:59 | |
| *** Dinesh_Bhor has quit IRC | 05:02 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:05 | |
| *** harlowja has quit IRC | 05:12 | |
| *** lpetrut has joined #openstack-nova | 05:12 | |
| *** imacdonn has quit IRC | 05:15 | |
| *** AlexeyAbashkin has joined #openstack-nova | 05:15 | |
| *** imacdonn has joined #openstack-nova | 05:15 | |
| *** danpawlik has joined #openstack-nova | 05:16 | |
| *** AlexeyAbashkin has quit IRC | 05:19 | |
| *** moshele has quit IRC | 05:21 | |
| *** danpawlik has quit IRC | 05:22 | |
| *** gyankum has joined #openstack-nova | 05:23 | |
| *** neha_alhat_ has quit IRC | 05:28 | |
| *** mdnadeem has joined #openstack-nova | 05:34 | |
| *** yingjun has joined #openstack-nova | 05:34 | |
| *** yingjun has quit IRC | 05:34 | |
| *** yamahata has quit IRC | 05:38 | |
| *** hiro-kobayashi has quit IRC | 05:39 | |
| *** vks1 has joined #openstack-nova | 05:41 | |
| *** elmaciej has joined #openstack-nova | 05:43 | |
| *** suresh12 has quit IRC | 05:47 | |
| *** gjayavelu has joined #openstack-nova | 05:48 | |
| *** yingjun has joined #openstack-nova | 05:48 | |
| *** danpawlik has joined #openstack-nova | 05:50 | |
| *** hiro-kobayashi has joined #openstack-nova | 05:54 | |
| *** danpawlik has quit IRC | 05:56 | |
| *** suresh12 has joined #openstack-nova | 05:56 | |
| *** Tom-Tom_ has joined #openstack-nova | 05:57 | |
| *** suresh12 has quit IRC | 06:01 | |
| *** Tom-Tom has quit IRC | 06:01 | |
| *** Tom-Tom has joined #openstack-nova | 06:02 | |
| *** Tom-Tom_ has quit IRC | 06:02 | |
| *** yingjun has quit IRC | 06:02 | |
| *** neha_alhat_ has joined #openstack-nova | 06:08 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 06:10 |
| *** sidx64 has joined #openstack-nova | 06:10 | |
| *** sidx64 has quit IRC | 06:10 | |
| *** gjayavelu has quit IRC | 06:14 | |
| *** AlexeyAbashkin has joined #openstack-nova | 06:17 | |
| *** Dinesh_Bhor has quit IRC | 06:18 | |
| *** Dinesh_Bhor has joined #openstack-nova | 06:19 | |
| *** zhurong has joined #openstack-nova | 06:21 | |
| *** lpetrut has quit IRC | 06:22 | |
| *** AlexeyAbashkin has quit IRC | 06:22 | |
| *** sidx64 has joined #openstack-nova | 06:22 | |
| *** fragatina has quit IRC | 06:23 | |
| *** elmaciej has quit IRC | 06:23 | |
| *** neha_alhat_ has quit IRC | 06:24 | |
| *** danpawlik has joined #openstack-nova | 06:29 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host field to InstanceActionEvent https://review.openstack.org/555146 | 06:30 |
| *** danpawlik has quit IRC | 06:34 | |
| *** lpetrut has joined #openstack-nova | 06:36 | |
| *** sidx64 has quit IRC | 06:40 | |
| *** vivsoni_ has quit IRC | 06:40 | |
| *** vivsoni_ has joined #openstack-nova | 06:41 | |
| *** vivsoni_ has quit IRC | 06:43 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 06:43 |
| *** vivsoni_ has joined #openstack-nova | 06:43 | |
| *** fragatina has joined #openstack-nova | 06:48 | |
| *** Tom-Tom_ has joined #openstack-nova | 06:49 | |
| *** alexchadin has joined #openstack-nova | 06:50 | |
| *** Tom-Tom__ has joined #openstack-nova | 06:50 | |
| *** Tom-Tom has quit IRC | 06:50 | |
| *** sahid has joined #openstack-nova | 06:53 | |
| *** Tom-Tom_ has quit IRC | 06:54 | |
| *** sar has quit IRC | 06:57 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Extract generate_hostid method into utils.py https://review.openstack.org/555282 | 06:59 |
| *** maciejjozefczyk has quit IRC | 07:01 | |
| *** jichen has quit IRC | 07:03 | |
| *** jichen_ has joined #openstack-nova | 07:03 | |
| *** jichen_ is now known as jichen | 07:03 | |
| *** Tom-Tom has joined #openstack-nova | 07:04 | |
| *** Tom-Tom__ has quit IRC | 07:07 | |
| *** chyka has joined #openstack-nova | 07:12 | |
| *** sidx64 has joined #openstack-nova | 07:13 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host field to InstanceActionEvent https://review.openstack.org/555146 | 07:13 |
| *** chyka has quit IRC | 07:17 | |
| *** josecastroleon has joined #openstack-nova | 07:17 | |
| *** pcaruana has joined #openstack-nova | 07:21 | |
| *** sar has joined #openstack-nova | 07:23 | |
| *** yingjun has joined #openstack-nova | 07:25 | |
| *** afaranha has joined #openstack-nova | 07:27 | |
| *** sidx64 has quit IRC | 07:28 | |
| *** Dinesh_Bhor has quit IRC | 07:30 | |
| *** zhurong has quit IRC | 07:30 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:30 | |
| *** sidx64 has joined #openstack-nova | 07:31 | |
| *** sidx64 has quit IRC | 07:32 | |
| *** sidx64 has joined #openstack-nova | 07:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/555576 | 07:40 |
| *** sidx64 has quit IRC | 07:42 | |
| *** avolkov has joined #openstack-nova | 07:43 | |
| *** sidx64 has joined #openstack-nova | 07:44 | |
| *** Dinesh_Bhor has quit IRC | 07:44 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:48 | |
| *** damien_r has joined #openstack-nova | 07:54 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:55 | |
| *** lajoskatona has quit IRC | 07:56 | |
| *** lpetrut has quit IRC | 07:56 | |
| *** danpawlik has joined #openstack-nova | 07:58 | |
| *** damien_r has quit IRC | 07:59 | |
| *** edmondsw has joined #openstack-nova | 07:59 | |
| *** danpawlik has quit IRC | 08:01 | |
| *** danpawlik has joined #openstack-nova | 08:01 | |
| *** ccamacho has joined #openstack-nova | 08:02 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:02 | |
| *** edmondsw has quit IRC | 08:03 | |
| *** lajoskatona has joined #openstack-nova | 08:11 | |
| *** tesseract has joined #openstack-nova | 08:20 | |
| *** ralonsoh has joined #openstack-nova | 08:34 | |
| *** hiro-kobayashi has quit IRC | 08:35 | |
| *** amoralej|off is now known as amoralej | 08:36 | |
| *** sidx64 has quit IRC | 08:36 | |
| *** andreas_s has joined #openstack-nova | 08:37 | |
| *** damien_r has joined #openstack-nova | 08:40 | |
| *** Tom-Tom has quit IRC | 08:41 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration https://review.openstack.org/500677 | 08:41 |
| *** Tom-Tom has joined #openstack-nova | 08:41 | |
| *** Tom-Tom has quit IRC | 08:43 | |
| *** Tom-Tom has joined #openstack-nova | 08:43 | |
| *** Tom-Tom_ has joined #openstack-nova | 08:44 | |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 08:45 |
| *** damien_r has quit IRC | 08:45 | |
| *** damien_r has joined #openstack-nova | 08:46 | |
| *** Tom-Tom has quit IRC | 08:47 | |
| *** jpena|off is now known as jpena | 08:49 | |
| *** sidx64 has joined #openstack-nova | 08:52 | |
| *** shunliz has joined #openstack-nova | 08:54 | |
| *** hoangcx has quit IRC | 08:56 | |
| *** andreas_s has quit IRC | 08:56 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | 08:56 |
| *** andreas_s_ has joined #openstack-nova | 08:58 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 08:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 08:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 08:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 08:59 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 08:59 |
| *** andreas__ has joined #openstack-nova | 08:59 | |
| bauzas | good morning folks | 08:59 |
| *** bauzas is now known as bauwser | 08:59 | |
| *** andrea___ has joined #openstack-nova | 09:01 | |
| *** bauwser is now known as bauzas | 09:01 | |
| *** bauzas is now known as bauwser | 09:02 | |
| *** andreas_s_ has quit IRC | 09:02 | |
| *** andrea___ has quit IRC | 09:03 | |
| *** andreas_s has joined #openstack-nova | 09:04 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | 09:04 |
| *** andreas__ has quit IRC | 09:04 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Trivial fix: add a log eariler https://review.openstack.org/555164 | 09:10 |
| *** andreas_s has quit IRC | 09:11 | |
| *** mdbooth has joined #openstack-nova | 09:12 | |
| *** lucas-afk is now known as lucasagomes | 09:13 | |
| *** alexchadin has quit IRC | 09:14 | |
| *** alexchadin has joined #openstack-nova | 09:18 | |
| *** alexchadin has quit IRC | 09:18 | |
| *** alexchadin has joined #openstack-nova | 09:18 | |
| *** alexchadin has quit IRC | 09:19 | |
| *** andreas_s has joined #openstack-nova | 09:19 | |
| *** alexchadin has joined #openstack-nova | 09:19 | |
| *** andreas_s has quit IRC | 09:19 | |
| *** alexchadin has quit IRC | 09:20 | |
| *** jichen has quit IRC | 09:20 | |
| *** alexchadin has joined #openstack-nova | 09:20 | |
| *** alexchadin has quit IRC | 09:20 | |
| *** rcernin has quit IRC | 09:20 | |
| *** alexchadin has joined #openstack-nova | 09:21 | |
| *** jichen has joined #openstack-nova | 09:21 | |
| *** jianghuaw has quit IRC | 09:22 | |
| *** andreas_s has joined #openstack-nova | 09:22 | |
| gibi | happy Friday nova | 09:23 |
| *** mgoddard has joined #openstack-nova | 09:23 | |
| *** gibi is now known as giblet | 09:23 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add the version description for InstanceActionEventList https://review.openstack.org/555711 | 09:25 |
| *** alexchadin has quit IRC | 09:25 | |
| *** ratailor has quit IRC | 09:25 | |
| *** lajoskatona has quit IRC | 09:26 | |
| *** lpetrut has joined #openstack-nova | 09:27 | |
| *** ratailor has joined #openstack-nova | 09:27 | |
| *** lpetrut_ has joined #openstack-nova | 09:28 | |
| *** lpetrut has quit IRC | 09:28 | |
| *** sdague has joined #openstack-nova | 09:34 | |
| *** derekh has joined #openstack-nova | 09:37 | |
| *** Tom-Tom_ has quit IRC | 09:40 | |
| *** Tom-Tom has joined #openstack-nova | 09:41 | |
| *** yingjun has quit IRC | 09:41 | |
| *** Dinesh_Bhor has quit IRC | 09:43 | |
| *** sidx64 has quit IRC | 09:43 | |
| *** hshiina has quit IRC | 09:43 | |
| *** lajoskatona has joined #openstack-nova | 09:44 | |
| *** licanwei_ has quit IRC | 09:45 | |
| *** Tom-Tom has quit IRC | 09:45 | |
| *** edmondsw has joined #openstack-nova | 09:47 | |
| *** liuzz has quit IRC | 09:47 | |
| * stephenfin senses a disturbance in the 'hw:cpu_thread_policy' force | 09:48 | |
| *** tssurya has joined #openstack-nova | 09:49 | |
| Spazmotic | Happy friday you ridiculously smart people :p | 09:50 |
| *** edmondsw has quit IRC | 09:51 | |
| *** lgsafety has joined #openstack-nova | 09:52 | |
| *** ratailor has quit IRC | 09:52 | |
| *** alexchadin has joined #openstack-nova | 09:52 | |
| *** sidx64 has joined #openstack-nova | 09:53 | |
| *** ratailor has joined #openstack-nova | 09:53 | |
| *** kalyan has joined #openstack-nova | 09:54 | |
| *** andreas_s has quit IRC | 09:57 | |
| *** alexchadin has quit IRC | 09:57 | |
| *** andreas_s_ has joined #openstack-nova | 10:00 | |
| *** tuanla____ has quit IRC | 10:01 | |
| *** lgsafety has quit IRC | 10:01 | |
| *** cdent has joined #openstack-nova | 10:03 | |
| kashyap | dansmith: Is adjusting even one blank space change (in the same area) a strict no-no? I was cautious while making that: https://review.openstack.org/#/c/534384/10/nova/virt/libvirt/driver.py | 10:04 |
| *** yamamoto has quit IRC | 10:05 | |
| bauwser | stephenfin: that cannot be true, my padawan | 10:06 |
| *** yamamoto has joined #openstack-nova | 10:06 | |
| bauwser | fear is the path to the dark side | 10:07 |
| *** rcernin has joined #openstack-nova | 10:08 | |
| * bauwser starts looking at implementing nested RPs, woot | 10:08 | |
| *** andreas_s_ has quit IRC | 10:09 | |
| *** andreas_s has joined #openstack-nova | 10:09 | |
| *** yamamoto has quit IRC | 10:11 | |
| *** avolkov has quit IRC | 10:11 | |
| *** avolkov has joined #openstack-nova | 10:11 | |
| *** andreas_s has quit IRC | 10:14 | |
| bauwser | jianghuaw_: around ? | 10:14 |
| bauwser | jianghuaw_: I'm starting to modeling inventories with multiple types | 10:14 |
| bauwser | thanks to NRP | 10:14 |
| bauwser | and I wonder how you modeled the tree | 10:14 |
| bauwser | because I was thinking of one child RP being the PCI device | 10:15 |
| bauwser | but then, depending on the GPU type, you could have different inventories | 10:15 |
| bauwser | so it would be yet another tree level, with each type being a child ? | 10:15 |
| *** Tom-Tom has joined #openstack-nova | 10:16 | |
| *** chyka has joined #openstack-nova | 10:16 | |
| bauwser | jianghuaw_: nothing in the spec mentions what if we have multiple types per pGPU https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/add-support-for-vgpu.html | 10:18 |
| kashyap | dansmith: melwitt: And yes, restricting the backport to one option with the _same_ config attribute sounds list a reasonable compromise | 10:18 |
| kashyap | (As to the rest of the questions, answering on the review.) | 10:18 |
| *** andreas_s has joined #openstack-nova | 10:18 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 10:18 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 10:18 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 10:18 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 10:18 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 10:18 |
| bauwser | jianghuaw_: that said, nvidia says that it can only support one type per pGPU in http://docs.nvidia.com/grid/latest/grid-vgpu-user-guide/index.html#homogeneous-grid-vgpus | 10:19 |
| bauwser | so I wonder if that really matteres | 10:19 |
| *** stephenfin is now known as finucannot | 10:19 | |
| *** alexchadin has joined #openstack-nova | 10:20 | |
| *** liverpooler has quit IRC | 10:21 | |
| *** chyka has quit IRC | 10:21 | |
| *** Tom-Tom has quit IRC | 10:21 | |
| *** andreas_s has quit IRC | 10:23 | |
| *** alexchadin has quit IRC | 10:25 | |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 10:27 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 10:27 |
| openstackgerrit | Huang Rui proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 10:27 |
| *** shunliz has quit IRC | 10:28 | |
| *** alexchadin has joined #openstack-nova | 10:28 | |
| *** andreas_s has joined #openstack-nova | 10:28 | |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 10:31 |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 10:31 |
| *** andreas_s has quit IRC | 10:32 | |
| *** sambetts|afk is now known as sambetts | 10:32 | |
| *** andreas_s has joined #openstack-nova | 10:34 | |
| *** andreas_s has quit IRC | 10:34 | |
| kashyap | sean-k-mooney: Thanks for the comment here: https://review.openstack.org/#/c/534384/10/nova/virt/libvirt/driver.py | 10:35 |
| openstackgerrit | Merged openstack/nova master: Not use thread alloc policy for emulator thread https://review.openstack.org/538700 | 10:37 |
| *** andreas_s has joined #openstack-nova | 10:37 | |
| *** alexchadin has quit IRC | 10:37 | |
| *** andreas_s_ has joined #openstack-nova | 10:39 | |
| *** lajoskatona has quit IRC | 10:40 | |
| *** dtantsur|afk is now known as dtantsur | 10:41 | |
| *** mdbooth has quit IRC | 10:41 | |
| *** andreas_s has quit IRC | 10:42 | |
| *** andreas_s has joined #openstack-nova | 10:42 | |
| *** andreas_s_ has quit IRC | 10:43 | |
| *** alexchadin has joined #openstack-nova | 10:44 | |
| *** jichen has quit IRC | 10:47 | |
| *** cdent has quit IRC | 10:48 | |
| *** alexchadin has quit IRC | 10:49 | |
| kashyap | dansmith: melwitt: While I understand the backport stance. FWIW, talking to danpb (who is SME on this topic) on IRC, he still thinks it's an artificial limiting for the sake of backports. | 10:51 |
| kashyap | I feel that we're over-evaluating. But whatever. | 10:52 |
| *** lajoskatona has joined #openstack-nova | 10:55 | |
| *** alexchadin has joined #openstack-nova | 10:55 | |
| *** alexchadin has quit IRC | 10:56 | |
| * kashyap goes to rework the thing | 10:56 | |
| *** tbachman has quit IRC | 10:58 | |
| *** vladikr has quit IRC | 10:59 | |
| *** alexchadin has joined #openstack-nova | 11:01 | |
| *** abhishekk has quit IRC | 11:03 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | 11:04 |
| *** elmaciej has joined #openstack-nova | 11:04 | |
| *** annp has quit IRC | 11:06 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | 11:06 |
| *** udesale has quit IRC | 11:06 | |
| *** yamamoto has joined #openstack-nova | 11:07 | |
| *** mdbooth has joined #openstack-nova | 11:08 | |
| *** alexchadin has quit IRC | 11:10 | |
| *** josecastroleon has quit IRC | 11:12 | |
| *** alexchadin has joined #openstack-nova | 11:12 | |
| *** yamamoto has quit IRC | 11:13 | |
| *** moshele has joined #openstack-nova | 11:21 | |
| *** moshele has quit IRC | 11:26 | |
| *** pcaruana has quit IRC | 11:27 | |
| *** elmaciej has quit IRC | 11:27 | |
| *** vks1 has quit IRC | 11:38 | |
| *** cdent has joined #openstack-nova | 11:43 | |
| sahid | jaypipes: i think we are good with the trusted vfs spec: can you have a final look? https://review.openstack.org/#/c/485522/ | 11:47 |
| *** maciejjozefczyk has quit IRC | 11:50 | |
| *** maciejjozefczyk has joined #openstack-nova | 11:50 | |
| *** jaosorior has quit IRC | 11:56 | |
| *** yamamoto has joined #openstack-nova | 11:56 | |
| *** yamamoto has quit IRC | 11:56 | |
| *** sree_ has joined #openstack-nova | 11:59 | |
| *** sree_ is now known as Guest19457 | 11:59 | |
| *** edmondsw has joined #openstack-nova | 12:03 | |
| *** liverpooler has joined #openstack-nova | 12:07 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Use microversion parse 0.2.1 https://review.openstack.org/550265 | 12:09 |
| *** ccamacho is now known as ccamacho|lunch | 12:10 | |
| *** yamamoto has joined #openstack-nova | 12:10 | |
| *** tbachman has joined #openstack-nova | 12:12 | |
| *** odyssey4me has quit IRC | 12:12 | |
| *** odyssey4me has joined #openstack-nova | 12:12 | |
| *** cdent has quit IRC | 12:13 | |
| *** sahid has quit IRC | 12:13 | |
| *** tbachman_ has joined #openstack-nova | 12:14 | |
| *** vladikr has joined #openstack-nova | 12:16 | |
| *** tbachman has quit IRC | 12:17 | |
| *** tbachman_ is now known as tbachman | 12:17 | |
| *** josecastroleon has joined #openstack-nova | 12:19 | |
| *** yamamoto has quit IRC | 12:25 | |
| *** efried is now known as fried_rice | 12:27 | |
| * fried_rice waves | 12:28 | |
| *** tovin07 has quit IRC | 12:29 | |
| fried_rice | Gate fixed? Time to recheck-all? | 12:29 |
| *** alexchadin has quit IRC | 12:29 | |
| *** pcaruana has joined #openstack-nova | 12:30 | |
| *** rcernin has quit IRC | 12:30 | |
| *** voelzmo has joined #openstack-nova | 12:31 | |
| *** alexchadin has joined #openstack-nova | 12:33 | |
| *** cdent has joined #openstack-nova | 12:37 | |
| *** lajoskatona has quit IRC | 12:38 | |
| *** tiendc has quit IRC | 12:40 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Stop using mox in virt/xenapi/image/test_vdi_through_dev.py https://review.openstack.org/342682 | 12:41 |
| fried_rice | eric-young: I don't see the os-brick release proposed yet. Need help? | 12:41 |
| *** pchavva has joined #openstack-nova | 12:42 | |
| *** lucasagomes is now known as lucas-hungry | 12:43 | |
| *** edleafe is now known as figleaf | 12:46 | |
| eric-young | fried_rice, there was some talk yesterday in the cinder channel about needing one more patch merged first. was going to check on status here in a bit | 12:47 |
| *** voelzmo has quit IRC | 12:47 | |
| fried_rice | eric-young: Roger that, just checking in :) | 12:47 |
| *** jaypipes is now known as leakypipes | 12:47 | |
| fried_rice | (I'm in no hurry, just saw the nova patch go by again this morning) | 12:48 |
| eric-young | fried_rice, I am in no super hurry either but it'd be good to get that patch off my plate :) | 12:49 |
| eric-young | also need to get zuul to approve it. seeing an error with tempest-full-py3 on POST_FAILURE | 12:50 |
| *** lajoskatona has joined #openstack-nova | 12:55 | |
| *** tssurya is now known as sususuryashines | 12:56 | |
| *** suresh12 has joined #openstack-nova | 12:58 | |
| leakypipes | sususuryashines: welcome to Friday :) | 13:01 |
| * leakypipes considering changing his nick to POST_FAILURE | 13:01 | |
| kashyap | leakypipes: You've got one hell of a plumbing expenses there :P | 13:02 |
| kashyap | s/expenses/bill/ | 13:02 |
| *** suresh12 has quit IRC | 13:02 | |
| sususuryashines | leakypipes : hehe, thanks! | 13:03 |
| leakypipes | kashyap: indeed. | 13:03 |
| *** mriedem has joined #openstack-nova | 13:03 | |
| *** hemna_ has quit IRC | 13:03 | |
| fried_rice | eric-young: That one should be recheckable now. | 13:07 |
| *** yamamoto has joined #openstack-nova | 13:08 | |
| *** yamamoto has quit IRC | 13:08 | |
| *** yamamoto has joined #openstack-nova | 13:08 | |
| *** yamamoto has quit IRC | 13:08 | |
| *** rubasov has joined #openstack-nova | 13:12 | |
| eric-young | thanks, will do | 13:12 |
| *** yamamoto has joined #openstack-nova | 13:15 | |
| *** alexchadin has quit IRC | 13:15 | |
| *** lyan has joined #openstack-nova | 13:16 | |
| *** lyan is now known as Guest43990 | 13:16 | |
| *** alexchadin has joined #openstack-nova | 13:17 | |
| sq4ind | hey guys, any way to recover VM which has been lost during live migration ? I cannot find it on any hypervisor | 13:19 |
| sq4ind | tried this already: https://docs.openstack.org/nova/latest/admin/node-down.html | 13:19 |
| sq4ind | no luck ;/ | 13:19 |
| *** yamamoto has quit IRC | 13:19 | |
| *** dtantsur is now known as dtantsur|brb | 13:20 | |
| openstackgerrit | Merged openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | 13:23 |
| *** eharney has joined #openstack-nova | 13:24 | |
| *** Guest19457 has quit IRC | 13:25 | |
| *** dklyle has quit IRC | 13:27 | |
| openstackgerrit | Patricia Domingues proposed openstack/nova master: load up the volume drivers by checking architecture https://review.openstack.org/541393 | 13:27 |
| *** _ix has joined #openstack-nova | 13:28 | |
| *** idlemind has joined #openstack-nova | 13:30 | |
| *** yamamoto has joined #openstack-nova | 13:30 | |
| *** sahid has joined #openstack-nova | 13:32 | |
| *** lucas-hungry is now known as lucasagomes | 13:32 | |
| dansmith | johnthetubaguy: https://review.openstack.org/#/c/539605 | 13:34 |
| *** _ix has quit IRC | 13:34 | |
| *** yamamoto has quit IRC | 13:35 | |
| giblet | fried_rice, leakypipes: when placement was designed was it intentionaly decided that placement does not provide information about which part of an allocation candidate fulfills which part of the request? | 13:38 |
| fried_rice | giblet: That seems like it would be a tough thing to design, implement, reason about. | 13:39 |
| giblet | fried_rice, leakypipes: I'm hit the issue that Neutron needs to know which part of an allocation record fulfills the original resource request of a given port | 13:39 |
| bauwser | fried_rice: leakypipes: I have a question for you about https://review.openstack.org/#/c/552924/ | 13:39 |
| leakypipes | giblet: not sure I follow you... an allocation request does indeed indicate which resources are served by which providers... | 13:39 |
| *** dansmith is now known as superdan | 13:39 | |
| fried_rice | giblet: Ah, yes, that's legitimate, and an issue that virt drivers will also have. It's going to come down to neutron needing to be able to map RP IDs/names to something that lets it know which real resource it corresponds to. | 13:40 |
| bauwser | fried_rice: leakypipes: if we split the compute resources in between NUMA nodes (like for vCPUs), if a guest is asking for 2 vCPUs, would it be possible to have one vCPU in one NUMA node and the other vCPU in a separate node ? | 13:40 |
| bauwser | like we do at the moment? | 13:41 |
| fried_rice | bauwser: only if you specify them in separate request groups. | 13:41 |
| *** esberglu has joined #openstack-nova | 13:41 | |
| *** yamahata has joined #openstack-nova | 13:41 | |
| bauwser | fried_rice: that's my concern | 13:41 |
| giblet | leakypipes: true, the allocation request is created against an RP, but the connection between an allocation request and the original resource request of a port is missing | 13:41 |
| fried_rice | bauwser: ...or if the driver models them in a single RP. I.e. the host is set up not to do NUMA at all. | 13:41 |
| bauwser | fried_rice: leakypipes: for the moment, say I have two NUMA nodes with 4 pCPUs each one | 13:42 |
| leakypipes | giblet: I'm not following you... | 13:42 |
| fried_rice | giblet: I think what we're getting at is: if you're having trouble making that connection, it's because you didn't split up your RPs enough. | 13:42 |
| bauwser | fried_rice: leakypipes: if 3 pCPUs are already taken, I can still have a instance using two pCPUs from two different NUMA nodes | 13:43 |
| bauwser | fried_rice: leakypipes: if we say we need to have a request query for that, then it would be a problem | 13:43 |
| giblet | fried_rice: OK that is a direction I have to investigate in the neutron port case | 13:43 |
| giblet | leakypipes: there is a neturon port, that will have a resource_request, that will end up as a granular resource request group in the GET allocation_candidates query | 13:44 |
| giblet | leakypipes: but parts of the returned allocation candidates does not mapps back to the original granular resource request group | 13:44 |
| giblet | leakypipes: I means placement does not specify which subset of an allocation candidate maps to a certain granular resource group | 13:45 |
| fried_rice | giblet: That's correct, but it also shouldn't need to. | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: RP list: member_of and resources parameters (v1.3, v1.4) https://review.openstack.org/511183 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 13:45 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 13:45 |
| *** yamamoto has joined #openstack-nova | 13:45 | |
| *** josecastroleon has quit IRC | 13:46 | |
| fried_rice | giblet: Because if you have two request groups that otherwise look the same, then it shouldn't matter which of those two pieces of the response you "use" for which. | 13:46 |
| kashyap | superdan: BTW, thanks for the validation question, that prompted me to file the bug that I needed to, for the new libvirt API: https://review.openstack.org/#/c/534384/10/nova/conf/libvirt.py | 13:46 |
| giblet | leakypipes, fried_rice: port1 and port2 both needs BW resource, an allocation candidate will have RP - BW pairs for both port1 and port2 but there is no way to tell which pair maps to which port | 13:46 |
| kashyap | superdan: And the libvirt dev already assigned it to himself and working on it :-) | 13:46 |
| superdan | kashyap: cool | 13:47 |
| fried_rice | But if the request groups were differentiated by, say, a custom trait indicating a physical network, then those traits will be in the provider summary, and you can use that to distinguish. | 13:47 |
| kashyap | superdan: Appreciate the eagle eyes. | 13:47 |
| giblet | fried_rice: true, the response is not ambigous. It is just haaard to make a generic code that does the mapping | 13:47 |
| leakypipes | giblet: the provider_summaries section of the allocation candidates response contains all the trait and usage information that callers would need to decide that mapping, though. | 13:47 |
| fried_rice | giblet: Well, you'd have to do the math, yeah. | 13:47 |
| fried_rice | which I agree is awkward | 13:48 |
| giblet | leakypipes, fried_rice: thanks, I understand now | 13:48 |
| giblet | a bit better... | 13:48 |
| fried_rice | bauwser: As we've currently got things conceived, if you want to do a don't-care-about-NUMA-affinity resource request on a system that's set up for NUMA, you should split your CPU resources into separate request groups. | 13:49 |
| fried_rice | bauwser: It's awkward, but without further invention, it's the only way to guarantee that you'd be able to get a result spread across multiple nodes if you need. | 13:49 |
| bauwser | fried_rice: not sure I understand the latter | 13:49 |
| bauwser | WDYM by splitting your resources into separate groups ? | 13:50 |
| kashyap | superdan: Do you absolutely want me to remove the white space change? https://review.openstack.org/#/c/534384/10/nova/virt/libvirt/driver.py | 13:50 |
| fried_rice | bauwser: Instead of resources=VCPU:2, which would always give you both VCPUs from the same provider, you would have to say resources1=VCPU:1&resources2=VCPU:1 | 13:50 |
| superdan | kashyap: I absolutely do | 13:50 |
| *** yamamoto has quit IRC | 13:50 | |
| kashyap | superdan: :-) Did it to keep the conditionals in line. But where is the next chance to "fix it"? | 13:50 |
| bauwser | fried_rice: that looks a terrible upgrade impact for operators IMHO | 13:50 |
| kashyap | A separate change just to do that is overkill | 13:51 |
| *** awaugama has joined #openstack-nova | 13:51 | |
| superdan | kashyap: it's not worth fixing, IMHO | 13:51 |
| kashyap | Okay, I'll curb my OCD there | 13:51 |
| bauwser | as an operator, I just want to provide flavors, either for specific NUMA calls, or just for asking resources | 13:51 |
| fried_rice | bauwser: Again, without further invention, that's the way it's got to be. Think about the implications if we supported splitting a single VCPU:2 across multiple RPs. | 13:51 |
| leakypipes | bauwser, giblet, fried_rice: I'm almost done reworking the cpu-resources spec with all the feedback from yesterday's 6-hour sage IRC conversation. Gimme about an hour and the new spec should answer a number of your questions. | 13:51 |
| *** yamamoto has joined #openstack-nova | 13:51 | |
| bauwser | fried_rice: we haven't modeled NUMA yet, hence my questions | 13:52 |
| *** sidx64 has quit IRC | 13:52 | |
| sean-k-mooney[m] | Fried_rice thats concering as if i have a gust that can't fit on a single numa node i cant boot that anymore if i dont request a multi numa topology | 13:52 |
| bauwser | fried_rice: if we go sharding the resources between NUMA nodes, then that's a terrible operator impact IMHO | 13:52 |
| fried_rice | bauwser: The example I gave was DISK_GB. If I say resources=DISK_GB:1024, but my compute host has multiple storage RPs (including shared), I definitely don't want GET /a_c to return me candidates where those DISK_GB are split across multiple. | 13:52 |
| bauwser | that needs at least to be properly documented | 13:52 |
| bauwser | fried_rice: again, I understand nested RPs | 13:53 |
| bauwser | fried_rice: so, yeah I agree | 13:53 |
| fried_rice | bauwser: That goes for nested and sharing alike. | 13:53 |
| bauwser | fried_rice: if you shard your resources between separate nested children, then it's understandable | 13:53 |
| bauwser | fried_rice: but here I'm talking of NUMA | 13:53 |
| sahid | leakypipes: mriedem, thanks for your efforts on reviewing the trusted vfs specs | 13:53 |
| fried_rice | bauwser: I don't disagree that this is awkward. But we would have to come up with some new semantic to allow spreading. resources=VCPU:2&allow_spread=resources | 13:53 |
| fried_rice | (ew) | 13:53 |
| bauwser | that looks to me the default behaviour to have IMHO | 13:54 |
| fried_rice | bauwser: You're talking about a user wanting to get a non-NUMA-affined instance on a NUMA-modeled compute host. | 13:54 |
| bauwser | by default, spread between NUMA nodes | 13:54 |
| fried_rice | bauwser: Unless we start special-casing by resource class, no can do. Because of the DISK_GB example. | 13:54 |
| bauwser | fried_rice: I'm talking of a user using a flavor that doesn't ask for NUMA resources, just vCPUs, yeah | 13:54 |
| fried_rice | ...which would be an issue *today* (whereas NUMA is a tomorrow thing) | 13:55 |
| sean-k-mooney[m] | bauwser: fried_rice this will be a behaviour change form today. Today we only numa affites if you ask for it | 13:55 |
| bauwser | what sean-k-mooney[m] wrote | 13:55 |
| bauwser | that would be a terrible upgrade impact | 13:55 |
| mriedem | sahid: np, i left some comments in the 2nd patch in the code series | 13:55 |
| bauwser | hence why I originally modeled my spec with vCPU resources being on the root RP | 13:55 |
| bauwser | and only NUMA specific resources being on the children | 13:56 |
| bauwser | to keep existing behaviour compatible | 13:56 |
| *** yamamoto has quit IRC | 13:56 | |
| fried_rice | bauwser: Okay, but then we would need some way to decrement the NUMA resources when the top-level resources are requested/consumed. | 13:56 |
| sahid | mriedem: ok i will address them, i also have noticed your comment on the first patch | 13:56 |
| bauwser | fried_rice: that's a separate NUMA resource class, see my spec | 13:56 |
| bauwser | the terms are maybe not good | 13:57 |
| leakypipes | sahid: np. thanks for your patience. | 13:57 |
| leakypipes | sahid: I should have the cpu-resources next revision up shortly for your review. | 13:57 |
| bauwser | but if I'm asking for resources:VCPU=2,NUMA_VCPU=2 then it'll decrement both | 13:57 |
| sahid | leakypipes: ok, sure i will | 13:58 |
| sahid | review it | 13:58 |
| sean-k-mooney[m] | Well i think we need to have vcpus under numa but i think we will need something likek a grouping level attribute that defaults to host. E. G. Host, numa, rp grouping | 13:58 |
| *** links has quit IRC | 13:58 | |
| fried_rice | bauwser: yup. But how is that better than asking for resources1:VCPU:1&resources2=VCPU:1 | 13:58 |
| bauwser | fried_rice: because existing flavors will still work for non-NUMA queries | 13:59 |
| bauwser | that's all the difference | 13:59 |
| sean-k-mooney[m] | Fried_rice how do you know how to create the groups | 13:59 |
| bauwser | here, you're asking to change all the operator's flavor to explicitly tell whether they care about NUMA sharding or not | 13:59 |
| fried_rice | bauwser: Existing flavors have resources:VCPU=2,NUMA_VCPU=2 ? | 13:59 |
| bauwser | existing flavors just have VCPU resources | 13:59 |
| fried_rice | right | 13:59 |
| figleaf | fried_rice: how could you possibly know which NUMA nodes to decrement resources from? | 14:00 |
| bauwser | like I wrote, we can have Nova to ask for both resource classes if it sees that | 14:00 |
| fried_rice | So look, there are ways to handle this with and without changing the placement API | 14:00 |
| fried_rice | yeah, that'd be one way. | 14:00 |
| sean-k-mooney[m] | Existing flavours without numa_nodes set dont have numa affinity for cpus | 14:00 |
| fried_rice | The other way is to have nova see that the flavor doesn't care about NUMA affinity and *nova* splits up the resource requests | 14:00 |
| bauwser | I mean, if a flavor tells only about vCPUs, then query Placement only for that, but make allocations for both VCPU and NUMA_SOMETHING_VCPU | 14:00 |
| bauwser | fried_rice: that's what I wrote in the spec | 14:01 |
| bauwser | nova could make the translation | 14:01 |
| bauwser | leakypipes: ^ I hope you follow | 14:01 |
| fried_rice | bauwser: Cool; either way, we're having nova massage the request. But I prefer the path where we're actually representing true inventory numbers. | 14:01 |
| sean-k-mooney[m] | Fried_rice how does nova know how to split it up | 14:01 |
| fried_rice | sean-k-mooney[m]: For VCPU, it's one per request group, plain and simple. | 14:02 |
| bauwser | sean-k-mooney[m]: there are two different things, the placement query and the allocation | 14:02 |
| bauwser | fried_rice: also, note I'd like to not depend on request groups for NUMA | 14:02 |
| *** andreas_s has quit IRC | 14:02 | |
| fried_rice | bauwser: Whyzat? | 14:02 |
| *** diga has joined #openstack-nova | 14:02 | |
| fried_rice | I don't think you can get around it, especially when you start talking about e.g. NICs. | 14:03 |
| sean-k-mooney[m] | Flavor has 16 vcpus and dose not care about numa. All hosts have 2 10 cpus. How does nova know it needs to split into two groups | 14:03 |
| bauwser | again, that requires operators opiniating about which specific placement they should do per flavor for non-NUMA specific instances | 14:03 |
| fried_rice | sean-k-mooney[m]: it doesn't split into two groups. It splits into 16 | 14:03 |
| bauwser | sean-k-mooney[m]: my vision was that we would ask placement only for VCPUs that are on the root RP | 14:03 |
| fried_rice | Sorry guys, got to run for a bit. | 14:03 |
| bauwser | sean-k-mooney[m]: but once we get a RP, we would make the allocation against a child | 14:04 |
| bauwser | well, erm | 14:04 |
| bauwser | nevermind | 14:04 |
| bauwser | it's stupid | 14:04 |
| sean-k-mooney[m] | So 16 request of 1 cpu | 14:04 |
| bauwser | what we need is to get all the children that match the query | 14:04 |
| bauwser | and only consume one of those | 14:04 |
| bauwser | fuuu, I'm getting headaches | 14:04 |
| sean-k-mooney[m] | leakypipes: is this something we should cover in your spec | 14:05 |
| figleaf | bauwser: "only in root" - that was one of the decisions we made back in Atlanta. A machine would be configured by the operator ahead of time. If a VCPU was assigned to a NUMA node, it could not be considered also to be part of the root compute node | 14:06 |
| bauwser | sean-k-mooney[m]: NUMA related things should be covered in https://review.openstack.org/#/c/552924/3/specs/rocky/approved/numa-topology-with-rps.rst | 14:06 |
| bauwser | figleaf: I don't disagree with that statement | 14:07 |
| sean-k-mooney[m] | figleaf: almost all server systems are numa system even systems with one socket | 14:07 |
| bauwser | figleaf: I'm just trying to evaluate the operator impact of a NoValidHost if we begin counting NUMA nodes for non-NUMA specific instances | 14:08 |
| figleaf | sean-k-mooney[m]: sure, but the choice on how to consume them (respecting NUMA, or ignoring) is an up-front decision by the operator. | 14:08 |
| *** ratailor has quit IRC | 14:08 | |
| sean-k-mooney[m] | bauwser: i think it also needs to be in jays cpu resources spec | 14:08 |
| bauwser | figleaf: so you agree with the fact that a classic flavor just asking for VCPUs shouldn't be accounting NUMA nodes, right? | 14:09 |
| *** dave-mccowan has joined #openstack-nova | 14:09 | |
| sean-k-mooney[m] | figleaf: no its a choice of the virt driver | 14:09 |
| bauwser | how the inventory is modeled is one thing | 14:09 |
| bauwser | and I'm fine with having VCPU resources on a NUMA node if that helps | 14:09 |
| figleaf | bauwser: of course, but it would only select a host that was configured *not* to use NUMA. IOW, the ram, vcpu, etc, was inventory of the compute | 14:09 |
| bauwser | but my point is, I don't want to get a NoValidHost if that restricts my existing flavor to specific NUMA nodes | 14:10 |
| openstackgerrit | Merged openstack/nova master: Change compute mgr placement check to region_name https://review.openstack.org/554759 | 14:10 |
| bauwser | figleaf: that's super costly then | 14:10 |
| openstackgerrit | Merged openstack/nova master: Remove translate and a TODO https://review.openstack.org/554827 | 14:11 |
| figleaf | bauwser: if you configure all your machines not to use NUMA, and request a NUMA-specific flavor, you will get NoValidHost. The reverse would also be true | 14:11 |
| bauwser | figleaf: because I want to have both compute workloads that can do NUMA affinity and regular workloads that don't care | 14:11 |
| bauwser | figleaf: again, that's a huuuuuge behavioural change | 14:11 |
| figleaf | bauwser: sure, then configure some of your machines for NUMA, and others for not-NUMA | 14:11 |
| figleaf | The balance is up to the operator | 14:11 |
| bauwser | it's suboptimal | 14:11 |
| sean-k-mooney[m] | figleaf: we don't have a compute node option for ignore numa | 14:12 |
| cdent | making the optimal solution is introducing an enormous amount of complexity that is making hard to finish _any_ solution | 14:12 |
| bauwser | sean-k-mooney[m]: I guess figleaf wants it | 14:12 |
| *** jpena is now known as jpena|lunch | 14:12 | |
| sean-k-mooney[m] | Figleaf but that is not something you can configure normally | 14:12 |
| bauwser | if you opt for non-NUMA, then you get a flat inventoty | 14:12 |
| bauwser | if you opt for NUMA topology, then you're stuck with NUMA-only workloads on that single node | 14:13 |
| *** yamahata has quit IRC | 14:13 | |
| bauwser | :/ | 14:13 |
| figleaf | sean-k-mooney[m]: it would be how resources are reported. If the virt driver reports that a NUMA node owns some inventory, then that NUMA node has to satisfy the request. There is no concept in placement for the kind of sharing you're talking about | 14:14 |
| bauwser | cdent: that's the whole purpose of the spec | 14:14 |
| *** amoralej is now known as amoralej|lunch | 14:14 | |
| cdent | bauwser: which "that's"? | 14:14 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Standardize CPU resource tracking https://review.openstack.org/555081 | 14:14 |
| bauwser | cdent: to see if we can model the NUMA inventories in any other ways that would help to solve the problem without requiring too much engineering | 14:14 |
| leakypipes | cfriesen, bauwser, sahid, sean-k-mooney[m], finucannot, fried_rice: ^^ ok, new revision pushed. lemme know if that looks a bit better. | 14:15 |
| sean-k-mooney[m] | figleaf: again how would yo do that. Numa is a feature of the hardware its not a software option so your talking about ordering hardware with out multiple memory controllers or configure mirrors dimms in the bios | 14:15 |
| bauwser | leakypipes: you missed a NUMA conversation | 14:15 |
| cdent | bauwser: okay. My statement was more along the lines of "trying to satisfy all this NUMA and other enhanced platform stuff, in general, is breaking the 80/20 rule" | 14:15 |
| *** tidwellr has joined #openstack-nova | 14:15 | |
| leakypipes | bauwser: not really. see the spec. :) | 14:15 |
| bauwser | cdent: the problem is that like sean-k-mooney[m] said, modern architectures are NUMA-based, but 80% of our workloads don't care | 14:16 |
| sean-k-mooney[m] | figleaf: not in placement but that is how nova works today. | 14:16 |
| sean-k-mooney[m] | leakypipes: cool will do | 14:16 |
| bauwser | leakypipes: hah, teaser! | 14:16 |
| openstackgerrit | Merged openstack/nova master: Add the version description for InstanceActionEventList https://review.openstack.org/555711 | 14:16 |
| bauwser | I like that | 14:16 |
| sean-k-mooney[m] | Lleakypipes yes this is related to that spec too | 14:16 |
| *** yamahata has joined #openstack-nova | 14:17 | |
| *** gouthamr has joined #openstack-nova | 14:17 | |
| figleaf | sean-k-mooney[m]: it is software that reports what hardware exists to placement. If it configure to respect the NUMA layout, it reports the inventory that NUMA nodes "own". If it is configured to sum up the resources, then it reports inventory of the compute node. | 14:17 |
| * leakypipes moves on to sahid's tx/rx spec | 14:17 | |
| bauwser | I quote "placement is the most ambitious crossover of the history" | 14:17 |
| sean-k-mooney[m] | cdent: well in this case we are breaking insance that dont care about numa by fitting them to numa nodes | 14:18 |
| *** gus has quit IRC | 14:18 | |
| bauwser | leakypipes: https://twitter.com/sylvainbauza/status/977187898167611392 | 14:19 |
| giblet | bauwser: lol | 14:20 |
| *** burt has joined #openstack-nova | 14:21 | |
| openstackgerrit | Merged openstack/nova master: remove _cleanup_running_deleted_instances repeat detach volume https://review.openstack.org/554090 | 14:21 |
| openstackgerrit | Merged openstack/nova master: deprecate fping_path config option https://review.openstack.org/526602 | 14:21 |
| mriedem | there is no reason we need to warn for this right? http://git.openstack.org/cgit/openstack/nova/tree/nova/compute/manager.py#n3340 | 14:22 |
| mriedem | the user is trying to do something the api allows them to do, so the operator shouldn't be warned about it | 14:22 |
| * bauwser goes reading the last iteration of leakypipes's spec | 14:22 | |
| mriedem | https://git.openstack.org/cgit/openstack/nova/tree/nova/compute/api.py#n2717 | 14:22 |
| bauwser | I think it's time for me writing a new rev' for NUMA and stating the concerns in the spexc | 14:22 |
| cdent | my laptop battery and my brain battery are dying | 14:25 |
| bauwser | cdent: take a power bank | 14:26 |
| cdent | bauwser: you got one for brains? hook me up | 14:26 |
| bauwser | cdent: that said, I can't perclue where to plug the bank | 14:26 |
| bauwser | s/perclue/guess | 14:27 |
| *** archit has joined #openstack-nova | 14:27 | |
| bauwser | note for self, stop buying expensive nespresso machines that are just subject to failures | 14:28 |
| bauwser | and start finding other ways for making coffee | 14:28 |
| *** elmaciej has joined #openstack-nova | 14:28 | |
| *** zhaochao has quit IRC | 14:29 | |
| *** dave-mcc_ has joined #openstack-nova | 14:29 | |
| *** cdent has quit IRC | 14:29 | |
| openstackgerrit | Tyler Blakeslee proposed openstack/nova master: Add __repr__ for NovaException https://review.openstack.org/555812 | 14:29 |
| *** gus has joined #openstack-nova | 14:30 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | 14:31 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Modify nova-manage cell_v2 list_cells to display "disabled" column https://review.openstack.org/555415 | 14:31 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416 | 14:31 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled option to create_cell command https://review.openstack.org/555417 | 14:31 |
| *** dave-mccowan has quit IRC | 14:32 | |
| *** mlavalle has joined #openstack-nova | 14:32 | |
| sahid | leakypipes: ++ on you spec cpu-resources | 14:35 |
| *** sahid has quit IRC | 14:36 | |
| *** sahid has joined #openstack-nova | 14:37 | |
| *** sree has joined #openstack-nova | 14:37 | |
| *** dtantsur|brb is now known as dtantsur | 14:37 | |
| bauwser | leakypipes: time for a question ? | 14:38 |
| bauwser | leakypipes: about https://review.openstack.org/#/c/555081/3/specs/rocky/approved/cpu-resources.rst@249 and beyond | 14:39 |
| leakypipes | bauwser: sure | 14:39 |
| leakypipes | sahid: +2 on tx/rx | 14:39 |
| leakypipes | sahid: I will let mriedem +W it though | 14:39 |
| bauwser | leakypipes: just noting a few things on the spec | 14:39 |
| bauwser | leakypipes: sahid: the TX/RX was next in my review queue | 14:40 |
| superdan | leakypipes: mriedem said he was leaving it to us in the meeting, so I think you can +W | 14:40 |
| bauwser | I have a fucking low number of reviews this week | 14:40 |
| leakypipes | superdan: got it. done. | 14:40 |
| leakypipes | bauwser: reading specs takes a long time... | 14:41 |
| sahid | superdan thanks for having so active on it | 14:41 |
| superdan | sahid: s/having/being/ | 14:41 |
| * superdan is still in review mode :) | 14:41 | |
| leakypipes | :) | 14:41 |
| sahid | :) | 14:41 |
| *** psachin has quit IRC | 14:42 | |
| *** tidwellr has quit IRC | 14:43 | |
| mriedem | kashyap: i updated https://wiki.openstack.org/wiki/LibvirtDistroSupportMatrix for queens and added the ubuntu 18.04 LTS package versions based on https://packages.ubuntu.com/bionic/allpackages | 14:44 |
| figleaf | leakypipes: quick question about the spec. When there are NUMA nodes with V/PCPU inventory, and a non-granular request is received, will there be allocation candidates returned for every possible combination of NUMA providers? | 14:44 |
| mriedem | looks like all of the other distros in that table are old | 14:44 |
| *** r-daneel has joined #openstack-nova | 14:44 | |
| kashyap | mriedem: Ah-ha. I have it open in one tab | 14:45 |
| kashyap | And was reviewing your historical changes there yesterday | 14:45 |
| mriedem | kashyap: so since we didn't bump in queens, | 14:45 |
| kashyap | Will get to it, once I respond to this bug-from-hell downstream | 14:45 |
| mriedem | we already have a 'next min version' in the driver, which is actually kind of old now, libvirt 1.3.1 | 14:45 |
| leakypipes | figleaf: yes, alloc candidates would contain alloc requests against each NUMA node that had space to fit the sum amount of requested VCPU/PCPU | 14:45 |
| mriedem | the question is then what will the next min version be in Solar | 14:45 |
| kashyap | What is Solar? | 14:45 |
| leakypipes | figleaf: I can add a note to clarify that. | 14:45 |
| kashyap | A new Ubuntu version? | 14:45 |
| mriedem | solar is next openstack | 14:46 |
| mriedem | (pending that name is approved by legal) | 14:46 |
| kashyap | Aah, I didn't check the release naming for the next one | 14:46 |
| leakypipes | I guess my suggestion of "Suck It, Trebek" didn't fly. | 14:46 |
| figleaf | leakypipes: that's what I thought. When an a-c is chosen and allocated against, is there anything that constrains the building of the VM to the particular NUMA node that was allocated against? | 14:46 |
| bauwser | figleaf: leakypipes: reviewed | 14:47 |
| leakypipes | figleaf: nothing besides the provider UUID, no. | 14:47 |
| openstackgerrit | Merged openstack/nova-specs master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/539605 | 14:47 |
| bauwser | figleaf: leakypipes: my main concern is that existing flavors that don't ask for NUMA specifics shouldn't be constrained on NUMA nodes | 14:48 |
| figleaf | leakypipes: and there is no relation between the RP UUID in placement and a particular NUMA node in hardware, right? | 14:48 |
| leakypipes | bauwser: they aren't "constrained". they just end up on a NUMA node is the host system is NUMA.. | 14:48 |
| leakypipes | s/is/if/ | 14:48 |
| bauwser | leakypipes: back up a second | 14:48 |
| bauwser | leakypipes: I'm taking the example I had previously | 14:49 |
| bauwser | say I ask for two VCPUs, period. | 14:49 |
| bauwser | what if my compute only has only vCPU left on one NUMA node, but one other vCPU left on another NUMA node ? | 14:49 |
| bauwser | at the moment, nova will satisfy that | 14:49 |
| leakypipes | bauwser: if the host system is NUMA, then the VCPU inventories will be against child provider representing NUMA nodes. And the allocation candidates will contain allocation requests that consume against one of those NUMA node resource providers. | 14:50 |
| bauwser | because the strategy is "spread" between all NUMA nodes, except if you clearly ask for another strategy | 14:50 |
| leakypipes | bauwser: in that case, that compute node would not be returned. | 14:50 |
| leakypipes | bauwser: since no single provider has capacity for 2 VCPU | 14:50 |
| bauwser | leakypipes: that's the whole point I discussed previously | 14:50 |
| bauwser | leakypipes: that's a behavioural change | 14:51 |
| bauwser | from a placement perspective, what was successful before is no longer possible | 14:51 |
| -openstackstatus- NOTICE: zuul.o.o has been restarted to pick up latest code base and clear memory usage. Both check / gate queues were saved, be sure to check your patches and recheck when needed. | 14:52 | |
| leakypipes | bauwser: then don't have the host configured to create NUMA node resource providers. | 14:52 |
| leakypipes | bauwser: and just use a single provider for the comute node. | 14:52 |
| *** yamamoto has joined #openstack-nova | 14:52 | |
| bauwser | leakypipes: so only specific NUMA workloads would run on a NUMA-specified compute, and non-NUMA workloads would end up on the rest ? | 14:53 |
| bauwser | if that's a config option, the default value would be "flat topology" | 14:53 |
| leakypipes | bauwser: not following you... how is that different from today? | 14:53 |
| bauwser | leakypipes: today you can end up having non-NUMA workloads that are running on a NUMA compute | 14:54 |
| *** danpawlik has quit IRC | 14:54 | |
| bauwser | actually | 14:54 |
| bauwser | sec | 14:54 |
| bauwser | doing huge pages and CPU pinning would be insane | 14:54 |
| bauwser | if some workloads would just pick any CPU | 14:54 |
| leakypipes | bauwser: yes, and there's nothing in the new system that would change that. the operator simply wouldn't set up the compute host to create NUMA child providers on that host (if this is really an issue that anyone is concerned about)( | 14:55 |
| bauwser | leakypipes: okay, I see your point, how the operator would drive whether they want a non-NUMA inventory or not ? | 14:55 |
| bauwser | thru a config option ? | 14:55 |
| leakypipes | bauwser: that's up to your spec, no? :) | 14:56 |
| bauwser | fair enough :) | 14:56 |
| bauwser | okay, I think that's a reasonable trade-off | 14:56 |
| *** sar has quit IRC | 14:56 | |
| *** david-lyle has joined #openstack-nova | 14:57 | |
| * leakypipes grabs more coffee, fuels up for the day of spec reviews | 14:57 | |
| *** yamamoto has quit IRC | 14:57 | |
| figleaf | bauwser: that's what I meant here [t 3hcx] | 14:58 |
| purplerbot | <figleaf> bauwser: "only in root" - that was one of the decisions we made back in Atlanta. A machine would be configured by the operator ahead of time. If a VCPU was assigned to a NUMA node, it could not be considered also to be part of the root compute node [2018-03-23 14:06:19.844755] [n 3hcx] | 14:58 |
| *** jpena|lunch is now known as jpena | 14:58 | |
| bauwser | figleaf: yeah, what changed now is that I think that most operators already shard their cloud in between hosts that supports NUMA workloads and others (thru aggregates I guess) | 15:00 |
| bauwser | because doing CPU pinning requires that your host is pretty dedicated for | 15:00 |
| bauwser | I'm not saying you were wrong :) | 15:00 |
| figleaf | bauwser: of course not! :) | 15:01 |
| *** alexchadin has quit IRC | 15:01 | |
| *** mdnadeem has quit IRC | 15:02 | |
| mlavalle | rybridges: hi, when you have a chance, plese look here: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/plugin.py#L232. As you can see, after adding the hosts to the aggregate, it adds the IPv$ inventory to the RP. Was the inventory created? | 15:02 |
| *** r-daneel has quit IRC | 15:04 | |
| *** pooja_jadhav has quit IRC | 15:05 | |
| *** ktibi has joined #openstack-nova | 15:06 | |
| sean-k-mooney[m] | leakypipes: so just finished the v3 of the spec. i like this much more but some nits inline | 15:09 |
| *** amoralej|lunch is now known as amoralej | 15:09 | |
| leakypipes | sean-k-mooney[m]: cool, ty sir | 15:09 |
| leakypipes | sean-k-mooney[m], cfriesen: wanted to make sure I got all the definitions crystal clear... | 15:09 |
| leakypipes | sean-k-mooney[m], cfriesen: did I properly describe vCPU threads, emulator threads and I/O threads? | 15:09 |
| *** tbachman has quit IRC | 15:11 | |
| ktibi | Hi, I try to add a compute on existing platform and compute return errors : Another thread already created a resource provider with the UUID XXXXX. I think it's because I remove this compute previously and added it with same hostname. | 15:11 |
| rybridges | mlavalle: The inventory is not being created. All the resource provider tied to the segment does get created, but the inventories, usages, and aggregates associated with the resource provider are all empty | 15:11 |
| sean-k-mooney[m] | leakypipes: just read them again and yes i think so. we dont support configuring io threads in nova but the definition is correct | 15:12 |
| ktibi | where do you think I need to purge or clean compute ? | 15:12 |
| ktibi | full log: http://paste.openstack.org/show/710134/ | 15:13 |
| rybridges | mlavalle: In fact, I dont even think the code gets to the line and executes it at all because the aggregate never even gets associated. I added a very simple log statement before and after this line: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/plugin.py#L225 and I see the one before being printed every time, but the one after never gets printed impying that after | 15:13 |
| rybridges | we call associate_aggregates() the control never returns | 15:13 |
| rybridges | I also put a log statement on this line and I never see it get printed: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/placement_client.py#L163 | 15:14 |
| rybridges | very strange | 15:14 |
| sean-k-mooney[m] | bauwser: do you have a minute to discuss the nic feature based scheduling spec | 15:14 |
| *** hongbin has joined #openstack-nova | 15:14 | |
| bauwser | sean-k-mooney[m]: sorry I saw your pings earlier | 15:15 |
| bauwser | sean-k-mooney[m]: I'm currently in a team meeting but shoot | 15:15 |
| leakypipes | ktibi: can you re-start the nova-compoute with the DEBUG level logging and pastebin the log please? | 15:15 |
| ktibi | leakypipes, y of course | 15:15 |
| leakypipes | ktibi: it does indeed look like you may need to issue a couple manual cleanup calls to the placement API. but tough to tell without the debug logs | 15:16 |
| sean-k-mooney[m] | bauwser: https://review.openstack.org/#/c/545951/, i dont know how to proceed with this. i am only ment to be spending 10% of my time upstream this cycle. | 15:16 |
| sean-k-mooney[m] | bauwser: if i was to start this spec from scratch i would do it differently today but i wont have time to rewrite it this cycle | 15:16 |
| bauwser | sean-k-mooney[m]: opening the spec | 15:17 |
| bauwser | I need to reload context in my mind | 15:17 |
| bauwser | aaaaaah I remember | 15:18 |
| ktibi | leakypipes, http://paste.openstack.org/show/710154/ | 15:18 |
| bauwser | sean-k-mooney[m]: well, I don't remember the specific outcome we had during the PTG | 15:18 |
| leakypipes | ktibi: cool. thanks. btw, is this queens? | 15:19 |
| bauwser | sean-k-mooney[m]: could you please refresh my mind ? | 15:19 |
| bauwser | sean-k-mooney[m]: IIRC, we discussed on the opportunity to move things to conductor, right? | 15:19 |
| ktibi | leakypipes, pike with kolla | 15:19 |
| bauwser | not nobody objected | 15:19 |
| leakypipes | ktibi: if you could pastebin the placement-api logs, too, that would be great | 15:19 |
| sean-k-mooney[m] | bauwser: ya so rofolfo work on this for 2 cycles. i was just ment to be rebasing it an resubmitting his work this cycle. if its beyond that however i have to escalate to my managment to see if i can work on it an rewrite it to track the nic features as tratis in placement. | 15:20 |
| bauwser | I see | 15:20 |
| bauwser | okay, I don't honestly want to throw you under the bus | 15:20 |
| *** itlinux has joined #openstack-nova | 15:21 | |
| bauwser | my only concern is that it creates more tech debt | 15:21 |
| sean-k-mooney[m] | bauwser: today there is no reason we could not use nested RPs + traits to track the nic feartres for sriov device. | 15:21 |
| bauwser | but I see leakypipes +2ing that spec | 15:21 |
| sean-k-mooney[m] | we have been storing that in the nova db since pike. the remainder of the spec is consuming it in the schduler. | 15:21 |
| bauwser | so I guess it's more or less a trade-off | 15:21 |
| sean-k-mooney[m] | bauwser: ya that is fair | 15:21 |
| bauwser | well, okay, I have to disappear in a short few because of kids, but once I'm back, I'll reevaluate the spec | 15:22 |
| sean-k-mooney[m] | well i can ask can i try to port this once it merges to placement | 15:22 |
| ktibi | leakypipes, I checked placement log, i can see only info log with GET on resource_providers/705b7a45-ce71-4455-8349-6cee4c051c0e/inventories | 15:22 |
| leakypipes | ktibi: question for you... it looks like there are no instances in the cell DB for that compute node. were there originally instances on it? | 15:23 |
| sean-k-mooney[m] | ok cool im technically off today but let me know what you decide and ill be back monday | 15:23 |
| ktibi | leakypipes, no when I remove this node, I disable service, evacuate all instances, remove service and shutdown the compute | 15:24 |
| *** bhagyashris has quit IRC | 15:24 | |
| leakypipes | ktibi: k. I'm wondering maybe the placement service for some reason didn't get updated with the removal of those instances... | 15:24 |
| sean-k-mooney[m] | bauwser: if i was able to also port to doing this in placement in rocky that would be the best outcome but im ment to work on someting else internally from april that 10%. | 15:24 |
| leakypipes | ktibi: can you please execute the following SQL query against the nova api database? | 15:24 |
| *** Shilpa has quit IRC | 15:25 | |
| *** cdent has joined #openstack-nova | 15:25 | |
| leakypipes | ktibi: SELECT COUNT(*) FROM allocations AS a JOIN inventories AS i ON a.resource_provider_id = i.resource_provider_id JOIN resource_providers AS rp ON i.resource_provider_id = rp.id WHERE rp.uuid = '91f9e750-351f-463e-aa00-c8843c37560e'; | 15:25 |
| *** melwitt is now known as jgwentworth | 15:26 | |
| ktibi | leakypipes, count : 0 | 15:26 |
| sean-k-mooney[m] | o/ ok ill stop working on my day off now... maybe... | 15:26 |
| leakypipes | ktibi: ok, good. | 15:26 |
| leakypipes | ktibi: what about this one? SELECT COUNT(*) FROM inventories AS i ON a.resource_provider_id = i.resource_provider_id JOIN resource_providers AS rp ON i.resource_provider_id = rp.id WHERE rp.uuid = '91f9e750-351f-463e-aa00-c8843c37560e'; | 15:26 |
| leakypipes | ktibi: oops, sorry | 15:26 |
| leakypipes | ktibi: what about this one? SELECT COUNT(*) FROM inventories AS i JOIN resource_providers AS rp ON i.resource_provider_id = rp.id WHERE rp.uuid = '91f9e750-351f-463e-aa00-c8843c37560e'; | 15:27 |
| ktibi | leakypipes, maybe that can help you http://paste.openstack.org/show/710164/ | 15:27 |
| ktibi | leakypipes, count : 0 too | 15:28 |
| leakypipes | k | 15:28 |
| leakypipes | ktibi: got it... | 15:28 |
| cdent | figleaf, fried_rice, leakypipes : just to confirm: I assume we would like the forbidden traits support to show up in the same microversion for /resource_providers and /allocation_candidates, yes? | 15:28 |
| leakypipes | ktibi: your services table has two records for compute06 | 15:28 |
| leakypipes | ktibi: delete the one where deleted_at is not NULL. | 15:29 |
| ktibi | yes I see :) I waiting your comfirmation to delete it ^^ | 15:29 |
| fried_rice | cdent: Sure, unless there's some reason to split 'em. | 15:29 |
| cdent | fried_rice: only if we want to make the code more split into stages | 15:29 |
| fried_rice | no, not necessary IMO. | 15:29 |
| fried_rice | cdent: In fact, I find myself slightly irked that we can't include multiple features in a single microversion - for reasons of our development process, not for any technical reason. | 15:30 |
| fried_rice | cdent: But in this case, it's the same feature, so we can totally get away with it. | 15:30 |
| cdent | yes and yes | 15:31 |
| leakypipes | ktibi: and then set disabled=0 on the other services table record with copmute06 host name. | 15:31 |
| leakypipes | ktibi: finally, can you show me what the compute_nodes table looks like please? just do this: SELECT id, uuid, hypervisor_hostname FROM compute_nodes; | 15:32 |
| ktibi | leakypipes, ho I can see two compute06 | 15:33 |
| ktibi | leakypipes, http://paste.openstack.org/show/710169/ | 15:34 |
| mlavalle | rybridges: in that log statement right above L225, could you log aggregate.uuid? The reason I am asking this is https://review.openstack.org/#/c/415031/, which fixes https://bugs.launchpad.net/nova/+bug/1652642 | 15:35 |
| openstack | Launchpad bug 1652642 in OpenStack Compute (nova) "Nova API doesn't return aggregate's uuid, which is needed when using the placement API" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | 15:35 |
| leakypipes | ktibi: DELETE FROM compute_nodes WHERE id = 15; | 15:35 |
| leakypipes | ktibi: that *should* fix things up... | 15:36 |
| ktibi | leakypipes, ok I restart nova compute on 06 | 15:36 |
| openstackgerrit | melanie witt proposed openstack/nova master: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555093 | 15:36 |
| mlavalle | rybridges: this is the Nova API version you require for aggregates to have a uuid: https://review.openstack.org/#/c/415031/9/nova/api/openstack/api_version_request.py@100 | 15:37 |
| ktibi | leakypipes, same error on compute06 : Another thread already created a resource provider with the UUID 91f9e750-351f-463e-aa00-c8843c37560e. Grabbing that record from the placement API. | 15:37 |
| ktibi | maybe need to restart placement ? | 15:37 |
| leakypipes | ktibi: well, that's not an error. :) the error is the "Cannot refresh inventory" thing. | 15:38 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Create volume attachment during boot from volume in compute https://review.openstack.org/541420 | 15:38 |
| leakypipes | ktibi: no, no need to restart placement. | 15:38 |
| leakypipes | ktibi: what we want to see is inventory records being created in placement. | 15:38 |
| ktibi | leakypipes, ok I'll test to migrate instance to this host | 15:39 |
| *** links has joined #openstack-nova | 15:40 | |
| *** gyee has joined #openstack-nova | 15:41 | |
| *** josecastroleon has joined #openstack-nova | 15:42 | |
| figleaf | cdent: yes, one microversion for that makes sense | 15:42 |
| *** chyka has joined #openstack-nova | 15:43 | |
| ktibi | leakypipes, ok fail :/ | 15:43 |
| ktibi | leakypipes, exception thrown: No valid host was found. Unable to move instance f8e70257-244e-4fa7-b545-0a888b742b84 to host compute06. There is not enough capacity on the host for the instance. | 15:43 |
| giblet | mriedem, leakypipes, mlavalle: is it OK for you If I publish a new version of the bandwidth spec soon, or you are just in the middle of reviewing the current PS? | 15:44 |
| mlavalle | giblet: I'm good | 15:44 |
| mriedem | giblet: i haven't gone back over the comments since i last reviewed it last week | 15:45 |
| mriedem | i could do that quick i guess... | 15:45 |
| ktibi | leakypipes, http://paste.openstack.org/show/710182/ | 15:46 |
| giblet | mriedem: I still need about an hour to do the update locally, if that is enough for you. Or you can go back after I published the new rev. | 15:46 |
| kashyap | mriedem: So just checked w/ the folks on -infra, the Gate already does have the "Bionic" (upcoming Ubuntu LTS) images. Do we have any non-voting Nova jobs for it? | 15:46 |
| kashyap | (Near as see, none.) | 15:46 |
| mriedem | kashyap: no | 15:46 |
| *** links has quit IRC | 15:46 | |
| mriedem | kashyap: i'm just trying to get us to using the queens UCA right now | 15:46 |
| mriedem | https://review.openstack.org/#/c/554317/ | 15:47 |
| * kashyap clicks | 15:47 | |
| giblet | mriedem: there will some new problems described in the update | 15:47 |
| giblet | mriedem: so you might want to read the new rev anyhow | 15:47 |
| mriedem | giblet: i've got about 20 minutes so i'll go over the replies quick | 15:48 |
| giblet | mriedem: OK, cool, thanks | 15:48 |
| -openstackstatus- NOTICE: Gerrit will be temporarily unreachable as we restart it to complete the rename of some projects. | 15:49 | |
| *** diga has quit IRC | 15:52 | |
| mriedem | giblet: yeah i can already see issues with https://review.openstack.org/#/c/502306/17/specs/rocky/approved/bandwidth-resource-provider.rst@100 | 15:52 |
| mriedem | so commenting | 15:52 |
| *** yamamoto has joined #openstack-nova | 15:53 | |
| mriedem | and gerrit just shit the bed | 15:54 |
| giblet | gerrit: you are not helpful at all! | 15:56 |
| giblet | mriedem: gerrit is back | 15:59 |
| *** yassine has quit IRC | 15:59 | |
| *** yamamoto has quit IRC | 15:59 | |
| mriedem | huzzah | 15:59 |
| bauwser | sean-k-mooney[m]: +Wd | 15:59 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: Snapshot https://review.openstack.org/543023 | 16:00 |
| *** yassine has joined #openstack-nova | 16:00 | |
| *** gyankum has quit IRC | 16:00 | |
| rybridges | mlavalle: you might be on to something. When I try to print aggregate.uuid, my log message gets swallowed and i dont see anything getting printed | 16:01 |
| bauwser | hum | 16:01 |
| bauwser | https://review.openstack.org/#/dashboard/?foreach=(project%253Aopenstack%252Fnova-specs)+status%253Aopen+NOT+owner%253Aself+NOT+label%253AWorkflow%3C%253D-1+label%253AVerified%3E%253D1%252Cjenkins+NOT+reviewedby%253Aself+is%253Amergeable&title=Nova+Specs+Review+Inbox+&Needs+final+%252B2=NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+label%253ACode-Review%3E%253D2&Small+Patches=NOT+label%253ACode-Review%3C%253D-1%252Cnova-cor | 16:01 |
| bauwser | delta%253A%3C%253D10&Needs+Feedback+(Changes+older+than+5+days+that+have+not+been+reviewed+by+anyone)=NOT+label%253ACode-Review%3C%253D2+age%253A5d&You+are+a+reviewer%252C+but+haven't+voted+in+the+current+revision=reviewer%253Aself&Passed+Jenkins%252C+No+Negative+Core+Feedback=NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+NOT+message%253A%22Closes-Bug%253A+%22&Wayward+Changes+(Changes+with+no+code+review+in+the+last+5+days) | 16:01 |
| bauwser | abel%253ACode-Review%3C%253D2+NOT+label%253ACode-Review%3C%253D-1%252Cnova-core+age%253A5d | 16:01 |
| bauwser | my gerrit dashboard is just getting me only two specs... | 16:02 |
| bauwser | WTF ? | 16:02 |
| kashyap | Give it a couple of mins to breathe | 16:02 |
| *** suresh12 has joined #openstack-nova | 16:04 | |
| rybridges | mlavalle: I should have the right version though... Everything I am running is just stock stable/ocata built from source. dont even have any patches yet | 16:06 |
| mriedem | giblet: replies in PS17 https://review.openstack.org/#/c/502306/17/ | 16:07 |
| *** jroll has quit IRC | 16:07 | |
| giblet | mriedem: looking | 16:07 |
| *** mriedem is now known as mriedem_afk | 16:09 | |
| giblet | mriedem_afk: good points. I will dig it | 16:09 |
| openstackgerrit | Merged openstack/nova-specs master: Reintroduced nic feature based scheduling for rocky https://review.openstack.org/545951 | 16:10 |
| *** jroll has joined #openstack-nova | 16:10 | |
| *** AlexeyAbashkin has quit IRC | 16:14 | |
| *** josecastroleon has quit IRC | 16:15 | |
| *** jroll has quit IRC | 16:15 | |
| *** rama_y has joined #openstack-nova | 16:16 | |
| *** jroll has joined #openstack-nova | 16:16 | |
| bauwser | leakypipes: just a comment on https://review.openstack.org/#/c/545057/5 preventing a +2, just add your thoughts on error handling and I'm +2 | 16:16 |
| * bauwser disappears for a couple of hours but will be back tonight | 16:16 | |
| *** harlowja has joined #openstack-nova | 16:16 | |
| rybridges | mlavalle: One other thing to note perhaps... When I do openstack aggregate show, I see the ID, but not the uuid in the output. If I want to see the uuid I need to go to the db and look at the aggregates table. | 16:17 |
| mlavalle | rybridges: give me a few minutes and I'll check in my dev system | 16:17 |
| mlavalle | I was going to propose you check with the CLI | 16:17 |
| mlavalle | so you got ahead of me ;-) | 16:17 |
| *** danpawlik has joined #openstack-nova | 16:18 | |
| rybridges | cool thanks! Yea i am taking a look at the CLI now. Even when I run aggregate list with the --debug option i still do not see the uuid from the db anywhere in the response. That is likely related to the problem | 16:19 |
| rybridges | The response body looks like this: {"aggregates": [{"name": "Neutron segment id b9d173e2-fff2-4a33-ac50-312e6f3be914", "availability_zone": null, "deleted": false, "created_at": "2018-03-23T15:59:51.000000", "updated_at": null, "hosts": [], "deleted_at": null, "id": 19, "metadata": {}}]} | 16:20 |
| *** penick is now known as JunoMan | 16:22 | |
| *** danpawlik has quit IRC | 16:23 | |
| *** jmlowe has quit IRC | 16:25 | |
| *** afaranha has quit IRC | 16:26 | |
| *** germs has quit IRC | 16:26 | |
| *** germs has joined #openstack-nova | 16:27 | |
| *** germs has quit IRC | 16:27 | |
| *** germs has joined #openstack-nova | 16:27 | |
| *** ralonsoh has quit IRC | 16:30 | |
| *** josecastroleon has joined #openstack-nova | 16:31 | |
| *** archit has quit IRC | 16:32 | |
| cfriesen | sahid: for what it's worth, I think your short-term proposal to run isolated emulator threads on 'CONF.cpu_shared_set' is reasonable, though I have one question--what would we do if that set is empty? Allocate a new host CPU like we do now and emit a log? | 16:33 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add request filter functionality to scheduler https://review.openstack.org/544730 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add aggregates list to Destination object https://review.openstack.org/544729 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make get_allocation_candidates() honor aggregate restrictions https://review.openstack.org/547990 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add AggregateList.get_by_metadata() query method https://review.openstack.org/544728 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add require_tenant_aggregate request filter https://review.openstack.org/545002 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Honor availability_zone hint via placement https://review.openstack.org/546282 | 16:34 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add an index on aggregate_metadata.value https://review.openstack.org/555851 | 16:34 |
| mlavalle | rybridges: this is what I see in my dev environment, which was built from trunk about a week ago: http://paste.openstack.org/show/710244/ | 16:34 |
| *** fried_rice is now known as fried_rolls | 16:35 | |
| openstackgerrit | Merged openstack/python-novaclient master: Updated from global requirements https://review.openstack.org/555590 | 16:35 |
| *** josecastroleon has quit IRC | 16:36 | |
| leakypipes | giblet: totally cool with me if you publish a new version of the bw spec | 16:37 |
| *** maciejjozefczyk has quit IRC | 16:37 | |
| leakypipes | ktibi: ok, so inventory isn't being created properly on the compute06 for some reason... | 16:37 |
| *** maciejjozefczyk has joined #openstack-nova | 16:37 | |
| leakypipes | ktibi: still trying to figure out why that is. | 16:37 |
| *** maciejjozefczyk has quit IRC | 16:37 | |
| *** tbachman has joined #openstack-nova | 16:38 | |
| giblet | mriedem_afk: I left my view in the reply https://review.openstack.org/#/c/502306/17/specs/rocky/approved/bandwidth-resource-provider.rst@100 | 16:38 |
| ktibi | leakypipes, yes need to clean all compute06 in all DB for works. | 16:38 |
| *** maciejjozefczyk has joined #openstack-nova | 16:38 | |
| ktibi | thx for your time :p | 16:38 |
| leakypipes | np | 16:40 |
| *** kaisers_ has joined #openstack-nova | 16:42 | |
| *** Tom-Tom has joined #openstack-nova | 16:42 | |
| rybridges | mlavalle: I did not build from trunk. I am running stable/ocata. My output looks like this: http://paste.openstack.org/show/710254/ | 16:42 |
| *** maciejjozefczyk has quit IRC | 16:43 | |
| openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Add `hide_hypervisor_id` flavor extra_spec https://review.openstack.org/555861 | 16:43 |
| mlavalle | rybridges: run the command with --debug, so we can see the response. the client might be out of synch with the API | 16:43 |
| *** Tom-Tom has quit IRC | 16:46 | |
| cdent | leakypipes: I suspect my forbidden changes in allocation candidates are going to collide a fair bit with the stuff you are working on for nested, especially since I think both sets of work are going to reveal some necessary changes with how trait handling is desired to work. I don't think it will be dire or anything, just a heads up | 16:46 |
| mlavalle | rybridges: I'll be away from my keyboard for a while. I have to pick up my wife. I'll ping you when I come back | 16:51 |
| *** mdbooth has quit IRC | 16:52 | |
| *** david-lyle has quit IRC | 16:52 | |
| leakypipes | cdent: ack, you are correct. | 16:52 |
| leakypipes | cdent: I should have the next revision of that code (https://review.openstack.org/#/q/status:open+project:openstack/nova+branch:master+topic:bp/nested-resource-providers-allocation-candidates) pushed within the next hour. | 16:53 |
| cdent | cool, my stuff is very spikey at the moment but almost working | 16:53 |
| leakypipes | cdent: just trying to get fried_rolls's update provider tree series reviewed right now | 16:53 |
| leakypipes | cdent: coolio. | 16:53 |
| *** Swami has joined #openstack-nova | 16:54 | |
| *** bnemec is now known as beekneemech | 16:54 | |
| *** yamamoto has joined #openstack-nova | 16:55 | |
| rybridges | mlavalle: Here is debug output http://paste.openstack.org/show/710270/ | 16:55 |
| *** lucasagomes is now known as lucas-afk | 16:57 | |
| *** yamamoto has quit IRC | 17:00 | |
| *** tbachman has quit IRC | 17:01 | |
| *** hemna_ has joined #openstack-nova | 17:01 | |
| cfriesen | bauwser: question for you..if an operator doesn't want to expost a host as NUMA-aware, why not just reboot with "numa=off" on the kernel boot args? | 17:05 |
| *** andreas_s has joined #openstack-nova | 17:06 | |
| *** masber has quit IRC | 17:06 | |
| *** sususuryashines has quit IRC | 17:07 | |
| openstackgerrit | Elod Illes proposed openstack/nova stable/ocata: Functional test for regression bug #1713783 https://review.openstack.org/505160 | 17:09 |
| openstack | bug 1713783 in OpenStack Compute (nova) ocata "After failed evacuation the recovered source compute tries to delete the instance" [High,In progress] https://launchpad.net/bugs/1713783 - Assigned to Balazs Gibizer (balazs-gibizer) | 17:09 |
| *** mriedem_afk has quit IRC | 17:14 | |
| openstackgerrit | Elod Illes proposed openstack/nova master: Functional test: cold migrate to compute down https://review.openstack.org/496280 | 17:15 |
| *** dtantsur is now known as dtantsur|afk | 17:16 | |
| *** mdamkot has joined #openstack-nova | 17:17 | |
| *** vks1 has joined #openstack-nova | 17:18 | |
| *** masber has joined #openstack-nova | 17:18 | |
| *** damien_r has quit IRC | 17:22 | |
| *** pcaruana has quit IRC | 17:23 | |
| *** archit has joined #openstack-nova | 17:24 | |
| *** itlinux has quit IRC | 17:27 | |
| *** ktibi has quit IRC | 17:29 | |
| openstackgerrit | Tyler Blakeslee proposed openstack/nova master: Add __repr__ for NovaException https://review.openstack.org/555812 | 17:29 |
| *** beekneemech has quit IRC | 17:29 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: mirror nova host aggregates to placement API https://review.openstack.org/545057 | 17:30 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 17:32 |
| *** bnemec has joined #openstack-nova | 17:32 | |
| *** suresh12 has quit IRC | 17:32 | |
| *** bnemec is now known as beekneemech | 17:33 | |
| *** beekneemech has quit IRC | 17:34 | |
| *** bnemec has joined #openstack-nova | 17:34 | |
| giblet | mriedem, leakypipes, mlavalle, rubasov: I've published a new rev from the bandwidth spec covering two new issues 1) simulating neutron backend selection with a new scheduler weigher 2) handling the mapping between physical resource consumption and the resource claim https://review.openstack.org/#/c/502306 | 17:35 |
| sahid | cfriesen: is the set is empty so we use the default behavior of hw:emulator_threads_policy="share" | 17:35 |
| *** maciejjozefczyk has joined #openstack-nova | 17:35 | |
| sahid | i will update the spec for monday | 17:35 |
| *** suresh12 has joined #openstack-nova | 17:36 | |
| *** suresh12 has quit IRC | 17:36 | |
| *** suresh12 has joined #openstack-nova | 17:36 | |
| *** suresh12 has quit IRC | 17:36 | |
| *** mdamkot has quit IRC | 17:36 | |
| *** masber has quit IRC | 17:37 | |
| *** maciejjozefczyk has quit IRC | 17:39 | |
| *** gjayavelu has joined #openstack-nova | 17:39 | |
| *** mgoddard has quit IRC | 17:40 | |
| *** mriedem has joined #openstack-nova | 17:41 | |
| leakypipes | giblet: reviewing it as we speak... | 17:41 |
| mriedem | rybridges: aggregate uuids were in a later microversion, openstack CLI defaults to 2.1, so if you want the uuid in the openstack aggregate show output, you need to opt into the microversion | 17:42 |
| mriedem | openstack aggregate list --openstack-compute-api-version 2.41 | 17:42 |
| mriedem | something like that | 17:42 |
| mriedem | --os-compute-api-version i mean | 17:43 |
| *** jpena is now known as jpena|off | 17:43 | |
| *** psachin has joined #openstack-nova | 17:44 | |
| rybridges | ok thanks mriedem. I can try that. Is there a way to tell nova to use that microversion on the backend as well? That may work for me as a user on the CLI, but the backend also needs to be able to use that microversion as well | 17:45 |
| *** derekh has quit IRC | 17:46 | |
| jgwentworth | maybe some progress has been made but yesterday rybridges said there weren't any hosts in the nova_api.aggregate_hosts table, proving that the request to add the host to the aggregate never made it there | 17:48 |
| mriedem | rybridges: i don't understand your question | 17:48 |
| mriedem | what is "the backend"? | 17:48 |
| jgwentworth | so not a microversion showing the uuids issue | 17:48 |
| openstackgerrit | Merged openstack/nova master: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/520248 | 17:49 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/555418 | 17:49 |
| *** harlowja has quit IRC | 17:50 | |
| mriedem | rybridges: specifying --os-compute-api-version 2.x on the CLI passes that in a header to the compute REST API, which then processes that single request with that microversion | 17:50 |
| *** sahid has quit IRC | 17:51 | |
| rybridges | mriedem: What I mean is.. I need neutron to also use the correct microversion when it creates aggregates. Like in this call: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/plugin.py#L224 | 17:52 |
| *** vks1 has quit IRC | 17:52 | |
| rybridges | I realize i can pass the microversion manually when I am running on the CLI, but how does neutron know to use the right microversion when it needs to create aggregates on my behalf? | 17:52 |
| rybridges | I am guessing there is some configuration somewhere in neutron.conf or nova.conf where I can specify the microversion? | 17:53 |
| mriedem | neutron doesn't have microversions | 17:53 |
| *** AlexeyAbashkin has joined #openstack-nova | 17:53 | |
| mriedem | and nova aggregates don't have anything to do with neutron | 17:53 |
| mriedem | nova doesn't pass a microversion through it's compute API to neutron | 17:53 |
| jgwentworth | you can pass microversions with the novaclient python bindings, so the neutron plugin could do that, I think | 17:53 |
| mriedem | microversions in config files isn't a thing | 17:53 |
| mriedem | neutron is a client, | 17:54 |
| jgwentworth | he's talking about some kinda neutron plugin that creates nova aggregates and adds hosts to aggregates when using the routed provider network segments stuff | 17:54 |
| mriedem | of the placement API (and compute API?), | 17:54 |
| mriedem | so neutron, as a client, will need to specify a microversion when it makes its own requests | 17:54 |
| mriedem | if they require some minimum microversion | 17:54 |
| mriedem | this is the same thing as cinder calling the compute os-server-external-events API with a microversion to pass volume-extended events | 17:55 |
| mriedem | https://github.com/openstack/cinder/blob/master/cinder/compute/nova.py#L207 | 17:55 |
| *** felipemonteiro has joined #openstack-nova | 17:55 | |
| *** yamamoto has joined #openstack-nova | 17:56 | |
| rybridges | ya jgwentworth gets what I am doing. So in my case, the openstack network subnet create command is creating the aggregate for me. I never ask nova explicitly myself to do it, it happens in the backend as part of the subnet create command. So I need to pass the microversion into the subnet create command maybe.. I can try that | 17:56 |
| *** yamahata has quit IRC | 17:57 | |
| *** AlexeyAbashkin has quit IRC | 17:57 | |
| jgwentworth | rybridges: here is what I was talking about, when the client object is instantiated, a microversion can be specified, but I think there's also a way to specify per call https://docs.openstack.org/python-novaclient/pike/reference/api/index.html#usage | 17:57 |
| jgwentworth | not seeing that in this doc tho | 17:58 |
| *** AlexeyAbashkin has joined #openstack-nova | 17:58 | |
| *** david-lyle has joined #openstack-nova | 17:59 | |
| *** jmlowe has joined #openstack-nova | 18:00 | |
| jgwentworth | actually, not seeing a way to specify a microversion per-call in the novaclient python API | 18:01 |
| *** yamamoto has quit IRC | 18:02 | |
| mriedem | fried_rolls: hopefully i've word-smithed this well enough https://review.openstack.org/#/c/555481/ | 18:02 |
| mriedem | rybridges: you should not have to pass through a compute api version to neutron | 18:03 |
| mriedem | if neutron is going to create aggregates in nova, and needs the uuid, then it must use at least microversion 2.41 | 18:03 |
| *** AlexeyAbashkin has quit IRC | 18:03 | |
| mriedem | else it's a bug | 18:03 |
| mriedem | there is not a way to specify a microversion per call in novaclient's python binding API, as it's stored on the client object when it's constructed, | 18:04 |
| mriedem | you could probably mutate the internals of the client object, but we don't have a supported hook for doing that | 18:04 |
| rybridges | okay thanks mriedem thtat is helpful to know | 18:04 |
| *** Tom-Tom has joined #openstack-nova | 18:05 | |
| rybridges | The uuid is not being returned by the python novaclient when an aggregate is created from neutron, that i know for sure | 18:05 |
| leakypipes | giblet: small request in review. | 18:05 |
| mriedem | where in the neutron code does neutron create a nova host aggregate? | 18:06 |
| rybridges | so maybe this code in neutron that instantiates the client is not correct and/or is not instantiating the python novaclient with the correct microversion | 18:06 |
| mriedem | https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/placement_client.py#L163 is NOT nova | 18:06 |
| mriedem | placement aggregates != nova host aggregates | 18:06 |
| mriedem | completely different REST APIs | 18:06 |
| rybridges | right here: https://github.com/openstack/neutron/blob/master/neutron/services/segments/plugin.py#L221 | 18:06 |
| rybridges | this is going thru nova client, not placement client | 18:06 |
| rybridges | to create the aggregate | 18:06 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 18:07 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: resource requests for nested providers https://review.openstack.org/554529 | 18:07 |
| *** oomichi has joined #openstack-nova | 18:07 | |
| leakypipes | fried_rolls: ok, ready for your re-re-re-review ^^ | 18:07 |
| leakypipes | figleaf: you too :) | 18:07 |
| mriedem | rybridges: https://github.com/openstack/neutron/blob/master/neutron/services/segments/plugin.py#L48 | 18:07 |
| mriedem | novaclient is created with 2.41 | 18:07 |
| mriedem | so it's using 2.41 | 18:07 |
| rybridges | heh yup | 18:08 |
| rybridges | thats pretty strange | 18:08 |
| rybridges | doesnt make sense why the uuid is not returned. i cannot print it. and any call that references it gets swallowed | 18:08 |
| mriedem | the POST /os-aggregates API ref says that uuid is returned https://developer.openstack.org/api-ref/compute/#create-aggregate | 18:09 |
| mriedem | so which version of novaclient are you using? | 18:09 |
| *** Tom-Tom has quit IRC | 18:09 | |
| mriedem | 7.1.2? https://github.com/openstack/requirements/blob/stable/ocata/upper-constraints.txt#L332 | 18:09 |
| rybridges | 7.1.3 looks like | 18:10 |
| mriedem | anyway, this is the object you get in novaclient https://github.com/openstack/python-novaclient/blob/7.1.2/novaclient/v2/aggregates.py#L21 | 18:10 |
| mriedem | it's just an object that extends dict and has the POST response body as attributes | 18:10 |
| mriedem | so 'uuid' should be in there | 18:10 |
| *** kaisers_ has quit IRC | 18:11 | |
| artom | cfriesen, sean-k-mooney[m], thanks for the reviews guys. Busy with some downstream stuff, so not as responsive as I'd like | 18:11 |
| artom | Can we settle on the scope of the spec though? Specifically, include or not PCI stuff | 18:12 |
| mriedem | this is the code that returns the aggregate response body https://github.com/openstack/nova/blob/stable/ocata/nova/api/openstack/compute/aggregates.py#L204 | 18:12 |
| artom | If we go with EPA as sean-k-mooney[m]'s saying, we'd kinda have to, but I'm worried about scope creep, and actually finishing it this cycle | 18:12 |
| mriedem | and i know that works because otherwise this response, used in our testing for 2.41, would fail https://github.com/openstack/nova/blob/stable/ocata/doc/api_samples/os-aggregates/v2.41/aggregate-post-resp.json | 18:13 |
| rybridges | Yea that makes sense mriedem. I realize things would break in tests if the uuid actually wasnt being returned. I did just see something interested though: http://paste.openstack.org/show/710369/ | 18:14 |
| rybridges | Looks like maybe my nova backend does not support microversions for some reason. Perhaps that is the problem. I cannot instantiate a client with a microversion at all | 18:14 |
| mriedem | first, you don't need to specify a specific microversion on the nova CLI since the nova CLI opts into, by default, the latest available between the server and client | 18:15 |
| *** maciejjozefczyk has joined #openstack-nova | 18:15 | |
| mriedem | as for that 'does not support microversions' stuff, idk what you guys do to hack things up for your own api extensions | 18:15 |
| *** tbachman has joined #openstack-nova | 18:15 | |
| *** slagle has quit IRC | 18:15 | |
| rybridges | we have 0 patches in the environment i am using right now | 18:15 |
| rybridges | 0 | 18:16 |
| *** sambetts is now known as sambetts|afk | 18:16 | |
| rybridges | jgwentworth said earlier that i can pass the microversion into the backend as an argument so i figured i would try it. thats all. | 18:16 |
| mriedem | "pass the microversion into the backend" isn't a thing | 18:16 |
| mriedem | just do a simple curl request to your compute endpoint URL | 18:17 |
| mriedem | that will give you the version document | 18:17 |
| mriedem | and https://github.com/openstack/python-novaclient/blob/5af4a8edd88187e0a21c665a6e91fcdcd8ced06c/novaclient/api_versions.py#L265 is complaining that there is no min/max version in your compute endpoint version doc | 18:18 |
| jgwentworth | I think we're just using different words. but in the novaclient python API bindings, you *can* specify a microversion when the client object is created, in fact, I think you have to. if it allows you not to, then it will default to the lowest/base microversion | 18:18 |
| cfriesen | artom: no worries, I'm in the same boat | 18:18 |
| mriedem | jgwentworth: yes and as i pointed out above, neutron is creating novaclient with version 2.41 | 18:19 |
| cfriesen | artom: leave PCI out for now | 18:19 |
| mriedem | hard-coded | 18:19 |
| jgwentworth | I was mistaken that there was some way to specify microversion *per call* after instantiating the client object. I am not seeing a way to do that | 18:19 |
| jgwentworth | okay, so that is taken care of. so rybridges that code is calling with 2.41 already | 18:19 |
| cfriesen | artom: there are additional complications for PCI that merit a separate spec | 18:19 |
| mriedem | the novaclient python API bindings default to 2.1, the CLI defaults to the latest available between the server and what the client understands | 18:19 |
| jgwentworth | right | 18:19 |
| *** suresh12 has joined #openstack-nova | 18:20 | |
| artom | cfriesen, cool, good to know you'll back be on that :) | 18:20 |
| artom | *back me | 18:20 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Support for forbidden traits in placement https://review.openstack.org/554665 | 18:20 |
| mriedem | fried_rolls: want to backport https://review.openstack.org/#/c/554759/ to queens? | 18:20 |
| *** maciejjozefczyk has quit IRC | 18:21 | |
| rybridges | Ok so if the CLI defaults to the latest version that the server understands, then my server does not understand 2.41. If it did, then I would see the uuid in the response body of this call: http://paste.openstack.org/show/710386/ | 18:21 |
| rybridges | either that or its not defaulting to the highest version available but rather the lowest base | 18:23 |
| jgwentworth | you're running ocata, yes? you should have it https://docs.openstack.org/nova/latest/reference/api-microversion-history.html#id37 | 18:23 |
| mriedem | {"version": {"status": "SUPPORTED", "updated": "2011-01-21T11:33:21Z", "links": [{"href": "http://XXX:8774/v2/", "rel": "self"}, {"href": "http://docs.openstack.org/", "type": "text/html", "rel": "describedby"}], "min_version": "", "version": "", "media-types": [{"base": "application/json", "type": "application/vnd.openstack.compute+json;version=2"}], "id": "v2.0"}} | 18:23 |
| mriedem | that's your server's compute endpoint url version doc | 18:23 |
| mriedem | note there is no min or version | 18:23 |
| mriedem | version == max | 18:23 |
| mriedem | so yeah, your cloud is f'ed | 18:23 |
| jgwentworth | yeah ... you have ocata right rybridges? | 18:24 |
| rybridges | yes i am running stable/ocata | 18:24 |
| jgwentworth | hm, wtf | 18:24 |
| superdan | this seems like #openstack fodder, no? | 18:24 |
| mriedem | after 2 days of debugging this issue, yes | 18:24 |
| mriedem | this is BASIC stuff | 18:24 |
| superdan | especially if this is just installation basically | 18:25 |
| superdan | yeah | 18:25 |
| mriedem | where are sean-k-mooney[m] and cfriesen around when i need to flood this channel with placement and NFV talk? | 18:25 |
| *** sdeath has joined #openstack-nova | 18:25 | |
| figleaf | leakypipes: ack. Will re-review in a few | 18:26 |
| *** psachin has quit IRC | 18:26 | |
| *** sree has quit IRC | 18:26 | |
| *** sree has joined #openstack-nova | 18:27 | |
| *** tbachman has quit IRC | 18:27 | |
| *** yamahata has joined #openstack-nova | 18:29 | |
| cdent | mriedem, superdan : if you like less code, the requirements dependency for removing microversion stuff in placement to a library has been improved: https://review.openstack.org/#/c/550265/ | 18:30 |
| cdent | sigh: s/improved/approved/ | 18:30 |
| mriedem | i like more code on fridays | 18:30 |
| openstackgerrit | Merged openstack/nova master: trivial: omit condition evaluations https://review.openstack.org/545248 | 18:31 |
| openstackgerrit | Merged openstack/nova master: Stop using mox in virt/xenapi/image/test_vdi_through_dev.py https://review.openstack.org/342682 | 18:31 |
| cdent | mriedem: you never were any fun | 18:31 |
| *** lajoskatona has quit IRC | 18:31 | |
| *** andreas_s has quit IRC | 18:31 | |
| *** sree has quit IRC | 18:31 | |
| *** maciejjozefczyk has joined #openstack-nova | 18:32 | |
| *** itlinux has joined #openstack-nova | 18:33 | |
| *** eharney has quit IRC | 18:35 | |
| *** tbachman has joined #openstack-nova | 18:36 | |
| cdent | leakypipes you might also like https://review.openstack.org/#/c/550265/ <- placement using microversion-parse 0.2.1 | 18:36 |
| *** chyka has quit IRC | 18:37 | |
| *** r-daneel has joined #openstack-nova | 18:37 | |
| *** chyka has joined #openstack-nova | 18:37 | |
| *** chyka_ has joined #openstack-nova | 18:41 | |
| *** chyka has quit IRC | 18:42 | |
| *** tssurya has joined #openstack-nova | 18:45 | |
| *** pchavva1 has joined #openstack-nova | 18:45 | |
| openstackgerrit | Eric Fried proposed openstack/nova stable/queens: Change compute mgr placement check to region_name https://review.openstack.org/555901 | 18:47 |
| *** andreas_s has joined #openstack-nova | 18:47 | |
| fried_rolls | mriedem: Done ^ | 18:47 |
| fried_rolls | leakypipes: ack | 18:47 |
| *** itlinux has quit IRC | 18:47 | |
| *** maciejjozefczyk has quit IRC | 18:47 | |
| *** lajoskatona has joined #openstack-nova | 18:47 | |
| *** jmlowe has quit IRC | 18:47 | |
| mriedem | thanks | 18:48 |
| jgwentworth | indeed looks like it's an installation problem. from the novaclient --debug output, it's talking to the /v2 nova endpoint which doesn't do microversions. it's supposed to be /v2.1 (in the service catalog) | 18:48 |
| cdent | fried_rolls: wasn't half the bug there that people should have to express either region_name or os_region_name if they are using the default? not just that they were using the wrong one? | 18:48 |
| *** itlinux has joined #openstack-nova | 18:49 | |
| fried_rolls | cdent: The bug was that, if you specified os_region_name, you got a deprecation warning, but it worked. But if you specified region_name, it just didn't work at all. | 18:49 |
| fried_rolls | but yeah, pretty sure you have to specify one or the other. | 18:50 |
| fried_rolls | which... is kind of a pity, really. | 18:50 |
| cdent | that seems wrong | 18:50 |
| *** r-daneel_ has joined #openstack-nova | 18:50 | |
| *** andreas_s has quit IRC | 18:51 | |
| *** r-daneel has quit IRC | 18:52 | |
| *** r-daneel_ is now known as r-daneel | 18:52 | |
| figleaf | leakypipes: can you explain what's going on in https://review.openstack.org/#/c/554529/2/nova/api/openstack/placement/objects/resource_provider.py@3055 ? | 18:53 |
| *** dave-mcc_ has quit IRC | 18:55 | |
| figleaf | leakypipes: oh, never mind. I think I got it | 18:56 |
| *** yamamoto has joined #openstack-nova | 18:58 | |
| sean-k-mooney[m] | mriedem: today on vacation | 19:02 |
| JunoMan | Sorry for wasting your time with the microversion confusion y'all. We missed that in the kilo release notes. | 19:02 |
| *** yamamoto has quit IRC | 19:02 | |
| mriedem | sean-k-mooney[m]: was just joking, probably missed the context | 19:03 |
| mriedem | ignore | 19:03 |
| sean-k-mooney[m] | mriedem: also after yesterday i think i need to not think about numa/nfv for at least a day or two | 19:03 |
| mriedem | superdan: the reader/writer fixes are ready to go imo https://review.openstack.org/#/q/topic:bug/1746509+(status:open+OR+status:merged) | 19:05 |
| sean-k-mooney[m] | mriedem: hehe ya i know. i may have installed riot.im on my phone and now get irc routed too me now. it has its pros and cons | 19:06 |
| jroll | JunoMan: excellent friday nick | 19:06 |
| *** itlinux has quit IRC | 19:08 | |
| *** lpetrut_ has quit IRC | 19:08 | |
| superdan | mriedem: okay will look in a few | 19:09 |
| *** harlowja has joined #openstack-nova | 19:09 | |
| openstackgerrit | Julia Kreger proposed openstack/nova master: WIP: Add microversion to ironic client wrapper call https://review.openstack.org/554762 | 19:11 |
| jgwentworth | will superdan strike down my code comment sentence? time will tell | 19:11 |
| superdan | sources from beyond the grave say.....PLAUSIBLE | 19:12 |
| jgwentworth | lol | 19:12 |
| *** tssurya has quit IRC | 19:13 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:14 | |
| *** itlinux has joined #openstack-nova | 19:15 | |
| *** hemna_ has quit IRC | 19:15 | |
| *** itlinux has quit IRC | 19:16 | |
| *** lajoskatona has quit IRC | 19:17 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM Driver: vSCSI volume driver https://review.openstack.org/526094 | 19:18 |
| *** AlexeyAbashkin has quit IRC | 19:19 | |
| *** tssurya has joined #openstack-nova | 19:22 | |
| *** felipemonteiro__ has joined #openstack-nova | 19:23 | |
| *** vivsoni__ has joined #openstack-nova | 19:23 | |
| *** suresh12 has quit IRC | 19:25 | |
| *** felipemonteiro has quit IRC | 19:26 | |
| *** vivsoni_ has quit IRC | 19:26 | |
| *** bkopilov has quit IRC | 19:27 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Always deallocate networking before reschedule if using Neutron https://review.openstack.org/555907 | 19:28 |
| *** avolkov has quit IRC | 19:28 | |
| *** bkopilov has joined #openstack-nova | 19:31 | |
| *** openstackgerrit has quit IRC | 19:34 | |
| *** suresh12 has joined #openstack-nova | 19:38 | |
| *** fried_rolls is now known as fried_rice | 19:40 | |
| *** jmlowe has joined #openstack-nova | 19:43 | |
| *** openstackgerrit has joined #openstack-nova | 19:45 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/555576 | 19:45 |
| *** eharney has joined #openstack-nova | 19:49 | |
| *** fragatina has quit IRC | 19:52 | |
| *** lpetrut has joined #openstack-nova | 19:53 | |
| *** moshele has joined #openstack-nova | 19:55 | |
| *** yamamoto has joined #openstack-nova | 19:58 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/queens: Add functional regression test for bug 1746509 https://review.openstack.org/555914 | 20:02 |
| openstack | bug 1746509 in OpenStack Compute (nova) "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" [Medium,In progress] https://launchpad.net/bugs/1746509 - Assigned to melanie witt (melwitt) | 20:02 |
| openstackgerrit | melanie witt proposed openstack/nova stable/queens: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555915 | 20:02 |
| *** gouthamr has quit IRC | 20:03 | |
| *** patriciadomin has joined #openstack-nova | 20:03 | |
| *** sidx64 has joined #openstack-nova | 20:04 | |
| *** yamamoto has quit IRC | 20:04 | |
| *** elmaciej has quit IRC | 20:05 | |
| openstackgerrit | melanie witt proposed openstack/nova stable/pike: Add functional regression test for bug 1746509 https://review.openstack.org/555058 | 20:06 |
| openstack | bug 1746509 in OpenStack Compute (nova) "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" [Medium,In progress] https://launchpad.net/bugs/1746509 - Assigned to melanie witt (melwitt) | 20:06 |
| openstackgerrit | melanie witt proposed openstack/nova stable/pike: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555923 | 20:06 |
| openstackgerrit | Konstantinos Samaras-Tsakiris proposed openstack/nova master: Add `hide_hypervisor_id` flavor extra_spec https://review.openstack.org/555861 | 20:06 |
| *** itlinux has joined #openstack-nova | 20:07 | |
| *** hemna_ has joined #openstack-nova | 20:07 | |
| *** itlinux has quit IRC | 20:09 | |
| *** jroll has quit IRC | 20:09 | |
| *** tesseract has quit IRC | 20:09 | |
| *** jroll has joined #openstack-nova | 20:10 | |
| *** r-daneel has quit IRC | 20:11 | |
| *** pchavva1 has quit IRC | 20:13 | |
| mlavalle | rybridges: yeah, you are not getting the uuid attribute from the API. If you look at the response in http://paste.openstack.org/show/710244/, I get a header with the API version: X-OpenStack-Nova-API-Version: 2.60. Yours doesn't even have that header | 20:14 |
| *** sidx64 has quit IRC | 20:14 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:14 | |
| jgwentworth | mlavalle: yeah, I think they have a wrong (old) url for nova in their service catalog, /v2 (does not support microversions) where it should be /v2.1 | 20:15 |
| jroll | yep | 20:17 |
| *** AlexeyAbashkin has quit IRC | 20:18 | |
| *** dave-mccowan has joined #openstack-nova | 20:19 | |
| *** yassine has quit IRC | 20:19 | |
| *** sidx64 has joined #openstack-nova | 20:20 | |
| *** pchavva has quit IRC | 20:20 | |
| *** yassine has joined #openstack-nova | 20:25 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add disabled option to create_cell command https://review.openstack.org/555417 | 20:26 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Modify nova-manage cell_v2 list_cells to display "disabled" column https://review.openstack.org/555415 | 20:26 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Add --enable and --disable options to nova-manage update_cell https://review.openstack.org/555416 | 20:27 |
| *** r-daneel has joined #openstack-nova | 20:29 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | 20:30 |
| *** jmlowe has quit IRC | 20:31 | |
| jgwentworth | fried_rice, superdan: I was looking at this review and it says that the powervm driver capabilities will be dynamic but I notice none of the other virt drivers have dynamic capabilities. would that not be problematic at all if the driver can change its capabilities on-the-fly? as far as the end user's experience/expectation? https://review.openstack.org/#/c/547169/3/nova/virt/powervm/driver.py@55 | 20:32 |
| *** _ix has joined #openstack-nova | 20:33 | |
| jgwentworth | because I thought usually we advertise those things through the hypervisor matrix doc | 20:34 |
| *** cdent has quit IRC | 20:35 | |
| superdan | jgwentworth: I don't think it would be a problem necessarily, | 20:37 |
| superdan | if we're updating traits in placement | 20:37 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | 20:37 |
| superdan | you restart a compute, it has a new libvirt version, exposes some new feature, tells placement, and suddenly things can boot there needing that feature | 20:37 |
| jgwentworth | okay, I see | 20:37 |
| superdan | most of our capabilities have been rather static thus far, but that's just because they're like "have they implemented resize yet" sort of things I think | 20:38 |
| *** sidx64_ has joined #openstack-nova | 20:38 | |
| jgwentworth | thanks | 20:40 |
| *** sidx64 has quit IRC | 20:41 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: Resize https://review.openstack.org/553583 | 20:47 |
| *** jmlowe has joined #openstack-nova | 20:52 | |
| *** awaugama has quit IRC | 20:52 | |
| *** danpawlik has joined #openstack-nova | 20:52 | |
| *** danpawlik has quit IRC | 20:54 | |
| *** yassine has quit IRC | 20:56 | |
| *** yamamoto has joined #openstack-nova | 21:00 | |
| *** yamamoto has quit IRC | 21:06 | |
| fried_rice | jgwentworth: Yuh, what superdan said. In this case, it's "dynamic" only per start of the compute service, based on which disk driver you picked. | 21:10 |
| *** fragatina has joined #openstack-nova | 21:11 | |
| openstackgerrit | Julia Kreger proposed openstack/nova master: Add microversion to ironic client wrapper call https://review.openstack.org/554762 | 21:11 |
| jgwentworth | fried_rice: ack | 21:12 |
| *** hongbin has quit IRC | 21:14 | |
| openstackgerrit | Merged openstack/nova master: Add functional regression test for bug 1746509 https://review.openstack.org/555092 | 21:15 |
| openstack | bug 1746509 in OpenStack Compute (nova) pike "TypeError: Can't upgrade a READER transaction to a WRITER mid-transaction" [High,In progress] https://launchpad.net/bugs/1746509 - Assigned to melanie witt (melwitt) | 21:15 |
| *** edmondsw has quit IRC | 21:17 | |
| rybridges | mlavalle: jgwentworth: Ya it was the URL. The problem is there are no errors anywhere. The logs do not warn us about the fact that it cannot instantiate a client with a microversion b/c the url is not pointing to 2.1 (like it would on the CLI). Also there is no error or stack trace when we try to reference aggregate.uuid which does not exist. Everything just silently fails which makes it extremely | 21:20 |
| rybridges | difficult to debug. We thought it was a bug at first because this was really strange behavior | 21:20 |
| *** moshele has quit IRC | 21:21 | |
| jgwentworth | rybridges: aggregate.uuid coming back from the novaclient python bindings? yeah, I suspect that no one has run into this + proposed novaclient patches to enhance the warnings/errors | 21:23 |
| *** lpetrut has quit IRC | 21:26 | |
| mriedem | presumably there are errors in the neutron logs | 21:27 |
| mriedem | which is the client in this case | 21:27 |
| *** esberglu has quit IRC | 21:28 | |
| mriedem | calling aggregate.uuid on the Aggregate object from novaclient would result in an AttributeError | 21:28 |
| mriedem | and all of that should be barfed into the neutron logs | 21:28 |
| openstackgerrit | Merged openstack/nova master: Move _make_instance_list call outside of DB transaction context https://review.openstack.org/555093 | 21:29 |
| mlavalle | rybridges: agree with jgwentworth, it is a bug. My assumption when writing the code was that the following was going to take care of the API version: https://github.com/openstack/neutron/blob/stable/ocata/neutron/services/segments/plugin.py#L159 | 21:29 |
| *** lpetrut has joined #openstack-nova | 21:30 | |
| *** bnemec is now known as beekneemech | 21:32 | |
| rybridges | mriedem: Attribute errors were not printed at all in our logs. No warnings either. Nothing. I was tailing nova-api, placement, neutron server and all of the agent logs on our hypervisors and at no point was there ever any error or anything. That is why I came here for help. If there had been anything in the logs I would have been able to solve it pretty easily i think. | 21:32 |
| jgwentworth | holy crap, I think that's the first time my patches have ever merged without CI issues | 21:32 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Allow scheduling only to enabled cells (Filter Scheduler) https://review.openstack.org/550527 | 21:33 |
| *** sidx64_ has quit IRC | 21:35 | |
| *** wolverineav has joined #openstack-nova | 21:36 | |
| mriedem | rybridges: ok, i don't know how that's possible, because http://paste.openstack.org/show/710652/ | 21:42 |
| mriedem | anywho | 21:43 |
| openstackgerrit | melanie witt proposed openstack/nova master: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/555945 | 21:43 |
| openstackgerrit | Merged openstack/nova master: trivial: Resolve 'oslo.context' deprecation warnings https://review.openstack.org/532822 | 21:47 |
| openstackgerrit | melanie witt proposed openstack/nova master: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/555945 | 21:55 |
| *** dave-mccowan has quit IRC | 21:57 | |
| *** burt has quit IRC | 22:00 | |
| *** _ix has quit IRC | 22:01 | |
| *** _ix has joined #openstack-nova | 22:02 | |
| *** yamamoto has joined #openstack-nova | 22:02 | |
| *** gouthamr has joined #openstack-nova | 22:03 | |
| jgwentworth | mriedem: queens backport is ready2rock https://review.openstack.org/#/c/555914 | 22:04 |
| *** yamamoto has quit IRC | 22:08 | |
| *** tssurya has quit IRC | 22:10 | |
| *** tssurya has joined #openstack-nova | 22:12 | |
| *** tssurya has quit IRC | 22:12 | |
| *** tssurya has joined #openstack-nova | 22:12 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Teardown networking when rolling back live migration even if shared disk https://review.openstack.org/555481 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DRY up test_rollback_live_migration_set_migration_status https://review.openstack.org/555489 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add code to bind a port against a dest host during live migration https://review.openstack.org/523604 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Port binding based on events during live migration https://review.openstack.org/434870 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | 22:24 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 22:24 |
| *** liverpooler has quit IRC | 22:32 | |
| mriedem | consider them rocked | 22:33 |
| *** figleaf is now known as edleafe | 22:34 | |
| *** EmilienM has quit IRC | 22:34 | |
| jgwentworth | \o/ | 22:35 |
| *** EmilienM has joined #openstack-nova | 22:36 | |
| *** slagle has joined #openstack-nova | 22:37 | |
| * fried_rice has found superdan patches to -1^W review | 22:41 | |
| *** r-daneel has quit IRC | 22:43 | |
| jgwentworth | my patch to move the live migration job in-tree failed the in-tree job but not the legacy old same job \:| | 22:44 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/555949 | 22:44 |
| jgwentworth | NoValidHost | 22:44 |
| *** chyka_ has quit IRC | 22:44 | |
| *** rama_y has quit IRC | 22:46 | |
| jgwentworth | Host filter ignoring hosts: ubuntu-xenial-inap-mtl01-0003165879 | 22:50 |
| jgwentworth | hmm | 22:50 |
| mriedem | jgwentworth: add me and i can look on monday | 22:52 |
| *** mriedem is now known as mriedem_afk | 22:52 | |
| jgwentworth | thank ye | 22:52 |
| *** david-lyle has quit IRC | 22:52 | |
| *** rama_y has joined #openstack-nova | 22:55 | |
| *** _ix has quit IRC | 22:55 | |
| *** gouthamr has quit IRC | 23:03 | |
| *** yamamoto has joined #openstack-nova | 23:04 | |
| *** lpetrut has quit IRC | 23:05 | |
| *** claudiub has quit IRC | 23:08 | |
| *** artom has quit IRC | 23:08 | |
| Spazmotic | Morning folks | 23:09 |
| *** yassine has joined #openstack-nova | 23:09 | |
| *** yamamoto has quit IRC | 23:10 | |
| * fried_rice waves at Spazmotic | 23:14 | |
| fried_rice | Is it... Saturday morning for you? | 23:14 |
| Spazmotic | Friday Evening.. Night shift hours :) | 23:14 |
| fried_rice | mm, UGT morning, mahbad. | 23:14 |
| Spazmotic | I wake up just in time to watch everyone head off for the weekend and I guard this room all weekend :p | 23:15 |
| *** yamahata has quit IRC | 23:16 | |
| *** tssurya has quit IRC | 23:20 | |
| *** suresh12 has quit IRC | 23:23 | |
| *** wolverineav has quit IRC | 23:24 | |
| *** wolverineav has joined #openstack-nova | 23:24 | |
| *** wolverineav has joined #openstack-nova | 23:25 | |
| *** suresh12 has joined #openstack-nova | 23:27 | |
| *** wolverineav has quit IRC | 23:30 | |
| *** wolverineav has joined #openstack-nova | 23:32 | |
| *** liusheng has quit IRC | 23:32 | |
| *** liusheng has joined #openstack-nova | 23:33 | |
| *** david-lyle has joined #openstack-nova | 23:50 | |
| *** felipemonteiro__ has quit IRC | 23:52 | |
| *** sdague has quit IRC | 23:53 | |
| *** jgwentworth is now known as melwitt | 23:54 | |
| *** mriedem_afk has quit IRC | 23:56 | |
| *** Swami has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!