| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2018-01-15 | |||
| 16:02:23 | bauzas | ok, will review it when I have time then | |
| 16:03:32 | jaypipes | bauzas: that said, doesn't the REST API limit the length of the metadata value? | |
| 16:03:44 | jaypipes | bauzas: in which case a microversion is gonna be necessary | |
| 16:03:52 | bauzas | jaypipes: yup, possibly | |
| 16:04:02 | bauzas | tbh, I haven't reviewed it | |
| 16:04:11 | bauzas | just looking at the main modification | |
| 16:04:39 | bauzas | jaypipes: for your point, that'll depend if we cut the string by the API or not | |
| 16:05:10 | bauzas | but yeah, if it's all about having the API giving more than 255 chars for that, then yes, it's a microversion needed | |
| 16:09:46 | efried | namedtuple. Fabulous. | |
| 16:12:20 | prometheanfire | nova doesn't look like it supports the new os_vif https://review.openstack.org/532987 | |
| 16:12:35 | prometheanfire | will bug the list if you want | |
| 16:12:39 | edleafe | efried: not that fabulous | |
| 16:12:49 | efried | edleafe You don't like? | |
| 16:12:57 | edleafe | efried: see the comment I just posted | |
| 16:14:59 | efried | edleafe You set an attribute on the class, not an instance thereof. | |
| 16:15:31 | edleafe | ugh, was just thinking that through more. Coffee hasn't fully kicked in yet | |
| 16:15:40 | openstackgerrit | Merged openstack/nova master: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/389798 | |
| 16:15:55 | openstackgerrit | Merged openstack/nova master: Add aggregates check in allocation candidates https://review.openstack.org/522407 | |
| 16:20:25 | stephenfin | prometheanfire: Oh, looking | |
| 16:20:37 | stephenfin | sean-k-mooney, ralonsoh: Seen https://review.openstack.org/#/c/532987/ ? | |
| 16:21:06 | prometheanfire | stephenfin: cool | |
| 16:21:06 | sean-k-mooney | stephenfin: looking now | |
| 16:21:23 | prometheanfire | hopefully it's just a overly specific test | |
| 16:21:38 | ralonsoh | stephenfin: I'm taking a look now | |
| 16:22:35 | sean-k-mooney | stephenfin: at first glance these see unrelated to os-vif | |
| 16:23:00 | stephenfin | sean-k-mooney: Quite possibly. I haven't looked into it in detail | |
| 16:23:12 | stephenfin | Just saw the multiple rechecks. Usually a sign of something more sinister | |
| 16:23:25 | sean-k-mooney | stephenfin: the network basic opts test were failing due to a an ip confict | |
| 16:24:06 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_certs object https://review.openstack.org/489408 | |
| 16:25:05 | sean-k-mooney | stephenfin: yes however that fact that tempest.scenario.test_network_basic_ops.TestNetworkBasicOps passes on multinode (one of the failing test) means its likely not a os-vif issue | |
| 16:25:59 | sean-k-mooney | stephenfin: ill dig a little deeper | |
| 16:27:14 | sean-k-mooney | stephenfin: there are a whole bunch of Too many open files errors in n-cpu logs | |
| 16:27:54 | sean-k-mooney | e.g. http://logs.openstack.org/87/532987/1/check/legacy-tempest-dsvm-neutron-full/cb9c0bb/logs/screen-n-cpu.txt.gz?level=ERROR#_Jan_12_14_00_56_222328 | |
| 16:29:18 | openstackgerrit | Eric Fried proposed openstack/nova master: Track associated sharing RPs in report client https://review.openstack.org/526539 | |
| 16:29:18 | openstackgerrit | Eric Fried proposed openstack/nova master: SchedulerReportClient.set_traits_for_provider https://review.openstack.org/532564 | |
| 16:29:19 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.populate_from_iterable https://review.openstack.org/520756 | |
| 16:29:19 | openstackgerrit | Eric Fried proposed openstack/nova master: Raise on API errors getting aggregates/traits https://review.openstack.org/526540 | |
| 16:29:20 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Scheduler[Report]Client.get_provider_tree https://review.openstack.org/521098 | |
| 16:29:20 | openstackgerrit | Eric Fried proposed openstack/nova master: Track tree-associated providers in report client https://review.openstack.org/526541 | |
| 16:29:21 | openstackgerrit | Eric Fried proposed openstack/nova master: ProviderTree.snapshot => ProviderSnapshot https://review.openstack.org/533244 | |
| 16:29:21 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: ComputeDriver.update_provider_tree() https://review.openstack.org/521187 | |
| 16:29:22 | openstackgerrit | Eric Fried proposed openstack/nova master: WIP: Use update_provider_tree from resource tracker https://review.openstack.org/520246 | |
| 16:29:40 | efried | jaypipes cdent edleafe ^ -- and FYI, the bottom patch is also needed by mgoddard to get ironic traits moving. | |
| 16:29:58 | jaypipes | efried: sure, just finishing up a rebase | |
| 16:30:23 | efried | gibi also ^ -- thanks for the comments on the ProviderTree.snapshot patch. I've now slotted it into the series where it's gonna wind up. | |
| 16:30:36 | edleafe | efried: ack | |
| 16:31:07 | cdent | noted | |
| 16:42:08 | artom | Pci knowledgable folks, if I have a PciDevice (a <hostdev> in the libvirt xml), what would be the best way to find its MAC? | |
| 16:42:32 | artom | sahid, stephenfin ^^ ? | |
| 16:43:49 | sean-k-mooney | stephenfin: ill stack with the latest release of os-vif and see if i can replicate the ci failure on my end | |
| 16:44:08 | sahid | artom: not sure i understand, we do set the mac address when configuring the libvirt <interface>, no? | |
| 16:44:38 | artom | sahid, I mean when passing through an entire PF | |
| 16:44:47 | artom | It's a <hostdev> in the XML, not an <interface> | |
| 16:44:52 | artom | So there's no MAC | |
| 16:44:53 | sahid | oh i see | |
| 16:45:05 | stephenfin | sean-k-mooney: Sounds good. I'll take a look myself tomorrow | |
| 16:45:11 | stephenfin | See if we can figure out what's going on | |
| 16:45:16 | sahid | artom: no idea sorry | |
| 16:45:29 | artom | sahid, no worries :) | |
| 16:46:01 | artom | Who's the most knowledgeable PCI person we have now that vladikr and nikola aren't with us anymore? | |
| 16:47:12 | sean-k-mooney | stephenfin: it could be litally that we are not hitting the openfile limit but it could be something else too. if its ulimit then it something we need to fix in devstack | |
| 16:48:47 | sahid | artom: in such situation it's the guest os which have the device, the host can't really see it | |
| 16:48:58 | sahid | i mean i guess it can't | |
| 16:49:01 | sean-k-mooney | stephenfin: fyi this local.conf should allow you to reproduce http://paste.openstack.org/show/645256/ | |
| 16:49:09 | sahid | so probably your change is qemu-guest-agent | |
| 16:49:27 | stephenfin | sean-k-mooney: Funny how it would trigger now. Surely we were using privsep before? | |
| 16:49:29 | sahid | s/have the device/own the device | |
| 16:50:21 | sean-k-mooney | stephenfin: we were but maybe with the change to useing the python libs we are a using more filedicriptors or b we are leaking them? | |
| 16:50:58 | stephenfin | We could validate that by turning on warnings. Python 3 warns for open fds | |
| 16:54:51 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/pike: Do not attempt volume swap when guest is stopped/suspended https://review.openstack.org/533733 | |
| 16:55:23 | sean-k-mooney | stephenfin: it could possibly be a provider difference? the test that were failing on rackspace were passing on inap when run as part of the grenade multinode job. anyway ill see if i can reproduce locally | |
| 16:58:18 | sean-k-mooney | stephenfin: apparently https://review.openstack.org/508498 broke Kuryr-Kubernetes so we might need to do a 1.8.1/1.9.0 release to fix them though im not conviced the fix should not be on there end yet | |
| 16:59:21 | stephenfin | sean-k-mooney: Oh, someone else is using os-vif. Interesting... | |
| 16:59:49 | sean-k-mooney | ya i was aware kuryr was using it but i did not think they were ever saving os-vif objects | |
| 17:12:32 | openstackgerrit | Lee Yarwood proposed openstack/nova master: libvirt: QEMU native LUKS decryption for volumes https://review.openstack.org/523958 | |
| 17:23:16 | jaypipes | sean-k-mooney: I'm currently replying to mdulko about that... | |
| 17:23:51 | jaypipes | sean-k-mooney: bottom line, even if we did do obj_make_compatible(), that wouldn't have magically caused the client side to not break. | |
| 17:24:35 | jaypipes | sean-k-mooney: they still will need to advertise what versions of the objects they need (Nova does it via the nova-specific oslo versionedobject registry, etc) | |
| 17:24:42 | jaypipes | sean-k-mooney: there's no free lunch here... | |
| 17:24:52 | sean-k-mooney | jaypipes: right v2 of the patch had the make compatible code but you would have to do the down grade and i think the compat code just created an empty list of ip | |
| 17:25:03 | jaypipes | right. | |
| 17:25:07 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 17:25:10 | jaypipes | anyway, writing an answer... | |
| 17:25:21 | jaypipes | dansmith might be interested in this thread, too.. | |
| 17:25:55 | sean-k-mooney | the main point i wanted to get across was the lib freeze is thusday so if we can fix it by then it needs to be fixed in kuryr kubernetes | |
| 17:26:15 | jaypipes | sean-k-mooney: "fix" is relative here, which is my point :) | |
| 17:26:23 | sean-k-mooney | we could revert the patch i guess since nova is not using that field | |
| 17:26:30 | jaypipes | sean-k-mooney: "fix" might be just a flat revert, yeah | |
| 17:27:28 | sean-k-mooney | well we could certenly do that but they are using the object i a way we specifcially intoduced that patch to prevent so longterm we would have to adress the migration issue | |
| 17:31:34 | jaypipes | sean-k-mooney: did you even know that kuryr-kubernetes was consuming os-vif objects? I certainly didn't... | |
| 17:35:52 | openstackgerrit | Eric Berglund proposed openstack/nova master: PowerVM driver: ovs vif https://review.openstack.org/422512 | |
| 17:38:48 | melwitt | bauzas: this was previously approved by mriedem, so I think we can re-approve it after the rebase https://review.openstack.org/#/c/531407 | |
| 17:45:34 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: Recreate mediated devices on reboot https://review.openstack.org/533642 | |
| 17:45:34 | openstackgerrit | Sylvain Bauza proposed openstack/nova master: libvirt: create vGPU for instance https://review.openstack.org/528832 | |
| 17:46:13 | bauzas | melwitt: okie | |
| 17:47:04 | melwitt | thanks bauzas | |
| 17:54:17 | itlinux | hello nova team.. I have a OOO deployment looking to implement cells2, by default cells2 has one already any tips on how to expand that? Thanks | |
| 17:55:13 | sean-k-mooney | jaypipes: i knew that kuryr was for port plugging but i did not think they were ever storing if the os-vif objects | |
| 17:56:11 | openstackgerrit | Jackie Truong proposed openstack/nova master: Add trusted_image_certificates to REST API https://review.openstack.org/486204 | |
| 17:56:21 | sean-k-mooney | jaypipes: the ips field that we moved form the subnet object to the vif object is never used in os-vif or nova hence why i taught it was safe to move it | |