| *** dviroel|out is now known as dviroel | 00:33 | |
| *** dviroel is now known as dviroel|out | 01:00 | |
| aneeeshp11 | Anyone using Software Factory for cinder/os-brick third party CI? I did install Software Factory and it is posting results to ci-sandbox project. Now planning to add the jobs for cinder. Could any one share playbooks or job yaml files that can be used for the cinder project? | 03:27 |
|---|---|---|
| opendevreview | liuhuajie proposed openstack/cinder master: Remove None for get method, the default value is None https://review.opendev.org/c/openstack/cinder/+/850449 | 04:59 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Delete volume and cloned-volume concurrently https://review.opendev.org/c/openstack/cinder/+/843309 | 05:08 |
| *** geguileo is now known as Guest5518 | 05:57 | |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add cinder-manage command to update service_uuid https://review.opendev.org/c/openstack/cinder/+/823089 | 07:40 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Storage node configuration https://review.opendev.org/c/openstack/cinder/+/822822 | 08:54 |
| opendevreview | Merged openstack/cinderlib stable/yoga: Don't use yum to install ceph https://review.opendev.org/c/openstack/cinderlib/+/850393 | 09:14 |
| opendevreview | Merged openstack/cinderlib stable/yoga: Update deprecated zuul syntax https://review.opendev.org/c/openstack/cinderlib/+/850394 | 09:14 |
| opendevreview | Merged openstack/cinderlib stable/xena: Don't use yum to install ceph https://review.opendev.org/c/openstack/cinderlib/+/850395 | 09:17 |
| opendevreview | Merged openstack/cinderlib stable/xena: Update deprecated zuul syntax https://review.opendev.org/c/openstack/cinderlib/+/850396 | 09:17 |
| opendevreview | Merged openstack/cinderlib stable/wallaby: Don't use yum to install ceph https://review.opendev.org/c/openstack/cinderlib/+/850399 | 09:27 |
| opendevreview | Merged openstack/cinderlib stable/wallaby: Update deprecated zuul syntax https://review.opendev.org/c/openstack/cinderlib/+/850400 | 09:27 |
| tosky | whoami-rajat: the victoria patches for cinderlib are ready to be merged too (and after they merge, I can fix the ussuri and train ones) | 09:47 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Storage node configuration https://review.opendev.org/c/openstack/cinder/+/822822 | 09:48 |
| whoami-rajat | tosky, just wanted to get some clarity on the fixes, rosmaita wrote that one patch is fixing the gate and another is removing deprecated zuul syntax, just wanted to know till which branch they have an impact? | 09:51 |
| tosky | the one which fixes the deprecated zuul syntax is needed everywhere | 09:53 |
| tosky | the other is needed until the relevant job is fixed | 09:54 |
| tosky | which is probably everywhere the ceph job uses ubuntu, because it was logically wrong to use the ansible yum task there (a mistake I did some time ago) | 09:54 |
| whoami-rajat | ack | 10:22 |
| *** dviroel|out is now known as dviroel | 11:34 | |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 11:48 |
| rosmaita | whoami-rajat: the yum->package fix needs to go back to ussuri (train is using centos-7, where using yum should be no problem) | 12:03 |
| rosmaita | the zuul syntax needs to go as far back as we can get it | 12:03 |
| whoami-rajat | ack, currently one of the victoria patch is waiting for another +2 and ussuri gate is failing | 12:03 |
| rosmaita | whoami-rajat: it's weird, victoria is passing without having https://review.opendev.org/q/I0278b42785d1 backported to it, but the ussuri failure looks like it needs that patch | 12:20 |
| whoami-rajat | TBH, cinderlib gate has been always weird for me | 12:22 |
| rosmaita | yeah, the trailing release model doesn't help | 12:24 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin | 13:59 |
| enriquetaso | thanks | 13:59 |
| hemna | doink | 13:59 |
| whoami-rajat | np | 14:00 |
| *** geguileor is now known as geguileo | 14:00 | |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver to return all iSCSI portals https://review.opendev.org/c/openstack/cinder/+/849022 | 14:02 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: WIP: test multiattach volume works after disconnect https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/850526 | 14:12 |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 14:31 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Check for CRLF terminators in pep8 env https://review.opendev.org/c/openstack/cinder/+/848759 | 14:44 |
| simondodsley | eharney: this change will potentially all the current and new NVMe drivers so it needs to be resolved before driver merge freeze | 15:00 |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Jul 20 15:01:49 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:01 |
| enriquetaso | Welcome back to the Cinder Bug Meeting | 15:02 |
| enriquetaso | Cinder has 6 new bugs reported. Mostly low bugs: | 15:02 |
| enriquetaso | #topic Infinidat | 15:02 |
| enriquetaso | Cinder has two bugs regarding Infinidat and one of them it's kind of a big bug: | 15:02 |
| enriquetaso | "Infinidat Cinder driver multi-attach feature is broken" | 15:02 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1982350 | 15:02 |
| enriquetaso | I think many drivers have made the same mistake in their multiattach implementations. This is a sign that we need a tempest test that actually tests multiattach | 15:03 |
| rosmaita | enriquetaso: ++ | 15:03 |
| enriquetaso | eharney ^ | 15:03 |
| eharney | i tossed up https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/850526 but it probably doesn't work yet | 15:03 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/850526 | 15:03 |
| enriquetaso | thanks eric! | 15:03 |
| enriquetaso | Please keep an eye on this test so we can merge it as soon as possible. | 15:03 |
| enriquetaso | maybe infinidat team would like to add some comments to this ? :) | 15:04 |
| enriquetaso | The second one is | 15:05 |
| enriquetaso | "Infinidat Cinder driver ignores driver_use_ssl option" | 15:05 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981982 | 15:05 |
| enriquetaso | It has a fix proposed to master, feel free to review it! | 15:05 |
| rosmaita | ok | 15:06 |
| eharney | infinidat also has https://review.opendev.org/c/openstack/cinder/+/849022 up which needs review, maybe someone looking at these reviews should hit all of them together | 15:06 |
| enriquetaso | sounds good | 15:08 |
| enriquetaso | Next, we have 3 generic-NFS and remotefs related bugs: | 15:08 |
| enriquetaso | #topic NFS Backup driver doesn't remove empty directories on backup deletion. | 15:09 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1982032 | 15:09 |
| enriquetaso | whoami-rajat: Using cinder backup with NFS backend, everything works fine, except that when we delete a backup the actual "content" of the backup gets deleted but not the directory structure on the NFS share | 15:09 |
| enriquetaso | Fix proposed to master: | 15:09 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/804603 | 15:09 |
| eharney | i called out some issues w/ race conditions on the patch, need to go look at rajat's updates and see if i can find any further concerns | 15:09 |
| whoami-rajat | yeah, I've revised with some of eharney's suggestions | 15:09 |
| whoami-rajat | thanks eharney | 15:10 |
| enriquetaso | thanks | 15:12 |
| enriquetaso | #topic [NFS] Nova raises an error on the server to resize command | 15:12 |
| enriquetaso | https://bugs.launchpad.net/cinder/+bug/1981562 | 15:12 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981562 | 15:12 |
| enriquetaso | I have not been able to look into it more, so I am not sure if there is anything to do from the cinder side. I have a devstack+nfs so I'll try this out today. | 15:12 |
| eharney | yeah, we need to determine if this is doing something that is not intended to be supported | 15:14 |
| eharney | a closer look would be good | 15:14 |
| enriquetaso | #action (enriquetaso) take a closer look and reproduce bug1981562 | 15:15 |
| enriquetaso | The last one is incomplete and waiting from the reporter side but I'd like to mention it | 15:15 |
| enriquetaso | #topic RemoteFs: Unexcepted error while running the command. | 15:15 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1981961 | 15:15 |
| eharney | we should be able to understand this one just by looking at the code | 15:16 |
| eharney | does the nfs driver call mkdir via rootwrap? if so it needs to be in the volume.filters file | 15:16 |
| enriquetaso | OK, so it's not a incomplete bug | 15:18 |
| enriquetaso | it's a valid bug | 15:18 |
| eharney | i think only os-brick calls mkdir, so this is probably not the right fix, but unclear what is going on | 15:18 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/850142 | 15:18 |
| enriquetaso | OK, maybe we can discuss it later in this week? | 15:19 |
| eharney | sure | 15:20 |
| eharney | it would repro on generic nfs, but may depend on how they have it configured | 15:20 |
| eharney | obviously our gate job is setup in a way where mkdir works | 15:20 |
| enriquetaso | I'm not completely aware of what the fix want's to achieve, but since I have the nfs devstack maybe I can check this as well | 15:21 |
| enriquetaso | makes sense | 15:21 |
| enriquetaso | thanks Eric | 15:21 |
| eharney | yes, mkdir being run as root needs to be done, but the code that does that is in brick, and there is a privsep layer involved there instead of just rootwrap etc, need to peel back the onion a bit | 15:21 |
| enriquetaso | okay | 15:22 |
| enriquetaso | eharney++ | 15:22 |
| enriquetaso | OK, moving one. The last bug of the day is from Storwize | 15:22 |
| enriquetaso | #topic [storwize] Driver Initialization error w.r.t default portset | 15:22 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1982078 | 15:23 |
| enriquetaso | Driver initialization failed with the below error: | 15:23 |
| enriquetaso | ERROR cinder.volume.manager cinder.exception.InvalidInput: Invalid input received: The storage device does not support iSCSI. Please configure the device to support iSCSI or switch to a driver using a different protocol. | 15:23 |
| enriquetaso | There's a fix proposed to master | 15:23 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/850310 | 15:23 |
| enriquetaso | #topic open discussion | 15:24 |
| enriquetaso | Please feel free to mention bugs now | 15:24 |
| eharney | https://bugs.launchpad.net/cinder/+bug/1973228 is one that could use some review | 15:24 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/842237 | 15:25 |
| enriquetaso | Please Cinder team review ^ | 15:26 |
| enriquetaso | i'll do it | 15:26 |
| enriquetaso | it's a medium size bug | 15:26 |
| dwhite44 | There is a tempest bug affecting some CIs, maybe relevant if someone is digging into the multi-attach bug from earlier: https://bugs.launchpad.net/tempest/+bug/1980816 | 15:28 |
| eharney | i don't know enough about tempest to really understand that bug, but good to know about | 15:30 |
| enriquetaso | oh, thanks for sharing that dwhite44 i'll keep an eye on bug1980816 | 15:31 |
| enriquetaso | thanks everyone! | 15:31 |
| enriquetaso | hello? | 15:31 |
| enriquetaso | #endmeeting | 15:31 |
| opendevmeet | Meeting ended Wed Jul 20 15:31:46 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:31 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-20-15.01.html | 15:31 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-20-15.01.txt | 15:31 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-07-20-15.01.log.html | 15:31 |
| enriquetaso | :P thanks | 15:31 |
| whoami-rajat | rosmaita, can you reply to the open concern here? https://review.opendev.org/c/openstack/releases/+/848841 | 15:35 |
| rosmaita | looking | 15:35 |
| whoami-rajat | https://review.opendev.org/c/openstack/releases/+/848841/1/deliverables/xena/cinderlib.yaml#13 | 15:36 |
| whoami-rajat | this comment specifically | 15:36 |
| rosmaita | whoami-rajat: i am still thinking | 15:57 |
| whoami-rajat | ok, I was also confused there | 15:58 |
| rosmaita | you know, there have been no functional changes since 4.1.0, so we could just not release cinderlib xena | 15:59 |
| rosmaita | i'm not sure whether 4fc56c8 is for the gate only, or if it has wider implications | 15:59 |
| rosmaita | that's https://review.opendev.org/c/openstack/cinderlib/+/847622 | 16:00 |
| rosmaita | so if that's all that's holding up the xena stable releases, i suggest drop the cinderlib.yaml and we can figure this out later | 16:01 |
| *** dviroel is now known as dviroel|lunch | 16:06 | |
| whoami-rajat | I | 16:18 |
| whoami-rajat | I'm not sure either about the impact of the patch | 16:18 |
| whoami-rajat | but i tend to agree that the motive of the patch was to fix gate so not really needed in the release | 16:18 |
| whoami-rajat | jbernard, what do you think about it? ^ | 16:19 |
| jbernard | whoami-rajat: if it just addresses issue in the gate, it would be invisible to most (if not all) consumers of the stable release no? | 16:23 |
| jbernard | whoami-rajat: if that's true, we can omit it from this round, imo | 16:24 |
| whoami-rajat | rosmaita, one ques, are we using cinderlib from source in other stable branches like cinder xena? or using the released version of cinderlib xena? | 16:30 |
| opendevreview | Alexander Deiter proposed openstack/cinder master: Fix Infinidat driver to use SSL options https://review.opendev.org/c/openstack/cinder/+/849846 | 16:47 |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 17:02 |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 17:05 |
| *** akekane_ is now known as abhishekk | 17:09 | |
| rosmaita | whoami-rajat: we only run cinderlib functional tests in cinder gates | 17:14 |
| rosmaita | cinderlib gates are from source (as you'd expect) | 17:15 |
| rosmaita | but to answer your question, looks like the cinder CI uses cinderlib from source | 17:17 |
| rosmaita | afaik, the only cinderlib job affected by that change is cinderlib-lvm-functional (which for some reason is passing on victoria without the patch, whereas ussuri is failing: https://review.opendev.org/c/openstack/cinderlib/+/850403 ) | 17:19 |
| *** dviroel|lunch is now known as dviroel | 17:23 | |
| whoami-rajat | so the cinderlib release doesn't affect gate situation in other projects | 17:43 |
| whoami-rajat | rosmaita, jbernard i think we should remove cinderlib from xena releases then | 17:44 |
| whoami-rajat | rosmaita, also regarding the failure, backporting the cinderlib fix to ussuri should fix it ? | 17:44 |
| jbernard | whoami-rajat: ok, ill update teh patch | 17:45 |
| whoami-rajat | thanks! | 17:45 |
| jbernard | whoami-rajat: done | 17:48 |
| jbernard | whoami-rajat: that one should be good to go now | 17:48 |
| rosmaita | whoami-rajat: i think so ... the fix is in wallaby, somehow victoria doesn't need it, but i'm pretty sure it's exactly the problem we are seeing in ussuri | 17:51 |
| rosmaita | i have been poking around trying to figure out why victoria doesn't seem to need it, can't figure anything | 17:52 |
| rosmaita | whoami-rajat: i will propose backports to victoria and wallaby, because i don't think we want to skip victoria with that change? | 17:54 |
| jbernard | whoami-rajat: i mentioned to elod that you would circle back on his stable-xena questions | 17:55 |
| jbernard | whoami-rajat: (i didn't mean to overwrite them) | 17:55 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Flattening of child volumes during deletion https://review.opendev.org/c/openstack/cinder/+/835384 | 17:56 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: RBD: Refactor mocks https://review.opendev.org/c/openstack/cinder/+/850548 | 17:56 |
| eharney | whoami-rajat: geguileo: rosmaita: we merged https://review.opendev.org/c/openstack/cinder/+/847335 w/ the cinder-mypy job failing, so cinder-mypy now fails on master | 17:59 |
| rosmaita | shoot | 18:00 |
| whoami-rajat | rosmaita, yep, don't want to miss a release even though we're not releasing it (i.e. victoria) | 18:15 |
| whoami-rajat | jbernard, ack, will leave a reply there | 18:15 |
| whoami-rajat | eharney, ah, sorry about that, i will propose a fix for it | 18:16 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Fix mypy job https://review.opendev.org/c/openstack/cinder/+/850574 | 18:53 |
| whoami-rajat | eharney, rosmaita fixed it ^ | 18:53 |
| rosmaita | whoami-rajat: good job! | 18:55 |
| whoami-rajat | :) | 18:56 |
| rosmaita | whoami-rajat: turns out that https://review.opendev.org/c/openstack/cinderlib/+/735935 wasn't introduced until wallaby, so we can't cherry-pick https://review.opendev.org/c/openstack/cinderlib/+/847623 directly to victoria | 18:56 |
| rosmaita | train, ussuri, and victoria are all in Extended Maintenance mode anyway at this point | 18:56 |
| rosmaita | if no more work is going to go into those, we could remove the affected job and EOL those branches | 18:56 |
| rosmaita | we'll have to see what geguileo thinks ... i'm not sure what the user community for cinderlib is like | 18:57 |
| rosmaita | hopefully, they like to use the latest release | 18:57 |
| rosmaita | :) | 18:57 |
| whoami-rajat | ah so if we want to fix it, we will need the privsep fix + this patch which introduced privsep, strange to see those branches failing then if it doesn't even support privsep | 18:59 |
| whoami-rajat | s/support privsep/ have privsep support | 19:00 |
| whoami-rajat | I'm not sure either but sooner or later they will be EOL so we should decide if we want to maintain it or not | 19:00 |
| whoami-rajat | wow, even Pike is EM till now ... https://releases.openstack.org/ | 19:04 |
| rosmaita | pike is eol for cinder, anyway | 19:32 |
| *** dviroel is now known as dviroel|afk | 19:34 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 19:57 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 20:02 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 20:35 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 21:06 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!