| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-01-24 | |||
| 16:46:30 | sean-k-mooney | but it does not have depend on so you mena we need to backprot each branch or changes in other repos | |
| 16:46:40 | dansmith | the information we need was not being exposed out of oslo utils before the patch he referenced | |
| 16:46:59 | sean-k-mooney | ok then thats kind of a problem | |
| 16:47:11 | sean-k-mooney | the olso backports would have to happen first | |
| 16:47:29 | dansmith | I don't think he's suggesting it upstream | |
| 16:47:31 | sean-k-mooney | and we would need to be able to work with older oslo | |
| 16:47:40 | opendevreview | Merged openstack/nova master: Make tenant network policy default to PROJECT_READER_OR_ADMIN https://review.opendev.org/c/openstack/nova/+/865071 | |
| 16:47:43 | dansmith | he's talking about old and crusty packages debian is keeping on life support | |
| 16:48:00 | bauzas | oh | |
| 16:48:08 | sean-k-mooney | ok we can propably talk about this out of the meeting | |
| 16:48:15 | bauzas | that's gonna be fun then | |
| 16:48:24 | dansmith | yes, especially since it's out of our support scope, very clearly | |
| 16:48:41 | sean-k-mooney | https://review.opendev.org/c/openstack/oslo.utils/+/706880 is in ussuri and above | |
| 16:48:49 | dansmith | he's talking about rocky | |
| 16:48:56 | sean-k-mooney | so i think train is the only supported release without it | |
| 16:49:13 | sean-k-mooney | dansmith: right but we have one release that does not have the oslo patch | |
| 16:49:32 | dansmith | sean-k-mooney: who is we? not upstream openstack | |
| 16:49:41 | zigo | I don't suggest it upstream, just trying to share my findings at the moment (I don't have any blockers ... yet, if I do I'll let you know). | |
| 16:49:42 | sean-k-mooney | we still supprot train for nova | |
| 16:49:52 | sean-k-mooney | and https://review.opendev.org/c/openstack/oslo.utils/+/706880 is not in it | |
| 16:50:00 | dansmith | elodilles: ? | |
| 16:50:10 | zigo | Rocky, I haven't started working on it yet... | |
| 16:50:12 | sean-k-mooney | based on the gerrit included in output | |
| 16:50:57 | zigo | sean-k-mooney: Correct. (but not really a pb for me...) | |
| 16:51:43 | sean-k-mooney | anywya lets loop back to this after the meeting | |
| 16:52:06 | sean-k-mooney | if we need it for train then we can backport it to train in oslo | |
| 16:52:13 | sean-k-mooney | but there are plenty of branches to get through first | |
| 16:52:16 | bauzas | is it me or there was a typo in the oslo.utils patch ? https://review.opendev.org/c/openstack/oslo.utils/+/706880/4/oslo_utils/imageutils.py#88 | |
| 16:52:50 | sean-k-mooney | not that i see | |
| 16:52:53 | bauzas | appened ? | |
| 16:53:28 | sean-k-mooney | its a list so i think that valid | |
| 16:53:44 | bauzas | I know I'm not a python expert, but I didn't know that a list object was having an 'appened' method | |
| 16:53:48 | sean-k-mooney | https://docs.python.org/3/tutorial/datastructures.html#more-on-lists | |
| 16:53:49 | bauzas | append, eyes | |
| 16:53:59 | sean-k-mooney | """Add an item to the end of the list. Equivalent to a[len(a):] = [x].""" | |
| 16:54:08 | bauzas | append, yes | |
| 16:54:14 | bauzas | appened, doesn't exist | |
| 16:54:22 | sean-k-mooney | oh | |
| 16:54:46 | bauzas | which tends me thinking this patch is nice but unsufficient | |
| 16:55:06 | sean-k-mooney | ya thats wrong i think but i shoudl not try and spot typos :) | |
| 16:55:19 | sean-k-mooney | that presumably was fixed | |
| 16:55:27 | bauzas | correct, probably in a fup | |
| 16:55:39 | zigo | Well, in this specific case, the code is executed in the format == 'json' path, so it *is* enough ... (but still wrong in that other case) | |
| 16:55:45 | bauzas | so it would require the fup to be dragged down too | |
| 16:56:15 | sean-k-mooney | aparenlty not https://github.com/openstack/oslo.utils/blob/stable/ussuri/oslo_utils/imageutils.py#L89 | |
| 16:56:19 | bauzas | zigo: correct, the bug is only on the string definition | |
| 16:56:25 | zigo | Well, stable/zed has the typo ... :/ | |
| 16:56:28 | bauzas | when you str() | |
| 16:56:34 | bauzas | or you print | |
| 16:56:48 | bauzas | so, technically, you don't need the fix | |
| 16:56:51 | sean-k-mooney | https://github.com/openstack/oslo.utils/commit/d49d5944824f15d00e04e1b9c7f8c3b03b440c95 | |
| 16:57:01 | sean-k-mooney | it was fixed 2 months ago | |
| 16:57:02 | bauzas | anyway, the meeting is close to the end | |
| 16:57:15 | bauzas | how to wrap up on this ? | |
| 16:57:31 | dansmith | I suggest #endmeeting | |
| 16:57:35 | sean-k-mooney | +1 | |
| 16:57:38 | elodilles | :) | |
| 16:57:49 | bauzas | then | |
| 16:57:54 | bauzas | thanks all | |
| 16:57:58 | bauzas | #endmeeting | |
| 16:57:58 | opendevmeet | Meeting ended Tue Jan 24 16:57:58 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:57:58 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-24-16.00.html | |
| 16:57:58 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-24-16.00.txt | |
| 16:57:58 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-01-24-16.00.log.html | |
| 16:58:07 | bauzas | wasn't that quick, unfortunately | |
| 17:00:31 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (db) https://review.opendev.org/c/openstack/nova/+/831193 | |
| 17:00:31 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 17:00:32 | opendevreview | ribaudr proposed openstack/nova master: Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 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: Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 17:00:34 | opendevreview | ribaudr proposed openstack/nova master: Check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 17:00:35 | opendevreview | ribaudr proposed openstack/nova master: Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 17:00:35 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 17:00:36 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 17:00:36 | opendevreview | ribaudr proposed openstack/nova master: Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 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 libvirt test to ensure metadata are working. https://review.opendev.org/c/openstack/nova/+/852086 | |
| 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: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: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:42 | opendevreview | ribaudr proposed openstack/nova master: Add instance.share_attach_error notification https://review.opendev.org/c/openstack/nova/+/860282 | |
| 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: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: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:46 | opendevreview | ribaudr proposed openstack/nova master: Add helper methods to rescue/unrescue shares https://review.opendev.org/c/openstack/nova/+/860286 | |
| 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: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:48 | opendevreview | ribaudr proposed openstack/nova master: Change microversion to 2.XX https://review.opendev.org/c/openstack/nova/+/852088 | |
| 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 | |