Monday, 2018-08-27

« Sunday, 2018-08-26 Index Tuesday, 2018-08-28 »
*** caixiaoyu has joined #openstack-cinder00:44
*** imacdonn has quit IRC01:18
*** imacdonn has joined #openstack-cinder01:18
*** jiaopengju has joined #openstack-cinder01:27
*** brinzhang has joined #openstack-cinder02:10
*** zhaochao has joined #openstack-cinder02:19
*** mmethot has quit IRC02:20
*** psachin has joined #openstack-cinder02:46
*** whoami-rajat has joined #openstack-cinder03:04
*** ircuser-1 has joined #openstack-cinder03:18
*** gkadam has joined #openstack-cinder03:41
*** Dinesh_Bhor has joined #openstack-cinder03:50
*** Dinesh_Bhor has quit IRC04:00
openstackgerritVivek Soni proposed openstack/cinder master: 3PAR: Added retries on volume deletion  https://review.openstack.org/58743804:05
*** Dinesh_Bhor has joined #openstack-cinder04:54
*** abhishekk has joined #openstack-cinder05:11
*** zhaochao has quit IRC05:19
*** zhaochao has joined #openstack-cinder05:20
*** e0ne has joined #openstack-cinder05:34
*** v12aml has quit IRC06:05
*** v12aml has joined #openstack-cinder06:09
*** dpawlik has joined #openstack-cinder06:10
*** dpawlik has quit IRC06:11
*** dpawlik_ has joined #openstack-cinder06:11
*** e0ne has quit IRC06:16
*** Bhujay has joined #openstack-cinder06:34
*** Bhujay has quit IRC06:40
*** pcaruana has joined #openstack-cinder06:49
*** Bhujay has joined #openstack-cinder06:49
*** Bhujay has quit IRC07:10
*** Dinesh_Bhor has quit IRC07:13
caixiaoyuHi tommylikehu, would you please help me review BP https://blueprints.launchpad.net/cinder/+spec/cinder-list-improvement .07:15
*** Bhujay has joined #openstack-cinder07:18
tommylikehucaixiaoyu:  we won't accept new individual filter parameters, all new filters will come in the format of "cinder list --filters xxxx=xxxxx"07:23
*** alexchadin has joined #openstack-cinder07:24
tommylikehuand for the cases of snapshot and source volume, i guess it already works if you list volume with command: cinder list --filters snapshot/source_volid=xxxx?07:26
tommylikehus/snapshot/snapshot_id07:26
*** rcernin has quit IRC07:31
caixiaoyuSorry, I didn't find "cinder list --filters". Is it a BP?07:33
caixiaoyu@ tommylikehu07:34
tommylikehuno it's in code already try the cinderclient07:34
caixiaoyuOh, I get it. Thank you.07:51
*** Dinesh_Bhor has joined #openstack-cinder07:55
openstackgerritMatan Sabag proposed openstack/cinder master: Fixed invalid number of arguments bug in ScaleIO driver  https://review.openstack.org/59665807:59
openstackgerritSam Wan proposed openstack/cinder master: _is_volume_creation_safe doesn't need 'provisioning'. Fixes bug 1789167  https://review.openstack.org/59665908:02
openstackbug 1789167 in Cinder "ScaleIO _is_volume_creation_safe should not input provisioning parameter" [Undecided,New] https://launchpad.net/bugs/178916708:02
openstackgerritziyu proposed openstack/cinder master: Correct the link of 'OpenStack Documentation Contributor Guide'  https://review.openstack.org/59660208:12
*** josecastroleon has joined #openstack-cinder08:27
*** josecastroleon has quit IRC08:30
*** lbirkhahn has joined #openstack-cinder08:37
openstackgerritwangxiyuan proposed openstack/cinder master: Remove auth_uri usage  https://review.openstack.org/59666808:41
*** e0ne has joined #openstack-cinder08:48
*** Bhujay has quit IRC08:48
*** Dinesh_Bhor has quit IRC09:07
*** Dinesh_Bhor has joined #openstack-cinder09:15
*** Luzi has joined #openstack-cinder09:19
*** vivsoni_ has quit IRC09:19
*** Bhujay has joined #openstack-cinder09:21
*** alexchadin has quit IRC09:43
*** lennyb_ has joined #openstack-cinder09:44
*** lennyb_ is now known as lennyb09:45
*** vivsoni has joined #openstack-cinder09:52
openstackgerritwangxiyuan proposed openstack/cinder master: Remove auth_uri usage  https://review.openstack.org/59666809:53
*** caixiaoyu has quit IRC09:56
*** Dinesh_Bhor has quit IRC10:08
*** Dinesh_Bhor has joined #openstack-cinder10:22
*** abhishekk has quit IRC10:30
*** pcaruana has quit IRC10:32
*** pcaruana has joined #openstack-cinder10:32
*** erlon has joined #openstack-cinder10:48
*** Dinesh_Bhor has quit IRC10:58
*** nicolasbock has joined #openstack-cinder11:13
*** gkadam has quit IRC11:23
*** gkadam has joined #openstack-cinder11:23
*** luizbag has joined #openstack-cinder11:26
*** jroll has quit IRC11:43
*** rtjure has quit IRC11:44
*** jroll has joined #openstack-cinder11:44
*** rtjure has joined #openstack-cinder11:58
*** vivsoni has quit IRC12:01
*** brinzh has joined #openstack-cinder12:03
*** brinzhang has quit IRC12:06
*** vivsoni has joined #openstack-cinder12:06
openstackgerritwangxiyuan proposed openstack/cinder master: Remove auth_uri usage  https://review.openstack.org/59666812:09
*** luizbag has quit IRC12:10
*** luizbag has joined #openstack-cinder12:11
*** tpsilva has joined #openstack-cinder12:13
*** alexchadin has joined #openstack-cinder12:18
*** luizbag has quit IRC12:18
*** luizbag has joined #openstack-cinder12:19
*** luizbag has quit IRC12:32
*** brinzh has quit IRC12:32
*** luizbag has joined #openstack-cinder12:33
*** luizbag has quit IRC12:33
*** luizbag has joined #openstack-cinder12:34
*** rtjure has quit IRC12:43
*** rtjure has joined #openstack-cinder12:48
*** gkadam has quit IRC12:51
*** alexchadin has quit IRC12:51
*** dustins has joined #openstack-cinder12:53
*** moshele has joined #openstack-cinder12:54
*** mriedem has joined #openstack-cinder12:55
*** cmart has joined #openstack-cinder13:09
*** mchlumsky has joined #openstack-cinder13:16
*** alexchadin has joined #openstack-cinder13:18
*** yumiriam has joined #openstack-cinder13:23
*** Luzi has quit IRC13:31
*** moshele has quit IRC13:40
*** moshele has joined #openstack-cinder13:41
*** eharney has joined #openstack-cinder13:43
*** alexchadin has quit IRC13:43
*** alexchadin has joined #openstack-cinder13:44
*** psachin has quit IRC13:50
openstackgerritChuck Short proposed openstack/os-brick master: Replace assertRaisesRegexp with assertRaisesRegex  https://review.openstack.org/59673613:53
*** mchlumsky has quit IRC13:55
*** mchlumsky has joined #openstack-cinder13:57
*** r-daneel has joined #openstack-cinder14:05
*** alexchadin has quit IRC14:11
*** gnufied has joined #openstack-cinder14:14
*** zhaochao has quit IRC14:17
*** r-daneel_ has joined #openstack-cinder14:24
*** r-daneel has quit IRC14:24
*** r-daneel_ is now known as r-daneel14:24
openstackgerritClaudiu Belu proposed openstack/cinder master: WIP: Adds mock autospec  https://review.openstack.org/58066314:25
luizbagdoes someone know why os-brick tox is using pylint 0.26.0?14:28
e0neluizbag: hi. does this version have any issues?14:28
luizbagit fails the pylint when base python is version 314:30
eharneyluizbag: some changes are needed to update it to a current version -- i started this but didn't finish it14:30
eharneyhttps://review.openstack.org/#/c/442857/14:30
smcginnisI think we need to raise that higher now like what was done on the cinder repo.14:35
smcginnisThough there are still issues there. I haven't been able to figure out why we get random failures where it sees new pylint errors on code that hasn't changed.14:36
*** ianychoi has quit IRC14:36
eharneyoh yeah, i forgot we did that in cinder already14:38
luizbagI'll bump pylint version on os-brick to 1.4.3 as well and test it, I'll use eharney's patch as a base for mine. Ok?14:41
eharneyluizbag: there should be a newer one that would be a better start14:41
luizbagthe latest version is 2.1.114:42
smcginnisIIRC, we need 2.0+ to work right with python 3.14:42
*** ianychoi has joined #openstack-cinder14:42
luizbagok, 2.1.1 here we go!14:43
smcginnisOh wait, in cinder we raised to 1.9.1. I think it was actually 1.8+ that was needed, but if we're going to raise it now, probably good to go with the newest.14:43
*** dpawlik_ has quit IRC14:46
openstackgerrithongwei.xu proposed openstack/cinder master: Extract user_id from context as the operator id  https://review.openstack.org/59412414:47
*** alexchadin has joined #openstack-cinder14:49
openstackgerrithongwei.xu proposed openstack/cinder master: Extract user_id from context as the operator id  https://review.openstack.org/59412414:49
*** eharney has quit IRC14:54
*** eharney has joined #openstack-cinder14:55
*** markvoelker has joined #openstack-cinder14:55
*** gcb_ has joined #openstack-cinder14:56
*** Bhujay has quit IRC15:02
smcginnisluizbag: If you look into raising pylint, looks like lintstack.py needs to be updated for the Run call. Apparently with the 2.0 release they changed the "exit=False" to "do_exit=False", just for funsies.15:04
*** r-daneel_ has joined #openstack-cinder15:17
*** r-daneel has quit IRC15:17
*** r-daneel_ is now known as r-daneel15:17
*** e0ne has quit IRC15:19
*** markstur has joined #openstack-cinder15:23
openstackgerritSean McGinnis proposed openstack/cinder master: Raise pylint to 2.1.1  https://review.openstack.org/59681015:25
*** pcaruana has quit IRC15:29
*** dpawlik has joined #openstack-cinder15:30
*** alexchadin has quit IRC15:32
*** dpawlik has quit IRC15:34
*** markvoelker has quit IRC15:36
*** Bhujay has joined #openstack-cinder15:38
openstackgerritHelen Walsh proposed openstack/cinder stable/queens: VMAX doc - important known issue  https://review.openstack.org/59681915:38
*** gcb_ has quit IRC15:48
*** nicolasbock has quit IRC15:49
*** Bhujay has quit IRC15:49
smcginnisjungleboyj, eharney: I'd like to get this through - https://review.openstack.org/#/c/593192/15:55
smcginnisAnd then backport that through the stable branches since it really doesn't need to be running there.15:55
openstackgerritJay Bryant proposed openstack/cinder master: Remove cinder-tox-compliance job  https://review.openstack.org/59319215:59
smcginnisHah, ok.16:00
*** markstur has quit IRC16:05
openstackgerritSean McGinnis proposed openstack/os-brick stable/queens: FC Allow for multipath volumes with different LUNs  https://review.openstack.org/59450116:16
*** eharney_ has joined #openstack-cinder16:18
*** eharney has quit IRC16:18
*** eharney_ is now known as eharney16:18
*** pcaruana has joined #openstack-cinder16:26
*** itlinux has joined #openstack-cinder16:29
kukacz_hi, is it a bad idea to assign "service" project as cinder_internal_tenant_project_id for the purpose of image caching? as cinder_internal_tenant_user_id I am planning to use "cinder"16:33
eharneyhttps://docs.openstack.org/cinder/latest/admin/blockstorage-image-volume-cache.html claims that the service project can be used16:37
kukacz_eharney: you mean this sentence?: "They can be the Block Storage service project ..."16:40
eharneyyes16:40
kukacz_I was reading that as some (hypothetical) "cinder" project, which I don't have16:40
*** e0ne has joined #openstack-cinder16:41
kukacz_it's enough for me to ensure there are no risks related to using the "service" project specifically16:41
kukacz_cause I had never used it for anything such yet16:42
*** Bhujay has joined #openstack-cinder17:00
*** openstackgerrit has quit IRC17:04
*** Bhujay has quit IRC17:05
kukacz_eharney: thank you17:08
itlinuxkukacz_: pure flash storage suggestion is to use that.. I have that setup on my env.17:11
kukacz_itlinux: you mean the combination of "service" project and "cinder" user?17:12
itlinuxno it's a new user and a project17:12
itlinuxwhich is used for cache17:13
itlinuxyou have to setup that id in cinder.conf17:13
kukacz_aha, I was trying to avoid creating another project + user. we already have these unused projects as "admin" and "service" so my idea was to use them if that's safe17:15
*** gnufied has quit IRC17:15
itlinuxI would not use those! that's my personal suggestion.17:18
itlinuxit should work by any mean.17:18
kukacz_yes, it works, I've already enabled it. just wonder what might be the specific risks related to using "service"17:25
kukacz_RedHat guide states "The administrator should configure the new internal cinder tenant manually for image cache ..."17:27
kukacz_I'm not using RedHat distribution, was just googling for more opinions17:28
kukacz_itlinux: thank you for help!17:28
*** dpawlik has joined #openstack-cinder17:47
*** dpawlik has quit IRC17:51
*** r-daneel has quit IRC18:04
*** r-daneel has joined #openstack-cinder18:12
*** openstackgerrit has joined #openstack-cinder18:16
openstackgerritMerged openstack/cinder master: Fixed invalid number of arguments bug in ScaleIO driver  https://review.openstack.org/59665818:16
openstackgerritMerged openstack/cinder master: Handle multiattach attribute when managing volumes  https://review.openstack.org/58675018:16
openstackgerritSean McGinnis proposed openstack/cinder stable/rocky: Fixed invalid number of arguments bug in ScaleIO driver  https://review.openstack.org/59685118:22
*** gnufied has joined #openstack-cinder18:32
itlinuxit does not matter if you are using Red Hat. this is tight to cinder kukacz_:18:32
*** _hemna has quit IRC18:36
*** itlinux has quit IRC18:54
*** eharney has quit IRC18:57
*** itlinux has joined #openstack-cinder19:11
*** pcaruana has quit IRC19:13
*** itlinux has quit IRC19:16
tpsilva@smcginnis are bugfixes already allowed to merge?19:24
*** e0ne has quit IRC19:27
smcginnistpsilva: Bug fixes are always welcome.19:28
smcginnistpsilva: If they are for stable/rocky, they just won't be able to get released until rocky is officially over and we can do our first stable release.19:28
tpsilva@smcginnis so if it merges, I'll need to backport to stable/rocky?19:28
*** e0ne has joined #openstack-cinder19:29
tpsilva@smcginnis I'm asking because we have a somewhat urgent fix that need to be backported to stable/queens19:29
smcginnistpsilva: Yes, if you are proposing a bug fix to master and it's something that needs to also be addressed in rocky, then it will need to get backported to stable/rocky after it merges to master.19:29
smcginnisThen after stable/rocky, then get backported to stable/queens.19:30
*** _hemna has joined #openstack-cinder19:30
tpsilva@smcginnis understood19:30
tpsilva@smcginnis here's the fix: https://review.openstack.org/#/c/596297/19:30
smcginnistpsilva: Cool, I'll take a look.19:30
tpsilva@smcginnis thank you!19:30
smcginnistpsilva: Is the cDOT NetApp CI the one for DataOntap?19:31
tpsilva@smcginnis yes19:31
openstackgerritMerged openstack/cinder master: Remove flashsystem_multipath_enabled opt  https://review.openstack.org/59534019:40
*** e0ne has quit IRC19:44
openstackgerritSean McGinnis proposed openstack/cinder stable/queens: ScaleIO: Prevent usage of unsafe volumes  https://review.openstack.org/59687919:46
openstackgerritAlyson proposed openstack/cinder master: Fix IPv6 for Cinder NetApp ONTAP drivers  https://review.openstack.org/59629719:48
*** eharney has joined #openstack-cinder19:59
*** moshele has quit IRC20:00
*** e0ne has joined #openstack-cinder20:00
*** nicolasbock has joined #openstack-cinder20:30
*** e0ne has quit IRC20:37
*** luizbag has quit IRC20:37
openstackgerritClaudiu Belu proposed openstack/cinder master: WIP: Adds mock autospec  https://review.openstack.org/58066320:46
*** r-daneel_ has joined #openstack-cinder20:57
*** r-daneel has quit IRC20:59
*** r-daneel_ is now known as r-daneel20:59
*** erlon has quit IRC21:02
*** markvoelker has joined #openstack-cinder21:04
*** markvoelker has quit IRC21:13
*** dustins has quit IRC21:24
*** rcernin has joined #openstack-cinder21:51
*** mriedem has quit IRC21:52
openstackgerritTommyLike proposed openstack/cinder master: Update doc for get-pools API  https://review.openstack.org/59187822:38
*** nicolasbock has quit IRC22:39
*** nicolasbock has joined #openstack-cinder22:44
*** cmart has quit IRC23:04
*** nicolasbock has quit IRC23:07
*** r-daneel has quit IRC23:08
*** nwonknu has quit IRC23:23
*** Tahvok_ has joined #openstack-cinder23:24
*** rha has quit IRC23:24
*** Tahvok has quit IRC23:24
*** Tahvok_ is now known as Tahvok23:24
*** zul has quit IRC23:29
*** nwonknu has joined #openstack-cinder23:32
*** tpsilva has quit IRC23:39
*** dpawlik has joined #openstack-cinder23:47
*** dpawlik has quit IRC23:52
« Sunday, 2018-08-26 Index Tuesday, 2018-08-28 »

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