Posted	Nick	Remark
#openstack-nova - 2020-05-19
18:22:48	openstackgerrit	Merged openstack/nova stable/rocky: Reset the cell cache for database access in Service https://review.opendev.org/720592
18:22:58	openstackgerrit	Merged openstack/nova stable/ussuri: Make quotas respect instance_list_per_project_cells https://review.opendev.org/729086
18:23:09	openstack	bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer)
18:23:09	openstackgerrit	Merged openstack/nova stable/rocky: Reproduce bug 1862633 https://review.opendev.org/713187
18:28:37	openstackgerrit	Dan Smith proposed openstack/nova master: Add admin doc information about image cache resource accounting https://review.opendev.org/729350
18:33:25	openstackgerrit	Merged openstack/nova stable/rocky: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/713196
20:13:45	openstackgerrit	Merged openstack/nova master: Bump hacking min version to 3.1.0 https://review.opendev.org/727589
21:00:50	openstackgerrit	Lee Yarwood proposed openstack/nova master: Don't age swap files in the image cache https://review.opendev.org/641093
22:20:45	openstackgerrit	Merged openstack/nova master: Reserve DISK_GB resource for the image cache https://review.opendev.org/728439
22:30:29	openstackgerrit	Merged openstack/nova master: Fix ut error on Aarch64(And other non-x86 platform) https://review.opendev.org/713163
23:24:40	openstackgerrit	melanie witt proposed openstack/nova master: Add admin doc information about image cache resource accounting https://review.opendev.org/729350
#openstack-nova - 2020-05-20
00:04:48	mriedem	dansmith: question in that docs patch
00:09:14	openstackgerrit	melanie witt proposed openstack/nova master: rbd_utils: increase _destroy_volume timeout https://review.opendev.org/705764
00:26:44	openstackgerrit	Merged openstack/nova stable/train: Revert "nova shared storage: rbd is always shared storage" https://review.opendev.org/726861
03:31:01	openstackgerrit	Qiu Fossen proposed openstack/nova stable/stein: Update scheduler instance info at confirm resize https://review.opendev.org/729505
04:46:03	openstackgerrit	Qiu Fossen proposed openstack/nova-specs master: specify mac for creating instance https://review.opendev.org/700429
04:58:37	openstackgerrit	Merged openstack/nova master: libvirt: Remove MIN_LIBVIRT_MULTIATTACH https://review.opendev.org/710238
06:41:50	openstackgerrit	Qiu Fossen proposed openstack/nova stable/stein: Update scheduler instance info at confirm resize https://review.opendev.org/729505
06:58:57	gibi	good morning nova
07:25:34	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/ussuri: Reserve DISK_GB resource for the image cache https://review.opendev.org/729523
07:33:36	openstackgerrit	Qiu Fossen proposed openstack/nova stable/rocky: Update scheduler instance info at confirm resize https://review.opendev.org/729527
07:51:07	openstack	bug 1869050 in OpenStack Compute (nova) stein "migration of anti-affinity server fails due to stale scheduler instance info" [Low,In progress] https://launchpad.net/bugs/1869050
07:51:07	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/stein: Reproduce bug 1869050 https://review.opendev.org/729530
07:57:02	openstackgerrit	Qiu Fossen proposed openstack/nova stable/stein: Create instance action when burying in cell0 https://review.opendev.org/729531
08:10:21	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/stein: Update scheduler instance info at confirm resize https://review.opendev.org/729538
08:10:50	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Use annotated ddt for test_cpu_policy_constraint https://review.opendev.org/692205
08:13:20	lyarwood	morning \o
08:16:27	openstackgerrit	Wenping Song proposed openstack/nova master: error may occur when filter scheduler with accelerator https://review.opendev.org/722651
08:17:20	gibi	lyarwood: o/
08:19:32	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/queens: Clean up allocation if unshelve fails due to neutron https://review.opendev.org/729540
08:19:32	openstack	bug 1862633 in OpenStack Compute (nova) "unshelve leak allocation if update port fails" [Medium,Fix released] https://launchpad.net/bugs/1862633 - Assigned to Balazs Gibizer (balazs-gibizer)
08:19:32	openstackgerrit	Balazs Gibizer proposed openstack/nova stable/queens: Reproduce bug 1862633 https://review.opendev.org/729539
08:28:22	aarents	good morning
08:29:44	aarents	dansmith, about your yesterday interrogation, Yes we benefit of cache purge, and yes we have host with big cache usage (unremovable) that prevent to put more instance than expected. this is random. we have plenty of case where, for example a customer boot from its big snapshot(not sharable with neighborhood), or we have plenty of small image in cache all 'Debian 9' but because we maintain security
08:29:50	aarents	update frequently on our public image, those image are different. We still have benefit to cache image, sometime a bit less.
08:36:44	openstackgerrit	Lee Yarwood proposed openstack/nova master: Don't age swap files in the image cache https://review.opendev.org/641093
08:43:21	brinzhang	sean-k-mooney, gibi: hi, I have a question. If the ARQs instance in SHELVED (not shelved_offload) state, do we need to delete it's bind arqs info?
08:44:00	brinzhang	when do shelve_instance action
08:46:10	brinzhang	IMO, if the instance in SHELVED status, it should always occupied the arqs resources, we dont delete it's binding arqs when we shelve it, right?
08:53:56	Aditi20	Hi, I have a doubt regarding openstack. Could you please provide me inputs?
08:53:57	Aditi20	is - https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/config.py this file creates libvirt.xml file for instances. And it doesn't have any support for nested-hv currently.
09:12:35	openstackgerrit	Huaqiang Wang proposed openstack/nova master: test_hardware: code formating for multiline construct https://review.opendev.org/728793
09:13:48	openstackgerrit	Lee Yarwood proposed openstack/nova master: Add admin doc information about image cache resource accounting https://review.opendev.org/729350
09:20:07	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Use annotated ddt for test_cpu_policy_constraint https://review.opendev.org/692205
09:29:03	openstackgerrit	Huaqiang Wang proposed openstack/nova master: Use annotated ddt for test_cpu_policy_constraint https://review.opendev.org/692205
09:36:42	sean-k-mooney	brinzhang: no we only delete the ARQs in shelve offload
09:37:18	brinzhang	sean-k-mooney: got it, I talked with alex_xu
09:38:28	brinzhang	sean-k-monney: thanks, I will submit a poc code for unshelve/shelve instance, hope you can review that when you have free time.
09:46:51	sean-k-mooney	my mothewr rang so ill review when im done
09:47:11	openstackgerrit	Jiri Suchomel proposed openstack/nova master: Remove deprecated nova.image.download hook https://review.opendev.org/725338
09:47:12	openstackgerrit	Jiri Suchomel proposed openstack/nova master: Add ability to download Glance images into the libvirt image cache via RBD https://review.opendev.org/574301
09:47:36	brinzhang	sean-k-mooney: thanks ^
09:48:35	openstackgerrit	Huaqiang Wang proposed openstack/nova master: [WIP] Use annotated ddt for test_cpu_policy_constraint https://review.opendev.org/692205
09:48:38	jsuchome	lyarwood: Hi, I've addressed your comments, in some parts we still disagree (same with some dansmith's ones btw)
09:50:48	lyarwood	jsuchome: ack, I'll take a look now
10:13:30	openstackgerrit	Huaqiang Wang proposed openstack/nova master: test_hardware: show test parameter details for causing a test failure https://review.opendev.org/692205
10:47:16	openstackgerrit	Brin Zhang proposed openstack/nova master: WIP: Cyborg shelve/unshelve support https://review.opendev.org/729563
11:08:11	openstackgerrit	Merged openstack/nova master: trivial: Remove remaining '_LE' instances https://review.opendev.org/728963
11:20:10	lyarwood	jsuchome: sorry was side tracked, just updated.
11:20:59	lyarwood	jsuchome: apologies for the mistakes before, I really think we need to break this up a little to make the logic more clear to reviewers
11:26:09	jsuchome	lyarwood: the main part is somewhat confusing, I agree ... I'm not sure though how to make it better
11:26:58	jsuchome	maybe by moving the verification bits into different method...
11:52:28	lyarwood	jsuchome: yeah I think that would be useful
11:53:04	brinzhang_	sean-k-mooney: IMO, you should move https://review.opendev.org/#/c/729563/1/nova/compute/api.py@4041 this common block code to the cyborg as a common function,
11:53:39	sean-k-mooney	yep that is more or less what i was referint too when i commented on your follow up
11:53:41	brinzhang_	sean-mooney: I was rebased on your patch, if I move this that you cannot call it in your patch
11:54:02	sean-k-mooney	i noticed you wer basically doint the same thing and extracting it to a common function we can both use makes sense
11:56:06	sean-k-mooney	brinzhang_: i was also being lazy and suggesting you could refactor my code :P but ill see if i can update it later today
11:56:07	brinzhang	after you update, I will update again, to call the common function ^
11:56:51	brinzhang	sean-k-mooney: ok, if I have time tomorrow, I will do
11:57:25	brinzhang	I am meeting now, after meetting , I will go home today.
11:57:44	sean-k-mooney	brinzhang: cool no worries i have a meeting in an hour. im not sure if i will get it done before then but ill try to update it in the afternoon
11:58:03	brinzhang	sean-k-mooney: thanks ^^
11:58:59	sean-k-mooney	brinzhang: while you are hear you said i should handel the excepiotn in rebuild like is done at X but the link you provded did not have any expcetion handeling
11:59:12	sean-k-mooney	can you let me know what you wanted me to do
11:59:36	sean-k-mooney	brinzhang: im refering to this comment https://review.opendev.org/#/c/715326/11/nova/conductor/manager.py@1232
12:00:01	brinzhang	create_and_bind_arqs will raise an exception, when bind failed, so I think that should catch the exception
12:00:49	sean-k-mooney	ah right
12:01:47	sean-k-mooney	i taught i was handeling that elsewhere but perhaps i missed updating this code path
12:01:55	sean-k-mooney	ill take a look
12:02:07	brinzhang	ah, you are busy in works
12:03:03	sean-k-mooney	not too busy but im trying to start wraping my head around the numa in plamcnet code changes so im trying to context switch away form cyborg for a day or two
12:03:31	sean-k-mooney	that said i have not really started on the numa in placment stuff yet so i still have the cyborg context loaded in my brain for the moment
12:04:33	sean-k-mooney	im still happy to review your cyborg patches but that why im slitly distracted as im preparing a second dev env to work on the numa stuff in parallel
12:04:50	brinzhang	I can understand you sean-k-mooney, I will try my best to do these thing, to reduce your work in cybrog
12:06:06	brinzhang	sean-k-money: hope you can keep cyborg service in your environment, maybe need your test in your environment for some actions
12:06:17	sean-k-mooney	brinzhang: well i actully dont mind working on the cycborg stuff its just on on our internal roadmap for the next major release e.g. 17.0 to have it productised so i have to blance it with stuff that is on our road map
12:06:37	sean-k-mooney	brinzhang: :) that i why im setting up a second multi node dev env for numa
12:06:57	sean-k-mooney	brinzhang: i have openstack deploy at home so im just created 2 more vms for working on numa
12:07:08	sean-k-mooney	im keeping my cyborg vms running too
12:07:25	brinzhang	thanks ^^
12:08:04	brinzhang	I have to continue my meetting sean-k-mooney
12:13:55	nightmare_unreal	hello, sometimes when i run openstack server migrate , it does not throw any error but doesn't actually migrate. But when I run cmd again it works ( if I run it 3rd time then it throws error which is normal)
12:14:28	nightmare_unreal	is it possible that sometimes the cmd are ignored or something ? I am not sure ( I will try to reproduce it)
12:14:43	sean-k-mooney	nightmare_unreal: no the command should not be ignored
12:14:54	sean-k-mooney	we auto rollback on some internal errors
12:15:15	sean-k-mooney	did you do resize --confrim on the second migration before the third
12:15:39	sean-k-mooney	if you did not confim the cold migration then i would expect the third to error
12:15:42	nightmare_unreal	if I don't do resize confirm then it throws error : which is expected and intended behavior
12:15:58	nightmare_unreal	but once i do confirm and run cmd again it works ( which it should IMO)

Earlier   Later