Wednesday, 2025-07-02

*** bauzas2 is now known as bauzas00:14
*** mhen_ is now known as mhen01:12
*** bauzas7 is now known as bauzas06:20
*** bauzas7 is now known as bauzas07:34
jbernard#startmeeting cinder14:04
opendevmeetMeeting started Wed Jul  2 14:04:06 2025 UTC and is due to finish in 60 minutes.  The chair is jbernard. Information about MeetBot at http://wiki.debian.org/MeetBot.14:04
opendevmeetUseful Commands: #action #agreed #help #info #idea #link #topic #startvote.14:04
opendevmeetThe meeting name has been set to 'cinder'14:04
simondodsleyo/14:04
jbernard#topic roll call14:04
jbernardo/14:04
hvlcchao1o/14:04
rosmaitao/14:04
jbernard#link https://etherpad.opendev.org/p/cinder-flamingo-meetings14:04
sp-bmilanovo/14:04
sfernandhi14:05
Saio/14:05
jbernardwelcome everyone14:09
jbernard#annoucements14:10
jbernardall new commits need a 'signed-off-by' field in teh commit message14:10
jbernardthis is done by adding the '-s' flag to git-commit14:10
jbernard#link https://docs.openstack.org/contributors/common/dco.html14:11
jbernardrosmaita: ^ anything to add14:11
rosmaitanot really, i am not sure about the status of automated checking, so core reviewers may need to check for a while14:11
simondodsleyi just used it this morning14:12
gireeshhi14:12
jbernardrosmaita: do we know if patches will fail gate if it's not present?14:13
rosmaitanot sure, because it is only required for changes after 30 June14:13
rosmaitaso a June patch with no changes that is approved today doesn't need to have it14:14
jbernardok14:14
sp-bmilanov1I seem to have been disconnected for ~10 minutes, are we still on the "Signed-off-by" topic? I just hit something that might be of interest14:15
jbernardsp-bmilanov1: yep14:15
rosmaitawe are, go ahead14:15
sp-bmilanov1nice, so, I wanted to cherry-pick some changes to stable/2024.1 and I can't do it from the Gerrit UI because of this new requirement.. I can do it via git's CLI though14:16
jbernardfun14:16
sp-bmilanov1but it seems better to do it via Gerrit's UI since it tracks it a bit more with the "cherry pick of" row in the UI14:17
jbernardsp-bmilanov1: you could raise this issue with the infra team, maybe they're aware already14:17
sp-bmilanov1will do, thanks14:18
rosmaitabut also, if you 'git cherry-pick -x <hash>' locally, you get the cherry-picked-from line in the commit message14:18
* sp-bmilanov1 is testing it out right now14:18
sp-bmilanov1https://review.opendev.org/c/openstack/cinder/+/953958 I guess that's good enough until infra figure it out14:21
jbernard#topic followup14:23
jbernardwe have had some gate issues in recent weeks14:23
jbernardi /think/ most of those issues have been resolved14:23
jbernardrosmaita: ^ can you confirm?14:23
jbernardpep8 and barbican patches have landed14:23
hemnamep14:24
rosmaitai need to check14:24
rosmaitalooks like pep8 is back in shape through 2024.1 (Caracal): https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-pep8&project=openstack/cinder14:26
hemnarosmaita something else seems busted though ? https://zuul.opendev.org/t/openstack/build/d65450201076455dab6328c4125af6ec14:26
rosmaitaand barbican seems ok through caracal, also: https://zuul.opendev.org/t/openstack/builds?job_name=cinder-tempest-plugin-lvm-lio-barbican&project=openstack/cinder14:27
rosmaitahemna: looking14:27
rosmaitahemna: i think there was some discussion about that in the glance channel recently14:27
hemnaok.  any ideas / resolution yet?14:28
rosmaitanot sure, reading now: https://meetings.opendev.org/irclogs/%23openstack-glance/%23openstack-glance.2025-07-01.log.html#openstack-glance.2025-07-01.log.html#t2025-07-01T17:37:0714:29
hemnaok ty14:29
rosmaitahemna: sounds like it may be fixed now? go ahead and give your patch a recheck and let's see what happens14:31
hemnaok will do 14:32
jbernard#topic sdk, missing cinder/volume commands14:33
jbernardsimondodsley: ^14:33
simondodsleywhilst working on some openstack ansible modules i noticed a bunch of cinder/volume commands do not seem to be represented in the SDK. Who can look at getting these added? Mainly around qos specs, but a few other as well. I have a link to an etherpad with a list in the meeting etherpad.14:33
jbernardsimondodsley: is that list comprehensive, or just the one's you noticed?14:35
jbernardi know rajat has spent time on the sdk/client14:36
simondodsleyjust what ive noticed, although i think i t is pretty comprehensive14:36
jbernard#link https://etherpad.opendev.org/p/cinder-openstacksdk-gaps14:36
simondodsleyi'm stuck creating new cinder ansible modules without these SDK entries. I can use http methods to do it but that is very untidy14:37
jbernardyeah..14:37
jbernard#action current sdk state, what to do about missing commands14:37
jbernardsimondodsley: i dont have a quick answer, but i should have a better understanding before next meeting14:38
simondodsleyok- thanks14:38
jbernardare you volunteering? :)14:38
simondodsleynot really - i took a look at the sdk code and didn't understand it14:39
jbernardill try to have a better answer by next meeting14:40
simondodsley👌14:41
jbernardunrelated, i did reach out to macrosan about their wiki updates, haven't gotten a response yet (related to fungi's topic last meeting)14:41
jbernard#topic open discussion14:41
simondodsleywhat is the state of reviews from cores?14:42
* sp-bmilanov1 just got back from #openstack-infra, the Signed-off-by issue with backporting is known and present in the announcement in the ML -- https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/KGEKDH3AKRM3CKQW7SEVVVQFV2X6OCF6/14:43
jbernardsimondodsley: we need more14:45
simondodsleycores or reviews???14:46
jbernardyes14:46
simondodsleythere are a lot of patches out there outstanding for some time14:46
rosmaitasp-bmilanov1: ok, so you *can* use the gerrit web UI to backport, but you need to manually add the signed-off-by line to the commit message14:46
fungii haven't tested that, but does it give you a chance to use the webui editor between starting the backport and creating the change?14:47
fungii guess it pulls up a commit message editor at the time you click the backport button?14:48
rosmaitayes, there is a "cherry pick change to another branch" dialog box14:49
rosmaitait has a "cherry pick commit message"14:49
rosmaitaso you could add the signed-off-by line in there if needed14:50
jbernardok, last call14:56
sp-bmilanov1ugh, I completely missed that, thanks all!14:56
jbernardalright, thank you everyone14:58
jbernard#endmeeting14:58
opendevmeetMeeting ended Wed Jul  2 14:58:35 2025 UTC.  Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)14:58
opendevmeetMinutes:        https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.html14:58
opendevmeetMinutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.txt14:58
opendevmeetLog:            https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.log.html14:58

Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!