*** senrique__ has quit IRC | 00:05 | |
*** tosky has quit IRC | 00:10 | |
*** senrique__ has joined #openstack-cinder | 00:17 | |
*** zenkuro has quit IRC | 00:25 | |
*** hamalq has quit IRC | 00:26 | |
*** sapd1 has quit IRC | 00:52 | |
openstackgerrit | Merged openstack/cinder master: [IBM DS8000]: Fix compatability issue in get_host https://review.opendev.org/c/openstack/cinder/+/765060 | 00:52 |
---|---|---|
*** n-saito has quit IRC | 01:20 | |
*** lbragstad_ is now known as lbragstad | 01:23 | |
*** ianychoi has joined #openstack-cinder | 01:24 | |
*** ianychoi_ has quit IRC | 01:26 | |
*** ajitha has joined #openstack-cinder | 01:30 | |
*** n-saito has joined #openstack-cinder | 01:41 | |
*** rratnaka has joined #openstack-cinder | 02:01 | |
*** senrique__ has quit IRC | 02:03 | |
openstackgerrit | 江治林 proposed openstack/python-cinderclient master: testtest https://review.opendev.org/c/openstack/python-cinderclient/+/780998 | 02:13 |
*** martinkennelly has quit IRC | 02:21 | |
*** psachin has joined #openstack-cinder | 02:27 | |
openstackgerrit | 江治林 proposed openstack/cinder master: when modify encrypted information of volume type, verify encryption id https://review.opendev.org/c/openstack/cinder/+/781003 | 02:42 |
*** GirishChilukuri has joined #openstack-cinder | 02:46 | |
*** udesale has joined #openstack-cinder | 02:54 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.dell_emc https://review.opendev.org/c/openstack/cinder/+/781006 | 02:56 |
*** priteau has quit IRC | 03:03 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.fusionstorage https://review.opendev.org/c/openstack/cinder/+/781007 | 03:05 |
*** GirishChilukuri has quit IRC | 03:06 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.fusionstorage https://review.opendev.org/c/openstack/cinder/+/781007 | 03:06 |
*** tinwood has quit IRC | 03:11 | |
*** tinwood has joined #openstack-cinder | 03:14 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.ibm https://review.opendev.org/c/openstack/cinder/+/781008 | 03:19 |
*** rcernin has quit IRC | 03:22 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.ibm https://review.opendev.org/c/openstack/cinder/+/781008 | 03:25 |
*** rcernin has joined #openstack-cinder | 03:33 | |
*** rcernin has quit IRC | 03:41 | |
*** rcernin has joined #openstack-cinder | 03:42 | |
*** GirishChilukuri has joined #openstack-cinder | 03:51 | |
openstackgerrit | Ivan Pchelintsev proposed openstack/cinder master: Add iSCSI IPv6 support to Dell EMC XtremIO driver https://review.opendev.org/c/openstack/cinder/+/781009 | 03:52 |
*** vishalmanchanda has joined #openstack-cinder | 04:26 | |
*** m75abrams has joined #openstack-cinder | 04:59 | |
*** knomura has joined #openstack-cinder | 05:05 | |
*** knomura has quit IRC | 05:16 | |
openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [SVF]:Storwize hyperswap volume is not deleting. https://review.opendev.org/c/openstack/cinder/+/771783 | 05:33 |
openstackgerrit | Tushar Trambak Gite proposed openstack/cinder master: Changed whitelist to allowlist in tox https://review.opendev.org/c/openstack/cinder/+/771572 | 05:39 |
*** Luzi has joined #openstack-cinder | 05:44 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.inspur https://review.opendev.org/c/openstack/cinder/+/781012 | 05:47 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.netapp https://review.opendev.org/c/openstack/cinder/+/781016 | 06:02 |
*** dsariel has joined #openstack-cinder | 06:09 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.nexenta https://review.opendev.org/c/openstack/cinder/+/781017 | 06:10 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.solidfire https://review.opendev.org/c/openstack/cinder/+/781018 | 06:13 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.solidfire https://review.opendev.org/c/openstack/cinder/+/781018 | 06:19 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.synology https://review.opendev.org/c/openstack/cinder/+/781022 | 06:25 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.test.unit.volume.drivers.vmware https://review.opendev.org/c/openstack/cinder/+/781023 | 06:28 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six from cinder.transter https://review.opendev.org/c/openstack/cinder/+/781024 | 06:37 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six from cinder.transter https://review.opendev.org/c/openstack/cinder/+/781024 | 06:39 |
*** lpetrut has joined #openstack-cinder | 06:41 | |
openstackgerrit | Merged openstack/cinder master: [DS8K]: Support revert to snapshot https://review.opendev.org/c/openstack/cinder/+/773937 | 06:45 |
openstackgerrit | Tushar Trambak Gite proposed openstack/cinder-specs master: Changed minversion in tox to 3.18.0 https://review.opendev.org/c/openstack/cinder-specs/+/781028 | 07:11 |
*** knomura has joined #openstack-cinder | 07:28 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify/Move project validation methods to api_utils https://review.opendev.org/c/openstack/cinder/+/752431 | 07:29 |
openstackgerrit | Girish Chilukuri proposed openstack/cinder master: [SVF]: Volume name is not validated for host https://review.opendev.org/c/openstack/cinder/+/753134 | 07:30 |
*** happyhemant has joined #openstack-cinder | 07:31 | |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Modify/Move project validation methods to api_utils https://review.opendev.org/c/openstack/cinder/+/752431 | 07:32 |
*** knomura has quit IRC | 07:33 | |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.volume.drivers https://review.opendev.org/c/openstack/cinder/+/781032 | 07:35 |
openstackgerrit | Rajat Dhasmana proposed openstack/cinder master: Add project validation to volume type access https://review.opendev.org/c/openstack/cinder/+/777574 | 07:36 |
openstackgerrit | Xuan Yandong proposed openstack/cinder master: remove six of cinder.volume.drivers https://review.opendev.org/c/openstack/cinder/+/781032 | 07:37 |
*** sfernand has quit IRC | 07:55 | |
*** dsariel has quit IRC | 08:00 | |
*** dsariel has joined #openstack-cinder | 08:01 | |
*** GirishChilukuri_ has joined #openstack-cinder | 08:07 | |
*** GirishChilukuri has quit IRC | 08:10 | |
*** rcernin has quit IRC | 08:11 | |
*** rpittau|afk is now known as rpittau | 08:23 | |
*** rratnaka14 has joined #openstack-cinder | 08:26 | |
rm_work | Anyone know the status on this mTLS change? Just awaiting review? https://review.opendev.org/c/openstack/python-cinderclient/+/776311 | 08:26 |
rm_work | May have missed client releases (last week?) but should still be able to get it in and cut a followup maybe? | 08:27 |
*** rratnaka has quit IRC | 08:29 | |
*** e0ne has joined #openstack-cinder | 08:48 | |
*** knomura has joined #openstack-cinder | 08:50 | |
*** knomura has quit IRC | 08:55 | |
*** knomura has joined #openstack-cinder | 09:01 | |
*** tosky has joined #openstack-cinder | 09:03 | |
*** knomura has quit IRC | 09:06 | |
*** ociuhandu has joined #openstack-cinder | 09:10 | |
*** rcernin has joined #openstack-cinder | 09:17 | |
*** knomura has joined #openstack-cinder | 09:19 | |
*** knomura has quit IRC | 09:23 | |
*** ociuhandu has quit IRC | 09:25 | |
*** knomura has joined #openstack-cinder | 09:29 | |
*** openstackgerrit has quit IRC | 09:33 | |
*** knomura_ has joined #openstack-cinder | 09:34 | |
*** knomura has quit IRC | 09:34 | |
*** knomura_ has quit IRC | 09:34 | |
*** knomura has joined #openstack-cinder | 09:35 | |
*** knomura has quit IRC | 09:36 | |
*** knomura has joined #openstack-cinder | 09:37 | |
*** knomura has quit IRC | 09:38 | |
*** knomura has joined #openstack-cinder | 09:38 | |
*** ociuhandu has joined #openstack-cinder | 09:39 | |
*** knomura_ has joined #openstack-cinder | 09:41 | |
*** knomura has quit IRC | 09:42 | |
*** knomura has joined #openstack-cinder | 09:44 | |
*** knomura_ has quit IRC | 09:46 | |
*** udesale_ has joined #openstack-cinder | 09:46 | |
*** knomura has quit IRC | 09:48 | |
*** rcernin_ has joined #openstack-cinder | 09:48 | |
*** knomura has joined #openstack-cinder | 09:49 | |
*** knomura has quit IRC | 09:53 | |
*** rcernin has quit IRC | 09:55 | |
*** dsariel has quit IRC | 09:55 | |
*** udesale has quit IRC | 09:55 | |
*** ociuhandu has quit IRC | 09:58 | |
*** ociuhandu has joined #openstack-cinder | 10:00 | |
*** ociuhandu has quit IRC | 10:16 | |
*** happyhemant has quit IRC | 10:18 | |
*** knomura has joined #openstack-cinder | 10:22 | |
*** knomura has quit IRC | 10:26 | |
*** ociuhandu has joined #openstack-cinder | 10:27 | |
*** knomura has joined #openstack-cinder | 10:32 | |
*** knomura_ has joined #openstack-cinder | 10:35 | |
*** udesale__ has joined #openstack-cinder | 10:36 | |
*** knomura has quit IRC | 10:37 | |
*** knomura_ has quit IRC | 10:38 | |
*** knomura has joined #openstack-cinder | 10:38 | |
*** udesale_ has quit IRC | 10:38 | |
*** knomura has quit IRC | 10:42 | |
*** knomura has joined #openstack-cinder | 10:43 | |
*** knomura has quit IRC | 10:47 | |
rosmaita | rm_work: we discussed this last week at the cinder meeting and the bug meeting: http://eavesdrop.openstack.org/meetings/cinder/2021/cinder.2021-03-10-14.01.log.html#l-64 and http://eavesdrop.openstack.org/irclogs/%23openstack-cinder/%23openstack-cinder.2021-03-10.log.html#t2021-03-10T15:10:19 | 10:54 |
rosmaita | looks like i forgot to leave a comment on your patch, so apologies for that | 10:54 |
rosmaita | short story is, we would like a spec explaining what you propose and why | 10:54 |
rosmaita | see https://docs.openstack.org/cinder/latest/contributor/contributing.html#new-feature-planning for info about proposing a spec | 10:55 |
*** knomura has joined #openstack-cinder | 10:56 | |
*** knomura has quit IRC | 11:00 | |
*** carloss has joined #openstack-cinder | 11:08 | |
*** knomura has joined #openstack-cinder | 11:13 | |
*** knomura has quit IRC | 11:18 | |
*** zenkuro has joined #openstack-cinder | 11:22 | |
*** knomura has joined #openstack-cinder | 11:24 | |
*** priteau has joined #openstack-cinder | 11:26 | |
*** knomura has quit IRC | 11:28 | |
*** rcernin_ has quit IRC | 11:32 | |
*** manoj_katari has joined #openstack-cinder | 11:35 | |
*** ociuhandu has quit IRC | 11:36 | |
*** ociuhandu has joined #openstack-cinder | 11:37 | |
*** ociuhandu has quit IRC | 11:39 | |
*** ociuhandu has joined #openstack-cinder | 11:39 | |
*** knomura has joined #openstack-cinder | 11:43 | |
*** knomura has quit IRC | 11:47 | |
*** zenkuro has quit IRC | 11:50 | |
*** zenkuro has joined #openstack-cinder | 11:50 | |
GirishChilukuri_ | rosmaita: Addressed review comments for below patches | 11:55 |
GirishChilukuri_ | https://review.opendev.org/c/openstack/cinder/+/753134 | 11:55 |
GirishChilukuri_ | https://review.opendev.org/c/openstack/cinder/+/771783 | 11:55 |
*** takamatsu has quit IRC | 11:57 | |
rosmaita | GirishChilukuri_: ack | 12:13 |
GirishChilukuri_ | rosmaita: Thanks | 12:15 |
*** GirishChilukuri has joined #openstack-cinder | 12:20 | |
*** GirishChilukuri_ has quit IRC | 12:20 | |
*** manoj_katari has quit IRC | 12:23 | |
*** ociuhandu has quit IRC | 12:27 | |
*** GirishChilukuri has quit IRC | 12:27 | |
*** GirishChilukuri has joined #openstack-cinder | 12:30 | |
*** e0ne has quit IRC | 12:35 | |
*** kinpaa12389 has joined #openstack-cinder | 12:44 | |
*** zenkuro has quit IRC | 12:49 | |
*** zenkuro has joined #openstack-cinder | 12:49 | |
*** senrique__ has joined #openstack-cinder | 12:54 | |
*** ociuhandu has joined #openstack-cinder | 12:58 | |
*** ociuhandu has quit IRC | 13:03 | |
*** GirishChilukuri has quit IRC | 13:14 | |
*** GirishChilukuri has joined #openstack-cinder | 13:19 | |
*** knomura has joined #openstack-cinder | 13:28 | |
*** knomura has quit IRC | 13:32 | |
*** almir-okato has joined #openstack-cinder | 13:36 | |
*** sapd1 has joined #openstack-cinder | 13:36 | |
*** knomura has joined #openstack-cinder | 13:44 | |
*** GirishChilukuri has quit IRC | 13:45 | |
*** e0ne has joined #openstack-cinder | 13:46 | |
*** knomura has quit IRC | 13:46 | |
*** GirishChilukuri has joined #openstack-cinder | 13:46 | |
*** knomura has joined #openstack-cinder | 13:46 | |
*** knomura has quit IRC | 13:50 | |
*** openstackgerrit has joined #openstack-cinder | 13:52 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: doc: Remove cryptsetup reference for compute setup https://review.opendev.org/c/openstack/cinder/+/781101 | 13:52 |
*** TusharTgite has joined #openstack-cinder | 13:53 | |
rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:57 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^ | 13:57 |
*** senrique__ is now known as enriquetaso | 13:58 | |
enriquetaso | thanks! | 13:58 |
*** ociuhandu has joined #openstack-cinder | 13:59 | |
*** sfernand has joined #openstack-cinder | 14:00 | |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix automatic quota sync for migrating volumes https://review.opendev.org/c/openstack/cinder/+/778182 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Add quota utils to cinder-manage https://review.opendev.org/c/openstack/cinder/+/776951 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Remove unused quota python classes https://review.opendev.org/c/openstack/cinder/+/778017 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Quota: Fix until_refresh config changes https://review.opendev.org/c/openstack/cinder/+/778210 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Remove nested quota leftovers https://review.opendev.org/c/openstack/cinder/+/779717 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix quota usage duplicate entries https://review.opendev.org/c/openstack/cinder/+/779946 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Fix automatic quota sync for temporary volumes https://review.opendev.org/c/openstack/cinder/+/781104 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Update cinder manage quota commands https://review.opendev.org/c/openstack/cinder/+/781105 | 14:00 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Prevent quota and reservations to go into negative https://review.opendev.org/c/openstack/cinder/+/781106 | 14:01 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Quota: Fix multiple race conditions https://review.opendev.org/c/openstack/cinder/+/781107 | 14:01 |
geguileo | rosmaita: thanks | 14:01 |
rosmaita | i think geguileo has fixed quotas! | 14:01 |
geguileo | rosmaita: I wish!! | 14:02 |
*** knomura has joined #openstack-cinder | 14:06 | |
*** rratnaka14 has quit IRC | 14:08 | |
*** rratnaka has joined #openstack-cinder | 14:09 | |
rosmaita | geguileo: well, at least you are leaving them better off than how you found them | 14:09 |
*** ociuhandu has quit IRC | 14:09 | |
*** knomura has quit IRC | 14:09 | |
*** knomura has joined #openstack-cinder | 14:10 | |
*** ociuhandu has joined #openstack-cinder | 14:11 | |
*** knomura has quit IRC | 14:11 | |
*** knomura has joined #openstack-cinder | 14:11 | |
geguileo | that's the idea | 14:13 |
*** knomura has quit IRC | 14:13 | |
*** knomura_ has joined #openstack-cinder | 14:13 | |
*** ociuhandu has quit IRC | 14:16 | |
*** knomura_ has quit IRC | 14:18 | |
*** ociuhandu has joined #openstack-cinder | 14:25 | |
*** vishalmanchanda has quit IRC | 14:25 | |
openstackgerrit | Merged openstack/cinder master: API validation: Use cinder_host for services checks https://review.opendev.org/c/openstack/cinder/+/777470 | 14:28 |
*** whoami-rajat_ has joined #openstack-cinder | 14:30 | |
*** vishalmanchanda has joined #openstack-cinder | 14:35 | |
*** abhishekk is now known as abhishekk|afk | 14:36 | |
*** rratnaka14 has joined #openstack-cinder | 14:59 | |
sfernand | rosmaita: are we going to have a etherpad with the review priorities for RC1? | 15:01 |
kinpaa12389 | can I continue here ? | 15:01 |
rosmaita | kinpaa12389: wait for the meeting to start | 15:01 |
rosmaita | sfernand: not sure, i have been focused on features | 15:02 |
rosmaita | once the RC is cut, though, we will be looking for release-critical bugs | 15:02 |
*** rratnaka has quit IRC | 15:02 | |
openstackgerrit | Merged openstack/cinder master: [SVF]:Storwize hyperswap volume is not deleting. https://review.opendev.org/c/openstack/cinder/+/771783 | 15:02 |
enriquetaso | hi | 15:02 |
rosmaita | hello | 15:03 |
enriquetaso | sfernand: https://etherpad.opendev.org/p/cinder-wallaby-features | 15:03 |
*** lpetrut has quit IRC | 15:03 | |
enriquetaso | i think that's the etherpad | 15:03 |
openstackgerrit | Merged openstack/cinder master: [SVF]: Volume name is not validated for host https://review.opendev.org/c/openstack/cinder/+/753134 | 15:03 |
rosmaita | i think sfernand is asking about follow-up bugs | 15:03 |
sfernand | I was thinking on a etherpad for follow ups and critical fixes | 15:03 |
sfernand | yep | 15:03 |
enriquetaso | oh, i don't have that etherpad i think... OK, i'm going to start the Cinder Bug meeting | 15:04 |
eharney | we should just use gerrit review-priority for rc1 bugs | 15:04 |
sfernand | just worried with follow ups that we can miss the deadline | 15:04 |
rosmaita | sfernand: that is a good concern | 15:04 |
rosmaita | eharney: i think only cores can set the priority | 15:05 |
rosmaita | i think sfernand wants a place for cores to look to decide if they should up the priority on patches | 15:05 |
eharney | ah | 15:05 |
rosmaita | and people really seem to like working with etherpads | 15:05 |
*** TusharTgite has quit IRC | 15:06 | |
sfernand | yep | 15:06 |
enriquetaso | #startmeeting cinder_bs | 15:06 |
openstack | Meeting started Wed Mar 17 15:06:24 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:06 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:06 |
*** openstack changes topic to " (Meeting topic: cinder_bs)" | 15:06 | |
openstack | The meeting name has been set to 'cinder_bs' | 15:06 |
enriquetaso | #topic roll call | 15:06 |
*** openstack changes topic to "roll call (Meeting topic: cinder_bs)" | 15:06 | |
enriquetaso | hi :P | 15:06 |
kinpaa12389 | hi | 15:06 |
rosmaita | o/ | 15:06 |
enriquetaso | Etherpad: | 15:06 |
enriquetaso | #link https://etherpad.opendev.org/p/cinder-bug-squad-meeting | 15:06 |
enriquetaso | Full list of bugs: | 15:07 |
enriquetaso | #link http://lists.openstack.org/pipermail/openstack-discuss/2021-March/021135.html | 15:07 |
enriquetaso | #topic bug_1:"SADeprecationWarning: The joinedload_all() function is deprecated, and will be removed in a future release. Please use method chaining with joinedload() instead" | 15:07 |
*** openstack changes topic to "bug_1:"SADeprecationWarning: The joinedload_all() function is deprecated, and will be removed in a future release. Please use method chaining with joinedload() instead" (Meeting topic: cinder_bs)" | 15:07 | |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1832164 | 15:08 |
openstack | Launchpad bug 1832164 in Cinder "SADeprecationWarning: The joinedload_all() function is deprecated, and will be removed in a future release. Please use method chaining with joinedload() instead" [Critical,In progress] - Assigned to Gorka Eguileor (gorka) | 15:08 |
*** ociuhandu has quit IRC | 15:08 | |
enriquetaso | wanted to name the bug because it is in critical condition but it is already handled by Gorka. | 15:08 |
zenkuro | rosmaita: thanks for your detailed comments on CI, Im applying fixes right now | 15:08 |
rosmaita | that one illustrates a point eric made last week | 15:08 |
rosmaita | namely, we need to review some old bugs | 15:08 |
enriquetaso | rosmaita++ | 15:09 |
rosmaita | enriquetaso: i can make it un-critical | 15:09 |
enriquetaso | My idea was to do an update of the old bugs but I could not do it for this week, I will prepare it for the next and maybe some for the PTG if they are important. | 15:09 |
rosmaita | yeah, i am not saying that you personally need to do it | 15:09 |
enriquetaso | Fix proposed: | 15:09 |
rosmaita | just that "we" need to do it | 15:09 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/780755 | 15:09 |
enriquetaso | rosmaita sure | 15:09 |
rosmaita | so the story is that sqlalchemy 1.4.0 drops that joinedload_all call | 15:10 |
rosmaita | and 1.4.0 was released on monday | 15:10 |
rosmaita | and our docs job was using it and breaking | 15:10 |
rosmaita | in the mean time, 1.4.0 is not included in upper-constraints | 15:10 |
rosmaita | our doc job was ignoring upper constraints (sort of) | 15:10 |
rosmaita | so, we have fixed our jobs to respect upper constraints | 15:11 |
rosmaita | what i am saying is that it is important that we fix this, because we will be broken in xena | 15:11 |
rosmaita | but it is not as critical as i thought yesterday | 15:11 |
rosmaita | and now i will shut up | 15:11 |
enriquetaso | ohh | 15:11 |
eharney | well, it's a pretty simple change | 15:11 |
enriquetaso | thanks rosmaita for the details \o/ | 15:12 |
enriquetaso | The following bug is already assigned and is being worked on, but I wanted to highlight it in case someone is using quotas. | 15:12 |
enriquetaso | #topic bug_2: "Automatic quota refresh counting temporary volumes" | 15:12 |
*** openstack changes topic to "bug_2: "Automatic quota refresh counting temporary volumes" (Meeting topic: cinder_bs)" | 15:12 | |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1919161 | 15:12 |
openstack | Launchpad bug 1919161 in Cinder "Automatic quota refresh counting temporary volumes" [High,New] - Assigned to Gorka Eguileor (gorka) | 15:13 |
enriquetaso | 'When using the automatic quota refresh via `until_refresh` and `max_age` configuration options the calculated quota usage by the refresh will not be correct if there are temporary volumes (such as those created from snapshots for backups)' | 15:13 |
enriquetaso | I couldn't find a patch for ^ so I think it doesn't have one yet | 15:13 |
enriquetaso | Moving on... I need an opinion on next one: | 15:14 |
enriquetaso | #topic bug_3: "id of encryption for volume type not verified" | 15:14 |
*** openstack changes topic to "bug_3: "id of encryption for volume type not verified" (Meeting topic: cinder_bs)" | 15:14 | |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1918879 | 15:14 |
openstack | Launchpad bug 1918879 in Cinder "id of encryption for volume type not verified" [Undecided,New] | 15:14 |
enriquetaso | The description of the bug just says "when try to update encryption for a given volume type, this api received one parameter is id, but it is not used,therefore, when it is a wrong encryption id, the api layer not check". | 15:14 |
enriquetaso | I guess the bug is incomplete, I'd like to ask about steps of how to reproduce the problem and probably if It's using LUKS or something else. However, I've never updated a volume type before so I'd like to be double sure about what to ask. | 15:14 |
eharney | proably https://review.opendev.org/c/openstack/cinder/+/779436 | 15:14 |
enriquetaso | great | 15:15 |
rosmaita | enriquetaso: i think start by asking exactly what api call, what they are passing in the request, and also what release they are talking about | 15:16 |
enriquetaso | #action enriquetaso: ask for more detailed information | 15:18 |
enriquetaso | next one: | 15:18 |
enriquetaso | #topic bug_4: "ensure_export model_update ignored" | 15:19 |
*** openstack changes topic to "bug_4: "ensure_export model_update ignored" (Meeting topic: cinder_bs)" | 15:19 | |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1918449 | 15:19 |
openstack | Launchpad bug 1918449 in Cinder "ensure_export model_update ignored" [Wishlist,Incomplete] | 15:19 |
enriquetaso | "The volume manager calls the driver's ensure_export() at startup time. There are several volume drivers that return a model_update from ensure_export, but the volume driver ignores that model_update." | 15:19 |
enriquetaso | I set the importance to wishlist, should I change it to medium? | 15:20 |
eharney | this still needs some justification that it is really a bug, and we need a discussion/decision about whether drivers should really be returning updates from ensure_export calls | 15:20 |
eharney | likely to end up as a ptg discussion i guess | 15:21 |
enriquetaso | Yes, please read Eric's comment | 15:21 |
eharney | before we resolve this bug, we really need to resolve: "how exactly does the ensure_export driver API behave?" which seems to be up in the air a bit | 15:21 |
rosmaita | https://github.com/openstack/cinder/blob/master/cinder/volume/driver.py#L1430 | 15:22 |
*** abhishekk|afk is now known as abhishekk | 15:22 | |
rosmaita | so the other ones explicitly say " can optionally return a dict of changes", this one does not | 15:22 |
eharney | right, and we have had code for ages that does not accept a return dict from it | 15:23 |
eharney | so it seems reasonable to start at the idea that it probably shouldn't be updating things that require it to return a dict, etc | 15:23 |
rosmaita | i agree | 15:23 |
eharney | which goes back to the questions i was asking before: why exactly should we _start_ doing that? still unclear | 15:23 |
*** ociuhandu has joined #openstack-cinder | 15:24 | |
enriquetaso | so, it's a good topic for the next PTG | 15:24 |
enriquetaso | OK.. | 15:25 |
enriquetaso | #topic Open Discussion | 15:25 |
*** openstack changes topic to "Open Discussion (Meeting topic: cinder_bs)" | 15:25 | |
kinpaa12389 | https://bugs.launchpad.net/cinder/+bug/1298135 | 15:25 |
openstack | Launchpad bug 1298135 in Cinder "Cinder should handle token expiration for long ops" [Medium,Confirmed] | 15:25 |
kinpaa12389 | can we discuss this ? | 15:25 |
enriquetaso | sure | 15:25 |
enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1298135 | 15:26 |
enriquetaso | "Cinder should handle token expiration for long ops" | 15:26 |
kinpaa12389 | so cinder <-> nova and cinder <-> glance support service_token | 15:26 |
kinpaa12389 | cinder <->swift does not.. | 15:26 |
kinpaa12389 | so I added that support and sent service_token to swift, modified swiftclient to consider session to talk with keystone middleware. | 15:27 |
kinpaa12389 | however this does not work. and token expiry trigger 401 Error. | 15:27 |
eharney | i'd like to understand why it does not | 15:27 |
kinpaa12389 | the swift proxy-server validates token for each backup chunk and it talks with keystone middleware to validate token.. This report failure as invalid token because its expired. | 15:28 |
* enriquetaso likes the tag 'bugsmash' :P | 15:28 | |
*** ociuhandu has quit IRC | 15:28 | |
kinpaa12389 | So, I see two ways to solve it: | 15:29 |
kinpaa12389 | 1) Either patch both Cinder and Swift to support service_auth and its tokens. | 15:29 |
kinpaa12389 | 2) Or use Keystone trust and refresh tokens like Glance already does for long running image uploads. | 15:29 |
eharney | my general sense is that this shouldn't require patching Cinder and Swift | 15:30 |
*** ociuhandu has joined #openstack-cinder | 15:30 | |
kinpaa12389 | eharney can you try this experiment. may be I am doing incorrect steps. | 15:30 |
rosmaita | i'm not sure what the current status of keystone trusts is | 15:30 |
eharney | i'm not sure that keystone trusts are needed for this | 15:30 |
eharney | i don't think we use trusts for service tokens used to talk to nova from cinder, do we? | 15:31 |
kinpaa12389 | no trusts only used for glance <->swift | 15:31 |
kinpaa12389 | its not in cinder | 15:31 |
kinpaa12389 | ok for option 1.. add expiration in keystone.conf e.g. 100 seconds and try to upload backup of 2GB in swift. | 15:32 |
eharney | i think there's more to it than that | 15:32 |
rosmaita | ok, so that's a problem right there | 15:32 |
eharney | i.e. the details in https://docs.openstack.org/cinder/latest/configuration/block-storage/service-token.html | 15:32 |
rosmaita | the service tokens also expire | 15:32 |
kinpaa12389 | right. | 15:32 |
kinpaa12389 | forgot to mention | 15:32 |
kinpaa12389 | u need to add [service_user] section and respective support in cinder/backup/drivers/swift.py | 15:33 |
kinpaa12389 | please update bug if it works for anyone whoever can try. | 15:33 |
enriquetaso | OK, i need to end the meeting | 15:34 |
enriquetaso | the last update was in 2019 | 15:35 |
rosmaita | oh meant to mention this one: https://bugs.launchpad.net/oslo.db/+bug/1814199 | 15:36 |
openstack | Launchpad bug 1814199 in Cinder "soft_delete is wrong" [High,In progress] - Assigned to Gorka Eguileor (gorka) | 15:36 |
enriquetaso | sure | 15:36 |
rosmaita | this is another old one that gorka has a patch up for | 15:36 |
enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/776974 | 15:36 |
rosmaita | so there's probably not much to say about it | 15:37 |
enriquetaso | Fix proposed ^ | 15:37 |
rosmaita | other than i promised gorka that i would review it | 15:37 |
enriquetaso | #action review https://review.opendev.org/c/openstack/cinder/+/776974 | 15:37 |
rosmaita | thanks! | 15:37 |
rosmaita | it's a bit more complicated than that other sqlalchemy patch | 15:37 |
enriquetaso | I'll try to reproduce the bug kinpaa12389, but i'm not familiar with the topic | 15:38 |
*** ociuhandu has quit IRC | 15:38 | |
enriquetaso | rosmaita++ | 15:39 |
rosmaita | kinpaa12389: you may not see much action on that until after RC-time | 15:39 |
rosmaita | we kinda have our hands full at the moment | 15:39 |
enriquetaso | true.. | 15:39 |
enriquetaso | OK, thank you all! | 15:39 |
enriquetaso | #endmeeting | 15:40 |
*** openstack changes topic to "The Block Storage Project | https://wiki.openstack.org/wiki/Cinder | https://tiny.cc/CinderPriorities" | 15:40 | |
openstack | Meeting ended Wed Mar 17 15:40:00 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:40 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_bs/2021/cinder_bs.2021-03-17-15.06.html | 15:40 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_bs/2021/cinder_bs.2021-03-17-15.06.txt | 15:40 |
openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_bs/2021/cinder_bs.2021-03-17-15.06.log.html | 15:40 |
rosmaita | i never answered sfernand's question about feature follow-up bugs | 15:40 |
enriquetaso | :P | 15:40 |
kinpaa12389 | ok | 15:40 |
rosmaita | i think i will take enriquetaso's advice and say add follow=up bugs to https://etherpad.opendev.org/p/cinder-wallaby-features | 15:40 |
sfernand | ok that makes sense | 15:41 |
rosmaita | cool | 15:43 |
*** psachin has quit IRC | 15:51 | |
rosmaita | zenkuro: ack ... feel free to ping me when your CI is ready | 15:52 |
*** josephillips has joined #openstack-cinder | 15:52 | |
*** abishop_ has joined #openstack-cinder | 15:54 | |
*** m75abrams has quit IRC | 15:54 | |
*** abishop has quit IRC | 15:57 | |
*** abishop_ is now known as abishop | 15:58 | |
*** lpetrut has joined #openstack-cinder | 15:59 | |
*** Luzi has quit IRC | 16:03 | |
*** ociuhandu has joined #openstack-cinder | 16:03 | |
*** ociuhandu_ has joined #openstack-cinder | 16:04 | |
*** ociuhandu has quit IRC | 16:04 | |
*** lpetrut has quit IRC | 16:12 | |
*** kinpaa12389 has quit IRC | 16:13 | |
*** hamalq has joined #openstack-cinder | 16:31 | |
*** knomura has joined #openstack-cinder | 16:40 | |
*** rratnaka has joined #openstack-cinder | 16:41 | |
*** rratnaka14 has quit IRC | 16:44 | |
*** knomura has quit IRC | 16:44 | |
openstackgerrit | Merged openstack/os-brick stable/train: Add oslo.context dependency https://review.opendev.org/c/openstack/os-brick/+/772347 | 16:54 |
*** ociuhandu_ has quit IRC | 16:57 | |
*** ociuhandu has joined #openstack-cinder | 16:58 | |
*** ociuhandu has quit IRC | 17:01 | |
*** ociuhandu has joined #openstack-cinder | 17:01 | |
*** rratnaka has quit IRC | 17:03 | |
*** rpittau is now known as rpittau|afk | 17:08 | |
openstackgerrit | Merged openstack/cinder master: NFS: Fix for groups and cloning https://review.opendev.org/c/openstack/cinder/+/780700 | 17:13 |
openstackgerrit | Merged openstack/cinder master: Add QoS Suport for Pure Storage https://review.opendev.org/c/openstack/cinder/+/727267 | 17:14 |
*** knomura has joined #openstack-cinder | 17:16 | |
openstackgerrit | Merged openstack/cinder master: NetApp ONTAP: Implement FlexGroup pool https://review.opendev.org/c/openstack/cinder/+/776713 | 17:16 |
*** dosaboy_ is now known as dosaboy | 17:16 | |
*** knomura has quit IRC | 17:20 | |
*** ociuhandu has quit IRC | 17:23 | |
*** GirishChilukuri has quit IRC | 17:39 | |
*** e0ne has quit IRC | 17:41 | |
*** zenkuro has quit IRC | 17:50 | |
*** zenkuro has joined #openstack-cinder | 17:50 | |
*** udesale__ has quit IRC | 17:59 | |
openstackgerrit | Merged openstack/cinder master: Add support for consistency groups in the Nimble Storage driver https://review.opendev.org/c/openstack/cinder/+/778886 | 18:07 |
openstackgerrit | Merged openstack/cinder master: Resolve SADeprecationWarning for joinedload_all https://review.opendev.org/c/openstack/cinder/+/780755 | 18:08 |
*** whoami-rajat_ is now known as whoami-rajat | 18:18 | |
*** josephillips has quit IRC | 18:49 | |
*** josephillips has joined #openstack-cinder | 18:52 | |
*** rcernin_ has joined #openstack-cinder | 19:26 | |
*** rcernin_ has quit IRC | 19:32 | |
*** vishalmanchanda has quit IRC | 19:35 | |
*** gmann is now known as gmann_afk | 19:41 | |
*** rcernin_ has joined #openstack-cinder | 19:57 | |
*** rcernin_ has quit IRC | 20:02 | |
*** rcernin_ has joined #openstack-cinder | 20:32 | |
*** rcernin_ has quit IRC | 20:37 | |
zenkuro | rosmaita: hi, can you please explain more about conflict related to support-matrix.ini? | 20:39 |
rosmaita | zenkuro: i'm just assuming that's where it will be, because no one else is touching the other files! | 20:39 |
zenkuro | rosmaita: you are saying that there will be a conflict that I will have to resolve once I pass review? | 20:40 |
*** whoami-rajat has quit IRC | 20:40 | |
rosmaita | several people have modified the support matrix, so what's probably happening is that git can't find a common ancestor between your change and theirs, and will want you to fix it manually | 20:40 |
rosmaita | zenkuro: yes, update your local repository and then rebase your change on master | 20:41 |
rosmaita | i think git will tell you there are conflicts in support-matrix.ini that you need to fix | 20:41 |
zenkuro | hm, okey. the final deadline is on the next week right? So I can apply your propositions and work on resolving the conflict | 20:42 |
rosmaita | well, the deadline for this is friday (this friday) | 20:42 |
rosmaita | so resolve the conflict now and git-review this patch | 20:43 |
rosmaita | and the followups can be done next week (before thursday) | 20:43 |
zenkuro | got it | 20:44 |
rosmaita | the key thing is resolving the merge conflict and getting the CI passed | 20:44 |
rosmaita | yeah, hope i wasn't confusing there | 20:44 |
*** rcernin_ has joined #openstack-cinder | 20:50 | |
rm_work | rosmaita: ok, I'll read through those today, I just didn't think this was anything past a bugfix, which last I checked didn't need specs or anything :D but can do that if people think it's a bigger deal | 20:52 |
rosmaita | rm_work: maybe i have you confused with someone else, are you talking about https://review.opendev.org/c/openstack/cinder/+/763760 ? | 20:54 |
rm_work | No | 20:55 |
rm_work | client mTLS bug | 20:55 |
rosmaita | oh, ok | 20:55 |
rm_work | You linked the right conversations | 20:55 |
rm_work | Which I did just read | 20:55 |
rosmaita | gotcha, that was like 8 hours ago, sorry | 20:55 |
rosmaita | yeah, it seems more like a feature than a bugfix | 20:56 |
rm_work | It looks like people are confused? mTLS is already supposed to be supported by everything across all clients. There were like two instances with edge cases, here and manila. We fixed both. | 20:56 |
rm_work | Very minor issues where the client didn't properly pass a couple of vars or didn't use keystoneauth correctly | 20:57 |
rosmaita | do you have the bug link handy? | 20:57 |
rm_work | https://bugs.launchpad.net/python-cinderclient/+bug/1915996 | 20:58 |
openstack | Launchpad bug 1915996 in python-cinderclient "Support passing client certificates for mTLS connections" [Medium,In progress] - Assigned to Sri Harsha mekala (harshayahoo) | 20:58 |
rm_work | it's worth mentioning for context I guess that this is the ONLY outstanding bug with mTLS across all of OSC | 20:59 |
rm_work | the entirety of OSC supports mTLS, and 95% of glanceclient does | 21:00 |
zenkuro | rosmaita: yah, got it. My ci failed on merge. | 21:00 |
rm_work | the issue is in this one spot where someone didn't properly pass the cert var which is expected by keystoneauth1 (and normally handled automagically internally) | 21:01 |
rm_work | I don't quite understand how this would be filed as a "feature", mTLS support was done many cycles ago as part of the keystone-session adoption, just apparently not tested in glanceclient | 21:02 |
*** rcernin_ has quit IRC | 21:04 | |
rm_work | The meat of the change is like 9 lines here: https://review.opendev.org/c/openstack/python-cinderclient/+/776311/18/cinderclient/client.py | 21:05 |
rm_work | which are almost entirely just passing an extra var | 21:05 |
rosmaita | ok, i think we misunderstood what you were proposing | 21:06 |
rm_work | I apologize that I was not able to make it to the meeting originally so I could have cleared this up :P | 21:06 |
rosmaita | that's ok, and to be honest, we are swamped with reviews right now and were looking for any excuse to put another one off to the future | 21:07 |
rm_work | we've tested this internally (where we have turned on mTLS as fully enforced across all API endpoints) and with this patch everything is good | 21:08 |
rm_work | I think as far as reviews go, this is a pretty trivial one, fortunately :P | 21:08 |
*** rcernin_ has joined #openstack-cinder | 21:09 | |
*** rcernin_ has quit IRC | 21:15 | |
rosmaita | i guess the main problem is that the bugs and commit messages talk about not supporting TLS, which i am pretty sure we do, whereas what is missing is mtls, which you want to add | 21:16 |
rosmaita | is that correct? | 21:16 |
rosmaita | zenkuro: yeah, my point was don't freak out if the conflict resolution removes your +2s, it should be an easy re-review | 21:20 |
zenkuro | yh, sure. Any way thanks for explaining. so I will NO try to add some new features in this patch =) | 21:21 |
rosmaita | right! | 21:21 |
rm_work | hmm yeah possibly the original author missed the "m" | 21:32 |
rm_work | the bug report is titled with mTLS tho? | 21:33 |
rm_work | and it says mTLS in the description of the ticket | 21:33 |
rosmaita | rm_work: well, it would help if you could correct those and supply the context that mtls is already supported (almost) everywhere ... was there a community goal or something that we missed? | 21:34 |
rm_work | ^^ so again, having just looked at them, I don't see what you're talking about | 21:35 |
rm_work | mTLS is mentioned as far as I see | 21:35 |
rm_work | https://bugs.launchpad.net/python-cinderclient/+bug/1915996 | 21:35 |
openstack | Launchpad bug 1915996 in python-cinderclient "Support passing client certificates for mTLS connections" [Medium,In progress] - Assigned to Sri Harsha mekala (harshayahoo) | 21:35 |
rm_work | see title ^^ | 21:35 |
rosmaita | sure, and "support x" sounds like a feature | 21:37 |
rosmaita | and "Cinder request to glance does not support TLS" sounds like an incorrect statement | 21:37 |
rm_work | in-so-much as the rest of OSC "supports" mTLS :P | 21:37 |
rm_work | where is that? | 21:38 |
rosmaita | https://bugs.launchpad.net/cinder/+bug/1917797 | 21:38 |
openstack | Launchpad bug 1917797 in Cinder "Cinder request to glance does not support TLS" [Low,Opinion] | 21:38 |
rm_work | hmmm | 21:38 |
rm_work | that's not the bug linked to the patch i mentioned | 21:39 |
rosmaita | https://review.opendev.org/c/openstack/cinder/+/778768 says it closes that bug | 21:40 |
rm_work | this is the patch I am interested in: https://review.opendev.org/c/openstack/python-cinderclient/+/776311 | 21:40 |
rosmaita | maybe i have too many windows open | 21:40 |
rm_work | yeah we may be talking across each other a bit :P | 21:41 |
rm_work | i don't think i was aware there was another issue | 21:41 |
*** rcernin_ has joined #openstack-cinder | 21:41 | |
rm_work | though Hamza (the other bug author) is in my org, lol | 21:41 |
rm_work | let me look into whatever the heck that is | 21:41 |
rosmaita | i guess i found it here: https://review.opendev.org/c/openstack/cinder/+/778768 | 21:42 |
rosmaita | sorry, bad paste | 21:42 |
rosmaita | https://bugs.launchpad.net/python-cinderclient/+bug/1915996/comments/3 | 21:42 |
openstack | Launchpad bug 1915996 in python-cinderclient "Support passing client certificates for mTLS connections" [Medium,In progress] - Assigned to Sri Harsha mekala (harshayahoo) | 21:42 |
rm_work | hmmm | 21:42 |
rm_work | interesting | 21:43 |
rm_work | yeah, it's possible *hamza's* bug is missing the "m" in TLS | 21:43 |
rm_work | obviously it works for https endpoints already | 21:43 |
rm_work | I will ask him to edit the title to say "mTLS" instead of "TLS" because that is wrong | 21:43 |
rm_work | ah I have the ability to fix that | 21:44 |
rm_work | done | 21:44 |
openstackgerrit | Adam Harwell proposed openstack/cinder master: Support mTLS when calling the glance API https://review.opendev.org/c/openstack/cinder/+/778768 | 21:48 |
rm_work | updated the patch title as well | 21:48 |
rm_work | I continue to believe this absolutely qualifies as a *bug*, not a new feature | 21:49 |
rosmaita | yeah, after your explanation, i think you have a point | 21:59 |
rosmaita | so the minor cleanup in the commit messages and bug title should help a lot | 21:59 |
rosmaita | we (or at least me) weren't aware that mTLS was supported almost everywhere in openstack | 22:00 |
rosmaita | so don't worry about the spec, the bugs & patches should be fine | 22:00 |
rm_work | yeah, I hope the tiny patches should speak for themselves now that that's more clear :) | 22:09 |
*** gmann_afk is now known as gmann | 22:39 | |
*** carloss has quit IRC | 23:27 | |
openstackgerrit | Andrei proposed openstack/cinder master: JovianDSS: add certs and snapshot restore https://review.opendev.org/c/openstack/cinder/+/763760 | 23:29 |
*** ajitha has quit IRC | 23:40 | |
*** sapd1 has quit IRC | 23:56 | |
*** hamalq has quit IRC | 23:57 | |
*** hamalq has joined #openstack-cinder | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!