opendevreview | Masayuki Igawa proposed openstack/os-brick stable/victoria: RBD: Fix disconnect_volume for encrypted volumes https://review.opendev.org/c/openstack/os-brick/+/864617 | 02:03 |
---|---|---|
opendevreview | Manojkatari proposed openstack/cinder stable/wallaby: [PURE] support IPv6 / add parameter pure_iscsi_cidr_list https://review.opendev.org/c/openstack/cinder/+/864185 | 05:59 |
*** amoralej|off is now known as amoralej | 07:00 | |
enriquetaso | Hello geguileo, sorry for bothering you but last week in the bug meeting we talked about https://bugs.launchpad.net/cinder/+bug/1995204 | 10:53 |
enriquetaso | https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-09-15.00.log.html#l-10 | 10:53 |
enriquetaso | and I'm still not sure the importance of the bug, would you mind to take a look a see if it's valid and makes sense? | 10:54 |
opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 11:06 |
opendevreview | Dmitriy Rabotyagov proposed openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 11:08 |
*** dviroel|afk is now known as dviroel | 11:20 | |
*** amoralej is now known as amoralej|off | 11:47 | |
geguileo | enriquetaso: sure, give me 10 minutes... I'm trying to configure my IRC client (weechat) to connect to slack... | 13:09 |
opendevreview | Dr. Jens Harbott proposed openstack/cinder master: WIP: Test Ubuntu 22.04 (Jammy) migration https://review.opendev.org/c/openstack/cinder/+/861609 | 13:11 |
TusharTgite | hi core reviewers need a review on this https://review.opendev.org/c/openstack/cinder/+/860212 https://review.opendev.org/c/openstack/cinder/+/860227 | 13:12 |
*** abhishekk is now known as akekane|home | 13:26 | |
senrique | geguileo, thanks! | 13:27 |
senrique | TusharTgite, done | 13:32 |
TusharTgite | senrique: thanks | 13:36 |
opendevreview | Merged openstack/cinder master: Set backup status to error on VolumeNotFound https://review.opendev.org/c/openstack/cinder/+/864100 | 13:38 |
opendevreview | Merged openstack/cinder master: DataCore: Add note about unique CHAP storage paths https://review.opendev.org/c/openstack/cinder/+/849869 | 13:38 |
opendevreview | Merged openstack/cinder master: Update mypy to 0.981 https://review.opendev.org/c/openstack/cinder/+/859877 | 13:38 |
*** amoralej|off is now known as amoralej | 13:46 | |
geguileo | senrique: ok, check, that's the expected behavior, which doesn't mean that it's not a problem... | 13:54 |
geguileo | senrique: it's the whole issue of each scheduler keeping an updated value only locally | 13:54 |
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 adiare | 13:59 |
enriquetaso | thanks | 13:59 |
whoami-rajat | np | 14:00 |
enriquetaso | hi | 15:03 |
enriquetaso | cinder bug meeting | 15:03 |
enriquetaso | very quick | 15:03 |
enriquetaso | #startmeeting cinder_bs | 15:03 |
opendevmeet | Meeting started Wed Nov 16 15:03:42 2022 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 |
enriquetaso | ganso, whoami-rajat | 15:03 |
enriquetaso | do you want to discuss the topic now? | 15:04 |
enriquetaso | It will be recorded on the bug meeting | 15:04 |
ganso | yea very quick | 15:04 |
ganso | it is just an FYI on https://review.opendev.org/c/openstack/cinder/+/812685 | 15:04 |
whoami-rajat | I've no issues now or later | 15:04 |
enriquetaso | #topic Filter reserved image properties | 15:04 |
ganso | we had a lot of interesting discussion on that patch, at some point we intended to have a discussion here | 15:04 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/812685 | 15:04 |
ganso | but it seems that yesterday we ended up agreeing on the approach | 15:05 |
rosmaita | ganso: what's your real name? (are you Rodrigo or Rafael?) | 15:05 |
ganso | so it might need not a discussion | 15:05 |
ganso | rosmaita: I'm Rodrigo | 15:05 |
rosmaita | ok, thanks | 15:05 |
ganso | so I suppose Rafael is changing the patch to have the metadata removed only on upload-back-to-glance | 15:06 |
ganso | if anyone disagrees or would like to discuss it further, we can do so, otherwise, might not need any further discussing | 15:06 |
whoami-rajat | I also kind of agree on that approach | 15:06 |
whoami-rajat | if glance doesn't want to supply some metadata to the consumer (nova, cinder) then glance should not supply it in the first place | 15:07 |
whoami-rajat | if glance is supplying it then it makes sense to keep it and only filter out when it creates issues | 15:07 |
rosmaita | not surprisingly, i agree with that | 15:08 |
whoami-rajat | that provides us flexibility, since once metadata is removed, we can't recover it but if it stays, we can always filter it later | 15:09 |
whoami-rajat | so +1 to rosmaita's suggestion | 15:10 |
whoami-rajat | ganso, anything else on this? don't want to block enriquetaso's meeting for long :) | 15:11 |
ganso | whoami-rajat: nothing else =) | 15:11 |
ganso | thanks enriquetaso for this meeting | 15:11 |
whoami-rajat | great, thanks | 15:11 |
enriquetaso | cool | 15:11 |
whoami-rajat | thanks enriquetaso ! | 15:11 |
enriquetaso | OK, Only one bug for today's meeting. List of bugs: | 15:11 |
enriquetaso | no worries | 15:12 |
enriquetaso | #link https://lists.openstack.org/pipermail/openstack-discuss/2022-November/031212.html | 15:12 |
enriquetaso | #topic Cinderlib Gate broken in Zed | 15:12 |
enriquetaso | #link https://bugs.launchpad.net/cinderlib/+bug/1996738 | 15:12 |
enriquetaso | As discussed last week at the cinder meeting. I've opened a bug for the cinderlib gate failures. | 15:12 |
enriquetaso | whoami-rajat, rosmaita | 15:12 |
enriquetaso | ^ | 15:12 |
enriquetaso | Let me know if the description is wrong so I can update it. | 15:12 |
rosmaita | looks good, thank you for filing the bug | 15:12 |
whoami-rajat | i don't see "horribly broken" in title but it should be fine | 15:13 |
enriquetaso | lol | 15:13 |
enriquetaso | i can update the title :P | 15:13 |
rosmaita | enriquetaso: you should! | 15:13 |
whoami-rajat | looks good, thanks for reporting it | 15:13 |
enriquetaso | #action: (enriquetaso) update the titel | 15:13 |
enriquetaso | title* | 15:13 |
enriquetaso | OK, moving on | 15:13 |
rosmaita | ok, i will assign it to myself with the goal of reporting status at the festival of bugs on friday | 15:13 |
enriquetaso | thanks | 15:14 |
whoami-rajat | rosmaita++ | 15:14 |
enriquetaso | #topic "[OpenStack Yoga] Creating a VM fails when stopping only one rabbitmq. | 15:14 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1990257 | 15:14 |
enriquetaso | I've removed cinder from that bug | 15:14 |
enriquetaso | it doesn't look like a cinder bug, but if you think it should, please let me know | 15:15 |
enriquetaso | #topic open discussion | 15:15 |
enriquetaso | any bugs to discuss ? | 15:15 |
rosmaita | circling back to that cinderclient snapshot-create fix, i think we should backport it as far as xena | 15:15 |
rosmaita | that's bug https://bugs.launchpad.net/python-cinderclient/+bug/1995883 | 15:15 |
whoami-rajat | we need more info on 1990257, if they're creating the instance BFV then it might be a cinder problem | 15:16 |
rosmaita | i think we need to un-target from Wallaby, and add Zed, Yoga, and Xena instead | 15:16 |
enriquetaso | ture | 15:16 |
enriquetaso | true | 15:16 |
whoami-rajat | i will add a comment there | 15:17 |
enriquetaso | #action(enriquetaso): un-target wallaby and add zed, yoga and xena for bug1990257 | 15:17 |
enriquetaso | thanks whoami-rajat | 15:17 |
enriquetaso | thanks for updating rosmaita | 15:18 |
rosmaita | np, i actually brought it up becasue i didn't know if you've used the "target to series" functionality in launchpad? | 15:19 |
enriquetaso | i'm trying to use them yes | 15:19 |
rosmaita | cool, we have an action item for jon bernard on this | 15:20 |
enriquetaso | jbernard :) | 15:20 |
rosmaita | it's showing that you can nominate kilo through wallaby ... x, y, z aren't available | 15:20 |
enriquetaso | #action: add a comment in 1990257 asking if they are using intance BFV | 15:20 |
enriquetaso | :/ | 15:21 |
whoami-rajat | enriquetaso, added a comment | 15:21 |
rosmaita | oh, that may be because you already did the targeting | 15:21 |
enriquetaso | i've updated it rosmaita | 15:21 |
enriquetaso | yes, sorry | 15:21 |
rosmaita | not sure why it picked Zed as the tracking branch, though | 15:21 |
enriquetaso | yes.. that's wrong | 15:22 |
enriquetaso | i'll let you do it, maybe multiple updates from diff people confuses launchpad | 15:22 |
rosmaita | no, it's a launchpad thing, nothing you did | 15:22 |
enriquetaso | thanks! | 15:23 |
rosmaita | the 2023.1 series isn't defined for cinderclient yet | 15:23 |
rosmaita | and Zed is showing as the focus of development ... so that's why the bug is tracking zed | 15:23 |
enriquetaso | the fix is in master tho | 15:24 |
rosmaita | yeah, but launchpad only knows about the releases we tell it about | 15:24 |
enriquetaso | u.u OK | 15:25 |
enriquetaso | Do we have any other bug to discuss? | 15:25 |
enriquetaso | OK, closing the meeting then :) | 15:27 |
enriquetaso | thank you!! | 15:27 |
whoami-rajat | thanks enriquetaso | 15:27 |
enriquetaso | #endmeeting | 15:27 |
opendevmeet | Meeting ended Wed Nov 16 15:27:31 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:27 |
opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.html | 15:27 |
opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.txt | 15:27 |
opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2022/cinder_bs.2022-11-16-15.03.log.html | 15:27 |
enriquetaso | thank whoami-rajat | 15:27 |
sfv880 | Hello reviewers, Sorry to bother you, I replied and fixed on https://review.opendev.org/c/openstack/cinder/+/850320 and https://review.opendev.org/c/openstack/cinder/+/850321 - could you please review? Thank you so much! | 15:29 |
rosmaita | sfv880: they both add more lines of tests than lines of code, which is a really good thing! | 15:31 |
sfv880 | rosmaita: Thank you! | 15:32 |
*** dviroel is now known as dviroel|lunch | 16:32 | |
*** dviroel|lunch is now known as dviroel | 17:27 | |
*** amoralej is now known as amoralej|off | 17:46 | |
*** dviroel is now known as dviroel|afk | 19:34 | |
opendevreview | Nahim Alves de Souza proposed openstack/cinder master: Netapp ONTAP: Fix volume attachment with multiple HA pairs https://review.opendev.org/c/openstack/cinder/+/863711 | 21:03 |
opendevreview | Merged openstack/devstack-plugin-ceph master: Use default Ubuntu Jammy ceph repos https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/863524 | 21:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!