Monday, 2019-02-25

*** erlon has quit IRC00:00
*** markvoelker has quit IRC00:06
*** erlon has joined #openstack-cinder00:27
*** sdake has joined #openstack-cinder00:46
*** ircuser-1 has joined #openstack-cinder00:58
*** sdake has quit IRC01:03
*** sdake has joined #openstack-cinder01:05
*** lixiaoy1 has joined #openstack-cinder01:10
*** sdake has quit IRC01:19
*** dave-mccowan has joined #openstack-cinder01:29
*** dave-mccowan has quit IRC01:35
*** phuongnh has joined #openstack-cinder01:44
openstackgerritGhanshyam Mann proposed openstack/cinder master: Define single place for irrelevant-files  https://review.openstack.org/63420801:50
openstackgerritGhanshyam Mann proposed openstack/cinder master: Define single place for irrelevant-files  https://review.openstack.org/63420801:51
*** wanghao has joined #openstack-cinder01:53
*** markvoelker has joined #openstack-cinder02:07
*** Dinesh_Bhor has joined #openstack-cinder02:20
*** jbernard has quit IRC02:23
*** jbernard has joined #openstack-cinder02:25
*** markvoelker has quit IRC02:37
openstackgerritRyan Liang proposed openstack/cinder master: Unity: Add consistent group support  https://review.openstack.org/57411903:00
*** sdake has joined #openstack-cinder03:03
*** whoami-rajat has joined #openstack-cinder03:12
*** markvoelker has joined #openstack-cinder03:35
*** sdake has quit IRC03:36
*** sdake has joined #openstack-cinder03:40
openstackgerritzengyingzhe proposed openstack/cinder master: Update support matrix of Huawei driver  https://review.openstack.org/63856103:43
*** udesale has joined #openstack-cinder03:51
*** sdake has quit IRC03:56
*** markvoelker has quit IRC04:03
*** udesale has quit IRC04:21
*** sdake has joined #openstack-cinder04:21
*** udesale has joined #openstack-cinder04:22
*** spsurya has joined #openstack-cinder04:24
*** sdake has quit IRC04:25
*** sdake has joined #openstack-cinder04:26
*** sdake has quit IRC04:30
*** sdake has joined #openstack-cinder04:32
*** sdake has quit IRC04:35
*** sdake has joined #openstack-cinder04:36
*** sdake has quit IRC04:40
*** sdake has joined #openstack-cinder04:44
*** sdake has quit IRC04:45
*** sdake has joined #openstack-cinder04:52
*** sdake has quit IRC04:55
*** sdake has joined #openstack-cinder04:57
*** sdake has quit IRC05:00
*** markvoelker has joined #openstack-cinder05:00
*** sdake has joined #openstack-cinder05:01
*** sdake has quit IRC05:10
*** sdake_ has joined #openstack-cinder05:10
*** sdake_ has quit IRC05:15
*** sdake has joined #openstack-cinder05:17
*** sdake has quit IRC05:21
*** sdake has joined #openstack-cinder05:22
*** sdake has quit IRC05:26
*** sdake has joined #openstack-cinder05:26
*** bhagyashris has joined #openstack-cinder05:29
*** sdake has quit IRC05:30
*** sdake_ has joined #openstack-cinder05:32
*** markvoelker has quit IRC05:33
*** sdake_ has quit IRC05:35
*** sdake has joined #openstack-cinder05:37
*** sdake has quit IRC05:45
*** sdake has joined #openstack-cinder05:46
*** sdake has quit IRC05:51
*** sdake has joined #openstack-cinder05:51
*** zengyingzhe has quit IRC05:51
*** sdake has quit IRC05:55
*** sdake has joined #openstack-cinder05:57
*** sdake has quit IRC06:00
*** dpawlik has joined #openstack-cinder06:01
*** sdake has joined #openstack-cinder06:02
*** avishay has joined #openstack-cinder06:02
*** sdake has quit IRC06:06
*** sdake_ has joined #openstack-cinder06:06
*** sdake_ has quit IRC06:10
openstackgerritBrin Zhang proposed openstack/cinder master: api-ref: add qos_specs_id to show default volume type response  https://review.openstack.org/62520806:18
avishaygeguileo: Hey, I have an os-brick question. I am hitting some bug where occasionally a thread doesn't wake up from the lock. As far as I can tell, it's probably some issue with eventlet/semaphore.06:23
avishayI am running with an Open iSCSI initiator that supports manual scan. Can I just manually remove the lock around connect/disconnect_volume in my setup for now?06:24
*** Luzi has joined #openstack-cinder06:51
*** vishakha has joined #openstack-cinder07:12
openstackgerritAlexey proposed openstack/cinder master: Update volume status during volume migration.  https://review.openstack.org/63899507:16
*** udesale has quit IRC07:27
*** udesale has joined #openstack-cinder07:27
*** markvoelker has joined #openstack-cinder07:30
*** arne_wiebalck has joined #openstack-cinder07:40
*** markvoelker has quit IRC08:03
*** lyaaaaarwood is now known as lyarwood08:06
openstackgerritMerged openstack/cinder master: Define single place for irrelevant-files  https://review.openstack.org/63420808:07
*** tkajinam has quit IRC08:14
*** takamatsu_ has quit IRC08:19
openstackgerritYury Kulazhenkov proposed openstack/os-brick master: rename ScaleIO connector to VxFlex OS  https://review.openstack.org/63553008:19
*** takamatsu_ has joined #openstack-cinder08:21
*** takamatsu_ has quit IRC08:32
*** takamatsu_ has joined #openstack-cinder08:38
*** e0ne has joined #openstack-cinder08:51
*** stakeda has joined #openstack-cinder08:52
*** lixiaoy1 has quit IRC08:53
openstackgerritYury Kulazhenkov proposed openstack/cinder master: remove support for deprecated options in VxFlex OS driver  https://review.openstack.org/63835808:55
*** doubletao has joined #openstack-cinder08:58
*** rcernin has quit IRC09:00
*** markvoelker has joined #openstack-cinder09:01
*** wanghao has quit IRC09:27
*** markvoelker has quit IRC09:34
*** bhagyashris has quit IRC10:02
*** erlon has quit IRC10:03
*** sdin has quit IRC10:04
*** sdinescu has joined #openstack-cinder10:04
geguileoavishay: I believe there was another reason why that lock was there10:12
geguileoavishay: iirc under some circumstances iscsiadm would block or fail (I have never seen it) if you add and remove target/portal/sessions concurrently10:12
avishaygeguileo: Interesting.  I saw your commit that adds guard_connection, but saw that it isn't used yet?  I thought this would replace that lock10:14
geguileoavishay: I think I explained why that lock was there...10:14
geguileoavishay: lock = context manager10:14
geguileothe patch to use it in Nova is on review10:15
geguileobut cannot pass CI until we release os-brick10:15
avishayI see.  Nova has a lock in addition to the @synchronized one?10:16
*** jmlowe has quit IRC10:16
*** jmlowe has joined #openstack-cinder10:17
geguileohttps://review.openstack.org/#/c/614190/10:18
geguileoavishay: ^10:18
avishayI see.  So in fact we have two locks, one in nova, and one in os-brick.10:20
avishaySo how do all these patches help if we still need the ones in os-brick?10:22
*** markvoelker has joined #openstack-cinder10:31
avishaygeguileo: ^10:32
geguileoavishay: have you read the comment in the nova patch and the context manager's os-bric commit message?10:33
avishaygeguileo: Yes, from what I understand the goal is to remove locks from Nova in certain cases. However, os-brick (at least with iSCSI) always locks.  That's what I don't get...10:35
*** trident has quit IRC10:47
*** kevko has joined #openstack-cinder10:48
kevkosmcginnis: hi, what about https://review.openstack.org/#/c/635195/ , could you please merge ?10:49
*** trident has joined #openstack-cinder10:50
*** stakeda has quit IRC10:50
kevkosmcginnis: there has been already +2 but pep8 failed .... now it is ready , pep8 fixed , and there is  +1 from zuul10:50
*** ganso has joined #openstack-cinder10:51
avishaygeguileo: What I'm confused about is in the nova patch you wrote it's to "allow Nova to [...] only lock when it's really necessary". My question is why is it ever necessary if os-brick has locks around connect_volume and disconnect_volume.10:52
geguileoavishay: it's explained in the nova patch10:53
geguileoand you can also check the shared_targets patches, as it should also be explained there10:53
avishayI read the patches, I read the spec, and I still don't get it.  Sorry if I'm dense, but I'd appreciate it if you could give me a quick explanation. :)10:54
geguileoon lock attach/detach the other lock map+attach/detach+unmpa10:55
geguileounmpa=unmap10:55
avishaygeguileo: Thank you, that's what I was missing11:02
*** markvoelker has quit IRC11:04
*** mauricio_ps has joined #openstack-cinder11:06
*** doubletao has quit IRC11:07
*** Dinesh_Bhor has quit IRC11:07
*** mauricio_ps has quit IRC11:08
*** mauricio_ps has joined #openstack-cinder11:08
*** udesale has quit IRC11:18
*** thgcorrea has joined #openstack-cinder11:20
*** Nhey has joined #openstack-cinder11:30
openstackgerritThiago Correa proposed openstack/cinder master: Improve documentation of goodness/filter function  https://review.openstack.org/63875211:37
openstackgerritThiago Correa proposed openstack/cinder master: Improve documentation of goodness/filter function  https://review.openstack.org/63875211:38
*** sdake has joined #openstack-cinder11:46
*** sdake has quit IRC11:50
*** sdake has joined #openstack-cinder11:51
*** sdake has quit IRC11:55
openstackgerritYury Kulazhenkov proposed openstack/cinder master: remove support for deprecated options in VxFlex OS driver  https://review.openstack.org/63835811:56
*** sdake has joined #openstack-cinder11:56
*** sdake has quit IRC12:00
*** markvoelker has joined #openstack-cinder12:01
*** sdake has joined #openstack-cinder12:02
*** sdake has quit IRC12:02
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844212:05
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844212:09
*** markvoelker has quit IRC12:35
*** udesale has joined #openstack-cinder12:42
*** e0ne has quit IRC12:47
*** phuongnh has quit IRC12:48
*** luizbag has joined #openstack-cinder12:57
*** lseki has joined #openstack-cinder13:04
*** Nhey has quit IRC13:04
*** abishop has joined #openstack-cinder13:04
*** rosmaita has joined #openstack-cinder13:05
openstackgerritVlad Gusev proposed openstack/cinder stable/rocky: Fix version return incorrect when endpoint url end without /  https://review.openstack.org/63838713:06
openstackgerritVlad Gusev proposed openstack/cinder stable/queens: Fix version return incorrect when endpoint url end without /  https://review.openstack.org/63838913:06
*** jistr is now known as jistr|mtg13:07
*** jistr|mtg is now known as jistr13:08
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844213:26
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844213:27
*** markvoelker has joined #openstack-cinder13:32
*** e0ne has joined #openstack-cinder13:35
*** udesale has quit IRC13:37
*** udesale has joined #openstack-cinder13:38
*** sri_ has joined #openstack-cinder13:42
*** jmlowe has quit IRC13:44
*** sapd1 has joined #openstack-cinder13:48
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844213:50
*** mmethot has joined #openstack-cinder14:03
*** markvoelker has quit IRC14:05
*** dpawlik has quit IRC14:06
*** mvkr has quit IRC14:24
*** jmlowe has joined #openstack-cinder14:29
*** zul has joined #openstack-cinder14:32
*** mvkr has joined #openstack-cinder14:35
*** eharney has joined #openstack-cinder14:40
*** mriedem has joined #openstack-cinder14:47
*** sapd1 has quit IRC14:59
*** udesale has quit IRC15:02
*** e0ne has quit IRC15:03
*** e0ne has joined #openstack-cinder15:05
*** gnufied has joined #openstack-cinder15:06
*** Luzi has quit IRC15:13
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844215:17
openstackgerritEric Harney proposed openstack/cinder-tempest-plugin master: Add metadata to snapshot for revert test  https://review.openstack.org/63913315:28
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844215:34
openstackgerritEric Harney proposed openstack/cinder master: Extend timeout for database migration tests  https://review.openstack.org/63520315:38
whoami-rajat^ Thanks Eric for the update :)15:40
e0nesmcginnis, jungleboyj, eharney: hi. what is our expectation about cross availability zone volume retype/migration15:50
e0neit works now, but we don't change AZ for volume15:50
e0neI think we should not allow it by default. maybe it should be allowed only for admin users15:51
eharneyreally? sounds like https://review.openstack.org/#/c/550775/ ?15:51
jungleboyje0ne:  So you can move the volume to a new AZ but the AZ for the volume isn't updated?15:51
eharneythis is a feature that's supposed to work15:51
e0nejungleboyj: yep15:51
jungleboyjeharney:  ++15:51
jungleboyje0ne:  That sounds like a big bug then.15:51
eharneyit sounds like a bug we already fixed once...15:51
e0neeharney: thanks, it's what I was looking for15:52
jungleboyj:-)15:52
jungleboyjYay for it already being fixed.15:52
e0nehonestly, I checked it with ocata only and just want to understand what is expected behaviour15:52
eharneyahh15:53
eharney*whew*15:53
e0neeharney: thanks for the help15:57
*** _alastor_ has joined #openstack-cinder16:02
openstackgerritRajat Dhasmana proposed openstack/cinder stable/rocky: DNM: Testing cinder legacy jobs on rocky still use xenial  https://review.openstack.org/63914216:02
openstackgerritEric Harney proposed openstack/cinder master: Trim tempest scenarios for ceph job  https://review.openstack.org/63914316:04
*** markvoelker has joined #openstack-cinder16:05
openstackgerritEric Harney proposed openstack/cinder master: Trim tempest scenarios for ceph job  https://review.openstack.org/63914316:15
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525516:17
openstackgerritEric Harney proposed openstack/cinder master: Trim tempest scenarios for ceph job  https://review.openstack.org/63914316:19
*** _hemna has joined #openstack-cinder16:23
*** imacdonn has joined #openstack-cinder16:23
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844216:23
openstackgerritMerged openstack/cinder master: Update support matrix of Huawei driver  https://review.openstack.org/63856116:25
*** e0ne has quit IRC16:26
*** _hemna has quit IRC16:30
*** avishay has quit IRC16:31
*** e0ne has joined #openstack-cinder16:31
openstackgerritEric Harney proposed openstack/cinder master: DNM: Trim tempest scenarios for ceph job  https://review.openstack.org/63914316:33
*** markvoelker has quit IRC16:34
*** ganso has left #openstack-cinder16:36
*** takamatsu_ has quit IRC16:39
*** dklyle_ has quit IRC16:40
*** takamatsu_ has joined #openstack-cinder16:41
*** takamatsu_ has quit IRC16:45
*** takamatsu_ has joined #openstack-cinder16:51
*** dklyle has joined #openstack-cinder16:55
*** carlos_silva has joined #openstack-cinder17:03
*** takamatsu_ has quit IRC17:06
*** takamatsu_ has joined #openstack-cinder17:07
*** spotz has joined #openstack-cinder17:10
*** luizbag has quit IRC17:16
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844217:17
*** eharney has quit IRC17:17
*** e0ne has quit IRC17:18
*** takamatsu_ has quit IRC17:21
*** takamatsu_ has joined #openstack-cinder17:22
*** erlon has joined #openstack-cinder17:23
*** takamatsu_ has quit IRC17:25
*** sdake has joined #openstack-cinder17:25
*** IvensZambrano has joined #openstack-cinder17:27
*** takamatsu_ has joined #openstack-cinder17:31
*** markvoelker has joined #openstack-cinder17:32
hemnaguwak17:40
*** dpawlik has joined #openstack-cinder17:43
*** takamatsu_ has quit IRC17:46
*** takamatsu_ has joined #openstack-cinder17:48
*** dpawlik has quit IRC17:52
*** takamatsu_ has quit IRC18:05
*** markvoelker has quit IRC18:05
*** sdake_ has joined #openstack-cinder18:06
*** sdake has quit IRC18:07
openstackgerritGorka Eguileor proposed openstack/cinderlib master: Add zuul jobs  https://review.openstack.org/63844218:09
*** sdake_ has quit IRC18:10
*** takamatsu_ has joined #openstack-cinder18:10
*** sdake has joined #openstack-cinder18:11
*** takamatsu_ has quit IRC18:14
*** sdake has quit IRC18:16
*** mvkr has quit IRC18:16
*** sdake_ has joined #openstack-cinder18:16
*** takamatsu_ has joined #openstack-cinder18:20
*** sdake_ has quit IRC18:20
*** sdake has joined #openstack-cinder18:22
*** sdake has quit IRC18:26
*** sdake_ has joined #openstack-cinder18:26
*** mvkr has joined #openstack-cinder18:33
*** sdake_ has quit IRC18:35
*** sdake has joined #openstack-cinder18:36
openstackgerritMerged openstack/cinder master: Extend timeout for database migration tests  https://review.openstack.org/63520318:36
*** sdake has quit IRC18:41
*** sdake_ has joined #openstack-cinder18:41
*** IvensZambrano has quit IRC18:42
*** e0ne has joined #openstack-cinder18:44
*** sdake_ has quit IRC18:45
*** sdake has joined #openstack-cinder18:46
*** jmlowe has quit IRC18:49
*** takamatsu_ has quit IRC18:50
*** sdake has quit IRC18:50
*** sdake has joined #openstack-cinder18:51
*** jmlowe has joined #openstack-cinder18:53
*** sdake has quit IRC18:55
*** takamatsu_ has joined #openstack-cinder18:56
*** sdake has joined #openstack-cinder18:56
*** sdake has quit IRC19:01
*** sdake has joined #openstack-cinder19:02
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525519:02
*** sdake has quit IRC19:05
*** sdake has joined #openstack-cinder19:07
*** sdake has quit IRC19:11
*** sdake has joined #openstack-cinder19:12
*** sdake has quit IRC19:15
*** eharney has joined #openstack-cinder19:16
*** sdake_ has joined #openstack-cinder19:17
*** sdake_ has quit IRC19:20
openstackgerritRajat Dhasmana proposed openstack/cinder master: Add migration for creating default volume type  https://review.openstack.org/63918019:20
*** gnufied has quit IRC19:30
*** luizbag has joined #openstack-cinder19:31
*** IvensZambrano has joined #openstack-cinder19:40
*** gnufied has joined #openstack-cinder19:41
openstackgerritMerged openstack/os-brick master: Fix FC case sensitive scanning  https://review.openstack.org/63817819:59
openstackgerritMerged openstack/os-brick master: Py3: Fix invalid escape sequencees  https://review.openstack.org/63849819:59
*** thgcorrea has quit IRC20:00
*** mauricio_ps has quit IRC20:01
*** markvoelker has joined #openstack-cinder20:02
*** vishakha has quit IRC20:10
*** IvensZambrano has quit IRC20:15
*** jmlowe has quit IRC20:21
erloneharney, hey Eric, can you revisit this patch when you have chance? Still have a -1 from you20:24
eharneyerlon: which one?20:26
erloneharney, https://review.openstack.org/#/c/633596/20:27
*** dosaboy has quit IRC20:34
*** dosaboy has joined #openstack-cinder20:36
*** markvoelker has quit IRC20:36
*** dosaboy has quit IRC20:37
*** _hemna has joined #openstack-cinder20:41
*** dosaboy has joined #openstack-cinder20:46
openstackgerritSean McGinnis proposed openstack/cinder master: Extend use of user message in volume action  https://review.openstack.org/46393620:49
*** _hemna has quit IRC20:50
*** lseki has quit IRC21:04
*** prg3 has joined #openstack-cinder21:07
*** jmlowe has joined #openstack-cinder21:07
*** prg3 has left #openstack-cinder21:09
*** e0ne has quit IRC21:21
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder master: Add get_driver_options method  https://review.openstack.org/63525521:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinderlib master: Convert the driver_options to a dictionary  https://review.openstack.org/63919321:27
hemnageguileo ^^21:27
*** sdake has joined #openstack-cinder21:28
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinderlib master: Convert the driver_options to a dictionary  https://review.openstack.org/63919321:28
*** markvoelker has joined #openstack-cinder21:33
*** luizbag has quit IRC21:34
*** rcernin has joined #openstack-cinder21:46
*** whoami-rajat has quit IRC21:47
*** sdake has quit IRC21:48
*** gnufied has quit IRC21:51
*** e0ne has joined #openstack-cinder21:54
*** e0ne has quit IRC21:55
*** sdake has joined #openstack-cinder21:59
*** sdake has quit IRC22:01
*** sdake has joined #openstack-cinder22:02
*** carlos_silva has quit IRC22:02
*** markvoelker has quit IRC22:04
*** e0ne has joined #openstack-cinder22:05
*** sdake has quit IRC22:05
*** sdake has joined #openstack-cinder22:07
*** e0ne has quit IRC22:08
*** sdake has quit IRC22:10
*** imacdonn_ has joined #openstack-cinder22:10
*** imacdonn has quit IRC22:11
*** s10 has joined #openstack-cinder22:11
*** supamatt has quit IRC22:11
*** sdake has joined #openstack-cinder22:12
*** supamatt has joined #openstack-cinder22:12
*** _alastor_ has quit IRC22:13
*** sdake has quit IRC22:13
openstackgerritSean McGinnis proposed openstack/cinder-tempest-plugin master: Multipath tempest scenarios  https://review.openstack.org/63920522:15
openstackgerritMerged openstack/python-cinderclient master: add python 3.7 unit test job  https://review.openstack.org/61003622:16
*** erlon has quit IRC22:17
*** _alastor_ has joined #openstack-cinder22:20
*** _alastor_ has quit IRC22:20
openstackgerritSean McGinnis proposed openstack/cinder master: Enable service capabilities update at less than 60 seconds  https://review.openstack.org/46993622:44
*** tkajinam has joined #openstack-cinder22:57
*** markvoelker has joined #openstack-cinder23:01
*** _alastor_ has joined #openstack-cinder23:03
openstackgerritMerged openstack/python-cinderclient master: Fix: cinder group-list not working with non-admin user  https://review.openstack.org/62735923:09
*** _alastor_ has quit IRC23:13
*** _alastor_ has joined #openstack-cinder23:14
*** _alastor_ has quit IRC23:16
*** _alastor_ has joined #openstack-cinder23:17
*** _alastor_ has quit IRC23:17
*** _alastor_ has joined #openstack-cinder23:22
*** markvoelker has quit IRC23:34
*** s10 has quit IRC23:34
openstackgerritEric Harney proposed openstack/cinder master: Disable etcd3 for lio-barbican job  https://review.openstack.org/63277323:47
*** _alastor_ has quit IRC23:57
*** _alastor_ has joined #openstack-cinder23:58

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