Tuesday, 2018-10-30

*** _alastor_ has joined #openstack-cinder00:10
*** _alastor_ has quit IRC00:14
*** lixiaoy1 has joined #openstack-cinder00:14
*** mriedem_away has quit IRC00:15
*** Liang__ has joined #openstack-cinder00:19
*** Liang__ is now known as LiangFang00:20
*** daikk115 has quit IRC00:23
*** _alastor_ has joined #openstack-cinder00:52
*** _alastor_ has quit IRC00:56
*** wangy has joined #openstack-cinder01:03
*** bhagyashris_ has joined #openstack-cinder01:04
*** Dinesh_Bhor has joined #openstack-cinder01:12
*** TuanDA has joined #openstack-cinder01:29
*** Dinesh_Bhor has quit IRC02:01
*** Dinesh_Bhor has joined #openstack-cinder02:02
*** wangy has quit IRC02:14
*** mhen has quit IRC02:14
*** wangy has joined #openstack-cinder02:16
*** mhen has joined #openstack-cinder02:19
openstackgerritYong Huang proposed openstack/cinder master: [Unity] Storage-assisted migration support  https://review.openstack.org/61262102:59
openstackgerritwang yong proposed openstack/cinder master: Cinder volume driver for Inspur AS13000 series  https://review.openstack.org/56297703:07
*** psachin has joined #openstack-cinder03:10
openstackgerritYikun Jiang proposed openstack/cinder-specs master: Add volume re-image api  https://review.openstack.org/60531703:20
*** wangy has quit IRC03:21
*** wangy has joined #openstack-cinder03:21
*** Dinesh_Bhor has quit IRC03:38
*** wangy has quit IRC03:39
*** Dinesh_Bhor has joined #openstack-cinder03:41
*** udesale has joined #openstack-cinder03:49
*** bswartz has quit IRC04:11
*** dave-mccowan has quit IRC04:19
*** bswartz has joined #openstack-cinder04:19
*** Dinesh_Bhor has quit IRC04:21
*** TuanDA has quit IRC04:23
*** TuanDA has joined #openstack-cinder04:23
*** sapd1 has quit IRC04:38
*** Dinesh_Bhor has joined #openstack-cinder04:49
*** lpetrut has joined #openstack-cinder04:52
*** sapd1 has joined #openstack-cinder04:59
*** Dinesh_Bhor has quit IRC05:00
*** wangy has joined #openstack-cinder05:03
openstackgerritMerged openstack/cinder master: Update sphinx extension logging  https://review.openstack.org/61115805:12
*** Dinesh_Bhor has joined #openstack-cinder05:15
*** lpetrut has quit IRC05:28
*** pcaruana has joined #openstack-cinder05:36
*** brinzhang has joined #openstack-cinder05:41
*** pcaruana has quit IRC05:47
*** wangy has quit IRC05:59
*** jiapei has left #openstack-cinder06:00
*** wangy has joined #openstack-cinder06:06
*** wangy has quit IRC06:11
*** wxy-xiyuan has quit IRC06:16
*** mnaser has quit IRC06:16
*** wxy-xiyuan has joined #openstack-cinder06:16
*** mnaser has joined #openstack-cinder06:17
*** hoonetorg has quit IRC06:29
*** wangy has joined #openstack-cinder06:42
*** hoonetorg has joined #openstack-cinder06:42
*** tonyb has quit IRC06:50
*** Luzi has joined #openstack-cinder06:52
*** TuanDA has quit IRC07:00
*** TuanDA has joined #openstack-cinder07:01
*** tomtom001 has quit IRC07:02
*** wangy has quit IRC07:04
*** wangy has joined #openstack-cinder07:06
*** abhishekk has joined #openstack-cinder07:10
openstackgerritRajat Dhasmana proposed openstack/cinder master: Improve/Clean api-ref parameters  https://review.openstack.org/61408607:11
*** wangy has quit IRC07:18
*** wangy has joined #openstack-cinder07:20
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/61408707:20
*** dpawlik has joined #openstack-cinder07:26
*** dpawlik has quit IRC07:27
*** dpawlik has joined #openstack-cinder07:27
openstackgerritYikun Jiang proposed openstack/cinder master: Consistent policy name - volume actions  https://review.openstack.org/61410507:39
*** arne_wiebalck has joined #openstack-cinder07:45
*** LiangFang has quit IRC07:45
*** Liang__ has joined #openstack-cinder07:47
*** Liang__ is now known as LiangFang07:51
*** arne_wiebalck has quit IRC07:53
*** arne_wiebalck has joined #openstack-cinder07:54
*** pcaruana has joined #openstack-cinder07:57
*** pcaruana is now known as pcaruana|elisa|07:59
*** gkadam has joined #openstack-cinder08:05
*** sapd1 has quit IRC08:10
*** sapd1 has joined #openstack-cinder08:12
*** Dinesh_Bhor has quit IRC08:16
*** wangy has quit IRC08:20
openstackgerritDai Dang Van proposed openstack/cinder master: Update backup flow for CephBackupDriver  https://review.openstack.org/61064508:28
*** daikk115 has joined #openstack-cinder08:29
*** Dinesh_Bhor has joined #openstack-cinder08:30
*** e0ne has joined #openstack-cinder08:40
*** daikk115 has quit IRC08:41
*** lpetrut has joined #openstack-cinder08:48
openstackgerritinspurericzhang proposed openstack/cinder-tempest-plugin master: [Trivial Fix] update home-page url  https://review.openstack.org/61414808:54
*** lixiaoy1 has quit IRC08:54
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/61408708:58
*** bhagyashris_ has quit IRC09:51
*** Dinesh_Bhor has quit IRC09:55
openstackgerrityenai proposed openstack/cinder master: Rollback the reservation when do retype  https://review.openstack.org/61416109:56
openstackgerritMerged openstack/cinder master: Handle rbd.OSError on broken RBD image  https://review.openstack.org/61274110:11
*** Dinesh_Bhor has joined #openstack-cinder10:15
*** TuanDA has quit IRC10:21
*** abhishekk has quit IRC10:26
*** v12aml has quit IRC10:27
*** luizbag has joined #openstack-cinder10:30
*** e0ne has quit IRC10:32
openstackgerritDmitriy Rabotjagov (noonedeadpunk) proposed openstack/cinder stable/rocky: Handle rbd.OSError on broken RBD image  https://review.openstack.org/61417110:32
*** v12aml has joined #openstack-cinder10:33
openstackgerritArne Wiebalck proposed openstack/cinder master: Support deferred deletion in RBD  https://review.openstack.org/60898410:36
*** dpawlik has quit IRC10:37
*** dpawlik has joined #openstack-cinder10:37
*** icey has quit IRC10:40
*** LiangFang has quit IRC10:42
openstackgerritArne Wiebalck proposed openstack/cinder master: [Trivial] Fix typo in comment in delete_volume  https://review.openstack.org/61418110:48
*** ganso has joined #openstack-cinder10:49
*** luizbag has quit IRC10:49
*** luizbag has joined #openstack-cinder10:50
*** icey has joined #openstack-cinder10:51
*** e0ne has joined #openstack-cinder10:54
*** erlon has joined #openstack-cinder11:04
openstackgerritGorka Eguileor proposed openstack/os-brick master: Context manager to handle shared_targets  https://review.openstack.org/61418911:09
*** gkadam has quit IRC11:12
*** Dinesh_Bhor has quit IRC11:19
*** udesale has quit IRC11:24
*** mvkr has quit IRC11:30
*** lixiaoy1 has joined #openstack-cinder11:50
*** v12aml has quit IRC11:52
*** v12aml has joined #openstack-cinder12:00
*** mvkr has joined #openstack-cinder12:02
*** brinzhang has quit IRC12:07
*** abishop has joined #openstack-cinder12:15
openstackgerritYikun Jiang proposed openstack/cinder master: Consistent policy name - volume actions  https://review.openstack.org/61410512:19
*** e0ne has quit IRC12:21
*** zul has quit IRC12:27
_hemnayough12:48
_hemnasmcginnis: https://review.openstack.org/#/c/613695/12:48
_hemnacan't we just deprecate the old non-python syntax for the goodness function?12:49
_hemnaI really never understood why we needed a new made up syntax, when we have python to use instead.12:49
smcginnis_hemna: Interesting idea. Not sure how much impact that would have on existing deployments.12:49
_hemnawell, deprecating it would still allow it to exist for a time12:49
smcginnis_hemna: Maybe we should add that to the meeting agenda.12:49
_hemnasince that patch adds eval support, why not just start the migration towards it12:50
smcginnis_hemna: I think this patch could also go through as is separately from deciding to deprecate the current syntax.12:50
_hemnait could, but it makes the new config option, yet another option that would need to be deprecated in the future12:51
smcginnisReally the eval usage scares me a little, but I think it's OK the way it was done. But maybe the eval security concerns were the reason for the different syntax before.12:51
_hemnaas it would be the default12:51
smcginnisOh, good point.12:51
smcginnisWe could potentially just drop the config option and just always allow this.12:51
_hemnayah12:52
_hemnathe eval call in that patch limits the things it can execute12:52
_hemnawhich should be the case12:52
smcginnisI didn't know about that capability until this patch, so that did alleviate a lot of my initial security concerns.12:54
*** lpetrut has quit IRC13:05
*** dustins has joined #openstack-cinder13:07
*** e0ne has joined #openstack-cinder13:08
*** udesale has joined #openstack-cinder13:08
*** psachin has quit IRC13:09
*** lpetrut has joined #openstack-cinder13:11
*** mriedem has joined #openstack-cinder13:13
_hemnasmcginnis: https://review.openstack.org/#/c/608988/13:13
_hemnadoes that one change the response code to the client/API ?13:14
_hemnasince we are returning [] and just logging the error now ?13:14
_hemnapreviously, the error would bubble up to the API and out to the client no?13:14
*** tpsilva has joined #openstack-cinder13:15
*** psachin has joined #openstack-cinder13:15
*** Liang__ has joined #openstack-cinder13:20
*** Liang__ is now known as LiangFang13:21
smcginnisYeah, it does change it. So rather than erroring out (which I don't think is desired behavior) it will return an empty list.13:22
smcginnisWhich I think is appropriate for "give me a list of volumes I can manage" for these, because there will never be any volumes you can manage with them.13:23
_hemnaso I guess we are ok with changing the API responses w/o a microversion?13:32
_hemnaunless we consider this a bug?13:32
_hemnaI guess the patch does say it's fixing a bug....13:33
jungleboyjYeah, I was ok with that change.  We have fixed API responses like that before without doing a microversion.  It should be ok to do.13:36
smcginnisYeah, I consider it a bug fix. I don't think we should have ever bubbled up errors to the end user. Makes a less than pleasant user experience I think.13:36
_hemnakewl13:36
_hemnaI +A'd it.13:36
_hemnajust wanted to touch base on that13:36
jungleboyj++ What can I manage?  It isn't supported, so nothing.  :-)13:36
smcginnisCool, thanks. Good to talk through these too.13:37
*** dklyle has quit IRC13:37
*** dklyle has joined #openstack-cinder13:37
smcginnis(and good to see my queue go down) ;)13:37
_hemna:)13:37
jungleboyj:-)  Yeah, I think we need to get back to having more of the quick technical discussions.13:38
smcginnis++13:40
openstackgerritGorka Eguileor proposed openstack/cinder stable/rocky: Fix non iSCSI attach serialization  https://review.openstack.org/61421513:48
openstackgerritErlon R. Cruz proposed openstack/cinder master: NetApp SolidFire: Revert volume to snapshot  https://review.openstack.org/61421713:53
*** wangy has joined #openstack-cinder13:57
*** wangy has quit IRC13:57
openstackgerritSean McGinnis proposed openstack/cinder master: Update driver_opts directive logging  https://review.openstack.org/61063213:59
openstackgerritSean McGinnis proposed openstack/cinder master: Update driver_opts directive logging  https://review.openstack.org/61063214:00
*** dklyle has quit IRC14:02
*** david-lyle has joined #openstack-cinder14:02
jungleboyj_hemna:  You around?14:17
*** itlinux has quit IRC14:20
*** gkadam has joined #openstack-cinder14:21
openstackgerritGorka Eguileor proposed openstack/cinder stable/queens: Fix non iSCSI attach serialization  https://review.openstack.org/61422714:23
*** eharney has joined #openstack-cinder14:28
*** pcaruana|elisa| has quit IRC14:33
*** e0ne has quit IRC14:33
*** e0ne has joined #openstack-cinder14:37
*** psachin has quit IRC14:38
*** jmlowe has quit IRC14:41
*** pcaruana|elisa| has joined #openstack-cinder14:45
openstackgerritMerged openstack/cinder master: Move sphinx-feature-classification to docs reqs  https://review.openstack.org/59830214:50
*** Luzi has quit IRC14:52
*** lixiaoy1 has quit IRC14:59
*** jmlowe has joined #openstack-cinder15:02
*** ianychoi_ is now known as ianychoi15:07
*** itlinux has joined #openstack-cinder15:11
openstackgerritSean McGinnis proposed openstack/cinder master: Remove setup.py check from pep8 job  https://review.openstack.org/61425015:12
openstackgerritMerged openstack/python-cinderclient master: Fix incorrect use of flake8:noqa  https://review.openstack.org/61366015:12
*** e0ne has quit IRC15:25
*** pcaruana|elisa| has quit IRC15:35
*** e0ne has joined #openstack-cinder15:35
*** dpawlik has quit IRC15:43
*** dpawlik has joined #openstack-cinder15:44
*** dpawlik has quit IRC15:49
*** pcaruana|elisa| has joined #openstack-cinder15:50
openstackgerritGorka Eguileor proposed openstack/os-brick master: Context manager to handle shared_targets  https://review.openstack.org/61418915:55
openstackgerritEric Harney proposed openstack/cinder stable/queens: LVM: Disable multiattach for LIO iSCSI target  https://review.openstack.org/61427815:56
*** pcaruana|elisa| has quit IRC16:00
*** _alastor_ has joined #openstack-cinder16:05
*** dustins has quit IRC16:11
*** lpetrut has quit IRC16:16
*** LiangFang has quit IRC16:20
*** dave-mccowan has joined #openstack-cinder16:23
*** e0ne has quit IRC16:30
*** woojay has quit IRC16:31
*** rha has quit IRC16:36
*** rha has joined #openstack-cinder16:39
*** chhagarw has joined #openstack-cinder16:39
*** udesale has quit IRC16:43
openstackgerritArne Wiebalck proposed openstack/cinder master: Support deferred deletion in RBD  https://review.openstack.org/60898416:44
*** mvkr has quit IRC16:55
*** itlinux has quit IRC16:57
*** abishop has quit IRC16:59
*** itlinux has joined #openstack-cinder17:07
*** gkadam has quit IRC17:08
_hemnajungleboyj: hey man what's up17:21
jungleboyj_hemna: I think I got my question answered.17:22
jungleboyjWorking with my co-worker in China to try to get iSCSI support put together for Ceph.17:22
_hemnaah ok17:23
_hemnacoolio17:23
*** lpetrut has joined #openstack-cinder17:23
_hemnajust use SUSE :P17:23
_hemnaSES has all of it built in.....17:23
jungleboyj_hemna:  :p17:23
jungleboyjHe was totally missing the control path in the discussion.17:24
jungleboyjHe started to have me thinking I was wrong but he has caught up with me now.17:24
*** gkadam has joined #openstack-cinder17:29
*** chhagarw has quit IRC17:29
*** abishop has joined #openstack-cinder17:31
*** viveksoni has joined #openstack-cinder17:35
*** gkadam has quit IRC17:40
openstackgerritMerged openstack/cinder master: Remove volume:get policy from test policy file  https://review.openstack.org/60949817:50
openstackgerritMerged openstack/cinder master: Increment versioning with pbr instruction  https://review.openstack.org/61056617:50
*** dpawlik has joined #openstack-cinder18:03
erlonsmcginnis, hey nice cleanup on the specs18:03
erlonmuch clearner list now18:04
jungleboyj:-)  We worked on that together yesterday.  :-)18:04
erlonjungleboyj, ow, nice, so, credits to you as well18:05
jungleboyj:-)  I was up in Minneapolis so we worked on cleaning up things that we could do together.18:05
jungleboyjNice to know it is recognized.18:05
erlonsmcginnis, jungleboyj, definitely, thanks for taking the time for doing that18:06
openstackgerritMerged openstack/cinder master: Remove deprecated iSCSI target config opts  https://review.openstack.org/59531818:08
openstackgerritMerged openstack/cinder master: Ensure VNX unit tests don't sleep  https://review.openstack.org/60048118:08
openstackgerritMerged openstack/cinder master: Remove deprecated HPE Lefthand config opts  https://review.openstack.org/59532818:08
openstackgerritMerged openstack/cinder master: [Trivial] Fix typo in comment in delete_volume  https://review.openstack.org/61418118:08
*** e0ne has joined #openstack-cinder18:10
*** viveksoni has quit IRC18:13
*** dustins has joined #openstack-cinder18:13
*** dpawlik has quit IRC18:20
openstackgerritMerged openstack/cinder master: Handle drivers that do not support list manageable  https://review.openstack.org/60898818:22
*** e0ne has quit IRC18:25
*** itlinux has quit IRC18:29
*** lpetrut has quit IRC18:29
*** dpawlik has joined #openstack-cinder18:36
*** dpawlik has quit IRC18:40
*** woojay has joined #openstack-cinder18:49
smcginniserlon: Haha, thanks. :)18:53
jungleboyj_hemna:  Now this is funny given the article:  https://usercontent.irccloud-cdn.com/file/Wm5tZzAA/image.png18:53
_hemnahehe man18:58
_hemnaI don't envy redhat employees at this point.18:58
jungleboyjMe either.  So many different thoughts and opinions.19:03
jungleboyjThose two companies are going to be watched like hawks.19:03
imacdonnsmcginnis: are you still -1 on https://review.openstack.org/611463 ?19:07
*** jmlowe has quit IRC19:07
smcginnisimacdonn: Yeah, either get rid of the new logging call or get rid of the print. We don't want both.19:11
imacdonnsmcginnis: I disagree19:12
smcginnisimacdonn: Why do we want double output there?19:13
imacdonnsmcginnis: The point of logging call is to dump the stack trace, so you have a chance of being able to figure out what caused the failure .. the print is just stdout progress indication for an interactive command19:13
*** e0ne has joined #openstack-cinder19:13
smcginnisDoesn't change that it's going to output it twice.19:14
imacdonnnote that it doesn't "crash" on exception .. it'll move on and try other migrations19:14
imacdonnit's not going to output it in the same place twice19:14
imacdonnthe log usually goes to, like, /var/log/cinder/cinder-manage.log19:14
smcginnisI still don't think we need it both places, but I'll remove my -1.19:17
imacdonnk, thanks19:18
imacdonnany other cores around to look at this ?19:18
jungleboyjimacdonn:  In a meeting.  Will catch up in a bit.19:19
imacdonnjungleboyj: you already +2'ed it .. but if you have any further opinion.... thanks19:19
jungleboyjimacdonn:  Oh, I guess we are good then since Sean did the +W.  I shall return to reconfiguring my server.19:38
imacdonnjungleboyj: yes, thanks :)19:38
*** jmlowe has joined #openstack-cinder19:48
*** gary_perkins has joined #openstack-cinder19:51
_alastor_Is "goodness" used to determine the probability of placement on a backend or is a higher "goodness" value host going to get all placements until another host has a higher "goodness"?19:57
smcginnis_alastor_: If the weigher doesn't skew one or the other, then yes, I think so.20:04
_alastor_smcginnis: umm, that was a "yes" answer to a "this-or-that" question :P20:06
smcginnisLike the second example here places it based on size of the volume plus the backend capacity - https://docs.openstack.org/cinder/latest/admin/blockstorage-driver-filter-weighing.html#driver-filter-and-weigher-usage-examples20:06
smcginnis_alastor_: I think I'm saying the probabilty is higher, but I'm not sure. I've been awake too long. :)20:07
smcginnis_alastor_: Here's Winston's original blueprint - https://blueprints.launchpad.net/cinder/+spec/filtering-weighing-with-driver-supplied-functions20:08
_alastor_smcginnis: ok, so it seems like the higher number always wins20:09
openstackgerritMerged openstack/cinder master: Restructure Huawei driver  https://review.openstack.org/59902920:11
*** pcaruana|elisa| has joined #openstack-cinder20:12
*** luizbag has quit IRC20:12
*** pcaruana|elisa| has quit IRC20:32
*** imacdonn has quit IRC20:33
*** imacdonn has joined #openstack-cinder20:34
*** mvkr has joined #openstack-cinder20:37
openstackgerritDhinesh Balasubramaniam proposed openstack/cinder master: Hedvig Cinder driver implementation  https://review.openstack.org/43334120:48
*** erlon has quit IRC21:02
*** dustins has quit IRC21:16
*** e0ne has quit IRC21:18
*** eharney has quit IRC21:21
openstackgerritMerged openstack/cinder master: fix misspelling of available  https://review.openstack.org/61394821:24
*** mriedem has quit IRC21:30
openstackgerritArchit Modi proposed openstack/cinder master: Fix a typo in multiattach doc  https://review.openstack.org/61435821:36
*** david-lyle has quit IRC22:08
*** dklyle has joined #openstack-cinder22:28
*** erlon has joined #openstack-cinder22:35
*** e0ne has joined #openstack-cinder22:49
*** tonyb has joined #openstack-cinder23:03
*** prometheanfire has joined #openstack-cinder23:12
prometheanfirenew verions of future seem to break cinder23:12
prometheanfirehttps://review.openstack.org/61431423:13
*** e0ne has quit IRC23:13
*** ganso has quit IRC23:17
*** tpsilva has quit IRC23:18
*** chhagarw has joined #openstack-cinder23:25
*** Liang__ has joined #openstack-cinder23:28
*** faizy_ has joined #openstack-cinder23:28
*** chhagarw has quit IRC23:30
*** Liang__ is now known as LiangFang23:30
*** faizy98 has quit IRC23:31
*** erlon has quit IRC23:35
*** erlon has joined #openstack-cinder23:48
openstackgerritFelipe Monteiro proposed openstack/cinder master: api-ref: Correct response body type for show host details  https://review.openstack.org/61437323:52
*** lixiaoy1 has joined #openstack-cinder23:58

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