| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-03-05 | |||
| 17:01:45 | mriedem | gibi: can we mark this complete? https://blueprints.launchpad.net/nova/+spec/use-nested-allocation-candidates | |
| 17:02:14 | gibi | mriedem: there is one intentionally open patch https://review.openstack.org/#/c/608298/ | |
| 17:02:32 | gibi | mriedem: it is an edge case that we don't need in nova right now | |
| 17:02:34 | mriedem | "This cannot happen today but will in the future, when cpu and memory resource are moved to the NUMA RP and servers" | |
| 17:02:42 | gibi | mriedem: exactly | |
| 17:02:47 | mriedem | correct, so i'd just say it becomes a part of those bps when they happen | |
| 17:03:01 | gibi | mriedem: OK. I agree. Let's mark the bp completed | |
| 17:04:37 | mriedem | done | |
| 17:04:55 | openstackgerrit | Chris Friesen proposed openstack/nova master: Handle missing exception in instance creation code https://review.openstack.org/641082 | |
| 17:05:12 | cfriesen | mriedem: I also robustified the other PCI test | |
| 17:05:17 | mriedem | oh my | |
| 17:05:20 | mriedem | that's a -5 | |
| 17:05:44 | mriedem | i hope you ran fast8 | |
| 17:05:47 | mriedem | but +2 | |
| 17:06:09 | cfriesen | I can break out the other test as a separate commit if you want... | |
| 17:06:16 | mriedem | i'm just joking | |
| 17:06:45 | mriedem | -5 is a joke unless we're talking about forced cold migration | |
| 17:11:39 | sean-k-mooney | i tought forced migrations tradtional add a couple fo factorals added on like so -5!! | |
| 17:12:47 | mriedem | -5?!?!??!! | |
| 17:12:54 | mriedem | jazz it up however you want | |
| 17:13:49 | melwitt | reshaper + func test is finally on its way to the sarlacc pits \o/ | |
| 17:14:22 | dansmith | melwitt: oh, I saw the comment wasn't on the latest PS, but I hadn't gone back to see what had changed | |
| 17:15:12 | melwitt | dansmith: understood. just giving you the 411 | |
| 17:21:28 | mriedem | mdbooth: you might want to take a look at this https://review.openstack.org/#/c/613039/ | |
| 17:21:33 | mriedem | if it has a chance of making stein | |
| 17:21:52 | mriedem | it does have test coverage... https://review.openstack.org/#/c/623483/ | |
| 17:21:58 | mdbooth | mriedem: ack, thanks | |
| 17:30:35 | openstackgerrit | Matthew Booth proposed openstack/nova master: Don't age swap files in the image cache https://review.openstack.org/641093 | |
| 17:35:38 | mriedem | aspiers: correct me if i'm wrong, but up to https://review.openstack.org/#/c/638680/ we advertise that a compute will support sev but up to that point we don't actually configure anything in a guest that requires sev, right? | |
| 17:36:16 | mriedem | so if we were to merge that code in stein, we'd be saying "these computes support sev" but you can't actually schedule servers to them and get sev in the guest, yeah? | |
| 17:38:19 | mriedem | i.e. something is yet to build on this https://review.openstack.org/#/c/636318/ | |
| 17:38:58 | mriedem | all of this being done in parallel series makes it hard to track | |
| 17:41:30 | dansmith | someone should +W this: https://review.openstack.org/#/c/641021/ | |
| 17:43:23 | melwitt | you can't +W it bc it's your patch. I can look | |
| 17:44:31 | bauzas | I can take a look if needed | |
| 17:44:36 | bauzas | I still have 30 mins left | |
| 17:44:47 | dansmith | it's easy, you all should be fighting over it | |
| 17:45:10 | bauzas | I haven't followed the cross-cell resize series but looks like it's unrelated | |
| 17:45:22 | bauzas | so I can chime in | |
| 17:46:41 | cfriesen | philosophical question for people. if we resize to the same instance type, should we validate the extra-specs again in case they've been changed since we booted? I kind of think yes. | |
| 17:47:52 | melwitt | bauzas: you can have it :) | |
| 17:47:58 | dansmith | cfriesen: I think yes | |
| 17:47:58 | sean-k-mooney | cfriesen: 1 yes 2 we proably should make exta-specs imunteable at some point once a flavor is in use | |
| 17:47:58 | cfriesen | (or is that disallowed and only actually happens on migrations?) | |
| 17:48:07 | bauzas | cfriesen: yes please | |
| 17:48:13 | cfriesen | it's confusing, there's something related to cell types in here | |
| 17:48:41 | melwitt | cfriesen: I think yes too | |
| 17:49:03 | bauzas | cfriesen: the problem is that we put too many things in extra sepcs | |
| 17:49:15 | sean-k-mooney | is there any reason we dont allow you to set extraspec when creating a flavor so we can make the whole thing readonly | |
| 17:49:19 | bauzas | we should at least validate those specs before calling the scheduler | |
| 17:49:56 | cfriesen | I'm looking at https://github.com/openstack/nova/blob/master/nova/compute/api.py#L3541 which seems to indicate that you're not supposed to resize to the same flavor, but there's an exception for a "compute" cell type | |
| 17:49:57 | sean-k-mooney | i guess maybe because resize is a cold migration? | |
| 17:50:16 | openstackgerrit | Merged openstack/nova master: Stub out port binding create/delete in NeutronFixture https://review.openstack.org/636413 | |
| 17:50:42 | mdbooth | lyarwood: Would you also be able to pontificate upon https://review.openstack.org/#/c/613039/ ? | |
| 17:51:12 | openstackgerrit | Merged openstack/python-novaclient master: Microversion 2.71 - show server group https://review.openstack.org/640657 | |
| 17:51:16 | mdbooth | lyarwood: FWIW I'll be +1 on that assuming the tests are good, but I don't have time to look now. | |
| 17:52:06 | cfriesen | sean-k-mooney: yeah, but the comments say that for migration "flavor_id" is None. | |
| 17:53:32 | sean-k-mooney | cfriesen: right well resize and colde migrate are the same thing in nova except resize change the flavor and migrate does not | |
| 17:53:59 | stephenfin | gibi: Sorry for the delay. +2'd now :) | |
| 17:54:24 | cfriesen | sean-k-mooney: yeah, but there's some cells complication apparently: https://bugs.launchpad.net/nova/+bug/1243816 | |
| 17:54:26 | openstack | Launchpad bug 1243816 in OpenStack Compute (nova) "migrate fails with cells" [Undecided,Fix released] - Assigned to Chris Behrens (cbehrens) | |
| 17:54:30 | sean-k-mooney | cfriesen: so resize to the same flavor whould be expected to do the same thing as migrate however migrate uses the embeded fravor | |
| 17:55:42 | sean-k-mooney | yes well for cells v1 the flavor may not have been the same or replicated across cell. cross cell resize for cellsv2 is something mriedem is working on currently | |
| 18:00:45 | cfriesen | Looks like normally you cannot resize to the same flavor, so my concern is unfounded. | |
| 18:01:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Rework all things metadata'y https://review.openstack.org/640730 | |
| 18:01:20 | openstackgerrit | Stephen Finucane proposed openstack/nova master: docs: Fix nits in remote console guide https://review.openstack.org/641098 | |
| 18:02:06 | stephenfin | efried: You'd probably be interested in those come Friday ^ | |
| 18:03:56 | efried | ack | |
| 18:04:08 | efried | Friday because... after freeze? | |
| 18:05:26 | openstackgerrit | Chris Friesen proposed openstack/nova master: Flavor extra spec and image properties validation from API https://review.openstack.org/620706 | |
| 18:22:04 | openstackgerrit | Chris Friesen proposed openstack/nova master: Improve libvirt image and snapshot handling https://review.openstack.org/616692 | |
| 18:22:35 | cfriesen | any chance of getting some core eyes on this? ^ It's a host filesystem cache optimization patch, has a +1 from mdbooth. (And if you say it's simply not going into S then I'll shut up about it till next week.) | |
| 18:27:55 | openstackgerrit | Merged openstack/nova master: libvirt: implement reshaper for vgpu https://review.openstack.org/599208 | |
| 18:28:03 | openstackgerrit | Merged openstack/nova master: Add functional test for libvirt vgpu reshape https://review.openstack.org/631559 | |
| 18:28:11 | openstackgerrit | Merged openstack/nova master: FUP: docs nit https://review.openstack.org/639647 | |
| 18:28:56 | dansmith | cfriesen: you mean host buffer cache optimization yeah/ | |
| 18:29:25 | dansmith | that looks like a nice win and might be why my toy all-in-one openstack at home completely destroys performance while taking snapshots | |
| 18:41:56 | melwitt | dansmith: if something tries to reference unset user_id field, "NotImplementedError: Cannot load 'user_id' in the base class" happens. do you think we need to log a message in addition to that? | |
| 18:43:23 | dansmith | melwitt: I dunno, that ends up in a lot of places where we have an implementation error, so it's nice to log something about why (i.e. mappings haven't been migrated) but it's also a failing on our part, since it shouldn't be used anywhere anyway | |
| 18:43:46 | dansmith | melwitt: so you decide I guess.. but exploding is the right thing regardless of the, uh, type of explosion It hink | |
| 18:44:56 | melwitt | dansmith: ok, yeah, that makes sense. just log and reraise | |
| 18:54:08 | mriedem | mdbooth: btw, i hope you've seen office space: https://photos.app.goo.gl/HgzGk3aR2yGHWMv69 | |
| 19:04:06 | melwitt | lol | |
| 19:06:02 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.openstack.org/602432 | |
| 19:06:54 | melwitt | cfriesen: from the commit message, "so that cache option can be set properly". that makes it sound like it's a bug without that change. is it a bug or no? | |
| 19:06:58 | mriedem | cfriesen: approved your validation change and i'm assuming you're still working on a patch to check get_pci_requests_from_flavor from resize? | |
| 19:08:28 | dansmith | ah dangit, I was just going through that | |
| 19:08:42 | dansmith | melwitt: it's a feature | |
| 19:09:01 | melwitt | oh k | |
| 19:09:09 | dansmith | melwitt: it's not a bug because it's not wrong, but setting the cache properly avoids trashing the host's page cache while doing it, improving performance for everything running while the snapshot is happening | |
| 19:09:21 | melwitt | ah, thanks | |
| 19:09:32 | cfriesen | melwitt: what he said. :) | |
| 19:09:42 | dansmith | aside from the consistency gain of not calling the same external binary manually in multiple places | |
| 19:10:04 | cfriesen | mriedem: yes, just working on tests | |
| 19:10:11 | melwitt | ok, kewl | |
| 19:14:05 | efried | whoops, guess I should have been paying attention here before I +A'd that sucker. | |
| 19:16:31 | melwitt | it's np | |
| 19:16:50 | efried | okay, can pull it if necessary. | |
| 19:17:26 | melwitt | nah, it's ok. looking at it further, it would take me longer to understand all the qemu-img stuff anyway | |
| 19:28:51 | artom | sean-k-mooney, around? I need your Neutron and/or os-vif expertise | |