Posted	Nick	Remark
#openstack-nova - 2019-04-15
20:14:03	mriedem	sync_instance_info should be a broadcast - no timeout
20:14:03	mnaser	I mean I think if we want (anti-)affinity, we have to afaik?
20:14:18	mriedem	no, it's just a cache in the scheduler
20:14:35	mriedem	if the data isn't cached in the scheduler it pulls it from the db per request - which could be slow
20:14:39	mnaser	oh yeah I remember that
20:14:47	mnaser	let me find the traceback
20:15:00	dansmith	mriedem: I have to submit something to lower-constraints itself before I can put it in nova yeah?
20:15:13	mriedem	dansmith: no, it's per-project
20:15:18	mriedem	not like the old g-r way
20:15:24	dansmith	sigh
20:15:30	mriedem	honestly i don't know why we have lower-constraints.txt and requirements.txt
20:15:32	dansmith	might as well not ever try to remember because it'll be different
20:15:47	openstackgerrit	Dan Smith proposed openstack/nova master: Add image type capability flags and trait conversions https://review.openstack.org/652710
20:15:51	mriedem	g-r was dropped so requirements.txt isn't automatically updated from g-r anymore
20:16:01	mnaser	https://www.irccloud.com/pastebin/drWAr4hV/
20:16:04	mriedem	and lower-constraints was added, but....
20:16:07	dansmith	right but I thought lower was, but whaterver
20:16:16	openstackgerrit	Merged openstack/nova master: Remove cells v1 jobs https://review.openstack.org/651289
20:16:17	mnaser	is resource_tracker not thee same thing? maybe I'm being misleading here
20:16:36	mriedem	mnaser: not that's something else, sec
20:17:10	mriedem	https://github.com/openstack/nova/blob/stable/stein/nova/compute/resource_tracker.py#L1323
20:17:19	mriedem	it's timing out pulling instances from the db
20:17:34	mriedem	well, "an instance"
20:17:50	mriedem	which....your db / mq must be getting hammered?
20:18:24	mnaser	mriedem: nothing sticks out
20:18:39	mriedem	that will kill the whole update_available_resource periodic though
20:18:50	mriedem	until the next time it runs i mean
20:19:35	mriedem	sync_instance_info is a different periodic that reports info to the scheduler
20:19:37	mriedem	per host
20:19:54	mnaser	I remember seeing a traceback involving it
20:19:55	mnaser	let me check again
21:17:33	mriedem	why doesn't tox yell at me anymore when i provide an env which doesn't exist?
21:30:51	mriedem	dansmith: do you remember this from 3+ years ago? https://review.openstack.org/#/c/256073/
21:31:08	mriedem	if so, did https://review.openstack.org/#/c/325684/ regress it?
21:32:12	sean-k-mooney	so it was fixed for 6 months 2 and a half years ago
21:33:19	sean-k-mooney	i must say i like the topic branch
21:33:29	sean-k-mooney	*topic name
21:34:42	mriedem	i just noticed that the base TestCase is still doing openstack_driver.DRIVER_CACHE = {} even though that's not used anymore
21:35:15	sean-k-mooney	i was going to say i assume you were doing git blame on something random and found that
21:36:35	sean-k-mooney	presuably we should create an updated version of dans original caching patch again for this
21:37:12	sean-k-mooney	i think we have several other dictionay caches at this point so we can proably reuse one for it
21:37:33	dansmith	mriedem: I don't remember it no
21:37:45	mriedem	i'll put up a logging patch that logs every time we create that object
21:37:48	mriedem	to see how bad it is
21:38:12	mriedem	dansmith: coincidentally that series is related to recent issues https://review.openstack.org/#/q/topic:okay-we-got-it+(status:open+OR+status:merged)
21:41:36	sean-k-mooney	mriedem: by the way i added a memorize decorator in https://review.openstack.org/#/c/599071/3/nova/api/validation/parameter_types.py so we could just move the into the nova utils and reuse it if we want
21:59:55	openstackgerrit	Matt Riedemann proposed openstack/nova master: Make nova.compute.rpcapi.ComputeAPI.router a singleton https://review.openstack.org/649197
22:06:02	openstackgerrit	Matt Riedemann proposed openstack/nova master: WIP: log when loading security group driver https://review.openstack.org/652783
23:39:17	openstackgerrit	Lee Yarwood proposed openstack/nova master: Restore connection_info after live migration rollback https://review.openstack.org/551349
23:39:18	openstackgerrit	Lee Yarwood proposed openstack/nova master: WIP compute: Use source_bdms to reset attachment_ids during LM rollback https://review.openstack.org/652800
23:48:24	openstackgerrit	Merged openstack/nova stable/rocky: Adding tests to demonstrate bug #1821824 https://review.openstack.org/649602
23:48:26	openstack	bug 1821824 in OpenStack Compute (nova) rocky "Forbidden traits in flavor properties don't work" [High,In progress] https://launchpad.net/bugs/1821824 - Assigned to Eric Fried (efried)
23:48:31	openstackgerrit	Merged openstack/nova stable/rocky: Fix bug preventing forbidden traits from working https://review.openstack.org/649603
23:48:37	openstackgerrit	Merged openstack/nova stable/rocky: Add missing libvirt exception during device detach https://review.openstack.org/651637
#openstack-nova - 2019-04-16
00:02:41	openstackgerrit	Lee Yarwood proposed openstack/nova master: compute: Use source_bdms to reset attachment_ids during LM rollback https://review.openstack.org/652800
00:53:31	openstackgerrit	zhufl proposed openstack/nova master: Remove conductor_api and _last_host_check from manager.py https://review.openstack.org/651059
01:05:49	openstackgerrit	Dustin Cowles proposed openstack/nova master: Introduces the openstacksdk to nova https://review.openstack.org/643664
03:01:13	openstackgerrit	Boxiang Zhu proposed openstack/nova-specs master: Add host and hypervisor_hostname flag to create server https://review.openstack.org/645458
04:05:04	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.openstack.org/651969
05:23:27	openstackgerrit	Merged openstack/nova master: Handle unsetting '[DEFAULT] dhcp_domain' https://review.openstack.org/652662
06:05:09	openstackgerrit	Boxiang Zhu proposed openstack/nova master: [WIP] Add host and hypervisor_hostname flag to create server https://review.openstack.org/645520
06:18:05	openstackgerrit	Merged openstack/nova master: Remove dead code https://review.openstack.org/650277
07:56:43	openstackgerrit	Boxiang Zhu proposed openstack/nova master: Fix live migration break group policy simultaneously https://review.openstack.org/651969
08:22:37	openstackgerrit	Merged openstack/nova master: Dropping the py35 testing https://review.openstack.org/643871
08:25:28	openstackgerrit	Merged openstack/nova master: Bump to hacking 1.1.0 https://review.openstack.org/651553
08:25:36	openstackgerrit	Merged openstack/nova master: Remove 'nova-cells' service https://review.openstack.org/651290
08:37:33	lyarwood	stephenfin: https://review.openstack.org/#/c/637224/ - would you mind taking a look at this today?
08:37:52	lyarwood	^ not a spec FWIW so feel free to ignore for now
08:43:36	kashyap	lyarwood: Morning, hope this answers your question: https://review.openstack.org/#/c/506720/8/specs/train/approved/allow-secure-boot-for-qemu-kvm-guests.rst@287
08:43:41	kashyap	(Thanks for the review, so far.)
08:47:15	stephenfin	lyarwood: added to the queue :)
08:52:41	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove '/os-cells' REST APIs https://review.openstack.org/651291
08:52:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling cells v1 in '/os-hypervisors' API https://review.openstack.org/651292
08:52:42	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling cells v1 in '/os-servers' API https://review.openstack.org/651293
08:52:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'nova-manage cell' commands https://review.openstack.org/651294
08:52:43	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for console authentication https://review.openstack.org/651295
08:52:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove old-style cell v1 instance listing https://review.openstack.org/651296
08:52:44	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'bdm_(update_or_create|destroy)_at_top' https://review.openstack.org/651297
08:52:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'instance_fault_create_at_top' https://review.openstack.org/651298
08:52:45	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'instance_info_cache_update_at_top' https://review.openstack.org/651299
08:52:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'get_keypair_at_top' https://review.openstack.org/651300
08:52:46	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'instance_update_at_top', 'instance_destroy_at_top' https://review.openstack.org/651301
08:52:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'instance_update_from_api' https://review.openstack.org/651302
08:52:47	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling 'update_cells' on 'BandwidthUsage.create' https://review.openstack.org/651303
08:52:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for instance naming https://review.openstack.org/651304
08:52:48	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove cells code https://review.openstack.org/651306
08:55:25	lyarwood	kashyap: ack thanks, replied. I was more suggesting we generate the file once and make copies for each instance to use after that.
08:56:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove 'instance_update_from_api' https://review.openstack.org/651302
08:56:05	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling 'update_cells' on 'BandwidthUsage.create' https://review.openstack.org/651303
08:56:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling cells v1 for instance naming https://review.openstack.org/651304
08:56:06	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove cells code https://review.openstack.org/651306
08:56:16	lyarwood	kashyap: and that was what led me to ask about where that template/master file could be stored.
08:56:24	openstackgerrit	Stephen Finucane proposed openstack/nova master: Stop handling 'InstanceUnknownCell' exception https://review.openstack.org/651307
08:56:32	openstackgerrit	Stephen Finucane proposed openstack/nova master: Remove unnecessary wrapper https://review.openstack.org/651308
08:56:39	openstackgerrit	Stephen Finucane proposed openstack/nova master: db: Remove cell APIs https://review.openstack.org/651309
08:56:53	lyarwood	kashyap: generating it for each instance is expense IMHO, I'm just looking for ways to avoid that.
08:56:53	openstackgerrit	Stephen Finucane proposed openstack/nova master: conf: Remove cells v1 options, group https://review.openstack.org/651310
08:57:16	kashyap	lyarwood: Let me be sure I'm not misreading you:
08:57:44	kashyap	lyarwood: You're saying to generate _once_ and reuse it? Or generate them ahead store them somewhere and _then_ use a unique VARS file per instance?
08:58:03	kashyap	s/ahead store/ahead and store/

Earlier   Later