Tuesday, 2021-04-13

« Monday, 2021-04-12 Index Wednesday, 2021-04-14 »
*** mailingsam has quit IRC00:19
*** martinkennelly has quit IRC00:20
*** enriquetaso has joined #openstack-cinder00:28
*** lbragstad_ has joined #openstack-cinder00:43
*** lbragstad has quit IRC00:45
openstackgerritMerged openstack/cinder stable/victoria: LVM: Fix delete volume error due to lvs failure  https://review.opendev.org/c/openstack/cinder/+/78522301:16
*** hamalq has quit IRC01:19
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: [stable-only] Cap bandit and fix lower-constraints  https://review.opendev.org/c/openstack/cinder/+/78597901:31
openstackgerritMerged openstack/cinder master: Backup manager: Synchronously call remove_export_snapshot  https://review.opendev.org/c/openstack/cinder/+/78227101:40
openstackgerritMerged openstack/cinder stable/victoria: Backup manager: Synchronously call remove_export  https://review.opendev.org/c/openstack/cinder/+/78297001:40
openstackgerritMerged openstack/cinder stable/victoria: Tests: Don't assert notifier not called  https://review.opendev.org/c/openstack/cinder/+/78181301:41
*** almir-okato has quit IRC01:41
*** carloss has quit IRC01:43
*** ajitha has joined #openstack-cinder01:57
*** zzzeek has quit IRC02:21
*** zzzeek has joined #openstack-cinder02:23
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: [stable-only] Cap bandit and fix lower-constraints  https://review.opendev.org/c/openstack/cinder/+/78597902:37
*** enriquetaso has quit IRC02:40
*** zenkuro has joined #openstack-cinder02:50
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: [stable-only] Cap bandit and fix lower-constraints  https://review.opendev.org/c/openstack/cinder/+/78597902:54
*** rcernin has quit IRC02:58
*** vishalmanchanda has joined #openstack-cinder03:17
*** rcernin has joined #openstack-cinder03:19
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: [stable-only] Cap bandit and fix lower-constraints  https://review.opendev.org/c/openstack/cinder/+/78597903:25
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: [stable-only] Cap bandit and fix lower-constraints  https://review.opendev.org/c/openstack/cinder/+/78597903:37
*** rcernin has quit IRC03:37
*** rcernin has joined #openstack-cinder03:37
*** zenkuro has quit IRC03:46
*** zenkuro has joined #openstack-cinder03:47
openstackgerritBrian Rosmaita proposed openstack/cinder stable/rocky: Drop lower-constraints job and update test-req  https://review.opendev.org/c/openstack/cinder/+/78569604:06
*** zzzeek has quit IRC04:59
*** zzzeek has joined #openstack-cinder05:00
*** zenkuro has quit IRC05:07
*** zenkuro has joined #openstack-cinder05:08
*** Luzi has joined #openstack-cinder05:50
*** udesale has joined #openstack-cinder05:58
*** zenkuro has quit IRC06:03
*** zenkuro has joined #openstack-cinder06:04
*** rcernin has quit IRC06:38
*** rcernin has joined #openstack-cinder06:38
*** rcernin has quit IRC06:55
*** pcaruana has quit IRC07:11
*** zenkuro has quit IRC07:14
*** rosmaita has quit IRC07:37
*** pcaruana has joined #openstack-cinder07:40
*** rpittau|afk is now known as rpittau07:43
*** tosky has joined #openstack-cinder07:49
*** rosmaita has joined #openstack-cinder07:52
*** ociuhandu has joined #openstack-cinder07:53
*** e0ne has joined #openstack-cinder07:54
*** rcernin has joined #openstack-cinder07:56
*** rcernin has quit IRC08:00
*** rcernin has joined #openstack-cinder08:08
*** martinkennelly has joined #openstack-cinder08:12
*** rcernin has quit IRC08:19
*** rcernin has joined #openstack-cinder08:24
*** dsariel has joined #openstack-cinder08:41
*** ociuhandu has quit IRC08:41
*** ociuhandu has joined #openstack-cinder08:41
*** ociuhandu has quit IRC08:46
*** Luzi has quit IRC08:47
*** Luzi has joined #openstack-cinder08:48
*** ociuhandu has joined #openstack-cinder08:50
*** rcernin has quit IRC08:58
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/ussuri: Add ports filtering support to Dell EMC XtremIO driver  https://review.opendev.org/c/openstack/cinder/+/78386509:13
openstackgerritPavlo Shchelokovskyy proposed openstack/cinder master: Fix instance locality scheduler filter  https://review.opendev.org/c/openstack/cinder/+/78602509:31
*** ociuhandu has quit IRC09:36
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/train: Add ports filtering support to Dell EMC XtremIO driver  https://review.opendev.org/c/openstack/cinder/+/78386609:37
*** ociuhandu has joined #openstack-cinder09:37
openstackgerritIvan Pchelintsev proposed openstack/cinder stable/victoria: Fix PowerStore iSCSI targets filtering  https://review.opendev.org/c/openstack/cinder/+/78393309:40
*** ociuhandu has quit IRC09:42
*** ociuhandu has joined #openstack-cinder09:49
*** rcernin has joined #openstack-cinder09:50
*** rpittau is now known as rpittau|bbl09:54
*** rcernin has quit IRC09:56
*** rcernin has joined #openstack-cinder10:14
*** rcernin has quit IRC10:20
*** zenkuro has joined #openstack-cinder10:26
*** sapd1 has joined #openstack-cinder10:41
*** zenkuro has quit IRC10:51
*** sapd1 has quit IRC11:11
*** tkajinam has quit IRC11:13
*** carloss has joined #openstack-cinder11:15
*** baojg has quit IRC11:16
*** baojg has joined #openstack-cinder11:16
MrClayPoleHi, I'm attempting to sanity check my understanding of how cinder manages iSCSI connections in OpenStack. This is the flow as I understand it. A requested for a new volume comes in through the cinder-api which then passed to the cinder-volume. Cinder-volume then used to storage drive (in our case Zadara) to make any changes required on the iSCSI SAN. Cinder-volume then used os_bricks on the compute node to run the11:20
MrClayPolerequired iscsd/iscsiadm commands to ensure the volume is correctly mapped?11:20
*** ociuhandu has quit IRC11:37
*** ociuhandu has joined #openstack-cinder11:38
*** ociuhandu has quit IRC11:50
*** ociuhandu has joined #openstack-cinder11:50
*** enriquetaso has joined #openstack-cinder11:52
*** ociuhandu has quit IRC11:55
*** ociuhandu has joined #openstack-cinder11:59
*** ociuhandu has quit IRC12:15
*** sfernand has joined #openstack-cinder12:19
*** e0ne has quit IRC12:19
*** rosmaita has left #openstack-cinder12:19
*** ociuhandu has joined #openstack-cinder12:22
*** rosmaita has joined #openstack-cinder12:25
*** rpittau|bbl is now known as rpittau12:31
*** ociuhandu has quit IRC12:35
*** ociuhandu has joined #openstack-cinder12:36
*** ociuhandu has quit IRC12:38
*** ociuhandu has joined #openstack-cinder12:38
openstackgerritBrian Rosmaita proposed openstack/cinder stable/victoria: Drop lower-constraints job  https://review.opendev.org/c/openstack/cinder/+/78570013:00
openstackgerritBrian Rosmaita proposed openstack/cinder stable/train: Drop lower-constraints job  https://review.opendev.org/c/openstack/cinder/+/78569813:04
openstackgerritFernando Ferraz Silva proposed openstack/cinder stable/stein: NetApp SolidFire: Fix clone and request timeout issues  https://review.opendev.org/c/openstack/cinder/+/76494213:06
openstackgerritFernando Ferraz Silva proposed openstack/cinder stable/stein: NetApp SolidFire: Fix clone and request timeout issues  https://review.opendev.org/c/openstack/cinder/+/76494213:10
*** ociuhandu has quit IRC13:11
openstackgerritFernando Ferraz Silva proposed openstack/cinder stable/stein: NetApp SolidFire: Fix error on cluster workload rebalancing  https://review.opendev.org/c/openstack/cinder/+/76427013:11
*** mailingsam has joined #openstack-cinder13:15
*** ociuhandu has joined #openstack-cinder13:18
*** tosky has quit IRC13:19
*** tosky has joined #openstack-cinder13:25
*** ociuhandu has quit IRC13:26
*** tosky has quit IRC13:29
*** tosky has joined #openstack-cinder13:29
*** ociuhandu has joined #openstack-cinder13:39
*** ociuhandu has quit IRC13:44
*** Luzi has quit IRC13:49
openstackgerritAjitha Robert proposed openstack/cinder master: Move nimble driver code to hpe folder  https://review.opendev.org/c/openstack/cinder/+/78605413:53
*** ociuhandu has joined #openstack-cinder13:55
*** lpetrut has joined #openstack-cinder13:56
*** ociuhandu has quit IRC14:03
*** ociuhandu has joined #openstack-cinder14:04
*** ociuhandu has quit IRC14:04
*** ociuhandu has joined #openstack-cinder14:05
*** dsariel has quit IRC14:05
*** dsariel has joined #openstack-cinder14:06
*** jv has quit IRC14:06
*** Luzi has joined #openstack-cinder14:10
openstackgerritCarlos Eduardo proposed openstack/cinder master: [NetApp] Fix iSCSI CHAP auth issue during volume attach  https://review.opendev.org/c/openstack/cinder/+/78586514:14
*** ociuhandu has quit IRC14:20
*** lpetrut has quit IRC14:20
*** ociuhandu has joined #openstack-cinder14:20
*** zenkuro has joined #openstack-cinder14:27
*** lbragstad_ is now known as lbragstad14:31
*** Luzi has quit IRC14:51
*** ociuhandu has quit IRC15:22
*** ociuhandu has joined #openstack-cinder15:22
*** ociuhandu_ has joined #openstack-cinder15:23
*** ociuhandu has quit IRC15:26
*** ociuhandu_ has quit IRC15:28
openstackgerritEric Harney proposed openstack/cinder stable/ussuri: Label temporary files created by image_utils  https://review.opendev.org/c/openstack/cinder/+/78593315:30
hemnaFYi, filed an issue with the k8s csi plugin for cinder.   Train release removed a deprecated ability in the API to filter by name, etc.15:38
hemnaso anyone that calls the api can't do any filtering w/o passing in microversions properly15:38
hemnawhich also includes the openstack client15:38
hemnahttps://github.com/kubernetes/cloud-provider-openstack/issues/148715:38
hemnahttps://github.com/kubernetes/cloud-provider-openstack/pull/148815:38
hemnaI'm not sure its a wise choice to allow removing deprecated APIs until at least the openstack client supports microversions for cinder api15:39
hemnawe found this in our deployments after I upgraded to Train15:39
hemnahttps://github.com/openstack/cinder/commit/ee1ac541253c42e3d94c907cb5b49a1172d68817. that patch broke the filtering for fetching volumes15:40
hemna(for clients w/o microversioning support)15:40
geguileohemna: but that's a configuration option right?  so it should have nothing to do with microversions, right?15:41
geguileoso all that's necessary is to have the right contents in /etc/cinder/resourcefilters.json15:42
geguileosorry, resource_filters.json15:42
hemnabut the default was to not have the columns in the resource_filters.json15:42
hemnaso the upgrade broke clients15:42
geguileowhat I mean to say, is that this is not related to microversions15:43
hemnaname~ was in the resource filters, which should have included name by default, but doesn't15:43
geguileoif the deployment configuration is changed it will work regardless of microversions15:43
hemnawell it is, because if the microversion was passed to support 3.34 it would have worked.15:43
hemnathe deployment configuration didn't change15:43
geguileoand if the configuration is updated, it works regardless15:44
hemnamy point is that the upgrade alone broke the API for clients until we 1) found out it was broken and 2) had to fix the resource_filters.json files in all of our deployments15:44
geguileothe config didn't change, but it should have, because it was using a deprecated thingy15:44
hemnabecause microversions aren't passed by openstack client15:45
hemnaif osc would pass microversions, this wouldn't have broke15:45
geguileotrue15:45
hemnait's specific to this particular call for filtering based on name~15:46
hemnaname fails15:46
hemnaw/o the microversion15:46
hemnaanyway, we 'fixed' it, in our deployment's resource_filters.json after customers found it :(15:46
geguileoand regardless of microversions, if the sys admin updates the configuration file, it also works15:46
geguileoafter all that was deprecated 3 releases before it was removed15:46
hemnaso, the problem is 2 fold.  1) removing deprecated APIs when 2) openstackclient doesn't support microversions.  That resulted in cinder resorting back to an API that didn't support the feature, that used to exist.15:47
hemnawhich begs the question, when a deprecated API is removed, maybe the minimum microversion should be bumped to that version, so the API doesn't break, when it used to work.15:48
hemnaI thought the assumption of the microversioning was to prevent APIs from breaking.15:48
hemnaespecially older versions.15:49
hemnaanyway, the osc is really hurting cinder IMHO15:49
hemnaalmost all of customers use it instead of cinder client, because they use it for other operations.15:49
hemnaand I end up having to scramble to fix previously working APIs :(15:50
*** ociuhandu has joined #openstack-cinder15:50
hemnaanyway, I hope the csi patch will get accepted.  it'll fix k8s at least15:51
*** GirishChilukuri_ has joined #openstack-cinder15:53
*** ociuhandu has quit IRC15:55
*** GirishChilukuri_ has quit IRC15:55
*** GirishChilukuri has joined #openstack-cinder15:56
*** GirishChilukuri_ has joined #openstack-cinder15:58
*** jv has joined #openstack-cinder15:58
*** GirishChilukuri has quit IRC16:02
*** GirishChilukuri_ has quit IRC16:19
*** hamalq has joined #openstack-cinder16:20
*** rpittau is now known as rpittau|afk16:24
*** rcernin has joined #openstack-cinder16:32
*** e0ne has joined #openstack-cinder16:35
*** rcernin has quit IRC16:37
*** e0ne has quit IRC16:38
*** e0ne has joined #openstack-cinder16:42
*** udesale has quit IRC17:15
*** zenkuro has quit IRC17:21
openstackgerritElod Illes proposed openstack/cinder stable/rocky: Cap bandit to v1.6.2 and fix constraints  https://review.opendev.org/c/openstack/cinder/+/76710617:21
*** e0ne has quit IRC17:38
openstackgerritBrian Rosmaita proposed openstack/cinder stable/train: Drop lower-constraints job  https://review.opendev.org/c/openstack/cinder/+/78569818:01
openstackgerritElod Illes proposed openstack/cinder stable/rocky: Cap bandit to v1.6.2 and fix constraints  https://review.opendev.org/c/openstack/cinder/+/76710618:18
*** mailingsam has quit IRC18:51
*** baojg has quit IRC19:13
*** baojg has joined #openstack-cinder19:14
*** ociuhandu has joined #openstack-cinder19:23
*** ajitha has quit IRC19:46
*** vishalmanchanda has quit IRC19:55
*** ociuhandu has quit IRC20:05
*** dsariel has quit IRC21:54
openstackgerritMerged openstack/cinder stable/victoria: PowerMax Driver - Promotion RDF Group number fix  https://review.opendev.org/c/openstack/cinder/+/78330821:57
openstackgerritMerged openstack/cinder stable/ussuri: Tests: Don't assert notifier not called  https://review.opendev.org/c/openstack/cinder/+/78597122:20
*** tkajinam has joined #openstack-cinder22:23
*** fnordahl has quit IRC22:37
*** rcernin has joined #openstack-cinder22:52
*** rcernin has quit IRC22:54
*** rcernin has joined #openstack-cinder22:54
*** tosky has quit IRC23:34
*** ajitha has joined #openstack-cinder23:56
« Monday, 2021-04-12 Index Wednesday, 2021-04-14 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!