*** bauzas_ is now known as bauzas | 05:13 | |
*** bauzas_ is now known as bauzas | 06:57 | |
*** bauzas_ is now known as bauzas | 09:42 | |
*** bauzas_ is now known as bauzas | 09:50 | |
*** bauzas_ is now known as bauzas | 11:21 | |
carloss | #startmeeting manila | 15:00 |
---|---|---|
opendevmeet | Meeting started Thu Jun 27 15:00:21 2024 UTC and is due to finish in 60 minutes. The chair is carloss. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
opendevmeet | The meeting name has been set to 'manila' | 15:00 |
haixin | o/ | 15:00 |
carloss | courtesy ping: dviroel vhari gouthamr carthaca msaravan pulluri ashrodri | 15:00 |
vhari | o/ | 15:00 |
carthaca | hi | 15:00 |
gireesh | hi | 15:01 |
msaravan | hi | 15:01 |
gouthamr | hello o/ | 15:01 |
jayaanand | hi | 15:02 |
ashrodri | o/ | 15:03 |
chuanm | hi | 15:03 |
ccokeke[m] | Hello | 15:04 |
carloss | o/ hello everyone! good quorum | 15:04 |
carloss | let's get started | 15:04 |
carloss | our meeting agenda for today: | 15:04 |
carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:04 |
carloss | first up | 15:05 |
carloss | #topic Announcements | 15:05 |
carloss | Schedule and Deadlines | 15:05 |
carloss | #link https://releases.openstack.org/dalmatian/schedule.html (Dalmatian release schedule) | 15:05 |
carloss | This is our spec freeze week | 15:06 |
carloss | We are currently targeting two specs for this deadline | 15:06 |
carloss | thank you everyone for providing feedback on the specs! | 15:06 |
carloss | I think we are getting close, but we still need to close some details on the two specifications | 15:07 |
carloss | Add spec lite for resource metadata backend operations | 15:08 |
carloss | #link https://review.opendev.org/c/openstack/manila-specs/+/916595 | 15:08 |
carloss | and | 15:08 |
carloss | Add spec for ensure shares | 15:08 |
carloss | #link https://review.opendev.org/c/openstack/manila-specs/+/915480 | 15:08 |
carloss | in case we can't hash out all of the details over this week, update and merge the specs | 15:09 |
carloss | we can have a one week extension | 15:09 |
carloss | that's all I had on the spec freeze deadline | 15:10 |
carloss | also, according to our first bugsquash event for this release is happening in two weeks | 15:11 |
carloss | it will happen in the week of Jul 08 to Jul 12 | 15:12 |
carloss | do you have any considerations to share on the dates? if a majority of us won't be able to spend the time in that week, we can discuss new dates close to this one | 15:12 |
carloss | I think we can follow the same format as before: using a shorter contribution period, i.e. starting on Wednesday and ending on a Friday | 15:13 |
carloss | any suggestions? thoughts? | 15:14 |
carloss | taking silence as no | 15:15 |
carloss | we will discuss it again in the next meeting | 15:16 |
carloss | and the last announcement I'd like to share: | 15:16 |
carloss | the release naming process is open! | 15:16 |
gouthamr | (next week's meeting is a holiday in the US) | 15:17 |
carloss | oh, maybe we can discuss this in openstack-manila earlier then? | 15:18 |
carloss | and send the email with at least 1 week notice | 15:18 |
gouthamr | ++ | 15:18 |
carloss | gouthamr: thanks | 15:18 |
carloss | so on the release naming process: you might have received this email: 2025.1 E OpenStack Release Naming Poll | 15:18 |
carloss | please make your vote count | 15:19 |
carloss | and help us chose what is going to be the next release name | 15:19 |
carloss | #link https://civs1.civs.us/cgi-bin/results.pl?id=E_d8469898c60545f2 (Voting poll results) | 15:19 |
carloss | current leader is Epoxy, but Ermine is very close | 15:20 |
* carloss thinks Ermine is as close as we can get from a Zorilla :p | 15:20 | |
carloss | so please participate in the voting process :) | 15:20 |
carloss | that's all I had for announcements. Do you have something you'd like to share? | 15:21 |
carloss | taking silence as no... | 15:22 |
carloss | #topic Review Focus | 15:22 |
carloss | #link https://etherpad.opendev.org/p/manila-dalmatian-review-focus (Dalmatian review focus etherpad) | 15:22 |
carloss | so as I said, this is spec freeze week, and we have some specs waiting to be merged | 15:23 |
carloss | I think both are fine in terms of people reviewing | 15:23 |
carloss | is there any concern that showed up on the reviews and you would like to bring up in this meeting? | 15:23 |
carloss | alright | 15:26 |
carloss | let's try to keep up with the reviews in the specs and aim to merge them this week | 15:27 |
carloss | if we can't, then I think merging them by the end of the next week would not be that bad | 15:27 |
carloss | is there another change you'd like to get some reviewers attention? | 15:28 |
carloss | #topic Bug Triage (vhari) | 15:29 |
carloss | #link https://etherpad.openstack.org/p/manila-bug-triage-pad-new (Bug Triage etherpad) | 15:29 |
vhari | ty carloss :) | 15:29 |
carloss | np | 15:30 |
carloss | thankfully launchpad is working today | 15:30 |
vhari | so all recent manila bugs are in progress - thanks to all contributors :) | 15:30 |
vhari | ack | 15:30 |
vhari | #link https://bugs.launchpad.net/python-manilaclient/+bug/2068631 | 15:30 |
vhari | looking for minor triage | 15:31 |
vhari | maybe a good low-hanging-fruit | 15:34 |
carloss | nice use case, thanks for the bug report carthaca | 15:35 |
carloss | this will require some changes in the API, but I think they are not major changes | 15:35 |
carloss | so +1 for low-hanging fruit | 15:35 |
carloss | in reality, I think this already works if we specify source share server id among the filters | 15:35 |
carthaca | API changes are not required for that one, this is client only ;) | 15:36 |
carloss | yeah | 15:36 |
carloss | #link https://github.com/openstack/manila/blob/master/manila/api/v1/share_servers.py#L74 | 15:36 |
carloss | this ^ should get us covered | 15:36 |
carloss | is there anyone willing to give this fix a try? | 15:37 |
carloss | likely a good candidate for bugsquash too :) | 15:37 |
vhari | carloss++ | 15:37 |
vhari | if anyone is interested pls assign the bug to yourself :) | 15:39 |
carloss | so prio can be low, as it is an RFE, we can target it to m-3 | 15:39 |
vhari | ack | 15:39 |
vhari | jumping to next bug https://bugs.launchpad.net/python-manilaclient/+bug/2068732 | 15:39 |
carloss | hmm, interesting | 15:43 |
* carloss looks at the code | 15:43 | |
carthaca | That one is about a very interesting API, because the ID of the share server you can specify in the url is an ID of an object that can already be gone xD | 15:43 |
carloss | yeah, a bit weird to think in that sense | 15:46 |
carloss | I think this is happening in the CLI because of this: https://github.com/openstack/python-manilaclient/blob/85f21732176ed6218e4b8b64ac3261dd40f54c7a/manilaclient/osc/v2/share_servers.py#L539 | 15:46 |
carloss | we are trying to look up the share server before we send the request | 15:47 |
carloss | which makes sense when the share server still exists, but doesn't when the migration was successful | 15:47 |
carloss | and the former share server was deleted | 15:47 |
carloss | > yeah, a bit weird to think in that sense | 15:49 |
carloss | I mean, the operation sounds confusing, but I think that's the expected behavior at the end | 15:49 |
carloss | xD | 15:49 |
carloss | iirc the get progress works for both source and destination | 15:50 |
carloss | in this case, I think we would need to: | 15:50 |
carloss | - try to get the share server that was specified | 15:51 |
carloss | - if not found, we send a request to get share servers and filter by the source share server ID | 15:51 |
carloss | - and then send the request to Manila using the destination share server | 15:51 |
carloss | it will add a couple more steps for the OSC to perform and the operation can end up taking some seconds, but I think it would work | 15:52 |
carloss | > - if not found, we send a request to get share servers and filter by the source share server ID | 15:52 |
carloss | if none was found, then we just say the share server wasn't found | 15:52 |
carloss | carthaca: wdyt? | 15:52 |
carloss | > iirc the get progress works for both source and destination | 15:53 |
carloss | I meant you can specify either the source or the destination share server id in the request, and it should work just fine | 15:53 |
carloss | lemme double check that | 15:53 |
carthaca | the migration get progress API works with the source share server id, no need to do additional calls, I would say | 15:53 |
carloss | yeah... I was confused because I remember we at least had the discussion of trying to make the get progress call work with both source and destination share server ids | 15:56 |
carloss | so this could and could not be a low hanging fruit. I'd say we can try dropping the share server lookup and sending the request directly to Manila | 15:57 |
carloss | the share/api is handling that just fine | 15:57 |
carloss | so I'd say this is also a low hanging fruit, and we can target this to the client release milestone | 15:58 |
carloss | carthaca: is there anyone from SAP willing to work on this? | 15:58 |
carloss | if not, we can try to find volunteers or bringing this up at the bugsquash too | 15:59 |
carloss | time check :) | 16:00 |
carthaca | sorry, I don't have volunteers right now | 16:00 |
carloss | carthaca: no worries | 16:00 |
carloss | we can look for people that can fix the bug | 16:00 |
carloss | thanks for the bug report | 16:00 |
carloss | vhari: thank you for working on the list of bugs for this week | 16:00 |
vhari | yw carloss | 16:01 |
carloss | let's get back to #openstack-manila | 16:01 |
carloss | #endmeeting | 16:01 |
opendevmeet | Meeting ended Thu Jun 27 16:01:24 2024 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2024/manila.2024-06-27-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2024/manila.2024-06-27-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2024/manila.2024-06-27-15.00.log.html | 16:01 |
*** bauzas_ is now known as bauzas | 17:13 | |
*** bauzas_ is now known as bauzas | 18:13 | |
*** bauzas_ is now known as bauzas | 20:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!