Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2018-06-19
15:33:25 mriedem jacqueslist
15:35:40 openstackgerrit huanhongda proposed openstack/nova stable/ocata: [Stable Only] Remove soft-deleted instances from quota_usages https://review.openstack.org/570653
15:35:41 kashyap Heh; that looks dodgy. /me stops going more off-topic, and heads offline. Tomorrow evening is my most difficult Dutch written exam; need to study.
15:49:50 mriedem zcorneli: dansmith: ok comments inline on the file-backed memory change https://review.openstack.org/#/c/567876/
15:49:53 mriedem mostly good,
15:50:07 mriedem but i did wonder about the AggregateRamFilter for host aggregates that set ram_allocation_ratio!=1.0
15:50:23 mriedem the spec didn't mention that nor the docs in the change
15:50:44 mriedem maybe it just means, don't put file-backed memory hosts in aggregates that have ram_allocation_ratio!=1.0
15:51:56 zcorneli mriedem: Looking through commentary now
15:52:42 mriedem i think in general we will just have some gaps that we could shore up with proper scheduling filtering, including not landing a flavor that requests huge pages on a file-backed host
15:52:48 mriedem but we could also do follow ups on those
16:07:41 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: Configure multiple vGPU types https://review.openstack.org/564214
16:07:42 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: mdevs returning parent and vendor PCI info https://review.openstack.org/562304
16:07:43 openstackgerrit Sylvain Bauza proposed openstack/nova master: libvirt: Get inventories per vGPU type https://review.openstack.org/564215
16:13:00 zcorneli mriedem: Most of the notes look good, will fix up what I can. Not sure how to handle the aggregates ram_allocation_ratio thing, except just documenting not to do that.
16:13:18 mriedem zcorneli: interested in dansmith's thoughts on that as well
16:13:40 dansmith mriedem: sorry I'm working on a downstream thing atm
16:13:41 mriedem zcorneli: as noted inline, i think we could robustify the scheduling landmines in follow ups if needed
16:13:50 mriedem in other words, i don't consider it blocking
16:15:29 zcorneli mriedem: I agree, I think it will act appropriately if it does get into any of those gaps (we're using immediate file allocation, instead of lazy, so qemu will fail to start up if you exceed the capacity of the backing store, for example). Then someone could look at the docs and see the notes about "Don't do this thing"
16:15:32 TheJulia out of curiosity, is the nova-v1-cells job in a healthy state?
16:15:44 TheJulia err, nova-cells-v1
16:15:45 zcorneli Not the greatest situation, but it's at least "correct"
16:16:30 dansmith mriedem: if there was some hypervisor that couldn't do memory overcommit, and you put it into an aggregate with overcommit set, that'd be the same situation here
16:16:59 dansmith I don't recall what the solution was for aggregate-based overcommit post placement though
16:17:25 mriedem there are a few specs i think...but not clear on that either
16:17:41 mriedem TheJulia: there was a known regression over the weekend, resolved yesterday
16:17:51 mriedem http://status.openstack.org/elastic-recheck/#1777478
16:17:53 TheJulia mriedem: thanks
16:18:39 mriedem zcorneli: the MemoryPagesUnsupported conflict will result in a reschedule to another host,
16:18:57 mriedem so yeah you might land on a host that supports huge pages but isn't configured for file-backed memory
16:19:11 mriedem assuming the operator hasn't used host aggregates to make those features mutually exclusive
16:19:46 zcorneli mriedem: Which seems OK, because file-backed isn't requested by the user / flavor.
16:19:58 mriedem it's just a lot of host aggregate whack-a-mole for the operator to have to deal with, and we aren't great about exlusion filtering
16:20:05 mriedem zcorneli: right, but huge pages are
16:20:45 mriedem that's why i was thinking, in the future we could report a capability for the host and a scheduler filter could say, 'oh this host is file-backed memory and the request has huge pages, i know those don't work, so exclude that host
16:21:05 zcorneli mriedem: So if they end up landing on a host without file-backed, but with huge pages, they've gotten a "correct" thing. But the scheduler definitely had to do some extra work in it.
16:21:23 mriedem well, or they got lucky
16:21:28 mriedem or the operator setup aggregates to make that happen
16:21:59 zcorneli Yea, agreed. I think we had talked something similar to this early on too, with the migration thing (<Rocky -> Rocky+file-backed).
16:22:01 mriedem anyway, we could optimize in the future if needed
16:22:09 zcorneli Having it be a capability so the scheduler can avoid that breakage.
16:22:11 dansmith mriedem: well, that would be baking some libvirt-ness into the scheduler a bit
16:22:15 mriedem i imagine operators will add their own scheduler filters for this
16:22:23 dansmith mriedem: because maybe hyper-v could do that, for example
16:22:35 mriedem dansmith: yeah i know
16:22:50 dansmith exposing some can-do-huge-pages thing and then having scheduler filter out those hosts if requested would be okay thoug,
16:22:54 mriedem dansmith: but (1) i'm sure it's not the only libvirt-specific thing in scheduler filters and (2) could be a long time before any other virt driver added support for this
16:22:56 dansmith and we could de-assert that if file-backed
16:23:04 dansmith sure
16:23:26 mriedem right, if file-backed, report 'can't do huge pages'
16:23:37 mriedem for libvirt
16:23:56 mriedem if operators do end up adding a custom filter, i wish they'd upstream it...
16:24:52 mriedem zcorneli: since you'll be writing limitations docs on this stuff, you could just mention something like (in the docs), "this could be optimized with a scheduler filter in the future"
16:25:08 mriedem that kind of leaves the door open to people that are hitting this that we're amenable to contributing a filter for stuff like this
16:25:50 zcorneli mriedem: Ack. Will add a note in the docs as I write up the limitations.
16:26:06 mriedem guess i should get back on https://review.openstack.org/#/c/538498/ finally...
16:26:15 mriedem but upt has scared me off
16:37:06 openstackgerrit Dan Smith proposed openstack/nova master: Be graceful about vif plugging in early ironic driver startup https://review.openstack.org/576580
16:48:20 openstackgerrit zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384
16:50:48 openstackgerrit zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384
17:04:26 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588
17:05:47 openstackgerrit zhangyangyang proposed openstack/nova master: Remove ChanceScheduler https://review.openstack.org/576384
17:06:44 openstackgerrit Merged openstack/nova master: Fix MigrateData object tests for compat routines https://review.openstack.org/576256
17:08:23 openstackgerrit Eric Fried proposed openstack/nova master: WIP: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588
17:25:20 openstackgerrit Eric Fried proposed openstack/nova master: RT stats deepcopy 1x/node instead of 1x/instance https://review.openstack.org/576591
17:33:16 openstackgerrit Balazs Gibizer proposed openstack/nova stable/queens: Trying to reproduce bug 1777540 https://review.openstack.org/576593
17:33:17 openstack bug 1777540 in OpenStack Compute (nova) ""IndexError: list index out of range" in ExceptionPayload.from_exception during resize failure" [Undecided,New] https://launchpad.net/bugs/1777540
17:33:36 gibi mriedem: if you are interested in my failed trials ^^
17:43:49 openstackgerrit Eric Fried proposed openstack/nova master: RT stats deepcopy 1x/node instead of 1x/instance https://review.openstack.org/576591
17:44:55 openstackgerrit Matt Riedemann proposed openstack/os-traits master: Add COMPUTE_TRUSTED_CERTS trait https://review.openstack.org/576595
18:52:04 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Convert driver supported capabilities to compute node provider traits https://review.openstack.org/538498
18:52:07 openstackgerrit Eric Fried proposed openstack/nova master: Squash: exclude nested alloc cands in older microversions https://review.openstack.org/576609
18:55:54 openstackgerrit Eric Fried proposed openstack/nova master: Squash: exclude nested alloc cands in older microversions https://review.openstack.org/576609
19:14:36 mriedem get your easy bug fix here! https://review.openstack.org/#/c/575222/
19:45:41 openstackgerrit Merged openstack/nova master: Return all nested providers in tree https://review.openstack.org/559480
19:51:49 efried mriedem: easy
19:54:00 openstackgerrit Artom Lifshitz proposed openstack/nova master: DNM: Use claim context during live migration https://review.openstack.org/576222
19:56:45 openstackgerrit Matt Riedemann proposed openstack/nova master: Fix regression when listing build_requests with marker and ip filter https://review.openstack.org/576161
19:56:46 mriedem zcorneli|afk: i just noticed this, https://review.openstack.org/#/c/574240/ - it was on the china bug smash etherpad
20:18:33 openstackgerrit Merged openstack/nova master: Add full traceback to ExceptionPayload in versioned notifications https://review.openstack.org/564092
21:05:59 openstackgerrit Eric Fried proposed openstack/nova master: Convert ironic virt driver to update_provider_tree https://review.openstack.org/576588
21:43:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Don't heal allocations for deleted servers https://review.openstack.org/575222
22:07:51 openstackgerrit Eric Fried proposed openstack/nova master: Add a microversion for consumer generation support https://review.openstack.org/565604
22:23:35 openstackgerrit Matt Riedemann proposed openstack/nova master: Wait for network-vif-plugged before starting live migration https://review.openstack.org/558001
22:23:36 openstackgerrit Matt Riedemann proposed openstack/nova master: Add check if neutron "binding-extended" extension is available https://review.openstack.org/523548
22:23:37 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "bind_ports_to_host" neutron API method https://review.openstack.org/523604
22:23:38 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "delete_port_binding" network API method https://review.openstack.org/552170
22:23:39 openstackgerrit Matt Riedemann proposed openstack/nova master: Add "activate_port_binding" neutron API method https://review.openstack.org/555947
22:23:40 openstackgerrit Matt Riedemann proposed openstack/nova master: Delete port bindings in setup_networks_on_host if teardown=True https://review.openstack.org/556333
22:23:41 openstackgerrit Matt Riedemann proposed openstack/nova master: Implement migrate_instance_start method for neutron https://review.openstack.org/556334
22:23:42 openstackgerrit Matt Riedemann proposed openstack/nova master: WIP: Add VIFMigrateData object for live migration https://review.openstack.org/515423
22:23:43 openstackgerrit Matt Riedemann proposed openstack/nova master: Add VIFMigrateData.get_dest_vif https://review.openstack.org/566931
22:23:44 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: factor out pre_live_migration plug_vifs call https://review.openstack.org/566932
22:23:45 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: use dest host port bindings during pre_live_migration https://review.openstack.org/566933
22:23:46 openstackgerrit Matt Riedemann proposed openstack/nova master: libvirt: use dest host vif migrate details for live migration https://review.openstack.org/551370
22:23:47 openstackgerrit Matt Riedemann proposed openstack/nova master: Annotate flows and handle PortBindingDeletionFailed in ComputeManager https://review.openstack.org/551371
22:23:48 openstackgerrit Matt Riedemann proposed openstack/nova master: Port binding based on events during live migration https://review.openstack.org/434870
22:23:49 openstackgerrit Matt Riedemann proposed openstack/nova master: conductor: use port binding extended API in during live migrate https://review.openstack.org/522537
22:29:05 zcorneli|afk mriedem: Yep. So not the first to find it at least.

Earlier   Later