| *** amodi has quit IRC | 00:11 | |
| *** liverpooler has quit IRC | 00:21 | |
| *** rcernin has joined #openstack-nova | 00:21 | |
| *** AlexeyAbashkin has joined #openstack-nova | 00:32 | |
| *** odyssey4me has quit IRC | 00:34 | |
| *** odyssey4me has joined #openstack-nova | 00:35 | |
| *** AlexeyAbashkin has quit IRC | 00:36 | |
| *** hshiina has joined #openstack-nova | 00:37 | |
| artom_ | efried, if you're still around, I didn't want to pollute the mailing list with that's essentially just a "+1", but that update_provider_tree design update that you sent was really nice | 00:43 |
|---|---|---|
| artom_ | For what it's worth, my opinion is that the precedent that this sets should be valued and repeated | 00:44 |
| Spaz-Home | Morning folks | 00:47 |
| *** harlowja has quit IRC | 00:49 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:49 | |
| *** liuzz has joined #openstack-nova | 00:50 | |
| *** wolverineav has quit IRC | 00:54 | |
| *** salv-orlando has joined #openstack-nova | 00:55 | |
| *** hamzy__ has joined #openstack-nova | 00:57 | |
| *** salv-orlando has quit IRC | 01:00 | |
| *** zhaochao has joined #openstack-nova | 01:02 | |
| *** david-lyle has joined #openstack-nova | 01:02 | |
| *** phuongnh has joined #openstack-nova | 01:03 | |
| *** gjayavelu has quit IRC | 01:08 | |
| *** itlinux has joined #openstack-nova | 01:13 | |
| *** sdague has quit IRC | 01:14 | |
| *** dtruong_ has joined #openstack-nova | 01:14 | |
| *** wolverineav has joined #openstack-nova | 01:16 | |
| *** tiendc has joined #openstack-nova | 01:17 | |
| *** tiendc has quit IRC | 01:17 | |
| *** dtruong has quit IRC | 01:17 | |
| *** tiendc has joined #openstack-nova | 01:18 | |
| *** jichen has joined #openstack-nova | 01:20 | |
| *** wolverineav has quit IRC | 01:21 | |
| *** wolverineav has joined #openstack-nova | 01:22 | |
| yikun | mriedem, OK, thanks. : ) | 01:24 |
| *** wolverineav has quit IRC | 01:26 | |
| *** phuongnh has quit IRC | 01:32 | |
| *** phuongnh has joined #openstack-nova | 01:33 | |
| *** yamahata has quit IRC | 01:34 | |
| *** chyka has joined #openstack-nova | 01:34 | |
| *** chyka has quit IRC | 01:39 | |
| *** suresh12 has quit IRC | 01:39 | |
| *** yufei has joined #openstack-nova | 01:41 | |
| *** gongysh has joined #openstack-nova | 01:44 | |
| *** suresh12 has joined #openstack-nova | 01:46 | |
| *** Kevin_Zheng_ is now known as Kevin_Zheng | 01:50 | |
| *** AlexeyAbashkin has joined #openstack-nova | 01:52 | |
| *** salv-orlando has joined #openstack-nova | 01:56 | |
| *** AlexeyAbashkin has quit IRC | 01:56 | |
| *** phuongnh has quit IRC | 01:58 | |
| *** phuongnh has joined #openstack-nova | 01:58 | |
| *** felipemonteiro has joined #openstack-nova | 01:59 | |
| *** salv-orlando has quit IRC | 02:01 | |
| *** mikal_ has joined #openstack-nova | 02:03 | |
| *** mikal has quit IRC | 02:06 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared and dedicated VMs in one host https://review.openstack.org/543805 | 02:06 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Support shared/dedicated vCPUs in one instance https://review.openstack.org/545734 | 02:06 |
| *** Zames has joined #openstack-nova | 02:08 | |
| *** lei-zh has joined #openstack-nova | 02:10 | |
| *** gongysh has quit IRC | 02:11 | |
| *** Zames has quit IRC | 02:12 | |
| *** suresh12 has quit IRC | 02:19 | |
| mriedem | 享受工作，朋友。 | 02:25 |
| mriedem | o/ | 02:25 |
| *** mriedem has quit IRC | 02:26 | |
| *** tuanla____ has joined #openstack-nova | 02:28 | |
| *** Spazmotic has joined #openstack-nova | 02:28 | |
| Spazmotic | Annnnnd morning from the Office. | 02:28 |
| *** Spaz-Work has quit IRC | 02:29 | |
| *** yufei has quit IRC | 02:29 | |
| *** yufei has joined #openstack-nova | 02:29 | |
| jianghuaw_ | Good morning. | 02:30 |
| jianghuaw_ | Spazmotic, I saw your message on live-migration. | 02:31 |
| jianghuaw_ | that's expected behavior to get the exception when block_migration is False but it's not shared storage. | 02:32 |
| jianghuaw_ | Look this: https://developer.openstack.org/api-ref/compute/ | 02:32 |
| Spazmotic | Will do, will take a look soon | 02:33 |
| jianghuaw_ | Set to False means the request will fail when the source and destination hosts are not on the shared storage. | 02:33 |
| jianghuaw_ | Searching "Live-Migrate Server" there:-) | 02:33 |
| *** germs has quit IRC | 02:33 | |
| jianghuaw_ | the above statement is included in the table of the detail for "Live-Migrate Server (os-migrateLive Action)" | 02:34 |
| *** germs has joined #openstack-nova | 02:35 | |
| *** germs has quit IRC | 02:35 | |
| *** germs has joined #openstack-nova | 02:35 | |
| Spazmotic | I see... Interesting | 02:37 |
| Spazmotic | I'll keep reading through it more completely so I can understand what it's doing differently then sir | 02:37 |
| Spazmotic | thanks | 02:37 |
| jianghuaw_ | np. you're always welcome:-) | 02:38 |
| *** lei-zh has quit IRC | 02:41 | |
| *** chyka has joined #openstack-nova | 02:42 | |
| *** psachin has joined #openstack-nova | 02:43 | |
| *** chyka has quit IRC | 02:46 | |
| *** psachin has quit IRC | 02:49 | |
| *** amodi has joined #openstack-nova | 02:50 | |
| *** felipemonteiro has quit IRC | 02:52 | |
| Spazmotic | Co workers had me doubting the migrate_data field again and how it gets passed around | 02:52 |
| Spazmotic | Live migrate you too confusing for your own good | 02:52 |
| *** jmlowe has joined #openstack-nova | 02:52 | |
| *** vladikr has quit IRC | 02:53 | |
| *** vladikr has joined #openstack-nova | 02:53 | |
| *** psachin has joined #openstack-nova | 02:54 | |
| *** hieulq has quit IRC | 02:54 | |
| *** itlinux has quit IRC | 02:56 | |
| *** hongbin has joined #openstack-nova | 02:57 | |
| *** salv-orlando has joined #openstack-nova | 02:57 | |
| *** salv-orlando has quit IRC | 03:01 | |
| *** fragatina has quit IRC | 03:02 | |
| *** itlinux has joined #openstack-nova | 03:09 | |
| *** suresh12 has joined #openstack-nova | 03:14 | |
| *** amodi has quit IRC | 03:16 | |
| *** suresh12 has quit IRC | 03:18 | |
| *** pooja-jadhav is now known as pooja_jadhav | 03:21 | |
| *** r-daneel has quit IRC | 03:36 | |
| *** nicolasbock has quit IRC | 03:40 | |
| *** yufei has quit IRC | 03:41 | |
| *** r-daneel has joined #openstack-nova | 03:53 | |
| *** r-daneel_ has joined #openstack-nova | 03:55 | |
| *** hongbin has quit IRC | 03:57 | |
| *** crushil has joined #openstack-nova | 03:57 | |
| *** r-daneel has quit IRC | 03:57 | |
| *** r-daneel_ is now known as r-daneel | 03:57 | |
| *** suresh12 has joined #openstack-nova | 03:58 | |
| *** salv-orlando has joined #openstack-nova | 03:58 | |
| *** udesale has joined #openstack-nova | 04:00 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288 | 04:00 |
| *** salv-orlando has quit IRC | 04:02 | |
| *** sapd has joined #openstack-nova | 04:03 | |
| *** vladikr has quit IRC | 04:03 | |
| *** vladikr has joined #openstack-nova | 04:04 | |
| *** germs has quit IRC | 04:05 | |
| *** germs has joined #openstack-nova | 04:05 | |
| *** germs has quit IRC | 04:05 | |
| *** germs has joined #openstack-nova | 04:05 | |
| *** germs has quit IRC | 04:06 | |
| *** fragatina has joined #openstack-nova | 04:08 | |
| *** Dinesh_Bhor has quit IRC | 04:09 | |
| *** fragatina has quit IRC | 04:10 | |
| *** janki has joined #openstack-nova | 04:13 | |
| *** mikal_ has quit IRC | 04:17 | |
| *** mikal has joined #openstack-nova | 04:20 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:21 | |
| *** hoangcx has quit IRC | 04:25 | |
| *** yufei has joined #openstack-nova | 04:26 | |
| *** chyka has joined #openstack-nova | 04:32 | |
| *** links has joined #openstack-nova | 04:32 | |
| *** harlowja has joined #openstack-nova | 04:35 | |
| *** chyka has quit IRC | 04:36 | |
| *** sree_ has joined #openstack-nova | 04:39 | |
| *** sree_ is now known as Guest98337 | 04:39 | |
| *** andreas_s has joined #openstack-nova | 04:40 | |
| *** lpetrut has joined #openstack-nova | 04:42 | |
| *** sridharg has joined #openstack-nova | 04:43 | |
| *** Guest98337 has quit IRC | 04:43 | |
| *** sree has joined #openstack-nova | 04:43 | |
| *** andreas_s has quit IRC | 04:45 | |
| *** ratailor has joined #openstack-nova | 04:47 | |
| *** salv-orlando has joined #openstack-nova | 04:59 | |
| *** salv-orlando has quit IRC | 05:03 | |
| *** lpetrut has quit IRC | 05:17 | |
| *** zhurong has joined #openstack-nova | 05:18 | |
| *** hoangcx has joined #openstack-nova | 05:19 | |
| *** suresh12 has quit IRC | 05:19 | |
| *** suresh12 has joined #openstack-nova | 05:20 | |
| *** lpetrut has joined #openstack-nova | 05:20 | |
| *** harlowja has quit IRC | 05:22 | |
| *** abhishekk has joined #openstack-nova | 05:27 | |
| *** moshele has joined #openstack-nova | 05:32 | |
| *** lpetrut has quit IRC | 05:34 | |
| *** moshele has quit IRC | 05:40 | |
| *** gjayavelu has joined #openstack-nova | 05:41 | |
| *** mdnadeem has joined #openstack-nova | 05:42 | |
| *** Zames has joined #openstack-nova | 05:47 | |
| *** suresh12 has quit IRC | 05:51 | |
| *** suresh12 has joined #openstack-nova | 05:53 | |
| *** Zames has quit IRC | 05:54 | |
| *** suresh12 has quit IRC | 05:59 | |
| *** salv-orlando has joined #openstack-nova | 05:59 | |
| *** Dinesh_Bhor has quit IRC | 06:00 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 06:00 |
| *** sidx64 has joined #openstack-nova | 06:03 | |
| *** salv-orlando has quit IRC | 06:04 | |
| *** sidx64 has quit IRC | 06:04 | |
| *** sidx64 has joined #openstack-nova | 06:06 | |
| *** germs has joined #openstack-nova | 06:06 | |
| *** germs has quit IRC | 06:06 | |
| *** germs has joined #openstack-nova | 06:06 | |
| *** salv-orlando has joined #openstack-nova | 06:10 | |
| *** germs has quit IRC | 06:11 | |
| *** claudiub|2 has joined #openstack-nova | 06:11 | |
| openstackgerrit | Merged openstack/nova master: placement: generation in provider aggregate APIs https://review.openstack.org/548249 | 06:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/548772 | 06:19 |
| *** crushil has quit IRC | 06:25 | |
| *** mingyu_ has quit IRC | 06:28 | |
| *** phuongnh has quit IRC | 06:28 | |
| *** phuongnh has joined #openstack-nova | 06:29 | |
| *** alexchadin has joined #openstack-nova | 06:31 | |
| *** suresh12 has joined #openstack-nova | 06:31 | |
| *** AlexeyAbashkin has joined #openstack-nova | 06:36 | |
| *** suresh12 has quit IRC | 06:36 | |
| *** gjayavelu has quit IRC | 06:38 | |
| *** fragatina has joined #openstack-nova | 06:43 | |
| *** gongysh has joined #openstack-nova | 06:43 | |
| *** fragatina has quit IRC | 06:43 | |
| *** fragatina has joined #openstack-nova | 06:43 | |
| *** fragatin_ has joined #openstack-nova | 06:44 | |
| *** kholkina has joined #openstack-nova | 06:46 | |
| *** fragatina has quit IRC | 06:48 | |
| *** OctopusZhang_ has joined #openstack-nova | 06:49 | |
| *** yufei has quit IRC | 06:49 | |
| *** OctopusZhang__ has joined #openstack-nova | 06:49 | |
| *** sidx64 has quit IRC | 06:50 | |
| *** jaosorior has joined #openstack-nova | 06:50 | |
| *** AlexeyAbashkin has quit IRC | 06:51 | |
| *** OctopusZhang_ has quit IRC | 06:53 | |
| *** eandersson has joined #openstack-nova | 06:54 | |
| eandersson | Maybe a silly idea... but would it be possible to hook into metadata changes (at least user initiated)? | 06:54 |
| *** sidx64 has joined #openstack-nova | 06:56 | |
| eandersson | We use the metadata from vm create to perform specific actions using notifications (e.g. compute.instance.create.end), but would be nice to be able to retroactively do the same on existing VMs using metadata. | 06:56 |
| *** p_d has joined #openstack-nova | 06:58 | |
| *** sticker has quit IRC | 07:05 | |
| *** sidx64_ has joined #openstack-nova | 07:11 | |
| *** sidx64 has quit IRC | 07:13 | |
| openstackgerrit | Julia Kreger proposed openstack/nova master: WIP: Update Ironic UTs to enable REST migration https://review.openstack.org/553699 | 07:13 |
| *** sar has joined #openstack-nova | 07:16 | |
| *** sidx64_ has quit IRC | 07:17 | |
| *** lpetrut has joined #openstack-nova | 07:17 | |
| *** sidx64 has joined #openstack-nova | 07:17 | |
| *** lpetrut_ has joined #openstack-nova | 07:19 | |
| *** lpetrut has quit IRC | 07:19 | |
| *** rcernin has quit IRC | 07:20 | |
| *** phuongnh has quit IRC | 07:27 | |
| *** phuongnh has joined #openstack-nova | 07:27 | |
| *** sidx64 has quit IRC | 07:29 | |
| *** sidx64 has joined #openstack-nova | 07:30 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 07:33 |
| *** liusheng has quit IRC | 07:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Updated from global requirements https://review.openstack.org/553211 | 07:34 |
| *** liusheng has joined #openstack-nova | 07:34 | |
| *** brault has quit IRC | 07:36 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Support live migration in pooled multi-nodes environment https://review.openstack.org/489451 | 07:42 |
| *** brault has joined #openstack-nova | 07:43 | |
| Spazmotic | I wish I understand the entirity of that patch XD | 07:45 |
| *** itlinux has quit IRC | 07:46 | |
| *** OctopusZhang has joined #openstack-nova | 07:47 | |
| *** OctopusZhang__ has quit IRC | 07:47 | |
| *** OctopusZhang is now known as yufei | 07:47 | |
| *** OctopusZhang__ has joined #openstack-nova | 07:48 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:50 | |
| *** yufei has quit IRC | 07:52 | |
| *** zhurong has quit IRC | 07:56 | |
| *** germs has joined #openstack-nova | 08:08 | |
| *** germs has quit IRC | 08:08 | |
| *** germs has joined #openstack-nova | 08:08 | |
| *** hoonetorg has quit IRC | 08:11 | |
| *** tssurya has joined #openstack-nova | 08:12 | |
| *** germs has quit IRC | 08:13 | |
| openstackgerrit | Merged openstack/nova master: Scheduler multiple workers support https://review.openstack.org/159382 | 08:14 |
| *** damien_r has joined #openstack-nova | 08:14 | |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 08:18 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | 08:19 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP https://review.openstack.org/553288 | 08:23 |
| *** ralonsoh has joined #openstack-nova | 08:24 | |
| *** sidx64 has quit IRC | 08:26 | |
| *** hoonetorg has joined #openstack-nova | 08:27 | |
| *** cdent has joined #openstack-nova | 08:27 | |
| *** cdent has left #openstack-nova | 08:28 | |
| *** p_d has quit IRC | 08:29 | |
| *** tesseract has joined #openstack-nova | 08:30 | |
| *** sahid has joined #openstack-nova | 08:31 | |
| *** jchhatbar has joined #openstack-nova | 08:32 | |
| *** janki has quit IRC | 08:32 | |
| *** moshele has joined #openstack-nova | 08:32 | |
| *** OctopusZhang has joined #openstack-nova | 08:45 | |
| *** OctopusZhang__ has quit IRC | 08:45 | |
| *** OctopusZhang is now known as yufei | 08:45 | |
| *** OctopusZhang__ has joined #openstack-nova | 08:46 | |
| *** jpena|off is now known as jpena | 08:49 | |
| *** yufei has quit IRC | 08:50 | |
| *** alexchadin has quit IRC | 08:50 | |
| *** moshele has quit IRC | 08:50 | |
| *** alexchadin has joined #openstack-nova | 08:50 | |
| *** alexchadin has quit IRC | 08:51 | |
| *** alexchadin has joined #openstack-nova | 08:51 | |
| *** alexchadin has quit IRC | 08:51 | |
| *** alexchadin has joined #openstack-nova | 08:52 | |
| *** alexchadin has quit IRC | 08:52 | |
| *** alexchadin has joined #openstack-nova | 08:53 | |
| *** alexchadin has quit IRC | 08:53 | |
| *** alexchadin has joined #openstack-nova | 08:54 | |
| *** alexchadin has quit IRC | 08:54 | |
| *** alexchadin has joined #openstack-nova | 08:59 | |
| *** ccamacho has joined #openstack-nova | 09:01 | |
| *** alexchadin has quit IRC | 09:04 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 09:06 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 09:06 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 09:06 |
| *** phuongnh has quit IRC | 09:09 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: DNM: Demo code for microversion parse extraction https://review.openstack.org/550265 | 09:15 |
| *** hshiina has quit IRC | 09:19 | |
| *** psachin has quit IRC | 09:22 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Provide framework for setting placement error codes https://review.openstack.org/546177 | 09:23 |
| *** jistr is now known as jistr|mtg | 09:24 | |
| *** alexchadin has joined #openstack-nova | 09:25 | |
| openstackgerrit | Andrey Volkov proposed openstack/osc-placement master: Get resource provider by uuid or name https://review.openstack.org/527791 | 09:26 |
| *** alexchadin has quit IRC | 09:29 | |
| *** mdbooth has joined #openstack-nova | 09:29 | |
| *** alexchadin has joined #openstack-nova | 09:30 | |
| *** Hazelesque has quit IRC | 09:32 | |
| *** Hazelesque has joined #openstack-nova | 09:33 | |
| *** psachin has joined #openstack-nova | 09:34 | |
| *** alexchadin has quit IRC | 09:35 | |
| *** sdague has joined #openstack-nova | 09:38 | |
| *** mgoddard_ has joined #openstack-nova | 09:38 | |
| *** lucas-afk is now known as lucasagomes | 09:38 | |
| *** derekh has joined #openstack-nova | 09:40 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553496 | 09:40 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730 | 09:40 |
| lyarwood | stephenfin: ^ FWIW Doug suggested this in https://bugs.launchpad.net/nova/+bug/1755783 | 09:41 |
| openstack | Launchpad bug 1755783 in OpenStack Compute (nova) "PCI configuration documentation renders with smart quotes" [Undecided,In progress] - Assigned to Lee Yarwood (lyarwood) | 09:41 |
| lyarwood | stephenfin: I think that's correct, ./me wait for the docs to build | 09:41 |
| stephenfin | lyarwood: That's kind of working around a oslo.config bug though | 09:42 |
| *** dtantsur|afk is now known as dtantsur | 09:42 | |
| *** esberglu has joined #openstack-nova | 09:43 | |
| stephenfin | That being said, we're stuck with older versions of oslo.config for older versions of nova, so that could be the only backportable solution | 09:43 |
| stephenfin | I've got no machine to work on NUMA-aware vSwitches, so I can work on that oslo.config fix today | 09:43 |
| lyarwood | stephenfin: indeed, so the bug is that even with back-ticks we still end up with smartquotes right? | 09:44 |
| *** esberglu has quit IRC | 09:47 | |
| *** annp has joined #openstack-nova | 09:47 | |
| *** liuzz has quit IRC | 09:47 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Update the nova-manage db archive_deleted_rows description https://review.openstack.org/553733 | 09:48 |
| openstackgerrit | jichenjc proposed openstack/nova master: Move test_allocation_candidates.py https://review.openstack.org/553149 | 09:49 |
| openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_provider.py https://review.openstack.org/553150 | 09:49 |
| openstackgerrit | jichenjc proposed openstack/nova master: Move test_resource_class_cache.py https://review.openstack.org/553153 | 09:49 |
| stephenfin | lyarwood: Yeah, we're not parsing the config option help text properly so only some of the formatting is being picked up | 09:50 |
| stephenfin | Biggest issue I see to resolving this is all the broken config option descriptions that likely exist at this point. Guess I'll have to try handle any warnings that might be raised | 09:51 |
| *** andreas_s has joined #openstack-nova | 09:54 | |
| *** tovin07 has quit IRC | 09:56 | |
| *** alexchadin has joined #openstack-nova | 09:56 | |
| *** inara` has quit IRC | 09:57 | |
| *** inara has joined #openstack-nova | 10:00 | |
| lyarwood | stephenfin: urgh, so that doesn't work, still seeing smartquotes | 10:01 |
| *** mgoddard_ has quit IRC | 10:01 | |
| *** mingyu has joined #openstack-nova | 10:06 | |
| *** jichen has quit IRC | 10:06 | |
| *** hoangcx has quit IRC | 10:07 | |
| *** germs has joined #openstack-nova | 10:09 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/queens: Make archive_deleted_rows handle a missing CONF.api_database.connection https://review.openstack.org/553742 | 10:10 |
| *** sidx64 has joined #openstack-nova | 10:10 | |
| stephenfin | lyarwood: You need to do that in 'docutils.conf' | 10:12 |
| stephenfin | lyarwood: See https://github.com/rtfd/readthedocs.org/issues/2940 | 10:12 |
| *** germs has quit IRC | 10:14 | |
| lyarwood | stephenfin: huh | 10:17 |
| lyarwood | stephenfin: FWIW I was using http://www.sphinx-doc.org/en/stable/config.html#confval-smartquotes | 10:18 |
| *** sapd has quit IRC | 10:18 | |
| stephenfin | lyarwood: "New in version 1.6.6" | 10:18 |
| stephenfin | https://github.com/openstack/requirements/blob/master/global-requirements.txt#L421 | 10:19 |
| lyarwood | *facepalm* | 10:19 |
| stephenfin | I wasn't aware of that option but it seems we can't use it anyway | 10:19 |
| stephenfin | :/ | 10:19 |
| lyarwood | I might just leave it in the change anyway for when we do get above 1.6.6 | 10:20 |
| stephenfin | lyarwood: That...could be a while. Some projects are seeing issues with it and I can't figure out why my "fix" won't pass CI https://review.openstack.org/#/c/552100/ | 10:20 |
| stephenfin | lyarwood: If you're backporting, maybe use the older 'html' prefixed option, assuming we're using <1.6 on those branches? | 10:21 |
| *** sree has quit IRC | 10:21 | |
| *** sree has joined #openstack-nova | 10:22 | |
| *** sree has quit IRC | 10:22 | |
| *** sree has joined #openstack-nova | 10:22 | |
| lyarwood | stephenfin: yup good point | 10:23 |
| lyarwood | stephenfin: `while setting up extension ['?', 'sphinx.ext.todo']: node class 'todolist' is already registered, its visitors will be overridden` | 10:24 |
| lyarwood | stephenfin: ^ tox -e docs - known issue? | 10:24 |
| stephenfin | lyarwood: I've just spotted that too. That was me (commit 936404b90a954609f0c94582df0e2fd700948cea) | 10:25 |
| stephenfin | Seems that feature (which I wrote) is broken in Sphinx | 10:25 |
| stephenfin | my turn to *facepalm* | 10:25 |
| stephenfin | I'm working on a fix as we speak. Gimme 5 | 10:25 |
| *** mgoddard has joined #openstack-nova | 10:26 | |
| stephenfin | lyarwood: To fix that, just revert said commit | 10:26 |
| *** andymccr_ is now known as andymccr | 10:27 | |
| lyarwood | stephenfin: it's fine, it had already built the config.html page I care about | 10:27 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730 | 10:31 |
| stephenfin | So oslo.config is doing what I'd expect it to do. Very odd... | 10:33 |
| stephenfin | Namely, emitting the raw lines as its sees them and letting the docutils rST parser do its work | 10:33 |
| *** nicolasbock has joined #openstack-nova | 10:36 | |
| *** tbachman has quit IRC | 10:39 | |
| *** ralonsoh_ has joined #openstack-nova | 10:41 | |
| *** ralonsoh has quit IRC | 10:41 | |
| *** mingyu has quit IRC | 10:44 | |
| *** udesale has quit IRC | 10:50 | |
| *** tbachman has joined #openstack-nova | 10:57 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Correct documentation for '[pci] passthrough_whitelist' https://review.openstack.org/552874 | 10:57 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751 | 10:57 |
| stephenfin | lyarwood: ^ | 10:57 |
| lyarwood | stephenfin: still end up with smartquotes and unformatted text in config.html | 11:02 |
| lyarwood | stephenfin: but the build passes now ;) | 11:02 |
| *** abhishekk has quit IRC | 11:04 | |
| *** jistr|mtg is now known as jistr | 11:07 | |
| *** bhujay has joined #openstack-nova | 11:08 | |
| *** mingyu has joined #openstack-nova | 11:18 | |
| *** mvk has quit IRC | 11:19 | |
| *** yamamoto has quit IRC | 11:24 | |
| *** vladikr has quit IRC | 11:29 | |
| stephenfin | lyarwood: Yeah, that's just to fix the broken local build. Did your patch work? | 11:31 |
| *** mdnadeem has quit IRC | 11:31 | |
| stephenfin | My reading of that was that the old option was removed in 1.6.0 and then that alias was added in 1.6.6. For 1.6.0 and 1.6.5, the docutils config file was the only way | 11:32 |
| stephenfin | But that's a guess | 11:32 |
| *** psachin has quit IRC | 11:33 | |
| *** mvk has joined #openstack-nova | 11:34 | |
| *** psachin has joined #openstack-nova | 11:38 | |
| *** mdnadeem has joined #openstack-nova | 11:39 | |
| *** sar has quit IRC | 11:55 | |
| *** mingyu_ has joined #openstack-nova | 11:57 | |
| *** mingyu has quit IRC | 11:57 | |
| *** yamamoto has joined #openstack-nova | 11:58 | |
| *** mingyu_ has quit IRC | 11:58 | |
| lyarwood | stephenfin: it did locally, just waiting on check job running, still queued at the moment. | 11:59 |
| *** odyssey4me has quit IRC | 12:04 | |
| *** odyssey4me has joined #openstack-nova | 12:04 | |
| *** ratailor has quit IRC | 12:05 | |
| *** bhujay has quit IRC | 12:05 | |
| *** mingyu has joined #openstack-nova | 12:10 | |
| *** germs has joined #openstack-nova | 12:10 | |
| *** germs has quit IRC | 12:10 | |
| *** germs has joined #openstack-nova | 12:10 | |
| *** mingyu_ has joined #openstack-nova | 12:11 | |
| *** mingyu has quit IRC | 12:11 | |
| *** edmondsw has joined #openstack-nova | 12:12 | |
| *** edleafe is now known as figleaf | 12:12 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova-specs master: Complex (Anti)-Affinity Policies https://review.openstack.org/546925 | 12:13 |
| *** sree has quit IRC | 12:13 | |
| *** sree has joined #openstack-nova | 12:13 | |
| *** germs has quit IRC | 12:14 | |
| *** sambetts|afk is now known as sambetts | 12:15 | |
| *** gaurangt has quit IRC | 12:20 | |
| *** Zames has joined #openstack-nova | 12:22 | |
| *** gaurangt has joined #openstack-nova | 12:22 | |
| *** tiendc has quit IRC | 12:23 | |
| *** annp has quit IRC | 12:24 | |
| lyarwood | stephenfin: http://logs.openstack.org/30/553730/2/check/build-openstack-sphinx-docs/28fd09c/html/configuration/config.html#pci - appears to be working | 12:24 |
| * lyarwood -> lunch | 12:24 | |
| *** sar has joined #openstack-nova | 12:27 | |
| *** jpena is now known as jpena|lunch | 12:27 | |
| *** Zames has quit IRC | 12:27 | |
| *** gwqubpq has joined #openstack-nova | 12:30 | |
| *** mingyu_ has quit IRC | 12:30 | |
| *** sidx64 has quit IRC | 12:30 | |
| *** tuanla____ has quit IRC | 12:30 | |
| *** yamamoto has quit IRC | 12:31 | |
| *** elmaciej has joined #openstack-nova | 12:32 | |
| *** Zames has joined #openstack-nova | 12:33 | |
| *** sidx64 has joined #openstack-nova | 12:34 | |
| *** vladikr has joined #openstack-nova | 12:35 | |
| *** yamamoto has joined #openstack-nova | 12:36 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy https://review.openstack.org/553776 | 12:39 |
| *** gongysh has quit IRC | 12:42 | |
| figleaf | ugh - tempest failed again. Is it worth rechecking yet again? | 12:45 |
| *** jchhatbar has quit IRC | 12:47 | |
| *** jchhatba_ has joined #openstack-nova | 12:47 | |
| *** moshele has joined #openstack-nova | 12:49 | |
| *** Zames has quit IRC | 12:50 | |
| tssurya | figleaf: guess not :( | 12:51 |
| *** gongysh has joined #openstack-nova | 12:51 | |
| * Spazmotic holds on on Depends-On submission | 12:51 | |
| Spazmotic | heh | 12:51 |
| *** moshele has quit IRC | 12:54 | |
| *** tssurya is now known as sususuryashines | 12:54 | |
| *** gongysh has quit IRC | 12:57 | |
| *** elmaciej_ has joined #openstack-nova | 12:57 | |
| *** gyankum has joined #openstack-nova | 12:59 | |
| *** dave-mccowan has joined #openstack-nova | 12:59 | |
| *** elmaciej has quit IRC | 12:59 | |
| *** salv-orlando has quit IRC | 13:01 | |
| Roamer` | a couple of years ago I proposed https://review.openstack.org/#/c/137830/ and then Things Happened(tm) and we stopped working on the StorPool Nova image backend. Looks like we'll write it after all (even if we have to maintain it out-of-tree); if I un-abandon that spec and propose it for inclusion, will somebody be very much against it? | 13:02 |
| *** dave-mcc_ has joined #openstack-nova | 13:04 | |
| *** dave-mccowan has quit IRC | 13:04 | |
| *** lyan has joined #openstack-nova | 13:05 | |
| *** lyan is now known as Guest14530 | 13:06 | |
| *** lucasagomes is now known as lucas-hungry | 13:06 | |
| *** mriedem has joined #openstack-nova | 13:06 | |
| *** efried is now known as fried_rice | 13:10 | |
| fried_rice | artom_: Thank you very much sir. | 13:10 |
| fried_rice | alex_xu_: Thanks! | 13:10 |
| fried_rice | figleaf: I win! | 13:11 |
| * fried_rice gloats a little bit | 13:11 | |
| figleaf | fried_rice: You probably sabotaged tempest so that you could win :) | 13:12 |
| fried_rice | figleaf: To save some heartache, consider rebasing on top of https://review.openstack.org/#/c/548934/ (which is already in the gate) instead of master. | 13:12 |
| fried_rice | figleaf: Oh, I totally did. I didn't know it was gonna *work* though. | 13:12 |
| figleaf | fried_rice: that patch has a tempest failure, too | 13:13 |
| figleaf | you forgot to un-bork tempest | 13:13 |
| *** jaypipes is now known as leakypipes | 13:16 | |
| fried_rice | those volumes... | 13:16 |
| *** esberglu has joined #openstack-nova | 13:17 | |
| *** salv-orlando has joined #openstack-nova | 13:17 | |
| *** sidx64 has quit IRC | 13:18 | |
| *** artom_ has quit IRC | 13:19 | |
| openstackgerrit | Merged openstack/nova master: New-style _set_inventory_for_provider https://review.openstack.org/537648 | 13:22 |
| *** cdent has joined #openstack-nova | 13:22 | |
| mriedem | https://review.openstack.org/553786 | 13:22 |
| *** liverpooler has joined #openstack-nova | 13:24 | |
| figleaf | mriedem: <fingers crossed> | 13:25 |
| *** andreas_s has quit IRC | 13:27 | |
| *** burt has joined #openstack-nova | 13:28 | |
| *** BlackDex has quit IRC | 13:29 | |
| *** eharney has joined #openstack-nova | 13:30 | |
| *** andreas_s_ has joined #openstack-nova | 13:31 | |
| *** udesale has joined #openstack-nova | 13:31 | |
| mnaser | how do we feel about having 'ComputeCapabilitiesFilter' but instead parses image properties rather than instance type | 13:32 |
| mnaser | (reason is, i have X customer which needs Y feature, i dont want to start maintaining instance types for every single one, but rather give the ability for the customer to add propreties) | 13:33 |
| *** ratailor has joined #openstack-nova | 13:33 | |
| *** jpena|lunch is now known as jpena | 13:33 | |
| *** r-daneel has quit IRC | 13:34 | |
| *** jchhatba_ has quit IRC | 13:35 | |
| *** andreas_s_ has quit IRC | 13:35 | |
| leakypipes | mnaser: https://review.openstack.org/#/c/541507/ will make your day. | 13:36 |
| mnaser | leakypipes: i love how the "example" is exactly what we're looking for .. trait:HW_CPU_X86_AVX2=required | 13:37 |
| mnaser | (like, the exact instruction set too) | 13:38 |
| leakypipes | mnaser: bingo. | 13:38 |
| mnaser | i guess i'll have to live with custom flavors till this code gets shipped | 13:39 |
| leakypipes | mnaser: should be done in <2 months | 13:39 |
| leakypipes | right arvindn05? ;P | 13:39 |
| mnaser | well isnt it nice we have 6 month release cycles so i can pick it up in a release in 6 months max | 13:39 |
| mnaser | ok it's friday i'll be nice | 13:40 |
| *** elmaciej has joined #openstack-nova | 13:40 | |
| leakypipes | mnaser: :) | 13:42 |
| *** elmaciej_ has quit IRC | 13:42 | |
| *** felipemonteiro has joined #openstack-nova | 13:44 | |
| *** david-lyle has quit IRC | 13:45 | |
| openstackgerrit | sahid proposed openstack/nova-specs master: virt: allow instances to be booted with trusted VFs https://review.openstack.org/485522 | 13:46 |
| *** tbachman has quit IRC | 13:46 | |
| *** udesale has quit IRC | 13:46 | |
| *** zhaochao has quit IRC | 13:47 | |
| *** andreas_s has joined #openstack-nova | 13:48 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates https://review.openstack.org/552098 | 13:51 |
| figleaf | fried_rice: ^^ rebased | 13:51 |
| *** psachin has quit IRC | 13:51 | |
| *** elmaciej has quit IRC | 13:51 | |
| *** andreas_s has quit IRC | 13:52 | |
| openstackgerrit | Merged openstack/nova-specs master: Fix spec name for "report CPU features as traits" https://review.openstack.org/552159 | 13:53 |
| mriedem | sahid: i see the problem in that linuxbridge multinode job | 13:53 |
| *** alexchadin has quit IRC | 13:54 | |
| mriedem | posted a new version | 13:54 |
| leakypipes | figleaf: +2 | 13:55 |
| *** moshele has joined #openstack-nova | 13:55 | |
| *** mvk has quit IRC | 13:55 | |
| *** sree_ has joined #openstack-nova | 13:56 | |
| *** mlavalle has joined #openstack-nova | 13:56 | |
| *** sree_ is now known as Guest42798 | 13:56 | |
| *** links has quit IRC | 13:56 | |
| sahid | mriedem: cool thanks, it's that change in "hosts", right? | 13:57 |
| *** moshele has quit IRC | 13:58 | |
| *** sree has quit IRC | 13:58 | |
| mriedem | yeah | 13:58 |
| mriedem | subtle differences in the ansible playbook for a multinode job | 13:59 |
| sahid | ok, i also working on backporting the change in one of our rhos version. so i hope we could do some internal tests | 14:00 |
| *** hongbin has joined #openstack-nova | 14:00 | |
| stephenfin | lyarwood: Been looking at this oslo.config bug https://bugs.launchpad.net/nova/+bug/1755783 It seems nested_parse is getting confused with our output for that option http://paste.openstack.org/show/703014/ | 14:02 |
| openstack | Launchpad bug 1755783 in OpenStack Compute (nova) "PCI configuration documentation renders with smart quotes" [Undecided,In progress] - Assigned to Stephen Finucane (stephenfinucane) | 14:02 |
| stephenfin | lyarwood: So it's probably going to need a significant rewrite of that extension to unconfuse it :( | 14:02 |
| *** BlackDex has joined #openstack-nova | 14:02 | |
| stephenfin | (all that stuff in the <paragraph> tag should have been parsed itself but wasn't) | 14:03 |
| dansmith | sahid: I thought you said you already tested this locally back in an earlier version | 14:03 |
| dansmith | sahid: does that mean you haven't actually tested this manually yet? | 14:03 |
| dansmith | specifically "I'm making tests locally and everything seemed to work" | 14:04 |
| mnaser | are live migration job failures common-ish in gate? aka safe to recheck? | 14:04 |
| fried_rice | figleaf: I'm sorry I'm just getting to this, but I've got some comments on that guy. | 14:06 |
| figleaf | fried_rice: go for it | 14:06 |
| *** lucas-hungry is now known as lucasagomes | 14:06 | |
| sahid | dansmith: that was for a previous version of that patch | 14:07 |
| dansmith | sahid: right, the version that _couldn't_ work...so you haven't tested the latest approach locally? | 14:08 |
| *** gouthamr has joined #openstack-nova | 14:08 | |
| fried_rice | figleaf, leakypipes: The thing so far that we might potentially want to hold up for is that the syntax isn't consistent with member_of in GET /resource_providers. Looks like this discussed earlier in the life of this patch, but not addressed. | 14:08 |
| sahid | dansmith: no sure what you mean by the version that _counldn't work | 14:09 |
| dansmith | heh | 14:09 |
| *** sree has joined #openstack-nova | 14:09 | |
| *** germs has joined #openstack-nova | 14:10 | |
| *** germs has quit IRC | 14:10 | |
| *** germs has joined #openstack-nova | 14:10 | |
| *** mingyu has joined #openstack-nova | 14:10 | |
| *** Guest42798 has quit IRC | 14:12 | |
| *** mingyu has quit IRC | 14:14 | |
| *** germs has quit IRC | 14:15 | |
| *** pchavva has joined #openstack-nova | 14:15 | |
| figleaf | fried_rice: what are the differences that you see? | 14:17 |
| *** felipemonteiro_ has joined #openstack-nova | 14:17 | |
| *** mingyu has joined #openstack-nova | 14:17 | |
| fried_rice | figleaf: GET /resource_providers?member_of=in:<list of UUIDs> -- yours has no 'in:'. Personally I think 'in:' is silly and would like to see it go away. But it's inconsistent. But I don't like it. But it's inconsistent. I'm torn. | 14:18 |
| *** tbachman has joined #openstack-nova | 14:18 | |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/553211 | 14:19 |
| figleaf | fried_rice: what do you mean? Because 'in:' is optional with a single UUID? | 14:20 |
| leakypipes | fried_rice: I used to think I was indecisive. Now I'm not so sure. | 14:20 |
| *** awaugama has joined #openstack-nova | 14:20 | |
| fried_rice | cdent: Do we have a blueprint for placement split? | 14:20 |
| stephenfin | Can someone send this on its merry way through the gate? https://review.openstack.org/#/c/553751/ Fixes an issue I introduced with 'tox -e docs' locally (which the gate no longer runs) | 14:20 |
| *** hongbin has quit IRC | 14:20 | |
| fried_rice | https://review.openstack.org/#/c/553149/ should be tagged with it. | 14:20 |
| leakypipes | fried_rice: no, it's more a collection of things. | 14:20 |
| *** felipemonteiro has quit IRC | 14:20 | |
| *** kholkina has quit IRC | 14:20 | |
| *** hongbin has joined #openstack-nova | 14:20 | |
| mriedem | mdbooth: dansmith: Kevin_Zheng: cfriesen: re: the abort queued live migration spec, i did some digging and i think the futures library/module is what we'd want https://review.openstack.org/#/c/536722/ | 14:21 |
| mriedem | throw futures into a thread pool executor and you can cancel them if they haven't already started | 14:21 |
| fried_rice | figleaf: I haven't gotten all the way through the patch yet (in a meeting atm) but it looks to me like your GET /a_c member_of doesn't use 'in:' at all. | 14:21 |
| leakypipes | stephenfin: gerrit is telling me it needs a rebase | 14:21 |
| dansmith | mriedem: that's one way to do it yeah | 14:21 |
| mriedem | reminds me of when i used to use https://docs.oracle.com/javase/7/docs/api/java/util/concurrent/package-summary.html | 14:21 |
| fried_rice | figleaf: In GET /r_p if I'm not mistaken, you *can't* specify a list without 'in:' | 14:21 |
| *** felipemonteiro__ has joined #openstack-nova | 14:21 | |
| stephenfin | leakypipes: It is? I'm not seeing anything in the UI | 14:22 |
| *** stephenfin is now known as finucannot | 14:22 | |
| *** dansmith is now known as superdan | 14:22 | |
| cdent | fried_rice: no, there's no spec or blueprint at this stage, mostly because of what leakypipes says: there's a suite of vaguely connected lots of things. Do you think we should have something? | 14:23 |
| mriedem | mnaser: yes we hit live migration failures in the gate intermittently | 14:23 |
| figleaf | fried_rice: https://review.openstack.org/#/c/552098/8/nova/api/openstack/placement/util.py@333 | 14:23 |
| mriedem | mnaser: there are some known issues with libvirt in the versions we use | 14:23 |
| mnaser | mriedem: ah bummer, i'll throw a recheck then | 14:23 |
| leakypipes | finucannot: it says Cannot Merge for me. | 14:23 |
| mriedem | mnaser: e.g. http://status.openstack.org/elastic-recheck/#1706377 | 14:23 |
| *** r-daneel has joined #openstack-nova | 14:23 | |
| *** mingyu has quit IRC | 14:23 | |
| fried_rice | cdent: Seems like it would be nice. At least a common topic so we can see everything. I guess that would be sufficient. | 14:24 |
| *** derekh has quit IRC | 14:24 | |
| cdent | i'll make something | 14:24 |
| *** Spazmotic has quit IRC | 14:24 | |
| *** mingyu has joined #openstack-nova | 14:25 | |
| *** felipemonteiro_ has quit IRC | 14:25 | |
| fried_rice | figleaf: Okay; So far I had only read the explanation comment in microversion.py and it's not listed in there. I'll stay quiet til I finish the patch. | 14:25 |
| mriedem | Kevin_Zheng: so i think we create a ThreadPoolExecutor, submit the live migration calls on that, which returns Future objects, and we map those to the migration id which we can later lookup to then call Future.cancel() | 14:25 |
| fried_rice | I see it in the reno. | 14:26 |
| finucannot | fried_rice: Oh, right. Weird that the UI didn't say that. | 14:26 |
| * finucannot goes to fix | 14:26 | |
| superdan | mriedem: we need to make sure we don't .cancel() after it's started really running | 14:26 |
| superdan | mriedem: does futures handle that or do we need to? | 14:26 |
| *** lyarwood is now known as lyaaaaaaaaaaaaar | 14:27 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: mirror nova host aggregate members to placement https://review.openstack.org/553597 | 14:27 |
| figleaf | fried_rice: if the comments can be made clearer, that can always be done in a follow-up patch, no? | 14:27 |
| leakypipes | lyaaaaaaaaaaaaar: ahoy, matey. | 14:27 |
| fried_rice | figleaf: Yes, certainly. | 14:27 |
| *** mingyu has quit IRC | 14:27 | |
| Kevin_Zheng | mriedem: Thanks for reviewing, I’m not familiar with this lib, have to do some reading first | 14:28 |
| * lyaaaaaaaaaaaaar swigs rum | 14:28 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751 | 14:28 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Correct documentation for '[pci] passthrough_whitelist' https://review.openstack.org/552874 | 14:28 |
| *** Spazmotic has joined #openstack-nova | 14:28 | |
| finucannot | johnthetubaguy, leakypipes: and fixed. A global requirements messed with the sphinx version, causing a merge conflict ^ | 14:28 |
| *** elmaciej has joined #openstack-nova | 14:29 | |
| *** gyankum has quit IRC | 14:29 | |
| mriedem | superdan: cancel() returns False if the task has already starte | 14:30 |
| mriedem | *started | 14:30 |
| superdan | mriedem: ack, col | 14:30 |
| superdan | *cool | 14:30 |
| mriedem | you can also add a callback function to the future | 14:30 |
| mriedem | what i'm not totally sure about is if the executor removes futures from the pool once they are done | 14:30 |
| mriedem | automatically | 14:30 |
| mriedem | it should | 14:30 |
| mriedem | but we'll need to track the futures in a dict, | 14:31 |
| superdan | presumably we could remove it from the end of the thread ourselves if needed | 14:31 |
| mriedem | and we could add a callback function so that when a future is done, we callback to cleanup our dict | 14:31 |
| *** RaoulHC has joined #openstack-nova | 14:31 | |
| mriedem | or just at the end of a live migration run, either way | 14:31 |
| *** andreas_s has joined #openstack-nova | 14:32 | |
| *** mingyu has joined #openstack-nova | 14:32 | |
| superdan | that's what I mean | 14:32 |
| mriedem | actually that's overthining it, | 14:33 |
| mriedem | *thinking | 14:33 |
| mriedem | i think we can just have _do_live_migration remove the entry from the dict when it runs and changes the migration status to 'preparing' | 14:33 |
| mriedem | since at that point, you can't cancel the live migration | 14:33 |
| leakypipes | finucannot: +Wallaby'd | 14:33 |
| finucannot | leakypipes: Cheers :) | 14:34 |
| *** mingyu has quit IRC | 14:34 | |
| * finucannot goes back to wrangling Sphinx and oslo.config | 14:34 | |
| cfriesen | what's the right channel for devstack questions? it seems to be installing the wrong version of packages for stable/queens | 14:35 |
| mriedem | finucannot: leakypipes: https://review.openstack.org/#/c/553751/2 | 14:35 |
| mriedem | cfriesen: -qa | 14:35 |
| *** jog0 is now known as jogo | 14:35 | |
| cfriesen | thx | 14:36 |
| finucannot | mriedem: Good catch. I'll pull it out of the queue and rework | 14:36 |
| finucannot | mriedem: Also, are you still using the 'tox -evenv' trick? Just install reno and save yourself a few minutes a week | 14:37 |
| finucannot | ;pip install --user reno' if you don't want to pollute your system (I think) | 14:37 |
| finucannot | *system Python install | 14:37 |
| leakypipes | mriedem: sorry man, had no idea about that :( | 14:37 |
| mriedem | i'll be damned if i'm going to change my reno creation workflow now | 14:37 |
| finucannot | :D fair | 14:38 |
| superdan | mriedem: ah yeah, true.. first thing it does is unqueue itself effectively.. makes sense | 14:39 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751 | 14:39 |
| finucannot | mriedem: as requested | 14:39 |
| *** derekh has joined #openstack-nova | 14:40 | |
| mriedem | finucannot: that's not using upper-constraints | 14:41 |
| mriedem | see https://review.openstack.org/#/c/532971/1/tox.ini | 14:41 |
| finucannot | mriedem: Neither is the standard [testenv] target | 14:41 |
| finucannot | which I copied | 14:41 |
| finucannot | Did I miss something? | 14:42 |
| mriedem | the install_command does... | 14:42 |
| mriedem | hmm | 14:42 |
| finucannot | Yeah, we hack 'install_command' to do that | 14:42 |
| mriedem | ok that's different from novaclient, but you're still missing runtime deps | 14:42 |
| mriedem | venv is just a place to do whatever, so it should have all requs | 14:42 |
| mriedem | *reqs | 14:42 |
| finucannot | They weren't there before though | 14:43 |
| *** mingyu has joined #openstack-nova | 14:43 | |
| *** felipemonteiro__ has quit IRC | 14:43 | |
| finucannot | If '[testenv:venv] deps' wasn't defined, it would defer to '[testenv] deps' https://github.com/openstack/nova/blob/master/tox.ini#L20 | 14:44 |
| *** felipemonteiro__ has joined #openstack-nova | 14:44 | |
| mriedem | right, hmm | 14:44 |
| openstackgerrit | Merged openstack/nova master: docs: Disable smartquotes https://review.openstack.org/553730 | 14:44 |
| finucannot | mriedem: We use 'usedevelop' | 14:44 |
| finucannot | which would install the package and all its runtime dependencies | 14:44 |
| finucannot | so we're good there | 14:44 |
| mriedem | ok, +2 | 14:44 |
| finucannot | (y) | 14:44 |
| mriedem | leakypipes: can you put a procedural -2 on https://review.openstack.org/#/c/553597/ until the spec is approved? | 14:45 |
| *** links has joined #openstack-nova | 14:48 | |
| *** mingyu has quit IRC | 14:49 | |
| leakypipes | mriedem: -W'd it. | 14:49 |
| leakypipes | mriedem: that was mostly me just experimenting in code to see what will work best for the mirroring. basically using it to inform the spec... | 14:50 |
| leakypipes | mriedem: working on the spec changes now. | 14:50 |
| mriedem | yeah i understand | 14:50 |
| *** amoralej is now known as amoralej|lunch | 14:54 | |
| *** bnemec is now known as beekneemech | 14:55 | |
| *** BlackDex has quit IRC | 14:55 | |
| *** mingyu has joined #openstack-nova | 14:56 | |
| cdent | mriedem: you recall how you imported oslo_service to get a single log_options config thing: https://github.com/openstack/nova/blob/master/nova/api/openstack/placement/wsgi.py#L60 ? | 14:59 |
| *** mingyu has quit IRC | 14:59 | |
| cdent | how horribly dirty is it to use a local config opt (that duplicates the oslo_service one) instead? | 14:59 |
| *** artom_ has joined #openstack-nova | 15:00 | |
| *** r-daneel has quit IRC | 15:01 | |
| *** r-daneel has joined #openstack-nova | 15:01 | |
| lyaaaaaaaaaaaaar | mnaser: https://review.openstack.org/#/q/topic:bug/1744361+status:open - btw the stable changes still reference different change-id's , happy to resubmit these if you don't have time. | 15:01 |
| *** mingyu has joined #openstack-nova | 15:02 | |
| mriedem | cdent: given i can't figure out how to add a hook into oslo.service for that, it's probably not the worst thing in the world | 15:04 |
| mnaser | lyaaaaaaaaaaaaar: appreciate if you do, I’m out of the office and running around with a bunch of stuff | 15:05 |
| mriedem | i think at one point when doing that i asked in the -oslo channel if i should even be doing this, i.e. piggyback on their option, and got no response | 15:05 |
| *** nicolasbock has quit IRC | 15:05 | |
| lyaaaaaaaaaaaaar | mnaser: np, happy to help | 15:05 |
| cdent | mriedem: in nova it's not a big deal since oslo_service is already there, but in placement it sets off a cascade of imports and requirments that is sad making. | 15:05 |
| mriedem | only thing i'm not sure about is how to signal a config option change here, | 15:06 |
| mriedem | because the old option isn't deprecated, | 15:06 |
| mriedem | only our usage of it, | 15:06 |
| mriedem | so i guess we could just log something on startup if/when it's used | 15:06 |
| mriedem | to say, 'move to this other nova-specific option' | 15:06 |
| *** mingyu has quit IRC | 15:06 | |
| mnaser | lyaaaaaaaaaaaaar: thank you very much | 15:07 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/queens: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553817 | 15:07 |
| *** melwitt is now known as jgwentworth | 15:08 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Revert "Refine waiting for vif plug events during _hard_reboot" https://review.openstack.org/553818 | 15:08 |
| *** ShilpaSD has quit IRC | 15:09 | |
| cdent | mriedem: I'm not sure I'm following? The option (in the config file) stays the same. What will be different is how the process has the code which makes the conf option legit and validates its value? | 15:09 |
| mriedem | cdent: i don't think you can (should) re-define an option in nova.conf that is already in oslo.service, | 15:09 |
| mriedem | because the oslo.service options are imported into nova.conf | 15:09 |
| mriedem | s/imported/registered/ | 15:09 |
| johnthetubaguy | superdan: mriedem: do we need the placeholder migrations for queens now? | 15:10 |
| cdent | right, but in placement itself there is no import of oslo_service | 15:10 |
| mriedem | cdent: i understand, but placement is still in nova | 15:10 |
| mriedem | and it's using nova's config | 15:10 |
| mriedem | so.. | 15:10 |
| cdent | so I'm wondering about a sort of hack in placement that says "this is a stub for a thing that is normall in olso_service, but since it's in default it's okay" | 15:10 |
| *** Guest14530 has quit IRC | 15:10 | |
| cdent | sure, but it is an indepedent process, so it doesn't really care, does it? | 15:11 |
| *** mgoddard has quit IRC | 15:11 | |
| mriedem | i defer to melwitt | 15:11 |
| cdent | it's not a big deal, it just seems wasteful to import oslo_service | 15:11 |
| mriedem | and i'm saying you don't have to, | 15:11 |
| mriedem | if we define our own option | 15:11 |
| mriedem | which is what placement would have to do anyway once it's split out | 15:12 |
| mriedem | to keep using this | 15:12 |
| *** lyan has joined #openstack-nova | 15:12 | |
| cdent | maybe we are talking about the same thing | 15:12 |
| *** mingyu has joined #openstack-nova | 15:12 | |
| *** lyan is now known as Guest63078 | 15:12 | |
| cdent | when you say "define our own option" do you mean "an option with a different name" or "an option with the same name but defined elsewhere"? | 15:12 |
| jgwentworth | Roamer`: I think you should send email to the dev mailing list with tag [nova] to ask about the possibility of the storpool imagebackend | 15:12 |
| mriedem | different name (or group) | 15:13 |
| cdent | huh, okay. That suggests to me that 'log_options' (which is mighty generic) is in the wrong package, but I guess we're stuck. | 15:14 |
| mriedem | idk, [api]/log_option | 15:15 |
| *** mingyu has quit IRC | 15:15 | |
| mriedem | *log_options? | 15:16 |
| *** BlackDex has joined #openstack-nova | 15:16 | |
| mriedem | [DEFAULT]/log_config | 15:16 |
| *** udesale has joined #openstack-nova | 15:16 | |
| * cdent shrugs | 15:16 | |
| Roamer` | jgwentworth, thanks, will do | 15:17 |
| cdent | 'log_options' seems like a thing that plenty of process, not just ones using oslo_service might like to do and configure, if they use oslo_config. but s'okay. | 15:18 |
| mriedem | johnthetubaguy: i think that's taken care of before the release | 15:18 |
| *** lpetrut_ has quit IRC | 15:19 | |
| mriedem | johnthetubaguy: although i don't see them | 15:19 |
| *** mingyu has joined #openstack-nova | 15:19 | |
| *** lpetrut has joined #openstack-nova | 15:19 | |
| johnthetubaguy | mriedem: but I don't see them in here: https://github.com/openstack/nova/tree/master/nova/db/sqlalchemy/migrate_repo/versions | 15:19 |
| cdent | fried_rice: https://blueprints.launchpad.net/nova/+spec/placement-extract | 15:19 |
| *** moshele has joined #openstack-nova | 15:19 | |
| mriedem | johnthetubaguy: nor do i | 15:19 |
| mriedem | johnthetubaguy: i think there is a script to generate them | 15:19 |
| fried_rice | cdent: Cool mon, thanks. | 15:20 |
| johnthetubaguy | mriedem: I think it has to be post release, so they are missing in stable, but present in master? it always confuses me | 15:20 |
| mriedem | yeah in case you backport a db migration, | 15:20 |
| mriedem | it then takes the placeholder spot in master | 15:21 |
| *** sree_ has joined #openstack-nova | 15:21 | |
| *** sree_ is now known as Guest71919 | 15:21 | |
| fried_rice | cdent: I guess you'll be going through and tagging all your patches accordingly? | 15:21 |
| fried_rice | I commented on https://review.openstack.org/#/c/553149/ | 15:21 |
| cdent | fried_rice: if that's what you really want | 15:22 |
| *** mingyu has quit IRC | 15:22 | |
| mriedem | johnthetubaguy: yikes https://review.openstack.org/#/q/file:%255Enova/db/sqlalchemy/api_migrations/migrate_repo/versions.*+status:open | 15:22 |
| mriedem | https://review.openstack.org/#/q/file:%255Enova/db/sqlalchemy/migrate_repo/versions.*+status:open | 15:23 |
| fried_rice | cdent: Do you feel as though that's pointless unnecessary busywork? Me, I'd like to be able to search on the topic and get the list of all the patches associated with the effort. And having the LP bot collate them in the bp whiteboard, also useful. | 15:23 |
| *** lpetrut has quit IRC | 15:23 | |
| fried_rice | cdent: I can do it if you like. Shouldn't be a big effort, eh? | 15:23 |
| superdan | johnthetubaguy: oh yeah I think we do, hopefully we haven't merged any yet? | 15:24 |
| *** sree has quit IRC | 15:24 | |
| *** moshele has quit IRC | 15:24 | |
| superdan | johnthetubaguy: I can do it, I usually do | 15:24 |
| cdent | fried_rice: I can see how it useful, so happy to do it, since I need to go in there anyway for something else. But because of the amount of searching I do to create the weekly report, I tend not to be very topic oriented (despite using the urls all the time), so it's not something that immediately leaps to mind for me | 15:24 |
| superdan | mriedem: ^ | 15:25 |
| *** janki has joined #openstack-nova | 15:25 | |
| fried_rice | cdent: Gotcha. | 15:25 |
| mriedem | i don't think we've merged any db migrations in rocky yet, but checking | 15:25 |
| mriedem | nope we're good | 15:26 |
| superdan | mriedem: will do it after this call | 15:26 |
| *** yamamoto has quit IRC | 15:26 | |
| *** amodi has joined #openstack-nova | 15:26 | |
| johnthetubaguy | superdan: sweet thanks, just spotted a change doing a migration, and it made me stop and think | 15:28 |
| superdan | johnthetubaguy: yeah, thanks | 15:28 |
| *** mingyu_ has joined #openstack-nova | 15:29 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: mirror nova host aggregates to placement API https://review.openstack.org/545057 | 15:29 |
| *** sar has quit IRC | 15:30 | |
| leakypipes | superdan, mriedem, bauzas, gibi, alex_xu_, figleaf: ^^ reworked mirroring spec. | 15:30 |
| *** links has quit IRC | 15:32 | |
| kashyap | mriedem: Was looking up the scroll this morning; about increasing timeouts due to QEMU / libvirt updates | 15:32 |
| kashyap | https://review.openstack.org/#/c/547676/ | 15:32 |
| kashyap | Even clarkb (not on this channel now) also reported at PTG having to increase tests' timeouts | 15:33 |
| kashyap | I think it must probably be the QEMU / libvirt slowdown after the Meltdown / Spectre fixes. /me goes to check w/ the libvirt folks | 15:33 |
| *** yamamoto has joined #openstack-nova | 15:36 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | 15:36 |
| *** amoralej|lunch is now known as amoralej | 15:37 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | 15:37 |
| *** derekh has quit IRC | 15:39 | |
| *** dave-mcc_ is now known as dave-mccowan | 15:40 | |
| mriedem | kashyap: yes mnaser already confirmed that | 15:41 |
| *** yamamoto has quit IRC | 15:41 | |
| kashyap | mriedem: I struggle to believe the libvirtd dameon _itself_ taking 1 _whole_ minute! | 15:41 |
| kashyap | mriedem: I'm checking w/ the Red Hat perf team to see if they have any numbers on this | 15:42 |
| mriedem | libvirt/qemu in the ocata UCA got updates and has to great a guest on first start to scan cpu features, which is not something newer libvirt in the pike UCA has to do | 15:42 |
| mriedem | *create | 15:42 |
| mriedem | https://launchpad.net/~ubuntu-cloud-archive/+archive/ubuntu/ocata-staging/+sourcepub/8774271/+listing-archive-extra | 15:42 |
| kashyap | I know, they ship with new CPU models, all ending with "*-IBRS" | 15:42 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova stable/pike: Unmap compute nodes when deleting host mappings in delete cell operation https://review.openstack.org/553829 | 15:42 |
| kashyap | (Yep, very familiar w/ those updates for a couple of months already. As I had to apply to a bunch of Fedora / RHEL machines) | 15:42 |
| *** sar has joined #openstack-nova | 15:43 | |
| *** udesale has quit IRC | 15:44 | |
| kashyap | This additional slow down even bolsters the case for the couple of Cloud Providers to enable nested virt in the Gate. | 15:45 |
| kashyap | Upstream KVM maintainer said he's going to enable it for Intel by default in the upstream kernel. | 15:45 |
| kashyap | ClarkB, myself and a bunch of others participated in a discussion with the Cloud Providers at PTG, some cloud providers (OVH) seemed amenable to that. | 15:46 |
| kashyap | Let's see... | 15:46 |
| *** chyka has joined #openstack-nova | 15:47 | |
| fried_rice | figleaf: Reviewed. Nothing blocking :) | 15:47 |
| fried_rice | figleaf: Are you still -1 on https://review.openstack.org/#/c/553476/ for the name of that article, having read my excellent excuses? | 15:48 |
| *** AlexeyAbashkin has quit IRC | 15:48 | |
| *** jgriffit1 is now known as jgriffith | 15:48 | |
| mriedem | volume test issues in the gate should be flushed once https://review.openstack.org/#/c/553794/ merges | 15:48 |
| jgwentworth | woot, thanks | 15:49 |
| *** mingyu has joined #openstack-nova | 15:49 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add placeholder migrations for Queens backports https://review.openstack.org/553831 | 15:49 |
| superdan | mriedem: johnthetubaguy ^ | 15:49 |
| mriedem | superdan: aren't there some tests that you need to modify for the placeholder range? | 15:49 |
| *** mingyu_ has quit IRC | 15:50 | |
| superdan | mriedem: um, no, jeez | 15:50 |
| superdan | (yes) | 15:50 |
| *** p_d has joined #openstack-nova | 15:50 | |
| cfriesen | kashyap: for what it's worth, I haven't had good experiences with stability of nested KVM with CentOS 7.3. | 15:51 |
| *** mingyu has quit IRC | 15:51 | |
| figleaf | fried_rice: yes, 'upt' is much too cryptic | 15:51 |
| kashyap | cfriesen: Got a link to the bugs you've filed? :-) | 15:51 |
| *** yamamoto has joined #openstack-nova | 15:51 | |
| kashyap | cfriesen: More seriously, some Cloud Providers in the room themselves said it was stable in their environments | 15:51 |
| fried_rice | figleaf: Having noticed your # noqa hack, I guess I can make it work with a longer name, eh? | 15:51 |
| cfriesen | kashyap: heh...nothing useful to report, just random intermittent crashes of the first-level guest. | 15:51 |
| kashyap | cfriesen: The cardinal rule here is to try with latest CentOS (stable 7.4) | 15:52 |
| kashyap | cfriesen: I used to track all the bugs I reported for nested virt upstream, a year or so ago: https://kashyapc.fedorapeople.org/kernel-kvm-bugs.txt | 15:52 |
| kashyap | All of them fixed now | 15:52 |
| cfriesen | cool, will have to give it a try with 7.4 | 15:52 |
| kashyap | cfriesen: But, one of the blocking points is the migration of the L1 guest (with the mere presence of L2 in it; it doesn't even have to run). And upstream is working on it as we speak | 15:53 |
| fried_rice | mriedem: What am I tracking for these volume failures to be resolved before I clog the gate with more rechecks? | 15:53 |
| *** r-daneel has quit IRC | 15:53 | |
| kashyap | cfriesen: s/one of the/the only/ | 15:53 |
| cfriesen | kashyap: live migration I assume? | 15:53 |
| *** mdurrant_ has joined #openstack-nova | 15:53 | |
| fried_rice | mriedem: https://review.openstack.org/#/c/553794/ ? | 15:53 |
| kashyap | cfriesen: Yep, live migrating L1 (while L2 is in it; online or offline) | 15:53 |
| figleaf | fried_rice: you don't have to make it *that* long! | 15:54 |
| fried_rice | figleaf: update-provider-tree will push me over 80. Anything else doesn't really make sense. upd-prov-tree? Ew. | 15:55 |
| mriedem | fried_rice: yes, as noted above | 15:55 |
| mriedem | "(10:48:59 AM) mriedem: volume test issues in the gate should be flushed once https://review.openstack.org/#/c/553794/ merges" | 15:55 |
| *** gmann__ has joined #openstack-nova | 15:55 | |
| fried_rice | mriedem: oh, see it now, thx | 15:55 |
| *** icey_ has joined #openstack-nova | 15:55 | |
| *** yamamoto has quit IRC | 15:55 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add placeholder migrations for Queens backports https://review.openstack.org/553831 | 15:56 |
| *** Exhar_ has joined #openstack-nova | 15:57 | |
| fried_rice | figleaf: Looks like your gabbi has some failures? | 15:57 |
| *** gibi_ has joined #openstack-nova | 15:57 | |
| fried_rice | figleaf: Oh, because now you have to send a dict-with-generation to set aggregates :) | 15:58 |
| figleaf | fried_rice: saw that. Working on it | 15:58 |
| fried_rice | figleaf: Added helpful link | 16:00 |
| fried_rice | (not that you needed it) | 16:00 |
| *** artom__ has joined #openstack-nova | 16:00 | |
| *** ratailor has quit IRC | 16:01 | |
| *** andreykurilin_ has joined #openstack-nova | 16:01 | |
| *** jistr_ has joined #openstack-nova | 16:01 | |
| *** szaher has quit IRC | 16:02 | |
| *** mingyu has joined #openstack-nova | 16:02 | |
| *** fanzhang_ has joined #openstack-nova | 16:02 | |
| *** fanzhang has quit IRC | 16:03 | |
| *** mdurrant__ has quit IRC | 16:03 | |
| *** lifeless has quit IRC | 16:03 | |
| *** gibi has quit IRC | 16:03 | |
| *** Exhar has quit IRC | 16:03 | |
| *** icey has quit IRC | 16:03 | |
| *** gmann_ has quit IRC | 16:03 | |
| *** rajinir has quit IRC | 16:03 | |
| *** redondo-mk has quit IRC | 16:03 | |
| *** knikolla has quit IRC | 16:03 | |
| *** andreykurilin has quit IRC | 16:03 | |
| *** sambetts has quit IRC | 16:03 | |
| *** aloga has quit IRC | 16:03 | |
| *** jistr has quit IRC | 16:03 | |
| *** gmann__ is now known as gmann_ | 16:03 | |
| *** icey_ is now known as icey | 16:03 | |
| *** harlowja has joined #openstack-nova | 16:03 | |
| *** sambetts_ has joined #openstack-nova | 16:03 | |
| *** elmaciej_ has joined #openstack-nova | 16:03 | |
| *** sambetts_ is now known as sambetts | 16:03 | |
| *** artom_ has quit IRC | 16:03 | |
| *** maciejjozefczyk has quit IRC | 16:04 | |
| *** dtruong_ has quit IRC | 16:04 | |
| *** jistr_ is now known as jistr | 16:04 | |
| *** dtruong has joined #openstack-nova | 16:05 | |
| *** lifeless has joined #openstack-nova | 16:05 | |
| *** elmaciej has quit IRC | 16:06 | |
| * mriedem goes to lunch | 16:06 | |
| *** p_d has quit IRC | 16:06 | |
| *** yamamoto has joined #openstack-nova | 16:06 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: update_provider_tree devref and docstring updates https://review.openstack.org/553476 | 16:06 |
| fried_rice | figleaf: Done ^ | 16:06 |
| *** fanzhang_ has quit IRC | 16:06 | |
| *** fanzhang has joined #openstack-nova | 16:07 | |
| *** sidx64 has joined #openstack-nova | 16:07 | |
| *** p_d has joined #openstack-nova | 16:08 | |
| *** ingy1 is now known as ingy | 16:09 | |
| *** p_d has quit IRC | 16:10 | |
| *** redondo-mk has joined #openstack-nova | 16:10 | |
| superdan | fried_rice: okay, so I get that you're hitting your peak on review efficiency and finding issues, | 16:10 |
| *** rajinir has joined #openstack-nova | 16:10 | |
| *** jgriffith is now known as big-dummy | 16:10 | |
| superdan | fried_rice: but that should fall short of finding issues in things I need to get merged | 16:10 |
| *** knikolla has joined #openstack-nova | 16:10 | |
| *** aloga has joined #openstack-nova | 16:11 | |
| *** yamamoto has quit IRC | 16:11 | |
| fried_rice | superdan: Send me that list. See where it gets ya. | 16:11 |
| superdan | heh | 16:11 |
| fried_rice | superdan: Are you talking about the member_of thing? | 16:11 |
| superdan | yeah :) | 16:11 |
| fried_rice | zuul had me beat on that one. | 16:11 |
| *** germs has joined #openstack-nova | 16:11 | |
| *** germs has quit IRC | 16:11 | |
| *** germs has joined #openstack-nova | 16:11 | |
| fried_rice | I'll have to try harder. | 16:12 |
| superdan | that was already approved, but failed the gate, now -1d | 16:12 |
| superdan | I'm not serious, of course, I'm glad you found stuff | 16:12 |
| fried_rice | yeah, the rebase needed to take into account the functionality of the thing it got rebased against. | 16:12 |
| fried_rice | TBH, I shoulda seen that on my first pass. | 16:12 |
| superdan | is this the microversion you two were racing for? | 16:12 |
| superdan | stackalytics should discount reviews where you're clearly taking shots at someone else to win a version race :P | 16:13 |
| *** mingyu has quit IRC | 16:14 | |
| cdent | reviewers should discount stackalytics | 16:15 |
| superdan | s/reviewers/everyone/ | 16:15 |
| *** germs has quit IRC | 16:16 | |
| cdent | superdan: I was trying to (quickly) come up with a regexp on your statement but I got lazy and instead printed the output. (meaning, yes, I agree) | 16:16 |
| superdan | heh | 16:17 |
| figleaf | superdan: it was losing the microversion race that made the member_of patch fail. For once, can't blame fried_rice | 16:18 |
| superdan | figleaf: ah | 16:18 |
| *** mingyu has joined #openstack-nova | 16:18 | |
| superdan | 1.21 is a better microversion anyway | 16:19 |
| fried_rice | It can drink | 16:19 |
| fried_rice | (wheat, unfiltered IIRC) | 16:20 |
| *** yamamoto has joined #openstack-nova | 16:21 | |
| *** bhujay has joined #openstack-nova | 16:22 | |
| *** bhujay has quit IRC | 16:24 | |
| *** mingyu has quit IRC | 16:24 | |
| *** bhujay has joined #openstack-nova | 16:25 | |
| *** yamamoto has quit IRC | 16:25 | |
| *** mingyu has joined #openstack-nova | 16:26 | |
| *** elmaciej has joined #openstack-nova | 16:30 | |
| *** elmaciej_ has quit IRC | 16:30 | |
| *** dtantsur is now known as dtantsur|afk | 16:31 | |
| *** mingyu has quit IRC | 16:31 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add 'member_of' param to GET /allocation_candidates https://review.openstack.org/552098 | 16:32 |
| figleaf | leakypipes: superdan: fried_rice: ^^ func tests fixed | 16:32 |
| figleaf | Had to resort to the gabbi HISTORY stuff, since I couldn't assume generation=0 | 16:33 |
| *** mingyu has joined #openstack-nova | 16:33 | |
| figleaf | Besides, we're not supposed to assume that, right? :) | 16:33 |
| *** felipemonteiro_ has joined #openstack-nova | 16:35 | |
| *** derekh has joined #openstack-nova | 16:35 | |
| *** sidx64 has quit IRC | 16:36 | |
| *** yamamoto has joined #openstack-nova | 16:36 | |
| *** mdnadeem has quit IRC | 16:37 | |
| *** mingyu has quit IRC | 16:37 | |
| *** felipemonteiro__ has quit IRC | 16:38 | |
| *** elmaciej has quit IRC | 16:39 | |
| *** mingyu has joined #openstack-nova | 16:40 | |
| *** elmaciej has joined #openstack-nova | 16:40 | |
| *** yamamoto has quit IRC | 16:41 | |
| *** mingyu has quit IRC | 16:42 | |
| *** felipemonteiro_ has quit IRC | 16:42 | |
| *** elmaciej has quit IRC | 16:45 | |
| *** elmaciej has joined #openstack-nova | 16:46 | |
| *** mdbooth has quit IRC | 16:46 | |
| *** mingyu has joined #openstack-nova | 16:46 | |
| fried_rice | figleaf: Ideally you should get it from the RP creation. I did that in a couple of places in my other patch. It's a PITA in gabbi. (Not such a problem in real code.) | 16:48 |
| fried_rice | FWIW, I wouldn't -1 hardcoded generation numbers in a gabbi test. | 16:48 |
| fried_rice | and would even write them myself :) | 16:49 |
| *** mingyu has quit IRC | 16:49 | |
| cdent | fried_rice: it's not a pita in gabbi, just do what figleaf has done | 16:51 |
| *** yamamoto has joined #openstack-nova | 16:51 | |
| *** sahid has quit IRC | 16:51 | |
| superdan | lyaaaaaaaaaaaaar: jgwentworth: looks like those reverts are failing some tests.. on two of the three patches at least | 16:52 |
| fried_rice | cdent: Sorry, I just meant you have to use a pretty long string to get there. | 16:52 |
| cdent | ow, my fingers | 16:52 |
| *** fried_rice is now known as fried_rolls | 16:52 | |
| cdent | is more explicit than e.g.. 'upt' | 16:52 |
| * cdent scurries away :) | 16:52 | |
| superdan | cdent: was that an idiocracy reference? | 16:53 |
| superdan | if so, kudos | 16:53 |
| lyaaaaaaaaaaaaar | superdan: yup looking | 16:53 |
| *** sususuryashines has quit IRC | 16:53 | |
| cdent | superdan: sadly no, entirely unintentional | 16:53 |
| superdan | cdent: ah then no kudos for you | 16:54 |
| cdent | /o\ | 16:54 |
| superdan | cdent: https://www.youtube.com/watch?v=r_4jrMwvZ2A | 16:54 |
| *** damien_r has quit IRC | 16:55 | |
| *** mingyu has joined #openstack-nova | 16:55 | |
| *** lpetrut has joined #openstack-nova | 16:55 | |
| superdan | kashyap: speaking of all that, any ETA on finishing that cpu features patch? | 16:56 |
| *** yamamoto has quit IRC | 16:56 | |
| superdan | oh I guess you just said you'd work on it next week in -infra | 16:57 |
| *** mingyu has quit IRC | 16:57 | |
| kashyap | superdan: I have been slow; you put me on the hook sometime mid next week. I am trying to figure out the unit test breakages :-( | 16:57 |
| superdan | I put you on the hook? | 16:57 |
| kashyap | superdan: You can keep me on the hook for next week, and shame me here if I don't finish it | 16:57 |
| kashyap | Haha | 16:57 |
| kashyap | We were writing the same thing :P | 16:57 |
| superdan | okay | 16:57 |
| *** mingyu has joined #openstack-nova | 17:00 | |
| *** esberglu has quit IRC | 17:04 | |
| *** mingyu_ has joined #openstack-nova | 17:04 | |
| *** harlowja has quit IRC | 17:05 | |
| *** gyee has joined #openstack-nova | 17:05 | |
| *** mingyu has quit IRC | 17:05 | |
| *** mingyu_ has quit IRC | 17:06 | |
| *** yamamoto has joined #openstack-nova | 17:06 | |
| *** yamamoto has quit IRC | 17:07 | |
| *** yamamoto has joined #openstack-nova | 17:07 | |
| *** yamamoto has quit IRC | 17:07 | |
| *** udesale has joined #openstack-nova | 17:07 | |
| *** Guest71919 has quit IRC | 17:07 | |
| *** sree has joined #openstack-nova | 17:08 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move placement exceptions into the placement package https://review.openstack.org/549862 | 17:08 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 17:08 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 17:08 |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 17:08 |
| mnaser | kashyap: yeah so just to follow up, it looks like older libvirt versions have to do a whole lot of stuff on start up to get cpu capabilities | 17:09 |
| *** mingyu has joined #openstack-nova | 17:10 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova master: ironic: stop lying to the RT when ironic is down https://review.openstack.org/545479 | 17:11 |
| jroll | this is what we were thinking, right? ^ | 17:11 |
| * jroll still needs to add a unit test for compute.manager but wants to make sure he's on the right track | 17:12 | |
| *** sree has quit IRC | 17:12 | |
| *** mingyu has quit IRC | 17:12 | |
| *** udesale has quit IRC | 17:16 | |
| *** mingyu has joined #openstack-nova | 17:16 | |
| *** AlexeyAbashkin has joined #openstack-nova | 17:17 | |
| *** fragatin_ has quit IRC | 17:18 | |
| superdan | fried_rolls: can you ack this again before I send it? https://review.openstack.org/#/c/552098/9 | 17:19 |
| *** mingyu has quit IRC | 17:20 | |
| kashyap | mnaser: Yeah; noted. | 17:22 |
| *** AlexeyAbashkin has quit IRC | 17:22 | |
| *** mingyu has joined #openstack-nova | 17:25 | |
| cdent | finucannot: do you mind if I unparent the db_driver conf removal change https://review.openstack.org/#/c/508487/5 it's parent looks stuck, but itself is indepdent | 17:25 |
| finucannot | cdent: go for it | 17:25 |
| openstackgerrit | melanie witt proposed openstack/nova master: only increment disk address unit for scsi devices https://review.openstack.org/538310 | 17:25 |
| openstackgerrit | Chris Dent proposed openstack/nova master: conf: Remove 'db_driver' config opt https://review.openstack.org/508487 | 17:25 |
| cdent | finucannot: thanks | 17:25 |
| cdent | johnthetubaguy: you happy to put a +w on https://review.openstack.org/#/c/508487/ now that it has no parent? you +2'd it a while back. | 17:26 |
| *** lyaaaaaaaaaaaaar is now known as lyarwood | 17:27 | |
| *** mingyu has quit IRC | 17:27 | |
| *** jaosorior has quit IRC | 17:28 | |
| *** sree has joined #openstack-nova | 17:29 | |
| *** mingyu has joined #openstack-nova | 17:30 | |
| *** yamahata has joined #openstack-nova | 17:30 | |
| *** mingyu has quit IRC | 17:31 | |
| *** artom__ is now known as artom | 17:33 | |
| cdent | superdan: I'm not fried_rolls but looking through his comments, and checking the changes since last I looked, assuming we are good with the "followon" nits that eric mentioned on ps8 being followons, looks good to me | 17:33 |
| *** sree has quit IRC | 17:33 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:35 | |
| *** lucasagomes is now known as lucas-pto | 17:35 | |
| superdan | cdent: yeah I just figured I'd give him another go at it, but if he doesn't show up soonish I can just send it | 17:37 |
| *** suresh12 has joined #openstack-nova | 17:37 | |
| *** RaoulHC has quit IRC | 17:40 | |
| *** jackie-truong has joined #openstack-nova | 17:41 | |
| superdan | mriedem: looks like it failed on glance again | 17:41 |
| mnaser | is legacy-tempest-dsvm-cells failing more than usual? | 17:41 |
| mriedem | mnaser: everything is failng more than usual | 17:42 |
| mriedem | until the cinder revert merges | 17:42 |
| mnaser | oh nice | 17:42 |
| mnaser | its friday | 17:42 |
| mnaser | okay, so ill hold rechecking stuff endlessly then | 17:42 |
| *** mingyu has joined #openstack-nova | 17:42 | |
| mriedem | mnaser: https://review.openstack.org/#/c/553794/ | 17:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: handle DiskNotFound during update_available_resource https://review.openstack.org/553067 | 17:42 |
| *** andreas_s has quit IRC | 17:42 | |
| mriedem | superdan: well that's just not cool | 17:42 |
| *** suresh12 has quit IRC | 17:43 | |
| mriedem | superdan: oh that was the one i saw fail, pre-lunch | 17:43 |
| mriedem | need to recheck | 17:43 |
| superdan | oh I thought you already did | 17:44 |
| *** mingyu has quit IRC | 17:44 | |
| superdan | heh I guess not, okay | 17:44 |
| superdan | I was comparing your log link to the latest and like "yep, looks exactly the same" | 17:44 |
| mriedem | the ovs multinode job, g-api started up right away on the subnode http://logs.openstack.org/08/553608/3/check/neutron-tempest-multinode-full/94a71ec/logs/subnode-2/devstacklog.txt.gz#_2018-03-16_14_40_43_011 | 17:44 |
| mriedem | so not sure what's the difference | 17:44 |
| *** imacdonn has quit IRC | 17:44 | |
| *** sree has joined #openstack-nova | 17:44 | |
| superdan | unless LB networking means the nodes can't talk to each other | 17:45 |
| *** imacdonn has joined #openstack-nova | 17:45 | |
| mriedem | it might | 17:45 |
| mriedem | https://review.openstack.org/#/c/553608/3/playbooks/legacy/neutron-tempest-linuxbridge/run.yaml@a33 | 17:45 |
| mriedem | i can throw that back in there | 17:45 |
| mriedem | but not sure if it will help | 17:45 |
| superdan | I really thought we had ruled this out as an easy thing to do | 17:45 |
| mriedem | well, the patch itself is pretty easy :) | 17:45 |
| superdan | maybe I was just psychic | 17:45 |
| *** jackie-truong has quit IRC | 17:45 | |
| mriedem | i.e. it's easy to copy the ovs multinode job, tweak for lb, and run it through ci until it works | 17:46 |
| mriedem | not like i'm spending a lot of time on it | 17:46 |
| mriedem | add PHYSICAL_NETWORK=default back in | 17:47 |
| mriedem | *adding | 17:47 |
| *** maciejjozefczyk has joined #openstack-nova | 17:47 | |
| *** suresh12 has joined #openstack-nova | 17:47 | |
| *** weshay is now known as weshay_brb | 17:47 | |
| *** mingyu has joined #openstack-nova | 17:48 | |
| *** sree has quit IRC | 17:49 | |
| *** mingyu has quit IRC | 17:49 | |
| *** sridharg has quit IRC | 17:50 | |
| *** mingyu has joined #openstack-nova | 17:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: api-ref: add a note in DELETE /os-services about deleting computes https://review.openstack.org/553598 | 17:55 |
| *** mingyu has quit IRC | 17:57 | |
| *** mingyu has joined #openstack-nova | 17:57 | |
| *** r-daneel has joined #openstack-nova | 17:59 | |
| *** mingyu has quit IRC | 17:59 | |
| jgwentworth | superdan, mriedem, leakypipes, fried_rolls, anyone else interested: here's a really rough draft on runways, please add comments and questions and if we can polish it a bit, I can send it out to the dev ML to start using the process https://etherpad.openstack.org/p/nova-runways-rocky | 18:00 |
| *** sree has joined #openstack-nova | 18:00 | |
| jgwentworth | noting that this is going to be really not perfect and we just want to start with something. I should probably write that on the etherpad too | 18:00 |
| *** jpena is now known as jpena|off | 18:01 | |
| *** harlowja has joined #openstack-nova | 18:01 | |
| *** gjayavelu has joined #openstack-nova | 18:03 | |
| *** harlowja_ has joined #openstack-nova | 18:04 | |
| *** sree has quit IRC | 18:05 | |
| *** harlowja has quit IRC | 18:06 | |
| *** amoralej is now known as amoralej|off | 18:06 | |
| *** suresh12 has quit IRC | 18:07 | |
| *** yamamoto has joined #openstack-nova | 18:08 | |
| *** hemna_ has joined #openstack-nova | 18:08 | |
| *** mingyu has joined #openstack-nova | 18:08 | |
| *** sridharg has joined #openstack-nova | 18:10 | |
| *** arvindn05 has quit IRC | 18:11 | |
| *** germs has joined #openstack-nova | 18:12 | |
| *** germs has quit IRC | 18:12 | |
| *** germs has joined #openstack-nova | 18:12 | |
| *** bhujay has quit IRC | 18:13 | |
| *** mingyu has quit IRC | 18:13 | |
| *** sridharg has quit IRC | 18:13 | |
| *** yamamoto has quit IRC | 18:14 | |
| *** mingyu has joined #openstack-nova | 18:15 | |
| *** germs has quit IRC | 18:17 | |
| *** fullmetaljackiet has joined #openstack-nova | 18:18 | |
| *** suresh12 has joined #openstack-nova | 18:20 | |
| *** mingyu_ has joined #openstack-nova | 18:20 | |
| *** felipemonteiro__ has joined #openstack-nova | 18:20 | |
| *** mingyu has quit IRC | 18:21 | |
| *** lyarwood is now known as lyaaaaaaaaaaaaar | 18:22 | |
| *** lyaaaaaaaaaaaaar has quit IRC | 18:22 | |
| *** felipemonteiro_ has quit IRC | 18:23 | |
| *** fragatina has joined #openstack-nova | 18:26 | |
| *** mingyu_ has quit IRC | 18:26 | |
| *** fragatina has quit IRC | 18:26 | |
| *** fragatina has joined #openstack-nova | 18:26 | |
| *** mingyu has joined #openstack-nova | 18:28 | |
| *** mingyu has quit IRC | 18:31 | |
| *** ralonsoh_ has quit IRC | 18:31 | |
| *** felipemonteiro_ has joined #openstack-nova | 18:32 | |
| *** felipemonteiro__ has quit IRC | 18:35 | |
| *** mingyu has joined #openstack-nova | 18:38 | |
| *** tssurya has joined #openstack-nova | 18:44 | |
| *** tssurya is now known as sususuryashines | 18:44 | |
| *** mingyu has quit IRC | 18:46 | |
| *** sambetts is now known as sambetts|afk | 18:48 | |
| *** mingyu has joined #openstack-nova | 18:49 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:50 | |
| *** cdent has quit IRC | 18:51 | |
| *** mingyu has quit IRC | 18:54 | |
| *** felipemonteiro_ has quit IRC | 18:54 | |
| *** felipemonteiro_ has joined #openstack-nova | 18:54 | |
| *** AlexeyAbashkin has quit IRC | 18:54 | |
| *** mingyu has joined #openstack-nova | 18:55 | |
| fried_rolls | superdan: Gah, I thought I did that before I went to roll, sorry about that. Done. | 18:58 |
| superdan | fried_rolls: thanks | 18:58 |
| *** mingyu has quit IRC | 18:58 | |
| *** weshay_brb is now known as weshay | 18:59 | |
| *** tesseract has quit IRC | 19:00 | |
| *** fried_rolls is now known as fried_rice | 19:02 | |
| *** mingyu has joined #openstack-nova | 19:02 | |
| fried_rice | superdan, cdent: and by the way, here's my contribution: https://youtu.be/RYwgtJ7kJTI?t=130 | 19:02 |
| superdan | fried_rice: now that's just silly | 19:03 |
| fried_rice | Well, compared to the high cinema you presented, of course. | 19:03 |
| fried_rice | superdan: I think the volume fails will continue to fail until https://review.openstack.org/#/c/553794/ merges? | 19:05 |
| *** janki has quit IRC | 19:05 | |
| superdan | fried_rice: intermittently I think | 19:05 |
| fried_rice | mm. Well, I was gonna wait to recheck mine, which is underneath his, so nyah. | 19:06 |
| superdan | well, whatever, I was trying to help | 19:06 |
| fried_rice | :) | 19:06 |
| superdan | gonna go cry in my corner now | 19:06 |
| fried_rice | What does that make the score? efried 1, dansmith 10000? | 19:07 |
| *** mingyu has quit IRC | 19:07 | |
| fried_rice | I'm making my comeback | 19:07 |
| mriedem | gah we don't have a Dict<?> field? | 19:07 |
| superdan | mriedem: DictOfStrings | 19:07 |
| mriedem | yeah but i've got a big ol bag of goodies | 19:08 |
| mriedem | i know what to do, | 19:08 |
| superdan | mriedem: a generic dict just makes it easy to throw complex types in and call it versioned | 19:08 |
| mriedem | i'm just complaining | 19:08 |
| mriedem | i know | 19:08 |
| mriedem | bag-o-dicts | 19:08 |
| * superdan slaps mriedem with a bag-o-dicts | 19:08 | |
| * mriedem enters his safe sapce | 19:08 | |
| mriedem | *space | 19:08 |
| mriedem | my typing has been really terrible lately | 19:09 |
| mriedem | talking to sean-k-mooney too much | 19:09 |
| *** yamamoto has joined #openstack-nova | 19:10 | |
| *** mingyu has joined #openstack-nova | 19:12 | |
| *** hemna_ has quit IRC | 19:12 | |
| *** felipemonteiro__ has joined #openstack-nova | 19:13 | |
| *** chyka has quit IRC | 19:14 | |
| *** mingyu has quit IRC | 19:15 | |
| *** crushil_ has joined #openstack-nova | 19:15 | |
| *** felipemonteiro__ has quit IRC | 19:16 | |
| *** felipemonteiro__ has joined #openstack-nova | 19:17 | |
| *** felipemonteiro_ has quit IRC | 19:17 | |
| *** wolverineav has joined #openstack-nova | 19:18 | |
| *** yamamoto has quit IRC | 19:20 | |
| mriedem | same g-api failed to start error in that job http://logs.openstack.org/08/553608/4/check/neutron-tempest-linuxbridge/0a5acff/logs/subnode-2/devstacklog.txt.gz#_2018-03-16_19_15_53_511 | 19:21 |
| mriedem | looking at devstack-gate, when it sets up multinode connectivity between the two hosts, it looks like it runs some ansible playbooks and those all assume ovs | 19:24 |
| superdan | mriedem: assume how? | 19:25 |
| *** beagles is now known as beagles_biab | 19:26 | |
| mriedem | https://github.com/openstack-infra/devstack-gate/tree/master/playbooks/roles | 19:26 |
| mriedem | although it looks like those are only used for nova-net multinode | 19:29 |
| mriedem | or if using dvr | 19:29 |
| mriedem | which this job isn't | 19:29 |
| *** andreas_s has joined #openstack-nova | 19:29 | |
| *** andreas_s has quit IRC | 19:33 | |
| *** lpetrut has quit IRC | 19:37 | |
| *** suresh12 has quit IRC | 19:38 | |
| mriedem | the subnode code is trying to hit g-api on the primary node, not the subnode where g-api is also running | 19:38 |
| *** mingyu has joined #openstack-nova | 19:39 | |
| *** mingyu has quit IRC | 19:39 | |
| *** sdague has quit IRC | 19:42 | |
| *** r-daneel_ has joined #openstack-nova | 19:45 | |
| *** maciejjozefczyk has quit IRC | 19:45 | |
| *** r-daneel has quit IRC | 19:47 | |
| *** r-daneel_ is now known as r-daneel | 19:47 | |
| superdan | aaaand, etherpad discussions fail. | 19:48 |
| superdan | fried_rice: we might need a way to draw a cut line on a long series and say "this is what we expect to get done during this window".. it'd be nice if that was always a whole blueprint, but maybe that's not always realistic | 19:49 |
| fried_rice | superdan: Yes, exactly so. | 19:50 |
| *** AlexeyAbashkin has joined #openstack-nova | 19:50 | |
| fried_rice | For two reasons: 1) reviewer bandwidth, 2) reasonable expectation to have the whole series completely ready at the same time. | 19:50 |
| fried_rice | There was a time when the upt series was over a dozen patches tall. | 19:50 |
| superdan | entirely possible that early patches will be reviewed to oblivion making later patches irrelevant until substantially changed | 19:50 |
| fried_rice | Yes, that also. | 19:51 |
| superdan | well, the point of this is to maintain focus so we can chew through long series easier, | 19:51 |
| superdan | so maybe it just needs to be "have approximately two weeks of stuff to review at the head of your series" | 19:51 |
| superdan | nobody will ever be able to measure that of course | 19:51 |
| fried_rice | No, but we'll get better at it with practice. | 19:52 |
| superdan | and hopefully most blueprints will wholly fit into two weeks, but.. | 19:52 |
| fried_rice | "Bottom N patches of bp X" | 19:52 |
| fried_rice | kind of thing. | 19:52 |
| mriedem | it's likely a discussion that happens when talking about the next thing to go into a slot, | 19:52 |
| superdan | well, aim for whole blueprints, but for special problem children like you... | 19:52 |
| mriedem | for single-patch small bp's, it's not an issue | 19:52 |
| superdan | yeah | 19:52 |
| mriedem | "is bp x ready for a slot? if so, are you going to be around for the next 2 weeks to respond to reviews? if so, i see that it's 20 patches already, what is the reasonable target?" | 19:53 |
| mriedem | this shouldn't be all that process heavy | 19:54 |
| mriedem | says the guy that loves process | 19:54 |
| superdan | yeah for serious | 19:54 |
| *** AlexeyAbashkin has quit IRC | 19:55 | |
| *** crushil_ has quit IRC | 19:55 | |
| mriedem | hey, | 19:55 |
| superdan | I expect we wing this for a cycle, and decide either to scrap it, or it becomes clear what some of the rules should be | 19:55 |
| mriedem | when it's not *my* process i don't care as much | 19:55 |
| superdan | heh | 19:55 |
| jgwentworth | I'll ping gibi_ and bauzas next week to add their feedback to this too | 19:57 |
| jgwentworth | since they're on PTO this week | 19:58 |
| superdan | jgwentworth: might want to clean up the pad a bit and move discussion to the bottom or something | 19:58 |
| jgwentworth | and it's friday late for them | 19:58 |
| superdan | it's a fairly confusing read right now | 19:58 |
| fried_rice | ++ | 19:59 |
| fried_rice | I'm done mucking with it; have at r. | 19:59 |
| superdan | a summary at the top or somethimg | 19:59 |
| jgwentworth | yeah, I was debating whether to copy-paste the whole thing into a -discussion etherpad (should have done that first) and leave this as the trimmed down thing | 19:59 |
| fried_rice | jgwentworth: I meant to mention, wouldn't mind if the definition was the first thing. | 19:59 |
| fried_rice | ("What is a runway?") | 19:59 |
| jgwentworth | oh, yeah, okay | 20:00 |
| *** suresh12 has joined #openstack-nova | 20:01 | |
| *** sree has joined #openstack-nova | 20:07 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:10 | |
| *** maciejjozefczyk has joined #openstack-nova | 20:11 | |
| *** sree has quit IRC | 20:11 | |
| *** germs has joined #openstack-nova | 20:13 | |
| *** felipemonteiro__ has quit IRC | 20:14 | |
| *** suresh12 has quit IRC | 20:14 | |
| openstackgerrit | Merged openstack/nova master: Update contributor guide for Rocky https://review.openstack.org/550288 | 20:15 |
| openstackgerrit | Merged openstack/nova master: Removed unnecessary parantheses in yield statements https://review.openstack.org/550379 | 20:15 |
| *** maciejjozefczyk has quit IRC | 20:16 | |
| *** yamamoto has joined #openstack-nova | 20:17 | |
| *** germs has quit IRC | 20:18 | |
| *** yamamoto has quit IRC | 20:22 | |
| *** pchavva has quit IRC | 20:27 | |
| *** moshele has joined #openstack-nova | 20:28 | |
| *** hemna_ has joined #openstack-nova | 20:29 | |
| *** oomichi has joined #openstack-nova | 20:31 | |
| *** artom has quit IRC | 20:32 | |
| *** chyka has joined #openstack-nova | 20:35 | |
| *** chyka has quit IRC | 20:36 | |
| *** chyka has joined #openstack-nova | 20:37 | |
| mriedem | cfriesen: any idea if you can live migrate between compute hosts with different libvirt virt_types today? like qemu->kvm and back? | 20:38 |
| mriedem | that pair might work, but i'm pretty sure something like kvm->xen wouldn't | 20:41 |
| *** hemna_ has quit IRC | 20:42 | |
| *** claudiub|2 has quit IRC | 20:47 | |
| *** awaugama has quit IRC | 20:48 | |
| *** moshele has quit IRC | 20:51 | |
| jgwentworth | superdan: this is a patch having to do with the hypervisor_type in the libvirt driver. it looked reasonable to me but I wasn't 100% sure whether filtering hypervisor_type by virt_type changes the semantics of hypervisor_type/capabilities. I'd appreciate your input about it https://review.openstack.org/#/c/531347/ | 20:52 |
| *** gjayavelu has quit IRC | 20:56 | |
| *** owalsh has quit IRC | 20:57 | |
| *** owalsh has joined #openstack-nova | 20:58 | |
| *** gjayavelu has joined #openstack-nova | 20:59 | |
| *** maciejjozefczyk has joined #openstack-nova | 20:59 | |
| *** dave-mccowan has quit IRC | 21:01 | |
| *** owalsh_ has joined #openstack-nova | 21:01 | |
| *** edmondsw has quit IRC | 21:03 | |
| *** owalsh- has joined #openstack-nova | 21:03 | |
| *** owalsh has quit IRC | 21:03 | |
| *** felipemonteiro_ has quit IRC | 21:05 | |
| *** owalsh_ has quit IRC | 21:06 | |
| *** gjayavelu has quit IRC | 21:10 | |
| *** owalsh- has quit IRC | 21:10 | |
| oomichi | hi, I have a question about Date(2012-09-27), Version(2012.1) on OpenStack Doc like https://docs.openstack.org/nova/latest/cli/nova-api-metadata.html | 21:10 |
| oomichi | They make me feel the doc seems super old, I am not sure we need to update or remove them | 21:11 |
| mriedem | oomichi: those have always annoyed me too | 21:12 |
| oomichi | mriedem: hehe, let me check the other projects too | 21:12 |
| mriedem | i'm not sure if they are needed for man page builds | 21:12 |
| mriedem | or were just originally informational | 21:12 |
| mriedem | https://docs.openstack.org/cinder/latest/man/cinder-manage.html | 21:13 |
| mriedem | cinder doesn't have the version or date in theirs | 21:13 |
| oomichi | mriedem: cool, thanks for picking it up | 21:14 |
| oomichi | it would be nice to send a mail to get answer from doc team | 21:14 |
| cfriesen | mriedem: I've never tried live migration between different libvirt types...I'd expect it to fail. | 21:14 |
| oomichi | will do that | 21:14 |
| mriedem | apparently there is a whole spec http://man7.org/linux/man-pages/man7/man-pages.7.html | 21:15 |
| mriedem | but i'm sure we don't follow the actual man page spec closely | 21:15 |
| oomichi | btw cinder's "Manual group:cloud computing" seems a little odd | 21:15 |
| *** owalsh has joined #openstack-nova | 21:15 | |
| mriedem | likely copied from nova | 21:15 |
| mriedem | when cinder was split out | 21:15 |
| mtreinish | oomichi, mriedem: it looks like it's just a table in the rst: https://git.openstack.org/cgit/openstack/nova/tree/doc/source/cli/nova-api.rst#n9 | 21:15 |
| smcginnis | Any man page specific stuff probably hasn't been touched since then. | 21:15 |
| mtreinish | the man pages are just sphinx rendering the rst: https://github.com/openstack/nova/blob/master/doc/source/conf.py#L137 with the manpage builder | 21:16 |
| *** gouthamr has quit IRC | 21:16 | |
| mtreinish | I just spent a bunch of time reformatting: https://stestr.readthedocs.io/en/latest/MANUAL.html to be a proper manpage when using that sphinx builder | 21:17 |
| mtreinish | I can give you a hand with that if it's something you'd want to do | 21:17 |
| *** yamamoto has joined #openstack-nova | 21:18 | |
| oomichi | mtreinish: I might want to remove them (Date, Version) if we can | 21:19 |
| mtreinish | yeah, I don't think that table adds anything. I think just push a patch to delete it, or change it to a comment so it doesn't render in the output | 21:19 |
| oomichi | mtreinish: thanks, let me try | 21:21 |
| *** yamamoto has quit IRC | 21:23 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.update_from_provider_tree https://review.openstack.org/533821 | 21:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 21:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix nits in update_provider_tree series https://review.openstack.org/531260 | 21:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Move refresh time from report client to prov tree https://review.openstack.org/535517 | 21:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Make generation optional in ProviderTree https://review.openstack.org/539324 | 21:24 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Add nested resources to server moving tests https://review.openstack.org/527728 | 21:24 |
| oomichi | mriedem: mtreinish: cinder team has removes Date and Version on the same reason which we feel: https://review.openstack.org/#/c/325465/ | 21:26 |
| *** AlexeyAbashkin has joined #openstack-nova | 21:29 | |
| *** itlinux has joined #openstack-nova | 21:31 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Clean up man page and remove version/date https://review.openstack.org/553903 | 21:32 |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove version/date from CLI documantation https://review.openstack.org/553903 | 21:33 |
| oomichi | mriedem: ^^^: done | 21:33 |
| *** fullmetaljackiet has quit IRC | 21:34 | |
| *** AlexeyAbashkin has quit IRC | 21:34 | |
| *** suresh12 has joined #openstack-nova | 21:39 | |
| mriedem | +2 | 21:40 |
| mriedem | POST_FAILURE on https://review.openstack.org/#/c/553794/ so it's going to have to get rechecked | 21:40 |
| *** suresh12 has quit IRC | 21:44 | |
| *** suresh12 has joined #openstack-nova | 21:46 | |
| *** elmaciej has quit IRC | 21:46 | |
| *** elmaciej has joined #openstack-nova | 21:47 | |
| *** itlinux has quit IRC | 21:48 | |
| *** sususuryashines has quit IRC | 21:48 | |
| *** suresh12 has quit IRC | 21:50 | |
| *** suresh12 has joined #openstack-nova | 21:50 | |
| *** suresh12 has quit IRC | 21:54 | |
| *** maciejjozefczyk has quit IRC | 21:54 | |
| *** suresh12 has joined #openstack-nova | 21:56 | |
| mriedem | cfriesen: found the code that won't allow it | 21:57 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L210 | 21:58 |
| mriedem | in the conductor task | 21:58 |
| mriedem | also checks the version | 21:58 |
| fried_rice | jgwentworth, superdan: Are we rolling out runways yet? I put one in the queue :) | 22:00 |
| *** suresh12 has quit IRC | 22:01 | |
| oomichi | thanks | 22:01 |
| jgwentworth | fried_rice: heh, you're ready2rock. gonna ping gibi and bauzas on monday and see if they have anything to add/change before unleashing it to the dev ML | 22:01 |
| fried_rice | cool cool. Meanwhile, same patches are in the rocky priorities etherpad, so I figger I'm covered :) | 22:02 |
| jgwentworth | speaking of, while you're here oomichi if you have any feedback about our rough draft about runways for rocky https://etherpad.openstack.org/p/nova-runways-rocky | 22:02 |
| jgwentworth | I'm sorry this hasn't been mentioned yet as I haven't written the PTG summary for the friday miscellaenous bag of topics yet | 22:02 |
| jgwentworth | *miscellaneous | 22:03 |
| *** suresh12 has joined #openstack-nova | 22:03 | |
| jgwentworth | alex_xu_ ^ | 22:03 |
| *** itlinux has joined #openstack-nova | 22:03 | |
| mriedem | jgwentworth: if you're going to attempt to summarize the friday misc stuff from the ptg, i'd recommend not trying to cover every single item, and only focus on highlights for any bigger or more contentious items | 22:04 |
| mriedem | otherwise the summary will essentually be the etherpad itself | 22:04 |
| jgwentworth | mriedem: yeah, I was wondering about that. thanks for the advice | 22:04 |
| oomichi | jgwentworth: OK, let me check. (that is already huge btw :-) | 22:04 |
| mriedem | i never tried to summarize all the misc stuff before | 22:04 |
| jgwentworth | good to know | 22:05 |
| *** chyka has quit IRC | 22:07 | |
| *** imacdonn has quit IRC | 22:07 | |
| *** aloga has quit IRC | 22:07 | |
| *** knikolla has quit IRC | 22:07 | |
| *** lifeless has quit IRC | 22:07 | |
| *** Guest63078 has quit IRC | 22:07 | |
| *** hongbin has quit IRC | 22:07 | |
| *** mlavalle has quit IRC | 22:07 | |
| *** gaurangt has quit IRC | 22:07 | |
| *** ccamacho has quit IRC | 22:07 | |
| *** slaweq has quit IRC | 22:07 | |
| *** krtaylor has quit IRC | 22:07 | |
| *** sshwarts has quit IRC | 22:07 | |
| *** jbernard has quit IRC | 22:07 | |
| *** yassine has quit IRC | 22:07 | |
| *** chyka has joined #openstack-nova | 22:07 | |
| *** imacdonn has joined #openstack-nova | 22:07 | |
| *** aloga has joined #openstack-nova | 22:07 | |
| *** knikolla has joined #openstack-nova | 22:07 | |
| *** lifeless has joined #openstack-nova | 22:07 | |
| *** Guest63078 has joined #openstack-nova | 22:07 | |
| *** hongbin has joined #openstack-nova | 22:07 | |
| *** mlavalle has joined #openstack-nova | 22:07 | |
| *** gaurangt has joined #openstack-nova | 22:07 | |
| *** ccamacho has joined #openstack-nova | 22:07 | |
| *** slaweq has joined #openstack-nova | 22:07 | |
| *** krtaylor has joined #openstack-nova | 22:07 | |
| *** sshwarts has joined #openstack-nova | 22:07 | |
| *** jbernard has joined #openstack-nova | 22:07 | |
| *** yassine has joined #openstack-nova | 22:07 | |
| *** lifeless has quit IRC | 22:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | 22:08 |
| 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:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: compute: use port binding extended API during live migration https://review.openstack.org/551371 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 22:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Turn on new port binding extended live migrate flow https://review.openstack.org/552173 | 22:08 |
| jgwentworth | superdan: ty for reviewing the hypervisor_type patch | 22:08 |
| *** lifeless has joined #openstack-nova | 22:08 | |
| *** r-daneel has quit IRC | 22:08 | |
| mriedem | superdan: i'll bug you about this on monday https://review.openstack.org/#/c/515423/9/nova/objects/migrate_data.py@186 but i assume it's because the field is defined in the parent class rather than each subclass | 22:10 |
| mriedem | so my option is moving that field into all subclasses | 22:10 |
| mriedem | which is easy and mechanical | 22:10 |
| *** chyka has quit IRC | 22:13 | |
| *** germs has joined #openstack-nova | 22:14 | |
| *** germs has quit IRC | 22:14 | |
| *** germs has joined #openstack-nova | 22:14 | |
| *** salv-orlando has quit IRC | 22:15 | |
| *** salv-orlando has joined #openstack-nova | 22:16 | |
| *** figleaf is now known as edleafe | 22:16 | |
| *** germs has quit IRC | 22:18 | |
| jgwentworth | TheJulia, jroll, mriedem: here's the nova/ironic PTG summary if you have anything to add/correct before I send it out to the ML https://etherpad.openstack.org/p/nova-ptg-rocky-ironic-summary | 22:18 |
| *** yamamoto has joined #openstack-nova | 22:20 | |
| *** yamamoto has quit IRC | 22:26 | |
| superdan | jgwentworth: yep | 22:33 |
| superdan | mriedem: left a comment, will help you look on monday | 22:33 |
| *** Guest63078 has quit IRC | 22:36 | |
| *** amodi has quit IRC | 22:44 | |
| *** liverpooler has quit IRC | 22:45 | |
| *** vladikr has quit IRC | 22:45 | |
| *** chyka has joined #openstack-nova | 22:46 | |
| * fried_rice waves | 22:47 | |
| *** fried_rice is now known as efried | 22:48 | |
| *** hongbin has quit IRC | 22:48 | |
| *** chyka has quit IRC | 22:49 | |
| openstackgerrit | Ken'ichi Ohmichi proposed openstack/nova master: Remove version/date from CLI documantation https://review.openstack.org/553903 | 22:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add --by-service to discover_hosts https://review.openstack.org/552691 | 22:55 |
| *** rhagarty has quit IRC | 22:57 | |
| *** superdan is now known as dansmith | 22:58 | |
| *** chyka has joined #openstack-nova | 23:02 | |
| openstackgerrit | Merged openstack/nova master: Follow the new PTI for document build https://review.openstack.org/553751 | 23:09 |
| *** yamamoto has joined #openstack-nova | 23:22 | |
| *** yamamoto has quit IRC | 23:27 | |
| *** wolverineav has quit IRC | 23:39 | |
| *** wolverineav has joined #openstack-nova | 23:39 | |
| *** r-daneel has joined #openstack-nova | 23:47 | |
| *** openstackgerrit has quit IRC | 23:48 | |
| *** jgwentworth is now known as melwitt | 23:48 | |
| *** chyka has quit IRC | 23:49 | |
| *** gjayavelu has joined #openstack-nova | 23:51 | |
| *** r-daneel has quit IRC | 23:51 | |
| *** gyee has quit IRC | 23:55 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!