| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/876907 | 04:25 |
|---|---|---|
| *** elodilles is now known as elodilles_pto | 06:49 | |
| whoami-rajat | hi jungleboyj enriquetaso eharney geguileo , can i have review on this patch? it unblocks cinderclient yoga gate https://review.opendev.org/c/openstack/python-cinderclient/+/874448 | 11:12 |
| enriquetaso | whoami-rajat, done | 11:15 |
| whoami-rajat | thanks enriquetaso ! | 11:15 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Include volume type constraints in internal API https://review.opendev.org/c/openstack/cinder/+/846146 | 11:31 |
| enriquetaso | eharney, hey, can you review my comment on https://review.opendev.org/c/openstack/cinder/+/877185 | 12:32 |
| enriquetaso | is autospec=true correct in that case? | 12:33 |
| opendevreview | Merged openstack/python-cinderclient stable/yoga: [stable-only] Skip backup test in functional py39 job https://review.opendev.org/c/openstack/python-cinderclient/+/874448 | 12:33 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fixed docstring description for test_boot_cloned_encryted_volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 12:49 |
| opendevreview | Khadija Kamran proposed openstack/cinder master: Replace plain mock.Mock() call with mock.Mock(spec) https://review.opendev.org/c/openstack/cinder/+/877185 | 13:18 |
| opendevreview | Khadija Kamran proposed openstack/cinder master: Replace plain mock.Mock() call with mock.Mock(spec) https://review.opendev.org/c/openstack/cinder/+/877185 | 13:33 |
| 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 |
| rosmaita | whoami-rajat: ty | 14:01 |
| whoami-rajat | np | 14:01 |
| whoami-rajat | jbernard, hey, would you be willing to take a look at zaitcev's patch as a second core (apart from me)? we need to merge it by this week https://review.opendev.org/c/openstack/cinder/+/852654 | 14:24 |
| opendevreview | Merged openstack/cinder master: Dell PowerFlex: Additionnal params for enabling self signed certificates https://review.opendev.org/c/openstack/cinder/+/858370 | 14:28 |
| opendevreview | Merged openstack/cinder master: PowerMax Driver - Fix for force flag https://review.opendev.org/c/openstack/cinder/+/797970 | 14:31 |
| enriquetaso | hi | 15:03 |
| rosmaita | o/ | 15:04 |
| whoami-rajat | hi | 15:04 |
| enriquetaso | #startmeeting cinder_bs | 15:04 |
| opendevmeet | Meeting started Wed Mar 15 15:04:58 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:04 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:04 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:04 |
| enriquetaso | full list of bugs: | 15:05 |
| enriquetaso | #link https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-01-15.01.log.html | 15:05 |
| enriquetaso | #topic Extending SCSI multipath doesn't work if Nova configuration changed | 15:05 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/2009157 | 15:06 |
| enriquetaso | geguileo, report this and next bug. Not fix proposed to master yet | 15:06 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/2009158 | 15:06 |
| enriquetaso | #topic [rbac] Reader user able to delete a user message | 15:08 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2009818 | 15:08 |
| enriquetaso | I think rosmaita and anyone involve in RBAC is affected by ^ | 15:08 |
| whoami-rajat | I will be working on the extend related bugs along with geguileo so I've assigned them to myself | 15:09 |
| rosmaita | enriquetaso: i need to verify that one still | 15:09 |
| enriquetaso | excellent! | 15:14 |
| enriquetaso | thanks | 15:14 |
| enriquetaso | moving on | 15:14 |
| enriquetaso | #topic Cinder Message API creates failure "'NoneType' object is not subscriptable" | 15:15 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2009483 | 15:15 |
| enriquetaso | Marked as duplicate of similar bug <https://bugs.launchpad.net/cinder/+bug/1978729> | 15:15 |
| enriquetaso | Based on rosmaita comment on https://review.opendev.org/c/openstack/cinder/+/876587 | 15:16 |
| enriquetaso | Moving on | 15:16 |
| rosmaita | yeah, i don't know that it's a duplicate | 15:16 |
| rosmaita | my point was that the proposal to detect 'None' gracefully might be covering up some other issue, like in that other bug | 15:16 |
| rosmaita | because i don't think that field should ever be None | 15:16 |
| enriquetaso | oh..maybe it's not related then.. i'll take a better look.. | 15:17 |
| rosmaita | it's sort of related, it's just the other bug had to do with the context not being serialized correctly | 15:17 |
| enriquetaso | isnt the api that should check if the fiel is None | 15:17 |
| rosmaita | don't think that's happening here | 15:17 |
| rosmaita | well, it's an internal API | 15:18 |
| rosmaita | i guess this is a place where mypy could help | 15:18 |
| rosmaita | make sure the 'action' or whatever cannot be None | 15:19 |
| enriquetaso | makes sense.. mypy should check this on the code, I dont know if mypy is covering this file.. I guess it needs further investigation, i'll take a deep look later | 15:20 |
| rosmaita | or, leave a comment in the bug and let the person working on it take a look | 15:20 |
| enriquetaso | even better | 15:23 |
| enriquetaso | thanks rosmaita | 15:23 |
| enriquetaso | moving on | 15:23 |
| enriquetaso | #topic [incomplete] [HPE] Volume name migration fails with keyerror | 15:23 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2008931 | 15:24 |
| enriquetaso | I think we need more information for the reporter eharney | 15:24 |
| enriquetaso | okay, ill talk to eric later | 15:24 |
| enriquetaso | last bug: | 15:24 |
| enriquetaso | #topic [nova] size_iops_sec does behave differently than mentioned in docs | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/nova/+bug/2008762 | 15:25 |
| enriquetaso | I've removed cinder project from that bug | 15:25 |
| enriquetaso | #topic open discussion | 15:25 |
| enriquetaso | any bugs to discuss today? | 15:25 |
| rosmaita | nothing from me | 15:29 |
| enriquetaso | :D | 15:29 |
| enriquetaso | okay! | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Mar 15 15:30:06 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-15-15.04.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-15-15.04.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-03-15-15.04.log.html | 15:30 |
| jbernard | whoami-rajat: will do | 15:34 |
| drencrom | Hi. Going back to the metadata from Glance discussion from the cinder meeting. I see that Glance also uses a mysql text field for the metadata. I think it should be reasonable to guess that glance will not send metadata that cannot be stored in its own database but it is true that the check for a particular string length migth not be very useful. | 16:09 |
| rosmaita | drencrom: you know, nova also saves the image metadata in a table when you build an instance from an image ... dyk what they do about this? | 17:04 |
| drencrom | rosmaita: no, but I can check | 17:35 |
| rosmaita | drencrom: thanks, that would be helpful | 17:35 |
| opendevreview | Ajay Pal proposed openstack/cinder master: Update block-storage-overview to reflect multiattach https://review.opendev.org/c/openstack/cinder/+/877070 | 18:32 |
| opendevreview | Ajay Pal proposed openstack/cinder master: Update block-storage-overview to reflect multiattach https://review.opendev.org/c/openstack/cinder/+/877544 | 18:59 |
| opendevreview | Ajay Pal proposed openstack/cinder master: Update block-storage-overview to reflect multiattach https://review.opendev.org/c/openstack/cinder/+/877544 | 19:01 |
| opendevreview | Pete Zaitcev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 23:33 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!