| *** haomaiwa_ has quit IRC | 00:01 | |
| *** haomaiwang has joined #openstack-cinder | 00:02 | |
| *** ganso_ has quit IRC | 00:05 | |
| *** garthb__ has quit IRC | 00:08 | |
| *** dims_ has joined #openstack-cinder | 00:08 | |
| *** pv_ has quit IRC | 00:16 | |
| *** xyang1 has joined #openstack-cinder | 00:26 | |
| *** xyang1 has quit IRC | 00:26 | |
| *** Yogi1 has joined #openstack-cinder | 00:29 | |
| *** jasondotstar has quit IRC | 00:33 | |
| *** smoriya has joined #openstack-cinder | 00:33 | |
| *** Yogi1 has quit IRC | 00:37 | |
| *** bill_az_ has quit IRC | 00:40 | |
| *** wyatt has joined #openstack-cinder | 00:42 | |
| *** mtanino has quit IRC | 00:44 | |
| *** Yogi1 has joined #openstack-cinder | 00:46 | |
| *** kfoo has joined #openstack-cinder | 00:50 | |
| *** wyatt has quit IRC | 00:50 | |
| breitz | anybody know where I find docs on the formatters that I find in various LOG.<whatever>() statements? | 00:53 |
|---|---|---|
| breitz | I can copy what I see in the code - but I'm interested in a little more education (doc reading) if possible | 00:54 |
| *** boris-42 has joined #openstack-cinder | 00:57 | |
| *** kfoo has quit IRC | 00:58 | |
| *** kfoo has joined #openstack-cinder | 01:00 | |
| *** xiaohui has joined #openstack-cinder | 01:00 | |
| *** haomaiwang has quit IRC | 01:01 | |
| *** Apoorva_ has quit IRC | 01:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 01:02 | |
| *** jasondotstar has joined #openstack-cinder | 01:03 | |
| *** zhonghua-li has joined #openstack-cinder | 01:08 | |
| breitz | found a pile of docs - and found what I was looking for | 01:08 |
| *** Yogi1 has quit IRC | 01:09 | |
| xyang | hemna: https://review.openstack.org/#/c/207677/ | 01:09 |
| xyang | hemna: marked it WIP | 01:09 |
| *** lixiaoy1 has joined #openstack-cinder | 01:11 | |
| *** kjelly has joined #openstack-cinder | 01:11 | |
| *** heyun has joined #openstack-cinder | 01:12 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add the ability to update type public status https://review.openstack.org/206788 | 01:13 |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 01:18 |
| *** dalgaaf has quit IRC | 01:19 | |
| thingee | jgriffith: there you are! thank you sir! | 01:20 |
| *** leeantho has quit IRC | 01:20 | |
| lixiaoy1 | thingee: I replied your comment in review https://review.openstack.org/#/c/199846/. Please have a look | 01:26 |
| thingee | lixiaoy1: tbarron got back to you https://review.openstack.org/#/c/203215/ | 01:29 |
| *** davechen has joined #openstack-cinder | 01:30 | |
| tbarron | lixiaoy1: thingee: LisaLi pls let me know if I'm getting your point on this one or not. I really do appreciate your help! | 01:31 |
| *** Lee1092 has joined #openstack-cinder | 01:31 | |
| lixiaoy1 | thingee: tbarron: let me have a look | 01:31 |
| tbarron | lixiaoy1: on https://review.openstack.org/#/c/199846 I tried to explain to winston-d_zZZ what I thought your use case was. | 01:32 |
| *** krypton has quit IRC | 01:32 | |
| *** annashen has joined #openstack-cinder | 01:32 | |
| *** krypton has joined #openstack-cinder | 01:33 | |
| tbarron | lixiaoy1: but I don't think I did a very good job. Oh I think that might be the other related change I'm talking about. | 01:35 |
| tbarron | lixiaoy1: in any case, I know winston-d_zZZ wanted to talk to you more about it. | 01:35 |
| *** annashen has quit IRC | 01:37 | |
| *** krypton has quit IRC | 01:37 | |
| *** zhonghua-li is now known as zhonghua-lee | 01:41 | |
| lixiaoy1 | tbarron: I read your comments in https://review.openstack.org/#/c/203215 scale backup spec. And yes that I think. host is used for cleaning up. not for normal operations. For example, if backup service host A created a backup, and backup service B can restore the backup, only requires services are same. | 01:42 |
| lixiaoy1 | tbarron: service same: for example, both are Ceph backup driver | 01:42 |
| tbarron | lixiaoy1: agree | 01:43 |
| tbarron | lixiaoy1: and actually chunkedbackup drivers (swift, nfs, posix as of today (yeah), and glusterfs soon) can likely restore backup from one another, thanks to DuncanT insisting | 01:45 |
| tbarron | lixiaoy1: but back to your main point ... | 01:45 |
| lixiaoy1 | tbarron: but about your last words in the comments that saving uuid of service, as currently we have backup service (like cinder.backup.drivers.ceph) in db, do we need to save the uuid again? | 01:47 |
| tbarron | lixiaoy1: probably not. Maybe in the future. I keep thinking about how to run multiple backup services without requiring a separate physical host for each one. | 01:48 |
| tbarron | lixiaoy1: so I was concerned about having more than one backup service with the same 'host' field. | 01:49 |
| tbarron | lixiaoy1: but thinking about that more, we could embed more info in host field if we need to. | 01:50 |
| lixiaoy1 | tbarron: yes | 01:50 |
| tbarron | lixiaoy1: we already do that with "pools" in volume server. | 01:50 |
| lixiaoy1 | tbarron: yeah, so no other fields needed | 01:50 |
| tbarron | lixiaoy1: so I don't think that worry I had is that relevant any more. | 01:50 |
| lixiaoy1 | tbarron: I agree. | 01:51 |
| tbarron | lixiaoy1: so I like your idea for how to do the cleanup in init_host. I will have to add some rpc stuff of course like I did in the rest of the | 01:51 |
| tbarron | lixiaoy1: POC so that there is loose coupling with volume drivers. | 01:52 |
| lixiaoy1 | tbarron: that's nice. | 01:52 |
| tbarron | thingee: ^^^^^ I can't have that ready tomorrow b/c I have to have a bunch of approvals to push each change up into gerrit :( | 01:52 |
| tbarron | lixiaoy1: thingee: but I can get that done well before the next deadline. | 01:53 |
| tbarron | lixiaoy1: the other big issue you raised was drivers like vmware that do a bunch of stuff in their backup and restore methods | 01:54 |
| tbarron | lixiaoy1: other than what I've assimilated into the backup manager attach/detach. | 01:54 |
| tbarron | lixiaoy1: they do something more than make their volume from their backend available as a block device file to the backup operation. | 01:55 |
| lixiaoy1 | : tbarron do you mean to migrate the codes in backup function of the volume driver to backup service? | 01:56 |
| tbarron | lixiaoy1: honestly, I don't know what they are doing yet, but it is true that my proposal doesn't do what they are doing. | 01:56 |
| tbarron | lixiaoy1: first we have to understand what they are doing over and above just making their backend volume "attachable" | 01:57 |
| *** annashen has joined #openstack-cinder | 01:57 | |
| tbarron | maybe need to talk to the owners of these drivers and find out. Are they trying to enable non-disruptive backup for their volumes in their own drivers? | 01:58 |
| tbarron | lixiaoy1: it isn't obvious to me from reading the code what is going on. | 01:58 |
| tbarron | lixiaoy1: I don't want to block a good idea for the project (if it is one) because of individual driver variations, unless these make sense. | 01:59 |
| tbarron | lixiaoy1: Does that seem right to you? | 01:59 |
| lixiaoy1 | tbarron: maybe can like non-disruptive backup, give a default implementation, and the interface that special drivers can implement by themselves | 01:59 |
| lixiaoy1 | tbarron: yeah, exactly. separate backup is a good idea, and is fundation for backup improvemenbt | 02:00 |
| tbarron | lixiaoy1: yes that might make sense. It is something that we'd need to work out if the team decides to move ahead on this front. | 02:00 |
| *** haomaiwa_ has quit IRC | 02:01 | |
| tbarron | lixiaoy1: thank you for endorsing the "good idea." A number of folks have done that as well. I think the question will be | 02:01 |
| tbarron | lixiaoy1: whether it makes sense to devote resources to this effort in Liberty or to defer it to a later release. | 02:02 |
| *** haomaiwang has joined #openstack-cinder | 02:02 | |
| lixiaoy1 | tbarron: I can help if you need. | 02:03 |
| tbarron | lixiaoy1: I think it is a legitimate question. Obviously I'd like to go ahead, but it will take a collective decision. | 02:03 |
| tbarron | lixiaoy1: you already are! But if we are to go ahead we'll need buyin and guidance from the core team. | 02:03 |
| lixiaoy1 | tbarron: :) | 02:03 |
| tbarron | lixiaoy1: that may sound like I'm making a sales job to them (and of course I am :-) ) but I honestly know that we have to pick and choose what to do in any particular OpenStack release. | 02:04 |
| lixiaoy1 | tbarron: seems cores also interested in it. I think once backup service is separated from volume, we can do a lot of things. | 02:04 |
| tbarron | lixiaoy1: ++ (unbiased :-) ) | 02:05 |
| lixiaoy1 | tbarron: do the work well, and obviously the work is worthy, to sale it will be easy. :) | 02:07 |
| *** salv-orlando has joined #openstack-cinder | 02:08 | |
| lixiaoy1 | tbarron: also thank you for explaining my patch to windson-d | 02:08 |
| tbarron | lixiaoy1: I agree. | 02:08 |
| lixiaoy1 | tbarron: which timezone are you in | 02:08 |
| tbarron | lixiaoy1: I am not sure I did a good job. He's been very helpful to me, so talk to him :) | 02:08 |
| tbarron | lixiaoy1: I'm UTC -4 right now. | 02:09 |
| *** bapalm has joined #openstack-cinder | 02:09 | |
| kjelly | Hi, guys. I am working on the bug. And there are many ways to solve the bug. The first way is https://review.openstack.org/#/c/206450/1/cinder/volume/api.py . Pros: Easy to implement. Only one database connection. Cons: Just a workaround. The second way is https://review.openstack.org/#/c/206450/2/cinder/volume/api.py . Pros: The code seems to be better. Cons: It needs extra database connection. Any suggestion for solve the issue ? | 02:11 |
| lixiaoy1 | I am in UTC+08:00 | 02:11 |
| lixiaoy1 | tbarron: I am in UTC+08:00 | 02:11 |
| tbarron | lixiaoy1: I was just going to ask. Good morning! | 02:11 |
| lixiaoy1 | tbarron: good night | 02:12 |
| tbarron | lixiaoy1: :-) thanks for the help! | 02:12 |
| lixiaoy1 | tbarron: my pleasure! | 02:13 |
| lixiaoy1 | kjelly: may I ask which bug? | 02:13 |
| lixiaoy1 | kjelly: oh got it. sorry | 02:13 |
| kjelly | lixiaoy1: https://bugs.launchpad.net/cinder/+bug/1477625 | 02:14 |
| openstack | Launchpad bug 1477625 in Cinder "DB access to show volumes may not be properly controlled" [Undecided,In progress] - Assigned to jelly (coding1314) | 02:14 |
| *** markvoelker has joined #openstack-cinder | 02:15 | |
| *** salv-orlando has quit IRC | 02:15 | |
| *** markvoelker has quit IRC | 02:15 | |
| *** bapalm has quit IRC | 02:19 | |
| *** markvoelker has joined #openstack-cinder | 02:19 | |
| openstackgerrit | Xiangfei Zhu proposed openstack/cinder: VMware: Fix re-attach volume error for VC 5.1 https://review.openstack.org/207380 | 02:19 |
| *** markvoelker_ has joined #openstack-cinder | 02:21 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 02:22 |
| *** markvoelker has quit IRC | 02:23 | |
| lixiaoy1 | kjelly: sorry can't help you. I am not familiar with the part. | 02:25 |
| *** mriedem has quit IRC | 02:25 | |
| *** jwcroppe has quit IRC | 02:28 | |
| *** jwcroppe has joined #openstack-cinder | 02:29 | |
| *** openstackgerrit has quit IRC | 02:31 | |
| kjelly | lixiaoy1: Don't mind. Thank you for spend time reviewing the issue. | 02:32 |
| *** openstackgerrit has joined #openstack-cinder | 02:32 | |
| *** zhonghua-lee has quit IRC | 02:32 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 02:33 |
| *** jwcroppe has quit IRC | 02:33 | |
| *** Yogi1 has joined #openstack-cinder | 02:36 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 02:37 |
| *** annashen_ has joined #openstack-cinder | 02:42 | |
| *** annashen has quit IRC | 02:44 | |
| *** Yogi1 has quit IRC | 02:44 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 02:45 |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| openstackgerrit | wanghao proposed openstack/cinder: Forbid creating volume from image which size is 0 https://review.openstack.org/207696 | 02:56 |
| *** jasondotstar has quit IRC | 02:58 | |
| *** garthb has joined #openstack-cinder | 02:58 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** haomaiwang has joined #openstack-cinder | 03:02 | |
| *** sakaYK has joined #openstack-cinder | 03:11 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: capacity-headroom https://review.openstack.org/206923 | 03:14 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 03:17 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 03:20 |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 03:20 |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 03:21 |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 03:22 |
| openstackgerrit | wanghao proposed openstack/python-cinderclient: Add to_dict method to Resource class https://review.openstack.org/207340 | 03:23 |
| *** markvoelker_ has quit IRC | 03:23 | |
| *** markvoelker_ has joined #openstack-cinder | 03:26 | |
| *** h00327910__ has quit IRC | 03:28 | |
| *** diogogmt has joined #openstack-cinder | 03:30 | |
| *** dims_ has quit IRC | 03:33 | |
| *** zhonghua-lee has joined #openstack-cinder | 03:33 | |
| *** diogogmt has quit IRC | 03:37 | |
| *** jwcroppe has joined #openstack-cinder | 03:43 | |
| *** brodyg has joined #openstack-cinder | 03:47 | |
| *** coolsvap|away is now known as coolsvap | 03:49 | |
| *** brodyg has quit IRC | 03:53 | |
| *** WyAt has joined #openstack-cinder | 03:55 | |
| *** edmondsw has quit IRC | 03:56 | |
| openstackgerrit | zhongjun proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 03:59 |
| *** salv-orlando has joined #openstack-cinder | 04:00 | |
| *** haomaiwang has quit IRC | 04:01 | |
| *** haomaiwang has joined #openstack-cinder | 04:02 | |
| *** salv-orlando has quit IRC | 04:07 | |
| *** haigang has joined #openstack-cinder | 04:07 | |
| *** zhonghua-lee has quit IRC | 04:07 | |
| *** shyama has quit IRC | 04:20 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:27 | |
| *** BharatK has joined #openstack-cinder | 04:28 | |
| *** sakaYK has quit IRC | 04:32 | |
| *** Apoorva has joined #openstack-cinder | 04:39 | |
| *** shyama has joined #openstack-cinder | 04:46 | |
| *** angela-s has quit IRC | 04:47 | |
| *** krypton has joined #openstack-cinder | 04:51 | |
| *** krypton has quit IRC | 04:51 | |
| *** krypton has joined #openstack-cinder | 04:51 | |
| *** annashen_ has quit IRC | 04:52 | |
| *** deepakcs has joined #openstack-cinder | 04:58 | |
| *** haomaiwang has quit IRC | 05:01 | |
| *** nikeshm has quit IRC | 05:01 | |
| *** haomaiwang has joined #openstack-cinder | 05:02 | |
| *** coolsvap is now known as coolsvap|away | 05:08 | |
| *** ub2 has quit IRC | 05:08 | |
| *** BharatK has quit IRC | 05:13 | |
| *** salv-orlando has joined #openstack-cinder | 05:13 | |
| *** coolsvap|away is now known as coolsvap | 05:17 | |
| *** salv-orlando has quit IRC | 05:20 | |
| *** salv-orlando has joined #openstack-cinder | 05:21 | |
| *** ishant has joined #openstack-cinder | 05:21 | |
| *** amickus has joined #openstack-cinder | 05:22 | |
| *** ig0r_ has joined #openstack-cinder | 05:25 | |
| deepakcs | wanghao: https://review.openstack.org/#/c/172813/17/nova/tests/unit/virt/libvirt/test_driver.py | 05:26 |
| deepakcs | wanghao: all testcode in there uses mox, so i just continued with it | 05:26 |
| deepakcs | wanghao: moving to mock will be another effort and hence another patch | 05:26 |
| *** salv-orlando has quit IRC | 05:27 | |
| wanghao | deepakcs: okay, I'm fine with it. | 05:29 |
| deepakcs | wanghao: so u plan to revert ur -1 ? | 05:30 |
| *** zhonghua-lee has joined #openstack-cinder | 05:31 | |
| wanghao | deepakcs: sure, I will do it soon. | 05:32 |
| *** krypton has quit IRC | 05:32 | |
| deepakcs | wanghao: thanks for ur understanding | 05:32 |
| *** krypton has joined #openstack-cinder | 05:32 | |
| *** annashen has joined #openstack-cinder | 05:32 | |
| wanghao | deepakcs: You're welcome:) | 05:33 |
| *** annashen_ has joined #openstack-cinder | 05:39 | |
| *** annashen has quit IRC | 05:42 | |
| *** jwcroppe has quit IRC | 05:43 | |
| *** annashen has joined #openstack-cinder | 05:45 | |
| *** lpetrut has joined #openstack-cinder | 05:47 | |
| *** markvoelker_ has quit IRC | 05:48 | |
| *** Apoorva has quit IRC | 05:48 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 05:53 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 05:53 |
| *** vlaza has joined #openstack-cinder | 05:58 | |
| *** annashen has quit IRC | 06:00 | |
| *** rushiagr_away is now known as rushiagr | 06:00 | |
| *** haomaiwang has quit IRC | 06:01 | |
| *** 18WAAC7EF has joined #openstack-cinder | 06:02 | |
| *** sgotliv has joined #openstack-cinder | 06:02 | |
| *** lpetrut has quit IRC | 06:03 | |
| *** harlowja_at_home has quit IRC | 06:04 | |
| *** lpetrut has joined #openstack-cinder | 06:05 | |
| *** coolsvap is now known as coolsvap|away | 06:05 | |
| *** BharatK has joined #openstack-cinder | 06:05 | |
| *** shausy has joined #openstack-cinder | 06:06 | |
| *** BharatK has quit IRC | 06:07 | |
| *** BharatK has joined #openstack-cinder | 06:07 | |
| *** ig0r__ has joined #openstack-cinder | 06:11 | |
| *** krypton has quit IRC | 06:11 | |
| *** ig0r_ has quit IRC | 06:12 | |
| *** coolsvap|away is now known as coolsvap | 06:13 | |
| *** jwcroppe has joined #openstack-cinder | 06:15 | |
| *** annashen_ has quit IRC | 06:15 | |
| *** BharatK has quit IRC | 06:21 | |
| *** lpetrut has quit IRC | 06:25 | |
| *** salv-orlando has joined #openstack-cinder | 06:26 | |
| *** sakaYK has joined #openstack-cinder | 06:29 | |
| *** ub has joined #openstack-cinder | 06:32 | |
| *** BharatK has joined #openstack-cinder | 06:34 | |
| *** zhonghua-lee has quit IRC | 06:36 | |
| *** salv-orlando has quit IRC | 06:37 | |
| *** shausy has quit IRC | 06:38 | |
| *** shausy has joined #openstack-cinder | 06:40 | |
| *** sakaYK has quit IRC | 06:43 | |
| *** shausy has quit IRC | 06:43 | |
| *** shausy has joined #openstack-cinder | 06:44 | |
| *** salv-orlando has joined #openstack-cinder | 06:46 | |
| *** ankit_ag has joined #openstack-cinder | 06:47 | |
| *** markvoelker has joined #openstack-cinder | 06:48 | |
| *** agarciam has joined #openstack-cinder | 06:49 | |
| *** belmoreira has joined #openstack-cinder | 06:52 | |
| *** markvoelker has quit IRC | 06:53 | |
| *** sakaYK has joined #openstack-cinder | 06:54 | |
| *** abehl has joined #openstack-cinder | 06:56 | |
| *** jwcroppe has quit IRC | 06:59 | |
| *** sgotliv has quit IRC | 07:00 | |
| *** 18WAAC7EF has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:01 | |
| *** coolsvap is now known as coolsvap|away | 07:02 | |
| *** boris-42 has quit IRC | 07:10 | |
| *** coolsvap|away is now known as coolsvap | 07:10 | |
| *** markus_z has joined #openstack-cinder | 07:11 | |
| *** aagrawal has joined #openstack-cinder | 07:13 | |
| *** aagrawal has quit IRC | 07:17 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 07:17 |
| *** aagrawal has joined #openstack-cinder | 07:18 | |
| *** ronis has joined #openstack-cinder | 07:18 | |
| *** aagrawal has quit IRC | 07:18 | |
| *** aagrawal has joined #openstack-cinder | 07:19 | |
| *** dalgaaf has joined #openstack-cinder | 07:19 | |
| *** aagrawal has quit IRC | 07:26 | |
| *** aagrawal has joined #openstack-cinder | 07:26 | |
| *** aagrawal has quit IRC | 07:31 | |
| *** aagrawal has joined #openstack-cinder | 07:32 | |
| *** salv-orlando has quit IRC | 07:33 | |
| *** shyama has quit IRC | 07:35 | |
| *** aagrawal has quit IRC | 07:40 | |
| *** aagrawal has joined #openstack-cinder | 07:41 | |
| *** aagrawal has quit IRC | 07:45 | |
| *** sakaYK has quit IRC | 07:45 | |
| *** aagrawal has joined #openstack-cinder | 07:46 | |
| *** salv-orlando has joined #openstack-cinder | 07:49 | |
| *** aagrawal has quit IRC | 07:50 | |
| *** aagrawal has joined #openstack-cinder | 07:50 | |
| *** humble__ has joined #openstack-cinder | 07:53 | |
| *** aagrawal has quit IRC | 07:55 | |
| *** sakaYK has joined #openstack-cinder | 07:56 | |
| *** aagrawal has joined #openstack-cinder | 07:56 | |
| *** aagrawal has quit IRC | 08:00 | |
| *** haomaiwang has quit IRC | 08:01 | |
| *** annashen has joined #openstack-cinder | 08:01 | |
| *** aagrawal has joined #openstack-cinder | 08:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 08:02 | |
| *** lpetrut has joined #openstack-cinder | 08:03 | |
| *** annashen has quit IRC | 08:05 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 08:06 |
| *** ishant has quit IRC | 08:06 | |
| *** afazekas has joined #openstack-cinder | 08:09 | |
| *** ndipanov has quit IRC | 08:10 | |
| *** aagrawal has quit IRC | 08:11 | |
| *** aagrawal has joined #openstack-cinder | 08:12 | |
| aarefiev | thingee: thanks, I'll work on implementation next week, I've already made poc | 08:15 |
| *** aagrawal has quit IRC | 08:17 | |
| *** aagrawal has joined #openstack-cinder | 08:17 | |
| *** jasondotstar has joined #openstack-cinder | 08:19 | |
| *** aagrawal has quit IRC | 08:24 | |
| *** aagrawal has joined #openstack-cinder | 08:24 | |
| *** salv-orlando has quit IRC | 08:27 | |
| openstackgerrit | wanghao proposed openstack/cinder: Enhance deletion efficiency when backup init host https://review.openstack.org/206359 | 08:30 |
| *** ndipanov has joined #openstack-cinder | 08:31 | |
| *** jistr has joined #openstack-cinder | 08:32 | |
| *** e0ne has joined #openstack-cinder | 08:35 | |
| *** rhagarty_ has quit IRC | 08:37 | |
| *** rhagarty has quit IRC | 08:37 | |
| *** rhagarty has joined #openstack-cinder | 08:38 | |
| *** e0ne has quit IRC | 08:40 | |
| *** sakaYK has quit IRC | 08:42 | |
| *** aagrawal has quit IRC | 08:44 | |
| *** aagrawal has joined #openstack-cinder | 08:45 | |
| *** bluex-pl has joined #openstack-cinder | 08:48 | |
| *** aagrawal has quit IRC | 08:49 | |
| *** markvoelker has joined #openstack-cinder | 08:49 | |
| *** aagrawal has joined #openstack-cinder | 08:49 | |
| *** markvoelker has quit IRC | 08:54 | |
| *** aagrawal has quit IRC | 08:54 | |
| *** aagrawal has joined #openstack-cinder | 08:54 | |
| *** lixiaoy1 has quit IRC | 08:56 | |
| *** davechen has left #openstack-cinder | 08:57 | |
| *** sakaYK has joined #openstack-cinder | 08:58 | |
| *** jordanP has joined #openstack-cinder | 08:59 | |
| *** haomaiwa_ has quit IRC | 09:01 | |
| *** aix has joined #openstack-cinder | 09:02 | |
| *** haomaiwang has joined #openstack-cinder | 09:02 | |
| *** shausy has quit IRC | 09:06 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds migration abortion https://review.openstack.org/207754 | 09:06 |
| *** shausy has joined #openstack-cinder | 09:07 | |
| *** sakaYK has quit IRC | 09:12 | |
| *** aagrawal has quit IRC | 09:13 | |
| *** e0ne has joined #openstack-cinder | 09:13 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 09:14 |
| *** aagrawal has joined #openstack-cinder | 09:14 | |
| *** hflai has quit IRC | 09:20 | |
| *** hflai has joined #openstack-cinder | 09:20 | |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate name and description string https://review.openstack.org/182994 | 09:22 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate string, integer limit for input parameter https://review.openstack.org/194968 | 09:22 |
| *** amickus has quit IRC | 09:22 | |
| *** sakaYK has joined #openstack-cinder | 09:23 | |
| *** aagrawal has quit IRC | 09:29 | |
| *** aagrawal has joined #openstack-cinder | 09:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 09:36 |
| *** aagrawal has quit IRC | 09:36 | |
| *** aagrawal has joined #openstack-cinder | 09:37 | |
| *** BharatK_ has joined #openstack-cinder | 09:37 | |
| *** BharatK has quit IRC | 09:39 | |
| *** humble__ is now known as humble_ | 09:41 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 09:44 |
| openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 09:46 |
| *** salv-orlando has joined #openstack-cinder | 09:48 | |
| *** garthb has quit IRC | 09:49 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with ConsistencyGroups. https://review.openstack.org/207544 | 09:52 |
| geguileo | bluex-pl: ping | 10:01 |
| *** annashen has joined #openstack-cinder | 10:01 | |
| geguileo | wanghao: ping | 10:02 |
| *** annashen has quit IRC | 10:06 | |
| *** e0ne has quit IRC | 10:07 | |
| *** sakaYK has quit IRC | 10:12 | |
| *** heyun has quit IRC | 10:16 | |
| *** humble_ has quit IRC | 10:18 | |
| *** aagrawal has quit IRC | 10:19 | |
| *** jasondotstar has quit IRC | 10:22 | |
| *** DTadrzak has quit IRC | 10:26 | |
| *** BharatK_ has quit IRC | 10:31 | |
| *** e0ne has joined #openstack-cinder | 10:31 | |
| *** jasondotstar has joined #openstack-cinder | 10:40 | |
| *** sgotliv has joined #openstack-cinder | 10:41 | |
| *** BharatK_ has joined #openstack-cinder | 10:45 | |
| *** zhenguo has quit IRC | 10:47 | |
| *** sakaYK has joined #openstack-cinder | 10:54 | |
| *** sakaYK has quit IRC | 10:57 | |
| *** sakaYK has joined #openstack-cinder | 10:58 | |
| *** jasondotstar has quit IRC | 11:01 | |
| *** bluex-pl has quit IRC | 11:02 | |
| *** zhonghua-lee has joined #openstack-cinder | 11:03 | |
| *** jasondotstar has joined #openstack-cinder | 11:03 | |
| *** bluex-pl has joined #openstack-cinder | 11:03 | |
| *** dims_ has joined #openstack-cinder | 11:04 | |
| *** bluex-pl has quit IRC | 11:04 | |
| *** bluex-pl has joined #openstack-cinder | 11:04 | |
| *** ig0r__ has quit IRC | 11:15 | |
| *** shyama has joined #openstack-cinder | 11:16 | |
| *** jasondotstar has quit IRC | 11:17 | |
| *** sakaYK has quit IRC | 11:18 | |
| *** ig0r_ has joined #openstack-cinder | 11:19 | |
| *** Tross has quit IRC | 11:19 | |
| *** Tross has joined #openstack-cinder | 11:19 | |
| bluex-pl | geguileo: hi | 11:20 |
| geguileo | bluex-pl: Hi, a question regarding the tests you and dulek did on GlusterFS | 11:20 |
| geguileo | bluex-pl: The locks you had to add, what were they for? | 11:20 |
| BharatK_ | DuncanT, "Posix Backup Driver" patch got merged. Thanks for your review. | 11:21 |
| BharatK_ | DuncanT, Also need your blessings for similar patch https://review.openstack.org/#/c/182254/, please review when you have some time. | 11:22 |
| bluex-pl | geguileo: it was for volume/manager.py and volume/drivers/remotefs.py if I remember correctly | 11:25 |
| geguileo | bluex-pl: Yes, but what were you locking on? | 11:25 |
| geguileo | bluex-pl: The snapshot ID? | 11:25 |
| geguileo | bluex-pl: For what operations? | 11:25 |
| geguileo | bluex-pl: I'm trying to figure out if that will be needed if locking is properly done in the manager | 11:26 |
| geguileo | bluex-pl: Or the API | 11:26 |
| *** afazekas has quit IRC | 11:31 | |
| bluex-pl | geguileo: it was only locked on volume id in remotefs ( https://review.openstack.org/#/c/185646/4/ ) | 11:35 |
| geguileo | bluex-pl: And it was for snapshots? | 11:35 |
| bluex-pl | yup | 11:36 |
| geguileo | bluex-pl: So if the manager "locks" the volume then the driver wouldn't need the lock, right? | 11:36 |
| *** Yogi1 has joined #openstack-cinder | 11:36 | |
| *** zhonghua-lee has quit IRC | 11:37 | |
| geguileo | bluex-pl: Because the problem there was that create_snapshot was missing the lock in the manager | 11:38 |
| *** coolsvap is now known as coolsvap|away | 11:39 | |
| *** Yogi1 has quit IRC | 11:43 | |
| bluex-pl | probably, only thing you need to check when creating snapshot was if it's not deleted at that moment so there should be lock for that | 11:43 |
| *** Yogi1 has joined #openstack-cinder | 11:44 | |
| *** e0ne has quit IRC | 11:48 | |
| *** e0ne has joined #openstack-cinder | 11:50 | |
| bluex-pl | geguileo: test was set up like this: https://gist.github.com/bluex-pl/ba50e5bd75a866c09e64 | 11:51 |
| geguileo | bluex-pl: Yes, I read it, just wanted to confirm everything | 11:51 |
| bluex-pl | ok | 11:52 |
| geguileo | bluex-pl: Because the snapshot problem can be solved without locks in the driver | 11:52 |
| geguileo | bluex-pl: Thanks! | 11:52 |
| bluex-pl | geguileo: no problem, but I don't think I was very helpful ;) | 11:54 |
| geguileo | bluex-pl: You were, I just needed confirmation and you gave it :-) | 11:54 |
| *** DTadrzak has joined #openstack-cinder | 12:01 | |
| *** annashen has joined #openstack-cinder | 12:02 | |
| *** timcl has joined #openstack-cinder | 12:02 | |
| bluex-pl | geguileo: btw regarding coordinator patch, LoopingCall isn't used because it was running on greenthreads and we wanted real threads to be sure that heartbeat is sent no matter what | 12:04 |
| geguileo | bluex-pl: I don't remember this conversation... XD | 12:05 |
| geguileo | bluex-pl: I only remember 2 comments on the Tooz Lock patch | 12:06 |
| tbarron | geguileo: in volume/drivers/remotefs.py I see @locked_volume_id_operation decorators around create_cloned_volume, copy_volume_to_image, | 12:06 |
| *** marcusvrn has joined #openstack-cinder | 12:06 | |
| geguileo | bluex-pl: One is an error (ret = False is missing) and the other is just a comment | 12:06 |
| *** annashen has quit IRC | 12:07 | |
| tbarron | create_volume_from snapshot, delete_snapshot, create_snapshot | 12:07 |
| bluex-pl | geguileo: oh, sorry it was harlowja comment | 12:07 |
| tbarron | just making sure that all can be handled from manager | 12:08 |
| geguileo | bluex-pl: I was suddenly lost ;-) | 12:08 |
| *** deepakcs has quit IRC | 12:08 | |
| geguileo | tbarron: And how are your checks going? | 12:08 |
| tbarron | geguileo: my checks? | 12:09 |
| geguileo | tbarron: Can they all be removed if we fix the API races and have a proper inter-node locking (DLM, DB or whatever)? | 12:09 |
| tbarron | geguileo: I dunno yet. I was asking you whether your assertion re. snapshots generalized to the other methods in case you had already thought that through. | 12:10 |
| geguileo | tbarron: I haven't looked at the driver yet | 12:10 |
| geguileo | tbarron: In the plan I have in my mind (the one I mentioned some minutes ago in a mail) looking at the drivers' locks is the last step | 12:11 |
| tbarron | geguileo: gotcha. Well I will do some looking and thinking before the conversation next week. We need to make sure that it's at a good time for you to be there virtually. | 12:11 |
| tbarron | geguileo: sure, I'll read it over. | 12:12 |
| geguileo | tbarron: If it's a good time that would be awesome | 12:12 |
| tbarron | geguileo: well, IMO essential. | 12:12 |
| geguileo | tbarron: And if it's not I'll just have to go to sleep late ;-) | 12:12 |
| *** subscope has quit IRC | 12:16 | |
| *** bswartz has quit IRC | 12:19 | |
| *** Yogi1 has quit IRC | 12:20 | |
| bluex-pl | geguileo: 2nd thing - regarding with lock(blocking) - do you think it's safe? current tooz.Lock.__enter__ didn't check if acquire was successful | 12:23 |
| *** jasondotstar has joined #openstack-cinder | 12:23 | |
| geguileo | bluex-pl: That's why I said after my patches land ;-) | 12:23 |
| bluex-pl | geguileo: ok, great | 12:24 |
| geguileo | bluex-pl: This one checks that acquire succeeded: https://review.openstack.org/#/c/207406/2 | 12:24 |
| *** BharatK_ has quit IRC | 12:24 | |
| geguileo | bluex-pl: This one allows providing blocking to the context manager: https://review.openstack.org/#/c/203057/ | 12:25 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers https://review.openstack.org/197999 | 12:25 |
| openstackgerrit | abhiram moturi proposed openstack/cinder: ZFSSA iSCSI driver to return project 'available' space https://review.openstack.org/207106 | 12:25 |
| *** haigang has quit IRC | 12:25 | |
| geguileo | bluex-pl: But since that's not merged the currect code to handle that in your patch is perfect as it is | 12:26 |
| bluex-pl | geguileo: nice one, I was wandering about sth similar | 12:30 |
| *** subscope has joined #openstack-cinder | 12:31 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot. https://review.openstack.org/207866 | 12:35 |
| *** edmondsw has joined #openstack-cinder | 12:36 | |
| *** coolsvap|away is now known as coolsvap | 12:36 | |
| *** dustins has joined #openstack-cinder | 12:36 | |
| *** Yogi1 has joined #openstack-cinder | 12:39 | |
| *** akerr has joined #openstack-cinder | 12:41 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 12:42 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: Cleanup for cinder tests with CGSnapshot. https://review.openstack.org/207866 | 12:42 |
| *** bapalm has joined #openstack-cinder | 12:42 | |
| *** coolsvap is now known as coolsvap|away | 12:43 | |
| *** jungleboyj has quit IRC | 12:44 | |
| *** coolsvap|away is now known as coolsvap | 12:45 | |
| *** cbader has quit IRC | 12:51 | |
| *** jasondotstar has quit IRC | 12:55 | |
| *** jasondotstar has joined #openstack-cinder | 12:59 | |
| *** bswartz has joined #openstack-cinder | 13:04 | |
| *** yuriy_n17 has joined #openstack-cinder | 13:07 | |
| *** coolsvap is now known as coolsvap|away | 13:09 | |
| *** salv-orlando has quit IRC | 13:13 | |
| *** coolsvap|away is now known as coolsvap | 13:16 | |
| *** mriedem has joined #openstack-cinder | 13:17 | |
| *** eharney has joined #openstack-cinder | 13:21 | |
| *** mriedem has quit IRC | 13:25 | |
| *** rushiagr is now known as rushiagr_away | 13:26 | |
| *** markvoelker has joined #openstack-cinder | 13:26 | |
| *** primechuck has joined #openstack-cinder | 13:26 | |
| *** smoriya has quit IRC | 13:26 | |
| *** ankit_ag has quit IRC | 13:27 | |
| *** mriedem has joined #openstack-cinder | 13:28 | |
| *** markvoelker_ has joined #openstack-cinder | 13:30 | |
| *** coolsvap is now known as coolsvap|away | 13:30 | |
| *** markvoelker has quit IRC | 13:31 | |
| *** dansmith is now known as superdan | 13:32 | |
| *** annegentle has joined #openstack-cinder | 13:33 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 13:36 |
| swamireddy | hi | 13:36 |
| *** timcl has quit IRC | 13:37 | |
| *** diogogmt has joined #openstack-cinder | 13:37 | |
| swamireddy | I have installed icehouse cinder and want to upgrade to Kilo release. Do we have any upgrade process for this? | 13:37 |
| *** timcl has joined #openstack-cinder | 13:41 | |
| *** coolsvap|away is now known as coolsvap | 13:48 | |
| *** coolsvap is now known as coolsvap|away | 13:51 | |
| *** rushiagr_away is now known as rushiagr | 13:52 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: ZFSSA iSCSI driver to return project 'available' space https://review.openstack.org/207106 | 13:54 |
| *** kfoo has left #openstack-cinder | 13:57 | |
| scottda | kjelly: I commented on Patch set #1 for https://review.openstack.org/#/c/206450/ | 13:58 |
| scottda | I think your logic is correct, and the patch will work to prevent non-admin users from getting volume info for volumes they do not own. | 13:58 |
| *** MIDENN_ has quit IRC | 14:00 | |
| *** jgriffith has quit IRC | 14:02 | |
| *** annashen has joined #openstack-cinder | 14:03 | |
| *** jgriffith has joined #openstack-cinder | 14:03 | |
| *** thangp has joined #openstack-cinder | 14:04 | |
| *** diogogmt has quit IRC | 14:06 | |
| *** annashen has quit IRC | 14:08 | |
| *** nikeshm has joined #openstack-cinder | 14:11 | |
| *** jgriffith has quit IRC | 14:13 | |
| *** rushiagr is now known as rushiagr_away | 14:16 | |
| *** diogogmt has joined #openstack-cinder | 14:17 | |
| *** kjelly_ has joined #openstack-cinder | 14:17 | |
| *** jgriffith has joined #openstack-cinder | 14:21 | |
| kjelly_ | Hi, guys. I am working on the bug. And there are many ways to solve the bug. The first way is https://review.openstack.org/#/c/206450/1/cinder/volume/api.py . Pros: Easy to implement. Only one database connection. Cons: Just a workaround. The second way is https://review.openstack.org/#/c/206450/2/cinder/volume/api.py . Pros: The code seems to be better. Cons: It needs extra database connection. Any suggestion for solve the | 14:22 |
| kjelly_ | issue ? | 14:22 |
| scottda | kjelly_: Did you see my comments on patch #1? | 14:23 |
| *** bapalm_ has joined #openstack-cinder | 14:23 | |
| *** mtanino has joined #openstack-cinder | 14:24 | |
| *** h00327910__ has joined #openstack-cinder | 14:24 | |
| *** annegentle has quit IRC | 14:26 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' argument for cinder backup-list https://review.openstack.org/207469 | 14:26 |
| *** bapalm has quit IRC | 14:26 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 14:27 |
| kjelly_ | scottda: yes | 14:28 |
| *** cbader has joined #openstack-cinder | 14:30 | |
| kjelly_ | scottda: Any suggestion for the issue ? | 14:33 |
| openstackgerrit | Jordan Pittier proposed openstack/cinder: Rework Scality SOFS driver to use RemoteFS class https://review.openstack.org/200610 | 14:33 |
| scottda | kjelly_: I'm not sure what the con for solution #1 means? Why is it just a workaround? | 14:35 |
| guitarzan | scottda: volume_get with an admin context is just asking for trouble | 14:36 |
| *** mylu has joined #openstack-cinder | 14:37 | |
| scottda | ok. so that means we should not elevate context in the get call? i.e. in the check for viewable_admin_meta? | 14:37 |
| kjelly_ | scottda: My thinking is the same as guitarzan. | 14:37 |
| guitarzan | scottda: correct | 14:37 |
| guitarzan | scottda: that's why I strongly prefer using a second call with an elevated context for the admin meta | 14:38 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-cinderclient: Add tests for python-cinderclient https://review.openstack.org/197643 | 14:38 |
| *** mylu has quit IRC | 14:38 | |
| guitarzan | the alternative is to rework the db api to be able to tell the difference between "yeah I really mean admin" and "not really admin, just give me the admin stuff" | 14:38 |
| guitarzan | and I don't like that idea much | 14:38 |
| *** mylu has joined #openstack-cinder | 14:39 | |
| kjelly_ | By the way, just as Cory Stone said, The real problem with this is that using an admin context eliminates the "project_only" flag in model_query in db/sqlalchemy/api.py#257 . Maybe we should refactor db.get | 14:39 |
| *** lcurtis has joined #openstack-cinder | 14:39 | |
| scottda | OK, that all makes sense. But I see kjelly_ dilemna since you want the second DB call and Thang does not. | 14:39 |
| thangp | scottda: I think the problem is where context is coming from | 14:40 |
| *** markvoelker_ has quit IRC | 14:40 | |
| thangp | scottda: if we really want admin metadata, the context should have been admin in the beginning | 14:40 |
| thangp | and we should not need context.elevated() | 14:41 |
| *** xyang1 has joined #openstack-cinder | 14:41 | |
| thangp | but there is a mix of using context.elevated() to get admin stuff with just a under-privilege user in a lot of places | 14:41 |
| scottda | well, do we ever want or need admin stuff for a non-admin user? That code must be there for a reason... | 14:42 |
| thangp | that's what I do not understand... why do we need admin_metadata for a non-admin | 14:42 |
| scottda | guitarzan: do you know why? | 14:43 |
| guitarzan | thangp: but there is some "user viewable" admin metadata | 14:43 |
| *** jgriffith has left #openstack-cinder | 14:43 | |
| *** jgriffith has quit IRC | 14:43 | |
| guitarzan | thangp: which probably shouldn't exist, but it does | 14:43 |
| thangp | guitarzan: then shouldnt it be under metadata then? | 14:44 |
| scottda | yes, that seems like the root of the problem. | 14:44 |
| thangp | instead of admin_metadata | 14:44 |
| guitarzan | honestly I don't remember exactly what was being put in admin_metadata | 14:44 |
| kjelly_ | thangp: admin metadata only have two key 'readonly', 'attached_mode' | 14:44 |
| guitarzan | ok, that makes sense then | 14:44 |
| guitarzan | because users can update regular metadata and we don't want them to update those | 14:44 |
| guitarzan | those probably shouldn't be meta but should instead be actual volume attributes | 14:44 |
| *** coolsvap|away is now known as coolsvap | 14:44 | |
| guitarzan | but that's a discussion that probably should have happened months ago before it got committed | 14:45 |
| *** mdenny has joined #openstack-cinder | 14:45 | |
| guitarzan | yay technical debt | 14:45 |
| thangp | so admin_metadata should be readable by everyone? | 14:45 |
| guitarzan | thangp: no, only those two specific keys | 14:45 |
| thangp | oh | 14:45 |
| thangp | very confusing | 14:45 |
| guitarzan | I completely agree | 14:45 |
| thangp | i wish we have metadata that is not editable | 14:46 |
| thangp | then it would fix this | 14:46 |
| scottda | yeah, make admin_metadata readable by all but writable only by the admin | 14:46 |
| guitarzan | well, I'm guessing the idea is there also could be admin meta that isn't actually readable | 14:46 |
| guitarzan | but I don't want to speculate too much | 14:47 |
| scottda | bleh | 14:47 |
| guitarzan | yep, feels quite yucky | 14:47 |
| scottda | what about moving 'readonly' and 'attached_mode' to volume_attributes? | 14:47 |
| scottda | how hard is that (technically speaking)? | 14:47 |
| guitarzan | I think that's worth bringing up | 14:47 |
| guitarzan | it's a db migration | 14:48 |
| guitarzan | I wonder if we could find the review that added those | 14:48 |
| guitarzan | I imagine (hope) that this topic came up then | 14:48 |
| thangp | the only problem with an extra db call is it might slow down volume_get. volume_get is called pretty often | 14:48 |
| guitarzan | I'm curious what DuncanT remembers about that | 14:48 |
| guitarzan | thangp: no argument there either :) | 14:48 |
| guitarzan | typically "one" call isn't too harmful, it's the n+1s that kill us | 14:49 |
| kjelly_ | So the patch will be useless because we will use volume_attributes instead of admin meta data? | 14:50 |
| thangp | what if we put the 'readonly' and 'attached_mode' were put into metadata and we just block any edit of it? | 14:51 |
| thangp | but i prefer to put it in the volume attribute myself | 14:51 |
| thangp | just a thought | 14:52 |
| *** jgriffith has joined #openstack-cinder | 14:52 | |
| *** salv-orlando has joined #openstack-cinder | 14:53 | |
| kjelly_ | thangp: putting it in the volume attribute is better. But we need more work. | 14:55 |
| *** breitz has quit IRC | 14:55 | |
| *** breitz has joined #openstack-cinder | 14:56 | |
| *** annegentle has joined #openstack-cinder | 14:57 | |
| *** chlong has quit IRC | 14:57 | |
| *** alwaysatthenoc has joined #openstack-cinder | 15:00 | |
| kjelly_ | thangp: By the way, volumes[“volume_metadata”] is list of “cinder.db.sqlalchemy.models.VolumeMetadata” if the volume is the return value of self.db.volume_get. the api expose some inner implement. Should we refactor it? How do you think? | 15:01 |
| scottda | kjelly_: thangp guitarzan Will any of you be attending the Cinder mid-cycle virtually next week? | 15:02 |
| scottda | I'll be there, and will bring this all up. | 15:02 |
| thangp | scottda: probably virtually | 15:02 |
| guitarzan | I'm not sure, I don't think so | 15:02 |
| *** jasondotstar has quit IRC | 15:03 | |
| scottda | It'd be nice to gather all of this contention up and put it on the agenda. Would someone like to do that? | 15:03 |
| thangp | kjelly: not sure what you are asking | 15:03 |
| kjelly_ | scottda: I'm not sure, I don't think so | 15:03 |
| thangp | scottda: I think my request is to move non-admin stuff out of admin_metadata | 15:03 |
| scottda | i.e. The bug (non-admin user can see other project's volume info), the problem with admin_meta (non-admin needs to read, admin only can write)... | 15:04 |
| thangp | I think that is the biggest issue | 15:04 |
| *** vlaza has left #openstack-cinder | 15:05 | |
| scottda | should we move non-admin stuff to volume_attributes? should we hack volume get? other ideas? etc | 15:05 |
| guitarzan | one thing that might be an issue as far as this bug is it's a big deal and will probably be backported | 15:05 |
| guitarzan | so we'll probably want to commit a fix | 15:05 |
| guitarzan | without a db migration | 15:05 |
| scottda | yeah, so perhaps there will be a short term solution (fix bug, even if it's a bit of a hack) and a long term solution (change the way we read and/or write admin_meta or move it to volume_attributes or ??) | 15:06 |
| thangp | scottda: +1 | 15:06 |
| *** subscope has quit IRC | 15:07 | |
| kjelly_ | scottda: +1 | 15:07 |
| thangp | scottda: it would be easier with a nosql db :) | 15:07 |
| thangp | to add extra stuff | 15:07 |
| thangp | and backport | 15:07 |
| jgriffith | it's funny... this admin-meta and everything we did is such a *simple* thing, but it's caused nothing but problems ever since we implemented it | 15:08 |
| scottda | thangp: yes, well, I'll let you fight for the nosql solution :) | 15:08 |
| guitarzan | jgriffith: indeed :) | 15:09 |
| *** chlong has joined #openstack-cinder | 15:11 | |
| guitarzan | although I think "admin meta, but viewable by user" is even worse | 15:12 |
| jgriffith | guitarzan: :( | 15:12 |
| * jgriffith makes note; "be sure to define usage and rules for things when we create them; and enforce in reviews" | 15:13 | |
| guitarzan | that would be a great note for sure | 15:13 |
| *** edtubill has joined #openstack-cinder | 15:16 | |
| *** dannywilson has joined #openstack-cinder | 15:16 | |
| *** dims_ has quit IRC | 15:17 | |
| *** dannywilson has quit IRC | 15:17 | |
| *** dims_ has joined #openstack-cinder | 15:17 | |
| *** dannywilson has joined #openstack-cinder | 15:17 | |
| scottda | well, kjelly_ If you'd like to add this issue and the possible solutions to the Cinder mid-cycle agenda, you can add my name and I'll help with the discussion. | 15:18 |
| *** ronis_ has joined #openstack-cinder | 15:18 | |
| scottda | https://etherpad.openstack.org/p/cinder-liberty-midcycle-meetup | 15:18 |
| *** bluex-pl has quit IRC | 15:19 | |
| *** ronis has quit IRC | 15:20 | |
| kjelly_ | scottda: Ok. Thank you. | 15:20 |
| *** dims_ is now known as dimsum__ | 15:21 | |
| jgriffith | dimsum__: You win best casual nic friday nic award | 15:21 |
| *** jungleboyj has joined #openstack-cinder | 15:22 | |
| jgriffith | I'm waiting for hemna to do hemuna-hemuna-hemuna | 15:22 |
| dimsum__ | jgriffith: :) | 15:23 |
| *** ronis_ has quit IRC | 15:23 | |
| dimsum__ | love that one too! | 15:23 |
| *** belmoreira has quit IRC | 15:24 | |
| *** jasondotstar has joined #openstack-cinder | 15:26 | |
| *** ganso_ has joined #openstack-cinder | 15:29 | |
| mtanino | thingee: hi! Do you plan to fix your spec and merge it within today? https://review.openstack.org/#/c/183947/3 | 15:33 |
| mtanino | I guess that spec is almost close. | 15:35 |
| *** daneyon has joined #openstack-cinder | 15:36 | |
| *** krypton has joined #openstack-cinder | 15:36 | |
| *** kjelly_ has quit IRC | 15:38 | |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Over subscription for HP LeftHand iSCSI driver https://review.openstack.org/207248 | 15:47 |
| *** dustins has quit IRC | 15:47 | |
| *** shausy has quit IRC | 15:47 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 15:49 |
| *** vokt has joined #openstack-cinder | 15:55 | |
| *** markvoelker has joined #openstack-cinder | 15:59 | |
| *** xyang1 has quit IRC | 16:00 | |
| *** ub has quit IRC | 16:01 | |
| *** xyang1 has joined #openstack-cinder | 16:01 | |
| *** coolsvap is now known as coolsvap|away | 16:02 | |
| *** markvoelker has quit IRC | 16:02 | |
| *** annashen has joined #openstack-cinder | 16:04 | |
| *** jasondotstar has quit IRC | 16:06 | |
| *** markus_z has quit IRC | 16:06 | |
| *** garthb has joined #openstack-cinder | 16:07 | |
| *** annashen has quit IRC | 16:09 | |
| *** mylu has quit IRC | 16:09 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' argument for cinder backup-list https://review.openstack.org/207469 | 16:10 |
| *** _cjones_ has joined #openstack-cinder | 16:11 | |
| edmondsw | jungleboyj, please look at this again when you get a chance: https://review.openstack.org/#/c/203237/ | 16:11 |
| *** alwaysatthenoc has quit IRC | 16:13 | |
| *** e0ne has quit IRC | 16:13 | |
| *** ashuk has joined #openstack-cinder | 16:13 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 16:14 |
| *** yuriy_n17 has quit IRC | 16:15 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 16:17 |
| jungleboyj | edmondsw: Ok, sorry I have been behind on reviews. | 16:17 |
| edmondsw | np, tx | 16:18 |
| edmondsw | I just keep having to rebase, so I'd like to get this in :) | 16:18 |
| *** mylu has joined #openstack-cinder | 16:19 | |
| *** ub has joined #openstack-cinder | 16:19 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 16:20 |
| *** leeantho has joined #openstack-cinder | 16:23 | |
| *** ub2 has joined #openstack-cinder | 16:23 | |
| *** mriedem has quit IRC | 16:24 | |
| *** ub has quit IRC | 16:24 | |
| *** annashen has joined #openstack-cinder | 16:25 | |
| *** ashuk has quit IRC | 16:25 | |
| thingee | mtanino: yes | 16:26 |
| *** jungleboyj has quit IRC | 16:29 | |
| hemna | wuh | 16:31 |
| *** primechuck has quit IRC | 16:33 | |
| *** mylu has quit IRC | 16:33 | |
| *** primechuck has joined #openstack-cinder | 16:33 | |
| *** primechuck has quit IRC | 16:33 | |
| *** primechuck has joined #openstack-cinder | 16:33 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add rootwrap filters https://review.openstack.org/207553 | 16:34 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/206877 | 16:35 |
| *** liuxg has joined #openstack-cinder | 16:35 | |
| *** Apoorva has joined #openstack-cinder | 16:36 | |
| *** cebruns_ has quit IRC | 16:37 | |
| *** hemna is now known as chewbacca | 16:40 | |
| *** ub2 has quit IRC | 16:40 | |
| *** chewbacca is now known as wookie | 16:41 | |
| *** wookie is now known as Guest9155 | 16:42 | |
| *** krypton has quit IRC | 16:42 | |
| *** jistr has quit IRC | 16:42 | |
| *** briancline has joined #openstack-cinder | 16:43 | |
| *** krtaylor has quit IRC | 16:44 | |
| *** ub has joined #openstack-cinder | 16:44 | |
| *** liuxg has quit IRC | 16:44 | |
| *** annashen_ has joined #openstack-cinder | 16:47 | |
| *** Guest9155 is now known as pipercolt | 16:47 | |
| *** mylu has joined #openstack-cinder | 16:47 | |
| *** amit213 has quit IRC | 16:50 | |
| *** amit213 has joined #openstack-cinder | 16:50 | |
| *** annashen has quit IRC | 16:50 | |
| mtanino | thingee: oops, sorry for my late and thank you for the confirmation :) | 16:50 |
| *** krtaylor has joined #openstack-cinder | 16:57 | |
| *** timcl1 has joined #openstack-cinder | 16:57 | |
| *** timcl has quit IRC | 16:59 | |
| *** ndipanov has quit IRC | 17:01 | |
| *** anish has quit IRC | 17:01 | |
| *** anish has joined #openstack-cinder | 17:02 | |
| *** ndipanov has joined #openstack-cinder | 17:02 | |
| *** e0ne has joined #openstack-cinder | 17:03 | |
| *** markvoelker has joined #openstack-cinder | 17:03 | |
| *** erlon has quit IRC | 17:03 | |
| *** ub has quit IRC | 17:03 | |
| *** skylerberg has joined #openstack-cinder | 17:05 | |
| *** ndipanov has quit IRC | 17:09 | |
| *** chlong has quit IRC | 17:09 | |
| *** dalgaaf has quit IRC | 17:09 | |
| *** dustins has joined #openstack-cinder | 17:09 | |
| *** annegentle is now known as theannegentle | 17:09 | |
| *** ub has joined #openstack-cinder | 17:10 | |
| skylerberg | Is the concensus that we are sticking with the current ABC + multiple inheritance approach for drivers? | 17:10 |
| *** krypton has joined #openstack-cinder | 17:11 | |
| *** sgotliv has quit IRC | 17:11 | |
| *** ndipanov has joined #openstack-cinder | 17:11 | |
| skylerberg | *consensus | 17:11 |
| *** Lee1092 has quit IRC | 17:13 | |
| *** mylu has quit IRC | 17:16 | |
| *** krypton has quit IRC | 17:17 | |
| *** krypton has joined #openstack-cinder | 17:18 | |
| swamireddy | I have installed icehouse cinder and want to upgrade to Kilo release. Do we have any upgrade process for this? | 17:19 |
| *** theannegentle has quit IRC | 17:19 | |
| *** theannegentle has joined #openstack-cinder | 17:20 | |
| *** krypton has quit IRC | 17:22 | |
| *** mylu has joined #openstack-cinder | 17:23 | |
| *** ub has quit IRC | 17:23 | |
| *** salv-orlando has quit IRC | 17:25 | |
| *** jasondotstar has joined #openstack-cinder | 17:25 | |
| *** markvoelker has quit IRC | 17:26 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add connector object to create_export https://review.openstack.org/202808 | 17:27 |
| *** krypton has joined #openstack-cinder | 17:28 | |
| *** e0ne has quit IRC | 17:33 | |
| *** xyang1 has quit IRC | 17:33 | |
| *** e0ne has joined #openstack-cinder | 17:33 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: use user-configured value for chunk size https://review.openstack.org/207107 | 17:37 |
| *** theannegentle has quit IRC | 17:45 | |
| *** lpetrut has quit IRC | 17:47 | |
| *** gouthamr has joined #openstack-cinder | 17:54 | |
| *** ronis_ has joined #openstack-cinder | 17:54 | |
| *** aix has quit IRC | 17:54 | |
| *** timcl1 has quit IRC | 17:54 | |
| *** ub has joined #openstack-cinder | 17:54 | |
| *** timcl has joined #openstack-cinder | 17:55 | |
| *** bswartz has quit IRC | 17:55 | |
| *** shyama has quit IRC | 17:55 | |
| *** bswartz has joined #openstack-cinder | 18:01 | |
| *** annashen has joined #openstack-cinder | 18:05 | |
| *** patrickeast has joined #openstack-cinder | 18:07 | |
| *** annashen has quit IRC | 18:10 | |
| *** theannegentle has joined #openstack-cinder | 18:11 | |
| *** breitz has quit IRC | 18:12 | |
| *** timcl has quit IRC | 18:13 | |
| *** mylu has quit IRC | 18:15 | |
| *** timcl has joined #openstack-cinder | 18:16 | |
| *** ndipanov has quit IRC | 18:18 | |
| thingee | jgriffith: ping | 18:19 |
| *** jordanP has quit IRC | 18:21 | |
| *** vokt has quit IRC | 18:24 | |
| *** pipercolt is now known as hemna | 18:26 | |
| hemna | xyang, looks like the devstack patch isn't working | 18:26 |
| hemna | xyang, http://paste.openstack.org/show/406619/ | 18:27 |
| *** salv-orlando has joined #openstack-cinder | 18:27 | |
| *** markvoelker has joined #openstack-cinder | 18:28 | |
| *** lpetrut has joined #openstack-cinder | 18:29 | |
| *** salv-orlando has quit IRC | 18:36 | |
| *** patrickeast has quit IRC | 18:37 | |
| *** e0ne has quit IRC | 18:37 | |
| *** markvoelker has quit IRC | 18:37 | |
| *** bswartz has quit IRC | 18:38 | |
| *** IlyaG has joined #openstack-cinder | 18:42 | |
| *** e0ne has joined #openstack-cinder | 18:45 | |
| *** e0ne has quit IRC | 18:47 | |
| *** kvidvans has quit IRC | 18:53 | |
| *** bswartz has joined #openstack-cinder | 18:54 | |
| hemna | ok I'm heading out. I'll see you guys Tuesday at the midcycle. | 18:56 |
| tbarron | hemna: cu there, have a good weekend | 18:56 |
| *** hemna is now known as hemnafk | 18:56 | |
| hemnafk | l8rs! | 18:57 |
| *** kjnelson has quit IRC | 18:58 | |
| *** timcl has quit IRC | 19:00 | |
| *** salv-orlando has joined #openstack-cinder | 19:01 | |
| *** kjnelson has joined #openstack-cinder | 19:04 | |
| *** lpetrut has quit IRC | 19:04 | |
| *** ub has quit IRC | 19:06 | |
| *** ub has joined #openstack-cinder | 19:07 | |
| *** jasondotstar has quit IRC | 19:12 | |
| *** patrickeast has joined #openstack-cinder | 19:16 | |
| openstackgerrit | Mike Perez proposed openstack/cinder-specs: Updating Get Volume Driver Capabilities Spec https://review.openstack.org/183947 | 19:17 |
| thingee | mtanino, jgriffith, DuncanT https://review.openstack.org/#/c/183947/3 | 19:18 |
| *** gouthamr has quit IRC | 19:18 | |
| thingee | please and thank you | 19:18 |
| mtanino | thingee: looking :) | 19:18 |
| * thingee disappears for a bit | 19:18 | |
| *** afazekas has joined #openstack-cinder | 19:18 | |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 19:19 |
| *** jasondotstar has joined #openstack-cinder | 19:19 | |
| *** thangp has quit IRC | 19:22 | |
| *** Guest100_ has joined #openstack-cinder | 19:23 | |
| *** jasondotstar has quit IRC | 19:25 | |
| *** ig0r_ has quit IRC | 19:26 | |
| *** afazekas has quit IRC | 19:26 | |
| *** e0ne has joined #openstack-cinder | 19:32 | |
| *** openstack has joined #openstack-cinder | 19:34 | |
| openstackgerrit | Tom Barron proposed openstack/cinder-specs: Scaling backup service blueprint spec https://review.openstack.org/203215 | 19:40 |
| *** xyang1 has joined #openstack-cinder | 19:40 | |
| *** Yogi1 has quit IRC | 19:46 | |
| *** angela-s has joined #openstack-cinder | 19:46 | |
| *** annashen has joined #openstack-cinder | 19:50 | |
| *** angela-s has quit IRC | 19:54 | |
| *** ronis_ has quit IRC | 20:01 | |
| *** annashen has quit IRC | 20:11 | |
| *** lpetrut has joined #openstack-cinder | 20:11 | |
| *** edtubill has left #openstack-cinder | 20:27 | |
| *** lpetrut has quit IRC | 20:28 | |
| *** e0ne has quit IRC | 20:30 | |
| *** xyang1 has quit IRC | 20:30 | |
| *** xyang1 has joined #openstack-cinder | 20:37 | |
| *** akerr has quit IRC | 20:38 | |
| *** bapalm_ has quit IRC | 20:38 | |
| *** xyang1 has quit IRC | 20:39 | |
| *** bswartz has quit IRC | 20:43 | |
| *** aix has joined #openstack-cinder | 20:47 | |
| *** e0ne has joined #openstack-cinder | 20:48 | |
| *** Guest100_ has quit IRC | 20:50 | |
| *** kjnelson has left #openstack-cinder | 20:56 | |
| *** dustins has quit IRC | 20:57 | |
| *** changbl has quit IRC | 21:09 | |
| *** Qzhang has joined #openstack-cinder | 21:13 | |
| *** Guest100_ has joined #openstack-cinder | 21:14 | |
| *** boris-42 has joined #openstack-cinder | 21:23 | |
| *** cbader has quit IRC | 21:25 | |
| *** e0ne has quit IRC | 21:30 | |
| *** bapalm_ has joined #openstack-cinder | 21:31 | |
| *** superflyy has joined #openstack-cinder | 21:31 | |
| *** vilobhmm has joined #openstack-cinder | 21:34 | |
| *** xyang1 has joined #openstack-cinder | 21:34 | |
| vilobhmm | thingee, jgriffith, xyang1: if you can review this https://review.openstack.org/#/c/206170/4 it would be helpful…has lots of +1, +2 just needs approval | 21:35 |
| *** DericHorn-HP has joined #openstack-cinder | 21:41 | |
| *** primechuck has quit IRC | 21:43 | |
| *** vokt has joined #openstack-cinder | 21:45 | |
| *** vokt has quit IRC | 21:46 | |
| *** xyang1 has quit IRC | 21:49 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 21:51 |
| *** bapalm_ has quit IRC | 21:55 | |
| *** tsekiyama has joined #openstack-cinder | 21:59 | |
| *** leeantho_ has joined #openstack-cinder | 22:01 | |
| *** leeantho has quit IRC | 22:02 | |
| *** primechuck has joined #openstack-cinder | 22:04 | |
| *** annashen has joined #openstack-cinder | 22:12 | |
| *** annashen has quit IRC | 22:16 | |
| *** xyang1 has joined #openstack-cinder | 22:24 | |
| *** lcurtis has quit IRC | 22:25 | |
| *** xyang1 has quit IRC | 22:27 | |
| *** theannegentle has quit IRC | 22:48 | |
| *** alexpilotti has joined #openstack-cinder | 22:51 | |
| *** superflyy has quit IRC | 22:52 | |
| alexpilotti | thingee: hi | 22:52 |
| *** annashen_ has quit IRC | 22:54 | |
| *** annashen has joined #openstack-cinder | 22:55 | |
| *** earlephilhower has quit IRC | 22:55 | |
| alexpilotti | thingee: the windows iSCSI / SMB / Samba CI has been improved to solve the issues that you pointed out, mostly caused by overload related transient issues | 22:56 |
| *** theannegentle has joined #openstack-cinder | 23:01 | |
| vilobhmm | jgriffith, jungleboyj : ping | 23:05 |
| *** theannegentle has quit IRC | 23:06 | |
| *** primechuck has quit IRC | 23:10 | |
| *** alexpilotti has quit IRC | 23:11 | |
| *** vilobhmm has quit IRC | 23:15 | |
| *** leeantho has joined #openstack-cinder | 23:16 | |
| *** _cjones_ has quit IRC | 23:18 | |
| *** garthb has quit IRC | 23:18 | |
| *** leeantho_ has quit IRC | 23:19 | |
| *** Yogi1 has joined #openstack-cinder | 23:21 | |
| *** DericHorn-HP has quit IRC | 23:22 | |
| *** DericHorn-HP has joined #openstack-cinder | 23:25 | |
| *** dannywilson has quit IRC | 23:30 | |
| *** patrickeast has quit IRC | 23:30 | |
| *** daneyon has quit IRC | 23:30 | |
| *** daneyon has joined #openstack-cinder | 23:31 | |
| *** Qzhang has quit IRC | 23:31 | |
| *** mdenny has quit IRC | 23:35 | |
| *** julim has quit IRC | 23:37 | |
| *** marcusvrn has quit IRC | 23:43 | |
| *** sghanekar has joined #openstack-cinder | 23:45 | |
| *** daneyon has quit IRC | 23:46 | |
| *** IlyaG has quit IRC | 23:46 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add multiple pools support to VMAX driver https://review.openstack.org/189990 | 23:49 |
| *** Yogi1 has quit IRC | 23:49 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Create CG from CG Snapshot https://review.openstack.org/195280 | 23:52 |
| *** tsekiyam_ has joined #openstack-cinder | 23:57 | |
| *** theannegentle has joined #openstack-cinder | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!