*** bauzas2 is now known as bauzas | 00:14 | |
*** mhen_ is now known as mhen | 01:12 | |
*** bauzas7 is now known as bauzas | 06:20 | |
*** bauzas7 is now known as bauzas | 07:34 | |
jbernard | #startmeeting cinder | 14:04 |
---|---|---|
opendevmeet | Meeting 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 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:04 |
opendevmeet | The meeting name has been set to 'cinder' | 14:04 |
simondodsley | o/ | 14:04 |
jbernard | #topic roll call | 14:04 |
jbernard | o/ | 14:04 |
hvlcchao1 | o/ | 14:04 |
rosmaita | o/ | 14:04 |
jbernard | #link https://etherpad.opendev.org/p/cinder-flamingo-meetings | 14:04 |
sp-bmilanov | o/ | 14:04 |
sfernand | hi | 14:05 |
Sai | o/ | 14:05 |
jbernard | welcome everyone | 14:09 |
jbernard | #annoucements | 14:10 |
jbernard | all new commits need a 'signed-off-by' field in teh commit message | 14:10 |
jbernard | this is done by adding the '-s' flag to git-commit | 14:10 |
jbernard | #link https://docs.openstack.org/contributors/common/dco.html | 14:11 |
jbernard | rosmaita: ^ anything to add | 14:11 |
rosmaita | not really, i am not sure about the status of automated checking, so core reviewers may need to check for a while | 14:11 |
simondodsley | i just used it this morning | 14:12 |
gireesh | hi | 14:12 |
jbernard | rosmaita: do we know if patches will fail gate if it's not present? | 14:13 |
rosmaita | not sure, because it is only required for changes after 30 June | 14:13 |
rosmaita | so a June patch with no changes that is approved today doesn't need to have it | 14:14 |
jbernard | ok | 14:14 |
sp-bmilanov1 | I 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 interest | 14:15 |
jbernard | sp-bmilanov1: yep | 14:15 |
rosmaita | we are, go ahead | 14:15 |
sp-bmilanov1 | nice, 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 though | 14:16 |
jbernard | fun | 14:16 |
sp-bmilanov1 | but 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 UI | 14:17 |
jbernard | sp-bmilanov1: you could raise this issue with the infra team, maybe they're aware already | 14:17 |
sp-bmilanov1 | will do, thanks | 14:18 |
rosmaita | but also, if you 'git cherry-pick -x <hash>' locally, you get the cherry-picked-from line in the commit message | 14:18 |
* sp-bmilanov1 is testing it out right now | 14:18 | |
sp-bmilanov1 | https://review.opendev.org/c/openstack/cinder/+/953958 I guess that's good enough until infra figure it out | 14:21 |
jbernard | #topic followup | 14:23 |
jbernard | we have had some gate issues in recent weeks | 14:23 |
jbernard | i /think/ most of those issues have been resolved | 14:23 |
jbernard | rosmaita: ^ can you confirm? | 14:23 |
jbernard | pep8 and barbican patches have landed | 14:23 |
hemna | mep | 14:24 |
rosmaita | i need to check | 14:24 |
rosmaita | looks like pep8 is back in shape through 2024.1 (Caracal): https://zuul.opendev.org/t/openstack/builds?job_name=openstack-tox-pep8&project=openstack/cinder | 14:26 |
hemna | rosmaita something else seems busted though ? https://zuul.opendev.org/t/openstack/build/d65450201076455dab6328c4125af6ec | 14:26 |
rosmaita | and barbican seems ok through caracal, also: https://zuul.opendev.org/t/openstack/builds?job_name=cinder-tempest-plugin-lvm-lio-barbican&project=openstack/cinder | 14:27 |
rosmaita | hemna: looking | 14:27 |
rosmaita | hemna: i think there was some discussion about that in the glance channel recently | 14:27 |
hemna | ok. any ideas / resolution yet? | 14:28 |
rosmaita | not 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:07 | 14:29 |
hemna | ok ty | 14:29 |
rosmaita | hemna: sounds like it may be fixed now? go ahead and give your patch a recheck and let's see what happens | 14:31 |
hemna | ok will do | 14:32 |
jbernard | #topic sdk, missing cinder/volume commands | 14:33 |
jbernard | simondodsley: ^ | 14:33 |
simondodsley | whilst 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 |
jbernard | simondodsley: is that list comprehensive, or just the one's you noticed? | 14:35 |
jbernard | i know rajat has spent time on the sdk/client | 14:36 |
simondodsley | just what ive noticed, although i think i t is pretty comprehensive | 14:36 |
jbernard | #link https://etherpad.opendev.org/p/cinder-openstacksdk-gaps | 14:36 |
simondodsley | i'm stuck creating new cinder ansible modules without these SDK entries. I can use http methods to do it but that is very untidy | 14:37 |
jbernard | yeah.. | 14:37 |
jbernard | #action current sdk state, what to do about missing commands | 14:37 |
jbernard | simondodsley: i dont have a quick answer, but i should have a better understanding before next meeting | 14:38 |
simondodsley | ok- thanks | 14:38 |
jbernard | are you volunteering? :) | 14:38 |
simondodsley | not really - i took a look at the sdk code and didn't understand it | 14:39 |
jbernard | ill try to have a better answer by next meeting | 14:40 |
simondodsley | 👌 | 14:41 |
jbernard | unrelated, 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 discussion | 14:41 |
simondodsley | what 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 | |
jbernard | simondodsley: we need more | 14:45 |
simondodsley | cores or reviews??? | 14:46 |
jbernard | yes | 14:46 |
simondodsley | there are a lot of patches out there outstanding for some time | 14:46 |
rosmaita | sp-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 message | 14:46 |
fungi | i 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 |
fungi | i guess it pulls up a commit message editor at the time you click the backport button? | 14:48 |
rosmaita | yes, there is a "cherry pick change to another branch" dialog box | 14:49 |
rosmaita | it has a "cherry pick commit message" | 14:49 |
rosmaita | so you could add the signed-off-by line in there if needed | 14:50 |
jbernard | ok, last call | 14:56 |
sp-bmilanov1 | ugh, I completely missed that, thanks all! | 14:56 |
jbernard | alright, thank you everyone | 14:58 |
jbernard | #endmeeting | 14:58 |
opendevmeet | Meeting ended Wed Jul 2 14:58:35 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:58 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.html | 14:58 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.txt | 14:58 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder/2025/cinder.2025-07-02-14.04.log.html | 14:58 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!