| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-19 | |||
| 11:10:42 | sean-k-mooney | gokhanis: i think your hitiing this https://bugzilla.redhat.com/show_bug.cgi?id=1547095 | |
| 11:11:47 | sean-k-mooney | gokhanis: apprently you can "fix" it by using the nolock mount option for nfsv3 | |
| 11:12:07 | sean-k-mooney | gokhanis: also relevent https://access.redhat.com/solutions/2780381 | |
| 11:12:50 | gokhanis | sean-k-mooney, I deployed my env with Openstack Ansible. I have create nfs share on zfs storage and I used it as cinder nfs backend. | |
| 11:12:56 | sean-k-mooney | gokhanis: form a downstream perspetive redhat discontinued support for nfsv3 in our porudct because of these locking issues | |
| 11:14:21 | sean-k-mooney | gokhanis: yep so on the zfs culster you need to either ensure that the zfs pool is exported as nfsv4, nfsv3 with the nfsv4 lock manager or you have to ensure the nfs share is mounted with the nolock option on the compute hosts | |
| 11:15:11 | sean-k-mooney | gokhanis: you can use https://docs.openstack.org/nova/latest/configuration/config.html#libvirt.nfs_mount_options to set teh nolock option i think | |
| 11:15:48 | sean-k-mooney | but if the fileserver supprot v4 then vers=4.2,lookupcache=pos | |
| 11:15:56 | sean-k-mooney | woudl be better then nolock | |
| 11:17:18 | sean-k-mooney | gokhanis: some more details on nolock https://github.com/oVirt/vdsm/commit/8aad4ebad10bab15c46ff743c105b6c2cba68d4e | |
| 11:17:32 | gokhanis | thanks sean-k-mooney I will check nfs side in my env and add options you have said. | |
| 12:39:22 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Configure and teardown ephemeral encryption secrets https://review.opendev.org/c/openstack/nova/+/826754 | |
| 12:39:22 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Add support to libvirt_info for LUKS based encryption https://review.opendev.org/c/openstack/nova/+/826755 | |
| 12:39:23 | opendevreview | melanie witt proposed openstack/nova master: imagebackend: Cache the key manager when disk is encrypted https://review.opendev.org/c/openstack/nova/+/826756 | |
| 12:39:23 | opendevreview | melanie witt proposed openstack/nova master: Support create with ephemeral encryption for qcow2 https://review.opendev.org/c/openstack/nova/+/870932 | |
| 12:39:24 | opendevreview | melanie witt proposed openstack/nova master: Support resize with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870933 | |
| 12:39:24 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to convert_image https://review.opendev.org/c/openstack/nova/+/870934 | |
| 12:39:25 | opendevreview | melanie witt proposed openstack/nova master: Add hw_ephemeral_encryption_secret_uuid image property https://review.opendev.org/c/openstack/nova/+/870935 | |
| 12:39:25 | opendevreview | melanie witt proposed openstack/nova master: Add encryption support to qemu-img rebase https://review.opendev.org/c/openstack/nova/+/870936 | |
| 12:39:26 | opendevreview | melanie witt proposed openstack/nova master: Support snapshot with ephemeral encryption https://review.opendev.org/c/openstack/nova/+/870937 | |
| 12:39:27 | opendevreview | melanie witt proposed openstack/nova master: Add reset_encryption_fields() and save_all() to BlockDeviceMappingList https://review.opendev.org/c/openstack/nova/+/870938 | |
| 12:39:27 | opendevreview | melanie witt proposed openstack/nova master: Update driver BDMs with ephemeral encryption image properties https://review.opendev.org/c/openstack/nova/+/870939 | |
| 12:39:29 | opendevreview | melanie witt proposed openstack/nova master: libvirt: Introduce support for qcow2 with LUKS https://review.opendev.org/c/openstack/nova/+/772273 | |
| 12:44:10 | opendevreview | melanie witt proposed openstack/nova master: DNM test ephemeral encryption + resize: qcow2, raw https://review.opendev.org/c/openstack/nova/+/862416 | |
| 12:45:31 | opendevreview | Rajesh Tailor proposed openstack/nova master: Handle InstanceInvalidState exception https://review.opendev.org/c/openstack/nova/+/861738 | |
| 12:57:02 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: At start-up allow skiping compareCPU() with a workaround https://review.opendev.org/c/openstack/nova/+/870794 | |
| 12:57:16 | kashyap | gibi: --^ Fixed the test, please have a re-look if you get a chance today | |
| 13:39:55 | opendevreview | Merged openstack/nova master: Clean up after ImportModulePoisonFixture https://review.opendev.org/c/openstack/nova/+/870993 | |
| 13:43:39 | gibi | bauzas, dansmith: the dstat log in https://review.opendev.org/c/openstack/nova/+/870950 shows that without that one test case that loads the image data there is no other memory hogger test case. At the highest memory load there is still 800MB free RAM in the VM. | |
| 13:51:20 | gibi | kashyap: +2, thanks | |
| 13:51:45 | sean-k-mooney | oh something merged thats a shock | |
| 13:52:19 | sean-k-mooney | its because test only change do not run the tempest jobs | |
| 13:52:24 | sean-k-mooney | similar for docs onlychanges | |
| 13:53:26 | kashyap | gibi: Thank you; (I fixed your other points too; sure you've seen it) | |
| 13:54:01 | kashyap | gibi: Is the nit worth respinning? | |
| 13:54:44 | sean-k-mooney | gibi: can you look at https://review.opendev.org/c/openstack/nova/+/869950 | |
| 13:54:45 | kashyap | sean-k-mooney: melwitt: or anyone else: I'm chaing a second +2/+W: please have a look at this workaround - https://review.opendev.org/c/openstack/nova/+/870794 | |
| 13:55:30 | sean-k-mooney | i have it open but i need to leave in 5 mins to drop back the tempory car i have and get back before my next meeting | |
| 13:55:37 | kashyap | gibi: sean-k-mooney: On _top_ of this workaround I'll add in the "replace old API with new API" patch - https://review.opendev.org/c/openstack/nova/+/869950 -- libvirt: Replace usage of compareCPU() with compareHypervisorCPU() | |
| 13:55:42 | sean-k-mooney | so it might be when i get back | |
| 13:55:45 | kashyap | Sure. | |
| 13:56:17 | sean-k-mooney | yep i want to merge both although i also think we should simplfy it so we only do one check | |
| 13:56:22 | gibi | kashyap: that nit from me does not worth a respin | |
| 13:56:50 | sean-k-mooney | e.g. only check with model+extra_flags | |
| 13:57:08 | kashyap | Yep; ie. remove the workaround-code | |
| 13:57:17 | kashyap | gibi: Nod; thx | |
| 13:57:34 | kashyap | (Remove later, i.e. not now :)) | |
| 13:57:59 | sean-k-mooney | anyway ill be back hopefully in an hour | |
| 13:58:10 | gibi | kashyap: so https://review.opendev.org/c/openstack/nova/+/869950 will be top of https://review.opendev.org/c/openstack/nova/+/870794 ? | |
| 13:58:21 | kashyap | gibi: Right | |
| 13:58:27 | kashyap | Makes sense to you? | |
| 13:58:35 | sean-k-mooney | well the workaround right now is disabling both check right | |
| 13:58:44 | gibi | kashyap: yeah that make sens | |
| 13:58:58 | sean-k-mooney | i have not looked at that patch in a few revisions | |
| 13:58:59 | kashyap | sean-k-mooney: Yes, only if you enable the workaround, obv. | |
| 13:59:27 | kashyap | sean-k-mooney: It's dead-simple - if workaround -> just skip the first _compare_cpu() call. | |
| 13:59:53 | kashyap | sean-k-mooney: And thanks, I see you've also +2ed the API-replacement patch, but I need to rebase it on top of the workaroud patch | |
| 14:00:09 | sean-k-mooney | and we still do the second call in init wiht the flags | |
| 14:00:49 | sean-k-mooney | ok am ill leave this for now and come back to it when i have time to review both again | |
| 14:01:39 | kashyap | Yeah, the second call correctly succeeds as it also takes into account the flags | |
| 14:02:22 | sean-k-mooney | yep so as long as that is alwasy there i was ok with removing the first call | |
| 14:02:38 | sean-k-mooney | i was not ok with removeing both of them and that why i wanted the workaround | |
| 14:02:51 | sean-k-mooney | anyway got to go be back later | |
| 14:05:58 | opendevreview | Sofia Enriquez proposed openstack/nova master: Implement is_luks_inside_qcow2 funtion https://review.opendev.org/c/openstack/nova/+/854030 | |
| 14:22:26 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Replace usage of compareCPU() with compareHypervisorCPU() https://review.opendev.org/c/openstack/nova/+/869950 | |
| 14:23:07 | kashyap | gibi: --^ (No hurry) Done, rebased it on top of the workaround. | |
| 14:33:17 | gibi | kashyap: done :) | |
| 14:34:10 | kashyap | (シ_ _)シ | |
| 14:34:16 | kashyap | (It's a Japanese bow) | |
| 14:47:56 | gibi | ahh, I see :) | |
| 15:00:36 | dansmith | gibi: nice | |
| 15:00:48 | dansmith | gibi: I'm working on the real fix, but it's giving me a hard time.. lots of layers to cut through | |
| 15:06:54 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: fixtures: Handle binding of first port https://review.opendev.org/c/openstack/nova/+/871048 | |
| 15:08:22 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/ussuri: Neutron fixture: don't clobber profile and vif_details if empty https://review.opendev.org/c/openstack/nova/+/871049 | |
| 15:08:25 | dansmith | ...aaand I got it working. sleep helps. | |
| 15:08:38 | dansmith | now to clean up the giant mess I've made | |
| 15:10:18 | gibi | dansmith: sounds good | |
| 15:28:02 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: fixtures: Handle binding of first port https://review.opendev.org/c/openstack/nova/+/796905 | |
| 15:28:42 | opendevreview | Alexey Stupnikov proposed openstack/nova stable/victoria: Neutron fixture: don't clobber profile and vif_details if empty https://review.opendev.org/c/openstack/nova/+/796906 | |
| 17:45:07 | bauzas | gibi: all good to know | |
| 17:45:49 | bauzas | hopefully https://review.opendev.org/c/openstack/tempest/+/870974 will be merged tonight | |
| 20:50:03 | opendevreview | Sofia Enriquez proposed openstack/nova master: WIP: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | |
| 21:11:49 | opendevreview | Sofia Enriquez proposed openstack/nova master: WIP: Implement encryption on backingStore https://review.opendev.org/c/openstack/nova/+/870012 | |
| #openstack-nova - 2023-01-20 | |||
| 09:10:15 | opendevreview | Hiroki Narukawa proposed openstack/nova master: libvirt: retry libvirt connection on live_migration_monitor https://review.opendev.org/c/openstack/nova/+/867077 | |
| 09:33:48 | opendevreview | Nobuhiro MIKI proposed openstack/os-traits master: Add new 'COMPUTE_ADDRESS_SPACE_*' traits https://review.opendev.org/c/openstack/os-traits/+/871226 | |
| 10:38:33 | kashyap | bauzas: sean-k-mooney: Can anyone please put this through? -- https://review.opendev.org/c/openstack/nova/+/870794/6 -- libvirt: At start-up allow skiping compareCPU() with a workaround | |
| 10:38:55 | bauzas | kashyap: sure, adding on my today's list | |
| 10:41:09 | kashyap | bauzas: FWIW, it is already tested in a real env by a downstream colleague | |
| 10:41:17 | bauzas | kashyap: yup, I knew | |
| 10:41:27 | kashyap | Ah, thx | |
| 10:41:46 | sean-k-mooney | kashyap: so that is not really what i was asking for by th eway with the workaround | |
| 10:41:58 | kashyap | sean-k-mooney: Sigh, what then? | |
| 10:42:01 | sean-k-mooney | your orginal patch did this https://review.opendev.org/c/openstack/nova/+/869587/4/nova/virt/libvirt/driver.py | |
| 10:42:08 | sean-k-mooney | removing both checks | |
| 10:42:14 | sean-k-mooney | unconditionally | |
| 10:42:24 | kashyap | sean-k-mooney: Right; but please see the discussion in the patch itself | |
| 10:42:25 | sean-k-mooney | i wanted you to add the workaroudn if you were removing both checks | |
| 10:42:43 | sean-k-mooney | the new one only removes the first check condtionally | |
| 10:42:45 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/870794/6/nova/virt/libvirt/driver.py | |
| 10:42:54 | kashyap | sean-k-mooney: Yes, the new one only removes the first check | |