Thursday, 2019-01-31

« Wednesday, 2019-01-30 Index Friday, 2019-02-01 »
*** imacdonn has quit IRC00:00
*** imacdonn has joined #openstack-cinder00:01
*** TxGirlGeek has quit IRC00:01
*** openstackgerrit has joined #openstack-cinder00:05
openstackgerritChris M proposed openstack/cinder master: Declare multiattach support for HPE MSA  https://review.openstack.org/63170200:05
*** sdake has joined #openstack-cinder00:06
*** sdake has quit IRC00:29
*** _alastor_ has quit IRC00:46
*** tosky has quit IRC00:46
*** sdake has joined #openstack-cinder00:50
*** khomesh|brb is now known as khomesh00:55
openstackgerritSean McGinnis proposed openstack/python-cinderclient master: Fix max version handling for help output  https://review.openstack.org/63402201:00
*** _alastor_ has joined #openstack-cinder01:04
*** khomesh is now known as khomesh|brb01:08
*** sdake has quit IRC01:09
*** bhagyashris has joined #openstack-cinder01:14
*** khomesh|brb is now known as khomesh01:17
*** doubletao has joined #openstack-cinder01:21
*** Dinesh_Bhor has joined #openstack-cinder01:48
*** TxGirlGeek has joined #openstack-cinder02:02
*** mriedem has quit IRC02:04
*** whoami-rajat has joined #openstack-cinder02:11
*** dims has quit IRC02:38
openstackgerritSofia Enriquez proposed openstack/cinder stable/ocata: ScaleIO Driver - adding cache and refactoring tests  https://review.openstack.org/63357002:38
*** dims has joined #openstack-cinder02:55
*** sapd1 has joined #openstack-cinder03:00
*** sdake has joined #openstack-cinder03:00
*** psachin has joined #openstack-cinder03:01
openstackgerritMerged openstack/cinder master: Move Fujitsu ETERNUS DX related file  https://review.openstack.org/63112103:02
*** khomesh is now known as khomesh|bfst03:04
openstackgerritMerged openstack/cinder master: fast8 improvements  https://review.openstack.org/63254003:06
*** mmethot has quit IRC03:14
*** enriquetaso has quit IRC03:15
*** mmethot has joined #openstack-cinder03:21
*** _alastor_ has quit IRC03:27
*** bhagyashris has quit IRC03:34
*** gkadam-bmgr has joined #openstack-cinder03:34
*** khomesh|bfst is now known as khomesh03:39
*** abhishekk has joined #openstack-cinder03:51
*** _alastor_ has joined #openstack-cinder03:54
*** udesale has joined #openstack-cinder03:55
*** udesale has quit IRC03:56
*** TxGirlGeek has quit IRC04:18
*** udesale has joined #openstack-cinder04:28
*** khomesh is now known as khomesh|brb04:50
*** _alastor_ has quit IRC05:08
*** doubletao has quit IRC05:11
*** khomesh|brb is now known as khomesh05:21
*** _alastor_ has joined #openstack-cinder05:25
*** sapd1 has quit IRC05:43
*** _alastor_ has quit IRC06:06
*** e0ne has joined #openstack-cinder06:12
*** sapd1 has joined #openstack-cinder06:30
*** zzzeek has quit IRC06:33
*** zzzeek has joined #openstack-cinder06:37
*** avishay has joined #openstack-cinder06:51
*** e0ne has quit IRC07:05
*** Luzi has joined #openstack-cinder07:06
*** markvoelker has joined #openstack-cinder07:26
avishaygeguileo: Hey!  I just tested the WWN validation patch.  The failed attach does in fact clean up the leftover device, and then retrying the connection works.07:30
*** avishay has quit IRC07:39
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR : Creation of volume from snapshot  https://review.openstack.org/63411907:42
*** takamatsu has joined #openstack-cinder07:46
*** markvoelker has quit IRC07:59
*** bhagyashris has joined #openstack-cinder08:08
*** tkajinam has quit IRC08:14
*** udesale has quit IRC08:14
*** udesale has joined #openstack-cinder08:15
*** phuongnh has joined #openstack-cinder08:18
*** sapd1 has quit IRC08:22
*** tosky has joined #openstack-cinder08:34
TheJuliajungleboyj: Any ideas why someone might see this? Seems odd to me that all of a sudden something has changed. https://review.openstack.org/#/c/63345908:39
*** Dinesh_Bhor has quit IRC08:44
*** Dinesh_Bhor has joined #openstack-cinder08:44
whoami-rajatrosmaita:  could you explain why the change was done in glance https://review.openstack.org/#/c/579335/ ? couldn't find anything on the bug or patch.08:48
*** markvoelker has joined #openstack-cinder08:56
*** e0ne has joined #openstack-cinder09:15
openstackgerritYachika_Ralhan proposed openstack/cinder master: Added new OSNEXUS Quantastor Cinder driver  https://review.openstack.org/61750309:28
*** markvoelker has quit IRC09:29
*** bhagyashris has quit IRC09:32
*** psachin has quit IRC09:40
*** luizbag has joined #openstack-cinder09:42
*** yachika has joined #openstack-cinder09:49
whoami-rajatTheJulia:  The code is failing here [1] while validating the schema for attachment [2].09:53
whoami-rajat[1] https://github.com/openstack/cinder/blob/master/cinder/api/contrib/volume_actions.py#L5409:53
whoami-rajat[2] https://github.com/openstack/cinder/blob/master/cinder/api/schemas/volume_actions.py#L53-L5409:53
*** sapd1 has joined #openstack-cinder09:55
*** ganso has joined #openstack-cinder09:55
*** erlon has joined #openstack-cinder10:10
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Publish backup capabilities to a scheduler  https://review.openstack.org/62088110:23
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Create backups via scheduler  https://review.openstack.org/63030510:23
openstackgerritIvan Kolodyazhny proposed openstack/cinder master: Create backups via scheduler  https://review.openstack.org/63030510:24
*** markvoelker has joined #openstack-cinder10:26
*** jistr is now known as jistr|chat10:31
*** abhishekk has quit IRC10:32
*** e0ne has quit IRC10:46
openstackgerritSilvan Kaiser proposed openstack/cinder master: [WIP] Add context to cloning snapshots in remotefs driver  https://review.openstack.org/57088510:47
*** avishay has joined #openstack-cinder10:50
*** e0ne has joined #openstack-cinder10:52
*** Dinesh_Bhor has quit IRC10:54
openstackgerritSilvan Kaiser proposed openstack/cinder master: [WIP] Add context to cloning snapshots in remotefs driver  https://review.openstack.org/57088510:54
*** markvoelker has quit IRC10:59
*** avishay has quit IRC11:02
TheJuliawhoami-rajat: I see, looks like that line was added 10 months ago. Yay for quietly breaking changes :)11:05
*** jistr|chat is now known as jistr11:05
*** sapd1_ has quit IRC11:18
openstackgerritGhanshyam Mann proposed openstack/cinder master: Switch tempest-slow to be run on python 3  https://review.openstack.org/63420511:19
whoami-rajatTheJulia: apologies :)11:20
TheJuliawhoami-rajat: c'est la vie11:20
*** sapd1 has quit IRC11:21
*** udesale has quit IRC11:21
*** phuongnh has quit IRC11:23
*** e0ne has quit IRC11:24
*** doubletao has joined #openstack-cinder11:27
*** ntpttr has quit IRC11:27
*** mchlumsky has quit IRC11:33
*** e0ne has joined #openstack-cinder11:34
*** mchlumsky has joined #openstack-cinder11:35
*** sdake has quit IRC11:41
*** sdake has joined #openstack-cinder11:48
openstackgerritSilvan Kaiser proposed openstack/cinder master: [WIP] Handle changed volume file names in remotefs snap existence check  https://review.openstack.org/63298211:49
*** markvoelker has joined #openstack-cinder11:57
rosmaitawhoami-rajat: will take a look when i get into the office this morning12:00
*** lemko has joined #openstack-cinder12:04
openstackgerritGhanshyam Mann proposed openstack/cinder master: Define single place for irrelevant-files  https://review.openstack.org/63420812:05
*** avishay has joined #openstack-cinder12:12
avishaygeguileo: Hey12:13
geguileoavishay: hi12:13
avishaygeguileo: Don't know if you saw my message before - I just tested the WWN validation patch.  The failed attach does in fact clean up the leftover device, and then retrying the connection works.12:13
geguileoavishay: we have a problem with the code...12:14
avishaywhat problem?12:14
geguileowell, not the code12:14
geguileojust the fact that we are raising an error12:15
geguileobut the damage is done by then12:15
geguileoas the new attached volume may have had data flushed to it12:15
geguileofrom the previous volume12:15
avishayI agree that it's not airtight, it's an improvement.  I also suggested detecting the leftover devices before scanning, but you said that should be nova's job?12:16
*** lpetrut has joined #openstack-cinder12:17
geguileoavishay: it should be nova's job and it would be slow and comples for us to do it12:17
geguileommmmm12:17
geguileomaybe not that costly....12:17
geguileoat least not for iSCSI12:17
avishaycan't we just look for /dev/disk/by-path/...12:17
geguileowe could check if there's a volume already with that LUN, then check the WWN, before going through the connection phase12:18
avishayThat's what I thought.  I still think checking after is a good sanity check that we could leave in.12:19
avishays/could/should/12:19
geguileothat check should be fast right? how long does it take to execute?12:20
geguileo(the check we have)12:20
avishayI think it should be very fast - just looking up each device in /sys/block and /dev/disk/by-id12:22
geguileothat's what I think as well, but I was wondering if you had actually confirmed it in logs or something12:24
avishayI can do that now12:24
*** udesale has joined #openstack-cinder12:28
*** markvoelker has quit IRC12:29
*** eharney has quit IRC12:33
avishay2019-01-31 12:33:10.018 INFO    BEFORE VERIFY(iscsi::_get_connect_result:541)12:33
avishay2019-01-31 12:33:10.019 INFO    AFTER VERIFY(iscsi::_get_connect_result:543)12:33
avishaygeguileo: Looks fast enough12:34
geguileoavishay: I don't know...12:40
geguileo};-)12:40
avishaySo I'm thinking about follow-up patch to check for the expected devices before scan?12:41
geguileosounds good12:42
avishaycool12:42
avishaythanks12:42
geguileothanks12:42
whoami-rajatrosmaita: got it thanks.12:50
*** e0ne has quit IRC13:01
*** e0ne has joined #openstack-cinder13:04
*** gnufied has quit IRC13:11
openstackgerritRaghavendra Tilay proposed openstack/cinder master: 3PAR: Provide capability to create volume either as a snapshot/independent copy  https://review.openstack.org/63411913:14
*** gkadam-bmgr is now known as gkadam13:15
*** gkadam has quit IRC13:20
*** e0ne has quit IRC13:25
*** jistr is now known as jistr|call13:25
whoami-rajatyikun smcginnis  This is a little urgently required by the horizon team to complete their BP. please take a look :)13:25
whoami-rajathttps://review.openstack.org/#/c/627549/13:25
*** markvoelker has joined #openstack-cinder13:26
*** e0ne has joined #openstack-cinder13:29
*** e0ne has quit IRC13:30
*** jistr|call is now known as jistr13:31
*** e0ne has joined #openstack-cinder13:32
*** e0ne has quit IRC13:34
*** e0ne has joined #openstack-cinder13:35
*** mriedem has joined #openstack-cinder13:37
*** eharney has joined #openstack-cinder13:46
*** pcaruana has quit IRC13:52
*** takamatsu has quit IRC13:54
*** davidsha has joined #openstack-cinder13:56
*** markvoelker has quit IRC13:58
*** Liang__ has joined #openstack-cinder14:00
*** Liang__ is now known as LiangFang14:00
*** pcaruana has joined #openstack-cinder14:02
*** sdake has quit IRC14:08
*** e0ne has quit IRC14:13
*** avishay has quit IRC14:17
jungleboyjTheJulia: Sorry we broke you.14:28
openstackgerritChuck Short proposed openstack/cinder master: Replace 'lvconvert' from rootwrap  https://review.openstack.org/63402014:30
*** zul has joined #openstack-cinder14:32
*** sdake has joined #openstack-cinder14:39
*** enriquetaso has joined #openstack-cinder14:39
*** e0ne has joined #openstack-cinder14:45
enriquetasowanghao ping ? :)14:46
whoami-rajatjungleboyj:  like glance_store and ironic, are there any other modules that use the attach functionality?14:49
jungleboyjwhoami-rajat:  Not that I can think of.14:50
jungleboyjos-brick obviously and then nova.14:50
whoami-rajatjungleboyj: i think they're well handled else we would have got notified. :)14:52
jungleboyjwhoami-rajat:  Right.  :-)14:52
whoami-rajatsmcginnis:  I approved the release note removal again and added a little detail, https://review.openstack.org/#/c/633998/ . I hope it's ok this time ?14:54
eharneywhoami-rajat: smcginnis: the reason i caught this was because we shipped an inaccurate release note on one of the stable releases saying that lio multiattach doesn't work when it does.  oops :/14:56
*** markvoelker has joined #openstack-cinder14:56
*** mvkr has quit IRC15:00
whoami-rajateharney:  does the backported code (if self.configuration.target_helper == 'lioadm': supports_multiattach = False) has any bad effect on the stable branch?15:00
eharneywhoami-rajat: wasn't that removed in a later backport?15:01
*** carlos_silva has joined #openstack-cinder15:04
smcginnisWait, we backported support for LIO multiattach?15:04
whoami-rajateharney: your code was backported to queens https://review.openstack.org/#/c/614278/ . and the fix was backported till rocky https://review.openstack.org/#/c/618472/ . was it missed for queens or was it intentional?15:06
eharneywell, we actually shipped it broken, then backported a patch to turn it off, then turned it back on after fixing the problem15:06
whoami-rajatsmcginnis:  to rocky.15:06
*** Luzi has quit IRC15:07
*** mvkr has joined #openstack-cinder15:13
*** TxGirlGeek has joined #openstack-cinder15:18
*** sdake has quit IRC15:24
*** avishay has joined #openstack-cinder15:25
*** lpetrut has quit IRC15:27
*** markvoelker has quit IRC15:29
*** meeeks has quit IRC15:33
*** gnufied has joined #openstack-cinder15:35
*** udesale has quit IRC15:40
*** LiangFang has quit IRC15:47
*** jmlowe has quit IRC15:49
*** _alastor_ has joined #openstack-cinder16:02
*** jmlowe has joined #openstack-cinder16:04
*** arne_wiebalck_ has joined #openstack-cinder16:07
*** arne_wiebalck_ has quit IRC16:07
*** enriquetaso has quit IRC16:14
*** sdake has joined #openstack-cinder16:20
*** markvoelker has joined #openstack-cinder16:27
*** imacdonn has quit IRC16:31
*** TxGirlGeek has quit IRC16:35
*** e0ne has quit IRC16:46
*** eharney has quit IRC16:49
*** spsurya has quit IRC16:54
*** markvoelker has quit IRC17:00
*** TxGirlGeek has joined #openstack-cinder17:03
*** avishay has quit IRC17:04
TheJuliajungleboyj: no worries :)17:13
*** takamatsu has joined #openstack-cinder17:14
TheJuliajungleboyj: It is an odd case to hit I believe, which is why it was only recently found I think17:15
*** enriquetaso has joined #openstack-cinder17:17
*** sdake has quit IRC17:28
*** davidsha has quit IRC17:38
*** lemko has quit IRC17:43
*** eharney has joined #openstack-cinder17:49
*** ntpttr has joined #openstack-cinder18:03
*** enriquetaso has quit IRC18:06
*** sdake has joined #openstack-cinder18:07
*** dave-mccowan has joined #openstack-cinder18:24
jungleboyjTheJulia:  Ok.  We need to be better about remembering that you guys have dependencies on us too.18:26
TheJuliajungleboyj: That would be much appreciated18:26
*** mvkr has quit IRC18:35
*** e0ne has joined #openstack-cinder18:39
*** e0ne has quit IRC18:48
*** TxGirlGeek has quit IRC18:51
*** TxGirlGeek has joined #openstack-cinder18:53
*** luizbag has quit IRC18:54
*** lpetrut has joined #openstack-cinder18:57
*** yachika has quit IRC19:07
openstackgerritMerged openstack/cinder master: Remove LIO multiattach release note  https://review.openstack.org/63399819:12
*** erlon has quit IRC19:23
*** markvoelker has joined #openstack-cinder19:27
*** pcaruana has quit IRC19:30
*** sapd1 has joined #openstack-cinder19:42
*** sapd1 has quit IRC19:48
*** TxGirlGeek has quit IRC19:48
*** abishop has quit IRC19:57
*** markvoelker has quit IRC20:00
*** arne_wiebalck_ has joined #openstack-cinder20:02
*** arne_wiebalck_ has quit IRC20:03
*** jmlowe has quit IRC20:04
*** arne_wiebalck_ has joined #openstack-cinder20:05
*** TxGirlGeek has joined #openstack-cinder20:05
*** jmlowe has joined #openstack-cinder20:24
*** carlos_silva has quit IRC20:43
*** markvoelker has joined #openstack-cinder20:57
*** TxGirlGeek has quit IRC21:00
*** arne_wiebalck_ has quit IRC21:17
*** TxGirlGeek has joined #openstack-cinder21:25
*** markvoelker has quit IRC21:30
*** eharney has quit IRC21:31
*** mchlumsky has quit IRC21:51
*** lpetrut has quit IRC21:54
*** lpetrut has joined #openstack-cinder21:58
*** lpetrut has quit IRC22:14
*** markvoelker has joined #openstack-cinder22:27
*** sdake has quit IRC22:30
openstackgerritMerged openstack/cinder master: Add project_id in group list and show API  https://review.openstack.org/62754922:35
*** markvoelker has quit IRC22:41
*** sdake has joined #openstack-cinder22:44
*** tkajinam has joined #openstack-cinder22:56
*** mriedem has quit IRC23:00
*** whoami-rajat has quit IRC23:00
« Wednesday, 2019-01-30 Index Friday, 2019-02-01 »

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