Tuesday, 2021-03-02

openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057700:12
enriquetasoxonsh looks cool, I'll try it00:15
*** tosky has quit IRC00:17
*** martinkennelly has quit IRC00:32
*** martinkennelly has joined #openstack-cinder00:36
*** hamalq has quit IRC00:41
*** martinkennelly has quit IRC00:46
*** martinkennelly has joined #openstack-cinder00:47
openstackgerritSofia Enriquez proposed openstack/cinder master: [doc] remove outdated package python-keystone  https://review.opendev.org/c/openstack/cinder/+/77811900:51
*** martinkennelly has quit IRC00:55
*** martinkennelly has joined #openstack-cinder00:56
*** jv_ has quit IRC01:07
*** n-saito has quit IRC01:17
*** n-saito has joined #openstack-cinder01:18
*** carloss has quit IRC01:42
*** n-saito has quit IRC01:42
*** n-saito has joined #openstack-cinder01:44
*** zzzeek has quit IRC01:49
*** zzzeek has joined #openstack-cinder01:51
*** martinkennelly has quit IRC01:57
rm_workThis bug is causing us some amount of consternation by the way, fix looks pretty simple if the review could get some eyes: https://review.opendev.org/c/openstack/python-cinderclient/+/77631102:17
rm_workthanks!02:17
*** rcernin has quit IRC02:34
*** zenkuro has quit IRC02:36
*** rcernin has joined #openstack-cinder02:47
openstackgerritLance Bragstad proposed openstack/cinder master: Implement project personas for volume action API  https://review.opendev.org/c/openstack/cinder/+/76019202:56
openstackgerritXuan Yandong proposed openstack/os-brick master: Remove six  https://review.opendev.org/c/openstack/os-brick/+/75459802:56
*** swegener has quit IRC03:04
openstackgerritLance Bragstad proposed openstack/cinder master: Implement project personas for volume API  https://review.opendev.org/c/openstack/cinder/+/76019903:20
*** vishalmanchanda has joined #openstack-cinder03:44
*** psachin has joined #openstack-cinder03:46
*** zzzeek has quit IRC04:32
*** zzzeek has joined #openstack-cinder04:33
*** venkatakrishnath has joined #openstack-cinder04:38
*** enriquetaso has quit IRC04:48
*** enriquetaso has joined #openstack-cinder04:49
*** udesale has joined #openstack-cinder05:08
*** enriquetaso has quit IRC05:11
*** whoami-rajat_ has joined #openstack-cinder05:25
*** m75abrams has joined #openstack-cinder05:25
*** brinzhang_ has quit IRC06:01
*** brinzhang_ has joined #openstack-cinder06:01
*** lbragstad_ has joined #openstack-cinder06:03
*** lbragstad has quit IRC06:06
*** zzzeek has quit IRC06:10
*** zzzeek has joined #openstack-cinder06:11
*** dsariel has joined #openstack-cinder06:12
openstackgerritXuan Yandong proposed openstack/os-brick master: Remove six  https://review.opendev.org/c/openstack/os-brick/+/75459806:21
*** whoami-rajat_ is now known as whoami-rajat06:54
*** rcernin has quit IRC06:57
*** m75abrams has quit IRC07:02
*** felipe_rodrigues has quit IRC07:17
*** jawad_axd has joined #openstack-cinder07:25
*** lpetrut has joined #openstack-cinder07:27
*** rpittau|afk is now known as rpittau08:22
*** tosky has joined #openstack-cinder08:35
openstackgerritRajat Dhasmana proposed openstack/cinder master: Refactor: move project validation to api_utils  https://review.opendev.org/c/openstack/cinder/+/75243108:42
*** ociuhandu has joined #openstack-cinder08:44
*** martinkennelly has joined #openstack-cinder08:50
openstackgerritXuan Yandong proposed openstack/os-brick master: Remove six  https://review.opendev.org/c/openstack/os-brick/+/75459809:03
*** GirishChilukuri has joined #openstack-cinder09:05
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057709:05
*** brinzhang0 has joined #openstack-cinder09:15
*** brinzhang_ has quit IRC09:18
*** zenkuro has joined #openstack-cinder09:31
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add project validation to volume type access  https://review.opendev.org/c/openstack/cinder/+/77757409:38
fricklerwhoami-rajat: seems your fix for https://bugs.launchpad.net/cinder/+bug/1753885 was incomplete, need to insert a "filters.pop('all_tenants')" here https://review.opendev.org/c/openstack/cinder/+/641717/6/cinder/api/common.py#b43009:43
openstackLaunchpad bug 1753885 in Cinder "invalid filter option "all_tenants" for non-aministrator user" [Critical,Fix released] - Assigned to Rajat Dhasmana (whoami-rajat)09:43
whoami-rajatfrickler: IIRC this method is only for validating the filters and not modifying them, we remove them at later stage,  are you facing any error related to this?09:47
*** dsariel has quit IRC09:47
*** dsariel has joined #openstack-cinder09:47
*** ociuhandu has quit IRC09:50
*** e0ne has joined #openstack-cinder10:02
*** e0ne has quit IRC10:02
*** zoharm has joined #openstack-cinder10:06
*** ociuhandu has joined #openstack-cinder10:06
fricklerwhoami-rajat: yes, "cinder --os-volume-api-version 3.30 show v1" works, "cinder --os-volume-api-version 3.31 show v1" fails. cinder-api log shows that the all_tenants key is removed in the earlier version but not in the later one10:13
frickleras a side effect, also a command like "openstack volume set" only works with an ID and not with a name when requiring 3.42 to allow resizing an attached volume10:15
*** udesale_ has joined #openstack-cinder10:15
whoami-rajatfrickler: this shouldn't be related to my patch since it's not bumping microversion, let me check10:15
fricklerwhoami-rajat: without your patch, api returns an error. with your patch, api returns an empty list, still not what is needed10:17
*** udesale has quit IRC10:19
whoami-rajatfrickler: as i can see the checks are all over the place, i will push a patch to make it consistent (and working)10:29
whoami-rajatthanks for reporting10:29
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add project validation to volume type access  https://review.opendev.org/c/openstack/cinder/+/77757410:48
*** m75abrams has joined #openstack-cinder10:51
*** carloss has joined #openstack-cinder11:01
openstackgerritZohar Mamedov proposed openstack/os-brick master: NVMeOF connector driver connection information compatibility fix  https://review.opendev.org/c/openstack/os-brick/+/77708611:14
*** psachin has quit IRC11:22
*** udesale_ has quit IRC11:40
*** GirishChilukuri has quit IRC11:42
*** udesale has joined #openstack-cinder11:43
*** GirishChilukuri has joined #openstack-cinder11:45
*** ociuhandu has quit IRC11:48
GirishChilukuri@core reviewers: Please review this code changes, Need +2 to merge the code changes.11:49
GirishChilukurihttps://review.opendev.org/c/openstack/cinder/+/77393711:49
GirishChilukurihttps://review.opendev.org/c/openstack/cinder/+/75313411:49
GirishChilukurihttps://review.opendev.org/c/openstack/cinder/+/77178311:49
GirishChilukurihttps://review.opendev.org/c/openstack/cinder/+/76506011:49
*** hoonetorg has quit IRC11:49
*** ociuhandu has joined #openstack-cinder11:50
*** ociuhandu has quit IRC11:50
*** ociuhandu has joined #openstack-cinder11:52
*** ociuhandu has quit IRC11:57
*** Luzi has joined #openstack-cinder11:58
*** hoonetorg has joined #openstack-cinder12:03
*** ociuhandu has joined #openstack-cinder12:12
*** ociuhandu has quit IRC12:17
*** ociuhandu has joined #openstack-cinder12:18
*** zzzeek has quit IRC12:20
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix automatic quota refresh for migrating volumes  https://review.opendev.org/c/openstack/cinder/+/77818212:22
*** ociuhandu has quit IRC12:22
*** zzzeek has joined #openstack-cinder12:23
*** GirishChilukuri has quit IRC12:25
openstackgerritMerged openstack/os-brick stable/victoria: FC: Fix not flushing on detach  https://review.opendev.org/c/openstack/os-brick/+/77709212:32
*** ociuhandu has joined #openstack-cinder12:35
*** enriquetaso has joined #openstack-cinder12:36
*** ociuhandu has quit IRC12:44
*** jv_ has joined #openstack-cinder13:05
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057713:18
*** ociuhandu has joined #openstack-cinder13:27
*** lbragstad_ is now known as lbragstad13:41
*** dsariel has quit IRC13:52
openstackgerritEric Harney proposed openstack/cinder master: mypy: generate stubs for imported libs  https://review.opendev.org/c/openstack/cinder/+/77500714:13
openstackgerritTushar Trambak Gite proposed openstack/cinder master: Reset state robustification for backup os-reset_status  https://review.opendev.org/c/openstack/cinder/+/77819314:19
openstackgerritLuigi Toscano proposed openstack/cinder-tempest-plugin master: Adds new scenario test for revert to snapshot  https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/62057714:29
*** dsariel has joined #openstack-cinder14:30
*** jawad_axd has quit IRC14:31
*** jawad_axd has joined #openstack-cinder14:32
*** Luzi has quit IRC14:36
whoami-rajatrosmaita: hey, these changes were not in the original tracker doc (not even added later) but this one seems important to get into victoria https://review.opendev.org/c/openstack/os-brick/+/77621614:44
whoami-rajatnot sure about this one https://review.opendev.org/c/openstack/python-cinderclient/+/77668814:45
*** atmark has joined #openstack-cinder14:55
atmarkhello, is it possible to configure to have volumes backed by rbd for set of computes and volumes backed by local disk for another set of computes?14:57
*** jawad_axd has quit IRC15:15
*** jawad_axd has joined #openstack-cinder15:16
*** jawad_axd has quit IRC15:24
*** zenkuro has quit IRC15:27
openstackgerritMerged openstack/cinder master: Add ports filtering support to Dell EMC XtremIO driver  https://review.opendev.org/c/openstack/cinder/+/77579815:30
*** lpetrut has quit IRC15:34
*** openstackgerrit has quit IRC15:35
*** openstackgerrit has joined #openstack-cinder15:41
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix until_refresh config changes  https://review.opendev.org/c/openstack/cinder/+/77821015:41
openstackgerritGorka Eguileor proposed openstack/cinder master: Fix until_refresh config changes  https://review.opendev.org/c/openstack/cinder/+/77821015:46
*** venkatakrishnath has quit IRC15:57
openstackgerritZohar Mamedov proposed openstack/os-brick master: NVMeOF connector driver connection information compatibility fix  https://review.opendev.org/c/openstack/os-brick/+/77708616:09
*** dsariel has quit IRC16:15
*** dviroel has quit IRC16:34
*** dsariel has joined #openstack-cinder16:35
*** e0ne has joined #openstack-cinder16:37
whoami-rajatjungleboyj:  smcginnis hey, can you take a look at this backport? it's good enhancement to have in victoria https://review.opendev.org/c/openstack/python-cinderclient/+/77668816:42
hemnawhat is the default timeout for our rpc api calls ?16:43
smcginniswhoami-rajat: "backport" and "enhancement" usuallly don't go together. ;) But I'll take a look after this call.16:43
jungleboyjsmcginnis:  Bwah ha ha.16:44
openstackgerritEric Harney proposed openstack/cinder master: Add warning message about slow volume backend  https://review.opendev.org/c/openstack/cinder/+/76675116:44
rosmaitahemna: i think 60 sec16:45
hemnaok16:45
jungleboyjrosmaita: So you are +2 on this:  https://review.opendev.org/c/openstack/python-cinderclient/+/77668816:46
whoami-rajatsmcginnis: yeah, not sure what to call it since it can't be disguised as a bugfix :/ thanks!16:47
hemnarosmaita ok I see the setting in oslo_messaging defaulting to 60 seconds16:47
hemnahttps://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/rpc/client.py#L39-L4116:47
rosmaitacool16:47
rosmaitajungleboyj: yes, but i'm only 1 vote16:47
rosmaitaand good thing smcginnis took a look16:48
jungleboyjYeah, now that I read that ... not ok.16:50
whoami-rajatyeah, it's not clear with the code but the v3 method is overriding the v2 one so we need to include --volume-id as a parameter in the v3 method as well16:52
jungleboyjwhoami-rajat:  So, I can see that the fact that this wasn't implemented was kind of a gap.16:56
whoami-rajatjungleboyj:  since '--volume-id' was deprecated in v2 also and '--volume' was a recommended way of doing, i think it's safe to not include that option in v3 but for a backport, that might cause inconsistencies16:59
jungleboyjRight.  Making that kind of a change in a backport isn't allowed.16:59
whoami-rajatyep, agreed17:00
smcginnisYeah, not allowed with stable policy to remove functionality on a stable release. And current release will need to be a major version bump. (hopefully that hasn't already gone out otherwise)17:00
*** m75abrams has quit IRC17:01
smcginnisRelease note on the original patch also should have called that out.17:01
whoami-rajatwas thinking the same thing ^^17:02
whoami-rajatabishop: ^^^17:03
*** udesale has quit IRC17:04
*** zoharm has quit IRC17:04
hemnaam getting an rpc timeout doing a backup, due to cinder's default behavior of creating a temporary clone of the volume being backed up, and that cloning operation taking longer than the default rpc timeout (which we have modified to 300 seconds)17:08
hemnahttps://github.com/openstack/cinder/blob/master/cinder/volume/rpcapi.py#L343-L35217:09
hemnathat guy takes too long to return17:09
abishopwhoami-rajat: not much to add, other than "ack"17:09
hemnasome clone operations are painful for backends17:09
hemnaand take a long time depending on the size of the volume being cloned17:09
*** ociuhandu_ has joined #openstack-cinder17:09
*** ociuhandu has quit IRC17:12
smcginnisWould love to get this in so I can automate the CI report generation: https://review.opendev.org/c/openstack/cinder/+/77645817:12
*** ociuhandu_ has quit IRC17:13
jungleboyj+2 from me.  :-)17:16
smcginnisThanks!17:16
*** lpetrut has joined #openstack-cinder17:28
openstackgerritMerged openstack/os-brick stable/victoria: ScaleIO: More connection info backward compatibility  https://review.opendev.org/c/openstack/os-brick/+/77621617:28
openstackgerritAlan Bishop proposed openstack/os-brick stable/ussuri: ScaleIO: More connection info backward compatibility  https://review.opendev.org/c/openstack/os-brick/+/77822417:41
*** rpittau is now known as rpittau|afk17:42
jbernardhemna: for the backend in question, is a clone necessary to perform the backup operation?17:43
jbernardhemna: perhaps that behaviour could be selectable for backends that struggle in the clone department17:44
*** tobias-urdin has quit IRC18:02
openstackgerritMerged openstack/cinder master: PowerMax Driver - Port status check  https://review.opendev.org/c/openstack/cinder/+/76164318:04
*** lpetrut has quit IRC18:07
*** hamalq has joined #openstack-cinder18:38
openstackgerritRatnakaram Rajesh proposed openstack/cinder master: Update code layout and missing Zadara features  https://review.opendev.org/c/openstack/cinder/+/77446319:01
*** rratnaka has joined #openstack-cinder19:03
*** rratnaka has quit IRC19:11
legochenhey team, I enabled the - image_volume_cache_enabled = True . I can see the cinder/volume/manager.py load this attribute properly after inject the dump codes there.20:06
legochenax_cache_entries: 020:06
legochen /opt/openstack/venv/cinder/lib/python3.6/site-packages/cinder/volume/manager.py:__init__:302 image_volume_cache_enabled: True, max_cache_size: 0, m20:06
legochenax_cache_entries: 020:06
legochenbut, don’t know why it doesn’t work in cinder/volume/flows/manager/create_volume.py20:08
legochenhttps://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L101220:08
legochenI also dump the variables’ values there - 2021-03-02 20:04:33,790 INFO [cinder.volume.flows.manager.create_volume] /opt/openstack/venv/cinder/lib/python3.6/site-packages/cinder/volume/flows/manager/create_volume.py:_create_from_image:1019 cloned: False , self.image_volume_cache:20:09
legochenNone , volume_is_encrypted: False20:09
legochenyou can see the self.image_volume_cache is set to None20:09
legochenrosmaita ^^ could you please give me some guidance about this :)20:10
rosmaitalegochen: will take a look in a bit20:11
legochenI also dump the value after this line. https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L38920:12
legochenthe result is NONE as well.20:12
rosmaitain the meantime, eric was wondering about that config problem you found yesterday ... did you define those config options in the [DEFAULT] section of the file?  that might be why they weren't working ... he thinks they should be defined in the backend's part of the config file, or in with the [backend_defaults}20:13
legochen this is my command to trigger this code - openstack volume create --type ScaleIO --size 5 --image rhel-7.9.11 bootable-img-120:13
legochenyou mean yesterday’s bug? I defined all them in [DEFAULT].20:14
rosmaitaok, that might be the problem, then20:15
legochenrosmaita - https://access.redhat.com/documentation/en-us/red_hat_openstack_platform/16.1/html/configuration_reference/cinder20:17
legochenI refer to this doc and see both “image_conversion_address_space_limit” and “image_conversion_cpu_limit” defined in [DEFAULT] section.20:18
legochenbtw, is there a group “[backend_defaults]” can use?20:19
legochenI’m okay to define  “image_conversion_address_space_limit” and “image_conversion_cpu_limit”  to my all backends and see if that could work.20:19
rosmaitaif you add a [backend_defaults] section, you can put options there that are shared by multiple backends20:20
legochenrosmatia, about the today’s issue - image_volume_cache_enabled . I defined this attribute in [DEFAULT] and all my backends, but, it doesn’t work as expected. I think the CONF load is good. Just don’t know why the “_create_from_image” function thinks the cache is not enabled.20:22
legochenokay, thanks for that. will try :)20:22
rosmaitalegochen: i think the image_volume_cache_enabled option is a per-backend option, so cinder won't find it in the [DEFAULT] section20:24
*** Underknowledge has quit IRC20:26
atmarkhello, is it possible to configure to have volumes backed by rbd specific set of computes and volumes backed by local disk for another set of computes?20:26
*** Underknowledge has joined #openstack-cinder20:26
legochenrosmaita - https://docs.openstack.org/cinder/rocky/configuration/block-storage/samples/cinder.conf.html20:26
legochenas the sample cinder.conf, it mentions image_volume_cache_enabled in [DEFAULT] section.20:27
legochenrosmaita, I moved  “image_conversion_address_space_limit” and “image_conversion_cpu_limit” to [backend_defaults] and all my backend sections…. howerever, it still doesn’t work20:30
rosmaitalegochen: thanks for trying that ... when you have a few minutes, please put a comment on the bug saying that you tried it20:31
rosmaita(will save someone else some time)20:31
legochensure, let me do it just now20:31
rosmaitathanks!20:31
rosmaitalegochen: so about the image_volume_cache_enabled setting ... it also shows up later in that sample config in the [backend_defaults] setting20:32
rosmaitai think the setting in [DEFAULT] is leftover from when you could only have one backend with cinder20:33
rosmaita(which was before i started working on it, so i'm not sure)20:33
rosmaitait is definitely confusing, though20:33
rosmaitaatmark: what time zone are you in?20:36
legochenrosmaita, looks like after I moved image_volume_cache_enabled to [backend_defaults], it works fine now20:39
rosmaita\o/20:39
rosmaitalegochen: you might want to file a bug about that, because we need to fix the docs or the sample config generation or something ... it's definitely confusing20:41
legochensure, will do. let me validate more about that first.20:42
rosmaitathanks!20:43
atmarkrosmaita: EST20:46
rosmaitaatmark: the agenda for tomorrow's cinder meeting at 1400-1500 UTC is pretty light, why don't you put it on the agenda and you can ask at the meeting?20:47
rosmaitayou may be able to do something with availability zones, but i am really not sure20:47
rosmaitahttps://etherpad.opendev.org/p/cinder-wallaby-meetings20:47
*** e0ne has quit IRC20:55
*** dviroel has joined #openstack-cinder20:55
*** abhishekk has quit IRC21:06
*** bhagyashri|rover has quit IRC21:06
*** abhishekk has joined #openstack-cinder21:06
*** bhagyashris has joined #openstack-cinder21:07
*** dsariel has quit IRC21:07
openstackgerritMerged openstack/cinder master: Fix CI_WIKI_NAME entries  https://review.opendev.org/c/openstack/cinder/+/77645821:17
openstackgerritMerged openstack/cinder stable/ussuri: Require oslo.serialization 3.1.2  https://review.opendev.org/c/openstack/cinder/+/77468021:17
openstackgerritMerged openstack/cinder stable/ussuri: [SVF]:Reduce slowness by caching pool information  https://review.opendev.org/c/openstack/cinder/+/76747821:18
*** hoonetorg has quit IRC21:21
legochenromaita, one more question about the output from openstack volume show <volume-id>.   I’d like to know where to configure the format of “os-vol-host-attr:host”.21:40
legochenin my use case, it shows -21:41
legochen| os-vol-host-attr:host          | my-host1.test.com@vxflexos-ssd#pd1:pd1-ssd21:41
legochenanother volume shows:21:41
legochen| os-vol-host-attr:host          | my-host2.test.com@vxflexos-ssd#pd1:pd1-ssd21:41
legochenlooks like the format is <hostname>@<backend_name>#<pool_name>21:42
legochenI’m curious is this defined by backend driver or can customize by admins?21:42
*** hoonetorg has joined #openstack-cinder21:42
legochenas I have multiple cinder api servers behind LB.21:43
openstackgerritMerged openstack/cinder stable/ussuri: RBD: Pass bytes type for mon_command inbuf  https://review.opendev.org/c/openstack/cinder/+/77754521:43
legochenso, the cinder API requests will dispatch to different api services.21:43
legochenI’m thinking can I let the host portion of “os-vol-host-attr:host   “ could be my LB CNAME instead of “hostname” of server who handling the request21:45
*** brinzhang0 has quit IRC21:47
*** rcernin has joined #openstack-cinder22:33
*** enriquetaso has quit IRC22:39
*** vishalmanchanda has quit IRC22:54
*** enriquetaso has joined #openstack-cinder23:10
*** zzzeek has quit IRC23:13
*** zzzeek has joined #openstack-cinder23:17
*** carloss has quit IRC23:40

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