| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-24 | |||
| 17:00:33 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (drivers and compute manager part) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 17:00:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 17:00:34 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 17:00:35 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 17:00:35 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 17:00:36 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 17:00:36 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 17:00:37 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to attach/detach shares https://review.opendev.org/c/openstack/nova/+/852085 | |
| 17:00:38 | opendevreview | ribaudr proposed openstack/nova master: Add virt/libvirt error test cases https://review.opendev.org/c/openstack/nova/+/852087 | |
| 17:00:38 | opendevreview | ribaudr proposed openstack/nova master: Add libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 17:00:40 | opendevreview | ribaudr proposed openstack/nova master: Support rebooting an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/854824 | |
| 17:00:40 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to reboot method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/854823 | |
| 17:00:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach_error notification https://review.opendev.org/c/openstack/nova/+/860283 | |
| 17:00:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 17:00:44 | opendevreview | ribaudr proposed openstack/nova master: Support resuming an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860285 | |
| 17:00:44 | opendevreview | ribaudr proposed openstack/nova master: Add share_info parameter to resume method for each driver (driver part) https://review.opendev.org/c/openstack/nova/+/860284 | |
| 17:00:46 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (driver part) https://review.opendev.org/c/openstack/nova/+/860287 | |
| 17:00:46 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 17:00:48 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.XX https://review.opendev.org/c/openstack/nova/+/852088 | |
| 17:00:48 | opendevreview | ribaudr proposed openstack/nova master: Support rescuing an instance with shares (compute and API part) https://review.opendev.org/c/openstack/nova/+/860288 | |
| 17:00:50 | opendevreview | ribaudr proposed openstack/nova master: Documentation https://review.opendev.org/c/openstack/nova/+/871642 | |
| 17:21:49 | priteau | dansmith: Hello. Is there a reason for no backport of https://review.opendev.org/c/openstack/nova/+/871622 to wallaby yet? It seems to backport cleanly with the resolution done in xena. | |
| 17:22:18 | dansmith | priteau: yes, because wallaby is EOL | |
| 17:22:55 | dansmith | well, EM | |
| 17:23:23 | priteau | But you will still propose a patch for it or not at all? | |
| 17:23:41 | dansmith | priteau: you go ahead :) | |
| 17:23:49 | priteau | OK | |
| 17:24:35 | priteau | The backports don't have the "cherry picked from commit" line by the way | |
| 17:24:48 | priteau | (maybe because it wasn't a cherry pick at all) | |
| 17:25:08 | opendevreview | Pierre Riteau proposed openstack/nova stable/wallaby: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871557 | |
| 17:26:17 | priteau | Hum, maybe it is new Gerrit behavior? | |
| 17:30:49 | dansmith | priteau: because they were all pre-canned ahead of time yeah | |
| 17:31:04 | gibi | priteau: I think the cherry picked from line is comfing from git cherry-pick -x | |
| 17:31:56 | priteau | No, I know what's happening. Gerrit only adds it when you cherry-pick a *merged* change. | |
| 17:32:17 | priteau | I cherry-picked from stable/xena to get the conflict resolution, this hasn't merged yet. | |
| 17:33:17 | gibi | priteau: true. if you want to add it then you can by doing the cherry-pick with git and using -x | |
| 17:34:19 | opendevreview | Pierre Riteau proposed openstack/nova stable/wallaby: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871557 | |
| 17:34:37 | priteau | Yes, I've done this many times. Here we go. | |
| 18:21:51 | gibi | Uggla: I re-reviewd the first half of the manial series mostly OK with it buth johnthetubaguy had some valid questions there so I left -1 to have the visibility | |
| 18:21:58 | gibi | I will continue tomorrow | |
| 18:22:09 | gibi | Uggla: do you have a set of functional tests added to the series? | |
| #openstack-nova - 2023-01-25 | |||
| 01:51:23 | opendevreview | Merged openstack/nova master: Check VMDK create-type against an allowed list https://review.opendev.org/c/openstack/nova/+/871612 | |
| 02:20:55 | gmann | gibi: sean-k-mooney: please review the placement RBAC change, https://review.opendev.org/c/openstack/placement/+/865618 | |
| 02:22:06 | gmann | I am sure it is in your list, just wanted to get review/merge this soon in case any thing we need to update/test we can do before m-3 | |
| 02:34:50 | sean-k-mooney[m] | gmann: i have one question in line regarding listing traits | |
| 02:35:54 | sean-k-mooney[m] | so im +1 but if we dont want to change listing traits im +2 on the change i think | |
| 02:36:00 | sean-k-mooney[m] | illl check back in my morning | |
| 02:36:46 | gmann | sean-k-mooney[m]: thanks. will check and reply | |
| 05:18:15 | gmann | sean-k-mooney[m]: replied https://review.opendev.org/c/openstack/placement/+/865618/2/placement/policies/trait.py#41 | |
| 08:27:15 | Uggla | gibi, hi I will have a look at the comments left by john. Strangely I did not noticed them before. (OO) | |
| 08:28:50 | Uggla | gibi, regarding functional test yes in the latest patches there are some of them. Or do you think about tempest tests ? | |
| 08:34:59 | gibi | Uggla: no, not tempest. But then I will see them as I progress with the review today... | |
| 08:48:34 | zigo | Hi there! | |
| 08:49:34 | zigo | I believe I have a working train patch for CVE-2022-47951, however, I had to change the default value of oslo.utils's QemuImgInfo() from format='human' to format='json'. | |
| 08:50:02 | zigo | What should I do, should I make the Nova call add format='json' to the call, or change the default in oslo.utils? | |
| 08:50:11 | zigo | gibi: Your opinion? | |
| 08:52:12 | zigo | Oh, I have my answer ... :) | |
| 08:52:16 | frickler | since the fix is for nova, I would keep it restricted to that. just my 0.03€ | |
| 08:52:16 | frickler | since the fix is for nova, I would keep it restricted to that. just my 0.03€ | |
| 08:52:28 | zigo | Latest version has: https://github.com/openstack/nova/blob/master/nova/virt/images.py#L48 (ie: format='json') | |
| 08:52:51 | zigo | So I'll do that ... | |
| 09:02:54 | gibi | zigo: better to call from nova with format='json' to limit the change to that single call. But I see you arrived to that solution anyhow | |
| 09:03:18 | zigo | Yeah ! | |
| 09:03:31 | zigo | Hopefully, I can just take these patches and do -> stein -> rocky, and then I'm good ! :) | |
| 09:05:31 | zigo | Contrary to what I wrote yesterday, only backporting https://review.opendev.org/c/openstack/nova/+/706897 was enough to get the VMDK check work in Train (plus that oslo.utils patch...). | |
| 09:15:28 | zigo | Shit, other failures ... :/ | |
| 09:18:38 | johnthetubaguy | For security fixes, does the usual branch ordering of merging the fixss apply, I don't remember? i.e. do we just merge each stable patch as it goes green, or we do them in order? | |
| 09:18:59 | johnthetubaguy | (i.e. xena seems ready to go now) | |
| 09:20:31 | zigo | I may need all of https://review.opendev.org/c/openstack/nova/+/711679 after all ... | |
| 09:21:22 | gibi | johnthetubaguy: I don't know about any exception from the stable policy for sec patches but maybe elodilles knows | |
| 09:22:11 | johnthetubaguy | I remember we can't do anything other than +2, by policy, as the review was on the security bug ticket (well for the maintained branches anyways). | |
| 09:30:09 | bauzas | what's the problem with the proposed fix ? | |
| 09:30:51 | johnthetubaguy | bauzas: So zigo has issues in train I think (which I am interested in, sadly), but I am more curious if we can merge stable branches out of order for a security fix like this? | |
| 09:31:25 | zigo | bauzas: Parts of nova changed between train and ussuri, but I think I can manage. | |
| 09:31:33 | zigo | Let me finish the backporting ... :) | |
| 09:31:51 | bauzas | johnthetubaguy: I'm rushing to review the stable branches | |
| 09:32:24 | bauzas | johnthetubaguy: technically, we have a CI job that prevents merging a patch on a stable branch if the parent isn't merged | |
| 09:32:47 | johnthetubaguy | ah, I didn't know that. | |
| 09:33:03 | bauzas | fwiw, ChatGPT is unable to find the typo https://review.opendev.org/c/openstack/oslo.utils/+/706880/4/oslo_utils/imageutils.py despite me giving him clues | |
| 09:33:09 | johnthetubaguy | I see arguments both ways for sure, but I wanted to check. | |
| 09:36:46 | bauzas | so, master is merged | |
| 09:36:59 | bauzas | zed was running on the gate but we got a failure | |
| 09:37:08 | johnthetubaguy | ack | |
| 09:37:09 | bauzas | and I just +2d yoga | |
| 09:37:29 | johnthetubaguy | OK, that was my question really, is that allowed? | |
| 09:37:51 | johnthetubaguy | I guess we just wait for the +W? | |
| 09:38:06 | bauzas | to merge some stable N-x patch before the parents ? | |
| 09:38:45 | bauzas | https://docs.openstack.org/project-team-guide/stable-branches.html#appropriate-fixes | |
| 09:38:53 | bauzas | " Whether the fix is already on master and all consequent stable branches: a change must be a backport of a change already merged onto master, unless the change simply does not make sense on master. Same applies to N-2 releases, where N is master, in which case both N-1 and N branches should have the patch merged and so on." | |
| 09:39:03 | gibi | nova-tox-validate-backport job is voting in the gate queue and I that will prevent merging an older branch before the patch on newer branch lands | |
| 09:39:04 | bauzas | but, there is an exception rule | |
| 09:39:13 | bauzas | " Some patches may get exception from rule 4 above. These are patches that do not touch production code, like test-only patches, or tox.ini changes that fix major gate breakage, etc.; or security patches that should not take much time to merge once the patches are published. In those cases, stable patches may be pushed into gate without waiting for all consequent branches to be fixed." | |
| 09:39:25 | bauzas | gibi: that's what I told to johnthetubaguy | |
| 09:39:42 | johnthetubaguy | yeah, I think we should just merge them all ASAP: "or security patches that should not take much time to merge once the patches are published" | |
| 09:39:44 | gibi | bauzas: so even though the policy allow secu patches to land our tooling did not | |
| 09:39:55 | bauzas | gibi: I was about to write this | |
| 09:39:59 | johnthetubaguy | ah, got you | |
| 09:40:05 | bauzas | somehow we are strictier than the policy | |
| 09:40:45 | johnthetubaguy | so what is the quickest path to merge? in parallel drop that CI job? | |
| 09:40:49 | bauzas | johnthetubaguy: either way, you know we'll need to publish releases | |