| *** spatel has quit IRC | 00:16 | |
| *** nicolasbock has quit IRC | 00:17 | |
| *** brinzhang has joined #openstack-nova | 00:19 | |
| *** tjgresha has quit IRC | 00:30 | |
| *** tjgresha has joined #openstack-nova | 00:30 | |
| *** brinzhang_ has joined #openstack-nova | 00:32 | |
| *** brinzhang has quit IRC | 00:36 | |
| *** spatel has joined #openstack-nova | 00:54 | |
| *** igordc has quit IRC | 00:59 | |
| *** hongbin has joined #openstack-nova | 01:03 | |
| *** sapd1 has joined #openstack-nova | 01:05 | |
| *** rabel has quit IRC | 01:06 | |
| *** rabel has joined #openstack-nova | 01:06 | |
| *** imacdonn has quit IRC | 01:13 | |
| *** imacdonn has joined #openstack-nova | 01:14 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | 01:23 |
|---|---|---|
| openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | 01:26 |
| *** bauzas_ has joined #openstack-nova | 01:29 | |
| *** artom has quit IRC | 01:30 | |
| *** altlogbot_0 has quit IRC | 01:30 | |
| *** bauzas has quit IRC | 01:30 | |
| *** bauzas_ is now known as bauzas | 01:30 | |
| *** altlogbot_1 has joined #openstack-nova | 01:32 | |
| *** jamesdenton has quit IRC | 01:32 | |
| *** jangutter has quit IRC | 01:33 | |
| *** jangutter has joined #openstack-nova | 01:33 | |
| *** hemna_ has quit IRC | 01:34 | |
| *** Kimmo_ has quit IRC | 01:35 | |
| *** adriant has joined #openstack-nova | 01:36 | |
| *** hemna_ has joined #openstack-nova | 01:38 | |
| *** _alastor_ has joined #openstack-nova | 01:43 | |
| *** hongbin has quit IRC | 01:57 | |
| openstackgerrit | Huachang Wang proposed openstack/nova-specs master: Use PCPU and VCPU in one instance https://review.opendev.org/668656 | 02:23 |
| *** tetsuro has joined #openstack-nova | 02:23 | |
| *** tbachman has quit IRC | 02:38 | |
| openstackgerrit | Qiu Fossen proposed openstack/nova master: Add user_id field to the migrations records https://review.opendev.org/669785 | 02:52 |
| *** yaawang has joined #openstack-nova | 02:55 | |
| *** tetsuro has quit IRC | 02:56 | |
| *** hongbin has joined #openstack-nova | 02:57 | |
| *** lbragstad has quit IRC | 02:57 | |
| *** icarusfactor has quit IRC | 03:03 | |
| *** whoami-rajat has joined #openstack-nova | 03:12 | |
| *** BjoernT has joined #openstack-nova | 03:17 | |
| *** hongbin has quit IRC | 04:04 | |
| *** udesale has joined #openstack-nova | 04:10 | |
| *** etp has joined #openstack-nova | 04:13 | |
| *** tbachman has joined #openstack-nova | 04:15 | |
| *** factor has joined #openstack-nova | 04:16 | |
| *** _alastor_ has quit IRC | 04:25 | |
| *** rcernin has quit IRC | 04:30 | |
| *** rcernin has joined #openstack-nova | 04:31 | |
| *** pcaruana has joined #openstack-nova | 04:35 | |
| *** spatel has quit IRC | 04:37 | |
| *** pcaruana has quit IRC | 04:38 | |
| *** tetsuro has joined #openstack-nova | 04:49 | |
| *** prometheanfire has left #openstack-nova | 04:56 | |
| *** gyee has quit IRC | 04:59 | |
| *** ileixe has quit IRC | 04:59 | |
| *** ileixe has joined #openstack-nova | 05:01 | |
| *** ileixe has quit IRC | 05:03 | |
| *** ileixe has joined #openstack-nova | 05:03 | |
| *** ricolin has joined #openstack-nova | 05:03 | |
| *** ccamacho has quit IRC | 05:06 | |
| *** ratailor has joined #openstack-nova | 05:17 | |
| openstackgerrit | Merged openstack/nova master: libvirt: manage COMPUTE_STATUS_DISABLED for hypervisor connection https://review.opendev.org/668707 | 05:19 |
| *** ivve has joined #openstack-nova | 05:19 | |
| *** BjoernT has quit IRC | 05:26 | |
| *** shilpasd has quit IRC | 05:30 | |
| *** luksky11 has joined #openstack-nova | 05:30 | |
| *** Luzi has joined #openstack-nova | 05:37 | |
| *** ratailor has quit IRC | 05:38 | |
| *** shilpasd has joined #openstack-nova | 05:40 | |
| *** ratailor has joined #openstack-nova | 05:45 | |
| *** udesale has quit IRC | 05:48 | |
| *** ccamacho has joined #openstack-nova | 05:53 | |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767 | 05:57 |
| *** rcernin has quit IRC | 05:57 | |
| *** tetsuro has quit IRC | 05:58 | |
| *** udesale has joined #openstack-nova | 06:06 | |
| *** tetsuro has joined #openstack-nova | 06:06 | |
| openstackgerrit | Bhagyashri Shewale proposed openstack/nova master: Ignore root_gb for BFV in simple tenant usage API https://review.opendev.org/612626 | 06:07 |
| *** udesale has quit IRC | 06:09 | |
| *** udesale has joined #openstack-nova | 06:10 | |
| *** dpawlik has joined #openstack-nova | 06:19 | |
| openstackgerrit | guangchao.pan proposed openstack/nova master: Add a name for the automatically created volume https://review.opendev.org/660096 | 06:26 |
| *** ccamacho has quit IRC | 06:29 | |
| *** ccamacho has joined #openstack-nova | 06:29 | |
| *** etp has quit IRC | 06:38 | |
| *** udesale has quit IRC | 06:40 | |
| *** cdent has joined #openstack-nova | 06:41 | |
| *** ociuhandu has joined #openstack-nova | 06:46 | |
| *** spatel has joined #openstack-nova | 06:48 | |
| *** ociuhandu has quit IRC | 06:52 | |
| *** spatel has quit IRC | 06:53 | |
| *** rpittau|afk is now known as rpittau | 06:58 | |
| *** slaweq has joined #openstack-nova | 07:02 | |
| *** pcaruana has joined #openstack-nova | 07:03 | |
| *** maciejjozefczyk has joined #openstack-nova | 07:05 | |
| *** ircuser-1 has joined #openstack-nova | 07:05 | |
| *** tssurya has joined #openstack-nova | 07:09 | |
| *** awalende has joined #openstack-nova | 07:16 | |
| *** ianw is now known as ianw_pto | 07:16 | |
| *** ttsiouts has joined #openstack-nova | 07:23 | |
| *** udesale has joined #openstack-nova | 07:24 | |
| *** helenafm has joined #openstack-nova | 07:25 | |
| *** ralonsoh has joined #openstack-nova | 07:30 | |
| openstackgerrit | Brin Zhang proposed openstack/nova master: Specify availability_zone to unshelve https://review.opendev.org/663851 | 07:34 |
| *** ttsiouts has quit IRC | 07:38 | |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: db: Add vpmems to instance_extra https://review.opendev.org/662697 | 07:39 |
| openstackgerrit | Luyao Zhong proposed openstack/nova master: object: Add VirtualPMEM and VirtualPMEMList objs https://review.opendev.org/634548 | 07:39 |
| *** brinzhang_ has quit IRC | 07:41 | |
| *** brinzhang_ has joined #openstack-nova | 07:42 | |
| *** brinzhang_ has quit IRC | 07:44 | |
| *** brinzhang_ has joined #openstack-nova | 07:44 | |
| *** priteau has joined #openstack-nova | 07:44 | |
| *** bhagyashris has joined #openstack-nova | 07:48 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.opendev.org/637955 | 07:50 |
| *** ratailor_ has joined #openstack-nova | 07:50 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move consts from neutronv2/api to constants module https://review.opendev.org/668945 | 07:50 |
| *** belmoreira has joined #openstack-nova | 07:51 | |
| *** ratailor has quit IRC | 07:52 | |
| *** ttsiouts has joined #openstack-nova | 07:57 | |
| *** dtantsur|afk is now known as dtantsur | 07:57 | |
| *** yaawang has quit IRC | 08:01 | |
| *** rcernin has joined #openstack-nova | 08:02 | |
| openstackgerrit | Shilpa Devharakar proposed openstack/nova master: Support filtering of hosts by forbidden aggregates https://review.opendev.org/667952 | 08:02 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use neutron contants in cmd/manage.py https://review.opendev.org/668946 | 08:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add 'resource_request' to neutronv2/constants https://review.opendev.org/668947 | 08:04 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Translatable output strings in heal allocation https://review.opendev.org/668925 | 08:04 |
| *** awalende has quit IRC | 08:04 | |
| *** awalende has joined #openstack-nova | 08:05 | |
| *** rcernin has quit IRC | 08:11 | |
| *** ttsiouts has quit IRC | 08:11 | |
| *** ttsiouts has joined #openstack-nova | 08:12 | |
| *** ttsiouts_ has joined #openstack-nova | 08:13 | |
| *** ociuhandu has joined #openstack-nova | 08:17 | |
| *** ttsiouts has quit IRC | 08:17 | |
| *** panda has quit IRC | 08:22 | |
| *** panda has joined #openstack-nova | 08:24 | |
| *** brinzhang_ has quit IRC | 08:24 | |
| *** brinzhang has joined #openstack-nova | 08:25 | |
| *** rcernin has joined #openstack-nova | 08:27 | |
| *** yaawang has joined #openstack-nova | 08:42 | |
| *** tkajinam has quit IRC | 08:42 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use the safe get_binding_profile https://review.opendev.org/669817 | 08:42 |
| *** priteau has quit IRC | 08:47 | |
| *** priteau has joined #openstack-nova | 08:48 | |
| *** mdbooth has joined #openstack-nova | 08:53 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova stable/rocky: Add support for osprofiler in placement wsgi https://review.opendev.org/667832 | 08:59 |
| *** belmoreira has quit IRC | 09:05 | |
| *** belmoreira has joined #openstack-nova | 09:12 | |
| *** belmoreira has quit IRC | 09:13 | |
| *** ratailor has joined #openstack-nova | 09:16 | |
| *** ratailor_ has quit IRC | 09:18 | |
| *** yaawang has quit IRC | 09:18 | |
| *** yaawang has joined #openstack-nova | 09:23 | |
| *** mdbooth has quit IRC | 09:32 | |
| *** mdbooth has joined #openstack-nova | 09:35 | |
| *** bhagyashris has quit IRC | 09:43 | |
| *** pcaruana has quit IRC | 09:48 | |
| openstackgerrit | Matthew Booth proposed openstack/nova master: Whether to use config drive depends on instance.configdrive https://review.opendev.org/485930 | 09:55 |
| *** ratailor_ has joined #openstack-nova | 09:58 | |
| *** derekh has joined #openstack-nova | 10:00 | |
| *** ratailor has quit IRC | 10:00 | |
| *** priteau has quit IRC | 10:03 | |
| *** tetsuro has quit IRC | 10:04 | |
| *** cdent has quit IRC | 10:07 | |
| *** nicolasbock has joined #openstack-nova | 10:10 | |
| *** ricolin has quit IRC | 10:14 | |
| *** ttsiouts_ has quit IRC | 10:16 | |
| *** ttsiouts has joined #openstack-nova | 10:17 | |
| *** cdent has joined #openstack-nova | 10:18 | |
| *** tetsuro has joined #openstack-nova | 10:21 | |
| *** ttsiouts has quit IRC | 10:21 | |
| *** ricolin has joined #openstack-nova | 10:22 | |
| *** ricolin has quit IRC | 10:23 | |
| *** shilpasd has quit IRC | 10:24 | |
| *** xek_ is now known as xek | 10:32 | |
| *** udesale has quit IRC | 10:34 | |
| *** luksky11 has quit IRC | 10:40 | |
| *** jamesdenton has joined #openstack-nova | 10:50 | |
| *** Kevin_Zheng has quit IRC | 10:50 | |
| *** tbachman has quit IRC | 10:52 | |
| *** tbachman has joined #openstack-nova | 10:52 | |
| *** tetsuro has quit IRC | 10:53 | |
| *** belmoreira has joined #openstack-nova | 11:01 | |
| *** shilpasd has joined #openstack-nova | 11:01 | |
| *** mdbooth has quit IRC | 11:03 | |
| *** mdbooth has joined #openstack-nova | 11:04 | |
| *** ttsiouts has joined #openstack-nova | 11:04 | |
| *** dosaboy has quit IRC | 11:05 | |
| *** nicolasbock has quit IRC | 11:06 | |
| *** dosaboy has joined #openstack-nova | 11:09 | |
| *** mdbooth has quit IRC | 11:10 | |
| *** tesseract has joined #openstack-nova | 11:11 | |
| *** tesseract has quit IRC | 11:13 | |
| *** tesseract has joined #openstack-nova | 11:15 | |
| *** luksky11 has joined #openstack-nova | 11:16 | |
| *** gokhani has quit IRC | 11:16 | |
| *** altlogbot_1 has quit IRC | 11:19 | |
| *** tssurya has quit IRC | 11:19 | |
| *** irclogbot_0 has quit IRC | 11:19 | |
| *** altlogbot_2 has joined #openstack-nova | 11:20 | |
| *** mdbooth has joined #openstack-nova | 11:22 | |
| *** zbr is now known as zbr|lunch | 11:22 | |
| *** tesseract has quit IRC | 11:23 | |
| *** altlogbot_2 has quit IRC | 11:25 | |
| *** mdbooth has quit IRC | 11:34 | |
| *** tbachman has quit IRC | 11:37 | |
| *** mdbooth has joined #openstack-nova | 11:39 | |
| *** belmoreira has quit IRC | 11:39 | |
| *** belmoreira has joined #openstack-nova | 11:43 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: nova-manage: heal port allocations https://review.opendev.org/637955 | 11:43 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Move consts from neutronv2/api to constants module https://review.opendev.org/668945 | 11:44 |
| *** ricolin has joined #openstack-nova | 11:44 | |
| *** tetsuro has joined #openstack-nova | 11:44 | |
| *** ttsiouts has quit IRC | 11:44 | |
| openstackgerrit | Ivaylo Mitev proposed openstack/nova master: VMware: Attach volumes using adapter type from instance https://review.opendev.org/616599 | 11:44 |
| *** ttsiouts has joined #openstack-nova | 11:45 | |
| *** ratailor_ has quit IRC | 11:45 | |
| *** artom has joined #openstack-nova | 11:48 | |
| *** ttsiouts has quit IRC | 11:49 | |
| *** jistr_ has joined #openstack-nova | 11:51 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use neutron contants in cmd/manage.py https://review.opendev.org/668946 | 11:52 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Use the safe get_binding_profile https://review.opendev.org/669817 | 11:52 |
| *** niceplace_ has joined #openstack-nova | 11:52 | |
| *** belmoreira has quit IRC | 11:52 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Add 'resource_request' to neutronv2/constants https://review.opendev.org/668947 | 11:52 |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Translatable output strings in heal allocation https://review.opendev.org/668925 | 11:53 |
| *** ttsiouts has joined #openstack-nova | 11:53 | |
| *** kinrui has joined #openstack-nova | 11:53 | |
| *** udesale has joined #openstack-nova | 11:55 | |
| *** jistr has quit IRC | 11:55 | |
| *** dannins has quit IRC | 11:55 | |
| *** fyx has quit IRC | 11:55 | |
| *** markmcclain has quit IRC | 11:55 | |
| *** niceplace has quit IRC | 11:55 | |
| *** hogepodge has quit IRC | 11:55 | |
| *** jbernard has quit IRC | 11:55 | |
| *** mordred has quit IRC | 11:55 | |
| *** guilhermesp has quit IRC | 11:55 | |
| *** seyeongkim has quit IRC | 11:55 | |
| *** BlackDex has quit IRC | 11:55 | |
| *** mnasiadka has quit IRC | 11:55 | |
| *** kmalloc has quit IRC | 11:55 | |
| *** TheJulia has quit IRC | 11:55 | |
| *** dustinc has quit IRC | 11:55 | |
| *** jrosser has quit IRC | 11:55 | |
| *** fungi has quit IRC | 11:55 | |
| *** lchabert has quit IRC | 11:56 | |
| *** melwitt has quit IRC | 11:56 | |
| *** asettle-PTO has quit IRC | 11:56 | |
| *** belmoreira has joined #openstack-nova | 11:57 | |
| *** zbr|lunch is now known as zbr | 11:59 | |
| *** rcernin has quit IRC | 11:59 | |
| *** irclogbot_3 has joined #openstack-nova | 12:00 | |
| *** dannins has joined #openstack-nova | 12:01 | |
| *** fyx has joined #openstack-nova | 12:01 | |
| *** hogepodge has joined #openstack-nova | 12:01 | |
| *** jbernard has joined #openstack-nova | 12:01 | |
| *** mordred has joined #openstack-nova | 12:01 | |
| *** guilhermesp has joined #openstack-nova | 12:01 | |
| *** seyeongkim has joined #openstack-nova | 12:01 | |
| *** BlackDex has joined #openstack-nova | 12:01 | |
| *** mnasiadka has joined #openstack-nova | 12:01 | |
| *** kmalloc has joined #openstack-nova | 12:01 | |
| *** TheJulia has joined #openstack-nova | 12:01 | |
| *** dustinc has joined #openstack-nova | 12:01 | |
| *** jrosser has joined #openstack-nova | 12:01 | |
| *** lchabert has joined #openstack-nova | 12:01 | |
| *** weshay_PTO is now known as weshay | 12:01 | |
| *** altlogbot_0 has joined #openstack-nova | 12:02 | |
| *** tetsuro has quit IRC | 12:03 | |
| *** altlogbot_0 has quit IRC | 12:05 | |
| *** irclogbot_3 has quit IRC | 12:05 | |
| *** altlogbot_1 has joined #openstack-nova | 12:08 | |
| *** pcaruana has joined #openstack-nova | 12:10 | |
| *** altlogbot_1 has quit IRC | 12:11 | |
| *** tbachman has joined #openstack-nova | 12:15 | |
| *** sapd1_x has joined #openstack-nova | 12:20 | |
| *** kinrui is now known as fungi | 12:21 | |
| *** tssurya has joined #openstack-nova | 12:27 | |
| *** helenafm has quit IRC | 12:33 | |
| *** jistr_ is now known as jistr | 12:33 | |
| *** belmoreira has quit IRC | 12:42 | |
| *** helenafm has joined #openstack-nova | 12:44 | |
| *** belmoreira has joined #openstack-nova | 12:46 | |
| *** belmoreira has quit IRC | 12:51 | |
| *** belmoreira has joined #openstack-nova | 12:52 | |
| openstackgerrit | Arthur Dayne proposed openstack/nova-specs master: Proposal for a safer noVNC console with password authentication https://review.opendev.org/623120 | 12:53 |
| *** altlogbot_3 has joined #openstack-nova | 12:54 | |
| *** altlogbot_3 has quit IRC | 12:57 | |
| *** sapd1_x has quit IRC | 13:00 | |
| openstackgerrit | sean mooney proposed openstack/nova master: make config drives sticky bug 1835822 https://review.opendev.org/669738 | 13:01 |
| openstack | bug 1835822 in OpenStack Compute (nova) "vms loose acess to config drive with CONF.force_config_drive=True after hard reboot" [Medium,In progress] https://launchpad.net/bugs/1835822 - Assigned to sean mooney (sean-k-mooney) | 13:01 |
| yonglihe | sean-k-mooney: I had finished a new logic per our discuss. Code released without unit test, so you could give advice in advance, https://review.opendev.org/#/c/627765/ | 13:07 |
| *** zul has joined #openstack-nova | 13:07 | |
| sean-k-mooney | i just finsihed ^ so i can take a look before i start my next thing | 13:08 |
| openstackgerrit | huanhongda proposed openstack/nova master: Fix for auth during live-migration (2) https://review.opendev.org/669867 | 13:10 |
| *** mriedem has joined #openstack-nova | 13:13 | |
| *** priteau has joined #openstack-nova | 13:14 | |
| *** lbragstad has joined #openstack-nova | 13:15 | |
| openstackgerrit | Merged openstack/nova master: Remove assumption of http error if consumer not exists https://review.opendev.org/669188 | 13:16 |
| yonglihe | sean-k-mooney, thanks, looking forward your comment. The 'reap_unknown' options seems not suitable which make that logic mess. I just collapse the orphan instance clean up use existing 'poweroff' and 'reap' option. Cause we had filter out instance not created by nova, this might acceptably. | 13:23 |
| *** maciejjozefczyk has quit IRC | 13:25 | |
| *** maciejjozefczyk has joined #openstack-nova | 13:28 | |
| yonglihe | @melanie witt, I had your comments addressed mostly, please take a look, thanks. | 13:30 |
| *** tbachman has quit IRC | 13:33 | |
| *** tbachman has joined #openstack-nova | 13:34 | |
| *** BjoernT has joined #openstack-nova | 13:34 | |
| *** brinzhang has quit IRC | 13:39 | |
| *** brinzhang has joined #openstack-nova | 13:40 | |
| *** mdbooth has quit IRC | 13:40 | |
| *** mdbooth has joined #openstack-nova | 13:40 | |
| sean-k-mooney | yonglihe: ya i think that is much closer to what i was expecting | 13:41 |
| sean-k-mooney | i left some comments inline | 13:41 |
| *** slaweq has quit IRC | 13:43 | |
| openstackgerrit | sean mooney proposed openstack/nova master: Whether to use config drive depends on instance.configdrive https://review.opendev.org/485930 | 13:43 |
| *** ttsiouts has quit IRC | 13:47 | |
| *** ttsiouts has joined #openstack-nova | 13:47 | |
| *** slaweq has joined #openstack-nova | 13:49 | |
| *** belmoreira has quit IRC | 13:49 | |
| *** ttsiouts has quit IRC | 13:52 | |
| *** BjoernT has quit IRC | 13:52 | |
| *** belmoreira has joined #openstack-nova | 13:54 | |
| mriedem | gmann: i'll update the policy rule name in https://review.opendev.org/#/c/645520/33 | 13:54 |
| *** mlavalle has joined #openstack-nova | 13:57 | |
| *** BjoernT has joined #openstack-nova | 13:57 | |
| *** Luzi has quit IRC | 13:57 | |
| *** ttsiouts has joined #openstack-nova | 13:58 | |
| *** tbachman has quit IRC | 14:02 | |
| *** tbachman has joined #openstack-nova | 14:08 | |
| openstackgerrit | Balazs Gibizer proposed openstack/nova master: Test heal port allocations in nova-next https://review.opendev.org/669879 | 14:10 |
| mriedem | i've got a series of pretty simple "implement update_provider_tree" changes for xen and hyperv here with +2s https://review.opendev.org/#/q/status:open+project:openstack/nova+branch:master+topic:update_provider_tree which eventually allows us to deprecate the non-upt compact code in the RT and lets us clean up that RT mess in U | 14:11 |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520 | 14:13 |
| *** spatel has joined #openstack-nova | 14:14 | |
| gibi | mriedem: do we have somebody from xen and hyperv to ack those changes? I looked at the xen one and it seems straitforward to me but I could wait for a xen developer to ack | 14:17 |
| *** mdbooth has quit IRC | 14:17 | |
| sean-k-mooney | gibi: im not sure we have anyone maintaining xen support at the moment | 14:18 |
| sean-k-mooney | out side of the core team that is | 14:18 |
| *** whoami-rajat has quit IRC | 14:18 | |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767 | 14:20 |
| *** mdbooth has joined #openstack-nova | 14:22 | |
| gibi | sean-k-mooney: OK, I left my +2 on the patch. | 14:23 |
| *** BlackDex has quit IRC | 14:25 | |
| mriedem | gibi: lpetrut from hyperv already acked the hyperv one and the CI passed, and it's a cherry pick from the out-of-tree hyperv driver | 14:26 |
| mriedem | we do'nt have any xen people nor ci | 14:26 |
| mriedem | but the xen change itself is very straightforward - just a port of get_inventory | 14:26 |
| gibi | mriedem: I'm +A on the hyperv as I saw lpetrut vote | 14:26 |
| mriedem | thanks | 14:26 |
| gibi | mriedem: OK, then I push the +A on the xen patch as well | 14:27 |
| mriedem | will be nice to eventually remove the spaghetti for that stuff from the RT | 14:27 |
| *** BlackDex has joined #openstack-nova | 14:27 | |
| gibi | mriedem: btw, I made my first stab at adding port healing integration test to nova-next https://review.opendev.org/669879 , Eagerly waiting for the gate to tell me why it won't work :) | 14:28 |
| mriedem | interesting | 14:29 |
| *** BjoernT_ has joined #openstack-nova | 14:30 | |
| mriedem | gibi: i think you put the changes on the wrong job in zuul.yaml | 14:31 |
| gibi | mriedem: you are right. :facepalm: | 14:31 |
| *** BjoernT has quit IRC | 14:32 | |
| mriedem | left another comment, but nova-next isn't zuul v3 native yet, | 14:33 |
| mriedem | but i think you could hack the devstack plugin/services/local.conf into the nova-next job using DEVSTACK_GATE_* variables | 14:33 |
| mriedem | until we convert nova-next to zuul v3 | 14:33 |
| *** dpawlik has quit IRC | 14:34 | |
| gibi | mriedem: thanks. I will do the translation to the old format | 14:34 |
| mriedem | that's probably easier for now | 14:34 |
| *** awalende has quit IRC | 14:34 | |
| mriedem | i might tinker with converting nova-next to zuul v3 separately, i've been meaning to do that anyway | 14:35 |
| *** awalende has joined #openstack-nova | 14:35 | |
| *** tbachman has quit IRC | 14:38 | |
| gibi | mriedem: cool, thanks for the quick review, you spared me some CI runs | 14:39 |
| *** awalende has quit IRC | 14:39 | |
| *** belmoreira has quit IRC | 14:42 | |
| *** mmethot has quit IRC | 14:44 | |
| *** belmoreira has joined #openstack-nova | 14:45 | |
| sean-k-mooney | mriedem: by the way we talked about actully enfroceing our policy of deprecating and removing virt drivers/backend without ci. have we pushed a patch to mark xen as deprected yet? | 14:46 |
| mriedem | we merged a patch to say xen is experimental but not yet deprecated | 14:47 |
| *** mmethot has joined #openstack-nova | 14:47 | |
| mriedem | https://github.com/openstack/nova/commit/92c7e0e0b4fcc596e86b493aafb28ee656cb046c#diff-775c4605e52aa43e2ad8c90489216dc8 | 14:47 |
| *** BlackDex has quit IRC | 14:47 | |
| sean-k-mooney | i still hope to either spend a day and fix the libvirt/lxc driver or push a patch to deprecate it. | 14:48 |
| *** BlackDex has joined #openstack-nova | 14:48 | |
| sean-k-mooney | but ya we will still have them for train in anycase | 14:48 |
| mriedem | if you care about lxc then maybe follow / talk to this guy https://review.opendev.org/#/q/owner:miguel%2540midokura.com+(status:open+OR+status:merged) | 14:49 |
| sean-k-mooney | oh he fixed the bug i was thinking of fixing | 14:50 |
| sean-k-mooney | cool | 14:50 |
| *** belmoreira has quit IRC | 14:51 | |
| sean-k-mooney | am im not sure how much i care about it but i am somewhat nostalgic in that lxc was second technically container tech i used and it has some usecases still | 14:51 |
| sean-k-mooney | the two thing i was thinking of doing was fixing that bug and trying to add a ci job | 14:51 |
| sean-k-mooney | basically if we still offically support it i would like to ci it but if no one is useing it then i dont mind if its removed. | 14:52 |
| *** ivve has quit IRC | 14:53 | |
| mriedem | i had a non-voting nova-lxc ci job for it long ago and had to blacklist a lot of tempest tests and even then it would fail and the failures would snowball hard on the compute | 14:54 |
| *** belmoreira has joined #openstack-nova | 14:54 | |
| mriedem | nbd mounts and such would get messed up from what i remember | 14:54 |
| mriedem | we don't officially support it btw, | 14:55 |
| mriedem | there is a warning that gets logged on the start of the driver saying it's not tested | 14:55 |
| sean-k-mooney | ah ok well its still there and i think its more useful the uml | 14:55 |
| sean-k-mooney | or some of the other non qmeu/kvm libvirt backends | 14:56 |
| sean-k-mooney | anyway its a low priority | 14:57 |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Add host and hypervisor_hostname flag to create server https://review.opendev.org/645520 | 14:57 |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767 | 14:57 |
| mriedem | speaking of priorities, how is that numa aware live migration series coming? | 14:59 |
| *** tbachman has joined #openstack-nova | 15:01 | |
| mriedem | cdent: did i dream this or did you at some point get nova's code coverage job working with unit + functional tests? it only runs unit tests today so we're not reporting all of the coverage. | 15:04 |
| cdent | I think I played with it after I got it working for placement but don't remember if it ever went beyond that. I can play with it again if that seems like a useful thing to do | 15:05 |
| *** belmoreira has quit IRC | 15:05 | |
| mriedem | i played with it at some point too and seem to remember things hanging or crashing somehow | 15:06 |
| cdent | let's blame eventlet | 15:07 |
| cdent | which might mean things are different now | 15:08 |
| * cdent starts it in background | 15:08 | |
| *** belmoreira has joined #openstack-nova | 15:08 | |
| mriedem | right, i think i tried configuring coverage with concurrency=eventlet but that didn't help | 15:10 |
| mriedem | https://coverage.readthedocs.io/en/v4.5.x/config.html#run | 15:11 |
| sean-k-mooney | mriedem: its proably failing due to our use of fixture. one of the oslo libs (oslo.fixtures?) i think calles exec directly with a code object and that breaks my debugger | 15:14 |
| sean-k-mooney | the coverage jobs hooks the debug entry point | 15:14 |
| openstackgerrit | Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.opendev.org/651969 | 15:14 |
| sean-k-mooney | its break on from oslo_utils.fixture import uuidsentinel as uuids | 15:15 |
| *** _alastor_ has joined #openstack-nova | 15:22 | |
| mriedem | i'm actually hitting errors with it trying to find the placement fixture | 15:25 |
| sean-k-mooney | yep | 15:27 |
| sean-k-mooney | that was the error i endup having | 15:27 |
| sean-k-mooney | its in the rewrite module when its actully trying to patch stuff | 15:27 |
| sean-k-mooney | the point in the code it raise the exception is actully kind of missleading but the final exception you get makes more sense | 15:28 |
| sean-k-mooney | im not really sure why "from oslo_utils.fixture import uuidsentinel as uuids" has to do with placement but im guessing we are messing with some gloabl state and its not happy | 15:31 |
| cdent | look at the functional job defs: you have to require placement explicitly | 15:32 |
| cdent | i've done that and I'm running with some moderate form of success | 15:32 |
| cdent | but it's not done yet | 15:32 |
| sean-k-mooney | how do you mean | 15:32 |
| mriedem | deps = | 15:32 |
| mriedem | -r{toxinidir}/test-requirements.txt | 15:32 |
| mriedem | git+https://opendev.org/openstack/placement#egg=openstack-placement | 15:32 |
| mriedem | good call | 15:32 |
| *** hamzy has quit IRC | 15:33 | |
| sean-k-mooney | i have placmenet in the functional-py36 env | 15:33 |
| cdent | we need sean-k-mooney 's 97 million core machine, because this is so slooooow | 15:34 |
| sean-k-mooney | what did you mean by require placement explcitly | 15:34 |
| mriedem | cdent: do you have the c extension? | 15:34 |
| mriedem | coverage --version in the venv should tell you | 15:34 |
| cdent | yes | 15:35 |
| sean-k-mooney | i can reboot that system into linux and test some things if ye like | 15:36 |
| sean-k-mooney | i was playing games on it last night so its still booted into windows | 15:36 |
| cdent | unit tests in RbdTestCase are blocking and timing out | 15:36 |
| cdent | non-linear unit tests are funny | 15:37 |
| cdent | mriedem: my guess is we can make it work but might need to blacklist a few of the "funny" tests | 15:37 |
| mriedem | yeah, i've got it running now too | 15:38 |
| mriedem | using all 8 cores...hammering | 15:38 |
| cdent | libvirt _get_new_connection is another issue | 15:41 |
| mriedem | i wonder if that one is eventlet | 15:41 |
| mriedem | i haven't run with concurrency=eventlet yet | 15:41 |
| cdent | it is | 15:42 |
| mriedem | likely need this in .coveragerc: | 15:42 |
| mriedem | concurrency = | 15:42 |
| mriedem | eventlet | 15:42 |
| mriedem | greenlet | 15:42 |
| mriedem | in the [run] section | 15:42 |
| cdent | it gets chundered up in eventlet tpool proxy call | 15:42 |
| cdent | my read on the docs was that was just for accounting | 15:42 |
| cdent | (and I added the eventlet bit) | 15:42 |
| cdent | but not greenlet | 15:42 |
| *** altlogbot_3 has joined #openstack-nova | 15:43 | |
| sean-k-mooney | technically its gevent under the covers of both right | 15:43 |
| *** mdbooth has quit IRC | 15:43 | |
| *** derekh has quit IRC | 15:44 | |
| *** mdbooth has joined #openstack-nova | 15:44 | |
| cdent | mriedem: I'm going to restart mine and let it run while I'm away, will post up any findings later | 15:44 |
| mriedem | yup, same | 15:44 |
| mriedem | i could see this being a sinkhole for my day | 15:44 |
| cdent | indeed | 15:46 |
| cdent | bbl | 15:46 |
| *** altlogbot_3 has quit IRC | 15:47 | |
| *** mdbooth has quit IRC | 15:49 | |
| *** helenafm has quit IRC | 15:52 | |
| *** priteau has quit IRC | 15:52 | |
| sean-k-mooney | for what its worth i just kicked it off on the 24 core server i have. i have turned off hyper treading for reasons | 15:55 |
| *** maciejjozefczyk has quit IRC | 15:55 | |
| *** igordc has joined #openstack-nova | 15:56 | |
| sean-k-mooney | hum the cover job seam to only be running the unit test | 15:58 |
| sean-k-mooney | i dont see where that is set in the tox env | 15:59 |
| sean-k-mooney | 89% is not too bad | 15:59 |
| sean-k-mooney | mriedem: cdent care to point out what i need to chagne to get it to run everything | 16:00 |
| mriedem | mine isn't done but this is what i have http://paste.openstack.org/show/754229/ | 16:02 |
| *** hamzy has joined #openstack-nova | 16:02 | |
| mriedem | i've mixed some cleanups into that | 16:02 |
| *** icarusfactor has joined #openstack-nova | 16:04 | |
| sean-k-mooney | ok its re running now and ya the functional tests are now running too | 16:04 |
| sean-k-mooney | actully ill add teh concurrancy stuff | 16:05 |
| *** factor has quit IRC | 16:06 | |
| *** factor__ has joined #openstack-nova | 16:06 | |
| *** ivve has joined #openstack-nova | 16:08 | |
| *** altlogbot_3 has joined #openstack-nova | 16:08 | |
| *** icarusfactor has quit IRC | 16:08 | |
| sean-k-mooney | should be done in another miniute or so just grabing a coffee | 16:09 |
| *** BjoernT_ has quit IRC | 16:09 | |
| *** altlogbot_3 has quit IRC | 16:13 | |
| sean-k-mooney | ya im seeing it stall with timeout near the end so we are not mocking something properly although it appears to be in the unit test which is weird | 16:13 |
| *** BjoernT_ has joined #openstack-nova | 16:13 | |
| sean-k-mooney | maybe we mess up some global state in the functional tests? | 16:14 |
| sean-k-mooney | i am running these under py37 which also could be related but probably not | 16:15 |
| sean-k-mooney | hum it look liek the libvirt issue might be real | 16:18 |
| sean-k-mooney | it looks like some of our mock dont work under python 3.7 and we actully try to connect to libvirt | 16:19 |
| *** whoami-rajat has joined #openstack-nova | 16:21 | |
| mriedem | gibi: now that you're leaving or already gone for the day, i've gone through your tests https://review.opendev.org/#/c/637955/34 | 16:23 |
| mriedem | sean-k-mooney: i would not be surprised, probably missing the FakeLibvirtFixture somewhere | 16:24 |
| mriedem | because several of the libvirt modules have a global libvirt variable which tests are supposed to stub out to be the fakelibvirt module | 16:25 |
| *** irclogbot_1 has joined #openstack-nova | 16:25 | |
| *** belmoreira has quit IRC | 16:27 | |
| *** irclogbot_1 has quit IRC | 16:27 | |
| mriedem | test_cleanup_volumes_pending_resize is mocking the wrong thing... | 16:28 |
| mriedem | RBDVolumeProxy rather than RbdProxy | 16:28 |
| sean-k-mooney | im seeing a whole bunch of placement time out where we call _get_guest_config which goes all the way down to calling _connect in nova.virt.libvirt.host | 16:28 |
| sean-k-mooney | ya i noticed that too | 16:28 |
| sean-k-mooney | im piping the full output to a file and ill through it up on google drive or something | 16:29 |
| sean-k-mooney | its more or less done at this point i think just waiting for all the timeouts to fire | 16:29 |
| *** factor__ has quit IRC | 16:29 | |
| *** factor__ has joined #openstack-nova | 16:30 | |
| sean-k-mooney | all the tox treads are more or less sleeping and then ever so foter a test times out and a bunch suceed | 16:30 |
| sean-k-mooney | test_min_version_file_backed_bad_ram_allocation_ratio apparently modifies teh db witout without using the right fixture | 16:33 |
| *** dtantsur is now known as dtantsur|afk | 16:34 | |
| sean-k-mooney | so ya there are a whole bunc of not so nice errors showing up | 16:34 |
| mriedem | i don't see _check_file_backed_memory_support hitting the db at all | 16:35 |
| sean-k-mooney | my output could be interleaved | 16:35 |
| sean-k-mooney | i have a couple of things that look like this | 16:37 |
| sean-k-mooney | http://paste.openstack.org/show/754231/ | 16:37 |
| *** rpittau is now known as rpittau|afk | 16:37 | |
| mriedem | that would be the libvirt host connection callback stuff that calls _set_host_enabled | 16:38 |
| *** ccamacho has quit IRC | 16:38 | |
| sean-k-mooney | ya which is happening because our mocking of the connection info is messed up | 16:38 |
| sean-k-mooney | we should be useing the fake libvirt conenction there or just mocking it out entirely sicne it a unit test | 16:38 |
| mriedem | i've had problems in the past with mocks on the libvirt Host stuff leading to weird intermittent failures | 16:39 |
| mriedem | especially with the hasVersion type calls | 16:39 |
| mriedem | like this: | 16:39 |
| sean-k-mooney | ya you mentioned that on my vidoe model patchs | 16:39 |
| mriedem | test_swap_volume_file | 16:39 |
| *** luksky11 has quit IRC | 16:39 | |
| mriedem | @mock.patch.object(host.Host, | 16:39 |
| mriedem | 'has_min_version', return_value=True) | 16:39 |
| mriedem | rather than mocking the drvr.host.has_min_version directly | 16:40 |
| mriedem | *drvr._host.has_min_version | 16:40 |
| mriedem | there are a billion libvirt driver unit tests so untangling that would be pretty hard | 16:40 |
| sean-k-mooney | well we could do a find an replace | 16:41 |
| *** spatel has quit IRC | 16:41 | |
| mriedem | i don't think it's that easy | 16:41 |
| sean-k-mooney | e.g. if there is one way we know works grep for the broken form | 16:41 |
| sean-k-mooney | proably not | 16:41 |
| *** eharney has joined #openstack-nova | 16:41 | |
| sean-k-mooney | but if you know a way that definetly works we can fix the unstable kind as we find them | 16:41 |
| sean-k-mooney | is there a way we could shorten these timeout for the tests by the way? | 16:42 |
| mriedem | yes | 16:42 |
| sean-k-mooney | e.g. globaly somehow | 16:42 |
| mriedem | OS_TEST_TIMEOUT i think | 16:42 |
| mriedem | it's in tox.ini | 16:42 |
| sean-k-mooney | or time out the tests them seleves | 16:43 |
| sean-k-mooney | that might cause issue with some of them but it would be worth trying | 16:43 |
| mriedem | well it would make this monster coverage thing fail faster yes | 16:44 |
| mriedem | dropping OS_TEST_TIMEOUT=60 would help | 16:44 |
| sean-k-mooney | i was thinking more doing set.flags in the base test case | 16:44 |
| mriedem | nova.conf doesn't have anything to do with test execution timeout... | 16:44 |
| *** factor__ has quit IRC | 16:44 | |
| sean-k-mooney | i was wondering if we could modify libvirt timeouts in nova.conf | 16:44 |
| mriedem | nope | 16:45 |
| openstackgerrit | Merged openstack/nova master: Implement update_provider_tree https://review.opendev.org/667417 | 16:45 |
| *** mgoddard has quit IRC | 16:45 | |
| *** mdbooth has joined #openstack-nova | 16:45 | |
| mriedem | configuring libvirt is kind of a libvirt.conf thing right? | 16:45 |
| sean-k-mooney | ya i guess i think these timeout are comeing form teh python binding or the c lib rather then libvirt | 16:45 |
| * mriedem gets lunch | 16:45 | |
| sean-k-mooney | ill give it another 15mins and if its still timing out ten ill run with OS_TEST_TIMEOUT=30 currently its timing out after 160s | 16:47 |
| *** _alastor_ has quit IRC | 16:48 | |
| *** ttsiouts has quit IRC | 16:48 | |
| *** mgoddard has joined #openstack-nova | 16:48 | |
| *** ttsiouts has joined #openstack-nova | 16:48 | |
| *** dpawlik has joined #openstack-nova | 16:49 | |
| *** mdbooth has quit IRC | 16:49 | |
| sean-k-mooney | actully i might make that OS_TEST_TIMEOUT=8 im not seeing any test over about 4.5 seconds | 16:50 |
| sean-k-mooney | oh there is 1 at 13s | 16:51 |
| *** tssurya has quit IRC | 16:51 | |
| *** ttsiouts has quit IRC | 16:53 | |
| *** _alastor_ has joined #openstack-nova | 16:55 | |
| dansmith | artom: I assume you're working on feedback on the resize revert patches yeah? | 16:55 |
| *** aarents has joined #openstack-nova | 16:57 | |
| *** udesale has quit IRC | 17:00 | |
| *** altlogbot_1 has joined #openstack-nova | 17:00 | |
| *** altlogbot_1 has quit IRC | 17:05 | |
| *** Sundar has joined #openstack-nova | 17:08 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912 | 17:09 |
| *** irclogbot_2 has joined #openstack-nova | 17:11 | |
| openstackgerrit | Merged openstack/nova master: xenapi: implement update_provider_tree https://review.opendev.org/667435 | 17:12 |
| openstackgerrit | Merged openstack/nova master: Deprecate non-update_provider_tree compat code https://review.opendev.org/667442 | 17:12 |
| Sundar | gibi, efried: Re. https://review.opendev.org/#/c/657464/, there was some discussion over naming, which got resolved in the review of the spec https://review.opendev.org/#/c/603955/12/specs/train/approved/nova-cyborg-interaction.rst@307 . Can we resume the review of 657464? | 17:13 |
| openstackgerrit | Merged openstack/nova master: Update COMPUTE_STATUS_DISABLED from set_host_enabled compute call https://review.opendev.org/668743 | 17:13 |
| *** irclogbot_2 has quit IRC | 17:13 | |
| efried | Sundar: As far as I'm concerned, absolutely. Not much I can do though, since I own the patch :) | 17:14 |
| *** dpawlik has quit IRC | 17:20 | |
| *** igordc has quit IRC | 17:29 | |
| openstackgerrit | Merged openstack/nova master: Add placement request pre-filter compute_status_filter https://review.opendev.org/668752 | 17:31 |
| openstackgerrit | Merged openstack/nova master: Refactor HostAPI.service_update https://review.opendev.org/668756 | 17:31 |
| openstackgerrit | Merged openstack/nova master: Sync COMPUTE_STATUS_DISABLED from API https://review.opendev.org/654596 | 17:32 |
| *** ralonsoh has quit IRC | 17:36 | |
| artom | dansmith, yeah | 17:38 |
| *** gyee has joined #openstack-nova | 17:39 | |
| *** tesseract has joined #openstack-nova | 17:40 | |
| dansmith | cool | 17:41 |
| *** hamzy has quit IRC | 17:47 | |
| *** hamzy has joined #openstack-nova | 17:52 | |
| openstackgerrit | Merged openstack/nova master: Follow up for pre-filter-disabled-computes series https://review.opendev.org/668986 | 17:54 |
| *** luksky11 has joined #openstack-nova | 18:09 | |
| *** BjoernT_ has quit IRC | 18:09 | |
| *** eharney has quit IRC | 18:17 | |
| *** eharney has joined #openstack-nova | 18:30 | |
| *** whoami-rajat has quit IRC | 18:30 | |
| *** melwitt has joined #openstack-nova | 18:34 | |
| *** irclogbot_2 has joined #openstack-nova | 18:37 | |
| *** hongbin has joined #openstack-nova | 18:37 | |
| *** tesseract has quit IRC | 18:39 | |
| *** irclogbot_2 has quit IRC | 18:39 | |
| *** panda has quit IRC | 18:43 | |
| *** eharney has quit IRC | 18:44 | |
| *** BjoernT has joined #openstack-nova | 18:45 | |
| *** eharney has joined #openstack-nova | 18:46 | |
| *** hamzy has quit IRC | 18:49 | |
| *** hamzy has joined #openstack-nova | 18:49 | |
| mriedem | +2 on the host/hypervisor_hostname on server create api change again https://review.opendev.org/#/c/645520/ | 18:51 |
| *** igordc has joined #openstack-nova | 18:55 | |
| *** ociuhandu has quit IRC | 19:02 | |
| *** factor has joined #openstack-nova | 19:06 | |
| openstackgerrit | Merged openstack/nova master: Cleanup quota user docs https://review.opendev.org/662573 | 19:13 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: nova-lvm: Disable [validation]/run_validation in tempest.conf https://review.opendev.org/662176 | 19:16 |
| *** panda has joined #openstack-nova | 19:19 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: Update AZ admin doc to mention the new way to specify hosts https://review.opendev.org/666767 | 19:30 |
| cdent | mriedem: I think you already know this, but my coverage run locked up and stayed locked (until I forced it dead just now) in nova.tests.unit.virt.libvirt.test_driver.LibvirtConnTestCase.test_check_can_live_migrate_dest_file_backed_discard_bad_libvirt | 19:34 |
| mriedem | i also killed mine | 19:37 |
| mriedem | but posted a change | 19:37 |
| mriedem | which will probably timeout :) | 19:37 |
| mriedem | ah yeah | 19:39 |
| mriedem | @mock.patch.object(fakelibvirt.Connection, 'getVersion') | 19:39 |
| mriedem | @mock.patch.object(fakelibvirt.Connection, 'getLibVersion') | 19:39 |
| mriedem | but that doesn't mock out conn = self.get_connection() | 19:40 |
| *** ttsiouts has joined #openstack-nova | 19:43 | |
| *** belmoreira has joined #openstack-nova | 19:44 | |
| *** ttx has quit IRC | 19:49 | |
| *** Hazelesque has quit IRC | 19:49 | |
| *** Hazelesque has joined #openstack-nova | 19:49 | |
| sean-k-mooney | ya i have seen the same | 19:50 |
| *** ttx has joined #openstack-nova | 19:50 | |
| sean-k-mooney | i ws just checking it before i pip out to the shops to get dinner but i was thinking maybe we could run the test in two batchs | 19:51 |
| sean-k-mooney | run the funtional test only first then unit tests only then have coverage combine the combine the two into one set of results | 19:52 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: libvirt: remove unused Service.get_by_compute_host mocks https://review.opendev.org/669934 | 19:54 |
| *** cdent has quit IRC | 19:57 | |
| mriedem | sean-k-mooney: huh https://github.com/nedbat/coveragepy/blob/6b72dcbd98c69a7e8c5333d2926c7f871161db7e/coverage/collector.py#L115 | 20:01 |
| mriedem | the docs are really not clear about that | 20:01 |
| *** lbragstad has quit IRC | 20:02 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: WIP: coverage job changes https://review.opendev.org/669912 | 20:04 |
| mriedem | efried: can you re-approve https://review.opendev.org/#/c/666042/ since i just had to rebase and fix the pep8 thing due to the newly enforced hacking rules | 20:06 |
| *** ivve has quit IRC | 20:06 | |
| efried | ... | 20:06 |
| efried | done | 20:08 |
| *** ttsiouts has quit IRC | 20:09 | |
| *** ttsiouts has joined #openstack-nova | 20:09 | |
| mriedem | thanks | 20:10 |
| *** ttsiouts has quit IRC | 20:14 | |
| *** xek has quit IRC | 20:21 | |
| *** irclogbot_0 has joined #openstack-nova | 20:25 | |
| *** irclogbot_0 has quit IRC | 20:27 | |
| *** hamzy has quit IRC | 20:37 | |
| *** _alastor_ has quit IRC | 20:37 | |
| *** _alastor_ has joined #openstack-nova | 20:37 | |
| *** eharney has quit IRC | 20:37 | |
| *** _alastor1 has joined #openstack-nova | 20:42 | |
| *** BjoernT_ has joined #openstack-nova | 20:43 | |
| *** BjoernT has quit IRC | 20:44 | |
| *** _alastor_ has quit IRC | 20:44 | |
| *** ttsiouts has joined #openstack-nova | 20:53 | |
| *** tbachman has quit IRC | 20:55 | |
| *** lbragstad has joined #openstack-nova | 20:56 | |
| *** ttsiouts has quit IRC | 21:04 | |
| *** ttsiouts has joined #openstack-nova | 21:04 | |
| mriedem | dansmith: you might have an opinion on this https://review.opendev.org/#/c/669545/ | 21:05 |
| dansmith | hrm | 21:07 |
| dansmith | I hate to add a traceback, but probably makes sense in the diaper case | 21:07 |
| dansmith | is there a bug filed that shows what the actual issue was? | 21:08 |
| dansmith | because we should definitely not take that path and log a traceback fro anything we can handle | 21:08 |
| mriedem | no bug as far as i know | 21:09 |
| *** ttsiouts has quit IRC | 21:09 | |
| mriedem | looks like something with nova - or some python-*client downstream - trying to get a response.status_code | 21:09 |
| mriedem | and response is None | 21:09 |
| mriedem | i thought of you b/c of the change i linked in there | 21:09 |
| *** jmlowe has joined #openstack-nova | 21:10 | |
| dansmith | yeah, and that case makes sense to not dump stack I think | 21:12 |
| dansmith | but if we're not handling it, dumping is the right thing to do I think | 21:12 |
| dansmith | point is, I think we should make sure we've fixed the problem that this would be dumping stack for | 21:13 |
| dansmith | in addition to dumping stack for the future | 21:13 |
| mriedem | if you're +2 i'll follow your lead | 21:14 |
| mriedem | into the bowels of hell | 21:14 |
| *** irclogbot_3 has joined #openstack-nova | 21:14 | |
| dansmith | commented | 21:15 |
| *** pcaruana has quit IRC | 21:19 | |
| *** irclogbot_3 has quit IRC | 21:19 | |
| *** _alastor1 has quit IRC | 21:21 | |
| *** Sundar has quit IRC | 21:21 | |
| *** belmoreira has quit IRC | 21:25 | |
| *** belmoreira has joined #openstack-nova | 21:26 | |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Replace a wiki link with API ref guide link https://review.opendev.org/669027 | 21:26 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Perf: Use dicts for ProviderTree roots https://review.opendev.org/637225 | 21:28 |
| openstackgerrit | Matt Riedemann proposed openstack/python-novaclient master: Add a guide to add a new microversion support https://review.opendev.org/667002 | 21:28 |
| openstackgerrit | Matt Riedemann proposed openstack/nova master: doc: Add links to novaclient contributor guide https://review.opendev.org/669028 | 21:36 |
| *** belmoreira has quit IRC | 21:36 | |
| *** irclogbot_3 has joined #openstack-nova | 21:39 | |
| *** _alastor1 has joined #openstack-nova | 21:39 | |
| *** belmoreira has joined #openstack-nova | 21:42 | |
| *** irclogbot_3 has quit IRC | 21:43 | |
| *** donnyd is now known as donnyd_pto | 21:43 | |
| *** jmlowe has quit IRC | 21:51 | |
| *** BjoernT_ has quit IRC | 21:52 | |
| *** jmlowe has joined #openstack-nova | 21:53 | |
| *** irclogbot_1 has joined #openstack-nova | 21:55 | |
| *** irclogbot_1 has quit IRC | 21:59 | |
| *** igordc has quit IRC | 22:00 | |
| *** belmoreira has quit IRC | 22:02 | |
| *** rcernin has joined #openstack-nova | 22:10 | |
| *** luksky11 has quit IRC | 22:12 | |
| *** tbachman has joined #openstack-nova | 22:18 | |
| *** tbachman has quit IRC | 22:31 | |
| *** awalende has joined #openstack-nova | 22:37 | |
| efried | dustinc: Bottom SDK patch is in merge conflict, not sure when that happened. I'm going to rebase it and the placement patch and leave the ironic series to you, mkay? | 22:39 |
| dustinc | Yeah I have it taken care of in local branch, will push it this week | 22:39 |
| dustinc | Thanks for heads up | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Introduces the openstacksdk to nova https://review.opendev.org/643664 | 22:40 |
| openstackgerrit | Eric Fried proposed openstack/nova master: Use OpenStack SDK for placement https://review.opendev.org/656023 | 22:40 |
| *** awalende has quit IRC | 22:42 | |
| efried | dustinc: ^ | 22:44 |
| efried | be careful not to regress to a previous commit, then. You should pull the bottom patch and rebase your series onto it. | 22:44 |
| *** altlogbot_0 has joined #openstack-nova | 22:45 | |
| dustinc | efried: hmm ok, yeah I had already rebased and resolved that conflict locally so will have to figure that out before I push then | 22:45 |
| efried | right. lmk if you need help. | 22:46 |
| openstackgerrit | Merged openstack/nova master: Fix GET /servers/detail host_status performance regression https://review.opendev.org/666042 | 22:49 |
| openstackgerrit | Merged openstack/nova master: libvirt: remove unused Service.get_by_compute_host mocks https://review.opendev.org/669934 | 22:49 |
| *** altlogbot_0 has quit IRC | 22:49 | |
| *** tkajinam has joined #openstack-nova | 22:52 | |
| *** slaweq has quit IRC | 22:54 | |
| *** hongbin has quit IRC | 22:59 | |
| *** tbachman has joined #openstack-nova | 23:14 | |
| openstackgerrit | Merged openstack/nova master: doc: Replace a wiki link with API ref guide link https://review.opendev.org/669027 | 23:23 |
| *** tbachman has quit IRC | 23:27 | |
| *** mlavalle has quit IRC | 23:32 | |
| *** takashin has joined #openstack-nova | 23:44 | |
| openstackgerrit | Takashi NATSUME proposed openstack/python-novaclient master: Deprecate cells v1 and extension commands and APIs https://review.opendev.org/669597 | 23:56 |
| *** hamzy has joined #openstack-nova | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!