| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-05 | |||
| 09:57:07 | aspiers | I managed to get into mokmanager that way | |
| 09:57:10 | aspiers | or whatever it's called | |
| 09:57:20 | puiterwijk | Oh | |
| 09:57:21 | puiterwijk | "fwsetup" | |
| 09:57:26 | puiterwijk | that's the grub command | |
| 09:57:37 | puiterwijk | menuentry 'System setup' $menuentry_id_option 'uefi-firmware' { | |
| 09:57:37 | puiterwijk | fwsetup | |
| 09:57:37 | puiterwijk | } | |
| 09:57:46 | stephenfin | melwitt: Heh, sorry. That auto-topic thing was killing me downstream and like mriedem I always do 'git checkout -b bug/NNN' (or 'bp/foo') manually first. Also, it was broken with storyboard | |
| 09:58:12 | aspiers | IRC is awesome for apparent non-sequiturs | |
| 09:58:13 | stephenfin | melwitt: mordred reviewed it though so if you _really_ want to blame someone, I suggest blaming him. Everyone else does it :P | |
| 09:58:24 | aspiers | morning stephenfin | |
| 09:58:26 | aspiers | ;-p | |
| 09:58:31 | stephenfin | yo | |
| 09:58:44 | aspiers | puiterwijk: fwsetup is accepted but does nothing | |
| 09:59:18 | kashyap | aspiers: So you got into Grub command-line... | |
| 09:59:41 | stephenfin | I can see four new towers going up just from our window | |
| 09:59:53 | aspiers | kashyap: yes | |
| 09:59:58 | aspiers | stephenfin: try London for a bit | |
| 10:00:21 | stephenfin | once that legislation is passed, sure :) | |
| 10:00:23 | aspiers | I get "error: you need to load the kernel first." | |
| 10:00:48 | stephenfin | til then, I'll stay where food supplies aren't a concern | |
| 10:00:54 | aspiers | :'( | |
| 10:01:12 | aspiers | entirely understandable | |
| 10:01:36 | aspiers | well, more on principle | |
| 10:01:50 | aspiers | no a-hole is going to take away my EU membership | |
| 10:11:43 | sean-k-mooney | i think its funny that they keep talking about people in northerin ireland losing ther eu citizenship give that almost everyone born there is entitled to irish citizen ship grantign them full eu citizenship too | |
| 10:12:09 | aspiers | Yeah they have other things to worry about but not that | |
| 10:12:34 | sean-k-mooney | well not funny in the convnetional sense more tragic that politions in london dont know thats a thing | |
| 10:12:53 | aspiers | half our politicians are totally clueless | |
| 10:13:16 | aspiers | let's not drag this channel down that rathole though | |
| 10:13:32 | sean-k-mooney | quite so | |
| 10:13:59 | cdent | half? | |
| 10:13:59 | sean-k-mooney | its stressful enough watching the news | |
| 10:14:18 | aspiers | cdent: probably more, I was being generous off the back of yesterday | |
| 10:15:14 | cdent | somebody else was trying to tell me that yesterday was a spark of hope, but I'm not sure. But I agree, turning an openstack channel into the brexit-therapy-network is probably a misuse | |
| 10:15:28 | aspiers | it's definitely a spark of hope | |
| 10:15:34 | aspiers | not sure how big but we'll see | |
| 10:19:20 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 10:24:10 | aspiers | kashyap, sean-k-mooney: turns out that my new q35 patch introduced a circular import loop https://review.opendev.org/#/c/680065/ | |
| 10:24:19 | aspiers | so I'm trying to figure out the best way of moving code around | |
| 10:24:33 | kashyap | aspiers: Did it? Will look in a bit; buried in untangling something :-( | |
| 10:25:02 | aspiers | see discussion with efried last night http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2019-09-04.log.html#t2019-09-04T20:57:14 | |
| 10:25:09 | sean-k-mooney | aspiers: between what | |
| 10:25:26 | aspiers | between virt.hardware and libvirt.utils | |
| 10:25:46 | aspiers | and also libvirt.driver in between which imports libvirt.utils | |
| 10:25:54 | sean-k-mooney | right virt.hardware cant call the libvirt code | |
| 10:26:06 | aspiers | so there are some options for how to fix this | |
| 10:26:34 | aspiers | hmm, need a cloud-based sequence diagram editor now to help discuss this | |
| 10:27:01 | sean-k-mooney | i see what your importing | |
| 10:27:25 | aspiers | sean-k-mooney: it affects the next apply-config patch too | |
| 10:27:48 | aspiers | one option is to move sev_enabled from utils to hardware | |
| 10:28:14 | sean-k-mooney | actully libvit.utils does not use teh hardware module | |
| 10:28:30 | sean-k-mooney | so you shoudl be able to import that form hardware.py | |
| 10:28:53 | sean-k-mooney | aspiers: well they should be in hardware.py anyway | |
| 10:28:58 | aspiers | sean-k-mooney: https://review.opendev.org/#/c/644565/51/nova/virt/libvirt/utils.py@38 | |
| 10:29:17 | sean-k-mooney | oh you adding it in another patch | |
| 10:29:25 | aspiers | another option is a new machine_types.py | |
| 10:29:35 | aspiers | a third option efried suggested is a new virt.libvirt.hardware | |
| 10:30:05 | openstackgerrit | Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResourceList objs https://review.opendev.org/678448 | |
| 10:30:06 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449 | |
| 10:30:06 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450 | |
| 10:30:07 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452 | |
| 10:30:07 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453 | |
| 10:30:08 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454 | |
| 10:30:08 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 10:30:09 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 10:30:09 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 10:30:10 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 10:30:11 | openstackgerrit | Luyao Zhong proposed openstack/nova master: doc: attaching virtual persistent memory to guests https://review.opendev.org/680300 | |
| 10:30:13 | aspiers | but I was struggling to figure out whether SEV is libvirt specific. At least machine types are QEMU-specific | |
| 10:30:40 | sean-k-mooney | sev is tecnically not | |
| 10:30:48 | sean-k-mooney | but the way you are checking for it is | |
| 10:30:55 | kashyap | aspiers: SEV is hardware-specific; machine types indeed are not "libvirt | |
| 10:31:04 | kashyap | ... -specific" | |
| 10:31:13 | kashyap | As you discovered (read the chat). | |
| 10:31:25 | kashyap | But they're typically handled by libvirt for Nova. | |
| 10:31:29 | aspiers | Right | |
| 10:32:41 | aspiers | The problem is that get_mem_encryption_constraint() and its helpers are tied to virt.hardware | |
| 10:33:11 | sean-k-mooney | aspiers: move sev_enabled out of utils into the driver | |
| 10:33:12 | aspiers | because they rely on _get_flavor_image_meta() in that file, and so do other functions there | |
| 10:34:06 | aspiers | sean-k-mooney: originally sev_enabled had to be outside driver.py | |
| 10:34:13 | aspiers | so it could be called from vif.py | |
| 10:34:25 | aspiers | but now I moved the iommu stuff into designer.py | |
| 10:34:31 | aspiers | so maybe I could move it back | |
| 10:34:36 | sean-k-mooney | all calls from vif.py are form the driver | |
| 10:34:46 | sean-k-mooney | so you can pass it in a flag if its needed | |
| 10:34:58 | aspiers | that wasn't easily doable before | |
| 10:35:04 | sean-k-mooney | but we dont pas the libvirt host object to any other funciton in utils | |
| 10:35:07 | aspiers | but now it's not needed anyway | |
| 10:35:30 | sean-k-mooney | the utils dont actully talk to libvirt as far as i can see | |
| 10:35:36 | aspiers | right | |
| 10:35:52 | sean-k-mooney | so the sev_enabled check does not fit with the rest of the module | |
| 10:36:41 | sean-k-mooney | so ya simple fix is just make it a member funciton of the driver then you can do self.host and just pass in the flavor and image | |
| 10:37:14 | aspiers | I don't think this is sufficient | |
| 10:37:27 | aspiers | how would https://review.opendev.org/#/c/644565/51/nova/virt/libvirt/utils.py@38 change? | |
| 10:37:34 | aspiers | oh sorry | |
| 10:37:37 | sean-k-mooney | i will remove the only use of hardware.py form the utils | |
| 10:37:42 | aspiers | wrong link | |
| 10:38:06 | aspiers | how would https://review.opendev.org/#/c/680065/4/nova/virt/hardware.py change? | |