Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2019-05-31
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 remaining vestiges of fake_libvirt_utils from unit tests. https://review.opendev.org/643896
21:00:08 openstackgerrit Eric Fried proposed openstack/nova master: Remove fake_libvirt_utils users in functional testing. https://review.opendev.org/644793
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 To use oot, you would have to say powervm_ext.driver.PowerVMDriver; and to use
22:30:49 efried yes
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)
02:32:24 openstackgerrit Merged openstack/nova master: Use instance mappings to count server group members https://review.opendev.org/638324
02:32:36 openstackgerrit Merged openstack/nova master: Add documentation for counting quota usage from placement https://review.opendev.org/653845
03:49:43 openstackgerrit Merged openstack/nova stable/stein: Fixes multi-registry config in Quobyte driver https://review.opendev.org/660706
04:34:25 openstackgerrit Merged openstack/nova stable/stein: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/661494
10:29:58 openstackgerrit Boris Bobrov proposed openstack/nova master: Add configs for AMD SEV https://review.opendev.org/636318
10:33:45 openstackgerrit Boris Bobrov proposed openstack/nova master: Add iommu driver to devices https://review.opendev.org/644564
#openstack-nova - 2019-06-02
02:47:15 openstackgerrit Guo Jingyu proposed openstack/nova master: Define and catch QemuGuestAgentNotRunning for set-password failure https://review.opendev.org/661466
07:30:41 openstackgerrit Adrian Chiris proposed openstack/nova stable/queens: Move get_pci_mapping_for_migration to MigrationContext https://review.opendev.org/661571
07:30:42 openstackgerrit Adrian Chiris proposed openstack/nova stable/queens: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/661572
11:49:29 openstackgerrit Merged openstack/nova stable/stein: Allow driver to properly unplug VIFs on destination on confirm resize https://review.opendev.org/661495
12:26:57 openstackgerrit zhaixiaojun proposed openstack/nova master: Exclude broken ironicclient versions 2.7.1 https://review.opendev.org/662404
20:38:50 openstackgerrit Dr. Clemens Hardewig proposed openstack/nova master: Fixes instance resizing/migration when swap volume is on lvm https://review.opendev.org/618621
#openstack-nova - 2019-06-03
02:11:28 openstackgerrit Takashi NATSUME proposed openstack/nova master: Remove mox in virt/test_block_device.py https://review.opendev.org/566153
03:10:47 openstackgerrit Merged openstack/nova-specs master: Fix a typo of 'keystoneauth1' https://review.opendev.org/614187

Earlier   Later