*** mriedem1 has joined #openstack-cinder | 00:05 | |
*** mriedem has quit IRC | 00:05 | |
jungleboyj | :-) Sorry, exhausted and a little slap happy. | 00:06 |
---|---|---|
*** mriedem1 is now known as mriedem | 00:08 | |
*** yangyapeng has quit IRC | 00:11 | |
*** yangyapeng has joined #openstack-cinder | 00:12 | |
*** ntpttr_laptop has quit IRC | 00:13 | |
*** abhitechie has joined #openstack-cinder | 00:16 | |
*** moshele has quit IRC | 00:27 | |
*** hemna_ has joined #openstack-cinder | 00:34 | |
*** gkadam has joined #openstack-cinder | 00:34 | |
*** nicolasbock has quit IRC | 00:35 | |
*** stakeda has joined #openstack-cinder | 00:37 | |
*** Apoorva_ has joined #openstack-cinder | 00:40 | |
*** nicolasbock has joined #openstack-cinder | 00:42 | |
*** Apoorva has quit IRC | 00:43 | |
*** markstur has quit IRC | 00:46 | |
*** markstur has joined #openstack-cinder | 00:47 | |
*** MeltedLux has joined #openstack-cinder | 00:50 | |
*** markstur has quit IRC | 00:51 | |
*** Apoorva_ has quit IRC | 00:54 | |
*** Shunli has joined #openstack-cinder | 00:58 | |
tommylikehu | hey jungleboyj , ildikov do you mean to use policy to limit multiattach volume's attachments that only only one could been set with R/W flag? | 01:09 |
jungleboyj | tommylikehu: using policy to control whether we have read only or read right attachments for volumes. We need to consider both. | 01:11 |
tommylikehu | jungleboyj: it's not related how many r/o or r/w attachments we can have for one specific volume, right? | 01:14 |
jungleboyj | No. That would be something to try to figure out in the future. Initially it would just be setting policies as to whether or who can do read only versus read write. | 01:17 |
tommylikehu | jungleboyj: same to this case? https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1405 | 01:17 |
openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix adjusted_free_virtual in CapacityFilter https://review.openstack.org/507539 | 01:23 |
*** shaner has quit IRC | 01:33 | |
*** shaner has joined #openstack-cinder | 01:34 | |
*** Liu_ has joined #openstack-cinder | 01:45 | |
*** crushil has joined #openstack-cinder | 01:46 | |
openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for backup resource https://review.openstack.org/507015 | 01:48 |
*** markstur has joined #openstack-cinder | 01:49 | |
*** markstur has quit IRC | 01:54 | |
*** Apoorva has joined #openstack-cinder | 01:54 | |
*** abhinavtechie has joined #openstack-cinder | 02:02 | |
*** lhx__ has joined #openstack-cinder | 02:03 | |
*** abhitechie has quit IRC | 02:05 | |
*** Liu_ has quit IRC | 02:08 | |
*** abhinavtechie has quit IRC | 02:12 | |
jungleboyj | tommylikehu: I think it would be a little more complicated than that as it would be using policy to determine who could attach read/write and read-only . | 02:21 |
tommylikehu | jungleboyj: ok | 02:22 |
jungleboyj | tommylikehu: Do we have any place else where we do anything like that/ | 02:22 |
tommylikehu | jungleboyj: so your point is to add more granularity in that case, not only the project owner or admin. | 02:26 |
*** lhx__ has quit IRC | 02:28 | |
*** lhx__ has joined #openstack-cinder | 02:29 | |
*** zhaochao has quit IRC | 02:29 | |
*** zhaochao has joined #openstack-cinder | 02:44 | |
*** erlon has quit IRC | 02:48 | |
*** thurloat has quit IRC | 02:50 | |
*** thurloat has joined #openstack-cinder | 02:50 | |
*** csuttles has quit IRC | 02:52 | |
*** csuttles has joined #openstack-cinder | 02:53 | |
*** csuttles has quit IRC | 02:54 | |
*** csuttles has joined #openstack-cinder | 02:55 | |
*** ntpttr_laptop has joined #openstack-cinder | 03:00 | |
jungleboyj | tommylikehu: Well, I think the starting point is to use policy with admin or not admin. | 03:11 |
jungleboyj | tommylikehu: I need to think about it more though. | 03:11 |
*** itlinux has joined #openstack-cinder | 03:12 | |
tommylikehu | jungleboyj: good! | 03:12 |
ildikov | jungleboyj: tommylikehu: we would like to have the ability to turn this on and off first | 03:14 |
ildikov | We just couldn't figure it out yesterday with johnthetubaguy that how R/W and R/O right now after a quick look | 03:15 |
tommylikehu | ildikov: extend attach volume has the same case | 03:15 |
tommylikehu | https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L1405 | 03:15 |
ildikov | So I thought bring up both here as a heads up where we need help on | 03:15 |
openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for snapshot resource https://review.openstack.org/507721 | 03:15 |
*** AlexeyAbashkin has quit IRC | 03:15 | |
ildikov | tommylikehu: yep, I saw the link | 03:15 |
ildikov | tommylikehu: we might want more variations than that in the future, but maybe some of that should happen on the Nova side | 03:16 |
*** AlexeyAbashkin has joined #openstack-cinder | 03:17 | |
ildikov | tommylikehu: I don't see our options clearly on this one yet as we didn't really play with R/W and R/O that much in the past | 03:17 |
jungleboyj | ildikov: You are still up? | 03:18 |
ildikov | jungleboyj: no, I slept 2-2.5 hours | 03:18 |
ildikov | jungleboyj: going to the airport right now | 03:19 |
jungleboyj | So I have been thinking about this since we talked about this earlier and I seem to have myself wrapped around the axle. | 03:19 |
tommylikehu | ildikov: safe tri[ | 03:19 |
tommylikehu | trip | 03:19 |
ildikov | tommylikehu: thank you! | 03:19 |
jungleboyj | ildikov: Hope you can get some sleep on the plane. | 03:20 |
ildikov | Will try :) | 03:20 |
*** nicolasbock has quit IRC | 03:21 | |
jungleboyj | tommylikehu: Maybe I am not understanding what you are thinking with the extend attached volume? | 03:21 |
tommylikehu | jungleboyj: guess you do. That one only provides seperate policy for difference cases( attached or nor) | 03:23 |
jungleboyj | With the extend attached volume it is ... ok that was just what I was about to say. | 03:24 |
tommylikehu | need more investigate on that. | 03:25 |
*** vivsoni has joined #openstack-cinder | 03:26 | |
jungleboyj | tommylikehu: Yeah, because here we are talking about two different options for a policy. | 03:29 |
jungleboyj | tommylikehu: So, the basic case would be to be able to enable or disable attaching read/write for a multi-attached volume. | 03:30 |
jungleboyj | The problem is that John is proposing that 'shared_volume' be the default case which seems to make having a policy for it useless. | 03:31 |
*** yangyapeng has quit IRC | 03:32 | |
*** yangyapeng has joined #openstack-cinder | 03:32 | |
jungleboyj | If we had 'shared_volume' as False by default then we could have a policy that when 'true' only admin could attach read/write or something. | 03:32 |
tommylikehu | ok | 03:34 |
jungleboyj | tommylikehu: Am I making any sense> | 03:34 |
tommylikehu | jungleboyj: sure | 03:34 |
tommylikehu | jungleboyj: but nothing come to mind for your case now, sorry | 03:36 |
*** yangyapeng has quit IRC | 03:37 | |
jungleboyj | tommylikehu: Ok, no worries. I will keep thinking about it and maybe chat with smcginnis and get his inputs. | 03:37 |
tommylikehu | jungleboyj: ha, great! | 03:38 |
*** itlinux has quit IRC | 03:41 | |
jungleboyj | tommylikehu: Let me know if you have any thoughts. | 03:43 |
*** lhx__ has quit IRC | 03:43 | |
tommylikehu | jungleboyj: no problem | 03:44 |
*** lhx__ has joined #openstack-cinder | 03:44 | |
*** ntpttr_laptop has quit IRC | 03:57 | |
*** crushil has quit IRC | 03:57 | |
*** lkuchlan has joined #openstack-cinder | 03:58 | |
*** markstur has joined #openstack-cinder | 04:08 | |
*** markstur has quit IRC | 04:13 | |
*** arnewiebalck_ has joined #openstack-cinder | 04:15 | |
*** wes_dillingham has quit IRC | 04:18 | |
*** zengyingzhe__ has joined #openstack-cinder | 04:20 | |
*** Apoorva_ has joined #openstack-cinder | 04:21 | |
*** csuttles_ has joined #openstack-cinder | 04:27 | |
*** csuttles has quit IRC | 04:29 | |
*** thurloat has quit IRC | 04:29 | |
*** Apoorva has quit IRC | 04:29 | |
*** hemna_ has quit IRC | 04:29 | |
*** mriedem has quit IRC | 04:29 | |
*** zengyingzhe_ has quit IRC | 04:29 | |
*** danpawlik has quit IRC | 04:29 | |
*** cburgess has quit IRC | 04:29 | |
*** lhinds has quit IRC | 04:29 | |
*** zeestrat has quit IRC | 04:29 | |
*** ericyoung has quit IRC | 04:29 | |
*** stephenfin has quit IRC | 04:29 | |
*** obre has quit IRC | 04:29 | |
*** thurloat has joined #openstack-cinder | 04:30 | |
*** stephenfin has joined #openstack-cinder | 04:30 | |
*** lhinds has joined #openstack-cinder | 04:30 | |
*** cburgess has joined #openstack-cinder | 04:30 | |
*** ericyoung has joined #openstack-cinder | 04:30 | |
*** john5223_ has quit IRC | 04:32 | |
*** mdbooth has quit IRC | 04:34 | |
*** crushil has joined #openstack-cinder | 04:34 | |
*** abhitechie has joined #openstack-cinder | 04:34 | |
*** obre has joined #openstack-cinder | 04:36 | |
*** zeestrat has joined #openstack-cinder | 04:36 | |
*** hemna_ has joined #openstack-cinder | 04:37 | |
*** danpawlik has joined #openstack-cinder | 04:37 | |
*** ganso has quit IRC | 04:38 | |
*** Apoorva_ has quit IRC | 04:40 | |
vivsoni | Anyone facing jenkins 'gate-cinder-python35' failure | 04:40 |
vivsoni | ft8.1: subunit.parser_StringException: Binary content: | 04:40 |
vivsoni | Packet data (application/octet-stream) | 04:40 |
vivsoni | i assume this is something to do with infra | 04:41 |
*** armax has joined #openstack-cinder | 04:42 | |
*** Shunli has quit IRC | 04:42 | |
*** chhavi has joined #openstack-cinder | 04:42 | |
vivsoni | http://logs.openstack.org/72/506072/7/gate/gate-cinder-python35/9f7f2c6/console.html | 04:42 |
*** markstur has joined #openstack-cinder | 04:46 | |
*** lhx__ has quit IRC | 04:46 | |
*** psachin has joined #openstack-cinder | 04:47 | |
*** mdbooth has joined #openstack-cinder | 04:48 | |
*** markstur has quit IRC | 04:50 | |
*** lhx__ has joined #openstack-cinder | 05:02 | |
*** lhx__ has quit IRC | 05:08 | |
*** zhurong has joined #openstack-cinder | 05:13 | |
*** crushil has quit IRC | 05:21 | |
*** markstur has joined #openstack-cinder | 05:23 | |
*** arnewiebalck_ has quit IRC | 05:25 | |
*** markstur has quit IRC | 05:28 | |
*** amoralej has quit IRC | 05:53 | |
*** ChubYann has quit IRC | 05:57 | |
*** markstur has joined #openstack-cinder | 06:00 | |
*** lkuchlan has quit IRC | 06:03 | |
*** lkuchlan has joined #openstack-cinder | 06:03 | |
*** lkuchlan is now known as lkuchlan_wfh | 06:03 | |
*** markstur has quit IRC | 06:04 | |
openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix free_capacity https://review.openstack.org/507746 | 06:10 |
openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix free_capacity https://review.openstack.org/507747 | 06:10 |
*** pgadiya has joined #openstack-cinder | 06:16 | |
openstackgerrit | Ghanshyam Mann proposed openstack/cinder master: Fix description for volume_type object in API Ref https://review.openstack.org/425461 | 06:18 |
*** armax has quit IRC | 06:19 | |
*** moshele has joined #openstack-cinder | 06:36 | |
*** markstur has joined #openstack-cinder | 06:36 | |
*** moshele has quit IRC | 06:38 | |
openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix free_capacity https://review.openstack.org/507752 | 06:38 |
*** rcernin has joined #openstack-cinder | 06:39 | |
*** e0ne has joined #openstack-cinder | 06:40 | |
*** armax has joined #openstack-cinder | 06:41 | |
*** markstur has quit IRC | 06:41 | |
*** armax has quit IRC | 06:50 | |
*** pgadiya has quit IRC | 06:55 | |
*** lhx__ has joined #openstack-cinder | 06:55 | |
*** kwathore has quit IRC | 07:05 | |
*** e0ne has quit IRC | 07:08 | |
*** bhagyashri_s has joined #openstack-cinder | 07:11 | |
*** ps_jadhav has joined #openstack-cinder | 07:11 | |
*** bhagyashri_s has quit IRC | 07:13 | |
*** bhagyashris has quit IRC | 07:14 | |
*** pooja_jadhav has quit IRC | 07:14 | |
*** markstur has joined #openstack-cinder | 07:14 | |
*** ps_jadhav has quit IRC | 07:14 | |
*** Dinesh_Bhor has quit IRC | 07:17 | |
*** markstur has quit IRC | 07:18 | |
openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for message, worker, cluster resources https://review.openstack.org/507768 | 07:19 |
*** bhagyashris has joined #openstack-cinder | 07:20 | |
*** pooja_jadhav has joined #openstack-cinder | 07:21 | |
*** Dinesh_Bhor has joined #openstack-cinder | 07:21 | |
*** tesseract has joined #openstack-cinder | 07:21 | |
*** moshele has joined #openstack-cinder | 07:37 | |
*** pooja-jadhav has joined #openstack-cinder | 07:46 | |
*** amoralej has joined #openstack-cinder | 07:47 | |
*** arnewiebalck_ has joined #openstack-cinder | 07:48 | |
*** pooja_jadhav has quit IRC | 07:48 | |
*** markstur has joined #openstack-cinder | 07:51 | |
*** alexchadin has joined #openstack-cinder | 07:53 | |
*** markstur has quit IRC | 07:55 | |
*** lkuchlan_wfh has quit IRC | 07:57 | |
*** lkuchlan_wfh has joined #openstack-cinder | 07:57 | |
*** zhurong has quit IRC | 08:11 | |
*** arnewiebalck_ has quit IRC | 08:13 | |
*** kwathore has joined #openstack-cinder | 08:19 | |
*** markstur has joined #openstack-cinder | 08:28 | |
*** zhurong has joined #openstack-cinder | 08:29 | |
*** markstur has quit IRC | 08:32 | |
openstackgerrit | Merged openstack/cinder master: 3PAR: Cinder volume revert to snapshot support https://review.openstack.org/506072 | 08:38 |
*** alexchadin has quit IRC | 08:41 | |
*** alexchadin has joined #openstack-cinder | 08:42 | |
*** alexchadin has quit IRC | 08:48 | |
*** chlong has quit IRC | 08:49 | |
*** lhx__ has quit IRC | 08:58 | |
*** lhx__ has joined #openstack-cinder | 08:59 | |
*** markstur has joined #openstack-cinder | 09:05 | |
*** alexchadin has joined #openstack-cinder | 09:07 | |
*** alexchadin has quit IRC | 09:08 | |
*** alexchadin has joined #openstack-cinder | 09:08 | |
*** markstur has quit IRC | 09:09 | |
arnewiebalck | Are there any activities already to integrate the Ceph/Luminous’ feature to defer the deletion of an image into Cinder? I’m thinking of: replace removal by move-to-trash and have a periodic task to clean up images marked as trash after a configurable timeout. This should help in case of accidental deletion and would make volume “deletion” instantaneous from the user’s pov. | 09:10 |
*** alexchadin has quit IRC | 09:11 | |
*** e0ne has joined #openstack-cinder | 09:16 | |
*** mvk has quit IRC | 09:34 | |
*** gkadam has quit IRC | 09:35 | |
*** josecastroleon has joined #openstack-cinder | 09:41 | |
*** markstur has joined #openstack-cinder | 09:41 | |
*** markstur has quit IRC | 09:46 | |
*** afazekas is now known as afazekas|seek4fo | 09:48 | |
*** sdague has joined #openstack-cinder | 09:50 | |
*** josecastroleon has quit IRC | 10:00 | |
*** stakeda has quit IRC | 10:02 | |
openstackgerrit | TommyLike proposed openstack/cinder master: [policy in code] Add support for group, g-snapshot resources https://review.openstack.org/507812 | 10:03 |
openstackgerrit | Dongdong Pei proposed openstack/cinder master: Fix free_capacity https://review.openstack.org/507752 | 10:05 |
*** pooja-jadhav is now known as pooja_jadhav | 10:11 | |
*** markstur has joined #openstack-cinder | 10:18 | |
*** zhurong has quit IRC | 10:20 | |
openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: change 'tenant' to 'project' in v2 doc https://review.openstack.org/507813 | 10:21 |
*** markstur has quit IRC | 10:23 | |
*** mvk has joined #openstack-cinder | 10:48 | |
openstackgerrit | Pooja Jadhav proposed openstack/cinder master: Extracted HTTP response codes to constants https://review.openstack.org/434898 | 10:48 |
chhavi | review request: https://review.openstack.org/#/c/507361/ | 10:50 |
chhavi | hemna_: https://review.openstack.org/#/c/504482/ | 10:51 |
*** markstur has joined #openstack-cinder | 10:56 | |
*** hemna_ has quit IRC | 10:57 | |
*** nicolasbock has joined #openstack-cinder | 11:00 | |
*** markstur has quit IRC | 11:01 | |
*** nicolasbock has quit IRC | 11:04 | |
*** strigazi has quit IRC | 11:23 | |
*** strigazi has joined #openstack-cinder | 11:24 | |
*** _pewp_ has quit IRC | 11:24 | |
*** pcaruana has joined #openstack-cinder | 11:27 | |
*** lhx_ has joined #openstack-cinder | 11:28 | |
*** _alastor__ has quit IRC | 11:28 | |
*** _alastor__ has joined #openstack-cinder | 11:30 | |
*** _pewp_ has joined #openstack-cinder | 11:31 | |
*** lhx__ has quit IRC | 11:31 | |
*** markstur has joined #openstack-cinder | 11:34 | |
*** markstur has quit IRC | 11:39 | |
*** lhx_ has quit IRC | 11:43 | |
*** abhitechie has quit IRC | 11:49 | |
*** dave-mccowan has joined #openstack-cinder | 11:50 | |
*** abishop has joined #openstack-cinder | 11:55 | |
*** dave-mccowan has quit IRC | 12:01 | |
*** dave-mccowan has joined #openstack-cinder | 12:04 | |
*** afazekas|seek4fo is now known as afazekas | 12:09 | |
*** markstur has joined #openstack-cinder | 12:11 | |
*** lkuchlan_wfh has quit IRC | 12:14 | |
*** wes_dillingham has joined #openstack-cinder | 12:14 | |
*** markstur has quit IRC | 12:16 | |
*** alexchadin has joined #openstack-cinder | 12:16 | |
*** edmondsw has joined #openstack-cinder | 12:17 | |
*** ganso has joined #openstack-cinder | 12:18 | |
*** moshele has quit IRC | 12:29 | |
*** lhx_ has joined #openstack-cinder | 12:30 | |
*** gouthamr has joined #openstack-cinder | 12:30 | |
*** yangyapeng has joined #openstack-cinder | 12:34 | |
*** gouthamr_ has joined #openstack-cinder | 12:36 | |
*** yangyapeng has quit IRC | 12:38 | |
*** gouthamr has quit IRC | 12:38 | |
*** lhx_ has quit IRC | 12:39 | |
*** gcb has quit IRC | 12:45 | |
*** amoralej is now known as amoralej|lunch | 12:48 | |
*** jmlowe has quit IRC | 12:48 | |
*** alexchadin has quit IRC | 12:55 | |
*** alexchadin has joined #openstack-cinder | 12:56 | |
*** eharney has joined #openstack-cinder | 13:01 | |
*** liverpooler has joined #openstack-cinder | 13:03 | |
*** tpsilva has joined #openstack-cinder | 13:03 | |
*** mriedem has joined #openstack-cinder | 13:09 | |
*** jistr is now known as jistr|call | 13:12 | |
*** moshele has joined #openstack-cinder | 13:13 | |
*** lucasxu has joined #openstack-cinder | 13:14 | |
*** MVenesio has joined #openstack-cinder | 13:16 | |
*** amoralej|lunch is now known as amoralej | 13:23 | |
*** lbragstad has joined #openstack-cinder | 13:27 | |
*** sbezverk has joined #openstack-cinder | 13:27 | |
*** chlong has joined #openstack-cinder | 13:29 | |
*** josecastroleon has joined #openstack-cinder | 13:30 | |
*** chlong has quit IRC | 13:31 | |
*** jmlowe has joined #openstack-cinder | 13:34 | |
*** lifeless has quit IRC | 13:46 | |
*** alexchadin has quit IRC | 13:51 | |
*** crushil has joined #openstack-cinder | 13:53 | |
*** lifeless has joined #openstack-cinder | 13:54 | |
*** dave-mccowan has quit IRC | 13:54 | |
*** dustins has joined #openstack-cinder | 13:54 | |
*** gkadam has joined #openstack-cinder | 13:56 | |
*** hemna_ has joined #openstack-cinder | 13:58 | |
*** rtjure has quit IRC | 13:58 | |
*** rtjure has joined #openstack-cinder | 14:03 | |
openstackgerrit | junbo.li proposed openstack/cinder master: tests: replace .testr.conf with .stestr.conf https://review.openstack.org/507866 | 14:07 |
*** felipemonteiro has joined #openstack-cinder | 14:10 | |
*** felipemonteiro_ has joined #openstack-cinder | 14:12 | |
*** haobing has joined #openstack-cinder | 14:14 | |
*** felipemonteiro has quit IRC | 14:16 | |
*** haobing has quit IRC | 14:22 | |
*** hemna_ has quit IRC | 14:25 | |
*** dave-mccowan has joined #openstack-cinder | 14:26 | |
*** ntpttr_laptop has joined #openstack-cinder | 14:27 | |
*** jistr|call is now known as jistr | 14:27 | |
openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Remove deprecated keymgr code https://review.openstack.org/468252 | 14:28 |
*** jmlowe has quit IRC | 14:31 | |
*** hemna_ has joined #openstack-cinder | 14:34 | |
*** jmlowe has joined #openstack-cinder | 14:37 | |
*** stvnoyes has quit IRC | 14:38 | |
*** dave-mccowan has quit IRC | 14:38 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Remove legacy driver name mappings https://review.openstack.org/507879 | 14:38 |
*** josecastroleon has quit IRC | 14:39 | |
*** dave-mccowan has joined #openstack-cinder | 14:44 | |
*** lhx_ has joined #openstack-cinder | 14:47 | |
*** jmlowe has quit IRC | 14:48 | |
*** jmlowe has joined #openstack-cinder | 14:48 | |
*** dave-mcc_ has joined #openstack-cinder | 14:49 | |
*** dave-mccowan has quit IRC | 14:51 | |
*** crose has joined #openstack-cinder | 14:52 | |
*** markstur has joined #openstack-cinder | 14:54 | |
*** lhx_ has quit IRC | 15:01 | |
*** ntpttr_laptop has quit IRC | 15:02 | |
*** jmlowe has quit IRC | 15:03 | |
*** jmlowe has joined #openstack-cinder | 15:05 | |
*** yangyapeng has joined #openstack-cinder | 15:08 | |
*** stvnoyes has joined #openstack-cinder | 15:10 | |
openstackgerrit | Ghanshyam Mann proposed openstack/cinder master: Fix api-ref for reset group snapshot https://review.openstack.org/507892 | 15:12 |
*** chlong has joined #openstack-cinder | 15:17 | |
*** josecastroleon has joined #openstack-cinder | 15:21 | |
*** moshele has quit IRC | 15:25 | |
*** gouthamr_ has quit IRC | 15:26 | |
*** lpetrut has joined #openstack-cinder | 15:27 | |
*** lhx_ has joined #openstack-cinder | 15:29 | |
*** jmlowe has quit IRC | 15:32 | |
*** kwathore has quit IRC | 15:36 | |
*** psachin has quit IRC | 15:40 | |
*** gkadam has quit IRC | 15:40 | |
*** rcernin has quit IRC | 15:43 | |
*** crushil_ has joined #openstack-cinder | 15:43 | |
*** josecastroleon has quit IRC | 15:44 | |
*** erlon has joined #openstack-cinder | 15:55 | |
*** xyang1 has joined #openstack-cinder | 15:56 | |
*** markstur_ has joined #openstack-cinder | 15:59 | |
*** bswartz has joined #openstack-cinder | 16:01 | |
*** markstur has quit IRC | 16:02 | |
*** sbezverk has quit IRC | 16:03 | |
*** esker has joined #openstack-cinder | 16:04 | |
*** Apoorva has joined #openstack-cinder | 16:04 | |
*** crushil_ has quit IRC | 16:05 | |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder master: Add ploop to parallels naming conversion https://review.openstack.org/507193 | 16:05 |
*** ntpttr_laptop has joined #openstack-cinder | 16:07 | |
*** ntpttr_laptop has quit IRC | 16:08 | |
*** r-daneel has joined #openstack-cinder | 16:12 | |
*** hemna_ has quit IRC | 16:15 | |
*** crushil has quit IRC | 16:23 | |
*** tesseract has quit IRC | 16:24 | |
*** dustins has quit IRC | 16:27 | |
*** lpetrut has quit IRC | 16:28 | |
*** jmlowe has joined #openstack-cinder | 16:35 | |
openstackgerrit | Ben Swartzlander proposed openstack/cinder master: NetApp ONTAP: Fix reporting of provisioned_capacity_gb https://review.openstack.org/505672 | 16:38 |
*** david-lyle has quit IRC | 16:40 | |
*** bhagyashris has quit IRC | 16:42 | |
*** gouthamr has joined #openstack-cinder | 16:43 | |
*** bhagyashris has joined #openstack-cinder | 16:43 | |
*** mvk has quit IRC | 16:44 | |
*** crushil has joined #openstack-cinder | 16:44 | |
*** hemna_ has joined #openstack-cinder | 16:45 | |
*** gouthamr_ has joined #openstack-cinder | 16:45 | |
jgriffith | Has anybody else had problems running single unit tests? | 16:48 |
jgriffith | smcginnis: ^^ | 16:48 |
jgriffith | I'm not sure sure but I think it conincided with stestr | 16:48 |
jungleboyj | jgriffith: I have always had issues getting one test to run. :-) | 16:48 |
jgriffith | but something like `tox -epy27 -- cinder.tests.unit.test_foo` no longer seems to work | 16:48 |
jgriffith | hmm | 16:48 |
*** gouthamr has quit IRC | 16:49 | |
lhx_ | jgriffith, ++ | 16:50 |
smcginnis | jgriffith: Did that ever? | 16:50 |
jgriffith | smcginnis: oh yes, it did; I use that heavily | 16:50 |
smcginnis | jgriffith: From what I've always done, you either need -n for a specific test or -r for a regex match. | 16:51 |
lhx_ | jgriffith, all the command in this page doesn't work? https://wiki.openstack.org/wiki/Testr#Debugging_.28pdb.29_Tests | 16:51 |
smcginnis | jgriffith: At least that's how it's been documented: https://docs.openstack.org/cinder/latest/contributor/testing.html#running-a-subset-of-tests-using-tox | 16:51 |
jgriffith | smcginnis: let me check -n | 16:51 |
tommylikehu | smcginnis: +1 | 16:51 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder master: Run backup compression on native thread https://review.openstack.org/507510 | 16:51 |
jgriffith | smcginnis: FYI https://wiki.openstack.org/wiki/Testr | 16:52 |
*** e0ne has quit IRC | 16:52 | |
jgriffith | :) | 16:52 |
*** xyang1 has quit IRC | 16:52 | |
jungleboyj | jgriffith -- worked a long time ago. Been a while since I tried it though. | 16:53 |
smcginnis | Huh, never tried that. | 16:53 |
smcginnis | Looks like debugging is the same, just don't need to get the list of tests. | 16:53 |
smcginnis | I always just activate the venv, then "python -m testtools.run cinder.tests.unit.test_stuff" | 16:54 |
jgriffith | smcginnis: hehe, that's what I've been doing :) | 16:54 |
jgriffith | I proposed a patch long long ago for --debug and pdb | 16:55 |
jgriffith | ok... well just me then | 16:55 |
jgriffith | never mind :) | 16:55 |
smcginnis | jgriffith: Patch to ostestr? | 16:55 |
jgriffith | smcginnis: yeah | 16:56 |
smcginnis | jgriffith: Don't suppose it's still out there? I think that would be very useful. | 16:56 |
jgriffith | it was actually removing the requirement for --debug IIRC | 16:56 |
jgriffith | smcginnis: it was years ago and merged, but I don't know if that whole thing works any more or not | 16:56 |
smcginnis | Huh, I'll have to give it a shot next time. | 16:57 |
jgriffith | It still works | 16:58 |
smcginnis | Nice - good to know. | 16:58 |
jgriffith | add your pdb statement then run `tox -epy27 -- -n cinder.tests.unit.test_foo` | 16:58 |
jgriffith | and yes, you're correct my problem is solved if I use -n | 16:59 |
smcginnis | Cool | 16:59 |
smcginnis | -r can come in handy too. | 16:59 |
*** Apoorva_ has joined #openstack-cinder | 17:03 | |
*** Apoorva has quit IRC | 17:06 | |
*** dustins has joined #openstack-cinder | 17:06 | |
*** dave-mcc_ has quit IRC | 17:06 | |
*** dave-mccowan has joined #openstack-cinder | 17:11 | |
openstackgerrit | Merged openstack/cinder master: Fix api-ref for reset group snapshot https://review.openstack.org/507892 | 17:12 |
*** david-lyle has joined #openstack-cinder | 17:13 | |
*** pcaruana has quit IRC | 17:14 | |
*** itlinux has joined #openstack-cinder | 17:16 | |
*** mchlumsky has quit IRC | 17:20 | |
*** esker has quit IRC | 17:24 | |
*** mchlumsky has joined #openstack-cinder | 17:24 | |
openstackgerrit | Hanxi Liu proposed openstack/cinder master: Address importing directory conflict https://review.openstack.org/507937 | 17:32 |
lhx_ | jungleboyj, smcginnis, could you help review ^? | 17:34 |
openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Update provider parameter in sample json files https://review.openstack.org/507940 | 17:47 |
*** e0ne has joined #openstack-cinder | 17:49 | |
openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Remove deprecated keymgr code https://review.openstack.org/468252 | 17:51 |
*** amoralej is now known as amoralej|off | 17:51 | |
*** itlinux has quit IRC | 17:51 | |
*** lhx_ has quit IRC | 18:07 | |
*** pcaruana has joined #openstack-cinder | 18:15 | |
*** ChubYann has joined #openstack-cinder | 18:18 | |
openstackgerrit | Kaitlin Farr proposed openstack/cinder master: Update provider parameter in sample json files https://review.openstack.org/507940 | 18:25 |
jungleboyj | mriedem: You around? | 18:29 |
*** dhellmann has left #openstack-cinder | 18:30 | |
mriedem | sort of | 18:33 |
mriedem | something something forum multiattach | 18:34 |
mriedem | go ahead | 18:34 |
mriedem | i haven't posted any forum topics yet | 18:34 |
mriedem | not sure what there will be to talk about besides people saying, "yes make it happen" | 18:34 |
mriedem | could give a background on prequisites, and questions to ops about what policy rules they need | 18:35 |
mriedem | jungleboyj: ^ | 18:35 |
jungleboyj | mriedem: Well and also an opportunity to share the progress we have made. | 18:36 |
jungleboyj | mriedem: Man, should I be scared I don't event to say what I want? | 18:36 |
mriedem | i saw your mention in the cinder meeting | 18:37 |
jungleboyj | mriedem: :-) | 18:37 |
jungleboyj | Ok, I will propose the session. | 18:38 |
smcginnis | mriedem, jungleboyj: I think it might be good to have the discussion with ops around R/W, R/O, and how to manage that. Expected user experience, etc. | 18:38 |
jungleboyj | smcginnis: Yeah, that is good. We know that they want multi-attach but I think we need to better understand how they are going to use it. | 18:39 |
ildikov | smcginnis: +1, my thought exactly | 18:47 |
*** lpetrut has joined #openstack-cinder | 18:47 | |
ildikov | smcginnis: we will go with a basic on/off as starting point most probably, but it would be beneficial to understand how they will use it beyond that | 18:48 |
smcginnis | ;) Lots of people are asking for it. I'm concerned that even though they are all asking for "multiattach", what that means to each of them may be slightly different. | 18:50 |
jungleboyj | smcginnis: Right. I wonder how many are going to assume we have done some magic to make it work with all backends/filesystems, etc . | 18:50 |
*** lpetrut has quit IRC | 18:50 | |
smcginnis | And that they can just format it ext4 and read/write from all their hosts. | 18:51 |
*** lpetrut has joined #openstack-cinder | 18:51 | |
jungleboyj | Yeah ... | 18:52 |
*** sbezverk has joined #openstack-cinder | 18:54 | |
*** moshele has joined #openstack-cinder | 18:55 | |
*** pcaruana has quit IRC | 19:00 | |
*** hemna_ has quit IRC | 19:02 | |
*** r-daneel has quit IRC | 19:02 | |
*** r-daneel has joined #openstack-cinder | 19:03 | |
jungleboyj | smcginnis: Have we really been working on multi-attach since 2013? | 19:03 |
smcginnis | Yerp | 19:04 |
smcginnis | In one form or another. | 19:04 |
jungleboyj | Because that is what the Spec and Blueprints go back to. | 19:04 |
jungleboyj | This stuff is all really old though. | 19:06 |
smcginnis | Yeah, we were "done" with multiattach around kilo I think. Until we realized it couldn't be used by anyone. | 19:07 |
*** arnewiebalck_ has joined #openstack-cinder | 19:07 | |
*** e0ne has quit IRC | 19:08 | |
*** Swanson has quit IRC | 19:08 | |
jungleboyj | Bwah ha ha. Maybe we have it right this time? | 19:08 |
*** vivsoni has quit IRC | 19:14 | |
*** vivsoni has joined #openstack-cinder | 19:14 | |
*** esker has joined #openstack-cinder | 19:15 | |
*** edmondsw has quit IRC | 19:15 | |
jungleboyj | Ok, I have our submissions in. | 19:16 |
*** edmondsw has joined #openstack-cinder | 19:18 | |
ildikov | Nice, thanks guys! | 19:20 |
jungleboyj | ildikov: No problem. | 19:20 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Clean up api-ref index page https://review.openstack.org/507959 | 19:22 |
*** liverpooler has quit IRC | 19:23 | |
*** edmondsw has quit IRC | 19:23 | |
*** liverpooler has joined #openstack-cinder | 19:23 | |
*** chhavi has quit IRC | 19:23 | |
*** jmlowe has quit IRC | 19:25 | |
*** shaner has quit IRC | 19:36 | |
*** liverpooler has quit IRC | 19:36 | |
*** shaner has joined #openstack-cinder | 19:36 | |
*** liverpooler has joined #openstack-cinder | 19:38 | |
*** arnewiebalck_ has quit IRC | 19:44 | |
*** eharney has quit IRC | 19:44 | |
*** liverpooler has quit IRC | 19:45 | |
*** esker has quit IRC | 19:46 | |
*** edmondsw has joined #openstack-cinder | 19:47 | |
*** edmondsw has quit IRC | 19:48 | |
*** edmondsw has joined #openstack-cinder | 19:48 | |
*** markvoelker has quit IRC | 19:48 | |
*** e0ne_ has joined #openstack-cinder | 19:49 | |
*** markvoelker has joined #openstack-cinder | 19:49 | |
*** shaner has quit IRC | 19:50 | |
*** shaner has joined #openstack-cinder | 19:50 | |
*** moshele has quit IRC | 19:56 | |
*** ChubYann has quit IRC | 19:58 | |
*** crose has quit IRC | 20:05 | |
*** ChubYann has joined #openstack-cinder | 20:12 | |
*** wes_dillingham has quit IRC | 20:12 | |
*** r-daneel has quit IRC | 20:14 | |
*** r-daneel has joined #openstack-cinder | 20:15 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Clean up api-ref index page https://review.openstack.org/507959 | 20:22 |
*** gouthamr_ has quit IRC | 20:32 | |
*** belmoreira has joined #openstack-cinder | 20:32 | |
*** gouthamr has joined #openstack-cinder | 20:35 | |
*** amoralej|off has quit IRC | 20:35 | |
*** amoralej has joined #openstack-cinder | 20:38 | |
*** gouthamr has quit IRC | 20:39 | |
jungleboyj | jgriffith: You around? | 20:40 |
*** gouthamr has joined #openstack-cinder | 20:42 | |
*** belmoreira has quit IRC | 20:44 | |
smcginnis | Funny, I was just going to ping jgriffith. | 20:44 |
*** gouthamr has quit IRC | 20:45 | |
*** lucasxu has quit IRC | 20:52 | |
jungleboyj | :-) I need someone with experience in iscsi and LVM. | 20:52 |
jungleboyj | :-( | 20:52 |
*** ChubYann has quit IRC | 20:54 | |
*** e0ne_ has quit IRC | 20:57 | |
*** e0ne has joined #openstack-cinder | 21:00 | |
*** mvk has joined #openstack-cinder | 21:02 | |
*** Swanson has joined #openstack-cinder | 21:02 | |
*** e0ne has quit IRC | 21:04 | |
*** eharney has joined #openstack-cinder | 21:04 | |
smcginnis | eharney: You around? | 21:04 |
eharney | smcginnis: yep | 21:05 |
*** e0ne has joined #openstack-cinder | 21:05 | |
smcginnis | eharney: So... this lvm_max_over_provisioning_ration setting. | 21:05 |
smcginnis | eharney: It's confusing AF. | 21:05 |
eharney | smcginnis: well, maybe | 21:05 |
smcginnis | eharney: You think we still need that? I think it may do more harm than good. | 21:05 |
eharney | smcginnis: i didn't really want it in the first place... it was a compromise instead of changing the default over provisioning ratio (which IMO we should still do) | 21:06 |
smcginnis | If we default to thin, and we overprovision by default, I kinda think we should treat LVM just like any other backend. If that doesn't work for them, then they can set the ratio to something more appropriate for there environment. | 21:07 |
*** ChubYann has joined #openstack-cinder | 21:07 | |
smcginnis | *their | 21:07 |
eharney | who can? | 21:08 |
smcginnis | eharney: They deployer. | 21:09 |
smcginnis | *The gah!! | 21:09 |
*** crushil has quit IRC | 21:09 | |
eharney | i still believe it isn't reasonable to ask deployers to change settings if they don't want the default ratio of 20x... this seems very unsafe to me | 21:11 |
smcginnis | Key part there was "if they don't want". If they are fine running thin provisioned then it should allow over provisioning. Kind of the whole point IMO. | 21:12 |
eharney | i don't see why this is the one case where we default to something that will blow your foot off if you aren't paying attention... it should be opt-in, not opt-out | 21:12 |
*** MVenesio has quit IRC | 21:13 | |
*** esker has joined #openstack-cinder | 21:13 | |
*** MVenesio has joined #openstack-cinder | 21:13 | |
eharney | maybe they don't know they don't want it until their array fills up | 21:13 |
smcginnis | It won't though. As long as they don't consume all the actual space. People are pretty used to how thin provisioning works from what I've seen. | 21:14 |
smcginnis | I think there's more of an issue with "why the heck can't a create more volumes when they're all thin provisioned?" | 21:14 |
eharney | well, that seems likely to be because cinder and a lot of drivers weren't even computing space statistics correctly, which is being fixed up now, right? | 21:15 |
eharney | given some of Gorka's recent findings, a lot of people using thin provisioning weren't relying on the calculations they thought they were anyway | 21:16 |
smcginnis | They were off, sure. But they would still allow some over provisioning. | 21:16 |
*** amoralej has quit IRC | 21:16 | |
eharney | at any rate, i wouldn't be terribly opposed to ditching the LVM-specific setting i guess | 21:17 |
*** MVenesio has quit IRC | 21:17 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Deprecate LVM overprovisioning ration and deprecate https://review.openstack.org/507985 | 21:18 |
eharney | now that we're getting backend_defaults config options, there's other ways to address the issue that weren't available before | 21:18 |
smcginnis | eharney: Let's see how it goes. ^^ :) | 21:18 |
eharney | since another motivation for having that option was that it wasn't possible to include a default conf setting in a cinder package | 21:18 |
*** amoralej has joined #openstack-cinder | 21:18 | |
smcginnis | eharney: True. Now could set to a "safer" number easier for all backends. | 21:18 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Deprecate LVM overprovisioning ratio and deprecate https://review.openstack.org/507985 | 21:19 |
smcginnis | Just can not type today. | 21:19 |
eharney | smcginnis: that commit summary is kind of redundant and redundant | 21:19 |
smcginnis | patrickeast: Please shut down your CI. | 21:19 |
smcginnis | eharney: Doh! I meant to mention not override. | 21:20 |
patrickeast | smcginnis: sure, one sec | 21:20 |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Stop overriding LVM overprovisioning ratio and deprecate https://review.openstack.org/507985 | 21:20 |
patrickeast | zuuls off | 21:21 |
smcginnis | patrickeast: Thank you. :) | 21:21 |
eharney | smcginnis: your bug report points out the other issue with this setting... nobody can remember if it's spelled "oversubscription" or "over_subscription" :) | 21:23 |
smcginnis | eharney: We seem to favor over_sub_script_ion | 21:23 |
eharney | lol | 21:23 |
*** e0ne has quit IRC | 21:24 | |
smcginnis | logical_volume_manager_over_sub_scription_rati_o | 21:24 |
*** abishop has quit IRC | 21:25 | |
* smcginnis can finally get back to testing scale on my tiny test machine | 21:26 | |
jgriffith | Oh!! I see I missed my favorite debate :) Sweet! | 21:33 |
jgriffith | and jungleboyj TBF, I didn't tweet it but multi-attach has been in use by Cinder consumers since K | 21:33 |
jgriffith | it actually works, just not with Nova | 21:33 |
jgriffith | and not if you're dumb :) | 21:33 |
*** itlinux has joined #openstack-cinder | 21:36 | |
jungleboyj | jgriffith: Really? | 21:37 |
jungleboyj | I knew some IBM customers were using it. | 21:37 |
jungleboyj | So, how are customers using it if not with Nova? | 21:38 |
jungleboyj | patrickeast: please fix your CI | 21:41 |
patrickeast | jungleboyj: its on the list.. | 21:41 |
jungleboyj | :-) | 21:41 |
ildikov | jgriffith: and there's also local changes in the Cinder code depending on what driver they use | 21:44 |
jgriffith | jungleboyj: They set the multi-attach flags and ignore the status checks on things | 21:44 |
ildikov | jgriffith: and even my never merged Nova patches made it into production | 21:44 |
jungleboyj | ildikov: Yeah, that was what I was thinking of that IBM had done. | 21:45 |
jgriffith | jungleboyj: IIRC both Gophercloud and Rexray do this | 21:45 |
ildikov | jgriffith: so it's really time to fix the mess now... :) | 21:45 |
*** dave-mccowan has quit IRC | 21:45 | |
jgriffith | It was time as soon as that code landed :) | 21:45 |
jungleboyj | Ok, so my tweet should be that we need to make it official. :-) | 21:45 |
jgriffith | where getting there | 21:45 |
jgriffith | but I'm trying to figure somethign out that used to be "ez" | 21:46 |
ildikov | jgriffith: I know, was just chewing on it for fun :) | 21:46 |
jgriffith | I'm introducing a new column to a table, and when I do that I want to also insert a value for said new column | 21:46 |
jgriffith | it used to be I would add the column, the do a list on the table and iterate through look for column.value == None and update it | 21:46 |
jgriffith | but that doesn't seem to be working oddly | 21:46 |
ildikov | In what way does that not work? | 21:47 |
jgriffith | https://gist.github.com/j-griffith/cf45938a373cab99d6dab4ae0f91a119 | 21:48 |
jgriffith | result is that I have my columns, but they're Null | 21:49 |
jgriffith | it's been so long since I've done any real DB migrations that it's sad | 21:49 |
*** wes_dillingham has joined #openstack-cinder | 21:49 | |
*** wes_dillingham has quit IRC | 21:49 | |
*** dustins has quit IRC | 21:50 | |
ildikov | But if you set it nullable=True and not giving it a default value it doesn't seem fully odd | 21:51 |
ildikov | But I didn't do too many DB migrations | 21:52 |
ildikov | at all... | 21:52 |
*** ntpttr_laptop has joined #openstack-cinder | 21:54 | |
*** mriedem is now known as mriedem_away | 21:57 | |
*** wes_dillingham has joined #openstack-cinder | 21:57 | |
*** dave-mccowan has joined #openstack-cinder | 21:57 | |
*** lpetrut has quit IRC | 21:59 | |
*** dave-mccowan has quit IRC | 22:03 | |
*** ntpttr_laptop has quit IRC | 22:23 | |
*** edmondsw has quit IRC | 22:23 | |
smcginnis | jgriffith: We have existing migrations that do something similar. | 22:27 |
smcginnis | Or at least we did. | 22:27 |
*** ntpttr_laptop has joined #openstack-cinder | 22:28 | |
*** markstur_ has quit IRC | 22:29 | |
*** markstur has joined #openstack-cinder | 22:29 | |
*** markstur has quit IRC | 22:29 | |
*** esker has quit IRC | 22:30 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder master: Stop overriding LVM overprovisioning ratio and deprecate https://review.openstack.org/507985 | 22:37 |
openstackgerrit | Merged openstack/cinder master: Api-ref: change 'tenant' to 'project' in v2 doc https://review.openstack.org/507813 | 22:58 |
*** rtjure has quit IRC | 23:01 | |
*** ntpttr_laptop has quit IRC | 23:01 | |
*** wes_dillingham has quit IRC | 23:03 | |
*** wes_dillingham has joined #openstack-cinder | 23:04 | |
*** wes_dillingham has quit IRC | 23:04 | |
*** felipemonteiro_ has quit IRC | 23:29 | |
*** mriedem_away is now known as mriedem | 23:31 | |
*** jmlowe has joined #openstack-cinder | 23:32 | |
*** wes_dillingham has joined #openstack-cinder | 23:33 | |
*** stakeda has joined #openstack-cinder | 23:34 | |
*** edmondsw has joined #openstack-cinder | 23:40 | |
*** edmondsw has quit IRC | 23:44 | |
*** moshele has joined #openstack-cinder | 23:44 | |
*** tpsilva has quit IRC | 23:46 | |
*** markstur has joined #openstack-cinder | 23:49 | |
*** jmlowe has quit IRC | 23:52 | |
*** jmlowe has joined #openstack-cinder | 23:53 | |
openstackgerrit | Merged openstack/cinder master: Clean up api-ref index page https://review.openstack.org/507959 | 23:53 |
*** lhx_ has joined #openstack-cinder | 23:57 | |
*** r-daneel has quit IRC | 23:58 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!