| melwitt | weird, the github mirror of nova-specs stopped being updated sometime back https://github.com/openstack/nova-specs | 00:00 |
|---|---|---|
| *** r-daneel has quit IRC | 00:01 | |
| *** hongbin has quit IRC | 00:03 | |
| *** weshay is now known as weshay_PTO | 00:04 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Use correct arguments in task inits https://review.openstack.org/543571 | 00:11 |
| *** acormier has joined #openstack-nova | 00:16 | |
| *** acormier has quit IRC | 00:16 | |
| *** acormier has joined #openstack-nova | 00:17 | |
| *** esberglu has quit IRC | 00:24 | |
| *** jobewan has quit IRC | 00:25 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:26 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Re-propose convert consoles code to use objects framework https://review.openstack.org/543662 | 00:26 |
| *** Dinesh_Bhor has quit IRC | 00:26 | |
| *** tetsuro has joined #openstack-nova | 00:29 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:32 | |
| *** tssurya has joined #openstack-nova | 00:33 | |
| *** oomichi has quit IRC | 00:34 | |
| *** vladikr has quit IRC | 00:35 | |
| *** vladikr has joined #openstack-nova | 00:35 | |
| *** pramodrj07 has quit IRC | 00:35 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:39 | |
| *** tssurya has quit IRC | 00:42 | |
| *** moshele has joined #openstack-nova | 00:42 | |
| *** fragatin_ has quit IRC | 00:42 | |
| *** fragatina has joined #openstack-nova | 00:42 | |
| *** jichen has joined #openstack-nova | 00:43 | |
| *** Dinesh_Bhor has quit IRC | 00:43 | |
| *** Dinesh_Bhor has joined #openstack-nova | 00:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova stable/ocata: Add 'delete_host' command in 'nova-manage cell_v2' https://review.openstack.org/513721 | 00:44 |
| *** vladikr has quit IRC | 00:45 | |
| *** vladikr has joined #openstack-nova | 00:45 | |
| *** yassine has joined #openstack-nova | 00:45 | |
| *** yassine is now known as Guest10778 | 00:45 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform live_migration_post_dest notification https://review.openstack.org/469784 | 00:46 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform servergroup.addmember notification https://review.openstack.org/541101 | 00:47 |
| *** tovin07 has joined #openstack-nova | 00:47 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Transform aggregate.update_metadata notification https://review.openstack.org/460625 | 00:47 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Abort Cold Migration https://review.openstack.org/334732 | 00:48 |
| *** oomichi has joined #openstack-nova | 00:55 | |
| *** sdague has quit IRC | 00:55 | |
| *** phuongnh has joined #openstack-nova | 00:56 | |
| *** zhaochao has joined #openstack-nova | 01:00 | |
| *** vladikr has quit IRC | 01:02 | |
| *** vladikr has joined #openstack-nova | 01:02 | |
| *** openstackgerrit has quit IRC | 01:03 | |
| *** slaweq has joined #openstack-nova | 01:03 | |
| *** wxy has joined #openstack-nova | 01:06 | |
| *** gyee has quit IRC | 01:07 | |
| *** slaweq has quit IRC | 01:08 | |
| *** yangyapeng has quit IRC | 01:12 | |
| *** yangyapeng has joined #openstack-nova | 01:12 | |
| *** itlinux has joined #openstack-nova | 01:15 | |
| *** tiendc has joined #openstack-nova | 01:16 | |
| *** suresh12 has quit IRC | 01:16 | |
| *** acormier_ has joined #openstack-nova | 01:16 | |
| *** openstackgerrit has joined #openstack-nova | 01:17 | |
| openstackgerrit | melanie witt proposed openstack/nova-specs master: Re-propose convert consoles code to use objects framework https://review.openstack.org/543662 | 01:17 |
| *** jcosmao has quit IRC | 01:19 | |
| *** andreykurilin has quit IRC | 01:20 | |
| *** acormier has quit IRC | 01:20 | |
| *** andreykurilin has joined #openstack-nova | 01:21 | |
| *** mdnadeem has joined #openstack-nova | 01:23 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova-specs master: Strict isolation of group of hosts for images https://review.openstack.org/381912 | 01:24 |
| *** suresh12 has joined #openstack-nova | 01:24 | |
| *** BrinZhang has joined #openstack-nova | 01:25 | |
| Dinesh_Bhor | cfriesen: Hi, could you please take a look at it whenever you get time: https://review.openstack.org/381912 . I have updated it according to our previous discussion. The flavor case will be covered in request_traits_for_flavor. | 01:28 |
| *** suresh12 has quit IRC | 01:29 | |
| *** acormier_ has quit IRC | 01:29 | |
| *** acormier has joined #openstack-nova | 01:30 | |
| *** vladikr has quit IRC | 01:30 | |
| *** vladikr has joined #openstack-nova | 01:31 | |
| *** mriedem has quit IRC | 01:31 | |
| *** lbragstad has quit IRC | 01:34 | |
| *** moshele has quit IRC | 01:36 | |
| *** Swami has quit IRC | 01:37 | |
| *** yamahata has quit IRC | 01:37 | |
| *** mattoliverau has left #openstack-nova | 01:37 | |
| *** itlinux has quit IRC | 01:43 | |
| *** Kvisle has quit IRC | 01:52 | |
| *** yangyapeng has quit IRC | 01:53 | |
| *** yangyapeng has joined #openstack-nova | 01:54 | |
| *** Kvisle has joined #openstack-nova | 01:55 | |
| *** itlinux has joined #openstack-nova | 01:56 | |
| *** oomichi has quit IRC | 02:02 | |
| *** moshele has joined #openstack-nova | 02:09 | |
| *** Sukhdev has quit IRC | 02:10 | |
| *** salv-orl_ has joined #openstack-nova | 02:16 | |
| *** salv-orlando has quit IRC | 02:19 | |
| *** AlexeyAbashkin has joined #openstack-nova | 02:21 | |
| *** vladikr has quit IRC | 02:22 | |
| *** itlinux has quit IRC | 02:22 | |
| *** vladikr has joined #openstack-nova | 02:22 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova-specs master: Fix missing blueprint URLs https://review.openstack.org/542543 | 02:23 |
| *** AlexeyAbashkin has quit IRC | 02:25 | |
| *** yamahata has joined #openstack-nova | 02:26 | |
| *** itlinux has joined #openstack-nova | 02:27 | |
| *** r-daneel has joined #openstack-nova | 02:33 | |
| *** bkopilov has quit IRC | 02:34 | |
| *** tovin07 has quit IRC | 02:35 | |
| *** links has joined #openstack-nova | 02:35 | |
| *** hongbin has joined #openstack-nova | 02:38 | |
| *** itlinux has quit IRC | 02:40 | |
| *** acormier_ has joined #openstack-nova | 02:49 | |
| *** acormier has quit IRC | 02:53 | |
| *** vladikr has quit IRC | 02:56 | |
| *** vladikr has joined #openstack-nova | 02:56 | |
| *** acormier_ has quit IRC | 02:59 | |
| *** acormier has joined #openstack-nova | 02:59 | |
| *** felipemonteiro has joined #openstack-nova | 02:59 | |
| *** acormier has quit IRC | 03:00 | |
| *** acormier has joined #openstack-nova | 03:00 | |
| *** felipemonteiro_ has joined #openstack-nova | 03:00 | |
| *** felipemonteiro has quit IRC | 03:04 | |
| *** harlowja has quit IRC | 03:04 | |
| *** acormier has quit IRC | 03:05 | |
| *** acormier has joined #openstack-nova | 03:07 | |
| *** jobewan has joined #openstack-nova | 03:08 | |
| *** acormier has joined #openstack-nova | 03:08 | |
| *** acormier has quit IRC | 03:16 | |
| *** felipemonteiro_ has quit IRC | 03:18 | |
| *** felipemonteiro__ has joined #openstack-nova | 03:18 | |
| *** takashin has quit IRC | 03:23 | |
| *** felipemonteiro__ has quit IRC | 03:26 | |
| *** moshele has quit IRC | 03:27 | |
| *** yangyapeng has quit IRC | 03:28 | |
| *** yangyapeng has joined #openstack-nova | 03:29 | |
| *** Tom-Tom has joined #openstack-nova | 03:29 | |
| *** suresh12 has joined #openstack-nova | 03:32 | |
| *** yangyapeng has quit IRC | 03:33 | |
| *** jobewan has quit IRC | 03:34 | |
| *** suresh12 has quit IRC | 03:37 | |
| *** brault has quit IRC | 03:39 | |
| *** sapd has quit IRC | 03:42 | |
| *** brault has joined #openstack-nova | 03:45 | |
| *** bkopilov has joined #openstack-nova | 03:49 | |
| *** lbragstad has joined #openstack-nova | 03:52 | |
| *** dave-mccowan has quit IRC | 03:56 | |
| *** udesale has joined #openstack-nova | 03:57 | |
| *** ttsiouts_ has joined #openstack-nova | 03:59 | |
| *** jmlowe has joined #openstack-nova | 04:04 | |
| *** links has quit IRC | 04:05 | |
| *** liuzz has quit IRC | 04:06 | |
| *** suresh12 has joined #openstack-nova | 04:14 | |
| *** vladikr has quit IRC | 04:14 | |
| *** vladikr has joined #openstack-nova | 04:15 | |
| *** abhishekk has joined #openstack-nova | 04:18 | |
| *** vladikr has quit IRC | 04:20 | |
| *** suresh12 has quit IRC | 04:23 | |
| *** links has joined #openstack-nova | 04:23 | |
| *** phuongnh has quit IRC | 04:29 | |
| *** phuongnh has joined #openstack-nova | 04:29 | |
| *** lpetrut has joined #openstack-nova | 04:37 | |
| *** psachin has joined #openstack-nova | 04:37 | |
| *** yamamoto has joined #openstack-nova | 04:44 | |
| *** yikun_jiang has quit IRC | 04:47 | |
| *** yikun_jiang has joined #openstack-nova | 04:48 | |
| openstackgerrit | Nakanishi Tomotaka proposed openstack/nova master: Use Placement API to check resource usage https://review.openstack.org/528953 | 04:50 |
| *** threestrands has quit IRC | 04:50 | |
| *** suresh12 has joined #openstack-nova | 04:58 | |
| *** tetsuro has quit IRC | 05:00 | |
| *** slaweq has joined #openstack-nova | 05:05 | |
| *** lpetrut has quit IRC | 05:05 | |
| *** slaweq has quit IRC | 05:10 | |
| *** lpetrut has joined #openstack-nova | 05:11 | |
| *** Tom-Tom_ has joined #openstack-nova | 05:18 | |
| *** mdnadeem has quit IRC | 05:18 | |
| *** ttsiouts_ has quit IRC | 05:19 | |
| *** takashin has joined #openstack-nova | 05:19 | |
| *** Tom-Tom has quit IRC | 05:21 | |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: add zvm document into https://review.openstack.org/532720 | 05:21 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm admin doc https://review.openstack.org/533125 | 05:21 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm CI information https://review.openstack.org/533512 | 05:21 |
| *** claudiub has joined #openstack-nova | 05:23 | |
| *** phuongnh has quit IRC | 05:24 | |
| *** phuongnh has joined #openstack-nova | 05:25 | |
| *** fragatina has quit IRC | 05:26 | |
| *** lpetrut has quit IRC | 05:29 | |
| *** acormier has joined #openstack-nova | 05:32 | |
| *** acormier has quit IRC | 05:32 | |
| *** acormier has joined #openstack-nova | 05:33 | |
| *** ratailor has joined #openstack-nova | 05:35 | |
| *** hongbin has quit IRC | 05:37 | |
| takashin | 05:41 | |
| *** janki has joined #openstack-nova | 05:43 | |
| *** moshele has joined #openstack-nova | 05:50 | |
| *** harlowja has joined #openstack-nova | 05:52 | |
| *** mdnadeem has joined #openstack-nova | 05:55 | |
| *** itlinux has joined #openstack-nova | 05:59 | |
| *** sridharg has joined #openstack-nova | 05:59 | |
| *** slaweq has joined #openstack-nova | 06:01 | |
| *** kholkina has joined #openstack-nova | 06:02 | |
| *** slaweq has quit IRC | 06:02 | |
| *** kholkina has quit IRC | 06:06 | |
| *** acormier has quit IRC | 06:07 | |
| *** itlinux has quit IRC | 06:08 | |
| *** hieulq has joined #openstack-nova | 06:17 | |
| *** acormier has joined #openstack-nova | 06:19 | |
| *** acormier has quit IRC | 06:22 | |
| *** acormier has joined #openstack-nova | 06:23 | |
| openstackgerrit | Dinesh Bhor proposed openstack/nova-specs master: Add hw:cpu_policy 'mixed' for better CPU utilization https://review.openstack.org/543805 | 06:24 |
| *** fragatina has joined #openstack-nova | 06:27 | |
| *** acormier has quit IRC | 06:33 | |
| *** harlowja has quit IRC | 06:33 | |
| openstackgerrit | Chen Hanxiao proposed openstack/nova master: _cleanup_resize: remove rbd swap disk if resize to flavor without swap https://review.openstack.org/534711 | 06:39 |
| *** yamamoto has quit IRC | 06:41 | |
| *** yamamoto has joined #openstack-nova | 06:43 | |
| *** vivsoni_ has quit IRC | 06:49 | |
| *** yamamoto has quit IRC | 06:50 | |
| *** yamamoto has joined #openstack-nova | 06:55 | |
| *** acormier has joined #openstack-nova | 06:56 | |
| *** kholkina has joined #openstack-nova | 06:58 | |
| openstackgerrit | jichenjc proposed openstack/nova master: List instances performace optimization https://review.openstack.org/507854 | 07:00 |
| *** lbragstad has quit IRC | 07:01 | |
| openstackgerrit | jichenjc proposed openstack/nova master: List instances performace optimization https://review.openstack.org/507854 | 07:03 |
| *** hiro-kobayashi has quit IRC | 07:04 | |
| *** suresh12 has quit IRC | 07:07 | |
| *** sshwarts has joined #openstack-nova | 07:09 | |
| *** lpetrut has joined #openstack-nova | 07:15 | |
| *** suresh12 has joined #openstack-nova | 07:15 | |
| *** jafeha has quit IRC | 07:16 | |
| *** jafeha has joined #openstack-nova | 07:17 | |
| *** hieulq has quit IRC | 07:17 | |
| *** acormier has quit IRC | 07:18 | |
| *** suresh12 has quit IRC | 07:20 | |
| *** Dinesh_Bhor has quit IRC | 07:21 | |
| *** fragatina has quit IRC | 07:25 | |
| *** rcernin has quit IRC | 07:25 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:25 | |
| *** hoangcx has quit IRC | 07:27 | |
| *** takashin has quit IRC | 07:28 | |
| *** takashin has joined #openstack-nova | 07:29 | |
| openstackgerrit | jichenjc proposed openstack/nova master: Avoid live migrate to same host https://review.openstack.org/542689 | 07:29 |
| *** hoonetorg has quit IRC | 07:34 | |
| *** takashin has quit IRC | 07:36 | |
| *** takashin has joined #openstack-nova | 07:36 | |
| *** Dinesh_Bhor has quit IRC | 07:37 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.openstack.org/541561 | 07:38 |
| openstackgerrit | jichenjc proposed openstack/nova master: Fix warn api_class is deprecated, use backend https://review.openstack.org/543830 | 07:41 |
| *** pcaruana has joined #openstack-nova | 07:41 | |
| *** alexchadin has joined #openstack-nova | 07:44 | |
| *** Dinesh_Bhor has quit IRC | 07:45 | |
| *** Guest10778 has quit IRC | 07:48 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:48 | |
| *** Dinesh_Bhor has joined #openstack-nova | 07:49 | |
| *** Guest10778 has joined #openstack-nova | 07:49 | |
| *** slaweq has joined #openstack-nova | 07:49 | |
| *** hoonetorg has joined #openstack-nova | 07:50 | |
| *** moshele has quit IRC | 07:51 | |
| *** moshele has joined #openstack-nova | 07:52 | |
| *** vivsoni has joined #openstack-nova | 07:54 | |
| *** lajoskatona has joined #openstack-nova | 08:00 | |
| *** tiendc has quit IRC | 08:01 | |
| *** tiendc has joined #openstack-nova | 08:02 | |
| *** slaweq_ has joined #openstack-nova | 08:03 | |
| *** Dinesh_Bhor has quit IRC | 08:04 | |
| *** takashin has left #openstack-nova | 08:05 | |
| *** jpena|off is now known as jpena | 08:06 | |
| *** slaweq_ has quit IRC | 08:08 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:08 | |
| *** Dinesh_Bhor has quit IRC | 08:09 | |
| *** ragiman has joined #openstack-nova | 08:12 | |
| *** hieulq has joined #openstack-nova | 08:18 | |
| *** Dinesh_Bhor has joined #openstack-nova | 08:20 | |
| *** ralonsoh has joined #openstack-nova | 08:20 | |
| *** yamahata has quit IRC | 08:23 | |
| *** xinliang has quit IRC | 08:23 | |
| *** BrinZhang has quit IRC | 08:24 | |
| *** tesseract has joined #openstack-nova | 08:26 | |
| *** tssurya has joined #openstack-nova | 08:28 | |
| *** jangutter has joined #openstack-nova | 08:28 | |
| *** amoralej|off is now known as amoralej | 08:29 | |
| *** ccamacho has joined #openstack-nova | 08:30 | |
| *** jangutter_ has joined #openstack-nova | 08:32 | |
| *** yamamoto has quit IRC | 08:35 | |
| *** damien_r has joined #openstack-nova | 08:35 | |
| *** hiro-kobayashi has joined #openstack-nova | 08:35 | |
| *** jangutter has quit IRC | 08:36 | |
| *** yamamoto has joined #openstack-nova | 08:39 | |
| *** moshele has quit IRC | 08:49 | |
| *** ttsiouts has quit IRC | 08:54 | |
| *** ttsiouts has joined #openstack-nova | 08:54 | |
| *** slaweq_ has joined #openstack-nova | 08:55 | |
| *** cdent has joined #openstack-nova | 08:55 | |
| bauzas | good morning Nova | 08:56 |
| cdent | o/ | 08:56 |
| *** priteau has joined #openstack-nova | 08:59 | |
| *** slaweq_ has quit IRC | 08:59 | |
| *** blkart has quit IRC | 09:03 | |
| *** blkart has joined #openstack-nova | 09:05 | |
| *** tetsuro has joined #openstack-nova | 09:07 | |
| stephenfin | o/ | 09:09 |
| gibi | morning | 09:09 |
| tetsuro | o/ | 09:09 |
| *** dtantsur|afk is now known as dtantsur | 09:09 | |
| hrw | morning | 09:12 |
| tssurya | o/ | 09:13 |
| gibi | stephenfin: what do you think, how can we push https://review.openstack.org/#/c/539605/ forward? | 09:14 |
| *** Tom-Tom_ has quit IRC | 09:15 | |
| *** Tom-Tom has joined #openstack-nova | 09:15 | |
| stephenfin | gibi: I'll take another look shortly. Last I checked, I had an open question on whether we wanted to do this on a port-by-port basis, instance-by-instance basis, or host-level basis | 09:15 |
| stephenfin | I'd favour the less granular approaches if suitable as they're going to less complex/won't add more darn extra specs :) | 09:16 |
| gibi | stephenfin: yeah, I tried to answer that question. I can accept both end of the spectrum i.e. host level or port level | 09:16 |
| gibi | stephenfin: I can accept that we don't want to add yet another flavor extra_spec | 09:17 |
| stephenfin | So I see. Let me reply to all the above and state my concerns there | 09:17 |
| stephenfin | gibi: Yeah, exactly when it's so specific to a given driver and use case | 09:17 |
| gibi | stephenfin: my internal requirements can be fulfilled with host level config + host aggregate | 09:17 |
| stephenfin | gibi: Ditto | 09:18 |
| gibi | stephenfin: cool, then we are on the same side | 09:18 |
| gibi | :) | 09:18 |
| gibi | it is a bumer that libvirt is revertin the support of the host level config | 09:19 |
| stephenfin | It's wonderful to have other cores looking at telco-related stuff again :) | 09:19 |
| *** acormier has joined #openstack-nova | 09:19 | |
| stephenfin | It's probably good for jaypipes' mental health | 09:19 |
| stephenfin | Indeed. They're very much against configuring any defaults at all in libvirt | 09:19 |
| *** Tom-Tom has quit IRC | 09:20 | |
| stephenfin | besides legacy stuff (as hrw discovered with the USB tablet under ARM issue) | 09:20 |
| gibi | I totally for jaypipes' mental well being | 09:22 |
| *** acormier has quit IRC | 09:23 | |
| gibi | stephenfin: if you summarize your view on the spec then I can second that based on this short chat and then we can call in jaypipes for a review | 09:23 |
| ameeda | Hi, just a question about gerrit review, I have 3 nova reviews waiting for reviewers (both of them got +2c, waiting for +2w) , do you follow scheduler to check them or I have to ask you to check them ? | 09:25 |
| openstackgerrit | Rong Han proposed openstack/nova master: Clean up ports and volumes when deleting ERROR instance https://review.openstack.org/340614 | 09:28 |
| *** slaweq_ has joined #openstack-nova | 09:36 | |
| *** Dinesh_Bhor has quit IRC | 09:37 | |
| *** slaweq_ has quit IRC | 09:41 | |
| stephenfin | gibi: ALL the comments left :) | 09:46 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: add zvm document into https://review.openstack.org/532720 | 09:46 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm admin doc https://review.openstack.org/533125 | 09:46 |
| openstackgerrit | jichenjc proposed openstack/nova master: WIP: Add zvm CI information https://review.openstack.org/533512 | 09:46 |
| *** dtantsur is now known as dtantsur|brb | 09:47 | |
| hrw | stephenfin: I will get some next stuff around libvirt soon ;D | 09:48 |
| gibi | stephenfin: thanks | 09:49 |
| *** derekh has joined #openstack-nova | 09:54 | |
| *** abhishekk has quit IRC | 09:54 | |
| *** hiro-kobayashi has quit IRC | 09:57 | |
| *** yamamoto has quit IRC | 09:58 | |
| *** moshele has joined #openstack-nova | 10:00 | |
| *** hieulq has quit IRC | 10:02 | |
| *** jichen has quit IRC | 10:08 | |
| *** tiendc has quit IRC | 10:10 | |
| *** elmaciej has joined #openstack-nova | 10:10 | |
| *** elmaciej_ has joined #openstack-nova | 10:14 | |
| *** phuongnh has quit IRC | 10:14 | |
| *** yamamoto has joined #openstack-nova | 10:15 | |
| openstackgerrit | Lee Yarwood proposed openstack/nova master: DNM: Test LM with encrypted volumes https://review.openstack.org/536350 | 10:16 |
| *** priteau has quit IRC | 10:17 | |
| *** elmaciej has quit IRC | 10:17 | |
| *** yamamoto has quit IRC | 10:22 | |
| *** ralonsoh has quit IRC | 10:23 | |
| *** ralonsoh has joined #openstack-nova | 10:26 | |
| *** alexchadin has quit IRC | 10:28 | |
| *** alexchadin has joined #openstack-nova | 10:29 | |
| *** ralonsoh_ has joined #openstack-nova | 10:32 | |
| *** alexchadin has quit IRC | 10:32 | |
| *** ralonsoh has quit IRC | 10:35 | |
| *** links has quit IRC | 10:36 | |
| *** alexchadin has joined #openstack-nova | 10:38 | |
| gibi | stephenfin: I think we are on the same page about https://review.openstack.org/#/c/539605 | 10:40 |
| gibi | stephenfin: thanks for the reply, I seconded it | 10:40 |
| *** alexchadin has quit IRC | 10:41 | |
| bauzas | I haven't started to review specs yet :) | 10:45 |
| bauzas | for the moment, bug triaging, my lobe | 10:45 |
| bauzas | love | 10:45 |
| *** links has joined #openstack-nova | 10:49 | |
| bauzas | stephenfin: sahid: not sure it's a bug, needing more details but still FYI https://bugs.launchpad.net/nova/+bug/1749045 | 10:50 |
| openstack | Launchpad bug 1749045 in OpenStack Compute (nova) "Used mem in numa_topology do not include mem used by instance which is not fix mem_page_size" [Undecided,Incomplete] | 10:50 |
| stephenfin | bauzas: Yup, that's a known issue. There's an issue in RH Bugzilla about it alright. sahid was working on it | 10:50 |
| bauzas | stephenfin: okay, could you please mark it as Confirmed then, unless it's a duplicate ? | 10:51 |
| stephenfin | bauzas: Yup, #1689753 looks pretty much the same | 10:52 |
| * bauzas looking | 10:53 | |
| stephenfin | That's something that (hopefully) placement will solve for us | 10:53 |
| bauzas | stephenfin: ok, marking 1749045 as duplicate of 1689753 | 10:53 |
| stephenfin | bauzas: Already done :P | 10:53 |
| bauzas | cool | 10:53 |
| bauzas | stephenfin: known issue too here https://bugs.launchpad.net/nova/+bug/1748799 | 10:54 |
| openstack | Launchpad bug 1748799 in OpenStack Compute (nova) "cpu binding conflict when multiple VMs execute unshelve at the same time" [Undecided,New] - Assigned to tangxing (tang-xing) | 10:54 |
| stephenfin | bauzas: Yup. That's the "we don't recalculate CPU pinning" issue | 10:54 |
| stephenfin | That's on the agenda for the PTG | 10:55 |
| bauzas | stephenfin: I know, any upstream bug I could say it's duplicate ? | 10:56 |
| stephenfin | bauzas: 1417667 ? | 10:56 |
| * bauzas looks | 10:56 | |
| stephenfin | Maybe update the title if shelve is also affected | 10:57 |
| *** owalsh_ has joined #openstack-nova | 10:57 | |
| *** ralonsoh__ has joined #openstack-nova | 10:57 | |
| *** ralonsoh_ has quit IRC | 10:58 | |
| bauzas | stephenfin: yep, that's actually a problem for *any* move ops | 10:59 |
| *** owalsh has quit IRC | 10:59 | |
| *** alexchadin has joined #openstack-nova | 11:00 | |
| *** owalsh has joined #openstack-nova | 11:01 | |
| *** alexchadin has quit IRC | 11:01 | |
| *** owalsh_ has quit IRC | 11:03 | |
| *** yamamoto has joined #openstack-nova | 11:13 | |
| *** owalsh_ has joined #openstack-nova | 11:14 | |
| *** owalsh has quit IRC | 11:18 | |
| *** bkopilov has quit IRC | 11:19 | |
| *** owalsh_ is now known as owalsh | 11:20 | |
| *** derekh has quit IRC | 11:23 | |
| *** derekh has joined #openstack-nova | 11:23 | |
| openstackgerrit | Stephen Finucane proposed openstack/nova-specs master: Add 'numa-aware-vswitches' spec https://review.openstack.org/541290 | 11:32 |
| *** yikun_jiang has quit IRC | 11:35 | |
| *** yikun_jiang has joined #openstack-nova | 11:35 | |
| *** alexchadin has joined #openstack-nova | 11:36 | |
| stephenfin | sean-k-mooney: When you're about, could you have a look at that to make sure the "problem description" section actually makes sense now? Still trying to configure a machine to evaluate this myself https://review.openstack.org/#/c/541290/2/specs/rocky/approved/numa-aware-vswitches.rst | 11:37 |
| *** ralonsoh__ is now known as ralonsoh | 11:43 | |
| *** tomhambleton_ has quit IRC | 11:43 | |
| *** tomhambleton_ has joined #openstack-nova | 11:43 | |
| *** sree has joined #openstack-nova | 11:47 | |
| *** tbachman has quit IRC | 11:49 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova-specs master: Enable NUMA Features for Libvirt/QEMU Driver https://review.openstack.org/533077 | 11:49 |
| openstackgerrit | Merged openstack/nova master: Move the nova-next job in-tree and update it https://review.openstack.org/541474 | 11:50 |
| *** mwhahaha has quit IRC | 11:51 | |
| *** mwhahaha has joined #openstack-nova | 11:52 | |
| *** acormier has joined #openstack-nova | 11:56 | |
| *** acormier has quit IRC | 11:57 | |
| *** acormier has joined #openstack-nova | 11:57 | |
| *** pooja_jadhav has quit IRC | 11:58 | |
| *** pooja_jadhav has joined #openstack-nova | 11:59 | |
| *** bhagyashri_s has joined #openstack-nova | 11:59 | |
| *** tetsuro has left #openstack-nova | 12:00 | |
| *** niraj_singh has joined #openstack-nova | 12:00 | |
| *** acormier has quit IRC | 12:01 | |
| *** bhagyashris has quit IRC | 12:02 | |
| *** nsingh has quit IRC | 12:02 | |
| *** sdague has joined #openstack-nova | 12:03 | |
| *** acormier has joined #openstack-nova | 12:03 | |
| *** acormier has quit IRC | 12:04 | |
| *** acormier has joined #openstack-nova | 12:05 | |
| *** alexchadin has quit IRC | 12:06 | |
| *** alexchadin has joined #openstack-nova | 12:06 | |
| *** jbryce has quit IRC | 12:11 | |
| *** jbryce has joined #openstack-nova | 12:11 | |
| *** bhujay has joined #openstack-nova | 12:19 | |
| *** sree has quit IRC | 12:21 | |
| *** dtantsur|brb is now known as dtantsur | 12:22 | |
| *** ratailor has quit IRC | 12:25 | |
| *** acormier has quit IRC | 12:26 | |
| *** acormier has joined #openstack-nova | 12:26 | |
| *** Tom-Tom has joined #openstack-nova | 12:28 | |
| *** acormier has quit IRC | 12:31 | |
| *** liusheng has quit IRC | 12:42 | |
| *** liusheng has joined #openstack-nova | 12:42 | |
| *** Tom-Tom has quit IRC | 12:45 | |
| *** bkopilov has joined #openstack-nova | 12:46 | |
| *** jpena is now known as jpena|lunch | 12:48 | |
| *** sree_ has joined #openstack-nova | 12:55 | |
| *** edmondsw has joined #openstack-nova | 12:56 | |
| *** sree_ is now known as Guest38218 | 12:56 | |
| *** sambetts|afk is now known as sambetts | 12:57 | |
| *** slaweq_ has joined #openstack-nova | 13:01 | |
| *** Guest38218 has quit IRC | 13:03 | |
| *** slaweq_ has quit IRC | 13:05 | |
| *** pchavva has joined #openstack-nova | 13:14 | |
| *** eharney has quit IRC | 13:14 | |
| *** jaypipes has joined #openstack-nova | 13:15 | |
| *** pcaruana has quit IRC | 13:16 | |
| *** jobewan has joined #openstack-nova | 13:17 | |
| *** lbragstad has joined #openstack-nova | 13:21 | |
| *** tbachman has joined #openstack-nova | 13:22 | |
| *** sree has joined #openstack-nova | 13:24 | |
| *** Tom-Tom has joined #openstack-nova | 13:24 | |
| *** READ10 has joined #openstack-nova | 13:24 | |
| *** Tom-Tom has quit IRC | 13:26 | |
| *** sree has quit IRC | 13:29 | |
| *** amoralej is now known as amoralej|lunch | 13:32 | |
| *** janki has quit IRC | 13:32 | |
| *** oanson has quit IRC | 13:33 | |
| *** oanson has joined #openstack-nova | 13:35 | |
| *** dave-mccowan has joined #openstack-nova | 13:36 | |
| *** liverpooler has joined #openstack-nova | 13:37 | |
| *** yamamoto has quit IRC | 13:41 | |
| *** vladikr has joined #openstack-nova | 13:44 | |
| *** alexchadin has quit IRC | 13:45 | |
| *** slaweq_ has joined #openstack-nova | 13:45 | |
| *** mriedem has joined #openstack-nova | 13:46 | |
| *** Eran_Kuris has quit IRC | 13:46 | |
| *** alexchadin has joined #openstack-nova | 13:46 | |
| *** jpena|lunch is now known as jpena | 13:48 | |
| *** fragatina has joined #openstack-nova | 13:49 | |
| *** slaweq_ has quit IRC | 13:50 | |
| *** sree has joined #openstack-nova | 13:52 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Test websocketproxy with TLS in the nova-next job https://review.openstack.org/513160 | 13:54 |
| *** jobewan has quit IRC | 13:55 | |
| *** zul has quit IRC | 13:56 | |
| *** sree has quit IRC | 13:57 | |
| *** zul has joined #openstack-nova | 13:57 | |
| *** esberglu has joined #openstack-nova | 13:58 | |
| esberglu | #startmeeting powervm_driver_meeting | 14:00 |
| openstack | Meeting started Tue Feb 13 14:00:21 2018 UTC and is due to finish in 60 minutes. The chair is esberglu. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: powervm_driver_meeting)" | 14:00 | |
| openstack | The meeting name has been set to 'powervm_driver_meeting' | 14:00 |
| esberglu | #topic In-tree Driver | 14:00 |
| *** openstack changes topic to "In-tree Driver (Meeting topic: powervm_driver_meeting)" | 14:00 | |
| *** mlavalle has joined #openstack-nova | 14:00 | |
| *** ralonsoh_ has joined #openstack-nova | 14:01 | |
| esberglu | #endmeeting | 14:01 |
| *** openstack changes topic to "This channel is for Nova development. For support of Nova deployments, please use #openstack. Please see: https://wiki.openstack.org/wiki/Nova/Queens_Release_Schedule" | 14:01 | |
| openstack | Meeting ended Tue Feb 13 14:01:15 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:01 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-02-13-14.00.html | 14:01 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-02-13-14.00.txt | 14:01 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/powervm_driver_meeting/2018/powervm_driver_meeting.2018-02-13-14.00.log.html | 14:01 |
| esberglu | Sorry thought I was in a different channel | 14:01 |
| *** eharney has joined #openstack-nova | 14:01 | |
| *** alexchadin has quit IRC | 14:02 | |
| *** Eran_Kuris has joined #openstack-nova | 14:02 | |
| *** liverpooler has quit IRC | 14:04 | |
| *** ralonsoh has quit IRC | 14:04 | |
| *** zhaochao has quit IRC | 14:09 | |
| *** abhishekk has joined #openstack-nova | 14:13 | |
| bauzas | mriedem: some possible race when trying to delete a BuildRequest instance https://bugs.launchpad.net/nova/+bug/1749167 | 14:19 |
| openstack | Launchpad bug 1749167 in OpenStack Compute (nova) "nova show can not get an instance information, and this instance can be queried from nova list." [Undecided,New] | 14:19 |
| bauzas | dansmith: when you're up, see above $ | 14:19 |
| bauzas | what I'm confused is that I can see a "too many connections'" SQL exception | 14:19 |
| *** yamahata has joined #openstack-nova | 14:20 | |
| openstackgerrit | Jiri Suchomel proposed openstack/nova master: Retry few times when creating the initial connection to libvirtd. https://review.openstack.org/416928 | 14:26 |
| *** yamamoto has joined #openstack-nova | 14:27 | |
| *** lajoskatona has quit IRC | 14:29 | |
| *** fragatina has quit IRC | 14:31 | |
| *** ralonsoh__ has joined #openstack-nova | 14:33 | |
| *** david-lyle has quit IRC | 14:33 | |
| *** openstackgerrit has quit IRC | 14:33 | |
| dansmith | bauzas: why is that confusing? I think it means there are too many connections :) | 14:34 |
| dansmith | I can also imagine that we hit that while trying to do the BR->RS->IM dance and thus it's in the cell and not mapped | 14:34 |
| bauzas | dansmith: yeah, I actually provided a comment | 14:35 |
| dansmith | I saw :) | 14:35 |
| bauzas | it impacted the scheduler call | 14:35 |
| dansmith | and probably other things | 14:35 |
| bauzas | so, yeah, let's ask to reproduce and if not, close it | 14:35 |
| bauzas | dansmith: unrelated, other evacuate thing https://bugs.launchpad.net/nova/+bug/1745977 | 14:36 |
| openstack | Launchpad bug 1745977 in OpenStack Compute (nova) "When source compute service up, will not destroy and clean up those instances which be evacuated then be deleted." [Undecided,New] | 14:36 |
| *** ralonsoh_ has quit IRC | 14:36 | |
| *** sree has joined #openstack-nova | 14:37 | |
| *** tbachman has quit IRC | 14:37 | |
| dansmith | okay | 14:38 |
| *** abhishekk has quit IRC | 14:39 | |
| *** r-daneel has quit IRC | 14:40 | |
| *** Tom-Tom has joined #openstack-nova | 14:41 | |
| *** Tom-Tom has quit IRC | 14:41 | |
| *** sree has quit IRC | 14:42 | |
| *** Tom-Tom has joined #openstack-nova | 14:42 | |
| *** sree has joined #openstack-nova | 14:43 | |
| *** acormier has joined #openstack-nova | 14:44 | |
| *** liverpooler has joined #openstack-nova | 14:45 | |
| *** slaweq_ has joined #openstack-nova | 14:47 | |
| *** links has quit IRC | 14:47 | |
| *** amoralej|lunch is now known as amoralej | 14:48 | |
| *** sree has quit IRC | 14:49 | |
| *** acormier_ has joined #openstack-nova | 14:49 | |
| *** acormier has quit IRC | 14:49 | |
| *** jobewan has joined #openstack-nova | 14:50 | |
| *** awaugama has joined #openstack-nova | 14:51 | |
| *** slaweq_ has quit IRC | 14:52 | |
| *** Shilpa has quit IRC | 14:53 | |
| *** mgoddard_ has joined #openstack-nova | 14:53 | |
| mriedem | lyarwood: want to get this pike backport? https://review.openstack.org/#/c/537811/ | 14:57 |
| mriedem | i'm going to push a pike release today | 14:57 |
| *** ying_zuo has quit IRC | 14:58 | |
| *** moshele has quit IRC | 14:58 | |
| mriedem | dansmith: can you get this one? https://review.openstack.org/#/c/541036/ | 14:58 |
| *** ying_zuo has joined #openstack-nova | 14:58 | |
| lyarwood | mriedem: ack looking | 14:58 |
| *** sapcc-bot2 has joined #openstack-nova | 14:59 | |
| *** sapcc-bot has quit IRC | 14:59 | |
| mriedem | i can't remember if you guys needed this for product either https://review.openstack.org/#/c/539003/ | 14:59 |
| *** sree has joined #openstack-nova | 14:59 | |
| dansmith | erm, not sure I've looked at that one in depth, we should ask her when she's around | 15:00 |
| *** Tom-Tom has quit IRC | 15:02 | |
| mriedem | lyarwood: i'm not sure about this one https://review.openstack.org/#/c/533733/ - i suppose it's ok since it fails fast in the api rather than silently on the compute, | 15:02 |
| *** sshwarts has quit IRC | 15:02 | |
| mriedem | the problem with swap volume is that i don't think anyone uses it directly, | 15:02 |
| mriedem | it happens as a result of a cinder volume retype or migration | 15:03 |
| lyarwood | mriedem: right but wouldn't this result in a cleaner failure in that case as well? | 15:03 |
| mriedem | so really, the doc updates should be in the cinder api-ref for retype saying that you can't retype an attached volume where the server instance is in one of those states | 15:03 |
| *** Tom-Tom_ has joined #openstack-nova | 15:04 | |
| mriedem | i don't know if/how cinder handles a 409 from swap volume, and if it rolls back the retype | 15:04 |
| lyarwood | mriedem: that was an assumption on my part, we'd actually return a 409 to cinder now at least. | 15:04 |
| *** sree has quit IRC | 15:04 | |
| mriedem | it is better to get a 409 though so cinder at least has a chance to do some kind of rollback | 15:04 |
| *** Tom-Tom__ has joined #openstack-nova | 15:05 | |
| mriedem | looks like cinder will delete the new volume (for a retype), and then set the old volume.migration_status='error' | 15:06 |
| *** tidwellr has joined #openstack-nova | 15:06 | |
| mriedem | and set the old volume status back to in-use | 15:06 |
| mriedem | (i think) | 15:06 |
| *** Tom-Tom_ has quit IRC | 15:08 | |
| *** Tom-Tom__ has quit IRC | 15:09 | |
| mriedem | lyarwood: ok +2 on that one; want to get https://review.openstack.org/#/c/541036/ and then i'll cut the release? | 15:10 |
| lyarwood | mriedem: yup will do, thanks | 15:11 |
| *** bhujay has quit IRC | 15:13 | |
| *** tbachman has joined #openstack-nova | 15:15 | |
| *** damien_r has quit IRC | 15:15 | |
| *** burt has joined #openstack-nova | 15:16 | |
| *** Eran_Kuris has quit IRC | 15:20 | |
| *** rmart04 has joined #openstack-nova | 15:21 | |
| mriedem | lyarwood: this is re-introducing a regression from queens https://review.openstack.org/#/c/543569/ | 15:23 |
| mriedem | i'm pretty sure anyway | 15:23 |
| *** Tom-Tom has joined #openstack-nova | 15:23 | |
| lyarwood | mriedem: How so? We also block swap_volume for encrypted volumes when using native LUKS in >=Queens | 15:24 |
| mriedem | lyarwood: https://github.com/openstack/nova/commit/cafe3d066ef7021c18961d4b239a10f61db23f2d#diff-f4019782d93a196a0d026479e6aa61b1 | 15:24 |
| mriedem | you're not backporting that regression fix, introduced by the native luks support patch | 15:24 |
| *** lpetrut has quit IRC | 15:24 | |
| mriedem | that broke several cinder backends | 15:24 |
| *** eharney has quit IRC | 15:25 | |
| *** openstackgerrit has joined #openstack-nova | 15:25 | |
| openstackgerrit | Boris Bobrov proposed openstack/python-novaclient stable/queens: Fix listing of instances above API max_limit https://review.openstack.org/543968 | 15:25 |
| lyarwood | mriedem: crap, yeah my bad | 15:26 |
| * lyarwood respins | 15:26 | |
| *** liverpooler has quit IRC | 15:26 | |
| *** r-daneel has joined #openstack-nova | 15:27 | |
| *** eharney has joined #openstack-nova | 15:28 | |
| *** eharney has quit IRC | 15:28 | |
| *** eharney has joined #openstack-nova | 15:28 | |
| *** jcosmao has joined #openstack-nova | 15:30 | |
| *** damien_r has joined #openstack-nova | 15:31 | |
| mriedem | stephenfin: you would probably enjoy the honors here https://review.openstack.org/#/c/513160/ | 15:31 |
| mriedem | nova-next job + tls proxy enabled | 15:31 |
| stephenfin | mriedem: Ooh, snazzy | 15:32 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Lazy-load instance attributes with read_deleted=yes https://review.openstack.org/543970 | 15:32 |
| mriedem | i think once we drop the legacy nova-next job that's defined in openstack-zuul-jobs, we should look at making nova-next (defined in tree) voting and gating | 15:33 |
| mriedem | and we should probably also make it run with py3 | 15:33 |
| openstackgerrit | Claudiu Belu proposed openstack/nova master: compute: Makes the resize_instance RPC call synchronous https://review.openstack.org/543971 | 15:33 |
| cdent | \o/ | 15:33 |
| mriedem | claudiub: re your allocation + resize delete thing, did you report a bug for that? | 15:33 |
| mriedem | if not, can you? | 15:33 |
| claudiub | I did. I just sent a patch for it as well | 15:34 |
| claudiub | ^ | 15:34 |
| claudiub | basically, the Except branch in ComputeManager's prep_resize is never executed. | 15:34 |
| mriedem | claudiub: i don't think that's the kind of fix we want | 15:34 |
| *** damien_r has quit IRC | 15:34 | |
| claudiub | in which the allocations gets cleaned up | 15:35 |
| mriedem | we likely need to handle the allocation cleanup in resize_instance | 15:35 |
| stephenfin | mriedem: Both the commit message and comment in 'nova-next/run.yaml' say the 'tls-proxy' service is enabled when it isn't. If you want to drop that, I can +W | 15:35 |
| stephenfin | Or vice versa. It's confusing otherwise :) | 15:35 |
| claudiub | that could work too | 15:36 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Test websocketproxy with TLS in the nova-next job https://review.openstack.org/513160 | 15:36 |
| mriedem | stephenfin: done | 15:37 |
| claudiub | we might have to do the rescheduling there too, then. | 15:37 |
| stephenfin | mriedem: likewise | 15:37 |
| mriedem | claudiub: if resize_instance fails, we're dead | 15:37 |
| *** slaweq_ has joined #openstack-nova | 15:38 | |
| claudiub | what do you mean? | 15:38 |
| claudiub | you mean that the resize operation should fail, and be put in error state? | 15:38 |
| mriedem | i don't think we need to try and make every part of a resize reschedulable | 15:38 |
| mriedem | yes | 15:38 |
| *** kholkina has quit IRC | 15:39 | |
| mriedem | changing the rpc cast to call might fix this issue, but likely introduces an rpc messaging timeout problem | 15:40 |
| mriedem | if resize_instance takes >1 minute | 15:40 |
| claudiub | well, not necessarely always. for example, if an InstanceFaultRollback is raised by the driver (at least in our scenario - if the new flavor's disk is smaller than the original), then the resize didn't happen, and the instance will be put back into Running state | 15:40 |
| claudiub | hm, I see. | 15:40 |
| mriedem | i see that the libvirt driver's migrate_disk_and_power_off method, called from resize_instance, is doing some checks like if you're trying to resize the disk down or if you're using lvm, it fails, | 15:42 |
| jaypipes | dims: https://twitter.com/jaypipes/status/963437624852668419 | 15:42 |
| *** slaweq_ has quit IRC | 15:42 | |
| mriedem | in that case we'd fail and not reschedule | 15:42 |
| mriedem | but in those cases, we don't want to reschedule | 15:43 |
| claudiub | fair enough | 15:43 |
| dims | jaypipes, y i was reading all about it yesterday ... :( | 15:43 |
| claudiub | well then, I'll send another PS. | 15:43 |
| mriedem | claudiub: you can maybe build a functional test similar to test_resize_to_same_host_prep_resize_fails | 15:48 |
| mriedem | where resize_instance fails and we make sure to cleanup allocations created by the scheduler for the destination host | 15:48 |
| claudiub | will do | 15:49 |
| *** liverpooler has joined #openstack-nova | 15:50 | |
| *** rmart04 has quit IRC | 15:50 | |
| *** Tom-Tom_ has joined #openstack-nova | 15:50 | |
| *** Tom-Tom has quit IRC | 15:53 | |
| *** david-lyle has joined #openstack-nova | 15:54 | |
| mriedem | dansmith: gibi: claudiub's bug got me thinking about how we might leak migration allocations during a failed cold migrate https://bugs.launchpad.net/nova/+bug/1749215/comments/2 | 15:59 |
| openstack | Launchpad bug 1749215 in OpenStack Compute (nova) "Allocations not deleted on failed resize_instance" [Medium,In progress] - Assigned to Claudiu Belu (cbelu) | 15:59 |
| mriedem | normally deleting an instance removes it's allocations from placement and we're happy, that's the last resort kind of cleanup thing, | 15:59 |
| mriedem | but we don't cleanup any allocations tracked against migration records for the instance when we delete the instance, | 16:00 |
| mriedem | granted if we rollback at the point of failure correctly, we shouldn't need to | 16:00 |
| *** burt has quit IRC | 16:00 | |
| *** burt has joined #openstack-nova | 16:01 | |
| *** itlinux has joined #openstack-nova | 16:01 | |
| dansmith | mriedem: well, the bug would be not in the instance delete path, but in the path where we left a migration allocation in place from the failed resize yeah? | 16:03 |
| mriedem | yeah that's what i mean by "if we rollback at the point of failure correctly, we shouldn't need to" | 16:03 |
| mriedem | looking at all of the resize flows, and where errors_out_migration_ctxt is used, it could be a lot of places | 16:03 |
| mriedem | which is what kind of worries me | 16:04 |
| *** openstackgerrit has quit IRC | 16:04 | |
| mriedem | anyway, it would likely break my brain to try and audit all of those places right now | 16:04 |
| *** tidwellr has quit IRC | 16:05 | |
| *** damien_r has joined #openstack-nova | 16:05 | |
| *** tidwellr has joined #openstack-nova | 16:06 | |
| *** felipemonteiro has joined #openstack-nova | 16:09 | |
| *** felipemonteiro_ has joined #openstack-nova | 16:10 | |
| *** moshele has joined #openstack-nova | 16:11 | |
| *** Tom-Tom_ has quit IRC | 16:11 | |
| *** felipemonteiro has quit IRC | 16:15 | |
| *** suresh12 has joined #openstack-nova | 16:17 | |
| *** tssurya has quit IRC | 16:18 | |
| mnaser | melwitt: dansmith https://review.openstack.org/#/c/539005/ thanks in advance, we just ran into this right now | 16:18 |
| cdent | mriedem: have you tried putting the contents of verify_placement.sh in the parent file in a ( subshell ) and then checking the exit code of the whole thing? | 16:18 |
| cdent | I'm not tracking what you've already tried, just saw the recent "external file not gonna work" report | 16:19 |
| *** mgoddard_ has quit IRC | 16:22 | |
| mriedem | cdent: nope | 16:23 |
| *** suresh12 has quit IRC | 16:23 | |
| mriedem | i think the recent failure is due to the legacy nova-next job running from a symlink in nova/tools/hooks | 16:23 |
| *** slaweq_ has joined #openstack-nova | 16:24 | |
| *** udesale has quit IRC | 16:27 | |
| *** slaweq_ has quit IRC | 16:29 | |
| *** sambetts is now known as sambetts|afk | 16:29 | |
| *** Sukhdev has joined #openstack-nova | 16:30 | |
| *** openstackgerrit has joined #openstack-nova | 16:33 | |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Add check for redundant import aliases https://review.openstack.org/543995 | 16:33 |
| mriedem | i wonder why the nova admin guide docs on schedulers gives a reference to the volume scheduler in cinder https://docs.openstack.org/nova/latest/admin/configuration/schedulers.html | 16:36 |
| *** psachin has quit IRC | 16:36 | |
| mriedem | the link is broken, but i don't know if i should fix it or just remove that part | 16:36 |
| *** imacdonn has quit IRC | 16:37 | |
| gibi | mriedem: does this trace mean that nova does not handle the exception? https://bugs.launchpad.net/nova/+bug/1749215/comments/3 | 16:37 |
| openstack | Launchpad bug 1749215 in OpenStack Compute (nova) "Allocations not deleted on failed resize_instance" [Medium,In progress] - Assigned to Claudiu Belu (cbelu) | 16:37 |
| *** imacdonn has joined #openstack-nova | 16:37 | |
| mriedem | gibi: depends on what you mean by handle | 16:37 |
| mriedem | that's in the _error_out_instance_on_exception context manager, | 16:38 |
| mriedem | which is setting the instance to ERROR due to an unexpected exception | 16:39 |
| mriedem | that's likely a bug, the instance shouldn't be in ERROR | 16:39 |
| mriedem | for "Resize error: Unable to resize disk down." | 16:39 |
| gibi | mriedem: in my log this is the last place nova mentions the ResizeError | 16:39 |
| openstackgerrit | Merged openstack/nova stable/pike: Make eventlet hub use a monotonic clock https://review.openstack.org/537811 | 16:39 |
| mriedem | gibi: sure, that's because resize_instance is running on the source host, via rpc cast from prep_resize on the dest host | 16:40 |
| gibi | mriedem: I in that context manager this is coming from we does not delete claims | 16:40 |
| mriedem | so if that fails, we should cleanup and exit | 16:40 |
| mriedem | the claim is made in prep_resize on the dest | 16:40 |
| mriedem | before we rpc cast to resize_instance | 16:41 |
| gibi | mriedem: ahh that was the missing piece | 16:41 |
| gibi | mriedem: thanks, now I have a place to look at | 16:41 |
| mriedem | what i'm not sure about is if/when the dest host RT updates its usage after the migration fails to remove the claim | 16:42 |
| mriedem | # Grab all in-progress migrations: | 16:43 |
| mriedem | migrations = objects.MigrationList.get_in_progress_by_host_and_node( | 16:43 |
| mriedem | context, self.host, nodename) | 16:43 |
| mriedem | self._pair_instances_to_migrations(migrations, instances) | 16:43 |
| mriedem | self._update_usage_from_migrations(context, migrations, nodename) | 16:43 |
| mriedem | that should update the dest host RT at some point | 16:43 |
| mriedem | once the migration fails, it will no longer be in that in-progress list | 16:44 |
| *** moshele has quit IRC | 16:44 | |
| *** itlinux has quit IRC | 16:46 | |
| gibi | mriedem: in my env I don't see that this Exception handler is called during the failed resize: https://github.com/openstack/nova/blob/fa6c0f9cb14f1b4ce4d9b1dbacb1743173089986/nova/compute/manager.py#L4124 | 16:46 |
| mriedem | gibi: it's not, | 16:47 |
| mriedem | because we've already rpc cast to the source host | 16:47 |
| mriedem | https://github.com/openstack/nova/blob/fa6c0f9cb14f1b4ce4d9b1dbacb1743173089986/nova/compute/manager.py#L4085 | 16:47 |
| *** pcaruana has joined #openstack-nova | 16:47 | |
| mriedem | that's why claudiub's original fix was to change that to an rpc call | 16:47 |
| mriedem | so the failure would come back to prep_resize on the dest and we'd run that cleanup code | 16:47 |
| mriedem | but rpc calls just mean messaging timeouts | 16:48 |
| mriedem | so i'd rather not do that | 16:48 |
| openstackgerrit | Merged openstack/python-novaclient stable/queens: Updated from global requirements https://review.openstack.org/543741 | 16:48 |
| gibi | mriedem: OK, then I understand what I see. | 16:48 |
| *** suresh12 has joined #openstack-nova | 16:48 | |
| gibi | mriedem: thanks again | 16:48 |
| gibi | mriedem: I have to run now but if the problem still open then I can look at it again tomorrow | 16:48 |
| *** suresh12_ has joined #openstack-nova | 16:49 | |
| *** mdnadeem has quit IRC | 16:49 | |
| *** eharney has quit IRC | 16:49 | |
| *** suresh12 has quit IRC | 16:49 | |
| *** suresh1__ has joined #openstack-nova | 16:49 | |
| *** suresh12_ has quit IRC | 16:50 | |
| *** suresh12 has joined #openstack-nova | 16:50 | |
| *** itlinux has joined #openstack-nova | 16:50 | |
| *** suresh12 has quit IRC | 16:50 | |
| *** suresh12 has joined #openstack-nova | 16:51 | |
| *** lpetrut has joined #openstack-nova | 16:53 | |
| *** chyka has joined #openstack-nova | 16:53 | |
| *** suresh1__ has quit IRC | 16:54 | |
| *** yasemin has quit IRC | 17:01 | |
| *** yasemin has joined #openstack-nova | 17:02 | |
| *** slaweq has quit IRC | 17:02 | |
| *** Sukhdev has quit IRC | 17:02 | |
| *** slaweq has joined #openstack-nova | 17:03 | |
| *** moshele has joined #openstack-nova | 17:03 | |
| *** hegemoOn has joined #openstack-nova | 17:04 | |
| hegemoOn | hello there | 17:04 |
| hegemoOn | is it possible to define the number of queue in an image metadata | 17:05 |
| hegemoOn | when using virtion-scsi ? | 17:05 |
| *** damien_r has quit IRC | 17:05 | |
| *** elmaciej_ has quit IRC | 17:06 | |
| *** tbachman_ has joined #openstack-nova | 17:06 | |
| *** moshele has quit IRC | 17:07 | |
| *** umbSublime has quit IRC | 17:07 | |
| *** slaweq has quit IRC | 17:07 | |
| *** tbachman has quit IRC | 17:07 | |
| *** tbachman_ is now known as tbachman | 17:07 | |
| mriedem | stephenfin: on https://review.openstack.org/#/c/531413/ - i think i might decouple the re-wording part so we can backport that, since i see some other config guide docs that reference that option | 17:09 |
| stephenfin | mriedem: Sure thing. The reservation was because we haven't removed deprecated aliases before, that I'm aware of (there's little cost in keeping them). Worth making sure it was intentional | 17:11 |
| mriedem | i'm sure we've removed deprecated aliases before | 17:11 |
| mriedem | maybe not recently, but it was pretty common cleanup back in the day | 17:11 |
| * mriedem now expects to see a bunch of 'remove deprecated aliases' patches from stephenfin | 17:11 | |
| stephenfin | :D | 17:12 |
| stephenfin | numa-aware-vswitches has me plenty busy :) | 17:12 |
| *** tbachman_ has joined #openstack-nova | 17:12 | |
| *** tbachman has quit IRC | 17:13 | |
| *** tbachman_ is now known as tbachman | 17:13 | |
| *** ragiman has quit IRC | 17:13 | |
| hegemoOn | in virsh xml you have <driver queues='n'> | 17:14 |
| hegemoOn | for virtio-scsi | 17:14 |
| hegemoOn | i cannot see a way to define n in nova | 17:15 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Fix and update compute schedulers config guide https://review.openstack.org/544010 | 17:16 |
| *** yamahata has quit IRC | 17:17 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Move db MAX constants to own file https://review.openstack.org/543469 | 17:19 |
| *** yamamoto has quit IRC | 17:21 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove the deprecated scheduler_driver_task_period option https://review.openstack.org/531413 | 17:21 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Clarify the help text for [scheduler]periodic_task_interval https://review.openstack.org/544015 | 17:21 |
| *** yamamoto has joined #openstack-nova | 17:24 | |
| *** yamamoto has quit IRC | 17:24 | |
| *** mdnadeem has joined #openstack-nova | 17:24 | |
| stephenfin | mriedem: Should I have +Wd this, given that you've a -2 on the following patch? https://review.openstack.org/#/c/539738/ | 17:24 |
| stephenfin | I can rebase and take it out of the gate if not | 17:25 |
| *** sridharg has quit IRC | 17:25 | |
| mriedem | it's a bug fix | 17:25 |
| mriedem | so i don't think so | 17:25 |
| stephenfin | Phew. Okidok | 17:25 |
| mriedem | i mean, it's not a problem | 17:25 |
| *** itlinux has quit IRC | 17:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Remove warning in feature support matrix page https://review.openstack.org/544017 | 17:27 |
| *** moshele has joined #openstack-nova | 17:28 | |
| *** ralonsoh__ has quit IRC | 17:30 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | 17:30 |
| *** itlinux has joined #openstack-nova | 17:31 | |
| openstackgerrit | James E. Blair proposed openstack/python-novaclient stable/ocata: Zuul: Remove project name https://review.openstack.org/544018 | 17:31 |
| openstackgerrit | Merged openstack/nova stable/pike: Query all cells for service version in _validate_bdm https://review.openstack.org/541036 | 17:32 |
| openstackgerrit | Eric Berglund proposed openstack/nova master: Use correct arguments in task inits https://review.openstack.org/543571 | 17:33 |
| *** moshele has quit IRC | 17:34 | |
| mriedem | lyarwood: dansmith: bauzas: so before i request the next pike release, do i have to bump the minor version to 16.1.0 for https://review.openstack.org/#/c/528330/ because it's a schema migration; i will forever remember the red hat product team lashing i got for *not* doing something like that back in newton | 17:34 |
| *** jpena is now known as jpena|off | 17:34 | |
| dansmith | um | 17:35 |
| dansmith | okay | 17:35 |
| dansmith | bump the minor _to_ what? | 17:36 |
| mriedem | 16.0.4 -> 16.1.0 | 17:36 |
| dansmith | oh I see | 17:36 |
| mriedem | i would normally just do 16.0.5 | 17:36 |
| mriedem | but caught hell for doing that once | 17:36 |
| dansmith | I don't think I know anything about that | 17:36 |
| *** AlexeyAbashkin has quit IRC | 17:38 | |
| *** yamamoto has joined #openstack-nova | 17:40 | |
| mriedem | alright 16.1.0 it is https://review.openstack.org/#/c/544020/ | 17:40 |
| *** jangutter has joined #openstack-nova | 17:41 | |
| *** naichuans has quit IRC | 17:42 | |
| *** liverpooler has quit IRC | 17:44 | |
| *** yamamoto has quit IRC | 17:45 | |
| *** jangutter_ has quit IRC | 17:45 | |
| *** slaweq has joined #openstack-nova | 17:45 | |
| *** umbSublime has joined #openstack-nova | 17:46 | |
| *** umbSublime has quit IRC | 17:46 | |
| *** umbSublime has joined #openstack-nova | 17:46 | |
| *** markvoelker has joined #openstack-nova | 17:46 | |
| *** liverpooler has joined #openstack-nova | 17:47 | |
| *** liverpoo1er has joined #openstack-nova | 17:48 | |
| *** slaweq has quit IRC | 17:50 | |
| *** yamahata has joined #openstack-nova | 17:51 | |
| *** pcaruana has quit IRC | 17:55 | |
| *** mgoddard_ has joined #openstack-nova | 17:58 | |
| *** Sukhdev has joined #openstack-nova | 18:00 | |
| *** derekh has quit IRC | 18:02 | |
| *** pcaruana has joined #openstack-nova | 18:06 | |
| *** yamamoto has joined #openstack-nova | 18:06 | |
| *** yamamoto has quit IRC | 18:06 | |
| *** david-lyle has quit IRC | 18:11 | |
| *** Sukhdev has quit IRC | 18:22 | |
| *** dtantsur is now known as dtantsur|afk | 18:24 | |
| *** slaweq has joined #openstack-nova | 18:25 | |
| *** slaweq has quit IRC | 18:26 | |
| *** slaweq has joined #openstack-nova | 18:26 | |
| lyarwood | mriedem: 16.1.0 is fine by me, this is still an optional schema migration until queens anyway right so anyone not running migrations after updating nova during stable/pike will pick it up when they upgrade to queens. | 18:29 |
| *** itlinux has quit IRC | 18:30 | |
| lyarwood | mriedem: FWIW with TripleO/RDO I can't see us ever running schema migrations with each minor update (16.0.4 to 16.1.0 etc) | 18:31 |
| dansmith | lyarwood: apparently a minor version change kicks the "do the db sync" flag on | 18:31 |
| lyarwood | dansmith: not in any of our tooling | 18:32 |
| lyarwood | dansmith: just the poor ops guy who notices the change | 18:32 |
| dansmith | lyarwood: supposedly bauzas beat up mriedem about it last time on that basis | 18:32 |
| *** mdnadeem has quit IRC | 18:33 | |
| lyarwood | owalsh: ^ re minor updates on TripleO, we don't run schema migrations at all right? | 18:34 |
| dansmith | if not then bauzas should pay for mriedem's therapist bills | 18:35 |
| lyarwood | dansmith: I can only assume his point was that we should still highlight to ops etc that the update contains a schema migration by bumping the minor release or something | 18:35 |
| *** slaweq has quit IRC | 18:39 | |
| owalsh | lyarwood: correct, just restart services | 18:39 |
| lyarwood | owalsh: cool thanks | 18:40 |
| dansmith | death match round 2, dublin | 18:40 |
| *** slaweq has joined #openstack-nova | 18:40 | |
| *** tesseract has quit IRC | 18:40 | |
| *** Sukhdev has joined #openstack-nova | 18:41 | |
| *** moshele has joined #openstack-nova | 18:44 | |
| *** itlinux has joined #openstack-nova | 18:44 | |
| *** pcaruana has quit IRC | 18:45 | |
| *** slaweq has quit IRC | 18:45 | |
| mriedem | fork in the kidneys, check | 18:53 |
| *** lpetrut has quit IRC | 18:53 | |
| *** eharney has joined #openstack-nova | 18:54 | |
| *** ttsiouts_ has joined #openstack-nova | 18:54 | |
| *** moshele has quit IRC | 18:56 | |
| *** itlinux has quit IRC | 18:57 | |
| *** itlinux has joined #openstack-nova | 18:59 | |
| *** david-lyle has joined #openstack-nova | 19:02 | |
| *** harlowja has joined #openstack-nova | 19:03 | |
| *** yamamoto has joined #openstack-nova | 19:06 | |
| *** rmcall has joined #openstack-nova | 19:07 | |
| *** Sukhdev has quit IRC | 19:07 | |
| *** moshele has joined #openstack-nova | 19:08 | |
| *** cdent has quit IRC | 19:08 | |
| *** moshele has quit IRC | 19:12 | |
| *** hongbin has joined #openstack-nova | 19:12 | |
| *** yamamoto has quit IRC | 19:15 | |
| *** yikun_jiang has quit IRC | 19:15 | |
| *** yikun_jiang has joined #openstack-nova | 19:15 | |
| *** amoralej is now known as amoralej|off | 19:19 | |
| *** david-lyle has quit IRC | 19:29 | |
| *** david-lyle has joined #openstack-nova | 19:32 | |
| *** tidwellr has quit IRC | 19:37 | |
| *** itlinux has quit IRC | 19:39 | |
| *** itlinux has joined #openstack-nova | 19:43 | |
| *** liverpoo1er has quit IRC | 19:44 | |
| *** liverpooler has quit IRC | 19:44 | |
| *** slaweq has joined #openstack-nova | 19:45 | |
| *** moshele has joined #openstack-nova | 19:45 | |
| *** itlinux has quit IRC | 19:46 | |
| *** slaweq has quit IRC | 19:49 | |
| *** moshele has quit IRC | 19:51 | |
| *** sandk has joined #openstack-nova | 19:54 | |
| *** john51 has quit IRC | 19:54 | |
| *** john51 has joined #openstack-nova | 19:55 | |
| openstackgerrit | Merged openstack/nova master: Replace Chinese quotes to English quotes https://review.openstack.org/543349 | 19:55 |
| dansmith | efried: jaypipes: seen the question on this? https://review.openstack.org/#/c/540111/3 | 19:56 |
| dansmith | I was about to reply, but then realized I was misunderstanding his concern and I think it's probably valid | 19:56 |
| dansmith | hoping that there's some detail of how you see that working that wouldn't actually break it | 19:57 |
| efried | dansmith: It's been on my list to look at, but was rapidly getting buried. Thanks for bringing it back to the top. Looking.... | 19:57 |
| *** salv-orlando has joined #openstack-nova | 19:58 | |
| mriedem | been wondering the same type of thing with traits, | 19:58 |
| mriedem | the ironic driver will blow away any traits that aren't on the ironic node | 19:59 |
| mriedem | rather than try to merge the | 19:59 |
| mriedem | *them | 19:59 |
| dansmith | well, there needs to be some amount of that I think, | 19:59 |
| dansmith | although we can't blow them all away in this case I think | 19:59 |
| dansmith | was hoping there was some "only blows away at the given level" detail or something | 19:59 |
| efried | mriedem: I remember that being discussed at length (for ironic traits), and the conclusion in that case was that the ironic inspector was the Source Of Truth, so it was kosher to blow away anything that crept in from elsewhere. | 20:00 |
| mriedem | idk, it seems quite limiting | 20:00 |
| dansmith | efried: it is for sure until the compute service starts needing to do some too, like for capabilities | 20:00 |
| mriedem | if we have an external system to model resources that other services outside of nova can interact with, it seems wrong to completely trample them | 20:00 |
| dansmith | in that case compute might be able to do its own merging, | 20:00 |
| dansmith | but for this inventory thing, nic bandwidth is probably a good example | 20:01 |
| mriedem | dansmith: yeah that's what i had to do in my poc patch for the capabilities thing | 20:01 |
| dansmith | yeah | 20:01 |
| efried | So I agree that we don't want to make it a rule that virt blows away children it doesn't recognize. | 20:01 |
| dansmith | so maybe for this we could get the inventory from the vif modeling somehow? | 20:01 |
| *** salv-orl_ has quit IRC | 20:01 | |
| mriedem | trying to balance the stance we've had in the past against things like metrics providers in-tree saying that's all best served outside of nova, | 20:01 |
| efried | But the design (and imple) is flexible enough that we don't need to make that rule at this level. | 20:01 |
| efried | s/impl/implementation/ | 20:02 |
| mriedem | and now we appear to have something that's outside of nova for external services, but we're saying we'll overwrite what they do | 20:02 |
| efried | vay | 20:02 |
| efried | (Sorry, that was /me frustrated at own inability to spell, twice) | 20:02 |
| efried | Okay, I get the concern. | 20:02 |
| dansmith | efried: so you're saying I can have a child of compute node and the update_tree() won't blow those away, just inveentory for the level I'm updating, yeah? | 20:02 |
| openstackgerrit | Merged openstack/nova master: Invalid query parameter could lead to HTTP 500 https://review.openstack.org/539164 | 20:02 |
| efried | dansmith: We'll make placement look *exactly* like whatever comes out the other side of update_provider_tree. | 20:03 |
| efried | But I still think that's okay. | 20:03 |
| dansmith | efried: not if that means we blow away stuff we didn't return.. so now I'm confused | 20:04 |
| efried | Because 1) virt gets to be the (primary) source of truth for the provider tree rooted at the compute node. | 20:04 |
| efried | And 2) virt kinda needs to know whether there's some other entity "out there" that's allowed to mess with some level of its tree. | 20:04 |
| dansmith | well, that's what I'm saying, | 20:04 |
| efried | If it knows that, then it can preserve those pieces of the tree unchanged. | 20:04 |
| dansmith | if we go that route, | 20:04 |
| efried | Because it receives them as part of the tree it gets on input. | 20:05 |
| dansmith | then we need to have the virt drvier capable of collecting external things we support, like bandwidth on a nic | 20:05 |
| efried | nononono | 20:05 |
| dansmith | oh, | 20:05 |
| efried | It doesn't need to be able to collect them, cause we already gave it to... yeah. | 20:05 |
| dansmith | you're saying ProviderTree already has the child things, | 20:05 |
| efried | yes, exactly. | 20:05 |
| dansmith | that's the confusion though: | 20:05 |
| efried | It has the whole picture as Placement knows about it right now. | 20:05 |
| efried | ...at least the picture that's rooted at the compute host RP. | 20:05 |
| dansmith | does ProviderTree.update(<not including things that were in the child>) blow those away? or are you saying the virt driver needs to look and include those things in its update? | 20:06 |
| *** tidwellr has joined #openstack-nova | 20:06 | |
| *** vladikr has quit IRC | 20:07 | |
| efried | ProviderTree.update_[thingy](rp_uuid) for thingy in (inventory, traits, aggregates) will replace thingy for just the RP indicated by rp_uuid. | 20:07 |
| efried | E.g. if you replace inventory via that method, but not traits, then traits stay whatever they were before. | 20:07 |
| efried | Also, those update_* methods don't muck with children. | 20:08 |
| dansmith | yeah but that's not the question | 20:08 |
| dansmith | okay, _that_ is the quesation | 20:08 |
| dansmith | whether update() touches the children | 20:08 |
| dansmith | um | 20:08 |
| dansmith | "affects the children" | 20:08 |
| efried | To muck with hierarchy, you have to ProviderTree.remove(), .new_root(), .new_child()... | 20:08 |
| efried | There is no update(). | 20:08 |
| dansmith | I know, I'm saving letters | 20:08 |
| dansmith | I think that's his question | 20:08 |
| dansmith | so I had assumed it would only affect the level you're on, and was replying, | 20:08 |
| dansmith | but then re-read the bit of your text he's taking exception to and wasn't sure | 20:09 |
| dansmith | so I think you just need to tighten that up a smidge | 20:09 |
| efried | Okay. That's a true statement ("...only affects the level you're on"). *except* that if you .remove(rp), you also remove all of rp's descendants. That's what the sentence is saying. | 20:09 |
| dansmith | sure | 20:09 |
| dansmith | and, | 20:09 |
| dansmith | I was going to say that if the compute node thinks it needs to remove its provider, then it _should_ affect the external thing that still thinks it is reporting resources for that provider | 20:10 |
| dansmith | i.e. compute is authoritative for the existence of itself | 20:10 |
| efried | right. | 20:10 |
| efried | It'll be a possibly-not-enforced-but-highly-recommended rule that these entities that share control of a tree never share control of a single RP within that tree. | 20:11 |
| efried | that way lies madness. | 20:11 |
| *** tidwellr has quit IRC | 20:11 | |
| dansmith | not sure how you would enforce tat, but agree, that should be the assumption | 20:11 |
| dansmith | *that | 20:11 |
| mriedem | stephenfin: still around? | 20:11 |
| efried | Okay, I'll post an edit. Thanks for the discussion dansmith, mriedem | 20:12 |
| *** vladikr has joined #openstack-nova | 20:12 | |
| *** moshele has joined #openstack-nova | 20:14 | |
| *** amodi has joined #openstack-nova | 20:14 | |
| *** moshele has quit IRC | 20:18 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup the manage-volumes admin doc https://review.openstack.org/544066 | 20:21 |
| jaypipes | efried, dansmith, mriedem: just finished reading back... yes, agree that the spec could use some clarity/language around the update_provider_tree() method implementations not calling ProviderTree.remove() for nodes in the tree that it doesn't manage. | 20:25 |
| dansmith | aye | 20:25 |
| efried | Roger wilco. | 20:25 |
| jaypipes | efried: I still don't understand this from tetsuro though: "Note that at least libvirt driver today removes CUSTOM_RCs which users set manually via placement API for a compute host when the periodic updating task happens." | 20:26 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup the manage-volumes admin doc https://review.openstack.org/544066 | 20:26 |
| efried | jaypipes: I think he's saying libvirt removes *inventory* in custom RCs it didn't create itself. | 20:26 |
| efried | I don't think we're talking about removing the actual resource classes from placement. | 20:27 |
| jaypipes | efried: is he just saying that if an operator manually calls POST /resource_providers/{cn_uuid}/inventories and adds a CUSTOM_XXX resource class, that it will be removed by the libvirt virt driver's get_inventory() method? | 20:27 |
| efried | I think so, yes. | 20:27 |
| jaypipes | efried: gotcha. ok. and yes, that is definitely by design... | 20:27 |
| efried | yup. | 20:27 |
| *** liverpooler has joined #openstack-nova | 20:28 | |
| *** liverpoo1er has joined #openstack-nova | 20:28 | |
| mriedem | that is eventually due to this check? https://github.com/openstack/nova/blob/master/nova/scheduler/client/report.py#L831 | 20:28 |
| jaypipes | efried: since clearly the virt driver owns the resources on the compute node. but it's worth saying that update_provider_tree() implementations will *not* remove a *node* from the tree of providers if it doesn't know about the node. | 20:28 |
| jaypipes | mriedem: yea | 20:28 |
| jaypipes | mriedem: oh, no... | 20:28 |
| efried | That's just an optimization that'll short-circuit if nothing has changed. | 20:29 |
| jaypipes | mriedem: that check is seeing if what we got from get_inventory() is the same as what is cached in the reportclient's provider tree. | 20:29 |
| jaypipes | what efried said. | 20:29 |
| mriedem | but if the driver reports some inventory change, for VCPU, MEMORY_MB and DISK_GB, | 20:29 |
| mriedem | and some external service had set some CUSTOM_FOO inventory, | 20:30 |
| mriedem | that's the code that's going to blast it away right? | 20:30 |
| jaypipes | mriedem: that conditional will be true if the operator set some CUSTOM_XXX resource class inventory and libvirt doesn't report that resource class. | 20:30 |
| *** READ10 has quit IRC | 20:30 | |
| *** mgoddard_ has quit IRC | 20:30 | |
| jaypipes | mriedem: it's just that inv_data is always exactly and only what the virt driver sets it to. | 20:30 |
| mriedem | yeah ok | 20:30 |
| jaypipes | mriedem: and that will overwrite anything the operator manually set. | 20:31 |
| mriedem | yup | 20:31 |
| jaypipes | mriedem: right, but again, that's get_inventory(). the update_provider_tree() method will blow away inventory for the specific (root) compute node provider and set it to whatever it thinks it should be. but it won't delete child nodes it doesn't manage. so if the operator (or external agent) set inventory for, say, NET_EFRESS_BYTES_PER_SECOND on a child provider node, the update_provider_tree() method won't delete that node and its inventory. | 20:32 |
| jaypipes | EGRESS... | 20:33 |
| *** tidwellr has joined #openstack-nova | 20:33 | |
| mriedem | EFRIED_BYTES | 20:33 |
| mriedem | jaypipes: got it | 20:33 |
| jaypipes | mriedem: but if an operator manually set the VCPU inventory on the compute node resource provider to something, the virt driver absolutely will overwrite that inventory to what it thinks is correct. | 20:34 |
| jaypipes | mriedem: ok, sorry for babbling | 20:34 |
| mriedem | yup makes sense | 20:34 |
| efried | jaypipes: tbc, we have no way (other than code reviews) to enforce that update_provider_tree doesn't muck with child nodes it doesn't manage. | 20:35 |
| *** itlinux has joined #openstack-nova | 20:36 | |
| mriedem | functional tests? | 20:36 |
| jaypipes | efried: correct. | 20:36 |
| efried | mriedem: Functional tests that would have to know something about the expected structure of the tree for the individual virt driver. | 20:36 |
| efried | structure & management | 20:36 |
| *** pramodrj07 has joined #openstack-nova | 20:37 | |
| jaypipes | efried: we can't programmatically determine resource ownership and policy for the various virt drivers and potential external agents ahead of time. all we can do is docstring the heck out of update_provider_tree() and explain the ramifications of calling ProviderTree.remove(). | 20:38 |
| efried | But yeah. That's my point. Whoever writes & reviews that test, and the code it operates on, has to know which bits are managed by virt and which by external agents. | 20:38 |
| *** pramodrj07 has quit IRC | 20:38 | |
| efried | jaypipes: ++ I need to add words to the docstring for this. | 20:38 |
| efried | I think there are words there, but they don't really talk about the implications, just about the effects. | 20:39 |
| *** awaugama has quit IRC | 20:39 | |
| jaypipes | efried: for instance, the PowerVM virt driver may just as well WANT to call ProviderTree.remove() on child nodes if the PowerVM virt driver is fully managing things like NICs and disk subsystems, etc.. | 20:39 |
| jaypipes | just no way to know ahead of time. | 20:39 |
| efried | exactly so. | 20:40 |
| jaypipes | efried: and, furthermore, the virt driver's update_provider_tree() method is pretty much exactly what I'd like us to propose to external agents and systems like Cinder/Neutron to use as their "interface" into the resource provider modeling of Placement's API. | 20:41 |
| efried | ++ | 20:41 |
| openstackgerrit | Jeremy Stanley proposed openstack/nova master: Include PCRE headers in bindep manifest https://review.openstack.org/544071 | 20:42 |
| fungi | just ran across that ^ due to https://ask.openstack.org/question/112467 | 20:42 |
| jaypipes | edleafe: unless they're being blocked as spam, I don't think the email notifications on your blog comments are working :( | 20:43 |
| edleafe | jaypipes: arrrrggghh | 20:50 |
| edleafe | recent update must have changed the settings | 20:50 |
| edleafe | lemme investigate | 20:50 |
| jaypipes | sean-k-mooney: so... | 20:53 |
| jaypipes | sean-k-mooney: are you or are you not in support of this spec? https://review.openstack.org/#/c/533077/3/specs/rocky/approved/libvirt-qemu-numa-support.rst | 20:54 |
| openstackgerrit | melissaml proposed openstack/nova-specs master: Replace Chinese quotes to English quotes https://review.openstack.org/544076 | 20:56 |
| *** tbachman has quit IRC | 20:57 | |
| openstackgerrit | Merged openstack/python-novaclient stable/ocata: Zuul: Remove project name https://review.openstack.org/544018 | 20:57 |
| *** amodi has quit IRC | 20:58 | |
| mriedem | fungi: https://review.openstack.org/#/c/542340/ | 20:58 |
| fungi | mriedem: excellent! i'll abandon mine. thanks | 21:00 |
| openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Make formatting of console consistent https://review.openstack.org/544081 | 21:02 |
| dansmith | efried: maybe you can answer this: https://review.openstack.org/#/c/540447/3 | 21:03 |
| dansmith | or jaypipes ^ | 21:04 |
| efried | on it | 21:04 |
| *** vladikr has quit IRC | 21:04 | |
| edleafe | jaypipes: should be working now. LMK if you don't get emails for future comments | 21:05 |
| efried | dansmith: Done. | 21:06 |
| dansmith | efried: replied, thanks | 21:08 |
| *** vladikr has joined #openstack-nova | 21:08 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | 21:09 |
| efried | mriedem, jaypipes, dansmith: There's that edit we talked about yesterday ^ | 21:09 |
| efried | edleafe: ^ | 21:10 |
| *** slaweq has joined #openstack-nova | 21:11 | |
| jaypipes | efried: cool, thanks | 21:12 |
| efried | jaypipes: In case you didn't read that backscroll (link in the commit message of ^) the plan is to propose a more or less new Rocky spec to talk about NRP-in-GET-alloc-cands stuff, rather than "reproposing" the NRP spec. | 21:13 |
| *** rmcall has quit IRC | 21:13 | |
| mriedem | i would recommend not reading the backscroll... | 21:14 |
| efried | Basically, the Q one lays the NRP groundwork, and the stuff coming off of that spiders in multiple directions - update_provider_tree, NRP-in-alloc-cands, wtf we're going to do about NUMA, etc. - each of which ought to have its own spec. | 21:14 |
| mriedem | but if you do, your reward is some nice simpsons clip | 21:14 |
| jaypipes | efried: ack | 21:16 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add admin guide doc on volume multiattach support https://review.openstack.org/544090 | 21:17 |
| openstackgerrit | Merged openstack/nova-specs master: Add generation support in aggregate association https://review.openstack.org/540447 | 21:17 |
| *** Sukhdev has joined #openstack-nova | 21:18 | |
| edleafe | efried: will review shortly | 21:19 |
| efried | d'oh, I forgot to tag the commit message with the bp. Editing... | 21:20 |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | 21:20 |
| *** ttsiouts_ has quit IRC | 21:20 | |
| efried | done ^ | 21:20 |
| *** ttsiouts_ has joined #openstack-nova | 21:21 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Clarify old microversion behavior https://review.openstack.org/544092 | 21:21 |
| efried | dansmith: There's that followup ^ | 21:22 |
| *** felipemonteiro_ has quit IRC | 21:22 | |
| dansmith | efried: I sent it to its maker, thanks | 21:24 |
| *** ttsiouts_ has quit IRC | 21:26 | |
| openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Wrap long console command https://review.openstack.org/544093 | 21:29 |
| *** ttx has quit IRC | 21:30 | |
| openstackgerrit | Merged openstack/nova master: doc: merge numa.rst to cpu-topologies.rst https://review.openstack.org/540585 | 21:30 |
| *** pchavva has quit IRC | 21:33 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | 21:38 |
| mriedem | efried: lgtm, thanks | 21:39 |
| openstackgerrit | Merged openstack/nova master: install-guide: Make formatting of console consistent https://review.openstack.org/544081 | 21:42 |
| openstackgerrit | Merged openstack/nova-specs master: Clarify old microversion behavior https://review.openstack.org/544092 | 21:43 |
| *** openstackstatus has quit IRC | 21:43 | |
| *** niraj_singh has quit IRC | 21:44 | |
| *** openstackstatus has joined #openstack-nova | 21:45 | |
| *** ChanServ sets mode: +v openstackstatus | 21:45 | |
| mriedem | dansmith: easy peasy https://review.openstack.org/#/c/543571/ | 21:45 |
| mriedem | likely RC2 fodder | 21:45 |
| dansmith | whoopsie | 21:47 |
| *** ttsiouts_ has joined #openstack-nova | 21:50 | |
| *** afazekas has quit IRC | 21:50 | |
| *** afazekas has joined #openstack-nova | 21:52 | |
| *** ttsiouts_ has quit IRC | 21:53 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Bindep does not catch missing libpcre3-dev on Ubuntu https://review.openstack.org/544108 | 21:54 |
| mriedem | esberglu: want to post a backport to queens for https://review.openstack.org/#/c/543571/ ? | 21:54 |
| openstackgerrit | Eric Berglund proposed openstack/nova stable/queens: Use correct arguments in task inits https://review.openstack.org/544109 | 21:54 |
| mriedem | with the -x option on git cherry-pick please | 21:54 |
| *** gyee has joined #openstack-nova | 22:00 | |
| *** tidwellr has quit IRC | 22:03 | |
| openstackgerrit | Eric Fried proposed openstack/nova-specs master: Update Provider Tree https://review.openstack.org/540111 | 22:03 |
| efried | dansmith, mriedem, jaypipes, edmondsw, edleafe: ^ | 22:03 |
| *** tidwellr has joined #openstack-nova | 22:04 | |
| *** rcernin has joined #openstack-nova | 22:05 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: Use correct arguments in task inits https://review.openstack.org/544109 | 22:05 |
| mriedem | esberglu: like that ^ | 22:05 |
| *** slaweq_ has joined #openstack-nova | 22:06 | |
| esberglu | mriedem: Was one command slower :) | 22:06 |
| esberglu | I just used the gerrit UI, noted for the future | 22:06 |
| mriedem | cherry picking from gerrit UI doesn't give you the -x thing until the change is merged | 22:06 |
| mriedem | so you have to resort to the CLI | 22:06 |
| *** r-daneel has quit IRC | 22:07 | |
| *** tidwellr has quit IRC | 22:08 | |
| *** tbachman has joined #openstack-nova | 22:08 | |
| *** eharney has quit IRC | 22:09 | |
| *** niraj_singh has joined #openstack-nova | 22:10 | |
| melwitt | mriedem: about https://review.openstack.org/#/c/539003, we (and everyone) need that, else we have the same exposure from CVE-2017-16239 if a "bad rebuild" is attempted twice | 22:16 |
| melwitt | mriedem: amoralej|off was asking in #rdo earlier today, which novaclient version should be used in u-c? he said he saw this change with a -2 on it and wanted to know if 9.1.1 is going to be the official version for queens | 22:22 |
| melwitt | https://review.openstack.org/#/c/538070 | 22:22 |
| *** itlinux has quit IRC | 22:23 | |
| *** acormier_ has quit IRC | 22:25 | |
| *** acormier has joined #openstack-nova | 22:25 | |
| *** tbachman has quit IRC | 22:26 | |
| *** itlinux has joined #openstack-nova | 22:27 | |
| *** acormier has quit IRC | 22:30 | |
| *** sandk has quit IRC | 22:30 | |
| *** tbachman has joined #openstack-nova | 22:36 | |
| *** felipemonteiro has joined #openstack-nova | 22:40 | |
| mlavalle | melwitt: hi. I am starting to put together the PTG agenda for the Neutron team. We have some topics that I think we should discuss with the Nova team. Can we have a cross team session? Maybe Thursday afternoon? | 22:40 |
| melwitt | mlavalle: yes, we have some topics we need to discuss with you all too (L145 in https://etherpad.openstack.org/p/nova-ptg-rocky). I think thursday afternoon should probably work | 22:43 |
| mlavalle | melwitt: can I add to that section of your etherpad the items I have in my radar screen? I'll ping you again tomorrow afternoon, once you've had a chance to look at them | 22:46 |
| melwitt | mlavalle: sure, please do | 22:46 |
| mlavalle | melwitt: :-) | 22:46 |
| *** slaweq_ has quit IRC | 22:46 | |
| *** itlinux has quit IRC | 22:46 | |
| *** slaweq_ has joined #openstack-nova | 22:47 | |
| *** slaweq_ has quit IRC | 22:51 | |
| mriedem | melwitt: yes on novaclient because 10.0.0 breaks horizon in queens | 22:55 |
| mriedem | and openstackclient | 22:55 |
| melwitt | okay, cool. thanks | 22:56 |
| mriedem | melwitt: ok for https://review.openstack.org/#/q/I3399a66fe9b1297cd6b0dca440145393ceaef41f - half the stable core team is from red hat so you'll have to raise the alarm there | 22:57 |
| *** tbachman has quit IRC | 22:58 | |
| mriedem | or just ping mtreinish | 22:58 |
| mriedem | he loves stable | 22:58 |
| melwitt | okay, will do. thanks for the heads up | 22:59 |
| *** sdague has quit IRC | 23:01 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova-specs master: Update Queens NRP spec to reflect reality https://review.openstack.org/544086 | 23:03 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Check for leaked server resource allocations in post_test_hook https://review.openstack.org/538510 | 23:08 |
| mtreinish | mriedem: Need me to look at some backports? | 23:08 |
| *** tbachman has joined #openstack-nova | 23:09 | |
| mriedem | https://review.openstack.org/#/q/I3399a66fe9b1297cd6b0dca440145393ceaef41f | 23:09 |
| *** hongbin has quit IRC | 23:10 | |
| *** suresh12 has quit IRC | 23:11 | |
| mtreinish | mriedem: when does ocata enter phase 3? | 23:12 |
| mriedem | LTS baby | 23:12 |
| mriedem | mtreinish: feb 26 i guess? https://releases.openstack.org/ | 23:13 |
| mriedem | this is related to a cve | 23:13 |
| *** mriedem is now known as mriedem_afk | 23:17 | |
| *** suresh12 has joined #openstack-nova | 23:17 | |
| openstackgerrit | Merged openstack/nova master: Use correct arguments in task inits https://review.openstack.org/543571 | 23:17 |
| *** jobewan has quit IRC | 23:21 | |
| openstackgerrit | Nicolas Bock proposed openstack/nova master: install-guide: Wrap long console command https://review.openstack.org/544093 | 23:22 |
| openstackgerrit | Merged openstack/nova master: Ensure resource classes correctly https://review.openstack.org/539738 | 23:25 |
| *** acormier has joined #openstack-nova | 23:29 | |
| openstackgerrit | Nicolas Bock proposed openstack/nova stable/pike: Fix SUSE Install Guide: Placement port https://review.openstack.org/544131 | 23:30 |
| *** acormier has quit IRC | 23:37 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Avoid inventory DELETE API (no conflict detection) https://review.openstack.org/539712 | 23:41 |
| efried | jaypipes: Yet another manual rebase +2 please ^ | 23:41 |
| *** Sukhdev has quit IRC | 23:42 | |
| *** slaweq has quit IRC | 23:45 | |
| *** tbachman has quit IRC | 23:46 | |
| *** felipemonteiro has quit IRC | 23:50 | |
| *** acormier has joined #openstack-nova | 23:50 | |
| *** acormier has quit IRC | 23:55 | |
| *** takashin has joined #openstack-nova | 23:56 | |
| *** stakeda has joined #openstack-nova | 23:58 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!