| *** amoralej|off is now known as amoralej | 06:39 | |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: WIP: HPE 3par: Unable to create clone of replicated vol https://review.opendev.org/c/openstack/cinder/+/884594 | 06:42 |
|---|---|---|
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM: Create for temporary test - Please ignore https://review.opendev.org/c/openstack/cinder/+/875643 | 07:11 |
| whoami-rajat | rosmaita, geguileo clarkb dansmith most of the concerns are already discussed but i think the expectation from an EM branch has always been that the project team is responsible for providing any critical fix to that branch (at least for cinder). I don't know if it's the lack of awareness about the meaning of EM or just the thought that until the branch is open, it will receive fixes for it's life before going to EOL | 09:51 |
| whoami-rajat | not to mention the gap keeps increasing between the latest EOL branch and the last stable maintained release, for example, we had rocky as EM some time ago and last stable release maintained as yoga (8 branches in EM) | 09:53 |
| opendevreview | Merged openstack/devstack-plugin-nfs master: Switch to bobcat testing runtime https://review.opendev.org/c/openstack/devstack-plugin-nfs/+/883255 | 09:54 |
| enriquetaso | cinder-plugin-ceph-tempest https://zuul.opendev.org/t/openstack/build/7309c0fd9fd64dfdaa0d301187d1cd9e : TIMED_OUT in 2h 03m 29s | 09:57 |
| * enriquetaso looking for the patch that fix this | 09:57 | |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Automate generation of attachments api-ref samples https://review.opendev.org/c/openstack/cinder/+/884420 | 10:21 |
| enriquetaso | https://review.opendev.org/c/openstack/cinder/+/884194 | 10:40 |
| enriquetaso | I think we should merge to fix the gate | 10:41 |
| eharney | i don't think it will pass until we merge https://review.opendev.org/c/openstack/cinder/+/884671 first | 10:43 |
| tosky | that's when I rebased it that I realized the other issue | 10:45 |
| *** amoralej is now known as amoralej|lunch | 12:08 | |
| ganso | rosmaita, whoami-rajat: hi! This already has a +2 and I think it would be important for you to take a look given you're most familiar with the original patch: https://review.opendev.org/c/openstack/cinder/+/882087 Thanks in advance! | 13:00 |
| rosmaita | ganso: ack | 13:00 |
| *** amoralej|lunch is now known as amoralej | 13:09 | |
| rosmaita | ganso: left a comment for you | 13:24 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder master: Fix glance metadata properties filtering https://review.opendev.org/c/openstack/cinder/+/882087 | 13:26 |
| ganso | rosmaita: done ^ :) | 13:27 |
| rosmaita | thanks! | 13:28 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
| whoami-rajat | Meeting Link: https://meet.google.com/der-fofi-iki | 14:00 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd | 14:00 |
| jungleboyj | Will join a bit late. | 14:00 |
| whoami-rajat | sure | 14:01 |
| opendevreview | Evelina Shames proposed openstack/cinder-tempest-plugin master: Test srbac on backups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/878587 | 14:04 |
| jungleboyj | whoami-rajat: What is the file you shared with me via google drive yesterday? | 14:31 |
| whoami-rajat | jungleboyj, it was the recording of last month's recording, were you able to open/download it? | 14:32 |
| jungleboyj | Oh, I wanted to make sure it wasn't spam before I clicked on it as I didn't recognize it. Will take care of that now. :-) | 14:32 |
| whoami-rajat | oh ok, yeah makes sense, it should be a google link for recording so should be fine | 14:42 |
| jungleboyj | :-) Just being careful. Seeing people hacked lately. | 14:50 |
| jungleboyj | Recording is uploaded: https://youtu.be/n4D5-GLCoUw | 14:50 |
| jungleboyj | Added to the etherpad. | 14:50 |
| whoami-rajat | great, thanks, I will share this meeting recording as well when i get it | 14:50 |
| sfv880 | Hello, could you please review when you have a minute ? https://review.opendev.org/c/openstack/cinder/+/881188 Thank you! | 14:53 |
| opendevreview | Eric Harney proposed openstack/cinder master: zuul: Extend c-p-ceph-tempest, make integrated-storage-focal nv https://review.opendev.org/c/openstack/cinder/+/884896 | 15:02 |
| whoami-rajat | just realized i forgot about the BS meeting ... i guess enriquetaso won't mind doing it on IRC this time | 15:02 |
| enriquetaso | no worries | 15:02 |
| * enriquetaso needs a coffee | 15:02 | |
| enriquetaso | #startmeeting cinder_bs | 15:03 |
| opendevmeet | Meeting started Wed May 31 15:03:16 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:03 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:03 |
| eharney | rosmaita: tosky: https://review.opendev.org/c/openstack/cinder/+/884896 | 15:03 |
| enriquetaso | Okay, full report of bugs: | 15:03 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-May/033901.html | 15:03 |
| rosmaita | eharney: ack | 15:03 |
| enriquetaso | #topic Creating a snapshot failed by schema validation when its description | 15:04 |
| enriquetaso | contains a new line control character | 15:04 |
| eharney | i analyzed this one a bit already | 15:04 |
| eharney | basically we have different validation rules for descriptions of volumes and snapshots for no apparent reason | 15:04 |
| eharney | one allows endlines and the other doesn't | 15:04 |
| enriquetaso | maybe we can talk about the CI gate failure | 15:04 |
| enriquetaso | and then continue with the bug meeting.. | 15:05 |
| eharney | ok | 15:05 |
| enriquetaso | #topic Gate failure | 15:05 |
| eharney | https://review.opendev.org/c/openstack/cinder/+/884896 should fix it and might actually make it through the gate | 15:06 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/cinder master: Add noop backup API class https://review.opendev.org/c/openstack/cinder/+/884897 | 15:06 |
| enriquetaso | so https://review.opendev.org/c/openstack/cinder/+/884896 is going to unblock https://review.opendev.org/c/openstack/cinder/+/884671 ? | 15:07 |
| eharney | yes | 15:07 |
| enriquetaso | okay | 15:09 |
| enriquetaso | we need https://review.opendev.org/c/openstack/cinder/+/884671 to unblock https://review.opendev.org/c/openstack/cinder/+/884194 | 15:10 |
| enriquetaso | so, please review ^^ | 15:10 |
| eharney | we don't need 844194 if we land the new one | 15:10 |
| enriquetaso | oh.. okay! | 15:10 |
| tosky | unless https://review.opendev.org/c/openstack/cinder/+/884671 happens to merge before | 15:12 |
| enriquetaso | :/ | 15:13 |
| * enriquetaso fingercrosses | 15:14 | |
| enriquetaso | okay, I'm going to continue with the bug meeting | 15:14 |
| enriquetaso | #topic Creating a snapshot failed by schema validation when its description contains a new line control character. | 15:14 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2021909 | 15:15 |
| eharney | we have different validation rules for descriptions of volumes and snapshots for no apparent reason | 15:15 |
| eharney | one allows endlines and the other doesn't | 15:15 |
| eharney | we should probably fix the snapshot check to allow endlines like the volume check does | 15:15 |
| enriquetaso | Okay, so it's a valid bug that needs to be fix regardless of the cinder backend | 15:16 |
| eharney | yes this is just api validation | 15:16 |
| enriquetaso | I'll update my comment on the bug report then | 15:16 |
| enriquetaso | thanks eharney | 15:16 |
| enriquetaso | @all feel free to work on the bug ^ | 15:17 |
| enriquetaso | moving on | 15:17 |
| enriquetaso | #topic Cached images duplicated per host. | 15:18 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2020733 | 15:18 |
| enriquetaso | Cinder cache using NFS backend creates duplicate cached images per host. | 15:18 |
| enriquetaso | No fix proposed to master yet. Looks like a bug on NFS backend that should be fix | 15:19 |
| eharney | i'm not sure if it's an nfs driver bug or an image cache bug | 15:19 |
| eharney | or if there's some reason this happens that means it isn't a bug | 15:19 |
| enriquetaso | "Currently if you bring up an instance with image A on host A, first launch will cause it to download from glance and cache. On subsequent builds if instance is allocated to host A then it will refer to cache." | 15:19 |
| enriquetaso | eharney, true, I'm not sure if this is a something that you can config or not | 15:21 |
| eharney | abishop worked on locking around the image cache before, he might have some thoughts on this one | 15:21 |
| enriquetaso | sounds good, maybe abishop can add some comments on the bug report if possible | 15:23 |
| enriquetaso | thanks! | 15:23 |
| enriquetaso | moving on | 15:23 |
| enriquetaso | #topic HPE 3par: Unable to create clones of replicated vol. | 15:23 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2021941 | 15:23 |
| enriquetaso | Fix proposed to master: | 15:23 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/884594 | 15:23 |
| enriquetaso | moving on | 15:24 |
| enriquetaso | #topic NetApp ONTAP | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2021746 | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2021741 | 15:25 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2021739 | 15:25 |
| enriquetaso | Some patches were merged that dont fully cover the problems they meant to solve | 15:26 |
| enriquetaso | so those bug reports were open | 15:26 |
| enriquetaso | no fix proposed to master yet for them | 15:26 |
| enriquetaso | moving on | 15:26 |
| enriquetaso | #topic Open discussion | 15:26 |
| enriquetaso | any bug to discuss ? | 15:26 |
| enriquetaso | for the las 4 minutes | 15:26 |
| hillpd | I wanted to discuss: https://bugs.launchpad.net/cinder/+bug/2017575 | 15:26 |
| enriquetaso | sure, go hillpd | 15:27 |
| hillpd | Passing the cinder.conf "host" value to os-brick makes sense going forward. Unfortunately, that would not be safe to backport due to how the host value gets used in existing deployments. | 15:27 |
| hillpd | To address this in prior releases, we are considering adding a "[cinder-oceanstor]" config option, "use_host_fqdn" to address this in the huawei driver. Is that a reasonable direction? | 15:27 |
| enriquetaso | are you looking to backport "[cinder-oceanstor]"? | 15:29 |
| enriquetaso | Remember from the cinder meeting today that: we will EOL all current EM branches | 15:30 |
| enriquetaso | <rosmaita> that means EOL Xena, Wallaby, Victoria, Ussuri, Train | 15:30 |
| enriquetaso | <rosmaita> and we will have no EM branches until Yoga goes to EM in 5 months | 15:30 |
| hillpd | yeah, we would be looking to backport that solution | 15:30 |
| enriquetaso | rosmaita, whoami-rajat, eharney any thoughts on ^? | 15:30 |
| eharney | i'd need to dig into it to understand the details better | 15:31 |
| enriquetaso | from comment https://bugs.launchpad.net/cinder/+bug/2017575/comments/4 | 15:32 |
| enriquetaso | i think it would be better if you discuss this with rajat whoami-rajat (it's really late for him now but maybe tomorrow on IRC) | 15:33 |
| hillpd | Okay, I can add this discussion to the bug and get their feedback. | 15:33 |
| enriquetaso | if that doesnt work, please bring this to the main meeting, where more people attend | 15:34 |
| enriquetaso | thank you | 15:34 |
| hillpd | ah, okay -- I tried that last week and was told to raise it in the bug squad :) | 15:34 |
| enriquetaso | https://etherpad.opendev.org/p/cinder-bobcat-meetings | 15:34 |
| enriquetaso | oh | 15:34 |
| enriquetaso | sorry about that | 15:34 |
| hillpd | no worries | 15:34 |
| enriquetaso | okay, I'll ask Rajat tomorrow | 15:35 |
| enriquetaso | here and I'll ping you | 15:35 |
| hillpd | thanks | 15:35 |
| enriquetaso | I'll added to my calendar | 15:35 |
| enriquetaso | what timezone are you in? | 15:35 |
| hillpd | GMT-6 | 15:35 |
| enriquetaso | excellent | 15:35 |
| enriquetaso | thanks! | 15:35 |
| enriquetaso | #endmeeting | 15:35 |
| opendevmeet | Meeting ended Wed May 31 15:35:56 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:35 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-31-15.03.html | 15:35 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-31-15.03.txt | 15:35 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-05-31-15.03.log.html | 15:35 |
| *** amoralej is now known as amoralej|off | 16:43 | |
| rosmaita | whoami-rajat: https://review.opendev.org/q/topic:cinder-eol-june2023 | 20:42 |
| opendevreview | Pavlo Shchelokovskyy proposed openstack/cinder master: Add noop backup API class https://review.opendev.org/c/openstack/cinder/+/884897 | 20:47 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!