| *** mhen_ is now known as mhen | 02:46 | |
| opendevreview | mengxiangzhi proposed openstack/cinder master: Support zte vstorage driver https://review.opendev.org/c/openstack/cinder/+/969207 | 05:56 |
|---|---|---|
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Add basic support for Policy-Based Replication (PBR) https://review.opendev.org/c/openstack/cinder/+/965442 | 09:05 |
| opendevreview | Nimesh Desai proposed openstack/cinder master: [IBM SVf Driver] Define meaningful names for error constants https://review.opendev.org/c/openstack/cinder/+/959683 | 12:02 |
| rosmaita | Cinder meeting in #openstack-cinder at 1400 UTC | 14:02 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov | 14:02 |
| rosmaita | Luzi sfernand simondodsley zaubea nileshthathagar flelain wizardbit agalica lutimura kaisers | 14:02 |
| Luzi | o/ | 14:02 |
| simondodsley | o/ | 14:02 |
| harsh | o/ | 14:02 |
| rosmaita | #startmeeting cinder | 14:02 |
| opendevmeet | Meeting started Wed Jan 21 14:02:43 2026 UTC and is due to finish in 60 minutes. The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:02 |
| opendevmeet | The meeting name has been set to 'cinder' | 14:02 |
| erlon | \o | 14:02 |
| sp-bmilanov | o/ | 14:02 |
| mhen | o/ | 14:02 |
| VolodymyrBoiko[m] | o/ | 14:02 |
| RanjithRethinam | o/ | 14:03 |
| sfernand | hi | 14:04 |
| rosmaita | hello everyone | 14:04 |
| rosmaita | jbernard can't be here today, but i will do the best i can | 14:04 |
| simondodsley | I'm sure you will crush it... | 14:05 |
| rosmaita | well, i will crush something | 14:05 |
| rosmaita | ok, let's get started | 14:06 |
| rosmaita | #topic announcements | 14:06 |
| rosmaita | we had a Festival of Reviews this past Friday | 14:06 |
| rosmaita | attendance was good, so thanks to everyone who attended | 14:06 |
| rosmaita | and people who didn't, the next one will be coming up the first week of February | 14:06 |
| rosmaita | #link https://meetings.opendev.org/#Cinder_Festival_of_Reviews | 14:07 |
| rosmaita | ^^ contains a ICS file so you can get it on your calendar and never miss it | 14:08 |
| rosmaita | in other news | 14:09 |
| rosmaita | i think the gates are mostly behaving (probably should not have said that out loud) | 14:09 |
| rosmaita | there were some issues with the python 3.13 jobs being moved to debian-trixie nodes, but i think those are mostly sorted out | 14:09 |
| rosmaita | we still don't have python 3.14 jobs running, but those are optional | 14:10 |
| rosmaita | i need to redo my patch for those | 14:11 |
| rosmaita | the one thing i can share is that even when the jobs run, they fail ... we have some py 3.14 incompatible code | 14:11 |
| rosmaita | but we don't need to address that today | 14:11 |
| rosmaita | and finally, i think Jon said last week that the midcycle will be the last week of February? | 14:12 |
| rosmaita | and, i think he said first week | 14:12 |
| simondodsley | let's have 2 | 14:13 |
| rosmaita | :D | 14:13 |
| rosmaita | well, looks like February ... i will remind him to send something out to the ML | 14:14 |
| rosmaita | #action jbernard send email about midcycle to the ML | 14:14 |
| rosmaita | that's all i've got ... anyone have any announcements? | 14:14 |
| erlon | rosmaita: did you get any of the conversation regarding why they're moving the jobs from ubuntu? | 14:14 |
| simondodsley | does this mean that 3rd party CIs should move to debian as well? | 14:15 |
| kaisers | o/ (sry, late) | 14:15 |
| rosmaita | gimme a sec to figure out an answer to both questions | 14:16 |
| rosmaita | ok, so at the beginning of the cycle, the TC decides what stable distros are considered "supported" | 14:17 |
| rosmaita | #link https://governance.openstack.org/tc/reference/runtimes/2026.1.html | 14:17 |
| rosmaita | gazpacho ^^ | 14:17 |
| rosmaita | so we have Ubuntu 24.04 and Debian 13 | 14:17 |
| rosmaita | i believe the default python on 24.04 is 3.12 | 14:17 |
| rosmaita | and the default on Debian 13 is 3.13 | 14:17 |
| rosmaita | so, it's easier to run the 3.13 jobs on Debian because the python you want is "right there" without having to do anything | 14:18 |
| erlon | Ahh, right makes sense | 14:18 |
| rosmaita | yeah, so we are still running ubuntu jobs for the other python versions we are supporting in gazpacho | 14:19 |
| rosmaita | so we are still testing with both ubuntu and debian | 14:20 |
| simondodsley | if we move to debain does that fix the novnc packaging bug | 14:20 |
| rosmaita | and it's optional to test with centos stream 10 or rocky linux 10 | 14:20 |
| simondodsley | https://bugs.launchpad.net/neutron/+bug/2109592 | 14:21 |
| yuval | I want to bring up this patch which we discussed last week: https://review.opendev.org/c/openstack/os-brick/+/957599 | 14:22 |
| simondodsley | The devstack fix for this bug does not sit well in SoftwareFactory and has broken our CI (unless someone using SF has figured out a fix) | 14:22 |
| rosmaita | simondodsley: maybe? zigo is really good about keeping stuff updated, so i imagine that debian is distributing the fixed novnc package | 14:22 |
| yuval | I updated the commit | 14:22 |
| zigo | I'm for gating on Python 3.14... :) | 14:23 |
| zigo | What is it about NoVNC ? | 14:23 |
| simondodsley | See the bug above | 14:23 |
| rosmaita | zigo: i'm working on it (the py 3.14 gating): https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/968510 | 14:23 |
| zigo | rosmaita: Great, thanks! | 14:24 |
| rosmaita | ok, anything else before we move on? | 14:24 |
| zigo | FYI, I remember I fixed novnc. | 14:24 |
| zigo | novnc (1:1.6.0-2) unstable; urgency=medium | 14:25 |
| zigo | * Kill python3-novnc: novnc doesn't have any Python things for a long time | 14:25 |
| zigo | already, it's just javascript. | 14:25 |
| zigo | -- Thomas Goirand <zigo@debian.org> Wed, 30 Apr 2025 14:51:43 +0200 | 14:25 |
| zigo | Ubuntu may not have the fix though. | 14:25 |
| simondodsley | so if i switch our SF CI to use Debian we may be able to come back to life??? | 14:26 |
| zigo | Indeed, it's not in 24.04, only in Trixie. | 14:26 |
| zigo | simondodsley: At least, novnc 1:1.6.0-2 in Trixie has the issue adressed, and no python3-novnc package. | 14:26 |
| simondodsley | zigo: thanks. | 14:27 |
| simondodsley | Does anyone have a nodepool `diskimages` definiton for a trixie image that will work for devstack? | 14:27 |
| zigo | rosmaita: One more thing: I'm receiving no review on my patch: https://review.opendev.org/c/openstack/cinder/+/971710 | 14:28 |
| zigo | We're using it in production already, and we're very happy of it... | 14:28 |
| rosmaita | simondodsley: i think ask in #openstack-infra, they have debian-trixie nodes available, not sure where the definitions are, though | 14:29 |
| rosmaita | zigo: noted | 14:30 |
| rosmaita | ok, moving on ... 2 topics: encryption regression and continuation of os-brick config opts discussion | 14:30 |
| rosmaita | #topic image encryption | 14:30 |
| rosmaita | mhen: that's you | 14:30 |
| mhen | please have a look at the last two sections of https://etherpad.opendev.org/p/2026-01-image-encryption-client-regression | 14:31 |
| mhen | you can skip the lengthy preface at the top for now, its just context | 14:31 |
| mhen | it's quite minor and easy to fix I think just want to know which way of going about it would be preferred when making the client patch | 14:33 |
| RanjithRethinam | Can someone pls review/approve this PR, https://review.opendev.org/c/openstack/cinder/+/971739 all review comments are addressed and CI is passing | 14:35 |
| rosmaita | thanks, mhen. i was just looking at the cinder API for this: https://docs.openstack.org/api-ref/block-storage/v3/#upload-volume-to-image | 14:35 |
| rosmaita | just wanted to verify that this is an osc issue, not something we (cinder) caused | 14:36 |
| mhen | container_format is optional as per API but the client *always* sends it with an opinionated default although unnecessary | 14:36 |
| rosmaita | the container_format is optional, so i would vote for osc not sending anything for container_format if the user has not specified one | 14:36 |
| rosmaita | it's a fine line between being opinionated and stupid! | 14:37 |
| rosmaita | while you're fixing that, maybe you could make sure they're not sending opinionated content for other unspecified options? | 14:38 |
| rosmaita | though i guess that should be a separate patch | 14:38 |
| mhen | yes, for this at least we have a reason (the change introduced by the image encryption) | 14:39 |
| mhen | dunno if the osc folks would be happy with removing other stuff | 14:39 |
| rosmaita | i was thinking of the 'force' flag, they send it all the time for some requests (forget which one) and it causes issues with microversion 3.66 | 14:40 |
| rosmaita | anyway, thank you for presenting both the issue and the solution! | 14:41 |
| rosmaita | anything else on this? | 14:41 |
| mhen | no | 14:41 |
| mhen | thanks | 14:41 |
| mhen | I will try to go that route as discussed | 14:41 |
| RanjithRethinam | https://review.opendev.org/c/openstack/cinder/+/971739 can this get some attention pls | 14:41 |
| rosmaita | great! moving on | 14:41 |
| rosmaita | #topic continue discussion of os-brick patch from last week | 14:41 |
| rosmaita | yuval: that's you | 14:42 |
| yuval | I am here, anything left not answered? | 14:42 |
| rosmaita | #link https://review.opendev.org/c/openstack/os-brick/+/957599 | 14:42 |
| yuval | thanks | 14:42 |
| rosmaita | i will have a minor comment about the release note | 14:43 |
| rosmaita | might as well say it here as a reminder to everyone | 14:43 |
| rosmaita | don't forget that the release notes are handled by automation that will re-arrange them | 14:43 |
| rosmaita | so even though you have a single note, the sections will be far apart (most likely) | 14:44 |
| rosmaita | so in yuval's case, in the "upgrade" section, he refers to the options but doesn't say which ones | 14:44 |
| rosmaita | because they're right there in the previous bullet point | 14:44 |
| rosmaita | but when the release notes are all rendered for the release, they will be far apart | 14:44 |
| rosmaita | so sometimes you have to be a bit repetitive in your release notes | 14:45 |
| yuval | I get you | 14:45 |
| yuval | ok will handle | 14:45 |
| yuval | thank you | 14:45 |
| rosmaita | sounds good, i notice that erlon gave you a +1 and he's careful, so i imagine that your commit message update is good | 14:46 |
| rosmaita | (i will read after the meeting) | 14:46 |
| rosmaita | ok, so mainly this was a reminder that the patch was updated as requested, and the author is available here if there are questions | 14:46 |
| rosmaita | thanks! | 14:46 |
| rosmaita | just a reminder to everyone, our release note docs (how to write) are here: https://docs.openstack.org/cinder/latest/contributor/releasenotes.html | 14:48 |
| rosmaita | and if you want to see the actual release notes, https://docs.openstack.org/releasenotes/cinder/index.html | 14:48 |
| rosmaita | and, finally, you can always check to see what your note looks like in html by either running tox locally, or looking at the results of the build-openstack-releasenotes job on your patch | 14:49 |
| rosmaita | so, not to pick on yuval, but you can see his current note here: https://fe6d6d8568001e0da1b7-c29773bdeee4530a738751d9e026e2a7.ssl.cf2.rackcdn.com/openstack/ddcd6ef273e0474d9f8c5c89c95514d1/docs/unreleased.html | 14:50 |
| rosmaita | ok, that's all the topics on the agenda | 14:51 |
| rosmaita | i think RanjithRethinam has frantically been trying to get peoples' attention | 14:51 |
| kaisers | I'll fall in line: Would be great to have some fb on a small revert: https://review.opendev.org/c/openstack/cinder/+/967672 | 14:53 |
| seunghunlee | Hello, can I ask some answers for the latests comments on #link https://review.opendev.org/c/openstack/cinder/+/820375 | 14:53 |
| rosmaita | kaisers: you may need to rebase to make sure you pick up the change that will let py313 job pass | 14:54 |
| opendevreview | Takashi Kajinami proposed openstack/cinder-specs master: Replace obsolete PCRE packages https://review.opendev.org/c/openstack/cinder-specs/+/974071 | 14:55 |
| kaisers | rosmaita will do, thnx for the pointer | 14:55 |
| sanketh_shetty | Requesting core reviewers to look into - https://review.opendev.org/c/openstack/cinder/+/973908. It a very small regression fix. | 14:55 |
| rosmaita | kaisers: it's the change to bindep.txt from https://review.opendev.org/c/openstack/cinder/+/970038 that you need | 14:56 |
| kaisers | ack | 14:56 |
| rosmaita | eharney: when you have some time, seunghunlee has a question about your question on https://review.opendev.org/c/openstack/cinder/+/820375 | 14:58 |
| eharney | yeah i need to followup there - IMO we probably don't want that patch because i think you can achieve the desired behavior by just setting up ceph correctly | 14:59 |
| rosmaita | seunghunlee: if you do a 'git blame' on the rbd driver, look at the section where the client version is checked ... the commit message may have some info for you | 14:59 |
| rosmaita | (at least that's my memory) | 15:00 |
| rosmaita | sanketh_shetty: still looking at your patch, but could you please wrap the text in the release note? right now line 4 is a single line, please break it up into a series of shorter lines | 15:02 |
| rosmaita | i think you can do our standard 79-char line length | 15:03 |
| rosmaita | ok, we have gone over time ... thanks everyone for attending! | 15:03 |
| rosmaita | #endmeeting | 15:04 |
| opendevmeet | Meeting ended Wed Jan 21 15:04:05 2026 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-21-14.02.html | 15:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-21-14.02.txt | 15:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2026/cinder.2026-01-21-14.02.log.html | 15:04 |
| seunghunlee | rosmita: Got it thank you. eharney: but does that ceph setup allows creating flatten volume from image? | 15:06 |
| eharney | seunghunlee: if you enable image format v2 by setting minimum client compat to mimic, you do not have to flatten the image to remove the glance image that a vol was cloned from | 15:06 |
| eharney | i need to add a note to the cinder docs about this probably | 15:06 |
| seunghunlee | eharney: hmm okay. Could you add a note as a comment on the patch as well? For the record. | 15:10 |
| seunghunlee | Thanks for your time by the way | 15:10 |
| mhen | can somebody take a look at https://bugs.launchpad.net/cinder/+bug/2138806 and tell me if this is a user/validation error (slashes actually not allowed) or a genuine bug in Cinder or swiftclient? | 15:17 |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM SVf Driver] Add basic support for Policy-Based Replication (PBR) https://review.opendev.org/c/openstack/cinder/+/965442 | 15:30 |
| erlon | eharney: hey, can you give another check in the 512e/4k disk geometry patch? https://review.opendev.org/c/openstack/cinder/+/658283 | 15:39 |
| opendevreview | Merged openstack/cinder master: [tests-only] update for oslo.concurrency 7.3.0 https://review.opendev.org/c/openstack/cinder/+/973693 | 17:04 |
| opendevreview | Dmitriy Rabotyagov proposed openstack/cinder master: Allow operators to control Ceph pool selection for backups https://review.opendev.org/c/openstack/cinder/+/958671 | 19:38 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: API: Allow newlines in snapshot descriptions https://review.opendev.org/c/openstack/cinder/+/926914 | 21:48 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: Test that snapshot descriptions can contain newlines https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/926913 | 21:56 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!