Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-04-15
20:08:48 mriedem you get what you get and you don't throw a fit?
20:11:00 mriedem mnaser: i'll reach out to our downstream product team and see if they have a similar need for this
20:11:06 artom mriedem, basically - https://github.com/openstack/nova/blob/master/releasenotes/notes/virt-device-tagged-attach-53e214d3b3fdd183.yaml
20:11:38 mnaser mriedem: cool, want me to restore it?
20:11:52 mriedem mnaser: i can if i'm going to work on it
20:11:56 mriedem so don't bother
20:12:05 mnaser mriedem: ok cool!
20:12:22 mnaser I have been dealing with other fun weird things
20:12:33 mnaser like resource reporting messages taking way too long
20:12:36 mnaser and timing out
20:12:43 mnaser but that only happening during live migrations
20:13:01 mnaser I don't wanna point at nova yet.. but it's pretty weird.
20:13:17 mriedem "resource reporting messages"?
20:13:33 mnaser eh the stuff that runs every little bit and reports the instance list to the scheduler
20:13:45 mnaser and then at some point a post_live_migration failed too with a timeout
20:13:47 mriedem oh, didn't know you relied on that
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

Earlier   Later