| *** elmaciej has joined #openstack-nova | 00:17 | |
| *** sdague has quit IRC | 00:19 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (1/2) https://review.openstack.org/430608 | 00:20 |
|---|---|---|
| *** takashin has joined #openstack-nova | 00:21 | |
| *** owalsh_ has joined #openstack-nova | 00:29 | |
| *** dougshel_ has quit IRC | 00:29 | |
| *** owalsh has quit IRC | 00:31 | |
| *** lbragstad has joined #openstack-nova | 00:34 | |
| *** Dinesh__Bhor has joined #openstack-nova | 00:35 | |
| *** Dinesh__Bhor has quit IRC | 00:36 | |
| *** slaweq has joined #openstack-nova | 00:39 | |
| *** dims has quit IRC | 00:42 | |
| *** slaweq has quit IRC | 00:44 | |
| *** liuzz has joined #openstack-nova | 00:46 | |
| *** lbragstad has quit IRC | 00:49 | |
| *** tbachman has joined #openstack-nova | 00:51 | |
| *** edmondsw has joined #openstack-nova | 00:56 | |
| *** sambetts_ has quit IRC | 00:59 | |
| *** lbragstad has joined #openstack-nova | 00:59 | |
| *** sambetts_ has joined #openstack-nova | 01:00 | |
| *** hshiina has joined #openstack-nova | 01:03 | |
| *** salv-orl_ has joined #openstack-nova | 01:04 | |
| *** chenyb4 has joined #openstack-nova | 01:07 | |
| *** salv-orlando has quit IRC | 01:07 | |
| *** mlavalle has joined #openstack-nova | 01:12 | |
| *** lbragstad has quit IRC | 01:14 | |
| *** hongbin_ has joined #openstack-nova | 01:14 | |
| *** zhaochao has joined #openstack-nova | 01:16 | |
| *** sambetts_ has quit IRC | 01:19 | |
| *** sambetts_ has joined #openstack-nova | 01:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 1 https://review.openstack.org/560296 | 01:22 |
| *** jichen has joined #openstack-nova | 01:23 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 2 https://review.openstack.org/560299 | 01:27 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 3 https://review.openstack.org/560727 | 01:28 |
| *** owalsh_ has quit IRC | 01:29 | |
| *** owalsh has joined #openstack-nova | 01:29 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 01:32 |
| *** edmondsw has quit IRC | 01:33 | |
| *** edmondsw has joined #openstack-nova | 01:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: List/show all server migration types (2/2) https://review.openstack.org/459483 | 01:35 |
| *** edmondsw has quit IRC | 01:38 | |
| *** dougshelley66 has joined #openstack-nova | 01:38 | |
| *** hiro-kobayashi has joined #openstack-nova | 01:45 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 4 https://review.openstack.org/560731 | 01:46 |
| *** blkart has joined #openstack-nova | 01:52 | |
| *** masuberu has joined #openstack-nova | 01:54 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Record the host info in EventReporter https://review.openstack.org/556746 | 01:56 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Extract generate_hostid method into utils.py https://review.openstack.org/555282 | 01:56 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add host/hostId to instance action events API https://review.openstack.org/557145 | 01:56 |
| *** masber has quit IRC | 01:58 | |
| *** masber has joined #openstack-nova | 02:00 | |
| *** masuberu has quit IRC | 02:01 | |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Address nits in I00d29e9fd80e6b8f7ba3bbd8e82dde9d4cb1522f https://review.openstack.org/561493 | 02:03 |
| *** mlavalle has quit IRC | 02:03 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 5 https://review.openstack.org/560734 | 02:04 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Deduplicate notification samples Rocky - 6 https://review.openstack.org/560737 | 02:06 |
| *** gongysh has joined #openstack-nova | 02:06 | |
| *** masber has quit IRC | 02:13 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add request_id to instance action notifications https://review.openstack.org/553288 | 02:17 |
| *** Sukhdev has joined #openstack-nova | 02:34 | |
| *** gaoyan has joined #openstack-nova | 02:39 | |
| *** psachin has joined #openstack-nova | 02:39 | |
| *** gaoyan has quit IRC | 02:44 | |
| *** janki has joined #openstack-nova | 02:45 | |
| *** bingqianglin has joined #openstack-nova | 02:47 | |
| *** edmondsw has joined #openstack-nova | 02:48 | |
| *** edmondsw has quit IRC | 02:52 | |
| *** elmaciej has quit IRC | 03:01 | |
| *** wolverineav has quit IRC | 03:06 | |
| *** wolverineav has joined #openstack-nova | 03:06 | |
| *** wolverineav has quit IRC | 03:11 | |
| *** liuzz_ has joined #openstack-nova | 03:16 | |
| *** dklyle has quit IRC | 03:18 | |
| *** liuzz has quit IRC | 03:19 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Fix doc link for api https://review.openstack.org/561499 | 03:25 |
| *** takashin has quit IRC | 03:27 | |
| *** Sukhdev has quit IRC | 03:33 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Move update_task_state out of try/except https://review.openstack.org/557152 | 03:34 |
| *** hongbin_ has quit IRC | 03:35 | |
| *** lpetrut has joined #openstack-nova | 03:36 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add config drive link to api-guide https://review.openstack.org/561113 | 03:37 |
| *** wolverineav has joined #openstack-nova | 03:42 | |
| *** lbragstad has joined #openstack-nova | 03:48 | |
| *** masber has joined #openstack-nova | 03:49 | |
| *** bingqianglin has quit IRC | 03:57 | |
| openstackgerrit | jichenjc proposed openstack/nova master: [Doc]Link policies file into api https://review.openstack.org/561503 | 04:03 |
| *** lpetrut has quit IRC | 04:05 | |
| *** sridharg has joined #openstack-nova | 04:18 | |
| *** bhagyashris has joined #openstack-nova | 04:19 | |
| *** bkopilov has quit IRC | 04:22 | |
| openstackgerrit | Nguyen Hai proposed openstack/nova-specs master: Enhance nova-specs webpage and clean up repo https://review.openstack.org/551802 | 04:27 |
| *** udesale has joined #openstack-nova | 04:28 | |
| *** Tom-Tom has joined #openstack-nova | 04:35 | |
| *** edmondsw has joined #openstack-nova | 04:36 | |
| *** edmondsw has quit IRC | 04:40 | |
| *** slaweq has joined #openstack-nova | 04:41 | |
| *** slaweq has quit IRC | 04:45 | |
| *** bingqianglin has joined #openstack-nova | 04:49 | |
| *** ratailor has joined #openstack-nova | 04:53 | |
| *** claudiub|2 has joined #openstack-nova | 04:55 | |
| *** vivsoni has joined #openstack-nova | 05:00 | |
| *** fragatina has quit IRC | 05:01 | |
| *** wolverineav has quit IRC | 05:01 | |
| *** wolverineav has joined #openstack-nova | 05:01 | |
| *** gongysh has quit IRC | 05:09 | |
| *** gongysh has joined #openstack-nova | 05:12 | |
| *** abhishekk has joined #openstack-nova | 05:20 | |
| *** bingqianglin1 has joined #openstack-nova | 05:21 | |
| *** bingqianglin has quit IRC | 05:23 | |
| *** bingqianglin1 is now known as bingqianglin | 05:23 | |
| *** takashin has joined #openstack-nova | 05:23 | |
| *** gongysh has quit IRC | 05:24 | |
| *** sidx64 has joined #openstack-nova | 05:25 | |
| *** dpawlik has joined #openstack-nova | 05:39 | |
| *** dpawlik has quit IRC | 05:39 | |
| *** gongysh has joined #openstack-nova | 05:45 | |
| *** tojuvone has joined #openstack-nova | 05:50 | |
| takashin | 05:54 | |
| *** lajoskatona has joined #openstack-nova | 05:54 | |
| *** sar has quit IRC | 05:55 | |
| *** vivsoni has quit IRC | 05:56 | |
| *** vivsoni has joined #openstack-nova | 05:57 | |
| *** takedakn has joined #openstack-nova | 05:58 | |
| *** liuzz has joined #openstack-nova | 05:59 | |
| *** liuzz has quit IRC | 05:59 | |
| *** liuzz has joined #openstack-nova | 06:00 | |
| *** liuzz_ has quit IRC | 06:02 | |
| *** takedakn has quit IRC | 06:03 | |
| *** wolverineav has quit IRC | 06:08 | |
| *** wolverineav has joined #openstack-nova | 06:08 | |
| *** mvk has quit IRC | 06:10 | |
| *** fragatina has joined #openstack-nova | 06:10 | |
| *** mvk has joined #openstack-nova | 06:10 | |
| *** wolverineav has quit IRC | 06:12 | |
| *** jogo has quit IRC | 06:18 | |
| *** wolverineav has joined #openstack-nova | 06:20 | |
| *** sar has joined #openstack-nova | 06:23 | |
| *** edmondsw has joined #openstack-nova | 06:24 | |
| *** edmondsw has quit IRC | 06:28 | |
| *** lpetrut has joined #openstack-nova | 06:29 | |
| *** jchhatbar has joined #openstack-nova | 06:29 | |
| *** salv-orlando has joined #openstack-nova | 06:31 | |
| *** salv-orl_ has quit IRC | 06:31 | |
| *** janki has quit IRC | 06:32 | |
| *** do3meli has joined #openstack-nova | 06:33 | |
| *** ircuser-1 has joined #openstack-nova | 06:34 | |
| *** udesale_ has joined #openstack-nova | 06:34 | |
| *** udesale_ has quit IRC | 06:35 | |
| *** udesale_ has joined #openstack-nova | 06:35 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 06:37 |
| *** udesale has quit IRC | 06:38 | |
| *** sambetts_ has quit IRC | 06:38 | |
| *** vivsoni has quit IRC | 06:39 | |
| *** sidx64_ has joined #openstack-nova | 06:39 | |
| *** sambetts_ has joined #openstack-nova | 06:40 | |
| *** sidx64 has quit IRC | 06:41 | |
| *** andreas_s has joined #openstack-nova | 06:41 | |
| *** wolverineav has quit IRC | 06:41 | |
| *** sidx64 has joined #openstack-nova | 06:43 | |
| *** sidx64_ has quit IRC | 06:44 | |
| *** wolverineav has joined #openstack-nova | 06:48 | |
| *** vivsoni has joined #openstack-nova | 06:48 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 06:50 |
| *** wolverineav has quit IRC | 06:50 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support relay RP for allocation candidates https://review.openstack.org/533437 | 06:50 |
| *** slaweq has joined #openstack-nova | 06:51 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.61 - support extra_specs in flavor API https://review.openstack.org/561520 | 06:53 |
| *** armaan_ has joined #openstack-nova | 06:54 | |
| *** armaan has quit IRC | 06:54 | |
| *** rcernin has quit IRC | 06:56 | |
| *** moshele has joined #openstack-nova | 07:00 | |
| *** brault has quit IRC | 07:07 | |
| *** fragatina has quit IRC | 07:08 | |
| arvindn05 | bauzas: wanted to discuss your comments on https://review.openstack.org/#/c/560718/ - Handle rebuild of instance with new image | 07:11 |
| *** pcaruana has joined #openstack-nova | 07:13 | |
| *** jogo has joined #openstack-nova | 07:15 | |
| *** bkopilov has joined #openstack-nova | 07:17 | |
| *** tesseract has joined #openstack-nova | 07:21 | |
| *** yingjun has joined #openstack-nova | 07:24 | |
| *** gibi_awa1 is now known as gibi | 07:26 | |
| gibi | good morning nova | 07:26 |
| *** Dinesh_Bhor has joined #openstack-nova | 07:27 | |
| * gibi is trying to catch up | 07:27 | |
| *** ccamacho has joined #openstack-nova | 07:28 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | 07:28 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 07:28 |
| *** udesale__ has joined #openstack-nova | 07:32 | |
| *** tesseract has quit IRC | 07:33 | |
| *** udesale_ has quit IRC | 07:35 | |
| *** tesseract has joined #openstack-nova | 07:35 | |
| *** sahid has joined #openstack-nova | 07:36 | |
| *** Tom-Tom has quit IRC | 07:38 | |
| *** ragiman has joined #openstack-nova | 07:41 | |
| *** avolkov has joined #openstack-nova | 07:41 | |
| *** amoralej|off is now known as amoralej | 07:42 | |
| *** ralonsoh has joined #openstack-nova | 07:44 | |
| *** mgoddard has joined #openstack-nova | 07:44 | |
| openstackgerrit | Michael Still proposed openstack/nova master: Move IVS VIF plugging into the new utilities module. https://review.openstack.org/557552 | 07:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move create_tap_dev to the new utility method. https://review.openstack.org/557835 | 07:45 |
| openstackgerrit | Michael Still proposed openstack/nova master: Move set_vf_interface_vlan to the new utility module. https://review.openstack.org/558487 | 07:45 |
| *** dpawlik has joined #openstack-nova | 07:47 | |
| *** threestrands has quit IRC | 07:47 | |
| *** itlinux has joined #openstack-nova | 07:47 | |
| openstackgerrit | jichenjc proposed openstack/nova-specs master: Adds spec for instance live resize https://review.openstack.org/141219 | 07:47 |
| *** lajoskatona has quit IRC | 07:51 | |
| *** jpena|off is now known as jpena | 07:51 | |
| *** evrardjp has joined #openstack-nova | 07:55 | |
| *** ralonsoh_ has joined #openstack-nova | 07:56 | |
| *** ralonsoh has quit IRC | 07:58 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:58 | |
| *** migi has quit IRC | 07:59 | |
| *** migi has joined #openstack-nova | 07:59 | |
| *** migi has quit IRC | 08:00 | |
| *** itlinux_ has joined #openstack-nova | 08:02 | |
| *** takashin has left #openstack-nova | 08:02 | |
| *** itlinux has quit IRC | 08:04 | |
| *** itlinux_ has quit IRC | 08:06 | |
| *** lajoskatona has joined #openstack-nova | 08:07 | |
| *** Tom-Tom has joined #openstack-nova | 08:08 | |
| *** sambetts_ has quit IRC | 08:09 | |
| *** lucas-afk is now known as lucasagomes | 08:10 | |
| *** jaosorior has joined #openstack-nova | 08:10 | |
| *** sambetts_ has joined #openstack-nova | 08:13 | |
| *** edmondsw has joined #openstack-nova | 08:13 | |
| *** edmondsw has quit IRC | 08:17 | |
| *** mdbooth has joined #openstack-nova | 08:19 | |
| *** mdnadeem has joined #openstack-nova | 08:19 | |
| *** gongysh has quit IRC | 08:22 | |
| *** gongysh has joined #openstack-nova | 08:25 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver https://review.openstack.org/554195 | 08:26 |
| *** derekh has joined #openstack-nova | 08:28 | |
| *** tssurya has joined #openstack-nova | 08:29 | |
| *** abhishekk_ has joined #openstack-nova | 08:31 | |
| *** abhishekk has quit IRC | 08:32 | |
| *** armaan has joined #openstack-nova | 08:34 | |
| *** armaan_ has quit IRC | 08:35 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: virt/ironic: Implement rescue and unrescue https://review.openstack.org/416487 | 08:36 |
| *** brault has joined #openstack-nova | 08:36 | |
| *** yingjun has quit IRC | 08:37 | |
| *** abhishekk_ is now known as abhishekk | 08:39 | |
| *** gongysh has quit IRC | 08:40 | |
| *** gongysh has joined #openstack-nova | 08:43 | |
| openstackgerrit | Silvan Kaiser proposed openstack/nova master: Exec systemd-run with privileges in Quobyte driver https://review.openstack.org/554195 | 08:46 |
| *** lajoskatona has quit IRC | 08:47 | |
| *** itlinux has joined #openstack-nova | 08:53 | |
| *** itlinux has quit IRC | 08:54 | |
| kaisers1 | Hi! I'm looking for Novas policies on bugfix backports but so far was unable to find documentation on this, can somebody pls drop me a hint where to find this? :) | 08:57 |
| *** itlinux has joined #openstack-nova | 08:57 | |
| *** brault_ has joined #openstack-nova | 09:00 | |
| bauzas | kaisers1: https://docs.openstack.org/project-team-guide/stable-branches.html#proposing-fixes | 09:01 |
| *** ralonsoh__ has joined #openstack-nova | 09:02 | |
| *** abhishekk has quit IRC | 09:02 | |
| *** brault has quit IRC | 09:02 | |
| *** lajoskatona has joined #openstack-nova | 09:02 | |
| bauzas | arvindn05: hola | 09:03 |
| bauzas | sure, I'm here now :) | 09:03 |
| bauzas | arvindn05: about the rebuild conditional, I saw it, but I'm not sure I like it, you know :) | 09:05 |
| *** ralonsoh_ has quit IRC | 09:05 | |
| kaisers1 | bauzas: thanks! | 09:10 |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: [WIP] Send out notification when instance metadata updated https://review.openstack.org/558140 | 09:12 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: Add rules column to instance_group_policy table. https://review.openstack.org/560832 | 09:16 |
| openstackgerrit | Yikun Jiang (Kero) proposed openstack/nova master: WIP: complex policy https://review.openstack.org/553776 | 09:16 |
| *** abhishekk has joined #openstack-nova | 09:19 | |
| *** cdent has joined #openstack-nova | 09:21 | |
| *** trinaths has joined #openstack-nova | 09:26 | |
| *** itlinux has quit IRC | 09:28 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: add support for virtio-net rx/tx queue sizes https://review.openstack.org/484997 | 09:33 |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562 | 09:33 |
| *** hiro-kobayashi has quit IRC | 09:34 | |
| *** bhagyashris has quit IRC | 09:34 | |
| *** dr_gogeta86 has joined #openstack-nova | 09:34 | |
| *** dr_gogeta86 has quit IRC | 09:34 | |
| *** dr_gogeta86 has joined #openstack-nova | 09:34 | |
| *** Dinesh_Bhor has quit IRC | 09:35 | |
| *** armaan has quit IRC | 09:41 | |
| *** lajoskatona has quit IRC | 09:48 | |
| *** elmaciej has joined #openstack-nova | 09:51 | |
| *** gongysh has quit IRC | 09:51 | |
| *** elmaciej has quit IRC | 09:55 | |
| *** ociuhandu has joined #openstack-nova | 09:56 | |
| *** ociuhandu has quit IRC | 09:58 | |
| *** bingqianglin has quit IRC | 10:03 | |
| *** lajoskatona has joined #openstack-nova | 10:04 | |
| *** dpawlik has quit IRC | 10:05 | |
| *** sahid has quit IRC | 10:05 | |
| *** dpawlik has joined #openstack-nova | 10:05 | |
| *** jichen has quit IRC | 10:08 | |
| *** sahid has joined #openstack-nova | 10:08 | |
| *** tianhui has quit IRC | 10:12 | |
| *** tianhui has joined #openstack-nova | 10:13 | |
| *** alexchadin has joined #openstack-nova | 10:16 | |
| *** elmaciej has joined #openstack-nova | 10:18 | |
| *** bhagyashris has joined #openstack-nova | 10:19 | |
| *** cdent has quit IRC | 10:21 | |
| *** tianhui_ has joined #openstack-nova | 10:24 | |
| *** tianhui has quit IRC | 10:26 | |
| *** fragatina has joined #openstack-nova | 10:32 | |
| *** nicolasbock has joined #openstack-nova | 10:33 | |
| *** armaan has joined #openstack-nova | 10:35 | |
| *** chenyb4 has quit IRC | 10:36 | |
| *** nicolasbock has quit IRC | 10:39 | |
| *** sdague has joined #openstack-nova | 10:40 | |
| *** stakeda has quit IRC | 10:42 | |
| *** armaan has quit IRC | 10:45 | |
| *** damien_r has joined #openstack-nova | 10:45 | |
| *** pooja_jadhav has joined #openstack-nova | 10:49 | |
| *** cdent has joined #openstack-nova | 10:50 | |
| *** damien_r has quit IRC | 10:54 | |
| *** tbachman has quit IRC | 10:56 | |
| *** fragatina has quit IRC | 10:57 | |
| *** slagle has joined #openstack-nova | 10:57 | |
| *** abhishekk has quit IRC | 11:02 | |
| *** lucasagomes is now known as lucas-hungry | 11:04 | |
| *** udesale__ has quit IRC | 11:05 | |
| *** sambetts_ is now known as sambetts | 11:06 | |
| openstackgerrit | Martin Midolesov proposed openstack/nova master: [vmwareapi] Expose cpu model on homogeneous cluster https://review.openstack.org/561588 | 11:07 |
| *** armaan has joined #openstack-nova | 11:13 | |
| *** bkopilov has quit IRC | 11:16 | |
| *** itlinux has joined #openstack-nova | 11:16 | |
| *** nicolasbock has joined #openstack-nova | 11:17 | |
| *** mdbooth has quit IRC | 11:18 | |
| *** mdbooth has joined #openstack-nova | 11:19 | |
| *** trinaths has quit IRC | 11:23 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Pass disk_info dict to libvirt_info https://review.openstack.org/529329 | 11:34 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Expose volume host type and path independent of libvirt config https://review.openstack.org/530786 | 11:34 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Don't generate fake disk_info in swap_volume https://review.openstack.org/530787 | 11:34 |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Local disk serial numbers for the libvirt driver https://review.openstack.org/529380 | 11:34 |
| *** alexchadin has quit IRC | 11:37 | |
| *** sidx64 has quit IRC | 11:38 | |
| *** sidx64 has joined #openstack-nova | 11:39 | |
| *** hshiina is now known as hshiina|afk | 11:42 | |
| *** armaan has quit IRC | 11:46 | |
| *** armaan has joined #openstack-nova | 11:47 | |
| *** armaan_ has joined #openstack-nova | 11:50 | |
| *** armaan has quit IRC | 11:50 | |
| *** dave-mcc_ has joined #openstack-nova | 11:59 | |
| *** markvoelker has joined #openstack-nova | 11:59 | |
| *** dave-mccowan has quit IRC | 12:01 | |
| *** Tom-Tom has quit IRC | 12:01 | |
| *** Tom-Tom has joined #openstack-nova | 12:02 | |
| *** lucas-hungry is now known as lucasagomes | 12:02 | |
| cdent | welcome back gibi, hope you had a great honeymoon | 12:04 |
| *** mvk has quit IRC | 12:05 | |
| *** QianYu has joined #openstack-nova | 12:05 | |
| *** Tom-Tom has quit IRC | 12:06 | |
| *** amoralej is now known as amoralej|lunch | 12:07 | |
| *** tbachman has joined #openstack-nova | 12:08 | |
| *** cdent has quit IRC | 12:08 | |
| *** armaan_ has quit IRC | 12:10 | |
| *** QianYu has quit IRC | 12:10 | |
| *** armaan has joined #openstack-nova | 12:11 | |
| *** armaan has quit IRC | 12:15 | |
| *** pchavva has joined #openstack-nova | 12:20 | |
| *** edmondsw has joined #openstack-nova | 12:21 | |
| *** mgoddard has quit IRC | 12:21 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: libvirt: Block swap volume attempts with encrypted volumes prior to Queens https://review.openstack.org/543569 | 12:22 |
| *** tbachman has quit IRC | 12:24 | |
| *** udesale__ has joined #openstack-nova | 12:24 | |
| *** udesale__ has quit IRC | 12:26 | |
| *** udesale has joined #openstack-nova | 12:26 | |
| *** leakypipes is now known as jaypipes | 12:27 | |
| *** dims has joined #openstack-nova | 12:30 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: Refactor a test method including 3 test cases https://review.openstack.org/561603 | 12:30 |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: Block swap volume attempts with encrypted volumes prior to Queens https://review.openstack.org/561604 | 12:30 |
| *** tbachman has joined #openstack-nova | 12:31 | |
| *** cdent has joined #openstack-nova | 12:33 | |
| *** alexchadin has joined #openstack-nova | 12:33 | |
| *** owalsh has quit IRC | 12:34 | |
| *** owalsh has joined #openstack-nova | 12:35 | |
| gibi | cdent: thanks! We had a great honeymoon indeed | 12:37 |
| cdent | \o/ | 12:37 |
| *** liuzz_ has joined #openstack-nova | 12:37 | |
| *** yamamoto has quit IRC | 12:38 | |
| efried | Say, anyone been after the gate yet this morning? I noticed it started failing pretty consistently on Friday (of course). | 12:38 |
| * gibi haven't had the chance to look at the gate state yet | 12:39 | |
| efried | cdent: You know things about grenade, yah? | 12:41 |
| *** amoralej|lunch is now known as amoralej | 12:41 | |
| cdent | efried: there's discussion on os-dev related to neutron and requirements handling breaking the gate | 12:41 |
| cdent | so I think it is being worked on already | 12:41 |
| *** liuzz has quit IRC | 12:41 | |
| efried | oh, okay. | 12:41 |
| efried | So it's not nova's fault? | 12:41 |
| *** owalsh has quit IRC | 12:42 | |
| cdent | as far as I was able to discern, no | 12:42 |
| *** owalsh has joined #openstack-nova | 12:42 | |
| efried | k, I guess I'll just wait, then. | 12:42 |
| * efried starts deleting zuul failure emails... | 12:42 | |
| *** armaan has joined #openstack-nova | 12:43 | |
| *** kholkina has joined #openstack-nova | 12:45 | |
| *** itlinux has quit IRC | 12:46 | |
| cdent | efried: this is at least one of the bugs: https://bugs.launchpad.net/devstack/+bug/1763966 | 12:46 |
| openstack | Launchpad bug 1763966 in grenade "python-psutil dependency error on ubuntu 16.04.4" [Undecided,New] | 12:46 |
| efried | ack | 12:47 |
| *** sidx64 has quit IRC | 12:47 | |
| *** oanson has quit IRC | 12:49 | |
| *** mvk has joined #openstack-nova | 12:49 | |
| bauzas | efried: yet again working on my NUMA spec | 12:51 |
| *** chenyb4 has joined #openstack-nova | 12:51 | |
| efried | mhm | 12:52 |
| bauzas | efried: tbh, I need to look again at your own group spec | 12:52 |
| * efried provides bauzas helpful link: http://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/granular-resource-requests.html | 12:52 | |
| bauzas | yeah I'm looking at it now | 12:53 |
| *** sidx64 has joined #openstack-nova | 12:55 | |
| *** armaan has quit IRC | 12:55 | |
| *** armaan has joined #openstack-nova | 12:56 | |
| *** mgoddard has joined #openstack-nova | 12:56 | |
| *** chenyb4 has quit IRC | 12:56 | |
| bauzas | efried: /me trying to understand https://specs.openstack.org/openstack/nova-specs/specs/rocky/approved/granular-resource-requests.html#requirement-3 | 12:56 |
| *** oanson has joined #openstack-nova | 12:57 | |
| efried | bauzas: You should note that jaypipes has proposed an amendment which appreciably changes things: https://review.openstack.org/#/c/560974/ -- but if I have my way, that delta will die a fiery death. | 12:57 |
| efried | bauzas: requirement 3 is saying that we want to be able to land our instance even if it means spreading VCPU resources across multiple NUMA nodes because we're reaching a point where they're getting pretty full and we can't fit all of them on one. | 12:58 |
| *** sdeath has quit IRC | 12:58 | |
| *** bkopilov has joined #openstack-nova | 12:59 | |
| *** LAKUNI1 has joined #openstack-nova | 12:59 | |
| bauzas | efried: what I don't understand is, if I have a tree with for example a RP, and then a child RP passing VGPU resources, would I still have the compute UUID RP being accepted if I'm saying some unnumbered group, like resources:VCPU=1&VGPU=1 ? | 12:59 |
| *** dklyle has joined #openstack-nova | 13:00 | |
| *** ArchiFleKs has quit IRC | 13:00 | |
| bauzas | or do I need to ask for resources1:VCPU=1&resources2:VGPU=1 ? | 13:00 |
| LAKUNI1 | ssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssssss | 13:00 |
| *** armaan has quit IRC | 13:00 | |
| artom | snek! | 13:00 |
| efried | bauzas: In that situation, either one would be fine. | 13:01 |
| bauzas | efried: because atm, the main RP is having VGPU inventories, but after my implementation, it would have a nested RP which would only pass VGPU | 13:01 |
| bauzas | efried: ok cool then | 13:01 |
| bauzas | efried: then why people should care about numbered req groups ? | 13:01 |
| *** yamamoto has joined #openstack-nova | 13:01 | |
| * bauzas trying to understand the usecases | 13:02 | |
| efried | bauzas: Because let's say for example you want two VFs on different networks. | 13:02 |
| efried | Networks are represented by traits on the PF RPs. | 13:02 |
| efried | We don't allow you to say resources=VF:1,VF:1 | 13:02 |
| *** vladikr has joined #openstack-nova | 13:02 | |
| bauzas | aha, I see it written The semantic for the (single) un-numbered grouping is unchanged. That is, it may still return results from different RPs in the same tree (or, when “shared” is fully implemented, the same aggregate). However, a numbered group will always return results from the same RP. This is to satisfy Requirement 2. | 13:03 |
| *** LAKUNI1 has left #openstack-nova | 13:04 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova stable/queens: only increment disk address unit for scsi devices https://review.openstack.org/561196 | 13:04 |
| bauzas | efried: ok, so, resources:VGPU:1,VCPU=1 is identical to resources1:VGPU=1&resources2:VGPU=1 but *not equivalent to* resources1:VGPU=1,VCPU=1 | 13:04 |
| bauzas | because if you're passing multiple resource classes to the same numbered group, then they need to be in the same RP | 13:04 |
| bauzas | right? | 13:05 |
| efried | bauzas: With the spec as written, that's correct. With jaypipes' amendment, those would be very different. | 13:05 |
| bauzas | argh | 13:05 |
| efried | Sorry, that was confusing. | 13:05 |
| bauzas | efried: https://review.openstack.org/#/c/560974/1/specs/rocky/approved/granular-resource-requests.rst@34 right? | 13:05 |
| bauzas | the conf option that would modify the behaviour ? | 13:05 |
| bauzas | I'm not sure I like that | 13:05 |
| *** armaan has joined #openstack-nova | 13:07 | |
| efried | The conf option would allow you to specify resource classes in the spec that allow their units to be spread across multiple RPs, when specified in the un-numbered request group. | 13:07 |
| *** sidx64 has quit IRC | 13:07 | |
| *** trozet has joined #openstack-nova | 13:07 | |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Numbered request groups use different providers https://review.openstack.org/560974 | 13:08 |
| efried | bauzas: I think jaypipes was trying to satisfy the "spread" use case. So you would put VCPU in that conf option, and then you could say resources=VCPU:4, and placement would *first* try to fit all the VCPUs on one RP, but if it couldn't, it would then try to do 2 and 2, and then 1,1,1,1 | 13:08 |
| bauzas | efried: I see | 13:08 |
| *** dklyle has quit IRC | 13:09 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova stable/pike: only increment disk address unit for scsi devices https://review.openstack.org/561611 | 13:09 |
| jaypipes | efried: sorry, just pushed up a new rev on that. didn't change much, though. | 13:09 |
| *** lyan has joined #openstack-nova | 13:09 | |
| bauzas | efried: I'm rushing to upload my spec so we could be discussing about that | 13:09 |
| *** lyan is now known as Guest98903 | 13:10 | |
| efried | jaypipes: ack. I was just responding to cdent's comments... | 13:10 |
| jaypipes | efried: yup, I responded to both you and cdent on the last revision review. | 13:10 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova stable/ocata: only increment disk address unit for scsi devices https://review.openstack.org/561613 | 13:11 |
| *** awaugama has joined #openstack-nova | 13:11 | |
| *** mriedem has joined #openstack-nova | 13:13 | |
| *** M0z122 has joined #openstack-nova | 13:17 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Proposes NUMA topology with RPs https://review.openstack.org/552924 | 13:19 |
| bauzas | efried: jaypipes: you could be interested in ^ | 13:19 |
| jaypipes | bauzas: s/could/will/ | 13:20 |
| efried | ack | 13:20 |
| *** ttsiouts_ has joined #openstack-nova | 13:22 | |
| *** fabian_ has joined #openstack-nova | 13:26 | |
| *** jroll has quit IRC | 13:26 | |
| *** QianYu has joined #openstack-nova | 13:26 | |
| *** jroll has joined #openstack-nova | 13:27 | |
| *** zhaochao has quit IRC | 13:27 | |
| M0z122 | Hi there, | 13:29 |
| *** mdbooth has quit IRC | 13:29 | |
| M0z122 | I'm interested in LVM backed instances, But it seems that cold migrate,live-migration,live snapshoting, resize are not implemented when images_type == lvm. Do you know if there are any onging dev on this, or any plan to dev it ? | 13:29 |
| * bauzas back to code | 13:31 | |
| *** superdan is now known as dansmith | 13:35 | |
| *** ratailor has quit IRC | 13:36 | |
| *** burt has joined #openstack-nova | 13:37 | |
| *** mgariepy has joined #openstack-nova | 13:38 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova stable/ocata: libvirt: Block swap volume attempts with encrypted volumes prior to Queens https://review.openstack.org/561604 | 13:40 |
| *** esberglu has joined #openstack-nova | 13:42 | |
| mriedem | M0z122: cfriesen has code for that i believe | 13:43 |
| *** armaan has quit IRC | 13:45 | |
| *** armaan has joined #openstack-nova | 13:46 | |
| *** psachin has quit IRC | 13:47 | |
| *** yingjun has joined #openstack-nova | 13:47 | |
| *** QianYu_ has joined #openstack-nova | 13:48 | |
| *** bhagyashris has quit IRC | 13:48 | |
| *** QianYu has quit IRC | 13:48 | |
| *** jmlowe_ has quit IRC | 13:49 | |
| *** bhagyashris has joined #openstack-nova | 13:49 | |
| *** eharney has joined #openstack-nova | 13:49 | |
| *** awaugama has quit IRC | 13:50 | |
| *** pooja-jadhav has joined #openstack-nova | 13:51 | |
| openstackgerrit | Merged openstack/nova master: Fix doc link for api https://review.openstack.org/561499 | 13:51 |
| *** alexchadin has quit IRC | 13:52 | |
| *** pooja_jadhav has quit IRC | 13:52 | |
| dansmith | mriedem: should we be holding off on approvals for the current fails? seems to have hit everything I've looked at this morning already | 13:52 |
| *** sar has quit IRC | 13:53 | |
| efried | dansmith: Doc-only fixes are kosher :) ---^ | 13:53 |
| *** gongysh has joined #openstack-nova | 13:53 | |
| dansmith | ...yeah, and/ | 13:53 |
| efried | dansmith: cdent was telling me earlier we think this has something to do with neutron and requirements, with this being one of the culprits: https://bugs.launchpad.net/devstack/+bug/1763966 | 13:54 |
| openstack | Launchpad bug 1763966 in devstack "python-psutil dependency error on ubuntu 16.04.4" [Critical,Fix released] - Assigned to hongbin (hongbin034) | 13:54 |
| *** derekh has quit IRC | 13:54 | |
| *** jmlowe has joined #openstack-nova | 13:54 | |
| dansmith | efried: right I know, | 13:54 |
| efried | not nova-specific, beind discussed in #openstack-dev | 13:54 |
| dansmith | I know | 13:55 |
| efried | okay. | 13:55 |
| *** armaan has quit IRC | 13:55 | |
| dansmith | I'm asking if it's 100%, because I assume it is, | 13:55 |
| dansmith | thus no reason to send anything into recheck or the gate | 13:55 |
| efried | It seems to be. Except for doc fixes. | 13:55 |
| efried | Right. | 13:55 |
| *** alexchadin has joined #openstack-nova | 13:55 | |
| *** armaan has joined #openstack-nova | 13:55 | |
| *** derekh has joined #openstack-nova | 13:56 | |
| mriedem | master is fine | 13:56 |
| efried | dansmith: So yeah, you should hold off +A on the zvm series. | 13:56 |
| efried | but only til the gate is fixed | 13:56 |
| mriedem | https://review.openstack.org/#/q/I8bf80efc04883cd754c19bea0303064080112c6e | 13:56 |
| dansmith | oh as of an hour ago, okay | 13:56 |
| efried | oh, okay, cool. | 13:57 |
| dansmith | all the ones I'm looking at were before that | 13:57 |
| *** r-daneel has quit IRC | 13:57 | |
| *** fabian_ has quit IRC | 13:57 | |
| edleafe | Scheduler subteam meeting in 1 minute in #openstack-meeting-alt | 13:59 |
| *** takashin has joined #openstack-nova | 13:59 | |
| *** ragiman has quit IRC | 14:03 | |
| *** liverpooler has joined #openstack-nova | 14:05 | |
| *** sar has joined #openstack-nova | 14:07 | |
| *** udesale has quit IRC | 14:11 | |
| *** mlavalle has joined #openstack-nova | 14:11 | |
| *** udesale has joined #openstack-nova | 14:12 | |
| M0z122 | mriedem: thanks, will ask to cfriesen | 14:12 |
| *** QianYu has joined #openstack-nova | 14:13 | |
| *** QianYu_ has quit IRC | 14:13 | |
| *** jmlowe has quit IRC | 14:13 | |
| *** felipemonteiro has joined #openstack-nova | 14:15 | |
| *** jmlowe has joined #openstack-nova | 14:16 | |
| *** ragiman has joined #openstack-nova | 14:18 | |
| *** mdbooth has joined #openstack-nova | 14:24 | |
| openstackgerrit | Merged openstack/nova-specs master: Spec for isolating configuration of placement database https://review.openstack.org/552927 | 14:26 |
| *** lpetrut_ has joined #openstack-nova | 14:28 | |
| *** pchavva has quit IRC | 14:29 | |
| *** lpetrut has quit IRC | 14:32 | |
| *** jmlowe has quit IRC | 14:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Stop printing flavor details on successful flavor-delete https://review.openstack.org/561634 | 14:33 |
| *** moshele has quit IRC | 14:33 | |
| *** cdent has quit IRC | 14:42 | |
| *** kosamara has joined #openstack-nova | 14:42 | |
| *** vladikr has quit IRC | 14:42 | |
| *** pchavva has joined #openstack-nova | 14:43 | |
| *** takashin has left #openstack-nova | 14:44 | |
| *** vladikr has joined #openstack-nova | 14:44 | |
| *** sahid has quit IRC | 14:44 | |
| *** QianYu_ has joined #openstack-nova | 14:46 | |
| *** QianYu has quit IRC | 14:46 | |
| *** dpawlik has quit IRC | 14:46 | |
| *** dpawlik has joined #openstack-nova | 14:48 | |
| kosamara | Hi mriedem, I've updated https://review.openstack.org/#/c/555861/ Can you take a look if you have the time? | 14:51 |
| *** sdeath has joined #openstack-nova | 14:52 | |
| mriedem | i'll put it in the browser tab queue | 14:52 |
| *** armaan has quit IRC | 14:53 | |
| *** armaan has joined #openstack-nova | 14:53 | |
| *** alexchadin has quit IRC | 14:55 | |
| *** jmlowe has joined #openstack-nova | 14:56 | |
| *** fabian_ has joined #openstack-nova | 14:56 | |
| *** gcb has joined #openstack-nova | 14:56 | |
| *** namnh has joined #openstack-nova | 14:57 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:59 | |
| *** beekneemech is now known as bnemec | 14:59 | |
| *** rajinir has joined #openstack-nova | 14:59 | |
| *** hongbin_ has joined #openstack-nova | 14:59 | |
| *** felipemonteiro has quit IRC | 15:02 | |
| *** r-daneel has joined #openstack-nova | 15:03 | |
| *** finucannot is now known as stephenfin | 15:03 | |
| *** ttsiouts_ has quit IRC | 15:05 | |
| *** QianYu has joined #openstack-nova | 15:07 | |
| *** QianYu_ has quit IRC | 15:07 | |
| *** cdent has joined #openstack-nova | 15:08 | |
| *** pcaruana has quit IRC | 15:09 | |
| *** kholkina has quit IRC | 15:10 | |
| *** tbachman has quit IRC | 15:11 | |
| *** ragiman has quit IRC | 15:11 | |
| kashyap | Hi folks, wonder if anyone has time to look at why is 'mox' raising UnexpectedMethodCallError with this unit test: http://paste.openstack.org/show/719304/ | 15:13 |
| kashyap | I posted two code fragments the test points to (from guest.py & driver.py in nova/libvirt) | 15:14 |
| *** sahid has joined #openstack-nova | 15:15 | |
| *** ralonsoh__ has quit IRC | 15:15 | |
| * kashyap has even double-checked the parameters of migrateToURI3() and they definitely didn't change in a while. So, "method is called with incorrect parameters" is definitely not one of the reasons for failure | 15:16 | |
| kashyap | mdbooth: Wonder if you have a minute to look at the above (read prior 3 lines of scroll for context)? Last I recall, you made the change to that surrounding code. :-) | 15:18 |
| * mdbooth reads back | 15:18 | |
| kashyap | mdbooth: So, that UnexpectedMethodCallError is raised when "if a method is called with incorrect parameters, or out of the specified order". | 15:19 |
| kashyap | Neither is the case, from my reading. | 15:19 |
| mdbooth | kashyap: What have you changed? | 15:20 |
| mdbooth | kashyap: pastebin your local diff | 15:20 |
| *** armaan has quit IRC | 15:20 | |
| kashyap | Fixing the fall out of bumping versions: https://review.openstack.org/#/c/558783/ | 15:20 |
| *** armaan has joined #openstack-nova | 15:21 | |
| mdbooth | kashyap: I'll fire it up locally | 15:21 |
| * kashyap has fixed the merge conflict locally; the 4 tests that are failing, mostly point to the above usage | 15:21 | |
| kashyap | mdbooth: Then, wait -- let me post the merge-conflict fixed series | 15:21 |
| * mdbooth sighs | 15:22 | |
| mdbooth | I can't pull it right now | 15:22 |
| kashyap | Yeah, it's merge conflict; I fixed it this morning locally. Pushing the new rev in a couple. I'll notify | 15:23 |
| * mdbooth looks for an ethernet cable | 15:23 | |
| efried | kashyap: Note that takashin is doing a whole slew of patches to remove mox. | 15:23 |
| *** ragiman has joined #openstack-nova | 15:24 | |
| kashyap | efried: So, he'll be replacing it w/ Mock, is it? | 15:24 |
| efried | kashyap: Yes. There's a bp for that, so I'm sure he would welcome the help if you'd like to propose a "fix" for this problem that happens to swap out mox for mock. | 15:24 |
| *** tssurya has quit IRC | 15:24 | |
| efried | kashyap: https://review.openstack.org/#/q/project:openstack/nova+branch:master+topic:bp/mox-removal | 15:25 |
| kashyap | efried: I'll see about it. Python unit tests is one my Python weakness area | 15:25 |
| kashyap | First trying to remove all the now-needless compat code due to the version bumps | 15:25 |
| *** tbachman has joined #openstack-nova | 15:25 | |
| efried | kashyap: No pressure :) | 15:26 |
| kashyap | Thanks for the (non-null) pointer | 15:27 |
| mdbooth | efried kashyap: I'd recommend against mixing those 2 changes, though. | 15:28 |
| kashyap | mdbooth: Yeah, I'm strictly following the "One logical change per patch" | 15:28 |
| kashyap | No madness | 15:28 |
| *** mdbooth_ has joined #openstack-nova | 15:28 | |
| kashyap | But appreciate the reminder :-) | 15:28 |
| *** awaugama has joined #openstack-nova | 15:29 | |
| mdbooth_ | kashyap: Don't know if ^^^ sent, but I recommend against mixing those 2 changes. | 15:29 |
| kashyap | mdbooth_: Yep, got sent. And then I wrote: < kashyap> mdbooth: Yeah, I'm strictly following the "One logical change per patch" rule | 15:29 |
| cfriesen | mriedem: M0z122: I think this was the last time I tried to push LVM resize/migration upstream: https://review.openstack.org/#/c/337334/ | 15:30 |
| mdbooth_ | cfriesen: Aside: do you get a performance benefit from lvm? | 15:31 |
| openstackgerrit | Merged openstack/nova master: Add config drive link to api-guide https://review.openstack.org/561113 | 15:32 |
| *** jchhatbar has quit IRC | 15:32 | |
| *** mdbooth has quit IRC | 15:32 | |
| cfriesen | mdbooth_: last time we checked, I think there was a difference, but I don't know that we've checked in a while. Have you seen more recent benchmarks? | 15:32 |
| mdbooth_ | cfriesen: No, but I vaguely recall that it was marginal at best. | 15:33 |
| *** jistr is now known as jistr|mtg | 15:33 | |
| mdbooth_ | cfriesen: Over raw, that is. | 15:33 |
| mdbooth_ | Just wondering if you had data which showed it's worth the bother. | 15:33 |
| M0z122 | cfriesen: many thks will have a look on it | 15:34 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add DifferentHostFilter to enabled_filters default https://review.openstack.org/561651 | 15:35 |
| *** fabian_ has quit IRC | 15:35 | |
| mdbooth_ | cfriesen: Intuitively, I'd expect that if you fallocate a raw file which isn't fragmented and open it O_DIRECT, all of which I think we do, the performance should be the same as LVM, as the kernel is basically doing the same in both cases. | 15:37 |
| *** armaan has quit IRC | 15:38 | |
| mdbooth_ | cfriesen: Numbers beat intuition of course, though :) | 15:38 |
| *** armaan has joined #openstack-nova | 15:38 | |
| *** felipemonteiro_ has quit IRC | 15:40 | |
| *** jdillaman has joined #openstack-nova | 15:41 | |
| *** andreas_s has quit IRC | 15:42 | |
| *** andreas_s has joined #openstack-nova | 15:43 | |
| *** do3meli has quit IRC | 15:45 | |
| *** Nisha_Agarwal has joined #openstack-nova | 15:46 | |
| *** lucasagomes is now known as lucas-brb | 15:47 | |
| *** jpena is now known as jpena|brb | 15:47 | |
| *** jackie-truong has joined #openstack-nova | 15:48 | |
| *** mgagne_ is now known as mgagne | 15:50 | |
| Nisha_Agarwal | Hi team, I have 2 stable pike devstack setups- one stacked last week on a freshly imaged system and another stacked around a month ago. When i was trying "nova boot" for ironic nodes on both the setups i land up in this error http://paste.openstack.org/show/719143/ due to which it send the "delete" event to ironic node as soon as it goes to "wait-call-back" state. On debugging we found that it happens because instance's context is set to | 15:52 |
| Nisha_Agarwal | None at some point. We commented out this line https://github.com/openstack/nova/blob/stable/pike/nova/objects/instance.py#L824 and the deploy goes thru. | 15:52 |
| *** yingjun has quit IRC | 15:55 | |
| *** QianYu has quit IRC | 15:55 | |
| dansmith | melwitt: did you see my comment on that bottom certs patch? I think she reverted the changes she made for you | 15:55 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add (Same|Different)HostFilter to defaults https://review.openstack.org/561651 | 15:57 |
| Nisha_Agarwal | dansmith, could you help me with above? Should i go ahead and open the defect against Nova for the OrphanedObject error reported on stable/pike? | 15:57 |
| *** andreas_s has quit IRC | 15:57 | |
| Nisha_Agarwal | or there is some configuration in the config file which i should be setting so that the issue is not hit? | 15:57 |
| *** tbachman has quit IRC | 15:58 | |
| *** namnh has quit IRC | 16:00 | |
| *** mdbooth_ is now known as mdbooth | 16:01 | |
| dansmith | Nisha_Agarwal: orphaned objects are those without a context set, which prevents them from doing things like loading or saving from the database | 16:01 |
| dansmith | Nisha_Agarwal: obviously no config option should cause/prevent that | 16:02 |
| *** andreas_s has joined #openstack-nova | 16:02 | |
| *** gcb has quit IRC | 16:02 | |
| *** derekh has quit IRC | 16:03 | |
| *** gongysh has quit IRC | 16:03 | |
| cfriesen | mriedem: we're running into problems with the code that adds a uuid to services on the fly. it seems that calling service.save() on a deleted service doesn't work. it chokes down in service_get() | 16:03 |
| *** jistr|mtg is now known as jistr | 16:03 | |
| mriedem | cfriesen: where do we save() a deleted service? | 16:04 |
| cfriesen | Service._from_db_object(), if there's no uuid | 16:04 |
| mriedem | might be a bug that melwitt has recently fixed | 16:04 |
| mriedem | cfriesen: https://review.openstack.org/#/c/555923/ | 16:05 |
| mriedem | is that it? | 16:05 |
| *** dpawlik has quit IRC | 16:05 | |
| dansmith | yeah | 16:06 |
| dansmith | was going to say | 16:06 |
| mriedem | if it's not that, then i will need to see a stacktrace rather than a dead cat | 16:06 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 16:07 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 16:07 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 16:07 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 16:07 |
| cfriesen | mriedem: so previously we were hitting the bug that commit was fixing...so we applied that fix and hit this other problem. Stacktrace at http://paste.openstack.org/show/719308/ | 16:07 |
| Nisha_Agarwal | dansmith, the issue is seen on both the stable pike setups as soon as nova calls this https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L1102 which in turn hits https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L452 | 16:08 |
| mriedem | gibi: welcome back - you might have an opinion on the last comment i left here about notifications https://review.openstack.org/#/c/486204/ | 16:08 |
| dansmith | cfriesen: is that service deleted? | 16:08 |
| mriedem | i'll throw it in the notification meeting | 16:08 |
| cfriesen | dansmith: yes | 16:08 |
| dansmith | cfriesen: that's why, but I thought we determined it shouldn't even be processing deleted services.. did you tweak something in there? | 16:08 |
| *** dpawlik has joined #openstack-nova | 16:09 | |
| cfriesen | dansmith: the sequence of events was: boot instance on compute node, migrate instance, delete compute node and service, upgrade to pike, create new compute node with same name, migrate instance back to that compute node. | 16:09 |
| cfriesen | dansmith: so now when we look up the service by name, there are two matching entries, one of which is deleted | 16:09 |
| dansmith | cfriesen: but we're using model_query which should filter out the deleted one, IIRC | 16:10 |
| openstackgerrit | Jay Pipes proposed openstack/nova-specs master: Numbered request groups use different providers https://review.openstack.org/560974 | 16:10 |
| *** QianYu has joined #openstack-nova | 16:10 | |
| mriedem | cfriesen: i need more of the stacktrace | 16:10 |
| mriedem | not the middle of it | 16:10 |
| stephenfin | bauzas: Want to weigh in on this comment? https://review.openstack.org/#/c/379525/9/nova/scheduler/weights/cpu.py | 16:11 |
| cfriesen | mriedem: will try to get it. | 16:11 |
| stephenfin | bauzas: We don't do this for RAM or disk. Should we? | 16:11 |
| *** gjayavelu has joined #openstack-nova | 16:12 | |
| mriedem | stephenfin: the core filter does | 16:12 |
| mriedem | vcpus_total = host_state.vcpus_total * cpu_allocation_ratio | 16:12 |
| *** gyee has joined #openstack-nova | 16:13 | |
| mriedem | stephenfin: also, it looks like the HostState object takes into account reserved_host_cpus? | 16:13 |
| *** dpawlik has quit IRC | 16:14 | |
| cfriesen | mriedem: I think the core/ram/disk filters account for allocation ratio (though disk is wonky), but maybe the weighers don't? | 16:14 |
| stephenfin | mriedem: *don't take into account? | 16:15 |
| mriedem | yeah doens't take into account | 16:15 |
| mriedem | the inventory on the resource provider for the compute node in placement accounts for reserved values | 16:15 |
| mriedem | but i don't see that on the HostState object or it's wrapped compute node | 16:16 |
| stephenfin | mriedem: Line 1136 of nova/compute/resource_tracker.py ? | 16:16 |
| cfriesen | looks like for ram and disk they just weigh them based on the "free" amount | 16:16 |
| stephenfin | (_update_usage_from_instances function) | 16:16 |
| stephenfin | So I'm guessing it gets it from the ComputeNode object | 16:16 |
| mriedem | stephenfin: hmm, yeah, but that's weird. | 16:17 |
| mriedem | cn.vcpus_used = CONF.reserved_host_cpus | 16:17 |
| mriedem | that's not accounting for the number of instances running on the ohst | 16:18 |
| mriedem | *host | 16:18 |
| stephenfin | mriedem: We do that at the end of the function with the call to '_update_usage_from_instance', I suspect | 16:18 |
| stephenfin | Which presumably appends rather than overwrites | 16:19 |
| stephenfin | cfriesen: Hmm, so as we don't have a "free"-style attribute for CPUs I guess they're correct and we've to figure that out ourselves | 16:19 |
| stephenfin | Or add such an attribute | 16:19 |
| *** QianYu has quit IRC | 16:20 | |
| mriedem | stephenfin: yeah looks like _update_usage() | 16:20 |
| *** andreas_s has quit IRC | 16:20 | |
| mriedem | which also takes into account overhead from the virt driver | 16:21 |
| mriedem | vcpus_usage += overhead.get('vcpus', 0) | 16:21 |
| mriedem | ok left some comments in your change for reference | 16:23 |
| stephenfin | mriedem: Excellent. Ta | 16:23 |
| mriedem | i won't profess to know what the right answer is | 16:23 |
| mriedem | it does match this though https://github.com/openstack/nova/blob/fb0b785169e5e422b06e82f2eb58e68f6d2008b3/nova/compute/resource_tracker.py#L786 | 16:23 |
| *** lbragstad has quit IRC | 16:24 | |
| * stephenfin goes to investigate | 16:24 | |
| *** QianYu_ has joined #openstack-nova | 16:24 | |
| mriedem | this is the claim code | 16:24 |
| mriedem | https://github.com/openstack/nova/blob/fb0b785169e5e422b06e82f2eb58e68f6d2008b3/nova/compute/claims.py#L185 | 16:24 |
| mriedem | which is what ultimately matters | 16:24 |
| *** andreas_s has joined #openstack-nova | 16:25 | |
| *** tbachman has joined #openstack-nova | 16:26 | |
| mriedem | the fun part is, | 16:26 |
| mriedem | if you're using placement, you shouldn't enable the CoreFilter, which means the vcpus limit isn't passed from scheduler to compute, and no vcpus claim is actually done in the resource tracker | 16:26 |
| *** hemna_ has joined #openstack-nova | 16:26 | |
| cfriesen | mriedem: here's the full trace: http://paste.openstack.org/show/719309/ | 16:26 |
| mriedem | the only actual RT claim we should ever have, if you're using placement, right now is for numa | 16:26 |
| mriedem | and maybe pci? | 16:26 |
| *** ejat has quit IRC | 16:27 | |
| *** ejat has joined #openstack-nova | 16:27 | |
| *** ejat has quit IRC | 16:27 | |
| *** ejat has joined #openstack-nova | 16:27 | |
| mriedem | cfriesen: that's the exact same thing that melwitt fixed | 16:27 |
| cfriesen | mriedem: no, melwitt's change was a reader->writer context thing | 16:27 |
| *** jmlowe has quit IRC | 16:28 | |
| mriedem | hmm, so more similar to Ic2f239f634f917a5771b0401a5073546c710c036 | 16:29 |
| mriedem | except in that case the instance is deleted but the service isn't | 16:29 |
| *** derekh has joined #openstack-nova | 16:30 | |
| mriedem | dansmith: didn't you recently have a read_deleted thing in the Instance object for evacuate? | 16:30 |
| dansmith | mriedem: idk | 16:30 |
| mriedem | https://github.com/openstack/nova/commit/619754f5c836ed1b58c807138836e6cf5a4e6904#diff-1b01e8e37be9d889295e5333b7857139 | 16:31 |
| *** udesale has quit IRC | 16:31 | |
| mriedem | cfriesen: https://review.openstack.org/#/q/Ide6cc5bb1fce2c9aea9fa3efdf940e8308cd9ed0 | 16:31 |
| *** QianYu_ has quit IRC | 16:32 | |
| *** jpena|brb is now known as jpena|off | 16:32 | |
| mriedem | maybe you have that already... | 16:32 |
| *** hemna_ has quit IRC | 16:32 | |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Bump MIN_{LIBVIRT,QEMU}_VERSION for "Rocky" https://review.openstack.org/558783 | 16:34 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_{LIBVIRT_KVM,QEMU}_S390_VERSION https://review.openstack.org/560363 | 16:34 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_KVM_PPC64_VERSION https://review.openstack.org/560364 | 16:34 |
| openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Drop MIN_LIBVIRT_VIRTUOZZO_VERSION https://review.openstack.org/560365 | 16:34 |
| cfriesen | mriedem: nope, don't have it. will try it out. thanks. | 16:34 |
| kashyap | mdbooth: ^ If you want to fetch now. | 16:34 |
| *** mdbooth has quit IRC | 16:38 | |
| stephenfin | mriedem: If I request 30 CPUs and I only have 22 available, I get a negative number for https://github.com/openstack/nova/blob/fb0b785169e5e422b06e82f2eb58e68f6d2008b3/nova/compute/resource_tracker.py#L786 | 16:39 |
| *** ragiman has quit IRC | 16:39 | |
| stephenfin | mriedem: Sounds like a bug to me at least | 16:39 |
| *** jpena|off is now known as jpena | 16:39 | |
| *** lpetrut_ has quit IRC | 16:39 | |
| *** andreas_s has quit IRC | 16:39 | |
| mriedem | stephenfin: if you request 30 CPUs and the compute node is reporting it only has 22, then the scheduler should filter out that host and you never hit that compute, right? | 16:40 |
| cfriesen | stephenfin: I think all of those should factor in allocation ratio somehow | 16:40 |
| cfriesen | mriedem: not if cpu_allocation_ratio is 2 | 16:40 |
| stephenfin | mriedem: I've 22 * 16 | 16:40 |
| stephenfin | At least, assuming I'm understanding overcommit correctly | 16:41 |
| stephenfin | The only thing it should fail on is a single instance having > 22 vCPUs | 16:41 |
| *** yamamoto has quit IRC | 16:41 | |
| stephenfin | (No self-overcommit) | 16:41 |
| mriedem | sure, misunderstanding. then i guess that's why the CoreFilter cares about allocation ratio yeah? https://github.com/openstack/nova/blob/fb0b785169e5e422b06e82f2eb58e68f6d2008b3/nova/scheduler/filters/core_filter.py#L49 | 16:42 |
| cfriesen | when we moved away from per-host-aggregate allocation ratios, it might have made sense to remove all knowledge of allocation ratios from anywhere but the compute node itself and just have it lie about how many it has. :) | 16:42 |
| mriedem | and placement for that matter | 16:42 |
| stephenfin | mriedem: I guess so, yeah. Fair play to Rong Han for spotting it | 16:42 |
| mriedem | maybe he already has that weighter out of tree and is just comparing :) | 16:42 |
| stephenfin | cfriesen: Agreed. Do wonder how that would affect CPU pinning though. Don't we use that? | 16:43 |
| cfriesen | how do we support disk allocation ratios with network disks, when different compute nodes could have different allocation ratios configured? | 16:43 |
| cfriesen | stephenfin: pinning ignores the ratios, always 1:1 | 16:43 |
| stephenfin | mriedem: I'd put money on it :) | 16:43 |
| * mriedem goes to snow blow the driveway | 16:44 | |
| stephenfin | cfriesen: Not in the ComputeNode was reporting N * ratio free CPUs though, right? | 16:44 |
| mriedem | (again) | 16:44 |
| stephenfin | Although that's neither here nor there. I'll fix this up here and then fix up the log | 16:44 |
| cfriesen | stephenfin: pinning doesn't use that count | 16:44 |
| stephenfin | which seems broken | 16:44 |
| stephenfin | cfriesen: Ah, potential clean up so | 16:44 |
| *** andreas_s has joined #openstack-nova | 16:44 | |
| cfriesen | stephenfin: at least, I'm about 90% sure it doesn't. :) | 16:44 |
| *** gjayavelu has quit IRC | 16:45 | |
| *** yamamoto has joined #openstack-nova | 16:46 | |
| *** fragatina has joined #openstack-nova | 16:46 | |
| *** sahid has quit IRC | 16:46 | |
| cfriesen | jaypipes: do you know how we handle disk_allocation_ratio as a per-compute-node config option when we're using shared disk resource providers? | 16:48 |
| * stephenfin heads out into the rain | 16:49 | |
| * cfriesen still has snow | 16:49 | |
| *** Nisha_Agarwal has quit IRC | 16:49 | |
| *** hemna_ has joined #openstack-nova | 16:50 | |
| *** yamamoto has quit IRC | 16:51 | |
| *** armaan has quit IRC | 16:52 | |
| *** gjayavelu has joined #openstack-nova | 16:56 | |
| *** andreas_s has quit IRC | 16:58 | |
| *** abhishekk has joined #openstack-nova | 16:58 | |
| *** derekh has quit IRC | 17:01 | |
| *** yamamoto has joined #openstack-nova | 17:01 | |
| *** calebb has quit IRC | 17:04 | |
| *** mgoddard has quit IRC | 17:05 | |
| *** mdnadeem has quit IRC | 17:05 | |
| *** yamamoto has quit IRC | 17:06 | |
| *** moshele has joined #openstack-nova | 17:09 | |
| *** yamamoto has joined #openstack-nova | 17:11 | |
| *** yamamoto has quit IRC | 17:11 | |
| *** mvk has quit IRC | 17:13 | |
| *** calebb has joined #openstack-nova | 17:15 | |
| *** AlexeyAbashkin has quit IRC | 17:17 | |
| *** elmaciej has quit IRC | 17:17 | |
| *** diegows has quit IRC | 17:18 | |
| *** jpena is now known as jpena|off | 17:19 | |
| *** yamahata has quit IRC | 17:20 | |
| *** felipemonteiro has joined #openstack-nova | 17:21 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 17:22 |
| *** hemna_ has quit IRC | 17:23 | |
| *** gjayavelu has quit IRC | 17:26 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Parse forbidden in extra_specs https://review.openstack.org/561677 | 17:28 |
| *** sridharg has quit IRC | 17:32 | |
| *** r-daneel has quit IRC | 17:34 | |
| *** r-daneel has joined #openstack-nova | 17:35 | |
| jaypipes | cfriesen: nobody is yet using shared disk resource providers, so I don't know :) | 17:37 |
| *** mgoddard has joined #openstack-nova | 17:41 | |
| *** jmlowe has joined #openstack-nova | 17:42 | |
| *** elmaciej has joined #openstack-nova | 17:42 | |
| *** lbragstad has joined #openstack-nova | 17:43 | |
| *** felipemonteiro_ has joined #openstack-nova | 17:45 | |
| *** abhishekk has quit IRC | 17:45 | |
| *** lpetrut_ has joined #openstack-nova | 17:46 | |
| *** elmaciej has quit IRC | 17:48 | |
| *** felipemonteiro has quit IRC | 17:48 | |
| *** wolverineav has joined #openstack-nova | 17:49 | |
| *** gjayavelu has joined #openstack-nova | 17:52 | |
| *** yamahata has joined #openstack-nova | 17:54 | |
| *** yamamoto has joined #openstack-nova | 17:57 | |
| *** hemna_ has joined #openstack-nova | 18:01 | |
| *** Tom-Tom has joined #openstack-nova | 18:01 | |
| *** yamamoto has quit IRC | 18:02 | |
| cfriesen | jaypipes: seems like we'd need to have the allocation ratio associated with the resource provider itself rather than the compute node | 18:04 |
| *** hemna_ has quit IRC | 18:07 | |
| cdent | cfriesen: yes, that would be the idea. efried has done some preliminary libvirt work, lemme find the link | 18:07 |
| cdent | also you aware of #openstack-placement | 18:07 |
| cdent | cfriesen: https://review.openstack.org/#/c/560459/ | 18:07 |
| efried | cdent, cfriesen: In case you didn't catch the last nova meeting, bhagyashris agreed to take these patches over and fix up the tests. | 18:08 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: tests for alloc candidates with nested and traits https://review.openstack.org/531899 | 18:09 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: resource requests for nested providers https://review.openstack.org/554529 | 18:09 |
| jaypipes | cfriesen: see https://review.openstack.org/#/c/544683/ | 18:10 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Document how to disable notifications https://review.openstack.org/561684 | 18:13 |
| *** arvindn05 has left #openstack-nova | 18:15 | |
| *** arvindn0_ has joined #openstack-nova | 18:15 | |
| *** arvindn0_ has left #openstack-nova | 18:16 | |
| dansmith | sean-k-mooney: around? | 18:16 |
| *** arvindn0_ has joined #openstack-nova | 18:16 | |
| *** arvindn0_ has quit IRC | 18:18 | |
| *** wolverineav has quit IRC | 18:18 | |
| *** arvindn05 has joined #openstack-nova | 18:18 | |
| *** wolverineav has joined #openstack-nova | 18:18 | |
| *** arvindn051 has joined #openstack-nova | 18:18 | |
| mriedem | cdent: thanks for https://pypi.org/project/wsgi_intercept/ - i'll see about getting g-r updated | 18:21 |
| *** arvindn05 has quit IRC | 18:21 | |
| *** arvindn05 has joined #openstack-nova | 18:21 | |
| mriedem | although i likely can't use that in this change until later since we might backport this patch | 18:23 |
| cdent | yeah, was worth finding anyway | 18:23 |
| *** liverpooler has quit IRC | 18:26 | |
| *** amoralej is now known as amoralej|off | 18:28 | |
| openstackgerrit | Merged openstack/nova master: Don't log a warning for InstanceNotFound with deleted VIFs https://review.openstack.org/554591 | 18:29 |
| *** avolkov has quit IRC | 18:31 | |
| *** eharney has quit IRC | 18:32 | |
| sean-k-mooney | dansmith: i was on a call but yes | 18:33 |
| dansmith | sean-k-mooney: hey, so we had a question on friday and I think you're the right person to ask | 18:33 |
| dansmith | sean-k-mooney: we still have non-os_vif handling of vhostuser interfaces in vif.pym | 18:34 |
| sean-k-mooney | somthing about not implemented error for os-vif | 18:34 |
| dansmith | but it's not clear to me/us if that is still needed or not | 18:34 |
| dansmith | yeah | 18:34 |
| sean-k-mooney | dansmith: that is used by both vpp and snabswitch | 18:34 |
| dansmith | okay | 18:34 |
| dansmith | why? | 18:34 |
| sean-k-mooney | in both cases vhost-user is not being used with ovs so they dont need to have somting plug it into a vswitch | 18:35 |
| sean-k-mooney | in the vpp case i belive they have an agent on the plathform that poll for the creation fo the socket and just connects when it shows up | 18:35 |
| *** Zames has joined #openstack-nova | 18:36 | |
| dansmith | so is it always going to be that way? | 18:36 |
| sean-k-mooney | dansmith: we could add a new plugin to os-vif if you want to move it out of tree | 18:36 |
| *** tesseract has quit IRC | 18:36 | |
| dansmith | seems like it would be less confusing | 18:36 |
| dansmith | maybe a comment about who/what uses that path would be nice | 18:36 |
| dansmith | regardless, can you have a look over https://review.openstack.org/#/c/484997 ? | 18:37 |
| sean-k-mooney | we didnt to that initally as they are not reference plugings but its basically a noop so i dont really see the harm in a generic_vhost_user pluging in os-vif | 18:37 |
| *** _pewp_ has quit IRC | 18:37 | |
| sean-k-mooney | oh the queue lenght stuff sure ill take a look now | 18:38 |
| dansmith | thanks | 18:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 18:39 |
| *** harlowja has joined #openstack-nova | 18:40 | |
| *** Zames has quit IRC | 18:40 | |
| *** _pewp_ has joined #openstack-nova | 18:42 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:44 | |
| *** eharney has joined #openstack-nova | 18:47 | |
| efried | bauzas: still around? | 18:47 |
| *** tssurya has joined #openstack-nova | 18:48 | |
| *** AlexeyAbashkin has quit IRC | 18:49 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:49 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: fix wrong driver name for vhostuser interface https://review.openstack.org/561562 | 18:50 |
| *** mvk has joined #openstack-nova | 18:52 | |
| *** hemna_ has joined #openstack-nova | 18:53 | |
| *** hemna_ has quit IRC | 18:53 | |
| *** hemna_ has joined #openstack-nova | 18:53 | |
| melwitt | dansmith: yeah, saw your comment and -1ed it earlier this morning to add visibility to the fact that the changes were reverted | 18:53 |
| *** mgoddard has quit IRC | 18:53 | |
| dansmith | melwitt: ack | 18:54 |
| *** lajoskatona has quit IRC | 18:56 | |
| *** yamamoto has joined #openstack-nova | 18:58 | |
| *** jmlowe has quit IRC | 18:58 | |
| *** AlexeyAbashkin has quit IRC | 18:59 | |
| mriedem | melwitt: i'm not sure if you want a simple specless blueprint for this or just let it slide as a wishlist bug fix https://review.openstack.org/#/c/555861/ - note we wouldn't backport that either way | 19:01 |
| *** dpawlik has joined #openstack-nova | 19:01 | |
| *** tbachman has quit IRC | 19:01 | |
| *** yamamoto has quit IRC | 19:02 | |
| melwitt | cfriesen, mriedem, dansmith: looking at the code, it makes sense what's happening. the service was deleted while the instance still referred to it, then when you list the instance, it tries to generate a UUID for the service (that is deleted) and the service_update method tries to 'get' the service record by id and it gets ServiceNotFound because model_query will automatically filter out deleted things https://github.com/openstack/nova/ | 19:02 |
| melwitt | blob/master/nova/db/sqlalchemy/api.py#L554 | 19:02 |
| *** tmcm has joined #openstack-nova | 19:02 | |
| melwitt | https://github.com/openstack/nova/blob/master/nova/db/sqlalchemy/api.py#L554 | 19:02 |
| tmcm | is it possible to run mitaka-based compute nodes with a queens nova-scheduler/placement? | 19:04 |
| tmcm | i'm looking at https://docs.openstack.org/nova/latest/user/placement.html but it's not clear to me from what i read there | 19:04 |
| melwitt | I would think if you're deleting services out from under live instances, after you create new services to replace them, wouldn't you have to manually associate the new service with existing instances? else it seems like other bad things could happen | 19:04 |
| *** openstackgerrit has quit IRC | 19:05 | |
| dansmith | they're only associated by name, right? | 19:05 |
| melwitt | the db api stuff implied the backref is by service id | 19:05 |
| melwitt | but lemme re-look | 19:06 |
| *** dpawlik has quit IRC | 19:06 | |
| dansmith | I don't think we have a service_id on instance, | 19:06 |
| dansmith | I thought I looked | 19:06 |
| dansmith | although I was looking at computenode specifically | 19:06 |
| mriedem | i wonder why "from_db_object" is public in their stacktraces....totally unrelated | 19:06 |
| melwitt | yeah looks like backref by host and binary | 19:06 |
| melwitt | so how did the service_update get the service id ... | 19:07 |
| dansmith | it would be if we read_deleted when we looked it up | 19:07 |
| dansmith | and got the first older deleted one by name | 19:08 |
| dansmith | I haven't looked back into his latest stack trace, fyi | 19:08 |
| melwitt | okay. yeah, the stacktrace shows ServiceNotFound being raised from service_update as service_update is trying to update via id and the id is not found because model_query | 19:08 |
| dansmith | right | 19:09 |
| mriedem | "the sequence of events was: boot instance on compute node, migrate instance, delete compute node and service, upgrade to pike, create new compute node with same name, migrate instance back to that compute node." | 19:09 |
| *** openstackgerrit has joined #openstack-nova | 19:10 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: placement: resource requests for nested providers https://review.openstack.org/554529 | 19:10 |
| melwitt | I haven't made the connection yet but somehow the service object associated with the instance is the deleted one. which I thought would be expected if you deleted the service out from under the instance. like, what would ever update the instance to point at a new service? I guess you say by name, which would find more than one matching service, and it's happening to find the deleted one first? | 19:10 |
| dansmith | melwitt: it can't be | 19:10 |
| dansmith | because it's only by name | 19:10 |
| dansmith | I didn't see where they ened up starting down this path, I thought it was in the "upgrade to pike" section, | 19:10 |
| melwitt | but two have the same name, the deleted one and the new one | 19:10 |
| dansmith | and that they were implying we were handling deleted services in the migration | 19:11 |
| dansmith | right | 19:11 |
| mriedem | also waiting to hear if they have this https://review.openstack.org/#/c/545988/ | 19:11 |
| melwitt | I haven't yet connected where the query is happening that would get the two returned | 19:11 |
| melwitt | because like you all said earlier, that should be filtering out deleted rows, we would expect | 19:11 |
| dansmith | yeah | 19:11 |
| mriedem | if they have https://review.openstack.org/#/c/545988/ they will query deleted records too | 19:12 |
| dansmith | mriedem: oh I didn't get the connection this morning you were making | 19:12 |
| *** Tom-Tom has quit IRC | 19:12 | |
| dansmith | mriedem: that we're using a read_deleted context to get the instance and thus the service too? | 19:12 |
| mriedem | yes | 19:12 |
| * dansmith really isn't paying attention | 19:12 | |
| dansmith | gotcha that makes sense | 19:12 |
| melwitt | mriedem: I think we should probably go with a specless blueprint on that one, so it gets bundled with the other new features | 19:13 |
| mriedem | ack | 19:14 |
| melwitt | thanks | 19:14 |
| mriedem | trying to think of a test to recreate cfriesen's issue, could start two compute services, create the instance on one, migrate to the other, delete the old compute service/node, fake the upgrade to pike by deleting the uuid on the 2 services, start a new compute service with the same name as the old one, and then migrate the instance back to the other host | 19:19 |
| mriedem | in his stacktrace, he's not doing a migration, simply listing instances | 19:19 |
| mriedem | which unconditionally joins on services https://github.com/openstack/nova/blob/f0400313f30843fba114584f582b93f7cb5e5e48/nova/api/openstack/compute/servers.py#L295 | 19:20 |
| melwitt | sounds like that should do it | 19:20 |
| *** moshele has quit IRC | 19:20 | |
| * melwitt wonders if there are any other gotchas to the read_deleted='yes' for lazy-loading instance attributes | 19:21 | |
| mriedem | cfriesen: have you reported an upstream bug for this issue? | 19:21 |
| *** sdeath has quit IRC | 19:22 | |
| *** dpawlik has joined #openstack-nova | 19:23 | |
| *** dpawlik has quit IRC | 19:23 | |
| mriedem | bauzas: https://bugs.launchpad.net/nova/+bug/1764460 | 19:23 |
| openstack | Launchpad bug 1764460 in OpenStack Compute (nova) queens "Cannot hard reboot an instance in error state" [High,Confirmed] | 19:23 |
| *** dpawlik has joined #openstack-nova | 19:24 | |
| mriedem | looks like a regression in the libvirt driver due to the mdev stuff in queens | 19:24 |
| mriedem | this is fun https://bugs.launchpad.net/nova/+bug/1764385 | 19:25 |
| openstack | Launchpad bug 1764385 in OpenStack Compute (nova) "no intimation to the admin that nova-api is stopped during execution of polling compute" [Undecided,Incomplete] | 19:25 |
| melwitt | on the libvirt one, looks like a simple fix at least | 19:26 |
| *** arvindn05 has quit IRC | 19:27 | |
| melwitt | I saw this old bug recently, evacuations that failed to schedule end up getting nuked during nova-compute restart https://bugs.launchpad.net/nova/+bug/1583504 | 19:27 |
| openstack | Launchpad bug 1583504 in OpenStack Compute (nova) "The instances which didn't be evacuated will be destroyed when the nova-compute service is restarted." [Undecided,Confirmed] - Assigned to Margarita Shakhova (shakhova-margarita) | 19:27 |
| melwitt | because the logic assumes that 'accepted' migration means it succeeded. I'm not sure what the original reasoning was for including 'accepted' in the logic (along with 'done') | 19:28 |
| mriedem | hmm, is that from before the evac robustification to use migration records? | 19:28 |
| mriedem | oh | 19:28 |
| mriedem | that recently changed, the logic in the compute i mean, | 19:29 |
| mriedem | it was discussed at the ptg in denver | 19:29 |
| melwitt | this was the proposed fix https://review.openstack.org/#/c/318731/2/nova/compute/manager.py | 19:29 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 19:29 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 19:29 |
| melwitt | orly | 19:29 |
| * melwitt looks | 19:29 | |
| mriedem | i'm thinking of this i think https://review.openstack.org/#/c/499237 | 19:30 |
| melwitt | code looks the same AFAICT https://github.com/openstack/nova/blob/master/nova/compute/manager.py#L640 | 19:30 |
| melwitt | oh, okay. that should help then | 19:30 |
| mriedem | right, if scheduling fails, set the migration status to 'error' | 19:31 |
| mriedem | so when restarting the compute, the migration is not 'accepted' | 19:31 |
| mriedem | or 'done' | 19:31 |
| melwitt | maybe that bug could be considered closed with that fix, but we'd probably wanna write a test to make sure | 19:31 |
| melwitt | yep, gotcha | 19:31 |
| mriedem | i think you'd just have to update this test https://review.openstack.org/#/c/499237/4/nova/tests/functional/regressions/test_bug_1713783.py | 19:32 |
| mriedem | to restart the compute | 19:32 |
| *** cdent has quit IRC | 19:32 | |
| *** tbachman has joined #openstack-nova | 19:32 | |
| melwitt | that would be sweet if that's all it'd take | 19:33 |
| cfriesen | mriedem: I haven't reported an upstream bug yet, was trying to figure out if it was an upstream issue or not. My coworker reports that with the two fixes from stable/pike it doesn't fix the problem immediately but after some time the record has been updated in the DB. | 19:33 |
| mriedem | after some time...like running the online data migrations? | 19:34 |
| mriedem | or archiving deleted rows? | 19:34 |
| cfriesen | melwitt: will the "read_deleted=yes" context change from https://git.openstack.org/cgit/openstack/nova/commit/?id=1407079d4008c6304799dd83f5bf4ba505d8e438 override the default model_query behaviour? | 19:34 |
| mriedem | yes | 19:34 |
| mriedem | context.read_deleted defaults to 'no' | 19:34 |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: Report the allocated size of preallocated file based disks https://review.openstack.org/561704 | 19:35 |
| *** imacdonn has quit IRC | 19:35 | |
| *** imacdonn has joined #openstack-nova | 19:35 | |
| *** tbachman_ has joined #openstack-nova | 19:36 | |
| cfriesen | mriedem: my coworker says he didn't do any online data migrations or archiving rows or anything "interesting". He suspects something querying the instances. | 19:37 |
| *** tbachman has quit IRC | 19:37 | |
| *** tbachman_ is now known as tbachman | 19:37 | |
| cfriesen | running the online data migration doesn't normally affect deleted services, but applying "with utils.temporary_mutation(self._context, read_deleted='yes'):" to the online migration apparently fixed it all right up. | 19:38 |
| melwitt | well, if the ordering of the returned dupe services (deleted one and the new one) is random, then eventually it would work fine if the non-deleted one was first, right? | 19:38 |
| mriedem | cfriesen: ok so now you're saying that it's fixed? | 19:38 |
| *** moshele has joined #openstack-nova | 19:38 | |
| mriedem | i.e. https://review.openstack.org/#/c/545988/ | 19:39 |
| mriedem | since everything is going to happen to that service with that mutated context, i expect it will update the deleted service with a uuid | 19:39 |
| melwitt | yeah like, eventually maybe the non-deleted one is returned "first" in the list and then it goes through with the update and then there won't be a problem from then on | 19:39 |
| mriedem | i just need to know if i should spend time trying to write a recreate test | 19:40 |
| cfriesen | mriedem: that commit did not immediately fix things (he still hit some ServiceNotFound errors after applying it), but eventually the service did get a uuid | 19:40 |
| melwitt | I think there's still a bug where for lazy-load of services it needs to not read deleted ones | 19:40 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Implement certificate_utils https://review.openstack.org/479949 | 19:41 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Plumb trusted_certs through libvirt driver image paths https://review.openstack.org/561262 | 19:41 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 19:41 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 19:41 |
| melwitt | I hope services don't allow dupe by name if they are both non-deleted | 19:41 |
| cfriesen | melwitt: arguably for the instance->service mapping the bug is that it maps by name | 19:41 |
| melwitt | because then there would be a problem anyways | 19:41 |
| mriedem | https://review.openstack.org/#/c/545988/ | 19:42 |
| melwitt | cfriesen: yeah, true. | 19:42 |
| mriedem | oops | 19:42 |
| mriedem | schema.UniqueConstraint("host", "topic", "deleted", | 19:42 |
| mriedem | name="uniq_services0host0topic0deleted"), | 19:42 |
| mriedem | schema.UniqueConstraint("host", "binary", "deleted", | 19:42 |
| mriedem | name="uniq_services0host0binary0deleted"), | 19:42 |
| tmcm | sorry to repeat myself, but is it possible to run mitaka-based compute nodes with a queens nova-scheduler/placement? i'm looking at https://docs.openstack.org/nova/latest/user/placement.html but it's not clear to me from what i read there | 19:42 |
| mriedem | tmcm: no | 19:42 |
| tmcm | ok, thanks | 19:42 |
| mriedem | it might work, but not by intention | 19:42 |
| mriedem | we support n-1 computes | 19:42 |
| mriedem | so controller can be queens and then computes can be pike+ | 19:42 |
| tmcm | understood, thanks | 19:43 |
| *** Tom-Tom has joined #openstack-nova | 19:43 | |
| mriedem | at certain points we randomly delete compat code that's older than n-1 | 19:43 |
| *** moshele has quit IRC | 19:43 | |
| *** awaugama has quit IRC | 19:43 | |
| mriedem | tmcm: this should be a bit more clear https://docs.openstack.org/nova/latest/user/upgrade.html#rolling-upgrade-process | 19:44 |
| cfriesen | mriedem: for a recreate test could we add a new DB entry with a deleted service that has the same name as the existing service? | 19:45 |
| *** cdent has joined #openstack-nova | 19:45 | |
| tmcm | mriedem: got it. i'd actually worked through that document already and my queens controller was happy from a db standpoint, i was just stuck at: | 19:46 |
| tmcm | +-------------------------------------------------------------------+ | 19:46 |
| tmcm | | Check: Resource Providers | | 19:46 |
| tmcm | | Result: Warning | | 19:46 |
| tmcm | | Details: There are no compute resource providers in the Placement | | 19:46 |
| tmcm | | service but there are 80 compute nodes in the deployment. | | 19:46 |
| tmcm | | This means no compute nodes are reporting into the | | 19:46 |
| tmcm | | Placement service and need to be upgraded and/or fixed. | | 19:46 |
| tmcm | | See | | 19:46 |
| tmcm | | https://docs.openstack.org/nova/latest/user/placement.html | | 19:46 |
| tmcm | | for more details. | | 19:46 |
| tmcm | +-------------------------------------------------------------------+ | 19:46 |
| tmcm | on the compute side | 19:46 |
| openstackgerrit | Merged openstack/nova master: List instances performace optimization https://review.openstack.org/507854 | 19:46 |
| tmcm | well, from the compute standpoint, i mean | 19:46 |
| tmcm | so if there was any way to force the mitaka compute nodes to use the placement service, i might be OK | 19:47 |
| artom | tmcm, there isn't, the code didn't exist in mitaka :) | 19:47 |
| *** Tom-Tom has quit IRC | 19:47 | |
| tmcm | got it | 19:47 |
| artom | (Err, right?) | 19:47 |
| tmcm | thanks to both of you | 19:47 |
| tmcm | i didn't see it in there | 19:47 |
| artom | This self-doubt thing is cripling | 19:48 |
| mriedem | artom: yes, new in newton | 19:49 |
| mriedem | optional in newton, required starting in ocata | 19:49 |
| cfriesen | mriedem: apparently "nova show" on the instance will result in adding the uuid onto the service, but "nova list" hits the ServiceNotFound error | 19:49 |
| artom | mriedem, thank you :) | 19:49 |
| melwitt | you'd have to upgrade your compute nodes to at least newton to make them report in to placement. might even be ocata, not sure | 19:49 |
| mriedem | cfriesen: those should both go down the same code path as far as i know, but maybe not | 19:49 |
| melwitt | I think it would be random depending on the ordering of the returned dupe services | 19:50 |
| melwitt | nevermind, it looks like it should try to generate a UUID for both services that match the instance via host, so it should fail reliably | 19:54 |
| sean-k-mooney | dansmith: just finished looking at the patch. the broad strokes look fine. i left some nits inline. main concern is the name attribute of the driver field. i think we should just not set it for safty across livemirate in the future. anyway i left comments in the patch but over all it looks ok | 19:54 |
| *** wolverineav has quit IRC | 19:54 | |
| sean-k-mooney | dansmith: i might try an apply it locally tomorow and see if i have any issues. anyway im going to head for the evening. | 19:56 |
| dansmith | sean-k-mooney: okay just to be sure, you know we're already setting it to vhost in almost all cases now right? | 19:57 |
| dansmith | sean-k-mooney: it's done in _get_virtio_mq_settings() | 19:57 |
| sean-k-mooney | dansmith: am i dont think we do that for vhost-user ports however | 19:57 |
| dansmith | sean-k-mooney: we do | 19:57 |
| dansmith | sean-k-mooney: even though it is wrong | 19:57 |
| dansmith | that's the point :) | 19:57 |
| sean-k-mooney | when we request vhost multiqueue only right | 19:58 |
| dansmith | no | 19:58 |
| dansmith | if we're in kvm, and MODEL_VIRTIO, we call that method and it sets it to vhost always I think | 19:58 |
| *** yamamoto has joined #openstack-nova | 19:58 | |
| TheJulia | efried: your python-ironicclient patch, Would you be okay if I updated the commit message? | 19:58 |
| dansmith | sean-k-mooney: oh, actually, I guess you're right | 19:59 |
| dansmith | sean-k-mooney: sahid said the opposite and I looked but didn't see | 19:59 |
| sean-k-mooney | dansmith: oh we did not used too | 19:59 |
| dansmith | sean-k-mooney: so yeah, this does change it to be more often, which is what I was trying to avoid when we talked last week | 19:59 |
| sean-k-mooney | dansmith: ya this is a normal vhost-user xml http://paste.openstack.org/show/719324/ | 19:59 |
| dansmith | but I missed that we only do it for multiqueue | 19:59 |
| efried | TheJulia: Sure, you can do whatever you like with it. I would be happy to keep working on it too if you prefer. (I saw you made a comment, but I haven't gotten to reading it yet.) | 19:59 |
| dansmith | sean-k-mooney: so good call on that, thanks | 19:59 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add wrinkle to FailedEvacuateStateTests https://review.openstack.org/561707 | 20:00 |
| sean-k-mooney | dansmith: well i dont think it would break anything today since its ignored but if they ever stop ignoring it i dont know if that would break us | 20:00 |
| *** cdent has quit IRC | 20:00 | |
| dansmith | sean-k-mooney: yeah | 20:00 |
| *** edmondsw_ has joined #openstack-nova | 20:00 | |
| dansmith | sean-k-mooney: I figured since I don't know the details, it'd be better to just avoid changing it | 20:01 |
| efried | TheJulia: Oh, it was rloo's note. Anyway, just let me know whatever you want me to do there. | 20:01 |
| dansmith | sean-k-mooney: I was asking him to just set it explicitly where he knows it should be vhost or vhostuser | 20:01 |
| *** r-daneel has quit IRC | 20:01 | |
| dansmith | sean-k-mooney: did you see the cleanup patch below? | 20:01 |
| efried | TheJulia: Caveat: Never used storyboard before... | 20:01 |
| dansmith | that was also based on ignoring, but it's vhostuser and you said vhost-user, so is it wrong? | 20:01 |
| TheJulia | efried: I can create it, its not a big deal | 20:01 |
| TheJulia | efried: but your welcome to if you want | 20:02 |
| * TheJulia bets we have at least one other story that this falls under | 20:02 | |
| TheJulia | Story clean-up is taking... many braincells. | 20:02 |
| efried | TheJulia: Go for it, co-author away. | 20:02 |
| *** lpetrut_ has quit IRC | 20:02 | |
| sean-k-mooney | dansmith: well neighter vhostuser or vhost-user is documented explictly as an allowed value but they used vhost-user when refing to it. | 20:02 |
| TheJulia | efried: don't need that :) But thanks. | 20:03 |
| dansmith | sean-k-mooney: okay you wanna comment on the patch below with that info then? | 20:03 |
| sean-k-mooney | dansmith: ill try and take a look at the libvirt source code tomorrow and see whent there parse has defiend | 20:03 |
| *** edmondsw has quit IRC | 20:03 | |
| dansmith | sean-k-mooney: okay | 20:03 |
| *** yamamoto has quit IRC | 20:04 | |
| *** wolverineav has joined #openstack-nova | 20:05 | |
| *** tmcm has quit IRC | 20:08 | |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add (Same|Different)HostFilter to defaults https://review.openstack.org/561651 | 20:11 |
| *** eharney has quit IRC | 20:11 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 20:13 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 20:13 |
| openstackgerrit | Lance Bragstad proposed openstack/nova master: Set scope_types for admin password policies https://review.openstack.org/553613 | 20:18 |
| openstackgerrit | Lance Bragstad proposed openstack/nova master: Add scope_types to server policies https://review.openstack.org/525772 | 20:18 |
| *** r-daneel has joined #openstack-nova | 20:19 | |
| lbragstad | johnthetubaguy_: jaypipes ^ look another shot at working a rough test structure into those | 20:19 |
| lbragstad | they will fail, but i'm curious about the approach you'd like me to take with the testing | 20:20 |
| lbragstad | or if what's proposed even makes sense | 20:20 |
| jaypipes | lbragstad: cool, thanks | 20:22 |
| jaypipes | lbragstad: will re-review shortly. | 20:22 |
| lbragstad | awesome - ty | 20:22 |
| *** Tom-Tom has joined #openstack-nova | 20:22 | |
| *** Tom-Tom has quit IRC | 20:27 | |
| *** dklyle has joined #openstack-nova | 20:28 | |
| *** Tom-Tom has joined #openstack-nova | 20:35 | |
| *** hemna_ has quit IRC | 20:36 | |
| *** vladikr has quit IRC | 20:39 | |
| *** Tom-Tom has quit IRC | 20:40 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:41 | |
| *** wolverineav has quit IRC | 20:42 | |
| *** wolverineav has joined #openstack-nova | 20:42 | |
| *** AlexeyAbashkin has quit IRC | 20:45 | |
| *** vladikr has joined #openstack-nova | 20:51 | |
| *** mtreinish has quit IRC | 20:51 | |
| *** tbachman has quit IRC | 20:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: List instances performace optimization https://review.openstack.org/561715 | 20:54 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 20:54 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 20:54 |
| *** harlowja has quit IRC | 20:55 | |
| *** edmondsw_ has quit IRC | 20:56 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Support forced separation of providers in granular https://review.openstack.org/561717 | 20:56 |
| *** mtreinish has joined #openstack-nova | 20:57 | |
| *** esberglu has quit IRC | 20:58 | |
| *** edmondsw has joined #openstack-nova | 20:59 | |
| *** burt has quit IRC | 20:59 | |
| *** fragatina has quit IRC | 20:59 | |
| *** yamamoto has joined #openstack-nova | 21:00 | |
| *** sambetts is now known as sambetts|afk | 21:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove vestigial system_metadata param from info_from_instance() https://review.openstack.org/561724 | 21:03 |
| *** edmondsw has quit IRC | 21:03 | |
| *** pchavva has quit IRC | 21:03 | |
| *** yamamoto has quit IRC | 21:06 | |
| *** Tom-Tom has joined #openstack-nova | 21:10 | |
| *** arvindn05 has joined #openstack-nova | 21:11 | |
| *** dpawlik has quit IRC | 21:12 | |
| *** arvindn05 has quit IRC | 21:14 | |
| *** Tom-Tom has quit IRC | 21:15 | |
| *** wolverineav has quit IRC | 21:16 | |
| *** wolverineav has joined #openstack-nova | 21:16 | |
| *** tbachman has joined #openstack-nova | 21:17 | |
| *** wolverineav has quit IRC | 21:17 | |
| *** wolverin_ has joined #openstack-nova | 21:17 | |
| *** fragatina has joined #openstack-nova | 21:22 | |
| *** dklyle has quit IRC | 21:22 | |
| *** felipemonteiro_ has quit IRC | 21:23 | |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | 21:27 |
| openstackgerrit | Jackie Truong proposed openstack/nova master: Add certificate validation docs https://review.openstack.org/560158 | 21:27 |
| *** mrjk has quit IRC | 21:28 | |
| *** mrjk has joined #openstack-nova | 21:29 | |
| *** tbachman has quit IRC | 21:31 | |
| *** tbachman has joined #openstack-nova | 21:32 | |
| jackie-truong | mriedem dansmith jaypipes melwitt johnthetubaguy_: Thanks for reviewing the certificate validation patches. 479949 and 561262 should just need one last look. The API patch is going through Zuul now. | 21:32 |
| jackie-truong | https://review.openstack.org/#/c/479949/ | 21:33 |
| jackie-truong | https://review.openstack.org/#/c/561262/ | 21:33 |
| *** jmlowe has joined #openstack-nova | 21:36 | |
| melwitt | thanks jackie-truong. I've rechecked the tempest tests so we can look at that too for the latest revision of the series https://review.openstack.org/#/c/515210 | 21:39 |
| *** esberglu has joined #openstack-nova | 21:42 | |
| jackie-truong | melwitt: Awesome, thanks | 21:42 |
| *** takashin has joined #openstack-nova | 21:44 | |
| *** Tom-Tom has joined #openstack-nova | 21:46 | |
| *** salv-orlando has quit IRC | 21:50 | |
| *** dklyle has joined #openstack-nova | 21:50 | |
| *** Tom-Tom has quit IRC | 21:50 | |
| *** rcernin has joined #openstack-nova | 21:51 | |
| *** dpawlik has joined #openstack-nova | 21:54 | |
| *** dpawlik has quit IRC | 21:58 | |
| *** Tom-Tom has joined #openstack-nova | 22:02 | |
| *** yamamoto has joined #openstack-nova | 22:02 | |
| *** BlackDex has quit IRC | 22:04 | |
| *** BlackDex has joined #openstack-nova | 22:05 | |
| *** david-lyle has joined #openstack-nova | 22:06 | |
| *** Tom-Tom has quit IRC | 22:06 | |
| *** dklyle has quit IRC | 22:08 | |
| *** yamamoto has quit IRC | 22:08 | |
| *** david-lyle has quit IRC | 22:11 | |
| *** Guest98903 has quit IRC | 22:11 | |
| mriedem | artom: gotta say, i'm not a fan of this https://review.openstack.org/#/c/561651/ | 22:15 |
| mriedem | if there is a problem with tempest, let's fix tempest | 22:15 |
| cfriesen | mriedem: I reproduced the Service UUID issue with vanilla stable/pike devstack. Writing up a bug now. | 22:15 |
| mriedem | artom: as noted inline, those tempest tests for these filters probably shouldn't even live in tempest | 22:15 |
| *** slaweq has quit IRC | 22:16 | |
| *** slaweq has joined #openstack-nova | 22:16 | |
| mriedem | jackie-truong: besides dumping on artom's patch i'm done for the day, sorry | 22:18 |
| mriedem | cfriesen: ok | 22:19 |
| jackie-truong | mriedem: np, thanks for all of the comments! | 22:19 |
| *** mlavalle has quit IRC | 22:20 | |
| *** slaweq has quit IRC | 22:21 | |
| melwitt | jackie-truong: looks like there's a problem :( http://logs.openstack.org/10/515210/31/check/barbican-simple-crypto-devstack-tempest/0ff6bcc/logs/testr_results.html.gz | 22:21 |
| melwitt | Additional properties are not allowed (u'trusted_image_certificates' was unexpected)", u'code': 400 | 22:21 |
| melwitt | looks like a schema validation fail? | 22:21 |
| jackie-truong | melwitt: Saw that. Looking into it | 22:24 |
| *** dougshelley66 has quit IRC | 22:27 | |
| *** dougshelley66 has joined #openstack-nova | 22:28 | |
| *** Tom-Tom has joined #openstack-nova | 22:36 | |
| *** dklyle has joined #openstack-nova | 22:40 | |
| *** Tom-Tom has quit IRC | 22:40 | |
| *** hemna_ has joined #openstack-nova | 22:44 | |
| *** Nisha_Agarwal has joined #openstack-nova | 22:45 | |
| cfriesen | melwitt: mriedem: here's the Service-without-UUID exception.ServiceNotFound issue: https://bugs.launchpad.net/nova/+bug/1764556 | 22:47 |
| openstack | Launchpad bug 1764556 in OpenStack Compute (nova) ""nova list" fails with exception.ServiceNotFound if service is deleted and has no UUID" [Undecided,New] | 22:47 |
| melwitt | thanks | 22:48 |
| cfriesen | reproducing is straightforward, just needs a fake DB entry in the services table | 22:48 |
| *** Nisha_Agarwal has quit IRC | 22:49 | |
| *** salv-orlando has joined #openstack-nova | 22:50 | |
| *** Tom-Tom has joined #openstack-nova | 22:51 | |
| *** hongbin_ has quit IRC | 22:55 | |
| *** salv-orlando has quit IRC | 22:56 | |
| *** Tom-Tom has quit IRC | 22:56 | |
| *** claudiub|2 has quit IRC | 22:57 | |
| *** dklyle has quit IRC | 22:58 | |
| *** andreas_s has joined #openstack-nova | 22:59 | |
| cfriesen | why does "openstack server --live" require a hostname? | 23:01 |
| cfriesen | make that "openstack server migrate --live" | 23:02 |
| *** yamamoto has joined #openstack-nova | 23:04 | |
| *** andreas_s has quit IRC | 23:04 | |
| artom | mriedem, yeah, I get it | 23:09 |
| *** yamamoto has quit IRC | 23:09 | |
| artom | We were having downstream discussions about it, and the argument that sort of got me was "if every CI has to change it, shouldn't it be in defaults?" | 23:10 |
| artom | Devstack enables it, downstream OSP CI needs to enable it, so... | 23:11 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (1) https://review.openstack.org/560821 | 23:11 |
| artom | But yeah, "CI needs it" isn't a massively compelling argument | 23:11 |
| *** jackie-truong has quit IRC | 23:12 | |
| artom | So I threw it up, just to see what discussion would come out of it, and to be able to say "upstream said" as opposed to "I said no" :) | 23:12 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_serversV21.py (2) https://review.openstack.org/560829 | 23:12 |
| *** yassine has joined #openstack-nova | 23:13 | |
| *** dklyle has joined #openstack-nova | 23:13 | |
| *** yassine is now known as Guest2631 | 23:13 | |
| melwitt | artom: I think it makes sense that CIs would enable most, if not all of the filters that exist, to cover the features and make sure they work with their products | 23:15 |
| *** Tom-Tom has joined #openstack-nova | 23:16 | |
| melwitt | for deciding whether to include certain filters in default_filters, I think that would be mostly about whether the filters are used by "most users" or are useful in a typical deployment | 23:16 |
| artom | melwitt, yeah, apparently it's really complicated for them to enabled it in their thousand+ jobs? | 23:17 |
| artom | melwitt, but I agree - CI and users have different needs | 23:17 |
| * artom is in the weird position of having to "defend" a patch he proposed himself, despite only half-believing in it | 23:18 | |
| *** moshele has joined #openstack-nova | 23:18 | |
| melwitt | I didn't think it's complicated in that I thought you just set enabled_filters to the filters you want enabled in nova.conf | 23:19 |
| artom | melwitt, yeah, but apparently they need to do that for *every* job? As in, there's no common deployment that covers all jobs? | 23:19 |
| *** Tom-Tom has quit IRC | 23:19 | |
| melwitt | that said, I don't really have an opinion about whether SameHost and DifferentHost belong in default filters because TBH I don't know if those are really commonly used by end users or not | 23:20 |
| artom | https://review.gerrithub.io/#/c/407794/ is the infrared change | 23:20 |
| artom | Which apparently got -2 by Arie? I have no idea what's going on | 23:20 |
| artom | melwitt, heh, don't we have those sosreports we can use for insights? At least for OSP users? | 23:21 |
| melwitt | yeah, we could I would think | 23:21 |
| artom | Man, the effort:complexity ratio of this thing is through the roof | 23:21 |
| artom | There's, like, 5 of us, just to enable a filter | 23:22 |
| melwitt | well, enabling a filter is supposed to be all pluggable, like you can do it in your nova.conf | 23:22 |
| melwitt | this is starting a discussion about "what belongs in defaults?" "should everything be in defaults?" "what's the downside of putting everything in defaults?" and I don't really know the answers to those questions off the top of my head | 23:23 |
| artom | Right, but the original problem is still "a filter that we need isn't enabled" | 23:24 |
| melwitt | by default, no. there are a lot of filters we could say that about. so to me, it's more like, should we just add all filters to defaults and is there any reason to not do that? I'd be asking scheduler subteam folk about it | 23:26 |
| *** jaosorior has quit IRC | 23:26 | |
| *** jaosorior has joined #openstack-nova | 23:27 | |
| *** sdeath has joined #openstack-nova | 23:27 | |
| *** sdague has quit IRC | 23:28 | |
| *** r-daneel has quit IRC | 23:29 | |
| *** jaosorior has quit IRC | 23:34 | |
| *** jaosorior has joined #openstack-nova | 23:34 | |
| *** rajinir has quit IRC | 23:34 | |
| artom | melwitt, yeah, it might be better to rephrase this as 2 questions: 1. do we have data that justifies adding same/different hostfilter to the defaults? | 23:36 |
| artom | 2. Should we just enable all filters by default? | 23:36 |
| artom | I can poke around our sosreports for 1 tomorrow, for 2... I guess tomorrow as well | 23:36 |
| cfriesen | melwitt: https://review.openstack.org/#/c/560690/ has been updated to address your final comment | 23:37 |
| melwitt | artom: yeah. the point of the default filters is to give someone a config for a typical cloud. if same/different are typically used, then they should be in defaults I think. CI are a different beast IMHO because they're aiming to run and verify as many features and capabilities as possible | 23:37 |
| melwitt | so you would expect them to enable everything they possibly could. that said, IIUC there's some mismatch between what tempest defaults to and what nova defaults to and that is what's actually causing this problem | 23:38 |
| melwitt | cfriesen: thanks, will take another look | 23:39 |
| cfriesen | melwitt: the default set of enabled filters is actually pretty small | 23:40 |
| *** dklyle has quit IRC | 23:40 | |
| melwitt | cfriesen: yes. I have learned that recently | 23:40 |
| melwitt | or re-learned it. I've seen default filter list in the past and eventually forgot it | 23:41 |
| *** jroll has quit IRC | 23:42 | |
| *** gyee has quit IRC | 23:43 | |
| *** salv-orlando has joined #openstack-nova | 23:51 | |
| *** Tom-Tom has joined #openstack-nova | 23:52 | |
| openstackgerrit | Merged openstack/nova master: Remove RequestContext.instance_lock_checked https://review.openstack.org/554378 | 23:55 |
| *** salv-orlando has quit IRC | 23:56 | |
| *** sdeath has quit IRC | 23:56 | |
| *** Tom-Tom has quit IRC | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!