| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-09-22 | |||
| 11:38:17 | sean-k-mooney | bauzas: can you hit this https://review.opendev.org/c/openstack/placement/+/857900 | |
| 11:38:24 | sean-k-mooney | we need it for https://review.opendev.org/c/openstack/releases/+/858866 | |
| 11:47:00 | sean-k-mooney | bauzas: we are goning to need an rc2 for placment | |
| 11:47:18 | sean-k-mooney | well maybe not | |
| 11:47:20 | sean-k-mooney | https://github.com/openstack/placement/blob/master/bindep.txt#L31-L36 | |
| 11:47:40 | sean-k-mooney | on master i need to fix the bindeps that is wrong and really out of data | |
| 11:47:43 | sean-k-mooney | *date | |
| 11:48:04 | sean-k-mooney | we should fix it on stable/zed too but it does not nessisarly need to be RC2 | |
| 11:48:11 | sean-k-mooney | we could fix it after the offical release | |
| 11:48:16 | sean-k-mooney | im going to go fix it now | |
| 11:48:30 | sean-k-mooney | and file a bug and we can decided then if we do an RC2 or not | |
| 11:49:25 | sean-k-mooney | ill propose a backport in eitehr case and we can either hold it until after the reelase of merge it depening on how we feel about this | |
| 11:49:37 | sean-k-mooney | this bindep will not install on ubuntu 22.04 | |
| 11:49:45 | sean-k-mooney | which is what we use for the python 3.10 jobs | |
| 11:50:01 | sean-k-mooney | python-dev does not exist in 22.04 | |
| 12:02:30 | bauzas | sean-k-mooney: damn | |
| 12:04:21 | bauzas | sean-k-mooney: you can review this too https://review.opendev.org/c/openstack/nova/+/858087/1 | |
| 12:05:26 | opendevreview | Merged openstack/nova master: Update master for stable/zed https://review.opendev.org/c/openstack/nova/+/858084 | |
| 12:05:26 | bauzas | sean-k-mooney: add your thoughts on the bindep in https://etherpad.opendev.org/p/nova-zed-rc-potential | |
| 12:05:30 | sean-k-mooney | oh yes ill do that now | |
| 12:05:57 | sean-k-mooney | done | |
| 12:12:48 | opendevreview | Merged openstack/placement master: Update master for stable/zed https://review.opendev.org/c/openstack/placement/+/857900 | |
| 13:19:39 | opendevreview | sean mooney proposed openstack/placement master: update bindep for ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/858927 | |
| 13:20:07 | sean-k-mooney | bauzas: ^ that shoudl be enouch to resolve the issue | |
| 13:24:43 | bauzas | hmm, ok, will look | |
| 13:25:37 | sean-k-mooney | stephenfin: bauzas so there are 3 fix patches https://review.opendev.org/c/openstack/placement/+/829573 for rhel only https://review.opendev.org/c/openstack/placement/+/845074 for debin only that just need 1 +2 by the way | |
| 13:25:48 | sean-k-mooney | and my patch which fixes both | |
| 13:27:04 | sean-k-mooney | we could mege both of the older patches or merge mine | |
| 13:27:53 | bauzas | sean-k-mooney: I have a meeting in 3 mins, so please discuss this in 30 mins | |
| 13:28:04 | sean-k-mooney | i need to go do other stuff so please let me know how what ye want to do | |
| 13:28:12 | sean-k-mooney | sure | |
| 13:42:40 | opendevreview | Merged openstack/nova master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/nova/+/858087 | |
| 13:46:01 | stephenfin | sean-k-mooney: Can you drop the precise identifiers also? https://review.opendev.org/c/openstack/placement/+/858927 | |
| 13:54:12 | sean-k-mooney | oh ya sure ill do that now | |
| 13:55:30 | sean-k-mooney | im going to replace it with test by the way since we currently realy on it being there implictly | |
| 13:58:15 | opendevreview | sean mooney proposed openstack/placement master: update bindep for ubuntu 22.04 https://review.opendev.org/c/openstack/placement/+/858927 | |
| 13:58:51 | sean-k-mooney | stephenfin: ^ happy with that if not ill resping again quickly | |
| 14:08:17 | stephenfin | sean-k-mooney: +2 though I do have more comments that might warrant a respin. Will re +2 if you do | |
| 14:08:47 | sean-k-mooney | im currently doing a review but i am also heere so let me take a look | |
| 14:09:12 | sean-k-mooney | ya | |
| 14:09:17 | sean-k-mooney | so im fine with droping python | |
| 14:09:23 | sean-k-mooney | becuase you knwo bindep needs it | |
| 14:09:44 | sean-k-mooney | so if you dont know why we have it i can remove it but i think its to ensure we have all the python version we need | |
| 14:10:45 | sean-k-mooney | i.e. i think the tox job does not preinstall the python versoin for the current job | |
| 14:11:01 | sean-k-mooney | and bindep is isntalling all of them so when tox runs the required version is there | |
| 14:11:07 | sean-k-mooney | does that make sesne | |
| 14:12:21 | sean-k-mooney | actully no | |
| 14:12:23 | sean-k-mooney | https://packages.ubuntu.com/focal/python3-all | |
| 14:12:32 | sean-k-mooney | so its just pulling the default python looking at the deps | |
| 14:12:44 | sean-k-mooney | althoguh that might be wrong | |
| 14:12:49 | sean-k-mooney | package depending on all supported Python 3 runtime versions | |
| 14:13:03 | sean-k-mooney | so that appears to by why we are using it | |
| 14:19:13 | bauzas | sean-k-mooney: I'm done with my meeting but I need to go taxidading | |
| 14:19:30 | bauzas | I'll look at your patches when I'm back | |
| 14:40:38 | zigo | My 1st Zed VM is up and pings ... :P | |
| 14:41:18 | sean-k-mooney | :) | |
| 14:49:57 | gmann | sean-k-mooney: I thought it was merged. gibi stephenfin please check this https://review.opendev.org/c/openstack/placement/+/845074 | |
| 14:50:20 | gmann | or bauzas ^^ | |
| 14:55:44 | stephenfin | gmann: We were just talking about that. https://review.opendev.org/c/openstack/placement/+/858927 does the same but for RHEL/Fedora also | |
| 14:55:54 | stephenfin | Is this blocking the gate? | |
| 14:56:36 | gmann | i think py310 job is failing whihc is voting now | |
| 14:57:24 | frickler | it is blocking https://review.opendev.org/c/openstack/placement/+/857901 which should be merged before anything else in master | |
| 14:58:02 | stephenfin | gmann: Are you core on placement? Want to look over this? https://review.opendev.org/c/openstack/placement/+/858927 | |
| 14:58:32 | stephenfin | I +W'd it to unblock the gate but it's always good to have another pair of eyes | |
| 14:58:45 | gmann | sure | |
| 14:59:27 | gmann | done | |
| 14:59:34 | opendevreview | Stephen Finucane proposed openstack/placement master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/placement/+/857901 | |
| 14:59:45 | stephenfin | ty | |
| 15:00:06 | stephenfin | rebased that and +W so they should merge together (CC: Frickler) | |
| 15:17:46 | bauzas | sean-k-mooney: gmann: stephenfin: fwiw, +Wd https://review.opendev.org/c/openstack/placement/+/858927 | |
| 15:18:25 | bauzas | tbc, we don't need to backport it to Zed, right? | |
| 15:18:29 | bauzas | this is just for Antelope | |
| 15:18:58 | gmann | bauzas: yes. only for Antelope | |
| 15:19:13 | bauzas | yeah, just verified https://governance.openstack.org/tc/reference/runtimes/zed.html | |
| 15:19:19 | bauzas | 20.04 | |
| 15:19:22 | bauzas | we're all good | |
| 16:00:20 | melwitt | sean-k-mooney: I wanted to get your thoughts on this bug fix, do you think this is an ok thing to do or any problems with it? I was unsure about whether it's proper to use qemu-img info to determine the disk format of a volume https://review.opendev.org/c/openstack/nova/+/858836 | |
| 16:02:03 | sean-k-mooney | is this related ot luks | |
| 16:02:27 | melwitt | sean-k-mooney: no, it's just nfs | |
| 16:02:28 | sean-k-mooney | oh no its differnt | |
| 16:02:45 | sean-k-mooney | ya i tough i swa an nfs related patch for something like this | |
| 16:02:52 | sean-k-mooney | oh i rememebr the bug | |
| 16:03:08 | sean-k-mooney | so snapshots on nfs have to be qcow right | |
| 16:03:23 | melwitt | right | |
| 16:03:33 | sean-k-mooney | like the os-assisted-snapshost api say it must be that | |
| 16:03:36 | sean-k-mooney | ok | |
| 16:03:42 | melwitt | yes it does | |
| 16:04:11 | sean-k-mooney | so when i was looking at the snapshot code i think i say we shoudl usjt update it to be qcow | |
| 16:04:19 | melwitt | but on our side we store the volume attachment connection_info and the format captured there represents the format of the volume (which is raw) not the snapshot (which is qcow2) | |
| 16:04:58 | sean-k-mooney | ya | |
| 16:05:03 | melwitt | yeah, the thing is how to "remember" that when the instance is rebooted for example | |
| 16:05:50 | sean-k-mooney | let me see if i can find the code i was suggesting changing but basicaly i was suggestin updating the format on the image | |
| 16:06:07 | sean-k-mooney | what your dowing sound valid | |
| 16:06:13 | sean-k-mooney | just not sure if its requried | |
| 16:06:18 | melwitt | oh, hm | |
| 16:07:54 | sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L2932-L2958 | |
| 16:08:00 | sean-k-mooney | i tought we could fix that in that code | |
| 16:09:04 | melwitt | ok, I will look at that. thanks for the pointer | |
| 16:10:16 | sean-k-mooney | honestly i think when we create teh snapshot the connection info should get updated to qcow | |
| 16:11:11 | melwitt | sean-k-mooney: I tried that first https://review.opendev.org/c/openstack/cinder/+/857528 and it's run into problems | |