| *** edmondsw has joined #openstack-nova | 00:05 | |
| *** salv-orlando has quit IRC | 00:09 | |
| *** salv-orlando has joined #openstack-nova | 00:09 | |
| *** edmondsw has quit IRC | 00:10 | |
| *** rtjure has quit IRC | 00:11 | |
| *** AlexeyAbashkin has joined #openstack-nova | 00:13 | |
| *** salv-orlando has quit IRC | 00:14 | |
| *** hongbin has quit IRC | 00:14 | |
| *** AlexeyAbashkin has quit IRC | 00:17 | |
| *** Sukhdev has joined #openstack-nova | 00:19 | |
| *** rtjure has joined #openstack-nova | 00:19 | |
| *** claudiub|2 has quit IRC | 00:25 | |
| *** itlinux has quit IRC | 00:26 | |
| openstackgerrit | Vipin Balachandran proposed openstack/nova master: VMware: Support volumes backed by VStorageObject https://review.openstack.org/521690 | 00:33 |
|---|---|---|
| *** amodi has joined #openstack-nova | 00:35 | |
| *** felipemonteiro has joined #openstack-nova | 00:40 | |
| *** yamamoto has quit IRC | 00:42 | |
| *** Sandy619 has joined #openstack-nova | 00:43 | |
| *** armax has joined #openstack-nova | 00:44 | |
| *** Sandy619 has quit IRC | 00:45 | |
| *** trinaths has joined #openstack-nova | 00:46 | |
| *** edmondsw has joined #openstack-nova | 00:49 | |
| *** edmondsw has quit IRC | 00:52 | |
| *** edmondsw has joined #openstack-nova | 00:52 | |
| *** gfhellma has quit IRC | 00:58 | |
| *** phuongnh has joined #openstack-nova | 01:04 | |
| *** suresh12 has quit IRC | 01:06 | |
| *** Guest18917 has quit IRC | 01:11 | |
| *** salv-orlando has joined #openstack-nova | 01:11 | |
| *** yassine has joined #openstack-nova | 01:11 | |
| *** yassine has quit IRC | 01:11 | |
| *** yassine has joined #openstack-nova | 01:12 | |
| *** AlexeyAbashkin has joined #openstack-nova | 01:12 | |
| *** yassine is now known as Guest25442 | 01:12 | |
| *** rtjure has quit IRC | 01:13 | |
| *** trungnv has joined #openstack-nova | 01:13 | |
| *** rtjure has joined #openstack-nova | 01:14 | |
| *** AlexeyAbashkin has quit IRC | 01:16 | |
| *** suresh12 has joined #openstack-nova | 01:18 | |
| *** edmondsw has quit IRC | 01:20 | |
| *** edmondsw has joined #openstack-nova | 01:20 | |
| *** Daisy_ has joined #openstack-nova | 01:22 | |
| *** Daisy_ has quit IRC | 01:24 | |
| *** Daisy_ has joined #openstack-nova | 01:24 | |
| *** edmondsw has quit IRC | 01:25 | |
| *** salv-orlando has quit IRC | 01:26 | |
| *** takashin has joined #openstack-nova | 01:26 | |
| *** masayukig has quit IRC | 01:28 | |
| tonyb | dansmith: Thanks. | 01:28 |
| *** Daisy_ has quit IRC | 01:29 | |
| *** gcb has joined #openstack-nova | 01:29 | |
| *** masayukig has joined #openstack-nova | 01:29 | |
| *** tianhui has joined #openstack-nova | 01:30 | |
| *** yangyapeng has joined #openstack-nova | 01:31 | |
| *** dave-mccowan has joined #openstack-nova | 01:31 | |
| *** Daisy_ has joined #openstack-nova | 01:32 | |
| *** yangyapeng has quit IRC | 01:35 | |
| *** trinaths has left #openstack-nova | 01:40 | |
| *** yamamoto has joined #openstack-nova | 01:42 | |
| *** gouthamr has quit IRC | 01:48 | |
| *** yangyapeng has joined #openstack-nova | 01:49 | |
| *** yamamoto has quit IRC | 01:49 | |
| *** yamamoto has joined #openstack-nova | 01:51 | |
| *** vladikr has quit IRC | 01:52 | |
| *** cheneydc has joined #openstack-nova | 01:52 | |
| *** yangyapeng has quit IRC | 01:54 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot" https://review.openstack.org/514128 | 02:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Add a description of 'key_name' in rebuild https://review.openstack.org/519993 | 02:00 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Update document related to host aggregate https://review.openstack.org/514499 | 02:01 |
| *** Apoorva has quit IRC | 02:09 | |
| *** Apoorva has joined #openstack-nova | 02:10 | |
| *** Apoorva has quit IRC | 02:11 | |
| *** Apoorva has joined #openstack-nova | 02:11 | |
| *** gongysh has joined #openstack-nova | 02:13 | |
| *** yangyapeng has joined #openstack-nova | 02:15 | |
| *** chyka has quit IRC | 02:19 | |
| *** yangyapeng has quit IRC | 02:19 | |
| *** salv-orlando has joined #openstack-nova | 02:22 | |
| *** trungnv has quit IRC | 02:23 | |
| *** gmann_afk is now known as gmann | 02:25 | |
| *** moshele has joined #openstack-nova | 02:26 | |
| *** salv-orlando has quit IRC | 02:27 | |
| *** chyka has joined #openstack-nova | 02:27 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | 02:29 |
| openstackgerrit | Merged openstack/nova master: Include project_id and user_id in AllocationList.get_all_by_consumer_id https://review.openstack.org/512420 | 02:29 |
| *** yamahata has quit IRC | 02:29 | |
| *** chyka has quit IRC | 02:32 | |
| *** trungnv has joined #openstack-nova | 02:34 | |
| *** yangyapeng has joined #openstack-nova | 02:34 | |
| *** gouthamr has joined #openstack-nova | 02:34 | |
| dims | mriedem_away : you need to send a translation for your team meeting email :) | 02:34 |
| *** suresh12 has quit IRC | 02:38 | |
| *** itlinux has joined #openstack-nova | 02:38 | |
| *** yangyapeng has quit IRC | 02:38 | |
| gmann | alex_xu: can you re review this -https://review.openstack.org/#/c/516104/9 | 02:45 |
| gmann | yikun: i replied on this, can you check - https://review.openstack.org/#/c/515413/ | 02:48 |
| *** moshele has quit IRC | 02:49 | |
| *** hongbin has joined #openstack-nova | 02:52 | |
| *** Sukhdev has quit IRC | 02:53 | |
| *** edmondsw has joined #openstack-nova | 02:55 | |
| *** vladikr has joined #openstack-nova | 02:57 | |
| *** felipemonteiro has quit IRC | 02:57 | |
| *** erlon has quit IRC | 02:58 | |
| *** moshele has joined #openstack-nova | 02:59 | |
| *** yangyapeng has joined #openstack-nova | 02:59 | |
| *** edmondsw has quit IRC | 03:00 | |
| Kevin_Zheng | gmann: yikun is on his trip to bug smash, maybe not available for the next 6 hours | 03:03 |
| *** yangyapeng has quit IRC | 03:04 | |
| gmann | Kevin_Zheng: ohk, thanks for info. | 03:04 |
| *** tianhui_ has joined #openstack-nova | 03:07 | |
| *** suresh12 has joined #openstack-nova | 03:07 | |
| *** gongysh has quit IRC | 03:09 | |
| *** tianhui has quit IRC | 03:09 | |
| *** dave-mccowan has quit IRC | 03:10 | |
| *** suresh12 has quit IRC | 03:11 | |
| *** yangyapeng has joined #openstack-nova | 03:18 | |
| *** links has joined #openstack-nova | 03:19 | |
| *** tianhui_ is now known as tianhui | 03:19 | |
| *** harlowja has quit IRC | 03:22 | |
| *** yangyapeng has quit IRC | 03:22 | |
| *** salv-orlando has joined #openstack-nova | 03:23 | |
| *** akekane has joined #openstack-nova | 03:26 | |
| *** akekane is now known as abhishekk | 03:26 | |
| *** salv-orlando has quit IRC | 03:28 | |
| *** Daisy_ has quit IRC | 03:29 | |
| *** Daisy_ has joined #openstack-nova | 03:29 | |
| *** Daisy_ has quit IRC | 03:29 | |
| *** Daisy_ has joined #openstack-nova | 03:30 | |
| *** takashin has quit IRC | 03:30 | |
| *** Apoorva has quit IRC | 03:31 | |
| *** takashin has joined #openstack-nova | 03:31 | |
| *** zerda3 has quit IRC | 03:33 | |
| *** Daisy_ has quit IRC | 03:34 | |
| *** vladikr has quit IRC | 03:35 | |
| *** vladikr has joined #openstack-nova | 03:35 | |
| *** gcb has quit IRC | 03:42 | |
| *** gszasz has quit IRC | 03:53 | |
| *** itlinux has quit IRC | 03:58 | |
| *** harlowja has joined #openstack-nova | 04:01 | |
| *** yangyapeng has joined #openstack-nova | 04:03 | |
| *** gouthamr has quit IRC | 04:05 | |
| *** yangyapeng has quit IRC | 04:08 | |
| *** moshele has quit IRC | 04:09 | |
| *** mdnadeem has joined #openstack-nova | 04:10 | |
| *** psachin has joined #openstack-nova | 04:11 | |
| *** yamamoto has quit IRC | 04:11 | |
| *** vladikr has quit IRC | 04:11 | |
| *** itlinux has joined #openstack-nova | 04:12 | |
| *** hongbin has quit IRC | 04:15 | |
| *** hongbin has joined #openstack-nova | 04:15 | |
| *** chyka has joined #openstack-nova | 04:16 | |
| *** Daisy_ has joined #openstack-nova | 04:17 | |
| *** chyka has quit IRC | 04:20 | |
| *** Daisy_ has quit IRC | 04:22 | |
| *** diga has joined #openstack-nova | 04:24 | |
| *** yangyapeng has joined #openstack-nova | 04:24 | |
| *** Daisy_ has joined #openstack-nova | 04:24 | |
| *** salv-orlando has joined #openstack-nova | 04:24 | |
| *** Sukhdev has joined #openstack-nova | 04:25 | |
| *** TuanLA has joined #openstack-nova | 04:27 | |
| *** Daisy_ has quit IRC | 04:28 | |
| *** yangyapeng has quit IRC | 04:28 | |
| *** salv-orlando has quit IRC | 04:29 | |
| *** Daisy_ has joined #openstack-nova | 04:30 | |
| *** sridharg has joined #openstack-nova | 04:32 | |
| *** Daisy_ has quit IRC | 04:35 | |
| openstackgerrit | Merged openstack/nova master: api-ref: Add a description of 'key_name' in rebuild https://review.openstack.org/519993 | 04:35 |
| *** tetsuro has joined #openstack-nova | 04:42 | |
| *** Daisy_ has joined #openstack-nova | 04:43 | |
| *** edmondsw has joined #openstack-nova | 04:44 | |
| *** itlinux has quit IRC | 04:44 | |
| *** gbarros has quit IRC | 04:46 | |
| *** gcb has joined #openstack-nova | 04:46 | |
| *** itlinux has joined #openstack-nova | 04:47 | |
| *** Daisy_ has quit IRC | 04:47 | |
| *** hongbin has quit IRC | 04:48 | |
| *** edmondsw has quit IRC | 04:48 | |
| *** yamamoto has joined #openstack-nova | 04:48 | |
| *** yangyapeng has joined #openstack-nova | 04:48 | |
| *** hshiina has joined #openstack-nova | 04:50 | |
| *** diga has quit IRC | 04:52 | |
| *** yangyapeng has quit IRC | 04:52 | |
| *** lbragstad has quit IRC | 04:58 | |
| *** Daisy_ has joined #openstack-nova | 04:59 | |
| *** ratailor has joined #openstack-nova | 04:59 | |
| *** markvoelker has quit IRC | 05:00 | |
| *** diga has joined #openstack-nova | 05:05 | |
| *** claudiub|2 has joined #openstack-nova | 05:08 | |
| *** yamahata has joined #openstack-nova | 05:10 | |
| *** janki has joined #openstack-nova | 05:13 | |
| *** trinaths has joined #openstack-nova | 05:15 | |
| *** ijw has joined #openstack-nova | 05:17 | |
| *** suresh12 has joined #openstack-nova | 05:20 | |
| *** salv-orlando has joined #openstack-nova | 05:25 | |
| *** sree has joined #openstack-nova | 05:27 | |
| *** amodi has quit IRC | 05:28 | |
| *** salv-orlando has quit IRC | 05:29 | |
| *** edmondsw has joined #openstack-nova | 05:29 | |
| *** itlinux has quit IRC | 05:31 | |
| openstackgerrit | Merged openstack/nova master: api-ref: Fix an example in "Delete Assisted Volume Snapshot" https://review.openstack.org/514128 | 05:31 |
| *** udesale has joined #openstack-nova | 05:32 | |
| *** yamamoto has quit IRC | 05:32 | |
| *** edmondsw has quit IRC | 05:33 | |
| *** yamamoto has joined #openstack-nova | 05:34 | |
| *** yangyapeng has joined #openstack-nova | 05:37 | |
| *** chyka has joined #openstack-nova | 05:39 | |
| *** pcaruana has joined #openstack-nova | 05:39 | |
| *** yamamoto_ has joined #openstack-nova | 05:40 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | 05:42 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: [placement] Add 'Location' parameters in API ref https://review.openstack.org/521541 | 05:42 |
| *** yamamoto has quit IRC | 05:43 | |
| *** chyka has quit IRC | 05:43 | |
| *** edmondsw has joined #openstack-nova | 05:48 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: remove reserve_quota_delta https://review.openstack.org/521469 | 05:48 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | 05:48 |
| *** harlowja has quit IRC | 05:52 | |
| *** edmondsw has quit IRC | 05:52 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Add description for resize down support https://review.openstack.org/516572 | 05:53 |
| *** lajoskatona has joined #openstack-nova | 05:53 | |
| openstackgerrit | jichenjc proposed openstack/nova master: mv generate_glance_url to get_image_endpoint_url https://review.openstack.org/511400 | 05:54 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group uuid and vgpu type uuid https://review.openstack.org/521717 | 05:54 |
| *** yamamoto_ has quit IRC | 05:56 | |
| *** rodolof has joined #openstack-nova | 05:57 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group uuid and vgpu type uuid https://review.openstack.org/521717 | 05:58 |
| *** sree has quit IRC | 06:00 | |
| *** hshiina2 has joined #openstack-nova | 06:00 | |
| *** markvoelker has joined #openstack-nova | 06:00 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group uuid and vgpu type uuid https://review.openstack.org/521717 | 06:00 |
| *** chyka has joined #openstack-nova | 06:00 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group uuid and vgpu type uuid https://review.openstack.org/521717 | 06:02 |
| *** hshiina has quit IRC | 06:03 | |
| *** sree has joined #openstack-nova | 06:04 | |
| *** chyka has quit IRC | 06:04 | |
| openstackgerrit | jichenjc proposed openstack/nova master: conf: remove deprecated 'project_id_regex' opt https://review.openstack.org/501057 | 06:06 |
| *** Sukhdev has quit IRC | 06:06 | |
| *** sree has quit IRC | 06:12 | |
| *** pcaruana has quit IRC | 06:13 | |
| *** armax has quit IRC | 06:14 | |
| *** chyka has joined #openstack-nova | 06:16 | |
| *** gongysh has joined #openstack-nova | 06:16 | |
| *** chyka has quit IRC | 06:20 | |
| *** dtruong_ has joined #openstack-nova | 06:22 | |
| *** mdnadeem has quit IRC | 06:22 | |
| *** cheneydc_ has joined #openstack-nova | 06:22 | |
| *** dtruong has quit IRC | 06:23 | |
| *** cheneydc has quit IRC | 06:23 | |
| *** damien_r1 has quit IRC | 06:23 | |
| *** janki has quit IRC | 06:23 | |
| *** damien_r has joined #openstack-nova | 06:23 | |
| *** janki has joined #openstack-nova | 06:23 | |
| *** rodolof has quit IRC | 06:23 | |
| *** annp has joined #openstack-nova | 06:23 | |
| *** rodolof has joined #openstack-nova | 06:23 | |
| *** salv-orlando has joined #openstack-nova | 06:24 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 06:24 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | 06:24 |
| *** edmondsw has joined #openstack-nova | 06:29 | |
| *** suresh12 has quit IRC | 06:31 | |
| *** hemna_ has quit IRC | 06:33 | |
| *** suresh12 has joined #openstack-nova | 06:33 | |
| *** edmondsw has quit IRC | 06:33 | |
| *** suresh12 has quit IRC | 06:38 | |
| *** mdnadeem has joined #openstack-nova | 06:38 | |
| *** gongysh has quit IRC | 06:40 | |
| *** moshele has joined #openstack-nova | 06:47 | |
| *** peter-hamilton has quit IRC | 06:47 | |
| *** edmondsw has joined #openstack-nova | 06:48 | |
| *** trinaths has left #openstack-nova | 06:52 | |
| *** edmondsw has quit IRC | 06:53 | |
| *** moshele has quit IRC | 06:54 | |
| *** trungnv has quit IRC | 06:57 | |
| *** mdnadeem has quit IRC | 06:57 | |
| *** trinaths has joined #openstack-nova | 06:58 | |
| *** Daisy_ has quit IRC | 06:59 | |
| *** rcernin has quit IRC | 06:59 | |
| *** Daisy_ has joined #openstack-nova | 06:59 | |
| *** dtantsur|afk is now known as dtantsur | 07:01 | |
| *** Daisy__ has joined #openstack-nova | 07:03 | |
| *** hshiina3 has joined #openstack-nova | 07:04 | |
| *** Daisy__ has quit IRC | 07:04 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 07:04 |
| *** Daisy__ has joined #openstack-nova | 07:04 | |
| *** Daisy_ has quit IRC | 07:04 | |
| *** josecastroleon has quit IRC | 07:04 | |
| *** josecastroleon1 has joined #openstack-nova | 07:04 | |
| *** hshiina2 has quit IRC | 07:05 | |
| *** edmondsw has joined #openstack-nova | 07:08 | |
| *** yamamoto has joined #openstack-nova | 07:10 | |
| *** mdnadeem has joined #openstack-nova | 07:10 | |
| *** yamamoto_ has joined #openstack-nova | 07:10 | |
| *** spectr has joined #openstack-nova | 07:11 | |
| *** spectr has quit IRC | 07:12 | |
| *** edmondsw has quit IRC | 07:12 | |
| *** yamamoto has quit IRC | 07:14 | |
| *** Daisy__ has quit IRC | 07:14 | |
| *** andreas_s has joined #openstack-nova | 07:19 | |
| *** Daisy_ has joined #openstack-nova | 07:22 | |
| *** trungnv has joined #openstack-nova | 07:23 | |
| *** edmondsw has joined #openstack-nova | 07:23 | |
| *** pcaruana has joined #openstack-nova | 07:30 | |
| *** chyka has joined #openstack-nova | 07:30 | |
| *** takashin has left #openstack-nova | 07:30 | |
| *** lpetrut has joined #openstack-nova | 07:32 | |
| *** udesale has quit IRC | 07:32 | |
| *** edmondsw has quit IRC | 07:33 | |
| *** sahid has joined #openstack-nova | 07:35 | |
| *** chyka has quit IRC | 07:35 | |
| *** ijw has quit IRC | 07:37 | |
| *** josecastroleon1 has quit IRC | 07:40 | |
| *** trungnv has quit IRC | 07:41 | |
| *** edmondsw has joined #openstack-nova | 07:48 | |
| *** trungnv has joined #openstack-nova | 07:52 | |
| *** edmondsw has quit IRC | 07:52 | |
| *** ralonsoh has joined #openstack-nova | 07:56 | |
| *** slaweq has joined #openstack-nova | 07:59 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 08:00 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 08:00 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 08:00 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 08:00 |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi(N-R-P): Add API to support vgpu resource provider create https://review.openstack.org/520313 | 08:01 |
| *** moshele has joined #openstack-nova | 08:02 | |
| *** edand has joined #openstack-nova | 08:03 | |
| *** priteau has joined #openstack-nova | 08:03 | |
| *** karthiks has quit IRC | 08:06 | |
| *** Daisy_ has quit IRC | 08:06 | |
| *** edmondsw has joined #openstack-nova | 08:08 | |
| *** gcb has quit IRC | 08:12 | |
| *** josecastroleon has joined #openstack-nova | 08:12 | |
| *** edmondsw has quit IRC | 08:13 | |
| *** gcb has joined #openstack-nova | 08:13 | |
| *** Daisy_ has joined #openstack-nova | 08:14 | |
| *** rmart04 has joined #openstack-nova | 08:16 | |
| *** salv-orlando has quit IRC | 08:18 | |
| *** slaweq has quit IRC | 08:18 | |
| *** hoonetorg has quit IRC | 08:18 | |
| *** salv-orlando has joined #openstack-nova | 08:18 | |
| *** gcb has quit IRC | 08:19 | |
| *** gcb has joined #openstack-nova | 08:19 | |
| *** udesale has joined #openstack-nova | 08:19 | |
| *** slaweq has joined #openstack-nova | 08:20 | |
| *** Daisy_ has quit IRC | 08:21 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: xenapi: Add parse function to get gpu group uuid and vgpu type uuid https://review.openstack.org/521717 | 08:21 |
| *** salv-orlando has quit IRC | 08:22 | |
| *** tesseract has joined #openstack-nova | 08:23 | |
| *** karthiks has joined #openstack-nova | 08:23 | |
| *** slaweq has quit IRC | 08:24 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 08:26 |
| *** edmondsw has joined #openstack-nova | 08:28 | |
| *** yamamoto_ has quit IRC | 08:29 | |
| *** salv-orlando has joined #openstack-nova | 08:31 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - test updated releasenotes and sphinx job https://review.openstack.org/520677 | 08:32 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Update bindep.txt for doc builds https://review.openstack.org/521755 | 08:32 |
| *** karthiks has quit IRC | 08:33 | |
| *** edmondsw has quit IRC | 08:33 | |
| *** karthiks has joined #openstack-nova | 08:33 | |
| *** hoonetorg has joined #openstack-nova | 08:35 | |
| *** cheneydc_ has quit IRC | 08:36 | |
| *** lpetrut has quit IRC | 08:36 | |
| *** cheneydc_ has joined #openstack-nova | 08:37 | |
| *** Daisy_ has joined #openstack-nova | 08:37 | |
| *** ijw has joined #openstack-nova | 08:38 | |
| *** slaweq has joined #openstack-nova | 08:38 | |
| *** lpetrut has joined #openstack-nova | 08:40 | |
| *** cheneydc_ has quit IRC | 08:40 | |
| *** cheneydc_ has joined #openstack-nova | 08:40 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 08:41 |
| *** ragiman has joined #openstack-nova | 08:42 | |
| *** jpena|off is now known as jpena | 08:43 | |
| *** ijw has quit IRC | 08:45 | |
| *** slaweq has quit IRC | 08:47 | |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | 08:47 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 08:47 |
| openstackgerrit | Rodolfo Alonso Hernandez proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 08:47 |
| *** slaweq has joined #openstack-nova | 08:47 | |
| *** aloga has quit IRC | 08:48 | |
| *** aloga has joined #openstack-nova | 08:48 | |
| *** edmondsw has joined #openstack-nova | 08:49 | |
| openstackgerrit | Van Hung Pham proposed openstack/nova master: Remove translation of log messages https://review.openstack.org/466995 | 08:53 |
| *** edmondsw has quit IRC | 08:53 | |
| frickler | the config ref for ocata shows options that are only available in pike and later, I'm assuming that this is because they are autogenerated using uncapped oslo.log reqs, how can that be fixed? https://bugs.launchpad.net/nova/+bug/1733515 | 08:57 |
| openstack | Launchpad bug 1733515 in OpenStack Compute (nova) "use_journal option not available in ocata" [Undecided,New] | 08:57 |
| *** yamamoto has joined #openstack-nova | 08:58 | |
| *** lpetrut has quit IRC | 08:59 | |
| *** Sandy619 has joined #openstack-nova | 09:05 | |
| *** Daisy_ has quit IRC | 09:05 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Object changes to support last-modified headers https://review.openstack.org/521639 | 09:05 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Add cache headers to placement api requests https://review.openstack.org/521640 | 09:05 |
| *** Daisy_ has joined #openstack-nova | 09:09 | |
| *** gszasz has joined #openstack-nova | 09:10 | |
| *** Daisy_ has quit IRC | 09:10 | |
| *** Daisy_ has joined #openstack-nova | 09:10 | |
| *** andreas_s has quit IRC | 09:13 | |
| openstackgerrit | xulei proposed openstack/nova master: Delete out-of-date record for resource provider https://review.openstack.org/521764 | 09:13 |
| *** andreas_s has joined #openstack-nova | 09:13 | |
| *** trungnv has quit IRC | 09:14 | |
| *** slaweq has quit IRC | 09:14 | |
| *** abhishekk has quit IRC | 09:16 | |
| *** diga has quit IRC | 09:17 | |
| *** akekane has joined #openstack-nova | 09:17 | |
| *** andreas_s has quit IRC | 09:18 | |
| *** sree has joined #openstack-nova | 09:19 | |
| *** dtantsur_ has joined #openstack-nova | 09:21 | |
| *** dtantsur has quit IRC | 09:22 | |
| *** dtantsur_ is now known as dtantsur | 09:22 | |
| *** sree has quit IRC | 09:24 | |
| *** andreas_s has joined #openstack-nova | 09:24 | |
| *** andreas_s has quit IRC | 09:29 | |
| *** Sandy619 has quit IRC | 09:30 | |
| *** mvk has quit IRC | 09:30 | |
| *** cdent has joined #openstack-nova | 09:30 | |
| *** mvk has joined #openstack-nova | 09:30 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 09:32 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 09:32 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 09:32 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 09:32 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: restrict to only enable one vGPU type https://review.openstack.org/521769 | 09:32 |
| *** lucas-afk is now known as lucasagomes | 09:32 | |
| openstackgerrit | Merged openstack/osc-placement master: CLI for inventories https://review.openstack.org/457533 | 09:33 |
| *** yingjun has quit IRC | 09:34 | |
| *** Daisy_ has quit IRC | 09:35 | |
| *** andreas_s has joined #openstack-nova | 09:36 | |
| *** slaweq has joined #openstack-nova | 09:39 | |
| *** slaweq has quit IRC | 09:39 | |
| *** slaweq has joined #openstack-nova | 09:40 | |
| *** andreas_s has quit IRC | 09:40 | |
| *** derekh has joined #openstack-nova | 09:41 | |
| *** lpetrut has joined #openstack-nova | 09:42 | |
| *** andreas_s has joined #openstack-nova | 09:44 | |
| *** yamahata has quit IRC | 09:46 | |
| *** andreas_s has quit IRC | 09:48 | |
| *** andreas_s has joined #openstack-nova | 09:49 | |
| *** andreas_s has quit IRC | 09:51 | |
| *** andreas_s has joined #openstack-nova | 09:52 | |
| *** trungnv has joined #openstack-nova | 09:56 | |
| *** yamamoto has quit IRC | 09:57 | |
| *** yamamoto has joined #openstack-nova | 10:00 | |
| *** cheneydc_ has quit IRC | 10:00 | |
| *** salv-orlando has quit IRC | 10:00 | |
| *** salv-orlando has joined #openstack-nova | 10:01 | |
| *** rcernin has joined #openstack-nova | 10:01 | |
| *** yamamoto has quit IRC | 10:02 | |
| *** yamamoto has joined #openstack-nova | 10:02 | |
| *** janki has quit IRC | 10:02 | |
| *** janki has joined #openstack-nova | 10:02 | |
| *** TuanLA has quit IRC | 10:05 | |
| *** salv-orlando has quit IRC | 10:05 | |
| *** slaweq has quit IRC | 10:09 | |
| *** andreas_s has quit IRC | 10:11 | |
| *** andreas_s has joined #openstack-nova | 10:12 | |
| *** slaweq has joined #openstack-nova | 10:13 | |
| *** slaweq has quit IRC | 10:14 | |
| *** slaweq has joined #openstack-nova | 10:15 | |
| *** andreas_s has quit IRC | 10:16 | |
| openstackgerrit | Zhenyu Zheng proposed openstack/nova master: Add instance action record for attach/detach/swap volumes https://review.openstack.org/517205 | 10:18 |
| *** gszasz has quit IRC | 10:21 | |
| *** andreas_s has joined #openstack-nova | 10:23 | |
| *** annp has quit IRC | 10:28 | |
| *** jhesketh has quit IRC | 10:28 | |
| *** andreas_s has quit IRC | 10:28 | |
| *** ratailor has quit IRC | 10:30 | |
| *** jhesketh has joined #openstack-nova | 10:30 | |
| *** gmann is now known as gmann_afk | 10:34 | |
| *** ralonsoh has quit IRC | 10:34 | |
| *** ralonsoh has joined #openstack-nova | 10:34 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/519066 | 10:38 |
| *** erlon has joined #openstack-nova | 10:40 | |
| *** yamamoto has quit IRC | 10:41 | |
| *** yamamoto has joined #openstack-nova | 10:41 | |
| *** ijw has joined #openstack-nova | 10:43 | |
| *** tester352 has joined #openstack-nova | 10:47 | |
| *** ijw has quit IRC | 10:47 | |
| *** ratailor has joined #openstack-nova | 10:47 | |
| *** edmondsw has joined #openstack-nova | 10:48 | |
| moshele | stephenfin: hi https://review.openstack.org/#/c/519066/ is ready for review again | 10:49 |
| tester352 | Hello! Can anyone help to resolve following issue: https://pastebin.com/raw/9GBXCkas ? In short - after failed attempt to create instance due to volume quota limit I get "Error: Unable to retrieve attachment information." in horizon. | 10:52 |
| *** edmondsw has quit IRC | 10:52 | |
| tester352 | could be a bug because of failed instance creation there should not remain remnants of instance | 10:53 |
| *** ratailor_ has joined #openstack-nova | 10:53 | |
| *** akekane has quit IRC | 10:53 | |
| *** andreas_s has joined #openstack-nova | 10:54 | |
| *** ratailor has quit IRC | 10:56 | |
| *** ralonsoh has quit IRC | 10:56 | |
| *** chyka has joined #openstack-nova | 10:57 | |
| *** ralonsoh has joined #openstack-nova | 10:57 | |
| openstackgerrit | Merged openstack/nova master: VMware: serial console log (completed) https://review.openstack.org/450636 | 10:57 |
| *** trinaths has quit IRC | 11:00 | |
| *** salv-orlando has joined #openstack-nova | 11:01 | |
| *** chyka has quit IRC | 11:01 | |
| *** slaweq has quit IRC | 11:02 | |
| tetsuro | is alex or jay or efried here? | 11:02 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Fix the usage of instance.snapshot notification sample https://review.openstack.org/520579 | 11:03 |
| *** slaweq has joined #openstack-nova | 11:04 | |
| *** salv-orlando has quit IRC | 11:06 | |
| *** gszasz has joined #openstack-nova | 11:06 | |
| 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 | 11:07 |
| *** edmondsw has joined #openstack-nova | 11:08 | |
| *** slaweq has quit IRC | 11:08 | |
| 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 | 11:09 |
| *** edand has quit IRC | 11:11 | |
| *** edmondsw has quit IRC | 11:12 | |
| *** andreas_s has quit IRC | 11:13 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | 11:14 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 11:14 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix missing marker functions https://review.openstack.org/514579 | 11:15 |
| *** andreas_s has joined #openstack-nova | 11:18 | |
| *** AlexeyAbashkin has joined #openstack-nova | 11:24 | |
| *** mvk has quit IRC | 11:27 | |
| *** edmondsw has joined #openstack-nova | 11:28 | |
| *** andreas_s has quit IRC | 11:29 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Remove quota reserve/commit/rollback https://review.openstack.org/521470 | 11:29 |
| *** andreas_s has joined #openstack-nova | 11:30 | |
| *** udesale has quit IRC | 11:30 | |
| *** edmondsw has quit IRC | 11:32 | |
| *** andreas_s has quit IRC | 11:39 | |
| *** andreas_s has joined #openstack-nova | 11:39 | |
| *** slaweq has joined #openstack-nova | 11:40 | |
| *** edand has joined #openstack-nova | 11:42 | |
| *** slaweq has quit IRC | 11:42 | |
| *** slaweq has joined #openstack-nova | 11:43 | |
| *** diga has joined #openstack-nova | 11:44 | |
| *** ijw has joined #openstack-nova | 11:44 | |
| *** slaweq_ has joined #openstack-nova | 11:47 | |
| *** slaweq has quit IRC | 11:47 | |
| *** edmondsw has joined #openstack-nova | 11:48 | |
| *** ijw has quit IRC | 11:49 | |
| *** tbachman has quit IRC | 11:50 | |
| *** vladikr has joined #openstack-nova | 11:50 | |
| *** slaweq_ has quit IRC | 11:51 | |
| *** slaweq has joined #openstack-nova | 11:52 | |
| *** edmondsw has quit IRC | 11:52 | |
| *** andreas_s has quit IRC | 11:53 | |
| *** slaweq_ has joined #openstack-nova | 11:53 | |
| *** chyka has joined #openstack-nova | 11:54 | |
| *** slaweq has quit IRC | 11:56 | |
| *** tbachman has joined #openstack-nova | 11:57 | |
| *** mvk has joined #openstack-nova | 11:57 | |
| *** slaweq_ has quit IRC | 11:58 | |
| *** chyka has quit IRC | 11:58 | |
| *** andreas_s has joined #openstack-nova | 11:58 | |
| mdbooth | Anybody know why I'm getting logged out of gerrit every 5 minutes? | 12:00 |
| *** tbachman has quit IRC | 12:01 | |
| cdent | mdbooth: because gerrit is trying to do you a favor? | 12:01 |
| *** salv-orlando has joined #openstack-nova | 12:01 | |
| mdbooth | cdent: Heh. It would be the first time ever, though :) | 12:02 |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove command nova-manage quota refresh https://review.openstack.org/521829 | 12:05 |
| *** andreas_s has quit IRC | 12:08 | |
| *** sdague has joined #openstack-nova | 12:08 | |
| *** vladikr has quit IRC | 12:13 | |
| *** slaweq has joined #openstack-nova | 12:14 | |
| *** andreas_s has joined #openstack-nova | 12:15 | |
| *** pooja_jadhav has quit IRC | 12:18 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove nova-manage account and nova-manage project https://review.openstack.org/521833 | 12:18 |
| *** gcb has quit IRC | 12:18 | |
| *** slaweq has quit IRC | 12:19 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Remove command nova-manage quota refresh https://review.openstack.org/521829 | 12:19 |
| *** vivsoni__ has joined #openstack-nova | 12:20 | |
| *** vivsoni_ has quit IRC | 12:20 | |
| openstackgerrit | Merged openstack/nova master: [placement] Fix a wrong redirection in placement doc https://review.openstack.org/517163 | 12:23 |
| openstackgerrit | jichenjc proposed openstack/nova master: remove nova-manage shell command https://review.openstack.org/521835 | 12:24 |
| *** phuongnh has quit IRC | 12:25 | |
| openstackgerrit | Merged openstack/nova master: Fix format in live-migration-usage.rst https://review.openstack.org/516416 | 12:26 |
| openstackgerrit | Merged openstack/nova master: Using --option ARGUMENT https://review.openstack.org/516238 | 12:26 |
| openstackgerrit | Merged openstack/nova master: Document the real behavior of notify_on_state_change https://review.openstack.org/516264 | 12:27 |
| openstackgerrit | Merged openstack/nova master: Add a note about versioned notification samples being per-release https://review.openstack.org/520193 | 12:27 |
| *** chyka has joined #openstack-nova | 12:27 | |
| *** slaweq has joined #openstack-nova | 12:29 | |
| *** chyka has quit IRC | 12:32 | |
| openstackgerrit | jichenjc proposed openstack/nova master: remove 'nova-manage host' and 'nova-manage agent' https://review.openstack.org/521837 | 12:33 |
| *** slaweq_ has joined #openstack-nova | 12:33 | |
| *** slaweq has quit IRC | 12:34 | |
| *** pooja_jadhav has joined #openstack-nova | 12:34 | |
| *** pooja-jadhav has joined #openstack-nova | 12:35 | |
| *** pooja-jadhav has quit IRC | 12:35 | |
| *** bhagyashris has quit IRC | 12:36 | |
| *** niraj_singh has quit IRC | 12:37 | |
| *** slaweq_ has quit IRC | 12:38 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 12:38 |
| efried | tetsuro Sorry, just saw your ping. What's up? | 12:39 |
| efried | dims Definitely an American-centric email, that. It being Thanksgiving, people eat turkey ("cousin of a yard bird") and watch football (for some reason mriedem_away is a Vikings fan). | 12:44 |
| *** yangyapeng has quit IRC | 12:44 | |
| tetsuro | cool | 12:45 |
| *** yangyapeng has joined #openstack-nova | 12:45 | |
| *** bhagyashris has joined #openstack-nova | 12:45 | |
| *** ijw has joined #openstack-nova | 12:45 | |
| tetsuro | i was debugging and thought i solve it, but seems that it was my misunderstanding. | 12:46 |
| tetsuro | Forgive me for messing up here... https://review.openstack.org/#/c/519081/ | 12:46 |
| *** moshele has quit IRC | 12:48 | |
| *** edmondsw has joined #openstack-nova | 12:48 | |
| *** lucasagomes is now known as lucas-hungry | 12:49 | |
| *** yangyapeng has quit IRC | 12:49 | |
| tetsuro | The day after tomorrow is a thanksgiving day in Japan. | 12:49 |
| *** ijw has quit IRC | 12:50 | |
| efried | tetsuro I don't see any harm done on https://review.openstack.org/#/c/519081/ -- what's the summary of this issue? | 12:51 |
| tetsuro | I just thought I had solved it and wondered where to throw a patch so was about to ask you on which patch I should make it. | 12:52 |
| tetsuro | But now I am still testing, so please forget my ping. | 12:53 |
| efried | tetsuro Okay, no worries. Appreciate your efforts. | 12:53 |
| tetsuro | now i should get back home from my office. | 12:54 |
| efried | tetsuro Looking at that sort, I can't imagine why it's there or what it thinks it's doing. | 12:54 |
| tetsuro | it is sorting not to do inner join at first, I think. Anyway, thanks for answering ! | 12:55 |
| tetsuro | not to do outer join at first. | 12:55 |
| *** vladikr has joined #openstack-nova | 12:55 | |
| tetsuro | I gotta go. see you tomorrow. | 12:57 |
| *** tetsuro has quit IRC | 12:57 | |
| *** chyka has joined #openstack-nova | 12:58 | |
| *** diga has quit IRC | 12:58 | |
| *** jpena is now known as jpena|lunch | 12:58 | |
| *** tianhui has quit IRC | 13:01 | |
| *** chyka has quit IRC | 13:03 | |
| *** efried has quit IRC | 13:04 | |
| *** tbachman has joined #openstack-nova | 13:10 | |
| *** dave-mccowan has joined #openstack-nova | 13:11 | |
| *** tianhui has joined #openstack-nova | 13:11 | |
| *** yangyapeng has joined #openstack-nova | 13:16 | |
| *** dave-mcc_ has joined #openstack-nova | 13:16 | |
| *** pchavva has joined #openstack-nova | 13:17 | |
| *** dave-mccowan has quit IRC | 13:17 | |
| *** efried has joined #openstack-nova | 13:17 | |
| *** tbachman has quit IRC | 13:18 | |
| *** belmoreira has joined #openstack-nova | 13:18 | |
| *** tbachman has joined #openstack-nova | 13:19 | |
| sean-k-mooney | mriedem_away: at the ptg we discussed the fact that rodolfo had created a bluprint in os-vif for this but we dont use them there so you suggested just tracking this in nova since os-vif is a nova deliverable. | 13:21 |
| *** moshele has joined #openstack-nova | 13:21 | |
| sean-k-mooney | mriedem_away: so yes https://blueprints.launchpad.net/nova/+spec/vif-port-profile is a nova blueprint for work that is predominately done in os-vif | 13:22 |
| *** sree has joined #openstack-nova | 13:25 | |
| *** mdnadeem has quit IRC | 13:26 | |
| *** moshele has quit IRC | 13:27 | |
| *** rcernin has quit IRC | 13:30 | |
| *** ratailor_ is now known as ratailor | 13:31 | |
| *** mhenkel has quit IRC | 13:32 | |
| *** suresh12 has joined #openstack-nova | 13:34 | |
| *** mhenkel has joined #openstack-nova | 13:38 | |
| *** suresh12 has quit IRC | 13:39 | |
| *** rodolof has quit IRC | 13:40 | |
| *** ijw has joined #openstack-nova | 13:40 | |
| *** udesale has joined #openstack-nova | 13:45 | |
| *** ijw has quit IRC | 13:45 | |
| *** jaypipes has joined #openstack-nova | 13:46 | |
| *** rmart04 has quit IRC | 13:48 | |
| efried | stephenfin gibi bauzas Anyone up for a couple of easy doc reviews? https://review.openstack.org/521216 https://review.openstack.org/521563 | 13:49 |
| *** mhenkel has quit IRC | 13:49 | |
| stephenfin | efried: Sure thing | 13:49 |
| efried | Ta verramuch | 13:49 |
| *** jpena|lunch is now known as jpena | 13:50 | |
| *** lucas-hungry is now known as lucasagomes | 13:51 | |
| *** salv-orlando has quit IRC | 13:51 | |
| *** salv-orlando has joined #openstack-nova | 13:52 | |
| *** slaweq has joined #openstack-nova | 13:52 | |
| *** slaweq has quit IRC | 13:53 | |
| *** slaweq has joined #openstack-nova | 13:53 | |
| *** jianghuaw has joined #openstack-nova | 13:55 | |
| *** salv-orlando has quit IRC | 13:55 | |
| *** salv-orlando has joined #openstack-nova | 13:56 | |
| *** mlavalle has joined #openstack-nova | 13:56 | |
| *** gszasz has quit IRC | 13:59 | |
| *** gszasz has joined #openstack-nova | 14:02 | |
| stephenfin | sean-k-mooney, ralonsoh: Say, can you get 82599ES (Niantic) NICs in embedded form? | 14:02 |
| stephenfin | e.g. as part of the board, rather than a discrete board | 14:02 |
| *** sahid has quit IRC | 14:04 | |
| *** mgariepy has quit IRC | 14:05 | |
| jianghuaw | gibi, hi. | 14:05 |
| *** mkoderer_ has joined #openstack-nova | 14:06 | |
| *** tpatzig_ has joined #openstack-nova | 14:06 | |
| *** dgonzalez_ has joined #openstack-nova | 14:06 | |
| *** david_1 has joined #openstack-nova | 14:06 | |
| ralonsoh | stephenfin: I'll ask this question to drivers team. But I don't think so: https://ark.intel.com/products/codename/32659/Niantic | 14:08 |
| *** dgonzalez_ has quit IRC | 14:08 | |
| *** tpatzig_ has quit IRC | 14:08 | |
| *** mkoderer_ has quit IRC | 14:08 | |
| *** david_1 has quit IRC | 14:08 | |
| *** sahid has joined #openstack-nova | 14:09 | |
| *** tbachman has quit IRC | 14:12 | |
| *** READ10 has joined #openstack-nova | 14:14 | |
| gibi | efried: hi! I'm still going through my mail backlog, fit those patches are still open when I finish that I can check these | 14:14 |
| gibi | jianghuaw: hi! | 14:14 |
| efried | gibi stephenfin already pushed them. Thanks! | 14:14 |
| *** lyan has joined #openstack-nova | 14:14 | |
| *** mgariepy has joined #openstack-nova | 14:14 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Change 'InstancePCIRequest' spec field https://review.openstack.org/449257 | 14:15 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add Neutron port capabilities to devspec in request https://review.openstack.org/451777 | 14:15 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Format NIC features using os-traits definitions https://review.openstack.org/466051 | 14:15 |
| jianghuaw | gibi, are you going to help reviewing this patch? https://review.openstack.org/#/c/512580/ | 14:15 |
| stephenfin | bauzas: Want to take a second look at https://review.openstack.org/#/c/466051/ | 14:15 |
| jianghuaw | gibi, we've got +2 and serveral +1s on it. But need another core help to review it. | 14:16 |
| jianghuaw | We mentioned this patches in last week. | 14:16 |
| gibi | jianghuaw: I will try but | 14:18 |
| *** mhenkel has joined #openstack-nova | 14:18 | |
| gibi | jianghuaw: I have a long backlog for today | 14:18 |
| *** toure_biab is now known as toure | 14:19 | |
| jianghuaw | gibi, I see. It's appreciated if you can put it in the todo queue. | 14:19 |
| mriedem_away | sean-k-mooney: ack thanks | 14:19 |
| jaypipes | I can't read "TripleO IPSEC Integration" without thinking "Triple-sec integration" and then thinking about Cointreau. | 14:20 |
| *** mriedem_away is now known as mriedem | 14:20 | |
| *** ratailor has quit IRC | 14:21 | |
| *** takedakn has joined #openstack-nova | 14:21 | |
| *** gbarros has joined #openstack-nova | 14:23 | |
| *** lbragstad has joined #openstack-nova | 14:24 | |
| stephenfin | ralonsoh: Thought so, but I wasn't sure. Thanks :) | 14:24 |
| *** andreas_s has quit IRC | 14:25 | |
| stephenfin | efried, jaypipes: Am I correct in saying the Memory/DiskFilter is going away/gone thanks to placement? If so, what about the weighers? | 14:25 |
| *** andreas_s has joined #openstack-nova | 14:25 | |
| efried | stephenfin Weighers won't be affected by placement. | 14:25 |
| efried | stephenfin Because placement doesn't do any weighing; everything's binary. | 14:26 |
| jaypipes | stephenfin: yes. and what efried said. | 14:26 |
| mriedem | jianghuaw: on https://review.openstack.org/#/c/512580/ - was there a specific reason to use MultiStrOpt over ListOpt? | 14:26 |
| *** mdnadeem has joined #openstack-nova | 14:26 | |
| efried | stephenfin As for those filters going away... I would expect the answer is yes. | 14:26 |
| efried | ...eventually. | 14:26 |
| stephenfin | efried: That's what I was thinking. So does this mean something like this wouldn't be affected by placement? https://review.openstack.org/#/c/379525/ | 14:26 |
| mriedem | stephenfin: the core/ram/disk filters are no longer part of the default list of enabled filters b/c of placement | 14:26 |
| bauzas | stephenfin: ralonsoh: so the problem is that we don't have a os-traits tag, right? | 14:26 |
| stephenfin | (not saying it's something we want, but placement is not the reason not to want it) | 14:26 |
| *** suresh12 has joined #openstack-nova | 14:26 | |
| efried | stephenfin Agree. | 14:27 |
| stephenfin | bauzas: Correct. We need to wait for the next release | 14:27 |
| bauzas | stephenfin: ralonsoh: honestly, we could just create a new release | 14:27 |
| bauzas | mriedem: ^ | 14:27 |
| stephenfin | Also, I wouldn't want to block that patch while we wait for it | 14:27 |
| *** clayton has quit IRC | 14:27 | |
| jaypipes | stephenfin: eventually the *inputs* to the weighers will be created from the returned results of the GET /allocation_candidates call's provider_summaries response, instead of a constructing from the ComputeNode object/cell DB table. | 14:27 |
| stephenfin | That's also an option :D | 14:27 |
| bauzas | mriedem: context is https://review.openstack.org/#/c/466051/12..15/nova/virt/libvirt/config.py | 14:27 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Enable cold migration with target host(1/2) https://review.openstack.org/408955 | 14:27 |
| stephenfin | jaypipes: But filters would still be a nova (vs placement thing), right? | 14:28 |
| bauzas | mriedem: I could just provide a new release for os-traits, fine with you ? | 14:28 |
| stephenfin | Sorry - weighters | 14:28 |
| mriedem | bauzas: sure | 14:28 |
| ralonsoh | bauzas: you are right, os-traits is not updated. But the patch can work as is | 14:28 |
| jaypipes | stephenfin: weighers will continue to be in nova-scheduler, yes | 14:28 |
| mriedem | bauzas: you'll be working this week if the gate explodes and the rest of us are on vacation? | 14:28 |
| bauzas | stephenfin: ralonsoh: I'll just provide a new release change for os-traits | 14:28 |
| * stephenfin goes to unabandon that patch | 14:28 | |
| *** felipemonteiro has joined #openstack-nova | 14:28 | |
| bauzas | mriedem: sure, I can | 14:28 |
| jianghuaw | mriedem, both should work. But I think bauzas suggested MultiStrOpt when reviewing the spec. | 14:29 |
| stephenfin | jaypipes, efried: If either of you want to review that once I do, I'd appreciate it https://review.openstack.org/#/c/379525/ | 14:29 |
| jaypipes | stephenfin: k | 14:29 |
| *** andreas_s has quit IRC | 14:30 | |
| *** felipemonteiro_ has joined #openstack-nova | 14:30 | |
| stephenfin | Cheers :) Seems I'm on the hook for nested RP reviews this week too, you'll be happy to hear | 14:30 |
| mriedem | stephenfin: https://review.openstack.org/#/c/379525/ isn't happening for queens | 14:30 |
| efried | stephenfin This is for a 3-year-old blueprint that's not approved for Q ? | 14:30 |
| stephenfin | /unhappy | 14:30 |
| bauzas | jianghuaw: I'd love to see you reviewing my inventory patch for the VGPU in libvirt | 14:30 |
| *** clayton has joined #openstack-nova | 14:30 | |
| stephenfin | Ahh, hold on so. I didn't realize there was a BP attached | 14:30 |
| jianghuaw | bauzas, sure. I will. Actually I've opened that patch. | 14:30 |
| stephenfin | In that case, I'll reopen and target for Rocky | 14:31 |
| *** suresh12 has quit IRC | 14:31 | |
| efried | SGTM | 14:31 |
| bauzas | jianghuaw: I have multiple problems with libvirt mdevs, but that change would be okay | 14:31 |
| *** markvoelker has quit IRC | 14:31 | |
| mriedem | bauzas: any specific reason to use MultiStrOpt over ListOpt here? https://review.openstack.org/#/c/512580/ | 14:31 |
| *** slaweq has quit IRC | 14:31 | |
| *** markvoelker has joined #openstack-nova | 14:31 | |
| stephenfin | jianghuaw, bauzas, mriedem: MultiStrOpt is deprecated, afaik | 14:31 |
| stephenfin | Or will be soon. Don't use it | 14:31 |
| bauzas | mriedem: I don't remember me commenting that | 14:32 |
| *** gszasz has quit IRC | 14:32 | |
| bauzas | lemme verify | 14:32 |
| mriedem | stephenfin: it doesn't say that it is https://github.com/openstack/oslo.config/blob/master/oslo_config/cfg.py#L1571 | 14:32 |
| bauzas | hem, no, I haven't said that https://review.openstack.org/#/c/512580/1/nova/conf/vgpu.py | 14:32 |
| stephenfin | mriedem: Hmm, odd. I was sure gcb talked about that at the PTG | 14:33 |
| mriedem | i'm asking in -oslo | 14:33 |
| *** edand has quit IRC | 14:33 | |
| bauzas | mriedem: stephenfin: yeah, don't see it in https://docs.openstack.org/oslo.config/latest/reference/opts.html#oslo_config.cfg.MultiStrOpt | 14:33 |
| bauzas | if that's deprecated, then please change that | 14:33 |
| *** felipemonteiro has quit IRC | 14:34 | |
| stephenfin | I'm pretty sure gcb did talk about it, so maybe it's on his list. In any case, I wouldn't use it. It's very confusing | 14:34 |
| bauzas | honestly, I don't see the difference | 14:34 |
| bauzas | but meh | 14:34 |
| bauzas | I trust you | 14:34 |
| bauzas | if you say ListOpt should be used, then -1 the change | 14:34 |
| stephenfin | Every other option is specified once except that special one :) | 14:34 |
| bauzas | AFAIK, you're an oslo core :p | 14:34 |
| stephenfin | Heh, I hate -1ing for that kind of stuff, but ok | 14:35 |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: cleanup mapping/reqspec after archive instance https://review.openstack.org/515034 | 14:35 |
| *** moshele has joined #openstack-nova | 14:35 | |
| jianghuaw | bauzas, stephenfin, mriedem: Got it. So will proceed to change it from MultiStrOpt to ListOpts. Thanks. | 14:36 |
| bauzas | ++ | 14:36 |
| bauzas | I'll need to rebase my change, but meh | 14:36 |
| stephenfin | ++ as commented, those things are hard to change after the fact | 14:36 |
| bauzas | jianghuaw: just ping me when it's uploaded so I can rebase my series | 14:36 |
| stephenfin | bauzas: I'm a pbr core - not oslo (yet ;)) | 14:36 |
| jianghuaw | bauzas, Sure. | 14:36 |
| bauzas | stephenfin: WAT? | 14:37 |
| bauzas | :p | 14:37 |
| stephenfin | Probably for the better 🙈 | 14:37 |
| *** hshiina3 has quit IRC | 14:37 | |
| bauzas | ralonsoh: FWIW, working on a new release for os-traits | 14:37 |
| ralonsoh | bauzas: thank you very much | 14:38 |
| *** larainema has quit IRC | 14:39 | |
| *** artom has joined #openstack-nova | 14:39 | |
| *** tbachman has joined #openstack-nova | 14:41 | |
| *** ijw has joined #openstack-nova | 14:41 | |
| *** amodi has joined #openstack-nova | 14:43 | |
| *** gszasz has joined #openstack-nova | 14:44 | |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 14:46 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: get vGPU stats from hypervisor https://review.openstack.org/512965 | 14:46 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: provide vGPU inventory in compute node https://review.openstack.org/516217 | 14:46 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: create vGPU for instance https://review.openstack.org/516899 | 14:46 |
| openstackgerrit | Jianghua Wang proposed openstack/nova master: XenAPI: restrict to only enable one vGPU type https://review.openstack.org/521769 | 14:46 |
| *** ijw has quit IRC | 14:46 | |
| jianghuaw | bauzas, ^ | 14:46 |
| *** edand has joined #openstack-nova | 14:47 | |
| *** links has quit IRC | 14:47 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Add CPUWeigher https://review.openstack.org/379525 | 14:48 |
| bauzas | jianghuaw: roger, will rebase | 14:48 |
| jianghuaw | bauzas, cool. | 14:49 |
| moshele | stephenfin: hi can you review https://review.openstack.org/#/c/519066/ ? | 14:49 |
| stephenfin | moshele: Big backlog today, but it's on the list :) | 14:49 |
| moshele | stephenfin: so tomorrow :) | 14:50 |
| stephenfin | Depends on how quickly I work through the backlog, heh | 14:50 |
| moshele | stephenfin: :) | 14:50 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/pike: Document the real behavior of notify_on_state_change https://review.openstack.org/521870 | 14:51 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 14:51 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 14:51 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 14:51 |
| dansmith | mriedem: you wanna write the reno for ^? I'm struggling to formulate english sentences at the moment | 14:52 |
| bauzas | mriedem: ralonsoh: stephenfin: https://review.openstack.org/#/c/521869/ | 14:52 |
| bauzas | mriedem: need your lovely +1 | 14:52 |
| mriedem | dansmith: sure | 14:53 |
| stephenfin | bauzas: Do we want to merge the requirements updates first | 14:53 |
| stephenfin | ? | 14:53 |
| stephenfin | https://review.openstack.org/#/q/project:openstack/os-traits+status:open | 14:53 |
| stephenfin | The rest are WIP | 14:53 |
| *** slaweq has joined #openstack-nova | 14:53 | |
| bauzas | stephenfin: I looked at those | 14:53 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/ocata: Document the real behavior of notify_on_state_change https://review.openstack.org/521871 | 14:53 |
| bauzas | stephenfin: and honestly, I don't see why we need to block a new release because of those | 14:54 |
| stephenfin | Sweet. Then you have your +1 | 14:54 |
| bauzas | stephenfin: most of cores will be on vacations by the next days | 14:54 |
| *** READ10 has quit IRC | 14:54 | |
| bauzas | so if we wanna merge a new release and modify g-r, then it needs to be soon | 14:54 |
| *** weshay is now known as weshay_pto | 14:55 | |
| bauzas | mriedem: FWIW, I'm not sure we need to bump g-r given nova reqs.txt is os-traits>=0.3.2 # Apache-2.0 | 14:56 |
| bauzas | you okay with that ? | 14:56 |
| bauzas | or do you think it would be better to just amend g-r ? | 14:56 |
| mriedem | if we're doing a release for new code in os-traits to use in nova, then we need to bump the minimum required version in g-r | 14:56 |
| *** ijw has joined #openstack-nova | 14:57 | |
| stephenfin | bauzas: What he said. If someone's on 0.3.2, it would break | 14:57 |
| bauzas | k | 14:57 |
| bauzas | I'll provide a new g-r then | 14:57 |
| stephenfin | While we're here, these two should probably go in at some point too (or be abandoned) https://review.openstack.org/#/q/project:openstack/os-traits+status:open+branch:stable/pike | 14:58 |
| * stephenfin has no idea who the stable cores for os-traits are? Same as nova? | 14:58 | |
| *** awaugama has joined #openstack-nova | 15:00 | |
| bauzas | that's correct | 15:00 |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: GET /RPs with aggs, inventories, traits https://review.openstack.org/521875 | 15:00 |
| mriedem | bauzas: -1 | 15:00 |
| efried | jaypipes cdent ^ Along the lines of what we discussed briefly yesterday. | 15:00 |
| bauzas | mriedem: oh man, good point | 15:01 |
| bauzas | mriedem: It's just a long day that I haven't provided a new release | 15:01 |
| bauzas | meh | 15:01 |
| jaypipes | efried: k. it's gonna be low priority though :) | 15:01 |
| mriedem | psh | 15:01 |
| mriedem | excuses excuses | 15:01 |
| cdent | efried: noted | 15:01 |
| bauzas | mriedem: no excuses, I'm a d-bag, that's it | 15:02 |
| stephenfin | bauzas: Well we didn't want to say anything, but... | 15:02 |
| * stephenfin also forgot about semantic versioning | 15:02 | |
| efried | cdent jaypipes Would I be correct in assuming that we get a significant performance improvement by reducing the number of placement API calls, assuming everything else (amount of db work, etc.) remains the same? | 15:02 |
| *** mvk has quit IRC | 15:03 | |
| bauzas | there, fixed | 15:03 |
| jaypipes | efried: for the nova-computes running Ironic virt driver, yeah. | 15:03 |
| *** yamamoto has quit IRC | 15:03 | |
| *** eharney has joined #openstack-nova | 15:03 | |
| jaypipes | efried: for the KVM ones, probably won't notice much of a difference. | 15:03 |
| cdent | efried: performance _where_ ? | 15:03 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 15:04 |
| efried | jaypipes cdent I'm talking about just placement calls. The above change set allows us to reduce 3N+1 calls down to one, to get the same data. (N is the number of providers in the tree.) | 15:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Send soft_delete from context manager https://review.openstack.org/476459 | 15:04 |
| efried | jaypipes cdent The data as a whole is identical, the work being done on the placement side is identical, etc. - just being done in one API call instead of many. | 15:05 |
| cdent | efried: you’re limiting the opportunities for the placement service to parallize across multiple instances of placement itself, which may or may not be a concern. I tend to prefer lots of smaller calls on many servers | 15:05 |
| jaypipes | efried: understood. and what I'm saying is that N is not a large number for KVM. For Ironic (and maybe powervm) N can be a large number. | 15:05 |
| openstackgerrit | Merged openstack/os-traits master: Update the documentation urls https://review.openstack.org/501122 | 15:05 |
| *** sree_ has joined #openstack-nova | 15:05 | |
| *** sree_ is now known as Guest1302 | 15:06 | |
| efried | jaypipes Ah, I understand. Nevertheless, even for a one-node tree, it would be taking 4 calls down to 1. | 15:06 |
| jaypipes | efried: understood. :) | 15:06 |
| efried | jaypipes I guess I'm asking if the overhead of the API calls is significant. | 15:06 |
| *** janki has quit IRC | 15:07 | |
| cdent | efried: if it is possible to measure instead of speculate that would be great, but my speculation would be that http overhead is not the limiting factor in the big picture in these situations | 15:07 |
| *** udesale has quit IRC | 15:07 | |
| cdent | certainly as a micro optimization, at webscale™, http request overhead matters | 15:08 |
| *** armax has joined #openstack-nova | 15:08 | |
| efried | cdent Okay, interesting. Well, if for no other reason than it makes the client code easier... | 15:08 |
| *** sree has quit IRC | 15:08 | |
| cdent | but only if it cannot be ameliorated by parallelization | 15:08 |
| *** ijw has quit IRC | 15:09 | |
| cdent | my gut reaction is that your prematurely optimizing, and that your efforts would be better served by trying to break stuff | 15:09 |
| cdent | so that when we fix stuff, we’re fixing the right stuff | 15:09 |
| cdent | but I’m sure telling grandma how to suck eggs | 15:09 |
| cdent | efried: Are you familiar with that phrase? I didn’t know it when I first heard it and it totally confused me. | 15:10 |
| efried | cdent No, but google helped me out. | 15:10 |
| *** mdnadeem has quit IRC | 15:10 | |
| efried | cdent In the context of the SchedulerReportClient, parallelization would entail some nontrivial rework, likely including spawing extra threads, which doesn't seem likely. | 15:10 |
| cdent | no, I mean parallization at the global level, not the per “query” level | 15:11 |
| cdent | we are more concerned with the performance of placement being good and predictable for everyone, at large | 15:11 |
| cdent | and given N is almost always quite small... | 15:12 |
| *** tianhui has quit IRC | 15:12 | |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: Implement new attach Cinder flow https://review.openstack.org/330285 | 15:13 |
| openstackgerrit | Ildiko Vancsa proposed openstack/nova master: libvirt: Allow multiple volume attachments https://review.openstack.org/267587 | 15:13 |
| efried | cdent Though even at its smallest, we would be reducing the number of calls by 4x | 15:13 |
| cdent | I hear you, but as we have no evidence, yet, that it matters, it’s early optimization, right? | 15:13 |
| efried | cdent If I was doing it just for performance, yeah. But I have *hard* evidence that it's gonna make my coding easier :) | 15:14 |
| cdent | while making the server more complex | 15:14 |
| *** takedakn has quit IRC | 15:14 | |
| efried | cdent See this series: https://review.openstack.org/#/c/521605/ | 15:15 |
| *** moshele has quit IRC | 15:15 | |
| * cdent nods | 15:15 | |
| efried | cdent But also https://review.openstack.org/#/c/521098/4/nova/scheduler/client/report.py as we discussed yesterday. | 15:16 |
| efried | Anyway, I did happen to notice a dearth of functional tests in test_report_client, so maybe I will do as you suggest and go back to breaking things. | 15:17 |
| cdent | efried: I’m not against your idea at all, I just don’t think it is urgent in the face of the usual constraints about review bandwidth, getting other stuff merged in this cycle, etc etc. I’d love it if we operated on a “this is a good idea, let’s just do it” mode but we’re not quite there. | 15:18 |
| *** andreas_s has joined #openstack-nova | 15:19 | |
| *** mvk has joined #openstack-nova | 15:19 | |
| efried | cdent I'm with you. Though TBH I'm not sure the amount of code will be more one way or the other. We either collate on the server side or the client side. | 15:20 |
| cdent | I’m biased to doing collation on the client side because I want the server to be as simple as possible, and will consistently resist adding new stuff, until overwhelmed and then blithely acquiese | 15:21 |
| *** Guest1302 has quit IRC | 15:24 | |
| *** fnordahl has quit IRC | 15:24 | |
| *** sree has joined #openstack-nova | 15:24 | |
| openstackgerrit | Merged openstack/nova master: placement: Document request headers in api-ref https://review.openstack.org/521563 | 15:25 |
| openstackgerrit | Merged openstack/nova master: placement: Document `in:` prefix for ?member_of= https://review.openstack.org/521216 | 15:26 |
| artom | dansmith, mriedem, is it worth it to "argue" about which filters should be policy at this point? | 15:26 |
| dansmith | artom: if you see something glaringly wrong, point it out | 15:26 |
| artom | I see there are many comments to that effect already, and I haven't read all of them yet | 15:26 |
| artom | dansmith, I wouldn't say glaringly. To me it's "does policy == things can change through a rebuild"? | 15:27 |
| artom | Or rather, is that what we want? | 15:27 |
| artom | Or are we going with the "real" semantic definition of policy, even it it means running some filters needlessly? | 15:27 |
| *** andreas_s has quit IRC | 15:27 | |
| *** esberglu has joined #openstack-nova | 15:27 | |
| efried | cdent Are you aware of a bug complaining that GET / of the placement API should be unauthenticated? | 15:28 |
| *** lajoskatona has quit IRC | 15:28 | |
| dansmith | artom: well, as we discussed on that review, it's hard to draw a neat box around that | 15:28 |
| efried | cdent AFAIK, we want every service to permit version discovery (which is what's at that URI) for free. <== mordred True? | 15:28 |
| *** sree has quit IRC | 15:29 | |
| artom | dansmith, yeah, which is why I'm asking here before leaving a bunch of somments | 15:29 |
| dansmith | artom: I want it to be "affects _where_ a thing should land" vs. "affects whether a host is "allowed" or "can" land" or something | 15:29 |
| artom | *comments | 15:29 |
| dansmith | artom: but clearly for the purposes of this backport, it's mostly just "does this break rebuild" | 15:29 |
| artom | For instance, the AZ filter... the AZ can't change with a rebuild, can it? Do we need to run it? | 15:29 |
| dansmith | artom: and I'm not running it on rebuild right? | 15:30 |
| * dansmith checks | 15:30 | |
| dansmith | oh I am | 15:30 |
| dansmith | that might be a mistake.. I was thinking that certain aggregate lines could have changed such that the operator wants to block the rebuild, | 15:30 |
| dansmith | but az is probably not right | 15:30 |
| mriedem | heh, maybe the variable should just be, RUN_ON_REBUILD | 15:31 |
| dansmith | artom: I was thinking https://review.openstack.org/#/c/521186/5/nova/scheduler/filters/aggregate_instance_extra_specs.py so I might have confused the az one | 15:31 |
| mordred | efried: yes - we want all the things to be able to do version discovery - lemme read more scrollback though | 15:31 |
| artom | The aggregate stuff as well... | 15:31 |
| efried | mordred Just the comment two up from that one | 15:31 |
| artom | I grok that aggregate metadata can be changed by an operator | 15:31 |
| efried | mordred Today placement's GET / requires auth | 15:31 |
| artom | But nothing about the instance can change to affect its aggregate placement, I think? | 15:32 |
| dansmith | artom: so we have a test that makes sure that rebuild fails if the operator has manually migrated one of your anti-affinity brothers to your same host.. I figure that means that late violation of such a policy should make rebuild fail | 15:32 |
| mriedem | artom: the image can change on rebuild | 15:32 |
| mriedem | so the image meta can change the host aggregates you can be in | 15:32 |
| mriedem | which was the original bug | 15:32 |
| artom | mriedem, ah, does that specify aggregate stuff? OK, that's ignorance on my part then | 15:32 |
| * artom goes looking for a comprehensive list of image properties | 15:33 | |
| mriedem | efried: didn't you have a devstack patch at one point that depended on https://review.openstack.org/#/c/490057/ and removed the [glance] auth options from nova.conf to see it passing? | 15:33 |
| efried | mriedem one sec... | 15:34 |
| mriedem | artom: good luck | 15:34 |
| mriedem | oh wait, nevermind, we codify image meta | 15:34 |
| mriedem | nova.objects.image_meta | 15:34 |
| dansmith | yeah it's in there | 15:34 |
| mriedem | was thinking extra specs | 15:34 |
| artom | Those are flavour though... (right?) | 15:35 |
| efried | mriedem https://review.openstack.org/#/c/490031/ (It was sdague's) | 15:35 |
| mordred | efried: yes. please make to all version discovery unauthenticated | 15:35 |
| dansmith | however, I think the point is that you can put extra_specs on the aggregate that looks at image properties to keep things in place | 15:35 |
| artom | And flavour can't change with a rebuild... | 15:35 |
| mriedem | artom: correct, | 15:35 |
| mriedem | flavor can only change on resize | 15:35 |
| mriedem | and we run through the scheduler on resize already | 15:35 |
| dansmith | artom: that's the point of the original bug, you might have landed on a linux host, rebuild to a windows image, and blow a license for the operator | 15:35 |
| sdague | efried: those test results current? if so we can get that moved in | 15:37 |
| efried | sdague They're ~6w old. I'll recheck. | 15:37 |
| sdague | efried: cool | 15:37 |
| artom | dansmith, going back to anti-affinity... we never checked the scheduler previously, so if during an instance's lifetime an anti-affinity brother migrated to the same host, the rebuild would not fail | 15:39 |
| artom | Whereas now it would | 15:39 |
| mriedem | efried: i'm a bit confused in that we don't even fill in the auth options for [glance] in nova.conf in our dsvm ci runs today, | 15:40 |
| dansmith | artom: we already have a test for it | 15:40 |
| mriedem | does nova really not talk to glance anywhere where it needs a token that doesn't come from the user context? | 15:40 |
| sdague | mriedem: correct | 15:40 |
| artom | dansmith, ah right, I was reading scrollback backwards | 15:40 |
| mriedem | same for cinder | 15:40 |
| sdague | mriedem: yes | 15:40 |
| efried | what he said ^ | 15:40 |
| artom | This is a minefield:S | 15:41 |
| dansmith | artom: it's in functional.test_server_group somewhere | 15:41 |
| sdague | neutron runs mixed mode, some times user token, some times system | 15:41 |
| *** slaweq has quit IRC | 15:41 | |
| sdague | and ironic is system level, because nova is it's multi tenancy | 15:41 |
| mriedem | the other thing i remembered i need to look at for this is the xenapi plugins for glance | 15:41 |
| efried | sdague Neutron some of each? Hum, I may have screwed up https://review.openstack.org/509892 | 15:41 |
| sdague | efried: yeh, there is a background network info cache refresh iirc | 15:42 |
| efried | sdague Actually, I didn't change how the auth plugin is retrieved for the neutron patch. So whatever it was doing before, it's still doing now. | 15:43 |
| sdague | and, there are times we "escalate" privs because neutron privs for users aren't always sufficient to do server builds | 15:43 |
| sdague | efried: cool | 15:43 |
| sdague | yeh, the neutron one is messier | 15:43 |
| efried | sdague This business - unchanged: https://review.openstack.org/#/c/509892/6/nova/network/neutronv2/api.py@150 | 15:44 |
| mriedem | sdague: do you remember any of the xenapi glance plugin whackiness? https://github.com/openstack/os-xenapi/blob/master/os_xenapi/dom0/etc/xapi.d/plugins/glance.py | 15:44 |
| mriedem | i can't tell if any of that would be affected by this | 15:44 |
| *** gouthamr has joined #openstack-nova | 15:44 | |
| sdague | I do, I can look | 15:45 |
| *** sree has joined #openstack-nova | 15:46 | |
| sdague | oh, ug, that's in a different tree now, one sec | 15:46 |
| mriedem | yeah as far as i can tell, | 15:47 |
| mriedem | https://github.com/openstack/os-xenapi/blob/master/os_xenapi/dom0/etc/xapi.d/plugins/glance.py#L630 | 15:47 |
| efried | That sucker is hardcoding glance URIs. And I can't even see how it's doing authentication. | 15:47 |
| mriedem | it's not getting a token, | 15:47 |
| mriedem | so extra_headers would have to include the token | 15:47 |
| sdague | right | 15:47 |
| mriedem | jianghuaw: still around? | 15:48 |
| sdague | https://github.com/openstack/nova/blob/25afc31b202d66c294a8ac84895e2876cdd6681b/nova/virt/xenapi/image/glance.py#L40 | 15:49 |
| sdague | I think that's where it all comes together | 15:49 |
| jianghuaw | mriedem, yes. | 15:49 |
| sdague | https://github.com/openstack/nova/blob/25afc31b202d66c294a8ac84895e2876cdd6681b/nova/image/glance.py#L99-L106 yeh | 15:50 |
| *** sree has quit IRC | 15:50 | |
| mriedem | ok, so xenapi just gets the glance identity headers and passes those through to the plugin for upload/download | 15:50 |
| sdague | yep | 15:50 |
| sdague | so it's the same basic model, it just does custom serialization | 15:51 |
| mriedem | ok | 15:51 |
| mriedem | jianghuaw: nevermind, we were just discussing https://review.openstack.org/#/c/490057/ and whether or not it would affect the xenapi glance plugin in os-xenapi | 15:51 |
| mriedem | but it looks like the user token is passed through | 15:51 |
| jianghuaw | mriedem, ok:-) | 15:51 |
| mriedem | xenserver ci did fail on that patch | 15:52 |
| mriedem | but it looks like due to devstack setup failing | 15:52 |
| mriedem | http://dd6b71949550285df7dc-dda4e480e005aaa13ec303551d2d8155.r49.cf1.rackcdn.com/57/490057/25/check/dsvm-tempest-neutron-network/c802d48/logs/devstacklog.txt.gz | 15:52 |
| mriedem | TypeError: __init__() got an unexpected keyword argument 'retry_on_request' | 15:52 |
| efried | mordred Can you point me to a doc somewhere that says discovery URIs should be auth-less? | 15:53 |
| *** mvk has quit IRC | 15:54 | |
| mriedem | weird that TypeError was fixed a few months ago Ic98f8cfeb6f78a7e066613787e7cbcea02fcec5c | 15:55 |
| mriedem | so why is the xenserver ci failing on that now? | 15:55 |
| mriedem | Skip the change: openstack/nova:master:refs/changes/57/490057/25 | 15:56 |
| mriedem | wtf | 15:56 |
| mriedem | https://review.openstack.org/#/c/490057/ probably just hasn't been rebased in forever | 15:59 |
| *** jistr is now known as jistr|mtg | 15:59 | |
| *** sree has joined #openstack-nova | 16:00 | |
| *** liuyulong has joined #openstack-nova | 16:00 | |
| mordred | efried: https://review.openstack.org/#/c/459710/17/guidelines/discoverability.rst - line 90 | 16:00 |
| *** marst has joined #openstack-nova | 16:00 | |
| mordred | efried: it hasn't landed yet because I still need to do another pass | 16:01 |
| mriedem | efried: there https://review.openstack.org/#/c/490057/ | 16:01 |
| jianghuaw | mriedem, yeah. That patch may need a rebased. I met one similar failure some days ago. | 16:02 |
| *** yangyapeng has quit IRC | 16:02 | |
| mordred | efried: but, tl;dr - it's fundamental metadata about a service and there is nothing about it that is different per user or secret in any way, so protecting it behind auth makes things harder for no benefit | 16:02 |
| *** yangyapeng has joined #openstack-nova | 16:03 | |
| *** links has joined #openstack-nova | 16:03 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce framework for RFB authentication https://review.openstack.org/345397 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: introduce the VeNCrypt RFB authentication scheme https://review.openstack.org/345398 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: console: provide an RFB security proxy implementation https://review.openstack.org/345399 | 16:03 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: doc: Document TLS security setup for noVNC proxy https://review.openstack.org/500544 | 16:03 |
| *** yamamoto has joined #openstack-nova | 16:03 | |
| *** andreas_s has joined #openstack-nova | 16:04 | |
| *** sree has quit IRC | 16:04 | |
| *** READ10 has joined #openstack-nova | 16:04 | |
| *** slaweq has joined #openstack-nova | 16:05 | |
| jianghuaw | jaypipes, bauzas: may you help to check this patch? https://review.openstack.org/#/c/518294/ I think we got some discussion on it before. But do need your review. Eric has +1 on it. | 16:05 |
| jaypipes | jianghuaw: yes, will get to that this afternoon. | 16:06 |
| *** janki has joined #openstack-nova | 16:06 | |
| jianghuaw | jaypipes, thanks:-) | 16:06 |
| efried | mordred Thanks | 16:06 |
| jaypipes | efried: do you have any local changes on that refactor series? cuz I want to fix up the one thing alex_xu wants to see w.r.t. trait validation. | 16:07 |
| *** mvk has joined #openstack-nova | 16:07 | |
| efried | jaypipes Nope, it's all you. | 16:07 |
| *** gaoyan has joined #openstack-nova | 16:07 | |
| jaypipes | roger | 16:07 |
| efried | jaypipes You saw my comment there too? | 16:07 |
| jaypipes | yup | 16:07 |
| jaypipes | efried: will try to accommodate both of you | 16:08 |
| efried | :) | 16:08 |
| *** yangyapeng has quit IRC | 16:08 | |
| *** andreas_s has quit IRC | 16:09 | |
| jaypipes | efried: shall I rebase all the way to https://review.openstack.org/#/c/521189/ ? | 16:09 |
| efried | jaypipes Yes please. | 16:09 |
| jaypipes | you got it. | 16:09 |
| *** yamamoto has quit IRC | 16:09 | |
| *** sree has joined #openstack-nova | 16:10 | |
| *** itlinux has joined #openstack-nova | 16:10 | |
| *** links has quit IRC | 16:12 | |
| *** gaoyan has quit IRC | 16:13 | |
| jianghuaw | efried, jaypipes: After switching to update_provider_tree(), the implementation in driver should cover the things done in _normalize_inventory_from_cn_obj(). right? | 16:14 |
| efried | jianghuaw That's my position, yes. Others have yet to confirm. | 16:14 |
| jaypipes | jianghuaw: yep, exactly correct. | 16:14 |
| *** sree has quit IRC | 16:14 | |
| jianghuaw | efried, jaypipes thanks. XenAPI will go with that way. Cheers:-) | 16:15 |
| efried | jianghuaw Note that it's gonna be a while before update_provider_tree is ready... | 16:15 |
| efried | jianghuaw Lots of code underneath it that jaypipes and I are working on. | 16:16 |
| *** jistr|mtg is now known as jistr | 16:16 | |
| *** moshele has joined #openstack-nova | 16:16 | |
| *** toure is now known as toure_biab | 16:16 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: WIP: placement: GET /RPs with aggs, inventories, traits https://review.openstack.org/521875 | 16:17 |
| jianghuaw | efried, I see. the change will depends your patches and Jay's patches. Just also try to prepare thing as much as possible for XenAPI. | 16:18 |
| efried | jianghuaw Yup, that's a good idea. Just be prepared to be patient (and flexible - the API very well might change). | 16:18 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 16:19 |
| jianghuaw | efried, sure. Thanks for the heads-up. | 16:19 |
| openstackgerrit | Stephen Finucane proposed openstack/os-vif master: Migrate from 'ip' commands to 'pyroute2' https://review.openstack.org/484386 | 16:21 |
| *** sree has joined #openstack-nova | 16:21 | |
| *** fnordahl has joined #openstack-nova | 16:22 | |
| *** andreas_s has joined #openstack-nova | 16:23 | |
| moshele | stephenfin: respond to you comment in https://review.openstack.org/#/c/519066/ | 16:24 |
| *** edmondsw has quit IRC | 16:24 | |
| *** edmondsw has joined #openstack-nova | 16:25 | |
| stephenfin | moshele: replied | 16:25 |
| *** sree has quit IRC | 16:25 | |
| *** jose-phillips has joined #openstack-nova | 16:26 | |
| stephenfin | ralonsoh: Rebased your 'pyroute2' patch for os-vif. If you can move that to a lib dir, I think that should keep both sean-k-mooney and sahid happy | 16:26 |
| ralonsoh | stephenfin: I'll do this, thanks | 16:27 |
| *** AlexeyAbashkin has quit IRC | 16:28 | |
| *** sree has joined #openstack-nova | 16:28 | |
| moshele | stephenfin: so you want message like https://github.com/openstack/nova/blob/master/nova/network/neutronv2/api.py#L517 ? | 16:28 |
| stephenfin | moshele: Exactly. Something to give a little more context (and make me more comfortable with ignoring the exception) | 16:29 |
| *** edmondsw has quit IRC | 16:29 | |
| *** josecastroleon has quit IRC | 16:29 | |
| moshele | stephenfin: undestood | 16:29 |
| *** psachin has quit IRC | 16:29 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 16:30 |
| efried | mriedem ^ | 16:30 |
| openstackgerrit | Merged openstack/nova master: Deduplicate keypair notification samples https://review.openstack.org/521562 | 16:31 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for neutron client https://review.openstack.org/509892 | 16:31 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for keystone conf & requests https://review.openstack.org/507693 | 16:31 |
| *** andreas_s has quit IRC | 16:31 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | 16:32 |
| *** liuyulong has quit IRC | 16:32 | |
| *** niraj_singh has joined #openstack-nova | 16:32 | |
| *** sree has quit IRC | 16:32 | |
| niraj_singh | i am working on service token for cinder nova interaction. | 16:32 |
| niraj_singh | Cinder backend is NFS and Nova backend is LVM. | 16:32 |
| niraj_singh | Conf option for nova and cinder: http://paste.openstack.org/show/626972/ | 16:32 |
| niraj_singh | i am creating volume snapshot that is attached to an instance. | 16:32 |
| niraj_singh | while creating snapshot i am getting error at compute side. | 16:32 |
| niraj_singh | Please find the log :http://paste.openstack.org/show/626966/ | 16:32 |
| niraj_singh | Any help? | 16:32 |
| *** hemna_ has joined #openstack-nova | 16:32 | |
| bauzas | ralonsoh: https://pypi.python.org/pypi/os-traits/0.4.0 is there | 16:33 |
| bauzas | ralonsoh: could you please provide a new patchset that would use it and just depend on https://review.openstack.org/#/c/521883/ ? | 16:34 |
| ralonsoh | bauzas: i'll submit another patch for https://review.openstack.org/#/c/466051/ | 16:34 |
| bauzas | ralonsoh: cool thanks | 16:34 |
| *** damien_r1 has joined #openstack-nova | 16:34 | |
| *** niraj_singh has quit IRC | 16:35 | |
| *** toure_biab is now known as toure | 16:36 | |
| *** damien_r has quit IRC | 16:36 | |
| efried | sdague I think this was one you and I talked about: https://review.openstack.org/#/c/512329/ | 16:37 |
| *** belmoreira has quit IRC | 16:39 | |
| *** sree has joined #openstack-nova | 16:42 | |
| *** fragatina has quit IRC | 16:43 | |
| gibi | jianghuaw: I see I missed the party in https://review.openstack.org/#/c/512580 and now it is on the gate. Sorry for not having the bandwidth to check it earlier | 16:43 |
| *** fragatina has joined #openstack-nova | 16:44 | |
| *** salv-orlando has quit IRC | 16:45 | |
| *** salv-orlando has joined #openstack-nova | 16:46 | |
| *** fragatina has quit IRC | 16:46 | |
| *** fragatina has joined #openstack-nova | 16:46 | |
| *** sree has quit IRC | 16:47 | |
| openstackgerrit | Moshe Levi proposed openstack/nova master: Don't overwrite binding-profile https://review.openstack.org/519066 | 16:48 |
| moshele | stephenfin: done ^ | 16:49 |
| *** salv-orlando has quit IRC | 16:50 | |
| *** gbarros has quit IRC | 16:50 | |
| *** fragatina has quit IRC | 16:50 | |
| *** moshele has quit IRC | 16:51 | |
| *** fragatina has joined #openstack-nova | 16:51 | |
| *** gbarros has joined #openstack-nova | 16:51 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: required traits for no sharing providers https://review.openstack.org/517027 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Add ProviderSummary.resource_class_names @property https://review.openstack.org/517646 | 16:52 |
| *** chyka has joined #openstack-nova | 16:52 | |
| openstackgerrit | Jay Pipes proposed openstack/nova master: handle traits with sharing providers https://review.openstack.org/517119 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with non overlapping sharing RPs https://review.openstack.org/519380 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test alloc_cands with one RP shared between two RPs https://review.openstack.org/519617 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test allocation candidates: multiple aggregates https://review.openstack.org/518633 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Test helper: validate provider summaries https://review.openstack.org/518982 | 16:52 |
| openstackgerrit | Jay Pipes proposed openstack/nova master: Fix accumulated nits in refactor series https://review.openstack.org/521189 | 16:52 |
| jaypipes | efried: et voila. ^ | 16:52 |
| efried | jaypipes ack | 16:52 |
| sean-k-mooney | jaypipes: did you just push shared RP support or just traits to model shared and non shared RPs? | 16:54 |
| *** andreas_s has joined #openstack-nova | 16:54 | |
| *** dtantsur is now known as dtantsur|afk | 16:55 | |
| *** ragiman has quit IRC | 16:55 | |
| *** fragatina has quit IRC | 16:55 | |
| sean-k-mooney | stephenfin: regarding embedded niantics i believe Intel sells embedded mezzanine form factor cards to dell/hp and there is also a version that is integrated into the motherboard too. | 16:57 |
| efried | sean-k-mooney We're kinda working shared in as we go - at least to the point of maintaining existing functionality. Though we won't officially "support" it in Q, I don't imagine. | 16:57 |
| *** chyka_ has joined #openstack-nova | 16:58 | |
| *** openstackstatus has quit IRC | 16:58 | |
| *** shaner has quit IRC | 16:58 | |
| *** chyka has quit IRC | 16:58 | |
| stephenfin | dansmith: Could you take a look at this? https://review.openstack.org/#/c/396184/ Think it's something I should backport after | 16:58 |
| *** andreas_s has quit IRC | 16:59 | |
| dansmith | stephenfin: oh gawd | 16:59 |
| dansmith | stephenfin: yeah, will look when I get off this call | 16:59 |
| stephenfin | (y) | 16:59 |
| sean-k-mooney | efried: ah cool. | 16:59 |
| *** shaner has joined #openstack-nova | 16:59 | |
| *** openstackstatus has joined #openstack-nova | 17:00 | |
| *** ChanServ sets mode: +v openstackstatus | 17:00 | |
| *** slaweq has quit IRC | 17:01 | |
| *** salv-orlando has joined #openstack-nova | 17:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix NoneType error when [service_user] is misconfigured https://review.openstack.org/521947 | 17:03 |
| mriedem | efried: you'll like this ^ let's not fail with NoneType errors during re-auth | 17:03 |
| mriedem | niraj_singh had a real issue and then left | 17:04 |
| *** ratailor has joined #openstack-nova | 17:04 | |
| efried | mriedem https://review.openstack.org/#/c/512329/ | 17:04 |
| sean-k-mooney | stephenfin: so looking at that you infer the request for pinning by checking "self.cpu_pinning is not None" what dose self.cpu_pinning contain? will that check work if i explcitly set the policy to shared in the flavour | 17:05 |
| stephenfin | sean-k-mooney: We _used_ to do that, then we added a field to actually store the policy | 17:05 |
| efried | mriedem And yeah, I just sent him an email. | 17:05 |
| stephenfin | With recent object versions, we check the policy field. However, that patch ensure the older object versions continue to work | 17:06 |
| efried | mriedem I bet he didn't realize the conf split that happened a couple months ago, and has his stuff in the wrong conf file. | 17:06 |
| *** yamamoto has joined #openstack-nova | 17:06 | |
| *** jianghuaw has quit IRC | 17:06 | |
| *** sridharg has quit IRC | 17:06 | |
| mriedem | efried: replied on yours | 17:06 |
| mriedem | i don't like raising a random exception here | 17:06 |
| efried | mriedem Having seen yours, I suspected that would be the case. | 17:07 |
| mriedem | well, | 17:07 |
| mriedem | it punishes the user for the operator screwing up | 17:07 |
| efried | mriedem I would rather fail early. | 17:07 |
| efried | mriedem Because otherwise they probably get the original bug | 17:07 |
| mriedem | plus, it could leak 500s out of the api | 17:07 |
| mriedem | the original bug is the user token times out and you can't re-auth, which is no different from not using the service user stuff | 17:08 |
| *** artom_ has joined #openstack-nova | 17:08 | |
| mriedem | with your change, the operator screws up and the api user is punished, plus we probably get 500s for this now | 17:08 |
| mriedem | because i'm sure there is REST API code calling glance/cinder/neutron client stuff and not handling the error you're raising | 17:09 |
| efried | mriedem The service will fail very early, likely before the user even got his hands on it. | 17:09 |
| *** yangyapeng has joined #openstack-nova | 17:09 | |
| *** tesseract has quit IRC | 17:09 | |
| mriedem | you're assuming operators are doing full api test coverage, including all of the proxy apis which are going to hit tis code | 17:09 |
| *** josecastroleon has joined #openstack-nova | 17:10 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: test: Store the OutputStreamCapture fixture https://review.openstack.org/515146 | 17:10 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-status: Migrate to cliff https://review.openstack.org/515147 | 17:10 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: trivial: Rename 'policy_check' -> 'policy' https://review.openstack.org/515148 | 17:10 |
| *** ratailor has quit IRC | 17:10 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: nova-policy: Migrate to cliff https://review.openstack.org/515149 | 17:10 |
| efried | mriedem It only has to be hit once from one API. Like if they create a flavor or an image or whatever. | 17:10 |
| efried | mriedem Which admins tend to do early on before they hand their cloud off to users, true? | 17:10 |
| efried | mriedem So I guess we don't cover the case where they've got an existing cloud and they just decide to enable this thing. | 17:10 |
| mriedem | i also wouldn't backport yours | 17:11 |
| *** yamamoto has quit IRC | 17:12 | |
| mriedem | because if i'm on stable and pick that up, and all of a sudden i start getting 500s, i'd be annoyed | 17:12 |
| *** artom has quit IRC | 17:13 | |
| *** yangyapeng has quit IRC | 17:14 | |
| efried | mriedem Is backporting a consideration? The service user thing is experimental, right? | 17:14 |
| mriedem | i don't consider it experimental | 17:14 |
| mriedem | it's been in since ocata and we run with it enabled in our nova-next job | 17:14 |
| mriedem | so yes i was going to backport | 17:14 |
| mriedem | i've also marketed this feature during 2 summit project update talks | 17:15 |
| efried | mriedem https://github.com/openstack/nova/blob/master/nova/conf/service_token.py#L43 | 17:15 |
| efried | So I guess we oughtta take that line out ^ | 17:15 |
| jaypipes | sean-k-mooney: that series touches both traits and shared resource providers | 17:15 |
| *** lucasagomes is now known as lucas-afk | 17:15 | |
| jaypipes | sean-k-mooney: it's a combo of me, efried and gibi. | 17:15 |
| mriedem | efried: yeah i'd be fine with that | 17:16 |
| mriedem | i'd leave it disabled by default | 17:16 |
| *** purplerbot has quit IRC | 17:18 | |
| *** purplerbot has joined #openstack-nova | 17:18 | |
| *** edand has quit IRC | 17:19 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Service token is not experimental https://review.openstack.org/521955 | 17:21 |
| efried | mriedem ^ | 17:21 |
| *** janki has quit IRC | 17:22 | |
| mriedem | wanna remove the 'this is disabled by default' line? | 17:23 |
| mriedem | then i'm +2 | 17:23 |
| *** Apoorva has joined #openstack-nova | 17:23 | |
| dansmith | artom_: good comments, thanks for those | 17:23 |
| dansmith | mriedem: you might want to look at those before you push up a rev and see if you have opinions | 17:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Remove deprecated 'multi_instance_display_name_template' opt https://review.openstack.org/499612 | 17:24 |
| openstackgerrit | Stephen Finucane proposed openstack/nova master: Simplify instance name generation https://review.openstack.org/516573 | 17:24 |
| artom_ | dansmith, yey, my brain works! | 17:24 |
| sean-k-mooney | stephenfin: sorry had to step away for a minute. well what i was wondering is wether "is not none" is enough e.g. can self.cpu_pinning contain "shared" | 17:27 |
| mriedem | efried: +2 on https://review.openstack.org/#/c/490057 - thanks for the quick updates | 17:29 |
| stephenfin | sean-k-mooney: cpu_pinning contains a dict of host to guest CPU mappings | 17:29 |
| efried | mriedem Thanks. | 17:30 |
| stephenfin | You're thinking of 'cpu_policy', which would contain 'shared' or 'dedicated' | 17:30 |
| * stephenfin is catching a lift and has to run himself | 17:30 | |
| openstackgerrit | Merged openstack/nova master: Merge flavor extensions controller code https://review.openstack.org/516104 | 17:30 |
| *** fragatina has joined #openstack-nova | 17:32 | |
| sean-k-mooney | stephenfin: yes i just noticed self.cpu_pinning is a proxy field for cpu_pinning_raw whic is a dict of integers presumable the vCPU to pCPU mappings | 17:33 |
| efried | mriedem I dup'd my bug (https://bugs.launchpad.net/nova/+bug/1724689) to yours and abandoned my patch. | 17:34 |
| openstack | Launchpad bug 1733642 in OpenStack Compute (nova) "duplicate for #1724689 AttributeError: 'NoneType' object has no attribute 'get_token'" [Medium,In progress] - Assigned to Matt Riedemann (mriedem) | 17:34 |
| mriedem | efried: ok, i was going to dupe my bug against yours but ok :) | 17:34 |
| artom_ | I feel like naming it "policy" is confusing us | 17:34 |
| artom_ | As I writing an internal email about it, I came up with "qualitative" (what we're calling policy) vs "quantitative" (ie, resources) | 17:35 |
| artom_ | Does that work better? (Or at all?) | 17:35 |
| *** artom_ is now known as artom | 17:36 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Service token is not experimental https://review.openstack.org/521955 | 17:36 |
| efried | mriedem That's done ^ | 17:36 |
| mriedem | +2 | 17:37 |
| efried | sdague Service token-y stuff while you're in the mood: https://review.openstack.org/#/c/521947/ https://review.openstack.org/521955 | 17:38 |
| dansmith | artom_: I don't really think that has much more meaning to me, but I'm obviously biased | 17:40 |
| dansmith | artom_: I'll let mriedem make the call on what the name should be | 17:40 |
| artom | Ugh, it'll be some obscure 80s rock band | 17:40 |
| mriedem | i agree it's confusing that _GroupAntiAffinityFilter is a policy filter but _GroupAffinityFilter isn't | 17:41 |
| dansmith | fine by me :P | 17:41 |
| sdague | efried: looking | 17:41 |
| mriedem | i mean, we could just call the damn thing RUN_FOR_REBUILD but that doesn't help a ton with the reasoning behind which filters should be run during rebuild or not | 17:41 |
| dansmith | and I expect we need to re-use much of the logic for some other things where we need to check with the scheduler | 17:42 |
| mriedem | example? | 17:42 |
| dansmith | although I guess resize does need to check the resourcey things | 17:42 |
| dansmith | I dunno | 17:42 |
| dansmith | feels far too targeted to just say it's for resize | 17:42 |
| mriedem | i think rebuild is just the odd duck | 17:42 |
| dansmith | well, if that's really the case, then maybe we should just be specific until we have a counterexample | 17:43 |
| mriedem | resize runs through the scheduler because it's going to do a claim on the chosen host | 17:43 |
| dansmith | yeah | 17:43 |
| mriedem | live migration doesn't do a claim | 17:43 |
| mriedem | but live migration does allocate on the dest host | 17:43 |
| mriedem | using the same flavor | 17:43 |
| sdague | efried: so, on https://review.openstack.org/#/c/490057/27/nova/api/openstack/compute/servers.py, while it's fine to pass context around, did you look into just pulling it from thread local storage? | 17:43 |
| sdague | efried: https://github.com/openstack/oslo.context/blob/18aa6ec496e54b2403c2cc65234ef6447021fdee/oslo_context/context.py#L490-L495 | 17:44 |
| efried | sdague I didn't. | 17:44 |
| efried | sdague tbh, mordred did the code-side work on that patch. I just did the test, and kept it up to date. | 17:44 |
| sdague | ok, the long pass through of some of these variables gets a little spidery. It's probably fine here, but something to think about in the future | 17:44 |
| mriedem | that's why i worried about something doing context.get_admin_context() and that eventually going through here | 17:45 |
| mriedem | but apparently we don't do that with images | 17:45 |
| efried | sdague Yeah, that would have made things a lot easier. Is that the same thing? | 17:45 |
| *** pcaruana has quit IRC | 17:48 | |
| sdague | mriedem: yeh, I could see that | 17:49 |
| sdague | it's one of those things where context was written originally to not have to pass it around | 17:49 |
| sdague | but that got forgotten at some point | 17:49 |
| *** ralonsoh has quit IRC | 17:49 | |
| sdague | so, meh | 17:49 |
| sdague | efried: +A on that patch, I'll look at the rest of the conversions on top once I get some lunch | 17:50 |
| efried | sdague Thank you sir. | 17:50 |
| efried | sdague (mriedem) Note that the cinder one might still be busted. But now that this one is going through, I'll get back to looking at that. | 17:50 |
| mriedem | efried: it does remind me that someone had a patch to add some code for nova to auth with a service user to do things with cinder like we do for neutron | 17:52 |
| mriedem | i think in one of the periodic tasks | 17:52 |
| efried | mriedem Yeah, I couldn't remember if that was niraj_singh, but I don't think so. | 17:52 |
| *** claudiub|2 has quit IRC | 17:52 | |
| mriedem | efried: https://review.openstack.org/#/c/384799/ | 17:52 |
| efried | mriedem Oh, actually I was thinking of someone who was trying to use service auth from *within* cinder. | 17:53 |
| *** READ10 has quit IRC | 17:53 | |
| *** sree has joined #openstack-nova | 17:53 | |
| artom | dansmith, mriedem, I think we need to get stephenfin to look at the PCI and NUMA bits when he comes back tomorrow | 17:53 |
| dansmith | okay I'm off tomorrow and mriedem is too I think | 17:53 |
| mriedem | pci is extra specs only | 17:53 |
| mriedem | i asked stephen about that yesterday | 17:54 |
| mriedem | numa is definitely a resource thing | 17:54 |
| mriedem | we do claims on numa limits in the resource tracker | 17:54 |
| artom | mriedem, can an admin change them though? | 17:54 |
| *** yamahata has joined #openstack-nova | 17:54 | |
| mriedem | an admin could change the pci passthrough settings in flavor extra specs, sure | 17:54 |
| artom | For NUMA the cpu and thread policy can change, I dunno if that can give rise to the situation where the host can no longer accept them | 17:54 |
| mriedem | changing extra specs on an existing flavor would be kind of a bad move i think | 17:55 |
| *** damien_r1 has quit IRC | 17:55 | |
| mriedem | but we allow it | 17:55 |
| dansmith | the flavor embedded in the instance will have the pci stuff then, which means we won't see the changed extra_specs from the top-level flavor in the rebuild | 17:55 |
| dansmith | the cpu and thread policy change in the image meta might be a reason to do the rebuild though, | 17:56 |
| dansmith | so I dunno what to say about that | 17:56 |
| *** sree has quit IRC | 17:57 | |
| artom | dansmith, oh right, that's true, flavours stay with instances now | 17:57 |
| *** hongbin has joined #openstack-nova | 17:58 | |
| artom | So anything involving flavor extra specs isn't policy then - ie, no need to check with it on rebuild, it hasn't changed for the actual instance | 17:58 |
| dansmith | well, no | 17:58 |
| *** derekh has quit IRC | 17:58 | |
| dansmith | flavors stay with the instance, flavours are not a thing in nova :P | 17:58 |
| artom | What do you mean not a thing? | 17:58 |
| artom | Oh | 17:59 |
| dansmith | artom: yeah, that's probably a good guideline, although some flavor extra_specs get overridden by image meta in some weird ways I think | 17:59 |
| artom | Ugh. | 18:00 |
| mriedem | NUMATopologyFilter gets values from flavors first, and if not set, checks the image meta | 18:00 |
| artom | Just set everything to policy and wait for the bug reports then >:( | 18:00 |
| mriedem | so flavor extra specs take precedence there | 18:00 |
| dansmith | artom: that's the way it is now, and the bug we're fixing :P | 18:00 |
| mriedem | oh wait nvm that | 18:00 |
| mriedem | re the question about the az filter | 18:01 |
| mriedem | https://review.openstack.org/#/c/446446/ | 18:01 |
| mriedem | "Unfortunately, as an operator | 18:02 |
| mriedem | can modify any metadata, it is possible to change the value of the | 18:02 |
| mriedem | availability_zone key for a specific aggregate and accordingly change what the | 18:02 |
| mriedem | user can see when looking at AZs" | 18:02 |
| *** jistr is now known as jistr|off|trng | 18:04 | |
| openstackgerrit | Merged openstack/nova master: vgpu: add enabled white list https://review.openstack.org/512580 | 18:05 |
| *** moshele has joined #openstack-nova | 18:07 | |
| *** yamamoto has joined #openstack-nova | 18:08 | |
| *** mvk has quit IRC | 18:09 | |
| *** david-lyle has quit IRC | 18:09 | |
| *** david-lyle has joined #openstack-nova | 18:09 | |
| mriedem | we persist the flavor with the request spec yes? | 18:10 |
| dansmith | flavor id you mean? | 18:10 |
| mriedem | the full flavor | 18:10 |
| mriedem | looks like yes | 18:11 |
| dansmith | hrm, I guess we do | 18:11 |
| mriedem | which means, re the TrustedFilter question, | 18:11 |
| mriedem | if the admin changes the flavor extra specs for the trusted filter after you've created an instance, when you rebuild or migrate that instance, the filter is going to work on the original flavor used to create the instance | 18:11 |
| mriedem | not the modified one | 18:11 |
| mriedem | *i think* | 18:12 |
| artom | That's what I understood when dansmith said that flavoUrs stay with the instance, yeah | 18:12 |
| *** yamamoto has quit IRC | 18:12 | |
| *** _pewp_ has quit IRC | 18:14 | |
| *** sean-k-mooney has quit IRC | 18:16 | |
| mriedem | my favorite burstlet flavour is tripe | 18:17 |
| mriedem | artom: dansmith: ok, some replies inline https://review.openstack.org/#/c/521186/ | 18:17 |
| *** _pewp_ has joined #openstack-nova | 18:18 | |
| mriedem | dansmith: btw, i changed _GroupAntiAffinityFilter to False, ran functional tests and nothing failed | 18:18 |
| dansmith | mriedem: it'll be non-deterministic | 18:19 |
| mriedem | gha | 18:19 |
| mriedem | *gah | 18:19 |
| dansmith | because if it selects the other host by chance it'll be happy | 18:19 |
| *** Swami has joined #openstack-nova | 18:19 | |
| dansmith | it was oddly repeatable for me.. two tests validating the same thing (for two microversions) one would always pass the other would always fail | 18:19 |
| dansmith | took me a while to figure out why | 18:19 |
| dansmith | I was able to shake it up with different levels of concurrency and running the tests in isolation | 18:20 |
| mriedem | test_rebuild_with_soft_anti_affinity ? | 18:20 |
| *** jpena is now known as jpena|off | 18:20 | |
| mriedem | or test_rebuild_with_anti_affinity | 18:20 |
| dansmith | soft I think | 18:20 |
| *** moshele has quit IRC | 18:21 | |
| mriedem | that one involves a weigher so yeah might be that one | 18:21 |
| dansmith | I can try to repro if it's important | 18:21 |
| mriedem | will run it a few times | 18:21 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | 18:21 |
| mriedem | i am dangerously close to needing some deterministic lunch | 18:22 |
| dansmith | mriedem: http://logs.openstack.org/86/521186/3/check/openstack-tox-functional/dc8ab81/testr_results.html.gz | 18:23 |
| dansmith | that was the failure I tried to repro locally that led me down the path I think | 18:23 |
| *** felipemonteiro__ has joined #openstack-nova | 18:23 | |
| *** slaweq has joined #openstack-nova | 18:24 | |
| *** felipemonteiro_ has quit IRC | 18:26 | |
| *** ijw has joined #openstack-nova | 18:31 | |
| *** ijw has quit IRC | 18:35 | |
| *** READ10 has joined #openstack-nova | 18:36 | |
| *** READ10 has quit IRC | 18:36 | |
| *** AlexeyAbashkin has joined #openstack-nova | 18:40 | |
| *** AlexeyAbashkin has quit IRC | 18:44 | |
| *** imacdonn has quit IRC | 18:45 | |
| *** imacdonn has joined #openstack-nova | 18:45 | |
| dansmith | artom_: mriedem well, now I can't get it to fail | 18:48 |
| *** pchavva has quit IRC | 18:49 | |
| mriedem | :/ | 18:49 |
| dansmith | oh, wait | 18:49 |
| mriedem | was that with same_host or something? | 18:49 |
| dansmith | I bet it's because I hit this before I was doing the force_hosts | 18:49 |
| mriedem | ah | 18:49 |
| dansmith | it was when I was going to do the check in conductor manager I think | 18:50 |
| mriedem | so we can probably change that one then | 18:50 |
| mriedem | which would help with the confusion | 18:50 |
| dansmith | yeah I guess so | 18:50 |
| mriedem | dansmith: artom: in general, what are your thoughts/feelings/hopes/dreams about just calling the hint _is_rebuild and the filter attribute RUN_FOR_REBUILD? then each filter decision is about that scope/use case rather than what policy means | 18:51 |
| dansmith | I dunno, it feels too specific, but I can't come up with another reason to do this really | 18:52 |
| mriedem | right, like you said, we could change it later if we have a need to generify it | 18:52 |
| dansmith | yeah | 18:53 |
| mriedem | we could also maybe remove the variable at some point later with rpc version changes, like passing an actual is_rebuild parameter to select_destinations or something | 18:53 |
| dansmith | yeah | 18:53 |
| mriedem | i'm not sure that would buy us much, except maybe more straight-forward | 18:53 |
| dansmith | ideally it'd be operation="rebuild" to give us some room for the future | 18:53 |
| dansmith | but yeah | 18:53 |
| mriedem | sure | 18:53 |
| mriedem | operation="mind crime" | 18:54 |
| dansmith | heh | 18:54 |
| mriedem | 80s queensryche reference for artom | 18:54 |
| *** mvk has joined #openstack-nova | 18:54 | |
| *** itlinux has quit IRC | 18:57 | |
| *** slaweq has quit IRC | 18:58 | |
| artom | I was -5 for half the 80s, though I do enjoy some of their music now :) | 18:58 |
| artom | Renaming it to RUN_FOR_REBUILD would definitely make the scope clearer | 18:58 |
| *** slaweq has joined #openstack-nova | 18:58 | |
| artom | I agree it's not pretty | 18:58 |
| *** Sukhdev has joined #openstack-nova | 18:59 | |
| artom | Though I'm not as repulsed by the api <-> scheduler coupling - the scheduler knowing about rebuilds isn't as far fetched as it sounds | 18:59 |
| artom | I think ideally I'd like more methods in the filters, like I said in my last comment | 18:59 |
| artom | So host_passes for "can this host run this instance now?", host_acceptable for "can this host accept this instance, resource considerations aside", and maybe other ones for cases we haven't thought of | 19:00 |
| artom | But that doesn't look very backportable | 19:00 |
| *** jeblair is now known as thecount | 19:01 | |
| *** thecount is now known as jeblair | 19:01 | |
| *** slaweq has quit IRC | 19:03 | |
| *** Gorian has quit IRC | 19:03 | |
| *** itlinux has joined #openstack-nova | 19:03 | |
| *** Sukhdev has quit IRC | 19:04 | |
| *** gbarros has quit IRC | 19:04 | |
| openstackgerrit | Merged openstack/nova master: cleanup mapping/reqspec after archive instance https://review.openstack.org/515034 | 19:04 |
| *** gbarros has joined #openstack-nova | 19:05 | |
| mriedem | mgagne: a couple of small changes to make here and then i think we're good https://review.openstack.org/#/c/503088 | 19:05 |
| mriedem | artom: the scheduler is definitely aware of moves for placement allocations, even if it's not super obvious | 19:06 |
| mgagne | mriedem: will work on it today. I think committed the Ironic release note by mistake, will reword as suggested | 19:06 |
| mriedem | mgagne: yeah heh, that was my comment :) | 19:07 |
| mgagne | didn't read yet :P, will do | 19:07 |
| artom | mriedem, yeah, it decides which instances go to which hosts, to operations on instances that have to do with hosts aren't outside its domain :) | 19:07 |
| artom | *so operations | 19:08 |
| *** Gorian has joined #openstack-nova | 19:08 | |
| artom | If we started telling it about keypair updates I'd be more concerned ;) | 19:08 |
| mriedem | artom: i mean this specifically https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L1035 | 19:08 |
| *** ijw has joined #openstack-nova | 19:08 | |
| *** yamamoto has joined #openstack-nova | 19:09 | |
| *** hoonetorg has quit IRC | 19:09 | |
| *** hoonetorg has joined #openstack-nova | 19:10 | |
| *** AlexeyAbashkin has joined #openstack-nova | 19:11 | |
| *** ijw has quit IRC | 19:12 | |
| *** yamamoto has quit IRC | 19:13 | |
| *** gbarros has quit IRC | 19:13 | |
| *** gbarros has joined #openstack-nova | 19:14 | |
| efried | cdent Still around? | 19:16 |
| *** AlexeyAbashkin has quit IRC | 19:16 | |
| cdent | yessir | 19:16 |
| mgagne | mriedem: so should I refactor _generate_configdrive or not? | 19:16 |
| efried | cdent Is there a way in gabbi for me to say "This must *not* be in the response" ? | 19:16 |
| cdent | in a body or header? | 19:17 |
| efried | body | 19:17 |
| efried | I know about checking the length of a chunk of the response; will fall back on that if nothing else is available. | 19:18 |
| cdent | there are a couple of additional options: | 19:18 |
| cdent | if the surrounding chunk of json is small and you know all of it, you can make the value of the jsonpath be the entire chunk, instead of just a scalar value | 19:19 |
| cdent | or you might be able to do something with regular expressions | 19:19 |
| cdent | (the value side of the jsonpath statement can be a regex) | 19:19 |
| cdent | efried: if you got a pointer to an almost there example I can probably say something more concrete | 19:20 |
| *** marst has quit IRC | 19:20 | |
| efried | cdent Like here: https://github.com/openstack/nova/blob/master/nova/tests/functional/api/openstack/placement/gabbits/resource-provider.yaml#L105 | 19:21 |
| efried | cdent I want to make sure there's no "traits" key | 19:21 |
| efried | etc. | 19:21 |
| * cdent thinks | 19:21 | |
| efried | response_forbidden_json_paths:\n\t$.resource_providers[0].traits | 19:22 |
| cdent | heh | 19:22 |
| cdent | is your intent to validate the entire response? If so, you might consider: | 19:23 |
| cdent | response_json_paths:\n$: @<my-big-response.json | 19:23 |
| mriedem | mgagne: no, i think what you have is ok | 19:24 |
| efried | cdent Right; though the docs kinda recommended against that. | 19:24 |
| mriedem | the error handling being different makes a refactor less worthwhile | 19:24 |
| openstackgerrit | Eric Young proposed openstack/nova master: Implement ScaleIO image backend https://review.openstack.org/407440 | 19:24 |
| mgagne | mriedem: got it | 19:24 |
| cdent | efried: they also recommend not validating entire responses | 19:24 |
| cdent | (in one test) | 19:24 |
| efried | cdent Specifically in this case I'd like to ensure I don't return the extra fields when I don't request them. (Related to the traits/aggregates/inventories thing from this morning) | 19:25 |
| efried | cdent (Yes, I realize I haven't abandoned this patch, but I'm taking it as an opportunity to learn gabbi.) | 19:25 |
| cdent | :) | 19:26 |
| cdent | another option would be to write your own CustomHandler that actually implements response_forbidden_json_paths | 19:26 |
| * cdent looks at the jsonpath_rw docs | 19:27 | |
| efried | cdent That would live in nova, or that would be contributed back to gabbi? | 19:27 |
| efried | oh, jsonpath_rw, that's a separate thing. /me looks too... | 19:28 |
| *** slaweq has joined #openstack-nova | 19:28 | |
| *** gszasz has quit IRC | 19:29 | |
| *** pchavva has joined #openstack-nova | 19:29 | |
| cdent | I was thinking live in nova initially, and then merged to gabbi if it felt sufficiently useful/nice | 19:30 |
| *** tianhui has joined #openstack-nova | 19:30 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: WIP(5): PowerVM driver: ovs vif https://review.openstack.org/422512 | 19:31 |
| cdent | efried: this might be a case where it would be better to have a unit test of your serializer | 19:32 |
| efried | cdent Also. | 19:33 |
| dansmith | mriedem: you have some changes against the set locally, right? will you make the hint change and the other things we discussed as part of that? | 19:34 |
| mriedem | dansmith: i don't have anything local | 19:34 |
| *** slaweq has quit IRC | 19:34 | |
| dansmith | oh okay I thought you were working on the reno and flipping some of the flags | 19:35 |
| efried | cdent Thanks for the help. A CustomHandler might be more than I'm willing to bite off at the moment, but I'll pester you if I decide to have a crack at it. | 19:35 |
| dansmith | I'll go ahead and do that other stuff then | 19:35 |
| mriedem | dansmith: that got sidetracked by the review comments and waiting to hear back from some people | 19:35 |
| dansmith | okay | 19:35 |
| *** tianhui has quit IRC | 19:37 | |
| openstackgerrit | Vipin Balachandran proposed openstack/nova master: VMware: Support volumes backed by VStorageObject https://review.openstack.org/521690 | 19:37 |
| *** NightKhaos has quit IRC | 19:39 | |
| *** ijw has joined #openstack-nova | 19:41 | |
| *** NightKhaos has joined #openstack-nova | 19:44 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] Symmetric GET and PUT /allocations/{consumer_uuid} https://review.openstack.org/510626 | 19:44 |
| openstackgerrit | Chris Dent proposed openstack/nova master: [placement] POST /allocations to set allocations for >1 consumers https://review.openstack.org/500073 | 19:44 |
| *** ijw has quit IRC | 19:47 | |
| *** josecastroleon has quit IRC | 19:48 | |
| *** ijw has joined #openstack-nova | 19:48 | |
| efried | cdent Sanity check me here: POST /resource_providers/{uuid}/inventories is not documented? | 19:49 |
| cdent | yeah, it was sort deprecated | 19:50 |
| cdent | there’s a review somewhere adding it back in, that’s under discussion | 19:51 |
| efried | cdent Oh, okay. So I won't hurry to document it. | 19:51 |
| openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.54 - Enable reset keypair while rebuild https://review.openstack.org/519572 | 19:51 |
| cdent | efried: here’s that review https://review.openstack.org/#/c/511342/ | 19:53 |
| *** ijw has quit IRC | 19:55 | |
| *** suresh12 has joined #openstack-nova | 20:00 | |
| *** cdent has quit IRC | 20:00 | |
| *** slaweq has joined #openstack-nova | 20:01 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: Update bindep.txt for doc builds https://review.openstack.org/521755 | 20:02 |
| *** tbachman has quit IRC | 20:02 | |
| openstackgerrit | Merged openstack/nova master: Get auth from context for glance endpoint https://review.openstack.org/490057 | 20:04 |
| *** suresh12 has quit IRC | 20:05 | |
| *** slaweq has quit IRC | 20:06 | |
| *** pchavva has quit IRC | 20:07 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova stable/pike: Update bindep.txt for doc builds https://review.openstack.org/521999 | 20:09 |
| *** tbachman has joined #openstack-nova | 20:09 | |
| *** jose-phillips has quit IRC | 20:09 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova stable/ocata: Update bindep.txt for doc builds https://review.openstack.org/522000 | 20:09 |
| openstackgerrit | Andreas Jaeger proposed openstack/nova stable/newton: Update bindep.txt for doc builds https://review.openstack.org/522001 | 20:09 |
| *** yamamoto has joined #openstack-nova | 20:10 | |
| *** AlexeyAbashkin has joined #openstack-nova | 20:11 | |
| *** jose-phillips has joined #openstack-nova | 20:12 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: Skip authentication on version discovery URI https://review.openstack.org/522002 | 20:12 |
| *** yamamoto has quit IRC | 20:14 | |
| *** AlexeyAbashkin has quit IRC | 20:15 | |
| *** salv-orlando has quit IRC | 20:15 | |
| *** salv-orlando has joined #openstack-nova | 20:16 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: skip authentication on root URI https://review.openstack.org/522002 | 20:16 |
| *** salv-orlando has quit IRC | 20:20 | |
| *** ijw has joined #openstack-nova | 20:22 | |
| *** tbachman has quit IRC | 20:22 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 20:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 20:23 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 20:23 |
| dansmith | mriedem: artom: ^ | 20:24 |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | 20:28 |
| artom | dansmith, looking | 20:29 |
| dansmith | I think I got everything but we talked about lots of things | 20:29 |
| *** sahid has quit IRC | 20:29 | |
| openstackgerrit | Andreas Jaeger proposed openstack/nova master: DNM - test updated releasenotes and sphinx job https://review.openstack.org/520677 | 20:30 |
| *** slaweq has joined #openstack-nova | 20:35 | |
| mriedem | dansmith: comments inline | 20:37 |
| mriedem | mostly just nits | 20:37 |
| mriedem | i can update next week if you're tired of this | 20:37 |
| *** salv-orlando has joined #openstack-nova | 20:37 | |
| *** vladikr has quit IRC | 20:38 | |
| *** vladikr has joined #openstack-nova | 20:38 | |
| *** suresh12 has joined #openstack-nova | 20:38 | |
| *** suresh12 has quit IRC | 20:40 | |
| *** slaweq has quit IRC | 20:40 | |
| *** ijw_ has joined #openstack-nova | 20:41 | |
| dansmith | oh I'm tired of it for sure but: | 20:41 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Alternate fix for validating image on rebuild https://review.openstack.org/521186 | 20:41 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Add regression test for rebuild with new image doubling allocations https://review.openstack.org/521153 | 20:41 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 20:41 |
| dansmith | apparently I broke the top patch in the process here | 20:41 |
| * dansmith re-runs tests | 20:41 | |
| *** sbezverk has quit IRC | 20:42 | |
| mriedem | did you drop the filter scheduler claim part? | 20:43 |
| *** ijw has quit IRC | 20:44 | |
| mriedem | oh nvm i'm looking at the wrong patch | 20:45 |
| mriedem | dansmith: yeah this needs updating https://review.openstack.org/#/c/521662/5/nova/scheduler/filter_scheduler.py | 20:45 |
| mriedem | +1 on the bottom change though | 20:47 |
| artom | Hah, I think you fixed some of what I was about to post before I had a chance to post it | 20:49 |
| * artom starts over with 7 | 20:49 | |
| dansmith | mriedem: yep, that's it | 20:50 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Fix doubling allocations on rebuild https://review.openstack.org/521662 | 20:51 |
| *** ijw_ has quit IRC | 20:52 | |
| openstackgerrit | Merged openstack/nova master: Service token is not experimental https://review.openstack.org/521955 | 20:52 |
| *** priteau has quit IRC | 20:54 | |
| *** swebster has joined #openstack-nova | 20:55 | |
| *** moshele has joined #openstack-nova | 20:59 | |
| mriedem | ok barring renos and such i think that series is good | 20:59 |
| *** suresh12 has joined #openstack-nova | 21:00 | |
| *** Guest25442 has quit IRC | 21:01 | |
| artom | dansmith, two nits from me. And I need to look at the tests eventually | 21:02 |
| *** moshele has quit IRC | 21:03 | |
| *** ijw has joined #openstack-nova | 21:04 | |
| dansmith | artom: mriedem is going to write the reno, so I'll let him update those things if he agrees | 21:04 |
| dansmith | I'm going to be fading out here in a bit | 21:04 |
| artom | Sounds good | 21:05 |
| mriedem | the numa one is probably a change, i forgot that was using image props | 21:08 |
| mriedem | but yeah next week at this point | 21:08 |
| mriedem | i'm out of here in less than an hour | 21:08 |
| *** ijw has quit IRC | 21:08 | |
| artom | Enjoy flightless birds and excessive consumption :) | 21:08 |
| mriedem | i don't actually care for turkey all that much | 21:08 |
| artom | Maybe I meant emus, for all you know | 21:08 |
| mriedem | gross | 21:09 |
| artom | Or kiwis | 21:09 |
| mriedem | i am no longer in australia so none of those animals apply | 21:09 |
| melwitt | we ate some emu while in australia. it was good | 21:09 |
| *** salv-orlando has quit IRC | 21:11 | |
| *** yamamoto has joined #openstack-nova | 21:11 | |
| *** sbezverk has joined #openstack-nova | 21:11 | |
| *** slaweq has joined #openstack-nova | 21:11 | |
| *** salv-orlando has joined #openstack-nova | 21:11 | |
| *** yamamoto has quit IRC | 21:15 | |
| mriedem | let me go tell laura | 21:16 |
| *** salv-orlando has quit IRC | 21:16 | |
| *** slaweq has quit IRC | 21:16 | |
| *** marst has joined #openstack-nova | 21:19 | |
| *** lpetrut has quit IRC | 21:21 | |
| *** sree has joined #openstack-nova | 21:21 | |
| *** vishwanathj has joined #openstack-nova | 21:23 | |
| *** sree has quit IRC | 21:26 | |
| *** tbachman has joined #openstack-nova | 21:26 | |
| *** tssurya_ has joined #openstack-nova | 21:31 | |
| *** tssurya_ has quit IRC | 21:32 | |
| *** eharney has quit IRC | 21:37 | |
| *** salv-orlando has joined #openstack-nova | 21:41 | |
| *** sree has joined #openstack-nova | 21:47 | |
| *** dave-mcc_ has quit IRC | 21:47 | |
| *** threestrands has joined #openstack-nova | 21:48 | |
| *** rcernin has joined #openstack-nova | 21:50 | |
| *** sree has quit IRC | 21:52 | |
| *** tbachman has quit IRC | 21:53 | |
| openstackgerrit | Hongbin Lu proposed openstack/nova master: placement: skip authentication on root URI https://review.openstack.org/522002 | 21:54 |
| *** awaugama has quit IRC | 21:56 | |
| *** ijw has joined #openstack-nova | 21:56 | |
| *** mvk has quit IRC | 21:59 | |
| *** ijw has quit IRC | 22:00 | |
| *** ijw has joined #openstack-nova | 22:00 | |
| *** Apoorva_ has joined #openstack-nova | 22:02 | |
| *** mriedem has quit IRC | 22:05 | |
| *** Apoorva has quit IRC | 22:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: Deprecate file injection https://review.openstack.org/522027 | 22:05 |
| *** ijw_ has joined #openstack-nova | 22:07 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: placement: GET /RPs with aggs, inventories, traits https://review.openstack.org/521875 | 22:09 |
| *** sree has joined #openstack-nova | 22:09 | |
| efried | jaypipes FYI, added gabbits to ^ | 22:09 |
| jaypipes | ack | 22:10 |
| efried | jaypipes Now I'm a gabbi expert. | 22:10 |
| jaypipes | :) | 22:10 |
| *** ijw has quit IRC | 22:10 | |
| efried | jaypipes And I will go away for a long weekend and eat turkey and forget all of it. | 22:10 |
| *** yangyapeng has joined #openstack-nova | 22:10 | |
| jaypipes | heh, understood :) | 22:10 |
| efried | jaypipes Anything you want my eyes on before I take off today? | 22:11 |
| efried | jaypipes FYI I'll be out tomorrow through the weekend, back Monday. | 22:11 |
| *** yamamoto has joined #openstack-nova | 22:12 | |
| *** sree has quit IRC | 22:14 | |
| *** yangyapeng has quit IRC | 22:14 | |
| jaypipes | efried: enjoy your time off and the holiday | 22:15 |
| jaypipes | efried: and no, I'm good :) | 22:15 |
| *** mvk has joined #openstack-nova | 22:15 | |
| efried | jaypipes You too (assuming you're having some) | 22:15 |
| jaypipes | thanks though, you've been super over the last month | 22:15 |
| efried | Most welcome. | 22:15 |
| jaypipes | efried: yes, definitely doing thanksgiving din-din :) | 22:15 |
| *** markvoelker has quit IRC | 22:16 | |
| *** yamamoto has quit IRC | 22:17 | |
| cfriesen | anyone know if nova is susceptible to https://bugzilla.redhat.com/show_bug.cgi?id=1439078 or are our compatibility checks stringent enough? | 22:21 |
| openstack | bugzilla.redhat.com bug 1439078 in qemu-kvm-rhev "After migration,VM crash in dst host with "qemu-kvm: error: failed to set MSR 0x38f to 0x7000000ff"" [Unspecified,Closed: notabug] - Assigned to dgilbert | 22:21 |
| jaypipes | cfriesen: lol, I read that as "assigned to dilbert" | 22:21 |
| cfriesen | basically live migration failed when going from non-HT host to HT host with the same CPU | 22:21 |
| cfriesen | because the number of PMU registers changed | 22:21 |
| jaypipes | cfriesen: it vaguely rings a bell. I'd ping sahid or kashyap in the morning. | 22:22 |
| cfriesen | sounds like a plan | 22:22 |
| *** amodi has quit IRC | 22:25 | |
| *** markvoelker has joined #openstack-nova | 22:27 | |
| *** marst has quit IRC | 22:30 | |
| *** slaweq has joined #openstack-nova | 22:31 | |
| *** slaweq has quit IRC | 22:32 | |
| *** slaweq has joined #openstack-nova | 22:32 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use ksa adapter for cinder client https://review.openstack.org/508345 | 22:34 |
| *** claudiub|2 has joined #openstack-nova | 22:35 | |
| *** tbachman has joined #openstack-nova | 22:36 | |
| *** burt has quit IRC | 22:37 | |
| openstackgerrit | Mathieu Gagné proposed openstack/nova master: Regenerate and pass configdrive when rebuild Ironic nodes https://review.openstack.org/503088 | 22:37 |
| *** suresh12 has quit IRC | 22:37 | |
| efried | Hasta la Monday, y'all | 22:38 |
| * efried waves | 22:38 | |
| *** efried is now known as fried_turkey | 22:38 | |
| *** suresh12 has joined #openstack-nova | 22:42 | |
| *** lyan has quit IRC | 22:45 | |
| *** ijw_ has quit IRC | 22:51 | |
| *** eharney has joined #openstack-nova | 22:52 | |
| *** sree has joined #openstack-nova | 22:58 | |
| *** sree has quit IRC | 23:01 | |
| *** itlinux has quit IRC | 23:05 | |
| *** bnemec has quit IRC | 23:09 | |
| *** jaypipes has quit IRC | 23:11 | |
| *** bnemec has joined #openstack-nova | 23:12 | |
| *** AlexeyAbashkin has joined #openstack-nova | 23:12 | |
| *** felipemonteiro__ has quit IRC | 23:13 | |
| *** esberglu has quit IRC | 23:13 | |
| *** yamamoto has joined #openstack-nova | 23:14 | |
| *** AlexeyAbashkin has quit IRC | 23:16 | |
| *** yamamoto has quit IRC | 23:18 | |
| *** shvepsy_ has quit IRC | 23:22 | |
| *** slaweq has quit IRC | 23:25 | |
| *** Sandy619 has joined #openstack-nova | 23:28 | |
| *** gmann_afk is now known as gmann | 23:28 | |
| *** sdague has quit IRC | 23:29 | |
| *** Sandy619 has quit IRC | 23:30 | |
| *** takashin has joined #openstack-nova | 23:35 | |
| gmann | jianghuaw_: | 23:56 |
| gmann | jianghuaw_: opps just ignore | 23:56 |
| *** salv-orlando has quit IRC | 23:59 | |
| *** salv-orlando has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!