Thursday, 2018-10-18

*** bswartz has joined #openstack-cinder00:44
*** brinzhang has joined #openstack-cinder01:04
*** brinzhang has quit IRC01:04
*** brinzhang has joined #openstack-cinder01:05
*** markvoelker has joined #openstack-cinder01:05
*** markvoelker has quit IRC01:09
*** brinzhang has quit IRC01:10
*** brinzhang has joined #openstack-cinder01:11
*** brinzhang has quit IRC01:12
*** brinzhang has joined #openstack-cinder01:13
*** Dinesh_Bhor has joined #openstack-cinder01:20
*** imacdonn has quit IRC01:22
*** imacdonn has joined #openstack-cinder01:22
*** LiangFang has joined #openstack-cinder01:23
*** tommylikehu has joined #openstack-cinder01:27
*** erlon__ has quit IRC01:29
*** zhanglong has joined #openstack-cinder01:32
*** tpsilva has quit IRC01:40
openstackgerritwang yong proposed openstack/cinder master: Cinder volume driver for Inspur AS13000 series  https://review.openstack.org/56297701:46
*** mhen has quit IRC01:48
*** mhen has joined #openstack-cinder01:50
*** Dinesh_Bhor has quit IRC01:53
*** Dinesh_Bhor has joined #openstack-cinder02:03
*** lixiaoy1 has joined #openstack-cinder02:23
lixiaoy1@smcginnis_vaca Hi Sean, we are going to submit this driver in Stein release. https://blueprints.launchpad.net/cinder/+spec/nvme-volume-driver Currently this target is set as T release. Can you change it to S release?02:23
openstackgerrithongwei.xu proposed openstack/cinder master: Extract user_id from context as the operator id  https://review.openstack.org/59412402:26
* yikun https://etherpad.openstack.org/p/cinder-stein-meeting-agendas#11602:38
*** Dinesh_Bhor has quit IRC02:38
yikun^ jungleboyj erlon after discussed with tommy and dived in to implementation, I sync the information in meeting etherpad~02:39
yikunyou can check this when you have time.02:40
jungleboyjyikun:  Thanks for the update and for getting tommylikehu in touch with me.02:41
jungleboyjSo we need a bug for that.02:41
yikunjungleboyj: ok, I'd like to do it, and I will try to fix it today.02:44
jungleboyjI am writing the bug.02:45
yikunCool, thanks for doing it, :)02:45
jungleboyjyikun: https://bugs.launchpad.net/cinder/+bug/179850302:46
openstackLaunchpad bug 1798503 in Cinder "When reverting volume to smaller snapshot size is incorrect" [High,Confirmed]02:46
yikunGot it02:46
*** Dinesh_Bhor has joined #openstack-cinder02:47
lixiaoy1@ jungleboyj, Hi Jay, we are going to sumit a new driver in Stein release. But currently in this bp https://blueprints.launchpad.net/cinder/+spec/nvme-volume-driver , the target is T release. Could you update it to Stein release?02:47
lixiaoy1jungleboyj: I have talked with e0ne, and this bp will be done by us02:49
jungleboyjlixiaoy1:  Updated.02:49
lixiaoy1jungleboyj: thank you!02:50
jungleboyjlixiaoy1:  You are welcome.02:50
*** psachin has joined #openstack-cinder02:53
*** dave-mccowan has quit IRC02:56
*** rosmaita has left #openstack-cinder02:58
*** rcernin has quit IRC03:02
*** deepak_mourya__ has joined #openstack-cinder03:12
*** brinzhang has quit IRC03:17
*** brinzhang has joined #openstack-cinder03:18
*** rcernin has joined #openstack-cinder03:28
jungleboyjspotz:  Things might have failed earlier because I missed the scripts/bootstrap-ansible.sh step.03:34
jungleboyjspotz:  It is just as well that I am starting with a fresh system though.03:34
jungleboyjspotz:  Nice how with ansible the error messages are generally clear.03:46
openstackgerritYikun Jiang proposed openstack/cinder master: Forbidden to revert volume to a different size snapshot  https://review.openstack.org/61149103:50
*** Dinesh_Bhor has quit IRC04:05
*** brinzhang has quit IRC04:20
*** TuanDA has joined #openstack-cinder04:40
*** Dinesh_Bhor has joined #openstack-cinder04:44
*** udesale has joined #openstack-cinder04:45
*** chhagarw has joined #openstack-cinder05:00
*** Dinesh_Bhor has quit IRC05:13
*** lbragstad_503 has quit IRC05:27
*** lbragstad_503 has joined #openstack-cinder05:27
*** deepak_mourya__ has quit IRC05:30
*** hamdyk has joined #openstack-cinder06:04
*** Luzi has joined #openstack-cinder06:15
*** TuanDA has quit IRC06:22
*** TuanDA has joined #openstack-cinder06:23
*** phillu has joined #openstack-cinder06:31
openstackgerritYikun Jiang proposed openstack/cinder master: Forbidden to revert volume to a different size snapshot  https://review.openstack.org/61149106:34
*** e0ne has joined #openstack-cinder06:38
*** phillu has quit IRC06:44
openstackgerritRyan Liang proposed openstack/cinder master: Unity: add option to delete attached snapshots  https://review.openstack.org/61151507:02
*** rcernin has quit IRC07:07
*** lpetrut has joined #openstack-cinder07:09
*** phillu_ has joined #openstack-cinder07:17
*** phillu_ has quit IRC07:18
*** phillu has joined #openstack-cinder07:18
*** jcosmao has quit IRC07:22
*** openstackgerrit has quit IRC07:35
*** phillu has quit IRC07:39
*** jwongz has joined #openstack-cinder07:39
jwongzhi,all07:39
jwongzrbd doesn't support replication group?07:40
jwongzI create a blueprint:https://blueprints.launchpad.net/cinder/+spec/ceph-rbd-replication07:41
jwongzwho can be assigned?07:41
*** phillu has joined #openstack-cinder07:42
*** chhagarw has quit IRC07:53
*** dpawlik has quit IRC07:55
*** e0ne has quit IRC08:02
*** TuanDA has quit IRC08:06
*** TuanDA has joined #openstack-cinder08:07
*** e0ne has joined #openstack-cinder08:13
*** dpawlik has joined #openstack-cinder08:13
*** openstackgerrit has joined #openstack-cinder08:14
openstackgerritBrin Zhang proposed openstack/cinder master: Replace resource properity of string with text in quota_usages  https://review.openstack.org/61153008:14
openstackgerritOpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/61153308:22
*** faizy98 has joined #openstack-cinder08:26
*** faizy_ has quit IRC08:30
*** Dinesh_Bhor has joined #openstack-cinder08:37
*** pooja-jadhav has joined #openstack-cinder08:46
*** pooja_jadhav has quit IRC08:46
*** lixiaoy1 has quit IRC08:53
*** dpawlik has quit IRC09:02
*** Dinesh_Bhor has quit IRC09:03
*** dpawlik has joined #openstack-cinder09:04
openstackgerritwanghao proposed openstack/cinder master: Support to query volume filter by changes since/before  https://review.openstack.org/35268809:10
*** gkadam has joined #openstack-cinder09:21
*** gkadam has quit IRC09:21
*** Dinesh_Bhor has joined #openstack-cinder09:28
*** LiangFang has quit IRC09:31
*** Dinesh_Bhor has quit IRC09:31
*** pooja-jadhav is now known as pooja_jadhav09:33
sapd1_whoami-rajat: I have resolved that error :(09:34
whoami-rajatsapd1_: oh sorry. i've been busy with a few tasks lately.09:36
whoami-rajatsapd1_: so you HAVE or HAVEN'T resolved the issue?09:37
sapd1_whoami-rajat: yes. because the mirror type I have set in cinder volumes pool is pool instead of image.09:37
sapd1_whoami-rajat: I have.09:37
sapd1_whoami-rajat: I don't think mirror type pool for pool is not working.09:38
sapd1_whoami-rajat: So I think this is a bug in cinder?09:38
whoami-rajatsapd1_: I'm not sure since i'm not an expert in ceph domain. But yes you can report a bug for the same. we can disucss this with more experienced people in cinder meetings.09:41
openstackgerritYikun Jiang proposed openstack/cinder master: Microversion 3.56 - Support volume re-image  https://review.openstack.org/60634609:50
sapd1_whoami-rajat: Because either pool or image mode can mirror image between two ceph clusters09:51
*** hamdyk has quit IRC09:54
*** hamdyk has joined #openstack-cinder09:55
*** dave-mccowan has joined #openstack-cinder10:19
whoami-rajatsapd1_:  yes. but just to confirm that pool is supported in cinder. that i'm unaware of.10:20
*** phillu has quit IRC10:23
sapd1_whoami-rajat: I will repost this bug.10:27
whoami-rajatsapd1_: sure. please share the bug link after reporting it.10:31
sapd1_whoami-rajat: I think that is not bug belong to cinder. Because cinder-volume is using librbd. so it depends on librbd. :D10:45
*** phillu has joined #openstack-cinder10:50
whoami-rajatsapd1_: ok good. but librbd supports the functionality.10:56
*** chhagarw has joined #openstack-cinder10:57
*** zhanglong has quit IRC10:59
*** udesale has quit IRC11:04
*** erlon__ has joined #openstack-cinder11:11
*** luizbag has joined #openstack-cinder11:15
*** icey has quit IRC11:30
*** icey has joined #openstack-cinder11:34
*** s10 has joined #openstack-cinder11:37
*** chhagarw has quit IRC11:39
*** ganso has joined #openstack-cinder11:48
*** e0ne has quit IRC11:50
openstackgerritMaciej Szwed proposed openstack/cinder master: Adding SPDK NVMe-oF target driver  https://review.openstack.org/57275911:50
openstackgerritMaciej Szwed proposed openstack/cinder master: Adding SPDK volume driver  https://review.openstack.org/56422911:50
openstackgerritMaciej Szwed proposed openstack/cinder master: SPDK drivers documentation  https://review.openstack.org/60927611:50
*** abishop has joined #openstack-cinder11:57
*** rosmaita has joined #openstack-cinder11:59
*** faizy98 has quit IRC12:00
*** faizy98 has joined #openstack-cinder12:01
*** tommylikehu has quit IRC12:07
*** TuanDA has quit IRC12:07
* smcginnis_vaca passing by12:09
smcginnis_vacaReading through the meeting logs. I do remember at the Design Summit in Barcelona we identified volume extend as being an issue and said we would not allow reverting to snapshot if the volume had been extended because of that.12:10
yikunsmcginnis_vaca: yes, the generic implementation use "dd" to copy snap-temp-volume to target volume, so it a problem to copy volume after extending12:13
yikunthe generic implementation means if the driver not support revert_snapshot_to_volume, we have a implementation revert_snapshot_to_volume_generic to instead.12:15
s10Does Cinder have an option to limit the size of volume which is allowed to be uploaded to the image? I see, that anyone can start action upload-to-image for large volume (for example, 4Tb), if policy allows it (volume_extension:volume_actions:upload_image is admin_or_owner by default). There is an image_size_cap option in Glance, which is defaulted to 1Tb, but cinder-volume starts volume export before upload, so image_conversion_dir will be filled upto 4T12:18
s10b if there is enough space, otherwise it will be DoS for cinder-volume node, if image_conversion_dir is not on a separate system partition, and only then it will face Glance cap.12:18
s10Where is function check_available_space() is being called in cinder-volume service while preparing to upload raw cinder volume to raw glance image?12:26
*** DHE has left #openstack-cinder12:27
*** eharney has joined #openstack-cinder12:27
yikuns10: now the check_available_space() is only called when we create volume from a image12:29
yikunhttps://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L76812:29
s10https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L1429 - looks like there is no such check12:29
*** dims has quit IRC12:30
openstackgerritYikun Jiang proposed openstack/cinder master: Microversion 3.56 - Support volume re-image  https://review.openstack.org/60634612:31
*** dims has joined #openstack-cinder12:33
*** lpetrut has quit IRC12:33
*** gkadam has joined #openstack-cinder12:34
s10I will fill a bug about this issue. I believe there should be a free space check.12:36
*** mriedem has joined #openstack-cinder12:38
*** psachin has quit IRC12:50
s10https://bugs.launchpad.net/cinder/+bug/179858912:52
openstackLaunchpad bug 1798589 in Cinder "No available space check for image_conversion_dir in cinder-volume on upload-to-image" [Undecided,New]12:52
spotzjungleboyj: let me know how it went12:54
jungleboyjspotz:  will do12:56
*** e0ne has joined #openstack-cinder13:07
openstackgerritChuck Short proposed openstack/cinder master: Remove touch command from volume.filters  https://review.openstack.org/61051413:19
jungleboyjspotz: If there isn't a .aio config file in the examples directory does that mean you can't do it as part of an AIO deployment or just that it is the same whether you are doing AIO or not.13:20
*** mvkr has quit IRC13:27
*** lbragstad_503 is now known as lbragstad13:32
*** lixiaoy1 has joined #openstack-cinder13:37
spotzjungleboyj: which role/project?13:39
jungleboyjspotz:  Barbican13:39
spotzjungleboyj: Hrm, let me look it may be aio friendly cause I know Magnum can be installed and that's a requirement for it13:40
jungleboyjspotz:  Just know we have a lot of discussion around it for Cinder and thought it might be good to include if I need to poke at it in the future.13:41
*** LiangFang has joined #openstack-cinder13:42
spotzjungleboyj: You might want to set it to the one infra host but looking at the role you should be fine13:44
jungleboyjspotz:  What do you mean.13:45
spotzjungleboyj: The difference between the .aio and .example is the number of infra hosts13:46
spotz1 vs 313:46
*** jcosmao has joined #openstack-cinder13:47
jungleboyjOh,  gotcha.13:49
spotzjungleboyj: checked on channle, work has been done for rocky/master - https://review.openstack.org/#/q/I10a07d5e4b2b3fbf122a79f15e27a9ca3479922113:51
*** mchlumsky has joined #openstack-cinder13:52
*** mvkr has joined #openstack-cinder13:57
*** mchlumsky has quit IRC13:57
*** gkadam has quit IRC13:59
*** mchlumsky has joined #openstack-cinder13:59
*** gkadam has joined #openstack-cinder13:59
lixiaoy1We are going to support a new Cinder driver. From document, snapshot support is mandatory. But the new backend can't support snapshot, if I raise NOT_implemented in create_snap, is that ok?14:14
lixiaoy1Anybody can help to answer?14:14
openstackgerritRuben de Smet proposed openstack/cinder master: Adds peer persistence feature to HPE 3PAR driver  https://review.openstack.org/61162014:14
*** phillu has quit IRC14:26
*** hamdykhader has joined #openstack-cinder14:26
*** hamdyk has quit IRC14:29
*** gkadam_ has joined #openstack-cinder14:33
*** lemko has joined #openstack-cinder14:35
*** gkadam has quit IRC14:35
*** hamdykhader has quit IRC14:38
jungleboyjspotz: Ok, I was using queens at this point.  So, I won't mess with that yet.14:51
spotzok14:51
*** jmlowe has quit IRC14:57
*** s10 has quit IRC14:59
*** Luzi has quit IRC15:01
*** itlinux has quit IRC15:01
openstackgerritChuck Short proposed openstack/cinder master: Remove 'ln' command from rootwrap filter  https://review.openstack.org/61164515:02
openstackgerritChuck Short proposed openstack/cinder master: Drop drv_cfg from rootwrap filters  https://review.openstack.org/61164615:04
*** dklyle has joined #openstack-cinder15:06
*** e0ne has quit IRC15:12
lixiaoy1Any cores help to review the spec "Driver reinitialization after failure" https://review.openstack.org/#/c/599505/ ?15:14
jungleboyjlixiaoy1:  Sure.  I will try to take alook.15:15
lixiaoy1jungleboyj: thank you very much15:15
*** eharney has quit IRC15:16
*** e0ne has joined #openstack-cinder15:18
jungleboyjlixiaoy1:  Thanks for the Ping.  Not doing a good job of keeping up with Specs lately.15:21
lixiaoy1jungleboyj: no, no, understand you are always occupied.15:22
jungleboyjlixiaoy1:  Yeah.  Especially lately.15:23
lixiaoy1jungleboyj: take care :)15:26
jungleboyjlixiaoy1:  Thanks.  You too.15:26
*** phillu has joined #openstack-cinder15:28
*** lixiaoy1 has quit IRC15:30
openstackgerritChuck Short proposed openstack/cinder master: Remove umount from volume.filters  https://review.openstack.org/61165415:36
*** gkadam_ has quit IRC15:38
openstackgerritChuck Short proposed openstack/cinder master: Remove mount from volume.filters  https://review.openstack.org/61165715:42
*** LiangFang has quit IRC15:47
*** itlinux has joined #openstack-cinder15:51
*** eharney has joined #openstack-cinder16:00
*** e0ne has quit IRC16:00
*** phillu has quit IRC16:02
*** mriedem is now known as mriedem_lunch16:10
*** dklyle has quit IRC16:16
*** dklyle has joined #openstack-cinder16:16
*** dklyle has quit IRC16:37
*** s10 has joined #openstack-cinder16:50
*** s10 has quit IRC16:51
*** mvkr has quit IRC16:58
*** ircuser-1 has quit IRC17:15
*** dklyle has joined #openstack-cinder17:30
*** mvkr has joined #openstack-cinder17:41
*** dklyle has quit IRC17:42
*** dklyle has joined #openstack-cinder17:50
*** mriedem_lunch is now known as mriedem17:51
mriedemyikes http://logs.openstack.org/20/611020/2/experimental/grenade-postgresql/8be4557/logs/screen-c-vol.txt.gz?level=TRACE17:54
*** dklyle has quit IRC18:02
mriedemi think what's happening here ^ is a volume attachment delete request times out, the attachment record is deleted, but then on restart of c-vol it's trying to lazy-load the volume->attachment->volume which blows up18:09
*** hedvig_01 has joined #openstack-cinder18:13
hedvig_01When I run the create backup command, I see "Trying to connect to iSCSI portal.  Couldn't find iscsi sessions because iscsiadm err: iscsiadm: No active sessions " in the backup log. Is it okay to see this log statement?18:14
hedvig_01UI shows no error.18:15
*** lbragstad has quit IRC18:17
*** dklyle has joined #openstack-cinder18:20
*** s10 has joined #openstack-cinder18:23
s10Can https://bugs.launchpad.net/cinder/+bug/1798589 be considered as security issue, because it allows in some cased to DoS cinder-volume host via disk space exhaustion?18:25
openstackLaunchpad bug 1798589 in Cinder "No available space check for image_conversion_dir in cinder-volume on upload-to-image" [Undecided,New]18:25
*** dklyle has quit IRC18:29
openstackgerritChuck Short proposed openstack/cinder master: Remove scsi_id from rootwrap filter  https://review.openstack.org/61169318:36
openstackgerritChuck Short proposed openstack/cinder master: Remove drbdadm from filters  https://review.openstack.org/61169418:38
*** lbragstad has joined #openstack-cinder18:48
openstackgerritChuck Short proposed openstack/cinder master: Switch ietadm to use olso.privsep  https://review.openstack.org/61169718:53
*** dklyle has joined #openstack-cinder19:07
*** dklyle has quit IRC19:14
*** itlinux has quit IRC19:16
*** lbragstad has quit IRC19:21
*** lbragstad has joined #openstack-cinder19:24
*** tpsilva has joined #openstack-cinder19:33
eharneyhmm.. zuul is commenting with pylint errors now?  https://review.openstack.org/#/c/606346/19:39
imacdonnit's non-voting19:40
eharneyright, but you will never be able to find review comments if there 100+ comments from that...19:40
imacdonnoh, I see19:40
eharneyand pylint always trips up on cinder.objects, so there will be a lot of them19:41
imacdonnyeah, that is a tad noisy19:41
imacdonnI don't usually pay attention to zuul comments ... just look at anything that failed, and dig into the logs19:42
eharneyi don't recall ever seeing comments like this before19:42
imacdonnoh, I'm slowly catching up .. it's actually putting comments in the files .. ugh19:43
eharneyyes19:43
* eharney asks -infra19:47
imacdonnyeah, that must be new .. didn't happen on a change I submitted yesterday: https://review.openstack.org/61146319:47
* imacdonn will listen over there19:47
*** e0ne has joined #openstack-cinder19:52
openstackgerritEric Harney proposed openstack/python-cinderclient master: WIP: Fix doc build error  https://review.openstack.org/61170820:01
*** eharney has quit IRC20:03
*** e0ne has quit IRC20:05
*** luizbag has quit IRC20:09
*** openstackgerrit has quit IRC20:36
*** pcaruana has quit IRC20:44
*** openstackgerrit has joined #openstack-cinder20:56
openstackgerritMerged openstack/cinder master: Imported Translations from Zanata  https://review.openstack.org/61153320:56
*** dustins has quit IRC20:57
*** erlon__ has quit IRC21:01
*** itlinux has joined #openstack-cinder21:06
*** abishop has quit IRC21:34
*** dklyle has joined #openstack-cinder21:36
*** rosmaita has left #openstack-cinder21:42
*** dklyle has quit IRC21:45
*** mriedem has quit IRC22:12
*** mchlumsky has quit IRC22:21
*** rcernin has joined #openstack-cinder22:33
*** lemko has quit IRC22:34
*** ircuser-1 has joined #openstack-cinder22:52
*** tpsilva has quit IRC23:03
*** jiaopengju has quit IRC23:05
*** jiaopengju has joined #openstack-cinder23:08
*** erlon__ has joined #openstack-cinder23:18
*** ganso has quit IRC23:22
*** jiaopengju has quit IRC23:35
*** jiaopengju has joined #openstack-cinder23:36

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