*** mhen_ is now known as mhen | 01:24 | |
carloss | #startmeeting manila | 15:00 |
---|---|---|
opendevmeet | Meeting started Thu May 29 15:00:07 2025 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 |
gouthamr | o/ | 15:00 |
carloss | courtesy ping: dviroel vhari carthaca msaravan Sai ashrodri gireesh | 15:00 |
Sai | o/ | 15:00 |
haixin | :) | 15:02 |
carloss | o/ hello zorillas | 15:03 |
carloss | our meeting agenda for today: | 15:03 |
carloss | #link https://wiki.openstack.org/wiki/Manila/Meetings#Next_meeting | 15:04 |
carloss | so this is still the bugsquash week | 15:04 |
carloss | we'll get to that in a bit | 15:04 |
carloss | but first | 15:04 |
carloss | #topic Announcements | 15:04 |
carloss | Schedule and Deadlines | 15:05 |
carloss | #link https://releases.openstack.org/flamingo/schedule.html (Flamingo Schedule) | 15:05 |
carloss | we're now two weeks away from our hackathon, which in the initial idea of the PTG targeted functional tests for the python-manilaclient repo | 15:05 |
carloss | targeting OSC, I mean | 15:06 |
carloss | we wanted to do that before we remove the manilaclient | 15:06 |
kpdev | hi | 15:06 |
* carloss o/ | 15:06 | |
carloss | do you have any concerns with regards to the hackathon date? as in: are you okay with it being in two weeks from now? | 15:06 |
gouthamr | sounds good; however we may need a bunch of pre-work to determine the gaps | 15:08 |
carloss | gouthamr: yep, I can work on that | 15:09 |
gouthamr | wait, you said two weeks from now | 15:09 |
gouthamr | but, on the schedule it is R-13 | 15:09 |
gouthamr | Jun 30 - Jul 04 | 15:09 |
carloss | argh, yes | 15:09 |
carloss | something wrong with my eyes today | 15:10 |
carloss | gouthamr: thanks | 15:10 |
carloss | next deadline is spec freeze, 3 weeks from now | 15:10 |
carloss | my bad | 15:11 |
carloss | that's all I had for $topic | 15:12 |
carloss | do you have any other announcements you'd like to share with the zorillas today? | 15:12 |
gouthamr | not an announcement, per se.. but a word on availability - mine is going to be iffy over the next couple of months.. i'd probably engage more async than usual | 15:13 |
gouthamr | hoping to still keep up with reviews, and will need some notice to have sync conversations | 15:13 |
carloss | gouthamr: ack, thanks for the heads-up | 15:16 |
carloss | and also for offering time in case we need :) | 15:17 |
carloss | let's switch to Bug triaging, and then we can talk bugsquash | 15:17 |
carloss | #topic Bug Triage | 15:18 |
carloss | #link https://etherpad.opendev.org/p/manila-bug-triage-pad-new (Bug triage etherpad) | 15:18 |
carloss | a bunch of new bugs | 15:18 |
carloss | #link https://bugs.launchpad.net/manila/+bug/2111813 ([NetApp] enable aes-encryption for cifs) | 15:19 |
carloss | we seem to have a fix proposed: | 15:19 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/951087 ([NetApp] Enable aes-encryption for cifs) | 15:19 |
kpdev | yes, assigned to self | 15:19 |
carloss | kpdev: thank you | 15:19 |
carloss | kpdev: is it breaking any workflows? | 15:20 |
kpdev | its was fix of some error during cifs share creation | 15:20 |
kpdev | mentioned in commit | 15:20 |
carloss | yep | 15:20 |
carloss | I think we can make it medium importance | 15:21 |
gouthamr | is this really a bug? | 15:21 |
gouthamr | sounds like a missing feature.. what's failing without this? | 15:21 |
kpdev | kind of bug or small feature. was getting error during share creation KRB5KDC_ERR_ETYPE_NOSUPP.. mentioned in commit | 15:22 |
kpdev | but PR is small, I created bug and fixed via it. | 15:22 |
gouthamr | sounds fine, some bugs are really RFEs | 15:22 |
gouthamr | but, did something change in ONTAP to require it? | 15:22 |
kpdev | also.. manila-grenade-skip-level-always still failing for recheck of all patches I retried since yesterday. | 15:22 |
carloss | > also.. manila-grenade-skip-level-always still failing for recheck of all patches I retried since yesterday. | 15:23 |
carloss | yep, that's one of the topics I wanted to mention, as I also noticed it when I was looking at changes from the bugsquash and rebasing them | 15:23 |
gouthamr | https://zuul.opendev.org/t/openstack/build/f9beac99a25043229c93ad2605d85e9d/log/job-output.txt#15713-15736 | 15:24 |
gouthamr | ^ i haven't checked the chatter to see if neutron folks know about this | 15:24 |
gouthamr | https://zuul.opendev.org/t/openstack/build/f9beac99a25043229c93ad2605d85e9d/log/controller/logs/screen-neutron-api.txt looks fine to me | 15:28 |
carloss | yes | 15:29 |
kpdev | >but, did something change in ONTAP to require it? ... No, nothing changed. it was downstreamed since last 2-3 years, upstreaming it now. | 15:29 |
gouthamr | kpdev: sounds like some ONTAP configuration in your cloud requires this | 15:30 |
carloss | ++ - maybe we should tag as an RFE | 15:31 |
gouthamr | Sai/gireesh: please take a look.. i'd be concerned if this is not available/expected to be turned on my default.. because i've not seen an issue creating cifs shares/access rules without AES encryption enabled.. | 15:31 |
kpdev | I dont see any special config did for this feature. but yes netapp internal testing and review would be better to confirm. | 15:32 |
carloss | ++ | 15:32 |
gouthamr | https://kb.netapp.com/on-prem/ontap/da/NAS/NAS-KBs/Can_we_enable_AES_encryption_on_CIFS_server | 15:32 |
Sai | I will dissolve with Gireesh on this. Will update him. Thanks. | 15:32 |
Sai | Discuss* | 15:32 |
carloss | Sai: thanks | 15:33 |
gouthamr | also this: https://docs.netapp.com/us-en/ontap/smb-admin/enable-disable-aes-encryption-kerberos-task.html | 15:33 |
gouthamr | "Whether AES encryption is enabled by default and whether you have the option to specify encryption types depends on your ONTAP version." - maybe the driver will need to do this based on the ONTAP versions | 15:34 |
gouthamr | also, the option is changing with newer versions | 15:35 |
gouthamr | with newer versions of ONTAP, you can specify the encryption type | 15:35 |
kpdev | hmm. so this will be 9.11.1 and earlier | 15:35 |
kpdev | I will update fix to consider versions | 15:36 |
carloss | kpdev++ | 15:37 |
carloss | thanks | 15:38 |
carloss | I think we have time for another untriaged bug | 15:38 |
carloss | #link https://bugs.launchpad.net/manila/+bug/2111905 (NetApp - Need to provide the option to override the “is-shared” field value for qos policy created during share creation) | 15:39 |
carloss | sounds like an RFE | 15:40 |
gouthamr | yeah | 15:40 |
carloss | i see gireesh already added himself to assignee | 15:40 |
carloss | likely low prio | 15:40 |
gouthamr | +1 | 15:41 |
carloss | Sai: are you aware if gireesh is planning to work on this? | 15:41 |
gireesh | yes, these are kind of RFE, one of the customer is requesting for these. We will work on these | 15:41 |
carloss | gireesh: ack, thank you! | 15:42 |
carloss | we have another untriaged bug that seems a bit high prio | 15:43 |
* carloss is hijacking all the time of bugsquash to bug triaging :/ | 15:43 | |
carloss | this is the last one I'll bring up today, I promise | 15:43 |
carloss | #link #link https://bugs.launchpad.net/manila/+bug/2111918 ([Manila][NetApp][REST]Share deletion is failing at NetApp volume creation saying: Field "retention_period" cannot be set in this operation") | 15:43 |
carloss | Sai thank for reporting this bug! | 15:44 |
carloss | I've set kpdev as an assignee mostly for an investigation reason, as I think he might have the most context on this change | 15:44 |
Sai | Np, Thank you carloss !! | 15:44 |
carloss | apparently, share deletion is failing in REST | 15:44 |
carloss | kpdev: could you please take a look? | 15:44 |
carloss | this might be a regression | 15:45 |
carloss | if it is, we'll need to have an owner to this fix | 15:45 |
kpdev | No, I had added fix there based on documentation. but this time we need correct soluton from netapp | 15:45 |
kpdev | since the field I mentioned no longer present in REST documentation. | 15:45 |
kpdev | so need to find another way to provide retention_period. NetApp needs to update on bug how to do it and then I will raise PR. | 15:46 |
carloss | so might be something varying from one ontap version to another? | 15:46 |
carloss | so, this is one of those issues the NetApp CI would easily catch :/ | 15:46 |
Sai | kpdev: Meanwhile, can you please update the documentation you referred earlier in the bug. Thanks | 15:47 |
kpdev | not sure about version, we are not using REST yet. | 15:47 |
gireesh | retention_period need to set in patch call not in post call. Once vserver is created after that we have to set this parameter - kpdev | 15:47 |
kpdev | The documentation redirect to new pages now, I had shared with you earlier | 15:47 |
kpdev | @gireesh: got it, can anyone from netapp update that on bug ? | 15:48 |
gireesh | ok, I'll do this | 15:48 |
carloss | ack, thank you | 15:48 |
carloss | please update the bug as well | 15:48 |
carloss | alright, let's move to the next topic | 15:49 |
carloss | #topic Bugsquash - Review-a-thon | 15:50 |
carloss | https://etherpad.opendev.org/p/manila-flamingo-m1-bugsquash (Flamingo m-1 bugsquash) | 15:50 |
carloss | #link https://etherpad.opendev.org/p/manila-flamingo-m1-bugsquash (Flamingo m-1 bugsquash) | 15:50 |
carloss | so we had the kick-off call on Tuesday and I am posting the video to the YouTube channel | 15:50 |
carloss | we've got review assignments, so I'd like to ask reviewers to take a look at the changes | 15:51 |
carloss | we have the CI situation at the moment, but there are some easy wins for us that are not relying on the CI being working rn | 15:51 |
carloss | I'm talking about documentation bugs | 15:52 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/945891 (Add mount snapshot support to user guide) | 15:53 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/934462 (Updates the share extend and shrink documentation) | 15:53 |
carloss | #link https://review.opendev.org/c/openstack/manila/+/946505 (use os cli for shared-file-crud-share.rst) | 15:53 |
carloss | two of these already have +2s/+1 from cores | 15:54 |
carloss | can I get more eyes on them? | 15:54 |
gouthamr | yep | 15:54 |
haixin | i will check this | 15:54 |
carloss | gouthamr haixin thank you very much! | 15:54 |
carloss | now for the other changes, which are being impacted by the CI situation | 15:55 |
carloss | I've looked at the job history and apparently it is failing for a while | 15:55 |
carloss | I tried correlating to the changes that were merged to openstack/neutron on a given date but these don't seem to be a problem | 15:56 |
carloss | and as gouthamr also mentioned, neutron API seems to be working just fine | 15:56 |
carloss | I posted some review comments to a bunch of changes, some are being resolved, but CI won't let these changes merges even if we have agreement from reviewers | 15:57 |
carloss | I'll investigate the failures | 15:57 |
carloss | in the meantime | 15:57 |
carloss | not a very good timing for the CI being broken | 15:59 |
carloss | but I think we are getting reviews in some changes | 15:59 |
carloss | and when CI issues are fixed, we can go ahead and merge the changes that are blocked by it | 15:59 |
carloss | questions? suggestions? | 16:00 |
carloss | alright, let's wrap up | 16:01 |
carloss | and get back to #openstack-manila | 16:01 |
carloss | thanks for participating | 16:01 |
carloss | and have a great day | 16:01 |
carloss | #endmeeting | 16:01 |
opendevmeet | Meeting ended Thu May 29 16:01:29 2025 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:01 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/manila/2025/manila.2025-05-29-15.00.html | 16:01 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/manila/2025/manila.2025-05-29-15.00.txt | 16:01 |
opendevmeet | Log: https://meetings.opendev.org/meetings/manila/2025/manila.2025-05-29-15.00.log.html | 16:01 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!