| opendevreview | Merged openstack/cinder master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/cinder/+/858276 | 04:07 |
|---|---|---|
| opendevreview | Aneesh Pachilangottil proposed openstack/cinder master: NVMEoF volume driver for Fungible Storage https://review.opendev.org/c/openstack/cinder/+/849143 | 04:46 |
| opendevreview | Aneesh Pachilangottil proposed openstack/cinder master: NVMEoF volume driver for Fungible Storage https://review.opendev.org/c/openstack/cinder/+/849143 | 04:47 |
| *** ianw is now known as ianw_pto | 05:09 | |
| opendevreview | Masayuki Igawa proposed openstack/cinder stable/zed: Ceph backup: Close source file https://review.opendev.org/c/openstack/cinder/+/858613 | 06:03 |
| *** tobias-urdin6 is now known as tobias-urdin | 08:49 | |
| opendevreview | Tushar Trambak Gite proposed openstack/os-brick master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/os-brick/+/855012 | 11:15 |
| opendevreview | Tushar Trambak Gite proposed openstack/python-brick-cinderclient-ext master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/python-brick-cinderclient-ext/+/854988 | 11:16 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder-tempest-plugin master: Update metadata in setup.cfg https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/854989 | 11:17 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Dell PowerStore: Add test for NVMEoF Adapter https://review.opendev.org/c/openstack/cinder/+/858678 | 11:37 |
| raghavendrat | hi reviewers ... Requesting feedback on https://review.opendev.org/c/openstack/cinder/+/756711 | 12:16 |
| raghavendrat | Zuul and CI have passed. Thanks. | 12:17 |
| whoami-rajat | tkajinam, hey, can you backport this change to stable zed? before yoga it needs to merge into zed https://review.opendev.org/c/openstack/cinder/+/824149 | 12:32 |
| HappyStacker | Hi all, I've a question for the community. We at Dell have a PowerFlex CI failing all the time. It turns out that a patch recently merged in master has introduced a regression and all the tempest tests which are trying to extend the volume will fail | 13:04 |
| HappyStacker | What's the right approach to handle it? We plan to scope a new feature for PowerFlex 8GB rounding but it's not done yet | 13:05 |
| opendevreview | Merged openstack/cinder master: IBM FlashSystem driver using py3.5 incompatible code https://review.opendev.org/c/openstack/cinder/+/855479 | 13:08 |
| opendevreview | Merged openstack/cinder stable/zed: Ceph backup: Close source file https://review.opendev.org/c/openstack/cinder/+/858613 | 13:08 |
| opendevreview | Merged openstack/cinder master: PowerStore Driver - New format of connection properties for NVMeOF connector https://review.opendev.org/c/openstack/cinder/+/852261 | 13:26 |
| enriquetaso | HappyStacker, you can revert the changes and notify cores | 13:54 |
| enriquetaso | whoami-rajat, rosmaita i think that's the way^ | 13:54 |
| HappyStacker | ok, isn't that complex to handle? how can I do it? | 13:55 |
| rosmaita | HappyStacker: what patch are you talking about? | 13:55 |
| whoami-rajat | HappyStacker, can you provide link of the patch that introduced regression? | 13:55 |
| HappyStacker | give me mn | 13:56 |
| HappyStacker | https://review.opendev.org/c/openstack/cinder/+/836973 | 13:56 |
| whoami-rajat | i think we can revert this and backport the reverted patch to stable/zed ? rosmaita | 13:58 |
| rosmaita | looking | 13:58 |
| rosmaita | HappyStacker: so this is dell's own patch touching only a dell driver? | 13:59 |
| HappyStacker | yes powerflex rounding issue | 13:59 |
| 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 | 14:00 |
| hemna | mep | 14:00 |
| HappyStacker | which is expected to be treated as a whole new feature | 14:00 |
| whoami-rajat | but it's a bugfix | 14:00 |
| rosmaita | HappyStacker: so not a quick bugfix? | 14:00 |
| rosmaita | maybe we can discuss in open discussion at the meeting | 14:01 |
| HappyStacker | ok | 14:01 |
| whoami-rajat | HappyStacker, I've been familiar with this problem for a long time and the real issue is not in the create volume call but in the extend call which doesn't return a model update | 14:02 |
| whoami-rajat | HappyStacker, but as rosmaita said, we will discuss this in open discussion in #openstack-meeting-alt | 14:03 |
| yuval | also here: Before the meeting ends, is anyone running openstack redhat platform system and I can ask him few questions? | 15:01 |
| enriquetaso | #startmeeting cinder_bs | 15:02 |
| opendevmeet | Meeting started Wed Sep 21 15:02:44 2022 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:02 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:02 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:02 |
| HappyStacker | yuval:I'm using RHOSP | 15:03 |
| enriquetaso | Full list of bugs: | 15:03 |
| enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030543.html | 15:03 |
| enriquetaso | #topic NFS volume snapshot | 15:04 |
| TusharTgite_ | hi core reviewers need areview on this already have one + 2 https://review.opendev.org/c/openstack/cinderlib/+/855010 , https://review.opendev.org/c/openstack/cinder/+/855350 , https://review.opendev.org/c/openstack/cinder-specs/+/854982 , | 15:04 |
| enriquetaso | does not update volume attachment format to qcow2 | 15:04 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1989514 | 15:04 |
| enriquetaso | Summary: Nova team found that after performing a volume snapshot on a stopped instance, that instance could no longer boot. | 15:04 |
| enriquetaso | I reproduced the problem, so I confirm the bug | 15:05 |
| enriquetaso | There's a fix proposed to master | 15:05 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/857528 | 15:05 |
| enriquetaso | But the fix doesn't looks right yet. There are some inconsistencies between nova and cinder. | 15:07 |
| enriquetaso | Nova adds a top-level layer on the volume (a qcow2 layer on top of raw volume when creating the snapshot). However, on the Cinder side, we still think that the volume is raw. | 15:07 |
| enriquetaso | After discussing with geguileo, whoami-rajat and rosmaita, the right approach would be that nova should not be updating the reference but Cinder. | 15:08 |
| enriquetaso | It's crazy that generic-nfs snapshots are not working | 15:09 |
| rosmaita | enriquetaso: ++ | 15:10 |
| enriquetaso | :P | 15:10 |
| enriquetaso | well, nothing else, the patch need further work before merge | 15:10 |
| enriquetaso | #action(enriquetaso) check why tempest isn't cheking this case scenario with generic-nfs | 15:11 |
| enriquetaso | moving on | 15:12 |
| enriquetaso | #topic Dell PowerMax - Optimized SL should be assigned when retyping a volume to a default volume type. | 15:12 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1989730 | 15:12 |
| enriquetaso | Summary: Optimized SL should be assigned when retyping a volume to default volume type. | 15:13 |
| enriquetaso | Fix proposed to master | 15:13 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/803773 | 15:13 |
| enriquetaso | I'll review, please feel free to review the patch | 15:13 |
| enriquetaso | moving on | 15:13 |
| enriquetaso | walshh_, feel free to add comments about the fix if you have some | 15:14 |
| enriquetaso | OK, moving on | 15:15 |
| enriquetaso | #topic Dell PowerFlex: Failed to attach a volume when using self-signed Certificates | 15:15 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1990136 | 15:15 |
| enriquetaso | I'd like to see a more clear bug report but I guess it's fine.. | 15:16 |
| enriquetaso | Fix proposed to master: | 15:16 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/858370 | 15:16 |
| HappyStacker | I can ùake it clear | 15:16 |
| enriquetaso | I think HappyStacker is working on that | 15:17 |
| HappyStacker | I am | 15:17 |
| HappyStacker | Let me add info on the description | 15:17 |
| enriquetaso | thanks HappyStacker, feel free to add a description of the bug if you have time at least :D | 15:17 |
| enriquetaso | thanks | 15:17 |
| rosmaita | the commit message on that patch is very nice | 15:17 |
| enriquetaso | yes it is ! | 15:18 |
| enriquetaso | Cinder team, please review the fix ^ when you have time | 15:18 |
| HappyStacker | thks, I'll carry it over to the bug description, is that fine? | 15:18 |
| enriquetaso | yep, thanks HappyStacker ! | 15:19 |
| enriquetaso | OK, last bug of today's meeting | 15:19 |
| HappyStacker | Done | 15:19 |
| enriquetaso | #topic [Yadro] Tatlin driver does not set project id for temp volumes | 15:19 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1990134 | 15:19 |
| enriquetaso | Summary: When doing snapshots Tatlin driver creates a temp volume with admin context, i.e. without project id. If at the same time we run "os-hosts" tests we get projects with null name and the tests fail with schema validation (project id must be string, not null) | 15:20 |
| enriquetaso | looks like something that Yadro team should fix soon | 15:20 |
| enriquetaso | fix proposed to master | 15:20 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/858293 | 15:20 |
| enriquetaso | I haven't looked deep on the fix but it may need a release note | 15:21 |
| enriquetaso | weird that CI passed | 15:22 |
| enriquetaso | OK.. moving on | 15:23 |
| enriquetaso | #topic open discussion | 15:23 |
| enriquetaso | feel free to add bugs to discuss now | 15:23 |
| enriquetaso | I've created this not-so-cool script for listing the launchpad bugs https://github.com/enriquetaso/launchpad-bug-reporter/blob/main/main.py | 15:25 |
| enriquetaso | My idea is to see if I can automate the bug report email or something like that in the future | 15:26 |
| enriquetaso | so any suggestions are welcome | 15:26 |
| HappyStacker | how does the output look like? | 15:26 |
| rosmaita | enriquetaso: that's a good idea | 15:27 |
| enriquetaso | the output doesn't look cool, just listing the bugs plain terminal output | 15:27 |
| enriquetaso | OH, i forgot about | 15:27 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1990053 | 15:27 |
| enriquetaso | Cinder NFS backend with image-volume cache creates volumes with wrong size | 15:28 |
| enriquetaso | NFS is having a lot of problems lately | 15:28 |
| enriquetaso | the bug report doesn't have any fix proposed yet, so I'll try to reproduce and I'll include it in next week email report | 15:29 |
| enriquetaso | OK, we're running out of time | 15:30 |
| enriquetaso | Thanks everyone | 15:30 |
| HappyStacker | there's a supposed fix | 15:30 |
| enriquetaso | oh | 15:30 |
| HappyStacker | https://review.opendev.org/c/openstack/cinder/+/855964 | 15:30 |
| enriquetaso | oh i vote -1 on it | 15:30 |
| enriquetaso | thanks HappyStacker, i need to look closer | 15:30 |
| rosmaita | your -1 was for them not filing a bug! | 15:31 |
| enriquetaso | yes! lol | 15:31 |
| enriquetaso | OK, feel free to review it too! | 15:31 |
| enriquetaso | I'm closing this meeting for now | 15:31 |
| enriquetaso | thanks!! | 15:31 |
| enriquetaso | #endmeeting | 15:32 |
| opendevmeet | Meeting ended Wed Sep 21 15:32:02 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:32 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-21-15.02.html | 15:32 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-21-15.02.txt | 15:32 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-09-21-15.02.log.html | 15:32 |
| rosmaita | thanks enriquetaso | 15:32 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: Revert "PowerFlex driver - fix the display of the incorrect volume" https://review.opendev.org/c/openstack/cinder/+/858621 | 15:40 |
| sfv880 | Hello whoami-rajat: I replied and fixed on https://review.opendev.org/c/openstack/cinder/+/850320 - thank you very much! | 16:25 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Add command to fix/generate idempotent IDs https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/839301 | 16:36 |
| opendevreview | Merged openstack/cinder-specs master: Update doc8 version https://review.opendev.org/c/openstack/cinder-specs/+/854982 | 20:55 |
| opendevreview | Eric Harney proposed openstack/cinder stable/yoga: Tests: Fix service stopping on cleanup https://review.opendev.org/c/openstack/cinder/+/858817 | 20:58 |
| opendevreview | Eric Harney proposed openstack/cinder stable/yoga: Tests: Monkey patch before logs are loaded https://review.opendev.org/c/openstack/cinder/+/858818 | 20:58 |
| opendevreview | Eric Harney proposed openstack/cinder stable/yoga: Tests: Fix test runner never finishing https://review.opendev.org/c/openstack/cinder/+/858819 | 20:58 |
| opendevreview | Merged openstack/cinder master: Use the json format output of qemu-img info https://review.opendev.org/c/openstack/cinder/+/799663 | 21:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!