| *** vladikr has quit IRC | 00:20 | |
| *** vladikr has joined #openstack-nova | 00:21 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: fixes mock autospec usage https://review.openstack.org/447505 | 00:21 |
|---|---|---|
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 00:21 |
| *** tbachman has quit IRC | 00:22 | |
| *** tbachman has joined #openstack-nova | 00:24 | |
| *** masuberu has joined #openstack-nova | 00:28 | |
| *** masber has quit IRC | 00:29 | |
| *** calebb has quit IRC | 00:33 | |
| *** calebb has joined #openstack-nova | 00:34 | |
| *** Guest69039 has quit IRC | 00:37 | |
| *** yassine has joined #openstack-nova | 00:38 | |
| *** yassine is now known as Guest8268 | 00:38 | |
| *** dave-mccowan has joined #openstack-nova | 00:41 | |
| *** hieulq has joined #openstack-nova | 00:52 | |
| *** liusheng has quit IRC | 00:52 | |
| *** salv-orlando has joined #openstack-nova | 00:52 | |
| openstackgerrit | Li Xipeng proposed openstack/nova master: Fix bug case by none token context https://review.openstack.org/522112 | 00:52 |
| *** _pewp_ has quit IRC | 00:57 | |
| *** _pewp_ has joined #openstack-nova | 00:58 | |
| *** salv-orlando has quit IRC | 00:58 | |
| *** tbachman has quit IRC | 01:02 | |
| *** tbachman has joined #openstack-nova | 01:03 | |
| *** tbachman has quit IRC | 01:08 | |
| *** phuongnh has joined #openstack-nova | 01:09 | |
| *** dave-mccowan has quit IRC | 01:09 | |
| *** tbachman has joined #openstack-nova | 01:10 | |
| *** ijw has joined #openstack-nova | 01:19 | |
| *** trungnv has joined #openstack-nova | 01:19 | |
| *** hongbin has joined #openstack-nova | 01:19 | |
| *** yangyapeng has joined #openstack-nova | 01:22 | |
| *** ijw has quit IRC | 01:23 | |
| *** dave-mccowan has joined #openstack-nova | 01:24 | |
| *** tbachman has quit IRC | 01:26 | |
| *** yamamoto has joined #openstack-nova | 01:29 | |
| *** lei-zh has joined #openstack-nova | 01:37 | |
| *** tbachman has joined #openstack-nova | 01:41 | |
| *** masuberu has quit IRC | 01:43 | |
| *** gbarros has joined #openstack-nova | 01:46 | |
| *** Tom-Tom has joined #openstack-nova | 01:46 | |
| *** tbachman_ has joined #openstack-nova | 01:46 | |
| *** tbachman has quit IRC | 01:46 | |
| *** tbachman_ is now known as tbachman | 01:47 | |
| *** salv-orlando has joined #openstack-nova | 01:54 | |
| *** tbachman has quit IRC | 01:55 | |
| *** salv-orlando has quit IRC | 02:00 | |
| *** pchavva has quit IRC | 02:03 | |
| *** litao__ has joined #openstack-nova | 02:03 | |
| *** liusheng has joined #openstack-nova | 02:14 | |
| *** gbarros has quit IRC | 02:22 | |
| *** sam_dw has quit IRC | 02:31 | |
| *** gbarros has joined #openstack-nova | 02:32 | |
| *** gcb has joined #openstack-nova | 02:33 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 02:37 |
| *** gmann_afk is now known as gmann | 02:37 | |
| *** fragatin_ has quit IRC | 02:38 | |
| *** fragatina has joined #openstack-nova | 02:39 | |
| *** threestrands_ has joined #openstack-nova | 02:49 | |
| *** threestrands_ has quit IRC | 02:49 | |
| *** threestrands_ has joined #openstack-nova | 02:49 | |
| *** moshele has joined #openstack-nova | 02:49 | |
| *** gbarros has quit IRC | 02:49 | |
| *** threestrands_ has quit IRC | 02:50 | |
| *** threestrands has quit IRC | 02:50 | |
| *** threestrands_ has joined #openstack-nova | 02:50 | |
| *** threestrands_ has quit IRC | 02:50 | |
| *** threestrands_ has joined #openstack-nova | 02:50 | |
| *** masber has joined #openstack-nova | 02:54 | |
| *** vladikr has quit IRC | 02:55 | |
| *** salv-orlando has joined #openstack-nova | 02:56 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Handle gathering exception in get_instance_sorted. https://review.openstack.org/522425 | 02:56 |
| openstackgerrit | Tao Li proposed openstack/nova master: Ignore some updates from virt driver https://review.openstack.org/523006 | 03:01 |
| *** dave-mccowan has quit IRC | 03:01 | |
| *** salv-orlando has quit IRC | 03:02 | |
| *** Tom-Tom has quit IRC | 03:02 | |
| *** moshele has quit IRC | 03:02 | |
| *** Tom-Tom has joined #openstack-nova | 03:02 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | 03:07 |
| *** dave-mccowan has joined #openstack-nova | 03:07 | |
| *** ralonsoh has quit IRC | 03:08 | |
| *** Tom-Tom has quit IRC | 03:08 | |
| *** masber has quit IRC | 03:09 | |
| openstackgerrit | Eli Qiao proposed openstack/nova master: Api-guide: Add Block Device Mapping https://review.openstack.org/522084 | 03:11 |
| openstackgerrit | Eli Qiao proposed openstack/nova master: Api-guide: Add concept for scheduler hints https://review.openstack.org/522106 | 03:11 |
| *** eliqiao has joined #openstack-nova | 03:11 | |
| eliqiao | @alex_xu can you help to take a quick look at https://review.openstack.org/#/c/522067/ (+2 already) when you are available ? | 03:14 |
| *** dave-mccowan has quit IRC | 03:18 | |
| *** sridharg has joined #openstack-nova | 03:23 | |
| *** vladikr has joined #openstack-nova | 03:24 | |
| *** masber has joined #openstack-nova | 03:29 | |
| *** abhishekk has joined #openstack-nova | 03:32 | |
| *** sree has joined #openstack-nova | 03:38 | |
| *** jichen has joined #openstack-nova | 03:43 | |
| *** yangyapeng has quit IRC | 03:45 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Fix the format file name https://review.openstack.org/523010 | 03:46 |
| *** mdnadeem has joined #openstack-nova | 03:49 | |
| *** psachin has joined #openstack-nova | 03:50 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: nova-manage db archive_deleted_rows is not multi-cell aware https://review.openstack.org/507486 | 03:52 |
| *** diga has joined #openstack-nova | 03:54 | |
| *** salv-orlando has joined #openstack-nova | 03:57 | |
| *** yikun_jiang is now known as kero_jiang | 03:59 | |
| *** salv-orlando has quit IRC | 04:04 | |
| *** yangyapeng has joined #openstack-nova | 04:09 | |
| *** lei-zh has quit IRC | 04:10 | |
| *** hoonetorg has quit IRC | 04:11 | |
| *** hongbin has quit IRC | 04:12 | |
| *** vladikr has quit IRC | 04:13 | |
| *** yangyapeng has quit IRC | 04:14 | |
| *** Dinesh_Bhor has joined #openstack-nova | 04:21 | |
| *** diga has quit IRC | 04:22 | |
| *** diga has joined #openstack-nova | 04:23 | |
| *** yangyapeng has joined #openstack-nova | 04:25 | |
| *** links has joined #openstack-nova | 04:26 | |
| *** suresh12 has joined #openstack-nova | 04:27 | |
| *** Tom-Tom has joined #openstack-nova | 04:42 | |
| *** yamamoto has quit IRC | 04:43 | |
| *** ijw has joined #openstack-nova | 04:47 | |
| openstackgerrit | Tuan Luong-Anh proposed openstack/nova master: Fix the format file name https://review.openstack.org/523010 | 04:48 |
| *** yamamoto has joined #openstack-nova | 04:49 | |
| *** hongbin has joined #openstack-nova | 04:51 | |
| *** udesale has joined #openstack-nova | 04:53 | |
| *** salv-orlando has joined #openstack-nova | 04:59 | |
| *** lei-zh has joined #openstack-nova | 05:03 | |
| *** salv-orlando has quit IRC | 05:05 | |
| openstackgerrit | Li Xipeng proposed openstack/nova master: Fix bug case by none token context https://review.openstack.org/522112 | 05:09 |
| *** jaosorior has joined #openstack-nova | 05:19 | |
| *** adisky_ has joined #openstack-nova | 05:29 | |
| *** ratailor has joined #openstack-nova | 05:30 | |
| *** yamamoto has quit IRC | 05:44 | |
| *** hongbin has quit IRC | 05:46 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 05:47 |
| *** dklyle has joined #openstack-nova | 05:47 | |
| *** yamamoto has joined #openstack-nova | 05:48 | |
| *** david-lyle has quit IRC | 05:48 | |
| *** udesale__ has joined #openstack-nova | 05:52 | |
| *** udesale has quit IRC | 05:53 | |
| *** chyka has joined #openstack-nova | 05:54 | |
| *** udesale has joined #openstack-nova | 05:55 | |
| *** vivsoni has quit IRC | 05:55 | |
| *** vivsoni has joined #openstack-nova | 05:55 | |
| *** udesale__ has quit IRC | 05:57 | |
| openstackgerrit | Li Xipeng proposed openstack/nova master: Fix bug as BDM failed when booting from volume https://review.openstack.org/522486 | 05:58 |
| *** TuanLA has joined #openstack-nova | 06:01 | |
| *** salv-orlando has joined #openstack-nova | 06:01 | |
| *** lei-zh has quit IRC | 06:04 | |
| *** lei-zh has joined #openstack-nova | 06:04 | |
| *** Alex_Staf has joined #openstack-nova | 06:06 | |
| *** salv-orlando has quit IRC | 06:07 | |
| openstackgerrit | Huan Xie proposed openstack/nova master: XenAPI: Don't use nicira-iface-id for XenServer VIF https://review.openstack.org/451641 | 06:11 |
| *** janki has joined #openstack-nova | 06:15 | |
| *** janki has quit IRC | 06:16 | |
| *** janki has joined #openstack-nova | 06:16 | |
| *** fragatina has quit IRC | 06:21 | |
| *** bhagyashris_ has joined #openstack-nova | 06:34 | |
| *** nsingh has joined #openstack-nova | 06:35 | |
| *** bhagyashri_s has quit IRC | 06:36 | |
| *** niraj_singh has quit IRC | 06:36 | |
| *** annp has joined #openstack-nova | 06:39 | |
| *** spectr has joined #openstack-nova | 06:44 | |
| *** moshele has joined #openstack-nova | 06:45 | |
| *** namnh has joined #openstack-nova | 06:47 | |
| *** rcernin has quit IRC | 06:47 | |
| *** suresh12 has quit IRC | 06:49 | |
| *** rcernin has joined #openstack-nova | 06:49 | |
| *** chyka has quit IRC | 06:53 | |
| *** pallavi has joined #openstack-nova | 06:54 | |
| *** spectr has quit IRC | 06:55 | |
| *** moshele has quit IRC | 06:56 | |
| pallavi | Hi | 06:56 |
| pallavi | I have installed openstack and but my VM is getting internet connection | 06:57 |
| pallavi | please help me to fix this issue | 06:58 |
| eliqiao | https://ask.openstack.org/ is a good place to ask questions. | 07:03 |
| *** salv-orlando has joined #openstack-nova | 07:03 | |
| *** edand has joined #openstack-nova | 07:05 | |
| *** lei-zh1 has joined #openstack-nova | 07:08 | |
| *** salv-orlando has quit IRC | 07:09 | |
| *** lei-zh has quit IRC | 07:11 | |
| *** gcb has quit IRC | 07:13 | |
| openstackgerrit | Li Xipeng proposed openstack/nova master: Fix bug as BDM failed when booting from volume https://review.openstack.org/522486 | 07:19 |
| openstackgerrit | Li Xipeng proposed openstack/nova master: Fix bug case by none token context https://review.openstack.org/522112 | 07:22 |
| *** moshele has joined #openstack-nova | 07:25 | |
| *** rcernin has quit IRC | 07:28 | |
| openstackgerrit | Chen Hanxiao proposed openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655 | 07:29 |
| *** ralonsoh has joined #openstack-nova | 07:30 | |
| *** andreas_s has joined #openstack-nova | 07:32 | |
| *** bhagyashris_ is now known as bhagyashris | 07:34 | |
| *** salv-orlando has joined #openstack-nova | 07:36 | |
| *** salv-orlando has quit IRC | 07:37 | |
| *** salv-orlando has joined #openstack-nova | 07:37 | |
| openstackgerrit | licanwei proposed openstack/nova master: cli 'nova-manage db sync' can't upgrade cell1 https://review.openstack.org/519275 | 07:39 |
| *** maciejjozefczyk has quit IRC | 07:43 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Get vgpu info from `allocations` https://review.openstack.org/521717 | 07:50 |
| *** lei-zh has joined #openstack-nova | 07:51 | |
| *** lei-zh1 has quit IRC | 07:51 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:57 | |
| *** sahid has joined #openstack-nova | 07:59 | |
| *** yamamoto has quit IRC | 08:04 | |
| *** takashin has left #openstack-nova | 08:06 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 08:06 |
| *** ttsiouts has quit IRC | 08:11 | |
| *** Tom-Tom has quit IRC | 08:11 | |
| *** pcaruana has joined #openstack-nova | 08:11 | |
| *** gcb has joined #openstack-nova | 08:12 | |
| *** sshwarts has joined #openstack-nova | 08:12 | |
| *** maciejjozefczyk has joined #openstack-nova | 08:13 | |
| *** jangutter has joined #openstack-nova | 08:18 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: fixes mock autospec usage https://review.openstack.org/447505 | 08:18 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 08:18 |
| *** pooja_jadhav has joined #openstack-nova | 08:18 | |
| *** belmoreira has joined #openstack-nova | 08:22 | |
| *** lpetrut has joined #openstack-nova | 08:22 | |
| *** slaweq has joined #openstack-nova | 08:24 | |
| *** sree_ has joined #openstack-nova | 08:29 | |
| *** sree_ is now known as Guest13269 | 08:29 | |
| *** sree has quit IRC | 08:31 | |
| khappone | Hello there. I sumbitted this bug a while back, and I wondered if there could be some eyes on this. It's a possible (well, it's already happened) dataloss bug in Nova with the unshelve workflow. https://bugs.launchpad.net/nova/+bug/1732428 | 08:32 |
| openstack | Launchpad bug 1732428 in OpenStack Compute (nova) "Unshelving a VM breaks instance metadata when using qcow2 backed images" [Undecided,New] | 08:32 |
| *** andreas_s has quit IRC | 08:33 | |
| *** yamamoto has joined #openstack-nova | 08:33 | |
| *** andreas_s has joined #openstack-nova | 08:33 | |
| *** jpena|off is now known as jpena | 08:38 | |
| *** josecastroleon has joined #openstack-nova | 08:40 | |
| *** andreas_s has quit IRC | 08:42 | |
| *** andreas_s has joined #openstack-nova | 08:43 | |
| *** andreas_s has quit IRC | 08:45 | |
| *** andreas_s has joined #openstack-nova | 08:46 | |
| *** yamamoto has quit IRC | 08:51 | |
| *** ttsiouts has joined #openstack-nova | 08:53 | |
| *** sree has joined #openstack-nova | 09:00 | |
| *** Guest13269 has quit IRC | 09:01 | |
| *** damien_r has joined #openstack-nova | 09:03 | |
| *** jchhatbar has joined #openstack-nova | 09:12 | |
| *** cheneydc has joined #openstack-nova | 09:12 | |
| *** janki has quit IRC | 09:13 | |
| *** gmann is now known as gmann_afk | 09:13 | |
| *** lpetrut has quit IRC | 09:26 | |
| *** lei-zh has quit IRC | 09:27 | |
| *** lei-zh has joined #openstack-nova | 09:27 | |
| *** derekh has joined #openstack-nova | 09:28 | |
| *** yamamoto has joined #openstack-nova | 09:31 | |
| *** spectr has joined #openstack-nova | 09:37 | |
| *** spectr has quit IRC | 09:37 | |
| *** yamamoto has quit IRC | 09:39 | |
| *** aloga has quit IRC | 09:41 | |
| *** aloga has joined #openstack-nova | 09:41 | |
| *** lpetrut has joined #openstack-nova | 09:44 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 09:44 |
| *** claudiub|2 has joined #openstack-nova | 09:44 | |
| *** gszasz has joined #openstack-nova | 09:46 | |
| *** pallavi has quit IRC | 09:47 | |
| *** cheneydc has quit IRC | 09:58 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 09:59 |
| *** annp has quit IRC | 10:03 | |
| *** jichen has quit IRC | 10:03 | |
| *** ociuhandu has joined #openstack-nova | 10:03 | |
| *** sree has quit IRC | 10:03 | |
| *** ociuhandu has quit IRC | 10:04 | |
| *** sree has joined #openstack-nova | 10:04 | |
| *** mvk has joined #openstack-nova | 10:05 | |
| *** sdague has joined #openstack-nova | 10:07 | |
| *** owalsh is now known as owalsh_afk | 10:08 | |
| *** dtantsur|afk is now known as dtantsur | 10:08 | |
| *** sree has quit IRC | 10:08 | |
| *** andreas_s has quit IRC | 10:09 | |
| *** andreas_s has joined #openstack-nova | 10:10 | |
| *** lpetrut has quit IRC | 10:11 | |
| *** sambetts_ has joined #openstack-nova | 10:16 | |
| *** andreas_s has quit IRC | 10:18 | |
| *** spectr has joined #openstack-nova | 10:22 | |
| *** spectr has quit IRC | 10:23 | |
| *** namnh has quit IRC | 10:25 | |
| *** yamamoto has joined #openstack-nova | 10:27 | |
| *** lpetrut has joined #openstack-nova | 10:29 | |
| *** andreas_s has joined #openstack-nova | 10:29 | |
| *** yamamoto has quit IRC | 10:32 | |
| *** lei-zh has quit IRC | 10:36 | |
| openstackgerrit | Hironori Shiina proposed openstack/nova master: Ironic: Check a last error of power action https://review.openstack.org/470754 | 10:39 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 10:46 |
| *** sridharg has quit IRC | 10:46 | |
| *** andreas_s has quit IRC | 10:48 | |
| *** andreas_s has joined #openstack-nova | 10:48 | |
| *** yamamoto has joined #openstack-nova | 10:53 | |
| *** yamamoto has quit IRC | 10:55 | |
| *** phuongnh has quit IRC | 10:55 | |
| *** pcaruana has quit IRC | 10:55 | |
| *** sambetts_ is now known as sambetts | 10:57 | |
| *** pcaruana has joined #openstack-nova | 10:59 | |
| *** yamamoto has joined #openstack-nova | 11:01 | |
| *** TuanLA has quit IRC | 11:02 | |
| *** ianw has quit IRC | 11:03 | |
| *** ianw_ has joined #openstack-nova | 11:03 | |
| *** ianw_ is now known as ianw | 11:04 | |
| *** yamamoto has quit IRC | 11:06 | |
| *** andreas_s has quit IRC | 11:07 | |
| *** finucannot is now known as stephenfin | 11:07 | |
| *** andreas_s has joined #openstack-nova | 11:07 | |
| *** udesale has quit IRC | 11:07 | |
| *** abhishekk has quit IRC | 11:09 | |
| *** _pewp_ has quit IRC | 11:10 | |
| *** _pewp_ has joined #openstack-nova | 11:14 | |
| *** andreas_s has quit IRC | 11:16 | |
| *** ratailor has quit IRC | 11:16 | |
| *** alexchadin has joined #openstack-nova | 11:19 | |
| *** chyka has joined #openstack-nova | 11:24 | |
| *** gszasz has quit IRC | 11:24 | |
| *** andreas_s has joined #openstack-nova | 11:27 | |
| openstackgerrit | licanwei proposed openstack/nova master: cli 'nova-manage db sync' can't upgrade cell1 https://review.openstack.org/519275 | 11:28 |
| *** sree has joined #openstack-nova | 11:28 | |
| *** andreas_s has quit IRC | 11:28 | |
| *** chyka has quit IRC | 11:29 | |
| *** andreas_s has joined #openstack-nova | 11:29 | |
| *** cdent has joined #openstack-nova | 11:30 | |
| *** yamamoto has joined #openstack-nova | 11:31 | |
| *** sree has quit IRC | 11:33 | |
| *** diga has quit IRC | 11:33 | |
| *** andreas_s has quit IRC | 11:34 | |
| *** rodolof has joined #openstack-nova | 11:34 | |
| *** spectr has joined #openstack-nova | 11:35 | |
| *** spectr has quit IRC | 11:35 | |
| *** yamamoto has quit IRC | 11:36 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi:(WIP:need ut) Get vgpu info from `allocations` https://review.openstack.org/521717 | 11:36 |
| *** cdent has quit IRC | 11:39 | |
| *** alexchadin has quit IRC | 11:40 | |
| *** alexchadin has joined #openstack-nova | 11:41 | |
| *** Kevin_Zheng has quit IRC | 11:41 | |
| *** threestrands_ has quit IRC | 11:42 | |
| *** erlon has joined #openstack-nova | 11:42 | |
| *** litao__ has quit IRC | 11:43 | |
| *** sree has joined #openstack-nova | 11:44 | |
| *** gcb has quit IRC | 11:45 | |
| *** gszasz has joined #openstack-nova | 11:49 | |
| *** alexchadin has quit IRC | 11:51 | |
| *** alexchadin has joined #openstack-nova | 11:51 | |
| *** icey has quit IRC | 11:53 | |
| *** patrickeast has quit IRC | 11:53 | |
| *** samueldmq has quit IRC | 11:53 | |
| *** ying_zuo has quit IRC | 11:53 | |
| *** zhenguo has quit IRC | 11:53 | |
| *** fyxim has quit IRC | 11:54 | |
| *** larainema has quit IRC | 11:54 | |
| *** kmalloc has quit IRC | 11:54 | |
| *** vdrok has quit IRC | 11:54 | |
| *** csatari has quit IRC | 11:54 | |
| *** jbryce has quit IRC | 11:54 | |
| *** ildikov has quit IRC | 11:54 | |
| *** ankit_ has quit IRC | 11:54 | |
| *** redondo-mk has quit IRC | 11:54 | |
| *** fmccrthy has quit IRC | 11:54 | |
| *** icey has joined #openstack-nova | 11:55 | |
| *** samueldmq has joined #openstack-nova | 11:55 | |
| *** zhenguo has joined #openstack-nova | 11:55 | |
| *** patrickeast has joined #openstack-nova | 11:55 | |
| *** fmccrthy has joined #openstack-nova | 11:56 | |
| *** ankit_ has joined #openstack-nova | 11:56 | |
| *** redondo-mk has joined #openstack-nova | 11:56 | |
| *** vdrok has joined #openstack-nova | 11:56 | |
| *** kmalloc has joined #openstack-nova | 11:56 | |
| *** fyxim has joined #openstack-nova | 11:56 | |
| *** csatari has joined #openstack-nova | 11:56 | |
| *** ildikov has joined #openstack-nova | 11:56 | |
| *** yamamoto has joined #openstack-nova | 12:01 | |
| *** cdent has joined #openstack-nova | 12:02 | |
| *** yangyapeng has quit IRC | 12:06 | |
| *** yamamoto has quit IRC | 12:06 | |
| *** cheneydc has joined #openstack-nova | 12:08 | |
| *** alexchadin has quit IRC | 12:09 | |
| *** dave-mcc_ has joined #openstack-nova | 12:10 | |
| *** alexchadin has joined #openstack-nova | 12:10 | |
| *** smatzek has joined #openstack-nova | 12:12 | |
| *** sree has quit IRC | 12:13 | |
| *** fried_turkey is now known as efried | 12:18 | |
| *** andreas_s has joined #openstack-nova | 12:22 | |
| *** szaher has quit IRC | 12:22 | |
| efried | Good morning Nova! | 12:23 |
| *** rodolof has quit IRC | 12:23 | |
| *** rodolof has joined #openstack-nova | 12:24 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ocata: Document the real behavior of notify_on_state_change https://review.openstack.org/521871 | 12:24 |
| *** giblet is now known as gibi | 12:24 | |
| *** szaher has joined #openstack-nova | 12:25 | |
| *** hshiina has joined #openstack-nova | 12:28 | |
| *** udesale has joined #openstack-nova | 12:29 | |
| *** links has quit IRC | 12:29 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 12:31 |
| *** alexchadin has quit IRC | 12:35 | |
| *** alexchadin has joined #openstack-nova | 12:36 | |
| *** andreas_s has quit IRC | 12:38 | |
| *** andreas_s has joined #openstack-nova | 12:41 | |
| *** nkorabli has joined #openstack-nova | 12:41 | |
| *** links has joined #openstack-nova | 12:43 | |
| *** sapcc-bot has quit IRC | 12:44 | |
| *** sapcc-bot has joined #openstack-nova | 12:44 | |
| *** carthaca_ has joined #openstack-nova | 12:44 | |
| *** mkoderer_ has joined #openstack-nova | 12:44 | |
| *** tpatzig_ has joined #openstack-nova | 12:44 | |
| *** dgonzalez_ has joined #openstack-nova | 12:44 | |
| *** david_1 has joined #openstack-nova | 12:44 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 12:44 |
| *** dgonzalez_ has quit IRC | 12:46 | |
| *** mkoderer_ has quit IRC | 12:46 | |
| *** tpatzig_ has quit IRC | 12:46 | |
| *** carthaca_ has quit IRC | 12:46 | |
| *** david_1 has quit IRC | 12:46 | |
| nkorabli | Hey there! Would love to clarify Numa cpu pinning process in Openstack. I've went through all the docs I could find on that - but still one thing is not clear to me. Is that possible to assign an instance to a particular Numa cell? It seems like by default, when you launch a single CPU vm using numa, it won't be spread evenly across all numa cells, it will be pinned to the `cellid=0`. | 12:46 |
| *** sridharg has joined #openstack-nova | 12:46 | |
| *** yamamoto has joined #openstack-nova | 12:47 | |
| nkorabli | @here | 12:47 |
| *** lpetrut has quit IRC | 12:48 | |
| sean-k-mooney | nkorabli: that depends on your definition of assigin to a numa cell | 12:50 |
| sean-k-mooney | nkorabli: you can request a vm to have a single virtual numa cell in which case openstack will create it and pin it to a single numa cell | 12:50 |
| sean-k-mooney | nkorabli: you cannot however say that i want to boot this vm on numa cell 0 of host X. that would break the cloud abstration | 12:51 |
| *** yamamoto has quit IRC | 12:51 | |
| *** jpena is now known as jpena|lunch | 12:55 | |
| *** chyka has joined #openstack-nova | 12:56 | |
| *** lennyb__ has joined #openstack-nova | 12:56 | |
| *** yangyapeng has joined #openstack-nova | 12:56 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 12:57 |
| *** lpetrut has joined #openstack-nova | 12:57 | |
| nkorabli | sean-k-moooney: I played a bit with that - I had single compute node with 2 numa cells(nodes). I launched a lot of VMs with a single CPU. Then checked how those VMs got distributed across those numa cells. Apparently, all of them got pinned to the first numa node and none to second. | 12:59 |
| sean-k-mooney | are you using openstack kilo | 12:59 |
| sean-k-mooney | or using a pci passthrough device. | 13:00 |
| nkorabli | I used Liberty for that | 13:00 |
| sean-k-mooney | thats your issue | 13:00 |
| nkorabli | I also noticed this in the code: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4099-L4104 | 13:00 |
| sean-k-mooney | liberty had a bug where the numa virtual numa node was used to select the phyical numa node | 13:00 |
| *** chyka has quit IRC | 13:00 | |
| sean-k-mooney | that was expcitly never intended to be the behavior | 13:00 |
| *** dtantsur is now known as dtantsur|bbl | 13:03 | |
| nkorabli | So, this behavior shouldn't be the same in older releases? I supposed that according to the link above - this issue hasn't been fixed yet..' | 13:03 |
| *** owalsh_afk is now known as owalsh | 13:03 | |
| *** gszasz has quit IRC | 13:04 | |
| sean-k-mooney | no you are miss understanding what i said. https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L4099-L4104 simply states that if we do not request a numa topology for a vm we leave the kernel spread the instaces. | 13:04 |
| sean-k-mooney | by default vms created in openstack are not assumed to have memory or cpus that come from a single numa node | 13:05 |
| *** vladikr has joined #openstack-nova | 13:07 | |
| sean-k-mooney | nkorabli: are several indirect ways that a numa topology can be requested on your behalf without you seting the hw:numa_nodes extra spec. | 13:08 |
| sean-k-mooney | nkorabli: for example if you request hugepages a request for a singel numa node will implicitly be created. unless your request a numa topology via hw:numa_nodes | 13:09 |
| nkorabli | If I specified `hw:numa_nodes=1` then it will use just one cell, right? (like `cellid=0`). I can't specify `hw:numa_nodes=2` for a single CPU instance either. So, I can't explicitly make use of both numa nodes when working with single CPU VMs? | 13:13 |
| *** Eran_Kuris has quit IRC | 13:14 | |
| *** mdnadeem has quit IRC | 13:15 | |
| sean-k-mooney | nkorabli: if you specify hw:numa_nodes=1 for a single core vm flavor that vm can be placed on either of numa nodes on the host system but it will be affinitesed to which ever node it first starts on | 13:15 |
| *** markvoelker has quit IRC | 13:16 | |
| sean-k-mooney | nkorabli: if you dont spceify hw:numa_nodes at all it will implcitly float across all numa nodes and cores at the whim of the host kernel scheduler | 13:16 |
| *** markvoelker has joined #openstack-nova | 13:16 | |
| *** Eran_Kuris has joined #openstack-nova | 13:17 | |
| sean-k-mooney | nkorabli: to be more precise it will float across the cores specified via the vcpu_pin_set option in the nova.conf | 13:18 |
| *** chyka has joined #openstack-nova | 13:18 | |
| sean-k-mooney | nkorabli: if vcpu_pin_set is not specified it defaults to all cores | 13:19 |
| nkorabli | alright, that's great! I was expecting behavior like this, just couldn't reproduce in Liberty. When I specified hw:numa_nodes=1, it started throwing errors like this: "No valid host was found. There are not enough hosts available." So, I thought it fills up the first node and never sees the second. | 13:21 |
| nkorabli | This line also made me think this way: https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L1976 | 13:23 |
| *** chyka has quit IRC | 13:23 | |
| nkorabli | If I understand you right, this is an issue that appears in Liberty only? | 13:24 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/config.py#L1976 is referring to the virtual numa topology of the guest. not the host numa topology | 13:25 |
| sean-k-mooney | the first release of openstack with numa support did a 1:1 mapping between the virtual cellid to the host cellid howerver that was a mistake that was correct in later releases | 13:26 |
| sean-k-mooney | nkorabli: there is also a rather little known option hw:numa_mempolicy which defaults to strict which allows you to specify if the guest memory must come from the same numa node as the vCPUS | 13:28 |
| sean-k-mooney | i cant find the upstream docs but its memtioned here in the redhat docs https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux_OpenStack_Platform/6/html/Administration_Guide/section-flavors.html | 13:29 |
| efried | edleafe Sched meeting is in half an hour, right? | 13:30 |
| *** gcb has joined #openstack-nova | 13:31 | |
| *** eharney has joined #openstack-nova | 13:34 | |
| cdent | efried: appears so | 13:36 |
| efried | cdent Takes me a couple weeks to get the time shift right. DST is stupid. | 13:36 |
| cdent | yes and yes | 13:36 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 13:37 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.resize_revert notification https://review.openstack.org/396811 | 13:39 |
| *** pchavva has joined #openstack-nova | 13:40 | |
| *** lpetrut has quit IRC | 13:42 | |
| *** lpetrut has joined #openstack-nova | 13:42 | |
| nkorabli | sean-k-mooney: thanks a lot! Let me test that out! | 13:43 |
| *** pchavva1 has joined #openstack-nova | 13:44 | |
| *** pchavva1 has quit IRC | 13:44 | |
| *** yamamoto has joined #openstack-nova | 13:45 | |
| *** yamamoto has quit IRC | 13:45 | |
| *** takashin has joined #openstack-nova | 13:48 | |
| *** lpetrut has quit IRC | 13:48 | |
| *** lpetrut has joined #openstack-nova | 13:49 | |
| *** yamamoto has joined #openstack-nova | 13:49 | |
| *** jpena|lunch is now known as jpena | 13:49 | |
| *** jbryce has joined #openstack-nova | 13:51 | |
| *** larainema has joined #openstack-nova | 13:51 | |
| *** ying_zuo has joined #openstack-nova | 13:51 | |
| *** gszasz has joined #openstack-nova | 13:52 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 13:52 |
| *** trozet has quit IRC | 13:52 | |
| edleafe | efried: yes, scheduler subteam meeting in 6 minutes in #openstack-meeting-alt | 13:54 |
| efried | thx | 13:54 |
| *** mriedem has joined #openstack-nova | 13:54 | |
| *** trozet has joined #openstack-nova | 13:54 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 13:56 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | 13:56 |
| *** jaypipes has joined #openstack-nova | 13:56 | |
| *** lpetrut has quit IRC | 13:57 | |
| *** yamamoto has quit IRC | 13:57 | |
| *** lei-zh has joined #openstack-nova | 13:59 | |
| *** foolish has joined #openstack-nova | 14:01 | |
| *** lyan has joined #openstack-nova | 14:01 | |
| *** karthiks has quit IRC | 14:01 | |
| *** eliqiao has quit IRC | 14:03 | |
| *** liverpooler has joined #openstack-nova | 14:05 | |
| *** gbarros has joined #openstack-nova | 14:10 | |
| *** alexchadin has quit IRC | 14:10 | |
| *** cheneydc_ has joined #openstack-nova | 14:11 | |
| *** cheneydc has quit IRC | 14:14 | |
| *** sree has joined #openstack-nova | 14:15 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform instance.exists notification https://review.openstack.org/403660 | 14:18 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add sample test for instance audit https://review.openstack.org/480955 | 14:18 |
| *** smatzek has quit IRC | 14:18 | |
| *** smatzek has joined #openstack-nova | 14:19 | |
| *** sree has quit IRC | 14:19 | |
| *** gbarros has quit IRC | 14:19 | |
| *** cheneydc_ has quit IRC | 14:20 | |
| *** cheneydc_ has joined #openstack-nova | 14:20 | |
| *** sree has joined #openstack-nova | 14:21 | |
| *** smatzek has quit IRC | 14:24 | |
| *** dtantsur|bbl is now known as dtantsur | 14:24 | |
| *** josecastroleon1 has joined #openstack-nova | 14:30 | |
| *** josecastroleon has quit IRC | 14:30 | |
| *** links has quit IRC | 14:32 | |
| *** belmoreira has quit IRC | 14:33 | |
| *** esberglu has joined #openstack-nova | 14:33 | |
| *** gcb has quit IRC | 14:33 | |
| alex_xu | efried: looks like https://review.openstack.org/#/c/498737/2/nova/tests/functional/db/test_resource_provider.py@2981 same with https://review.openstack.org/#/c/518633/15/nova/tests/functional/db/test_allocation_candidates.py@1719 | 14:41 |
| alex_xu | efried: I didn't found one similar to this yet https://review.openstack.org/#/c/498737/2/nova/tests/functional/db/test_resource_provider.py@3207 | 14:42 |
| *** udesale has quit IRC | 14:42 | |
| *** burt has joined #openstack-nova | 14:45 | |
| mriedem | gibi: are you looking for changes on this patch? https://review.openstack.org/#/c/516782/ | 14:46 |
| mriedem | i see you didn't -1 it | 14:46 |
| mriedem | alex_xu: regarding https://review.openstack.org/#/c/522305/ i'm inclined to just fix the bug since it's an admin-only API by default, so not really an interop concern, and it's a 1-line fix which we can then backport | 14:46 |
| mriedem | i'll need to consider updating the microversion dev ref docs on admin APIs | 14:47 |
| alex_xu | mriedem: ok, got it | 14:47 |
| gibi | mriedem: It would be nice to close the bug 1730730 report as well with the patch but jaypipes seems has different oppinion | 14:50 |
| openstack | bug 1730730 in OpenStack Compute (nova) "AllocationCandidates.get_by_filters returns garbage with only sharing providers" [Undecided,Triaged] https://launchpad.net/bugs/1730730 | 14:50 |
| gibi | jaypipes: is it OK for you if I quickly resping https://review.openstack.org/#/c/516782/ and add the closes-bug tag to the commit message? | 14:52 |
| *** takedakn has joined #openstack-nova | 14:52 | |
| jaypipes | gibi: go for it. | 14:52 |
| gibi | jaypipes: OK | 14:53 |
| *** mlavalle has joined #openstack-nova | 14:54 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | 14:54 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM support matrix update https://review.openstack.org/523140 | 14:54 |
| gibi | mriedem, alex_xu, jaypipes: here is the quick update, i'm +2 now on https://review.openstack.org/516782 | 14:55 |
| alex_xu | gibi: done | 14:56 |
| *** salv-orlando has quit IRC | 14:57 | |
| *** yamamoto has joined #openstack-nova | 14:57 | |
| *** salv-orlando has joined #openstack-nova | 14:58 | |
| *** awaugama has joined #openstack-nova | 14:58 | |
| *** yamahata has quit IRC | 14:59 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add x-openstack-request-id in API ref https://review.openstack.org/523007 | 15:00 |
| *** yamahata has joined #openstack-nova | 15:00 | |
| *** hshiina has quit IRC | 15:01 | |
| *** lei-zh has quit IRC | 15:01 | |
| *** lyan has quit IRC | 15:02 | |
| *** cheneydc_ has quit IRC | 15:02 | |
| *** takashin_ has joined #openstack-nova | 15:03 | |
| *** salv-orlando has quit IRC | 15:03 | |
| *** fragatina has joined #openstack-nova | 15:03 | |
| *** yamamoto has quit IRC | 15:04 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix invalid minRam error message https://review.openstack.org/523142 | 15:04 |
| *** lyan has joined #openstack-nova | 15:05 | |
| *** takashin has quit IRC | 15:05 | |
| *** hamzy has quit IRC | 15:06 | |
| *** burt has quit IRC | 15:07 | |
| *** gbarros has joined #openstack-nova | 15:10 | |
| *** fragatin_ has joined #openstack-nova | 15:12 | |
| *** marst has joined #openstack-nova | 15:12 | |
| *** andreas_s has quit IRC | 15:12 | |
| *** bnemec has quit IRC | 15:13 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 15:13 |
| *** fragatina has quit IRC | 15:13 | |
| *** Alex_Staf has quit IRC | 15:13 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test alloc_cands with indirectly sharing RPs https://review.openstack.org/519601 | 15:14 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 15:14 |
| *** felipemonteiro has joined #openstack-nova | 15:14 | |
| *** felipemonteiro_ has joined #openstack-nova | 15:16 | |
| *** toure_biab is now known as toure | 15:16 | |
| *** andreas_s has joined #openstack-nova | 15:17 | |
| *** jdillaman has quit IRC | 15:18 | |
| *** smatzek has joined #openstack-nova | 15:18 | |
| *** felipemonteiro has quit IRC | 15:20 | |
| *** andreas_s has quit IRC | 15:21 | |
| *** smatzek has quit IRC | 15:21 | |
| *** smatzek has joined #openstack-nova | 15:21 | |
| *** cdent has quit IRC | 15:24 | |
| *** yangyapeng has quit IRC | 15:24 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Refined fix for validating image on rebuild https://review.openstack.org/521186 | 15:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 15:24 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 15:24 |
| efried | dansmith Is https://review.openstack.org/#/c/507638/ related to the "Migration fixups" priority at https://etherpad.openstack.org/p/nova-ptg-queens-placement L49? | 15:27 |
| efried | dansmith Perhaps a better question: what all is related to the "migration fixups" priority at ^ ? | 15:27 |
| dansmith | that's the last one | 15:27 |
| dansmith | all the others merged already | 15:27 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement query param schema for delete assisted vol https://review.openstack.org/513668 | 15:28 |
| openstackgerrit | Lajos Katona proposed openstack/nova master: Extend ServerMovingTests with custom resources https://review.openstack.org/497399 | 15:28 |
| efried | dansmith Cool. Looks like it needs a rebase? | 15:28 |
| dansmith | hmm, yeah I guess so | 15:29 |
| efried | dansmith (cdent) Is POST /allocations for multiple consumers (https://review.openstack.org/#/c/500073/) not related to the same effort? | 15:30 |
| dansmith | it is | 15:30 |
| efried | okay, thanks. | 15:30 |
| *** hongbin has joined #openstack-nova | 15:35 | |
| *** sree has quit IRC | 15:37 | |
| *** suresh12 has joined #openstack-nova | 15:40 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make live migration hold resources with a migration allocation https://review.openstack.org/507638 | 15:42 |
| *** suresh12 has quit IRC | 15:44 | |
| openstackgerrit | jiangpf proposed openstack/nova master: Encode libvirt domain XML in UTF-8 https://review.openstack.org/522161 | 15:45 |
| efried | jaypipes edleafe dansmith Draft: http://paste.openstack.org/show/627496/ -- As owners of the Three Priorities, would you please vet this for proper inclusion of linkitude? | 15:45 |
| *** andreas_s has joined #openstack-nova | 15:46 | |
| *** edand has quit IRC | 15:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unnecessary self.flags and ConfPatcher https://review.openstack.org/522993 | 15:49 |
| *** cdent has joined #openstack-nova | 15:49 | |
| *** hamzy has joined #openstack-nova | 15:50 | |
| *** andreas_s has quit IRC | 15:50 | |
| edleafe | efried: except for the double spaces after periods, it looks good | 15:53 |
| efried | edleafe Thanks. Appreciate you taking a look. I'll work on the spacing thing. | 15:53 |
| edleafe | It probably looks fine on a typewriter | 15:54 |
| *** bnemec has joined #openstack-nova | 15:54 | |
| efried | edleafe That's how I proofread all my work. | 15:55 |
| *** rodolof has quit IRC | 15:58 | |
| *** yangyapeng has joined #openstack-nova | 15:58 | |
| *** slaweq has quit IRC | 15:58 | |
| *** salv-orlando has joined #openstack-nova | 15:58 | |
| *** slaweq has joined #openstack-nova | 15:59 | |
| *** slaweq has quit IRC | 15:59 | |
| *** slaweq has joined #openstack-nova | 16:00 | |
| *** slaweq has quit IRC | 16:00 | |
| *** slaweq has joined #openstack-nova | 16:00 | |
| *** yangyapeng has quit IRC | 16:02 | |
| *** jchhatbar has quit IRC | 16:03 | |
| *** salv-orlando has quit IRC | 16:04 | |
| *** josecastroleon1 has quit IRC | 16:04 | |
| *** slaweq has quit IRC | 16:05 | |
| *** moshele has quit IRC | 16:06 | |
| *** foolish has quit IRC | 16:09 | |
| *** belmoreira has joined #openstack-nova | 16:09 | |
| *** takedakn has quit IRC | 16:09 | |
| *** kero_jiang has quit IRC | 16:09 | |
| *** gbarros has quit IRC | 16:10 | |
| *** alex_xu has quit IRC | 16:11 | |
| *** yikun has joined #openstack-nova | 16:12 | |
| *** yikun has quit IRC | 16:13 | |
| *** xinliang has quit IRC | 16:14 | |
| *** yikun has joined #openstack-nova | 16:14 | |
| *** sree has joined #openstack-nova | 16:15 | |
| *** andreas_s has joined #openstack-nova | 16:18 | |
| *** damien_r has quit IRC | 16:19 | |
| *** tbachman has joined #openstack-nova | 16:22 | |
| *** suresh12 has joined #openstack-nova | 16:23 | |
| *** andreas_s has quit IRC | 16:23 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Versioned notifications for service create and delete https://review.openstack.org/519588 | 16:24 |
| *** gbarros has joined #openstack-nova | 16:24 | |
| *** damien_r has joined #openstack-nova | 16:24 | |
| *** gbarros has quit IRC | 16:25 | |
| *** sdague has quit IRC | 16:26 | |
| *** shaohe_feng has quit IRC | 16:27 | |
| *** alex_xu has joined #openstack-nova | 16:27 | |
| *** takashin_ has left #openstack-nova | 16:27 | |
| *** xinliang has joined #openstack-nova | 16:28 | |
| *** sshwarts has quit IRC | 16:28 | |
| *** gszasz has quit IRC | 16:29 | |
| *** shaohe_feng has joined #openstack-nova | 16:31 | |
| *** sdague has joined #openstack-nova | 16:33 | |
| *** xinliang has quit IRC | 16:34 | |
| *** belmoreira has quit IRC | 16:35 | |
| jaypipes | efried: sure, looks fine to me, thank you! | 16:36 |
| efried | jaypipes Thanks. | 16:36 |
| *** xinliang has joined #openstack-nova | 16:39 | |
| *** chyka has joined #openstack-nova | 16:40 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Address nits from service create/destroy notification review https://review.openstack.org/523162 | 16:43 |
| *** shaner has quit IRC | 16:44 | |
| cdent | nice placement functional test to get in: https://review.openstack.org/#/c/493865/ | 16:44 |
| *** jpena is now known as jpena|brb | 16:44 | |
| *** shaner has joined #openstack-nova | 16:45 | |
| *** sree has quit IRC | 16:50 | |
| *** sree has joined #openstack-nova | 16:50 | |
| *** suresh12 has quit IRC | 16:54 | |
| *** gbarros has joined #openstack-nova | 16:55 | |
| *** sree has quit IRC | 16:56 | |
| *** moshele has joined #openstack-nova | 16:58 | |
| *** toure is now known as toure_biab | 16:59 | |
| *** AlexeyAbashkin has quit IRC | 16:59 | |
| *** salv-orlando has joined #openstack-nova | 17:00 | |
| openstackgerrit | Merged openstack/nova master: Implement query param schema for flavor index https://review.openstack.org/515413 | 17:00 |
| *** gyee_ has joined #openstack-nova | 17:00 | |
| openstackgerrit | Merged openstack/nova master: Implement query param schema for fping index https://review.openstack.org/513890 | 17:00 |
| *** sree has joined #openstack-nova | 17:04 | |
| *** felipemonteiro_ has quit IRC | 17:05 | |
| *** salv-orlando has quit IRC | 17:06 | |
| *** slaweq has joined #openstack-nova | 17:08 | |
| *** sree has quit IRC | 17:09 | |
| *** pcaruana has quit IRC | 17:09 | |
| *** psachin has quit IRC | 17:11 | |
| *** slaweq has quit IRC | 17:13 | |
| dansmith | mriedem: this is about to pass check: https://review.openstack.org/#/c/521186/ | 17:13 |
| dansmith | we gonna get that in soon? | 17:13 |
| mriedem | haven't looked at the latest, | 17:15 |
| *** salv-orlando has joined #openstack-nova | 17:15 | |
| mriedem | was going through your live migration swap allocs change quick | 17:15 |
| *** salv-orlando has quit IRC | 17:16 | |
| *** jmlowe has quit IRC | 17:16 | |
| *** salv-orlando has joined #openstack-nova | 17:16 | |
| *** sree has joined #openstack-nova | 17:19 | |
| *** sree has quit IRC | 17:24 | |
| *** moshele has quit IRC | 17:25 | |
| *** jpena|brb is now known as jpena | 17:25 | |
| *** sahid has quit IRC | 17:27 | |
| mriedem | dansmith: ok looks like the only change in https://review.openstack.org/#/c/521186/ is the NUMATopologyFilter (which is looking at image properties so we should run it on rebuild) and the commit message title | 17:30 |
| mriedem | artom: are you good with ^ now? | 17:32 |
| mriedem | i think the only thing remaining is a release note | 17:32 |
| *** damien_r has quit IRC | 17:32 | |
| *** lyan has quit IRC | 17:36 | |
| *** lyan has joined #openstack-nova | 17:37 | |
| *** efried is now known as efried_rollin | 17:38 | |
| *** chyka has quit IRC | 17:39 | |
| *** chyka has joined #openstack-nova | 17:40 | |
| *** lpetrut has joined #openstack-nova | 17:41 | |
| *** gszasz has joined #openstack-nova | 17:42 | |
| *** lyan has quit IRC | 17:43 | |
| *** toure_biab is now known as toure | 17:44 | |
| *** Apoorva has joined #openstack-nova | 17:45 | |
| *** harlowja has joined #openstack-nova | 17:47 | |
| *** armax has joined #openstack-nova | 17:53 | |
| *** cdent has quit IRC | 17:53 | |
| *** sree has joined #openstack-nova | 17:53 | |
| openstackgerrit | Merged openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 17:53 |
| *** lyan has joined #openstack-nova | 17:55 | |
| *** cdent has joined #openstack-nova | 17:56 | |
| *** sree has quit IRC | 17:57 | |
| dansmith | mriedem: yep, which you were going to do | 18:00 |
| *** ralonsoh has quit IRC | 18:00 | |
| mriedem | yup | 18:00 |
| *** penick has joined #openstack-nova | 18:02 | |
| *** derekh has quit IRC | 18:02 | |
| *** belmoreira has joined #openstack-nova | 18:04 | |
| *** markvoelker has quit IRC | 18:10 | |
| *** fragatin_ has quit IRC | 18:11 | |
| *** sridharg has quit IRC | 18:11 | |
| belmoreira | mriedem, dansmith I have been working in enabling placement in newton with a nova_api DB per cell. I have some comments. Are you around? | 18:12 |
| dansmith | belmoreira: depends, are they nice comments? | 18:13 |
| belmoreira | :) a problem actually. And I would like your guidance | 18:14 |
| belmoreira | A problem that I found is the deletion of the build_request in nova_api. Conductor that runs in the cell will try to delete it from the local nova_api | 18:14 |
| *** suresh12 has joined #openstack-nova | 18:14 | |
| dansmith | does the api cell have a dedicated nova_api db? | 18:15 |
| dansmith | I guess it must | 18:15 |
| *** liangy has joined #openstack-nova | 18:16 | |
| belmoreira | yes, on cellsV1 we also have the same concept | 18:16 |
| dansmith | what I mean is, you have a placement for the api cell, and a placement for each compute cell | 18:16 |
| mriedem | belmoreira: is this when we delete an instance? it gets routed to the child cell api and that tries to delete the build request? | 18:16 |
| dansmith | so I'm guessing the api puts in a buildrequest, but the compute cell tries to delete it during boot | 18:16 |
| dansmith | which means you keep getting the buildrequest listed out of the api, and maybe compute goes to delete it, finds it deleted, and assumes the instance was deleted? | 18:17 |
| belmoreira | dansmith, the plan was to enable placement also in api_cell just to be already there for the pike upgrade | 18:17 |
| *** felipemonteiro has joined #openstack-nova | 18:17 | |
| *** felipemonteiro_ has joined #openstack-nova | 18:18 | |
| *** lyan has quit IRC | 18:18 | |
| *** moshele has joined #openstack-nova | 18:18 | |
| belmoreira | dansmith the issue is because the build_request is not deleted, the instance is created but remains in build state when the user list its instances | 18:18 |
| dansmith | okay so the first half of what I said | 18:19 |
| belmoreira | yes | 18:19 |
| dansmith | belmoreira: so you'll need something in the instance sync back to the api cell that deletes the build request | 18:19 |
| belmoreira | dansmith what I did was to change delete_build_request to delete the request from the nova_api DB in the top cell | 18:20 |
| belmoreira | basically adding a new conf option to point to that DB that is used in this function | 18:20 |
| *** jpena is now known as jpena|off | 18:20 | |
| dansmith | belmoreira: I think you want to delete the buildrequest here: https://github.com/openstack/nova/blob/master/nova/cells/messaging.py#L998 | 18:20 |
| dansmith | if you can test that, I think we could probably merge that change | 18:21 |
| *** bkopilov has quit IRC | 18:21 | |
| belmoreira | but it does it in the nova_api DB? | 18:22 |
| *** felipemonteiro has quit IRC | 18:22 | |
| dansmith | that code will run at the top-level nova-cells service, which would be configured to point to the top-level nova_api db right? | 18:23 |
| belmoreira | I wasn't aware that nova-cells would sync nova_api DB. Only nova DB in top cell. | 18:24 |
| dansmith | well, it doesn't do anything other than what you tell it to | 18:24 |
| *** lpetrut has quit IRC | 18:25 | |
| dansmith | historically there was no api db, so it didn't do anything there, but if you put code in there to find/delete the buildrequest then it will start to do some of that :) | 18:25 |
| belmoreira | I see :) | 18:25 |
| belmoreira | do we already have an example that nova_api DB is updated? | 18:25 |
| dansmith | in nova-cells? not that I know of | 18:26 |
| dansmith | it'll need to be graceful about buildrequest being deleted already, for the people that have a unified api db for all their cellsv1 cells | 18:27 |
| dansmith | are you asking because of consistency or just because you're not sure how to go about the lookup/delete/ | 18:28 |
| belmoreira | my current patch changes the the delete_build_request in conductor/manager to delete it in the nova_api cell. I wasn't aiming to merge it upstream :) | 18:29 |
| dansmith | because it should be roughly the thing done in delete_build_request | 18:29 |
| dansmith | okay but wouldn't you rather a change that doesn't need a new config option and _is_ upstream? | 18:29 |
| belmoreira | I started this conversation because I'm not sure if we are missing something else | 18:30 |
| belmoreira | or this can have any consequence when upgrading then to ocata/pike | 18:30 |
| belmoreira | dansmith sure I would prefer a proper fix | 18:31 |
| *** lpetrut has joined #openstack-nova | 18:33 | |
| belmoreira | dansmith thanks. I will have a look on how to have it in cells/messaging | 18:35 |
| dansmith | belmoreira: | 18:36 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Fix cellsv1 build request deletion if multiple api databases are used https://review.openstack.org/523187 | 18:36 |
| dansmith | belmoreira: see if that ^ works and/or tweak until it does and then we can move on with it | 18:36 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Fix cellsv1 build request deletion if multiple api databases are used https://review.openstack.org/523187 | 18:37 |
| belmoreira | dansmith: thanks a lot. I will let you know | 18:40 |
| dansmith | belmoreira: cool | 18:40 |
| *** fragatina has joined #openstack-nova | 18:47 | |
| *** fragatin_ has joined #openstack-nova | 18:48 | |
| artom | mriedem, haven't had a chance to look at the latest version yet | 18:50 |
| artom | Will do that now | 18:50 |
| *** cdent has quit IRC | 18:51 | |
| *** fragatina has quit IRC | 18:51 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:52 | |
| *** gouthamr has joined #openstack-nova | 18:52 | |
| *** belmoreira has quit IRC | 18:52 | |
| *** belmoreira has joined #openstack-nova | 18:53 | |
| *** shaner has quit IRC | 18:54 | |
| *** moshele has quit IRC | 18:54 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Use oslo_db Session in resource_provider.py https://review.openstack.org/523192 | 18:54 |
| jaypipes | mriedem, efried_rollin: ^ | 18:54 |
| *** AlexeyAbashkin has quit IRC | 18:56 | |
| *** moshele has joined #openstack-nova | 18:57 | |
| mriedem | that is a large change, but looks like it's inevitable | 18:58 |
| *** eharney has quit IRC | 18:58 | |
| *** jmlowe has joined #openstack-nova | 18:59 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix 'force' parameter in os-quota-sets PUT schema https://review.openstack.org/523194 | 18:59 |
| mriedem | sdague: ^ is that 1 line os-quota-sets 'force' regression fix. the functional api samples are just to show the regression | 19:00 |
| *** lpetrut has quit IRC | 19:01 | |
| *** vladikr has quit IRC | 19:03 | |
| *** linkmark has joined #openstack-nova | 19:04 | |
| *** shaner has joined #openstack-nova | 19:05 | |
| artom | Ohhh, the "need to open every PS individually to leave comments on each" thing is gone! | 19:05 |
| artom | Man, my life just keeps on improving | 19:06 |
| artom | This morning I learned about pssh | 19:06 |
| jaypipes | mriedem: well, I figured it would be silly to fix only the conn.execute() calls in the lookup_entry_in_table() function and leave the other call locations. | 19:06 |
| *** penick has quit IRC | 19:06 | |
| mriedem | jaypipes: if we need to backport that fix, then it would be best to fix the one place and do the massive refactor on top | 19:08 |
| *** penick has joined #openstack-nova | 19:08 | |
| jaypipes | mriedem: there's latent bugs in all those other call locations. | 19:08 |
| jaypipes | mriedem: you want to backport >1 patch or just one? | 19:09 |
| jaypipes | mriedem: also, I'm happy to create a backport patch for pike/ocata | 19:09 |
| *** lpetrut has joined #openstack-nova | 19:09 | |
| mriedem | i single backport patch would be ideal, however, with the amount of refactoring that's gone on already in queens, the backport is probably going to suck, but if you want to own the backport then that's ok with me for a single change | 19:10 |
| mriedem | i don't know that it needs to go to ocata | 19:10 |
| mriedem | i think this actually crept up in CI because posting allocations during scheduling failed | 19:11 |
| mriedem | which wasn't in ocata | 19:11 |
| jaypipes | mriedem: I'll handle the backport to pike, then, no prob. | 19:12 |
| sdague | mriedem: +2, though I'm still rather down on all the deepcopy and modify pattern, I think it creates more of these bugs in the future | 19:13 |
| sdague | vs. just do a full expand every time there is a new tweak on the schema | 19:13 |
| *** dtantsur is now known as dtantsur|afk | 19:14 | |
| *** pcaruana has joined #openstack-nova | 19:16 | |
| *** penick has quit IRC | 19:17 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:17 | |
| mriedem | sdague: thanks | 19:22 |
| mriedem | dansmith: here is my proposed reno https://gist.github.com/mriedem/3da0a2edf18be0487e8becad31293077 | 19:22 |
| *** AlexeyAbashkin has quit IRC | 19:22 | |
| mriedem | for https://review.openstack.org/#/c/521186/ | 19:22 |
| openstackgerrit | Merged openstack/nova master: finish refactor AllocCandidates._get_by_filters() https://review.openstack.org/516782 | 19:22 |
| openstackgerrit | Merged openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 19:23 |
| openstackgerrit | Merged openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 19:23 |
| *** salv-orlando has quit IRC | 19:28 | |
| *** salv-orlando has joined #openstack-nova | 19:28 | |
| *** sree has joined #openstack-nova | 19:28 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refined fix for validating image on rebuild https://review.openstack.org/521186 | 19:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 19:31 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 19:31 |
| *** burt has joined #openstack-nova | 19:31 | |
| mriedem | jaypipes: i added a reno on https://review.openstack.org/521186 which you were +2 on | 19:31 |
| *** penick has joined #openstack-nova | 19:32 | |
| cfriesen | when updating the default quota class set, if I set a value lower than the current usage will that prevent any more resources from being consumed? updating quota-sets has the "force" option for this case, but I don't see that for quota class sets. | 19:33 |
| *** sree has quit IRC | 19:33 | |
| *** yangyapeng has joined #openstack-nova | 19:34 | |
| mriedem | cfriesen: quota classes aren't tied to a tenant | 19:36 |
| mriedem | so there is no 'current usage' on a quota class | 19:37 |
| mriedem | as for "if I set a value lower than the current usage will that prevent any more resources from being consumed" - i'm not sure | 19:37 |
| mriedem | it might | 19:37 |
| cfriesen | mriedem: I was envisioning usage as being basically per tenant. | 19:38 |
| *** felipemonteiro_ has quit IRC | 19:38 | |
| mriedem | we check limits based on (1) is there a specific limit for this project, (2) if not, is there a specific limit for this quota class, (3) if not, what's the configured limit | 19:38 |
| cfriesen | so you have the class default as 10, the tenant currently has 9, I change the class default to 8 | 19:38 |
| mriedem | i think that will cause the next instance create to be overquota | 19:39 |
| *** yangyapeng has quit IRC | 19:39 | |
| cfriesen | that matches my understanding, thanks. it's odd we have "force" for that when setting tenant quotas but not the global default | 19:39 |
| mriedem | the point in changing the limits to be below the usage (like the force parameter in PUT /os-quota-sets) is to get to the desired limit regardless of current usage | 19:39 |
| *** moshele has quit IRC | 19:39 | |
| mriedem | if we checked that on the global default, we'd have to check usage for all projects | 19:40 |
| mriedem | and, | 19:40 |
| mriedem | we could be tracking quota for projects that no longer actually exist in keystone | 19:40 |
| cfriesen | true enough | 19:40 |
| openstackgerrit | Merged openstack/nova master: Implement query param schema for delete assisted vol https://review.openstack.org/513668 | 19:40 |
| openstackgerrit | Merged openstack/nova master: Versioned notifications for service create and delete https://review.openstack.org/519588 | 19:41 |
| *** artom has quit IRC | 19:48 | |
| *** artom has joined #openstack-nova | 19:49 | |
| *** eharney has joined #openstack-nova | 19:53 | |
| *** eharney has quit IRC | 19:53 | |
| *** eharney has joined #openstack-nova | 19:53 | |
| *** ijw has quit IRC | 19:56 | |
| *** sambetts is now known as sambetts|afk | 19:59 | |
| *** dave-mcc_ is now known as dave-mccowan | 20:04 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Refined fix for validating image on rebuild https://review.openstack.org/523212 | 20:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/523213 | 20:08 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix doubling allocations on rebuild https://review.openstack.org/523214 | 20:08 |
| *** vladikr has joined #openstack-nova | 20:11 | |
| *** vladikr has quit IRC | 20:13 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: SEA https://review.openstack.org/523216 | 20:15 |
| *** penick has quit IRC | 20:16 | |
| *** vladikr has joined #openstack-nova | 20:16 | |
| *** sree has joined #openstack-nova | 20:16 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:17 | |
| cfriesen | mriedem: when querying quota details including usage for a tenant/user, does it only count resources created by that user? | 20:19 |
| *** slaweq has joined #openstack-nova | 20:19 | |
| *** slaweq has quit IRC | 20:20 | |
| *** slaweq has joined #openstack-nova | 20:20 | |
| mriedem | cfriesen: you should talk to melwitt | 20:20 |
| *** AlexeyAbashkin has quit IRC | 20:21 | |
| *** sree has quit IRC | 20:21 | |
| *** moshele has joined #openstack-nova | 20:22 | |
| mriedem | cfriesen: in general quotas are on the usage in the project, | 20:27 |
| mriedem | except for keypairs | 20:27 |
| mriedem | you can see how usage is counted by the methods called for each CountableResource here https://github.com/openstack/nova/blob/master/nova/quota.py#L1903 | 20:28 |
| cfriesen | mriedem: thanks | 20:28 |
| *** suresh12 has quit IRC | 20:30 | |
| mriedem | so i guess PUT /os-quota-sets/{tenant_id} is a bit weird if you're updating limits for keypairs, since those are per-user, so that should really be something like PUT /os-qouta-sets/{tenant_id}/key_pairs/{user_id} - or something like that | 20:30 |
| *** jmlowe has quit IRC | 20:31 | |
| *** suresh12 has joined #openstack-nova | 20:31 | |
| *** yamahata has quit IRC | 20:32 | |
| cfriesen | quotas in general is a mess. the mapping between the HTTP API and the internal representation is really indirect. Looking at the quota for a user/tenant, it takes multiple queries to figure if the limit came from a tenant/user quota, a tenant quota, the default quota class, or the default in the code or config file. | 20:32 |
| *** threestrands_ has joined #openstack-nova | 20:32 | |
| *** threestrands_ has quit IRC | 20:32 | |
| *** threestrands_ has joined #openstack-nova | 20:32 | |
| *** threestrands_ has quit IRC | 20:33 | |
| *** threestrands_ has joined #openstack-nova | 20:34 | |
| *** threestrands has joined #openstack-nova | 20:34 | |
| *** amodi has joined #openstack-nova | 20:34 | |
| *** threestrands has quit IRC | 20:34 | |
| *** threestrands has joined #openstack-nova | 20:34 | |
| artom | mriedem, dansmith, I think we may have gotten a filter wrong, but I could just be being an idiot. Review left. | 20:34 |
| mriedem | cfriesen: that's what i explained before | 20:35 |
| mriedem | cfriesen: and why i wrote https://review.openstack.org/#/c/411035/ | 20:35 |
| *** efried_rollin is now known as efried | 20:38 | |
| *** tasker has quit IRC | 20:39 | |
| *** threestrands_ has quit IRC | 20:44 | |
| mriedem | artom: so just AggregateMultiTenancyIsolation at this point | 20:47 |
| mriedem | ? | 20:47 |
| artom | mriedem, yeah. Am I correct in wondering why it's True? | 20:48 |
| mriedem | yeah i think that's a carry over from PS5 when it used to be called POLICY_FILTER | 20:48 |
| mriedem | and got lumped in as 'policy' | 20:48 |
| mriedem | https://review.openstack.org/#/c/521186/5..7/nova/scheduler/filters/aggregate_multitenancy_isolation.py | 20:48 |
| artom | Cool, so I'm not in the field | 20:49 |
| mriedem | ok so i think i'll just change that one unless dansmith objects | 20:50 |
| dansmith | I'm sure I don't | 20:51 |
| artom | dansmith is like this pigeon at this point: https://imgur.com/gallery/Y5Ps3 | 20:52 |
| dansmith | pretty much | 20:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Refined fix for validating image on rebuild https://review.openstack.org/521186 | 20:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 20:53 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 20:53 |
| mriedem | artom: are you happy now? | 20:53 |
| artom | mriedem, sorry what? I've gone and joined dansmith and the pigeon on the dog: https://i.imgur.com/ijgWRzw.jpg | 20:56 |
| artom | mriedem, in seriousness though, yes. Thanks for being patient :) | 20:56 |
| *** vladikr has quit IRC | 20:56 | |
| mriedem | approved, thanks for being thorough | 20:57 |
| mriedem | just like henry david | 20:57 |
| openstackgerrit | Merged openstack/nova master: Updated from global requirements https://review.openstack.org/522929 | 20:57 |
| artom | Oh and this means I get to do all those downstream CVE backports I signed for | 20:58 |
| artom | So, actually, I have another nit I'd like to bikeshed over for the entire week | 20:59 |
| *** penick has joined #openstack-nova | 21:02 | |
| *** yamahata has joined #openstack-nova | 21:04 | |
| *** sree has joined #openstack-nova | 21:05 | |
| *** vladikr has joined #openstack-nova | 21:05 | |
| *** Apoorva_ has joined #openstack-nova | 21:05 | |
| *** Apoorva has quit IRC | 21:09 | |
| *** sree has quit IRC | 21:09 | |
| *** liverpooler has quit IRC | 21:11 | |
| *** gbarros has quit IRC | 21:12 | |
| *** gbarros has joined #openstack-nova | 21:15 | |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: POC: tests: autospecs all the mock.patch usages https://review.openstack.org/470775 | 21:15 |
| claudiub|2 | mriedem: hi. if you're still interested about mock autospecing, the above patch enforces autospec on all mock.patch calls. it does have a few hacks though, if you're willing to accept the compromise. | 21:17 |
| *** smatzek has quit IRC | 21:18 | |
| *** damien_r has joined #openstack-nova | 21:22 | |
| *** amodi has quit IRC | 21:22 | |
| *** gbarros has quit IRC | 21:22 | |
| *** ijw has joined #openstack-nova | 21:22 | |
| *** moshele has quit IRC | 21:22 | |
| belmoreira | dansmith: was testing what you proposed. I though I had a good solution... now I feel embarrassed :) | 21:23 |
| belmoreira | dansmith: It works great! thanks | 21:23 |
| *** gbarros has joined #openstack-nova | 21:24 | |
| dansmith | belmoreira: cool, glad to hear it | 21:24 |
| belmoreira | dansmith: do you think this can still be added into newton? | 21:26 |
| dansmith | belmoreira: not backported upstream most likely, but it should apply easily (as you know) | 21:26 |
| belmoreira | ok, thanks again | 21:27 |
| *** hamzy has quit IRC | 21:27 | |
| *** ijw has quit IRC | 21:30 | |
| cfriesen | mriedem: did we ever figure out what to do about keypairs and users/tenants? (like where one user creates a heat stack and another within the same tenant can't delete it) | 21:32 |
| cfriesen | can't find anything in the etherpads | 21:32 |
| mriedem | cfriesen: you can now rebuild an instance with a new key_name | 21:34 |
| mriedem | https://specs.openstack.org/openstack/nova-specs/specs/queens/approved/rebuild-keypair-reset.html | 21:34 |
| *** belmoreira has quit IRC | 21:34 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 21:35 |
| *** Tahvok has quit IRC | 21:35 | |
| cfriesen | mriedem: right, I knew about that part. but I thought some people were talking about how keypairs should really be owned by the tenant. Could be it never actually got written down. | 21:37 |
| mriedem | cfriesen: i'm not really remembering that | 21:37 |
| mriedem | cfriesen: maybe sdague does | 21:37 |
| cfriesen | thx | 21:38 |
| *** ijw has joined #openstack-nova | 21:39 | |
| *** pcaruana has quit IRC | 21:41 | |
| sdague | keypairs are the only resource not owned by the tenant | 21:43 |
| *** ijw has quit IRC | 21:43 | |
| sdague | it makes for a very weird model of ownership, because the point was supposed to be that tenants owned all the things, and users were just permissions to be part of a tenant | 21:44 |
| *** hamzy has joined #openstack-nova | 21:44 | |
| *** gszasz has quit IRC | 21:49 | |
| *** pchavva has quit IRC | 21:53 | |
| *** burt has quit IRC | 21:53 | |
| *** gbarros has quit IRC | 21:55 | |
| *** awaugama has quit IRC | 21:56 | |
| *** cdent has joined #openstack-nova | 21:57 | |
| *** esberglu has quit IRC | 21:57 | |
| *** esberglu has joined #openstack-nova | 21:58 | |
| *** lpetrut has quit IRC | 21:59 | |
| *** penick has quit IRC | 22:04 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 22:04 |
| *** rtjure has quit IRC | 22:08 | |
| *** ijw has joined #openstack-nova | 22:09 | |
| *** markvoelker has joined #openstack-nova | 22:09 | |
| *** cdent has quit IRC | 22:10 | |
| *** ijw has quit IRC | 22:10 | |
| *** edand has joined #openstack-nova | 22:11 | |
| *** penick has joined #openstack-nova | 22:12 | |
| *** edand has quit IRC | 22:16 | |
| melwitt | cfriesen, mriedem: I recall some discussion about how user A in a tenant can't ever view user B's keypairs in the same tenant. I just can't remember where/when that was | 22:19 |
| melwitt | it's so baked in that it would probably be difficult to change | 22:19 |
| cfriesen | melwitt: true. what about the the weirdness of per-tenant keypair quotas when keypairs belong to users across all tenants? | 22:21 |
| *** ijw has joined #openstack-nova | 22:22 | |
| melwitt | cfriesen: currently, I think you can set keypair quota per tenant but what it means is "how many keypairs can each user in the tenant have" | 22:23 |
| melwitt | are you thinking of a per tenant quota that means "how many keypairs can this tenant have in total"? | 22:23 |
| cfriesen | melwitt: but a user can exist in multiple tenants, so which one applies? | 22:23 |
| melwitt | whichever one the user is authing as at the moment | 22:24 |
| melwitt | i.e. the tenant passed in the REST API request | 22:24 |
| cfriesen | okay, that makes sense in a weird sort of way. | 22:24 |
| cfriesen | it does mean your usage in one tenant could go up due to activity in another tenant | 22:25 |
| melwitt | ah yeah, that's true. keypairs are counted per user. | 22:27 |
| mriedem | yikun: good morning. :) do we really need https://review.openstack.org/#/c/517273/ to add paging support for the os-migrations API? can't we just find the starting cell for the marker by iterating the cells until we find the one with the marker migration? that's obviously less efficient since we don't have a mapping in the API database, but i don't think performance here is as big a concern as it is for paging instances acros | 22:27 |
| mriedem | ltiple cells. | 22:27 |
| zigo | I'm getting a weird error in my tempest test env: qemu-img: file system may not support O_DIRECT\nqemu-img: Could not open '/var/lib/nova/instances/_base/c44b0b620ae7c6fd8111e0abb5a8d1fc39fcdf08.converted': Could not open '/var/lib/nova/instances/_base/c44b0b620ae7c6fd8111e0abb5a8d1fc39fcdf08.converted': Invalid argument | 22:30 |
| zigo | It looks like qemu-img doesn't like me running on a filesystem using overlay for / over tmpfs ... | 22:31 |
| zigo | Is there any workaround ? | 22:31 |
| * penick blinks | 22:32 | |
| zigo | penick: Blinking reading what I wrote ? :) | 22:33 |
| melwitt | zigo: what release of nova are you running? | 22:33 |
| zigo | melwitt: 16.0.3 | 22:34 |
| melwitt | I found this stackoverflow thing about it https://serverfault.com/questions/537440/qemu-dont-start-quest-with-disk-image-on-tmpfs which reminded me I've seen code in the libvirt driver that's supposed to handle this | 22:34 |
| *** suresh12 has quit IRC | 22:34 | |
| zigo | I'm trying to functional-validate Pike for Debian. | 22:34 |
| *** rcernin has joined #openstack-nova | 22:34 | |
| zigo | melwitt: It seems related to qemu-img not converting the image before booting it, though... | 22:35 |
| melwitt | this part is checking whether the filesystem supports direct io and will set cache=writethrough in that case to avoid the crash https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L406-L414 | 22:36 |
| zigo | stack dump: http://paste.openstack.org/show/627515/ | 22:36 |
| melwitt | (which is the workaround, cache=writethrough but it should be working around for you automatically) | 22:36 |
| *** damien_r has quit IRC | 22:37 | |
| zigo | Hum ... | 22:38 |
| zigo | melwitt: Is there any way I can just force nova to never use direct IO then? | 22:38 |
| zigo | Even an ugly hack, just to check ... | 22:38 |
| mriedem | dansmith: you might have some ideas here https://review.openstack.org/#/c/517273/21/nova/compute/migration_list.py - i think using the instance list merge sort framework that you wrote is maybe overkill for paging migrations across cells, but i'm unsure if the alternative (including a merge sort of the results in the api) is any less complicated. | 22:38 |
| dansmith | mriedem: definitely if we don't allow sort keys or directions (agreed we shouldn't) | 22:40 |
| zigo | melwitt: I'll try to always set writethrough and see what happens... | 22:40 |
| dansmith | mriedem: looks like it was blindly copied in there huh? | 22:40 |
| melwitt | zigo: something that might work is if you set [libvirt]/disk_cachemodes = file=writethrough | 22:40 |
| dansmith | mriedem: I can apply more brainpower to this tomorrow if needed | 22:40 |
| zigo | melwitt: Thanks, trying now ! | 22:41 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP: PowerVM Driver: SEA https://review.openstack.org/523216 | 22:42 |
| *** jmlowe has joined #openstack-nova | 22:44 | |
| zigo | melwitt: Bad luck, it's still the same log in nova-compute.log ... :/ | 22:45 |
| mriedem | dansmith: there is a patch below it which extracts some of the common object wrapper bits, but otherwise yeah it's mostly a copy of instance_list | 22:45 |
| dansmith | mriedem: yeah, I'm not really in favor of genericifying that stuff unless we need to, it just becomes more complex | 22:45 |
| mriedem | and i'm thinking if the user can't pass sort keys/dirs, (which they can't for os-migrations), then it's less work for us to just find the marker, page from there (like we used to do for instances) and merge sort the results at the end | 22:45 |
| dansmith | yup | 22:45 |
| * zigo tries to mount /var/lib/nova/instances on ext4, just to see what happens... | 22:45 | |
| *** esberglu has quit IRC | 22:46 | |
| mriedem | i know yikun wrote up some test cases so i think he was mostly worried about the merge sort prat | 22:46 |
| mriedem | *part | 22:46 |
| melwitt | zigo: guh :( okay, so we have some bug there where we're not setting cache mode in that particular code path or something like that. I'd have to dig into it more to know what's going on there | 22:46 |
| *** esberglu has joined #openstack-nova | 22:46 | |
| zigo | melwitt: Let me check with /var/lib/nova/instances mounted on ext4, just to make sure that's the issue. | 22:46 |
| melwitt | zigo: what images_type are you using? just flat? or is it volume or ceph? | 22:47 |
| zigo | melwitt: You mean for the image that I'm trying to boot? It's a "nornal" qcow2 image ... | 22:48 |
| zigo | ie: the official Debian OpenStack Stretch image that I maintain in cdimages.debian.org/cdimages/openstack ... | 22:48 |
| zigo | flat file in glance ... | 22:49 |
| melwitt | oh okay, I see | 22:49 |
| *** esberglu has quit IRC | 22:51 | |
| *** hamzy has quit IRC | 22:55 | |
| *** Alex_Staf has joined #openstack-nova | 22:56 | |
| *** Apoorva_ has quit IRC | 22:57 | |
| *** Apoorva has joined #openstack-nova | 22:58 | |
| *** ijw has quit IRC | 22:59 | |
| *** esberglu has joined #openstack-nova | 22:59 | |
| zigo | melwitt: I'm getting a different error due to other issues in my setup (neutron issue apparently), but I'm not seeing the qemu-img problem anymore after mounting over ext4 ! | 22:59 |
| zigo | So definitively there's a problem... | 23:00 |
| zigo | Should I file a bug? | 23:01 |
| melwitt | zigo: yeah, if you could file a bug that would help us look into it | 23:03 |
| *** jmlowe has quit IRC | 23:03 | |
| *** esberglu has quit IRC | 23:04 | |
| *** hamzy has joined #openstack-nova | 23:08 | |
| *** gmann_afk is now known as gmann | 23:11 | |
| gmann | mriedem: melwitt happy to get more review on novaclient zullv3 migration - https://review.openstack.org/#/q/topic:zuulv3-novaclient | 23:13 |
| *** ijw has joined #openstack-nova | 23:15 | |
| zigo | melwitt: https://bugs.launchpad.net/nova/+bug/1734784 | 23:18 |
| openstack | Launchpad bug 1734784 in OpenStack Compute (nova) "Cannot boot instances on filesystem without O_DIRECT support (fails on tmpfs)" [Undecided,New] | 23:18 |
| *** liangy has quit IRC | 23:19 | |
| *** mlavalle has quit IRC | 23:19 | |
| melwitt | zigo: ack, thanks | 23:22 |
| *** dave-mccowan has quit IRC | 23:24 | |
| *** ircuser-1 has joined #openstack-nova | 23:24 | |
| *** slaweq has quit IRC | 23:25 | |
| *** linkmark has quit IRC | 23:26 | |
| *** slaweq has joined #openstack-nova | 23:27 | |
| *** slaweq has quit IRC | 23:33 | |
| *** chyka_ has joined #openstack-nova | 23:33 | |
| *** jmlowe has joined #openstack-nova | 23:35 | |
| *** chyka has quit IRC | 23:36 | |
| *** suresh12 has joined #openstack-nova | 23:44 | |
| *** suresh12 has quit IRC | 23:44 | |
| *** suresh12 has joined #openstack-nova | 23:44 | |
| *** john51 has quit IRC | 23:45 | |
| *** john51 has joined #openstack-nova | 23:45 | |
| *** claudiub|2 has quit IRC | 23:49 | |
| *** marst has quit IRC | 23:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Deprecate file injection https://review.openstack.org/522027 | 23:57 |
| *** mriedem has quit IRC | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
