| *** neophy has quit IRC | 00:03 | |
| *** rdx_matt has joined #openstack-cinder | 00:04 | |
| *** rdx_matt has quit IRC | 00:08 | |
| *** haomaiwang has joined #openstack-cinder | 00:11 | |
| *** garthb has quit IRC | 00:14 | |
| *** dave-mccowan has joined #openstack-cinder | 00:14 | |
| *** mylu has joined #openstack-cinder | 00:15 | |
| *** leeantho has quit IRC | 00:21 | |
| *** gouthamr has joined #openstack-cinder | 00:24 | |
| *** mtanino_ has joined #openstack-cinder | 00:28 | |
| *** gouthamr_ has joined #openstack-cinder | 00:28 | |
| *** mtanino has quit IRC | 00:29 | |
| *** gouthamr has quit IRC | 00:30 | |
| *** arch-nemesis has quit IRC | 00:32 | |
| *** raildo is now known as raildo-afk | 00:33 | |
| *** Suyash has quit IRC | 00:34 | |
| *** qeelee has joined #openstack-cinder | 00:42 | |
| *** qeelee has quit IRC | 00:46 | |
| *** laughterwym has joined #openstack-cinder | 00:52 | |
| *** itlinux has quit IRC | 00:55 | |
| *** cheneydc has joined #openstack-cinder | 00:55 | |
| *** apoorvad has quit IRC | 00:58 | |
| *** sdague has quit IRC | 00:59 | |
| *** mtanino has joined #openstack-cinder | 01:03 | |
| *** mtanino_ has quit IRC | 01:04 | |
| *** rdx_matt has joined #openstack-cinder | 01:05 | |
| *** rdx_matt has quit IRC | 01:09 | |
| *** yangyape_ has joined #openstack-cinder | 01:11 | |
| *** iceyao has joined #openstack-cinder | 01:11 | |
| *** laughterwym has quit IRC | 01:14 | |
| *** laughterwym has joined #openstack-cinder | 01:15 | |
| *** bardia has quit IRC | 01:16 | |
| *** edmondsw has quit IRC | 01:21 | |
| *** EinstCrazy has joined #openstack-cinder | 01:22 | |
| *** haomaiwang has quit IRC | 01:26 | |
| *** alejandrito has joined #openstack-cinder | 01:26 | |
| *** mylu has quit IRC | 01:29 | |
| *** yangxi has joined #openstack-cinder | 01:31 | |
| *** mylu has joined #openstack-cinder | 01:33 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Add ConsistencyGroup Into Quota Management https://review.openstack.org/307024 | 01:33 |
|---|---|---|
| *** ebalduf has quit IRC | 01:38 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder: Add ConsistencyGroup Into Quota Management https://review.openstack.org/307024 | 01:47 |
| openstackgerrit | Merged openstack/cinder: Implement replicate_volume & failover on SolidFire https://review.openstack.org/299747 | 01:52 |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs: Add ConsistencyGroup Into Quota Management https://review.openstack.org/306946 | 01:53 |
| *** itlinux has joined #openstack-cinder | 01:53 | |
| openstackgerrit | Merged openstack/cinder: Add failed_over detection to SolidFire driver https://review.openstack.org/299748 | 01:54 |
| *** salv-orlando has joined #openstack-cinder | 01:56 | |
| openstackgerrit | Merged openstack/cinder: Handle TFlow short comings in SF driver https://review.openstack.org/300078 | 01:58 |
| openstackgerrit | wanghao proposed openstack/cinder: Backup tenant attribute support https://review.openstack.org/258374 | 01:58 |
| *** EinstCrazy has quit IRC | 02:00 | |
| *** yangxi has quit IRC | 02:01 | |
| *** cheneydc1 has joined #openstack-cinder | 02:01 | |
| *** cheneydc has quit IRC | 02:02 | |
| *** cheneydc1 is now known as cheneydc | 02:02 | |
| *** salv-orlando has quit IRC | 02:02 | |
| *** EinstCrazy has joined #openstack-cinder | 02:02 | |
| *** IlyaG has joined #openstack-cinder | 02:02 | |
| *** itlinux has quit IRC | 02:04 | |
| *** itlinux has joined #openstack-cinder | 02:05 | |
| *** haplo37 has joined #openstack-cinder | 02:05 | |
| *** rdx_matt has joined #openstack-cinder | 02:05 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found https://review.openstack.org/296176 | 02:06 |
| *** Julien-zte has joined #openstack-cinder | 02:08 | |
| *** rdx_matt has quit IRC | 02:10 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Mask chap password in log https://review.openstack.org/271595 | 02:11 |
| *** itlinux has quit IRC | 02:12 | |
| *** itlinux has joined #openstack-cinder | 02:13 | |
| *** itlinux has quit IRC | 02:15 | |
| *** haplo37 has quit IRC | 02:15 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add unit tests for Huawei drivers https://review.openstack.org/283029 | 02:15 |
| *** Lee1092 has joined #openstack-cinder | 02:18 | |
| *** haplo37 has joined #openstack-cinder | 02:20 | |
| *** salv-orlando has joined #openstack-cinder | 02:22 | |
| *** iceyao has quit IRC | 02:24 | |
| *** iceyao has joined #openstack-cinder | 02:24 | |
| *** salv-orlando has quit IRC | 02:26 | |
| *** gouthamr_ has quit IRC | 02:26 | |
| openstackgerrit | wangxiyuan proposed openstack/python-cinderclient: Support update consistencygroups quota https://review.openstack.org/307537 | 02:28 |
| *** asselin_ has joined #openstack-cinder | 02:28 | |
| *** mtanino has quit IRC | 02:29 | |
| *** asselin__ has quit IRC | 02:29 | |
| *** gouthamr has joined #openstack-cinder | 02:36 | |
| *** EinstCrazy has quit IRC | 02:37 | |
| *** houming has joined #openstack-cinder | 02:38 | |
| *** int32bit has joined #openstack-cinder | 02:39 | |
| *** alejandrito has quit IRC | 02:41 | |
| *** cknight has joined #openstack-cinder | 02:41 | |
| *** lixiaoy1 has quit IRC | 02:41 | |
| *** lixiaoy1 has joined #openstack-cinder | 02:42 | |
| *** laughter_ has joined #openstack-cinder | 02:49 | |
| *** laughte__ has joined #openstack-cinder | 02:50 | |
| *** gouthamr has quit IRC | 02:50 | |
| *** laughterwym has quit IRC | 02:52 | |
| *** ebalduf has joined #openstack-cinder | 02:53 | |
| *** EinstCrazy has joined #openstack-cinder | 02:53 | |
| *** laughter_ has quit IRC | 02:54 | |
| *** EinstCra_ has joined #openstack-cinder | 02:54 | |
| *** EinstCrazy has quit IRC | 02:54 | |
| *** zhipeng has joined #openstack-cinder | 02:59 | |
| *** dave-mccowan has quit IRC | 03:00 | |
| *** yangxi has joined #openstack-cinder | 03:01 | |
| *** mylu has quit IRC | 03:06 | |
| *** yangxi has quit IRC | 03:06 | |
| *** rdx_matt has joined #openstack-cinder | 03:06 | |
| *** rdx_matt has quit IRC | 03:11 | |
| *** mylu has joined #openstack-cinder | 03:22 | |
| *** salv-orlando has joined #openstack-cinder | 03:23 | |
| *** haplo37_ has joined #openstack-cinder | 03:23 | |
| *** haplo37_ has quit IRC | 03:23 | |
| *** yangxi has joined #openstack-cinder | 03:28 | |
| *** salv-orlando has quit IRC | 03:30 | |
| *** jdurgin has joined #openstack-cinder | 03:31 | |
| *** uberjay has quit IRC | 03:34 | |
| *** yangxi_ has joined #openstack-cinder | 03:34 | |
| *** ntpttr has quit IRC | 03:34 | |
| *** cknight has quit IRC | 03:35 | |
| *** iceyao_ has joined #openstack-cinder | 03:36 | |
| *** iceyao has quit IRC | 03:36 | |
| *** uberjay has joined #openstack-cinder | 03:37 | |
| *** yangxi has quit IRC | 03:37 | |
| *** iceyao has joined #openstack-cinder | 03:39 | |
| *** jdurgin has quit IRC | 03:39 | |
| *** zhipeng has quit IRC | 03:40 | |
| *** links has joined #openstack-cinder | 03:41 | |
| *** itlinux has joined #openstack-cinder | 03:41 | |
| *** iceyao_ has quit IRC | 03:42 | |
| *** iceyao has quit IRC | 03:44 | |
| *** itlinux has quit IRC | 03:45 | |
| *** itlinux has joined #openstack-cinder | 03:46 | |
| *** int32bit has quit IRC | 04:03 | |
| *** rdx_matt has joined #openstack-cinder | 04:07 | |
| *** laughte__ has quit IRC | 04:07 | |
| *** laughterwym has joined #openstack-cinder | 04:08 | |
| *** sheel has joined #openstack-cinder | 04:09 | |
| *** rdx_matt has quit IRC | 04:12 | |
| *** laughterwym has quit IRC | 04:13 | |
| *** yangxi_ has quit IRC | 04:17 | |
| *** IlyaG has quit IRC | 04:18 | |
| *** yangxi has joined #openstack-cinder | 04:19 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Fix wrong volume type of restore action https://review.openstack.org/303709 | 04:26 |
| *** haomaiwang has joined #openstack-cinder | 04:27 | |
| *** salv-orlando has joined #openstack-cinder | 04:27 | |
| *** haplo37 has quit IRC | 04:31 | |
| *** haplo37 has joined #openstack-cinder | 04:32 | |
| *** yangxi has quit IRC | 04:34 | |
| *** lpetrut has joined #openstack-cinder | 04:37 | |
| *** Poornima has joined #openstack-cinder | 04:40 | |
| *** garthb has joined #openstack-cinder | 04:40 | |
| *** asselin_ has quit IRC | 04:41 | |
| *** sgotliv_ has joined #openstack-cinder | 04:41 | |
| *** salv-orlando has quit IRC | 04:43 | |
| *** jwcroppe has quit IRC | 04:50 | |
| *** jwcroppe has joined #openstack-cinder | 04:51 | |
| *** Poornima has quit IRC | 04:52 | |
| *** Poornima has joined #openstack-cinder | 04:52 | |
| *** yangxi has joined #openstack-cinder | 04:57 | |
| *** mylu has quit IRC | 05:02 | |
| *** mylu has joined #openstack-cinder | 05:05 | |
| *** laughterwym has joined #openstack-cinder | 05:08 | |
| *** rdx_matt has joined #openstack-cinder | 05:08 | |
| *** rcernin has joined #openstack-cinder | 05:10 | |
| *** ircuser-1 has quit IRC | 05:11 | |
| *** laughterwym has quit IRC | 05:12 | |
| *** laughterwym has joined #openstack-cinder | 05:12 | |
| *** rdx_matt has quit IRC | 05:13 | |
| *** laughterwym has quit IRC | 05:13 | |
| *** nkrinner has joined #openstack-cinder | 05:13 | |
| *** laughterwym has joined #openstack-cinder | 05:15 | |
| *** laughterwym has joined #openstack-cinder | 05:16 | |
| *** chlong has quit IRC | 05:16 | |
| *** IlyaG has joined #openstack-cinder | 05:18 | |
| *** yangxi has quit IRC | 05:20 | |
| *** yangxi has joined #openstack-cinder | 05:22 | |
| *** IlyaG has quit IRC | 05:23 | |
| *** laughterwym has quit IRC | 05:24 | |
| *** laughterwym has joined #openstack-cinder | 05:25 | |
| *** laughterwym has quit IRC | 05:30 | |
| *** mylu has quit IRC | 05:30 | |
| *** shausy has joined #openstack-cinder | 05:30 | |
| *** lpetrut has quit IRC | 05:30 | |
| *** lpetrut has joined #openstack-cinder | 05:30 | |
| *** yangxi has quit IRC | 05:33 | |
| *** garthb has quit IRC | 05:35 | |
| *** yangxi has joined #openstack-cinder | 05:35 | |
| *** ChubYann has quit IRC | 05:37 | |
| *** salv-orlando has joined #openstack-cinder | 05:40 | |
| *** ntpttr has joined #openstack-cinder | 05:43 | |
| *** Poornima has quit IRC | 05:43 | |
| *** Poornima has joined #openstack-cinder | 05:43 | |
| *** shausy has quit IRC | 05:44 | |
| *** TheBall has quit IRC | 05:44 | |
| *** TheBall has joined #openstack-cinder | 05:44 | |
| *** haplo37 has quit IRC | 05:45 | |
| *** haplo37 has joined #openstack-cinder | 05:46 | |
| *** lpetrut has quit IRC | 05:47 | |
| *** mjura has joined #openstack-cinder | 05:48 | |
| *** salv-orl_ has joined #openstack-cinder | 05:52 | |
| *** salv-orl_ has quit IRC | 05:52 | |
| *** salv-orlando has quit IRC | 05:53 | |
| *** salv-orlando has joined #openstack-cinder | 05:53 | |
| *** e0ne has joined #openstack-cinder | 05:54 | |
| *** salv-orlando has quit IRC | 05:54 | |
| *** salv-orlando has joined #openstack-cinder | 05:55 | |
| *** mjura has quit IRC | 05:56 | |
| *** mjura has joined #openstack-cinder | 05:56 | |
| *** yangxi has quit IRC | 05:58 | |
| *** yangxi has joined #openstack-cinder | 06:01 | |
| *** Poornima has quit IRC | 06:04 | |
| *** laughterwym has joined #openstack-cinder | 06:04 | |
| *** e0ne has quit IRC | 06:05 | |
| *** lpetrut has joined #openstack-cinder | 06:05 | |
| *** rdx_matt has joined #openstack-cinder | 06:09 | |
| *** e0ne has joined #openstack-cinder | 06:10 | |
| *** lpetrut has quit IRC | 06:10 | |
| *** rdx_matt has quit IRC | 06:14 | |
| lixiaoy1 | DuncanT: ping | 06:19 |
| *** haplo37 has quit IRC | 06:20 | |
| *** Poornima has joined #openstack-cinder | 06:20 | |
| *** wanghao has quit IRC | 06:22 | |
| *** huyang has quit IRC | 06:22 | |
| *** huyang has joined #openstack-cinder | 06:22 | |
| *** wanghao has joined #openstack-cinder | 06:23 | |
| *** lpetrut has joined #openstack-cinder | 06:25 | |
| *** yangxi has quit IRC | 06:28 | |
| *** bjornar has joined #openstack-cinder | 06:29 | |
| *** yangxi has joined #openstack-cinder | 06:29 | |
| openstackgerrit | Accela Zhao proposed openstack/cinder: Inherit multiattach from volume source https://review.openstack.org/276312 | 06:29 |
| *** e0ne has quit IRC | 06:30 | |
| *** ankit_ag has joined #openstack-cinder | 06:31 | |
| *** e0ne has joined #openstack-cinder | 06:31 | |
| *** anshul has joined #openstack-cinder | 06:35 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Remove CG db access in Storwize driver https://review.openstack.org/296264 | 06:40 |
| openstackgerrit | Accela Zhao proposed openstack/cinder: Dedup code for test_cgsnapshots.py https://review.openstack.org/276338 | 06:44 |
| *** e0ne has quit IRC | 06:47 | |
| *** lpetrut has quit IRC | 06:48 | |
| *** pots1 has joined #openstack-cinder | 06:51 | |
| *** Poornima has quit IRC | 06:57 | |
| *** jwcroppe has quit IRC | 06:58 | |
| *** bjornar has quit IRC | 07:02 | |
| DuncanT | lixiaoy1: Good morning | 07:10 |
| *** Poornima has joined #openstack-cinder | 07:10 | |
| *** rdx_matt has joined #openstack-cinder | 07:10 | |
| lixiaoy1 | DuncanT: I saw your comments about my specs, https://review.openstack.org/#/c/248593/11 | 07:10 |
| DuncanT | lixiaoy1: Yes | 07:10 |
| lixiaoy1 | DuncanT: I didn't understand it. Could you kindly give more description? | 07:11 |
| lixiaoy1 | DuncanT: meanwhile I am wondering if it should be included in Castellan bp? | 07:12 |
| DuncanT | lixiaoy1: So at the moment, for all of the scenarios where we support retype, an admin can initiate the retype, or the owning tenant can request it | 07:13 |
| DuncanT | lixiaoy1: In the case of encrypted retype, that isn't necessarily going to be true | 07:14 |
| DuncanT | lixiaoy1: Barbican can be configured only to give key material to the tenant, not to admin | 07:14 |
| *** rdx_matt has quit IRC | 07:14 | |
| DuncanT | lixiaoy1: And bring-your-own-key, where the key isn't even stored unless the volume is attached, is coming too | 07:15 |
| DuncanT | lixiaoy1: This adds some limitations to retype, and an error path, that isn't there today. This needs to be called out, I think | 07:17 |
| lixiaoy1 | DuncanT: yes, this leads problem when admin initiates retype encrypted volumes | 07:18 |
| lixiaoy1 | DuncanT: I will do more investigations. | 07:18 |
| DuncanT | lixiaoy1: That fact needs to be reflected in the spec, and what the error reporting is (initially and once we have user facing errors) should be detailed | 07:19 |
| lixiaoy1 | DuncanT: OK, got it. | 07:20 |
| DuncanT | lixiaoy1: I don't think it's a big change, just a couple of notes to add | 07:20 |
| lixiaoy1 | DuncanT: yeah. Thank you for pointing it out.. | 07:20 |
| *** ebalduf has quit IRC | 07:20 | |
| lixiaoy1 | DuncanT: another things about encrypted volume is about creating encrypted from source volume or snapshot | 07:21 |
| DuncanT | lixiaoy1: It gets the same key today, right? | 07:21 |
| lixiaoy1 | DuncanT: currently Cinder only checks whether new volume uses same backend storage as snapshot, or source volume, and then call driver function to create encrypted volume | 07:22 |
| lixiaoy1 | DuncanT: this definitely is a problem. as it just writes raw data into encrypted volumes | 07:22 |
| DuncanT | When you create a volume from snap or clone one, it always ends up on the same backend, doesn't it? | 07:22 |
| *** arecknag has joined #openstack-cinder | 07:23 | |
| lixiaoy1 | DuncanT: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L251 | 07:23 |
| lixiaoy1 | DuncanT: yes | 07:23 |
| *** tesseract has joined #openstack-cinder | 07:24 | |
| DuncanT | So as long as it copies over the key id etc, why is a raw copy a problem? | 07:24 |
| *** tesseract is now known as Guest22945 | 07:24 | |
| *** andymaier has joined #openstack-cinder | 07:25 | |
| lixiaoy1 | DuncanT: for example, I have two volume types: encrypted and unencrypted. Volume A is created with encrypted type. And then I create a encrypted volume from volume A, | 07:25 |
| lixiaoy1 | DuncanT: Cinder checks that the two volumes are in same backend storage, and then it calls driver.create_cloned_volume to create a new volume | 07:25 |
| lixiaoy1 | DuncanT: as a result, in storage disks, volume B has same data as volume A. This is not correct, as volume B is encrypted, and volume A is not encrypted, they should have different data in storage disks | 07:26 |
| DuncanT | lixiaoy1: Ah, ok, yes, that's a broken scenario too. Do we have a bug open for it? | 07:27 |
| lixiaoy1 | DuncanT: this is same to creating encrypted volume from image, I added comments to bug https://bugs.launchpad.net/cinder/+bug/1482464 | 07:28 |
| openstack | Launchpad bug 1482464 in Cinder "cinder wrote unencrypted data to encrypted volumes when creating from an image" [High,Triaged] - Assigned to Lisa Li (lisali) | 07:28 |
| lixiaoy1 | DuncanT: but I have a concern about how to fix it? As if we want to make the data correct, we have to map the encrypted volumes to cinder volume host, and copied data from source to dest through cinder volume host | 07:29 |
| DuncanT | I think it should be broken out into a separate bug, since it will be fixed in a separate patch, probably | 07:30 |
| lixiaoy1 | DuncanT: ok, I will reopen a new bug | 07:30 |
| DuncanT | lixiaoy1: If we're using lux encryption? yes | 07:30 |
| lixiaoy1 | DuncanT: yes | 07:30 |
| lixiaoy1 | DuncanT: but volume creation from source volume and snapshot, currently Cinder just does the tasks in storage backend | 07:31 |
| DuncanT | lixiaoy1: cinder can do clone via a dd currently if the storage backend can't do it for us - we'll have to do the same thing here | 07:31 |
| lixiaoy1 | DuncanT: that is good | 07:32 |
| DuncanT | lixiaoy1: It's going to be complicated and require plenty of test scenarios to ensure we've got it right, but I think it is all fixable | 07:33 |
| lixiaoy1 | DuncanT: yeah. once the fix solution is decided, work is not hard | 07:34 |
| lixiaoy1 | DuncanT: The third thing I am thinking is about encryption in backup. This is related to encryption function in traditional storages | 07:34 |
| lixiaoy1 | DuncanT: currently encryption function is also provided in vendor storage, like HP, EMC, IBM products etc. | 07:35 |
| DuncanT | Encrypted backup has been mentioned by some product manager types before. Need to figure out how that ties in with backup of encrypted volumes too | 07:36 |
| lixiaoy1 | DuncanT: seems it is called at-rest encryption?! that is if users create a encrypted volume from storage products, and later he wants to backup it to ceph/swift, | 07:36 |
| *** lpetrut has joined #openstack-cinder | 07:36 | |
| DuncanT | Most vendor encryption is at-rest encryption, yes. LUKS covers the network too | 07:36 |
| lixiaoy1 | DuncanT: in Cinder, it backups the raw/unencrypted data to ceph/swift. | 07:36 |
| lixiaoy1 | DuncanT: currently swift also provides at-rest encryption, I am wondering whether Cinder needs to provide the function to backup such volume in encrypted format in swift/ceph | 07:37 |
| DuncanT | lixiaoy1: I'm not sure. That's kind of a question for security folks, about whether what is there is sufficient for the various security standards, and if not, how to fix it | 07:38 |
| DuncanT | lixiaoy1: I'll poke the security team at the summit, see what they can tell us | 07:39 |
| lixiaoy1 | DuncanT: OK, thanks | 07:39 |
| lixiaoy1 | DuncanT: thank you for your time. nice day! | 07:41 |
| *** avishay has joined #openstack-cinder | 07:45 | |
| *** yangxi has quit IRC | 07:47 | |
| *** qeelee has joined #openstack-cinder | 07:53 | |
| *** yangxi has joined #openstack-cinder | 07:55 | |
| *** yangxi has quit IRC | 07:57 | |
| *** anshul has quit IRC | 07:59 | |
| *** anshul has joined #openstack-cinder | 08:00 | |
| *** yangxi has joined #openstack-cinder | 08:03 | |
| *** jseiler_ has joined #openstack-cinder | 08:03 | |
| *** geguileo has joined #openstack-cinder | 08:04 | |
| *** yangxi has quit IRC | 08:04 | |
| *** yangxi has joined #openstack-cinder | 08:05 | |
| *** qeelee has quit IRC | 08:07 | |
| *** bjornar has joined #openstack-cinder | 08:07 | |
| *** qeelee has joined #openstack-cinder | 08:07 | |
| *** jseiler has quit IRC | 08:07 | |
| *** adrianofr has quit IRC | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** adrianofr has joined #openstack-cinder | 08:10 | |
| *** rdx_matt has joined #openstack-cinder | 08:10 | |
| *** e0ne has joined #openstack-cinder | 08:13 | |
| *** jistr has joined #openstack-cinder | 08:16 | |
| *** salv-orlando has quit IRC | 08:34 | |
| *** qeelee has quit IRC | 08:36 | |
| openstackgerrit | Merged openstack/cinder: Pylint: refactor ignored message lists https://review.openstack.org/301314 | 08:39 |
| openstackgerrit | xiaoqin proposed openstack/cinder: Remove CG db access in Storwize driver https://review.openstack.org/296264 | 08:45 |
| *** laughterwym has quit IRC | 08:45 | |
| *** laughterwym has joined #openstack-cinder | 08:46 | |
| *** Julien-zte has quit IRC | 08:46 | |
| *** qeelee has joined #openstack-cinder | 08:49 | |
| yangxi | haha | 08:50 |
| *** jordanP has joined #openstack-cinder | 08:54 | |
| *** sheel has quit IRC | 08:55 | |
| *** laughterwym has quit IRC | 08:55 | |
| *** Poornima has quit IRC | 08:58 | |
| *** salv-orlando has joined #openstack-cinder | 08:59 | |
| *** salv-orlando has quit IRC | 09:00 | |
| *** salv-orlando has joined #openstack-cinder | 09:01 | |
| *** Poornima has joined #openstack-cinder | 09:03 | |
| *** ociuhandu has joined #openstack-cinder | 09:05 | |
| *** adrianofr has quit IRC | 09:22 | |
| *** yangxi has quit IRC | 09:22 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races on extend and volume_upload_image https://review.openstack.org/216378 | 09:23 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from consistency groups https://review.openstack.org/259429 | 09:23 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from migrate and retype https://review.openstack.org/221442 | 09:23 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add ordering possibilities to conditional update https://review.openstack.org/231936 | 09:23 |
| *** hogepodge has joined #openstack-cinder | 09:23 | |
| *** yangxi has joined #openstack-cinder | 09:24 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement unmanage volume for RBD driver https://review.openstack.org/307342 | 09:25 |
| *** laughterwym has joined #openstack-cinder | 09:26 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Check flashcopy mapping before deleting volume https://review.openstack.org/268701 | 09:28 |
| openstackgerrit | wanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata https://review.openstack.org/147738 | 09:34 |
| *** klkumar has joined #openstack-cinder | 09:42 | |
| *** qeelee has quit IRC | 09:50 | |
| *** klkumar has quit IRC | 09:53 | |
| openstackgerrit | Cao ShuFeng proposed openstack/cinder: Fix wrong volume type of restore action https://review.openstack.org/303709 | 09:54 |
| *** IlyaG has joined #openstack-cinder | 09:55 | |
| *** xavior has joined #openstack-cinder | 09:55 | |
| xavior | hi all, does any one know how to use HTTPClient for python-cinderclient | 09:55 |
| xavior | by default it is uisng SessionClient as auth_plugin is not provided | 09:56 |
| xavior | any link/help to use auth_plugin??? | 09:56 |
| xavior | thank you | 09:56 |
| *** IlyaG has quit IRC | 09:59 | |
| *** cheneydc has quit IRC | 10:01 | |
| *** sdague has joined #openstack-cinder | 10:08 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods https://review.openstack.org/286598 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add service_nodes table and related methods https://review.openstack.org/286599 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add workers table https://review.openstack.org/303018 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add worker's DB operations https://review.openstack.org/303019 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cleanable base object and cleanup request VO https://review.openstack.org/303020 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make c-vol use workers table for cleanup https://review.openstack.org/303021 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Allow triggering cleanup from API https://review.openstack.org/303022 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make c-bak use workers table for cleanup https://review.openstack.org/303023 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add auto cleanup mechanism https://review.openstack.org/303024 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add job distribution API features https://review.openstack.org/294263 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add ServiceNode Versioned Object https://review.openstack.org/286600 | 10:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Change Job Distribution for HA A-A https://review.openstack.org/286601 | 10:09 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
| *** ndipanov has quit IRC | 10:14 | |
| *** yangyap__ has joined #openstack-cinder | 10:15 | |
| *** EinstCrazy has joined #openstack-cinder | 10:15 | |
| *** ndipanov has joined #openstack-cinder | 10:18 | |
| *** EinstCra_ has quit IRC | 10:18 | |
| *** yangyape_ has quit IRC | 10:19 | |
| *** yangyap__ has quit IRC | 10:19 | |
| *** EinstCrazy has quit IRC | 10:20 | |
| xavior | dims: hi you around? | 10:20 |
| *** gcb has joined #openstack-cinder | 10:28 | |
| *** asselin_ has joined #openstack-cinder | 10:37 | |
| *** markvoelker has joined #openstack-cinder | 10:37 | |
| *** markvoelker has quit IRC | 10:42 | |
| *** asselin_ has quit IRC | 10:43 | |
| *** jwcroppe has joined #openstack-cinder | 10:43 | |
| *** jwcroppe has quit IRC | 10:49 | |
| *** tpsilva has joined #openstack-cinder | 10:52 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add ability to filter by volume_glance_metadata https://review.openstack.org/147738 | 11:06 |
| *** xavior has quit IRC | 11:07 | |
| dims | ouch looks like xavior dropped off | 11:09 |
| *** julim has joined #openstack-cinder | 11:12 | |
| *** zhangjn has joined #openstack-cinder | 11:12 | |
| *** yangxi has quit IRC | 11:16 | |
| *** mjura has quit IRC | 11:30 | |
| *** salv-orlando has quit IRC | 11:34 | |
| *** mtanino has joined #openstack-cinder | 11:38 | |
| *** markvoelker has joined #openstack-cinder | 11:38 | |
| *** yhayashi has quit IRC | 11:39 | |
| *** markvoelker has quit IRC | 11:42 | |
| *** lprice1 has joined #openstack-cinder | 11:44 | |
| *** lprice has quit IRC | 11:44 | |
| *** mjura has joined #openstack-cinder | 11:45 | |
| *** salv-orlando has joined #openstack-cinder | 11:47 | |
| *** Guest22945 is now known as tesseract | 11:54 | |
| *** tesseract is now known as Guest73397 | 11:54 | |
| *** raildo-afk is now known as raildo | 11:58 | |
| *** haomaiwang has quit IRC | 12:00 | |
| *** jwcroppe has joined #openstack-cinder | 12:05 | |
| *** jwcroppe has quit IRC | 12:11 | |
| *** EinstCrazy has joined #openstack-cinder | 12:13 | |
| *** iceyao has joined #openstack-cinder | 12:15 | |
| *** jwcroppe has joined #openstack-cinder | 12:16 | |
| *** mtreinish has quit IRC | 12:18 | |
| *** markvoelker has joined #openstack-cinder | 12:19 | |
| *** mtreinish has joined #openstack-cinder | 12:21 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/307747 | 12:22 |
| *** ankit_ag has quit IRC | 12:25 | |
| openstackgerrit | Avishay Traeger proposed openstack/cinder: Map volume/snapshot manage extensions to v3 https://review.openstack.org/307324 | 12:34 |
| *** mjura has quit IRC | 12:40 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Deprecating XML config file https://review.openstack.org/294076 | 12:41 |
| *** jdurgin has joined #openstack-cinder | 12:43 | |
| *** houming has quit IRC | 12:50 | |
| *** haomaiwang has joined #openstack-cinder | 12:52 | |
| *** mjura has joined #openstack-cinder | 12:53 | |
| *** dave-mccowan has joined #openstack-cinder | 12:54 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:55 | |
| *** haomaiwa_ has quit IRC | 12:55 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:56 | |
| *** haomaiwang has quit IRC | 12:59 | |
| *** akerr has joined #openstack-cinder | 12:59 | |
| *** Julien-zte has joined #openstack-cinder | 13:02 | |
| *** gouthamr has joined #openstack-cinder | 13:02 | |
| openstackgerrit | Gorka Eguileor proposed openstack/python-cinderclient: Use six.moves.urllib.parse urlencode https://review.openstack.org/302961 | 13:03 |
| *** avishay has quit IRC | 13:03 | |
| *** rlrossit has joined #openstack-cinder | 13:03 | |
| *** eharney has joined #openstack-cinder | 13:04 | |
| *** cknight has joined #openstack-cinder | 13:09 | |
| *** akerr is now known as akerr_away | 13:11 | |
| *** akerr_away is now known as akerr | 13:11 | |
| *** Yogi1 has joined #openstack-cinder | 13:16 | |
| e0ne | hi. could anybody from cores take a look on https://review.openstack.org/231663 (remove xml api), please? | 13:16 |
| scottda | e0ne: If you have time and are still interested, we could use another set of eyes on cinderclient support for microversions: https://review.openstack.org/#/c/301941 | 13:17 |
| e0ne | scottda: sure, I'll do it in few minutes | 13:18 |
| scottda | thanks. /v3 endpoint support looks close, I think smcginnis might be ready to +A if you are interested in that as well: https://review.openstack.org/#/c/300028 | 13:19 |
| *** porrua has joined #openstack-cinder | 13:20 | |
| *** mriedem_away is now known as mriedem | 13:25 | |
| *** mylu has joined #openstack-cinder | 13:26 | |
| *** crose has joined #openstack-cinder | 13:28 | |
| *** laughter_ has joined #openstack-cinder | 13:29 | |
| *** xyang1 has joined #openstack-cinder | 13:30 | |
| *** laughter_ has quit IRC | 13:31 | |
| *** laughter_ has joined #openstack-cinder | 13:31 | |
| *** BigWillie has joined #openstack-cinder | 13:31 | |
| *** mriedem has quit IRC | 13:33 | |
| *** mriedem has joined #openstack-cinder | 13:35 | |
| *** mriedem has quit IRC | 13:36 | |
| *** openstackstatus has joined #openstack-cinder | 13:36 | |
| *** ChanServ sets mode: +v openstackstatus | 13:36 | |
| *** mriedem has joined #openstack-cinder | 13:36 | |
| scottda | smcginnis: Remember when I said a quick way to determine if a microversion was necessary was to ask "is this a backwards incompatible change?" I was wrong on that ....https://review.openstack.org/#/c/243041/7/guidelines/microversion_specification.rst | 13:37 |
| scottda | https://www.irccloud.com/pastebin/3K1WrOJv/ | 13:37 |
| smcginnis | scottda: So really any change? | 13:38 |
| scottda | smcginnis: That seems to be what the API WG is stating, so yes. | 13:38 |
| scottda | We can mull this over, and discuss at the Summit. I'll change the devref after we get to some consensus. | 13:39 |
| -openstackstatus- NOTICE: We have recovered one of our cloud providers, but there is a huge backlog of jobs to process. Please have patience until your jobs are processed | 13:39 | |
| smcginnis | "Y should be changed when you make any change to the API" - huh, I guess so. | 13:39 |
| openstackgerrit | Alex Meade proposed openstack/cinder: init_host offload to default to true https://review.openstack.org/306402 | 13:40 |
| ameade | geguileo, tbarron: https://review.openstack.org/#/c/306402/ | 13:41 |
| ameade | simplified it to just change the default | 13:41 |
| ameade | thanks for revs | 13:41 |
| tbarron | ameade: cool, will look | 13:42 |
| geguileo | ameade: You should add release note to mention that default has been changed | 13:42 |
| ameade | oh shoot i lost that in my update | 13:43 |
| ameade | good catch | 13:43 |
| geguileo | ameade: I know you had it before :-) | 13:43 |
| openstackgerrit | Alex Meade proposed openstack/cinder: init_host offload to default to true https://review.openstack.org/306402 | 13:44 |
| ameade | pushed | 13:44 |
| geguileo | ameade: I don't think the bug is relevant anymore | 13:46 |
| ameade | i updated it | 13:46 |
| ameade | but it's possible we dont need it at all | 13:46 |
| geguileo | ameade: I don't think it's relevant | 13:47 |
| ameade | does the current wording of the bug make it relevant? | 13:49 |
| ameade | but yeah i suppose it's not exactly a 'bug' at this point | 13:50 |
| *** Poornima has quit IRC | 13:52 | |
| *** Poornima has joined #openstack-cinder | 13:52 | |
| ameade | ugh new gerrit | 13:52 |
| ameade | trying to edit commit message | 13:53 |
| *** geguileo has quit IRC | 13:53 | |
| smcginnis | :) | 13:53 |
| *** diablo_rojo has joined #openstack-cinder | 13:53 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: init_host offload to default to true https://review.openstack.org/306402 | 13:53 |
| *** jordanP has quit IRC | 13:53 | |
| ameade | lol was on old patchset -_- | 13:54 |
| ameade | whoops lost gorka | 13:54 |
| e0ne | scottda: looking on your patches... | 13:55 |
| e0ne | scottda: why do we need changes here https://review.openstack.org/#/c/301941/12/cinderclient/tests/unit/test_shell.py? | 13:55 |
| *** jgregor has joined #openstack-cinder | 13:56 | |
| *** yangyape_ has joined #openstack-cinder | 13:57 | |
| *** crose has quit IRC | 13:57 | |
| *** Poornima has quit IRC | 13:57 | |
| scottda | e0ne: That looks like a typo or rebase error. I'll fix that up. Good catch. | 13:59 |
| e0ne | scottda: thanks! everything else looks good for m | 13:59 |
| e0ne | s/for m/for me | 13:59 |
| scottda | e0ne: Thanks for looking. | 13:59 |
| *** mtanino has quit IRC | 14:00 | |
| e0ne | scottda: you're welcome | 14:00 |
| *** haplo37 has joined #openstack-cinder | 14:03 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Support api-microversions https://review.openstack.org/301941 | 14:03 |
| e0ne | scottda: fyi https://review.openstack.org/#/c/300028/12/cinderclient/v3/shell.py - I'll remove my -1 once you answer my question | 14:03 |
| *** links has quit IRC | 14:03 | |
| *** laughter_ has quit IRC | 14:04 | |
| *** mylu has quit IRC | 14:05 | |
| openstackgerrit | Alex Meade proposed openstack/cinder: User messages API for error cases https://review.openstack.org/298052 | 14:05 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 14:06 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove support for 1.x volume RPC API https://review.openstack.org/297701 | 14:06 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 14:06 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 14:06 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 14:06 |
| *** dustins has joined #openstack-cinder | 14:07 | |
| *** mylu has joined #openstack-cinder | 14:07 | |
| scottda | e0ne: I've answered. I'm not sure it's a very good answer... | 14:07 |
| *** haplo37 has quit IRC | 14:08 | |
| xyang1 | smcginnis: what can be backported to Liberty now? only critical bugs and security patches? | 14:09 |
| e0ne | scottda: thanks | 14:09 |
| bswartz | I thought it was security patches only | 14:09 |
| xyang1 | bswartz: Liberty is 6-12 months, so phase II? | 14:11 |
| smcginnis | xyang1, bswartz: Yep, security only mode. | 14:11 |
| smcginnis | Though there is wording in there that keeps the door open for serious user impact, IIRC. | 14:11 |
| *** merooney has joined #openstack-cinder | 14:11 | |
| xyang1 | smcginnis: bswartz http://docs.openstack.org/project-team-guide/stable-branches.html | 14:12 |
| bswartz | oh it looks like the policy has evolved | 14:13 |
| xyang1 | smcginnis: bswartz isn't mitaka phase I, liberty phase II? | 14:13 |
| bswartz | yes it looks that way | 14:13 |
| bswartz | and kilo is in phase III | 14:13 |
| bswartz | previously, phase III didn't exist | 14:13 |
| xyang1 | bswartz: ok | 14:14 |
| openstackgerrit | Alex Meade proposed openstack/cinder: User messages API for error cases https://review.openstack.org/298052 | 14:14 |
| *** wanghao has quit IRC | 14:14 | |
| *** laughterwym has joined #openstack-cinder | 14:14 | |
| *** baumann has joined #openstack-cinder | 14:15 | |
| smcginnis | SO yeah, the "critical bugfixes" bit is what I was thinking of. | 14:15 |
| *** wanghao has joined #openstack-cinder | 14:15 | |
| smcginnis | It should mostly just be for security patches, but if there is something significant enough we can allow that through. | 14:15 |
| xyang1 | smcginnis: so there is room for negotiation?:) | 14:15 |
| smcginnis | xyang1: Hah, yep, I guess so. | 14:16 |
| smcginnis | Suddenly I feel a little nervous. ;) | 14:16 |
| xyang1 | smcginnis: :) | 14:16 |
| smcginnis | scottda: Now in a day or two that should make it through gate. :/ | 14:17 |
| scottda | smcginnis: :) | 14:18 |
| *** laughterwym has quit IRC | 14:19 | |
| *** al_indigo has left #openstack-cinder | 14:20 | |
| openstackgerrit | Merged openstack/python-cinderclient: Change extension module naming to a shorter one https://review.openstack.org/257306 | 14:20 |
| *** haplo37 has joined #openstack-cinder | 14:21 | |
| *** avishay has joined #openstack-cinder | 14:22 | |
| *** salv-orlando has quit IRC | 14:23 | |
| *** asselin_ has joined #openstack-cinder | 14:24 | |
| *** ebalduf has joined #openstack-cinder | 14:26 | |
| *** jordanP has joined #openstack-cinder | 14:27 | |
| *** merooney has quit IRC | 14:29 | |
| ameade | I think it's crazy we only support a release for 6 months, esp without any LTSes | 14:31 |
| *** vml has joined #openstack-cinder | 14:32 | |
| gouthamr | ameade: +1 | 14:33 |
| ameade | just wanted that opinion on the record lol | 14:33 |
| *** geguileo has joined #openstack-cinder | 14:34 | |
| smcginnis | ameade: I know there was some discussion not long ago about extending that. | 14:34 |
| bswartz | I support the LTS concept but only if done right | 14:34 |
| smcginnis | ameade: The problem is having the resources to do that. | 14:34 |
| ameade | sure | 14:34 |
| ameade | it's weird that a distro can find a bug, fix it, but not provide the fix to upstream | 14:35 |
| smcginnis | I think general consensus was that falls on the vendors that want to charge for support. | 14:35 |
| bswartz | smcginnis: +1 | 14:35 |
| *** jungleboyj has joined #openstack-cinder | 14:36 | |
| *** sheel has joined #openstack-cinder | 14:36 | |
| *** vml is now known as majkel | 14:37 | |
| *** mtanino has joined #openstack-cinder | 14:39 | |
| openstackgerrit | Merged openstack/cinder: Brocade FCZM: Fix LOGOUT_PAGE reference https://review.openstack.org/301264 | 14:40 |
| smcginnis | scottda: I think we're OK either way, but since we're waiting on Jenkins anyway, want to quick fix the nits I pointed out here: https://review.openstack.org/#/c/301941/ | 14:40 |
| scottda | smcginnis: sure, one minute... | 14:41 |
| smcginnis | scottda: If you do, I think we can just +2/+A that one and wait for it to make it's way through the queue. | 14:41 |
| *** mylu has quit IRC | 14:42 | |
| majkel | Hi guys, I'm trying to setup Cinder LVM + iSCSI (using tgtd on Ubuntu) in Mitaka, but have some slight problems (oslo_messaging.rpc.dispatcher InvalidInput: Invalid input received: Connector doesn't have required information: initiator) when trying to attach to a volume. | 14:42 |
| majkel | Any idea what can be wrong? ;) | 14:43 |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Support api-microversions https://review.openstack.org/301941 | 14:44 |
| scottda | smcginnis: Done&Done | 14:44 |
| smcginnis | scottda: Thanks! | 14:44 |
| scottda | smcginnis: So, are we not supposed to use Depends-On for in-the-same-repo dependencies ? It seems that might cause issues with unit tests passing, at least in some cases. | 14:45 |
| *** enriquetaso has joined #openstack-cinder | 14:46 | |
| *** akerr_ has joined #openstack-cinder | 14:47 | |
| *** akerr has quit IRC | 14:47 | |
| *** e0ne has quit IRC | 14:47 | |
| *** avishay has quit IRC | 14:48 | |
| *** mylu has joined #openstack-cinder | 14:52 | |
| *** nkrinner has quit IRC | 14:54 | |
| *** lprice has joined #openstack-cinder | 14:58 | |
| *** lprice1 has quit IRC | 15:00 | |
| *** mjura has quit IRC | 15:02 | |
| *** diablo_rojo has quit IRC | 15:02 | |
| smcginnis | scottda: Sorry, pulled away... | 15:02 |
| smcginnis | scottda: Yeah, that's really meant for cross-repo dependencies that you can't normal represent in git. | 15:02 |
| smcginnis | scottda: So things like marking a dependency for a cinderclient patch so it doesn't merge until the cinder side does. | 15:03 |
| smcginnis | scottda: Otherwise is just git branching off of git branching within the same repo. | 15:03 |
| *** garthb has joined #openstack-cinder | 15:06 | |
| *** mylu has quit IRC | 15:07 | |
| *** diablo_rojo has joined #openstack-cinder | 15:10 | |
| *** mylu has joined #openstack-cinder | 15:10 | |
| *** anshul has quit IRC | 15:11 | |
| *** arch-nemesis has joined #openstack-cinder | 15:14 | |
| *** bjornar has quit IRC | 15:15 | |
| *** jwcroppe_ has joined #openstack-cinder | 15:16 | |
| *** jwcroppe has quit IRC | 15:17 | |
| openstackgerrit | Merged openstack/python-cinderclient: Add /v3 endpoint support for cinderclient https://review.openstack.org/300028 | 15:20 |
| scottda | smcginnis: OK, thanks. I guess we would just list the dependencies as a comment in the commit message.. | 15:20 |
| *** qeelee has joined #openstack-cinder | 15:22 | |
| smcginnis | scottda: Well, if it's a real dependency in the same repo, the dependent patch should just be branched off of its dependencies. | 15:25 |
| smcginnis | git checkout -b patch1; make changes; git checkout -b patch2; make changes requiring patch1 | 15:25 |
| *** jordanP has quit IRC | 15:26 | |
| bswartz | smcginnis: while that's ideal I've seen a lot of abuse of the "Depends-On" gerrit feature to do intra-repo dependencies | 15:26 |
| scottda | smcginnis: OK, so that would show up in the topic branch, right? | 15:26 |
| smcginnis | bswartz: Yeah, I try to catch that, but not always. | 15:26 |
| *** qeelee has quit IRC | 15:26 | |
| eharney | bswartz: haypo did that when he was posting dozens of "convert to python3" patches, i think it actually made sense there | 15:26 |
| bswartz | smcginnis: there is one benefit that Depends-On has which is that you don't need to rebase on new patchsets of the parent change | 15:26 |
| *** majkel has quit IRC | 15:27 | |
| eharney | right | 15:27 |
| smcginnis | scottda: Honest, I'm not sure how that shows up with the new UI. It used to be very clear when there were dependencies like that. | 15:27 |
| smcginnis | bswartz: True | 15:27 |
| smcginnis | OK, so I guess there are some valid reasons for it. :) | 15:27 |
| smcginnis | But it's original intent was to be able to express dependencies in changes in other repos. | 15:28 |
| bswartz | yeah | 15:28 |
| scottda | So, that "don't need to rebase" is pretty cool. Just out of curiousity, how is using "Depends-On" intra-repo bad? | 15:28 |
| *** Julien-zte has quit IRC | 15:28 | |
| scottda | I mean, that might not be the original intent, but what does it hurt? | 15:28 |
| smcginnis | scottda: I don't think it can enforce real dependencies, but maybe it can now. | 15:28 |
| bswartz | and I don't like to see it abused to much -- especially because those "Depends-On" messages become part of the permanent git commit history of the project | 15:28 |
| eharney | scottda: well, some people's CI systems don't know how to process Depends-On, which isn't great, but... not huge | 15:28 |
| bswartz | it can enforce dependencies, but eharney is right about CI systems | 15:29 |
| bswartz | the major downside IMO is the pollution of the commit messages | 15:29 |
| scottda | eharney: Fair enough. But it seems that CI systems being able to use it would allow CI testing to actually work, since the dependent patches would need to pull in the parent to pass CI. | 15:30 |
| eharney | bswartz would love https://review.openstack.org/#/c/300189/ | 15:30 |
| *** yangyape_ has quit IRC | 15:30 | |
| Swanson | Pycharm offering to install requirements for me is just it mocking me, right? Does this actually ever work? | 15:30 |
| eharney | but, in that particular case, i would be removing those as they merge before that patch merges | 15:30 |
| bswartz | ARGH | 15:30 |
| scottda | Ha! | 15:30 |
| eharney | here i'm using it as a "track numerous assorted changes" feature | 15:30 |
| * bswartz searches for the eye rinse station | 15:31 | |
| eharney | lol | 15:31 |
| *** avishay has joined #openstack-cinder | 15:35 | |
| avishay | scottda: took a couple tries, but got my microversions working. thanks for the help yesterday. | 15:35 |
| scottda | avishay: Anytime. Let me know if I can help further. BTW, will you be in Austin? | 15:36 |
| *** apoorvad has joined #openstack-cinder | 15:42 | |
| avishay | scottda: unfortunately won't make this one. bad timing. should be in barcelona. | 15:43 |
| *** links has joined #openstack-cinder | 15:45 | |
| scottda | avishay: OK. Hope to see you in Spain then. | 15:47 |
| *** ircuser-1 has joined #openstack-cinder | 15:48 | |
| bswartz | scottda: you're assuming Cataluna will still be part of Spain -- I heard they were seceding -- or trying to | 15:49 |
| avishay | I was there in September/October when they voted - was very interesting | 15:50 |
| scottda | bswartz: Good point. As ever, most of my errors come from bad assumptions. | 15:50 |
| smcginnis | Last fall I saw a TON of flags all over the city for Cataluna. | 15:50 |
| avishay | smcginnis: yep, in almost every window, and not just in barcelona | 15:51 |
| avishay | but the biggest fear was that barcelona would be out of the spanish football league :) | 15:51 |
| smcginnis | avishay: Hah! Got to think about the important things. ;) | 15:52 |
| *** crose has joined #openstack-cinder | 15:52 | |
| bswartz | It's all fun and games until someone messes with the futbol league | 15:52 |
| avishay | the spanish government really used that as an excuse for people to vote against succession | 15:53 |
| hemna_ | Nobody expects the Spanish inquisition. | 15:54 |
| avishay | hemna_: :) | 15:55 |
| *** links has quit IRC | 15:57 | |
| sheel | https://review.openstack.org/#/c/278605/36 microversion is waiting for merge.. could some lenient teacher please pass it in exams? | 15:59 |
| sheel | this as well https://review.openstack.org/#/c/273938/ | 15:59 |
| sheel | thanks | 15:59 |
| *** Guest73397 has quit IRC | 16:00 | |
| *** iceyao has quit IRC | 16:02 | |
| *** martyturner has joined #openstack-cinder | 16:02 | |
| *** zzzeek has quit IRC | 16:02 | |
| *** zzzeek has joined #openstack-cinder | 16:02 | |
| *** andymaier has quit IRC | 16:04 | |
| *** leeantho has joined #openstack-cinder | 16:07 | |
| *** anshul has joined #openstack-cinder | 16:08 | |
| *** sdake has joined #openstack-cinder | 16:14 | |
| *** sasukeh has joined #openstack-cinder | 16:17 | |
| openstackgerrit | Sheel Rana proposed openstack/cinder: Microversion of Bootable filter in cinder list https://review.openstack.org/278605 | 16:18 |
| *** anteaya has quit IRC | 16:20 | |
| *** sdake_ has joined #openstack-cinder | 16:25 | |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder-specs: Allow an attached volume to be extended https://review.openstack.org/272524 | 16:27 |
| *** mylu has quit IRC | 16:27 | |
| *** mylu has joined #openstack-cinder | 16:27 | |
| *** sdake has quit IRC | 16:29 | |
| openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fail if oslo.versionedobjects issues Invalid UUID warnings https://review.openstack.org/300189 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from test_db_api unit tests https://review.openstack.org/300730 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Tests: Fix invalid UUIDs in EMC VNX tests https://review.openstack.org/293119 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: fix invalid uuid in emc extremio unit tests https://review.openstack.org/297848 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid uuids in tintri unit tests https://review.openstack.org/297379 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix vol type/spec uuid warnings in api.contrib https://review.openstack.org/304771 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix qos_manage uuid warnings in api.contrib tests https://review.openstack.org/304770 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: fix invalid uuid warnings from test_cmd https://review.openstack.org/297866 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix invalid uuids in EMC scaleio unit tests https://review.openstack.org/297375 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix CG uuid warnings in api.contrib unit tests https://review.openstack.org/304769 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from api.contrib backup tests https://review.openstack.org/304767 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix admin_actions uuid warnings in api.contrib https://review.openstack.org/304768 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings from db unit tests https://review.openstack.org/297255 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix uuid warnings in various api contrib unit tests https://review.openstack.org/304774 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix snapshot test uuid warnings in api.contrib https://review.openstack.org/304773 | 16:32 |
| openstackgerrit | Tom Barron proposed openstack/cinder: Fix volume test uuid warnings in api.contrib https://review.openstack.org/304772 | 16:32 |
| *** mylu has quit IRC | 16:33 | |
| *** IlyaG has joined #openstack-cinder | 16:35 | |
| *** IlyaG has quit IRC | 16:35 | |
| *** IlyaG has joined #openstack-cinder | 16:35 | |
| *** arecknag has quit IRC | 16:35 | |
| *** andymaier has joined #openstack-cinder | 16:36 | |
| akerr_ | scottda: ameade's devstack patch adding v3 endpoint merged yesterday. Mind giving https://review.openstack.org/#/c/301235/ another review? | 16:37 |
| *** IlyaG has quit IRC | 16:37 | |
| *** jistr has quit IRC | 16:39 | |
| *** aimeeU has joined #openstack-cinder | 16:40 | |
| *** salv-orlando has joined #openstack-cinder | 16:48 | |
| *** mtecer has quit IRC | 16:48 | |
| *** yangyape_ has joined #openstack-cinder | 16:49 | |
| *** yangyape_ has quit IRC | 16:50 | |
| *** salv-orlando has quit IRC | 16:51 | |
| *** klkumar has joined #openstack-cinder | 16:53 | |
| *** martyturner has quit IRC | 16:57 | |
| *** pots1 has quit IRC | 16:59 | |
| *** martyturner has joined #openstack-cinder | 17:03 | |
| *** haomaiwa_ has quit IRC | 17:03 | |
| *** haomaiwang has joined #openstack-cinder | 17:03 | |
| *** rcernin has quit IRC | 17:04 | |
| *** sasukeh has quit IRC | 17:04 | |
| *** zhangjn has quit IRC | 17:06 | |
| *** avishay has quit IRC | 17:07 | |
| *** mriedem has quit IRC | 17:08 | |
| *** zhangjn has joined #openstack-cinder | 17:09 | |
| *** garthb has quit IRC | 17:09 | |
| *** EinstCrazy has quit IRC | 17:14 | |
| *** garthb has joined #openstack-cinder | 17:14 | |
| scottda | akerr_: I'm running the tests now... | 17:14 |
| *** mriedem has joined #openstack-cinder | 17:14 | |
| *** ChubYann has joined #openstack-cinder | 17:18 | |
| *** garthb_ has joined #openstack-cinder | 17:19 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Add api-version to get server versions https://review.openstack.org/303627 | 17:19 |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Support api-microversions https://review.openstack.org/301941 | 17:19 |
| *** garthb has quit IRC | 17:21 | |
| *** bjornar has joined #openstack-cinder | 17:23 | |
| *** salv-orlando has joined #openstack-cinder | 17:24 | |
| *** yangyape_ has joined #openstack-cinder | 17:26 | |
| *** rdx_matt has quit IRC | 17:33 | |
| *** rlrossit has quit IRC | 17:39 | |
| *** dave-mccowan has quit IRC | 17:45 | |
| *** rlrossit has joined #openstack-cinder | 17:45 | |
| *** bardia has joined #openstack-cinder | 17:46 | |
| *** haomaiwang has quit IRC | 17:48 | |
| *** haomaiwang has joined #openstack-cinder | 17:48 | |
| *** ndipanov has quit IRC | 17:52 | |
| *** ivase has joined #openstack-cinder | 17:57 | |
| *** Suyash has joined #openstack-cinder | 17:57 | |
| *** martyturner has quit IRC | 17:59 | |
| *** lpetrut has quit IRC | 18:01 | |
| *** martyturner has joined #openstack-cinder | 18:01 | |
| akerr_ | scottda: thanks | 18:01 |
| *** garthb__ has joined #openstack-cinder | 18:02 | |
| *** garthb_ has quit IRC | 18:02 | |
| *** mtreinish has quit IRC | 18:03 | |
| *** alkhodos has joined #openstack-cinder | 18:03 | |
| *** dave-mccowan has joined #openstack-cinder | 18:05 | |
| *** mtreinish has joined #openstack-cinder | 18:08 | |
| *** klkumar has quit IRC | 18:09 | |
| *** asselin_ has quit IRC | 18:16 | |
| scottda | smcginnis: It looks like https://review.openstack.org/#/c/301941 lost your +A due to rebase. | 18:19 |
| jgriffith | smcginnis: sigh... I just can't seem to get OpenStack to work :( | 18:20 |
| *** krtaylor has quit IRC | 18:20 | |
| smcginnis | jgriffith: Sadness | 18:20 |
| jgriffith | smcginnis: looks like I'll have to take my CI offline again until I can figure out why Nova keeps bombing out | 18:20 |
| smcginnis | scottda: Opening now... | 18:20 |
| smcginnis | jgriffith: What's happening? Nova just not happy with something? | 18:20 |
| jgriffith | smcginnis: so I've gotten past the point of the instances failing to build randomly | 18:21 |
| jgriffith | smcginnis: but now I keep getting instances that boot up but can't reach the external network | 18:21 |
| jgriffith | smcginnis: so apt-get update and apt-get install fails | 18:21 |
| smcginnis | Neutron issues again? | 18:22 |
| jgriffith | smcginnis: well... that's the thing; I rebuilt everything going back to my trusty nova-net | 18:22 |
| jgriffith | smcginnis: seems like maybe it's a Nova problem | 18:22 |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP https://review.openstack.org/307974 | 18:23 |
| jgriffith | I think I'm going to go back to using Kilo | 18:23 |
| smcginnis | jgriffith: Oh interesting. So maybe it wasn't neutron before either. | 18:23 |
| jgriffith | It ran for months with no issues | 18:23 |
| smcginnis | jgriffith: LOL | 18:23 |
| jgriffith | smcginnis: right | 18:23 |
| Swanson | jgriffith, smcginnis that's what I was having issues with. I was running openstack on openstack and any instances created on the sub openstack and couldn't get to em. Nova and Neutron. | 18:23 |
| jgriffith | Swanson: intersting, sounds like we hit the same thing | 18:23 |
| jgriffith | Swanson: Liberty? | 18:23 |
| Swanson | jgriffith, liberty and mitaka. Didn't try kilo. | 18:24 |
| jgriffith | Swanson: oh.. wait, that's different | 18:24 |
| jgriffith | Swanson: I can ssh to the Instances fine, but they can't reach the outside world intermittently | 18:24 |
| jgriffith | Swanson: Mitaka had the same issues for me FWIW | 18:24 |
| Swanson | jgriffith, depends on where I start from. | 18:24 |
| *** jwcroppe has joined #openstack-cinder | 18:24 | |
| jgriffith | Swanson: thus my logic in going back to Kilo... it worked :) | 18:24 |
| smcginnis | Local (ufw) firewall issues? | 18:25 |
| jgriffith | smcginnis: can't be... since it's intermittent | 18:25 |
| smcginnis | But then Kilo wouldn't work either too. | 18:25 |
| jgriffith | smcginnis: so out of the last 55 runs; I have 31 that had no issue... and the rest all failed to apt-get update | 18:25 |
| jgriffith | smcginnis: booted up fine, ansible logged in no problem, but they couldn't reach the outside | 18:26 |
| *** jwcroppe_ has quit IRC | 18:27 | |
| smcginnis | jgriffith: Relevant? https://bugzilla.redhat.com/show_bug.cgi?id=983611 | 18:28 |
| openstack | bugzilla.redhat.com bug 983611 in guest-images "Cloud guest images needs to have "NOZEROCONF=yes" in /etc/sysconfig/network" [High,Closed: errata] - Assigned to jboggs | 18:28 |
| smcginnis | Wow, what bug tracker doesn't that bot handle. :) | 18:28 |
| jgriffith | LOL | 18:28 |
| jgriffith | Hmm | 18:29 |
| jgriffith | if I were still on Neutron I'd be suspect | 18:29 |
| *** Yogi1 has quit IRC | 18:29 | |
| jgriffith | smcginnis: and it's got the blanket RHEL statement that it's "fixed in a recent update" | 18:29 |
| *** krtaylor has joined #openstack-cinder | 18:30 | |
| smcginnis | jgriffith: ;) | 18:30 |
| jgriffith | smcginnis: so unless there's some time travel involved :) | 18:30 |
| Swanson | pycharm 2016.1 is bozoing my pastes. At least with mock involved. | 18:30 |
| smcginnis | jgriffith: So it was intermittent. Did you see only certain IP addresses failing? Or same address would work some times and not others? | 18:30 |
| jgriffith | smcginnis: oh... tha'ts for RHEL guest images though | 18:30 |
| smcginnis | jgriffith: Yeah, thought by chance it might apply to Ubuntu as well. | 18:31 |
| jgriffith | smcginnis: so that was my latest change. I sliced my range into just a handful of IP's that worked in the past | 18:31 |
| jgriffith | smcginnis: some failed, some worked. All had been used succesfully at one point or another previously | 18:31 |
| jgriffith | well... | 18:32 |
| smcginnis | Networking sux | 18:32 |
| jgriffith | I don't have that data I guess to say definitively | 18:32 |
| jgriffith | about the pass/fail... not about Networking... that definitely sux | 18:32 |
| smcginnis | Yeah, I don't mean nova-net or neutron. Just networking in general when you have weird issues. | 18:33 |
| jgriffith | agreed | 18:33 |
| * smcginnis is speaking as a former telecom and network admin | 18:33 | |
| jgriffith | smcginnis: nobody should have to endure a past like that | 18:34 |
| jgriffith | smcginnis: glad to see you're making progress with your recovery :) | 18:34 |
| smcginnis | ;) | 18:34 |
| *** pck_ is now known as pck | 18:35 | |
| Swanson | Nice ctrl-alt-shift-v is the new ctrl-v. | 18:36 |
| smcginnis | Swanson: Are you having fun? :) | 18:36 |
| Swanson | Simple paste is the good paste. Smart paste is the child of a brother and sister. | 18:37 |
| *** ociuhandu has quit IRC | 18:38 | |
| smcginnis | Swanson: Stop sniffing the paste. | 18:39 |
| sbezverk | jgriffith we did openstack presentation to the networking people, I mean real ones, they asked us who the hell invented it ;-) | 18:39 |
| *** dave-mccowan has quit IRC | 18:40 | |
| *** sdake_ has quit IRC | 18:41 | |
| *** lpetrut has joined #openstack-cinder | 18:43 | |
| jgriffith | sbezverk: LOL | 18:43 |
| jgriffith | sbezverk: actually I don't know whether to laugh or cry | 18:44 |
| jgriffith | sbezverk: I made the mistake of asking our networking folks here to help me with Neutron a while back.. they asked "are you guys smoking crack" | 18:44 |
| jgriffith | literally... that's what they asked. Followed by; how is anybody that's not an OpenStack expert supposed to figure this mess out? | 18:45 |
| *** itlinux has quit IRC | 18:45 | |
| jgriffith | sbezverk: I guess it's a job security/opportunity for a select few :) | 18:45 |
| Swanson | jgriffith, That's how oracle started. | 18:45 |
| jgriffith | Swanson: ha! | 18:45 |
| jgriffith | Swanson: sbezverk at least I laugh about it now... I used to get so upset and frustrated. The meds are helping :) | 18:46 |
| *** sdake has joined #openstack-cinder | 18:48 | |
| jgriffith | sbezverk: hey wait!!! You work at Cisco don't you? Networking is your specialty!! | 18:48 |
| jgriffith | sbezverk: make you a deal, I'll fix that cinder problem if you come out here and deploy a working Neutron stack for me ;) | 18:49 |
| smcginnis | Hah! | 18:49 |
| Swanson | jgriffith, then image it and ship it around. | 18:49 |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP https://review.openstack.org/307974 | 18:49 |
| jgriffith | Oh... well here's an important one: WARNING: Option "--pub_key" is deprecated; use "--pub-key" | 18:50 |
| Swanson | Madness should not be a requirement to install this stuff. | 18:50 |
| *** ebalduf has quit IRC | 18:52 | |
| *** dustins has quit IRC | 18:53 | |
| *** dustins has joined #openstack-cinder | 18:53 | |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP https://review.openstack.org/307974 | 18:54 |
| *** dustins has quit IRC | 18:56 | |
| *** dustins has joined #openstack-cinder | 18:57 | |
| *** martyturner has quit IRC | 19:01 | |
| *** martyturner has joined #openstack-cinder | 19:02 | |
| *** salv-orlando has quit IRC | 19:03 | |
| *** sheel has quit IRC | 19:05 | |
| *** neophy has joined #openstack-cinder | 19:08 | |
| ccesario | hey, does someone usage storwize_svc driver ? | 19:08 |
| smcginnis | jungleboyj might ^^ | 19:09 |
| *** esker has joined #openstack-cinder | 19:16 | |
| *** akerr_ has quit IRC | 19:16 | |
| *** akerr has joined #openstack-cinder | 19:24 | |
| *** apoorvad has quit IRC | 19:24 | |
| *** mylu has joined #openstack-cinder | 19:25 | |
| Swanson | pycharm 2016.1: Now with extra slowness! | 19:26 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Don't reset volume status when resetting migration status https://review.openstack.org/288732 | 19:27 |
| *** sgotliv_ has quit IRC | 19:29 | |
| *** gouthamr has quit IRC | 19:29 | |
| *** anshul has quit IRC | 19:30 | |
| *** ociuhandu has joined #openstack-cinder | 19:32 | |
| *** mtreinish has quit IRC | 19:41 | |
| *** mtreinish has joined #openstack-cinder | 19:42 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/python-cinderclient: Support name option for volume restore https://review.openstack.org/275910 | 19:43 |
| *** mtreinish has quit IRC | 19:49 | |
| *** neophy has quit IRC | 19:52 | |
| *** gouthamr has joined #openstack-cinder | 19:58 | |
| *** gouthamr has quit IRC | 19:58 | |
| *** gouthamr has joined #openstack-cinder | 19:59 | |
| *** aimeeU has quit IRC | 20:02 | |
| *** lpetrut has quit IRC | 20:04 | |
| *** salv-orlando has joined #openstack-cinder | 20:06 | |
| *** mylu has quit IRC | 20:08 | |
| *** mylu has joined #openstack-cinder | 20:08 | |
| *** salv-orlando has quit IRC | 20:11 | |
| jungleboyj | ccesario: We are part of the team that develops it. | 20:11 |
| *** esker has quit IRC | 20:13 | |
| *** esker has joined #openstack-cinder | 20:13 | |
| *** dave-mccowan has joined #openstack-cinder | 20:14 | |
| *** dave-mcc_ has joined #openstack-cinder | 20:15 | |
| openstackgerrit | Kendall Nelson proposed openstack/os-brick: WIP: Splitting Out Connectors from connector.py :WIP https://review.openstack.org/307974 | 20:15 |
| *** dave-mccowan has quit IRC | 20:18 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Remove support for 1.x volume RPC API https://review.openstack.org/297701 | 20:19 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Update manage_existing to use volume object https://review.openstack.org/258132 | 20:19 |
| *** akerr has quit IRC | 20:21 | |
| *** mylu has quit IRC | 20:21 | |
| *** sgotliv_ has joined #openstack-cinder | 20:27 | |
| *** mylu has joined #openstack-cinder | 20:29 | |
| *** porrua has quit IRC | 20:32 | |
| *** delattec has quit IRC | 20:34 | |
| *** sgotliv_ has quit IRC | 20:34 | |
| *** mylu has quit IRC | 20:35 | |
| *** jungleboyj has quit IRC | 20:36 | |
| *** BigWillie has quit IRC | 20:36 | |
| *** martyturner has quit IRC | 20:36 | |
| *** salv-orlando has joined #openstack-cinder | 20:45 | |
| *** andymaier has quit IRC | 20:54 | |
| *** gouthamr has quit IRC | 20:55 | |
| *** mylu has joined #openstack-cinder | 20:55 | |
| *** delattec has joined #openstack-cinder | 20:57 | |
| *** crose has quit IRC | 21:01 | |
| *** mylu has quit IRC | 21:02 | |
| *** dims_ has joined #openstack-cinder | 21:02 | |
| *** dims has quit IRC | 21:02 | |
| *** haplo37 has quit IRC | 21:03 | |
| patrickeast | Swanson: oh yea, i upgrade to the new version and lasted about 30 min before going back | 21:03 |
| patrickeast | sooo slow | 21:03 |
| *** mylu has joined #openstack-cinder | 21:04 | |
| *** pots1 has joined #openstack-cinder | 21:06 | |
| *** rlrossit has quit IRC | 21:06 | |
| *** pots1 has quit IRC | 21:06 | |
| smcginnis | patrickeast: Hey, just scanning through some bug fixes. Did you see response to you here: https://review.openstack.org/#/c/289298/ | 21:06 |
| patrickeast | smcginnis: probably not : / | 21:07 |
| * patrickeast looking | 21:07 | |
| smcginnis | patrickeast: No rush, just thought I would point it out since I ran across it. | 21:07 |
| *** dave-mcc_ has quit IRC | 21:08 | |
| *** mylu has quit IRC | 21:08 | |
| *** mylu has joined #openstack-cinder | 21:09 | |
| *** haomaiwang has quit IRC | 21:13 | |
| *** dave-mccowan has joined #openstack-cinder | 21:15 | |
| *** jdurgin has quit IRC | 21:17 | |
| *** ivase has quit IRC | 21:19 | |
| *** andymaier has joined #openstack-cinder | 21:24 | |
| openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 21:25 |
| *** sdake_ has joined #openstack-cinder | 21:30 | |
| alkhodos | Hi guys. We have a customer running multiple cinders against single backend and this causes some issues, so I was wondering is there a reason to do so and what are the benefits of having multiple cinder nodes? Perhaps resiliency, maybe something else? | 21:31 |
| *** diablo_rojo has quit IRC | 21:31 | |
| *** mylu has quit IRC | 21:32 | |
| *** sdake has quit IRC | 21:33 | |
| patrickeast | alkhodos: the reasons i've heard are for high availability and performance (needing to handle lots of volume operations concurrently) | 21:34 |
| patrickeast | alkhodos: but active/active cinder-volume services aren't really supported yet, geguileo can probably give the best description of the issues and work remaining to get there | 21:34 |
| alkhodos | patrickeast: that's what I thought too, but customer is using Juno or Kilo, so obviously no HA present on cinder side there... | 21:35 |
| patrickeast | yep | 21:35 |
| patrickeast | well | 21:35 |
| patrickeast | as you found, you *can* deploy that way | 21:35 |
| patrickeast | and it sort of mostly works... .right up until it doesn't | 21:35 |
| guitarzan | we run many c-vols but none of them manage the same volume | 21:36 |
| guitarzan | so it's not "HA" in that sense | 21:36 |
| patrickeast | right, in some cases you can do it and things work great | 21:36 |
| guitarzan | alkhodos: one of the reasons we do is simply because it's easy from a deployment perspective | 21:36 |
| *** dave-mccowan has quit IRC | 21:37 | |
| alkhodos | guitarzan: can you explain what do you mean by none of them manage the same volume ? | 21:37 |
| guitarzan | alkhodos: our backend is clustered | 21:37 |
| guitarzan | so we have lots of little storage nodes | 21:37 |
| alkhodos | guitarzan: I see, so it's 1 cinder per 1 backend in a sort? | 21:37 |
| alkhodos | guitarzan: or 1 storage node | 21:38 |
| guitarzan | alkhodos: kind of, except that our backend itself has a "middle layer" so it's exposed as one backend | 21:38 |
| guitarzan | but this is probably not the same use case as you | 21:38 |
| *** mylu has joined #openstack-cinder | 21:39 | |
| guitarzan | I'm curious what issues they're running into | 21:39 |
| guitarzan | I think you have to do some sneaky things to allow multiple c-vols to be able to manage the same volume | 21:40 |
| guitarzan | and if you're not doing that, I'm not sure what the problem is with multiple c-vols | 21:40 |
| alkhodos | guitarzan: the problem is that we have some problems with concurrent requests | 21:42 |
| guitarzan | alkhodos: on different volumes? | 21:42 |
| alkhodos | guitarzan: and while we can manage cinder to use single/low amount of threads, multiple cinders against single backend cause trouble | 21:42 |
| alkhodos | guitarzan: well, the problem is the Rest API | 21:42 |
| guitarzan | interesting | 21:42 |
| guitarzan | ah, the backend itself gets wrapped in a ball? | 21:43 |
| alkhodos | guitarzan: yep | 21:43 |
| guitarzan | gotcha | 21:43 |
| alkhodos | guitarzan: so I'm trying to understand why would they want to use multi cinders | 21:43 |
| guitarzan | could simply be performance reasons | 21:43 |
| alkhodos | guitarzan: and if it isn't completely necessary - ask them not to | 21:43 |
| alkhodos | guitarzan: ok, I see | 21:43 |
| guitarzan | gotcha | 21:43 |
| alkhodos | guitarzan: thank you for the feedback :) | 21:44 |
| guitarzan | and if one dies, you can still create new volumes | 21:44 |
| alkhodos | guitarzan: ye, that was my understanding | 21:45 |
| guitarzan | it would be nice to hear what their specific concerns are | 21:46 |
| alkhodos | guitarzan: we are going to ask them today, they are from Japan, so this makes communication a bit slow | 21:47 |
| guitarzan | fun | 21:47 |
| patrickeast | ^ fwiw i've also seen deployments that just do it because thats what fuel and tripleo do... and they didn't know they shouldn't are could do something more safe | 21:49 |
| patrickeast | shouldn't or could** | 21:49 |
| Swanson | If you aren't hammering the same volumes and you're backend device can handle the workload is there a real problem with using multiple backends? (Big caveats, sure.) | 21:50 |
| guitarzan | patrickeast: that's interesting | 21:51 |
| patrickeast | Swanson: i'll defer to geguileo for that one :D | 21:51 |
| guitarzan | Swanson: no, if the volumes are distinct, there really shouldn't be any issues | 21:51 |
| guitarzan | "works for me" anyway | 21:51 |
| guitarzan | and I have lots of c-vols :) | 21:52 |
| Swanson | "you're"? I typed "you're" instead of "your"? Going to go hang myself. | 21:52 |
| *** Lee1092 has quit IRC | 21:52 | |
| *** mriedem has quit IRC | 21:52 | |
| *** mylu has quit IRC | 21:53 | |
| guitarzan | dont go all grammar nazi on yourself | 21:53 |
| *** jgregor has quit IRC | 21:54 | |
| *** ametts has quit IRC | 21:57 | |
| *** xyang1 has quit IRC | 22:01 | |
| openstackgerrit | Xinli Guan proposed openstack/cinder: IBM FlashSystem: Cleanup host resource leaking https://review.openstack.org/307451 | 22:01 |
| *** bjornar has quit IRC | 22:05 | |
| *** karthikp has joined #openstack-cinder | 22:06 | |
| *** mylu has joined #openstack-cinder | 22:07 | |
| *** cknight has quit IRC | 22:08 | |
| *** baumann has quit IRC | 22:10 | |
| *** Julien-zte has joined #openstack-cinder | 22:11 | |
| *** mtreinish has joined #openstack-cinder | 22:15 | |
| *** tpsilva has quit IRC | 22:17 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP: Update LVM code so nova can use it https://review.openstack.org/308054 | 22:18 |
| karthikp | Hi, I am working on patch whcih requires me to me to pass a value from the vendor specific implementation to the driver.py | 22:19 |
| karthikp | I was thinking of adding a **kwargs to the abstract method: def create_volume(self, volume) from driver.py so that the vendors could pass up a value down from their code below | 22:20 |
| *** enriquetaso has quit IRC | 22:22 | |
| karthikp | This patch requires a vendor to send a different volume size after its creation ..because of bigger granular size for vol | 22:24 |
| *** salv-orlando has quit IRC | 22:26 | |
| *** sdake_ is now known as sake | 22:26 | |
| *** IlyaG has joined #openstack-cinder | 22:32 | |
| *** IlyaG has quit IRC | 22:32 | |
| hemna_ | karthikp, why do you need kwargs? | 22:35 |
| hemna_ | I think that opens up the API for lots of nonsense | 22:35 |
| hemna_ | what do you mean "pass a value to a vendor specific implementation to the driver.py" ? | 22:36 |
| hemna_ | karthikp, what aren't volume_types good enough for your vendor specific implementation? | 22:37 |
| hemna_ | it's partially why we have some 80+ vendor drivers and haven't had a need to change the create_volume() API | 22:37 |
| hemna_ | </rant> | 22:37 |
| karthikp | hemna_, Some vendors like ScaleIO have a bigger granular size(suppose 5GB) for their Volume size, so if we give cinder create 1 ..they need a way to log that saying the volume is of 5gb and 1 gb | 22:40 |
| hemna_ | LOG.warning(_LW("using 5Gb")) | 22:40 |
| hemna_ | I don't see a need for an API change | 22:41 |
| hemna_ | and at the bottom of create_volume, you do a model update with the size | 22:41 |
| hemna_ | Cinder's db is happy and so is ScaleIO | 22:42 |
| karthikp | hemna_, can they do that from their version of create_volume right? | 22:43 |
| karthikp | they can* | 22:43 |
| jgriffith | karthikp: I'm with hemna_ we've had this debate in the past, return a size update and call it good. Have fun figuring out quota management and filtering issues though :) | 22:43 |
| guitarzan | have your driver just error any creates that aren't mod 5 :) | 22:45 |
| jgriffith | guitarzan: :) | 22:45 |
| jgriffith | there's a whole host of options :) | 22:45 |
| jgriffith | I don't understand how adding a kwargs helps you out here anyway | 22:45 |
| jgriffith | user isn't supposed to know anything about what's on the backend, it's limitations or otherwise | 22:45 |
| hemna_ | hehe | 22:46 |
| hemna_ | mod 5 :) | 22:46 |
| jgriffith | karthikp: if you really want to solve this, you put a filter in the scheduler that says "I can only service requests in increments of 5" | 22:46 |
| jgriffith | guitarzan: may have been joking, but that's closer to correct than anything else IMO | 22:46 |
| guitarzan | that's a pretty good idea too | 22:46 |
| guitarzan | I wasn't really joking | 22:47 |
| jgriffith | guitarzan: even better :) | 22:47 |
| guitarzan | a custom scheduler is another great idea though | 22:47 |
| jgriffith | guitarzan: couldn't tell via irc | 22:47 |
| guitarzan | yeah, I put a smiley :) | 22:47 |
| jgriffith | I vote for custom scheduler.. that's what it's there for | 22:47 |
| guitarzan | ya, that's good | 22:47 |
| guitarzan | still doesn't help from an api response perspective, but what can ya do? | 22:48 |
| jgriffith | guitarzan: use a different backend :) | 22:48 |
| guitarzan | haha | 22:48 |
| karthikp | a model update to 5gb may be a little weird because user requested 1 gb and got 5gb Vol in return | 22:48 |
| jgriffith | karthikp: right | 22:48 |
| jgriffith | karthikp: so do a filter, and just error when it's not an increment of 5 | 22:49 |
| jgriffith | or it goes to another backend | 22:49 |
| jgriffith | karthikp: or give your customers a custom patch and explain to them what the limitation is and what they're in for | 22:49 |
| *** dalgaaf has quit IRC | 22:49 | |
| jgriffith | karthikp: I still didn't catch what you think kwargs would solve, or what you would have the user pass in to solve this? | 22:50 |
| karthikp | jgriffith, I am part of OSIC ..I saw this as bug on launchpad from possibly somebody from ScaleIO | 22:50 |
| jgriffith | karthikp: sorry.. not sure what OSIC is? | 22:51 |
| jgriffith | karthikp: I was just asking about your proposal? | 22:51 |
| jgriffith | karthikp: do you have link to the bug? | 22:51 |
| *** cknight has joined #openstack-cinder | 22:52 | |
| karthikp | jgriffith, my idea was to provide a way up to the driver -> to the volume manager and log it as warning | 22:52 |
| jgriffith | karthikp: provide "what" up to the driver? | 22:52 |
| jgriffith | karthikp: sorry, not following what you'd put in the kwargs, or *who* would put it there (whatever *it* is) | 22:53 |
| karthikp | jgriffith, sorry wasnt thinking right ..could have done that either way from even drivre code as hemna pointed out | 22:53 |
| *** Julien-zte has quit IRC | 22:54 | |
| karthikp | jgriffith, the volume size that actually got created(5gb) | 22:54 |
| *** mriedem has joined #openstack-cinder | 22:54 | |
| karthikp | https://bugs.launchpad.net/cinder/+bug/1561560 | 22:55 |
| openstack | Launchpad bug 1561560 in Cinder "Report back actual volume / snapshot size. " [Wishlist,Confirmed] - Assigned to Karthik Prabhu Vinod (karthik-prabhu-vinod) | 22:55 |
| hemna_ | karthikp, you can do that today. return a model update. | 22:55 |
| hemna_ | in your vendor specific driver. | 22:55 |
| karthikp | hemna_, i am a part of OSIC and was checking what I really wanted to do here for the bug | 23:02 |
| hemna_ | I don't know what OSIC is or that it has anything to do with what you are trying to accomplish. The bug is invalid IMHO | 23:03 |
| *** andymaier has quit IRC | 23:07 | |
| tbarron | https://osic.org/ | 23:07 |
| *** sdague has quit IRC | 23:07 | |
| tbarron | karthikp: glad you are looking at cinder issues! | 23:07 |
| hemna_ | I must be living under a rock. never heard of it. | 23:08 |
| tbarron | karthikp: I think a lot of us (myself included) want to make sure not to compromise the core code in cinder b/c of particular driver implementations. So that's a lot of what is going on with this particular issue that you are tackling at the moment. | 23:09 |
| tbarron | karthikp: but pls. do push us, ask questions, look at stuff in new ways, ...! | 23:10 |
| karthikp | tbarron, I completely understand, i was trying to see how this could be achieved | 23:13 |
| openstackgerrit | Merged openstack/cinder: SMBFS: Fix initialize connection issues caused by in-use images https://review.openstack.org/289298 | 23:13 |
| *** mylu has quit IRC | 23:14 | |
| *** david-lyle_ has joined #openstack-cinder | 23:14 | |
| *** david-lyle has quit IRC | 23:14 | |
| karthikp | tbarron, hemna_ jgriffith thanks a lot for your input... I am not sure if this a invalid | 23:14 |
| *** mylu has joined #openstack-cinder | 23:15 | |
| jgriffith | karthikp: NP, it's a tricky one; trying to figure out how to accommodate the one backend device | 23:15 |
| jgriffith | karthikp: I think the suggested options are all reasonable | 23:15 |
| jgriffith | karthikp: on the bright side, at least your not asking to make it "smaller" like the last person I talked to on this subject :) | 23:16 |
| tbarron | so is it reasonable to say: 1) driver returns "actual size" in model update, 2) if actual size exceeds allocated size, manager logs a warning ? | 23:17 |
| tbarron | or just INFO | 23:19 |
| *** arch-nemesis has quit IRC | 23:19 | |
| jgriffith | tbarron: I prefer error or custom filter honestly | 23:19 |
| jgriffith | tbarron: but at least updating is somewhat better than just ignoring as it does now | 23:20 |
| jgriffith | tbarron: and warning vs info... I dunno | 23:20 |
| jgriffith | tbarron: wait.... what? | 23:20 |
| tbarron | jgriffith: sure, I'm not opposed to those. I was just reading the "bug", which I think is really just a minor RFE | 23:20 |
| jgriffith | RFE? | 23:20 |
| tbarron | request for enhancement | 23:20 |
| jgriffith | :) | 23:21 |
| tbarron | jgriffith: gotta learn those NetApp acronyms man :-) | 23:21 |
| karthikp | tbarron, jgriffith Sorry I am new..so trying to figure it different ways. thanks | 23:22 |
| jgriffith | karthikp: no worries | 23:22 |
| tbarron | jgriffith: ++ | 23:22 |
| karthikp | will sleep on these solutions :) | 23:22 |
| *** sake is now known as sdake | 23:22 | |
| tbarron | k, you folks have a good night ... | 23:22 |
| *** karthikp has quit IRC | 23:27 | |
| *** jwcroppe_ has joined #openstack-cinder | 23:29 | |
| *** jwcroppe has quit IRC | 23:31 | |
| *** mylu has quit IRC | 23:32 | |
| *** mylu has joined #openstack-cinder | 23:33 | |
| *** mylu has quit IRC | 23:36 | |
| *** mylu has joined #openstack-cinder | 23:36 | |
| *** cburgess has quit IRC | 23:40 | |
| *** cknight1 has joined #openstack-cinder | 23:42 | |
| *** pleia2 has quit IRC | 23:43 | |
| *** pleia2 has joined #openstack-cinder | 23:43 | |
| *** cknight has quit IRC | 23:45 | |
| *** HenryG has quit IRC | 23:45 | |
| *** cburgess has joined #openstack-cinder | 23:45 | |
| *** dobson has quit IRC | 23:46 | |
| *** yhayashi has joined #openstack-cinder | 23:48 | |
| *** HenryG has joined #openstack-cinder | 23:48 | |
| *** dobson has joined #openstack-cinder | 23:49 | |
| *** jwcroppe has joined #openstack-cinder | 23:53 | |
| *** dustins has quit IRC | 23:54 | |
| *** jwcroppe_ has quit IRC | 23:55 | |
| *** karthikp has joined #openstack-cinder | 23:56 | |
| *** david-lyle_ is now known as david-lyle | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!