| *** armax has joined #openstack-nova | 00:02 | |
| *** esberglu has joined #openstack-nova | 00:04 | |
| *** smatzek has joined #openstack-nova | 00:07 | |
| *** itlinux has quit IRC | 00:07 | |
| *** esberglu has quit IRC | 00:08 | |
| *** mlavalle has quit IRC | 00:08 | |
| *** gfhellma has joined #openstack-nova | 00:10 | |
| *** smatzek has quit IRC | 00:11 | |
| *** tetsuro has joined #openstack-nova | 00:13 | |
| *** gyee_ has quit IRC | 00:13 | |
| *** amodi has quit IRC | 00:14 | |
| *** armax has quit IRC | 00:19 | |
| *** mriedem has joined #openstack-nova | 00:20 | |
| mriedem | efried: jaypipes: https://review.openstack.org/#/c/516778/ | 00:20 |
|---|---|---|
| mriedem | I think I'm following that, but some questions in the tests. | 00:20 |
| *** claudiub|2 has quit IRC | 00:22 | |
| *** slaweq has joined #openstack-nova | 00:24 | |
| *** sdague has quit IRC | 00:25 | |
| *** s1061123_ has quit IRC | 00:26 | |
| *** slaweq has quit IRC | 00:28 | |
| *** andreas_s has joined #openstack-nova | 00:29 | |
| *** hieulq has quit IRC | 00:30 | |
| *** s1061123 has joined #openstack-nova | 00:30 | |
| *** hieulq has joined #openstack-nova | 00:31 | |
| *** gouthamr has quit IRC | 00:31 | |
| *** markvoelker has quit IRC | 00:31 | |
| *** chyka has quit IRC | 00:31 | |
| *** armax has joined #openstack-nova | 00:33 | |
| *** andreas_s has quit IRC | 00:33 | |
| *** zhurong has joined #openstack-nova | 00:34 | |
| *** armax_ has joined #openstack-nova | 00:34 | |
| efried | mriedem Thanks, responded. | 00:36 |
| *** yangyapeng has quit IRC | 00:37 | |
| *** armax has quit IRC | 00:38 | |
| *** armax_ is now known as armax | 00:38 | |
| mriedem | efried: i'm good with a cleanup at the end of the series | 00:38 |
| efried | mriedem Thanks. | 00:38 |
| mriedem | efried: will wait for alex_xu to get back on that one, i see he's done a lot of testing with it | 00:38 |
| *** gbarros has quit IRC | 00:39 | |
| efried | mriedem Roger wilco. Thanks for helping get that series moving. | 00:39 |
| *** Sukhdev has joined #openstack-nova | 00:43 | |
| *** markvoelker has joined #openstack-nova | 00:43 | |
| *** yingjun has joined #openstack-nova | 00:44 | |
| *** Swami has quit IRC | 00:45 | |
| *** armax has quit IRC | 00:46 | |
| *** mriedem has quit IRC | 00:48 | |
| *** furlongm has quit IRC | 00:50 | |
| *** gmann_afk is now known as gmann | 00:50 | |
| *** yamamoto_ has quit IRC | 00:57 | |
| *** liverpooler has joined #openstack-nova | 00:57 | |
| *** Apoorva has joined #openstack-nova | 01:02 | |
| *** yamamoto has joined #openstack-nova | 01:03 | |
| *** Apoorva has quit IRC | 01:06 | |
| *** chyka has joined #openstack-nova | 01:07 | |
| *** phuongnh has joined #openstack-nova | 01:07 | |
| *** Apoorva has joined #openstack-nova | 01:08 | |
| *** ellioth has joined #openstack-nova | 01:08 | |
| *** yamamoto has quit IRC | 01:08 | |
| *** gouthamr has joined #openstack-nova | 01:10 | |
| *** chyka has quit IRC | 01:11 | |
| *** cheneydc has joined #openstack-nova | 01:17 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Update Instance action's updated_at when action event updated. https://review.openstack.org/507473 | 01:21 |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: [WIP] Add pagination and changes-since for instance-actions https://review.openstack.org/326326 | 01:21 |
| *** gfhellma has quit IRC | 01:23 | |
| *** tinwood has quit IRC | 01:29 | |
| *** tinwood has joined #openstack-nova | 01:30 | |
| *** zhurong has quit IRC | 01:34 | |
| *** hongbin has quit IRC | 01:35 | |
| *** jaypipes has quit IRC | 01:36 | |
| *** Sukhdev has quit IRC | 01:38 | |
| openstackgerrit | Yikun Jiang proposed openstack/nova master: Add pagination and Changes-since filter support for os-migrations. https://review.openstack.org/330406 | 01:40 |
| *** yamamoto has joined #openstack-nova | 01:48 | |
| *** naichuans has quit IRC | 01:51 | |
| *** liuyulong has joined #openstack-nova | 01:53 | |
| *** yamamoto has quit IRC | 01:53 | |
| *** Apoorva_ has joined #openstack-nova | 01:57 | |
| *** Apoorva has quit IRC | 02:00 | |
| *** Apoorva_ has quit IRC | 02:01 | |
| *** salv-orlando has joined #openstack-nova | 02:02 | |
| *** salv-orl_ has quit IRC | 02:05 | |
| *** lyan has joined #openstack-nova | 02:07 | |
| *** andreas_s has joined #openstack-nova | 02:09 | |
| *** trungnv has quit IRC | 02:11 | |
| *** armax has joined #openstack-nova | 02:13 | |
| *** yamamoto has joined #openstack-nova | 02:14 | |
| *** andreas_s has quit IRC | 02:14 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 02:18 |
| *** takashin has quit IRC | 02:20 | |
| *** yangyapeng has joined #openstack-nova | 02:24 | |
| *** slaweq has joined #openstack-nova | 02:24 | |
| *** slaweq has quit IRC | 02:29 | |
| *** annp has joined #openstack-nova | 02:30 | |
| *** fragatina has quit IRC | 02:30 | |
| *** lyan has quit IRC | 02:33 | |
| *** fragatina has joined #openstack-nova | 02:33 | |
| *** vivsoni has joined #openstack-nova | 02:36 | |
| *** fragatina has quit IRC | 02:37 | |
| *** namnh has joined #openstack-nova | 02:45 | |
| *** vivsoni has quit IRC | 02:50 | |
| *** vivsoni_ has joined #openstack-nova | 02:50 | |
| *** yamahata has joined #openstack-nova | 02:51 | |
| *** itlinux has joined #openstack-nova | 02:53 | |
| *** armax has quit IRC | 02:55 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 02:57 |
| *** armax has joined #openstack-nova | 02:59 | |
| *** salv-orl_ has joined #openstack-nova | 03:00 | |
| *** lyan has joined #openstack-nova | 03:01 | |
| *** salv-orlando has quit IRC | 03:02 | |
| *** links has joined #openstack-nova | 03:03 | |
| *** akekane has joined #openstack-nova | 03:13 | |
| *** akekane is now known as abhishekk | 03:14 | |
| openstackgerrit | Chen Hanxiao proposed openstack/python-novaclient master: inject file: add description of injecting multiple files. https://review.openstack.org/515655 | 03:16 |
| *** fragatina has joined #openstack-nova | 03:21 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 03:21 |
| *** hongbin has joined #openstack-nova | 03:22 | |
| *** yamamoto has quit IRC | 03:23 | |
| *** yamamoto has joined #openstack-nova | 03:25 | |
| *** lyan has quit IRC | 03:25 | |
| *** fragatina has quit IRC | 03:25 | |
| *** yamamoto has quit IRC | 03:28 | |
| *** psachin has joined #openstack-nova | 03:31 | |
| *** andreas_s has joined #openstack-nova | 03:33 | |
| *** itlinux has quit IRC | 03:36 | |
| *** Sukhdev has joined #openstack-nova | 03:37 | |
| *** andreas_s has quit IRC | 03:38 | |
| *** yamamoto has joined #openstack-nova | 03:40 | |
| *** esberglu has joined #openstack-nova | 03:40 | |
| *** esberglu has quit IRC | 03:41 | |
| *** esberglu has joined #openstack-nova | 03:41 | |
| *** esberglu has quit IRC | 03:41 | |
| *** esberglu has joined #openstack-nova | 03:41 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/nova-specs master: Add validation to improve pep8 check for nova-specs project https://review.openstack.org/519561 | 03:44 |
| *** yamamoto has quit IRC | 03:45 | |
| *** itlinux has joined #openstack-nova | 03:46 | |
| *** esberglu has quit IRC | 03:46 | |
| *** andreas_s has joined #openstack-nova | 03:49 | |
| *** eharney has quit IRC | 03:49 | |
| *** eharney has joined #openstack-nova | 03:50 | |
| *** andreas_s has quit IRC | 03:53 | |
| *** sridharg has joined #openstack-nova | 03:59 | |
| *** takashin has joined #openstack-nova | 04:01 | |
| *** gbarros has joined #openstack-nova | 04:02 | |
| *** vivsoni_ has quit IRC | 04:04 | |
| *** vivsoni has joined #openstack-nova | 04:04 | |
| *** gbarros has quit IRC | 04:05 | |
| *** eharney has quit IRC | 04:09 | |
| *** yamamoto has joined #openstack-nova | 04:10 | |
| *** yamamoto has quit IRC | 04:10 | |
| *** vivsoni has quit IRC | 04:12 | |
| *** dave-mccowan has quit IRC | 04:19 | |
| *** itlinux has quit IRC | 04:24 | |
| *** nikhil has quit IRC | 04:25 | |
| openstackgerrit | yuanyue proposed openstack/nova master: Add log in conductor to mark host for instance creation https://review.openstack.org/520369 | 04:32 |
| *** itlinux has joined #openstack-nova | 04:33 | |
| *** itlinux has quit IRC | 04:33 | |
| *** cheneydc has quit IRC | 04:39 | |
| *** hieulq has quit IRC | 04:42 | |
| *** phuongnh has quit IRC | 04:42 | |
| *** phuongnh has joined #openstack-nova | 04:42 | |
| *** hieulq has joined #openstack-nova | 04:42 | |
| *** itlinux has joined #openstack-nova | 04:49 | |
| *** andreas_s has joined #openstack-nova | 04:50 | |
| *** udesale has joined #openstack-nova | 04:50 | |
| *** diga has joined #openstack-nova | 04:52 | |
| *** vladikr has quit IRC | 04:54 | |
| *** andreas_s has quit IRC | 04:54 | |
| *** cheneydc has joined #openstack-nova | 04:59 | |
| *** zerda3 has joined #openstack-nova | 05:00 | |
| *** yamamoto has joined #openstack-nova | 05:10 | |
| *** jaosorior has joined #openstack-nova | 05:12 | |
| *** fragatina has joined #openstack-nova | 05:12 | |
| *** fragatina has quit IRC | 05:14 | |
| *** fragatina has joined #openstack-nova | 05:15 | |
| *** yamamoto has quit IRC | 05:17 | |
| *** janki has joined #openstack-nova | 05:22 | |
| *** claudiub has joined #openstack-nova | 05:22 | |
| *** hongbin has quit IRC | 05:23 | |
| *** armax has quit IRC | 05:23 | |
| *** yangyapeng has quit IRC | 05:24 | |
| BenderRodriguez | I need help | 05:25 |
| BenderRodriguez | WHenever I try to create an instance, I get the following error and the VM is stuck in the ERROR state when doing openstack server show... | 05:25 |
| BenderRodriguez | The error message shown in nova-conductor.log is as follows: NoValidHost: No valid host was found. There are not enough hosts available. | 05:25 |
| BenderRodriguez | What does this mean? | 05:25 |
| *** Sukhdev has quit IRC | 05:26 | |
| *** vivsoni has joined #openstack-nova | 05:30 | |
| *** trungnv has joined #openstack-nova | 05:34 | |
| *** gouthamr has quit IRC | 05:34 | |
| *** itlinux has quit IRC | 05:35 | |
| *** ratailor has joined #openstack-nova | 05:36 | |
| *** ratailor has quit IRC | 05:36 | |
| *** ratailor has joined #openstack-nova | 05:36 | |
| *** itlinux has joined #openstack-nova | 05:38 | |
| *** Sukhdev has joined #openstack-nova | 05:41 | |
| *** Sukhdev has quit IRC | 05:42 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Microversion 2.54 - Enable reset keypair while rebuild https://review.openstack.org/519572 | 05:45 |
| *** itlinux has quit IRC | 05:51 | |
| *** yangyapeng has joined #openstack-nova | 05:52 | |
| *** yangyapeng has quit IRC | 05:56 | |
| *** diga has quit IRC | 05:58 | |
| *** psachin_ has joined #openstack-nova | 06:00 | |
| *** psachin has quit IRC | 06:01 | |
| *** yangyapeng has joined #openstack-nova | 06:10 | |
| *** krtaylor has quit IRC | 06:14 | |
| *** yangyapeng has quit IRC | 06:15 | |
| *** lpetrut_ has joined #openstack-nova | 06:19 | |
| *** slaweq has joined #openstack-nova | 06:26 | |
| *** slaweq has quit IRC | 06:30 | |
| *** krtaylor has joined #openstack-nova | 06:31 | |
| *** yangyapeng has joined #openstack-nova | 06:36 | |
| *** ijw has joined #openstack-nova | 06:36 | |
| openstackgerrit | Ghanshyam Mann proposed openstack/nova master: Merge flavor extensions controller code https://review.openstack.org/516104 | 06:39 |
| *** zhurong has joined #openstack-nova | 06:40 | |
| *** yangyapeng has quit IRC | 06:41 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 06:42 |
| *** belmoreira has joined #openstack-nova | 06:43 | |
| *** ijw has quit IRC | 06:44 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach interface https://review.openstack.org/512190 | 06:46 |
| *** yangyapeng has joined #openstack-nova | 06:51 | |
| *** ellioth has quit IRC | 06:54 | |
| *** lokioden has joined #openstack-nova | 06:56 | |
| *** diga has joined #openstack-nova | 06:57 | |
| *** markvoelker has quit IRC | 06:59 | |
| *** pcaruana has joined #openstack-nova | 07:04 | |
| *** ijw has joined #openstack-nova | 07:12 | |
| *** phuongnh has quit IRC | 07:15 | |
| *** phuongnh has joined #openstack-nova | 07:16 | |
| *** ijw has quit IRC | 07:17 | |
| *** rcernin has quit IRC | 07:18 | |
| *** esberglu has joined #openstack-nova | 07:18 | |
| *** esberglu has quit IRC | 07:19 | |
| *** esberglu has joined #openstack-nova | 07:19 | |
| *** Oku_OS-away is now known as Oku_OS | 07:19 | |
| *** esberglu has quit IRC | 07:19 | |
| *** psachin_ has quit IRC | 07:20 | |
| *** esberglu has joined #openstack-nova | 07:20 | |
| *** esberglu has quit IRC | 07:25 | |
| *** namnh has quit IRC | 07:27 | |
| *** takashin has left #openstack-nova | 07:28 | |
| *** andreas_s has joined #openstack-nova | 07:29 | |
| *** yamamoto has joined #openstack-nova | 07:29 | |
| *** andreas_s has quit IRC | 07:29 | |
| *** andreas_s has joined #openstack-nova | 07:29 | |
| *** lpetrut_ has quit IRC | 07:31 | |
| *** psachin has joined #openstack-nova | 07:33 | |
| *** sahid has joined #openstack-nova | 07:34 | |
| *** yamamoto has quit IRC | 07:34 | |
| *** yamamoto has joined #openstack-nova | 07:36 | |
| *** psachin has quit IRC | 07:38 | |
| *** lpetrut_ has joined #openstack-nova | 07:39 | |
| *** psachin has joined #openstack-nova | 07:39 | |
| *** tetsuro has quit IRC | 07:41 | |
| *** trungnv has quit IRC | 07:42 | |
| *** zhurong has quit IRC | 07:43 | |
| *** ijw has joined #openstack-nova | 07:47 | |
| *** diga has quit IRC | 07:53 | |
| *** ijw has quit IRC | 07:53 | |
| *** kmalloc has quit IRC | 07:54 | |
| *** markvoelker has joined #openstack-nova | 08:00 | |
| *** AlexeyAbashkin has joined #openstack-nova | 08:01 | |
| *** edand_ has joined #openstack-nova | 08:03 | |
| *** edand_ has quit IRC | 08:08 | |
| *** lpetrut_ has quit IRC | 08:11 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 08:13 |
| *** alexchadin has joined #openstack-nova | 08:24 | |
| *** jmlowe has quit IRC | 08:27 | |
| *** amotoki has joined #openstack-nova | 08:31 | |
| *** yamamoto has quit IRC | 08:34 | |
| *** vivsoni has quit IRC | 08:36 | |
| *** lpetrut_ has joined #openstack-nova | 08:43 | |
| *** yamamoto has joined #openstack-nova | 08:44 | |
| *** vivsoni has joined #openstack-nova | 08:45 | |
| *** belmoreira has quit IRC | 08:47 | |
| *** jpena|off is now known as jpena | 08:49 | |
| *** ralonsoh has joined #openstack-nova | 08:49 | |
| *** yamamoto has quit IRC | 08:49 | |
| *** lpetrut_ has quit IRC | 08:50 | |
| *** lpetrut_ has joined #openstack-nova | 08:50 | |
| *** sdatko has joined #openstack-nova | 08:51 | |
| *** gaoyanami has joined #openstack-nova | 08:52 | |
| *** lpetrut_ has quit IRC | 08:57 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 09:00 |
| *** alexchadin has quit IRC | 09:05 | |
| *** alexchadin has joined #openstack-nova | 09:05 | |
| *** slaweq has joined #openstack-nova | 09:06 | |
| *** yamamoto has joined #openstack-nova | 09:14 | |
| *** yamamoto has quit IRC | 09:19 | |
| *** yamamoto has joined #openstack-nova | 09:19 | |
| *** derekh has joined #openstack-nova | 09:20 | |
| *** lucas-afk is now known as lucasagomes | 09:33 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: (WIP)xenapi(N-R-P): Add API to support compute node resource provider update and create https://review.openstack.org/521041 | 09:37 |
| *** cheneydc has quit IRC | 09:44 | |
| *** sridharg has quit IRC | 09:46 | |
| *** alexchadin has quit IRC | 09:46 | |
| *** alexchadin has joined #openstack-nova | 09:47 | |
| *** alexchadin has quit IRC | 09:57 | |
| *** alexchadin has joined #openstack-nova | 09:58 | |
| *** yingjun has quit IRC | 10:00 | |
| *** esberglu has joined #openstack-nova | 10:01 | |
| *** esberglu has quit IRC | 10:05 | |
| *** sapd__ has quit IRC | 10:09 | |
| *** sapd__ has joined #openstack-nova | 10:14 | |
| *** fragatin_ has joined #openstack-nova | 10:25 | |
| *** fragatina has quit IRC | 10:27 | |
| *** slaweq_ has joined #openstack-nova | 10:28 | |
| *** lpetrut_ has joined #openstack-nova | 10:30 | |
| *** slaweq_ has quit IRC | 10:32 | |
| *** yamamoto has quit IRC | 10:36 | |
| *** gmann is now known as gmann_afk | 10:37 | |
| *** annp has quit IRC | 10:37 | |
| *** isp has joined #openstack-nova | 10:42 | |
| isp | Hi, im using nova.servers.list() to list the instances I have, what could I get the info of each vm? something like nova.servers.show(vmName) or where could I get that functions implemented? | 10:43 |
| *** yamamoto has joined #openstack-nova | 10:50 | |
| *** phuongnh has quit IRC | 10:54 | |
| *** udesale has quit IRC | 10:56 | |
| *** dtantsur|afk is now known as dtantsur | 10:57 | |
| *** yamamoto has quit IRC | 10:59 | |
| *** abhishekk has quit IRC | 11:00 | |
| *** alexchadin has quit IRC | 11:04 | |
| *** alexchadin has joined #openstack-nova | 11:05 | |
| *** [1]lokioden has joined #openstack-nova | 11:13 | |
| *** claudiub|2 has joined #openstack-nova | 11:13 | |
| *** tetsuro has joined #openstack-nova | 11:13 | |
| *** lokioden has quit IRC | 11:14 | |
| *** [1]lokioden is now known as lokioden | 11:14 | |
| *** lpetrut_ has quit IRC | 11:14 | |
| *** claudiub has quit IRC | 11:15 | |
| *** lpetrut has joined #openstack-nova | 11:15 | |
| *** alexchadin has quit IRC | 11:17 | |
| *** slaweq has quit IRC | 11:28 | |
| *** chyka has joined #openstack-nova | 11:33 | |
| *** carthaca_ has joined #openstack-nova | 11:34 | |
| *** dgonzalez_ has joined #openstack-nova | 11:34 | |
| *** dgonzalez_ has quit IRC | 11:35 | |
| *** carthaca_ has quit IRC | 11:35 | |
| *** chyka has quit IRC | 11:38 | |
| *** tbachman has quit IRC | 11:43 | |
| *** yamamoto has joined #openstack-nova | 11:44 | |
| *** tetsuro has quit IRC | 11:44 | |
| *** snasir has quit IRC | 11:53 | |
| *** ratailor has quit IRC | 11:54 | |
| *** snasir has joined #openstack-nova | 11:54 | |
| *** yamahata has quit IRC | 11:55 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [CLI] Fix token auth type https://review.openstack.org/424256 | 12:00 |
| openstackgerrit | yuanyue proposed openstack/nova master: Add log in conductor to mark host for instance creation https://review.openstack.org/520369 | 12:03 |
| *** yingjun has joined #openstack-nova | 12:05 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [CLI] Fix token auth type https://review.openstack.org/424256 | 12:07 |
| *** sdague has joined #openstack-nova | 12:09 | |
| *** janki has quit IRC | 12:11 | |
| *** dtantsur is now known as dtantsur|brb | 12:12 | |
| *** yamamoto has quit IRC | 12:13 | |
| *** udesale has joined #openstack-nova | 12:13 | |
| efried | BenderRodriguez This is really a question for the #openstack channel. But check the compute logs for errors; that should give you a hint as to what went wrong. | 12:16 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 12:23 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper https://review.openstack.org/521071 | 12:24 |
| *** lucasagomes is now known as lucas-hungry | 12:33 | |
| *** yamamoto has joined #openstack-nova | 12:33 | |
| *** yamamoto has quit IRC | 12:37 | |
| *** sahid has quit IRC | 12:38 | |
| *** smatzek has joined #openstack-nova | 12:39 | |
| *** clayton has quit IRC | 12:44 | |
| *** esberglu has joined #openstack-nova | 12:45 | |
| *** yangyapeng has quit IRC | 12:45 | |
| *** yangyapeng has joined #openstack-nova | 12:46 | |
| *** gcb has quit IRC | 12:47 | |
| *** yangyapeng has quit IRC | 12:48 | |
| *** yangyapeng has joined #openstack-nova | 12:49 | |
| *** esberglu has quit IRC | 12:49 | |
| *** jpena is now known as jpena|lunch | 12:50 | |
| *** yangyapeng has quit IRC | 12:53 | |
| efried | Ahdang! | 12:55 |
| *** efried is now known as fried_rice | 12:55 | |
| *** slaweq has joined #openstack-nova | 12:57 | |
| *** psachin has quit IRC | 13:01 | |
| *** yamamoto has joined #openstack-nova | 13:01 | |
| *** yamamoto has quit IRC | 13:01 | |
| *** dtantsur|brb is now known as dtantsur | 13:03 | |
| *** yamamoto has joined #openstack-nova | 13:03 | |
| *** liuyulong has quit IRC | 13:16 | |
| *** edmondsw has joined #openstack-nova | 13:17 | |
| fried_rice | alex_xu Can you please review the nrp series? https://review.openstack.org/#/c/377138/ | 13:18 |
| *** links has quit IRC | 13:19 | |
| *** edleafe is now known as figleaf | 13:21 | |
| *** tbachman_ has joined #openstack-nova | 13:24 | |
| *** hieulq has quit IRC | 13:28 | |
| *** clayton has joined #openstack-nova | 13:29 | |
| *** janki has joined #openstack-nova | 13:30 | |
| *** lucas-hungry is now known as lucasagomes | 13:30 | |
| *** andreas_s has quit IRC | 13:33 | |
| *** andreas_s has joined #openstack-nova | 13:33 | |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 13:33 |
| *** markvoelker has quit IRC | 13:34 | |
| openstackgerrit | Merged openstack/python-novaclient master: Remove setting of version/release from releasenotes https://review.openstack.org/520733 | 13:34 |
| *** markvoelker has joined #openstack-nova | 13:34 | |
| *** slaweq has quit IRC | 13:34 | |
| *** sbezverk has quit IRC | 13:35 | |
| *** sbezverk has joined #openstack-nova | 13:36 | |
| *** andreas_s has quit IRC | 13:38 | |
| *** clayton has quit IRC | 13:38 | |
| *** clayton has joined #openstack-nova | 13:39 | |
| *** jaypipes has joined #openstack-nova | 13:39 | |
| *** dklyle has quit IRC | 13:48 | |
| alex_xu | fried_rice: yea, that is my next plan | 13:49 |
| fried_rice | alex_xu Cool | 13:50 |
| *** [1]lokioden has joined #openstack-nova | 13:50 | |
| *** lokioden has quit IRC | 13:52 | |
| *** [1]lokioden is now known as lokioden | 13:52 | |
| *** jpena|lunch is now known as jpena | 13:52 | |
| *** slaweq has joined #openstack-nova | 13:53 | |
| *** artom has quit IRC | 13:58 | |
| *** dtantsur is now known as dtantsur|brb | 14:03 | |
| *** yamamoto has quit IRC | 14:04 | |
| *** gabor_antal_ has quit IRC | 14:04 | |
| *** lyan has joined #openstack-nova | 14:07 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | 14:08 |
| *** gouthamr has joined #openstack-nova | 14:10 | |
| *** jaypipes is now known as leakypipes | 14:13 | |
| fried_rice | leakypipes Happy Friday! | 14:14 |
| leakypipes | morning fried_rice! :) | 14:14 |
| *** yamamoto has joined #openstack-nova | 14:15 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova master: VMware: fix memory stats https://review.openstack.org/516634 | 14:18 |
| *** mriedem has joined #openstack-nova | 14:18 | |
| *** esberglu has joined #openstack-nova | 14:21 | |
| *** dave-mcc_ has joined #openstack-nova | 14:24 | |
| *** gbarros has joined #openstack-nova | 14:27 | |
| *** udesale has quit IRC | 14:31 | |
| *** smatzek has quit IRC | 14:31 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper https://review.openstack.org/521071 | 14:32 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 14:32 |
| *** smatzek has joined #openstack-nova | 14:32 | |
| *** smatzek has quit IRC | 14:32 | |
| *** smatzek has joined #openstack-nova | 14:32 | |
| *** karthiks has quit IRC | 14:36 | |
| *** smatzek has quit IRC | 14:37 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper https://review.openstack.org/521071 | 14:39 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 14:39 |
| *** READ10 has joined #openstack-nova | 14:40 | |
| *** yamamoto has quit IRC | 14:44 | |
| *** hongbin has joined #openstack-nova | 14:48 | |
| ArchiFleKs | hi do you know how to enable soft-(anti)-affinity in nova ? I have the soft_affinity_weight_multiplier define in nova but the api response : openstack server group create: error: argument --policy: invalid choice: u'soft-anti-affinity' (choose from 'affinity', 'anti-affinity'). I'm on stable/pike | 14:48 |
| mriedem | ArchiFleKs: the soft policies have to be used with a microversion when creating the group | 14:49 |
| mriedem | https://developer.openstack.org/api-ref/compute/#create-server-group | 14:49 |
| mriedem | "This policy was added in microversion 2.15." | 14:49 |
| *** yamamoto has joined #openstack-nova | 14:50 | |
| mriedem | so send -H "X-OpenStack-Nova-API-Version: 2.15" | 14:50 |
| *** amodi has joined #openstack-nova | 14:51 | |
| ArchiFleKs | mriedem: my current version says 2.53 | 14:52 |
| *** yamamoto has quit IRC | 14:54 | |
| *** yingjun has quit IRC | 14:55 | |
| mriedem | current version? | 14:56 |
| mriedem | are you sending a microversion header in the request when creating the server group? | 14:56 |
| mriedem | what tool are you using to create the server group? nova CLI? curl, other? | 14:56 |
| *** ratailor has joined #openstack-nova | 14:57 | |
| *** yamahata has joined #openstack-nova | 14:58 | |
| *** smatzek has joined #openstack-nova | 14:58 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Check quota before creating volume snapshots https://review.openstack.org/520158 | 14:59 |
| *** smatzek has quit IRC | 14:59 | |
| *** psachin has joined #openstack-nova | 14:59 | |
| *** smatzek has joined #openstack-nova | 14:59 | |
| *** janki has quit IRC | 15:00 | |
| *** yingjun has joined #openstack-nova | 15:01 | |
| ArchiFleKs | mriedem: "status": "CURRENT", "updated": "2013-07-23T11:33:21Z", "links": [{"href": "https://compute:8774/v2.1/", "rel": "self"}], "min_version": "2.1", "version": "2.53", "id": "v2.1"}]}. I'd like to do it with openstack cli | 15:01 |
| *** dtantsur|brb is now known as dtantsur | 15:02 | |
| ArchiFleKs | can I pass microversion with --os-compute-api-version ? | 15:02 |
| *** mlavalle has joined #openstack-nova | 15:03 | |
| mriedem | ArchiFleKs: that's just the version doc | 15:03 |
| mriedem | telling you what the compute endpoint supports | 15:03 |
| mriedem | microversions are per-request | 15:03 |
| mriedem | and have different meanings | 15:03 |
| mriedem | nova CLI defaults to the latest available microversion in the compute endpoint and what the client side code understands, | 15:03 |
| mriedem | openstack CLI does not do that | 15:04 |
| mriedem | so with OSC, you'll need to specify the --os-compute-api-version option | 15:04 |
| mriedem | openstack --os-compute-api-version 2.15 server group create ... | 15:04 |
| mriedem | something like that | 15:04 |
| *** artom has joined #openstack-nova | 15:04 | |
| mriedem | fyi https://developer.openstack.org/api-guide/compute/microversions.html https://docs.openstack.org/nova/latest/reference/api-microversion-history.html | 15:05 |
| ArchiFleKs | mriedem: thanks | 15:06 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.all_by_uuid() https://review.openstack.org/520243 | 15:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643 | 15:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree https://review.openstack.org/520663 | 15:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 15:08 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 15:08 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates https://review.openstack.org/521097 | 15:08 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 15:08 |
| *** Apoorva has joined #openstack-nova | 15:08 | |
| *** eharney has joined #openstack-nova | 15:08 | |
| ArchiFleKs | mriedem: I tried to do that with os-compute-api-version but it stil fails, I can see that the compute version used is 2.15 in the --debug log though | 15:08 |
| *** yingjun has quit IRC | 15:08 | |
| *** artom_ has joined #openstack-nova | 15:09 | |
| *** andreas_s has joined #openstack-nova | 15:09 | |
| ArchiFleKs | mriedem: but it works with nova cli | 15:09 |
| mriedem | can you put the osc --debug output in a paste? | 15:09 |
| *** yingjun has joined #openstack-nova | 15:10 | |
| *** coreywright has quit IRC | 15:10 | |
| *** armax has joined #openstack-nova | 15:11 | |
| *** udesale has joined #openstack-nova | 15:11 | |
| *** udesale has quit IRC | 15:12 | |
| *** artom has quit IRC | 15:12 | |
| fried_rice | leakypipes I have a test error in https://review.openstack.org/#/c/520643/ which I'm fixing, but would you please take a look and let me know if you're satisfied with my other responses, so I can make all the changes at once? | 15:12 |
| *** andreas_s has quit IRC | 15:13 | |
| *** marst has joined #openstack-nova | 15:15 | |
| *** jaosorior has quit IRC | 15:19 | |
| *** Apoorva has quit IRC | 15:21 | |
| zigo | bauzas: Hi there! Do you know if Nova 16.0.3 includes CVE-2017-16239_Validate_new_image_via_scheduler_during_rebuild.patch ? | 15:21 |
| zigo | It looks like it to me... | 15:21 |
| zigo | But I'd like to be 100% sure. | 15:21 |
| * zigo looks at the git log | 15:21 | |
| bauzas | zigo: AFAIR, yes | 15:22 |
| leakypipes | fried_rice: yup, doing now. | 15:22 |
| zigo | bauzas: The git log shows it does. I'll push that to Debian unstable then. | 15:22 |
| *** coreywright has joined #openstack-nova | 15:23 | |
| *** yingjun has quit IRC | 15:23 | |
| *** david-lyle has joined #openstack-nova | 15:24 | |
| *** liuyulong has joined #openstack-nova | 15:24 | |
| ArchiFleKs | mriedem: http://paste.openstack.org/show/626622/ | 15:24 |
| zigo | bauzas: Do you know if that bug can be triggered by someone just having a tenant in the cloud, or is it less grave? | 15:25 |
| openstackgerrit | Nguyen Van Trung proposed openstack/nova-specs master: Add validation to improve pep8 check for nova-specs project https://review.openstack.org/519561 | 15:26 |
| zigo | bauzas: I'm just trying to figure out if I should upload to stretch-security or just to the next point release... | 15:26 |
| bauzas | zigo: I need to disappear for a couple of minutes (children at school) | 15:26 |
| zigo | Sure ! | 15:27 |
| zigo | mriedem: Maybe you could reply to me about this? | 15:28 |
| mriedem | ArchiFleKs: that looks like a bug in openstackclient | 15:28 |
| mriedem | i wonder if it's whitelisting the policy options | 15:28 |
| zigo | mriedem: The question was: for the last CVE on Nova (re-schedule issue), can it be triggered by a non-priviledged tenant, or is it just an admin issue? | 15:28 |
| mriedem | zigo: tenant | 15:29 |
| zigo | mriedem: Allright, so this grants a debian-security upload then. | 15:29 |
| zigo | Thanks. | 15:29 |
| mriedem | create server with image A, rebuild server with image B | 15:29 |
| *** ratailor has quit IRC | 15:31 | |
| *** itlinux has joined #openstack-nova | 15:31 | |
| mriedem | leakypipes: fried_rice: fyi, cyborg is looking for some granuler provider filtering stuff in placement, https://etherpad.openstack.org/p/cyborg-feature-tag - looking over that, it sounds like they are looking for a solver scheduler though | 15:34 |
| fried_rice | mriedem Cool, looking... | 15:34 |
| fried_rice | mriedem What's a "solver scheduler"? | 15:34 |
| mriedem | https://www.youtube.com/watch?v=Fsw-hjtQ8e4 | 15:34 |
| mriedem | https://github.com/openstack/nova-solver-scheduler | 15:34 |
| *** lyan has quit IRC | 15:35 | |
| *** yamamoto has joined #openstack-nova | 15:35 | |
| mriedem | alternative to filter scheduler with pluggable filters/weighers for custom optimizations | 15:35 |
| mriedem | long before placement | 15:35 |
| zigo | mriedem: If you don't mind the comment: it'd be nice, next time, to include the CVE number in the patch description. I'll talk about it to Tristant. | 15:35 |
| leakypipes | mriedem: you lose your conversation rights with leakypipes for bringing up solver scheduler. | 15:35 |
| mriedem | zigo: the CVE number didn't exist when i wrote the patches | 15:36 |
| mriedem | sorry | 15:36 |
| leakypipes | mriedem: I'm just kidding :) | 15:36 |
| *** ayoung has joined #openstack-nova | 15:37 | |
| mriedem | i was saying sorry to zigo | 15:37 |
| mriedem | :) | 15:37 |
| mriedem | not the leakiest of pipes | 15:37 |
| *** vladikr has joined #openstack-nova | 15:37 | |
| leakypipes | well, I retract my kidding, then :P | 15:37 |
| mriedem | leakypipes: if it makes you feel better, they also want this in k8s | 15:37 |
| mriedem | so sharing the load | 15:37 |
| leakypipes | mriedem: heh | 15:38 |
| mriedem | openstack can still ruin k8s yet! | 15:38 |
| ayoung | mriedem, well one way we can catch up to Kubernetes is to fix our RBAC | 15:38 |
| ayoung | and on that note, can we please merge https://review.openstack.org/#/c/384148/ as I've been chasing this now for over a year. The tests run right now, and I'd like to avoid more breakages | 15:39 |
| mriedem | ayoung: will fixing our rbac get me to a thinner slimmer openstack for the edge? because that's all anyone cares about | 15:39 |
| ayoung | mriedem, absolutely | 15:39 |
| *** yamamoto has quit IRC | 15:39 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: instance_mapping/request_spec records exist even after the instances are archived https://review.openstack.org/515034 | 15:40 |
| ayoung | mriedem, it will help with all members of U2 | 15:40 |
| *** derekh has quit IRC | 15:40 | |
| artom_ | mriedem, we should start with an openstack with sunglasses for bono, then we can think about the edge | 15:40 |
| *** artom_ is now known as artom | 15:40 | |
| *** gaoyanami has quit IRC | 15:40 | |
| ayoung | Start with the Drummer. He's the sensible one | 15:40 |
| mriedem | https://www.youtube.com/watch?v=UJXqnYCWW7Y | 15:40 |
| ayoung | Anyone, fixing this bug requires getting changes into absolutely every project in OpenStack, and it is not a fun thing to chase | 15:41 |
| *** itlinux has quit IRC | 15:42 | |
| ayoung | its coming up more and more, we are getting dupes of bug filings for it, and the requests for RBAC changes that cannot be done without proper scoping are piling up | 15:42 |
| ayoung | I have a T-shirt for whomever finally pulls the trigger on it | 15:43 |
| *** itlinux has joined #openstack-nova | 15:44 | |
| mriedem | ArchiFleKs: this is the bug https://github.com/openstack/python-openstackclient/blob/master/openstackclient/compute/v2/server_group.py#L58 | 15:45 |
| zigo | Now I get this when building Nova 16.0.3: http://paste.openstack.org/show/626628/ | 15:45 |
| mriedem | ArchiFleKs: osc is hard-coding the choices for the server group policies | 15:45 |
| zigo | This really looks like a broken test, rather than an issue in Nova itself. | 15:46 |
| mriedem | ArchiFleKs: https://bugs.launchpad.net/python-openstackclient/+bug/1732938 | 15:47 |
| openstack | Launchpad bug 1732938 in python-openstackclient "Unable to create server group with soft-anti-affinity policy using microversion 2.15" [Undecided,New] | 15:47 |
| *** ayoung has quit IRC | 15:48 | |
| mriedem | zigo: yes, although the test is mocking out that call | 15:49 |
| mriedem | test_validate_volume_no_mtab_entry | 15:49 |
| mriedem | oops | 15:49 |
| mriedem | @mock.patch.object(psutil, "disk_partitions") | 15:49 |
| *** ayoung has joined #openstack-nova | 15:49 | |
| mriedem | zigo: oh that's not fixed in pike | 15:50 |
| mriedem | sec | 15:50 |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: Fix quobyte test_validate_volume_no_mtab_entry https://review.openstack.org/521118 | 15:50 |
| mriedem | zigo: ^ | 15:50 |
| zigo | mriedem: Wonderful, that's very helpful for me, and much better than blacklisting the test. | 15:51 |
| mriedem | easy bp win https://review.openstack.org/#/c/450636/ | 15:57 |
| *** artom_ has joined #openstack-nova | 16:00 | |
| ArchiFleKs | mriedem: thanks a lot i'll follow that | 16:01 |
| *** david-lyle has quit IRC | 16:02 | |
| *** lyan has joined #openstack-nova | 16:02 | |
| *** david-lyle has joined #openstack-nova | 16:03 | |
| *** artom has quit IRC | 16:04 | |
| bauzas | mriedem: indeed, will look into the vmware log change | 16:05 |
| *** Oku_OS is now known as Oku_OS-away | 16:05 | |
| *** AlexeyAbashkin has quit IRC | 16:05 | |
| *** artom__ has joined #openstack-nova | 16:06 | |
| *** gyee_ has joined #openstack-nova | 16:09 | |
| *** artom_ has quit IRC | 16:09 | |
| *** yamamoto has joined #openstack-nova | 16:20 | |
| *** slaweq has quit IRC | 16:21 | |
| *** yamamoto has quit IRC | 16:25 | |
| *** jaypipes has joined #openstack-nova | 16:27 | |
| *** leakypipes has quit IRC | 16:28 | |
| *** jaypipes is now known as leakypipes | 16:28 | |
| *** artom__ is now known as artom | 16:29 | |
| cfriesen | mriedem: are you planning on updating https://review.openstack.org/#/c/520660/ to query the volume image metadata? (Using _get_bdm_image_metadata() maybe?). If not would you like me to take a crack at it? | 16:37 |
| mriedem | yeah planning on it, was just looking at something related to that | 16:37 |
| *** jistr has quit IRC | 16:39 | |
| fried_rice | Is the multi-node business (e.g. in resource_tracker _init_comput_host) *only* ironic? | 16:42 |
| fried_rice | johnthetubaguy leakypipes mriedem ^ | 16:42 |
| *** jistr has joined #openstack-nova | 16:43 | |
| leakypipes | fried_rice: no. also vCenter, IIRC. | 16:44 |
| fried_rice | k | 16:44 |
| *** gfhellma has joined #openstack-nova | 16:44 | |
| leakypipes | fried_rice: but Ironic is the canonical example, yes. | 16:44 |
| dansmith | leakypipes: no, it's only ironic | 16:44 |
| leakypipes | dansmith: oh? | 16:44 |
| * leakypipes waits for superdan to appear. | 16:44 | |
| *** Guest95810 has quit IRC | 16:44 | |
| dansmith | there was a vmware use of it long ago but it's been gone for a while | 16:45 |
| *** gfhellma has quit IRC | 16:45 | |
| leakypipes | ah, k | 16:45 |
| * dansmith is nowhere near super today | 16:45 | |
| *** gfhellma_ has joined #openstack-nova | 16:45 | |
| leakypipes | dansmith: no? :( | 16:45 |
| dansmith | leakypipes: I'm still suffering from Australia Pox | 16:45 |
| dansmith | the country really does try to kill people | 16:45 |
| leakypipes | ouch. | 16:46 |
| *** ijw has joined #openstack-nova | 16:46 | |
| *** yamahata has quit IRC | 16:47 | |
| mriedem | i think the vcenter multinode thing was removed in kilo | 16:48 |
| *** lucasagomes is now known as lucas-afk | 16:48 | |
| mriedem | liberty: https://github.com/openstack/nova/commit/2f7403bd7200a01e350cde9182c273562e0c9c62#diff-9c54c69030bce3e6af0deba002ab6b27 | 16:50 |
| *** Apoorva has joined #openstack-nova | 16:51 | |
| fried_rice | Dig, thanks y'all. | 16:51 |
| *** ijw has quit IRC | 16:52 | |
| *** chyka has joined #openstack-nova | 16:53 | |
| leakypipes | fried_rice: you on the unit test failure beginning in https://review.openstack.org/#/c/520643/? | 16:59 |
| fried_rice | leakypipes Yes, it's in my sandbox. | 17:00 |
| leakypipes | kk | 17:00 |
| leakypipes | just checkin. | 17:00 |
| fried_rice | leakypipes I'm not hurrying to push it up, since there's so many patches in front of that subseries. | 17:00 |
| leakypipes | k, np. you'd just asked me to review that and the ones above it. | 17:00 |
| leakypipes | in any case, I've added my thoughts. | 17:00 |
| leakypipes | fried_rice: ^ | 17:01 |
| fried_rice | leakypipes Yeah, the ones not marked WIP should be ready, and (assuming no review-prompted changes) should only need rebases once that UT fix goes in. | 17:01 |
| *** psachin has quit IRC | 17:02 | |
| *** fried_rice is now known as fried_rolls | 17:06 | |
| mriedem | cfriesen: oh another thing, today we *always* update instance.image_ref to the new image id provided during rebuild, so even if you rebuild a bfv instance with the same image as the original, we break that contractual thing in the API where instance.image_ref == None means volume-backed | 17:09 |
| *** tbachman_ is now known as tbachman | 17:13 | |
| tbachman | fried_rolls: menu change? | 17:14 |
| cfriesen | yeah...was thinking that we'd have to modify your patch to make that update conditional on BFV, and potentially deal with fallout from code expecting instance.image_ref to be set | 17:14 |
| cfriesen | mriedem: ^ | 17:15 |
| mriedem | code as in rebuild code in the compute service? | 17:17 |
| *** yamamoto has joined #openstack-nova | 17:18 | |
| *** yamamoto has quit IRC | 17:19 | |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [functional] Remove duplication of boot helper https://review.openstack.org/521071 | 17:20 |
| openstackgerrit | Andrey Kurilin proposed openstack/python-novaclient master: [ci] Use pseudo-random names for new resources https://review.openstack.org/479777 | 17:20 |
| cfriesen | mriedem: yes | 17:21 |
| mriedem | problem with that is, we could have backlevel computes, assuming there would need to be a change in the computes to handle that | 17:22 |
| cfriesen | mriedem: alternately we'd have to switch to some other method of representing BFV, or else have nova-api null it out for BFV instances | 17:22 |
| mriedem | but it shouldn't be a problem b/c i think we pass the new image_id down through rpc | 17:22 |
| *** hemna_ has quit IRC | 17:23 | |
| *** hemna_ has joined #openstack-nova | 17:23 | |
| cfriesen | mriedem: just realized that _get_bdm_image_metadata() will give the image metadata, but not the image id itself. | 17:24 |
| mriedem | yeah it should because of compute.utils.get_image_metadata_from_volume | 17:26 |
| mriedem | oh nvm you're right | 17:26 |
| mriedem | i think | 17:26 |
| cfriesen | that loops over VIM_IMAGE_ATTRIBUTES and pops them | 17:26 |
| *** andreas_s has joined #openstack-nova | 17:30 | |
| cfriesen | mriedem: I think we want something like this...totally untested and partly copied from_get_bdm_image_metadata() : http://paste.openstack.org/show/626644/ | 17:30 |
| cfriesen | could even move it down a bit after we calculate root_bdm and save doing it twice. | 17:33 |
| *** ralonsoh has quit IRC | 17:33 | |
| *** Apoorva has quit IRC | 17:34 | |
| *** andreas_s has quit IRC | 17:35 | |
| *** brault has quit IRC | 17:39 | |
| cfriesen | mriedem: new version, cleaned up a bit: http://paste.openstack.org/show/626648/ | 17:41 |
| *** akekane has joined #openstack-nova | 17:41 | |
| *** akekane is now known as abhishekk | 17:42 | |
| mriedem | you shouldn't need "if snapshot_id:" | 17:42 |
| mriedem | a bdm that is attached to an instance should always have volume_id set | 17:42 |
| mriedem | if you bfv with a snapshot, nova creates a volume from the snapshot and stores the created volume_id on the bdm | 17:42 |
| cfriesen | ah, okay | 17:43 |
| mriedem | feel free to update my change with a co-author | 17:43 |
| mriedem | i'm in the middle of something else | 17:43 |
| cfriesen | sure | 17:43 |
| cfriesen | should solve the other bug as well I think, by not going through the scheduler if they're the same | 17:43 |
| *** salv-orlando has joined #openstack-nova | 17:46 | |
| *** salv-orl_ has quit IRC | 17:46 | |
| *** dtantsur is now known as dtantsur|afk | 17:48 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: WIP: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | 17:48 |
| cfriesen | crud, messed that up | 17:50 |
| openstackgerrit | Chris Friesen proposed openstack/nova master: WIP: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | 17:52 |
| alaski | \quit | 17:54 |
| alaski | \quit | 17:54 |
| alaski | grr | 17:54 |
| dansmith | cover. blown. | 17:54 |
| *** gfhellma_ has quit IRC | 18:02 | |
| *** gbarros has quit IRC | 18:02 | |
| *** esberglu has quit IRC | 18:03 | |
| *** abhishekk has quit IRC | 18:07 | |
| leakypipes | mriedem: bottom five patches in validate_provider_summaries series are signed off by alex now, starting here: https://review.openstack.org/#/c/516778 | 18:10 |
| *** sapcc-bot2 has quit IRC | 18:12 | |
| *** sapcc-bot has joined #openstack-nova | 18:12 | |
| *** jpena is now known as jpena|off | 18:14 | |
| *** Apoorva has joined #openstack-nova | 18:17 | |
| *** Apoorva has quit IRC | 18:17 | |
| *** tbachman has quit IRC | 18:17 | |
| *** Apoorva has joined #openstack-nova | 18:18 | |
| *** Swami has joined #openstack-nova | 18:19 | |
| *** yamamoto has joined #openstack-nova | 18:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 18:19 |
| mriedem | dansmith: ^ | 18:19 |
| mriedem | gonna work on the todo in there to not use placement for getting the usage | 18:20 |
| dansmith | mriedem: you wanna I work on the actual fix then? | 18:20 |
| mriedem | if you have a pretty clear idea on how to start it, sure | 18:20 |
| dansmith | I don't have pretty clear anything these days, but I'll get it rolling | 18:20 |
| dansmith | mriedem: you specifically don't want to revert and re-fix then? | 18:21 |
| mriedem | idk | 18:21 |
| mriedem | also, bypassing filters is one thing, but we'd also have to apply that to bypassing placement | 18:21 |
| dansmith | for my backports it'd be much cleaner if we reverted, | 18:21 |
| dansmith | but I understand the optics of reverting a cve | 18:21 |
| *** gbarros has joined #openstack-nova | 18:24 | |
| *** yamamoto has quit IRC | 18:27 | |
| dansmith | mriedem: imagepropertiesfilter is what will kick out your test instance right? | 18:36 |
| mriedem | in which test? | 18:36 |
| mriedem | test_rebuild_with_image_novalidhost | 18:36 |
| dansmith | your new one | 18:37 |
| dansmith | test_rebuild_with_new_image | 18:37 |
| mriedem | in the new test, the scheduler doesn't kick it out | 18:37 |
| mriedem | it asserts that it's incorrectly doubling allocations | 18:37 |
| dansmith | but you're changing a thing that properties filter would kick out/ | 18:37 |
| mriedem | if i maxed out inventory on the node before the rebuild, then placement would kick it out | 18:37 |
| mriedem | ? | 18:38 |
| mriedem | that's the test that went with the cve | 18:38 |
| dansmith | right, what am I thinking. | 18:38 |
| mriedem | are you expecting the new test to create, fill up compute capacity, then rebuild and fail? | 18:39 |
| dansmith | no | 18:39 |
| mriedem | ok - we probably want a test like that anyway, what i'm testing in the new one is something else, and equally bad | 18:39 |
| dansmith | I still have mitaka on the brain, I was forgetting that master is already "fixed" | 18:39 |
| *** gfhellma has joined #openstack-nova | 18:40 | |
| dansmith | mriedem: so your test_rebuild_with_image_novalidhost passes if I remove your requested_destination= change | 18:44 |
| dansmith | does that test actually have more than one compute in it? | 18:45 |
| mriedem | no it doesn't | 18:45 |
| mriedem | it passes because the rebuild fails yes? | 18:46 |
| dansmith | well, I dunno, I'll have to look deeper | 18:47 |
| dansmith | it fails if I also remove the host=None trigger | 18:47 |
| dansmith | so it's failing to schedule even without the destination set | 18:48 |
| dansmith | I guess that makes sense actually if there's only one host | 18:50 |
| mriedem | hmm, if you remove the host=None from the API, we pass instance.host which should bypass the scheduler | 18:50 |
| dansmith | right, which is just the old behavior, which means it doesn't fail and so the test fails | 18:51 |
| mriedem | the test will fail if you remove host=None b/c it's asserting that rebuild raises a 500 | 18:51 |
| mriedem | yeah | 18:51 |
| dansmith | it's just hard to be confident that it's working when you set destination=theonlyone | 18:51 |
| mriedem | yeah, we can easily spin up another compute in the test before the rebuild part | 18:52 |
| dansmith | well, about that | 18:52 |
| dansmith | I'm also wondering if we should have asserted that the host we got from the scheduler is the same host | 18:52 |
| dansmith | because we're assuming that since we asked for the same host we either got it back or got nothing, | 18:52 |
| dansmith | but if we got back something different we'd kindof half evacuate | 18:52 |
| dansmith | you know, another way this breaks existing rebuilds, | 18:54 |
| mriedem | so assert that once the rebuild is done, the original instance.host matches the new instance.host | 18:54 |
| dansmith | is if the admin has set the compute=disabled so that no new things go to that compute, you suddenly can't rebuild anymore, which should be possible | 18:54 |
| dansmith | assert in the test you mean? | 18:54 |
| mriedem | yeah in the test | 18:55 |
| dansmith | that makes it non-deterministic though | 18:55 |
| dansmith | because sometimes we'll get back the same host by chance | 18:55 |
| mriedem | if we had 2 hosts, | 18:55 |
| mriedem | and used a weigher so that the scheduler would normally always pick the *other* host | 18:55 |
| mriedem | then assert that the rebuild never actually moves the instance | 18:55 |
| dansmith | yeah, just more complicated | 18:56 |
| mriedem | also, yes, i didn't think about rebuilding on a disabled compuete | 18:56 |
| mriedem | *compute | 18:56 |
| mriedem | suppose you thought about that b/c of the evacuate thing | 18:56 |
| dansmith | I was thinking about ways to ensure we get the other host in the test :) | 18:57 |
| dansmith | actually, the default fake driver only has one vcpu, so we might always get the other one by virtue of that | 18:58 |
| mriedem | SmallFakeDriver has 2 vcpus | 18:58 |
| mriedem | that had to be updated b/c of our doubling up tests | 18:58 |
| dansmith | I made them use the Medium one | 19:00 |
| dansmith | https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1216 | 19:00 |
| dansmith | we changed the small one too? | 19:00 |
| mriedem | yeah, when the tests were switched from using the chance scheduler to the filter scheduler | 19:02 |
| mriedem | since filter scheduler is using placement and does the claims | 19:02 |
| mriedem | I12de2e195022593ea2a3e2894f2c3b5226930d4f | 19:02 |
| dansmith | hrm | 19:05 |
| dansmith | hmm, seems like all weighers are enabled by default, which would mean we'd pick the other node by default | 19:09 |
| *** lpetrut has quit IRC | 19:09 | |
| mriedem | b/c packing? | 19:13 |
| mriedem | we have some functional tests which provide a custom weigher so the test can control which hosts are picked | 19:13 |
| mriedem | blarg, i can't change https://review.openstack.org/#/c/521153/ to use the hypervisors api to check for a doubling up of allocations using the *_used values on the compute node b/c rebuild doesn't do an RT claim | 19:15 |
| mriedem | so there goes that idea | 19:15 |
| dansmith | I think I'm not really starting up the other computes like I think | 19:16 |
| *** fried_rolls is now known as fried_rice | 19:17 | |
| *** tbachman has joined #openstack-nova | 19:20 | |
| ericyoung | I've got a review up for adding ScaleIO ephemeral volume support. | 19:21 |
| ericyoung | it needs to be rebased, any idea when people might start looking at it? | 19:21 |
| ericyoung | https://review.openstack.org/#/c/407440/ | 19:21 |
| *** marst has quit IRC | 19:21 | |
| *** marst_ has joined #openstack-nova | 19:21 | |
| ericyoung | if soo, I will prioritize the rebase | 19:21 |
| *** fragatin_ has quit IRC | 19:22 | |
| dansmith | mriedem: also note the use of samehost=foo will require SameHostFilter to be enabled, which isn't by default | 19:24 |
| mriedem | yeah | 19:26 |
| *** tdurakov has joined #openstack-nova | 19:27 | |
| *** marst_ has quit IRC | 19:27 | |
| openstackgerrit | Chris Friesen proposed openstack/nova master: WIP: Fail fast if changing image on a volume-backed server rebuild https://review.openstack.org/520660 | 19:28 |
| *** itlinux has quit IRC | 19:31 | |
| fried_rice | leakypipes Talk to me about how the resources from get_available_resource are dealt with today, vis-à-vis placement. | 19:31 |
| fried_rice | Do we *just* peel out the CPU, mem, and disk, and let the rest percolate through the old-school way? | 19:31 |
| leakypipes | fried_rice: depends on the virt drive. | 19:31 |
| leakypipes | driver... | 19:31 |
| leakypipes | fried_rice: for libvirt, yes, that's what we do. for ironic, we also handle the custom resource class if the Ironic node's resource_class attribute is set. | 19:32 |
| leakypipes | fried_rice: get_available_resource() is still called for both of those drivers, though. the result of get_available_resource() is used to track NUMA and PCI device resources. | 19:32 |
| *** itlinux has joined #openstack-nova | 19:33 | |
| fried_rice | leakypipes The ironic thing is done via get_inventory, though, nah? | 19:34 |
| leakypipes | fried_rice: yes, sorry, that's what I thought you were asking about. | 19:34 |
| *** gouthamr has quit IRC | 19:35 | |
| leakypipes | fried_rice: we don't "peel anything out" of the get_available_resource() response. rather, the get_available_resource() return is simply ignored by placement/reportclient. | 19:35 |
| fried_rice | leakypipes Eh? /me scours code... | 19:36 |
| fried_rice | leakypipes I can easily see how one would convert from get_inventory() to update_provider_tree() -- what I'm wondering about (more long term, really) is how/when one would be able to get rid of get_available_resource() altogether. | 19:36 |
| fried_rice | Sounds like this series won't be the thing that allows that to happen. | 19:36 |
| leakypipes | fried_rice: no, it won't. we will need to handle NUMA stuff before that's a reality. | 19:37 |
| leakypipes | fried_rice: but no *new* code is being allowed in the get_available_resource() calls... | 19:37 |
| *** Sukhdev has joined #openstack-nova | 19:38 | |
| fried_rice | leakypipes Humph, indeed I can't find where get_available_resource is parlayed into placement inventory -- but then how does the scheduler work for non-get_inventory()-implementin virt drivers? | 19:39 |
| openstackgerrit | Merged openstack/nova master: begin refactor AllocCandidates._get_by_filters() https://review.openstack.org/516778 | 19:40 |
| fried_rice | leakypipes Woot! ^ | 19:41 |
| leakypipes | :) | 19:41 |
| leakypipes | fried_rice: I think all virt drivers implement get_inventory() by now. | 19:42 |
| fried_rice | leakypipes Nope - PowerVM for sure not. | 19:42 |
| fried_rice | in tree or out of tree. | 19:42 |
| fried_rice | leakypipes Yeah, and yet when I look in the placement DB on my PowerVM compute node, I see a RP entry for the compute node and inventory for VCPU, MEMORY_MB, and DISK_GB. | 19:43 |
| fried_rice | At least, I thought I did. Looking again... | 19:43 |
| leakypipes | fried_rice: then there would be no resource providers in the placement/API DB. which AllocationCandidates.get_by_filters() would return [], [], which will trigger shceudler to do a ComputeNodeList.get_all() which will use the vcpus, memory_mb, and local_gb fields in the cell compute_nodes tables and those fields are populated with the get_available_resource() results. | 19:44 |
| fried_rice | leakypipes And that... populates the placement DB with those values? | 19:44 |
| fried_rice | Cause I confirmed | 19:45 |
| fried_rice | leakypipes http://paste.openstack.org/show/626656/ | 19:45 |
| fried_rice | leakypipes And no get_inventory in https://github.com/openstack/nova-powervm/blob/master/nova_powervm/virt/powervm/driver.py | 19:46 |
| leakypipes | fried_rice: https://github.com/openstack/nova/blob/master/nova/compute/resource_tracker.py#L824 | 19:47 |
| leakypipes | fried_rice: there is a hack in the scheduler client that sets the inventory based on the vcpus, memory_mb and local_gb of the compute node if the get_inventory() method is missing. | 19:47 |
| leakypipes | fried_rice: see the code comment directly above that... | 19:48 |
| leakypipes | # Eventually all virt drivers will return an inventory dict in the | 19:48 |
| leakypipes | # format that the placement API expects and we'll be able to remove | 19:48 |
| leakypipes | # this code branch | 19:48 |
| fried_rice | leakypipes _compute_node_to_inventory_dict - booya. | 19:48 |
| fried_rice | leakypipes Thanks for keeping me sane. | 19:48 |
| leakypipes | no problemo. | 19:49 |
| dansmith | mriedem: um.. I can't update a service to disable it in 2.38 | 19:50 |
| *** itlinux has quit IRC | 19:50 | |
| dansmith | mriedem: and the schema makes it sound like we can't except in >=2.53, but... we've been able to do that forever, no? | 19:50 |
| dansmith | v211 has a disabled_reason=, but no status= property for me to set to disabled... | 19:51 |
| *** fragatina has joined #openstack-nova | 19:54 | |
| *** itlinux has joined #openstack-nova | 19:55 | |
| dansmith | ah, I guess it was an action before | 19:57 |
| *** Apoorva has quit IRC | 20:01 | |
| openstackgerrit | Merged openstack/nova master: build ProviderSummary objects in sep function https://review.openstack.org/516779 | 20:02 |
| *** slaweq has joined #openstack-nova | 20:05 | |
| *** lpetrut has joined #openstack-nova | 20:06 | |
| *** vivsoni has quit IRC | 20:09 | |
| *** vivsoni has joined #openstack-nova | 20:09 | |
| *** gbarros has quit IRC | 20:15 | |
| mriedem | dansmith: yeah, 2.53 is a PUT api | 20:22 |
| dansmith | yeah, I didn't realize | 20:22 |
| dansmith | I'm close to a prototype to push up, fyi | 20:22 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Add Selection objects https://review.openstack.org/499239 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Refactor the code to check for sufficient hosts https://review.openstack.org/520242 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Return Selection objects from the scheduler driver https://review.openstack.org/495854 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Modify select_destinations() to return objects and alts https://review.openstack.org/510159 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Change RPC for select_destinations() https://review.openstack.org/516707 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Move the claim_resources method to scheduler utils https://review.openstack.org/511357 | 20:24 |
| openstackgerrit | Ed Leafe proposed openstack/nova master: Make conductor pass and use host_lists https://review.openstack.org/511358 | 20:24 |
| mriedem | dansmith: ok. i had to step away for a bit to clear my head - hadn't taken a break since i started today. i think i'm going to write a regression test for https://bugs.launchpad.net/nova/+bug/1732947 since i think rebuild + bfv with new image is something we need to fix anyway | 20:26 |
| openstack | Launchpad bug 1732947 in OpenStack Compute (nova) "volume-backed instance rebuild with no image change is still going through scheduler" [Undecided,In progress] - Assigned to Chris Friesen (cbf123) | 20:26 |
| mriedem | and rebuild + bfv with original image for that matter | 20:26 |
| dansmith | okay | 20:26 |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 20:27 |
| dansmith | mriedem: when you get a sec, let me explain my thinking in that test to make sure it's sound | 20:28 |
| mriedem | i think i get it | 20:30 |
| mriedem | we still have the allocation doubling up issue, but that's a separate problem | 20:31 |
| *** READ10 has quit IRC | 20:31 | |
| dansmith | yeah | 20:32 |
| dansmith | so if this is what you want, I need to go change the rest of the filters that are placement related to skip like this one does | 20:33 |
| *** vladikr has quit IRC | 20:33 | |
| dansmith | mriedem: there's also the question of just doing this, or putting a straight revert in front | 20:36 |
| dansmith | as this is, I'd have to backport the broken one first before I can backport this | 20:37 |
| dansmith | which I can do for sure, but I'd vote to cleanly revert and re-fix | 20:37 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.uuid_set() https://review.openstack.org/520243 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Scheduler set_inventory_for_provider does nested https://review.openstack.org/520643 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_tree https://review.openstack.org/520663 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient._get_providers_in_aggregates https://review.openstack.org/521097 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 20:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 20:38 |
| fried_rice | leakypipes Nits all fixed up ^ | 20:38 |
| leakypipes | fried_rice: ah, you went with uuid_set(). noice. | 20:39 |
| fried_rice | leakypipes I took the hint :) | 20:39 |
| leakypipes | fried_rice: it was a gentle hint. :) | 20:39 |
| mriedem | dansmith: comments inline | 20:39 |
| mriedem | dansmith: and yeah, not sure about the revert at this point | 20:40 |
| fried_rice | leakypipes This -1 was on the wrong patch btw - it stuck around after the rebase: https://review.openstack.org/#/c/520663/ | 20:40 |
| fried_rice | leakypipes ...just so you didn't bypass that guy for thinking you'd already seen it. | 20:41 |
| leakypipes | fried_rice: gotcha. | 20:41 |
| leakypipes | fried_rice: soon as tests complete, will re-review the series. | 20:41 |
| fried_rice | leakypipes Dig | 20:42 |
| mriedem | i'll work on the recreate test for https://bugs.launchpad.net/nova/+bug/1732947 and come back | 20:42 |
| openstack | Launchpad bug 1732947 in OpenStack Compute (nova) "volume-backed instance rebuild with no image change is still going through scheduler" [Undecided,In progress] - Assigned to Chris Friesen (cbf123) | 20:42 |
| *** itlinux has quit IRC | 20:43 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 20:45 |
| dansmith | mriedem: abstractmethod implies that we change the base filter to inherit from ABCMeta which is more change | 20:45 |
| dansmith | mriedem: we could also flip this and make the filters all skip if we're doing this policy check, unless they're opt-in as policy filters | 20:46 |
| dansmith | mriedem: and then just have them set POLICY=True in their class definition | 20:46 |
| mriedem | yeah i was thinking we could avoid the abc if we just have a default in the base and override in the subclasses | 20:46 |
| mriedem | like we do for calling placement for the scheduler drivers | 20:47 |
| dansmith | mriedem: and you want the default to be not-policy so we just override the ones that are policy? | 20:47 |
| mriedem | if that means fewer overrides, then i think yes | 20:47 |
| dansmith | I think it will | 20:48 |
| mriedem | we want the default to be whatever the majority of filters want | 20:48 |
| mriedem | ok | 20:48 |
| dansmith | either way, I think the overrides should be in the policy ones, not the resource ones | 20:48 |
| mriedem | yeah, like the affinity filters right? | 20:48 |
| dansmith | er, I guess I have the override in the placementy one right now | 20:48 |
| dansmith | lemme try to actually count how many overrides in each case | 20:48 |
| mriedem | placementy == ram/disk/core filters? | 20:49 |
| dansmith | yeah, mostly resource filters, except computefilter isn't really resourcey but is in that bucket | 20:49 |
| mriedem | yeah that one is tricky | 20:49 |
| mriedem | i was thinking about what you said about the computefilter not working for rebuild now, | 20:50 |
| mriedem | but that's only if you're rebuilding with a new image, | 20:50 |
| mriedem | and rebuilding with a new image on a disabled compute, that might not be the worst thing if we don't allow it | 20:50 |
| mriedem | but it is a change | 20:50 |
| *** Sukhdev has quit IRC | 20:51 | |
| dansmith | I think there are about 9 policy filters | 20:51 |
| *** eharney has quit IRC | 20:51 | |
| dansmith | maybe the best thing to do would be to just mark them all as either policy or not | 20:51 |
| *** jmlowe has joined #openstack-nova | 20:55 | |
| *** jape2 has joined #openstack-nova | 20:57 | |
| artom | We still a need default - can't operators have custom filters? | 20:58 |
| mriedem | they can | 20:58 |
| mriedem | but, | 20:58 |
| mriedem | release note | 20:58 |
| artom | Really, that's enough? | 20:58 |
| mriedem | there isn't an api contract on out of tree things | 20:58 |
| jape2 | Hi. is this the right place to ask for help regarding hyperv and dvr support for openstack pike? | 20:58 |
| dansmith | artom: yeah, still default to false (i.e. skip) but explicitly mark all our own filters | 20:58 |
| artom | mriedem, right, but we should still avoid breaking them, especially in easy cases like this | 20:59 |
| mriedem | jape2: try #openstack-hyper-v | 20:59 |
| artom | Anyways, dansmith nipped that in the bud | 20:59 |
| jape2 | thanks! | 20:59 |
| *** jape2 has left #openstack-nova | 20:59 | |
| *** Apoorva has joined #openstack-nova | 21:01 | |
| *** jmlowe has quit IRC | 21:01 | |
| *** gfhellma_ has joined #openstack-nova | 21:02 | |
| *** ianw has quit IRC | 21:02 | |
| *** gfhellma has quit IRC | 21:02 | |
| *** Hazelesque_ is now known as Hazelesque | 21:03 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | 21:03 |
| *** gfhellma_ has quit IRC | 21:03 | |
| *** gfhellma has joined #openstack-nova | 21:03 | |
| fried_rice | figleaf dansmith mriedem leakypipes Anything for the weekly placement update email? | 21:06 |
| mriedem | bugs | 21:06 |
| mriedem | no idk, i haven't been too involved with placement lately | 21:07 |
| fried_rice | figleaf I see movement on the series starting https://review.openstack.org/#/c/499239/ -- but still pinned -2. Is this ready to be unwedged yet? | 21:07 |
| mriedem | no | 21:07 |
| *** smatzek has quit IRC | 21:09 | |
| *** ianw has joined #openstack-nova | 21:09 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 21:10 |
| dansmith | mriedem: ^ updated approach, will start looking to see what unit test hell I've signed up for | 21:11 |
| dansmith | artom: ^ | 21:11 |
| *** vivsoni has quit IRC | 21:13 | |
| *** vivsoni has joined #openstack-nova | 21:14 | |
| *** AlexeyAbashkin has joined #openstack-nova | 21:19 | |
| mriedem | this is pretty funny http://www.kttc.com/story/36870998/2017/11/17/police-string-of-burglaries-in-ne-rochester-lands-one-man-behind-bars | 21:21 |
| mriedem | my kids school was in lockdown yesterday because of this joker | 21:21 |
| mriedem | " She asked him what he wanted and he just threw a shoe at her and left." | 21:22 |
| cfriesen | stealing someone's glasses? spraying deodorant on someone? this is a mentally ill guy, not a criminal mastermind | 21:23 |
| *** AlexeyAbashkin has quit IRC | 21:24 | |
| fried_rice | mriedem This lightweight should rob my house. No taser required. | 21:26 |
| cfriesen | mriedem: about that rebuild with boot-from-vol case, looks like a lot of the rebuild unit tests don't set up root_bdm properly. taking a look | 21:26 |
| mriedem | cfriesen: he also got an umbrella and some air jordans | 21:26 |
| mriedem | cfriesen: i'm writing a functional regression test for it | 21:26 |
| cfriesen | there you go...set for life | 21:26 |
| cfriesen | sweet | 21:27 |
| *** Apoorva_ has joined #openstack-nova | 21:30 | |
| *** salv-orl_ has joined #openstack-nova | 21:32 | |
| cfriesen | mriedem: okay, so looks like there are no blockdevicemappings set up for these tests. could we set up a suitable blockdevicemapping for instance server1? | 21:32 |
| *** Apoorva has quit IRC | 21:33 | |
| mriedem | i don't grok the last part of your question - my functional test will probably create a volume-backed instance, using the CinderFixture | 21:34 |
| cfriesen | I'm trying to figure out the most efficient way to solve all these failing unit tests. there's an instance that is created that is boot-from-vol, but compute_utils.get_root_bdm(context, instance) returns None | 21:34 |
| cfriesen | sorry, boot-from-image | 21:35 |
| *** salv-orlando has quit IRC | 21:35 | |
| artom | dansmith, left some minor comments | 21:35 |
| cfriesen | I was wondering if we set up a bdm for the instance, maybe it'd fix up a bunch of these tests | 21:35 |
| cfriesen | mriedem: I'd rather not fake out compute_utils.get_root_bdm() in 50 tests if I can avoid it. | 21:37 |
| mriedem | cfriesen: i agree, | 21:38 |
| mriedem | but atm, i'm focused on this functional regression test, so i'll have to dig into unit tests later | 21:38 |
| mriedem | because i think we want to decouple the fix for bug 1732947 from the one that raises a 400 if you request a rebuild with a new image for a volume-backed server | 21:39 |
| openstack | bug 1732947 in OpenStack Compute (nova) "volume-backed instance rebuild with no image change is still going through scheduler" [Undecided,In progress] https://launchpad.net/bugs/1732947 - Assigned to Chris Friesen (cbf123) | 21:39 |
| mriedem | it's...complicated | 21:39 |
| dansmith | artom: ack | 21:39 |
| cfriesen | mriedem: no worries | 21:39 |
| *** eharney has joined #openstack-nova | 21:42 | |
| *** andreas_s has joined #openstack-nova | 21:43 | |
| *** andreas_s has quit IRC | 21:47 | |
| *** tbachman has quit IRC | 21:53 | |
| *** Sukhdev has joined #openstack-nova | 21:53 | |
| *** slaweq has quit IRC | 21:55 | |
| *** slaweq has joined #openstack-nova | 21:55 | |
| *** tbachman has joined #openstack-nova | 21:57 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200 | 21:57 |
| mriedem | cfriesen: ^ | 21:57 |
| figleaf | fried_rice: that series is stuck until the entire thing has been reviewed. It's been nothing but rebase fun! | 21:58 |
| fried_rice | figleaf Ack. See ML. | 21:58 |
| *** slaweq has quit IRC | 21:59 | |
| *** gfhellma_ has joined #openstack-nova | 22:00 | |
| *** amodi has quit IRC | 22:00 | |
| *** artom has quit IRC | 22:03 | |
| *** gfhellma has quit IRC | 22:03 | |
| *** fragatina has quit IRC | 22:03 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 22:03 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | 22:04 |
| *** fragatina has joined #openstack-nova | 22:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add regression test for rebuilding a volume-backed server https://review.openstack.org/521200 | 22:06 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 22:06 |
| figleaf | fried_rice: also, I'm out all next week, so if you or anyone else wants to run the scheduler meeting on Monday, have at it. | 22:07 |
| fried_rice | figleaf Duly noted. | 22:07 |
| *** edmondsw has quit IRC | 22:19 | |
| *** edmondsw has joined #openstack-nova | 22:19 | |
| *** edmondsw_ has joined #openstack-nova | 22:23 | |
| *** edmondsw has quit IRC | 22:24 | |
| *** edmondsw_ has quit IRC | 22:27 | |
| *** hoonetorg has quit IRC | 22:28 | |
| *** gbarros has joined #openstack-nova | 22:30 | |
| *** dave-mcc_ has quit IRC | 22:30 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 22:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 22:38 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | 22:38 |
| fried_rice | leakypipes ASCII diagrams and string renames --^ | 22:44 |
| openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | 22:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | 22:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | 22:52 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | 22:52 |
| fried_rice | leakypipes And hopefully that brings all the non-WIPs up to scratch (tests are in for the ones below ^ ) | 22:52 |
| *** lpetrut has quit IRC | 22:57 | |
| *** yamamoto has joined #openstack-nova | 23:04 | |
| *** yamamoto has quit IRC | 23:05 | |
| * figleaf is signing off until after Thanksgiving weekend | 23:07 | |
| *** figleaf is now known as edleafe | 23:07 | |
| *** lyan has quit IRC | 23:09 | |
| *** imacdonn has quit IRC | 23:09 | |
| *** imacdonn has joined #openstack-nova | 23:10 | |
| *** leakypipes has quit IRC | 23:10 | |
| *** yamamoto has joined #openstack-nova | 23:11 | |
| *** yamamoto has quit IRC | 23:16 | |
| *** ayoung has quit IRC | 23:24 | |
| *** Yingxin has quit IRC | 23:26 | |
| *** Apoorva_ has quit IRC | 23:27 | |
| *** Apoorva has joined #openstack-nova | 23:28 | |
| *** gouthamr has joined #openstack-nova | 23:33 | |
| *** gouthamr_ has joined #openstack-nova | 23:36 | |
| *** gouthamr has quit IRC | 23:38 | |
| *** sdague has quit IRC | 23:40 | |
| *** edmondsw has joined #openstack-nova | 23:44 | |
| *** snasir has quit IRC | 23:45 | |
| *** edmondsw has quit IRC | 23:48 | |
| *** snasir has joined #openstack-nova | 23:48 | |
| *** gbarros has quit IRC | 23:53 | |
| *** pcaruana has quit IRC | 23:54 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!