| *** yamamoto has quit IRC | 00:00 | |
| *** armaan has quit IRC | 00:03 | |
| *** armaan has joined #openstack-nova | 00:04 | |
| *** armaan has quit IRC | 00:05 | |
| *** armaan has joined #openstack-nova | 00:05 | |
| *** Swami has quit IRC | 00:11 | |
| *** mriedem has quit IRC | 00:18 | |
| *** tbachman has joined #openstack-nova | 00:19 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:22 | |
| *** markvoelker has quit IRC | 00:25 | |
| openstackgerrit | Merged openstack/nova master: libvirt: Lift the restriction of choices for `cpu_model_extra_flags` https://review.openstack.org/563926 | 00:26 |
|---|---|---|
| *** yingjun has joined #openstack-nova | 00:31 | |
| *** hoangcx has joined #openstack-nova | 00:35 | |
| openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 00:37 |
| *** liuzz has joined #openstack-nova | 00:40 | |
| *** yingjun has quit IRC | 00:50 | |
| *** yingjun has joined #openstack-nova | 00:53 | |
| *** yamamoto has joined #openstack-nova | 00:57 | |
| *** fragatina has quit IRC | 00:57 | |
| *** jichen has joined #openstack-nova | 00:58 | |
| *** Guest72284 has quit IRC | 01:01 | |
| *** yamamoto has quit IRC | 01:03 | |
| *** zhaochao has joined #openstack-nova | 01:05 | |
| *** bzhao__ has joined #openstack-nova | 01:07 | |
| *** tiendc has joined #openstack-nova | 01:08 | |
| *** Zames_ has joined #openstack-nova | 01:11 | |
| *** dpawlik has joined #openstack-nova | 01:11 | |
| *** Zames_ has quit IRC | 01:13 | |
| *** dpawlik has quit IRC | 01:15 | |
| *** Guest72284 has joined #openstack-nova | 01:16 | |
| *** hongbin has joined #openstack-nova | 01:17 | |
| *** trungnv has joined #openstack-nova | 01:25 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689 | 01:25 |
| *** gjayavelu has quit IRC | 01:40 | |
| *** Tom-Tom has quit IRC | 01:41 | |
| *** tbachman has quit IRC | 01:46 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 01:47 |
| *** tbachman has joined #openstack-nova | 01:48 | |
| *** felipemonteiro has joined #openstack-nova | 01:50 | |
| *** liverpooler has joined #openstack-nova | 01:50 | |
| *** Nel1x has joined #openstack-nova | 01:51 | |
| *** vic-wu has joined #openstack-nova | 01:55 | |
| *** felipemonteiro has quit IRC | 01:56 | |
| *** dklyle has quit IRC | 01:57 | |
| *** david-lyle has joined #openstack-nova | 01:57 | |
| *** yamamoto has joined #openstack-nova | 01:59 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 02:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 02:00 |
| *** Tom-Tom has joined #openstack-nova | 02:02 | |
| *** yamahata has quit IRC | 02:02 | |
| *** yamamoto has quit IRC | 02:03 | |
| *** dougshelley66 has quit IRC | 02:04 | |
| *** yamamoto has joined #openstack-nova | 02:05 | |
| *** lei-zh has joined #openstack-nova | 02:06 | |
| *** dougshelley66 has joined #openstack-nova | 02:07 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 02:08 |
| *** masber has joined #openstack-nova | 02:09 | |
| *** yamamoto has quit IRC | 02:09 | |
| *** spsurya has joined #openstack-nova | 02:10 | |
| *** tinwood has quit IRC | 02:10 | |
| *** tinwood has joined #openstack-nova | 02:11 | |
| *** dave-mccowan has quit IRC | 02:13 | |
| *** mingyu has joined #openstack-nova | 02:16 | |
| *** psachin has joined #openstack-nova | 02:16 | |
| *** gyee has quit IRC | 02:17 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 02:17 |
| *** yamamoto has joined #openstack-nova | 02:20 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: WIP new migration threads control https://review.openstack.org/563505 | 02:22 |
| *** dpawlik has joined #openstack-nova | 02:25 | |
| *** mikal_ has quit IRC | 02:26 | |
| *** mikal has joined #openstack-nova | 02:26 | |
| *** markvoelker has joined #openstack-nova | 02:26 | |
| *** felipemonteiro has joined #openstack-nova | 02:29 | |
| *** dpawlik has quit IRC | 02:29 | |
| *** liverpooler has quit IRC | 02:31 | |
| *** dougshelley66 has quit IRC | 02:33 | |
| *** sdeath has joined #openstack-nova | 02:36 | |
| *** dougshelley66 has joined #openstack-nova | 02:36 | |
| *** felipemonteiro has quit IRC | 02:37 | |
| *** yingjun has quit IRC | 02:39 | |
| *** mikal has quit IRC | 02:41 | |
| *** david-lyle has quit IRC | 02:45 | |
| *** blkart has quit IRC | 02:45 | |
| *** mikal has joined #openstack-nova | 02:49 | |
| *** masuberu has joined #openstack-nova | 02:52 | |
| *** masber has quit IRC | 02:52 | |
| openstackgerrit | Merged openstack/nova master: libvirt: fix setting tx_queue_size when rx_queue_size is not set https://review.openstack.org/565573 | 02:54 |
| *** dougshelley66 has quit IRC | 02:55 | |
| *** tuanla____ has joined #openstack-nova | 02:55 | |
| *** markvoelker has quit IRC | 02:56 | |
| *** markvoelker has joined #openstack-nova | 02:57 | |
| *** blkart has joined #openstack-nova | 03:02 | |
| *** lei-zh has quit IRC | 03:02 | |
| *** lei-zh has joined #openstack-nova | 03:03 | |
| *** armaan has quit IRC | 03:03 | |
| *** armaan_ has joined #openstack-nova | 03:03 | |
| *** yingjun has joined #openstack-nova | 03:05 | |
| *** BrinZhang has joined #openstack-nova | 03:07 | |
| *** sapd has quit IRC | 03:08 | |
| *** slaweq has joined #openstack-nova | 03:11 | |
| *** namnh has joined #openstack-nova | 03:14 | |
| *** slaweq has quit IRC | 03:16 | |
| *** moshele has joined #openstack-nova | 03:17 | |
| *** moshele has quit IRC | 03:19 | |
| *** hongbin has quit IRC | 03:19 | |
| *** READ10 has quit IRC | 03:24 | |
| *** takashin has quit IRC | 03:25 | |
| *** yingjun has quit IRC | 03:27 | |
| *** nicolasbock has quit IRC | 03:31 | |
| *** vivsoni_ has quit IRC | 03:33 | |
| *** slaweq has joined #openstack-nova | 03:35 | |
| *** armaan_ has quit IRC | 03:39 | |
| *** hamzy has joined #openstack-nova | 03:40 | |
| *** fragatina has joined #openstack-nova | 03:42 | |
| *** fragatin_ has joined #openstack-nova | 03:44 | |
| *** fragatina has quit IRC | 03:47 | |
| *** slaweq has quit IRC | 03:49 | |
| *** slaweq has joined #openstack-nova | 03:50 | |
| *** takashin has joined #openstack-nova | 03:54 | |
| *** slaweq has quit IRC | 03:55 | |
| *** vivsoni has joined #openstack-nova | 03:55 | |
| *** slaweq has joined #openstack-nova | 03:55 | |
| *** slaweq has quit IRC | 03:59 | |
| *** lei-zh has quit IRC | 04:00 | |
| *** vivsoni has quit IRC | 04:01 | |
| *** slaweq has joined #openstack-nova | 04:05 | |
| *** slaweq has quit IRC | 04:10 | |
| *** vivsoni has joined #openstack-nova | 04:10 | |
| *** slaweq has joined #openstack-nova | 04:15 | |
| *** slaweq has quit IRC | 04:20 | |
| *** slaweq has joined #openstack-nova | 04:20 | |
| *** lei-zh has joined #openstack-nova | 04:22 | |
| *** armaan has joined #openstack-nova | 04:23 | |
| *** janki has joined #openstack-nova | 04:29 | |
| *** slaweq has quit IRC | 04:30 | |
| *** trozet has quit IRC | 04:30 | |
| *** lei-zh has quit IRC | 04:30 | |
| *** moshele has joined #openstack-nova | 04:32 | |
| *** udesale has joined #openstack-nova | 04:33 | |
| *** slaweq has joined #openstack-nova | 04:35 | |
| *** masuberu has quit IRC | 04:47 | |
| *** abhishekk has joined #openstack-nova | 04:48 | |
| *** links has joined #openstack-nova | 04:50 | |
| *** moshele has quit IRC | 04:55 | |
| *** yingjun has joined #openstack-nova | 04:56 | |
| *** slaweq has quit IRC | 05:00 | |
| *** Nel1x has quit IRC | 05:00 | |
| *** slaweq has joined #openstack-nova | 05:05 | |
| *** phuongnh has joined #openstack-nova | 05:14 | |
| *** slaweq has quit IRC | 05:15 | |
| *** ratailor has joined #openstack-nova | 05:17 | |
| *** yamahata has joined #openstack-nova | 05:17 | |
| *** mikal_ has joined #openstack-nova | 05:20 | |
| *** yingjun has quit IRC | 05:22 | |
| *** belmoreira has joined #openstack-nova | 05:22 | |
| *** mikal has quit IRC | 05:22 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | 05:23 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | 05:23 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | 05:23 |
| *** vic-wu has quit IRC | 05:34 | |
| *** Eran_Kuris_ has joined #openstack-nova | 05:34 | |
| *** sridharg has joined #openstack-nova | 05:36 | |
| *** bhujay has joined #openstack-nova | 05:46 | |
| *** moshele has joined #openstack-nova | 05:47 | |
| *** dklyle has joined #openstack-nova | 05:47 | |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 05:48 |
| *** yingjun has joined #openstack-nova | 05:48 | |
| *** moshele has quit IRC | 05:51 | |
| *** Taseer has joined #openstack-nova | 06:00 | |
| *** udesale_ has joined #openstack-nova | 06:03 | |
| *** udesale_ has quit IRC | 06:04 | |
| *** udesale_ has joined #openstack-nova | 06:05 | |
| *** udesale has quit IRC | 06:05 | |
| *** udesale__ has joined #openstack-nova | 06:07 | |
| *** jaosorior has joined #openstack-nova | 06:07 | |
| *** udesale_ has quit IRC | 06:09 | |
| *** hjensas has quit IRC | 06:11 | |
| *** sapd has joined #openstack-nova | 06:16 | |
| *** armaan has quit IRC | 06:20 | |
| *** armaan has joined #openstack-nova | 06:21 | |
| *** fragatina has joined #openstack-nova | 06:25 | |
| *** tiendc has quit IRC | 06:27 | |
| *** fragatin_ has quit IRC | 06:27 | |
| *** do3meli has joined #openstack-nova | 06:27 | |
| openstackgerrit | jichenjc proposed openstack/nova master: [doc] Add soft_deleted flag https://review.openstack.org/562897 | 06:28 |
| *** dklyle has quit IRC | 06:29 | |
| *** moshele has joined #openstack-nova | 06:34 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove ec2 object definitions https://review.openstack.org/557150 | 06:39 |
| openstackgerrit | jichenjc proposed openstack/nova master: remove ec2 db functions https://review.openstack.org/557572 | 06:39 |
| *** jaosorior has quit IRC | 06:41 | |
| *** udesale_ has joined #openstack-nova | 06:43 | |
| *** udesale_ has quit IRC | 06:43 | |
| *** udesale has joined #openstack-nova | 06:44 | |
| *** udesale__ has quit IRC | 06:45 | |
| *** annp has joined #openstack-nova | 06:48 | |
| *** rcernin_ has joined #openstack-nova | 06:50 | |
| *** rcernin has quit IRC | 06:51 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Enhance api-guide general info some updates https://review.openstack.org/561773 | 06:52 |
| *** MasterofJOKers_ has joined #openstack-nova | 06:55 | |
| *** MasterofJOKers has quit IRC | 06:57 | |
| *** MasterofJOKers_ is now known as MasterofJOKers | 06:57 | |
| *** slaweq has joined #openstack-nova | 06:58 | |
| *** slaweq has quit IRC | 06:58 | |
| *** slaweq has joined #openstack-nova | 06:58 | |
| *** slaweq has quit IRC | 06:58 | |
| *** slaweq has joined #openstack-nova | 06:58 | |
| openstackgerrit | inspurericzhang proposed openstack/nova master: Replace Chinese punctuation with English punctuation https://review.openstack.org/565655 | 06:59 |
| *** yamahata has quit IRC | 07:02 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid raise InstanceNotRunning exception https://review.openstack.org/541152 | 07:04 |
| *** hjensas has joined #openstack-nova | 07:04 | |
| *** rcernin_ has quit IRC | 07:05 | |
| *** sahid has joined #openstack-nova | 07:06 | |
| *** dpawlik has joined #openstack-nova | 07:06 | |
| *** dpawlik has quit IRC | 07:11 | |
| *** armaan has quit IRC | 07:12 | |
| *** armaan has joined #openstack-nova | 07:13 | |
| *** yamahata has joined #openstack-nova | 07:14 | |
| *** dpawlik has joined #openstack-nova | 07:15 | |
| *** armaan has quit IRC | 07:18 | |
| *** armaan has joined #openstack-nova | 07:18 | |
| *** giblet is now known as gibi | 07:20 | |
| gibi | good morning nova | 07:22 |
| *** tesseract has joined #openstack-nova | 07:25 | |
| *** armaan has quit IRC | 07:25 | |
| *** armaan has joined #openstack-nova | 07:26 | |
| *** yingjun has quit IRC | 07:26 | |
| *** armaan has quit IRC | 07:27 | |
| *** armaan has joined #openstack-nova | 07:28 | |
| *** ragiman_ has joined #openstack-nova | 07:32 | |
| *** armaan has quit IRC | 07:33 | |
| *** dklyle has joined #openstack-nova | 07:34 | |
| *** damien_r has joined #openstack-nova | 07:34 | |
| *** armaan has joined #openstack-nova | 07:34 | |
| *** belmoreira has quit IRC | 07:37 | |
| *** damien_r has quit IRC | 07:38 | |
| *** damien_r has joined #openstack-nova | 07:38 | |
| *** yingjun has joined #openstack-nova | 07:41 | |
| *** armaan has quit IRC | 07:41 | |
| *** ccamacho has joined #openstack-nova | 07:48 | |
| *** aloga has quit IRC | 07:52 | |
| *** aloga has joined #openstack-nova | 07:52 | |
| *** hemna_ has quit IRC | 07:56 | |
| *** mgoddard has joined #openstack-nova | 07:56 | |
| *** takashin has left #openstack-nova | 08:00 | |
| *** belmoreira has joined #openstack-nova | 08:01 | |
| *** tssurya has joined #openstack-nova | 08:04 | |
| *** bhujay has quit IRC | 08:05 | |
| *** sam_nowitzki has joined #openstack-nova | 08:05 | |
| *** bhujay has joined #openstack-nova | 08:08 | |
| *** pcaruana has joined #openstack-nova | 08:08 | |
| lyarwood | randomhack: https://review.openstack.org/#/c/460243/ fixed this during Queens | 08:09 |
| *** belmoreira has quit IRC | 08:10 | |
| lyarwood | randomhack: it's the same underlying failure as this swap volume bug https://bugs.launchpad.net/nova/+bug/1739593 | 08:10 |
| openstack | Launchpad bug 1739593 in OpenStack Compute (nova) pike "[OSSA-2018-001] Swapping encrypted volumes can lead to data loss and a possible compute host DOS attack (CVE-2017-18191)" [High,Fix committed] - Assigned to Lee Yarwood (lyarwood) | 08:10 |
| *** mdnadeem has joined #openstack-nova | 08:10 | |
| lyarwood | I can't mark it as a dup for some reason | 08:11 |
| *** andreas_s has joined #openstack-nova | 08:11 | |
| *** belmoreira has joined #openstack-nova | 08:11 | |
| *** salv-orlando has joined #openstack-nova | 08:12 | |
| *** bhujay has quit IRC | 08:13 | |
| *** yamahata has quit IRC | 08:16 | |
| *** mdbooth has joined #openstack-nova | 08:18 | |
| kashyap | stephenfin: Yes, I intend to handle those two clean-ups | 08:19 |
| kashyap | Noted here: https://review.openstack.org/#/c/564012/4/nova/virt/libvirt/driver.py | 08:19 |
| *** lucas-afk is now known as lucasagomes | 08:19 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/queens: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565668 | 08:20 |
| *** aloga has quit IRC | 08:21 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/pike: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565670 | 08:21 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 08:22 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 08:24 |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 08:25 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P):Get vgpu info from `allocations` https://review.openstack.org/521717 | 08:27 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova stable/ocata: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565672 | 08:27 |
| openstackgerrit | fupingxie proposed openstack/nova master: Update the parameter explain when update a volume attachment https://review.openstack.org/565181 | 08:29 |
| *** bhujay has joined #openstack-nova | 08:32 | |
| *** abhishekk has quit IRC | 08:35 | |
| *** abhishekk has joined #openstack-nova | 08:38 | |
| *** derekh has joined #openstack-nova | 08:39 | |
| *** lei-zh has joined #openstack-nova | 08:41 | |
| *** xinliang has quit IRC | 08:42 | |
| *** xinliang has joined #openstack-nova | 08:42 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Initial change set of z/VM driver https://review.openstack.org/523387 | 08:43 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: Spawn and destroy function of z/VM driver https://review.openstack.org/527658 | 08:43 |
| *** bhujay has quit IRC | 08:43 | |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add snapshot function https://review.openstack.org/534240 | 08:43 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add power actions https://review.openstack.org/543340 | 08:43 |
| openstackgerrit | jichenjc proposed openstack/nova master: z/VM Driver: add get console output https://review.openstack.org/543344 | 08:43 |
| *** sree_ has joined #openstack-nova | 08:46 | |
| *** sree_ is now known as Guest67087 | 08:46 | |
| *** salv-orlando has quit IRC | 08:47 | |
| *** salv-orlando has joined #openstack-nova | 08:48 | |
| *** Dinesh_Bhor has quit IRC | 08:48 | |
| *** namnh has quit IRC | 08:50 | |
| *** phuongnh has quit IRC | 08:50 | |
| *** trungnv has quit IRC | 08:50 | |
| *** tuanla____ has quit IRC | 08:50 | |
| *** hoangcx has quit IRC | 08:50 | |
| *** phuongnh has joined #openstack-nova | 08:51 | |
| *** tuanla____ has joined #openstack-nova | 08:51 | |
| *** namnh has joined #openstack-nova | 08:51 | |
| *** hoangcx has joined #openstack-nova | 08:51 | |
| *** trungnv has joined #openstack-nova | 08:51 | |
| *** salv-orlando has quit IRC | 08:52 | |
| kashyap | stephenfin: Ah, you've already posted those two | 08:55 |
| *** dklyle has quit IRC | 08:56 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 08:56 |
| kashyap | dansmith: We need to backport "libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565043" back to Ocata, or else the existing documention would be a "lie". | 09:01 |
| lyarwood | kashyap: argh my bad, I did mean to review these on Monday | 09:02 |
| kashyap | lyarwood: No problem; I only _just_ backported that | 09:03 |
| kashyap | As the main change got merged while I was asleep | 09:03 |
| kashyap | So you're right on time | 09:03 |
| kashyap | Thanks! | 09:03 |
| lyarwood | kashyap: huh, maybe you just said they'd be coming soon then :) | 09:03 |
| lyarwood | kashyap: and ack, I'll get to them today | 09:03 |
| kashyap | lyarwood: Thanks; you'll see the changes needing reviews here: https://review.openstack.org/#/q/topic:bug/1750829+(status:open+OR+status:merged) | 09:04 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 09:07 |
| *** sam_nowitzki has quit IRC | 09:10 | |
| *** rmart04 has joined #openstack-nova | 09:12 | |
| *** dklyle has joined #openstack-nova | 09:15 | |
| *** belmoreira has quit IRC | 09:16 | |
| *** armaan has joined #openstack-nova | 09:23 | |
| *** dklyle has quit IRC | 09:26 | |
| *** lei-zh1 has joined #openstack-nova | 09:33 | |
| *** lei-zh has quit IRC | 09:35 | |
| *** rmart04_ has joined #openstack-nova | 09:36 | |
| *** rmart04 has quit IRC | 09:36 | |
| *** rmart04_ is now known as rmart04 | 09:36 | |
| sahid | jaypipes: if you have a moment, I have a fix here, it's about how we compute memory we could be in situation where guests get oomkilled | 09:39 |
| sahid | https://review.openstack.org/#/c/532168/ | 09:39 |
| *** lei-zh1 has quit IRC | 09:41 | |
| *** lei-zh1 has joined #openstack-nova | 09:41 | |
| *** armaan has quit IRC | 09:43 | |
| *** yingjun has quit IRC | 09:45 | |
| *** Tom-Tom has quit IRC | 09:47 | |
| *** Tom-Tom has joined #openstack-nova | 09:48 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Convert fping API to processutils.execute() https://review.openstack.org/564144 | 09:48 |
| openstackgerrit | Michael Still proposed openstack/nova master: Convert xenapi's xvp console to processutils. https://review.openstack.org/564145 | 09:48 |
| openstackgerrit | Michael Still proposed openstack/nova master: Convert certificate generation to processutils. https://review.openstack.org/564146 | 09:48 |
| *** rmart04 has quit IRC | 09:50 | |
| *** dtantsur|afk is now known as dtantsur | 09:50 | |
| *** rmart04 has joined #openstack-nova | 09:52 | |
| *** Tom-Tom has quit IRC | 09:52 | |
| *** phuongnh has quit IRC | 09:55 | |
| *** tuanla____ has quit IRC | 09:55 | |
| *** namnh has quit IRC | 09:55 | |
| *** tuanla____ has joined #openstack-nova | 09:56 | |
| *** namnh has joined #openstack-nova | 09:56 | |
| *** phuongnh has joined #openstack-nova | 09:56 | |
| *** Guest72284 has quit IRC | 09:57 | |
| *** jichen has quit IRC | 09:58 | |
| *** namnh has quit IRC | 09:59 | |
| *** andreas_s has quit IRC | 09:59 | |
| *** BobBall has joined #openstack-nova | 10:02 | |
| *** armaan has joined #openstack-nova | 10:03 | |
| *** jmccarthy has joined #openstack-nova | 10:04 | |
| *** lei-zh1 has quit IRC | 10:06 | |
| *** Tom-Tom has joined #openstack-nova | 10:07 | |
| *** lei-zh1 has joined #openstack-nova | 10:07 | |
| *** andreas_s has joined #openstack-nova | 10:09 | |
| *** hoangcx has quit IRC | 10:09 | |
| *** andreas_s_ has joined #openstack-nova | 10:11 | |
| *** Tom-Tom has quit IRC | 10:11 | |
| *** andreas_s has quit IRC | 10:14 | |
| openstackgerrit | sahid proposed openstack/nova master: compute: introduce cpu_shared_set option https://review.openstack.org/561912 | 10:15 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: always pass emulator threads policy https://review.openstack.org/561913 | 10:15 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set https://review.openstack.org/510897 | 10:15 |
| *** salv-orlando has joined #openstack-nova | 10:15 | |
| *** andreas_s_ has quit IRC | 10:15 | |
| *** swamireddy has quit IRC | 10:17 | |
| *** andreas_s has joined #openstack-nova | 10:19 | |
| *** andreas_s has quit IRC | 10:19 | |
| *** andreas_s has joined #openstack-nova | 10:19 | |
| *** armaan has quit IRC | 10:21 | |
| *** andreas_s has quit IRC | 10:21 | |
| *** andreas_s has joined #openstack-nova | 10:22 | |
| *** salv-orlando has quit IRC | 10:22 | |
| *** andreas_s has quit IRC | 10:23 | |
| *** andreas_s has joined #openstack-nova | 10:25 | |
| *** rmart04 has quit IRC | 10:31 | |
| *** mikal has joined #openstack-nova | 10:32 | |
| *** sree_ has joined #openstack-nova | 10:32 | |
| *** abhishekk has quit IRC | 10:32 | |
| *** sree_ is now known as Guest15027 | 10:32 | |
| *** lei-zh1 has quit IRC | 10:33 | |
| *** mikal_ has quit IRC | 10:33 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move image conversion to privsep. https://review.openstack.org/554437 | 10:35 |
| openstackgerrit | Michael Still proposed openstack/nova master: We don't need utils.trycmd any more. https://review.openstack.org/554439 | 10:35 |
| openstackgerrit | Michael Still proposed openstack/nova master: We no longer need rootwrap. https://review.openstack.org/554438 | 10:35 |
| *** Guest67087 has quit IRC | 10:35 | |
| *** annp has quit IRC | 10:37 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552 | 10:37 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility module. https://review.openstack.org/557835 | 10:37 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 10:37 |
| *** nicolasbock has joined #openstack-nova | 10:42 | |
| *** rmart04 has joined #openstack-nova | 10:45 | |
| *** stephenfin has quit IRC | 10:47 | |
| *** tbachman has quit IRC | 10:49 | |
| *** stephenfin has joined #openstack-nova | 10:51 | |
| *** tuanla____ has quit IRC | 10:51 | |
| *** wolverineav has joined #openstack-nova | 10:54 | |
| *** phuongnh has quit IRC | 10:56 | |
| *** dougshelley66 has joined #openstack-nova | 10:57 | |
| openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | 10:58 |
| *** andreas_s has quit IRC | 11:00 | |
| *** andreas_s has joined #openstack-nova | 11:02 | |
| *** belmoreira has joined #openstack-nova | 11:02 | |
| *** r-daneel has quit IRC | 11:04 | |
| *** sdeath has quit IRC | 11:05 | |
| *** andreas_s has quit IRC | 11:07 | |
| *** xinliang has quit IRC | 11:08 | |
| *** jaosorior has joined #openstack-nova | 11:08 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make association_refresh configurable https://review.openstack.org/565526 | 11:09 |
| jbernard | melwitt: re the patch (yesterday, rbd busy snapshots), it's a valid rbd state, but tempest does not tolerate it | 11:11 |
| moshele | stephenfin: hi can you review the pike backport https://review.openstack.org/#/c/563501/ ? | 11:11 |
| *** r-daneel has joined #openstack-nova | 11:11 | |
| jbernard | melwitt: the order of operations during teardown is non-deterministic | 11:12 |
| openstackgerrit | Vladyslav Drok proposed openstack/nova master: ironic: Report resources as reserved when needed https://review.openstack.org/517921 | 11:13 |
| *** hjensas has quit IRC | 11:13 | |
| *** randomhack_ has joined #openstack-nova | 11:13 | |
| *** Tom-Tom has joined #openstack-nova | 11:14 | |
| *** andreas_s has joined #openstack-nova | 11:15 | |
| *** sdeath has joined #openstack-nova | 11:15 | |
| *** jaosorior has quit IRC | 11:16 | |
| openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | 11:17 |
| *** andreas_s_ has joined #openstack-nova | 11:19 | |
| *** janki has quit IRC | 11:19 | |
| *** sdeath has quit IRC | 11:20 | |
| *** andreas_s has quit IRC | 11:20 | |
| *** swamireddy has joined #openstack-nova | 11:20 | |
| *** andreas_s_ has quit IRC | 11:20 | |
| *** armaan has joined #openstack-nova | 11:20 | |
| *** andreas_s has joined #openstack-nova | 11:20 | |
| *** yassine has joined #openstack-nova | 11:20 | |
| *** yassine is now known as Guest80281 | 11:21 | |
| *** jmccarthy has quit IRC | 11:22 | |
| *** Tom-Tom has quit IRC | 11:23 | |
| *** Tom-Tom has joined #openstack-nova | 11:24 | |
| kashyap | melwitt: When you're back, mriedem is right about that extraCPU flags blueprint: a future patch needs to add the ability to remove specific CPU flags. | 11:24 |
| *** jmccarthy has joined #openstack-nova | 11:25 | |
| *** dklyle has joined #openstack-nova | 11:25 | |
| *** sdeath has joined #openstack-nova | 11:26 | |
| *** Tom-Tom has quit IRC | 11:28 | |
| *** andreas_s has quit IRC | 11:29 | |
| *** udesale has quit IRC | 11:30 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012 | 11:30 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_REALTIME_VERSION https://review.openstack.org/565707 | 11:30 |
| *** andreas_s has joined #openstack-nova | 11:30 | |
| *** lucasagomes is now known as lucas-hungry | 11:32 | |
| kashyap | stephenfin: sahid: ^ On the last commit above (on REALTIME), can you double-confirm what I wrote in the commit message? (I.e. no tests need to cleaned-up.) | 11:33 |
| *** dougshelley66 has quit IRC | 11:35 | |
| kashyap | (Also the one before it, 'CPU_POLICY' one, lost +W due to rebase.) | 11:35 |
| *** bhujay has joined #openstack-nova | 11:39 | |
| *** mvk has quit IRC | 11:40 | |
| *** salv-orlando has joined #openstack-nova | 11:46 | |
| *** dklyle has quit IRC | 11:50 | |
| *** Guest80281 has quit IRC | 11:52 | |
| *** salv-orlando has quit IRC | 11:52 | |
| *** salv-orlando has joined #openstack-nova | 11:53 | |
| *** yassine_ has joined #openstack-nova | 11:53 | |
| *** Guest15027 has quit IRC | 11:53 | |
| *** sree has joined #openstack-nova | 11:54 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Convert configdrive to use processutils. https://review.openstack.org/565723 | 11:55 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_QEMU_POSTCOPY_VERSION https://review.openstack.org/565724 | 11:55 |
| *** andreas_s has quit IRC | 11:56 | |
| *** salv-orlando has quit IRC | 11:57 | |
| *** andreas_s has joined #openstack-nova | 11:57 | |
| *** sree has quit IRC | 11:59 | |
| *** andreas_s has quit IRC | 11:59 | |
| *** andreas_s has joined #openstack-nova | 12:00 | |
| *** bhujay has quit IRC | 12:00 | |
| *** bhujay has joined #openstack-nova | 12:02 | |
| *** Taseer has quit IRC | 12:02 | |
| *** tbachman has joined #openstack-nova | 12:04 | |
| *** rmart04 has quit IRC | 12:05 | |
| *** bhujay has quit IRC | 12:08 | |
| *** tbachman has quit IRC | 12:09 | |
| *** dougshelley66 has joined #openstack-nova | 12:09 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | 12:10 |
| *** edmondsw has joined #openstack-nova | 12:13 | |
| *** edmondsw has quit IRC | 12:13 | |
| *** edmondsw has joined #openstack-nova | 12:13 | |
| *** rmart04 has joined #openstack-nova | 12:13 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: any traits in allocation_candidate query https://review.openstack.org/565730 | 12:13 |
| *** andreas_s has quit IRC | 12:14 | |
| *** tbachman has joined #openstack-nova | 12:15 | |
| *** andreas_s has joined #openstack-nova | 12:16 | |
| *** dtantsur is now known as dtantsur|brb | 12:23 | |
| *** fragatina has quit IRC | 12:24 | |
| *** eharney has quit IRC | 12:24 | |
| *** fragatina has joined #openstack-nova | 12:24 | |
| *** yamamoto has quit IRC | 12:25 | |
| *** yamamoto has joined #openstack-nova | 12:25 | |
| *** andreas_s has quit IRC | 12:26 | |
| *** moshele has quit IRC | 12:26 | |
| *** moshele has joined #openstack-nova | 12:26 | |
| *** andreas_s_ has joined #openstack-nova | 12:27 | |
| *** alex_xu has quit IRC | 12:29 | |
| *** andreas__ has joined #openstack-nova | 12:29 | |
| *** pcaruana has quit IRC | 12:30 | |
| *** Tom-Tom has joined #openstack-nova | 12:31 | |
| *** andreas_s_ has quit IRC | 12:32 | |
| *** liverpooler has joined #openstack-nova | 12:33 | |
| *** janki has joined #openstack-nova | 12:35 | |
| *** Tom-Tom has quit IRC | 12:36 | |
| *** alex_xu has joined #openstack-nova | 12:36 | |
| *** vladikr has joined #openstack-nova | 12:38 | |
| stephenfin | kashyap: Can do | 12:39 |
| *** randomhack has quit IRC | 12:40 | |
| *** salv-orlando has joined #openstack-nova | 12:43 | |
| *** armaan has quit IRC | 12:44 | |
| *** lucas-hungry is now known as lucasagomes | 12:44 | |
| *** rmart04_ has joined #openstack-nova | 12:44 | |
| jmccarthy | Hmm anyone try cold migrate an instance (cinder/lvm backed) with master ? I'm not sure what is the story - but I have this /var/lib/nova/instances/<id>/disk.info left behind on the source host afterwards (after resize verify) and I think it shouldn't be ? | 12:45 |
| *** belmoreira has quit IRC | 12:45 | |
| *** salv-orl_ has joined #openstack-nova | 12:45 | |
| *** rmart04 has quit IRC | 12:45 | |
| *** rmart04_ is now known as rmart04 | 12:45 | |
| *** salv-orlando has quit IRC | 12:46 | |
| *** psachin has quit IRC | 12:47 | |
| kashyap | stephenfin: Gracias | 12:48 |
| *** andreas__ has quit IRC | 12:48 | |
| *** mvk has joined #openstack-nova | 12:50 | |
| jmccarthy | Seems like it's related to this, which is fixed .. https://bugs.launchpad.net/nova/+bug/1666831 - where can one check whether tests for this bug are currently passing ? | 12:52 |
| openstack | Launchpad bug 1666831 in OpenStack Compute (nova) ocata "Nova recreates instance directory after migration/resize" [Low,Fix committed] - Assigned to Lee Yarwood (lyarwood) | 12:52 |
| *** andreas_s has joined #openstack-nova | 12:53 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 12:54 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 12:54 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | 12:54 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487 | 12:54 |
| *** trozet has joined #openstack-nova | 12:55 | |
| *** andreas_s has quit IRC | 12:56 | |
| *** andreas_s has joined #openstack-nova | 12:57 | |
| *** andreas_s has quit IRC | 12:57 | |
| *** andreas_s has joined #openstack-nova | 12:59 | |
| *** lyan has joined #openstack-nova | 13:01 | |
| *** lyan is now known as Guest27658 | 13:01 | |
| *** READ10 has joined #openstack-nova | 13:02 | |
| *** salv-orl_ has quit IRC | 13:03 | |
| *** salv-orlando has joined #openstack-nova | 13:03 | |
| *** mriedem has joined #openstack-nova | 13:03 | |
| *** udesale has joined #openstack-nova | 13:03 | |
| mriedem | stephenfin: i know it's past it's runway slot now, but the console objects series from mel is ready to go i think if you have some time to go through it, i know you've been through the earlier patches in the series so you have some context: https://review.openstack.org/#/q/topic:bp/convert-consoles-to-objects+status:open | 13:04 |
| stephenfin | mriedem: Sure, I'd be happy to | 13:05 |
| mriedem | thanks | 13:06 |
| mriedem | kashyap: is there going to be more to https://blueprints.launchpad.net/nova/+spec/libvirt-cpu-model-extra-flags ? | 13:06 |
| mriedem | or is the bp complete? | 13:06 |
| kashyap | mriedem: Your comment on Friday was right | 13:06 |
| kashyap | mriedem: There is the one last item | 13:06 |
| kashyap | That you quoted to Mel on IRC | 13:07 |
| mriedem | kashyap: even though the last patch says "(A future Nova patch will also allow ability to remove CPU flags.)" | 13:07 |
| kashyap | mriedem: We need ability to remove flags as well. That's one more item | 13:07 |
| kashyap | Indeed | 13:07 |
| mriedem | ok | 13:07 |
| *** pchavva has joined #openstack-nova | 13:07 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: Support mixing required traits with any traits https://review.openstack.org/565741 | 13:07 |
| kashyap | mriedem: If you are wondering why we need that. Here is one case I wrote up: | 13:08 |
| *** salv-orlando has quit IRC | 13:08 | |
| kashyap | You might ask: "Why would you want to remove a CPU flag though?" One | 13:08 |
| kashyap | scenario for that is: An Operator wants to generate a baseline CPU | 13:08 |
| kashyap | config. And a certain CPU flag is causing performance issue or other | 13:08 |
| kashyap | nuisance, and if the Operator isolated the problem to _that_ specific | 13:08 |
| kashyap | CPU flag, then she may want to remove the flag. | 13:08 |
| *** pcaruana has joined #openstack-nova | 13:08 | |
| kashyap | mriedem: And thanks for shepherding through the minor nits and merging! | 13:09 |
| *** Tom-Tom has joined #openstack-nova | 13:09 | |
| gibi | mriedem: hi! regarding having one bp with two specs. Jay asked me to split the spec. Does it automatically means I have to have two bp? | 13:10 |
| *** Tom-Tom has quit IRC | 13:12 | |
| mriedem | yes | 13:12 |
| *** Tom-Tom has joined #openstack-nova | 13:12 | |
| gibi | mriedem: OK, then I will file a separate bp too | 13:12 |
| *** dave-mccowan has joined #openstack-nova | 13:12 | |
| *** zz9pzza has quit IRC | 13:13 | |
| *** Shilpa has joined #openstack-nova | 13:14 | |
| kashyap | mriedem: When you get a minute, I have a different topic, which needs a opinion. It's quick: | 13:16 |
| kashyap | It's about this: https://review.openstack.org/#/c/565242/ -- libvirt: Remove support for Intel CMT `perf` events | 13:16 |
| kashyap | mriedem: At PTG, we agreed (based on info at that time) to log a warning when Nova's `enabled_perf_events` config is used that it is deprecated, and remove it later | 13:16 |
| *** yassine_ has quit IRC | 13:17 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Placement: support mixing required traits with any traits https://review.openstack.org/565741 | 13:17 |
| *** yassine has joined #openstack-nova | 13:17 | |
| gibi | mriedem: fixed ^^ | 13:17 |
| kashyap | mriedem: However ... upon closer code inspection, we cannot deprecate the config itself, as there are *other* `perf` events besides Intel's CMT events. | 13:17 |
| *** yassine is now known as Guest25377 | 13:17 | |
| stephenfin | mriedem, kashyap: I've just left a +1 on this because I think we should take the opportunity to clarify things a little while we're here https://review.openstack.org/#/c/565724/1 | 13:17 |
| kashyap | So here comes my change: What i am proposing is to _remove_ the support for those three Intel CMT events, which is encoded in a dict | 13:17 |
| *** dave-mcc_ has joined #openstack-nova | 13:17 | |
| stephenfin | However, it's not totally related so I can bump to a +2 if you disagree | 13:17 |
| kashyap | (And retain the config itself) | 13:17 |
| kashyap | stephenfin: Let me take a quick look | 13:18 |
| *** eharney has joined #openstack-nova | 13:19 | |
| *** mingyu has quit IRC | 13:19 | |
| *** dave-mccowan has quit IRC | 13:19 | |
| kashyap | stephenfin: Your point is right about it not being clear; but it's a separate change. | 13:20 |
| kashyap | stephenfin: However, | 13:20 |
| kashyap | stephenfin: ... your nit to clean-up _is_post_copy_available() is reasonable, and I should make it in this change. | 13:20 |
| kashyap | It's not a functional change, but just making it like the one below. | 13:21 |
| *** bkopilov_ has quit IRC | 13:22 | |
| *** bkopilov has quit IRC | 13:22 | |
| kashyap | (Even the test change, it still is maybe reasonable to adjust in the current patch.) | 13:22 |
| kashyap | (On a call; will look once done.) | 13:22 |
| *** randomhack_ has quit IRC | 13:23 | |
| mriedem | kashyap: i don't know why PERF_EVENTS_CPU_FLAG_MAPPING are special; you should also probably ask alex_xu about this since he helped work on this getting in | 13:24 |
| kashyap | mriedem: Yep, I actually meant to "Cc" alex_xu on the comment above. | 13:24 |
| kashyap | mriedem: The original config `enabled_perf_events` was added to monitor the flags noted in the CPU_FLAG_MAPPING | 13:24 |
| kashyap | That dict has the three `perf` events for Intel CMT | 13:24 |
| dansmith | kashyap: I know, I just don't think it's worth it.. just MHO | 13:25 |
| kashyap | The way I'm proposing is: (a) remove the broken-by-design Intel CMT (PERF_EVENTS_CPU_FLAG_MAPPING); (b) retain support for other events for `enabled_perf_events` | 13:25 |
| kashyap | dansmith: Hmm, it is a strict improvment, and is correcting a misleading lie that can impact instance boot | 13:26 |
| kashyap | Seems all the worth to me | 13:26 |
| kashyap | Anyway, the backports are done | 13:26 |
| kashyap | And they all pass too | 13:26 |
| kashyap | (It's just a couple of simple lines, FWIW) | 13:27 |
| kashyap | But otherwise, in general I agree with you that minor things shouldn't be backported all the way back like that. | 13:27 |
| *** belmoreira has joined #openstack-nova | 13:33 | |
| *** bkopilov_ has joined #openstack-nova | 13:34 | |
| *** awaugama has joined #openstack-nova | 13:34 | |
| *** bkopilov has joined #openstack-nova | 13:34 | |
| bauzas | folks, for those who don't know yet what I call the French May-hem, we're entering a month where a lot of French folks are out | 13:37 |
| dansmith | bauzas: so may is different how then? | 13:37 |
| alex_xu | kashyap: mriedem will check that tomorrow | 13:38 |
| bauzas | dansmith: yesterday was a holiday, plus 3 days | 13:38 |
| kashyap | alex_xu: I still need to fix unit tests; I proposed it just get the core change discussion going | 13:38 |
| * alex_xu is working on the family health problem, have limit bandwidth this week | 13:38 | |
| dansmith | bauzas: that was thinly veiled sarcasm | 13:38 |
| kashyap | alex_xu: No worries; take care of that. It's more important than this. | 13:39 |
| bauzas | yup :( | 13:39 |
| alex_xu | kashyap: thanks | 13:39 |
| bauzas | anyway, just trying to explain that if people want to discuss with me, ping me directly | 13:40 |
| * kashyap notices thin-veiling from dansmith | 13:40 | |
| bauzas | http://www.completefrance.com/home/news/national-holidays-in-france-in-may-1-4050587 | 13:40 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 13:43 |
| jaypipes | sahid: looking at your patch now. | 13:49 |
| sahid | thanks jaypipes, let me know if you need any clarifications | 13:52 |
| *** ratailor has quit IRC | 13:53 | |
| *** Eran_Kuris_ has quit IRC | 13:56 | |
| *** yamamoto has quit IRC | 13:57 | |
| *** belmoreira has quit IRC | 13:59 | |
| jaypipes | sahid: did this bug pop up in a RH customer recently or something? | 14:02 |
| *** abalutoiu__ has joined #openstack-nova | 14:04 | |
| *** mlavalle has joined #openstack-nova | 14:06 | |
| *** abalutoiu_ has quit IRC | 14:07 | |
| *** felipemonteiro has joined #openstack-nova | 14:07 | |
| mriedem | arvindn05: efried: jaypipes: bauzas: alex_xu: i've replied on the rebuild + image-defined traits thread; at this point i basically need to detach myself from the issue and ask melwitt or dansmith or others with fresh perspective to weigh in | 14:07 |
| sahid | jaypipes: yes BZ#1519540, basically two issues, one on Nova and the other in libvirt | 14:08 |
| sahid | https://bugzilla.redhat.com/show_bug.cgi?id=1519540 | 14:08 |
| openstack | bugzilla.redhat.com bug 1519540 in openstack-nova "nova reporting incorrect available memory" [High,On_dev] - Assigned to sferdjao | 14:08 |
| efried | mriedem: ack | 14:08 |
| *** alex_xu has quit IRC | 14:11 | |
| *** yamamoto has joined #openstack-nova | 14:11 | |
| jaypipes | sahid: k, reviewed. | 14:15 |
| jaypipes | sahid: I'd be interested in getting stephenfin's eye on that too. | 14:15 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_QEMU_POSTCOPY_VERSION https://review.openstack.org/565724 | 14:17 |
| kashyap | stephenfin: Alright, rewrote the unit test the above way. ^ Clearer? | 14:17 |
| *** alex_xu has joined #openstack-nova | 14:17 | |
| mriedem | mdbooth: looks like it's time for https://blueprints.launchpad.net/nova/+spec/local-disk-serial-numbers to go into a runway slot, are you going to be around for the next 2 weeks to respond to reviews? | 14:17 |
| mdbooth | mriedem: Yes indeed! | 14:18 |
| *** hongbin has joined #openstack-nova | 14:18 | |
| mdbooth | I'm out for the next 2 Mondays, but around apart from that and able to work on this. | 14:19 |
| mriedem | done, you've got until the 16th | 14:19 |
| mdbooth | Thanks | 14:19 |
| artom | What would cause a devstack-installed compute to not report to placement? | 14:21 |
| artom | I see it in both hypervisor-list and service list, so it's "in" the cloud | 14:21 |
| jaypipes | artom: spite. | 14:21 |
| mriedem | misconfig | 14:21 |
| artom | But can't boot VMs on it because I see no resource provider for it | 14:21 |
| mriedem | nova-compute can't talk to placement api maybe | 14:21 |
| mriedem | check [placement] section of nova-cpu.conf | 14:21 |
| mriedem | or just read the n-cpu logs | 14:21 |
| mriedem | see channel topic | 14:22 |
| artom | But this is devstack ;) | 14:22 |
| *** sambetts|afk is now known as sambetts | 14:23 | |
| artom | Hrmm, there's *no* [placement] in nova-cpu.conf | 14:23 |
| artom | That would explain things | 14:23 |
| *** Nil_ has joined #openstack-nova | 14:24 | |
| jaypipes | artom: like I said... spite ;P | 14:24 |
| artom | jaypipes, against me or against placement? | 14:24 |
| * artom fully expects "yes" as an answer. | 14:24 | |
| jaypipes | artom: 42 | 14:25 |
| *** hemna_ has joined #openstack-nova | 14:26 | |
| *** rmart04_ has joined #openstack-nova | 14:26 | |
| *** rmart04 has quit IRC | 14:26 | |
| *** rmart04_ is now known as rmart04 | 14:26 | |
| artom | mriedem, thanks for the pointers, I copied the controller's [placement] to nova-cpu.conf on the compute and fixed it | 14:28 |
| artom | So nova-cpu.conf is new-ish? I haven't actually looked at a devstack in a while. It's for the compute service, presumably? | 14:29 |
| *** alex_xu has quit IRC | 14:29 | |
| sahid | jaypipes: yep i think you are right on your review. thanks | 14:30 |
| *** felipemonteiro_ has joined #openstack-nova | 14:30 | |
| *** esberglu has joined #openstack-nova | 14:30 | |
| *** belmoreira has joined #openstack-nova | 14:31 | |
| mriedem | artom: been around since pike | 14:32 |
| artom | mriedem, shows how up to date I am :/ | 14:32 |
| jroll | mriedem: gdi, this is why you need a bouncer, meant to ping you on this: https://review.openstack.org/#/c/565736/ | 14:33 |
| jroll | s/meant/tried/ | 14:33 |
| mriedem | artom: i don't know why you'd not have placement configured properly, devstack should have set that all up for you | 14:33 |
| mriedem | gfdi jim, i'm a doctor not a guy that sits up all night waiting for irc pings | 14:34 |
| jroll | heh | 14:34 |
| *** felipemonteiro has quit IRC | 14:34 | |
| *** openstackgerrit has quit IRC | 14:34 | |
| artom | mriedem, I followed https://docs.openstack.org/devstack/latest/guides/neutron.html, maybe it's out of date for placement? Specifically, ENABLED_SERVICES on the compute | 14:34 |
| *** links has quit IRC | 14:34 | |
| mriedem | yes definitely | 14:35 |
| mriedem | placement-client should be in that list | 14:35 |
| mriedem | like https://docs.openstack.org/devstack/latest/guides/multinode-lab.html#configure-compute-nodes | 14:36 |
| artom | mriedem, aha, indeed | 14:36 |
| artom | Shall I apply force to a change in the textual representation of the documentation? | 14:37 |
| mriedem | yes | 14:37 |
| mriedem | be a good citizen | 14:37 |
| artom | Service guarantees citizenship! | 14:37 |
| jaypipes | sahid: ok, cool. test cases should be identical, though, so that's a plus :) | 14:38 |
| *** jmlowe has quit IRC | 14:39 | |
| *** felipemonteiro_ has quit IRC | 14:42 | |
| Shilpa | stephenfin: Hi | 14:42 |
| *** felipemonteiro_ has joined #openstack-nova | 14:42 | |
| stephenfin | Shilpa: o/ | 14:43 |
| kashyap | artom: Also, do post your renewed multi-node configs somewhere | 14:43 |
| jaypipes | mriedem, stephenfin, dansmith: your reviews on https://review.openstack.org/#/c/561315/ would be appreciated. thx in advance :) | 14:43 |
| kashyap | artom: For that poor soul who has to later setup to debug an "urgent" issue | 14:44 |
| Shilpa | stephenfin: i have verified tempest failure for https://review.openstack.org/#/c/550172/3 locally, and added few observations at http://paste.openstack.org/show/720231/, kindly go through the same. | 14:44 |
| kashyap | Mine are here: https://kashyapc.fedorapeople.org/virt/openstack/multi-node-configs/ (But they should be revamped) | 14:44 |
| stephenfin | Shilpa: Can do | 14:44 |
| stephenfin | kashyap: Yup, v2 of that looks much better. Thanks for addressing that. | 14:45 |
| stephenfin | Still have comments on the preceding patch, fyi | 14:45 |
| kashyap | stephenfin: No problem. Thanks for _spotting_ it. | 14:45 |
| *** vladikr has quit IRC | 14:45 | |
| kashyap | stephenfin: Oh, I see. Let me look | 14:46 |
| *** dtantsur|brb is now known as dtantsur | 14:46 | |
| *** dpawlik has quit IRC | 14:46 | |
| artom | kashyap, sure. The "tricky" bit was the hardware setup | 14:46 |
| *** felipemonteiro__ has joined #openstack-nova | 14:46 | |
| artom | Since I need to run on baremetal and don't control the network, I ended up cross-connecting the two machines on a second NIC and setting that as PUBLIC_INTERFACE | 14:47 |
| kashyap | Nested Virt for the win! | 14:47 |
| artom | kashyap, for NUMA live migration? Suuure ;) | 14:47 |
| kashyap | stephenfin: Okido, addressing that too | 14:47 |
| kashyap | artom: For testing PCI device assignment & NUMA, it is possible to an extent: https://www.berrange.com/posts/2017/02/16/setting-up-a-nested-kvm-guest-for-developing-testing-pci-device-assignment-with-numa/ | 14:48 |
| artom | Btw, I spent way too much setting up a thing where I could snapshot my entire devstack OS and revert, in case devstack failed and I needed to go back without having to reinstall from a USB key, but then my stack run essentially worked the first time, so :/ | 14:49 |
| jmccarthy | Hmm I seem to have some behaviour where disk.info shows up on the source host after a resize/cold migrate sort of like this review/bug: https://review.openstack.org/#/c/437356/ | 14:50 |
| jmccarthy | Although I have my nova repo checked out and 'tox -e py27 -- test_cleanup_resize' tests all pass locally, should these also cover cinder for the case: volume_driver = cinder.volume.drivers.lvm.LVMVolumeDriver ? | 14:50 |
| *** felipemonteiro_ has quit IRC | 14:50 | |
| *** vladikr has joined #openstack-nova | 14:50 | |
| *** mvk has quit IRC | 14:52 | |
| Shilpa | stephenfin: if you are busy elsewhere we can discuss the same tomorrow. | 14:57 |
| stephenfin | Shilpa: Yeah, if you wouldn't mind. Have a lot on my plate today | 14:57 |
| Shilpa | stephenfin: no issues, bye for now | 14:58 |
| *** openstackgerrit has joined #openstack-nova | 14:58 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop BAD_LIBVIRT_CPU_POLICY_VERSIONS https://review.openstack.org/564012 | 14:58 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_QEMU_POSTCOPY_VERSION https://review.openstack.org/565724 | 14:58 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_REALTIME_VERSION https://review.openstack.org/565707 | 14:58 |
| jmccarthy | s/these also cover cinder/these also cover nova/ | 14:59 |
| *** bhujay has joined #openstack-nova | 15:01 | |
| *** rmart04 has quit IRC | 15:01 | |
| *** jmlowe has joined #openstack-nova | 15:01 | |
| *** tbachman has quit IRC | 15:01 | |
| *** jmlowe has quit IRC | 15:03 | |
| *** udesale has quit IRC | 15:03 | |
| *** jmlowe has joined #openstack-nova | 15:04 | |
| melwitt | mriedem: I'll wade through the rebuild thread later | 15:05 |
| melwitt | mriedem: also, thanks for swapping the runway blueprint. I forgot to do it last night | 15:05 |
| *** gjayavelu has joined #openstack-nova | 15:05 | |
| mriedem | melwitt: re the rebuild + required image traits thing, at this point for simplicitly i'd probably just concede to the idea of failing in the api if the new image has required traits which aren't in the original image used to create the server | 15:06 |
| *** vladikr has quit IRC | 15:06 | |
| mriedem | and then hope that no one ever asks to support that later | 15:06 |
| mriedem | and if they do, you're no longer working on nova | 15:06 |
| *** alex_xu has joined #openstack-nova | 15:07 | |
| melwitt | based on my limited understanding of traits, I don't immediately see why the original image is tied to rebuilding with a new image and why it would need to reject it. so I shall read the thread to learn why :P | 15:07 |
| *** belmoreira has quit IRC | 15:09 | |
| cfriesen__ | sean-k-m1: sean-k-mooney[m]: do you know the current status of https://blueprints.launchpad.net/nova/+spec/sriov-physical-function-passthrough (where a device can be allocated as either a VF or the full PF) | 15:10 |
| *** germs has joined #openstack-nova | 15:10 | |
| *** germs has quit IRC | 15:10 | |
| *** germs has joined #openstack-nova | 15:10 | |
| *** moshele has quit IRC | 15:11 | |
| mriedem | melwitt: b/c the host the instance is running on during rebuild is filtered using the original image and it's required traits | 15:11 |
| melwitt | oh, right, because rebuild can't move | 15:12 |
| melwitt | I see | 15:13 |
| *** cfriesen__ is now known as cfriesen | 15:14 | |
| *** vladikr has joined #openstack-nova | 15:14 | |
| *** zhaochao has quit IRC | 15:16 | |
| *** vladikr_ has joined #openstack-nova | 15:20 | |
| *** yamamoto has quit IRC | 15:21 | |
| *** vladikr has quit IRC | 15:21 | |
| *** salv-orlando has joined #openstack-nova | 15:23 | |
| *** jaosorior has joined #openstack-nova | 15:23 | |
| *** dpawlik has joined #openstack-nova | 15:24 | |
| *** mingyu has joined #openstack-nova | 15:24 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 15:26 |
| *** shewless has joined #openstack-nova | 15:27 | |
| efried | stephenfin: You gonna go around and change all the other deprecations in the docs? There were about a dozen I counted yesterday. | 15:28 |
| *** dpawlik has quit IRC | 15:29 | |
| stephenfin | efried: I can but where? | 15:31 |
| efried | stephenfin: I went and looked for them yesterday. I can go find 'em again. | 15:31 |
| efried | stephenfin: But I guess if I'm gonna do that, I may as well fix 'em myself :) | 15:32 |
| shewless | Hi there. I have an instance which is allocated 2GB of RAM but when I look at the qemu process it looks like it's comsuming closer to 8GB of RAM. This is a huge discrepency. Can anyone help me figure out what's going on? | 15:33 |
| shewless | It kind of looks like we are seeing an extra 6GB of memory usage (over what's been allocated) PER instance? | 15:33 |
| stephenfin | efried: That...would make sense, yes :) | 15:35 |
| shewless | I thought that in general a qemu process would use less or equal to what's been provisioned for that instance. Is that wrong? | 15:36 |
| efried | stephenfin: This produced 29 results, which will need to be inspected manually to see if a .. deprecated:: marker is actually appropriate: | 15:38 |
| efried | # rgrep -i deprecated doc/source | awk -F: '$1 ~ /rst$/' | 15:38 |
| *** felipemonteiro__ has quit IRC | 15:39 | |
| *** tbachman has joined #openstack-nova | 15:41 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove IronicHostManager and baremetal scheduling options https://review.openstack.org/565805 | 15:42 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove [scheduler]/host_manager config option https://review.openstack.org/565806 | 15:42 |
| mriedem | jroll: ^ | 15:42 |
| jroll | woot | 15:42 |
| sahid | shewless: we know that there are some kind of overheads depending of the number of vCPUs, devices... but 6GB looks crazy, perhaps you should ask #virt in OFTC servers with the exact arguments you are passing to QEMU | 15:43 |
| *** salv-orlando has quit IRC | 15:43 | |
| *** salv-orlando has joined #openstack-nova | 15:43 | |
| shewless | sahid thanks I'll check there | 15:43 |
| *** mingyu has quit IRC | 15:43 | |
| *** salv-orlando has quit IRC | 15:44 | |
| openstackgerrit | sahid proposed openstack/nova master: network: update pci request spec to handle trusted tags https://review.openstack.org/458820 | 15:45 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: configure trust mode for vfs https://review.openstack.org/458514 | 15:45 |
| openstackgerrit | sahid proposed openstack/nova master: pci: don't consider case when match tags specs https://review.openstack.org/565808 | 15:45 |
| *** tbachman_ has joined #openstack-nova | 15:45 | |
| *** andreas_s has quit IRC | 15:45 | |
| *** tbachman has quit IRC | 15:46 | |
| *** tbachman_ is now known as tbachman | 15:46 | |
| *** andreas_s has joined #openstack-nova | 15:46 | |
| *** mingyu has joined #openstack-nova | 15:47 | |
| *** lucasagomes is now known as lucas-afk | 15:47 | |
| *** jmccarthy has quit IRC | 15:48 | |
| *** vladikr_ has quit IRC | 15:48 | |
| *** jaosorior has quit IRC | 15:49 | |
| efried | stephenfin: That deprecated role ain't especially pretty IMO. | 15:50 |
| *** andreas_s has quit IRC | 15:51 | |
| stephenfin | In what regard? | 15:51 |
| *** ragiman_ has quit IRC | 15:51 | |
| efried | let's see if your delta has rendered yet... | 15:51 |
| efried | stephenfin: http://logs.openstack.org/67/565367/9/check/build-openstack-sphinx-docs/c46fab1/html/cli/nova-consoleauth.html | 15:53 |
| efried | stephenfin: I guess I was expecting a box, like .. note:: | 15:53 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: update add-consumer-generation to focus on API https://review.openstack.org/565565 | 15:53 |
| sahid | cfriesen__ it's implemented | 15:53 |
| efried | stephenfin: It's aight, I suppose. | 15:53 |
| stephenfin | efried: Yeah, that's openstackdocstheme. Wonder what the reasoning for that was | 15:56 |
| efried | stephenfin: I'm sure it's related to it never being used, but I wonder if it's cause or effect :) | 15:57 |
| *** tssurya has quit IRC | 15:57 | |
| *** hamzy has quit IRC | 15:58 | |
| stephenfin | efried: Could be. I thought oslo.config used it but it actually uses '.. warning' too. I should fix that https://docs.openstack.org/nova/queens/configuration/config.html | 15:59 |
| openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 15:59 |
| *** gyee has joined #openstack-nova | 15:59 | |
| *** sahid has quit IRC | 15:59 | |
| *** damien_r has quit IRC | 16:02 | |
| *** janki has quit IRC | 16:02 | |
| *** dpawlik has joined #openstack-nova | 16:04 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822 | 16:05 |
| efried | stephenfin: Here it is ^ but I'll probably abandon it, unless we want to fix the theme to make it prettier. | 16:05 |
| *** efried is now known as efried_rollin | 16:05 | |
| *** dpawlik has quit IRC | 16:09 | |
| jroll | jaypipes: do you know if we need to report these as zero, or can we just drop them all? https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L764 | 16:10 |
| *** vladikr has joined #openstack-nova | 16:10 | |
| evrardjp | mriedem: thanks for the patch in OSA :) | 16:13 |
| *** do3meli has quit IRC | 16:13 | |
| jroll | jaypipes: ah, I think unit tests answered that for me, never mind. :) | 16:14 |
| mriedem | evrardjp: np | 16:16 |
| *** tesseract has quit IRC | 16:18 | |
| arvindn05 | mriedem: replied back to the thread on rebuild with image traits. Will look for further update from melwitt or dansmith | 16:20 |
| *** felipemonteiro has joined #openstack-nova | 16:20 | |
| mriedem | evrardjp: s/OSA/literally everything/ +(status:open+OR+status:merged) | 16:20 |
| mriedem | https://review.openstack.org/#/q/topic:exact-filters+(status:open+OR+status:merged) | 16:20 |
| *** yamamoto has joined #openstack-nova | 16:21 | |
| evrardjp | mriedem: I haved guessed that ;) | 16:22 |
| evrardjp | mriedem: but still thank you. | 16:22 |
| evrardjp | :D | 16:22 |
| arvindn05 | mriedem: i think we are on the same page in case we choose to reject the request at the API layer. the spec had the same intent | 16:22 |
| EmilienM | mriedem: thanks for https://review.openstack.org/#/c/565824/ - I took over, just see my comment for the FYI | 16:23 |
| arvindn05 | > If the image's required traits have changed from the original image, we can reject the rebuild request. Effectively, since the qualitative requirements have changed the user should create a new instance. | 16:24 |
| arvindn05 | https://review.openstack.org/#/c/560718/6/specs/rocky/approved/glance-image-traits.rst - line 100. I will make it more clear like i did in the ML | 16:24 |
| mriedem | EmilienM: ack thanks | 16:25 |
| *** salv-orlando has joined #openstack-nova | 16:26 | |
| mriedem | EmilienM: i suspect the depends-on is backward anyway for any of these deployment projects that gate on ironic, which tripleo does? | 16:26 |
| mriedem | but i didn't want the nova change depending on 20 random other deployment projects | 16:26 |
| EmilienM | mriedem: yeah it's fine | 16:27 |
| *** yamamoto has quit IRC | 16:27 | |
| *** salv-orlando has quit IRC | 16:29 | |
| *** salv-orlando has joined #openstack-nova | 16:30 | |
| *** Guest25377 has quit IRC | 16:33 | |
| *** salv-orlando has quit IRC | 16:34 | |
| openstackgerrit | Arvind Nadendla proposed openstack/nova master: update scheduler to use image-traits https://review.openstack.org/560596 | 16:38 |
| *** bhujay has quit IRC | 16:41 | |
| *** tbachman has quit IRC | 16:41 | |
| *** tbachman has joined #openstack-nova | 16:46 | |
| openstackgerrit | Jim Rollenhagen proposed openstack/nova master: Ironic: report 0 for vcpus/memory_mb/disk_gb resources https://review.openstack.org/565841 | 16:46 |
| jroll | mriedem: one more in your ironic chain for you ^ | 16:46 |
| mriedem | question inline | 16:51 |
| jroll | ah thanks | 16:53 |
| mriedem | dansmith: so for this CLI to fix allocations in placement for instances scheduled using the CachingScheduler, i'm trying to think where that should live, since none of the nova-manage subcommands really fit that; we could do something like, | 16:56 |
| mriedem | a) nova-manage placement ... | 16:56 |
| mriedem | b) placement-manage ... | 16:56 |
| mriedem | c) throw it in osc-placement | 16:56 |
| mriedem | although (c) isn't really meant for something like this | 16:57 |
| *** tssurya has joined #openstack-nova | 16:57 | |
| dansmith | yeah, it's a pretty oddball thing | 16:57 |
| mriedem | we're fixing allocations for nova instances, | 16:57 |
| dansmith | it's also a placement client sort of thing, so it's weird to have it in nova-manage, | 16:57 |
| mriedem | so thinking: nova-manage placement fix_allocations | 16:57 |
| dansmith | but placement-manage would be the opposite | 16:57 |
| mriedem | right, we need the instance uuid to find it's host/node to get the node uuid to find the resource provider it should have allocations against | 16:58 |
| dansmith | mriedem: maybe we need a generic nova-manage group, like "nova-manage fix-our-mess" or "nova-manage maintenance ..." | 16:58 |
| mriedem | so doing instance lookups from a placement-manage is weird | 16:58 |
| *** mingyu has quit IRC | 16:58 | |
| dansmith | could use that generic group for syncing aggregate membership, fixing cinder attachments (historically) | 16:58 |
| mriedem | jaypipes: where were you going to throw the nova-manage sync CLI for aggregates? | 16:59 |
| dansmith | mriedem: wherever you tell him to | 16:59 |
| dansmith | :P | 16:59 |
| mriedem | was just wondering if his gears were grinding on that yet | 17:00 |
| *** derekh has quit IRC | 17:00 | |
| *** weshay|ruck is now known as weshay | 17:00 | |
| *** mgoddard has quit IRC | 17:00 | |
| *** mingyu has joined #openstack-nova | 17:02 | |
| *** tbachman has quit IRC | 17:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove [scheduler]/host_manager config option https://review.openstack.org/565806 | 17:03 |
| mriedem | i'm partial to 'nova-manage placement ...' | 17:03 |
| mriedem | but don't care too much | 17:03 |
| mriedem | gonna get lunch | 17:04 |
| dansmith | melwitt: mriedem: tssurya: I've been completely covered up in other stuff for the last week.. I have nothing cellsy to talk about, unless the cern peeps have things to report? | 17:04 |
| dansmith | mriedem: well, I just don't know that we'll have enough in there to justify the group, but whatever you want | 17:04 |
| *** openstackgerrit has quit IRC | 17:04 | |
| mriedem | dansmith: we'll have at least 2 things, this and jay's sync aggs thing | 17:05 |
| mriedem | i don't have any cells stuff to talk about | 17:05 |
| *** moshele has joined #openstack-nova | 17:06 | |
| tssurya | dansmith: not really except that we moved to cellsv2, and basic operations take more time than for cellsv1 , we are trying to optimize whatever possible, but since belmiro won't be around today as its a Euro unfriendly meeting timing, we can give the overall experience in the next week's meeting | 17:07 |
| *** hamzy has joined #openstack-nova | 17:07 | |
| dansmith | tssurya: okay, yeah would be good to hear an analysis of what takes longer so we can think about what to do | 17:07 |
| *** yamahata has joined #openstack-nova | 17:08 | |
| tssurya | dansmith: yep | 17:08 |
| *** dpawlik has joined #openstack-nova | 17:08 | |
| *** Swami has joined #openstack-nova | 17:08 | |
| dansmith | cellsv1 cheated with cache coherency, so not surprising that some things will take longer to be correct ;) | 17:09 |
| tssurya | dansmith: haha, yea I get it but querying to all 70 DBs versus top cell having all info is also the toll maybe | 17:09 |
| dansmith | tssurya: you guys are on pike yeah? | 17:09 |
| tssurya | dansmith: skipped pike, moved straight to queens :) | 17:10 |
| tssurya | from ocata | 17:10 |
| *** felipemonteiro_ has joined #openstack-nova | 17:10 | |
| dansmith | oh | 17:10 |
| dansmith | yeah, so you should have the parallel instance listing, | 17:11 |
| dansmith | although we do iterate the cells in other places where we could optimize | 17:11 |
| dansmith | tssurya: https://review.openstack.org/#/c/509003/ | 17:11 |
| dansmith | that would help you more than most | 17:11 |
| jaypipes | mriedem: nova-manage placement ... works for me. | 17:11 |
| tssurya | dansmith: exactly, we are trying to optimize such things, however we also had some DB side issues, some being slower than the others, | 17:11 |
| tssurya | dansmith: oh I have never come across that patch | 17:12 |
| tssurya | thanks | 17:12 |
| *** dpawlik has quit IRC | 17:12 | |
| *** gjayavelu has quit IRC | 17:12 | |
| dansmith | tssurya: that would make us only query the cells a particular tenant has instances in, which I think in your case would be a big win.. like going from 70 to 2 or something | 17:13 |
| *** germs has quit IRC | 17:14 | |
| *** sdeath has quit IRC | 17:14 | |
| tssurya | dansmith: wow, too bad we did not know about this patch being there all along | 17:14 |
| dansmith | tssurya: well, my fault for letting it languish | 17:15 |
| *** felipemonteiro has quit IRC | 17:15 | |
| tssurya | I didn't even know CellMappingList.get_by_project_id existed | 17:15 |
| dansmith | tssurya: it doesn't, look below :) | 17:15 |
| tssurya | what we did was to go through InstanceMappings | 17:15 |
| dansmith | tssurya: could be an easy backport | 17:15 |
| tssurya | and nova list became fast, | 17:15 |
| dansmith | tssurya: yeah that's what it does | 17:15 |
| tssurya | dansmith: yes this is exactly what we did, we patched it the same way | 17:17 |
| dansmith | cool | 17:17 |
| tssurya | except if I had known you had this we would have backported :D | 17:17 |
| dansmith | heh | 17:17 |
| dansmith | I will get this rebased today | 17:17 |
| *** germs has joined #openstack-nova | 17:17 | |
| *** germs has quit IRC | 17:17 | |
| *** germs has joined #openstack-nova | 17:17 | |
| *** dtantsur is now known as dtantsur|afk | 17:17 | |
| tssurya | dansmith: thanks ! would be really nice to have this upstream | 17:17 |
| *** gjayavelu has joined #openstack-nova | 17:18 | |
| *** germs has quit IRC | 17:18 | |
| *** germs has joined #openstack-nova | 17:19 | |
| *** germs has quit IRC | 17:19 | |
| *** germs has joined #openstack-nova | 17:19 | |
| melwitt | tssurya: so even with the all cell query executed in parallel, you saw it being slow to list instances? | 17:20 |
| dansmith | melwitt: they have some very slow databases, which will hold up the query unnecessarily | 17:20 |
| *** gjayavelu has quit IRC | 17:20 | |
| melwitt | I see | 17:20 |
| tssurya | melwitt: once we had the list using InstanceMapping it was fast, but yes even with parallel querying under the current master code it is slow than cellsv1 | 17:21 |
| tssurya | due to DB issues like dan said | 17:21 |
| melwitt | gotcha | 17:21 |
| dansmith | it makes it as slow as the slowest db, of course | 17:22 |
| dansmith | which isn't really avoidable in general, but since they have lots of cells, but few per tenant, they will get more gain from this than people that don't do that | 17:22 |
| dansmith | tssurya: some of your cell dbs are far away from the controller too right? | 17:23 |
| tssurya | dansmith: eys | 17:23 |
| tssurya | yes* | 17:23 |
| *** tbachman has joined #openstack-nova | 17:23 | |
| dansmith | yeah, so that's related | 17:23 |
| dansmith | punishes everyone | 17:23 |
| dansmith | tssurya: since the beginning we've said we will probably have to do some caching at the top for situations like that, | 17:23 |
| dansmith | but we also always said we'd wait until we had something to measure | 17:23 |
| *** yamamoto has joined #openstack-nova | 17:23 | |
| dansmith | so that bill may be coming due soon | 17:23 |
| tssurya | dansmith: yea we are measuring stuff, belmiro will give a upgrade talk at the summit | 17:24 |
| dansmith | cool | 17:24 |
| *** gjayavelu has joined #openstack-nova | 17:25 | |
| melwitt | to the earlier question about the cells meeting, I don't have anything other than a request for review on this patch to make affinity work with multiple cells https://review.openstack.org/540258 | 17:25 |
| tssurya | for instance another point I wanted to know was this : https://bugs.launchpad.net/nova/+bug/1767303, we filed a new bug to only go through cells that are needed now that we have disabled the host_state cache. I will put a POC once we test the performance in our envo. | 17:26 |
| openstack | Launchpad bug 1767303 in OpenStack Compute (nova) "Scheduler connects to all cells DBs to gather compute nodes info" [Undecided,New] - Assigned to Surya Seetharaman (tssurya) | 17:26 |
| tssurya | but yea, we can also talk in the next week's meeting after we see if scheduling time improves | 17:27 |
| mgagne | tssurya: is there a blog post or something I can read about your migration to CellsV2? Or should I wait until summit to hear it first-hand at one of the talk? | 17:27 |
| tssurya | mgagne: we will try to put up one soon :) we are still putting out fires | 17:28 |
| mgagne | tssurya: all fine, I'm currently sitting in a fire too =) | 17:28 |
| dansmith | tssurya: cool, makes sense | 17:28 |
| tssurya | mgagne: your also on cellsv2 ? | 17:28 |
| dansmith | mgagne: how many cells do you have? | 17:28 |
| tssurya | you are* | 17:29 |
| mgagne | tssurya: running CellsV1 Mitaka | 17:29 |
| tssurya | mgagne: ack | 17:29 |
| mgagne | dansmith: more than 1 =) | 17:29 |
| *** gjayavelu has quit IRC | 17:29 | |
| *** yamamoto has quit IRC | 17:29 | |
| mgagne | dansmith: I'm not (yet) concerned with performance, only the migration process with multiple cells | 17:29 |
| *** mgoddard has joined #openstack-nova | 17:36 | |
| *** mdnadeem has quit IRC | 17:38 | |
| mriedem | tssurya: do you know if cern was/is using the CachingScheduler before upgrading to ocata+? | 17:38 |
| tssurya | mriedem: no we weren't and aren't | 17:39 |
| mriedem | ok | 17:39 |
| *** shewless has quit IRC | 17:39 | |
| tssurya | dansmith: not urgent, but whenever you get the time could you re-instate the +2 on https://review.openstack.org/#/c/559299/ , its the clean up on the bp which has been held up for some time. | 17:41 |
| *** sambetts is now known as sambetts|afk | 17:41 | |
| *** mdbooth has quit IRC | 17:41 | |
| dansmith | tssurya: done, sorry | 17:42 |
| tssurya | dansmith: thank you! :) | 17:42 |
| *** dpawlik has joined #openstack-nova | 17:46 | |
| *** hjensas has joined #openstack-nova | 17:47 | |
| *** hjensas has quit IRC | 17:47 | |
| *** hjensas has joined #openstack-nova | 17:47 | |
| *** dpawlik has quit IRC | 17:50 | |
| *** moshele has quit IRC | 17:51 | |
| *** moshele has joined #openstack-nova | 18:08 | |
| *** mgariepy has quit IRC | 18:09 | |
| mriedem | jroll: is oath using the CachingScheduler for baremetal scheduling? | 18:10 |
| *** moshele has quit IRC | 18:13 | |
| *** mgariepy has joined #openstack-nova | 18:15 | |
| jroll | mriedem: no, that would be insane, we wrote our own scheduler driver of course :| | 18:16 |
| jroll | mriedem: (it's subclassed from filter scheduler and we're likely dropping it in ocata, so carry on with the config drops and such) | 18:16 |
| mriedem | jroll: was more about the ML thread on this wrt caching scheduler | 18:17 |
| mriedem | and mgagne replies about performance of baremetal scheduling with the caching scheduler | 18:18 |
| jroll | mriedem: we'll find out soon about performance on the filter scheduler in ocata. in juno we did this crazy claims thing I proposed upstream and then dropped a while back | 18:18 |
| jroll | https://review.openstack.org/#/c/204641/ | 18:19 |
| mgagne | I guess I'm alone going at ludicrous speed with CachingScheduler =) | 18:20 |
| jroll | heh | 18:20 |
| jroll | with racing fixed with the resource classes stuff, I'm not sure the scheduler performance matters so much | 18:20 |
| mgagne | it was impossible to boot like 100 baremetal at the same time with FilterScheduler, now it's possible with CachingScheduler. | 18:21 |
| jroll | as baremetal machines take minimum 5 minutes to deploy and boot anyway | 18:21 |
| jroll | right | 18:21 |
| jroll | mgagne: what version are you on? | 18:21 |
| mriedem | mitaka | 18:21 |
| mgagne | jroll: the best and greatest Mitaka ;) | 18:21 |
| mriedem | mgagne: do you see a fair number of reschedules? | 18:22 |
| jroll | ah yeah. I suspect in master, a bunch of concurrent boots are just fine | 18:22 |
| mgagne | afaik, no. we run only one scheduler as cache is done in-memory and not shared | 18:22 |
| * jroll assumes you're running into collisions, not slow selection | 18:23 | |
| mgagne | we used to see a lot of rescheduling with FilterScheduler IIRC | 18:23 |
| mriedem | well, CachingScheduler is FilterScheduler | 18:23 |
| mriedem | except with the cache | 18:23 |
| mgagne | jroll: remember that OOM issue you had with scheduler with ironic? ;) | 18:23 |
| jroll | heh | 18:23 |
| jroll | that was loading instances and such | 18:24 |
| *** ccamacho has quit IRC | 18:24 | |
| mgagne | scheduler taking a gazillion bytes of ram | 18:24 |
| mgagne | jroll: yea, great stuff | 18:24 |
| jroll | idk, I'd like to see some benchmarking on master before we make decisions about CachingScheduler being useful on master | 18:24 |
| mriedem | i've asked for benchmarking between placement + filterscheduler and cachingscheduler since at least ocata/pike at every summit | 18:25 |
| mgagne | jroll: being useful is not really the issue, it's more that a migration path is not ready yet for those using CachingScheduler | 18:25 |
| mriedem | mgagne: i'm working on that CLI right now | 18:25 |
| *** yamamoto has joined #openstack-nova | 18:25 | |
| jroll | right, if we backport that CLI we should be all good | 18:25 |
| mriedem | and i did remember when replying to the ML that newton and ocata computes will write allocations to placement, | 18:25 |
| mriedem | it was only in pike when we disabled that because the filter scheduler creates the allocations now | 18:26 |
| mgagne | mriedem: tyvm for working on it. IMO, if it's not merged, it's not done. hence my concern with removing stuff that could break my use case. | 18:26 |
| mriedem | mgagne: sure, but also, if you're on mitaka, you have plenty of time for this to land before you're ready to roll to it | 18:26 |
| mriedem | i.e. the cachingscheduler and ironic_host_manager removal | 18:27 |
| mgagne | mriedem: I'm more on a defensive stance with that regard, can never be too careful | 18:27 |
| *** eharney has quit IRC | 18:27 | |
| *** mingyu has quit IRC | 18:27 | |
| mriedem | i understand :) | 18:28 |
| mgagne | on principal, I can't agree with someone suggesting a new feature/fix not yet merged to replace something that is going to be removed. | 18:28 |
| *** fragatina has quit IRC | 18:30 | |
| *** READ10 has quit IRC | 18:30 | |
| mgagne | or principle? | 18:30 |
| *** yamamoto has quit IRC | 18:31 | |
| artom | The latter | 18:31 |
| *** openstackgerrit has joined #openstack-nova | 18:35 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add CellMapping.get_by_project_id() query method https://review.openstack.org/509002 | 18:35 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make get_instance_objects_sorted() be smart about cells https://review.openstack.org/509003 | 18:35 |
| dansmith | tssurya: ^ | 18:35 |
| tssurya | dansmith: ack, and thanks | 18:35 |
| dansmith | I think we may want to consider a flag to let you choose between "always scatter to all cells" and "only hit cells for which you have instances" | 18:35 |
| dansmith | for your case, the latter will be a big improvement, but for people that spread across cells, doing the extra mapping lookup will be a waste | 18:36 |
| tssurya | dansmith: sounds good, yea makes sense | 18:36 |
| mgagne | artom: :) | 18:37 |
| tssurya | but are you talking about doing this in the listing context alone ? or a generic flag for the scatter-gather utility | 18:37 |
| *** gjayavelu has joined #openstack-nova | 18:37 | |
| dansmith | scatter-gather can't do it generically, | 18:37 |
| dansmith | because it only applies to instances | 18:37 |
| dansmith | so it would just be instance listing that would honor that | 18:38 |
| jroll | how ready for primetime is cells v2 in ocata? | 18:38 |
| tssurya | yea I mean scatter-gather already has the selected cells querying option, what I mean was this "only hit cells for which you have instances" sounds really appealing in a lot of places | 18:38 |
| tssurya | like quota calculating while booting | 18:39 |
| tssurya | calculation* | 18:39 |
| tssurya | meant* | 18:39 |
| tssurya | so was wondering if there was a way to flag this more generically | 18:39 |
| dansmith | tssurya: sure, it makes sense to do it in those sorts of places, I just think it has to be baked into each one that is tenant-scoped | 18:39 |
| tssurya | dansmith: hmm, yea | 18:39 |
| dansmith | tssurya: anywhere we can do tenant-scoped lookups should look at that flag, if that's what you mean | 18:40 |
| dansmith | jroll: not very | 18:40 |
| jroll | whomp whomp | 18:40 |
| tssurya | dansmith: yes, | 18:40 |
| melwitt | jroll: multi cell is not a thing in ocata, I'd say | 18:41 |
| jroll | melwitt: yeah, that was my actual question, thanks | 18:42 |
| melwitt | and, affinity scheduling doesn't work with multi cell yet, there's a bug (I have a patch up to fix it) https://review.openstack.org/540258 | 18:43 |
| melwitt | I know that's something y'all need | 18:43 |
| jroll | tis, thanks for that | 18:45 |
| *** sridharg has quit IRC | 18:46 | |
| *** mgoddard has quit IRC | 18:49 | |
| *** imacdonn has quit IRC | 18:54 | |
| *** imacdonn has joined #openstack-nova | 18:54 | |
| *** felipemonteiro__ has joined #openstack-nova | 18:59 | |
| *** felipemonteiro_ has quit IRC | 18:59 | |
| melwitt | hm, seeing an issue where a BFV instance being deleted fails to detach the volume because attachment_id = None http://logs.openstack.org/01/565601/2/check/tempest-full/f1004d9/controller/logs/screen-n-cpu.txt.gz#_May_02_03_50_00_958050 | 19:02 |
| cfriesen | anyone know offhand what's causing this failure when running tox against newton? http://paste.openstack.org/show/720243/ | 19:03 |
| melwitt | did you do a tox -e<whatevs> -r first? | 19:04 |
| melwitt | usually have to do that when you switch branches | 19:04 |
| cfriesen | will try that, thanks | 19:05 |
| melwitt | usually you'll have different deps versions required per branch and that's where those errors come from. if the env has a module version missing an attribute that another branch needs | 19:06 |
| *** spsurya has quit IRC | 19:09 | |
| melwitt | so for the BFV volume detach, it's using the old attach API, that means bdm.attachment_id was not set or None | 19:11 |
| melwitt | and then the old os-detach API is called with attachment_id = null and that yields a 400 from cinder | 19:11 |
| *** efried_rollin is now known as efried | 19:13 | |
| *** fragatina has joined #openstack-nova | 19:13 | |
| melwitt | now to figure out why attachment_id isn't set for a BFV instance created from a snapshot | 19:13 |
| *** fragatina has quit IRC | 19:14 | |
| *** fragatina has joined #openstack-nova | 19:15 | |
| *** fragatina has quit IRC | 19:16 | |
| *** fragatina has joined #openstack-nova | 19:18 | |
| *** fragatina has quit IRC | 19:18 | |
| *** fragatina has joined #openstack-nova | 19:19 | |
| *** fragatina has quit IRC | 19:19 | |
| *** fragatin_ has joined #openstack-nova | 19:21 | |
| mriedem | melwitt: i know exactly why | 19:22 |
| arvindn05 | efried: thanks for the review :). For the comment on the commit message, i just squash my individual commits to create the single commit that gerrit requires..hence the Added unit and functional tests | 19:22 |
| mriedem | melwitt: https://review.openstack.org/#/c/541420/ | 19:22 |
| melwitt | yesss, I was hoping there was already a patch for dat | 19:22 |
| mriedem | melwitt: either way, when detaching we should know if we're doing new style detach (attachment_id is not None) or old style detach | 19:22 |
| efried | arvindn05: You mean in your sandbox you had multiple commits, one that did the code and one that did the test, and you squashed them together before proposing them to gerrit? | 19:23 |
| arvindn05 | yup | 19:23 |
| efried | arvindn05: Yeah, that doesn't change my opinion :) There's no need for a commit message on a change that clearly has unit and functional tests to say that it has unit and functional tests. | 19:23 |
| melwitt | mriedem: right. we're doing old style because attachment_id is None but, attachment_id sent over to cinder in the old style os-detach action call is also None which gets a 400 response | 19:23 |
| efried | arvindn05: But you notice I'm not blocking on it either. | 19:23 |
| melwitt | so old style detach fails | 19:24 |
| mriedem | we don't send an attachment_id to os-detach in cinder | 19:24 |
| mriedem | it's not a parameter for that api | 19:24 |
| mriedem | well, i don't think it is | 19:24 |
| melwitt | it's in the log here http://logs.openstack.org/01/565601/2/check/tempest-full/f1004d9/controller/logs/screen-n-cpu.txt.gz#_May_02_03_50_00_958050 | 19:24 |
| mriedem | for new style detach, we'd call attachment_delete | 19:24 |
| mriedem | log schmog, that's fake news | 19:25 |
| melwitt | it's the only thing we're sending | 19:25 |
| arvindn05 | efried: cool....i dont like the gerrit workflow...more used to github which allows individual commits and lets the maintainer decide on squash vs rebase vs commit as is | 19:25 |
| arvindn05 | but going with the flow :) | 19:25 |
| melwitt | WHAT SAY YOU | 19:25 |
| efried | arvindn05: You're welcome (even encouraged) to split up reviews in gerrit, though generally if along test/impl lines the test should come first. | 19:26 |
| efried | arvindn05: Me, I've never used github, so no basis for comparison there. | 19:26 |
| mriedem | oh yeah i guess os-detach does take an attachment_id parameter, which is optional | 19:26 |
| melwitt | it's weird that of all the params, that's the only thing we're sending. seems wrong | 19:27 |
| mriedem | melwitt: we can't detach the volume on a volume-backed instance, is that what you're trying to do? | 19:27 |
| mriedem | *root volume | 19:27 |
| *** yamamoto has joined #openstack-nova | 19:27 | |
| melwitt | mriedem: maybe. this is a delete_on_termination=True instance created from a snapshot (the tempest test) and when we delete the instance, it tries to detach the volume and delete it. and it fails to detach it | 19:28 |
| mriedem | this schema check looks wrong | 19:28 |
| mriedem | InvalidInput: Invalid input received: Invalid input for field/attribute attachment_id. Value: None. None is not of type 'string' | 19:28 |
| melwitt | a BFV instance, that is | 19:28 |
| mriedem | this schema doesn't say attachment_id is required https://github.com/openstack/cinder/blob/master/cinder/api/schemas/volume_actions.py#L73 | 19:29 |
| mriedem | and it allows null | 19:29 |
| mriedem | oh no sorry | 19:29 |
| mriedem | that says you don't have to put a body in it | 19:29 |
| melwitt | so what's the correct flow for deleting a BFV instance with delete_on_termination=True? has to detach and then delete the volume, the volume can't be deleted if it's in-use | 19:30 |
| arvindn05 | efried: in gerrit, subsequent changes are always amends for a single review which means on the git side, you do lose history. If you wanted to say revert changes back to commit #2, we would need to download the code from gerrit for PS#2...anyway just my rant from using both github and gerrit :) | 19:31 |
| efried | arvindn05: You can do commits in series, which means you use a different change-id for the second commit but set the first commit as its parent. | 19:32 |
| mriedem | melwitt: the attachment_id getting passed to cinder isn't from the bdm, it's from the volume['attachments'] information, which is a thing that existed before the new style attach flow https://github.com/openstack/nova/blob/master/nova/compute/api.py#L4026 | 19:32 |
| *** rmart04 has joined #openstack-nova | 19:33 | |
| *** yamamoto has quit IRC | 19:33 | |
| efried | arvindn05: example: https://review.openstack.org/#/c/565279/2 is first, https://review.openstack.org/#/c/564351/5 is second, https://review.openstack.org/#/c/517757/33 is third in the series. | 19:33 |
| mriedem | this code is likely all old and busted and should be removed https://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L537-L559 | 19:34 |
| mriedem | since we don't support multiattach volumes with old style attachments | 19:34 |
| openstackgerrit | Merged openstack/nova master: Remove ExactCoreFilter ExactDiskFilter ExactRamFilter https://review.openstack.org/508886 | 19:34 |
| mriedem | that was all prep work written in mitaka https://review.openstack.org/#/c/193134/ | 19:35 |
| melwitt | arvindn05: yeah, with gerrit, you can't have multiple commits in one "review". so to do that, you have to make a stack of gerrit reviews (one review per commit) and then upload them together. you just do all of your commits as usual, then 'git review' will upload all of them as separate reviews that depend on each other | 19:35 |
| *** linkmark has joined #openstack-nova | 19:35 | |
| mriedem | melwitt: so it looks like we have a bug that we still pass an attachment_id to os-detach even if we don't have an attachment_id | 19:36 |
| -openstackstatus- NOTICE: The Gerrit service at review.openstack.org will be offline starting at 20:00 (in roughly 25 minutes) for a server move and operating system upgrade: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130118.html | 19:36 | |
| *** damien_r has joined #openstack-nova | 19:36 | |
| melwitt | mriedem: okay, so an empty body os-detach will still do the right thing and detach the volume | 19:36 |
| mriedem | maybe, not really sure, we don't have tests that try to detach a root volume | 19:37 |
| melwitt | I'll try depends-on your patch and see if it makes the tempest test change work (I added an assert that the volume was gone and that failed because the volume couldn't be detached and thus deleted) | 19:37 |
| mriedem | melwitt: ok, but you need this to get ceph working on stable branches? | 19:38 |
| melwitt | mriedem: on master. I don't know why it's failing 100% now over this "volume dependent on snapshot" issue when it didn't use to, but that's what's going on | 19:39 |
| mriedem | ok deps-on my patch and then we'll go from there | 19:39 |
| arvindn05 | melwitt: efried: ahh...k. will keep that in mind. Good workflow to use when all of your commits are complete...if the last one is WIP, we will have to mark it in gerrit review i guess | 19:39 |
| melwitt | but because the test does not first remove the volume snapshot, the second instance can't be deleted. I Don't Know How This Ever Worked(tm) for ceph | 19:40 |
| arvindn05 | learn something new about gerrit everyday :) | 19:40 |
| efried | arvindn05: Happens all the time. Even ones that *aren't* the last one get marked WIPs. | 19:40 |
| efried | arvindn05: And the whole series doesn't merge at once; they merge bottom up like you would expect. | 19:40 |
| efried | each is a dependency of the next. | 19:40 |
| melwitt | arvindn05: yeah, the usual way is to just prepend your commit message subject line on the WIP one with "WIP" | 19:40 |
| mriedem | melwitt: the test wasn't skipped for the ceph job before was it? | 19:41 |
| arvindn05 | thanks! | 19:41 |
| melwitt | mriedem: it might have been. I can't think of any other way this makes sense but I didn't find that yet. tbh I don't remember where to find the skip list for ceph | 19:42 |
| *** eharney has joined #openstack-nova | 19:42 | |
| mriedem | i don't think it ever landed, i remember cleaning up something about this recently | 19:43 |
| mriedem | https://review.openstack.org/#/c/550497/ | 19:43 |
| *** mvk has joined #openstack-nova | 19:45 | |
| melwitt | so both skip list changes are abandoned ... does that mean there's no skip list? I'm confused | 19:46 |
| *** linkmark has quit IRC | 19:46 | |
| melwitt | so that implies that test is new? *looks* | 19:47 |
| mriedem | it's not new | 19:47 |
| mriedem | it's only recently been renamed | 19:47 |
| mriedem | but the idempotent id hasn't changed | 19:47 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886 | 19:47 |
| melwitt | hrm | 19:47 |
| mriedem | mgagne: dansmith: ^ here is my start, sans tests and docs and lots of todos/notes | 19:47 |
| mriedem | melwitt: this is the job definition, and no it doesn't have a skip list http://git.openstack.org/cgit/openstack-infra/openstack-zuul-jobs/tree/playbooks/legacy/tempest-dsvm-py35-full-devstack-plugin-ceph/run.yaml#n2 | 19:48 |
| mriedem | jbernard was working on one but as you can see those patches were all abandoned | 19:48 |
| melwitt | doubleyew tee eff | 19:48 |
| mriedem | so for awhile the job was just totally broken | 19:48 |
| melwitt | I don't know how this test could ever pass with ceph | 19:48 |
| melwitt | and the job hasn't been broken for _that_ long, I didn't think | 19:49 |
| melwitt | anywho | 19:49 |
| mriedem | melwitt: could be that it wasn't using DEVSTACK_GATE_TEMPEST_FULL | 19:50 |
| *** lpetrut has joined #openstack-nova | 19:50 | |
| mriedem | tempest tox -e full runs tempest api tests concurrently and then scenario tests serially | 19:51 |
| mriedem | and this is a scenario tes | 19:51 |
| mriedem | *test | 19:51 |
| efried | arvindn05: Do we always have access to the old image metadata during a rebuild? | 19:51 |
| melwitt | okay, I'll take a look for that | 19:51 |
| *** felipemonteiro_ has joined #openstack-nova | 19:51 | |
| mriedem | if DEVSTACK_GATE_TEMPEST_* isn't specified, it looks like it should just be running smoke tests | 19:51 |
| *** slaweq has quit IRC | 19:52 | |
| arvindn05 | nope...we have the old image ref though....we can load the image meta from that | 19:52 |
| *** mingyu has joined #openstack-nova | 19:52 | |
| *** slaweq has joined #openstack-nova | 19:52 | |
| *** slaweq has quit IRC | 19:52 | |
| mriedem | efried: arvindn05: you should, it's in the request spec | 19:52 |
| efried | okay | 19:52 |
| *** slaweq has joined #openstack-nova | 19:52 | |
| mriedem | https://github.com/openstack/nova/blob/master/nova/objects/request_spec.py#L49 | 19:53 |
| mriedem | melwitt: maybe this https://github.com/openstack/devstack-plugin-ceph/commit/430fff8313a364f951423768d9ee8fdba4e79a27 | 19:53 |
| mriedem | nvm that's just manila | 19:53 |
| arvindn05 | for rebuild at the api level a request spec is not created yet i think | 19:54 |
| arvindn05 | compute.api.API#rebuild | 19:54 |
| arvindn05 | i am looking at that method to add the validation...is it the right place? | 19:54 |
| mriedem | the request spec is from when the instance was created | 19:54 |
| *** felipemonteiro__ has quit IRC | 19:55 | |
| mriedem | also, instance.image_meta is the current instance image meta for the instance | 19:55 |
| mriedem | so you don't even need the request spec | 19:56 |
| arvindn05 | great | 19:56 |
| arvindn05 | i would just need to load the new image meta then to compare each other | 19:57 |
| *** lpetrut has quit IRC | 19:57 | |
| *** rmart04 has quit IRC | 19:57 | |
| mriedem | you have the new image here https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3013 | 19:58 |
| *** slaweq has quit IRC | 19:59 | |
| arvindn05 | btw is there an instance.image_meta? i dont see it under objects.instance.Instance | 20:00 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/objects/instance.py#L238 | 20:02 |
| arvindn05 | ahh..its not a field...but a property got it | 20:03 |
| -openstackstatus- NOTICE: The Gerrit service at review.openstack.org will be offline over the next 1-2 hours for a server move and operating system upgrade: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130118.html | 20:05 | |
| *** ChanServ changes topic to "The Gerrit service at review.openstack.org will be offline over the next 1-2 hours for a server move and operating system upgrade: http://lists.openstack.org/pipermail/openstack-dev/2018-May/130118.html" | 20:05 | |
| *** mingyu has quit IRC | 20:06 | |
| melwitt | mriedem: indeed, here's a job pass on stable/queens from today http://logs.openstack.org/68/565668/1/check/legacy-tempest-dsvm-full-devstack-plugin-ceph/b6047e4/job-output.txt.gz#_2018-05-02_09_46_32_680582 | 20:07 |
| mriedem | hmm, tempest and devstack-plugin-ceph are branchless, | 20:08 |
| mriedem | so whatever is broken must be in master only for nova or cinder | 20:08 |
| melwitt | yeah, exactly. odd. going to see if I can find what's different | 20:08 |
| *** felipemonteiro_ has quit IRC | 20:09 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:10 | |
| mriedem | melwitt: my money is on that schema change in the cinder api which is queens only, merged in march | 20:10 |
| melwitt | you mean rocky only? | 20:10 |
| mriedem | before that, the api would just pass attachment_id=None down to the volume manager which handled it https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L1296 | 20:10 |
| mriedem | yeah, i'll never adapt to rocky | 20:10 |
| melwitt | QueensMan | 20:10 |
| melwitt | okay, I think I missed where you pointed out the schema change. *scrolls back* | 20:11 |
| melwitt | ah yep, so there used to be no schema validation I guess? agreed that has to be the reason for the failure now | 20:13 |
| mriedem | melwitt: https://bugs.launchpad.net/cinder/+bug/1768650 | 20:15 |
| openstack | Launchpad bug 1768650 in Cinder "POST /volumes/{volume_id}/action os-detach incorrectly requires attachment_id is not None" [Undecided,New] | 20:15 |
| mriedem | i'll push a patch to cinder | 20:15 |
| melwitt | sounds great | 20:16 |
| *** damien_r has quit IRC | 20:17 | |
| *** slaweq has joined #openstack-nova | 20:26 | |
| *** spsurya has joined #openstack-nova | 20:27 | |
| *** openstackgerrit has quit IRC | 20:27 | |
| *** yamamoto has joined #openstack-nova | 20:29 | |
| *** tssurya has quit IRC | 20:32 | |
| *** tssurya has joined #openstack-nova | 20:33 | |
| *** pcaruana has quit IRC | 20:34 | |
| *** moshele has joined #openstack-nova | 20:34 | |
| *** damien_r has joined #openstack-nova | 20:34 | |
| *** yamamoto has quit IRC | 20:34 | |
| mriedem | yeehaw http://paste.openstack.org/show/720250/ | 20:41 |
| melwitt | old school, email it | 20:42 |
| mriedem | heh, i thought about it | 20:42 |
| *** tssurya has quit IRC | 20:43 | |
| *** damien_r has quit IRC | 20:46 | |
| *** jmlowe has quit IRC | 20:53 | |
| *** dougshelley66 has quit IRC | 20:55 | |
| *** vladikr_ has joined #openstack-nova | 20:59 | |
| *** awaugama has quit IRC | 21:00 | |
| *** vladikr has quit IRC | 21:01 | |
| *** moshele has quit IRC | 21:02 | |
| *** felipemonteiro__ has joined #openstack-nova | 21:06 | |
| *** felipemonteiro_ has quit IRC | 21:09 | |
| *** wolverineav has quit IRC | 21:10 | |
| *** wolverineav has joined #openstack-nova | 21:11 | |
| *** moshele has joined #openstack-nova | 21:14 | |
| *** pchavva has quit IRC | 21:14 | |
| *** wolverineav has quit IRC | 21:15 | |
| *** linkmark has joined #openstack-nova | 21:16 | |
| *** dougshelley66 has joined #openstack-nova | 21:23 | |
| *** r-daneel has quit IRC | 21:25 | |
| cfriesen | has anyone ever looked at supporting "-cpu host,migratable=no,+invtsc" in order to get the "nonstop_tsc" flag in the guest? The tricky bit seems to be that nova might want to block attempts to migrate or snapshot the instance if it's enabled since it's going to fail down in qemu. | 21:29 |
| *** hamzy has quit IRC | 21:29 | |
| *** yamamoto has joined #openstack-nova | 21:31 | |
| *** jmlowe has joined #openstack-nova | 21:31 | |
| *** moshele has quit IRC | 21:32 | |
| *** yamamoto has quit IRC | 21:37 | |
| *** dpawlik has joined #openstack-nova | 21:42 | |
| *** jmlowe has quit IRC | 21:45 | |
| *** dpawlik has quit IRC | 21:47 | |
| *** tssurya has joined #openstack-nova | 21:49 | |
| *** sdeath has joined #openstack-nova | 21:51 | |
| *** sdeath has joined #openstack-nova | 21:51 | |
| *** tssurya has quit IRC | 21:53 | |
| *** jmlowe has joined #openstack-nova | 21:57 | |
| *** pchavva has joined #openstack-nova | 22:01 | |
| *** felipemonteiro_ has joined #openstack-nova | 22:01 | |
| *** felipemonteiro__ has quit IRC | 22:01 | |
| *** esberglu has quit IRC | 22:06 | |
| dansmith | melwitt: HEY. This country was built with patch-on-ML! | 22:07 |
| melwitt | ;D | 22:08 |
| *** ChanServ changes topic to "Current runways: xenapi-image-handler-option-improvement / convert-consoles-to-objects / add-zvm-driver-rocky -- This channel is for Nova development. For support of Nova deployments, please use #openstack." | 22:10 | |
| -openstackstatus- NOTICE: Gerrit maintenance has concluded successfully | 22:10 | |
| lbragstad | melwitt: o/ johnthetubaguy was telling me that you are a quota expert | 22:12 |
| *** Guest27658 has quit IRC | 22:14 | |
| *** mlavalle has quit IRC | 22:14 | |
| melwitt | lbragstad: o/ an accidental one, maybe | 22:14 |
| *** slaweq has quit IRC | 22:15 | |
| lbragstad | :) i was working with john on documenting CERN's usecase for hierarchical quotas/limit with the unified limit approach | 22:15 |
| lbragstad | i think i got it all out on paper now, https://review.openstack.org/#/c/565412/ | 22:15 |
| *** slaweq has joined #openstack-nova | 22:16 | |
| lbragstad | but figured you'd be pretty good at poking holes in what we have so far | 22:16 |
| lbragstad | (there are a ton of diagrams, but the rendered version isn't bad- http://logs.openstack.org/12/565412/3/check/build-openstack-sphinx-docs/22ea042/html/specs/keystone/rocky/strict-two-level-enforcement-model.html ) | 22:16 |
| melwitt | a-ha, very cool. I will definitely take a look | 22:17 |
| * lbragstad just realized the code formatting failed | 22:17 | |
| lbragstad | thanks melwitt | 22:17 |
| melwitt | oh, heh, so failed == excluded from the output entirely. that's helpful of it :P | 22:18 |
| melwitt | lbragstad: np, thanks for linking me | 22:19 |
| lbragstad | no problem - latest patch set should render the python example for service to incorporate the limit usage | 22:20 |
| melwitt | excellent | 22:20 |
| *** openstackgerrit has joined #openstack-nova | 22:22 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get anchors for sharing providers https://review.openstack.org/565279 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: Object changes for granular https://review.openstack.org/564351 | 22:22 |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 22:22 |
| *** sdeath has quit IRC | 22:23 | |
| openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 22:25 |
| *** edmondsw has quit IRC | 22:29 | |
| *** edmondsw has joined #openstack-nova | 22:30 | |
| *** yamamoto has joined #openstack-nova | 22:32 | |
| *** edmondsw has quit IRC | 22:35 | |
| *** edmondsw has joined #openstack-nova | 22:35 | |
| *** rcernin has joined #openstack-nova | 22:36 | |
| *** yamamoto has quit IRC | 22:38 | |
| *** edmondsw has quit IRC | 22:39 | |
| efried | mriedem: Good job distancing yourself from the image traits rebuild issue. | 22:51 |
| efried | That lasted, what, four hours? | 22:52 |
| mriedem | almost 9 | 22:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add nova-manage placement heal_allocations CLI https://review.openstack.org/565886 | 22:55 |
| mriedem | mgagne: now with functional tests ^ | 22:55 |
| mgagne | mriedem: cool stuff! are there multiple cells in the test? | 22:56 |
| mriedem | yes, except that doesn't currently work b/c there is a bug in our fixtures, i need to test it with https://review.openstack.org/#/c/524264/ or https://review.openstack.org/#/c/558160/ underneath | 22:57 |
| mriedem | but, in-laws are here and i'm going to get yelled at soon | 22:57 |
| *** mriedem is now known as mriedem_inlaws | 22:58 | |
| mgagne | mriedem: there is no rush, have a good one ;) | 22:58 |
| *** spsurya has quit IRC | 23:04 | |
| *** redondo-mk_ has joined #openstack-nova | 23:09 | |
| *** andrewbogott_ has joined #openstack-nova | 23:09 | |
| *** mnaser_ has joined #openstack-nova | 23:09 | |
| *** andreaf has quit IRC | 23:09 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Use revision_number on adding/removing security group https://review.openstack.org/535510 | 23:11 |
| *** Anticime1 has joined #openstack-nova | 23:12 | |
| *** hongbin has quit IRC | 23:12 | |
| *** jlvillal_ has joined #openstack-nova | 23:15 | |
| *** afazekas_ has joined #openstack-nova | 23:15 | |
| *** MrWatson has joined #openstack-nova | 23:15 | |
| *** _d34dh0r53_ has joined #openstack-nova | 23:15 | |
| melwitt | guh, all that hacking on CellDatabases that I was trying to fix with https://review.openstack.org/508432 makes me sad | 23:15 |
| *** rnoriega_ has joined #openstack-nova | 23:16 | |
| *** lbragstad[m] has quit IRC | 23:16 | |
| *** rmk has quit IRC | 23:16 | |
| *** redondo-mk has quit IRC | 23:16 | |
| *** andrewbogott has quit IRC | 23:16 | |
| *** mnaser has quit IRC | 23:16 | |
| *** NostawRm has quit IRC | 23:16 | |
| *** rnoriega has quit IRC | 23:16 | |
| *** afazekas has quit IRC | 23:16 | |
| *** jlvillal has quit IRC | 23:16 | |
| *** Anticimex has quit IRC | 23:16 | |
| *** d34dh0r53 has quit IRC | 23:16 | |
| *** andrewbogott_ is now known as andrewbogott | 23:16 | |
| *** redondo-mk_ is now known as redondo-mk | 23:16 | |
| *** mnaser_ is now known as mnaser | 23:16 | |
| *** rmk has joined #openstack-nova | 23:17 | |
| *** rmk has quit IRC | 23:17 | |
| *** rmk has joined #openstack-nova | 23:17 | |
| *** jlvillal_ is now known as jlvillal | 23:19 | |
| *** jlvillal is now known as Guest10014 | 23:19 | |
| arvindn05 | melwitt: thanks for the reply on the rebuild thread....wanted to get your thoughts on whether the approach is easy to maintain going forward | 23:21 |
| arvindn05 | melwitt: my main concern is if we decide to support forbidden/granular traits etc for images at some point, this part of the logic will need to be updated | 23:21 |
| melwitt | well, rejecting a valid request doesn't seem correct and would likely require later maintenance too | 23:22 |
| arvindn05 | aand maintained going forward... | 23:22 |
| melwitt | meaning, the placement queries would have to be updated? | 23:22 |
| *** lbragstad[m] has joined #openstack-nova | 23:23 | |
| arvindn05 | not just that....the current approach is to get allocations for current instance and then match them in the conductor | 23:24 |
| *** andreaf has joined #openstack-nova | 23:25 | |
| arvindn05 | the matching happens in the conductor for required traits for now...but if we need to handle granular/forbidden traits etc this matching logic will need to be updated | 23:25 |
| melwitt | can you point me to the code you're referring to? | 23:27 |
| arvindn05 | my point is that since required traits are qualitative requirements, if they change its better to ask the scheduler rather than try to duplicate logic in placement and in conductor | 23:27 |
| arvindn05 | there is not code yet....but eric summarized in this post http://lists.openstack.org/pipermail/openstack-dev/2018-April/129740.html | 23:27 |
| melwitt | okay. well, I'm just saying verifying the traits in placement seems like the only correct thing to do | 23:28 |
| arvindn05 | ideally yes, but a rebuild request never gets to placement api with any of the options described in the thread i think | 23:30 |
| arvindn05 | is there an different alternative where verifying the traits can happen in placement code in case of rebuild? | 23:31 |
| *** dave-mcc_ has quit IRC | 23:32 | |
| melwitt | what? some of the options involve calling placement | 23:32 |
| *** yamamoto has joined #openstack-nova | 23:34 | |
| arvindn05 | sorry if its unclear...i meant if there is way to utilize the matching logic of placement instead of trying to duplicate that in the conductor/schedulers | 23:35 |
| *** dave-mccowan has joined #openstack-nova | 23:36 | |
| melwitt | if there's a way to ask placement "do these providers have these traits?" then that's what we need right? we need to ask if the providers hosting our current allocations have the traits requested in the image | 23:39 |
| *** yamamoto has quit IRC | 23:40 | |
| melwitt | if we could ask it that way, placement would do the matching. but I don't have deep knowledge the placement APIs | 23:40 |
| melwitt | that is, I don't know if there's an available API like that | 23:41 |
| arvindn05 | As i understand it its not just "do these providers have these traits?"...its also "do we have allocations from these providers for those traits?" | 23:41 |
| arvindn05 | since we cannot do allocations during rebuild...we need to make sure our existing allocations satisfy all the trait requirements | 23:42 |
| melwitt | it is, and that's what I tried to say. we have to 1) know our allocations 2) know what providers are hosting our allocations 3) know what traits the providers in 2) have | 23:42 |
| *** dpawlik has joined #openstack-nova | 23:43 | |
| melwitt | so I guess it would be 1) query for own allocations by consumer == our instance 2) use the providers returned by that to ask 2) do these providers have these traits. again, I'm making up the second query, I don't know if one like that exists | 23:44 |
| melwitt | maybe I made up the first one too, but you get the idea | 23:44 |
| arvindn05 | yea...that was what efried had... | 23:45 |
| *** Swami has quit IRC | 23:46 | |
| *** vladikr__ has joined #openstack-nova | 23:46 | |
| *** dpawlik has quit IRC | 23:47 | |
| arvindn05 | the only 2 cons for this approach | 23:47 |
| arvindn05 | 1.the rebuild may or may not succeed, depending on how well the original allocations match up with the new requirements. | 23:47 |
| arvindn05 | 2. we need to need to account for handling preferred traits or granular resource traits if we decide to implement them for images at some point... | 23:47 |
| arvindn05 | see http://lists.openstack.org/pipermail/openstack-dev/2018-May/130087.html | 23:47 |
| arvindn05 | to see why 1 may occur.... | 23:48 |
| *** vladikr_ has quit IRC | 23:48 | |
| melwitt | are traits the same as properties? do they replace properties or? | 23:51 |
| melwitt | *image properties | 23:51 |
| arvindn05 | they are an addition to image properties... | 23:53 |
| arvindn05 | maybe long term they can start to replace some of the image properties.... | 23:53 |
| melwitt | okay. was just thinking about whether 1. would be a regression on how it behaves today. you can rebuild with a new image with new image properties if they pass scheduler filtering | 23:53 |
| melwitt | if image property == image trait then if you reject any rebuild with new traits, then that's a step backward from what's possible today | 23:54 |
| arvindn05 | nope traits are independent of image properties...they use the same key:value and are expressed as such in the image properties...but not the same | 23:56 |
| *** pchavva has quit IRC | 23:57 | |
| *** vladikr_ has joined #openstack-nova | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!