| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-06-21 | |||
| 10:17:04 | sean-k-mooney | stephenfin: oh dvs is a vmware thing | |
| 10:17:25 | sean-k-mooney | jangutter: switch statement unless you are channeling sql | |
| 10:17:43 | jangutter | sean-k-mooney: eep, was channeling Pascal of all things. | |
| 10:18:08 | sean-k-mooney | ha fair enough | |
| 10:18:49 | sean-k-mooney | you know if python had static type swe could have overload a function on teh argument type too | |
| 10:19:16 | sean-k-mooney | so many ways to solve this issue in other languages | |
| 10:20:36 | jangutter | stephenfin, sean-k-mooney: I haven't cross-checked, but how many of the VIFs in use by Neutron Stadium things are still legacy? | |
| 10:20:59 | jangutter | stephenfin, sean-k-mooney: I guess I'll have to go through the plugins in https://governance.openstack.org/tc/reference/projects/neutron.html | |
| 10:24:28 | sean-k-mooney | i dont think that many alot use ovs underneat | |
| 10:25:00 | jangutter | sean-k-mooney: yah, and a _lot_ of these are 'noop' plugging. | |
| 10:25:01 | sean-k-mooney | at this point i would be happy to just add drivers to os vif for the legecay stuff to get it out of nova and then we can drop them form os-vif later | |
| 10:25:13 | sean-k-mooney | jangutter: we have a driver for that :) | |
| 10:25:25 | jangutter | sean-k-mooney: yep. | |
| 10:29:57 | stephenfin | sean-k-mooney: RE: https://review.opendev.org/#/c/666630/3/nova/network/os_vif_util.py@545 | |
| 10:30:13 | stephenfin | We're still going to have a lot of lookups in https://review.opendev.org/#/c/666630/3/nova/network/os_vif_util.py@513 | |
| 10:30:37 | stephenfin | and I really don't want to change that as noted above | |
| 10:30:56 | sean-k-mooney | you dont have too | |
| 10:31:27 | sean-k-mooney | that shoudl be relitivly quick but you should make it a set | |
| 10:31:31 | sean-k-mooney | not a tuple | |
| 10:32:00 | stephenfin | okay, let me try | |
| 10:32:08 | stephenfin | In the interim, want to make sure https://review.opendev.org/#/c/666631/ makes sense? | |
| 10:32:10 | stephenfin | :D | |
| 10:32:25 | sean-k-mooney | stephenfin: if you define the set at module sope as a constant taht woudl also be better | |
| 10:32:44 | stephenfin | Even though it's only used in one place? | |
| 10:32:47 | sean-k-mooney | e.g. if vif_type in LEGACY_VIFS | |
| 10:33:10 | sean-k-mooney | sure why not its cheap | |
| 10:33:33 | sean-k-mooney | litally put it like 10 lines up on line 495 | |
| 10:34:49 | sean-k-mooney | i dont think our min version is at 3.2.0 yet | |
| 10:34:52 | sean-k-mooney | for libvirt | |
| 10:34:58 | sean-k-mooney | i think its currently at 3.0.0 | |
| 10:35:10 | stephenfin | Yeah, it is. That's why I just updated the docstring for one | |
| 10:36:01 | sean-k-mooney | oh sorry you fix the todo below that | |
| 10:36:06 | stephenfin | yup | |
| 10:36:52 | sean-k-mooney | do we set the vlan in the libvirt xml | |
| 10:37:24 | sean-k-mooney | as libvirt will only do that if we do | |
| 10:38:58 | sean-k-mooney | ok i think we do | |
| 10:39:17 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/vif.py#L352-L354 | |
| 10:40:12 | sean-k-mooney | ya we should be setting it here https://github.com/openstack/nova/blob/master/nova/virt/libvirt/designer.py#L116 | |
| 10:47:39 | stephenfin | okay, I really don't like using the dict lookup. I know it's slightly faster but it feel out of place in that file, given the amount of if-elses we have | |
| 10:48:00 | stephenfin | I have dragged the legacy VIFs out to their own variable though and used sets instead of tuples for 'in' lookups | |
| 10:48:15 | sean-k-mooney | ok | |
| 10:48:18 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630 | |
| 10:48:19 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another https://review.opendev.org/666631 | |
| 10:49:20 | sean-k-mooney | when we were using getattr before it was a dict lookup which is why i wanted to keep it that way | |
| 10:50:19 | stephenfin | Understandable. I think the additional clarity is worth the small cost though | |
| 10:56:32 | sean-k-mooney | stephenfin: will you get around to rebaseing https://review.opendev.org/#/c/663382/ soon or will i do it? | |
| 10:56:53 | stephenfin | oh, forgot about that. I'll do that today, yeah | |
| 10:57:50 | sean-k-mooney | ok i was talking to a partner about it yesterday so the sooner it lands upstream the sooner we can backport and the sooner they will be happy | |
| 10:58:52 | sean-k-mooney | ok im actuly goint ot go lie down now for a bit ill be back in a few hours | |
| 11:01:40 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814 | |
| 11:05:15 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Stop using getattr for VIF lookups https://review.opendev.org/666630 | |
| 11:05:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Remove dead minimum libvirt checks https://review.opendev.org/666814 | |
| 11:05:16 | openstackgerrit | Stephen Finucane proposed openstack/nova master: vif: Resolve a TODO and update another https://review.opendev.org/666631 | |
| 11:06:24 | openstackgerrit | Merged openstack/os-traits master: Sync Sphinx requirement https://review.opendev.org/666386 | |
| 11:13:00 | jangutter | stephenfin: coincidentally, that piece of getattr code is one of the first bits of python code that made me go "wow, you can do that, that's awesome!" | |
| 11:13:44 | jangutter | stephenfin: and also makes the code un-greppable, meaning even though it's cool code, I'm ++ in taking it out. | |
| 11:29:47 | stephenfin | jangutter: Exact same thing here, fwiw :) | |
| 11:30:01 | stephenfin | there are some other extraneous uses of getattr that I'll root out over time | |
| 11:32:00 | openstackgerrit | Adam Spiers proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 11:32:01 | openstackgerrit | Adam Spiers proposed openstack/nova master: Apply SEV-specific guest config when SEV is required https://review.opendev.org/644565 | |
| 11:32:01 | openstackgerrit | Adam Spiers proposed openstack/nova master: Add |
|
| 11:32:02 | openstackgerrit | Adam Spiers proposed openstack/nova master: Enable booting of libvirt guests with AMD SEV memory encryption https://review.opendev.org/666616 | |
| 11:37:08 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Unplug VIFs as part of cleanup of networks https://review.opendev.org/663382 | |
| 11:37:25 | stephenfin | sean-k-mooney: I'm going to need a functional test for ^ | |
| 11:37:32 | stephenfin | The initial version of the fix was totally wrong /o\ | |
| 11:59:19 | tssurya | The novaclient CI seems unhappy when running the openstack-tox-docs job (http://logs.openstack.org/92/666792/1/check/openstack-tox-docs/04b52fb/job-output.txt.gz#_2019-06-21_09_04_06_646600) | |
| 11:59:50 | tssurya | is there a bug reported already or should I open one | |
| 13:36:02 | openstackgerrit | Matt Riedemann proposed openstack/nova master: api-guide: better explain scheduler hints. https://review.opendev.org/522106 | |
| 13:36:43 | mriedem | gibi: easy docs change that you would probably be good for ^ | |
| 13:36:49 | mriedem | i'm already +2 | |
| 13:37:14 | mriedem | note that change is from nov 2017... | |
| 13:42:02 | bauzas | mriedem: +Wipped | |
| 13:42:15 | bauzas | easy peasy enough | |
| 13:42:48 | bauzas | FWIW, still working on providing the placement audit command | |
| 13:43:18 | openstackgerrit | Matthew Booth proposed openstack/nova master: Add functional test for bug 1550919 https://review.opendev.org/631294 | |
| 13:43:19 | openstackgerrit | Matthew Booth proposed openstack/nova master: libvirt: Don't delete disks on shared storage during evacuate https://review.opendev.org/578846 | |
| 13:43:19 | openstack | bug 1550919 in OpenStack Compute (nova) "[Libvirt]Evacuate fail may cause disk image be deleted" [Medium,In progress] https://launchpad.net/bugs/1550919 - Assigned to Matthew Booth (mbooth-9) | |
| 13:43:45 | dansmith | bauzas: allow me to provide some english help | |
| 13:43:54 | dansmith | bauzas: something is not "easy peasy" *enough* | |
| 13:44:02 | bauzas | :) | |
| 13:44:10 | dansmith | it's either peasy or not peasy, there is no gray area or gradient of peasiness | |
| 13:45:00 | bauzas | dansmith: but but... https://context.reverso.net/traduction/anglais-francais/peasy | |
| 13:45:17 | stephenfin | mriedem: What do you think I should log here? https://review.opendev.org/#/c/666630/5/nova/network/os_vif_util.py@535 Just "something bad happened" or maybe nothing at all? | |
| 13:45:43 | stephenfin | (Asking you since you've been involved in the earlier patches for both) | |
| 13:46:42 | mriedem | bauzas: easy peasy is fine | |
| 13:46:44 | mriedem | easy enough is fine | |
| 13:46:52 | dansmith | bauzas: exactly.. all of those examples are 100% peasy, right? no fractional peasiness | |
| 13:47:02 | dansmith | ^^ this man speaks the truth | |
| 13:47:02 | mriedem | but easy peasy enough is something that will cause a shared american/french tree of friendship to whither and die | |
| 13:47:09 | dansmith | lol | |
| 13:47:11 | bauzas | lol | |
| 13:47:29 | bauzas | gotcha | |
| 13:47:43 | bauzas | so, either "easy peasy" or "peasy enough" | |
| 13:47:46 | dansmith | yes. | |
| 13:47:49 | dansmith | no | |
| 13:47:54 | dansmith | easy peasy, or easy enough | |
| 13:47:57 | dansmith | not peasy enough | |
| 13:48:00 | bauzas | haha | |
| 13:48:09 | dansmith | gah | |
| 13:48:36 | mriedem | stephenfin: it's the same thing as you'd get before for those vif types so meh | |