Wednesday, 2018-01-10

*** AlexeyAbashkin has joined #openstack-cinder00:01
*** AlexeyAbashkin has quit IRC00:05
*** claudiub has quit IRC00:10
*** e0ne has quit IRC00:18
*** yangyapeng has quit IRC00:27
*** yangyapeng has joined #openstack-cinder00:27
*** yangyapeng has quit IRC00:31
*** andymccr has quit IRC00:36
*** felipemonteiro has quit IRC00:40
*** andymccr has joined #openstack-cinder00:40
*** wanghao_ has quit IRC00:41
*** wanghao has joined #openstack-cinder00:42
*** markvoelker has joined #openstack-cinder00:43
*** markvoelker has quit IRC00:45
*** markvoelker has joined #openstack-cinder00:49
*** markvoelker has quit IRC00:50
*** andymccr has quit IRC00:51
*** Dinesh_Bhor has joined #openstack-cinder00:52
*** andymccr has joined #openstack-cinder00:55
*** david-lyle has joined #openstack-cinder00:58
*** yangyapeng has joined #openstack-cinder01:00
*** Dinesh_Bhor has quit IRC01:02
*** elmaciej has quit IRC01:06
*** yangyapeng has quit IRC01:06
*** dave-mccowan has joined #openstack-cinder01:07
*** markvoelker has joined #openstack-cinder01:08
*** markvoelker has quit IRC01:09
*** rmcall has joined #openstack-cinder01:10
mriedemjgriffith: you can't use the pike UCA (default in devstack) with the multiattach stuff https://bugs.launchpad.net/nova/+bug/174220401:13
openstackLaunchpad bug 1742204 in OpenStack Compute (nova) "intermittent attach failures due to libvirt locking error" [Undecided,New]01:13
mriedemleft a reply in there01:13
*** yangyapeng has joined #openstack-cinder01:13
mriedemqemu 2.10 gives the write lock unless you have libvirt>=3.10, and the pike UCA has libvirt 3.601:13
*** Dinesh_Bhor has joined #openstack-cinder01:14
*** hoangcx has quit IRC01:16
*** hoangcx has joined #openstack-cinder01:17
*** yangyapeng has quit IRC01:18
*** daidv has quit IRC01:18
*** daidv has joined #openstack-cinder01:19
*** caisan has joined #openstack-cinder01:21
*** yangyapeng has joined #openstack-cinder01:22
*** zhurong has joined #openstack-cinder01:24
*** felipemonteiro has joined #openstack-cinder01:25
*** sbezverk has left #openstack-cinder01:32
*** dave-mccowan has quit IRC01:42
*** Apoorva_ has joined #openstack-cinder01:43
*** erlon has quit IRC01:44
*** Apoorva has quit IRC01:46
*** felipemonteiro_ has joined #openstack-cinder01:47
*** Apoorva_ has quit IRC01:47
*** felipemonteiro__ has joined #openstack-cinder01:51
*** felipemonteiro_ has quit IRC01:51
*** felipemonteiro has quit IRC01:51
*** caisan_ has joined #openstack-cinder01:54
*** felipemonteiro__ has quit IRC01:55
*** caisan has quit IRC01:57
*** sdague has quit IRC02:01
*** gman-tx has joined #openstack-cinder02:03
*** felipemonteiro__ has joined #openstack-cinder02:07
*** david-lyle has quit IRC02:07
*** dalgaaf has quit IRC02:09
*** liverpooler has joined #openstack-cinder02:10
*** dalgaaf has joined #openstack-cinder02:11
*** zhaochao has quit IRC02:14
*** mriedem has quit IRC02:16
*** dave-mccowan has joined #openstack-cinder02:16
*** felipemonteiro__ has quit IRC02:18
*** rmcall has quit IRC02:21
*** harlowja has quit IRC02:24
*** zhaochao has joined #openstack-cinder02:28
*** Apoorva has joined #openstack-cinder02:29
*** abishop has quit IRC02:38
openstackgerritTommyLike proposed openstack/cinder master: Revert volume consumed capacity when rescheduling  https://review.openstack.org/53134502:40
*** mordred has quit IRC02:42
*** caisan has joined #openstack-cinder02:43
*** Dinesh_Bhor has quit IRC02:43
*** mordred has joined #openstack-cinder02:44
*** caisan_ has quit IRC02:46
vivsonimriedem: hi02:47
*** dave-mccowan has quit IRC02:47
*** liverpooler has quit IRC02:47
*** Apoorva has quit IRC02:52
*** lbragstad has quit IRC02:54
*** Dinesh_Bhor has joined #openstack-cinder03:02
*** Dinesh_Bhor has quit IRC03:02
*** liverpooler has joined #openstack-cinder03:08
*** gman-tx has quit IRC03:17
openstackgerritZhao Chao proposed openstack/python-cinderclient master: Migrate to keystoneauth identity cli opts.  https://review.openstack.org/53221703:18
*** caisan_ has joined #openstack-cinder03:20
*** caisan has quit IRC03:23
*** zhurong has quit IRC03:23
*** monjoshi has joined #openstack-cinder03:28
*** Apoorva has joined #openstack-cinder03:30
*** jistr has quit IRC03:31
*** monjoshi has quit IRC03:32
*** zhurong has joined #openstack-cinder03:32
openstackgerritJesse Wu proposed openstack/cinder master: Synology: Rename iSCSI target config options  https://review.openstack.org/53238503:35
*** liverpooler has quit IRC03:42
*** andreykurilin has quit IRC03:44
*** andreykurilin has joined #openstack-cinder03:44
*** abhishekk has joined #openstack-cinder03:47
*** lyarwood has quit IRC03:49
*** lyarwood has joined #openstack-cinder03:50
*** AlexeyAbashkin has joined #openstack-cinder03:52
*** caisan has joined #openstack-cinder03:52
*** Dinesh_Bhor has joined #openstack-cinder03:55
*** caisan_ has quit IRC03:55
*** AlexeyAbashkin has quit IRC03:57
*** caisan has quit IRC04:02
openstackgerritzheng yin proposed openstack/cinder master: backup/ceph: remove redundant pool  https://review.openstack.org/48994904:18
*** chhavi__ has joined #openstack-cinder04:25
*** AlexeyAbashkin has joined #openstack-cinder04:25
*** AlexeyAbashkin has quit IRC04:29
*** Dinesh_Bhor has quit IRC04:34
openstackgerritjunbo.li proposed openstack/cinder master: Fix version details API does not return 200 OK  https://review.openstack.org/53239404:36
*** links has joined #openstack-cinder04:37
*** Dinesh_Bhor has joined #openstack-cinder04:37
*** Dinesh_Bhor has quit IRC04:38
*** itlinux has quit IRC04:39
*** zhurong has quit IRC04:40
*** links has quit IRC04:40
*** gouthamr has quit IRC04:41
*** namnh has joined #openstack-cinder04:44
*** david-lyle has joined #openstack-cinder04:46
*** links has joined #openstack-cinder04:46
*** david-lyle has quit IRC04:53
*** caisan has joined #openstack-cinder04:55
*** Apoorva has quit IRC04:58
*** ykarel has joined #openstack-cinder04:58
*** monjoshi has joined #openstack-cinder04:59
*** Dinesh_Bhor has joined #openstack-cinder05:00
*** hoonetorg has quit IRC05:00
*** hoonetorg has joined #openstack-cinder05:14
*** markstur has quit IRC05:14
*** harlowja has joined #openstack-cinder05:24
*** caisan_ has joined #openstack-cinder05:27
*** caisan has quit IRC05:30
*** gouthamr has joined #openstack-cinder05:32
*** psachin has joined #openstack-cinder05:39
*** markvoelker has joined #openstack-cinder05:39
*** wanghao has quit IRC05:47
*** lkuchlan has joined #openstack-cinder05:47
*** wanghao has joined #openstack-cinder05:48
*** sandanar has joined #openstack-cinder05:51
*** Dinesh_Bhor has quit IRC05:59
*** edmondsw has joined #openstack-cinder06:00
*** caisan has joined #openstack-cinder06:01
*** caisan_ has quit IRC06:04
*** edmondsw has quit IRC06:04
*** claudiub has joined #openstack-cinder06:05
*** zhurong has joined #openstack-cinder06:16
*** moshele has joined #openstack-cinder06:23
*** gouthamr has quit IRC06:23
*** pcaruana has joined #openstack-cinder06:27
openstackgerritJesse Wu proposed openstack/cinder master: Synology: Rename iSCSI target config option  https://review.openstack.org/53202106:32
*** caisan_ has joined #openstack-cinder06:33
*** Dinesh_Bhor has joined #openstack-cinder06:33
*** pcaruana has quit IRC06:33
*** pcaruana has joined #openstack-cinder06:34
*** caisan has quit IRC06:36
openstackgerritTommyLike proposed openstack/cinder master: Validate metadata before reserve quota  https://review.openstack.org/53242706:38
*** gcb has quit IRC06:55
*** ykarel_ has joined #openstack-cinder06:57
*** ykarel has quit IRC07:00
*** sar has joined #openstack-cinder07:06
*** sar is now known as sbr07:06
*** sbr is now known as sbra07:06
*** lkuchlan has quit IRC07:07
*** masber has quit IRC07:08
*** stakeda has joined #openstack-cinder07:12
*** rcernin has quit IRC07:15
*** fnordahl has quit IRC07:20
*** lkuchlan has joined #openstack-cinder07:22
*** fnordahl has joined #openstack-cinder07:22
*** ykarel__ has joined #openstack-cinder07:24
*** ykarel_ has quit IRC07:27
*** moshele has quit IRC07:29
*** armaan has joined #openstack-cinder07:30
*** moshele has joined #openstack-cinder07:31
*** sbra has quit IRC07:34
*** armax has quit IRC07:35
*** threestrands has quit IRC07:36
*** Dinesh_Bhor has quit IRC07:39
*** caisan has joined #openstack-cinder07:40
*** elmaciej has joined #openstack-cinder07:42
*** caisan_ has quit IRC07:43
*** ykarel__ is now known as ykarel|lunch07:43
*** lkuchlan has quit IRC07:44
openstackgerritHamdy Khader proposed openstack/cinder master: Adding abstract class NVMeOF  https://review.openstack.org/50494407:45
*** sbra has joined #openstack-cinder07:45
*** armax has joined #openstack-cinder07:47
*** lkuchlan_wfh has joined #openstack-cinder07:47
*** harlowja has quit IRC07:49
*** hoonetorg has quit IRC07:50
*** lkuchlan_wfh has quit IRC07:53
*** kholkina has joined #openstack-cinder07:55
*** AlexeyAbashkin has joined #openstack-cinder07:57
*** Dinesh_Bhor has joined #openstack-cinder07:57
*** armax has quit IRC08:01
*** elmaciej has quit IRC08:01
*** gkadam has joined #openstack-cinder08:04
*** lkuchlan has joined #openstack-cinder08:07
*** hoonetorg has joined #openstack-cinder08:07
*** armax has joined #openstack-cinder08:08
*** pgadiya has joined #openstack-cinder08:11
*** Dinesh_Bhor has quit IRC08:13
*** tesseract has joined #openstack-cinder08:14
*** Dinesh_Bhor has joined #openstack-cinder08:17
*** Dinesh_Bhor has quit IRC08:19
*** Dinesh_Bhor has joined #openstack-cinder08:20
*** caisan_ has joined #openstack-cinder08:21
*** caisan has quit IRC08:24
*** alexchadin has joined #openstack-cinder08:24
*** netrix_ has joined #openstack-cinder08:35
*** ykarel|lunch is now known as ykarel08:37
*** netrix_ has left #openstack-cinder08:51
*** Dinesh_Bhor has quit IRC08:53
*** caisan has joined #openstack-cinder08:55
*** jistr has joined #openstack-cinder08:57
*** caisan_ has quit IRC08:58
*** stakeda has quit IRC08:59
*** moshele has quit IRC09:04
*** Dinesh_Bhor has joined #openstack-cinder09:04
*** Dinesh_Bhor has quit IRC09:06
*** tommylikehu has quit IRC09:07
*** markvoelker has quit IRC09:08
*** moshele has joined #openstack-cinder09:08
*** tommylikehu has joined #openstack-cinder09:08
*** owalsh has quit IRC09:09
*** owalsh has joined #openstack-cinder09:10
*** lkuchlan has quit IRC09:13
kwathore@Team: Requesting you to review below patch: https://review.openstack.org/#/c/532178/09:15
vivsonikwathore: i have reviewed09:25
*** wanghao has quit IRC09:28
*** wanghao has joined #openstack-cinder09:28
*** wanghao has quit IRC09:28
*** wanghao has joined #openstack-cinder09:29
*** Dinesh_Bhor has joined #openstack-cinder09:29
*** wanghao has quit IRC09:29
*** wanghao has joined #openstack-cinder09:29
*** wanghao has quit IRC09:30
*** wanghao has joined #openstack-cinder09:31
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-cinder09:32
*** Dinesh_Bhor has quit IRC09:32
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:34
*** monjoshi has quit IRC09:34
*** e0ne has joined #openstack-cinder09:34
*** wanghao has joined #openstack-cinder09:34
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:35
*** caisan_ has joined #openstack-cinder09:41
*** sandanar has quit IRC09:42
*** sandanar has joined #openstack-cinder09:43
*** ykarel has quit IRC09:44
*** caisan has quit IRC09:44
*** ykarel has joined #openstack-cinder09:44
*** lpetrut has joined #openstack-cinder09:50
*** caisan_ has quit IRC10:06
openstackgerritAndreas Scheuring proposed openstack/cinder master: Docs: ibm_storage_driver: Fix volume_driver path in documentation  https://review.openstack.org/53247710:12
openstackgerritAndreas Scheuring proposed openstack/cinder master: ibm_volume_driver: Make ds8k default proxy  https://review.openstack.org/53248110:20
*** threestrands has joined #openstack-cinder10:21
*** alexchadin has quit IRC10:32
*** alexchadin has joined #openstack-cinder10:33
openstackgerritChhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread  https://review.openstack.org/51831610:43
*** namnh has quit IRC10:51
*** zhurong has quit IRC10:56
*** abhishekk has quit IRC10:57
*** elmaciej has joined #openstack-cinder11:02
*** lkuchlan has joined #openstack-cinder11:03
*** alexchadin has quit IRC11:06
*** sbra has quit IRC11:06
*** alexchadin has joined #openstack-cinder11:06
*** markvoelker has joined #openstack-cinder11:08
*** alexchadin has quit IRC11:09
*** yangyapeng has quit IRC11:12
*** AlexeyAbashkin has quit IRC11:18
*** AlexeyAbashkin has joined #openstack-cinder11:18
*** yangyapeng has joined #openstack-cinder11:25
*** armaan has quit IRC11:33
*** armaan has joined #openstack-cinder11:33
*** armaan has quit IRC11:34
*** armaan has joined #openstack-cinder11:34
*** chhavi__ has quit IRC11:35
*** alexchadin has joined #openstack-cinder11:39
*** moshele has quit IRC11:39
*** moshele has joined #openstack-cinder11:40
*** alexchadin has quit IRC11:41
*** markvoelker has quit IRC11:42
*** moshele has quit IRC11:42
*** nicolasbock has joined #openstack-cinder11:57
*** sdague has joined #openstack-cinder11:58
*** alexchadin has joined #openstack-cinder12:00
*** pgadiya has quit IRC12:20
*** sdague has quit IRC12:31
*** moshele has joined #openstack-cinder12:35
*** markvoelker has joined #openstack-cinder12:39
*** gcb has joined #openstack-cinder12:43
*** vivsoni has quit IRC12:54
*** vivsoni has joined #openstack-cinder12:54
*** links has quit IRC12:58
*** edmondsw has joined #openstack-cinder13:01
*** abishop has joined #openstack-cinder13:03
*** markvoelker has quit IRC13:12
*** mvk has quit IRC13:14
*** moshele has quit IRC13:20
*** daidv has quit IRC13:32
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Refactored cinder tempest tests to cinder_tempest_plugin  https://review.openstack.org/52982413:34
*** sbra has joined #openstack-cinder13:35
*** alexchadin has quit IRC13:38
*** mchlumsky has joined #openstack-cinder13:42
*** ykarel is now known as ykarel|away13:43
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest  https://review.openstack.org/53120613:45
*** moshele has joined #openstack-cinder13:45
*** pchavva has joined #openstack-cinder13:48
*** ykarel|away has quit IRC13:49
*** moshele has quit IRC13:51
*** alexchadin has joined #openstack-cinder13:52
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest  https://review.openstack.org/53120613:53
*** esker has joined #openstack-cinder13:54
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Refactored cinder tempest tests to cinder_tempest_plugin  https://review.openstack.org/52982413:57
*** sdague has joined #openstack-cinder13:59
*** armaan has quit IRC14:00
*** armaan has joined #openstack-cinder14:01
*** lhx_ has joined #openstack-cinder14:02
*** esker has quit IRC14:03
*** stvnoyes has joined #openstack-cinder14:05
*** lkuchlan has quit IRC14:08
*** lkuchlan has joined #openstack-cinder14:09
*** esker has joined #openstack-cinder14:09
*** markvoelker has joined #openstack-cinder14:10
*** gouthamr has joined #openstack-cinder14:17
*** esker has quit IRC14:19
*** mvk has joined #openstack-cinder14:22
*** esker has joined #openstack-cinder14:29
*** chhavi__ has joined #openstack-cinder14:34
*** mriedem has joined #openstack-cinder14:38
*** lbragstad has joined #openstack-cinder14:38
*** jmlowe has quit IRC14:39
*** armax has quit IRC14:41
*** markvoelker has quit IRC14:43
*** 18WAAA398 has quit IRC14:43
*** lkwan has quit IRC14:43
*** alexchadin has quit IRC14:43
*** esker has quit IRC14:46
*** eharney has joined #openstack-cinder14:48
*** markvoelker has joined #openstack-cinder14:51
*** gman-tx has joined #openstack-cinder14:56
-openstackstatus- NOTICE: Gerrit is being restarted due to slowness and to apply kernel patches14:57
*** felipemonteiro__ has joined #openstack-cinder14:57
*** felipemonteiro_ has joined #openstack-cinder14:59
openstackgerritChhavi Agarwal proposed openstack/cinder master: Run backup-restore operations on native thread  https://review.openstack.org/51831614:59
*** openstackgerrit has quit IRC15:01
*** felipemonteiro__ has quit IRC15:03
*** markvoelker has quit IRC15:03
*** dave-mccowan has joined #openstack-cinder15:09
*** moshele has joined #openstack-cinder15:12
*** dave-mcc_ has joined #openstack-cinder15:12
*** erlon has joined #openstack-cinder15:13
*** armax has joined #openstack-cinder15:14
*** dave-mccowan has quit IRC15:14
*** moshele has quit IRC15:17
*** elmaciej has quit IRC15:18
*** psachin has quit IRC15:22
*** kholkina has quit IRC15:30
*** felipemonteiro_ has quit IRC15:33
*** felipemonteiro_ has joined #openstack-cinder15:34
*** Apoorva has joined #openstack-cinder15:37
*** markvoelker has joined #openstack-cinder15:37
*** markstur has joined #openstack-cinder15:40
*** markstur has quit IRC15:41
*** markstur has joined #openstack-cinder15:41
*** openstackgerrit has joined #openstack-cinder15:41
openstackgerritZhao Chao proposed openstack/python-cinderclient master: Migrate to keystoneauth identity cli opts.  https://review.openstack.org/53221715:41
*** walshh has joined #openstack-cinder15:42
*** dustins has joined #openstack-cinder15:43
*** david-lyle has joined #openstack-cinder15:43
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest  https://review.openstack.org/53120615:44
*** sandanar has quit IRC15:45
*** zhaochao has quit IRC15:47
*** markstur_ has joined #openstack-cinder15:55
*** felipemonteiro__ has joined #openstack-cinder15:57
*** markstur has quit IRC15:58
*** jmlowe has joined #openstack-cinder16:00
*** moshele has joined #openstack-cinder16:00
*** felipemonteiro_ has quit IRC16:01
*** moshele has quit IRC16:02
*** tpsilva_ has joined #openstack-cinder16:02
*** dklyle has joined #openstack-cinder16:04
*** david-lyle has quit IRC16:04
*** sbra has quit IRC16:04
*** walshh has left #openstack-cinder16:04
*** mfisch` has joined #openstack-cinder16:06
*** walshh has joined #openstack-cinder16:07
*** walshh has quit IRC16:07
*** itlinux has joined #openstack-cinder16:09
*** mfisch has quit IRC16:10
*** lkuchlan has quit IRC16:17
*** markvoelker has quit IRC16:19
*** mayur_ind has joined #openstack-cinder16:20
mayur_indHi folks,16:22
mayur_indIf my backend is providing multiple target support , then where should I store the information of these targets??16:22
*** markvoelker has joined #openstack-cinder16:22
*** markvoelker has quit IRC16:27
*** armaan has quit IRC16:27
*** armaan has joined #openstack-cinder16:28
*** armaan has quit IRC16:28
*** armaan has joined #openstack-cinder16:28
*** ntpttr_laptop has joined #openstack-cinder16:39
*** ntpttr_laptop__ has joined #openstack-cinder16:40
*** ntpttr_laptop has quit IRC16:42
*** felipemonteiro__ has quit IRC16:48
*** felipemonteiro__ has joined #openstack-cinder16:48
jungleboyjjgriffith:  ^^16:52
*** AlexeyAbashkin has quit IRC16:55
*** crose has joined #openstack-cinder16:56
*** moshele has joined #openstack-cinder16:57
smcginnismayur_ind: You just return each target the host should log in to in initialize_connection.16:57
smcginnismayur_ind: See https://github.com/openstack/cinder/blob/master/cinder/interface/volume_driver.py#L17016:57
*** liverpooler has joined #openstack-cinder16:57
mayur_indthnks @smcginnis, but I want to maintain the list of targets, can I maintain this list in cinder.conf17:00
*** sbra has joined #openstack-cinder17:01
*** Apoorva has quit IRC17:01
*** dklyle has quit IRC17:03
*** e0ne has quit IRC17:05
smcginnismayur_ind: You can have a config option for your driver, but seems less user friendly to make the deployer configure that. But if that's what you've got to do...17:09
*** dave-mcc_ has quit IRC17:11
*** pcaruana has quit IRC17:11
*** lpetrut has quit IRC17:12
*** moshele has quit IRC17:16
*** harlowja has joined #openstack-cinder17:16
*** david-lyle has joined #openstack-cinder17:16
*** ktibi has joined #openstack-cinder17:17
ktibiHi all, I have tempest error for VolumesDeleteCascade with cinder and ceph.Cinder log say "ObjectNotFound: error opening pool 'container'" Indeed, I don't have container pool on ceph. Why cinder search this pool ?17:18
*** dave-mccowan has joined #openstack-cinder17:19
*** vivsoni__ has joined #openstack-cinder17:21
*** mayur_ind has left #openstack-cinder17:21
*** mfisch` has quit IRC17:23
jgriffithanybody consider themeselves an expert with debugging the cinder object code?17:26
smcginnisdulek or geguileo?17:26
geguileosmcginnis: hi17:26
jgriffithsometimes "volume.multiattach = True; volume.save()" updates the entry in the db; sometimes it does not :(17:26
jgriffithmost of the time is does not17:27
geguileothat's really weird!!!17:27
jgriffithI though maybe I was going crazy(er)17:27
jgriffithgeguileo: yeah.. I don't get it at all; I'm going to put some checks in the volume.save method of the object module and see if there's anything telling there17:28
geguileothe way it works is that when you set multiattach = True the oslo OVO code updates the dictionary that contains the fields that have been changed17:28
geguileoso when the save method is called it checks what changes have been made and if there are changes it stores them17:28
geguileojgriffith: let me have a look at the volume OVO code17:28
jgriffithgeguileo: hmm.. perhaps I'm inadvertantly updating it somewhere else and losing the delta?  Maybe I should look at that17:28
geguileojgriffith: it's more likely that it only saves when there are additional changes and for some reason when it's only the multiattach field it's not getting saved17:29
* geguileo looks into the OVO code17:29
jgriffithgeguileo: ok, I won't distract you (too much) since you're kind enough to look :)17:30
dulekjgriffith: You're sure it's not DB method? At a glance OVO code looks fine.17:30
jgriffithgeguileo: one thing that's interesting is I have one method that it works in every time (create) and retype is the one that's problematic; which might lend ot your theory of multiple changes17:30
jgriffithdulek: yep.. here; I can share the diff of what I have, just a sec17:31
dulekjgriffith: Hm, naaah, DB code looks even finer…17:31
geguileojgriffith: everything looks good on the OVO side17:31
geguileojgriffith: I can only think that we are calling refresh between those 2 calls17:32
jgriffithhttps://gist.github.com/j-griffith/6405fa107b1b5abbb1409dc38f6c1e6617:32
geguileojgriffith: easy, you are calling volume.save instead of volume.save()17:33
geguileoon L12417:33
jgriffithgeguileo: ha!17:33
jgriffithgeguileo: well that's embarassing17:34
geguileojgriffith: not really, it has happened to me before...  XDXDXD17:34
smcginnis:)17:34
dulek:D17:34
geguileoThat's why it's easy for me to spot it17:34
dulekgeguileo was faster.17:34
jgriffithgeguileo: thank you very much!  and you as well dulek !17:35
ildikovjgriffith: I do things like this all the time, I think mriedem cries every time he touches the code after me...17:35
geguileojgriffith: no problem17:35
jgriffithLOL17:35
jgriffiththis is why I miss C/C++ and love Golang17:35
dulekjgriffith: No problem, feel free to ping anytime. :)17:35
*** itlinux has quit IRC17:35
jgriffithpython just lets you do stupid things and laughs at you behind your back :)17:36
ildikovjgriffith: with C/C++ I only got to the crying part myself... :)17:36
jgriffithildikov: LOL, well yeah; perhaps that was a bad example :)17:37
jgriffithok, geguileo all fixed up; thanks again17:37
geguileonp17:37
*** ntpttr_laptop__ has quit IRC17:38
ildikovjgriffith: nah, I get your point; not your fault that C was the first thing I was introduced to at university after the butterflies, rainbows and unicorns at high school... :)17:39
*** ntpttr_laptop has joined #openstack-cinder17:39
smcginnisHungary has some interesting high schools apparently. :)17:40
jgriffithLOL17:40
ildikovjgriffith: maybe I should've been a graphic artist after all...17:40
*** itlinux has joined #openstack-cinder17:41
*** felipemonteiro_ has joined #openstack-cinder17:43
ildikovsmcginnis: we have, just don't ask other Hungarians about it :)17:43
ildikovsmcginnis: and well, I had other ideas than IT until the very last minute17:45
*** felipemonteiro__ has quit IRC17:46
*** lhx_ has quit IRC17:47
*** gkadam has quit IRC17:47
*** armaan has quit IRC17:47
*** armaan has joined #openstack-cinder17:48
smcginnisildikov: Me too, but when I graduated there really wasn't such thing as IT. :D17:52
smcginnisThough I did have a good fortran class in high school on Apple IIc's. Pretty innovative for the time.17:52
ildikovsmcginnis: fair enough :)17:52
ildikovoh, that sounds cool17:53
ildikovalthough I was all into drawing and dancing at high school and couldn't care less about computers... :D17:54
ildikovnot fully true, but almost17:54
smcginnisI won't say what I was into...17:54
ildikovsmcginnis: I really want to know now!17:55
smcginnisHah17:55
ildikovsmcginnis: you could tell me for a vote ;)17:56
smcginnislol17:56
smcginnisMaybe some evening at the PTG. ;)17:56
ildikovhmm, voting ends before that, but fine :)17:57
Swansonsmcginnis, IIc? We had to make due with Apple IIs just because they hadn't invented anything else yet.17:58
Swansonmake due or make do?17:59
smcginnisDo?17:59
SwansonHas to be, right?17:59
*** Apoorva has joined #openstack-cinder18:00
*** Apoorva has quit IRC18:00
*** Apoorva has joined #openstack-cinder18:01
*** vivsoni__ has quit IRC18:02
*** moshele has joined #openstack-cinder18:03
*** david-lyle has quit IRC18:07
*** moshele has quit IRC18:09
*** gouthamr has quit IRC18:09
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228018:14
*** felipemonteiro_ has quit IRC18:19
*** felipemonteiro_ has joined #openstack-cinder18:19
*** hemna_ has joined #openstack-cinder18:25
*** openstack has joined #openstack-cinder18:33
*** ChanServ sets mode: +o openstack18:33
*** AlexeyAbashkin has quit IRC18:35
*** tesseract has quit IRC18:36
*** harlowja has quit IRC18:37
*** jmlowe has quit IRC18:40
*** jmlowe has joined #openstack-cinder18:41
*** esker has joined #openstack-cinder18:47
*** gouthamr has joined #openstack-cinder18:47
*** e0ne has joined #openstack-cinder18:48
chandankumareharney: you were asking about job description na18:49
chandankumareharney: it appears here http://zuulv3.openstack.org/jobs.html18:50
eharneychandankumar: great, thanks18:51
*** gouthamr has quit IRC18:53
*** mvk has quit IRC18:57
*** david-lyle has joined #openstack-cinder18:59
*** raunak has joined #openstack-cinder19:01
*** moshele has joined #openstack-cinder19:02
*** felipemonteiro__ has joined #openstack-cinder19:03
*** felipemonteiro_ has quit IRC19:06
*** moshele has quit IRC19:07
*** harlowja has joined #openstack-cinder19:13
*** hemna_ has quit IRC19:16
*** harlowja_ has joined #openstack-cinder19:16
*** harlowja has quit IRC19:19
*** thingee has quit IRC19:23
*** thingee has joined #openstack-cinder19:29
openstackgerritJohn Griffith proposed openstack/cinder master: Enable multiattach capability  https://review.openstack.org/53228019:29
*** rmcall has joined #openstack-cinder19:30
*** Apoorva_ has joined #openstack-cinder19:32
*** mvk has joined #openstack-cinder19:33
*** Apoorva has quit IRC19:34
*** ktibi has quit IRC19:42
*** rmcall has quit IRC19:43
*** hemna_ has joined #openstack-cinder19:45
*** moshele has joined #openstack-cinder19:45
*** rmcall has joined #openstack-cinder19:46
geguileomnaser: 1602 volumes (801 from Cinder and 801 non cinder ones) takes 10 seconds to stat in my toy system19:48
mnaserhmm19:48
mnasergeguileo: what version of ceph?19:48
geguileomnaser: the system is a devstack all in one including the ceph cluster all in 2 cores and 8GB19:49
*** moshele has quit IRC19:49
geguileomnaser: 12.2.219:50
mnasergeguileo: ok, one second, just wanna check one thing please19:50
smcginnismnaser: You're running pike but patched it with the newer fixes, right?19:50
geguileomnaser: that's Luminous from December19:50
mnasersmcginnis: running latest tagged pike19:50
geguileomnaser: but that one doesn't have my fixes19:50
geguileoI think...19:50
*** markstur_ has quit IRC19:51
mnasergeguileo: im running - time for i in `rbd -p volumes ls`; do rbd info --format json volumes/$i; done19:52
mnaseri know that's not the same efficency19:53
geguileomnaser: that's irrelevant19:53
geguileomnaser: what's important is how fast the cinder code runs, not the bash tool19:53
mnaserright, but if the bash runs in 10s here and 100s here, it might be a ceph issue or whatever19:53
mnaserill write a small python script based exactly on the usage code then19:53
mnaseri'd like to see how long it takes to run here vs there19:53
geguileomnaser: is yours a production env or can you touch the code?19:54
mnaserprod19:54
geguileomnaser: because the easier way to see the timing is adding 2 logging lines (that's how I'm doing it now)19:54
mnasergeguileo: oh yeah good point19:54
mnaseryeah id rather not restart cinder-volume19:54
mnaseri can easily write a small python 'recreator'19:54
geguileomnaser: you go to the cinder/volume/drivers/rbd.py file and add one line at the begining of _update_volume_stats and another at the end19:54
geguileomnaser: well, on your recreator remember to use the proxies and eventlets we are using in Cinder19:55
geguileootherwise the comparison is not relevant19:55
openstackgerritChandan Kumar proposed openstack/cinder-tempest-plugin master: Added cinder tempest plugin zuulv3 jobs to execute tempest  https://review.openstack.org/53120619:55
*** markstur has joined #openstack-cinder19:55
*** nicolasbock has quit IRC19:56
geguileomnaser: by the way, if you want to compare the python code performance with the fastest implementation you can just run the single RBD call20:05
geguileomnaser: time sudo rbd ls -l volumes20:05
geguileomnaser: On my system that takes 6.386s versus the 10seconds it takes in Python20:06
mnasergeguileo: i thought the ls and the get size are two independent calls?20:06
mnaserthe .list() gets a list and then .size() gets the size for it20:06
geguileomnaser: they are, unless you use long format20:06
mnaseroh i see20:06
geguileomnaser: that is correct, but internally you can do both in one CLI call, and then internally the C code will make all the calls20:06
geguileomnaser: So 1 CLI call triggers many RBD calls20:06
mnaserok, see that is taking quite sometime to run20:07
mnaseri wonder if it has to do with the fact that this is 10.x20:07
mnaserok running it with time20:08
geguileomnaser: Running it like that is comparing the C++ optimized implementation versus the Python non optimized implementation that relies on proxing the calls to the C library to prevent greenthread starvation20:08
mnasergeguileo: of course, but your `rbd ls -l volumes` for 1600 volumes takes 6.3 seconds, and mine takes 4520:09
mnaserso.. that eliminates the issue of cinder20:09
mnaserand onto rbd, so i dont think there's any issues with cinder, there has to be something else20:09
mnasergeguileo: sorry to have you go through that, but thank you for taking the time to check it, i appreciate it20:12
geguileomnaser: I'll try to think of a way to speed it up anyway, since I think that 10 seconds for 1600 volumes is a little bit more than what I'd like20:13
mnasergeguileo: ill do my research and see if its an environment reason (or maybe an optimization of jewel that allows for much quicker listing)20:14
jdillamanmnaser: 45 seconds is way too long for essentially 2 round-trip calls to list the directory20:14
geguileomnaser: I'll have a look if there's a way to cache this like erlon suggested in the meeting20:14
geguileojdillaman: it's not 2 round trips, it's 1601 round trips20:14
mnaser^ yeah from what i understand thats what it is20:14
geguileojdillaman: first to get the list of images then one call for each of the images to get the sizes20:15
jdillamanmnaser: oh -- are you doing the long form20:15
mnaseryep, 0.154 ms latency to mons20:15
geguileomnaser: the long form is the equivalent of the stats code in Cinder20:15
jdillamanmnaser: the luminous release does 10 concurrently (by default)20:15
mnasermaybe that was the optimization thats not there in jewel20:16
mnaserthe list is happening serially20:16
jdillamancorrect20:16
mnaserthis can explain that my 45 seconds divided by 10 is 4.5 seconds (or somewhat close to 6.3s that geguileo is seeing)20:16
geguileo'i don't think that's it20:16
geguileoBecause my Cinder code does it in 10 seconds and it's doing it serially20:16
geguileoAnd I have 1602 images20:17
mnaseri wonder if the # of objects has an impact in this? this is diving into the levels of ceph beyond my knowledge20:17
mnaser67031 GB data, 21626 kobjects for this cluster20:17
geguileoit's late here, but I'll keep thinking of a way to improve the speed20:18
mnaserthank you20:18
mnaserhave a good night geguileo ! :)20:18
geguileomnaser: have a nice day20:18
*** rmcall has quit IRC20:25
*** dave-mccowan has quit IRC20:27
*** elmaciej has joined #openstack-cinder20:29
*** imacdonn has quit IRC20:32
*** imacdonn has joined #openstack-cinder20:33
*** dave-mccowan has joined #openstack-cinder20:39
openstackgerritJohn Griffith proposed openstack/cinder master: Add multiattach policy  https://review.openstack.org/53156920:53
*** gman-tx has quit IRC20:56
*** gman-tx has joined #openstack-cinder20:57
*** AlexeyAbashkin has joined #openstack-cinder21:00
*** eharney has quit IRC21:03
*** felipemonteiro__ has quit IRC21:03
*** felipemonteiro__ has joined #openstack-cinder21:04
*** AlexeyAbashkin has quit IRC21:04
*** dustins has quit IRC21:10
mriedemjgriffith: ildikov: L48 for the RW/RO secondary attachment stuff for the PTG https://etherpad.openstack.org/p/nova-ptg-rocky21:11
ildikovmriedem: I added stuff to L26 yesterday :)21:12
mriedemoh ill update yours21:12
ildikovmriedem: cool, sorry I should've pinged21:13
*** moshele has joined #openstack-cinder21:13
mriedemtotally fine21:14
*** e0ne has quit IRC21:14
*** Apoorva_ has quit IRC21:15
*** Apoorva has joined #openstack-cinder21:16
*** moshele has quit IRC21:16
*** dustins has joined #openstack-cinder21:18
*** felipemonteiro_ has joined #openstack-cinder21:20
*** felipemonteiro__ has quit IRC21:23
*** sbra has quit IRC21:27
*** dave-mccowan has quit IRC21:29
*** gouthamr has joined #openstack-cinder21:32
*** felipemonteiro_ has quit IRC21:32
*** felipemonteiro_ has joined #openstack-cinder21:33
*** gouthamr_ has joined #openstack-cinder21:35
*** gouthamr has quit IRC21:37
*** chhavi__ has quit IRC21:41
*** pcaruana has joined #openstack-cinder21:42
*** jmlowe has quit IRC21:42
*** pcaruana has quit IRC21:43
*** armaan has quit IRC21:48
*** threestrands_ has joined #openstack-cinder21:52
*** threestrands_ has quit IRC21:53
*** threestrands_ has joined #openstack-cinder21:53
*** threestrands_ has joined #openstack-cinder21:53
*** threestrands has quit IRC21:54
*** dustins has quit IRC21:54
*** armaan has joined #openstack-cinder22:00
*** tpsilva_ has quit IRC22:01
*** masber has joined #openstack-cinder22:02
openstackgerritJohn Griffith proposed openstack/cinder master: Fix readonly mode for new attachment API calls  https://review.openstack.org/53264522:03
*** crose has quit IRC22:03
*** gouthamr_ has quit IRC22:06
*** rcernin has joined #openstack-cinder22:11
erlongeguileo: not sure if that would work in your case but fswatch allows you to monitor for file/dir changes and get you notified on the events, so, if that works as I imagined you could listen in the dir for new files and update your file list22:20
erlonhttps://github.com/emcrisostomo/fswatch/blob/master/README.md22:22
-openstackstatus- NOTICE: The zuul system is being restarted to apply security updates and will be offline for several minutes. It will be restarted and changes re-equeued; changes approved during the downtime will need to be rechecked or re-approved.22:22
*** jmlowe has joined #openstack-cinder22:28
*** elmaciej has quit IRC22:28
*** felipemonteiro__ has joined #openstack-cinder22:29
*** mchlumsky has quit IRC22:29
*** felipemonteiro_ has quit IRC22:33
*** abishop has quit IRC22:36
*** gouthamr has joined #openstack-cinder22:37
*** threestrands_ has quit IRC22:41
*** gouthamr has quit IRC22:42
*** jmlowe has quit IRC22:52
openstackgerritEric Young proposed openstack/cinder master: Modify tests to account for ScaleIO creating volumes = 8GB  https://review.openstack.org/53268422:54
*** threestrands has joined #openstack-cinder22:55
*** threestrands has quit IRC22:55
*** threestrands has joined #openstack-cinder22:55
*** edmondsw has quit IRC22:59
*** rmcall has joined #openstack-cinder23:09
*** felipemonteiro__ has quit IRC23:10
*** threestrands has quit IRC23:11
*** jmlowe has joined #openstack-cinder23:12
*** gman-tx has quit IRC23:18
*** threestrands has joined #openstack-cinder23:20
*** elmaciej has joined #openstack-cinder23:29
*** raunak has quit IRC23:29
*** armax has quit IRC23:36
*** liverpooler has quit IRC23:44
*** jmlowe has quit IRC23:44
openstackgerritMatt Riedemann proposed openstack/cinder master: Add the nova-multiattach job to the experimental queue  https://review.openstack.org/53269523:45
mriedemlet's see if this zuulv3 stuff works ^23:45
*** jmlowe has joined #openstack-cinder23:46
*** sdague has quit IRC23:47
*** pchavva has quit IRC23:50
*** freerunner has quit IRC23:52
*** andreykurilin has quit IRC23:52
*** andreykurilin has joined #openstack-cinder23:53
*** freerunner has joined #openstack-cinder23:53
*** edmondsw has joined #openstack-cinder23:55
*** elmaciej has quit IRC23:59

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