| whoami-rajat | rosmaita, hey, if you're still around can you take a look at this backport? https://review.opendev.org/c/openstack/cinder/+/821739 | 01:06 |
|---|---|---|
| whoami-rajat | we can ask them to update the patch but if you're OK with as it | 01:07 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Follow up: Hitachi and OEM: Update documents https://review.opendev.org/c/openstack/cinder/+/875681 | 02:37 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Follow up: Hitachi and OEM: Update documents https://review.opendev.org/c/openstack/cinder/+/875681 | 02:40 |
| TusharTgite | whoami-rajat: hi could you review this patch https://review.opendev.org/c/openstack/cinder/+/860209 | 03:31 |
| opendevreview | Eric Harney proposed openstack/cinder master: test_rbd_iscsi: Make tests compatible with python 3.11 https://review.opendev.org/c/openstack/cinder/+/869396 | 04:14 |
| opendevreview | Merged openstack/cinder stable/xena: PowerMax Driver - Get manageable volumes fix https://review.opendev.org/c/openstack/cinder/+/821739 | 04:44 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 04:57 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 05:09 |
| opendevreview | Kumar Kanishka proposed openstack/cinder master: [IBM-Storwize-SVf] Invalid Exception raised due to invalid portset value https://review.opendev.org/c/openstack/cinder/+/874633 | 06:38 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Follow up: Hitachi: Change option name `hitachi_pool` to `hitachi_pools` https://review.opendev.org/c/openstack/cinder/+/875603 | 08:26 |
| opendevreview | Merged openstack/cinder master: Tests: Fix typo'd unit test name https://review.opendev.org/c/openstack/cinder/+/865035 | 09:49 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Include volume type constraints in internal API https://review.opendev.org/c/openstack/cinder/+/846146 | 11:10 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Delete volume and cloned-volume concurrently https://review.opendev.org/c/openstack/cinder/+/843309 | 11:10 |
| *** chuanm6 is now known as chuanm | 11:26 | |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] : Enable support for mirror-pool option for replication volume-type https://review.opendev.org/c/openstack/cinder/+/870006 | 12:32 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add the clone_across_pools driver capability https://review.opendev.org/c/openstack/cinder/+/847131 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: declare the clone_across_pools capability https://review.opendev.org/c/openstack/cinder/+/847132 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: cosmetic: comment headings instead of empty lines https://review.opendev.org/c/openstack/cinder/+/873507 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop _attach_volume() and _detach_volume() https://review.opendev.org/c/openstack/cinder/+/787296 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool driver: remove the obsolete backup_volume() https://review.opendev.org/c/openstack/cinder/+/803823 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: drop copy_image_to_volume() and copy_volume_to_image() https://review.opendev.org/c/openstack/cinder/+/870550 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the "rename volume" unit test emulation https://review.opendev.org/c/openstack/cinder/+/870681 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: fix the retype volume flow https://review.opendev.org/c/openstack/cinder/+/870551 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 12:49 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 12:49 |
| Roamer` | ...so I'd just like to point out that even though the very last change in this queue is a new driver feature, the other nine are more or less trivial bugfixes, some of which are sorely needed... and we deploy all of them at customers' sites, and our CI system applies most of them before running each test | 12:50 |
| tosky | geguileo: so, do you want me to fold https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/870889 inside https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/855576? Should we keep both jobs? I forgot what we agreed (if we agreed on something) | 13:28 |
| geguileo | tosky: iirc in order to reduce the number of jobs we agreed that the latest patch was the most relevant, since it tested the more complex code (shared targets + multipathing + new connection information format) | 13:31 |
| tosky | geguileo: oook, as the based job definition is in the first patch, do you want to me to just update the first one with the additional options and call the resulting job "the" nvme job? | 13:32 |
| geguileo | tosky: sounds good to me | 13:32 |
| tosky | uhm, it's probably easier to just use your patch instead | 13:34 |
| opendevreview | Luigi Toscano proposed openstack/cinder-tempest-plugin master: zuul: new LVM/nvmet job (relevant scenario) https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/870889 | 13:43 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Follow up: Hitachi: Change option name `hitachi_pool` to `hitachi_pools` https://review.opendev.org/c/openstack/cinder/+/875603 | 13:58 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 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 | 14:00 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Follow up: Hitachi and OEM: Update documents https://review.opendev.org/c/openstack/cinder/+/875681 | 14:13 |
| enriquetaso | #startmeeting cinder_bs | 15:01 |
| opendevmeet | Meeting started Wed Mar 1 15:01:22 2023 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 | Hello, Welcome to the Bug meeting | 15:01 |
| enriquetaso | Only 3 bugs reported | 15:01 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032516.html | 15:01 |
| enriquetaso | #topic Volume multiattach exposed to non-admin users via API | 15:02 |
| enriquetaso | We discussed this bug on the main meeting, so please check the logs: | 15:02 |
| enriquetaso | #link https://meetings.opendev.org/meetings/cinder/2023/cinder.2023-03-01-14.00.log.html#l-53 | 15:03 |
| enriquetaso | and please review Rajat's patches! | 15:03 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/874865 | 15:03 |
| enriquetaso | Moving on | 15:04 |
| enriquetaso | #topic image_utils: code hardening around decompression | 15:04 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2008429 | 15:04 |
| enriquetaso | rosmaita, talked about it in last week video meeting | 15:05 |
| rosmaita | i think i tagged it low hanging fruit | 15:05 |
| enriquetaso | It's a low-hanging-fruit bug so hopefully some potential intern may work on it | 15:05 |
| rosmaita | :) | 15:05 |
| enriquetaso | but anyone can work on it! | 15:06 |
| enriquetaso | thanks rosmaita | 15:06 |
| Tony_Saad | Hey, is it worth mentioning again our Antelope Dell driver bugs https://review.opendev.org/c/openstack/cinder/+/768105 https://review.opendev.org/c/openstack/cinder/+/797970 https://review.opendev.org/c/openstack/cinder/+/821739 https://review.opendev.org/c/openstack/cinder/+/858370 | 15:06 |
| enriquetaso | i see you are spanning any meeting you can :P | 15:07 |
| enriquetaso | remember to review Dell patches ^ | 15:08 |
| Tony_Saad | this is the first time I join this meeting haha I dont know which is best to bring up the topic | 15:08 |
| enriquetaso | One is already merged tho, and the last one.. i would like to see and update on the sean comment | 15:08 |
| Tony_Saad | thanks! | 15:09 |
| enriquetaso | np | 15:09 |
| enriquetaso | Last bug | 15:09 |
| enriquetaso | #topic [yadro] tatlin_client is_port_assigned method broken | 15:10 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2008735 | 15:10 |
| enriquetaso | No patch proposed to master yet | 15:10 |
| enriquetaso | I dont know if anyone from yadro would like to say something here | 15:10 |
| eharney | more evidence that we still benefit from pylint ^ | 15:10 |
| enriquetaso | true, we should make openstack-tox-pylint work again | 15:12 |
| eharney | i'm working on it, need reviews on patches like https://review.opendev.org/c/openstack/cinder/+/871289 | 15:12 |
| enriquetaso | ++ | 15:15 |
| enriquetaso | thanks eharney for working on this | 15:16 |
| enriquetaso | #topic open discussion | 15:16 |
| enriquetaso | Feel free to proposed any bug report or bug now | 15:16 |
| enriquetaso | I see https://review.opendev.org/c/openstack/cinder/+/871286 is also related to pylint, maybe we can tag them or something to keep it track together | 15:17 |
| eharney | that one is more complex, pylint seems to be very broken on 3.10 and 3.11 for us, still figuring that out | 15:17 |
| eharney | also, thanks for reviewing my 3.11 unit test fixes... one of them failed in the unit test gate and i have to figure out why | 15:18 |
| enriquetaso | :( | 15:19 |
| enriquetaso | thanks to you eharney | 15:20 |
| eharney | unfortunately i can't run tox unit tests locally today... anyone else hit that? | 15:20 |
| eharney | error: Multiple top-level packages discovered in a flat-layout: ['etc', 'roles', 'cinder', 'playbooks', 'releasenotes']. | 15:20 |
| enriquetaso | To be honest, I'm not able to deploy devstack in general.. `tox -e py3` returns `ERROR: Could not install packages due to an OSError: [Errno 13] Permission denied: 'RECORD'` | 15:23 |
| eharney | this fails for me with just a cinder clone and running any tox env (tox using 3.11) | 15:24 |
| eharney | uhy | 15:24 |
| enriquetaso | oh true `ERROR: could not install deps [virtualenv>=20.17.1, tox >= 4.0.0]; v = InvocationError("/usr/bin/python -m pip install | 15:24 |
| enriquetaso | 'virtualenv>=20.17.1' 'tox >= 4.0.0'", 1)` | 15:24 |
| enriquetaso | god | 15:24 |
| enriquetaso | i'm using tox 3.28 | 15:25 |
| eharney | i don't think it's tox itself | 15:26 |
| eharney | the error is coming from setuptools | 15:26 |
| rosmaita | jeez ... you may want to change your tox.ini to tox<4.0.0 and see what happens | 15:27 |
| opendevreview | Eric Harney proposed openstack/cinder master: Set packages in setup.cfg https://review.opendev.org/c/openstack/cinder/+/875939 | 15:29 |
| eharney | rosmaita: this fixes it for me ^ | 15:29 |
| eharney | dunno if it's correct though, just a quick pass.. | 15:29 |
| eharney | anyway i guess we can end the bug meeting | 15:30 |
| enriquetaso | yes | 15:30 |
| enriquetaso | thanks you! | 15:30 |
| enriquetaso | I'll try both things | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Mar 1 15:30:39 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-01-15.01.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-01-15.01.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-01-15.01.log.html | 15:30 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Fix temp volume project id for Tatlin storage https://review.opendev.org/c/openstack/cinder/+/858293 | 16:30 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin Unified driver - remove unused code https://review.opendev.org/c/openstack/cinder/+/862489 | 16:31 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/862490 | 16:31 |
| gmann | rosmaita: whoami-rajat : on multiattach things , read the meeting discussion but just replying /continue here. please help me to understand. so if multiattach and VT is passed together I think it is valid to return 400 and ask users to request one way | 17:01 |
| rosmaita | gmann: was just going to reply on the ML | 17:02 |
| gmann | but if anyone existing users passing only multiattach=True in request, does request/operation is successful ? | 17:02 |
| gmann | rosmaita: k | 17:02 |
| rosmaita | gmann: https://lists.openstack.org/pipermail/openstack-discuss/2023-March/032530.html | 17:03 |
| gmann | I am just considering tempest test case where multiattach=True in request is successful case. | 17:03 |
| rosmaita | well, the problem is that tempest just does the attach, then does a teardown | 17:03 |
| rosmaita | tempest doesn't care if the data gets corrupted or something | 17:04 |
| rosmaita | (it's not tempest's job) | 17:04 |
| gmann | ohk so it is not sully tested till the issue happen | 17:04 |
| gmann | fully | 17:04 |
| rosmaita | yeah, and you can get lucky and not have any problem | 17:04 |
| rosmaita | but in general, mutltiattach should depend on the volume type | 17:05 |
| gmann | rosmaita: whoami-rajat: replied on ML, if this is always unsuccessful (unknown successful ) operation as whole, I agree not to microversion which I think you can only confirm. My and tempest test knowledge is limited for this operation. | 17:21 |
| rosmaita | gmann: thanks | 17:22 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Skip update_features when features = 0 https://review.opendev.org/c/openstack/cinder/+/865855 | 18:15 |
| opendevreview | Eric Harney proposed openstack/cinder master: RBD: Skip update_features when features = 0 https://review.opendev.org/c/openstack/cinder/+/865855 | 18:15 |
| opendevreview | Eric Harney proposed openstack/cinder master: Hitachi HBSD: Fix invalid "raise None" https://review.opendev.org/c/openstack/cinder/+/875977 | 18:42 |
| opendevreview | Andrei proposed openstack/cinder master: JovianDSS: improve storage net ifc picking alg https://review.opendev.org/c/openstack/cinder/+/862432 | 19:54 |
| opendevreview | Tony Saad proposed openstack/cinder master: PowerMax Driver - RDF checks https://review.opendev.org/c/openstack/cinder/+/768105 | 20:50 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!