| *** masber has joined #openstack-nova | 00:03 | |
| *** masber has quit IRC | 00:03 | |
| *** masber has joined #openstack-nova | 00:03 | |
| *** masber has quit IRC | 00:04 | |
| *** masber has joined #openstack-nova | 00:04 | |
| *** masber has quit IRC | 00:04 | |
| *** masber has joined #openstack-nova | 00:06 | |
| *** masber has quit IRC | 00:06 | |
| *** masber has joined #openstack-nova | 00:08 | |
| *** yamamoto has joined #openstack-nova | 00:10 | |
| *** slaweq has joined #openstack-nova | 00:11 | |
| *** yamamoto has quit IRC | 00:15 | |
| *** slaweq has quit IRC | 00:16 | |
| *** hiro-kobayashi has joined #openstack-nova | 00:22 | |
| *** yassine has joined #openstack-nova | 00:35 | |
| *** yassine is now known as Guest17713 | 00:35 | |
| *** germs has joined #openstack-nova | 00:37 | |
| *** germs has quit IRC | 00:37 | |
| *** germs has joined #openstack-nova | 00:37 | |
| *** felipemonteiro__ has joined #openstack-nova | 00:41 | |
| *** germs has quit IRC | 00:41 | |
| *** gbarros has joined #openstack-nova | 00:44 | |
| *** felipemonteiro__ has quit IRC | 00:47 | |
| *** hshiina has joined #openstack-nova | 00:59 | |
| *** hongbin has joined #openstack-nova | 01:01 | |
| *** edmondsw has joined #openstack-nova | 01:07 | |
| *** edmondsw has quit IRC | 01:11 | |
| *** yamamoto has joined #openstack-nova | 01:12 | |
| *** phuongnh has joined #openstack-nova | 01:14 | |
| *** yamamoto has quit IRC | 01:17 | |
| *** edmondsw has joined #openstack-nova | 01:22 | |
| *** gbarros has quit IRC | 01:25 | |
| *** takashin has joined #openstack-nova | 01:30 | |
| *** trungnv has joined #openstack-nova | 01:33 | |
| *** edmondsw has quit IRC | 01:34 | |
| *** edmondsw has joined #openstack-nova | 01:35 | |
| *** edmondsw has quit IRC | 01:37 | |
| *** edmondsw has joined #openstack-nova | 01:37 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 01:38 |
|---|---|---|
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 01:38 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 01:38 |
| *** edmondsw has quit IRC | 01:38 | |
| *** tiendc has joined #openstack-nova | 01:39 | |
| *** baoli has joined #openstack-nova | 01:45 | |
| *** yamamoto has joined #openstack-nova | 01:49 | |
| *** bkopilov has quit IRC | 01:56 | |
| *** baoli has quit IRC | 01:59 | |
| *** baoli has joined #openstack-nova | 02:00 | |
| *** Tom-Tom has joined #openstack-nova | 02:02 | |
| *** baoli has quit IRC | 02:04 | |
| *** yamamoto has quit IRC | 02:09 | |
| *** yamamoto has joined #openstack-nova | 02:10 | |
| *** baoli has joined #openstack-nova | 02:17 | |
| *** lei-zh has joined #openstack-nova | 02:23 | |
| *** Guest17713 has quit IRC | 02:28 | |
| *** baoli has quit IRC | 02:28 | |
| *** baoli has joined #openstack-nova | 02:29 | |
| *** baoli has quit IRC | 02:34 | |
| *** vivsoni has quit IRC | 02:35 | |
| *** germs has joined #openstack-nova | 02:38 | |
| *** germs has quit IRC | 02:42 | |
| *** Guest17713 has joined #openstack-nova | 02:43 | |
| *** vivsoni has joined #openstack-nova | 02:45 | |
| *** yamamoto_ has joined #openstack-nova | 02:48 | |
| *** yamamoto has quit IRC | 02:50 | |
| *** psachin has joined #openstack-nova | 02:53 | |
| *** gbarros has joined #openstack-nova | 02:57 | |
| *** Nel1x has quit IRC | 02:58 | |
| *** gbarros has quit IRC | 03:01 | |
| *** takashin has quit IRC | 03:25 | |
| *** WonderfulAlice has joined #openstack-nova | 03:28 | |
| WonderfulAlice | Hey guys I see that nova net-list is now deprecated. What do you use now instead to create and assign different network interfaces? | 03:30 |
| *** bkopilov has joined #openstack-nova | 03:31 | |
| WonderfulAlice | I'm trying to follow this guild (https://docs.ovh.com/gb/en/public-cloud/use_vrack_and_private_networks_with_public_cloud_instances/#with-the-openstack-api) but it seems to now be dated with the new updates. | 03:32 |
| *** udesale has joined #openstack-nova | 03:44 | |
| *** threestrands has joined #openstack-nova | 03:59 | |
| *** liuzz_ has joined #openstack-nova | 03:59 | |
| *** threestrands_ has joined #openstack-nova | 04:00 | |
| *** WonderfulAlice has left #openstack-nova | 04:00 | |
| *** threestrands_ has quit IRC | 04:01 | |
| *** threestrands_ has joined #openstack-nova | 04:02 | |
| *** liuzz has quit IRC | 04:02 | |
| *** kevinbenton has quit IRC | 04:03 | |
| *** threestrands has quit IRC | 04:04 | |
| *** kevinbenton has joined #openstack-nova | 04:05 | |
| *** Tom-Tom_ has joined #openstack-nova | 04:06 | |
| *** Tom-Tom has quit IRC | 04:07 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (3) https://review.openstack.org/568184 | 04:07 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_compute_api.py (3) https://review.openstack.org/568184 | 04:09 |
| *** slaweq has joined #openstack-nova | 04:11 | |
| *** lei-zh has quit IRC | 04:12 | |
| *** slaweq has quit IRC | 04:16 | |
| *** toabctl has joined #openstack-nova | 04:21 | |
| *** hoonetorg has quit IRC | 04:21 | |
| *** lpetrut has joined #openstack-nova | 04:25 | |
| *** hongbin has quit IRC | 04:36 | |
| *** bhujay has joined #openstack-nova | 04:37 | |
| *** takashin has joined #openstack-nova | 04:38 | |
| *** hoonetorg has joined #openstack-nova | 04:39 | |
| *** germs has joined #openstack-nova | 04:39 | |
| *** threestrands_ has quit IRC | 04:39 | |
| *** pooja_jadhav has joined #openstack-nova | 04:40 | |
| *** links has joined #openstack-nova | 04:41 | |
| *** germs has quit IRC | 04:44 | |
| *** armaan has joined #openstack-nova | 04:50 | |
| *** udesale has quit IRC | 04:51 | |
| *** armaan has quit IRC | 04:52 | |
| *** armaan has joined #openstack-nova | 04:52 | |
| *** udesale has joined #openstack-nova | 04:56 | |
| *** fragatina has quit IRC | 04:57 | |
| *** threestrands has joined #openstack-nova | 05:04 | |
| *** sridharg has joined #openstack-nova | 05:06 | |
| *** fragatina has joined #openstack-nova | 05:07 | |
| *** threestrands has quit IRC | 05:07 | |
| *** lpetrut has quit IRC | 05:14 | |
| *** ratailor has joined #openstack-nova | 05:17 | |
| *** fragatina has quit IRC | 05:22 | |
| *** fragatina has joined #openstack-nova | 05:22 | |
| *** udesale_ has joined #openstack-nova | 05:23 | |
| *** hshiina2 has joined #openstack-nova | 05:25 | |
| *** udesale has quit IRC | 05:25 | |
| *** hshiina has quit IRC | 05:29 | |
| *** lei-zh has joined #openstack-nova | 05:30 | |
| *** stakeda has quit IRC | 05:36 | |
| *** evin has quit IRC | 05:42 | |
| *** fragatina has quit IRC | 05:43 | |
| *** lei-zh has quit IRC | 05:47 | |
| *** lei-zh has joined #openstack-nova | 05:48 | |
| openstackgerrit | caoyuan proposed openstack/nova master: Update auth_url in install docs https://review.openstack.org/568002 | 05:49 |
| *** abhishekk has joined #openstack-nova | 05:58 | |
| *** belmoreira has joined #openstack-nova | 06:08 | |
| *** blkart has joined #openstack-nova | 06:08 | |
| *** evin has joined #openstack-nova | 06:11 | |
| *** slaweq has joined #openstack-nova | 06:11 | |
| *** stakeda has joined #openstack-nova | 06:12 | |
| *** tbachman has joined #openstack-nova | 06:14 | |
| *** slaweq has quit IRC | 06:16 | |
| *** tbachman_ has joined #openstack-nova | 06:18 | |
| *** tbachman has quit IRC | 06:19 | |
| *** tbachman_ is now known as tbachman | 06:19 | |
| *** hshiina2 is now known as hshiina | 06:21 | |
| *** andreas_s has joined #openstack-nova | 06:28 | |
| openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Update docs, API samples for noVNC 1.0.0 https://review.openstack.org/550173 | 06:29 |
| takashin | alex_xu: Would you review https://review.openstack.org/#/c/529520/ ? It got one +2. | 06:29 |
| *** pcaruana has joined #openstack-nova | 06:31 | |
| *** tbachman has quit IRC | 06:31 | |
| *** moshele has joined #openstack-nova | 06:33 | |
| *** annp has joined #openstack-nova | 06:34 | |
| takashin | ls | 06:38 |
| *** germs has joined #openstack-nova | 06:40 | |
| *** germs has quit IRC | 06:40 | |
| *** germs has joined #openstack-nova | 06:40 | |
| *** germs has quit IRC | 06:44 | |
| *** do3meli has joined #openstack-nova | 06:48 | |
| *** slaweq has joined #openstack-nova | 06:49 | |
| *** maciejjozefczyk has joined #openstack-nova | 06:53 | |
| *** maciejjozefczyk has quit IRC | 06:53 | |
| *** tssurya has joined #openstack-nova | 06:54 | |
| *** armaan_ has joined #openstack-nova | 07:05 | |
| *** sahid has joined #openstack-nova | 07:08 | |
| *** armaan has quit IRC | 07:09 | |
| *** tesseract has joined #openstack-nova | 07:10 | |
| *** dpawlik has joined #openstack-nova | 07:13 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Metadata-API fails to retrieve avz for instances created before Pike https://review.openstack.org/567878 | 07:14 |
| *** amoralej|off is now known as amoralej | 07:19 | |
| *** mgoddard has joined #openstack-nova | 07:27 | |
| *** armaan has joined #openstack-nova | 07:29 | |
| *** armaan_ has quit IRC | 07:32 | |
| *** trungnv has quit IRC | 07:35 | |
| *** fragatina has joined #openstack-nova | 07:35 | |
| *** trungnv has joined #openstack-nova | 07:35 | |
| *** damien_r has joined #openstack-nova | 07:36 | |
| *** rcernin has quit IRC | 07:39 | |
| *** janki has joined #openstack-nova | 07:40 | |
| *** ragiman has joined #openstack-nova | 07:41 | |
| *** AlexeyAbashkin has joined #openstack-nova | 07:46 | |
| openstackgerrit | Radoslav Gerganov proposed openstack/nova stable/queens: Add random sleep between retry calls to placement https://review.openstack.org/568218 | 07:47 |
| openstackgerrit | Lenny Verkhovsky proposed openstack/nova master: Adding NVMEoF for libvirt driver https://review.openstack.org/482640 | 07:47 |
| gibi | good morning nova | 07:48 |
| *** jpena|off is now known as jpena | 07:53 | |
| *** kevinbenton40 has joined #openstack-nova | 07:54 | |
| *** fragatina has quit IRC | 07:54 | |
| *** kevinbenton has quit IRC | 07:56 | |
| *** kevinbenton40 has quit IRC | 07:57 | |
| *** tiendc has quit IRC | 07:57 | |
| *** annp has quit IRC | 07:57 | |
| *** tiendc has joined #openstack-nova | 07:58 | |
| *** annp has joined #openstack-nova | 07:58 | |
| *** lucas-afk is now known as lucasagomes | 07:59 | |
| *** kevinbenton has joined #openstack-nova | 08:01 | |
| *** belmoreira has quit IRC | 08:01 | |
| *** takashin has left #openstack-nova | 08:02 | |
| *** belmoreira has joined #openstack-nova | 08:04 | |
| *** gibi has quit IRC | 08:05 | |
| *** gibi has joined #openstack-nova | 08:07 | |
| *** gibi is now known as Guest44118 | 08:07 | |
| bauzas | good morning stackers | 08:09 |
| bauzas | me is back from holidays | 08:09 |
| * kashyap waves | 08:10 | |
| *** ivve has quit IRC | 08:11 | |
| kevinbenton | is there anything special nova does to prevent libvirt/kvm from deleting a pre-created network interface? | 08:11 |
| kevinbenton | when stopping an instance | 08:11 |
| *** dougsz has joined #openstack-nova | 08:12 | |
| *** janki has quit IRC | 08:14 | |
| *** janki has joined #openstack-nova | 08:14 | |
| *** mdnadeem has joined #openstack-nova | 08:16 | |
| *** dtantsur|afk is now known as dtantsur | 08:17 | |
| *** lpetrut has joined #openstack-nova | 08:21 | |
| *** janki has quit IRC | 08:22 | |
| *** janki has joined #openstack-nova | 08:22 | |
| *** derekh has joined #openstack-nova | 08:29 | |
| *** alex_xu has quit IRC | 08:32 | |
| *** alex_xu has joined #openstack-nova | 08:33 | |
| *** cdent has joined #openstack-nova | 08:37 | |
| *** udesale_ has quit IRC | 08:41 | |
| *** Shilpa has joined #openstack-nova | 08:42 | |
| *** udesale has joined #openstack-nova | 08:43 | |
| *** Guest44118 is now known as gibi | 08:44 | |
| *** moshele has quit IRC | 08:47 | |
| *** moshele has joined #openstack-nova | 08:48 | |
| *** sapcc-bot4 has joined #openstack-nova | 09:00 | |
| *** purplerbot has quit IRC | 09:02 | |
| *** purplerbot_ has joined #openstack-nova | 09:02 | |
| *** purplerbot_ is now known as purplerbot__ | 09:02 | |
| *** purplerbot__ is now known as purplerbot | 09:02 | |
| *** annp has quit IRC | 09:04 | |
| *** sridharg has quit IRC | 09:04 | |
| *** sapcc-bot3 has quit IRC | 09:04 | |
| *** annp has joined #openstack-nova | 09:05 | |
| *** masber has quit IRC | 09:06 | |
| *** dtantsur is now known as dtantsur|brb | 09:08 | |
| lyarwood | https://review.openstack.org/#/c/567899/ - Any cores able to +W this bugfix for libvirt LM with RAW disks? We (well, I) backported the offending change back across all open stable branches FWIW. | 09:08 |
| lyarwood | bauzas: \o_ welcome back ^ if you have time this morning | 09:09 |
| bauzas | lyarwood: will try | 09:09 |
| *** dpawlik has quit IRC | 09:10 | |
| *** sridharg has joined #openstack-nova | 09:16 | |
| *** hiro-kobayashi has quit IRC | 09:19 | |
| *** lei-zh has quit IRC | 09:23 | |
| *** lei-zh has joined #openstack-nova | 09:23 | |
| openstackgerrit | sahid proposed openstack/nova master: libvirt: place emulator threads on CONF.cpu_shared_set https://review.openstack.org/510897 | 09:24 |
| *** avolkov has joined #openstack-nova | 09:31 | |
| *** mgoddard has quit IRC | 09:33 | |
| *** lei-zh has quit IRC | 09:37 | |
| *** moshele has quit IRC | 09:37 | |
| *** lei-zh has joined #openstack-nova | 09:38 | |
| *** moshele has joined #openstack-nova | 09:38 | |
| *** sidx64 has joined #openstack-nova | 09:43 | |
| *** armaan has quit IRC | 09:43 | |
| *** jmccarthy has joined #openstack-nova | 09:44 | |
| *** armaan has joined #openstack-nova | 09:44 | |
| *** abalutoiu has joined #openstack-nova | 09:47 | |
| *** dpawlik has joined #openstack-nova | 09:48 | |
| *** diga has joined #openstack-nova | 09:53 | |
| *** dpawlik has quit IRC | 09:54 | |
| *** tiendc has quit IRC | 09:54 | |
| *** dpawlik has joined #openstack-nova | 09:55 | |
| *** jaosorior has joined #openstack-nova | 09:56 | |
| *** lei-zh has quit IRC | 09:56 | |
| *** sambetts|afk is now known as sambetts | 10:02 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 10:04 |
| *** mgoddard has joined #openstack-nova | 10:04 | |
| *** yamamoto_ has quit IRC | 10:13 | |
| *** bkopilov has quit IRC | 10:13 | |
| *** yamamoto has joined #openstack-nova | 10:14 | |
| sean-k-mooney | bauzas: johnthetubaguy could ye take a look at this trival backport for os-vif stable/queens branch when ye have time. https://review.openstack.org/#/c/567942/ | 10:18 |
| *** yamamoto_ has joined #openstack-nova | 10:19 | |
| *** yamamoto has quit IRC | 10:22 | |
| *** nicolasbock has joined #openstack-nova | 10:29 | |
| *** Tom-Tom_ has quit IRC | 10:29 | |
| *** Tom-Tom has joined #openstack-nova | 10:30 | |
| *** Tom-Tom has quit IRC | 10:35 | |
| *** dtantsur|brb is now known as dtantsur | 10:38 | |
| *** jmccarthy has quit IRC | 10:41 | |
| *** armaan has quit IRC | 10:49 | |
| *** armaan has joined #openstack-nova | 10:49 | |
| *** janki has quit IRC | 10:57 | |
| *** armaan has quit IRC | 10:59 | |
| *** armaan has joined #openstack-nova | 10:59 | |
| *** sidx64 has quit IRC | 10:59 | |
| *** sidx64 has joined #openstack-nova | 11:01 | |
| *** sar_ has joined #openstack-nova | 11:01 | |
| *** bhujay has quit IRC | 11:02 | |
| *** bhujay has joined #openstack-nova | 11:02 | |
| *** evin has quit IRC | 11:03 | |
| *** edmondsw has joined #openstack-nova | 11:03 | |
| *** bhujay has quit IRC | 11:04 | |
| *** mdnadeem has quit IRC | 11:04 | |
| *** bhujay has joined #openstack-nova | 11:05 | |
| *** links has quit IRC | 11:05 | |
| *** jmccarthy has joined #openstack-nova | 11:06 | |
| *** andreas_s has quit IRC | 11:10 | |
| *** andreas_s has joined #openstack-nova | 11:10 | |
| *** bhujay has quit IRC | 11:12 | |
| *** andreas_s has quit IRC | 11:15 | |
| *** Tom-Tom has joined #openstack-nova | 11:15 | |
| *** dpawlik has quit IRC | 11:16 | |
| *** mdnadeem has joined #openstack-nova | 11:16 | |
| *** links has joined #openstack-nova | 11:17 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Escalate UUID validation warning to error in test https://review.openstack.org/540386 | 11:18 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Suppress UUID warning in map_instance unit tests https://review.openstack.org/568263 | 11:18 |
| *** phuongnh has quit IRC | 11:18 | |
| *** dpawlik has joined #openstack-nova | 11:23 | |
| *** needsleep is now known as TheJulia | 11:25 | |
| *** andreas_s has joined #openstack-nova | 11:26 | |
| *** tomtom001 has quit IRC | 11:28 | |
| *** tomtom001 has joined #openstack-nova | 11:28 | |
| *** jpena is now known as jpena|lunch | 11:31 | |
| *** dougsz has quit IRC | 11:34 | |
| *** bkopilov has joined #openstack-nova | 11:34 | |
| *** janki has joined #openstack-nova | 11:37 | |
| *** mgoddard has quit IRC | 11:38 | |
| *** diga has quit IRC | 11:40 | |
| *** Tom-Tom has quit IRC | 11:41 | |
| *** Tom-Tom has joined #openstack-nova | 11:41 | |
| *** jmccarthy has quit IRC | 11:42 | |
| *** wolverineav has joined #openstack-nova | 11:44 | |
| *** yamamoto_ has quit IRC | 11:44 | |
| *** amoralej is now known as amoralej|lunch | 11:45 | |
| *** yamamoto has joined #openstack-nova | 11:45 | |
| *** Tom-Tom has quit IRC | 11:46 | |
| *** nicolasbock has quit IRC | 11:49 | |
| *** sar_ has quit IRC | 11:49 | |
| *** yamamoto has quit IRC | 11:50 | |
| *** yamamoto has joined #openstack-nova | 11:51 | |
| *** yamamoto has quit IRC | 11:51 | |
| *** links has quit IRC | 11:52 | |
| *** links has joined #openstack-nova | 11:52 | |
| *** sar_ has joined #openstack-nova | 11:52 | |
| *** abhishekk has quit IRC | 11:54 | |
| *** jmccarthy has joined #openstack-nova | 11:54 | |
| *** hshiina is now known as hshiina|afk | 11:55 | |
| openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 11:58 |
| *** weshay is now known as weshay_interview | 11:59 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova service-list use scatter-gather routine https://review.openstack.org/568271 | 12:02 |
| *** Tom-Tom has joined #openstack-nova | 12:05 | |
| *** sidx64 has quit IRC | 12:08 | |
| *** lpetrut has quit IRC | 12:10 | |
| *** Tom-Tom has quit IRC | 12:10 | |
| *** abalutoiu has quit IRC | 12:10 | |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return nested providers in get_by_request https://review.openstack.org/567113 | 12:10 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add traits check in nested provider cadidates https://review.openstack.org/567150 | 12:10 |
| openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands with sharing providers https://review.openstack.org/567508 | 12:10 |
| *** mgoddard has joined #openstack-nova | 12:11 | |
| *** gyankum has joined #openstack-nova | 12:11 | |
| *** cdent has quit IRC | 12:11 | |
| *** dougsz has joined #openstack-nova | 12:12 | |
| *** nicolasbock has joined #openstack-nova | 12:12 | |
| *** mdbooth has joined #openstack-nova | 12:14 | |
| openstackgerrit | Naichuan Sun proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 12:16 |
| *** sidx64 has joined #openstack-nova | 12:19 | |
| *** Nel1x has joined #openstack-nova | 12:22 | |
| *** abalutoiu has joined #openstack-nova | 12:23 | |
| *** jpena|lunch is now known as jpena | 12:25 | |
| *** Eran_Kuris has quit IRC | 12:26 | |
| gyankum | ping openstack-nova, How I can enable connection_trace for nova in RHOSP8. When I am setting connection_trace=true in nova.conf under Database section and restarting the nova-scheduler via pacemaker, nova-scheduler and nova-conduction services are not coming up. | 12:28 |
| *** lpetrut has joined #openstack-nova | 12:30 | |
| *** alexchadin has joined #openstack-nova | 12:32 | |
| *** abalutoiu_ has joined #openstack-nova | 12:36 | |
| *** amoralej|lunch is now known as amoralej | 12:39 | |
| *** jmccarthy has quit IRC | 12:40 | |
| *** moshele has quit IRC | 12:40 | |
| *** abalutoiu has quit IRC | 12:40 | |
| *** Eran_Kuris has joined #openstack-nova | 12:41 | |
| *** yamamoto has joined #openstack-nova | 12:42 | |
| *** lyan has joined #openstack-nova | 12:42 | |
| *** gyankum has quit IRC | 12:42 | |
| *** pchavva has joined #openstack-nova | 12:43 | |
| *** yamamoto has quit IRC | 12:48 | |
| openstackgerrit | Surya Seetharaman proposed openstack/nova master: Make nova service-list use scatter-gather routine https://review.openstack.org/568271 | 12:54 |
| *** sidx64 has quit IRC | 12:55 | |
| *** sahid has quit IRC | 12:57 | |
| *** sahid has joined #openstack-nova | 12:57 | |
| *** wolverineav has quit IRC | 12:57 | |
| *** jmccarthy has joined #openstack-nova | 12:57 | |
| *** weshay_interview is now known as weshay | 13:01 | |
| *** dklyle has quit IRC | 13:02 | |
| *** dklyle has joined #openstack-nova | 13:03 | |
| *** felipemonteiro__ has joined #openstack-nova | 13:04 | |
| *** yamamoto has joined #openstack-nova | 13:05 | |
| *** dklyle has quit IRC | 13:06 | |
| *** mriedem has joined #openstack-nova | 13:06 | |
| *** alexchadin has quit IRC | 13:09 | |
| *** felipemonteiro_ has joined #openstack-nova | 13:11 | |
| *** dklyle has joined #openstack-nova | 13:12 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961 | 13:14 |
| *** yamamoto has quit IRC | 13:14 | |
| *** felipemonteiro__ has quit IRC | 13:15 | |
| *** mvk has quit IRC | 13:16 | |
| *** dklyle has quit IRC | 13:17 | |
| *** cdent has joined #openstack-nova | 13:18 | |
| efried | finucannot: you around? | 13:18 |
| *** rmart04 has joined #openstack-nova | 13:20 | |
| *** trozet has joined #openstack-nova | 13:20 | |
| *** Nel1x has quit IRC | 13:21 | |
| *** Tom-Tom has joined #openstack-nova | 13:26 | |
| *** tetsuro has joined #openstack-nova | 13:26 | |
| *** felipemonteiro__ has joined #openstack-nova | 13:27 | |
| *** felipemonteiro_ has quit IRC | 13:27 | |
| *** baoli has joined #openstack-nova | 13:30 | |
| *** yamamoto has joined #openstack-nova | 13:31 | |
| *** superdan is now known as dansmith | 13:33 | |
| *** belmorei_ has joined #openstack-nova | 13:33 | |
| *** baoli has quit IRC | 13:34 | |
| *** baoli has joined #openstack-nova | 13:34 | |
| *** esberglu has joined #openstack-nova | 13:35 | |
| *** belmoreira has quit IRC | 13:35 | |
| *** yamamoto has quit IRC | 13:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | 13:41 |
| *** armaan has quit IRC | 13:42 | |
| *** mvk has joined #openstack-nova | 13:42 | |
| *** armaan has joined #openstack-nova | 13:43 | |
| *** armaan has quit IRC | 13:47 | |
| *** armaan has joined #openstack-nova | 13:47 | |
| *** armaan has quit IRC | 13:48 | |
| *** armaan has joined #openstack-nova | 13:49 | |
| *** jaypipes_ is now known as jaypipes | 13:51 | |
| *** yamamoto has joined #openstack-nova | 13:53 | |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 13:54 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 13:54 |
| openstackgerrit | Bence Romsics proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 13:54 |
| *** sar_ has quit IRC | 13:54 | |
| *** vladikr has joined #openstack-nova | 13:55 | |
| *** Eran_Kuris has quit IRC | 13:55 | |
| *** yamamoto has quit IRC | 13:57 | |
| *** jgwentworth is now known as melwitt | 13:57 | |
| melwitt | efried: finucannot is on PTO today | 13:58 |
| efried | melwitt: ack, thx | 13:58 |
| *** takashin has joined #openstack-nova | 13:59 | |
| *** beekneemech is now known as bnemec | 13:59 | |
| *** awaugama has joined #openstack-nova | 14:01 | |
| *** links has quit IRC | 14:03 | |
| *** hongbin has joined #openstack-nova | 14:05 | |
| *** r-daneel has joined #openstack-nova | 14:05 | |
| *** baoli has quit IRC | 14:05 | |
| *** burt has joined #openstack-nova | 14:05 | |
| *** baoli has joined #openstack-nova | 14:06 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add contributor docs on deprecating and removing compute REST APIs https://review.openstack.org/567687 | 14:12 |
| *** baoli has quit IRC | 14:12 | |
| *** yamamoto has joined #openstack-nova | 14:13 | |
| *** baoli has joined #openstack-nova | 14:14 | |
| *** udesale has quit IRC | 14:14 | |
| *** cdent_ has joined #openstack-nova | 14:16 | |
| *** liverpooler has joined #openstack-nova | 14:16 | |
| *** cdent has quit IRC | 14:17 | |
| *** cdent_ is now known as cdent | 14:17 | |
| *** tbachman has joined #openstack-nova | 14:17 | |
| mriedem | alex_xu: can you take a look at this spec again? https://review.openstack.org/#/c/546925/ | 14:19 |
| openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 14:20 |
| alex_xu | mriedem: yea, I will try to review tomorrow | 14:21 |
| mriedem | alex_xu: thanks | 14:21 |
| *** yamamoto has quit IRC | 14:21 | |
| alex_xu | mriedem: np | 14:21 |
| mriedem | mdbooth: are you going to be able to address review comments on this series soon? https://review.openstack.org/#/c/526347/ | 14:21 |
| *** alexchadin has joined #openstack-nova | 14:21 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Transform missing delete notifications https://review.openstack.org/410297 | 14:22 |
| mdbooth | mriedem: I'm about half way through. | 14:22 |
| mdbooth | Probably be a day or so. | 14:22 |
| mdbooth | I was expecting to get bumped to the back of the runway if that's what you're asking. | 14:23 |
| mriedem | mdbooth: ok, yeah | 14:23 |
| *** mlavalle has joined #openstack-nova | 14:23 | |
| *** gbarros has joined #openstack-nova | 14:24 | |
| mriedem | mdbooth: anyway, regardless of the runway, ping me when those 3 or so bottom changes are updated | 14:29 |
| openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Update docs, API samples for noVNC 1.0.0 https://review.openstack.org/550173 | 14:29 |
| mdbooth | mriedem: Will do, thanks. | 14:29 |
| *** do3meli has quit IRC | 14:31 | |
| *** takashin has left #openstack-nova | 14:32 | |
| *** eharney has joined #openstack-nova | 14:33 | |
| *** gyankum has joined #openstack-nova | 14:34 | |
| *** gyankum has quit IRC | 14:34 | |
| *** wolverineav has joined #openstack-nova | 14:37 | |
| *** yamamoto has joined #openstack-nova | 14:38 | |
| *** tetsuro has quit IRC | 14:39 | |
| kashyap | If anyone wants to take a look a simple clean-up: https://review.openstack.org/#/c/567258/ -- "libvirt: Remove usage of migrateToURI2() API" | 14:39 |
| *** wolverineav has quit IRC | 14:39 | |
| *** wolverineav has joined #openstack-nova | 14:39 | |
| *** rajinir has joined #openstack-nova | 14:40 | |
| *** prometheanfire has left #openstack-nova | 14:40 | |
| *** yamamoto has quit IRC | 14:42 | |
| *** gbarros has quit IRC | 14:47 | |
| *** armaan has quit IRC | 14:53 | |
| *** armaan has joined #openstack-nova | 14:53 | |
| *** belmorei_ has quit IRC | 14:54 | |
| *** ratailor has quit IRC | 14:57 | |
| *** yamamoto has joined #openstack-nova | 14:59 | |
| *** gcb has joined #openstack-nova | 15:01 | |
| *** BobBall has joined #openstack-nova | 15:01 | |
| mriedem | lyarwood: when I +2ed https://review.openstack.org/#/c/567899/ i had a false positive on the live migration recreate patch sitting on top of it; i fixed that this morning and now the job is still failing even with your patch | 15:02 |
| mriedem | this is when force_config_drive=true, images_type=raw, config_drive_format=vfat | 15:02 |
| mriedem | vdrok: ^ | 15:02 |
| *** pcaruana has quit IRC | 15:02 | |
| vdrok | hrm | 15:03 |
| mriedem | http://logs.openstack.org/60/567860/6/check/nova-live-migration/702f809/logs/screen-n-cpu.txt.gz?level=TRACE#_May_14_14_39_42_271510 | 15:03 |
| *** yamamoto has quit IRC | 15:03 | |
| vdrok | different error though, that's good :) | 15:03 |
| mriedem | tonyb: kashyap: remember https://bugs.launchpad.net/nova/+bug/1246201 at all and the related libvirt bug? i couldn't actually find a specific libvirt bug/patch in the related libvirt ML threads, but i'm wondering which version of libvirt fixes that so we can deprecate the workaround of using config_drive_format=vfat | 15:05 |
| openstack | Launchpad bug 1246201 in OpenStack Compute (nova) juno "Live migration fails when the instance has a config_drive_format=iso9660" [High,Fix released] - Assigned to Matt Riedemann (mriedem) | 15:05 |
| openstackgerrit | Dan Smith proposed openstack/nova master: Make scheduler client allow multiple member_of query parameters https://review.openstack.org/568313 | 15:05 |
| mriedem | "If your hypervisor driver is libvirt, and you want live migrate to work without shared storage, then use VFAT." | 15:05 |
| kashyap | Hey there | 15:05 |
| kashyap | mriedem: Where are you quoting that from? | 15:06 |
| mriedem | that should probably read, "and you want live migrate to work with a config drive but without shared storage" | 15:06 |
| mriedem | the "config_drive_format" option help text | 15:06 |
| kashyap | mriedem: So, looking at the Launchpad bug, it redirects me to this upstream libvirt bug that I asked to file: | 15:07 |
| kashyap | https://bugzilla.redhat.com/show_bug.cgi?id=1203032 | 15:07 |
| openstack | bugzilla.redhat.com bug 1203032 in libvirt "Enhancing block/disk migration in libvirt" [Low,Closed: nextrelease] - Assigned to pboldin | 15:07 |
| BobBall | hi melwitt - thought I'd join here rather than discussing on the ticket. It seems that tempest is using a flavour with root disk = 0 (flavor ref 42 which is new and called m1.nano) | 15:07 |
| kashyap | mriedem: And the FixedInVersion field says: libvirt-1.2.17 | 15:07 |
| BobBall | melwitt: Because we are requesting a 0-size root disk, it means that we don't try and resize, hence the success | 15:07 |
| mriedem | kashyap: hmm, presumably we can do away with some workaround code then | 15:08 |
| kashyap | mriedem: Yeah, definitely. Can add it to my list if you like | 15:09 |
| kashyap | If you are already on it, won't step on your toes | 15:09 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L7495 | 15:09 |
| melwitt | BobBall: yeah, just saw your reply. that's news to me that tempest uses a root_gb=0 flavor for resizes ... although, is it resizing from a 0 disk to a different 0 disk with different cpu/ram? | 15:09 |
| BobBall | melwitt: Yes - that's right - the resize target appears to be 84 (I think) which is m1.micro: root_gb=0 but more RAM | 15:10 |
| vdrok | mriedem: tho that error happens on spawn, not on migration now | 15:10 |
| mriedem | vdrok: yeah | 15:11 |
| vdrok | yeah we can do iso9660 now i think but we need to pass a new param to libvirt-python migrate method, migrate_disks, specifying cdrom device iiuc | 15:11 |
| mriedem | vdrok: i think kashyap removed that | 15:11 |
| melwitt | BobBall: I see. seems like that would be problematic in tempest in general and miss covering functionality in drivers if it's not doing any disk resize | 15:11 |
| kashyap | mriedem: Just to note my availability, this week until next Monday, I will be "sparsely available" due to a prior committment. From Monday I should be more available. | 15:11 |
| kashyap | (But I will be responsive on IRC, though) | 15:11 |
| kashyap | vdrok: Since Rocky onwards | 15:12 |
| melwitt | BobBall: thanks for tracking that down | 15:12 |
| mriedem | vdrok: i might be thinking of this https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6910 | 15:12 |
| BobBall | melwitt: This appears to be set up in devstack's tempest plugin... So yes, I believe this is a common failing | 15:12 |
| kashyap | ... the libvirt migrate API needs the 'params' dict passed. | 15:12 |
| vdrok | ah so most of the things are in place already | 15:13 |
| kashyap | Yeah | 15:13 |
| kashyap | And note: this in-flight patch will remove completely the old API | 15:14 |
| *** tomtom001 has quit IRC | 15:14 | |
| melwitt | BobBall: ack. I'll look into whether we could do minimal disk resizing in tempest | 15:14 |
| *** dklyle has joined #openstack-nova | 15:14 | |
| *** tomtom001 has joined #openstack-nova | 15:14 | |
| kashyap | vdrok: As it's not needed, and the new one variant satsified by Nova's current min libvirt version is backwards compatible w/ the older variant: | 15:14 |
| kashyap | https://review.openstack.org/#/c/567258/ | 15:14 |
| melwitt | BobBall: the other question I had was, is the new image handler thing the only way to fix resize for xenapi? I didn't get how that's connected | 15:14 |
| *** jangutter has joined #openstack-nova | 15:15 | |
| BobBall | melwitt: The fix for resize is common to all image handlers for XenServer - it's not related to the new image handler and could justifiably be split out into a separate patch. We just kept it here because this is where we were doing the manual testing | 15:15 |
| melwitt | BobBall: okay. I think it would be best to open a bug and fix the breakage separately rather than roll it into the blueprint work | 15:16 |
| BobBall | melwitt: Bah :D I knew you were going to say that :D I'll do it now so it doesn't have to wait for tomorrow with PRC's timezone. | 15:17 |
| *** derekh has quit IRC | 15:17 | |
| melwitt | I'm too predictable | 15:18 |
| kashyap | Unrelated: Does anyone here use "lp-tools" to file Launchpad bugs via command-line? | 15:18 |
| kashyap | (Launch, manage, query, etc.) | 15:18 |
| *** gbarros has joined #openstack-nova | 15:19 | |
| kashyap | Which reminds me ... some projects have moved to Storyboard. One day, probably Nova will too | 15:19 |
| *** FL1SK has quit IRC | 15:19 | |
| melwitt | but yeah, I'd like to get the driver fixed ASAP and not have to wait behind the feature work that's in-progress | 15:19 |
| *** yamamoto has joined #openstack-nova | 15:19 | |
| *** derekh has joined #openstack-nova | 15:20 | |
| *** derekh has quit IRC | 15:20 | |
| *** derekh has joined #openstack-nova | 15:21 | |
| BobBall | melwitt: We were hoping that patch for the feature was literally about to get merged once the tests all pass :) | 15:21 |
| mriedem | vdrok: so we used to use this utility method to call mkfs.vfat https://review.openstack.org/#/c/527510/3/nova/utils.py | 15:22 |
| *** dklyle has quit IRC | 15:22 | |
| *** gcb has quit IRC | 15:22 | |
| mriedem | i don't see anything checking to see if the path exists first or creates the path first | 15:22 |
| mriedem | reading https://linux.die.net/man/8/mkfs.vfat i'm not sure if -C should be specified or not | 15:23 |
| mriedem | http://manpages.ubuntu.com/manpages/xenial/man8/mkfs.vfat.8.html | 15:24 |
| mriedem | unless it's complaining that /opt/stack/data/nova/instances/51858976-6a84-467a-895b-39a22b43b27f/ doesn't exist, but i'm pretty sure it does at this point | 15:25 |
| mriedem | or mkfs.vfat doesn't exist | 15:25 |
| *** psachin has quit IRC | 15:26 | |
| *** yamamoto has quit IRC | 15:26 | |
| mriedem | trying something | 15:26 |
| vdrok | yeah I see 'unable to open <path>' when specifying non-existing path | 15:26 |
| vdrok | I was testing this on pike when privsep stuff did not exist | 15:28 |
| mriedem | i figured it was because we don't install dosfstools in devstack | 15:28 |
| *** jmccarthy1 has joined #openstack-nova | 15:32 | |
| *** jmccarthy1 has left #openstack-nova | 15:32 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test raw image live migration with vfat config drive https://review.openstack.org/567860 | 15:33 |
| *** felipemonteiro__ has quit IRC | 15:34 | |
| openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: define a new image handler to use vdi streaming https://review.openstack.org/486475 | 15:35 |
| openstackgerrit | Bob Ball proposed openstack/nova master: XenAPI: Pass expected return codes to resize2fs https://review.openstack.org/568318 | 15:35 |
| *** jmccarthy has quit IRC | 15:35 | |
| melwitt | BobBall: I just looked at the processutils code and it defaults check_exit_code=[0]. I wonder if we should just do that in nova/privsep/fs.py instead of making callers pass [0] if they need the default. seems like we could hit this again easily with people thinking they could pass None if they don't care to check the exit codes more specifically | 15:37 |
| BobBall | melwitt: resize2fs takes a check_exit_code - we could indeed put the default there, or decide not to pass it on if None was provided? Or perhaps default to True? | 15:39 |
| BobBall | (IMO check_exit_code=True makes more sense than check_exit_code=[0] | 15:39 |
| melwitt | BobBall: hang on, sorry | 15:39 |
| melwitt | I'm seeing there's a default check_exit_code=False in resize2fs in nova/virt/disk/api.py and I wonder why the code wasn't written to call that instead of calling the privsep module methods directly | 15:41 |
| *** ragiman has quit IRC | 15:41 | |
| *** ragiman has joined #openstack-nova | 15:42 | |
| *** yamamoto has joined #openstack-nova | 15:42 | |
| BobBall | melwitt: I believe because we don't want to run the e2fsck twice? | 15:43 |
| *** cdent has quit IRC | 15:43 | |
| melwitt | BobBall: yeah, and it can't accept the flags kwarg | 15:43 |
| *** jpena is now known as jpena|brb | 15:43 | |
| BobBall | Ah - yes. I hadn't spotted that bit :) | 15:43 |
| *** alexchadin has quit IRC | 15:44 | |
| BobBall | The second call could more easily use the general resize2fs although that would run an unwanted e2fsck again | 15:44 |
| BobBall | and on large disks that can take 10s of minutes... | 15:44 |
| melwitt | no, it's okay, I'm just now realizing that virt/api.py method is for ... someone else apparently | 15:45 |
| *** yamamoto has quit IRC | 15:47 | |
| *** andreas_s has quit IRC | 15:48 | |
| *** lpetrut has quit IRC | 15:49 | |
| *** nicolasbock has quit IRC | 15:50 | |
| BobBall | Joyfulness... | 15:51 |
| melwitt | yeah :| | 15:52 |
| BobBall | How about we just pass check_exit_code=True then? A good enough improvement over check_exit_code=[0]? | 15:52 |
| cfriesen | so we just hit https://bugs.launchpad.net/nova/+bug/1659062 in production, and it left the instance in a state where "reset-state" and "rebuild" didn't work to recover it. | 15:52 |
| openstack | Launchpad bug 1659062 in OpenStack Compute (nova) "Failed evacuations leave neutron ports on destination host" [Wishlist,Opinion] | 15:52 |
| *** nicolasbock has joined #openstack-nova | 15:53 | |
| cfriesen | anyone have any ideas on the best way to handle this? fixups in init_instance() on the source? | 15:53 |
| melwitt | BobBall: we could. I was just looking at the old pre-privsep code and it used to be that you could omit check_exit_code and get the default and I think that would make things easier for people, but that can be a separate improvement | 15:54 |
| BobBall | melwitt: I'm happy to make the default for fs.resize2fs be check_exit_code=True and remove it from the XenAPI code if that's the preferred fix. | 15:55 |
| BobBall | melwitt: As it's currently non-optional, all callers must be passing a value therefore having the default set to True won't hurt anyone | 15:56 |
| *** ragiman has quit IRC | 15:56 | |
| melwitt | BobBall: yeah, that's true. others might disagree and think the current way of "everybody has to pass it" is better, so I'm thinking not to add any potential debate to your fix | 15:58 |
| BobBall | melwitt: OK - so do you now think we should leave the fix as-is for now? (i.e. passing [0]?) | 15:59 |
| melwitt | yeah, I think that's fine | 15:59 |
| *** wolverineav has quit IRC | 16:00 | |
| *** andreas_s has joined #openstack-nova | 16:00 | |
| BobBall | +1 | 16:02 |
| *** mgoddard has quit IRC | 16:03 | |
| melwitt | mriedem: simple fix for xenapi driver regression https://review.openstack.org/568318 tldr: resize broke from the privsep changes but tempest didn't catch it because tempest tests resize with a 0 disk flavor => other 0 disk flavor and xenapi driver skips disk resize in that case | 16:04 |
| *** ccamacho has quit IRC | 16:05 | |
| *** andreas_s has quit IRC | 16:06 | |
| *** janki has quit IRC | 16:08 | |
| mriedem | done | 16:09 |
| *** lucasagomes is now known as lucas-afk | 16:10 | |
| melwitt | thanks | 16:10 |
| *** gyee has joined #openstack-nova | 16:13 | |
| *** sahid has quit IRC | 16:13 | |
| openstackgerrit | Eric Fried proposed openstack/nova master: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | 16:14 |
| *** vivsoni has quit IRC | 16:18 | |
| *** pcaruana has joined #openstack-nova | 16:20 | |
| *** dave-mccowan has joined #openstack-nova | 16:23 | |
| *** jpena|brb is now known as jpena | 16:24 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement granular policy rules for placement https://review.openstack.org/524425 | 16:27 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add granular policy rules for /resource_classes* https://review.openstack.org/565578 | 16:27 |
| *** armaan has quit IRC | 16:27 | |
| *** armaan has joined #openstack-nova | 16:28 | |
| *** dave-mccowan has quit IRC | 16:28 | |
| *** jmlowe has quit IRC | 16:29 | |
| *** dave-mccowan has joined #openstack-nova | 16:30 | |
| *** wolverineav has joined #openstack-nova | 16:31 | |
| *** felipemonteiro has joined #openstack-nova | 16:32 | |
| *** rmart04 has quit IRC | 16:33 | |
| *** damien_r has quit IRC | 16:33 | |
| *** lpetrut has joined #openstack-nova | 16:36 | |
| *** sambetts is now known as sambetts|afk | 16:36 | |
| *** jmccarthy has joined #openstack-nova | 16:37 | |
| mriedem | vdrok: it was the missing dosfstools | 16:38 |
| mriedem | http://logs.openstack.org/60/567860/7/check/nova-live-migration/09d27ca/ passes now | 16:38 |
| *** sar_ has joined #openstack-nova | 16:39 | |
| *** wolverineav has quit IRC | 16:40 | |
| mriedem | lyarwood: ^ so you can ignore my earlier ping, +2 on your fix again | 16:42 |
| *** zcorneli has joined #openstack-nova | 16:46 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: add lower-constraints job https://review.openstack.org/555961 | 16:47 |
| *** wolverineav has joined #openstack-nova | 16:50 | |
| *** itlinux has joined #openstack-nova | 16:57 | |
| *** dougsz has quit IRC | 16:57 | |
| *** AlexeyAbashkin has quit IRC | 16:57 | |
| *** pchavva has quit IRC | 16:58 | |
| *** derekh has quit IRC | 17:00 | |
| *** hemna_ has joined #openstack-nova | 17:07 | |
| openstackgerrit | Sylvain Bauza proposed openstack/nova-specs master: Proposes Multiple GPU types https://review.openstack.org/557065 | 17:07 |
| *** hemna_ has quit IRC | 17:07 | |
| bauzas | jaypipes: dansmith: you reviewed https://review.openstack.org/557065 here is a new revision, but I also provided an implementation change with https://review.openstack.org/#/c/564214/ | 17:07 |
| *** mdnadeem has quit IRC | 17:07 | |
| *** hemna_ has joined #openstack-nova | 17:07 | |
| bauzas | jaypipes: dansmith: do we really need a spec for that ? | 17:07 |
| bauzas | for sure, it's a new conf opt, but no upgrade change | 17:08 |
| dansmith | bauzas: you're asking if you need a spec that you have a spec for? | 17:08 |
| dansmith | seems like a reasonable thing to have a spec for, imho | 17:09 |
| bauzas | dansmith: I meant that given we already discussed alot about the possibilities, now we have the implementation change, looks like it's just an implementation point | 17:09 |
| bauzas | but anyway | 17:09 |
| bauzas | the spec is there for folks | 17:10 |
| bauzas | tbh, the most important for me is https://review.openstack.org/#/c/564214/ | 17:10 |
| *** tesseract has quit IRC | 17:11 | |
| *** sapd1 has joined #openstack-nova | 17:12 | |
| *** dtantsur is now known as dtantsur|afk | 17:14 | |
| *** efried has quit IRC | 17:19 | |
| *** efried has joined #openstack-nova | 17:19 | |
| *** jpena is now known as jpena|off | 17:19 | |
| *** baoli has quit IRC | 17:21 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 17:25 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 17:25 |
| *** baoli has joined #openstack-nova | 17:26 | |
| *** vivsoni__ has joined #openstack-nova | 17:27 | |
| *** yamamoto has joined #openstack-nova | 17:28 | |
| mnaser | can i please get eyes on https://review.openstack.org/#/c/566425 ? | 17:30 |
| *** imacdonn has quit IRC | 17:30 | |
| *** imacdonn has joined #openstack-nova | 17:30 | |
| *** FL1SK has joined #openstack-nova | 17:30 | |
| *** yamamoto has quit IRC | 17:32 | |
| *** germs has joined #openstack-nova | 17:33 | |
| openstackgerrit | Dan Smith proposed openstack/nova master: Honor availability_zone hint via placement https://review.openstack.org/546282 | 17:39 |
| *** Tom-Tom has quit IRC | 17:41 | |
| *** Tom-Tom has joined #openstack-nova | 17:41 | |
| mnaser | dansmith, efried, mriedem: thank you, also i like how dansmith and mriedem came up to the same exact thoguht process | 17:42 |
| dansmith | heh | 17:43 |
| mriedem | we finish each other's | 17:43 |
| dansmith | sentences. | 17:44 |
| efried | you guys are gross | 17:44 |
| dansmith | hah | 17:44 |
| dansmith | we're both planning to get pre-summit haircuts on wednesday | 17:44 |
| mriedem | mine is tomorrow | 17:45 |
| dansmith | aww | 17:45 |
| *** fragatina has joined #openstack-nova | 17:45 | |
| * dansmith calls to reschedule | 17:45 | |
| *** vivsoni__ has quit IRC | 17:46 | |
| *** fragatina has quit IRC | 17:46 | |
| *** germs has quit IRC | 17:46 | |
| *** fragatina has joined #openstack-nova | 17:47 | |
| *** fragatina has quit IRC | 17:47 | |
| *** cdent has joined #openstack-nova | 17:48 | |
| *** fragatina has joined #openstack-nova | 17:48 | |
| *** tssurya has quit IRC | 17:48 | |
| *** yamamoto has joined #openstack-nova | 17:49 | |
| *** yamamoto has quit IRC | 17:53 | |
| mnaser | i cant be the only one who constantly types 'reno add', get an error then 'reno new' | 17:54 |
| *** jmccarthy has left #openstack-nova | 17:55 | |
| *** jmlowe has joined #openstack-nova | 17:56 | |
| *** pchavva has joined #openstack-nova | 17:57 | |
| mriedem | tox -e venv -- reno new <slug> | 17:58 |
| *** amoralej is now known as amoralej|off | 17:59 | |
| *** r-daneel_ has joined #openstack-nova | 18:00 | |
| *** r-daneel has quit IRC | 18:01 | |
| *** r-daneel_ is now known as r-daneel | 18:01 | |
| mriedem | dansmith: https://review.openstack.org/#/c/567899/ fixes a regression that was backported through to ocata so we should probably get that fix in and backported, | 18:01 |
| mriedem | i had a recreate with our live migration job and the patch on top shows this fixes it | 18:01 |
| zcorneli | dansmith: For file backed memory, any new thoughts on how to handle the old->new migration issue? | 18:04 |
| *** ircuser-1 has joined #openstack-nova | 18:05 | |
| openstackgerrit | Merged openstack/nova master: Suppress UUID warning in map_instance unit tests https://review.openstack.org/568263 | 18:07 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Added ability to configure default architecture for ImagePropertiesFilter https://review.openstack.org/566425 | 18:07 |
| dansmith | mriedem: ack | 18:08 |
| dansmith | zcorneli: remind me, the new xml is calculated on the sending node, to be used by the destination node right? | 18:08 |
| zcorneli | dansmith: That's what I've seen, and how the code seems to expect it. | 18:09 |
| zcorneli | dansmith: Haven't seen any XML modification on the destination yet. | 18:09 |
| dansmith | zcorneli: yeah, so, if that's the case, then an older node migrating to a newer one may break things by sending it a non-file-backed xml, when the node is configured to use file-backed | 18:10 |
| dansmith | because it's ignorant | 18:10 |
| dansmith | zcorneli: so, if there's a way for the new node to reject the migration gracefully if it detects that situation, that'd be ideal | 18:10 |
| dansmith | zcorneli: so on the destination, some way to check that the source node is new enough to be honoring this.. maybe with the flag in the migratedata object, if that gets passed to the destination from the source at that point? | 18:12 |
| zcorneli | dansmith: Artom proposed rejecting the migration (He's working on something similar with NUMA?), but you mentioned that's not necessarily nice on the scheduler. | 18:12 |
| *** harlowja has joined #openstack-nova | 18:12 | |
| zcorneli | I can check with him to see how he's checking for it, and use something similar (No point to reinventing the wheel) | 18:12 |
| dansmith | zcorneli: we need to avoid banning cross-version migrations altogether | 18:12 |
| dansmith | zcorneli: it'd be nice if the scheduler could know about it, but I think it's probably too much for the first go-round | 18:13 |
| zcorneli | dansmith: I think we can just ban it when it would break things. Just need to do some investigation on how the migration_data is passed around, etc. | 18:13 |
| dansmith | zcorneli: yeah, so I think maybe it's okay to just reject the migration on the destination if that happens, and then make a big stink in the release note that you probably shouldn't turn that on for any new nodes until everything is upgraded | 18:14 |
| *** mdbooth has quit IRC | 18:15 | |
| zcorneli | dansmith: Works for me. Maybe a note in the config entry too. | 18:16 |
| dansmith | zcorneli: yeah, also put this wrinkle in the spec | 18:17 |
| *** mvk has quit IRC | 18:22 | |
| *** sridharg has quit IRC | 18:25 | |
| *** damien_r has joined #openstack-nova | 18:25 | |
| *** idlemind has joined #openstack-nova | 18:27 | |
| *** hamzy has quit IRC | 18:30 | |
| *** hamzy has joined #openstack-nova | 18:33 | |
| *** damien_r has quit IRC | 18:36 | |
| *** wolverineav has quit IRC | 18:36 | |
| mriedem | dansmith: the source node modifies the xml to send to the dest | 18:37 |
| mriedem | based on the migrate_data sent back from the dest during pre_live_migration | 18:37 |
| dansmith | yeah | 18:37 |
| dansmith | and the migratedata is created for the first time on the dest right? | 18:38 |
| mriedem | yes | 18:38 |
| mriedem | in check_dest_can_live_migrate.... something like that | 18:38 |
| dansmith | yeah, so that'll make it hard for a new dest to know it should fail | 18:38 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6444 | 18:38 |
| mriedem | conductor will retry hosts up to some live migration max retry limit (defaults to unlimited), | 18:39 |
| dansmith | yeah, but we have to have a way for the destination to know it needs to fail | 18:39 |
| mriedem | and conductor is probably not an appropriate place for this since it's libvirt-driver specific | 18:40 |
| *** mgoddard has joined #openstack-nova | 18:40 | |
| mriedem | if we expressed file-backed as a driver capability, it could be a trait on the compute node and we could schedule using a placement request filter | 18:40 |
| *** Guest16323 is now known as mgagne | 18:40 | |
| *** mgagne has joined #openstack-nova | 18:40 | |
| mriedem | is there a spec for this? | 18:41 |
| dansmith | mriedem: yeah, that was my first thing on the "make the scheduler know" but then I decided it was maybe too much | 18:41 |
| dansmith | mriedem: yeah | 18:41 |
| dansmith | https://review.openstack.org/#/c/563704/ | 18:41 |
| dansmith | mriedem: artom has this problem with the numa stuff too | 18:41 |
| mriedem | yes and i do too for the port binding stuff | 18:41 |
| dansmith | ah, okay | 18:42 |
| mriedem | https://review.openstack.org/#/c/522537/ is my conductor change btw | 18:42 |
| mriedem | which i told artom to base his numa stuff from | 18:42 |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Added ability to configure default architecture for ImagePropertiesFilter https://review.openstack.org/566425 | 18:42 |
| *** markvoelker_ has joined #openstack-nova | 18:42 | |
| mnaser | ^ that should be ready to review again if i didn't break something | 18:42 |
| dansmith | mriedem: so, the cap/trait thing is maybe a little harder than you describe, because we would have to look at the instance's existing host, see if it had that trait, and then ask for hosts with that | 18:42 |
| dansmith | you can't tell just from the instance config or anything | 18:42 |
| mnaser | passed pep8 locally | 18:43 |
| dansmith | mriedem: but if we have three such examples already then maybe it makes sense to go ahead with the scheduler-based approach | 18:43 |
| *** markvoelker has quit IRC | 18:43 | |
| dansmith | mriedem: although we still need to fail in all cases if you force the migration and skip the scheduler, but I guess a very late and painful failure on the destination is reasonable if you turned off all the safeguards anyway | 18:43 |
| mriedem | yes i'm fine with kicking people in the balls if they force the host and bypass the scheduler | 18:44 |
| dansmith | mriedem: er, actually the trait thing is harder | 18:44 |
| dansmith | mriedem: the problem is if you're moving from an older node to a newer one, | 18:44 |
| mriedem | in my port binding case, i'm lucky i guess in that we don't rely on the dest to do stuff pre-live-migration, we handle setting up the dest host port bindings in conductor | 18:44 |
| dansmith | so basically we'd have to know if they're on a host that can't do it, and then ask for other hosts that can't do it | 18:44 |
| mriedem | dansmith: you could use forbidden traits for that | 18:45 |
| dansmith | mriedem: yeah I know, but but it means we have to expose two traits (eventually): can-do-this and is-configured-for-this | 18:45 |
| dansmith | and then schedule for !can-do-this if they're on a host without can-do-this | 18:45 |
| dansmith | because people will want to schedule on is-configured-for-this too, like a cheaper flavor that gets file-backed memory | 18:46 |
| mriedem | can't that just be a single trait? | 18:46 |
| dansmith | no | 18:46 |
| dansmith | because a host is either configured to do this or not, it can't reasonably have instances with both right now | 18:46 |
| dansmith | otherwise we have to expose two pools of memory in placement for a host | 18:46 |
| mriedem | right, that's the file_backed_memory option right? | 18:47 |
| mriedem | it's a boolean | 18:47 |
| dansmith | that's the "is this enabled" flag yeah | 18:47 |
| dansmith | but for migration, | 18:47 |
| mriedem | so if file_backed_memory is True for a given compute service, we report that it has the file-backed-memory trait | 18:47 |
| dansmith | right, but that's not what we need for live migration | 18:47 |
| dansmith | you can migrate to/from a host with file-backed even if you don't, but only if you are new enough to *know* about it and tweak the new xml properly | 18:48 |
| mriedem | not sure i'm following; if the instance is on a source host with the file-backed-memory trait, we need to find a dest host with that same trait set right? | 18:48 |
| dansmith | no | 18:48 |
| dansmith | you can migrate from file-backed to non-file-backed, and vice versa | 18:48 |
| dansmith | but you can't migrate from an older node that doesn't know anything about this, | 18:48 |
| *** mgoddard has quit IRC | 18:48 | |
| dansmith | because it will generate non-filed-backed libvirt xml and send it to the dest, forcing it to be memory-backed even if that host is configured for file-backed | 18:48 |
| *** dklyle has joined #openstack-nova | 18:49 | |
| dansmith | it's really kindof a dumb thing about how we do live migrations | 18:49 |
| dansmith | we foist our desired xml on the destination even if it violates something about how it is configured | 18:49 |
| *** markvoelker has joined #openstack-nova | 18:49 | |
| dansmith | I can explain this quickly in a hangout if it'd be easier | 18:50 |
| mriedem | if the instance is on an old source compute that doesn't report the file-backed trait, i'm not sure why we wouldn't just say filter out all dest hosts that also don't have that trait; i understand you can go back and forth if the computes are new enough to know if they have this or not, so maybe we need a compute rpc version check in here too? | 18:50 |
| *** markvoelker_ has quit IRC | 18:50 | |
| jaypipes | bauzas: k, reviewed the vgpu type spec. no vote. just asked a question. | 18:50 |
| jaypipes | bauzas: the answer to that question is very important for me to know. | 18:51 |
| dansmith | mriedem: but that's the can-do-it flag, if we filter out hosts without that trait | 18:51 |
| dansmith | mriedem: we could do a service version for this if you want I guss | 18:52 |
| dansmith | feels a little strange for a virt-specific thing, | 18:52 |
| *** mvk has joined #openstack-nova | 18:52 | |
| dansmith | and rpc version bump is less useful if we're not actually adding a parameter for something | 18:52 |
| mriedem | yes it does, and i'm also not sure if/how it would make sense here | 18:53 |
| dansmith | we could do one of two things: | 18:53 |
| mriedem | i guess we can use a service version check to know if the source compute is too old to make a reliable guess about the trait | 18:53 |
| dansmith | 1. Not let you enable file-backed if the minimum service version says you have older computes (which sucks) | 18:53 |
| dansmith | 2. Have the destination look at the service version of the sending node and, if it's older and we're configured for file-backed, reject during pre-check | 18:54 |
| mriedem | with (2) we still have to build in a trait placement req filter right? | 18:55 |
| dansmith | no | 18:55 |
| dansmith | it would just mean we force the scheduler to burn a retry on it | 18:55 |
| mriedem | i guess we dont' have to, it would make scheduling more efficient | 18:55 |
| dansmith | so we could do it, or do without | 18:55 |
| dansmith | yeah | 18:55 |
| dansmith | I still think we should have two traits though | 18:55 |
| dansmith | "can do this" and "is doing this currently" | 18:55 |
| dansmith | well | 18:56 |
| dansmith | er, yeah.. we want the first for smart scheduling on live migrations, and the second for normal flavor-implies-slower-cheaper-hardware | 18:56 |
| mriedem | i figured "Have the destination look at the service version of the sending node and, if it's older and we're configured for file-backed, reject during pre-check" was for "can do this" | 18:58 |
| *** r-daneel has quit IRC | 18:59 | |
| *** r-daneel_ has joined #openstack-nova | 18:59 | |
| dansmith | that is the safety net, if the scheduler is ignorant, or if the operator does a force | 18:59 |
| dansmith | we have to have that | 18:59 |
| mriedem | and the file-backed trait was for "can do it and is configured to do it" | 18:59 |
| dansmith | if we have a can-do-this trait, then we can make the scheduler less ignorant | 18:59 |
| mriedem | yeah i know...but trait explosion | 18:59 |
| dansmith | actually | 18:59 |
| dansmith | I guess after rocky, we don't need the can-do-this trait | 19:00 |
| mriedem | right | 19:00 |
| dansmith | so maybe better to just reno it, put the safety check in because we need it | 19:00 |
| dansmith | yeah, okay | 19:00 |
| dansmith | sorry | 19:00 |
| mriedem | so i think you just need a single trait and the safety valve | 19:00 |
| dansmith | yeah | 19:00 |
| dansmith | zcorneli: you following this? | 19:00 |
| zcorneli | dansmith: Just getting back from lunch, reading through all of it now. | 19:00 |
| dansmith | zcorneli: okay, let me summarize: | 19:01 |
| dansmith | zcorneli: every node has a service record, with a version on it, from a globally monotonic version (objects.service.SERVICE_VERSION) | 19:01 |
| *** r-daneel_ is now known as r-daneel | 19:01 | |
| dansmith | zcorneli: the destination node can look up the service record for the sending node, see if it's new enough to be educated about file-backed memory | 19:01 |
| dansmith | zcorneli: if it's not, and the destination is configured for file-backed, then abort the migration | 19:02 |
| dansmith | zcorneli: the rest of the chatter is mostly for later stuff after the initial implementation | 19:02 |
| *** eharney has quit IRC | 19:03 | |
| zcorneli | dansmith: That seems reasonable to me. Is that lookup something that would be reasonable to use from the pre_live_migration method? | 19:05 |
| dansmith | zcorneli: it's a little unconventional, but reasonable yeah | 19:05 |
| dansmith | zcorneli: and you and artom both need to do it for your respective things, so ... it'll be conventional shortly ;) | 19:05 |
| dansmith | zcorneli: https://github.com/openstack/nova/blob/master/nova/objects/service.py#L325-L325 | 19:06 |
| dansmith | zcorneli: srv = objects.Service.get_by_compute_host(ctxt, instance.host); assert(srv.version >= 31) | 19:07 |
| openstackgerrit | Eric Fried proposed openstack/nova master: __str__ methods for RequestGroup, ResourceRequest https://review.openstack.org/568353 | 19:07 |
| dansmith | zcorneli: at the top of that file, bump SERVICE_VERSION and add an entry to the log | 19:07 |
| mriedem | zcorneli: not pre_live_migration, | 19:08 |
| mriedem | check_can_live_migrate_destination | 19:08 |
| mriedem | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L6444 | 19:08 |
| dansmith | er, yeah | 19:08 |
| mriedem | src_compute_info is a ComputeNode object (i think) that will have the host and node name to lookup the service versoin | 19:09 |
| dansmith | yeah, either way | 19:09 |
| mriedem | oh right instance.host/node would also have that, right | 19:10 |
| dansmith | and, compute host == service, we don't need to care about the node | 19:10 |
| *** sapd1 has quit IRC | 19:10 | |
| mriedem | sure for libvirt we don't | 19:10 |
| dansmith | we wouldn't for ironic either since the service version is on the compute host, but anyway | 19:10 |
| mriedem | true | 19:11 |
| mriedem | zcorneli: also, this is why supporting rolling upgrades is hard, in case you were wondering | 19:11 |
| mriedem | :) | 19:11 |
| dansmith | heh | 19:11 |
| zcorneli | mriedem: Yea, can definitely see that. | 19:11 |
| *** jmlowe has quit IRC | 19:13 | |
| zcorneli | dansmith: mriedem: If I'm following this right, I'll increment the service version as part of the patch, and check within check_can_live_migrate_destination that if file_backed_memory is enabled, and the source service version is < the new service version, fail. | 19:14 |
| dansmith | yup | 19:14 |
| zcorneli | Sounds pretty reasonable. Seems like a good way to handle things. | 19:15 |
| mriedem | mnaser: your patch is bombing out on unit tests hard | 19:15 |
| mnaser | mriedem: oh thats weird i didnt really touch that | 19:15 |
| mriedem | http://logs.openstack.org/25/566425/3/check/openstack-tox-py27/f1bc5f0/job-output.txt.gz#_2018-05-14_18_53_55_549849 | 19:15 |
| *** yamamoto has joined #openstack-nova | 19:15 | |
| dansmith | mnaser: not just failing, bombing.. not just bombing, but bombing *hard*. | 19:15 |
| mnaser | i had a feeling | 19:15 |
| mnaser | that was gonna happen | 19:15 |
| mnaser | import nova.objects.fields inside conf stuff | 19:16 |
| mriedem | the nova.objects.fields import in nova.conf.scheduler is killing the test discovery | 19:16 |
| mriedem | circular import somewhere probably | 19:16 |
| mnaser | yeah.. | 19:16 |
| dansmith | since config is so early, | 19:16 |
| dansmith | you might have to move those arch definitions somewhere neutral | 19:17 |
| dansmith | like Switzerland | 19:17 |
| mnaser | yeah that's what i was worried about | 19:17 |
| dansmith | or virt/hardware | 19:17 |
| mnaser | lol | 19:17 |
| mriedem | nazi arch list? | 19:17 |
| dansmith | mnaser: still, very worthwhile to get the choices= coverage I think | 19:17 |
| mnaser | who's okay with approving a change doing that in the same patch | 19:17 |
| * mnaser waits for inevitable silence | 19:17 | |
| dansmith | I would do it separately but I don't care that much | 19:18 |
| mnaser | nova/virt/hardware imports conf too | 19:18 |
| dansmith | virt/arch.py then | 19:18 |
| mnaser | it should probably be its own patch because i'll probably have to change all the references | 19:18 |
| dansmith | mnaser: you get paid by the patch so two patches is more better right? | 19:19 |
| dansmith | mnaser: If I was my own boss, I'd make up all kinds of silly metrics upon which to base my compensation | 19:20 |
| mnaser | dansmith, mriedem: ideally i'd like to have this to be backportable to queens at least... because it prevents us from multiarch.. how about i do a patch without choices= and then follow up that moves stuff to virt/arch then adding the choices in a third patch, allowing us to have 1 backportable patch that doesnt break the world? | 19:20 |
| *** yamamoto has quit IRC | 19:20 | |
| mnaser | dansmith: but i have to be in the top 10 in stackalytics to get my bonus | 19:20 |
| dansmith | mnaser: I'd rather it be one patch than go in (and especially be backported) without choices= | 19:20 |
| mnaser | (i honestly had a candidate that said they couldn't change their email because "their marketing department at old company" needed to show off cores upstream) | 19:21 |
| dansmith | if you have a config you backport that has no restrictions and then introduce a restriction, you broke compat | 19:21 |
| mnaser | as you can imagine that was eye opening.. | 19:21 |
| mnaser | dansmith: ok i see, i guess i'll throw the dice and hope for mercy by the stable maint gods | 19:22 |
| dansmith | mnaser: mriedem is a stable maint god | 19:23 |
| dansmith | and I'm a stable maint peasant | 19:23 |
| dansmith | so do whatever he says, but I think he'll be on the same page :) | 19:23 |
| bauzas | jaypipes: around ? | 19:24 |
| * dansmith -> lunch | 19:24 | |
| bauzas | jaypipes: about https://review.openstack.org/#/c/557065/ | 19:25 |
| bauzas | good question ^ | 19:25 |
| mriedem | mnaser: agree with dan, single change with the config option added with choices is best, since it defaults to None it should be ok | 19:25 |
| *** mgoddard has joined #openstack-nova | 19:25 | |
| * artom tries to catch up on the live migration convo | 19:26 | |
| jaypipes | bauzas: yes? | 19:27 |
| artom | I think with NUMA it's a bit different - in the sense that an instance can switch from file-backed to non-file backed, and the user might never now (latencies notwithstanding) | 19:27 |
| bauzas | jaypipes: so, about your point, maybe we should have some aliases for PCI devices ? | 19:27 |
| bauzas | like we did for PCI passthrought ? | 19:28 |
| artom | For NUMA... an instance *has* a NUMA topology, and we can't get rid of it, so if both source and dest can't handle live migrating the instance NUMA topology, so to speak, it's fair to bail out | 19:29 |
| bauzas | jaypipes: and then the compute would pass custom traits using those aliases ? | 19:29 |
| bauzas | like CUSTOM_GPU_M608Q | 19:29 |
| bauzas | mriedem: heh, btw. I just paid for a Huawei P20 Pro. Hope you'll get some money thanks to me :p | 19:32 |
| *** gbarros has quit IRC | 19:33 | |
| *** jmlowe has joined #openstack-nova | 19:34 | |
| *** gbarros has joined #openstack-nova | 19:34 | |
| zcorneli | dansmith: mriedem: Looking at check_can_live_migrate_destination, there doesn't look like anything in there actually fails out a migration, just collects data for the source to decide on. | 19:34 |
| jaypipes | bauzas: why? | 19:35 |
| jaypipes | bauzas: I mean, why would we create such a custom trait? | 19:36 |
| *** dave-mccowan has quit IRC | 19:36 | |
| jaypipes | bauzas: it doesn't mean anything. | 19:36 |
| jaypipes | bauzas: it's just a vendor ID. | 19:36 |
| mriedem | bauzas: i won't hold my breath | 19:36 |
| jaypipes | bauzas: we should be putting the standard GPU traits in the flavor | 19:36 |
| bauzas | jaypipes: maybe, I'm not specifically torn on that | 19:37 |
| mriedem | zcorneli: you have to raise MigrationPreCheckError | 19:38 |
| bauzas | jaypipes: tbc, the spec is for passing multiple vGPU types, but not on *asking* for a specific type | 19:38 |
| *** belmoreira has joined #openstack-nova | 19:38 | |
| artom | Unrelated logging question - if a method isn't passed a context, will LOG.debug still print the request ID? | 19:38 |
| mriedem | zcorneli: this is an rpc call to the dest compute https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L222 | 19:38 |
| bauzas | jaypipes: for that, I'd love to see Xen folks | 19:38 |
| *** felipemonteiro_ has joined #openstack-nova | 19:38 | |
| artom | I thought it did some black magic with the context, so if you're not passing it to a method, any LOG calls from within that method won't have the request ID | 19:38 |
| mriedem | zcorneli: if we fail with MigrationPreCheckError we'll handle it here https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L337 and continue looking for another valid destination host | 19:39 |
| bauzas | jaypipes: my point is that, for the moment, operators wanting to provide flavors for a specific type could just use placement-client and use custom traits, right? | 19:39 |
| mriedem | artom: yes | 19:39 |
| mriedem | artom: the context is stored in memory | 19:39 |
| mriedem | which causes all sorts of confusion wrt the request id | 19:39 |
| bauzas | jaypipes: so, the question you have is more about how nova could use standard traits, then | 19:39 |
| jaypipes | bauzas: why would they do that, though? | 19:39 |
| bauzas | jaypipes: why what ? using custom traits ? | 19:40 |
| mriedem | artom: https://github.com/openstack/oslo.context/blob/master/oslo_context/context.py#L40 | 19:40 |
| bauzas | jaypipes: because custom traits already exist | 19:40 |
| jaypipes | bauzas: the user wants a vGPU that has certain capabilities (resolution, max display heads, support for OpenCL, etc) | 19:40 |
| bauzas | jaypipes: sure, I understand your point | 19:40 |
| jaypipes | bauzas: the user doesn't want a specific M60-8Q Nvidia vGPU "type". | 19:40 |
| artom | mriedem, so there's a global context per thread? | 19:40 |
| bauzas | jaypipes: I don't disagree with that :) | 19:40 |
| bauzas | jaypipes: tbh, I'm fine with that | 19:41 |
| *** itlinux has quit IRC | 19:41 | |
| jaypipes | bauzas: when we discussed this with the Xen folks on the original VGPU spec, jianghuaw_ and I specifically agreed that vgpu "types" (which are nothing more than vendor jargon) would *not* be used as traits. | 19:41 |
| bauzas | jaypipes: my plan was about passing aliases, but for sure, we could have other conf opts for saying : "this trait = that PCI id" | 19:41 |
| bauzas | jaypipes: but like I said in my gerrit reply, I don't think it's related to the spec I wrote, maybe in a separate spec ? | 19:42 |
| zcorneli | mriedem: Ah, ok, needed to look up one more layer. Thanks! | 19:42 |
| bauzas | or we could opt something like "this PCI id = those traits" | 19:42 |
| *** felipemonteiro has quit IRC | 19:42 | |
| bauzas | the either | 19:42 |
| mriedem | artom: i just know that request id gets logged automagically | 19:44 |
| mriedem | and periodic tasks can bleed to use the in-memory user request context which makes logging confusing at times | 19:44 |
| mriedem | artom: see https://review.openstack.org/#/c/524306/2 | 19:45 |
| artom | mriedem, OK, thanks :) | 19:45 |
| openstackgerrit | Zack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory https://review.openstack.org/563704 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI for traits (v1.6) https://review.openstack.org/514643 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Resource class set (v1.7) https://review.openstack.org/514644 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Usages per project and user (v1.8, v1.9) https://review.openstack.org/514646 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: CLI allocation candidates (v1.10) https://review.openstack.org/514647 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: New dict format of allocations (v1.11, v1.12) https://review.openstack.org/542819 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Transactionally update allocations (v1.13) https://review.openstack.org/546674 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Add nested resource providers (v1.14) https://review.openstack.org/546675 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Limit allocation candidates (v1.15, v1.16) https://review.openstack.org/548043 | 19:53 |
| openstackgerrit | Matt Riedemann proposed openstack/osc-placement master: Allocation candidates parameter: required (v1.17) https://review.openstack.org/548326 | 19:53 |
| *** dave-mccowan has joined #openstack-nova | 19:53 | |
| *** awaugama has quit IRC | 19:55 | |
| *** yamamoto has joined #openstack-nova | 19:57 | |
| mnaser | mriedem, dansmith: sorry for bugging you but i just wanna get this right.. https://github.com/openstack/nova/blob/master/nova/objects/fields.py#L96 Architecture inherits BaseNovaEnum and if i import BaseNovaEnum inside a seperate module (say, nova.virt.arch) then inheriting/importing that brings me back to the same problem | 19:58 |
| dansmith | mnaser: don't move the field, | 19:58 |
| dansmith | just move ALL | 19:58 |
| openstackgerrit | Eric Fried proposed openstack/nova master: __str__ methods for RequestGroup, ResourceRequest https://review.openstack.org/568353 | 19:59 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Granular requests to get_allocation_candidates https://review.openstack.org/515811 | 19:59 |
| mriedem | right just the constant definition | 20:00 |
| openstackgerrit | Chris Dent proposed openstack/nova master: Extract part of PlacementFixture to placement https://review.openstack.org/568359 | 20:00 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Refactor _build_device_metadata https://review.openstack.org/533804 | 20:00 |
| openstackgerrit | Artom Lifshitz proposed openstack/nova master: Consider hostdev devices when building metadata https://review.openstack.org/533805 | 20:00 |
| openstackgerrit | Zack Cornelius proposed openstack/nova-specs master: Libvirt file backed memory https://review.openstack.org/563704 | 20:00 |
| mnaser | dansmith: ooo. so .. from nova.virt.arch import <something>, class Architecture(BaseNovaEnum): <insert all existing definition>, ALL = <something> ? | 20:01 |
| dansmith | mnaser: from nova.virt import arch | 20:01 |
| dansmith | mnaser: class Architecture(NovaBaseEnum): ALL = arch.ALL | 20:01 |
| mnaser | ooo okay, and should i rename the other arches to be something like.. MIPS = arch.MIPS so they arent defined twice? | 20:02 |
| *** yamamoto has quit IRC | 20:02 | |
| mnaser | without breaking any old dependency on those | 20:02 |
| dansmith | mnaser: I don't think you need to do that, do you? | 20:02 |
| mnaser | i don't, but then i'd have all the architectures defined twice? | 20:02 |
| dansmith | anyone that references nova.objects.fields.Architecture.MIPS should change to arch.MIPS | 20:02 |
| lyarwood | mriedem: finally back online, thanks for chasing down the LM failure! | 20:02 |
| mnaser | oooh okay | 20:02 |
| mnaser | so change the references rather than copy pasta | 20:02 |
| mnaser | okay, i follow now | 20:02 |
| mnaser | it makes sense | 20:02 |
| mriedem | lyarwood: np, just working on backports now | 20:03 |
| dansmith | mnaser: ugh, well, there are a ton unfortunately | 20:03 |
| efried | mnaser, dansmith: Why not a local import? | 20:03 |
| mnaser | yeah, mostly in tests | 20:03 |
| dansmith | mnaser: % grep fields.Architecture -r nova | wc -l | 20:03 |
| dansmith | 344 | 20:03 |
| mnaser | but still a but of nova/virt changes | 20:04 |
| dansmith | mnaser: so I guess just do MIPS = arch.MIPS for all of those for your backportable change | 20:04 |
| bauzas | jaypipes: /me goes offline, but lemme know if you want to see the above point about having a conf opt like "this PCI ID = those traits" added in https://review.openstack.org/#/c/557065/ | 20:04 |
| mnaser | i'll do that and then i'll do a follow up cleanup which can sit in master and create a merge conflict for everyone's code :) | 20:04 |
| mnaser | (sorry in advance) | 20:04 |
| dansmith | mnaser: yeah :) | 20:04 |
| mnaser | but actually what do you mean by a local import efried ? can we import things inside a class? | 20:06 |
| *** jmlowe has quit IRC | 20:06 | |
| mnaser | kinda like `import nova.virt.arch.*` inside Architecture? | 20:06 |
| mriedem | the conf option is global module level so i'm not sure you can do a local import there | 20:06 |
| efried | mnaser: It probably won't work for what you're doing here, because it's the import of fields from within the conf setup that's blowing you up. | 20:07 |
| mriedem | oh right that | 20:07 |
| efried | yeah, what mriedem said. | 20:07 |
| mnaser | figured i'd ask anyways :) | 20:07 |
| *** jmlowe has joined #openstack-nova | 20:08 | |
| efried | mnaser: Yes, you can do imports from within a class method. But doing it within a class (outside of a method) won't really help you, because that'll be processed when the file is read rather than at runtime, which is what you would need. | 20:08 |
| *** liverpooler has quit IRC | 20:10 | |
| *** itlinux has joined #openstack-nova | 20:12 | |
| *** gbarros has quit IRC | 20:14 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/queens: libvirt: Report the virtual size of RAW disks https://review.openstack.org/568363 | 20:14 |
| *** arvindn05 has left #openstack-nova | 20:16 | |
| *** efried has quit IRC | 20:17 | |
| *** felipemonteiro_ has quit IRC | 20:17 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:17 | |
| *** arvindn05 has joined #openstack-nova | 20:19 | |
| *** yamamoto has joined #openstack-nova | 20:19 | |
| openstackgerrit | Chris Dent proposed openstack/nova master: Extract part of PlacementFixture to placement https://review.openstack.org/568359 | 20:20 |
| *** avolkov has quit IRC | 20:22 | |
| *** yamamoto has quit IRC | 20:24 | |
| *** itlinux has quit IRC | 20:24 | |
| *** itlinux has joined #openstack-nova | 20:25 | |
| *** mgoddard has quit IRC | 20:26 | |
| *** eharney has joined #openstack-nova | 20:26 | |
| *** cdent has quit IRC | 20:26 | |
| *** felipemonteiro__ has joined #openstack-nova | 20:29 | |
| *** sidx64 has joined #openstack-nova | 20:29 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/pike: libvirt: Report the virtual size of RAW disks https://review.openstack.org/568369 | 20:30 |
| *** dklyle has quit IRC | 20:31 | |
| *** damien_r has joined #openstack-nova | 20:31 | |
| *** felipemonteiro_ has quit IRC | 20:32 | |
| *** belmoreira has quit IRC | 20:33 | |
| *** baoli has quit IRC | 20:37 | |
| *** sidx64_ has joined #openstack-nova | 20:37 | |
| *** baoli has joined #openstack-nova | 20:37 | |
| *** sidx64 has quit IRC | 20:39 | |
| *** damien_r has quit IRC | 20:39 | |
| *** jmlowe has quit IRC | 20:39 | |
| dansmith | mriedem: hmm, gate reset? | 20:40 |
| *** yamamoto has joined #openstack-nova | 20:40 | |
| dansmith | ugh yep | 20:41 |
| *** jmlowe has joined #openstack-nova | 20:41 | |
| dansmith | mriedem: was going to try to be all good and go hit your backports right after the master one merged, but.. :/ | 20:41 |
| *** baoli has quit IRC | 20:41 | |
| *** belmoreira has joined #openstack-nova | 20:42 | |
| *** pcaruana has quit IRC | 20:43 | |
| mriedem | :/ | 20:45 |
| *** yamamoto has quit IRC | 20:45 | |
| *** felipemonteiro_ has joined #openstack-nova | 20:46 | |
| *** felipemonteiro__ has quit IRC | 20:46 | |
| *** edmondsw has quit IRC | 20:47 | |
| *** germs has joined #openstack-nova | 20:47 | |
| *** germs has quit IRC | 20:47 | |
| *** germs has joined #openstack-nova | 20:47 | |
| *** edmondsw has joined #openstack-nova | 20:47 | |
| *** belmoreira has quit IRC | 20:47 | |
| *** damien_r has joined #openstack-nova | 20:47 | |
| *** germs has quit IRC | 20:51 | |
| *** edmondsw has quit IRC | 20:52 | |
| *** Sukhdev has joined #openstack-nova | 20:55 | |
| openstackgerrit | Mohammed Naser proposed openstack/nova master: Added ability to configure default architecture for ImagePropertiesFilter https://review.openstack.org/566425 | 20:56 |
| *** FL1SK has quit IRC | 21:00 | |
| *** yamamoto has joined #openstack-nova | 21:01 | |
| *** efried has joined #openstack-nova | 21:04 | |
| *** hemna_ has quit IRC | 21:04 | |
| *** FL1SK has joined #openstack-nova | 21:05 | |
| *** yamamoto has quit IRC | 21:06 | |
| *** r-daneel has quit IRC | 21:07 | |
| *** r-daneel has joined #openstack-nova | 21:07 | |
| *** armaan has quit IRC | 21:08 | |
| *** armaan has joined #openstack-nova | 21:08 | |
| *** lpetrut has quit IRC | 21:11 | |
| *** armaan has quit IRC | 21:11 | |
| *** armaan has joined #openstack-nova | 21:11 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova stable/ocata: libvirt: Report the virtual size of RAW disks https://review.openstack.org/568382 | 21:17 |
| *** bnemec has quit IRC | 21:17 | |
| *** bnemec has joined #openstack-nova | 21:18 | |
| *** felipemonteiro_ has quit IRC | 21:20 | |
| *** jmlowe has quit IRC | 21:21 | |
| *** pchavva has quit IRC | 21:21 | |
| *** yamamoto has joined #openstack-nova | 21:22 | |
| *** sidx64_ has quit IRC | 21:22 | |
| *** yamamoto has quit IRC | 21:28 | |
| *** itlinux has quit IRC | 21:34 | |
| *** jmlowe has joined #openstack-nova | 21:36 | |
| *** jmlowe has quit IRC | 21:41 | |
| *** yamamoto has joined #openstack-nova | 21:45 | |
| mriedem | finucannot: for when you're back, https://specs.openstack.org/openstack/nova-specs/specs/queens/implemented/websocket-proxy-to-host-security.html mentions SPICE and serial consoles also, but isn't the only support that was added for VNC? | 21:47 |
| *** yamamoto has quit IRC | 21:49 | |
| *** takashin has joined #openstack-nova | 21:51 | |
| *** rcernin has joined #openstack-nova | 21:53 | |
| mriedem | dansmith: i want to put something in the cells v2 forum etherpad, have you created it yet? https://wiki.openstack.org/wiki/Forum/Vancouver2018 | 21:56 |
| mriedem | if not i can create it quick | 21:56 |
| *** harlowja has quit IRC | 21:56 | |
| dansmith | mriedem: I have not | 21:56 |
| mriedem | https://etherpad.openstack.org/p/YVR18-cellsv2-migration-sync-with-operators | 21:58 |
| *** dklyle has joined #openstack-nova | 21:59 | |
| *** edmondsw has joined #openstack-nova | 22:03 | |
| dansmith | mriedem: thanks | 22:04 |
| *** efried1 has joined #openstack-nova | 22:05 | |
| *** yamamoto has joined #openstack-nova | 22:06 | |
| *** efried has quit IRC | 22:07 | |
| *** efried1 is now known as efried | 22:07 | |
| *** jistr has quit IRC | 22:07 | |
| *** edmondsw has quit IRC | 22:08 | |
| *** dklyle has quit IRC | 22:08 | |
| *** jistr has joined #openstack-nova | 22:10 | |
| *** yamamoto has quit IRC | 22:10 | |
| *** esberglu has quit IRC | 22:12 | |
| *** esberglu has joined #openstack-nova | 22:13 | |
| *** threestrands has joined #openstack-nova | 22:16 | |
| *** esberglu has quit IRC | 22:17 | |
| *** mlavalle has quit IRC | 22:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658 | 22:20 |
| *** slaweq has quit IRC | 22:20 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659 | 22:21 |
| *** slaweq has joined #openstack-nova | 22:21 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in libvirt/test_driver.py (1) https://review.openstack.org/566555 | 22:21 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: api-ref: Example verification for servers.inc https://review.openstack.org/529520 | 22:21 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_live_migrate.py https://review.openstack.org/564656 | 22:22 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528 | 22:22 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/*/test_volumes.py https://review.openstack.org/564655 | 22:22 |
| *** mriedem has quit IRC | 22:22 | |
| *** yamamoto has joined #openstack-nova | 22:26 | |
| *** hemna_ has joined #openstack-nova | 22:29 | |
| *** yamamoto has quit IRC | 22:33 | |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Fix 500 error while passing 4-byte unicode data https://review.openstack.org/407514 | 22:33 |
| openstackgerrit | Takashi NATSUME proposed openstack/nova master: Adds view builders for keypairs controller https://review.openstack.org/347289 | 22:34 |
| *** hongbin has quit IRC | 22:40 | |
| *** yamamoto has joined #openstack-nova | 22:49 | |
| *** dklyle has joined #openstack-nova | 22:50 | |
| *** yamamoto has quit IRC | 22:54 | |
| *** zcorneli has quit IRC | 22:54 | |
| *** r-daneel has quit IRC | 22:54 | |
| *** dklyle has quit IRC | 22:56 | |
| *** baoli has joined #openstack-nova | 23:00 | |
| *** dave-mccowan has quit IRC | 23:02 | |
| *** baoli has quit IRC | 23:04 | |
| *** lyan has quit IRC | 23:04 | |
| *** yamamoto has joined #openstack-nova | 23:10 | |
| *** yamamoto has quit IRC | 23:16 | |
| *** rajinir has quit IRC | 23:19 | |
| *** yamamoto has joined #openstack-nova | 23:32 | |
| *** dklyle has joined #openstack-nova | 23:35 | |
| *** yamamoto has quit IRC | 23:37 | |
| *** oanson has quit IRC | 23:37 | |
| *** Nel1x has joined #openstack-nova | 23:48 | |
| *** oanson has joined #openstack-nova | 23:48 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!