*** hemna_ has quit IRC | 00:08 | |
*** takashin has joined #openstack-nova | 00:09 | |
*** fragatina has quit IRC | 00:12 | |
*** fragatina has joined #openstack-nova | 00:15 | |
*** fragatina has quit IRC | 00:19 | |
*** Dinesh_Bhor has joined #openstack-nova | 00:19 | |
*** Bonjourm8 has quit IRC | 00:23 | |
*** mingyu_ has joined #openstack-nova | 00:24 | |
*** mingyu has quit IRC | 00:26 | |
*** suresh12 has quit IRC | 00:37 | |
*** suresh12 has joined #openstack-nova | 00:38 | |
*** felipemonteiro__ has joined #openstack-nova | 00:39 | |
*** hoangcx has joined #openstack-nova | 00:41 | |
*** hiro-kobayashi has joined #openstack-nova | 00:47 | |
*** Nel1x has quit IRC | 00:48 | |
*** tuanla____ has joined #openstack-nova | 00:57 | |
*** phuongnh has joined #openstack-nova | 01:00 | |
*** gjayavelu has quit IRC | 01:02 | |
*** Nel1x has joined #openstack-nova | 01:03 | |
*** dpawlik has joined #openstack-nova | 01:06 | |
*** zhaochao has joined #openstack-nova | 01:07 | |
*** mlavalle has quit IRC | 01:08 | |
*** vish_18 has quit IRC | 01:08 | |
*** dpawlik has quit IRC | 01:11 | |
*** hoangcx has quit IRC | 01:13 | |
*** hoangcx has joined #openstack-nova | 01:13 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (1) https://review.openstack.org/564639 | 01:13 |
---|---|---|
*** trungnv has quit IRC | 01:14 | |
*** trungnv has joined #openstack-nova | 01:14 | |
*** Nel1x has quit IRC | 01:20 | |
*** tiendc has joined #openstack-nova | 01:21 | |
openstackgerrit | Merged openstack/nova master: Proxy is_volume through DriverBlockDevice https://review.openstack.org/564017 | 01:27 |
*** suresh12 has quit IRC | 01:31 | |
*** Bonjourm8 has joined #openstack-nova | 01:31 | |
*** Nel1x has joined #openstack-nova | 01:34 | |
*** arvindn05 has quit IRC | 01:35 | |
*** felipemonteiro__ has quit IRC | 01:35 | |
*** bkopilov_ has quit IRC | 01:35 | |
*** bkopilov__ has quit IRC | 01:35 | |
*** arvindn05 has joined #openstack-nova | 01:40 | |
*** sapd has joined #openstack-nova | 01:40 | |
*** carl_baldwin_ has quit IRC | 01:55 | |
*** lei-zh has joined #openstack-nova | 01:56 | |
*** hongbin has joined #openstack-nova | 02:02 | |
*** namnh has joined #openstack-nova | 02:05 | |
*** Bonjourm8 has quit IRC | 02:11 | |
*** psachin` has joined #openstack-nova | 02:13 | |
*** yamamoto has joined #openstack-nova | 02:17 | |
*** dave-mccowan has quit IRC | 02:20 | |
*** yamamoto has quit IRC | 02:22 | |
*** brault has joined #openstack-nova | 02:31 | |
*** lei-zh has quit IRC | 02:33 | |
*** lei-zh has joined #openstack-nova | 02:34 | |
*** brault has quit IRC | 02:36 | |
*** suresh12 has joined #openstack-nova | 02:36 | |
*** gjayavelu has joined #openstack-nova | 02:36 | |
*** hongbin_ has joined #openstack-nova | 02:37 | |
*** felipemonteiro__ has joined #openstack-nova | 02:39 | |
*** hongbin has quit IRC | 02:40 | |
*** suresh12 has quit IRC | 02:40 | |
*** esberglu has joined #openstack-nova | 02:48 | |
*** esberglu has quit IRC | 02:48 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (2) https://review.openstack.org/564643 | 02:52 |
openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 02:53 |
*** bkopilov_ has joined #openstack-nova | 02:55 | |
*** bkopilov__ has joined #openstack-nova | 02:56 | |
*** felipemonteiro__ has quit IRC | 02:58 | |
*** fragatina has joined #openstack-nova | 02:58 | |
*** hoangcx has quit IRC | 03:03 | |
*** yamamoto has joined #openstack-nova | 03:04 | |
*** hoangcx has joined #openstack-nova | 03:05 | |
*** hoangcx has quit IRC | 03:05 | |
*** dpawlik has joined #openstack-nova | 03:07 | |
*** gcb has joined #openstack-nova | 03:08 | |
*** hoangcx_ has joined #openstack-nova | 03:08 | |
*** hoangcx_ has quit IRC | 03:09 | |
*** dpawlik has quit IRC | 03:12 | |
*** hemna_ has joined #openstack-nova | 03:14 | |
*** fragatina has quit IRC | 03:20 | |
*** takashin has quit IRC | 03:25 | |
*** dpawlik has joined #openstack-nova | 03:26 | |
*** dpawlik has quit IRC | 03:31 | |
*** yamamoto has quit IRC | 03:31 | |
*** nicolasbock has quit IRC | 03:40 | |
*** gyankum has joined #openstack-nova | 03:41 | |
*** lpetrut has joined #openstack-nova | 03:43 | |
*** takashin has joined #openstack-nova | 03:46 | |
*** sridharg has joined #openstack-nova | 03:48 | |
*** sapd has quit IRC | 03:48 | |
*** gyee has quit IRC | 03:48 | |
*** janki has joined #openstack-nova | 03:50 | |
*** tuanla____ has quit IRC | 03:50 | |
*** tuanla____ has joined #openstack-nova | 03:51 | |
*** udesale has joined #openstack-nova | 03:54 | |
*** hongbin_ has quit IRC | 03:55 | |
*** suresh12 has joined #openstack-nova | 04:01 | |
*** namnh has quit IRC | 04:01 | |
*** tiendc has quit IRC | 04:01 | |
*** tiendc has joined #openstack-nova | 04:01 | |
*** namnh has joined #openstack-nova | 04:01 | |
*** fragatina has joined #openstack-nova | 04:03 | |
*** fragatina has quit IRC | 04:04 | |
*** fragatina has joined #openstack-nova | 04:04 | |
*** mingyu_ has quit IRC | 04:06 | |
*** gbarros has joined #openstack-nova | 04:14 | |
*** lpetrut has quit IRC | 04:15 | |
*** gyee has joined #openstack-nova | 04:17 | |
*** lei-zh has quit IRC | 04:17 | |
*** moshele has joined #openstack-nova | 04:18 | |
openstackgerrit | Merged openstack/nova master: Deprecate the nova-consoleauth service https://review.openstack.org/565367 | 04:21 |
*** moshele has quit IRC | 04:22 | |
*** gyee has quit IRC | 04:24 | |
*** gjayavelu has quit IRC | 04:31 | |
*** links has joined #openstack-nova | 04:38 | |
*** vivsoni__ has quit IRC | 04:41 | |
*** vivsoni has joined #openstack-nova | 04:42 | |
*** abhishekk has joined #openstack-nova | 04:43 | |
*** ratailor has joined #openstack-nova | 04:49 | |
*** claudiub has joined #openstack-nova | 04:53 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_xenapi.py (3) https://review.openstack.org/564645 | 04:53 |
*** suresh12 has quit IRC | 04:58 | |
*** Zames_ has joined #openstack-nova | 04:59 | |
*** Dinesh__Bhor has joined #openstack-nova | 05:01 | |
*** Dinesh_Bhor has quit IRC | 05:02 | |
*** Zames_ has quit IRC | 05:02 | |
*** yamamoto has joined #openstack-nova | 05:06 | |
*** lei-zh has joined #openstack-nova | 05:07 | |
*** mingyu has joined #openstack-nova | 05:11 | |
*** lpetrut has joined #openstack-nova | 05:13 | |
*** hemna_ has quit IRC | 05:15 | |
*** ratailor has quit IRC | 05:17 | |
openstackgerrit | Merged openstack/nova master: Give volume DriverBlockDevice classes a common prefix https://review.openstack.org/526346 | 05:17 |
*** ratailor has joined #openstack-nova | 05:21 | |
*** lpetrut has quit IRC | 05:24 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (1) https://review.openstack.org/564658 | 05:24 |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in test_conductor.py (2) https://review.openstack.org/564659 | 05:25 |
*** Eran_Kuris has joined #openstack-nova | 05:27 | |
*** dpawlik has joined #openstack-nova | 05:27 | |
*** Nel1x has quit IRC | 05:27 | |
*** dpawlik has quit IRC | 05:32 | |
*** trinaths has joined #openstack-nova | 05:40 | |
*** ccamacho has joined #openstack-nova | 05:41 | |
openstackgerrit | jichenjc proposed openstack/nova master: [doc] merge quota document https://review.openstack.org/566783 | 05:46 |
*** ratailor_ has joined #openstack-nova | 05:50 | |
*** ratailor has quit IRC | 05:52 | |
*** kaisers1 has quit IRC | 05:54 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove mox in tests/unit/api/openstack/compute https://review.openstack.org/566528 | 05:54 |
*** gbarros has quit IRC | 05:57 | |
*** markvoelker has quit IRC | 06:01 | |
*** pcaruana has joined #openstack-nova | 06:01 | |
*** tssurya has joined #openstack-nova | 06:03 | |
*** armaan has joined #openstack-nova | 06:07 | |
*** moshele has joined #openstack-nova | 06:09 | |
*** lpetrut has joined #openstack-nova | 06:10 | |
*** kaisers1 has joined #openstack-nova | 06:11 | |
*** armaan has quit IRC | 06:11 | |
*** armaan has joined #openstack-nova | 06:12 | |
*** wolverineav has joined #openstack-nova | 06:13 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP] Add queued for delete to instance_mappings table. https://review.openstack.org/566788 | 06:13 |
*** Dinesh__Bhor has quit IRC | 06:14 | |
*** dineshbhor__ has joined #openstack-nova | 06:14 | |
*** dineshbhor__ has quit IRC | 06:15 | |
*** Dinesh_Bhor has joined #openstack-nova | 06:16 | |
*** penick has quit IRC | 06:17 | |
*** evin has joined #openstack-nova | 06:17 | |
*** ratailor_ has quit IRC | 06:19 | |
*** udesale has quit IRC | 06:23 | |
*** annp has joined #openstack-nova | 06:23 | |
*** udesale has joined #openstack-nova | 06:23 | |
*** armaan has quit IRC | 06:24 | |
*** armaan has joined #openstack-nova | 06:25 | |
*** liuzz_ has joined #openstack-nova | 06:29 | |
*** liuzz has quit IRC | 06:29 | |
*** brault has joined #openstack-nova | 06:32 | |
*** rajinir has quit IRC | 06:35 | |
*** penick has joined #openstack-nova | 06:36 | |
*** brault has quit IRC | 06:36 | |
*** andreas_s has joined #openstack-nova | 06:37 | |
*** mingyu_ has joined #openstack-nova | 06:37 | |
*** mingyu has quit IRC | 06:37 | |
*** alexchadin has joined #openstack-nova | 06:39 | |
*** bhujay has joined #openstack-nova | 06:42 | |
*** mingyu_ has quit IRC | 06:42 | |
*** gjayavelu has joined #openstack-nova | 06:43 | |
*** moshele has quit IRC | 06:44 | |
*** kaisers1 has quit IRC | 06:45 | |
*** markvoelker has joined #openstack-nova | 07:01 | |
*** dpawlik has joined #openstack-nova | 07:02 | |
*** brault has joined #openstack-nova | 07:02 | |
*** brault has quit IRC | 07:04 | |
*** brault has joined #openstack-nova | 07:04 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795 | 07:06 |
*** alexchad_ has joined #openstack-nova | 07:08 | |
*** rcernin has quit IRC | 07:08 | |
*** alexchadin has quit IRC | 07:10 | |
*** sapd has joined #openstack-nova | 07:11 | |
*** zhuli has quit IRC | 07:12 | |
*** zhuli_ has joined #openstack-nova | 07:13 | |
*** gjayavelu has quit IRC | 07:13 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP/POC] Add queued_for_delete field to InstanceMapping object https://review.openstack.org/566795 | 07:14 |
*** armaan has quit IRC | 07:14 | |
*** udesale has quit IRC | 07:15 | |
*** udesale has joined #openstack-nova | 07:15 | |
*** trinaths has quit IRC | 07:15 | |
*** claudiub|2 has joined #openstack-nova | 07:19 | |
*** avolkov has joined #openstack-nova | 07:21 | |
*** kaisers1 has joined #openstack-nova | 07:22 | |
*** claudiub has quit IRC | 07:22 | |
*** udesale_ has joined #openstack-nova | 07:34 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | 07:35 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487 | 07:35 |
*** amoralej|off is now known as amoralej | 07:35 | |
*** markvoelker has quit IRC | 07:36 | |
*** suresh12 has joined #openstack-nova | 07:37 | |
*** udesale has quit IRC | 07:37 | |
*** suresh12 has quit IRC | 07:41 | |
*** gcb has quit IRC | 07:41 | |
*** alexchad_ has quit IRC | 07:48 | |
*** jpena|off is now known as jpena | 07:52 | |
*** yamamoto has quit IRC | 07:56 | |
*** kashyap` is now known as kashyap | 07:57 | |
kashyap | dansmith: Sorry, had to be AFK a bit early last evening. Just catching up with the scroll | 07:58 |
*** gongysh has joined #openstack-nova | 08:02 | |
kashyap | dansmith: melwitt: Okido, let's go with the exhibit (a) which is: "deprecate in Rocky and hard-fail in Stein" | 08:02 |
*** yamamoto has joined #openstack-nova | 08:03 | |
*** lpetrut has quit IRC | 08:05 | |
*** jmccarthy has joined #openstack-nova | 08:07 | |
*** mdnadeem has joined #openstack-nova | 08:13 | |
*** lucas-pto is now known as lucasagomes | 08:16 | |
*** hiro-kobayashi has quit IRC | 08:22 | |
*** dtantsur|afk is now known as dtantsur | 08:27 | |
*** ralonsoh has joined #openstack-nova | 08:29 | |
openstackgerrit | Surya Seetharaman proposed openstack/nova master: [WIP/POC] Updating queued_for_delete from instance_destroy() https://review.openstack.org/566813 | 08:29 |
*** markvoelker has joined #openstack-nova | 08:33 | |
*** ratailor has joined #openstack-nova | 08:34 | |
*** nsingh has joined #openstack-nova | 08:38 | |
*** ShilpaSD has joined #openstack-nova | 08:38 | |
*** claudiub has joined #openstack-nova | 08:39 | |
*** stakeda_ has joined #openstack-nova | 08:39 | |
*** ratailor_ has joined #openstack-nova | 08:39 | |
*** vivsoni_ has joined #openstack-nova | 08:39 | |
*** ratailor has quit IRC | 08:40 | |
*** yamamoto_ has joined #openstack-nova | 08:40 | |
*** jmccarthy has quit IRC | 08:40 | |
*** bandini has quit IRC | 08:40 | |
*** zigo_ has quit IRC | 08:40 | |
*** stakeda has quit IRC | 08:41 | |
*** gryf has quit IRC | 08:41 | |
*** lyarwood has quit IRC | 08:41 | |
*** jmccarthy has joined #openstack-nova | 08:41 | |
*** lyarwood has joined #openstack-nova | 08:41 | |
*** gryf has joined #openstack-nova | 08:41 | |
*** yamamoto has quit IRC | 08:41 | |
*** niraj_singh has quit IRC | 08:41 | |
*** linkmark has quit IRC | 08:41 | |
*** bandini has joined #openstack-nova | 08:41 | |
*** claudiub|2 has quit IRC | 08:41 | |
*** vivsoni has quit IRC | 08:41 | |
*** linkmark has joined #openstack-nova | 08:42 | |
*** Shilpa has quit IRC | 08:42 | |
*** alexchadin has joined #openstack-nova | 08:44 | |
*** derekh has joined #openstack-nova | 08:44 | |
*** zigo has joined #openstack-nova | 08:45 | |
kashyap | bauzas: When you are about, can you merge this simple Ocata backport? Already has a +2 from Sean McGinnis: https://review.openstack.org/#/c/565672/ | 08:45 |
kashyap | bauzas: (Pinging you, as you ACKed the other branch backports) | 08:46 |
kashyap | Or even lyarwood ^ | 08:46 |
kashyap | That's the outstanding backport for that bug. | 08:46 |
*** yamamoto_ has quit IRC | 08:56 | |
lyarwood | kashyap: looking | 08:56 |
lyarwood | kashyap: done | 08:57 |
kashyap | lyarwood: Splendid! | 08:57 |
kashyap | Thank you | 08:58 |
*** Zames_ has joined #openstack-nova | 08:58 | |
*** jmccarthy has quit IRC | 09:01 | |
*** armaan has joined #openstack-nova | 09:01 | |
*** hiro-kobayashi has joined #openstack-nova | 09:01 | |
*** Zames_ has quit IRC | 09:02 | |
*** lpetrut has joined #openstack-nova | 09:03 | |
*** mdbooth has joined #openstack-nova | 09:05 | |
*** jmccarthy has joined #openstack-nova | 09:05 | |
*** markvoelker has quit IRC | 09:07 | |
*** armaan has quit IRC | 09:08 | |
*** yamamoto has joined #openstack-nova | 09:08 | |
*** gaoyan has joined #openstack-nova | 09:11 | |
*** goutham has joined #openstack-nova | 09:13 | |
*** goutham has left #openstack-nova | 09:13 | |
openstackgerrit | Theodoros Tsioutsias proposed openstack/nova-specs master: Add PENDING vm state https://review.openstack.org/554212 | 09:13 |
*** lpetrut has quit IRC | 09:14 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Consider nested RPs in get_all_with_shared https://review.openstack.org/556450 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support shared and nested allocation candidates https://review.openstack.org/556514 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested alloc cands without shared RPs https://review.openstack.org/561842 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all resources in provider_summaries https://review.openstack.org/558045 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Return resource providers without inventories https://review.openstack.org/559554 | 09:16 |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Support nested allocation candidates in placement https://review.openstack.org/565487 | 09:16 |
*** moshele has joined #openstack-nova | 09:16 | |
*** claudiub|2 has joined #openstack-nova | 09:19 | |
*** moshele has quit IRC | 09:20 | |
*** abhishekk has quit IRC | 09:20 | |
*** gongysh has quit IRC | 09:21 | |
*** moshele has joined #openstack-nova | 09:21 | |
*** abhishekk has joined #openstack-nova | 09:21 | |
*** sambetts|afk is now known as sambetts | 09:22 | |
*** claudiub has quit IRC | 09:22 | |
openstackgerrit | Vishakha Agarwal proposed openstack/nova master: Re-using the code of os brick cinder https://review.openstack.org/565703 | 09:23 |
*** wolverineav has quit IRC | 09:30 | |
*** wolverineav has joined #openstack-nova | 09:30 | |
*** lei-zh has quit IRC | 09:32 | |
*** Zames_ has joined #openstack-nova | 09:33 | |
*** Dinesh_Bhor has quit IRC | 09:33 | |
*** mingyu has joined #openstack-nova | 09:33 | |
*** wolverineav has quit IRC | 09:35 | |
*** mingyu has quit IRC | 09:35 | |
*** mingyu_ has joined #openstack-nova | 09:35 | |
*** Zames_ has quit IRC | 09:35 | |
*** bkopilov__ has quit IRC | 09:39 | |
*** bkopilov_ has quit IRC | 09:39 | |
*** ragiman has joined #openstack-nova | 09:42 | |
jmccarthy | mriedem: jgwentworth: Thanks for your help :) I tested the review, and in my setup it fixed the issue - https://review.openstack.org/#/c/566367 | 09:44 |
*** gongysh has joined #openstack-nova | 09:44 | |
*** Zames_ has joined #openstack-nova | 09:56 | |
*** hiro-kobayashi has quit IRC | 09:57 | |
*** Zames_ has quit IRC | 09:57 | |
*** udesale_ has quit IRC | 09:59 | |
*** mgoddard has joined #openstack-nova | 09:59 | |
*** Zames_ has joined #openstack-nova | 10:02 | |
*** armaan has joined #openstack-nova | 10:04 | |
*** markvoelker has joined #openstack-nova | 10:04 | |
*** Zames_ has quit IRC | 10:04 | |
*** alexchadin has quit IRC | 10:09 | |
*** abhishekk has quit IRC | 10:10 | |
*** tetsuro has joined #openstack-nova | 10:13 | |
*** yamamoto has quit IRC | 10:15 | |
openstackgerrit | Tetsuro Nakamura proposed openstack/nova master: Add tests for sharing rps in test_common_rc() https://review.openstack.org/566842 | 10:17 |
tetsuro | efried, jaypipes, bauzas: ^ found a new bug in sharing case. might be related to https://review.openstack.org/#/c/560459/3? | 10:19 |
*** mingyu_ has quit IRC | 10:19 | |
*** ratailor__ has joined #openstack-nova | 10:19 | |
*** yamamoto has joined #openstack-nova | 10:20 | |
*** ratailor_ has quit IRC | 10:22 | |
*** udesale has joined #openstack-nova | 10:27 | |
*** udesale has quit IRC | 10:27 | |
*** udesale has joined #openstack-nova | 10:28 | |
*** gaoyan has quit IRC | 10:29 | |
*** rcernin has joined #openstack-nova | 10:35 | |
*** nicolasbock has joined #openstack-nova | 10:36 | |
*** namnh has quit IRC | 10:36 | |
*** markvoelker has quit IRC | 10:38 | |
*** tuanla_____ has joined #openstack-nova | 10:40 | |
openstackgerrit | Merged openstack/nova stable/ocata: libvirt: Make `cpu_model_extra_flags` case-insensitive for real https://review.openstack.org/565672 | 10:40 |
*** tuanla____ has quit IRC | 10:42 | |
*** elod has quit IRC | 10:47 | |
*** yamamoto has quit IRC | 10:48 | |
*** elod has joined #openstack-nova | 10:49 | |
openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | 10:50 |
openstackgerrit | Balazs Gibizer proposed openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | 10:51 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Add three new '[libvirt] live_migration_*' options https://review.openstack.org/456571 | 10:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Gather 'live_migration_scheme', 'live_migration_inbound_addr' https://review.openstack.org/456572 | 10:52 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Convert 'live_migration_inbound_addr' to HostAddressOpt https://review.openstack.org/456573 | 10:52 |
stephenfin | owalsh: ^ | 10:52 |
owalsh | stephenfin: ack, thanks | 10:53 |
*** alexchadin has joined #openstack-nova | 10:55 | |
*** stakeda_ has quit IRC | 10:55 | |
*** ragiman has quit IRC | 10:58 | |
openstackgerrit | Theodoros Tsioutsias proposed openstack/nova master: Add scheduling notification https://review.openstack.org/566470 | 10:59 |
*** takashin has left #openstack-nova | 11:01 | |
openstackgerrit | Takashi NATSUME proposed openstack/nova master: Remove unused methods in nova/compute/utils.py https://review.openstack.org/566855 | 11:02 |
*** wolverineav has joined #openstack-nova | 11:03 | |
*** amoralej is now known as amoralej|lunch | 11:04 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: conf: Use new-style choice values https://review.openstack.org/530924 | 11:06 |
*** gongysh has quit IRC | 11:06 | |
*** moshele has quit IRC | 11:12 | |
*** ragiman has joined #openstack-nova | 11:12 | |
*** phuongnh has quit IRC | 11:14 | |
*** tuanla_____ has quit IRC | 11:15 | |
*** lucasagomes is now known as lucas-hungry | 11:20 | |
*** mrjk_ has quit IRC | 11:27 | |
*** mrjk_ has joined #openstack-nova | 11:28 | |
*** claudiub has joined #openstack-nova | 11:28 | |
tetsuro | Hmm... this makes me change my mind that this is a good opportunity to rework the allocation candidates logic in a totally different way. So it is fair that you ask that if we can amend nested alloc cands series without considering the existing path for sharing providers. | 11:30 |
*** tiendc has quit IRC | 11:31 | |
*** claudiub|2 has quit IRC | 11:31 | |
*** markvoelker has joined #openstack-nova | 11:34 | |
*** yamamoto has joined #openstack-nova | 11:37 | |
artom | sean-k-m1, sean-k-mooney[m], one of you around? Have some questions about the Intel NFV CI | 11:43 |
*** mingyu has joined #openstack-nova | 11:46 | |
*** jpena is now known as jpena|lunch | 11:46 | |
*** sean-k-m1 is now known as sean-k-mooney | 11:50 | |
sean-k-mooney | artom: hi | 11:50 |
artom | sean-k-mooney, heya - live migration with NUMA (either pinning or actual NUMA topologies) can be tested in the Intel NFV CI, right? | 11:51 |
sean-k-mooney | sorry i have been on vacation since last tuseday. my mum ha a minor operation so i was being a good son and makeing her tea and mining the house all week | 11:51 |
artom | sean-k-mooney, awww :) | 11:51 |
sean-k-mooney | artom: so yes nfv ci | 11:51 |
artom | Hope she's doing well | 11:52 |
sean-k-mooney | artom: she is doing fine. am yes you could test numa + livemigration in the intel nfv ci but i think the livemigration tests are not in cluded in teh tempest regex in the multinode job | 11:53 |
theanalyst | t | 11:53 |
*** annp has quit IRC | 11:53 | |
artom | sean-k-mooney, could you point me to the code? | 11:54 |
artom | https://github.com/openstack/intel-nfv-ci-tests I assume | 11:55 |
sean-k-mooney | artom: the ci is currently configured using an internal git repo but im going to take a look at the tempest.conf | 11:55 |
* artom went looking for himself like a big boy :) | 11:55 | |
sean-k-mooney | artom: that is a part of it but not the ci config | 11:55 |
sean-k-mooney | artom: that host our tempest plugin | 11:55 |
*** pchavva has joined #openstack-nova | 11:55 | |
*** sridharg has quit IRC | 11:56 | |
sean-k-mooney | artom: tempest-dsvm-intel-nfv-xenial/ uses that plugin but its not multinode | 11:56 |
artom | sean-k-mooney, does it go through gerrit? My plan is to write a live migration NUMA test (unless those already exist? A quick look at the code doesn't turn anything up), and make it Depends-On: my own patches for that | 11:56 |
*** sridharg has joined #openstack-nova | 11:57 | |
sean-k-mooney | artom: the statdard livemigration test would partly test this. the tempest-dsvm-ovsdpdk-nfv-networking-xenial replaces the default flvors with ones that use cpu pinning + hugepages + 2 numa nodes | 11:57 |
sean-k-mooney | artom: this is one of the tempest.confs from a run on the ci http://52.27.155.124/portland/2018-04-27/553072/6/check/tempest-dsvm-ovsdpdk-nfv-networking-xenial/ff9bb78/logs/tempest_conf.txt.gz | 11:58 |
sean-k-mooney | so yes live_migration = False is set | 11:59 |
sean-k-mooney | we could modify that via a patch to networking-ovs-dpdk to enable livemigration via its devstack plugin. | 11:59 |
artom | sean-k-mooney, so at least as a first step I could need to get live_migration set to True? | 11:59 |
artom | *would need | 12:00 |
sean-k-mooney | artom: yep, we could also do that via changing the internal repo but i dont have access to it because i dont know where it is now | 12:00 |
*** andreas_s has quit IRC | 12:01 | |
artom | sean-k-mooney, heh, are the contacts listed on https://wiki.openstack.org/wiki/ThirdPartySystems/Intel_NFV_CI still good? | 12:01 |
sean-k-mooney | networking-ovs-dpdk however i have full contol over. yes the contacts are correct | 12:01 |
*** lyan has joined #openstack-nova | 12:01 | |
*** lyan is now known as Guest87997 | 12:02 | |
artom | sean-k-mooney, wait, so which one do you *not* have access to? | 12:02 |
*** andreas_s_ has joined #openstack-nova | 12:02 | |
*** andreas_s_ has quit IRC | 12:02 | |
*** andreas_s_ has joined #openstack-nova | 12:02 | |
sean-k-mooney | artom: i do not have acess to the zuul 2.5 configure repo that stores the job config. i am the maintainer of networking-ovs-dpdk and have full aceess to modify the job via that repo using the devstack plugin | 12:03 |
artom | sean-k-mooney, ah, gotcha | 12:04 |
artom | And those are both internal Intel, right? | 12:04 |
sean-k-mooney | networking-ovs-dpdk is hosted by openstack infra but is not technically part of openstack. https://github.com/openstack/networking-ovs-dpdk | 12:05 |
*** mriedem has joined #openstack-nova | 12:05 | |
sean-k-mooney | artom: we should be able to use the test-config section of the devstack plugin to enable live migration https://github.com/openstack-dev/devstack/blob/master/doc/source/plugins.rst#pluginsh-contract | 12:05 |
artom | Also, http://52.27.155.124/portland/2018-04-20/563037/2/check/tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial/3e525a4/logs/tempest_conf.txt.gz seems to enable live_migration? | 12:05 |
artom | Which woujld | 12:06 |
artom | Which wou45ld | 12:06 |
artom | Which would make sense since it's multinode, and the other one isn't | 12:06 |
sean-k-mooney | tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial used to be the same job just run against neutron | 12:06 |
sean-k-mooney | sorry yes i ment tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial | 12:07 |
artom | (Excuse the random characters, daughter decided my laptop is a toy) | 12:07 |
sean-k-mooney | artom: no worries | 12:07 |
sean-k-mooney | still more understandable then my spelling at times | 12:08 |
*** markvoelker has quit IRC | 12:08 | |
*** edmondsw has joined #openstack-nova | 12:08 | |
*** ragiman has quit IRC | 12:09 | |
sean-k-mooney | artom: do you know if devstack-gate sets any env varibles that we could use to enable "ci testing" config changes within the devstack plugin. | 12:11 |
artom | sean-k-mooney, I'm not sure I understand the question | 12:11 |
*** andreas_s_ has quit IRC | 12:12 | |
sean-k-mooney | well if we add the sections to enable livemirgation in the networking-ovs-dpdk devstack plugin i am wondering is there a way to enable it automatically in a ci env and not on a normal install | 12:13 |
*** andreas_s has joined #openstack-nova | 12:13 | |
sean-k-mooney | if we are just setting tempest settings then there is no real harm always doing it i was just wondering if there was a way to detect that we are running in a ci | 12:14 |
*** andreas_s has quit IRC | 12:14 | |
mriedem | sean-k-mooney: https://github.com/openstack-infra/devstack-gate/blob/b3da8a393c68cc62924ee2f752f442d5c85ea8ed/devstack-vm-gate.sh#L566 | 12:15 |
*** markvoelker has joined #openstack-nova | 12:15 | |
*** andreas_s_ has joined #openstack-nova | 12:15 | |
mriedem | all non-grenade multinode jobs run live migration by default | 12:15 |
*** andreas_s_ has quit IRC | 12:15 | |
*** ratailor__ has quit IRC | 12:15 | |
mriedem | grenade doesn't because grenade by default only runs smoke tests, and live migration tests aren't smoke tests | 12:15 |
*** andreas_s_ has joined #openstack-nova | 12:16 | |
mriedem | we do have a grenade job specifically that tests just live migration back and forth from n-1 to n and back | 12:16 |
artom | mriedem, I suppose I could just test numa-aware live migration with cpu pinning since we don't need special hardware for that, but there'd be no way to assert anything about the instance XML, which is kinda what we want | 12:17 |
sean-k-mooney | mriedem: ah you are right looking at http://52.27.155.124/portland/2018-04-27/553072/6/check/tempest-dsvm-multinode-ovsdpdk-nfv-networking-xenial/943bdb9/logs/tempest_conf.txt.gz livemigration is turned on the quest then is why is it not running those tests | 12:17 |
*** amoralej|lunch is now known as amoralej | 12:18 | |
mriedem | sean-k-mooney: blacklist? | 12:18 |
sean-k-mooney | mriedem: proably ill have to look into that | 12:18 |
mriedem | [[ ^(?!.*(?:tempest\.scenario\.test_network_advanced_server_ops\.TestNetworkAdvancedServerOps\.test_server_connectivity_suspend_resume.*)|(?:tempest\.scenario\.test_network_advanced_server_ops\.TestNetworkAdvancedServerOps\.test_server_connectivity_pause_unpause.*)|(?:.*\.admin\.test_live_migration.*)|(?:.*\.TestNetworkAdvancedServerOps.test_server_connectivity_cold_migration_revert.*)|(?:.*\.TestNetworkAdvancedServerOps.test | 12:18 |
mriedem | ver_connectivity_cold_migration.*)|(?:.*\.TestNetworkAdvancedServerOps.test_server_connectivity_resize.*)|(?:.*\.TestNetworkBasicOps\.test_update_router_admin_state.*)|(?:.*\.TestNetworkBasicOps\.test_network_basic_op.*)|(?:.*\.TestNetworkBasicOps\.test_update_instance_port_admin_state.*))((?:tempest\.scenario\.test_network_basic_ops.*)|(?:tempest\.scenario\.test_network_advanced_server_ops.*)|(?:tempest\.api\.compute\.admin\ | 12:18 |
mriedem | t_live_migration.*)|(?:tempest\.api\.network.*)).*$ != '' ]] | 12:18 |
mriedem | yes it's blacklisted live migration | 12:19 |
sean-k-mooney | artom: you need a multi numa host vm as we have a restiction in the libvirt drivver that maps each guest numa node to a different host numa node | 12:19 |
artom | Which makes sense, because it's broken | 12:19 |
sean-k-mooney | artom: actully the reson the multinode job was created was to test livemigration | 12:19 |
artom | But... we know it doens't work :) | 12:19 |
mriedem | artom: what do you need to get out of the instance xml? | 12:19 |
mriedem | the numa config? | 12:20 |
sean-k-mooney | artom: it was turned off temporaly feburay last year and then the ci moved team and i guess it never got turned back on | 12:20 |
artom | mriedem, yeah | 12:20 |
artom | sean-k-mooney, how did it ever pass before? | 12:20 |
mriedem | artom: we have the instance diagnostics api... | 12:20 |
sean-k-mooney | artom: livemigration works it just does not respect pinning after | 12:20 |
artom | sean-k-mooney, ah, indeed | 12:20 |
sean-k-mooney | e.g. the vm will actully livemigrate we just broke all SLAs | 12:21 |
*** mdbooth has quit IRC | 12:21 | |
mriedem | artom: you could potentially add a numa_details field to the instance diagnostics response and use that for validating things in tempest (likely a tempest plugin) | 12:21 |
artom | mriedem, I don't think those are enough | 12:21 |
sean-k-mooney | mriedem: the issue with multi numa testing in the upstream ci is we cannot spawn a 2 numa node guest on a host with 1 numa node | 12:22 |
sean-k-mooney | using the libvirt driver | 12:22 |
*** dave-mccowan has joined #openstack-nova | 12:22 | |
sean-k-mooney | that is a limitaion that i personally think should never have been there but that is a different issue | 12:22 |
*** ragiman has joined #openstack-nova | 12:23 | |
artom | sean-k-mooney, so is the intel CI multi-physical-node? Or multinode on VMs, so we can't have VMs with more than 1 node? | 12:23 |
artom | mriedem, would it really be a plugin at that point though? If it's 100% through the API, it's legit tempest | 12:24 |
*** dave-mcc_ has joined #openstack-nova | 12:24 | |
sean-k-mooney | artom: mriedem: all the host vms for the intel nfv ci have 2 numa nodes to allow numa testing, the phyical hosts have 2-4 numa nodes dending on the server the nodepool vm lands on | 12:24 |
*** lucas-hungry is now known as lucasagomes | 12:24 | |
mriedem | artom: meaning, it's only compute api | 12:24 |
mriedem | there isn't a real need to make all other projects gate on a numa test in the common tempest repo | 12:25 |
artom | mriedem, so how does it work currently for nova-only tempest tests? | 12:25 |
*** bkopilov_ has joined #openstack-nova | 12:25 | |
mriedem | those are legacy | 12:26 |
mriedem | and some are interop tests | 12:26 |
mriedem | the real basic stuff is interop | 12:26 |
artom | mriedem, oh, they're not accepting new tests that are single-service? | 12:26 |
mriedem | depends | 12:26 |
artom | Christ, it's getting less useful by the minute | 12:26 |
artom | (Sorry tempest folks!) | 12:26 |
*** dave-mccowan has quit IRC | 12:26 | |
mriedem | i just know that over time, single service, non-interop tests were supposed to be moved into the project tree or a tempest plugin for that repo | 12:27 |
mriedem | most other projects already have tempest plugins, i know cinder and neutron have had their own for a long time | 12:27 |
artom | I suppose it kinda make sense... | 12:27 |
*** bkopilov__ has joined #openstack-nova | 12:27 | |
artom | Keep the "useful to everyone" stuff in-tree, the rest can be out of scope in plugins | 12:28 |
artom | Anyways | 12:28 |
artom | So, I think first step for me is to get live migration re-enabled in the intel NFV CI | 12:28 |
mriedem | i'm no QA gate keeper, but just don't be surprised if that's what they tell you | 12:28 |
artom | They'll pass, sortof | 12:29 |
sean-k-mooney | artom: do you want to test multi numa guests or just guest with a numa topology | 12:29 |
artom | sean-k-mooney, uh, there's a difference? | 12:29 |
sean-k-mooney | artom: hw:numa_nodes=1 should work in the upstream ci | 12:29 |
artom | Ah, in that sense | 12:29 |
artom | Hrmm | 12:29 |
artom | True, true | 12:29 |
artom | mriedem, well, in the short term at least, nothing would stop me from proposing a patch to show the test passing | 12:30 |
sean-k-mooney | artom: cpu pinning will not work in the upstream ci however which is that the feature you really want to test? | 12:30 |
artom | And it it gets -2, then we can think about plugins | 12:30 |
artom | sean-k-mooney, well, everything, ideally | 12:31 |
artom | Even hugepages | 12:31 |
artom | I'm not writing any new NUMA code | 12:31 |
artom | Just calling the old one when live migrating | 12:31 |
*** alexchadin has quit IRC | 12:31 | |
artom | So technically just showing that it gets called for 1 NUMA-ish thing (and does the right thing) would be enough | 12:31 |
artom | But... the more coverage the better | 12:31 |
mriedem | melwitt: fyi i've marked https://blueprints.launchpad.net/nova/+spec/convert-consoles-to-objects complete | 12:31 |
mriedem | artom: sure | 12:31 |
*** jmlowe has joined #openstack-nova | 12:32 | |
mriedem | artom: also, a new test would get by the intel 3rd party ci blacklist which is currently based on test names | 12:32 |
artom | mriedem, oh, hah, ineed. Sneaky :D | 12:32 |
mriedem | so you'll probably have to do something like have your nova series, and then have a DNM nova patch on top that depends on the tempest change, | 12:33 |
*** alexchadin has joined #openstack-nova | 12:33 | |
mriedem | because the intel CI runs on nova changes, but probably not tempest changes | 12:33 |
sean-k-mooney | i have to run to a meeting but ill be back in an hour or 2 | 12:33 |
artom | mriedem, yep, and a patch to the intel CI plugin that does stuff like check instance XML | 12:34 |
*** claudiub has quit IRC | 12:34 | |
artom | mriedem, Or. Or! A patch to nova that adds what I need to the diagnostics API | 12:34 |
artom | Not sure what would be simpler. | 12:34 |
mriedem | hypervisor-specific stuff in tempest sucks, | 12:34 |
mriedem | which is why i suggested adding a new field to the diagnostics api | 12:35 |
mriedem | alternatively, | 12:35 |
mriedem | does any of this numa stuff for the guest get modeled in placement? | 12:35 |
mriedem | as a consumed resource? | 12:35 |
alex_xu | + | 12:35 |
artom | mriedem, some of it, I think? But allocations are still on the compute via resource tracker, I believe | 12:35 |
mriedem | alex likes it | 12:35 |
mriedem | the numa resource allocations would be on numa resource providers in the compute node provider tree | 12:36 |
mriedem | but given an instance (consumer) uuid, you can get it's resource class allocations against which providers in placement | 12:36 |
mriedem | so your test could assert that the instance has NUMA resource class allocations | 12:36 |
alex_xu | mriedem: my daugther just smash my keyboard... | 12:36 |
efried | nice find tetsuro | 12:37 |
mriedem | ha, lot of that going on today | 12:37 |
artom | alex_xu, she clearly didn't smash hard enough since we can all read what you're typing | 12:37 |
mriedem | artom: but i don't think the numa stuff is done, or close(?) | 12:37 |
artom | mriedem, I'm not sure placement would be enough, since we would ideally check specific CPUs, not just quantities | 12:37 |
artom | And pinning can't be checked at all | 12:38 |
mriedem | artom: i'm not sure if there is a placement solution in the works for that yet or not, but in that case you could just hack up the diagnostics api | 12:38 |
artom | mriedem, placement is a pool I swim in, but I still breath through a snorkel, so I don't know what liquid surrounds me | 12:38 |
artom | At some point I will need to grow gills to breath through the placement pool fluid | 12:39 |
alex_xu | artom: yea, a little hulk | 12:39 |
* artom thinks he has what he needs - time to drive kiddos to daycare | 12:40 | |
*** jmccarthy has quit IRC | 12:42 | |
*** jmccarthy1 has joined #openstack-nova | 12:42 | |
*** jmccarthy1 has left #openstack-nova | 12:43 | |
*** jmccarthy has joined #openstack-nova | 12:43 | |
openstackgerrit | Takahito Hirose proposed openstack/python-novaclient master: api_version decorator becomes an error in Python 3.5.0. https://review.openstack.org/564702 | 12:44 |
*** mdbooth has joined #openstack-nova | 12:46 | |
*** openstackgerrit has quit IRC | 12:49 | |
*** gyankum has quit IRC | 12:53 | |
*** jpena|lunch is now known as jpena | 12:57 | |
*** wznoinsk has joined #openstack-nova | 13:03 | |
*** brault has quit IRC | 13:04 | |
*** armaan has quit IRC | 13:05 | |
*** armaan has joined #openstack-nova | 13:06 | |
*** claudiub has joined #openstack-nova | 13:09 | |
*** mvenesio has joined #openstack-nova | 13:13 | |
*** bswrchrd has joined #openstack-nova | 13:14 | |
*** yamamoto has quit IRC | 13:16 | |
*** yamamoto has joined #openstack-nova | 13:16 | |
*** openstackgerrit has joined #openstack-nova | 13:16 | |
openstackgerrit | Kashyap Chamarthy proposed openstack/nova master: libvirt: Deprecate support for monitoring Intel CMT `perf` events https://review.openstack.org/565242 | 13:16 |
kashyap | mriedem: When you get a minute, I read the scrollback from yesterday here, and went with the: "deprecate in Rocky and hard-fail in Stein" | 13:17 |
*** moshele has joined #openstack-nova | 13:18 | |
*** yingjun has joined #openstack-nova | 13:19 | |
kashyap | I don't think I got the "assert_called_once_with" quite right here: https://review.openstack.org/#/c/565242/5/nova/tests/unit/virt/libvirt/test_driver.py@6623 | 13:19 |
*** dklyle has quit IRC | 13:19 | |
wznoinsk | mriedem, hi | 13:20 |
zzzeek | jaypipes: what would cause lock wait timeout exceeded for an INSERT? | 13:20 |
jaypipes | zzzeek: another thread executing LOCK TABLES <table>? | 13:21 |
zzzeek | jaypipes: just that? nothing more subtle? ceilometer is doing it | 13:21 |
* zzzeek greps the source, sigh | 13:22 | |
jaypipes | zzzeek: got a log output or something more for me? :) | 13:22 |
zzzeek | jaypipes: i have the error message and the query id have to spend time looking for the logs. | 13:22 |
*** armaan has quit IRC | 13:23 | |
zzzeek | jaypipes: but there's nothing like, the "auto increment" feature or somethign locks | 13:23 |
jaypipes | zzzeek: the only other thing I can think of would be threads attempting to execute huge transactions. | 13:23 |
jaypipes | zzzeek: all concurrently | 13:23 |
*** mingyu has quit IRC | 13:23 | |
zzzeek | jaypipes: right and then innodb locks ...a set of potential rows? | 13:23 |
*** armaan has joined #openstack-nova | 13:23 | |
jaypipes | zzzeek: no, autoinc won't produce that lock wait timeout generally, unless like I said, you have multiple threads simultaneously attempting to commit huge transactions (with thousands or tens of thousands of data modifications in each trx) | 13:24 |
jaypipes | zzzeek: yes, innodb will do its gap locks if the PK isn't autoinc. | 13:25 |
zzzeek | jaypipes: ok but in that csae, what is the lock that the INSERT is waiting for? OK gap locks. got it | 13:25 |
jaypipes | zzzeek: but again... you need some serious concurrency and huge trx to see this impact IME | 13:25 |
*** nsingh has left #openstack-nova | 13:25 | |
*** eharney has joined #openstack-nova | 13:25 | |
zzzeek | jaypipes: this is a load test | 13:25 |
*** liverpooler has joined #openstack-nova | 13:25 | |
BlackDex | Hello there. Does queens support active/active rw in multiple instance using ceph storage and the correct kvm version | 13:25 |
BlackDex | ? | 13:25 |
jaypipes | zzzeek: my guess would be ceilometer is attempting to commit batches of record changes. maybe try reducing the length of time between those commits? | 13:26 |
zzzeek | jaypipes: I dont even know wehre ceilometer's database code is | 13:26 |
jaypipes | zzzeek: what version? | 13:26 |
zzzeek | master | 13:26 |
jaypipes | zzzeek: lemme grep and see. | 13:27 |
jaypipes | zzzeek: been a very long time since I looked at ceilometer. | 13:27 |
zzzeek | jaypipes: $ find ceilometer/ -name "*.py" -exec grep -l sql {} \; | 13:27 |
zzzeek | [classic@photon2 ceilometer]$ | 13:27 |
zzzeek | zero | 13:27 |
zzzeek | they've hidden it | 13:27 |
*** claudiub has quit IRC | 13:28 | |
jaypipes | zzzeek: gnocchi is now the backend data storage for meters, though, right? | 13:28 |
*** felipemonteiro__ has joined #openstack-nova | 13:28 | |
zzzeek | that's pretty impressive the string "sql" does not appear in their source base at all | 13:28 |
jaypipes | zzzeek: ceilometer is just the polling thing right? | 13:28 |
zzzeek | jaypipes: right. but the log is the "ceilometer agent-notification" | 13:28 |
jaypipes | zzzeek: https://github.com/openstack/ceilometer/blob/master/ceilometer/gnocchi_client.py | 13:28 |
*** gongysh has joined #openstack-nova | 13:28 | |
zzzeek | jaypipes: table name is "event" | 13:30 |
zzzeek | jaypipes: isn't that the old mysql driver? | 13:30 |
jaypipes | zzzeek: no idea :( | 13:30 |
zzzeek | jaypipes: ok | 13:30 |
*** moshele has quit IRC | 13:31 | |
*** bswrchrd has quit IRC | 13:32 | |
*** bswrchrd has joined #openstack-nova | 13:32 | |
*** burt has joined #openstack-nova | 13:32 | |
*** burt has quit IRC | 13:33 | |
jaypipes | zzzeek: is this happening in like a tempest run or something? or is this in a prod env? | 13:33 |
jaypipes | zzzeek: https://github.com/openstack/ceilometer/blob/master/ceilometer/polling/manager.py#L46 <-- maybe try setting that to False and seeing if lock wait timeouts go down (due to smaller trx sizes) | 13:33 |
zzzeek | jaypipes: top seekrit :) | 13:33 |
* jaypipes grasping at straws here, though... doesn't know ceilomer. | 13:33 | |
*** burt has joined #openstack-nova | 13:33 | |
zzzeek | jaypipes: dont worry, you've been a great help :) | 13:34 |
*** yingjun has quit IRC | 13:34 | |
zzzeek | jaypipes: the issue here is writing to an "event" table and I think that is the ancient mysql backend | 13:34 |
*** rmart04 has joined #openstack-nova | 13:34 | |
jaypipes | zzzeek: yeah, sounds like it. | 13:34 |
*** felipemonteiro_ has joined #openstack-nova | 13:36 | |
*** dtantsur is now known as dtantsur|bbl | 13:36 | |
jaypipes | zzzeek: you sure this is master? https://github.com/openstack/ceilometer/commit/9323f07f977f320882f8b536c3b54835274826fc | 13:36 |
zzzeek | jaypipes: in the error I'm seeing? it is purportedly at least queens | 13:37 |
*** ralonsoh has quit IRC | 13:37 | |
*** burt has quit IRC | 13:37 | |
*** burt has joined #openstack-nova | 13:38 | |
jaypipes | zzzeek: I think you may need to reach out to jd__. | 13:39 |
zzzeek | jaypipes: yep | 13:39 |
*** felipemonteiro__ has quit IRC | 13:39 | |
*** burt has quit IRC | 13:39 | |
*** burt has joined #openstack-nova | 13:40 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove 'NUMATopologyLimits.obj_from_db_obj' https://review.openstack.org/537412 | 13:40 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_to_dict' functions https://review.openstack.org/537413 | 13:40 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.openstack.org/537414 | 13:40 |
*** mlavalle has joined #openstack-nova | 13:40 | |
*** Bonjourm8 has joined #openstack-nova | 13:41 | |
*** Bonjourm8 has quit IRC | 13:42 | |
*** yingjun has joined #openstack-nova | 13:44 | |
*** yingjun has quit IRC | 13:45 | |
*** yingjun has joined #openstack-nova | 13:46 | |
*** yingjun has quit IRC | 13:46 | |
*** yingjun has joined #openstack-nova | 13:47 | |
*** yingjun has quit IRC | 13:48 | |
*** yingjun has joined #openstack-nova | 13:49 | |
*** amotoki has quit IRC | 13:51 | |
artom | dansmith, is there a trick in func tests to start a compute service with a specific version? I *could* mock object.Service.get_by_compute_host, which is what I want to return an "older" Service, but then it messes up other stuff that calls it | 13:58 |
*** hongbin has joined #openstack-nova | 13:59 | |
*** ccamacho has quit IRC | 14:00 | |
*** ccamacho1 has joined #openstack-nova | 14:00 | |
*** ccamacho1 has quit IRC | 14:00 | |
*** ccamacho has joined #openstack-nova | 14:00 | |
*** awaugama has joined #openstack-nova | 14:00 | |
*** esberglu has joined #openstack-nova | 14:00 | |
*** awaugama has quit IRC | 14:01 | |
*** awaugama has joined #openstack-nova | 14:01 | |
*** dpawlik_ has joined #openstack-nova | 14:03 | |
*** kosamara has quit IRC | 14:03 | |
*** tianhui has joined #openstack-nova | 14:03 | |
*** wwriverrat_ has joined #openstack-nova | 14:03 | |
*** alexchadin has quit IRC | 14:03 | |
*** lbragstad has quit IRC | 14:05 | |
*** mriedem has quit IRC | 14:05 | |
*** dpawlik has quit IRC | 14:05 | |
*** wwriverrat has quit IRC | 14:05 | |
*** wwriverrat_ is now known as wwriverrat | 14:05 | |
*** lbragstad has joined #openstack-nova | 14:05 | |
*** tianhui_ has quit IRC | 14:05 | |
*** armaan has quit IRC | 14:07 | |
*** armaan has joined #openstack-nova | 14:07 | |
*** psachin` has quit IRC | 14:07 | |
*** moshele has joined #openstack-nova | 14:09 | |
kashyap | Cany unit test experts comment on what I can do differenlty here: https://review.openstack.org/#/c/565242/5/nova/tests/unit/virt/libvirt/test_driver.py@6623 | 14:11 |
kashyap | When I assert that, I get a: "AssertionError: Expected 'warning' to be called once. Called 4 times." | 14:12 |
dansmith | artom: yeah, that'd be failure-prone.. I'd just start it and then update its record manually | 14:13 |
dansmith | also, mocks in functional tests aren't good | 14:13 |
artom | dansmith, yeah... | 14:14 |
*** moshele has quit IRC | 14:14 | |
artom | dansmith, if you're up for it I can WIP-up what I got and you can give early feedback? | 14:14 |
dansmith | artom: okay | 14:14 |
artom | I'm basing it on existing tests, so... | 14:14 |
*** mingyu has joined #openstack-nova | 14:14 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: WIP: Service version check for NUMA live migration https://review.openstack.org/566723 | 14:15 |
artom | dansmith, ^^ | 14:15 |
*** yamamoto has quit IRC | 14:16 | |
dansmith | artom: you want me to comment about the mock then? | 14:17 |
artom | I want you to be happy :) | 14:17 |
artom | dansmith, in seriousness though, just... if I'm way off base, let me know so I can adjust my approach right away, instead of going down this rabbit hole | 14:17 |
*** hamzy has quit IRC | 14:18 | |
*** hamzy has joined #openstack-nova | 14:18 | |
*** moshele has joined #openstack-nova | 14:19 | |
dansmith | artom: just commented what I said above but with pseudocode | 14:19 |
dansmith | does that make sense? | 14:19 |
*** yamamoto has joined #openstack-nova | 14:19 | |
kashyap | Can anyone remind me again, mentioning text like these in Config file help is OK, right? | 14:20 |
kashyap | "Note that support for Intel CMT events (`cmt`, `mbmbt`, `mbml`) is deprecated in Nova, and will be removed in "Stein" release." | 14:20 |
kashyap | Because the config file help text is per release, it is okay... | 14:20 |
openstackgerrit | Julia Kreger proposed openstack/nova master: ironic: add instance_uuid before any other spawn activity https://review.openstack.org/563722 | 14:20 |
artom | dansmith, ah, yeah, that's probably smarter. Cheers! | 14:20 |
artom | kashyap, you can iterate through the calls to see what they were, maybe something else is logging at warn level that you haven't considered? | 14:21 |
kashyap | artom: This is purely help text. All I am asking is, is it okay to call out future release names like what I noted above is okay in the help text | 14:22 |
*** mriedem has joined #openstack-nova | 14:22 | |
artom | kashyap, I was answered your earlier question about the calls assetion :) | 14:22 |
artom | *answering | 14:23 |
artom | *asserting | 14:23 |
kashyap | artom: Aaah, darn. My memory is like a gold fish | 14:23 |
* artom is becoming sean-k-mooney | 14:23 | |
artom | kashyap, https://docs.python.org/3/library/unittest.mock.html#unittest.mock.Mock.mock_calls | 14:23 |
kashyap | artom: It is the specific warning: https://review.openstack.org/#/c/565242/5/nova/tests/unit/virt/libvirt/test_driver.py@6623 | 14:23 |
kashyap | On that line | 14:23 |
artom | kashyap, yeah, and as a debugging aid I'm suggesting you examine what the calls were | 14:24 |
kashyap | Which should log this: https://review.openstack.org/#/c/565242/5/nova/virt/libvirt/driver.py@4799 | 14:24 |
kashyap | artom: Yep, digging...Thx for the (non-null) pointer | 14:24 |
jroll | kashyap: if there's other warn calls happening, you could also do mock_warn.assert_has_calls([mock.call('Monitoring...')]) | 14:24 |
artom | kashyap, I'm thinking something else called LOG.warning somewhere along that test's execution | 14:25 |
kashyap | jroll: I don't think it's other warn calls, because I was calling it with a specific warning message | 14:25 |
*** armaan has quit IRC | 14:25 | |
artom | If those were legit calls, you can adjust your tests to only assert on the call you care about | 14:25 |
kashyap | jroll: But let me try your trick. | 14:25 |
jroll | kashyap: what artom said :) | 14:25 |
artom | If they weren't legit, you fix your code :) | 14:25 |
jroll | ++ | 14:26 |
*** armaan has joined #openstack-nova | 14:26 | |
* kashyap looks | 14:26 | |
kashyap | Thx for the comments, folks | 14:26 |
artom | Btw, asserting on *log messages* is horrible testing practive | 14:27 |
artom | *practice | 14:27 |
artom | I know Nova is side-effect land, so we don't have much choice | 14:27 |
artom | But in an ideal world, we'd be asserting stuff on output, given a certain input | 14:27 |
kashyap | artom: I was actually asked to do it. I firt did the self.assertTrue(mock_warn.called) | 14:27 |
kashyap | And the libvirt driver at least doesn't do a lot of those. Only 3 or so of them -- asserting on log message. | 14:28 |
artom | kashyap, yeah, I fully understand the reality of what we're working on | 14:28 |
kashyap | And if it's any consolation, this warning will be removed in the next release. | 14:28 |
*** moshele has quit IRC | 14:29 | |
*** mchlumsky has joined #openstack-nova | 14:30 | |
*** Nil_ has joined #openstack-nova | 14:33 | |
openstackgerrit | Merged openstack/nova master: Changing scheduler sync event from INFO to DEBUG https://review.openstack.org/566392 | 14:34 |
*** Eran_Kuris has quit IRC | 14:38 | |
*** dklyle has joined #openstack-nova | 14:42 | |
*** wwriverrat has quit IRC | 14:43 | |
*** rcernin has quit IRC | 14:47 | |
*** rajinir has joined #openstack-nova | 14:49 | |
*** ragiman has quit IRC | 14:50 | |
*** yingjun has quit IRC | 14:51 | |
*** sapd1 has joined #openstack-nova | 14:52 | |
*** udesale has quit IRC | 14:53 | |
*** hemna_ has joined #openstack-nova | 14:55 | |
*** r-daneel has joined #openstack-nova | 14:55 | |
jmccarthy | mriedem: Thanks again for the help, this experimental review did the trick for me https://review.openstack.org/#/c/566367 | 14:58 |
*** gongysh has quit IRC | 14:59 | |
*** gyankum has joined #openstack-nova | 15:00 | |
*** rmart04 has quit IRC | 15:01 | |
mriedem | jmccarthy: np, thanks for verifying it | 15:05 |
kashyap | jroll: Your trick tells me my assertion was not even in the call list. ("not all found in call list"). | 15:06 |
jroll | interesting | 15:06 |
*** jungleboyj has joined #openstack-nova | 15:07 | |
* kashyap dukes around a bit more | 15:07 | |
*** r-daneel has quit IRC | 15:08 | |
*** links has quit IRC | 15:08 | |
mriedem | jmccarthy: just tested mel's suggestion in there, i'll get that patch updated later today | 15:21 |
*** bhujay has quit IRC | 15:21 | |
jmccarthy | mriedem: Ah ok cool - I can give that a whirl also to confirm (but probably not till later) | 15:22 |
*** andreas_s_ has quit IRC | 15:25 | |
*** andreas_s has joined #openstack-nova | 15:26 | |
*** zcorneli has joined #openstack-nova | 15:26 | |
*** felipemonteiro has joined #openstack-nova | 15:30 | |
*** felipemonteiro__ has joined #openstack-nova | 15:31 | |
*** gjayavelu has joined #openstack-nova | 15:31 | |
*** felipemonteiro_ has quit IRC | 15:32 | |
*** felipemonteiro has quit IRC | 15:35 | |
*** andreas_s has quit IRC | 15:35 | |
*** andreas_s has joined #openstack-nova | 15:40 | |
*** lucasagomes is now known as lucas-bbl | 15:42 | |
*** burt has quit IRC | 15:42 | |
*** burt has joined #openstack-nova | 15:42 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use query_parameters in granular gabbit https://review.openstack.org/566927 | 15:46 |
openstackgerrit | Aditya Vaja proposed openstack/nova master: remove IVS plug/unplug as they're moved to separate plugin https://review.openstack.org/534371 | 15:48 |
*** ccamacho has quit IRC | 15:49 | |
*** zhaochao has quit IRC | 15:50 | |
*** gyan_ has joined #openstack-nova | 15:51 | |
stephenfin | jaypipes, bauzas, sean-k-mooney: Fancy taking another swing at https://review.openstack.org/#/c/541290/ today? | 15:52 |
*** andreas_s has quit IRC | 15:53 | |
jaypipes | stephenfin: ack, will do. sorry, was swamped with other stuff yesterday and didn't get to that. | 15:53 |
stephenfin | no worries :) /me is going through bandwidth-resource-provider again atm | 15:54 |
*** gyankum has quit IRC | 15:54 | |
openstackgerrit | Brianna Poulos proposed openstack/nova master: Add notification support for trusted_certs https://review.openstack.org/563269 | 15:55 |
*** cdent has joined #openstack-nova | 15:58 | |
*** gjayavelu has quit IRC | 16:00 | |
*** edmondsw has quit IRC | 16:00 | |
*** cfriesen_ has quit IRC | 16:01 | |
*** cfriesen_ has joined #openstack-nova | 16:01 | |
*** bpoulos has joined #openstack-nova | 16:01 | |
*** edmondsw has joined #openstack-nova | 16:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData object for live migration https://review.openstack.org/515423 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 16:01 |
*** janki has quit IRC | 16:01 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932 | 16:01 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933 | 16:01 |
*** zcorneli has quit IRC | 16:03 | |
*** dtantsur|bbl is now known as dtantsur | 16:04 | |
*** gyee has joined #openstack-nova | 16:04 | |
*** bhujay has joined #openstack-nova | 16:05 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537 | 16:07 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: DNM: test live_migration_wait_for_vif_plug=True https://review.openstack.org/558006 | 16:07 |
*** cfriesen_ has quit IRC | 16:08 | |
*** wwriverrat has joined #openstack-nova | 16:09 | |
*** andreas_s has joined #openstack-nova | 16:16 | |
*** moshele has joined #openstack-nova | 16:17 | |
*** harlowja has joined #openstack-nova | 16:17 | |
*** cfriesen_ has joined #openstack-nova | 16:19 | |
*** yamamoto has quit IRC | 16:21 | |
*** andreas_s has quit IRC | 16:25 | |
*** andreas_s has joined #openstack-nova | 16:25 | |
*** ccamacho has joined #openstack-nova | 16:28 | |
*** gjayavelu has joined #openstack-nova | 16:30 | |
*** imacdonn has joined #openstack-nova | 16:34 | |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove 'NUMATopologyLimits.obj_from_db_obj' https://review.openstack.org/537412 | 16:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_to_dict' functions https://review.openstack.org/537413 | 16:35 |
openstackgerrit | Stephen Finucane proposed openstack/nova master: objects: Remove legacy '_from_dict' functions https://review.openstack.org/537414 | 16:35 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: check image type before removing snapshots in _cleanup_resize https://review.openstack.org/566367 | 16:36 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove old rbd snapshot removal error handling https://review.openstack.org/566369 | 16:36 |
*** fragatina has quit IRC | 16:36 | |
*** gjayavelu has quit IRC | 16:36 | |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Add InstanceNUMATopology to LibvirtLiveMigrateData https://review.openstack.org/566398 | 16:37 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 16:37 |
*** andreas_s has quit IRC | 16:38 | |
*** jmccarthy has left #openstack-nova | 16:39 | |
*** yamamoto has joined #openstack-nova | 16:39 | |
*** jmccarthy has joined #openstack-nova | 16:39 | |
*** jmccarthy has left #openstack-nova | 16:39 | |
*** Guest47185 has quit IRC | 16:41 | |
*** dklyle has quit IRC | 16:48 | |
*** EmilienM is now known as EmilienM_PTO | 16:49 | |
*** mingyu has quit IRC | 16:49 | |
*** mingyu has joined #openstack-nova | 16:49 | |
*** gyan_ has quit IRC | 16:52 | |
*** sapd1 has quit IRC | 16:53 | |
openstackgerrit | Eric Fried proposed openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 16:54 |
*** derekh has quit IRC | 16:57 | |
*** ccamacho has quit IRC | 16:58 | |
*** ccamacho has joined #openstack-nova | 16:58 | |
openstackgerrit | Merged openstack/nova master: libvirt: Fix misleading debug msg "Instance is running" https://review.openstack.org/565234 | 17:01 |
*** sambetts is now known as sambetts|afk | 17:02 | |
*** dklyle has joined #openstack-nova | 17:04 | |
*** mgoddard has quit IRC | 17:04 | |
*** jpena is now known as jpena|off | 17:08 | |
*** gjayavelu has joined #openstack-nova | 17:08 | |
*** zcorneli has joined #openstack-nova | 17:09 | |
*** fragatina has joined #openstack-nova | 17:09 | |
*** yamamoto has quit IRC | 17:10 | |
openstackgerrit | Dan Smith proposed openstack/nova master: WIP: Use oslo.messaging per-call monitoring https://review.openstack.org/566696 | 17:11 |
*** zcorneli has quit IRC | 17:13 | |
*** zcorneli has joined #openstack-nova | 17:16 | |
*** gyee has quit IRC | 17:17 | |
*** cfriesen_ has quit IRC | 17:20 | |
*** gjayavelu has quit IRC | 17:23 | |
*** slaweq has quit IRC | 17:25 | |
*** slaweq has joined #openstack-nova | 17:26 | |
*** hamzy has quit IRC | 17:28 | |
*** mdnadeem has quit IRC | 17:29 | |
openstackgerrit | Merged openstack/nova-specs master: Network bandwidth resource provider https://review.openstack.org/502306 | 17:30 |
gibi | \o/ ^^ | 17:30 |
*** harlowja has quit IRC | 17:31 | |
*** slaweq has quit IRC | 17:31 | |
*** mdbooth has quit IRC | 17:36 | |
efried | mriedem: In ocata and/or pike, in a bfv situation, is/was there a bug where the scheduler would require the host to have localdisk inventory in the amount requested by the flavor, even though it wasn't ultimately allocated/consumed? | 17:37 |
*** dklyle has quit IRC | 17:37 | |
*** bhujay has quit IRC | 17:39 | |
*** slaweq has joined #openstack-nova | 17:41 | |
*** cdent has quit IRC | 17:42 | |
*** slaweq has quit IRC | 17:45 | |
*** amoralej is now known as amoralej|off | 17:46 | |
*** tssurya has quit IRC | 17:50 | |
*** jlvillal is now known as jlvacation | 17:51 | |
*** r-daneel has joined #openstack-nova | 17:52 | |
*** gjayavelu has joined #openstack-nova | 17:56 | |
mriedem | efried: https://bugs.launchpad.net/nova/+bug/1469179 ? | 17:57 |
openstack | Launchpad bug 1469179 in OpenStack Compute (nova) "instance.root_gb should be 0 for volume-backed instances" [Medium,In progress] - Assigned to Dan Smith (danms) | 17:57 |
*** felipemonteiro__ has quit IRC | 18:10 | |
*** felipemonteiro__ has joined #openstack-nova | 18:10 | |
*** yamamoto has joined #openstack-nova | 18:11 | |
*** dtantsur is now known as dtantsur|afk | 18:19 | |
*** gyee has joined #openstack-nova | 18:19 | |
*** hamzy has joined #openstack-nova | 18:20 | |
*** Bonjourm8 has joined #openstack-nova | 18:20 | |
*** yamamoto has quit IRC | 18:21 | |
*** cfriesen_ has joined #openstack-nova | 18:21 | |
efried | Thanks mriedem | 18:23 |
*** harlowja has joined #openstack-nova | 18:25 | |
*** jaosorior has quit IRC | 18:26 | |
*** sridharg has quit IRC | 18:29 | |
*** strigazi has quit IRC | 18:30 | |
*** strigazi has joined #openstack-nova | 18:31 | |
*** psachin` has joined #openstack-nova | 18:33 | |
*** pchavva has quit IRC | 18:39 | |
*** Swami has joined #openstack-nova | 18:39 | |
*** tssurya has joined #openstack-nova | 18:42 | |
dansmith | tssurya: have you looked over this yet? https://review.openstack.org/#/c/509003/ | 18:44 |
dansmith | mriedem: melwitt: do you both agree that having a config flag for gating that ^ vs. just hitting all cells is a good idea? | 18:44 |
tssurya | dansmith: oops, not yet, will review now | 18:44 |
dansmith | tssurya: okay I want to make sure it will work for you | 18:44 |
*** Swami has quit IRC | 18:45 | |
mriedem | umm, | 18:47 |
mriedem | since most people aren't >1 cell yet, that adds an extra db hit to every GET /servers operation right? | 18:47 |
*** openstackgerrit has quit IRC | 18:49 | |
dansmith | yeah, which is why I think it probably needs a config knob | 18:50 |
dansmith | it should be a huge benefit to cern, but not most people | 18:51 |
tssurya | dansmith: this works for us since we have a lot of cells, but I think from a developer perspective we wouldn't want to retrieve the mappings every time , so I vote for a config option -> same logic in us doing refresh handler for disable cells to maintain the cache. | 18:51 |
*** Bonjourm8 has quit IRC | 18:52 | |
dansmith | aye | 18:52 |
jaypipes | dansmith: my thought was that such an option could easily be added if it becomes an issue later. | 19:00 |
mriedem | dansmith: well, you could create 1000 fake instances and compare the average time to list them either way and see if the extra join is a noticeable impact | 19:01 |
dansmith | mriedem: extra join? | 19:02 |
mriedem | instance mappings -> cell mappings | 19:02 |
jaypipes | mriedem: it's not a join. just an extra query against instance_mappings IIRC | 19:02 |
mriedem | the upfront extra join to filter cell mappings based on project_id | 19:02 |
dansmith | it's a whole extra hit, not just a join | 19:02 |
mriedem | the "new extra db kerfuffle" | 19:02 |
dansmith | I prefer to just make it default to continue using the cache, and let people like cern opt into this behavior | 19:03 |
melwitt | mriedem: sweet, thanks for closing out the bp | 19:09 |
*** dklyle has joined #openstack-nova | 19:09 | |
jaypipes | stephenfin: still around? | 19:10 |
*** Swami has joined #openstack-nova | 19:10 | |
*** Swami_ has joined #openstack-nova | 19:10 | |
jaypipes | stephenfin: what precisely do you mean by "mechanisms" in "Like provider networks, tenant networks can use one of five mechanisms: *flat*, *VLAN*, *local*, *VXLAN*, or *GRE*."? In particular, what is "local mechanism"? | 19:11 |
melwitt | dansmith: config option sounds okay ... the only bad side is, yet-another-config-option. and I'm thinking how can we group cells-related options, CONF.cells is already taken by cells v1 | 19:12 |
jaypipes | melwitt: CONF.cern? ;) | 19:13 |
mriedem | CONF.api.something | 19:13 |
melwitt | heh | 19:13 |
mriedem | CONF.api.multicell_project_per_cell | 19:13 |
tssurya | jaypipes: :P | 19:13 |
jaypipes | tssurya: :) | 19:13 |
*** dave-mcc_ has quit IRC | 19:14 | |
mriedem | fortunately we already have a project_id index in the instance_mappings table | 19:15 |
mriedem | this unblocks osc-placement if someone wants to give the +W https://review.openstack.org/#/c/565694/ | 19:17 |
jaypipes | mriedem: done | 19:19 |
mriedem | thanks | 19:19 |
*** dave-mccowan has joined #openstack-nova | 19:19 | |
*** david-lyle has joined #openstack-nova | 19:27 | |
arvindn05 | mriedem: quick question on your comment for https://review.openstack.org/#/c/560596 - update scheduler to use image-traits | 19:30 |
*** psachin` has quit IRC | 19:30 | |
arvindn05 | >>have a functional test for a volume-backed instance to show that we don't consider the required image traits | 19:30 |
arvindn05 | for volume backed instance, there is no image right? so where do the image traits come in? | 19:30 |
arvindn05 | tests for launching volume backed instance should already capture this case correct? | 19:31 |
*** tbachman has joined #openstack-nova | 19:33 | |
*** openstackgerrit has joined #openstack-nova | 19:33 | |
openstackgerrit | Chris Dent proposed openstack/nova master: Optional separate database for placement API https://review.openstack.org/362766 | 19:33 |
openstackgerrit | Chris Dent proposed openstack/nova master: Isolate placement database config https://review.openstack.org/541435 | 19:33 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Ensure that os-traits sync is attempted only at start of process https://review.openstack.org/553857 | 19:33 |
openstackgerrit | Chris Dent proposed openstack/nova master: WIP: Add PLACEMENT_DB_ENABLED=True to the nova-next job https://review.openstack.org/564067 | 19:34 |
mriedem | arvindn05: the root volume has an image in it | 19:35 |
mriedem | which could have required traits, | 19:35 |
mriedem | but yes that doesn't get put into the request spec so we don't filter on the image traits for volume-backed instances | 19:35 |
arvindn05 | mriedem: ahh..ok..dont know a lot about volume backed instances | 19:36 |
mriedem | arvindn05: a test could be that i have a single compute and an image with a required trait CUSTOM_FOO, and the compute doesn't have that trait. trying to create a server with just the image fails with novalidhost, but trying to create a volume-backed instance using that image passes because we don't filter on the image required traits in that case | 19:36 |
arvindn05 | i am guessing this should be in a single method? because the first part of the test is already part of the current functional test, but we will need to duplicate so we dont get intermittent success/failures? | 19:38 |
arvindn05 | or i can just create a test for volume backed instance case and use an image with trait and prove that it does not throw novalidhost | 19:40 |
*** slaweq has joined #openstack-nova | 19:41 | |
mriedem | test_image_traits_based_scheduling_no_valid_host already tests the first thing i said, right | 19:41 |
arvindn05 | mriedem: is this a good example of how i can create a volume backed instance https://github.com/openstack/nova/blob/master/nova/tests/functional/test_servers.py#L1330 | 19:41 |
mriedem | so you'd basically copy test_image_traits_based_scheduling_no_valid_host but _create_server_with_traits would include block_device_mapping_v2 | 19:41 |
mriedem | yes and no, | 19:42 |
*** awaugama has quit IRC | 19:42 | |
mriedem | that relies on https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1323 to boot from an existing bootable volume in the fixture | 19:42 |
mriedem | so for your test, the image 155d900f-4e14-4e4c-a73d-069cbf4541e6 would need to have the required trait on it | 19:43 |
mriedem | see https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1409 | 19:43 |
mriedem | i think you get the idea though | 19:44 |
mriedem | and it's something that will need to be in whatever image property documentation exists for required traits | 19:44 |
*** bpoulos has quit IRC | 19:45 | |
arvindn05 | yup...we have a patch for all the documentation...will handle it as part of that | 19:45 |
*** slaweq has quit IRC | 19:45 | |
openstackgerrit | Merged openstack/osc-placement master: Fix error message in test assert https://review.openstack.org/565694 | 19:49 |
*** moshele has quit IRC | 19:51 | |
efried | melwitt: Is there a bug for the intermittent nova-next failure? | 19:51 |
melwitt | efried: there was this one from yesterday https://launchpad.net/bugs/1769286 | 19:52 |
openstack | Launchpad bug 1769286 in devstack "NoVNCConsoleTestJSON.test_novnc intermittently fails with: SecurityProxyNegotiationFailed: Failed to negotiate security type with server: No compute auth available: No matching auth scheme: allowed types: 'AuthType.NONE', desired types: '19'" [High,Fix released] - Assigned to melanie witt (melwitt) | 19:52 |
melwitt | efried: are you seeing new failures? | 19:53 |
efried | melwitt: Hum, okay, so this is not the same one: http://logs.openstack.org/57/517757/38/check/nova-next/4d5fdba/job-output.txt.gz#_2018-05-08_18_40_52_205203 | 19:53 |
mriedem | that's not vnc | 19:53 |
mriedem | and is a known latent bug | 19:53 |
mriedem | http://status.openstack.org/elastic-recheck/#1732199 | 19:53 |
efried | mriedem: Thanks. I knew there's no way it could have anything to do with my 1.3KLOC patch. | 19:54 |
melwitt | :) | 19:54 |
melwitt | is e-r working these days? I feel like it's been at least a year since I saw it comment on any review | 19:54 |
efried | I thought once something was registered with elastic-recheck it posted comments to the gerrit review... | 19:54 |
efried | yeah, what melwitt said. | 19:54 |
arvindn05 | mriedem: Can i add a IMAGE_WITH_TRAITS_BACKED_VOL to CinderFixture for adding the unit test? or is there a different approach i should take... | 19:55 |
arvindn05 | *CinderFixtureNewAttachFlow | 19:57 |
*** wolverineav has quit IRC | 19:58 | |
*** wolverineav has joined #openstack-nova | 19:58 | |
melwitt | I remember asking about this VolumePathsNotFound during extend_volume http://logs.openstack.org/57/517757/38/check/nova-next/4d5fdba/logs/screen-n-cpu.txt.gz?level=TRACE#_May_08_18_35_13_576026 in the cinder channel awhile back, but no one had any ideas. it looks like something to do with iscsiadm not able to find or read the session | 19:59 |
*** munimeha has joined #openstack-nova | 20:00 | |
melwitt | and I had no idea what that means or what could cause it | 20:00 |
*** wolverineav has quit IRC | 20:03 | |
mriedem | elastic-recheck used to not comment on changes because we'd hit a 15 minute timeout to process logs | 20:05 |
melwitt | I've asked in -infra | 20:05 |
melwitt | looks like anytime indexing is behind, it won't comment as it won't want to post stale info http://status.openstack.org/elastic-recheck/ | 20:05 |
melwitt | maybe it's perpetually behind | 20:06 |
mriedem | arvindn05: the fixture is using a "real" image in the fake image service registry, so you could just update that image with required traits https://github.com/openstack/nova/blob/master/nova/tests/unit/image/fake.py#L44 | 20:06 |
mriedem | in your test, like you're doing in setUp for the other image | 20:07 |
*** esberglu has quit IRC | 20:07 | |
*** avolkov has quit IRC | 20:10 | |
arvindn05 | cool...one other question though i am looking at https://developer.openstack.org/api-ref/block-storage/v3/#list-accessible-volumes-with-details | 20:11 |
arvindn05 | "volume_image_metadata": { | 20:12 |
arvindn05 | "kernel_id": "8a55f5f1-78f7-4477-8168-977d8519342c", | 20:12 |
arvindn05 | "image_id": "b48c53e1-9a96-4a5a-a630-2e74ec54ddcc", | 20:12 |
arvindn05 | .... | 20:12 |
arvindn05 | }, | 20:12 |
arvindn05 | does volume image metadata ever even contain the image meta props like hw_architecture etc? | 20:12 |
arvindn05 | if the image meta data for a volume is just the image attributes rather than any of the standard fields of ImageMetaProps then again the test may not make sense | 20:13 |
mriedem | the volume's volume_image_metadata is a copy of the image metadata from the image or snapshot used to create the volume | 20:15 |
mriedem | so if we ever wanted/needed to actually filter volume-backed instances based on image metadata, we can totally do that by getting the image meta via the volume_image_metadata, we just don't do that today | 20:16 |
mriedem | http://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/flows/manager/create_volume.py#n559 | 20:17 |
arvindn05 | so if i have an image with trait:foo, then for a volume based on that image the meta data would look like below | 20:19 |
arvindn05 | "volume_image_metadata": { | 20:19 |
arvindn05 | "image_id": "b48c53e1-9a96-4a5a-a630-2e74ec54ddcc", | 20:19 |
arvindn05 | "trait:foo" "required" | 20:19 |
arvindn05 | .... | 20:19 |
arvindn05 | }, | 20:19 |
arvindn05 | or would the trait:foo field be missing and looked up when the image is loaded with the ID | 20:19 |
*** esberglu has joined #openstack-nova | 20:19 | |
mriedem | arvindn05: maybe you should fire up a devstack and test this out for yourself | 20:19 |
arvindn05 | if its the above with trait:foo being part of volume_image_metadata then i would need to add to CinderFixtureNewAttachFlow else i can do your suggestion of updating the image directly | 20:20 |
*** pcaruana has quit IRC | 20:21 | |
mriedem | https://github.com/openstack/nova/blob/master/nova/tests/fixtures.py#L1409 is not attempting to be an exhaustive list of what goes into volume_image_metadata for the cinder fixture | 20:21 |
arvindn05 | mriedem: havent worked with cinder before...so trying to cover the bases...will update my devstack to have cinder and try out as well | 20:21 |
arvindn05 | yea...if you knew off the top of the head if volume_image_metadata enumerates all ImageMetaProps(if defined on the image) that would help me get a headstart with the test | 20:23 |
*** dklyle has quit IRC | 20:32 | |
*** edmondsw has quit IRC | 20:33 | |
openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Fix the policy argument in server-group-create https://review.openstack.org/564699 | 20:37 |
*** hemna_ has quit IRC | 20:41 | |
*** ttsiouts_ has joined #openstack-nova | 20:46 | |
*** slaweq has joined #openstack-nova | 20:49 | |
*** andreas_s has joined #openstack-nova | 20:52 | |
openstackgerrit | karim proposed openstack/nova master: Pushing image traits to ironic node https://review.openstack.org/565620 | 20:55 |
*** andreas_s has quit IRC | 20:57 | |
openstackgerrit | Eric Fried proposed openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822 | 20:58 |
*** tssurya has quit IRC | 20:59 | |
mriedem | melwitt: i don't know if this is a bug fix or a feature or what https://review.openstack.org/#/c/564193/ | 21:05 |
* melwitt looks | 21:06 | |
openstackgerrit | Merged openstack/nova master: Bump pypowervm minimum to 1.1.15 https://review.openstack.org/564276 | 21:06 |
openstackgerrit | Artom Lifshitz proposed openstack/nova master: Service version check for NUMA live migration https://review.openstack.org/566723 | 21:08 |
melwitt | mriedem: yeah, looks like a driver parity thing. not sure if it's a feature either, not an expert on what image meta props really are | 21:08 |
mriedem | yeah it's adding support for hw_video_ram to the vmware driver | 21:10 |
mriedem | so not a bug | 21:11 |
mriedem | the commit message totally makes it sound like it's fixing something, but it's not | 21:11 |
mriedem | "fix this new thing that i'm adding" | 21:11 |
melwitt | yeah, agreed. good doc finds in your comment | 21:12 |
*** hemna_ has joined #openstack-nova | 21:13 | |
*** felipemonteiro_ has joined #openstack-nova | 21:13 | |
melwitt | we usually do blueprints for driver parity things but this is indeed a lot smaller than most of the parity things we usually see. kind of more along the lines of adding the aarch64 support stuff maybe? | 21:13 |
melwitt | so maybe don't need a blueprint but do need release note and doc update at a minimum | 21:16 |
mriedem | yeah i don't think we need a blueprint either | 21:17 |
mriedem | just reno and doc updates | 21:17 |
*** felipemonteiro__ has quit IRC | 21:17 | |
melwitt | ++ | 21:17 |
melwitt | looks like the doc update would be in glance though | 21:18 |
mriedem | for the image properties one yeah, | 21:18 |
mriedem | but we should also document the flavor extra spec in our docs | 21:18 |
melwitt | yeah -- just not sure it has to fall on their change though, since it's currently undocumented even for the libvirt driver | 21:19 |
mriedem | melwitt: ok, i'll do them a solid and push a patch below theirs that adds the flavor extra spec doc, and they can then update it | 21:20 |
melwitt | sounds super | 21:20 |
*** swamireddy has quit IRC | 21:20 | |
*** liverpooler has quit IRC | 21:22 | |
*** david-lyle has quit IRC | 21:23 | |
openstackgerrit | Merged openstack/nova master: Remove stale pip-missing-reqs tox test https://review.openstack.org/565535 | 21:32 |
*** dhellmann has left #openstack-nova | 21:37 | |
*** felipemonteiro__ has joined #openstack-nova | 21:38 | |
*** felipemonteiro_ has quit IRC | 21:38 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Don't confuse CPU pinning/NUMA as Hyper-V only https://review.openstack.org/561949 | 21:40 |
mriedem | stephenfin: we need to get -W working in the docs builds again ^ | 21:40 |
*** markvoelker_ has joined #openstack-nova | 21:41 | |
*** markvoelker has quit IRC | 21:42 | |
*** ttsiouts_ has quit IRC | 21:48 | |
*** hjensas is now known as hjensas|afk | 21:49 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822 | 21:50 |
openstackgerrit | Merged openstack/nova master: placement: Object changes for granular https://review.openstack.org/564351 | 21:51 |
sean-k-mooney | mriedem: huh when did they make that a config option on the host instead of using the there custom extra spec option | 21:51 |
mriedem | sean-k-mooney: context? | 21:51 |
sean-k-mooney | hyperv numa affinity | 21:52 |
mriedem | that's a question for claudiu | 21:52 |
mriedem | or git blame | 21:52 |
sean-k-mooney | hyperv used (like 4/5 releases ago) have a addtion hw:somting extra spec that allowed you to set the numa affinity of the guest memory | 21:53 |
sean-k-mooney | i went looking for it simi recently but could not find it so i assumed it was removed but i guest its now a host config option | 21:54 |
mriedem | dansmith: i left comments in https://review.openstack.org/#/c/509002/ but didn't want to -1 | 21:55 |
sean-k-mooney | mriedem: ah it was hw:numa_mempolicy=preferred|strict http://git.openstack.org/cgit/openstack/nova-specs/tree/specs/liberty/approved/hyper-v-vnuma-enable.rst#n78 | 21:57 |
sean-k-mooney | ah it looks like they did not complete the spec until ocata and that extra spec got lost along the way. that is a shame i wanted to add that to the libvirt driver at some point. | 22:01 |
*** burt has quit IRC | 22:01 | |
dansmith | mriedem: ack, thanks | 22:04 |
arvindn05 | mriedem: FYI tested with image backed volume the image meta data needs to be part of the volume_image_metadata...so will need to modify the fixture..will write the funtional test with that in mind :) | 22:09 |
arvindn05 | "volume_image_metadata": { | 22:09 |
arvindn05 | "image_id": "39f216fd-ebba-4a6a-b741-c83cd02a00ec", | 22:09 |
arvindn05 | "trait:CUSTOM_TRUSTED": "required", | 22:09 |
arvindn05 | ... | 22:09 |
arvindn05 | } | 22:09 |
*** esberglu has quit IRC | 22:12 | |
openstackgerrit | Matt Riedemann proposed openstack/nova master: vmware:Validation video ram against the max video ram allowed. https://review.openstack.org/564193 | 22:17 |
openstackgerrit | Matt Riedemann proposed openstack/nova master: Add docs for hw_video:ram_max_mb flavor extra spec https://review.openstack.org/567070 | 22:17 |
*** Guest87997 has quit IRC | 22:21 | |
*** rcernin has joined #openstack-nova | 22:25 | |
*** yassine has joined #openstack-nova | 22:28 | |
*** yassine is now known as Guest83896 | 22:29 | |
openstackgerrit | Merged openstack/osc-placement master: RP delete inventories (v1.5) https://review.openstack.org/514642 | 22:29 |
*** felipemonteiro_ has joined #openstack-nova | 22:34 | |
*** felipemonteiro__ has quit IRC | 22:37 | |
*** Guest83896 has quit IRC | 22:37 | |
melwitt | mriedem: I dunno if you saw my reply yesterday about the stable releases. I was thinking maybe we just do the releases after we get the regression fixes we know about in? what was your thought on it? do you think we should go ahead and flush the stable review queues too? | 22:38 |
*** hongbin has quit IRC | 22:48 | |
*** Guest83896 has joined #openstack-nova | 22:52 | |
*** moshele has joined #openstack-nova | 22:55 | |
*** slaweq has quit IRC | 22:57 | |
*** swamireddy has joined #openstack-nova | 22:59 | |
*** felipemonteiro__ has joined #openstack-nova | 23:04 | |
*** felipemonteiro_ has quit IRC | 23:04 | |
openstackgerrit | Merged openstack/nova master: Handle PortNotFoundClient exception when getting ports https://review.openstack.org/565183 | 23:09 |
melwitt | interesting, this old experimental queue job failing because it wants to install the liberty cloud archive which isn't supported on xenial http://logs.openstack.org/37/554437/18/experimental/legacy-tempest-dsvm-nova-libvirt-kvm-apr/e50a428/logs/devstacklog.txt.gz#_2018-05-08_22_58_11_828 | 23:12 |
*** felipemonteiro__ has quit IRC | 23:14 | |
openstackgerrit | Merged openstack/python-novaclient master: Fix the policy argument in server-group-create https://review.openstack.org/564699 | 23:18 |
*** vivsoni has joined #openstack-nova | 23:21 | |
*** vivsoni_ has quit IRC | 23:24 | |
*** r-daneel has quit IRC | 23:26 | |
*** munimeha has quit IRC | 23:30 | |
*** bkopilov_ has quit IRC | 23:31 | |
*** bkopilov__ has quit IRC | 23:32 | |
*** abalutoiu_ has quit IRC | 23:41 | |
*** bkopilov__ has joined #openstack-nova | 23:44 | |
*** mlavalle has quit IRC | 23:44 | |
*** bkopilov_ has joined #openstack-nova | 23:44 | |
openstackgerrit | Merged openstack/nova master: placement: Granular GET /allocation_candidates https://review.openstack.org/517757 | 23:47 |
openstackgerrit | Merged openstack/nova master: doc: Don't confuse CPU pinning/NUMA as Hyper-V only https://review.openstack.org/561949 | 23:47 |
openstackgerrit | Merged openstack/nova master: Use .. deprecated:: theme for deprecations https://review.openstack.org/565822 | 23:47 |
*** moshele has quit IRC | 23:50 | |
*** bkopilov__ has quit IRC | 23:52 | |
*** bkopilov_ has quit IRC | 23:52 | |
mriedem | melwitt: didn't see it, but these are the regressions i had in mind: https://review.openstack.org/#/c/564461/ https://review.openstack.org/#/c/566367/ https://review.openstack.org/#/c/563213/ | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!