Tuesday, 2019-04-16

*** markvoelker has quit IRC00:00
*** tejdeep has quit IRC00:11
*** mszwed has joined #openstack-cinder00:12
*** erlon has joined #openstack-cinder00:28
*** lseki has quit IRC00:37
*** Liang__ has joined #openstack-cinder00:48
*** eharney has quit IRC01:14
*** ruffian_sheep has joined #openstack-cinder01:30
*** baojg has joined #openstack-cinder01:35
*** zhubx has quit IRC01:49
*** zhubx has joined #openstack-cinder01:49
*** whoami-rajat has joined #openstack-cinder01:53
ruffian_sheepmszwed | whoami-rajat:Hi,do u know what is the result the CI should give to community?01:57
ruffian_sheepmszwed | whoami-rajat:http://47.254.90.248/cinder/log.html This one?Or http://47.254.90.248/cinder/tox2.log01:57
*** erlon has quit IRC01:58
ruffian_sheepwhoami-rajat:http://47.254.90.248/cinder/log2.html This is the latest result I run the tempest by setting the  regex pattern with test_*volume*02:28
ruffian_sheepwhoami-rajat:If it's enough to merger my code or I need to pass the rest of them?02:29
*** anks2k has joined #openstack-cinder02:42
*** zhubx has quit IRC02:49
*** boxiang has joined #openstack-cinder02:49
*** baojg has quit IRC02:53
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test for check dependencies between incr backups  https://review.openstack.org/65277103:06
*** enriquetaso has quit IRC03:24
whoami-rajatruffian_sheep: Hi, since i'm not much of an expert in the CI domain, can you attend the cinder weekly meeting on wednesday at 16:00 UTC? you can propose all your doubts there.03:55
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test for check dependencies between incr backups  https://review.openstack.org/65277104:03
*** _erlon_ has quit IRC04:05
*** imacdonn has quit IRC04:06
*** imacdonn has joined #openstack-cinder04:07
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Add test to restore from an incremental backup  https://review.openstack.org/65281704:37
openstackgerritAshish Koushik proposed openstack/cinder master: Add new OSNEXUS Quantastor Cinder driver  https://review.openstack.org/61750304:52
*** e0ne has joined #openstack-cinder04:53
*** e0ne has quit IRC04:56
*** anks2k has quit IRC05:01
*** benj_ has quit IRC05:15
*** e0ne has joined #openstack-cinder05:21
*** e0ne has quit IRC05:25
*** benj_ has joined #openstack-cinder05:26
*** baojg has joined #openstack-cinder05:37
*** Luzi has joined #openstack-cinder05:37
*** lpetrut has joined #openstack-cinder06:02
*** lpetrut has quit IRC06:16
*** lpetrut has joined #openstack-cinder06:16
*** raghavendrat has quit IRC06:23
*** pcaruana has joined #openstack-cinder06:26
*** masayukig has quit IRC06:27
*** kmalloc has quit IRC06:28
*** TheJulia has quit IRC06:29
*** kmalloc has joined #openstack-cinder06:30
*** kmalloc has quit IRC06:40
*** TheJulia has joined #openstack-cinder06:42
*** markvoelker has joined #openstack-cinder06:43
*** benj_ has quit IRC06:44
*** benj_ has joined #openstack-cinder06:45
*** sahid has joined #openstack-cinder06:45
*** TheJulia has quit IRC06:47
*** xyang has quit IRC06:48
*** TheJulia has joined #openstack-cinder06:52
*** xyang has joined #openstack-cinder06:52
*** kmalloc has joined #openstack-cinder06:53
*** luksky has joined #openstack-cinder06:54
*** masayukig has joined #openstack-cinder06:55
*** brinzhang has joined #openstack-cinder06:57
*** e0ne has joined #openstack-cinder07:06
*** e0ne has quit IRC07:09
openstackgerritBrin Zhang proposed openstack/cinder master: Rollback the volume_types table when failed to update quota_usages  https://review.openstack.org/61685007:15
*** e0ne has joined #openstack-cinder07:16
*** rcernin has quit IRC07:20
openstackgerritzhufl proposed openstack/cinder master: Fix missing print format error  https://review.openstack.org/65293307:35
*** helenafm has joined #openstack-cinder07:41
*** e0ne has quit IRC07:52
*** e0ne has joined #openstack-cinder07:58
*** e0ne has quit IRC07:59
*** xyang has quit IRC08:04
*** xyang has joined #openstack-cinder08:05
*** masayukig_ has joined #openstack-cinder08:05
*** masayukig has quit IRC08:05
*** masayukig_ is now known as masayukig08:05
*** phasespace has joined #openstack-cinder08:09
*** gkadam has joined #openstack-cinder08:09
*** gkadam has quit IRC08:10
*** e0ne has joined #openstack-cinder08:10
*** tkajinam has quit IRC08:24
*** anks2k has joined #openstack-cinder08:28
*** luksky has quit IRC08:36
*** luksky has joined #openstack-cinder08:39
anks2kGreetings, I tried adding a new volume driver that is yet to be merged to file “cinder/doc/source/reference/support-matrix.ini” But Zuul reported KeyError: 'driver.quantastor' during feature.implementations method . Would appreciate some direction. https://review.openstack.org/61750308:47
*** luksky has quit IRC09:07
*** luksky has joined #openstack-cinder09:19
*** thgcorrea has quit IRC09:27
*** ruffian_sheep has quit IRC09:31
*** davidsha has joined #openstack-cinder09:38
*** brinzhang has quit IRC10:00
*** rcernin has joined #openstack-cinder10:01
*** luksky has quit IRC10:03
*** Liang__ has quit IRC10:05
*** luksky has joined #openstack-cinder10:33
*** dviroel has joined #openstack-cinder10:42
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax docs - additional information on new and existing functionality  https://review.openstack.org/65155910:53
*** baojg has quit IRC10:56
*** baojg has joined #openstack-cinder10:57
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax docs - add information on new and existing functionality  https://review.openstack.org/65155910:57
*** baojg has quit IRC11:01
*** chhagarw has joined #openstack-cinder11:08
chhagarw@here do we have a limit of max number of concurrent clone operations which can run per volume11:09
whoami-rajatanks2k:  Hi, i've left comments on the patch.11:10
anks2kwhoami-rajat: Thanks, will rebase and continue11:14
*** carloss has joined #openstack-cinder11:16
*** pcaruana has quit IRC11:17
whoami-rajatanks2k: np11:18
*** baojg has joined #openstack-cinder11:23
*** phasespace has quit IRC11:35
*** thgcorrea has joined #openstack-cinder11:48
*** lpetrut has quit IRC11:52
*** erlon has joined #openstack-cinder11:59
*** lpetrut has joined #openstack-cinder12:00
*** sahid has quit IRC12:04
*** pcaruana has joined #openstack-cinder12:05
*** baojg has quit IRC12:05
*** sahid has joined #openstack-cinder12:05
*** gaoyan has joined #openstack-cinder12:21
*** lemko has joined #openstack-cinder12:23
*** Luzi has quit IRC12:24
*** ybunker has joined #openstack-cinder12:39
*** Liang__ has joined #openstack-cinder12:45
*** Luzi has joined #openstack-cinder12:46
*** enriquetaso has joined #openstack-cinder12:49
rosmaitaneed a cinder core who's not sean or rajat to +W https://review.openstack.org/#/c/652717/2 -- fixes the the requirements-check failures: http://zuul.openstack.org/builds?project=openstack%2Fcinder&job_name=requirements-check13:03
*** pcaruana has quit IRC13:07
*** jmlowe has quit IRC13:19
openstackgerritAshish Koushik proposed openstack/cinder master: Add new OSNEXUS Quantastor Cinder driver  https://review.openstack.org/61750313:21
*** mriedem has joined #openstack-cinder13:21
*** eharney has joined #openstack-cinder13:22
smcginnisanks2k: Thanks for getting that added.13:25
*** lseki has joined #openstack-cinder13:25
anks2ksmcginnis: Thanks for your direction. Our CI has also been modified to provide stestr html reports, running python3 and also made implicit mime changes to test artefacts so that you can open in browser from s3 now. Soon, CI would report the latest patch results and also please note, currently integration tests are run using an ipv6 address to show plugin is fully compatible with ipv6.13:28
*** egonzalez has quit IRC13:30
*** egonzalez has joined #openstack-cinder13:30
*** boxiang has quit IRC13:32
*** lennyb has quit IRC13:32
*** pcaruana has joined #openstack-cinder13:33
*** boxiang has joined #openstack-cinder13:33
smcginnisanks2k: Awesome! Thanks for taking care of all that, that's great.13:36
openstackgerritSean McGinnis proposed openstack/cinder master: Filters fetching backups for incremental backup  https://review.openstack.org/48472913:38
*** jmlowe has joined #openstack-cinder13:38
openstackgerritSean McGinnis proposed openstack/cinder master: Drop use of git.openstack.org  https://review.openstack.org/65277413:44
*** lennyb has joined #openstack-cinder13:45
openstackgerritEric Harney proposed openstack/cinder master: Bump hacking version to 1.1.x  https://review.openstack.org/65270313:46
*** sapd1_x has joined #openstack-cinder14:06
*** markvoelker has quit IRC14:07
*** Luzi has quit IRC14:09
*** Liang__ has quit IRC14:17
*** gaoyan has quit IRC14:20
openstackgerritPablo Caruana proposed openstack/cinder stable/rocky: NetApp: Return all iSCSI targets-portals  https://review.openstack.org/65118314:21
*** rcernin has quit IRC14:27
*** dpawlik has quit IRC14:28
*** gaoyan has joined #openstack-cinder14:35
*** baojg has joined #openstack-cinder14:35
*** lpetrut has quit IRC14:36
*** dklyle has quit IRC14:49
*** luksky has quit IRC14:50
*** dklyle has joined #openstack-cinder14:50
*** helenafm has quit IRC14:57
*** anks2k has quit IRC15:06
*** Liang__ has joined #openstack-cinder15:06
*** Liang__ is now known as LiangFang15:08
*** enriquetaso has quit IRC15:09
*** anks2k has joined #openstack-cinder15:11
*** gaoyan has quit IRC15:23
*** pcaruana has quit IRC15:27
*** pcaruana has joined #openstack-cinder15:30
*** e0ne has quit IRC15:31
*** pcaruana has quit IRC15:40
*** sahid has quit IRC15:45
*** pcaruana has joined #openstack-cinder15:46
openstackgerritMerged openstack/cinder master: Fix missing print format error  https://review.openstack.org/65293315:51
gmannit seems display_name is not in valid filter for volume list but it work as DB still has this field https://github.com/openstack/cinder/blob/master/doc/source/admin/generalized_filters.rst15:51
gmannis it for backward compatibility ?15:51
*** pcaruana has quit IRC15:55
*** pcaruana has joined #openstack-cinder16:02
*** ybunker has quit IRC16:07
*** pcaruana has quit IRC16:08
smcginnisgmann: Yeah, display_name was deprecated from v1, but we have handling to accept it in place of "name" in v2 and v3.16:12
gmannsmcginnis: ok. tempest seems still using that, changing it to 'name'16:13
smcginnisgmann: I think so many people still use it that we probably will never fully remove it even though it's been deprecated for years. But probably good if tempest is updated. Thanks for checking on that.16:14
gmannsmcginnis: +1 thanks16:15
*** anks2k has quit IRC16:21
*** LiangFang has quit IRC16:30
whoami-rajatsmcginnis: gmann  i think it isn't properly handled in filtering API. the behavior isn't consistent for admins(allows) and non-admins(returns 400)16:33
whoami-rajatsmcginnis: gmann it should be handled at the API level rather allowing it to the DB IMO.16:34
gmannwhoami-rajat: it is working for admin but did not check for non-admin16:34
whoami-rajatgmann: returning 400 for non-admins as it isn't a valid filter for volumes [1]16:35
whoami-rajat[1] https://github.com/openstack/cinder/blob/master/etc/cinder/resource_filters.json#L2-L416:35
*** pcaruana has joined #openstack-cinder16:36
gmannbut it seems strip-out the invalid filter not 400 - https://github.com/openstack/cinder/blob/7a14b019a5d225d17b929198c6c77bc0352329d7/cinder/utils.py#L73916:36
*** enriquetaso has joined #openstack-cinder16:38
*** ybunker has joined #openstack-cinder16:39
whoami-rajatgmann: that was the functionality before filtering API was introduced. after MV 3.31 it is handled here16:39
whoami-rajathttps://github.com/openstack/cinder/blob/master/cinder/api/common.py#L429-L43616:39
whoami-rajatit will ignore invalid filters if MV<3.3116:41
gmannok16:41
*** anks2k has joined #openstack-cinder16:47
*** markvoelker has joined #openstack-cinder16:52
*** markvoelker has quit IRC16:56
*** anks2k has quit IRC16:56
*** davidsha has quit IRC16:57
hemna_looking through the docs, it doesn't look like we really talk about setting default_volume_type16:59
hemna_even though it's pretty much required16:59
*** itlinux has joined #openstack-cinder16:59
hemna_https://docs.openstack.org/cinder/stein/configuration/index.html17:18
hemna_nothing there for volume types17:18
*** baojg has quit IRC17:24
*** sapd1_x has quit IRC17:26
hemna_dang, I guess we have no documentation on volume types at all17:29
hemna_wtf17:29
hemna_either that or I can't find it17:29
*** e0ne has joined #openstack-cinder17:32
whoami-rajathemna_: the volume type info is divided into several docs as per usage like  in configuring multiple backends ... but yeah i also couldn't find any defining doc of volume types17:32
whoami-rajatbut i found a useful link, they've briefly defined it at netapp's site17:33
whoami-rajathttp://netapp.github.io/openstack-deploy-ops-guide/icehouse/content/section_cinder-key-concepts.html17:33
openstackgerritSean McGinnis proposed openstack/cinder master: Remove 'message' from format argument from exception  https://review.openstack.org/57531817:37
hemna_I couldn't find any useful info on setting them up or why you'd use them, how they work17:38
hemna_as well as setting default_volume_type17:38
hemna_which is basically required17:38
hemna_as upload-to-image is broken w/o it17:38
*** dklyle has quit IRC17:45
*** e0ne has quit IRC17:49
whoami-rajathemna_: hmm, yeah that's important and missing.17:52
*** erlon has quit IRC17:54
*** erlon has joined #openstack-cinder17:56
chhagarwdo we set the default_encoding type for cinder18:04
chhagarwor its imported as default based on the system18:05
*** erlon has quit IRC18:07
*** lemko has quit IRC18:13
*** luksky has joined #openstack-cinder18:21
openstackgerritEric Harney proposed openstack/cinder master: NFS: Log exception when update_migrated_volume fails  https://review.openstack.org/65307418:26
hemna_I guess I can work on a page that talks about volume types18:33
*** irclogbot_2 has quit IRC18:39
*** irclogbot_0 has joined #openstack-cinder18:40
*** chhagarw has quit IRC18:46
openstackgerritSean McGinnis proposed openstack/cinder master: Fix wrong quota deduction when concurrent deletion for None host volume.  https://review.openstack.org/52689319:09
*** enriquetaso has quit IRC19:13
openstackgerritEric Harney proposed openstack/cinder master: NFS: Log exception when update_migrated_volume fails  https://review.openstack.org/65307419:16
*** eharney has quit IRC19:17
*** baojg has joined #openstack-cinder19:25
*** itlinux has quit IRC19:30
*** baojg has quit IRC19:30
*** jmlowe has quit IRC19:33
*** itlinux has joined #openstack-cinder19:34
*** jsquare has quit IRC19:42
*** jsquare has joined #openstack-cinder19:42
*** chhagarw has joined #openstack-cinder19:49
*** ybunker has quit IRC19:53
*** dklyle has joined #openstack-cinder19:58
*** jmlowe has joined #openstack-cinder20:02
*** thgcorrea has quit IRC20:05
*** eharney has joined #openstack-cinder20:14
*** pcaruana has quit IRC20:19
*** mchlumsky_ has quit IRC21:23
*** baojg has joined #openstack-cinder21:26
*** enriquetaso has joined #openstack-cinder21:26
*** chhagarw has quit IRC21:27
*** baojg has quit IRC21:30
openstackgerritSean McGinnis proposed openstack/python-cinderclient master: Remove deprecations and prepare for major release  https://review.openstack.org/60943921:39
*** itlinux has quit IRC21:53
*** itlinux has joined #openstack-cinder22:07
*** luksky has quit IRC22:35
*** rcernin has joined #openstack-cinder22:38
openstackgerritSofia Enriquez proposed openstack/cinder master: Support Incremental Backup Completion In RBD  https://review.openstack.org/62794122:51
*** tkajinam has joined #openstack-cinder22:54
*** whoami-rajat has quit IRC23:02
*** enriquetaso has quit IRC23:05
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Update Brocade FCZM driver's driver options  https://review.openstack.org/64460823:16
openstackgerritWalter A. Boring IV (hemna) proposed openstack/os-brick master: Add generate_connector_list  https://review.openstack.org/64366723:17
*** baojg has joined #openstack-cinder23:27
*** baojg has quit IRC23:31
*** itlinux has quit IRC23:35
*** Liang__ has joined #openstack-cinder23:53

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!