| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-05-31 | |||
| 19:28:50 | mriedem | dansmith: there you go, all 3 ^ | |
| 19:28:57 | mriedem | assuming you can fast approve the func test one | |
| 19:31:12 | dansmith | right on | |
| 19:33:15 | mnaser | mriedem: fwiw -- https://review.opendev.org/662552 | |
| 19:34:01 | mriedem | delicious ansible | |
| 19:37:32 | mriedem | mnaser: question in there | |
| 19:37:40 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Extract SEV-specific bits on host detection https://review.opendev.org/636334 | |
| 19:37:41 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Add iommu driver to devices https://review.opendev.org/644564 | |
| 19:37:41 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Add configs for AMD SEV https://review.opendev.org/636318 | |
| 19:37:42 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Use fake flavor instead of empty dict in test https://review.opendev.org/662555 | |
| 19:37:42 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Detect that SEV is required and enable iommu for devices https://review.opendev.org/644565 | |
| 19:37:43 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Use launchSecurity element when SEV was required https://review.opendev.org/662557 | |
| 19:37:43 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Pass extra_specs to flavor in vif tests https://review.opendev.org/662556 | |
| 19:37:44 | openstackgerrit | Boris Bobrov proposed openstack/nova master: Enable memory locking if SEV is requested https://review.opendev.org/662558 | |
| 19:39:00 | mnaser | mriedem: good call, done | |
| 19:42:08 | mriedem | melwitt: thing in https://review.opendev.org/#/c/638324/ which you can tack into your FUP change | |
| 20:00:52 | openstackgerrit | Eric Fried proposed openstack/nova master: Hide hypervisor id on windows guests https://review.opendev.org/579897 | |
| 20:01:14 | openstackgerrit | Eric Fried proposed openstack/nova master: Hide hypervisor id on windows guests https://review.opendev.org/579897 | |
| 20:03:08 | efried | mriedem, melwitt: ^ | |
| 20:03:46 | efried | sounded like they weren't super interested in continuing to follow that patch given the controversy, I went ahead and fixed my nits. | |
| 20:04:13 | efried | The changes are small and test-only, but a fair number of lines, so I'd prefer not to +2 it myself at this point. | |
| 20:07:24 | mriedem | i think they just didn't care b/c they've already been running with it in prod for a year or more | |
| 20:09:41 | mriedem | but yeah thanks for updating it | |
| 20:13:39 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Update quota known issues docs https://review.opendev.org/662570 | |
| 20:18:10 | efried | mriedem: I'm updating the nested magic 1 spec, what do you want that footnote to look like? | |
| 20:19:39 | mriedem | just a footnote? | |
| 20:19:52 | mriedem | not a huge deal to me, that won't make me +2 the spec or anything | |
| 20:20:02 | mriedem | as i said, i feel unqualified to approve that spec | |
| 20:20:32 | efried | I don't know what you mean by "just a footnote" | |
| 20:21:10 | efried | you mean plain non-RST'd non-linked text? | |
| 20:22:05 | mnaser | I can't quite grasp if https://github.com/openstack/nova/blob/5868303f2c33021310eb4fade97828ef1757dc6b/nova/network/linux_net.py is nova-network or nova's networking code. | |
| 20:22:23 | mnaser | (I'm cleaning up OSA's long list of old dependencies, and we install bridge-utils which provides brctl, and im just wondering if its necessary) | |
| 20:22:39 | efried | mnaser: Pretty sure it's nova networking, but I could be wrong. | |
| 20:22:52 | efried | by which I mean: pretty sure it's still needed | |
| 20:22:57 | mnaser | I mean I dont think nova should be creating or deleting any bridges? | |
| 20:23:08 | mnaser | right, the neutron role should install bridge-utils, not nova in this case | |
| 20:23:22 | mnaser | (it'll end up there, but just want to make sure its the right thing that puts it there) | |
| 20:30:22 | melwitt | mriedem: k thanks | |
| 20:34:26 | mriedem | mnaser: os-vif likely handles the low-level bridge stuff now | |
| 20:34:36 | mriedem | sean-k-mooney would be the person to ask such questions | |
| 20:35:05 | mriedem | efried: i guess using a footnote http://docutils.sourceforge.net/docs/user/rst/quickref.html#footnotes | |
| 20:35:33 | mriedem | i guess what was in the spec was a citation? | |
| 20:35:34 | efried | mriedem: Yes, I'm using that third form. | |
| 20:37:03 | mriedem | ok just tell me to stfu on the spec then | |
| 20:37:30 | efried | mriedem, I live to make you happy. It's easy to change to something you consider prettier. Just tell me what that is and I'll do it. | |
| 20:38:19 | efried | note that the theme is going to mung the appearance - that's what's giving you the magical backreference link instead of the bold full-sized text on the footnote itself | |
| 20:38:36 | efried | so if I changed to [1] it's still going to be "ugly". | |
| 20:40:16 | mnaser | mriedem: ah good call, fyi -- https://github.com/openstack/os-vif/commit/1f6fed6a69e9fd386e421f3cacae97c11cdd7c75#diff-17b53d4f2c04032fb60eec093a443af0 | |
| 20:40:19 | mnaser | so .. no longer needed now | |
| 20:41:09 | openstackgerrit | Matt Riedemann proposed openstack/nova master: Cleanup quota user docs https://review.opendev.org/662573 | |
| 20:48:58 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/stein: Add regression recreate test for bug 1830747 https://review.opendev.org/662574 | |
| 20:49:01 | openstack | bug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |
| 20:50:49 | openstackgerrit | Merged openstack/nova master: extract baselineCPU API call from _get_cpu_traits() https://review.opendev.org/661782 | |
| 20:50:57 | openstackgerrit | Merged openstack/nova master: xenapi: log quality warning in init_host https://review.opendev.org/662295 | |
| 21:00:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.opendev.org/644793 | |
| 21:00:08 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove remaining vestiges of fake_libvirt_utils from unit tests. https://review.opendev.org/643896 | |
| 21:02:21 | openstackgerrit | Matt Riedemann proposed openstack/nova stable/rocky: Add regression recreate test for bug 1830747 https://review.opendev.org/662578 | |
| 21:02:22 | openstack | bug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |
| 21:29:32 | openstackgerrit | melanie witt proposed openstack/nova master: Follow up for counting quota usage from placement https://review.opendev.org/662056 | |
| 21:29:47 | melwitt | mriedem: lmk if that code comment makes sense now ^ | |
| 22:05:18 | openstackgerrit | Julia Kreger proposed openstack/nova stable/rocky: [ironic] Don't remove instance info twice in destroy https://review.opendev.org/662584 | |
| 22:23:10 | mnaser | efried: I realize this is probably not much of your thing but I noticed some commits to openstack-ansible with the PowerVM driver around 1.5 years ago by IBM-ers .. is there someone I can talk to about the usage of it (if any left..?) | |
| 22:23:44 | efried | mnaser: Who were the IBMers? | |
| 22:24:24 | efried | mnaser: The PowerVM driver is still changing hands. Evidence is that the new maintainers are still... very new and ramping up. But they are ramping up. | |
| 22:24:52 | efried | If you're asking about the actual PowerVM driver, I can probably help you. (I know nothing about ansible, though) | |
| 22:25:00 | mnaser | efried: I see https://review.opendev.org/#/c/518980/ and https://review.opendev.org/#/c/455350/ and https://review.opendev.org/#/c/450298/ as small things here and there | |
| 22:25:23 | mnaser | well, OSA apparently supports deploying it.. but it has absolutely zero testing, so I'm a bit curious if we're just maintaining something that no one is using or if there's usage behind it | |
| 22:25:36 | mnaser | and 'maintain' is a strong word because more like have no idea if any of it works :) | |
| 22:26:06 | efried | interesting question. | |
| 22:26:24 | efried | So yeah, Adam and Eric are long gone at this point. | |
| 22:27:04 | efried | And as of today, I know for sure the OOT driver is broken, because of this patch to fix it: https://review.opendev.org/#/c/662393/ | |
| 22:27:39 | efried | does osa "support" the in-tree or out-of-tree driver? | |
| 22:28:35 | mnaser | efried: looking at the code, we install nova-powervm and set the compute driver to 'powervm.driver.PowerVMDriver' | |
| 22:28:59 | efried | wow | |
| 22:29:13 | efried | mnaser: at what openstack version? | |
| 22:30:01 | mnaser | efried: nova-powervm is installed without constraints so I guess it'll always pull in latest | |
| 22:30:30 | mnaser | and this code has been around for a while, still in master | |
| 22:30:31 | efried | cause I'm pretty sure you can't use the oot driver at that path | |
| 22:30:46 | mnaser | yeah, googling around apparently it should be powervm_ext | |
| 22:30:49 | efried | yes | |
| 22:30:49 | efried | To use oot, you would have to say powervm_ext.driver.PowerVMDriver; and to use | |
| 22:30:59 | efried | so here's my best guess | |
| 22:31:16 | mnaser | so it is functionally broken since $release_that_added_ext | |
| 22:31:34 | efried | yeah, that was like 2015 or 16 | |
| 22:31:45 | efried | I'm guessing we put this stuff together to force some distro (like RHEL) to include nova-powervm in packaging, knowing it would never work as is | |
| 22:32:06 | efried | so that we could install PowerVC (management GUI) into that distro | |
| 22:32:14 | efried | PowerVC would fix up all the stuff | |
| 22:33:11 | efried | anyway, if you wanted to rip all that stuff out, I think you would be justified in proposing that patch, but it would be nice to poke the ML first. | |
| 22:33:45 | mnaser | I think ill push a patch and poke the ML because if no one proposes fixing it then it sitting around doesn't help | |
| 22:41:52 | efried | agreed | |
| 22:46:52 | mnaser | efried: http://lists.openstack.org/pipermail/openstack-discuss/2019-May/006797.html fyi :) | |
| 22:47:01 | efried | ack | |
| 23:23:23 | openstackgerrit | Dustin Cowles proposed openstack/nova master: Introduces SDK to IronicDriver and uses for node.get https://review.opendev.org/642899 | |
| #openstack-nova - 2019-06-01 | |||
| 00:21:33 | openstackgerrit | Merged openstack/nova master: Remove fake_libvirt_utils from virt driver tests. https://review.opendev.org/643894 | |
| 00:57:25 | openstackgerrit | Eric Fried proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.opendev.org/644793 | |
| 02:07:58 | openstackgerrit | Merged openstack/nova master: Remove fake_libvirt_utils from libvirt imagebackend tests. https://review.opendev.org/643895 | |
| 02:25:16 | openstackgerrit | Guo Jingyu proposed openstack/nova master: Define and catch QemuGuestAgentNotRunning for set-password failure https://review.opendev.org/661466 | |
| 02:32:06 | openstackgerrit | Merged openstack/nova master: Move _fill_provider_mapping to the scheduler_utils https://review.opendev.org/655108 | |
| 02:32:13 | openstackgerrit | Merged openstack/nova master: Add regression recreate test for bug 1830747 https://review.opendev.org/661822 | |
| 02:32:14 | openstack | bug 1830747 in OpenStack Compute (nova) "Error 500 trying to migrate an instance after wrong request_spec" [High,In progress] https://launchpad.net/bugs/1830747 - Assigned to Matt Riedemann (mriedem) | |