| *** gyee has quit IRC | 00:08 | |
| *** dklyle has joined #openstack-nova | 00:12 | |
| *** gbarros has joined #openstack-nova | 00:13 | |
| *** edmondsw has joined #openstack-nova | 00:13 | |
| *** dklyle has quit IRC | 00:14 | |
| *** dklyle has joined #openstack-nova | 00:14 | |
| *** tbachman has joined #openstack-nova | 00:15 | |
| *** gbarros has quit IRC | 00:15 | |
| *** edmondsw has quit IRC | 00:18 | |
| *** yingjun has joined #openstack-nova | 00:24 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 00:29 |
|---|---|---|
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 00:29 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151 | 00:29 |
| *** yingjun has quit IRC | 00:30 | |
| *** tssurya has quit IRC | 00:30 | |
| *** yingjun has joined #openstack-nova | 00:36 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:36 | |
| *** liuzz has joined #openstack-nova | 00:41 | |
| *** mikal_ has joined #openstack-nova | 00:45 | |
| *** Dinesh_Bhor has quit IRC | 00:45 | |
| *** oomichi has quit IRC | 00:47 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:48 | |
| *** mikal has quit IRC | 00:49 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 00:56 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151 | 00:56 |
| *** Dinesh_Bhor has joined #openstack-nova | 01:00 | |
| *** jmlowe has joined #openstack-nova | 01:01 | |
| *** hshiina has joined #openstack-nova | 01:06 | |
| *** moshele has quit IRC | 01:11 | |
| *** zhaochao has joined #openstack-nova | 01:12 | |
| *** sq4ind has quit IRC | 01:13 | |
| *** sapd has joined #openstack-nova | 01:24 | |
| *** phuongnh has joined #openstack-nova | 01:27 | |
| openstackgerrit | Merged openstack/nova master: Convert configdrive to use processutils. https://review.openstack.org/565723 | 01:30 |
| *** Nel1x has joined #openstack-nova | 01:34 | |
| *** hongbin_ has joined #openstack-nova | 01:39 | |
| openstackgerrit | Erik Olof Gunnar Andersson proposed openstack/nova master: Remove remaning log translation in scheduler https://review.openstack.org/567455 | 01:43 |
| *** lei-zh has joined #openstack-nova | 01:50 | |
| *** edmondsw has joined #openstack-nova | 02:01 | |
| *** nicolasbock has quit IRC | 02:01 | |
| *** liverpooler has joined #openstack-nova | 02:03 | |
| *** tiendc has joined #openstack-nova | 02:04 | |
| *** edmondsw has quit IRC | 02:06 | |
| *** wwriverrat has quit IRC | 02:10 | |
| *** tinwood has quit IRC | 02:10 | |
| *** wwriverrat has joined #openstack-nova | 02:12 | |
| *** tinwood has joined #openstack-nova | 02:12 | |
| *** mdnadeem has joined #openstack-nova | 02:18 | |
| *** links has joined #openstack-nova | 02:28 | |
| *** andreas_s_ has joined #openstack-nova | 02:33 | |
| *** andreas_s_ has quit IRC | 02:37 | |
| *** sq4ind has joined #openstack-nova | 02:39 | |
| *** liverpooler has quit IRC | 02:43 | |
| *** psachin` has joined #openstack-nova | 02:45 | |
| *** yamamoto has joined #openstack-nova | 02:55 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 02:56 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 02:56 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 02:56 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 02:56 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 02:56 |
| openstackgerrit | Merged openstack/nova master: Add tests for alloc cands with poor local disk https://review.openstack.org/566842 | 02:56 |
| *** vladikr has quit IRC | 03:00 | |
| *** vladikr has joined #openstack-nova | 03:01 | |
| *** vladikr has quit IRC | 03:10 | |
| *** tiendc has quit IRC | 03:10 | |
| *** vladikr has joined #openstack-nova | 03:10 | |
| *** tiendc has joined #openstack-nova | 03:12 | |
| *** dave-mccowan has quit IRC | 03:27 | |
| *** annp has joined #openstack-nova | 03:32 | |
| *** takashin has quit IRC | 03:33 | |
| *** annp has quit IRC | 03:36 | |
| *** annp has joined #openstack-nova | 03:37 | |
| *** annp_ has joined #openstack-nova | 03:37 | |
| *** annp_ has quit IRC | 03:38 | |
| *** annp has quit IRC | 03:38 | |
| *** hongbin_ has quit IRC | 03:40 | |
| *** links has quit IRC | 03:43 | |
| *** lpetrut has joined #openstack-nova | 03:44 | |
| *** edmondsw has joined #openstack-nova | 03:50 | |
| *** edmondsw has quit IRC | 03:54 | |
| *** fragatina has quit IRC | 03:55 | |
| *** fragatina has joined #openstack-nova | 03:56 | |
| *** gyankum has joined #openstack-nova | 03:56 | |
| *** gyan_ has joined #openstack-nova | 03:56 | |
| *** fragatina has quit IRC | 04:00 | |
| *** yingjun has quit IRC | 04:10 | |
| *** wwriverrat has quit IRC | 04:11 | |
| *** takashin has joined #openstack-nova | 04:16 | |
| *** lpetrut has quit IRC | 04:24 | |
| *** mdnadeem has quit IRC | 04:24 | |
| *** Nel1x has quit IRC | 04:25 | |
| *** udesale has joined #openstack-nova | 04:27 | |
| *** moshele has joined #openstack-nova | 04:30 | |
| *** moshele has quit IRC | 04:33 | |
| *** mdnadeem has joined #openstack-nova | 04:35 | |
| *** hamzy has joined #openstack-nova | 04:38 | |
| openstackgerrit | Merged openstack/nova master: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/555945 | 04:39 |
| *** wolverineav has joined #openstack-nova | 04:43 | |
| *** links has joined #openstack-nova | 04:49 | |
| *** mdnadeem has quit IRC | 04:53 | |
| *** moshele has joined #openstack-nova | 04:54 | |
| *** mdnadeem has joined #openstack-nova | 04:55 | |
| *** Dinesh_Bhor has quit IRC | 05:00 | |
| *** moshele has quit IRC | 05:01 | |
| *** ratailor has joined #openstack-nova | 05:03 | |
| *** janki has joined #openstack-nova | 05:03 | |
| *** rcernin has joined #openstack-nova | 05:05 | |
| *** rmk has quit IRC | 05:05 | |
| *** abhishekk has joined #openstack-nova | 05:08 | |
| *** yamamoto has quit IRC | 05:09 | |
| *** yamamoto has joined #openstack-nova | 05:10 | |
| *** wolverineav has quit IRC | 05:10 | |
| *** wolverineav has joined #openstack-nova | 05:11 | |
| *** wolverineav has quit IRC | 05:13 | |
| *** wolverin_ has joined #openstack-nova | 05:13 | |
| *** wolverin_ has quit IRC | 05:15 | |
| *** wolverineav has joined #openstack-nova | 05:15 | |
| *** mdnadeem_ has joined #openstack-nova | 05:17 | |
| *** ccamacho has quit IRC | 05:17 | |
| *** suresh12 has joined #openstack-nova | 05:19 | |
| *** mdnadeem has quit IRC | 05:20 | |
| *** ratailor_ has joined #openstack-nova | 05:21 | |
| *** fragatina has joined #openstack-nova | 05:22 | |
| *** fragatina has quit IRC | 05:23 | |
| *** fragatina has joined #openstack-nova | 05:23 | |
| *** ratailor has quit IRC | 05:25 | |
| *** lei-zh has quit IRC | 05:28 | |
| *** lei-zh has joined #openstack-nova | 05:30 | |
| *** Roamer` has quit IRC | 05:33 | |
| *** tbachman_ has joined #openstack-nova | 05:33 | |
| *** tbachman has quit IRC | 05:36 | |
| *** tbachman_ is now known as tbachman | 05:36 | |
| *** trinaths has joined #openstack-nova | 05:37 | |
| *** rm_work has quit IRC | 05:37 | |
| *** rm_work has joined #openstack-nova | 05:37 | |
| *** edmondsw has joined #openstack-nova | 05:38 | |
| *** trinaths has quit IRC | 05:38 | |
| *** yingjun has joined #openstack-nova | 05:40 | |
| *** edmondsw has quit IRC | 05:42 | |
| openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 05:45 |
| *** suresh12 has quit IRC | 05:47 | |
| *** Dinesh_Bhor has joined #openstack-nova | 05:48 | |
| *** mdnadeem_ has quit IRC | 05:52 | |
| *** moshele has joined #openstack-nova | 05:52 | |
| *** wolverin_ has joined #openstack-nova | 05:56 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 05:57 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 05:57 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 05:57 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 05:57 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 05:57 |
| *** ratailor__ has joined #openstack-nova | 05:57 | |
| *** swamireddy has quit IRC | 05:58 | |
| *** jhesketh has quit IRC | 05:58 | |
| *** wolverineav has quit IRC | 05:58 | |
| *** Guest83896 has quit IRC | 05:58 | |
| *** hjensas has quit IRC | 05:58 | |
| *** ratailor_ has quit IRC | 06:00 | |
| *** markmcclain has quit IRC | 06:00 | |
| *** markmcclain has joined #openstack-nova | 06:00 | |
| *** xinliang has joined #openstack-nova | 06:03 | |
| *** mdnadeem_ has joined #openstack-nova | 06:04 | |
| *** rajinir has quit IRC | 06:05 | |
| *** avolkov has joined #openstack-nova | 06:07 | |
| *** Tom-Tom has joined #openstack-nova | 06:10 | |
| *** yingjun has quit IRC | 06:11 | |
| *** wolverin_ has quit IRC | 06:12 | |
| *** ccamacho has joined #openstack-nova | 06:14 | |
| *** fragatin_ has joined #openstack-nova | 06:17 | |
| *** trinaths has joined #openstack-nova | 06:18 | |
| *** fragati__ has joined #openstack-nova | 06:18 | |
| openstackgerrit | Hamdy Khader proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 06:19 |
| *** fragatina has quit IRC | 06:20 | |
| *** armaan has quit IRC | 06:20 | |
| *** armaan has joined #openstack-nova | 06:21 | |
| *** fragatin_ has quit IRC | 06:21 | |
| *** fragati__ has quit IRC | 06:27 | |
| *** trinaths has quit IRC | 06:36 | |
| *** trungnv has quit IRC | 06:38 | |
| *** trungnv has joined #openstack-nova | 06:38 | |
| *** phuongnh has quit IRC | 06:39 | |
| *** phuongnh has joined #openstack-nova | 06:40 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | 06:41 |
| *** trinaths has joined #openstack-nova | 06:43 | |
| *** sridharg has joined #openstack-nova | 06:47 | |
| *** trungnv has quit IRC | 06:49 | |
| *** trungnv has joined #openstack-nova | 06:49 | |
| *** lpetrut has joined #openstack-nova | 06:49 | |
| *** yamamoto has quit IRC | 06:49 | |
| *** andreas_s has joined #openstack-nova | 06:49 | |
| *** pcaruana has joined #openstack-nova | 06:51 | |
| *** mdnadeem_ has quit IRC | 06:52 | |
| *** andreas_s has quit IRC | 06:54 | |
| *** slaweq has quit IRC | 06:55 | |
| *** slaweq has joined #openstack-nova | 06:55 | |
| *** wolverineav has joined #openstack-nova | 06:55 | |
| *** Dinesh_Bhor has quit IRC | 06:58 | |
| *** threestrands_ has quit IRC | 06:58 | |
| *** yamamoto has joined #openstack-nova | 06:59 | |
| *** annp has joined #openstack-nova | 07:00 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:04 | |
| *** annp has quit IRC | 07:07 | |
| *** MrWatson has quit IRC | 07:10 | |
| *** NostawRm has joined #openstack-nova | 07:10 | |
| *** wolverineav has quit IRC | 07:10 | |
| *** mdnadeem_ has joined #openstack-nova | 07:10 | |
| *** bswrchrd has quit IRC | 07:11 | |
| *** bswrchrd has joined #openstack-nova | 07:11 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689 | 07:16 |
| *** aloga has joined #openstack-nova | 07:21 | |
| *** armaan has quit IRC | 07:22 | |
| *** armaan has joined #openstack-nova | 07:22 | |
| *** Tom-Tom has quit IRC | 07:23 | |
| *** tesseract has joined #openstack-nova | 07:24 | |
| *** zcorneli has quit IRC | 07:25 | |
| *** jangutter has quit IRC | 07:25 | |
| *** edmondsw has joined #openstack-nova | 07:26 | |
| *** edmondsw has quit IRC | 07:30 | |
| *** trungnv has quit IRC | 07:32 | |
| *** trungnv has joined #openstack-nova | 07:32 | |
| *** wolverineav has joined #openstack-nova | 07:34 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | 07:36 |
| *** jpena|off is now known as jpena | 07:40 | |
| *** wolverineav has quit IRC | 07:41 | |
| *** armaan has quit IRC | 07:44 | |
| *** armaan has joined #openstack-nova | 07:44 | |
| *** suresh12 has joined #openstack-nova | 07:48 | |
| *** annp has joined #openstack-nova | 07:48 | |
| *** suresh12 has quit IRC | 07:52 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | 07:55 |
| *** lucas-afk is now known as lucasagomes | 07:55 | |
| *** annp has quit IRC | 07:57 | |
| *** jangutter has joined #openstack-nova | 07:58 | |
| *** mgoddard has joined #openstack-nova | 08:00 | |
| *** vivsoni has quit IRC | 08:02 | |
| *** jmccarthy has joined #openstack-nova | 08:04 | |
| *** toabctl has quit IRC | 08:04 | |
| *** rmart04 has joined #openstack-nova | 08:05 | |
| *** annp has joined #openstack-nova | 08:06 | |
| *** toabctl has joined #openstack-nova | 08:08 | |
| *** Tom-Tom has joined #openstack-nova | 08:10 | |
| *** dougsz has joined #openstack-nova | 08:10 | |
| openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: fix against pep8 https://review.openstack.org/567502 | 08:11 |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: Add random sleep between retry calls to placement https://review.openstack.org/567506 | 08:20 |
| *** imacdonn has quit IRC | 08:23 | |
| *** imacdonn has joined #openstack-nova | 08:23 | |
| *** ratailor__ has quit IRC | 08:25 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 08:25 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 08:25 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Fix broken allocation candidates with traits https://review.openstack.org/567151 | 08:25 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 08:25 |
| *** swamireddy has joined #openstack-nova | 08:29 | |
| *** jhesketh has joined #openstack-nova | 08:29 | |
| *** avolkov has quit IRC | 08:30 | |
| *** yingjun has joined #openstack-nova | 08:30 | |
| *** tiendc has quit IRC | 08:31 | |
| *** zhaochao has quit IRC | 08:34 | |
| *** lbragstad[m] has quit IRC | 08:36 | |
| *** vivsoni has joined #openstack-nova | 08:36 | |
| *** sean-k-mooney[m] has quit IRC | 08:36 | |
| *** Kevin_Zheng has quit IRC | 08:43 | |
| *** Kevin_Zheng has joined #openstack-nova | 08:43 | |
| *** derekh has joined #openstack-nova | 08:45 | |
| *** hiro-kobayashi has quit IRC | 08:47 | |
| *** swamireddy has quit IRC | 08:47 | |
| *** jhesketh has quit IRC | 08:47 | |
| *** udesale has quit IRC | 08:47 | |
| *** udesale has joined #openstack-nova | 08:47 | |
| *** andreas_s has joined #openstack-nova | 08:48 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | 08:51 |
| *** mdbooth has joined #openstack-nova | 08:54 | |
| *** hiro-kobayashi has joined #openstack-nova | 08:54 | |
| *** guest___ has joined #openstack-nova | 08:57 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid warning log when image not exist https://review.openstack.org/567522 | 08:57 |
| guest___ | hi | 08:57 |
| guest___ | can someone help me with GPU? | 08:57 |
| guest___ | virtualization? | 08:57 |
| *** trinaths has quit IRC | 09:01 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: deprecate the config for image handler class path https://review.openstack.org/497201 | 09:03 |
| *** swamireddy has joined #openstack-nova | 09:06 | |
| *** jhesketh has joined #openstack-nova | 09:06 | |
| *** johnthetubaguy has quit IRC | 09:09 | |
| *** johnthetubaguy has joined #openstack-nova | 09:10 | |
| *** edmondsw has joined #openstack-nova | 09:14 | |
| moshele | stephenfin: hi | 09:14 |
| stephenfin | moshele: o/ | 09:14 |
| moshele | stephenfin: how are you? long time since we talked :) | 09:15 |
| stephenfin | Can't complain. Keeping busy :) | 09:15 |
| moshele | stephenfin: the nvme stuff was merged in cinder last night https://review.openstack.org/#/c/505556/ can you review the nova side https://review.openstack.org/#/c/482640/ ? | 09:16 |
| stephenfin | I sure can | 09:17 |
| moshele | stephenfin: thank :) are you going to summit ? | 09:17 |
| stephenfin | Yup! Got a talk on NUMA fun (which I really need to finish). You? | 09:18 |
| moshele | stephenfin: me too, so I will see you there. I need to catch up on nova project :) | 09:19 |
| *** edmondsw has quit IRC | 09:19 | |
| openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 09:24 |
| *** hiro-kobayashi has quit IRC | 09:25 | |
| *** trinaths has joined #openstack-nova | 09:27 | |
| *** wolverineav has joined #openstack-nova | 09:28 | |
| *** lbragstad[m] has joined #openstack-nova | 09:30 | |
| *** udesale has quit IRC | 09:31 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832 | 09:31 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy to InstacenGroup object and api models. https://review.openstack.org/563375 | 09:31 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add policy field to ServerGroup notification object https://review.openstack.org/563401 | 09:31 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 09:31 |
| *** yingjun has quit IRC | 09:36 | |
| *** cdent has joined #openstack-nova | 09:40 | |
| *** stakeda has quit IRC | 09:42 | |
| *** masayukig has joined #openstack-nova | 09:43 | |
| guest___ | hi guys | 09:44 |
| *** wolverineav has quit IRC | 09:48 | |
| *** lei-zh has quit IRC | 09:51 | |
| *** tbachman_ has joined #openstack-nova | 10:07 | |
| *** tbachman has quit IRC | 10:08 | |
| *** tbachman_ is now known as tbachman | 10:08 | |
| *** yamamoto has quit IRC | 10:10 | |
| *** RaoulHC has joined #openstack-nova | 10:10 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 10:12 |
| *** sean-k-mooney[m] has joined #openstack-nova | 10:13 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | 10:13 |
| *** yamamoto has joined #openstack-nova | 10:17 | |
| *** tbachman_ has joined #openstack-nova | 10:18 | |
| *** tbachman has quit IRC | 10:19 | |
| *** tbachman_ is now known as tbachman | 10:19 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 10:22 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: document change for supporting image handler https://review.openstack.org/563077 | 10:25 |
| *** udesale has joined #openstack-nova | 10:27 | |
| *** udesale has quit IRC | 10:28 | |
| *** udesale has joined #openstack-nova | 10:28 | |
| openstackgerrit | Merged openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 10:29 |
| *** armaan has quit IRC | 10:29 | |
| *** armaan has joined #openstack-nova | 10:32 | |
| *** Dinesh_Bhor has quit IRC | 10:33 | |
| *** sean-k-mooney has quit IRC | 10:37 | |
| *** abhishekk has quit IRC | 10:43 | |
| *** evin has quit IRC | 10:48 | |
| *** mvenesio has joined #openstack-nova | 10:50 | |
| *** eharney has quit IRC | 10:50 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (2) https://review.openstack.org/567557 | 10:52 |
| *** vladikr has quit IRC | 10:54 | |
| *** vladikr has joined #openstack-nova | 10:55 | |
| *** jmlowe has quit IRC | 10:57 | |
| *** tbachman has quit IRC | 11:00 | |
| *** takashin has left #openstack-nova | 11:00 | |
| *** edmondsw has joined #openstack-nova | 11:02 | |
| *** elod has quit IRC | 11:05 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: Microversion 2.63 - Use new format policy in server group https://review.openstack.org/567534 | 11:05 |
| *** elod has joined #openstack-nova | 11:05 | |
| *** edmondsw has quit IRC | 11:07 | |
| *** lucasagomes is now known as lucas-hungry | 11:09 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: VMware: Live migration of instances https://review.openstack.org/270116 | 11:11 |
| *** nicolasbock has joined #openstack-nova | 11:12 | |
| *** yikun__ has quit IRC | 11:16 | |
| *** Nel1x has joined #openstack-nova | 11:17 | |
| *** udesale_ has joined #openstack-nova | 11:18 | |
| *** udesale_ has quit IRC | 11:18 | |
| *** udesale_ has joined #openstack-nova | 11:18 | |
| *** udesale_ has quit IRC | 11:18 | |
| *** udesale_ has joined #openstack-nova | 11:19 | |
| *** udesale has quit IRC | 11:21 | |
| *** bkopilov_ has quit IRC | 11:22 | |
| *** bkopilov has quit IRC | 11:23 | |
| *** yamamoto has quit IRC | 11:23 | |
| *** eharney has joined #openstack-nova | 11:27 | |
| *** Tom-Tom has quit IRC | 11:29 | |
| *** Tom-Tom has joined #openstack-nova | 11:29 | |
| *** rcernin has quit IRC | 11:31 | |
| *** jchhatbar has joined #openstack-nova | 11:34 | |
| *** Tom-Tom has quit IRC | 11:34 | |
| *** Nel1x has quit IRC | 11:35 | |
| *** janki has quit IRC | 11:35 | |
| *** bkopilov_ has joined #openstack-nova | 11:35 | |
| *** bkopilov has joined #openstack-nova | 11:36 | |
| *** jchhatba_ has joined #openstack-nova | 11:38 | |
| *** jchhatbar has quit IRC | 11:41 | |
| *** mvenesio has quit IRC | 11:41 | |
| *** cdent has quit IRC | 11:42 | |
| *** mvenesio has joined #openstack-nova | 11:42 | |
| *** StevenK_ has joined #openstack-nova | 11:44 | |
| *** StevenK has quit IRC | 11:44 | |
| *** StevenK_ is now known as StevenK | 11:44 | |
| *** armaan has quit IRC | 11:44 | |
| *** armaan has joined #openstack-nova | 11:45 | |
| *** yamamoto has joined #openstack-nova | 11:47 | |
| *** jchhatba_ has quit IRC | 11:51 | |
| *** cdent has joined #openstack-nova | 11:51 | |
| *** bhujay has joined #openstack-nova | 11:52 | |
| bhagyashri_s | cdent: Hi, I am trying to write the functional test case for change https://review.openstack.org/#/c/560459 . I have written functinal test for the change # link http://paste.openstack.org/show/720747/ in /nova/tests/functional/test_servers.py | 11:55 |
| *** lucas-hungry is now known as lucasagomes | 11:56 | |
| cdent | hi bhagyashri_s. is it not doing what you want? | 11:56 |
| *** wolverineav has joined #openstack-nova | 11:57 | |
| cdent | bhagyashri_s: for what you are trying to test, a functional test in test_servers is probably not what you want | 12:02 |
| *** sambetts|afk is now known as sambetts | 12:02 | |
| cdent | because what you're trying to test is that the virt driver is creating the right data (in the ProviderTree), not create resource providers | 12:03 |
| cdent | bhagyashri_s: basically what Eric says in his comment | 12:03 |
| *** StevenK has quit IRC | 12:04 | |
| *** jpena is now known as jpena|lunch | 12:06 | |
| *** phuongnh has quit IRC | 12:10 | |
| bhagyashri_s | cdent: Firstly the change is regarding the shared resource provider thing . When user configure the shared resource provider then at that time the DISK_GB inventory should report from shared rp and not from the compute node rp | 12:10 |
| *** amoralej is now known as amoralej|lunch | 12:10 | |
| bhagyashri_s | cdent: and for that libvirt driver update_provider_tree() removes the DISK_GB inventory from the cn rp if the shared rp is configure | 12:11 |
| * cdent nods | 12:12 | |
| cdent | so in your test you should need to create a server. instead you just need to confirm that the correct resource providers and inventory are created | 12:12 |
| cdent | sorry: shouldn't need! | 12:12 |
| *** mvk has quit IRC | 12:14 | |
| bhagyashri_s | cdent so in my functional test case i am trying to update by calling the resource tackers update_provider_tree which basically updates the inventory internally using def _update() method which calls the drivers update_provider_tree() method and in my test case how should i instantiate the LibvirtDriver | 12:14 |
| *** StevenK has joined #openstack-nova | 12:16 | |
| openstackgerrit | Petersingh Anburaj proposed openstack/nova master: Making consistent used of GiB and MiB in Doc https://review.openstack.org/559985 | 12:17 |
| *** tbachman has joined #openstack-nova | 12:17 | |
| cdent | bhagyashri_s It sounds like you are trying to test from too high a level. Maybe try: Set up the sharing resource provider and put it an aggregate. Then create a ProviderTree representing a compute node, put it in aggregate too, then pass that to the method on the virt driver and make sure that the provider tree that comes back doesn't have local disk. I'm not sure of the setup you'll need to do for that, without doing a | 12:18 |
| cdent | more digging | 12:18 |
| cdent | if/when efried comes back you might check with him, but the virt drivers are an area I've done very little work with | 12:19 |
| *** tssurya has joined #openstack-nova | 12:21 | |
| *** pchavva has joined #openstack-nova | 12:22 | |
| openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Follow the new PTI for document build https://review.openstack.org/551802 | 12:23 |
| jroll | someone wake up mriedem before I forget to tell him a thing :P | 12:23 |
| *** mvenesio has quit IRC | 12:23 | |
| *** felipemonteiro has joined #openstack-nova | 12:23 | |
| *** mvenesio has joined #openstack-nova | 12:24 | |
| cdent | destroy all irc | 12:25 |
| *** felipemonteiro_ has joined #openstack-nova | 12:27 | |
| efried | ō/ | 12:27 |
| bhagyashri_s | cdent: ok ty | 12:27 |
| efried | reading back... | 12:27 |
| *** AlexeyAbashkin has joined #openstack-nova | 12:28 | |
| *** felipemonteiro has quit IRC | 12:29 | |
| efried | cdent, bhagyashri_s: Yes we want a unit test, but we also want a functional test. The functional test should do just what your paste shows. When we talked about this a couple of days ago, I offered to help you debug it IF you posted your WIP test case to the change set so I could download it and be at the same baseline, rather than copy/pasting from pastebin and trying to figure out where to put it and how to run it. | 12:33 |
| *** Roamer` has joined #openstack-nova | 12:36 | |
| *** AlexeyAbashkin has quit IRC | 12:37 | |
| *** lyan has joined #openstack-nova | 12:37 | |
| bhagyashri_s | efried: because of some environmental issue iam not able to push patches so just paste the code in paste.openstack.org | 12:37 |
| *** lyan is now known as Guest81722 | 12:38 | |
| *** evin has joined #openstack-nova | 12:38 | |
| *** trinaths has quit IRC | 12:39 | |
| bhagyashri_s | efried: once it will resolved i will push it, I catch the issue in my functional test case, as the change is in libvirt code so through the functional test case the driver is not loaded and that's why the inventory is not properly updated so my doubt is how i will initiate the driver so that it report the correct inventory if the shared resource provider is configured | 12:42 |
| bhagyashri_s | efried: sorry for inconvenience i am not able to push the patch | 12:43 |
| bauzas | FWIW, folks I'm on PTO till end of the week | 12:43 |
| bauzas | call it May-hem | 12:43 |
| efried | bhagyashri_s: okay, can you pastebin the whole file including imports and tell me the path where you put it? | 12:44 |
| bauzas | efried: jaypipes: cdent: I'm on slides mode for my YVR talk but something came out of my mind. Do we plan to have quota resources using Placement resource classes ? | 12:47 |
| *** liverpooler has joined #openstack-nova | 12:47 | |
| bauzas | efried: jaypipes: cdent: for example, we now have VGPU RC | 12:47 |
| *** AlexeyAbashkin has joined #openstack-nova | 12:47 | |
| bauzas | efried: jaypipes: cdent: I guess operators would love to get quotas for vGPUs | 12:47 |
| bauzas | so, how to help them ? | 12:47 |
| efried | I still don't know what a quota is | 12:47 |
| *** AlexeyAbashkin has quit IRC | 12:48 | |
| bauzas | efried: it's old story | 12:48 |
| bauzas | efried: https://docs.openstack.org/nova/pike/admin/quotas.html | 12:48 |
| bhagyashri_s | efried: #link http://paste.openstack.org/show/720752/ path: nova/tests/functional/test_servers.py | 12:48 |
| *** janki has joined #openstack-nova | 12:49 | |
| *** diga has joined #openstack-nova | 12:49 | |
| bauzas | efried: and https://docs.openstack.org/nova/latest/user/quotas.html | 12:50 |
| *** edmondsw_ has joined #openstack-nova | 12:51 | |
| *** liverpooler has quit IRC | 12:51 | |
| *** liverpooler has joined #openstack-nova | 12:52 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: ironic: Support resize and cold migration https://review.openstack.org/500677 | 12:53 |
| *** bswrchrd has quit IRC | 12:55 | |
| *** hshiina is now known as hshiina|afk | 12:58 | |
| *** bhujay has quit IRC | 13:02 | |
| *** pcaruana|afk| has joined #openstack-nova | 13:03 | |
| *** NotPcaruana has joined #openstack-nova | 13:04 | |
| *** vladikr has quit IRC | 13:05 | |
| *** gyankum has quit IRC | 13:05 | |
| *** gyan_ has quit IRC | 13:05 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: Rename parameters in call() of nova/image/glance.py https://review.openstack.org/508533 | 13:05 |
| *** vladikr has joined #openstack-nova | 13:05 | |
| *** avolkov has joined #openstack-nova | 13:08 | |
| *** tssurya has quit IRC | 13:16 | |
| moshele | melwitt: hi, the cinder changes for nvmeof is merged in cinder so can you review the nova change https://review.openstack.org/#/c/482640/ | 13:17 |
| *** tssurya has joined #openstack-nova | 13:17 | |
| efried | bauzas: So you're thinking about implementing quotas in placement, such that a user and/or project ID is allowed only a certain amount of a given resource class? | 13:21 |
| bauzas | efried: well, yes | 13:22 |
| efried | bauzas: It seems like a reasonable idea. I don't recall having heard of any plans along those lines, but that doesn't mean it's not on the radar for someone. | 13:22 |
| efried | bauzas: queue it up for Denver? | 13:22 |
| bauzas | efried: my point is about the fact we created a new resource class, VGPU | 13:23 |
| bauzas | efried: now the scheduler verifies it | 13:23 |
| jaypipes | bauzas: that's a good discussion. I'd like to have melwitt in it, though. | 13:23 |
| efried | agree | 13:23 |
| jaypipes | bauzas: shall we wait for her? | 13:23 |
| bauzas | jaypipes: sure, but I'm in vacations atm :) | 13:23 |
| bauzas | in the beach :) | 13:23 |
| *** psachin` has quit IRC | 13:23 | |
| bauzas | so when she's here, not sure I'll be here too :p | 13:24 |
| *** abhishekk has joined #openstack-nova | 13:24 | |
| bauzas | but anyway | 13:24 |
| bauzas | nothing really important atm | 13:24 |
| bauzas | it's more for my talk | 13:24 |
| jaypipes | bauzas: alex_xu was also working on quotas and placement things | 13:24 |
| bauzas | gtk | 13:24 |
| bauzas | AFAIK, we don't support custom quotas, right? | 13:24 |
| jaypipes | bauzas: I'm not sure any more. | 13:26 |
| bauzas | yup | 13:26 |
| bauzas | it was possible previously | 13:26 |
| bauzas | but when we provided the quotas for cells v2, then we stopped that | 13:26 |
| bauzas | that's what I remember at least | 13:26 |
| jaypipes | bauzas: right. I was under the impression that we (nova) would start using the placement API for various usage queries. but I don't think that's been done yet. | 13:31 |
| *** jistr is now known as jistr|mtg | 13:32 | |
| *** vladikr has quit IRC | 13:32 | |
| *** vladikr has joined #openstack-nova | 13:33 | |
| *** tssurya has quit IRC | 13:34 | |
| *** NotPcaruana has quit IRC | 13:34 | |
| *** pcaruana|afk| has quit IRC | 13:34 | |
| jaypipes | Kevin_Zheng: lmao 'All of a sudden, everyone uses the word "granular"' | 13:35 |
| jaypipes | efried: ^ | 13:36 |
| *** mriedem has joined #openstack-nova | 13:37 | |
| mriedem | stephenfin: bauzas: can one of you take a look at this? https://review.openstack.org/#/c/566367/ needs a final +2 and then we need to backport it through to ocata | 13:38 |
| bauzas | mriedem: I'm on PTO today and tomorrow, but see if I can help | 13:38 |
| *** dklyle has quit IRC | 13:40 | |
| *** moshele has quit IRC | 13:40 | |
| jroll | jaypipes: bauzas: yeah, I know it's on alex_xu's list to do quotas on resource classes, not sure if he has anything proposed upstream yet | 13:43 |
| stephenfin | mriedem: Sure | 13:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add NUMANetworkInfo object https://review.openstack.org/564439 | 13:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add '[neutron] physnets' and related options https://review.openstack.org/564440 | 13:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: libvirt: Start populating NUMACell.network_info field https://review.openstack.org/564441 | 13:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add NetworkRequest.physnet, tunnelled https://review.openstack.org/564442 | 13:43 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Unchain _get_phynet_info from _get_port_vnic_info https://review.openstack.org/564443 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Add 'populate_networks' to network API https://review.openstack.org/564444 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: network: Add '_get_network_tunnel_status' https://review.openstack.org/564445 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: db: Add network_requests to instance_extra https://review.openstack.org/564446 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add Instance.network_requests https://review.openstack.org/564447 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: hardware: Start accounting for network requests https://review.openstack.org/564448 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: rt: Start utilizing Instance.network_requests https://review.openstack.org/564449 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Add RequestSpec.requested_networks https://review.openstack.org/564450 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: api: Remove unnecessary default parameter https://review.openstack.org/564451 | 13:44 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: scheduler: Start utilizing network_requests https://review.openstack.org/564452 | 13:44 |
| jaypipes | jroll: last I heard, he was focusing on quota-by-aggregate and ran into a number of issues... | 13:44 |
| jroll | jaypipes: yar, he's working through it | 13:44 |
| jaypipes | jroll: sorry, had to wait until stephenfin's patch-cloud passed. | 13:44 |
| jroll | heh | 13:44 |
| stephenfin | ☁ ☁ ☁ | 13:44 |
| jaypipes | which reminds me I need to complete reviews on those. :) | 13:44 |
| stephenfin | jaypipes: Would be appreciated. Fixed the Tempest issue (thanks gibi/mriedem) and addressed what comments were there so far | 13:45 |
| jaypipes | gibi: I've read your ML post about the instance migration UUID thing three times and looked at the map_instances() function. and I still don't get it :( | 13:45 |
| stephenfin | jaypipes: we're using a munged UUID as a marker so we catch batch up the migrations | 13:46 |
| *** links has quit IRC | 13:46 | |
| *** eharney has quit IRC | 13:46 | |
| *** andreas_s has quit IRC | 13:47 | |
| stephenfin | cos it's munged, it's not a valid UUID and oslo.whatever complains | 13:47 |
| *** andreas_s has joined #openstack-nova | 13:47 | |
| *** tssurya has joined #openstack-nova | 13:47 | |
| jaypipes | stephenfin: right, but I don't understand why we're doing the replace('-','') thing at all | 13:47 |
| stephenfin | edleafe tried to fix it before but dansmith found the solution distressing | 13:47 |
| *** jpena|lunch is now known as jpena | 13:47 | |
| stephenfin | because there's a UNIQUE constraint on that column | 13:47 |
| stephenfin | if we didn't do that, it would clash with the instance (or whatever that UUID is taken from) | 13:48 |
| jaypipes | stephenfin: ok, but why are we changing it? | 13:48 |
| jaypipes | stephenfin: ah... so all migrations have the same uuid? | 13:48 |
| *** eharney has joined #openstack-nova | 13:48 | |
| stephenfin | jaypipes: My memory on this is hazy but I don't think that's it | 13:49 |
| stephenfin | The revert of the edleafe's solution is here 9038738d0134c0a1d59757ed3205d84f76416869 | 13:49 |
| stephenfin | (commit ID) | 13:49 |
| tssurya | jaypipes: we use this as a marker for nova-manage map_instances | 13:49 |
| jaypipes | stephenfin: k, thx for the commit ID. I'll look it up and see if I can understand the problem better. | 13:51 |
| stephenfin | Oh, yeah, now I remember | 13:51 |
| stephenfin | So we store a special string for InstanceMappingList.project_id | 13:51 |
| stephenfin | and then for the UUID value, we store the list of the last InstanceMappingList that we processed | 13:51 |
| *** mlavalle has joined #openstack-nova | 13:52 | |
| stephenfin | but munged, so it won't conflict with that InstanceMappingList's UUID | 13:52 |
| dansmith | I haven't read the thread, | 13:52 |
| *** mvk has joined #openstack-nova | 13:52 | |
| dansmith | but we can't change this unless you provide a migration path for people that might be in the middle of the process when they apply the patch | 13:52 |
| *** andreas_s has quit IRC | 13:53 | |
| dansmith | and if this is just because of the damn o.vo UUIDField warning (which I didn't want in the first place) I think it's not worth a change | 13:53 |
| *** mrf has joined #openstack-nova | 13:55 | |
| mrf | Hi, | 13:55 |
| mrf | After a Resize i got Instance at Error with the next drop in nova-compute: InstanceNotFound: Instance 299fd82f-f081-4c81-8226-70d208f10a8f could not be found. any ideas? | 13:55 |
| *** hongbin has joined #openstack-nova | 13:55 | |
| *** armaan has quit IRC | 14:01 | |
| *** armaan has joined #openstack-nova | 14:02 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Explain how the marker works for instance-cell mapping https://review.openstack.org/567597 | 14:05 |
| *** tbachman has quit IRC | 14:05 | |
| stephenfin | gibi, dansmith: ^ | 14:05 |
| mriedem | bauzas: fyi in case you haven't seen this yet https://openstack-in-production.blogspot.com/2018/05/introducing-gpus-to-cern-cloud.html | 14:08 |
| *** tetsuro has joined #openstack-nova | 14:08 | |
| *** awaugama has joined #openstack-nova | 14:08 | |
| *** r-daneel has joined #openstack-nova | 14:08 | |
| bauzas | mriedem: just discussed on twitter with tim | 14:08 |
| bauzas | but thanks | 14:09 |
| *** esberglu has joined #openstack-nova | 14:09 | |
| *** yamamoto has quit IRC | 14:10 | |
| openstackgerrit | Eric Berglund proposed openstack/nova stable/queens: Stringify instance UUID https://review.openstack.org/567599 | 14:13 |
| tetsuro | jaypipes, efried: Could you have a look at https://review.openstack.org/#/c/567113/5, if you have time? | 14:13 |
| *** jistr|mtg is now known as jistr | 14:17 | |
| *** lpetrut has quit IRC | 14:20 | |
| *** andreas_s has joined #openstack-nova | 14:22 | |
| *** tetsuro has quit IRC | 14:22 | |
| *** rajinir has joined #openstack-nova | 14:23 | |
| *** andreas_s has quit IRC | 14:27 | |
| *** edmondsw_ is now known as edmondsw | 14:28 | |
| *** cdent has quit IRC | 14:34 | |
| *** diga has quit IRC | 14:34 | |
| *** ccamacho1 has joined #openstack-nova | 14:35 | |
| *** ccamacho has quit IRC | 14:35 | |
| *** cdent has joined #openstack-nova | 14:35 | |
| *** dklyle has joined #openstack-nova | 14:36 | |
| mriedem | dansmith: question in https://review.openstack.org/#/c/509003/ about how the cell_mappings kwarg gets through to the DB API query method, i.e. why you need to do this https://review.openstack.org/#/c/509003/12/nova/tests/unit/api/openstack/fakes.py | 14:42 |
| mriedem | otherwise the rest are nits which can be deal with in a follow up | 14:42 |
| mriedem | *dealt | 14:42 |
| *** sean-k-mooney has joined #openstack-nova | 14:42 | |
| dansmith | ack, will look in a bit | 14:43 |
| sean-k-mooney | i have an interesting (annoying) issue with my teams openstack devcluster ... what does the value of the deleted column in the nova.instnaces table mean? | 14:46 |
| dansmith | sean-k-mooney: deleted=id means the instance is deleted | 14:46 |
| dansmith | deleted=0 means it's not | 14:46 |
| sean-k-mooney | dansmith: thanks. i have an instance that shows up in opentack server list but cant be found when i do a openstack server delete or nova force-delete | 14:47 |
| sean-k-mooney | dansmith: that was exactly what i was looking for ill see if that "deletes it" | 14:47 |
| dansmith | maybe it's missing a mapping? | 14:47 |
| sean-k-mooney | dansmith: or when my database crashed because the binary logs filled up the disk it currupted the entries... | 14:48 |
| dansmith | or that :) | 14:48 |
| sean-k-mooney | dansmith: it was mapped in the nova_api db. i tried deleteing it from there to see if it would be enough but it looks like i have to remove it from the cell0 db too or at least mark it as deleted | 14:49 |
| dansmith | yeah, list will still find it without a mapping unless it's actually deleted | 14:49 |
| *** ccamacho1 has quit IRC | 14:50 | |
| sean-k-mooney | dansmith: its currently messsing with my nodepool deployment because it keeps trying to delete it and cant hence my db hacking | 14:50 |
| dansmith | ack | 14:50 |
| *** felipemonteiro_ has quit IRC | 14:50 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567612 | 14:51 |
| *** mingyu has joined #openstack-nova | 14:53 | |
| *** lpetrut has joined #openstack-nova | 14:54 | |
| jroll | mriedem: fyi, https://review.openstack.org/567577 | 14:55 |
| jroll | thanks for pointing me at that bug, is not good | 14:55 |
| *** janki has quit IRC | 14:56 | |
| openstackgerrit | Eric Berglund proposed openstack/nova stable/queens: Stringify instance UUID https://review.openstack.org/567599 | 14:56 |
| mriedem | jroll: ok, are there also going to need to be nova changes? | 14:57 |
| jroll | mriedem: not in our driver, I'm not sure what you all do / need to do to shut down the connection on instance destroy | 14:57 |
| mriedem | jroll: that reply is confusing | 14:59 |
| jroll | lemme try again | 14:59 |
| tssurya | mriedem: thanks for triaging https://bugs.launchpad.net/nova/+bug/1768876 , shall I go ahead with the migration tool then for populating the ones that are missing? | 14:59 |
| mriedem | if there is nothing to do in the ironic driver, then i don't think there is anything to change in nova | 14:59 |
| mriedem | tssurya: i was just going to ask you about that, | 14:59 |
| openstack | Launchpad bug 1768876 in OpenStack Compute (nova) queens "metadata-api fails to get availability zone for instances created before pike" [High,Triaged] | 15:00 |
| mriedem | tssurya: it will require some thought, because i'm not sure what got put into instance.availability_zone before that change in pike, | 15:00 |
| *** mvenesio has quit IRC | 15:00 | |
| mriedem | i.e. is it None if the user doesn't specify an AZ when creating the instance? or does it default to 'nova'? | 15:00 |
| tssurya | wasn't it None ? | 15:00 |
| jroll | mriedem: the driver gives back some console URL, right? and nova's proxy thing connects to it? does that connection need to be closed or do we just let it die? if it needs to be closed, does nova do that already? (I guess this is outside the scope of the bug, but could be an improvement) | 15:00 |
| mriedem | i'm not sure, but it will matter if we add an online data migration routine since we'll need to know which instances need to be migrated | 15:00 |
| tssurya | mriedem: yea, I will try to dig more into the tree then | 15:01 |
| mriedem | jroll: not sure, probably a good question for melwitt | 15:01 |
| jroll | mriedem: ok, I'm sure it isn't urgent anyway. can mark that closes-bug instead if you like, but gonna wait for some reviews/CI results first | 15:02 |
| *** andreas_s has joined #openstack-nova | 15:02 | |
| *** abhishekk has quit IRC | 15:02 | |
| mriedem | i'll mark invalid for nova, | 15:03 |
| mriedem | and just point to the SB entry | 15:03 |
| *** jmlowe has joined #openstack-nova | 15:03 | |
| jroll | wfm | 15:04 |
| mriedem | unrelated, but looks like issues in storyboard aren't found via google search? | 15:04 |
| jroll | no clue | 15:05 |
| *** mgoddard has quit IRC | 15:05 | |
| jroll | that would be sad | 15:05 |
| mriedem | like, trump level sad? | 15:05 |
| mriedem | hash tag sad?! | 15:05 |
| jroll | SAD! | 15:06 |
| *** andreas_s has quit IRC | 15:07 | |
| *** janki has joined #openstack-nova | 15:07 | |
| artom | MALA! | 15:09 |
| *** mgoddard has joined #openstack-nova | 15:09 | |
| artom | Make Launchpad Great Again! | 15:09 |
| melwitt | jroll: the driver hands back a URL essentially that points at the console proxy. then the end user visits it and that initiates the connection to the proxy. AFAIK it's closed when the end user closes the console. I'm not aware that nova ever does anything to close it | 15:09 |
| melwitt | I like launchpad fine, never understood to push for storyboard | 15:10 |
| jroll | melwitt: ok cool, thanks | 15:10 |
| artom | NIH? | 15:10 |
| mriedem | melwitt: shhh | 15:10 |
| mriedem | short answer on SB indexing in google is no | 15:11 |
| mriedem | not yet | 15:11 |
| mriedem | see -dev for details | 15:11 |
| *** yamamoto has joined #openstack-nova | 15:11 | |
| melwitt | mriedem: I didn't think we'd need a blueprint for this https://blueprints.launchpad.net/nova/+spec/add-documentation-rules-api-ref ? | 15:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567617 | 15:15 |
| mriedem | melwitt: ha no | 15:15 |
| *** david-lyle has joined #openstack-nova | 15:17 | |
| *** david-lyle has quit IRC | 15:18 | |
| *** amoralej|lunch is now known as amoralej | 15:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567619 | 15:18 |
| *** yamamoto has quit IRC | 15:19 | |
| *** sridharg has quit IRC | 15:19 | |
| mriedem | melwitt: just leave a comment that you don't need a blueprint to write contributor docs and mark the bp obsolete since that's as close to invalid as we get in launchpad | 15:19 |
| *** jogo has quit IRC | 15:19 | |
| *** armaan has quit IRC | 15:20 | |
| melwitt | mriedem: k thanks | 15:20 |
| mriedem | i also don't really want to review said contributor doc additions, but... | 15:20 |
| *** armaan has joined #openstack-nova | 15:20 | |
| mriedem | simply linking from the contributor docs to the api-ref wiki would be good enough IMO | 15:20 |
| *** _pewp_ has quit IRC | 15:22 | |
| melwitt | okay, I'll suggest that on the bp | 15:24 |
| *** shaohe_feng has quit IRC | 15:24 | |
| *** alex_xu has quit IRC | 15:24 | |
| *** lpetrut has quit IRC | 15:27 | |
| *** dklyle has quit IRC | 15:28 | |
| *** NostawRm has quit IRC | 15:28 | |
| *** NostawRm has joined #openstack-nova | 15:28 | |
| *** wwriverrat has joined #openstack-nova | 15:30 | |
| *** suresh12 has joined #openstack-nova | 15:32 | |
| *** dklyle has joined #openstack-nova | 15:33 | |
| *** wwriverrat has quit IRC | 15:35 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/567623 | 15:39 |
| mdbooth | mriedem: Thanks for the review, btw. Working on addressing your comments, without embarking on another big cleanup. | 15:39 |
| mriedem | mdbooth: np; i thought i had figured out why the compute manager test in the 3rd or 4th patch in the series was failing (due to the fake driver wanting legacy bdms) but that didn't fix it locally for me | 15:40 |
| *** moshele has joined #openstack-nova | 15:41 | |
| mdbooth | Yeah, I got to the bottom of that. It's an annoying testing artifact. | 15:41 |
| *** wwriverrat has joined #openstack-nova | 15:42 | |
| mdbooth | The initializer pre-populates the dict subclass with a bunch of fields. This isn't required in the new class as it has only proxy fields, and can't be done for all proxied fields because is_volume is a property and isn't writable. | 15:43 |
| dansmith | mriedem: that stub_instance thing is because of a failure, not because I was being overly thorough :) | 15:43 |
| mdbooth | Anyway, I'll come up with something. Thanks again. | 15:43 |
| dansmith | mriedem: I'll remove and run tests to highlight which one it was | 15:43 |
| *** jogo has joined #openstack-nova | 15:44 | |
| dansmith | I think it's probably fping or something, which mocks something with the stub_instance that would have shadowed the inner call we make with cell_mappings | 15:44 |
| mriedem | dansmith: ok, as noted i didn't run all tests, just a sample, but anyway | 15:44 |
| dansmith | mriedem: yup | 15:44 |
| mriedem | dansmith: do you plan on doing a cleanup followup? otherwise i'm happy to. | 15:45 |
| mdbooth | I suspect I'm going to end up with some test-only code in block_device.py, though, because the tests need some way to serialise a whole object for comparison, whereas runtime code just accesses fields by name. | 15:45 |
| mriedem | since they're my nits | 15:45 |
| dansmith | mriedem: I haven't looked at all your comments, but if there are legit ones I'll do it yeah | 15:45 |
| mriedem | mdbooth: we shouldn't need test-only code in block_device.py. if we need special serialize code for those types of objects, surely we could house that in test utils separately | 15:46 |
| mriedem | e.g. we have special comparison methods for versioned objects | 15:46 |
| mdbooth | mriedem: It might come to that, but that's a bit of big cleanup I was hoping to avoid. | 15:46 |
| mdbooth | I think I'd end up having to poke a bunch of block_device tests. | 15:47 |
| mdbooth | But yeah, that was my first thought. | 15:47 |
| mriedem | ok i guess i don't know the root issue and what the fix would be to understand why a lot of stuff would have to change just for comparing local image bdms | 15:47 |
| *** suresh12 has quit IRC | 15:48 | |
| mdbooth | Well, there's already a bunch of test-only code in that module as it happens, it's just not obvious that it's test only. | 15:48 |
| mdbooth | I guess I could sneak it in :) | 15:49 |
| *** lpetrut has joined #openstack-nova | 15:49 | |
| mdbooth | Basically anything which calls self.update() in _transform for a proxy attribute is test-only. | 15:50 |
| mdbooth | But... I really don't want to clean it all up right now :) | 15:50 |
| mriedem | ok, so can't you follow suit for the new local image bdm? | 15:50 |
| mdbooth | mriedem: I can, yes, but I was going to be honest and mark it as test-only :) | 15:52 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L226 and https://github.com/openstack/nova/blob/master/nova/virt/block_device.py#L242 could clearly be changed to just use self._fields to avoid the duplicatoin | 15:52 |
| *** pcaruana has quit IRC | 15:52 | |
| dansmith | mriedem: 8 fails: https://pastebin.com/jUSj03qW | 15:52 |
| mdbooth | mriedem: Yep. | 15:52 |
| mriedem | gd fping | 15:53 |
| dansmith | ikr | 15:53 |
| mriedem | dansmith: looks like it's time i delete that code which i've been meaning to anyway | 15:53 |
| dansmith | \o/ | 15:53 |
| mriedem | as a start for the nova-net REST API delete pattern | 15:53 |
| mriedem | will do that in a bit | 15:53 |
| * mriedem realizes some stole the pen from his office and will pay dearly | 15:54 | |
| mriedem | *someone | 15:54 |
| mriedem | mtreinish: fyi, tox -e py27 -- <foo> doesn't work in nova in stable/pike, | 15:55 |
| mriedem | i think because of ostestr? | 15:55 |
| mriedem | or our tox.ini isn't passing the regex through properly | 15:55 |
| mtreinish | mriedem: yeah I think someone bumped constraints on pike to pull in os-testr 1.0.0 which broke the loophole nova was using for that (that was the thing mikal was complaining about in denver) | 15:56 |
| mriedem | https://github.com/openstack/nova/commit/bb7628fdf3b7481a0ca487cf6ede0babc4890ac5#diff-b91f3d5bd63fcd17221b267e851608e8 is in pike | 15:56 |
| mtreinish | I'm not sure why that was bumped on a stable branch though, but I've been pinged by a few people because of things that broke because of it | 15:56 |
| mtreinish | mriedem: in the short term just do tox -epy27 -- --regex <foo> | 15:57 |
| mriedem | doing that now | 15:57 |
| mriedem | that doesn't seem to work | 15:58 |
| mriedem | unless my regex is wrong | 15:58 |
| mriedem | tox -e py27 -- --regex test_cleanup_resize_not_same_host | 15:59 |
| mtreinish | oh, it's probably the quotes around posargs in the tox.ini | 16:00 |
| mtreinish | it probably is treating it as a single parameter with a space instead of a flag | 16:00 |
| mriedem | f it, zuul can tell me if i screwed up | 16:00 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/567625 | 16:00 |
| *** rmart04 has quit IRC | 16:02 | |
| *** _pewp_ has joined #openstack-nova | 16:04 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/567625 | 16:06 |
| mriedem | cfriesen_: i know you're using postgres, have you ever hit the issue described here? https://groups.google.com/forum/#!topic/sqlalchemy/sRgvpjUkzPU | 16:07 |
| mriedem | looks like eventlet shenanigans with pg and sqla | 16:07 |
| cfriesen_ | on a call, will take a look in a bit | 16:08 |
| *** germs has joined #openstack-nova | 16:10 | |
| *** germs has quit IRC | 16:10 | |
| *** germs has joined #openstack-nova | 16:10 | |
| *** mdnadeem_ has quit IRC | 16:11 | |
| *** gyee has joined #openstack-nova | 16:13 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/567630 | 16:17 |
| *** sambetts is now known as sambetts|afk | 16:21 | |
| tssurya | mriedem: looks like the avz setting if its not specified by the user depends on CONF.default_schedule_zone (https://github.com/openstack/nova/blob/23fce8c13f8347a8ad862ab38ae9d8259db21349/nova/compute/api.py#L487) | 16:22 |
| *** nicolasbock has quit IRC | 16:22 | |
| mriedem | tssurya: ok which defaults to None https://docs.openstack.org/nova/pike/configuration/config.html#DEFAULT.default_schedule_zone | 16:22 |
| tssurya | mriedem: yes, | 16:23 |
| mriedem | so, i kind of wonder how we didn't break the meaning / contract of None with the change in pike to always set AZ | 16:23 |
| mriedem | "* None, which means that the instance can move from one availability zone to another during its lifetime if it is moved from one compute node to another." | 16:23 |
| *** nicolasbock has joined #openstack-nova | 16:24 | |
| mriedem | i.e. before the change in pike, i could create an instance w/o specifying AZ and freely migrate to any compute node, but now if randomly land in AZ1, i'm bound to AZ1 even though i never specified that | 16:25 |
| mriedem | which also likely means operators are going to use the force flag when live migrating those types of instances | 16:25 |
| *** fragatin_ has joined #openstack-nova | 16:25 | |
| tssurya | mriedem: uh-huh, I see | 16:26 |
| mriedem | well, i assume that's true anyway, i haven't tried it, but it should be easy to reproduce with a functional test | 16:26 |
| mriedem | create 2 computes in separate AZs, create the instance w/o specifying AZ, and then live migrate it and see if scheduling fails | 16:27 |
| mriedem | if that works in ocata but not > ocata then we regressed | 16:27 |
| tssurya | mriedem: ack, I will look into this | 16:27 |
| *** mvenesio has joined #openstack-nova | 16:28 | |
| *** suresh12 has joined #openstack-nova | 16:29 | |
| *** masayuki- has joined #openstack-nova | 16:29 | |
| *** tbachman has joined #openstack-nova | 16:29 | |
| *** pcaruana has joined #openstack-nova | 16:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 16:37 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 16:37 |
| mriedem | efried: i cleaned those up so tests pass now ^ | 16:38 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Address feedback from instance_list smart-cell behavior https://review.openstack.org/567643 | 16:38 |
| dansmith | mriedem: ^ | 16:38 |
| efried | mriedem: ack | 16:39 |
| *** mvk has quit IRC | 16:41 | |
| *** suresh12 has quit IRC | 16:43 | |
| efried | mriedem: It doesn't look like you actually fixed https://review.openstack.org/#/c/557835/8/nova/tests/unit/network/test_utils.py | 16:43 |
| efried | mriedem: Unless I'm missing something obvious, you added the import but didn't change the offending references. | 16:43 |
| efried | So now you'll fail pep8 *and* py27/35 :) | 16:44 |
| *** pcaruana has quit IRC | 16:44 | |
| mriedem | you're blind | 16:44 |
| mriedem | everything passed locally | 16:44 |
| efried | mriedem: sorry, I'm blind, ignore me. | 16:44 |
| mriedem | efried: no joke https://review.openstack.org/#/c/558487/ | 16:46 |
| mriedem | sahid would be PISSED! | 16:46 |
| mriedem | apparently that doesn't have any tests? | 16:47 |
| mriedem | https://github.com/openstack/nova/commit/27614dc595d0fac2e57300c85bb34da266849a46 | 16:47 |
| efried | mriedem: I clearly looked into it https://review.openstack.org/#/c/558487/2/nova/network/linux_net.py@a1813 | 16:48 |
| efried | mriedem: Why doesn't my comment show up with yours? Seems like a gerrit ui bug. | 16:48 |
| sean-k-mooney | mriedem: we should proably port that to os-vif at some point rather then doing it in the libvirt vif.py | 16:48 |
| mriedem | sean-k-mooney: yes, both jaypipes and i asked for that in the original patch | 16:48 |
| efried | anyway, IIRC what I discovered was that that method had been removed somewhere else. | 16:48 |
| mriedem | so in his first change, he means to move create_tap_dev, but also moved the trusted vf tests, and then in the 2nd patch deletes the trusted_vf code but not the tests.... | 16:49 |
| sean-k-mooney | mriedem: adding a new pluging wont be two hard ill open a bug for it to track it. we also need to move a few others | 16:50 |
| efried | and yeah, I see that method in master. | 16:50 |
| mriedem | which is also what was causing the pep8/unit test failures | 16:50 |
| mriedem | so i'll fix this up | 16:50 |
| *** vladikr has quit IRC | 16:57 | |
| *** eharney_ has joined #openstack-nova | 16:57 | |
| *** vladikr has joined #openstack-nova | 16:57 | |
| *** dougsz has quit IRC | 16:59 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Address feedback from instance_list smart-cell behavior https://review.openstack.org/567643 | 16:59 |
| *** mdbooth has quit IRC | 17:00 | |
| *** derekh has quit IRC | 17:00 | |
| *** lucasagomes is now known as lucas-afk | 17:00 | |
| *** eharney has quit IRC | 17:01 | |
| *** edmondsw has quit IRC | 17:02 | |
| *** mrf has quit IRC | 17:02 | |
| *** andreas_s has joined #openstack-nova | 17:03 | |
| *** edmondsw has joined #openstack-nova | 17:03 | |
| *** moshele has quit IRC | 17:04 | |
| *** mgoddard has quit IRC | 17:07 | |
| *** andreas_s has quit IRC | 17:07 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 17:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 17:08 |
| mriedem | efried: ok those 2 should be golden now | 17:09 |
| efried | mriedem: ack | 17:10 |
| *** david-lyle has joined #openstack-nova | 17:11 | |
| *** udesale_ has quit IRC | 17:12 | |
| *** suresh12 has joined #openstack-nova | 17:13 | |
| *** suresh12 has quit IRC | 17:13 | |
| *** suresh12 has joined #openstack-nova | 17:13 | |
| *** dklyle has quit IRC | 17:14 | |
| *** david-lyle has quit IRC | 17:15 | |
| *** dklyle_ has joined #openstack-nova | 17:15 | |
| *** eharney_ is now known as eharney | 17:15 | |
| *** felipemonteiro has joined #openstack-nova | 17:17 | |
| *** suresh12 has quit IRC | 17:18 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:20 | |
| efried | mriedem: +2 on the first one, but didn't +W it - question inline. Can you confirm that that test case was added as part of this patch? | 17:21 |
| efried | (I don't see it in master or in the base) | 17:21 |
| cfriesen_ | mriedem: regarding the DB thing. Doesn't ring a bell, but I'm pretty sure we take down the DB, then switch the VIP, then bring up the DB again on the new side. | 17:22 |
| *** felipemonteiro has quit IRC | 17:24 | |
| cfriesen_ | cfriesen: we use shared storage rather than a master/slave postgres setup | 17:24 |
| cfriesen_ | mriedem: ^ (apparently I like talking to myself) | 17:25 |
| *** cfriesen_ is now known as cfriesen | 17:25 | |
| *** jmccarthy has left #openstack-nova | 17:29 | |
| cfriesen | is there a reason why we default to "novalocal" as the dhcp_domain even when using neutron? It means the hostname retrieved via metadata has ".novalocal" appended to it by default. | 17:32 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Add tests for alloc_cands with member_of https://review.openstack.org/561399 | 17:36 |
| mriedem | cfriesen: ack | 17:40 |
| mriedem | cfriesen: https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21754/novaneutron-ops-cross-project-session | 17:41 |
| *** mgoddard has joined #openstack-nova | 17:42 | |
| *** suresh12 has joined #openstack-nova | 17:44 | |
| *** Tom-Tom has joined #openstack-nova | 17:45 | |
| *** Tom-Tom has quit IRC | 17:45 | |
| *** suresh12 has quit IRC | 17:45 | |
| *** suresh12 has joined #openstack-nova | 17:45 | |
| *** dklyle_ has quit IRC | 17:46 | |
| *** janki has quit IRC | 17:48 | |
| mriedem | efried: yes it's new and it also looks wrong | 17:49 |
| mriedem | because ex.assert_has_calls(calls) looks like the wrong assertion, since execute should have been called twice | 17:50 |
| mriedem | so that should have probably been ex.return_value.assert_has_calls | 17:50 |
| mriedem | oh nvm, that's on set_device_mtu which is unrelated | 17:52 |
| *** rcernin has joined #openstack-nova | 17:52 | |
| efried | mriedem: ... yeah. | 17:52 |
| efried | so like, where tf did this test case come from? | 17:52 |
| efried | It's not wrong, it's just... random. | 17:52 |
| mriedem | probably rebase damage from I9872056c440a706b89dd51d3b9f2418951723efa | 17:53 |
| mriedem | yeah it was dropped here https://review.openstack.org/#/c/557543/1/nova/tests/unit/network/test_linux_net.py | 17:53 |
| mriedem | i'll update | 17:54 |
| efried | mriedem: Yup, I see it. | 17:54 |
| efried | mriedem: No need to update, what are you going to do, split it into a separate patch? | 17:54 |
| mriedem | yeah | 17:54 |
| efried | mriedem: Okay. FWIW I'd be fine just commenting in the review that it was accidentally dropped in a bad rebase, add that link ^, and be done. | 17:55 |
| efried | but if you prefer splitting, soit. | 17:55 |
| *** zcorneli has joined #openstack-nova | 17:56 | |
| *** suresh12_ has joined #openstack-nova | 17:57 | |
| openstackgerrit | Alvaro Lopez Garcia proposed openstack/nova master: Ensure that periodic reclaim cleans DB deleted instances https://review.openstack.org/323250 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 17:57 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add test_set_device_mtu_default back in https://review.openstack.org/567659 | 17:57 |
| *** dklyle_ has joined #openstack-nova | 17:58 | |
| *** wolverineav has quit IRC | 17:58 | |
| efried | mriedem: +2 all the way up. Don't know if you still feel good about approving the bottom one... | 17:58 |
| mriedem | i feel great | 17:58 |
| *** wolverineav has joined #openstack-nova | 17:59 | |
| mriedem | you could also just fast approve https://review.openstack.org/#/c/567659/ | 17:59 |
| mriedem | it's just adding a unit test back in | 17:59 |
| efried | mriedem: done. | 18:00 |
| *** mdnadeem_ has joined #openstack-nova | 18:00 | |
| *** suresh12 has quit IRC | 18:00 | |
| *** lpetrut has quit IRC | 18:02 | |
| *** wolverineav has quit IRC | 18:03 | |
| *** armaan has quit IRC | 18:04 | |
| *** armaan has joined #openstack-nova | 18:04 | |
| *** oomichi has joined #openstack-nova | 18:04 | |
| *** lpetrut has joined #openstack-nova | 18:05 | |
| *** tesseract-RH has joined #openstack-nova | 18:08 | |
| *** moshele has joined #openstack-nova | 18:08 | |
| *** tesseract has quit IRC | 18:09 | |
| *** tesseract-RH has quit IRC | 18:11 | |
| *** tesseract has joined #openstack-nova | 18:11 | |
| *** clutchbeyers has joined #openstack-nova | 18:11 | |
| *** moshele has quit IRC | 18:12 | |
| clutchbeyers | This is a post-newton question: Does anyone know if we set availability_zone in spec_obj on vm provision if the user does not select an availability_zone? | 18:13 |
| clutchbeyers | set in the db after launch* | 18:14 |
| *** liuzz_ has joined #openstack-nova | 18:15 | |
| *** masuberu has joined #openstack-nova | 18:16 | |
| *** liuzz has quit IRC | 18:17 | |
| *** masber has quit IRC | 18:19 | |
| *** tbachman has quit IRC | 18:19 | |
| *** mingyu has quit IRC | 18:20 | |
| *** bandini has quit IRC | 18:20 | |
| *** mingyu has joined #openstack-nova | 18:21 | |
| *** bandini has joined #openstack-nova | 18:22 | |
| *** moshele has joined #openstack-nova | 18:22 | |
| *** jpena is now known as jpena|off | 18:25 | |
| *** mingyu has quit IRC | 18:25 | |
| *** amoralej is now known as amoralej|lunch | 18:26 | |
| *** amoralej|lunch is now known as amoralej|off | 18:26 | |
| mriedem | clutchbeyers: i know that starting in pike we set the instance.availability_zone based on the compute host that was selected by the scheduler https://review.openstack.org/#/c/446053/ - not sure if that extends to the request spec | 18:28 |
| mriedem | the request spec is really about what the user requested | 18:28 |
| mriedem | so i don't think setting reqspec.az if the user didn't request one would be correct there | 18:29 |
| melwitt | before that change, instance.availability_zone used to get set to the "default availability zone" if one wasn't requested at boot time | 18:31 |
| *** fragatin_ has quit IRC | 18:31 | |
| *** mgoddard has quit IRC | 18:33 | |
| mriedem | melwitt: i know, see https://bugs.launchpad.net/nova/+bug/1768876 | 18:36 |
| openstack | Launchpad bug 1768876 in OpenStack Compute (nova) queens "metadata-api fails to get availability zone for instances created before pike" [High,Triaged] | 18:36 |
| efried | mriedem, melwitt: Do y'all have a problem if I fast-approve takashin's mox removal patches? They're test-only; if they pass zuul they clearly work; and I'd like to save other reviewers the pain of combing over these (they're pretty tedious). | 18:36 |
| melwitt | efried: passing zuul is the easy part -- reviewing to ensure test coverage is not lost isn't. so I've not thought of them as easy, one person reviews | 18:37 |
| melwitt | but that's just MHO | 18:37 |
| efried | melwitt: roger that. YHO is what I was asking for. Will leave them for other cores to scrutinize. | 18:38 |
| *** RaoulHC has quit IRC | 18:38 | |
| melwitt | mriedem: guh, another regression | 18:39 |
| openstackgerrit | Merged openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 18:40 |
| *** jmccarthy has joined #openstack-nova | 18:41 | |
| *** mdnadeem_ has quit IRC | 18:46 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Change the field type of InstanceMapping.instance_uuid https://review.openstack.org/567669 | 18:47 |
| *** tesseract has quit IRC | 18:49 | |
| *** rcernin has quit IRC | 18:50 | |
| *** felipemonteiro__ has joined #openstack-nova | 18:51 | |
| *** felipemonteiro_ has quit IRC | 18:51 | |
| arvindn05 | mriedem: good call on checking for scheduling volume backed instances | 18:51 |
| clutchbeyers | thanks for that explanation. we have a use-case where whatever az the scheduler chooses, we do not want a resize to migrate across az's. Sounds like the team has made the decision to keep that logic, and I support that. We can fork our own logic for that | 18:52 |
| arvindn05 | for volume back instances the volume meta data is provided as image meta data. see https://github.com/openstack/nova/blob/master/nova/compute/api.py#L1060 | 18:52 |
| mriedem | clutchbeyers: if you look at my comments in https://bugs.launchpad.net/nova/+bug/1768876 - starting in pike you can't migrate across az's | 18:53 |
| openstack | Launchpad bug 1768876 in OpenStack Compute (nova) queens "metadata-api fails to get availability zone for instances created before pike" [High,Triaged] | 18:53 |
| mriedem | which is actually a regression | 18:53 |
| jmccarthy | mriedem: You about ? Trying to get 'test_cleanup_resize_not_same_host' to pass locally with tox, I don't really know this area - any hints how to resolve this conflict ? https://paste.fedoraproject.org/paste/ntZxhXGlWmzNiL--RZU~Ww | 18:53 |
| mriedem | jmccarthy: see my backports? | 18:53 |
| mriedem | https://review.openstack.org/#/q/Ib10081150e125961cba19cfa821bddfac4614408 | 18:53 |
| *** fragatina has joined #openstack-nova | 18:54 | |
| *** fragatina has quit IRC | 18:54 | |
| jmccarthy | mriedem: Oh ! Ok cool lemme check that out | 18:54 |
| mriedem | arvindn05: i could be wrong but i don't think that actually gets stored in the RequestSpec.image field | 18:54 |
| *** fragatina has joined #openstack-nova | 18:55 | |
| jmccarthy | mreidem: Perfect - Thanks again ! Quite what to do there was giving me a headache lol | 18:55 |
| arvindn05 | from what i saw...the volume meta data is treated exactly the same as image meta data,...the requestSpec.image has volume meta data in the case | 18:55 |
| mriedem | arvindn05: i'm not so sure https://github.com/openstack/nova/commit/0a7427dc58eeb271646a962604106e0e84b870b6 | 18:56 |
| arvindn05 | the only difference is that for volume backed instance the "id" field in the RequestSpec.image is not present... | 18:56 |
| mriedem | but i don't know what you're seeing | 18:56 |
| mriedem | oh that might be true | 18:56 |
| arvindn05 | yup...the code there looks at the id field as well to determine...so looks like its consistent | 18:57 |
| mriedem | oh i guess we use boot_meta here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L882 | 18:57 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L132 | 18:58 |
| arvindn05 | now the only question is should we honor that metadata for volume boot? | 18:58 |
| mriedem | idk | 18:59 |
| arvindn05 | https://blueprints.launchpad.net/python-cinderclient/+spec/support-modify-volume-image-metadata an old blueprint discussion | 19:00 |
| mriedem | i'm not sure how much precedent we have for honoring image meta during scheduling that comes from a volume | 19:00 |
| arvindn05 | 20:32:54 <DuncanT> fmaldonado_: volume glance meta is for all of the same things glance properties are for | 19:00 |
| arvindn05 | it seems that within the code, the volume meta data is treated exactly the same as image meta data...so i am leaning towards leaving it this way and adding few more unit and functional test cases | 19:01 |
| *** tssurya has quit IRC | 19:03 | |
| clutchbeyers | mriedem: thats great to hear on the cross-az migration, we handle that in newton by use of the nova filters and extra specs that dont allow things to cross. But our specific use case is in the resize. If one spawns an instance without giving it an az, a resize will cold migrate (shutdown) and treat it like a new vm spin up and select another az with new flavors. Are you saying this behavior also does not allow for cross- | 19:04 |
| *** felipemonteiro_ has joined #openstack-nova | 19:04 | |
| mriedem | clutchbeyers: the change in pike would apply to any move operation through the scheduler, | 19:05 |
| mriedem | so resize, live migrate, unshelve and evacuate | 19:05 |
| *** felipemonteiro__ has quit IRC | 19:07 | |
| clutchbeyers | mriedem: thank you for the clarification! | 19:09 |
| *** rmk has joined #openstack-nova | 19:09 | |
| *** rmk has quit IRC | 19:09 | |
| *** rmk has joined #openstack-nova | 19:09 | |
| *** rcernin has joined #openstack-nova | 19:09 | |
| mriedem | clutchbeyers: note that we could consider this a regression in pike. it's in your favor i guess, but depending on how we handle this bug we might change it back. | 19:10 |
| mriedem | dansmith: maybe something we need to put in your forum session on cells... | 19:11 |
| *** jmccarthy has left #openstack-nova | 19:12 | |
| dansmith | mriedem: I'm not sure what you're saying is a regression | 19:13 |
| dansmith | or what it has to do with cells | 19:13 |
| dansmith | that you can't migrate across az boundaries/ | 19:13 |
| *** mvk has joined #openstack-nova | 19:13 | |
| mriedem | dansmith: https://bugs.launchpad.net/nova/+bug/1768876 | 19:13 |
| openstack | Launchpad bug 1768876 in OpenStack Compute (nova) queens "metadata-api fails to get availability zone for instances created before pike" [High,Triaged] | 19:13 |
| clutchbeyers | mriedem: it my honest opinion, if a tenant does not specify an az on vm provision, then they arent too concnered about controlling their fault domain. So when they do decide to resize and it moves to another az, they theoretically shouldn't care | 19:14 |
| clutchbeyers | But for those that specifically call out an az, they have a fault domain architecture in mind and wouldnt want to disrupt that | 19:14 |
| *** felipemonteiro_ has quit IRC | 19:14 | |
| dansmith | mriedem: okay, I think we talked about that in dublin, about potentially just forcing instance.az to whatever we chose if you didn't ask for one, keeping you in whatever az you happened to land in | 19:14 |
| *** felipemonteiro_ has joined #openstack-nova | 19:14 | |
| melwitt | I think that's what the original behavior was -- only allows migrating across AZ if none was selected deliberately | 19:15 |
| dansmith | melwitt: you mean only keeps you in your az if you asked for one initially | 19:15 |
| clutchbeyers | there is some assumptions being made there with how the tenant is designing their app/fault tolerance | 19:15 |
| *** rcernin has quit IRC | 19:15 | |
| melwitt | dansmith: yeah | 19:16 |
| dansmith | clutchbeyers: I totally agree that if you asked for one, you shouldn't be migrated out of it | 19:16 |
| dansmith | I get that mriedem's metadata thing is a regression, | 19:17 |
| dansmith | I'm missing what, if any, scheduler behavior regressed | 19:17 |
| mriedem | dansmith: i don't remember talking about this specifically in dublin, | 19:17 |
| mriedem | the only AZ thing I remember talking about wrt cells was the cross_az_attach volume upcall thing | 19:17 |
| *** pchavva has quit IRC | 19:17 | |
| mriedem | dansmith: what i'm pointing out in this comment https://bugs.launchpad.net/nova/+bug/1768876/comments/4 | 19:18 |
| openstack | Launchpad bug 1768876 in OpenStack Compute (nova) queens "metadata-api fails to get availability zone for instances created before pike" [High,Triaged] | 19:18 |
| dansmith | I'm pretty sure it was dublin, with bauzas and I | 19:18 |
| mriedem | i need to test it still to confirm it is a regression | 19:18 |
| *** zcorneli_ has joined #openstack-nova | 19:18 | |
| dansmith | okay, I'm confused because I thought the second part of your comment (that an instance would be confined to whatever AZ it landed in) wasn't yet the case, but we considered doing that | 19:19 |
| dansmith | in the dublin discussion where we were talking about how to handle instance.az when az wasn't requested | 19:20 |
| *** zcorneli has quit IRC | 19:20 | |
| mriedem | maybe we were talking about that in the context of volume cross_az_attach? | 19:21 |
| dansmith | I dunno, I thought not, but I don't see it on the etherpad | 19:21 |
| dansmith | I _know_ I've had a convo with bauzas about this in the not too distant past | 19:22 |
| dansmith | maybe the az discussion there is merging together in my head | 19:22 |
| mriedem | maybe related to the ops list thread on not allowing deleting/renaming an AZ that has instances in it? | 19:22 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 19:23 |
| dansmith | this was face-to-face or face-to-screen | 19:23 |
| dansmith | with bazuas | 19:23 |
| dansmith | mriedem: what change in pike do you think is making us keep things in the same az? | 19:24 |
| *** tbachman has joined #openstack-nova | 19:24 | |
| mriedem | https://review.openstack.org/#/c/446053/ | 19:24 |
| dansmith | ah, yeah | 19:25 |
| mriedem | if the instance.availability_zone was set based on CONF.default_availability_zone when the user didn't request one specifically, before that change, and that defaults to None, i believe that means the instance could freely move between AZs | 19:25 |
| mriedem | anyway, i can write a simple functional test to try this once i get this 'remove fping' patch up | 19:25 |
| dansmith | yeah, so, this is the change I was talking about, | 19:25 |
| dansmith | which is probably when I was talking to bauzas about it (before writing this) | 19:25 |
| dansmith | I just forgot that we had made it | 19:26 |
| mriedem | heh, time flies :) | 19:26 |
| dansmith | apparently | 19:26 |
| mriedem | i don't think i was aware of this change either until yesterday | 19:26 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove support for /os-fping REST API https://review.openstack.org/567682 | 19:26 |
| melwitt | I just checked and the default is 'nova', not None. not sure if that makes a difference | 19:26 |
| mriedem | melwitt: not for CONF.default_availability_zone | 19:27 |
| mriedem | there are 2 "default" AZ options | 19:27 |
| dansmith | so, going back to my "up for debate" comment above, | 19:27 |
| mriedem | one for the user's instance and one for the scheduler | 19:27 |
| melwitt | that's what I'm looking at? | 19:27 |
| dansmith | what's the actual problem with keeping it where it is? | 19:27 |
| *** mingyu has joined #openstack-nova | 19:27 | |
| melwitt | default_schedule_zone is the None one | 19:27 |
| melwitt | (has no default) | 19:27 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 19:28 |
| mriedem | i always have to read this again https://docs.openstack.org/nova/latest/user/aggregates.html#availability-zones-azs | 19:28 |
| *** moshele has quit IRC | 19:28 | |
| *** eharney has quit IRC | 19:29 | |
| mriedem | melwitt: oh that must have changed, | 19:29 |
| mriedem | see pike | 19:29 |
| mriedem | https://docs.openstack.org/nova/pike/configuration/config.html | 19:29 |
| mriedem | oh nvm it didn't change https://docs.openstack.org/nova/pike/configuration/config.html#DEFAULT.default_availability_zone | 19:29 |
| mriedem | wtf my eyes | 19:29 |
| *** wolverineav has joined #openstack-nova | 19:29 | |
| melwitt | tbh the only reason I remember the 'nova' availability zone being default is from using devstack | 19:30 |
| dansmith | [12:27:34] <dansmith>what's the actual problem with keeping it where it is? | 19:30 |
| mriedem | oh my comment in the bug is wrong | 19:30 |
| mriedem | default_schedule_zone is for instances | 19:30 |
| mriedem | default_availability_zone is for compute services | 19:30 |
| mriedem | dansmith: i don't know that there is, | 19:31 |
| mriedem | except it's a behavior change | 19:31 |
| mriedem | but, | 19:31 |
| mriedem | (2:11:06 PM) mriedem: dansmith: maybe something we need to put in your forum session on cells... | 19:31 |
| melwitt | dansmith: the concern is if that broke the original behavior of letting an instance move across AZs if someone didn't specify an AZ to begin with, if they don't care about AZ | 19:31 |
| mriedem | :) | 19:31 |
| melwitt | but we're not 100% sure yet if that behavior changed, mriedem is gonna test it out in a func test | 19:31 |
| mriedem | maybe it's just an FYI to operators at this point, idk | 19:31 |
| *** suresh12_ has quit IRC | 19:31 | |
| dansmith | no I think it did change | 19:31 |
| dansmith | but Idon't think it's user-visible | 19:32 |
| dansmith | what I do think makes more sense about the new behavior, | 19:32 |
| melwitt | it only would be if all the compute hosts in the same AZ are at capacity and there are free ones in another AZ, right? | 19:32 |
| dansmith | is that if I boot an instance, ssh in, check the AZ that it's in, and then boot an instance in another az to make sure I have separation, | 19:32 |
| dansmith | then something moves my instance into the same az and I'm surprised | 19:32 |
| melwitt | and they're stuck because it won't let the AZ be crossed | 19:32 |
| dansmith | which won't happen now | 19:32 |
| dansmith | melwitt: yeah but they have no idea if there is space in the cloud, or in their az | 19:33 |
| mriedem | one fear i have is operators will rely on the force flag for live migrate or evacuate to bypass the scheduler now to move the instance into another compute just because that other compute is in another AZ, | 19:33 |
| mriedem | but idk | 19:34 |
| melwitt | yeah, not until they contact support I guess, and admin can see the migration should be able to work and it doesn't | 19:34 |
| mriedem | if only we could force a cold migration to bypass the scheduler.... :) | 19:34 |
| dansmith | mriedem: fair point, but not super strong, IMHO | 19:34 |
| mriedem | dansmith: agree it's not strong | 19:34 |
| mriedem | if you have 10 computes in AZ1 and 10 in AZ2, then just pick one of the ones that's already in the same AZ | 19:35 |
| mriedem | also, kablam https://review.openstack.org/567682 | 19:35 |
| *** tssurya has joined #openstack-nova | 19:37 | |
| *** mlavalle has quit IRC | 19:37 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 19:38 |
| *** mlavalle has joined #openstack-nova | 19:39 | |
| *** sv1_X has joined #openstack-nova | 19:40 | |
| clutchbeyers | mriedem: cold migrations bypassing the scheduler was a joke right? :) | 19:40 |
| *** tidwellr has joined #openstack-nova | 19:41 | |
| *** tssurya has quit IRC | 19:41 | |
| *** oomichi has quit IRC | 19:42 | |
| mriedem | clutchbeyers: L645 https://etherpad.openstack.org/p/nova-ptg-rocky | 19:42 |
| mriedem | oh the -5 didn't make it into the notes | 19:43 |
| *** lpetrut has quit IRC | 19:43 | |
| *** tssurya has joined #openstack-nova | 19:44 | |
| *** hoonetorg has quit IRC | 19:46 | |
| *** wolverineav has quit IRC | 19:48 | |
| *** wolverineav has joined #openstack-nova | 19:49 | |
| *** Ahud07 has joined #openstack-nova | 19:50 | |
| melwitt | dansmith: I was just reading the ML thread about the map_instances marker thing, and I think what takashin was suggesting is to print the next marker before returning 1, and then let that be passed as a starting point when the command is run again https://github.com/openstack/nova/blob/master/nova/cmd/manage.py#L1193 | 19:50 |
| *** dklyle_ is now known as dklyle | 19:50 | |
| melwitt | would that not work for some reason? | 19:51 |
| *** wolverineav has quit IRC | 19:53 | |
| *** Ahud07 has left #openstack-nova | 19:55 | |
| *** wolverineav has joined #openstack-nova | 19:56 | |
| edleafe | melwitt: if the batches were all done in a row, it might. But there could be long gaps of time between batches, so the marker needs to be stored | 19:57 |
| *** tidwellr has quit IRC | 19:58 | |
| melwitt | I see | 19:58 |
| *** felipemonteiro__ has joined #openstack-nova | 19:59 | |
| *** eharney has joined #openstack-nova | 20:00 | |
| *** hoonetorg has joined #openstack-nova | 20:00 | |
| *** wolverineav has quit IRC | 20:00 | |
| *** felipemonteiro_ has quit IRC | 20:02 | |
| *** sv1_X has quit IRC | 20:03 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 20:07 |
| *** burt has quit IRC | 20:07 | |
| dansmith | melwitt: yeah I know he was talking about that and I agree with edleafe that it's not a great solution | 20:07 |
| dansmith | melwitt: the bit I'm not sure about was the "it could be exposed accidentally through the api" part | 20:07 |
| *** burt has joined #openstack-nova | 20:08 | |
| melwitt | okay, I think I missed the "exposed accidentally through the api" part | 20:09 |
| melwitt | I didn't see anyone mention something like that | 20:09 |
| dansmith | he said "in some compute rest apis, it returns the marker" | 20:10 |
| melwitt | I think he means like in instance list, it returns the marker if there's additional pages available | 20:10 |
| *** liverpooler has quit IRC | 20:10 | |
| melwitt | like if you have > 1000 (api max limit) instances in the list, it will give you a "next marker" so you can query again and pass the marker to get the next page | 20:11 |
| dansmith | you mean you think he's offering the REST API's pagination as an example that could be used for the cli to be user-supplied-marker behavior? | 20:11 |
| melwitt | that's what I think he was saying, yeah. if doing it similar to that could avoid storing the marker | 20:12 |
| dansmith | I guess I can see it being read like that, but to me it sounded like he was saying it's bad to store it in-band because there's a way you can see it externally | 20:12 |
| melwitt | if map_instances hands you back the next marker, says "I did a batch but there's more" and you can call again and give the marker and it does the next batch | 20:12 |
| dansmith | I understand how the user could supply a marker, yeah | 20:14 |
| dansmith | I just think that's fairly terrible UX for a CLI, especially a chatty one that has all kinds of other crap going to stdio like python warnings | 20:15 |
| *** burt has quit IRC | 20:15 | |
| melwitt | oh, hm. yeah, I guess if I re-read it and connect the 3rd paragraph to the 2nd, it could sound that way. but I thought he was stating it's not ideal to store the marker and then next gave a suggestion on how we could avoid storing it | 20:15 |
| dansmith | I'll send another clarification mail | 20:16 |
| arvindn05 | efried: gibi: mriedem: updated the code review for https://review.openstack.org/560596 - "update scheduler to use image-traits" | 20:16 |
| melwitt | yeah, maybe it doesn't fit in so well for the CLI, but it's an interesting idea. probably the only way to avoid having to store a marker record | 20:16 |
| efried | arvindn05: ack | 20:17 |
| arvindn05 | efried: ty...hopefully the final PS for this change :) | 20:19 |
| efried | arvindn05: Hah! | 20:19 |
| efried | arvindn05: You've just karmically guaranteed that won't be the case! | 20:19 |
| dansmith | melwitt: it's what we had before we replaced it with this, because it was annoying | 20:19 |
| dansmith | https://review.openstack.org/#/c/301916/ | 20:20 |
| melwitt | ah, okay | 20:20 |
| mriedem | fwiw, | 20:20 |
| arvindn05 | efried: lol...well to job security then...hehe | 20:20 |
| efried | :) | 20:20 |
| mriedem | the new heal_allocations CLI i added has a note in there about taking a marker from the user, and dumping out the last instance processed so the next run can start from the marker rather than process a bunch of instances already healed | 20:20 |
| *** armaan has quit IRC | 20:21 | |
| dansmith | mriedem: we have all these other instances where we're able to keep track of things for the admin, it feels pretty sucky to go back to "you must have a mouse and a scratch pad to use this thing" experience | 20:22 |
| dansmith | discovery hosts, instance mapping, reqspec fixup | 20:22 |
| *** armaan has joined #openstack-nova | 20:22 | |
| melwitt | that's a really old change too, I somehow stayed blissfully unaware of how the map_instances CLI worked historically | 20:22 |
| dansmith | all keep track of it for you because we're talking about potentially an ass-ton of 50-item iterations | 20:22 |
| mriedem | i didn't realize we had a marker for the reqspec data migration | 20:23 |
| dansmith | yup | 20:23 |
| dansmith | it wasn't as controversial because it wasn't fighting a UC field | 20:23 |
| dansmith | but it worked basically the same way | 20:23 |
| mriedem | i'm not even sure where i'd store the marker in my case | 20:24 |
| mriedem | i'm iterating the instances table per cell | 20:24 |
| dansmith | yeah, I understand yours is a little less obvious, but these all have reasonable places | 20:24 |
| mriedem | and we can't put a fake instance in the instaces db | 20:24 |
| mriedem | *table | 20:24 |
| dansmith | we could have a scratch pad table we use for this kind of thing.. just "varchar(255) key, text value" | 20:25 |
| dansmith | no index, never expect to have much data in there, | 20:25 |
| dansmith | just some context for things like this | 20:25 |
| mriedem | well, i thought about system_metadata, | 20:25 |
| mriedem | because i could query for a specific system_metadata with a marker key, get that instance, delete the marker entry and continue, but it's not great | 20:26 |
| dansmith | you could yeah, but it could be a little racy with things potentially loading/saving that instance for other things maybe | 20:26 |
| dansmith | although you're just doing active things, so.. | 20:26 |
| *** armaan has quit IRC | 20:27 | |
| dansmith | I would like to circle back, | 20:27 |
| dansmith | and point out that the only reason we're discussing this is because we emit a few warnings during unit tests :) | 20:27 |
| mriedem | that's not why i'm talking about it, | 20:30 |
| mriedem | i don't care about the map_instances thing | 20:30 |
| *** oomichi has joined #openstack-nova | 20:30 | |
| efried | arvindn05: I was totally going to let the typos slide, but that last docstring is wrong enough to warrant a respin. | 20:30 |
| efried | arvindn05: karma ^^ | 20:30 |
| melwitt | it's true that the unit test warnings drew attention to the marker-record-storing pattern. but I think lots of people have been unaware of that pattern and it's pretty surprising, or at least it was to me the first time I learned about it | 20:31 |
| arvindn05 | arvindn05: ...good catches...will fix ASAP | 20:31 |
| dansmith | ...because it has just been working? :) | 20:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687 | 20:32 |
| melwitt | yep, I guess so. no problems, no eyeballs drawn to it | 20:32 |
| *** moshele has joined #openstack-nova | 20:32 | |
| *** moshele has quit IRC | 20:34 | |
| mriedem | dansmith: want to hit these backports https://review.openstack.org/#/q/Idea86d6bb648b1e6fef8813dbe569724ce81a750 - then we can safely remove the legacy live migration job from project-config and openstack-zuul-jobs | 20:35 |
| mriedem | dansmith: shouldn't require 2 +2s i don't think | 20:35 |
| dansmith | OMG YES | 20:35 |
| dansmith | <-- loves hitting backports | 20:35 |
| mriedem | otherwise right now we're getting hit on master b/c we're running 2 live migration jbos | 20:35 |
| * melwitt picks up more baskets of backports to offer dansmith | 20:35 | |
| mriedem | it will take that as (1) serious and (2) literal | 20:35 |
| dansmith | mriedem: hah | 20:35 |
| * dansmith scowls at melwitt | 20:36 | |
| melwitt | :D | 20:36 |
| mriedem | i've been holding https://review.openstack.org/#/q/Ib10081150e125961cba19cfa821bddfac4614408 | 20:36 |
| mriedem | then we can do a (sweet) release | 20:36 |
| mriedem | this is ready too https://review.openstack.org/#/c/567271/ | 20:36 |
| openstackgerrit | Merged openstack/nova master: Remove mox in test_compute_api.py (1) https://review.openstack.org/564415 | 20:39 |
| *** takashin has joined #openstack-nova | 20:40 | |
| mriedem | oomichi: ha https://review.openstack.org/#/c/564415/3/nova/tests/unit/compute/test_compute_api.py@1013 | 20:40 |
| mriedem | oomichi: check out the vmware virt driver unit tests at some point | 20:40 |
| oomichi | mriedem: hehe, more longer ? :) | 20:41 |
| openstackgerrit | Merged openstack/nova master: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/566367 | 20:43 |
| mriedem | ok now i can bug dan about those backports | 20:44 |
| *** trozet has joined #openstack-nova | 20:44 | |
| melwitt | nova meeting in 15 minutes | 20:45 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: always create consumer records https://review.openstack.org/567678 | 20:48 |
| *** clutchbeyers has quit IRC | 20:50 | |
| oomichi | mriedem: btw where does 410 come from ? related to https://review.openstack.org/#/c/567687 | 20:52 |
| oomichi | 410 means http410 status code | 20:52 |
| *** mvenesio has quit IRC | 20:53 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 20:53 |
| mriedem | https://www.w3.org/Protocols/rfc2616/rfc2616-sec10.html | 20:54 |
| arvindn05 | efried: not going to say anything...but new PS uploaded :) | 20:54 |
| efried | looking now | 20:54 |
| mriedem | oomichi: "The requested resource is no longer available at the server and no forwarding address is known. This condition is expected to be considered permanent." | 20:54 |
| efried | arvindn05: +2 from me. Now you have to get by mriedem... | 20:55 |
| arvindn05 | nice...thanks for the quick turnaround efried | 20:55 |
| oomichi | mriedem: I didn't see http410 case and api-wg guideline doesn't mention about http410 case, so I was curious about that | 20:57 |
| mriedem | oomichi: it's what we used when we removed os-cloudpipe and os-certificates, and it makes more sense then 404 | 20:58 |
| mriedem | *than | 20:58 |
| mriedem | 404 is what we used for capping / deprecating with microversions | 20:58 |
| mriedem | technically we also use 404 when a route is not found | 20:58 |
| oomichi | mriedem: yeah, 404 is my base thinking. Nice to have conversation widely | 20:59 |
| mriedem | oomichi: feel free to bring it up in the ML thread | 21:00 |
| mriedem | but at this point it would be weird to use 410 for those other removed apis but 404 for fping | 21:00 |
| mriedem | IMO | 21:00 |
| *** felipemonteiro__ has quit IRC | 21:00 | |
| oomichi | OK, let me do that | 21:01 |
| *** felipemonteiro has joined #openstack-nova | 21:02 | |
| mriedem | sean-k-mooney: this is for you https://bugs.launchpad.net/nova/+bug/1770378 | 21:07 |
| openstack | Launchpad bug 1770378 in OpenStack Compute (nova) "windows can‘t start when use vhostuser interface" [Undecided,New] | 21:07 |
| mriedem | sean-k-mooney: i noticed that we have 0 docs in nova or neutron on using vhostuser for networking | 21:08 |
| mriedem | is that somewhere else? | 21:08 |
| *** slaweq has quit IRC | 21:09 | |
| mriedem | oh hmm here it is https://docs.openstack.org/neutron/latest/contributor/internals/ovs_vhostuser.html | 21:09 |
| sean-k-mooney | mriedem: stephen added docs to the networking guide | 21:09 |
| mriedem | right but https://docs.openstack.org/neutron/latest/search.html?q=vhostuser | 21:10 |
| *** slaweq has joined #openstack-nova | 21:10 | |
| mriedem | oh b/c it's vhost-user | 21:10 |
| sean-k-mooney | mriedem: this is where the docs used to be https://docs.openstack.org/neutron/pike/admin/config-ovs-dpdk.html | 21:12 |
| sean-k-mooney | anyway regarding the bug you should be able to run windows guest on vhost-user ports however i do remember issues in the past due to virtio diriver versions | 21:13 |
| *** awaugama has quit IRC | 21:13 | |
| sean-k-mooney | mriedem: this might be the fix for the issue http://dpdk.org/dev/patchwork/patch/15541/ | 21:14 |
| *** edmondsw has quit IRC | 21:14 | |
| *** edmondsw has joined #openstack-nova | 21:15 | |
| *** wolverineav has joined #openstack-nova | 21:16 | |
| *** felipemonteiro has quit IRC | 21:17 | |
| *** hemna_ has quit IRC | 21:18 | |
| *** vladikr has quit IRC | 21:18 | |
| *** edmondsw has quit IRC | 21:19 | |
| *** avolkov has quit IRC | 21:21 | |
| *** moshele has joined #openstack-nova | 21:22 | |
| *** felipemonteiro has joined #openstack-nova | 21:23 | |
| *** slaweq has quit IRC | 21:24 | |
| *** slaweq has joined #openstack-nova | 21:25 | |
| *** slaweq has quit IRC | 21:30 | |
| *** vladikr has joined #openstack-nova | 21:31 | |
| *** liverpooler has joined #openstack-nova | 21:37 | |
| *** eharney has quit IRC | 21:39 | |
| *** dave-mccowan has joined #openstack-nova | 21:39 | |
| mnaser | mriedem: https://review.openstack.org/#/c/565611/ ok for me to merge this or wait till https://review.openstack.org/#/q/Idea86d6bb648b1e6fef8813dbe569724ce81a750 is in? | 21:41 |
| sean-k-mooney | mriedem: i have responded to https://bugs.launchpad.net/nova/+bug/1770378 and set it to incomplete until they respond with the version of ovs and dpdk they are using | 21:41 |
| openstack | Launchpad bug 1770378 in OpenStack Compute (nova) "windows can‘t start when use vhostuser interface" [Undecided,Incomplete] | 21:41 |
| *** felipemonteiro has quit IRC | 21:44 | |
| *** felipemonteiro has joined #openstack-nova | 21:46 | |
| openstackgerrit | Merged openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 21:48 |
| *** cfriesen has quit IRC | 21:49 | |
| mriedem | takashin: this is the event https://www.openstack.org/summit/vancouver-2018/summit-schedule/events/21749/missing-features-in-openstack-for-public-clouds | 21:50 |
| takashin | mriedem: Thanks. I will attend it. | 21:50 |
| mriedem | https://etherpad.openstack.org/p/YVR-publiccloud-wg-brainstorming | 21:51 |
| mriedem | that's the only etherpad i'm aware of right now | 21:51 |
| mriedem | sean-k-mooney: thanks for commenting on that vhostuser bug | 21:53 |
| mriedem | mnaser: that's good to go now | 21:53 |
| mnaser | mriedem: +w :) | 21:53 |
| *** cfriesen has joined #openstack-nova | 21:58 | |
| mriedem | melwitt: i saw this in triage today https://bugs.launchpad.net/nova/+bug/1770348 | 22:03 |
| openstack | Launchpad bug 1770348 in OpenStack Compute (nova) "Unable to access console via dashboard" [Undecided,Incomplete] | 22:03 |
| mriedem | mnaser: thanks | 22:03 |
| openstackgerrit | Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247 | 22:06 |
| *** ivve has quit IRC | 22:09 | |
| *** hemna_ has joined #openstack-nova | 22:10 | |
| *** slaweq has joined #openstack-nova | 22:11 | |
| *** felipemonteiro has quit IRC | 22:14 | |
| *** slaweq has quit IRC | 22:16 | |
| melwitt | mriedem: looking | 22:16 |
| openstackgerrit | Merged openstack/nova stable/queens: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567612 | 22:16 |
| openstackgerrit | Merged openstack/nova stable/ocata: Migrate tempest-dsvm-multinode-live-migration job in-tree https://review.openstack.org/567619 | 22:16 |
| *** esberglu has quit IRC | 22:19 | |
| openstackgerrit | Merged openstack/nova stable/pike: log stale allocations as WARNING instead of DEBUG https://review.openstack.org/567271 | 22:21 |
| openstackgerrit | Hongbin Lu proposed openstack/nova-specs master: Choose default network on ambiguity https://review.openstack.org/520247 | 22:25 |
| melwitt | mriedem: that would have been my first question too, whether they're using devstack or a real deployment. what's probably wrong is they need to pass the cell nova.conf to the console proxy service instead of the old global nova.conf | 22:28 |
| *** wolverineav has quit IRC | 22:29 | |
| melwitt | I'm thinking maybe I need to make the release note better to call that out | 22:29 |
| *** mrf has joined #openstack-nova | 22:31 | |
| mrf | Hi | 22:31 |
| mrf | does images_rbd_pool at nova.conf support multiple pools? | 22:31 |
| openstackgerrit | Merged openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 22:31 |
| melwitt | mrf: I don't believe so. mnaser could you confirm? ^ | 22:33 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add nova-slow job https://review.openstack.org/567697 | 22:33 |
| mriedem | there's your slow job | 22:33 |
| mnaser | mriedem: it does not unfortunately, it can only use a single rbd image backend | 22:33 |
| mnaser | err | 22:33 |
| mnaser | mrf: ** ^ | 22:33 |
| *** esberglu has joined #openstack-nova | 22:33 | |
| melwitt | mriedem: awesome thanks | 22:33 |
| *** esberglu has quit IRC | 22:33 | |
| melwitt | wznoinsk: ^ | 22:33 |
| mriedem | ok stable backports for this https://review.openstack.org/#/q/Ib10081150e125961cba19cfa821bddfac4614408 are ready | 22:36 |
| mriedem | but probably no one around | 22:36 |
| mriedem | oh tonyb mtreinish ^ :) | 22:39 |
| *** cfriesen has quit IRC | 22:41 | |
| *** tbachman has quit IRC | 22:41 | |
| *** tbachman has joined #openstack-nova | 22:41 | |
| *** tssurya_ has joined #openstack-nova | 22:42 | |
| *** jaypipes_ has joined #openstack-nova | 22:42 | |
| *** wwriverrat_ has joined #openstack-nova | 22:42 | |
| *** tssurya has quit IRC | 22:42 | |
| *** wwriverrat has quit IRC | 22:42 | |
| *** jaypipes has quit IRC | 22:42 | |
| *** wwriverrat_ is now known as wwriverrat | 22:42 | |
| mrf | mmm i got a trouble with 2 computes nodes , maybe some could help each one got diferent images_rbd_pool | 22:42 |
| *** vdrok has quit IRC | 22:42 | |
| *** dulek has quit IRC | 22:42 | |
| mrf | and now i cant migrate instances beetween them | 22:43 |
| *** vdrok has joined #openstack-nova | 22:43 | |
| tonyb | mriedem: Thanks, I'll review them after this meeting | 22:45 |
| *** Guest81722 has quit IRC | 22:46 | |
| *** dulek has joined #openstack-nova | 22:46 | |
| *** jaypipes__ has joined #openstack-nova | 22:48 | |
| *** jaypipes_ has quit IRC | 22:49 | |
| *** nicolasbock has quit IRC | 22:50 | |
| *** nicolasbock has joined #openstack-nova | 22:50 | |
| *** vladikr has quit IRC | 22:50 | |
| *** cfriesen has joined #openstack-nova | 22:51 | |
| *** cdent has quit IRC | 22:51 | |
| *** bandini has quit IRC | 22:54 | |
| *** hongbin has quit IRC | 22:55 | |
| *** cfriesen has quit IRC | 22:55 | |
| *** cfriesen_ has joined #openstack-nova | 22:55 | |
| *** bandini has joined #openstack-nova | 22:57 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 23:02 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 23:03 |
| *** vladikr has joined #openstack-nova | 23:04 | |
| openstackgerrit | melanie witt proposed openstack/nova master: Avoid warning log when image not exist https://review.openstack.org/567522 | 23:06 |
| mriedem | AHA! | 23:17 |
| melwitt | what | 23:17 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py#L43 | 23:18 |
| mrf | O_O | 23:18 |
| mriedem | we don't have a regression | 23:18 |
| *** moshele has quit IRC | 23:18 | |
| mriedem | if you didn't request an az on server create, the az filter doesn't take the current instance az into account | 23:18 |
| melwitt | well I'll be | 23:18 |
| *** liverpooler has quit IRC | 23:18 | |
| mriedem | and clutchbeyers already left | 23:19 |
| melwitt | disaster averted | 23:19 |
| *** tssurya_ has quit IRC | 23:20 | |
| mriedem | we still have the up-call and data migration to deal with :) | 23:25 |
| mriedem | we still have a regression since pike, just an easier one to deal with | 23:26 |
| *** germs has quit IRC | 23:26 | |
| mriedem | will push up this functional test since i bothered writing it, and then i'm off | 23:26 |
| *** germs has joined #openstack-nova | 23:27 | |
| *** germs has quit IRC | 23:27 | |
| *** germs has joined #openstack-nova | 23:27 | |
| melwitt | :( wah wah | 23:29 |
| melwitt | thanks mriedem | 23:29 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add implicit instance AZ + live migration test https://review.openstack.org/567701 | 23:30 |
| mrf | mmm i got a trouble with 2 computes nodes , maybe some could help each one got diferent images_rbd_pool, looking virshdump it point to rbd pool, can it be manually change the xmlfile to the new pool and them move the object at ceph level? | 23:30 |
| melwitt | mrf: maybe. you could try asking an advance ceph operator like mnaser or ask on the openstack-operators mailing list where people can respond | 23:32 |
| *** mriedem has quit IRC | 23:32 | |
| mrf | thank so much | 23:33 |
| melwitt | good luck | 23:34 |
| *** mlavalle has quit IRC | 23:34 | |
| openstackgerrit | Tsuyoshi Nagata proposed openstack/nova master: nova improvement of maximum attach volumes more than 26 vols https://review.openstack.org/567472 | 23:37 |
| *** yamamoto has joined #openstack-nova | 23:37 | |
| *** liuzz has joined #openstack-nova | 23:40 | |
| *** liuzz_ has quit IRC | 23:40 | |
| *** liverpooler has joined #openstack-nova | 23:42 | |
| *** yamamoto has quit IRC | 23:42 | |
| *** rajinir has quit IRC | 23:43 | |
| mrf | Why instance migration with blockmigration is not supported in a stopped vm? Cannot 'os-migrateLive' instance af02564c-8579-4226-96b7-99e3b650c06b while it is in vm_state stopped (HTTP 409) (Request-ID: req-aee842d9-593b-43fc-9487-e5eb191bc732) | 23:43 |
| melwitt | mrf: just a guess, but I think the instance has to be running in order to do a live migration. that is, libvirt must need the instance running in order to do it | 23:48 |
| mrf | yes but if i want only migrate storage of instance | 23:49 |
| mrf | state can me started or stopped | 23:49 |
| mrf | similar to other hypervisors like xenserver, hyperv, vmware | 23:49 |
| mrf | you can do a storage migration without know if machine is live or stopped | 23:49 |
| mrf | block-migration must not depend of machine state | 23:50 |
| melwitt | maybe you want regular cold migration then? https://developer.openstack.org/api-ref/compute/#migrate-server-migrate-action | 23:51 |
| mrf | it drops : [root@controller ~(keystone_admin)]# openstack server migrate --block-migration af02564c-8579-4226-96b7-99e3b650c06b | 23:54 |
| mrf | --live must be specified if --block-migration or --disk-overcommit is specified | 23:54 |
| melwitt | I think you don't need to pass --block-migration for a cold migrate | 23:56 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!