| *** Yogi1 has quit IRC | 00:09 | |
| *** diogogmt has quit IRC | 00:09 | |
| *** akerr_ has quit IRC | 00:10 | |
| *** angela-s has quit IRC | 00:10 | |
| *** Ctina_ has quit IRC | 00:11 | |
| *** dannywilson has quit IRC | 00:11 | |
| xyang1 | jgriffith: Maybe find something sounds better than meatball?:) | 00:12 |
|---|---|---|
| xyang1 | jgriffith: I see what you are saying. i just don't have a better solution to solve the ABC mess | 00:13 |
| *** lixiaoy1 has joined #openstack-cinder | 00:18 | |
| *** takedakn has quit IRC | 00:23 | |
| *** garthb_ has quit IRC | 00:25 | |
| *** tsekiyama has quit IRC | 00:26 | |
| *** mtanino has quit IRC | 00:32 | |
| *** dims has joined #openstack-cinder | 00:34 | |
| *** merooney has joined #openstack-cinder | 00:36 | |
| *** daneyon has joined #openstack-cinder | 00:39 | |
| *** mriedem has joined #openstack-cinder | 00:43 | |
| *** leeantho_ has quit IRC | 00:44 | |
| *** mriedem1 has quit IRC | 00:45 | |
| *** merooney has quit IRC | 00:45 | |
| *** daneyon_ has joined #openstack-cinder | 00:46 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Prevent uploading encrypted volume to images https://review.openstack.org/213616 | 00:46 |
| openstackgerrit | LisaLi proposed openstack/cinder: Prevent uploading encrypted volume to images https://review.openstack.org/213616 | 00:48 |
| *** mriedem has quit IRC | 00:48 | |
| *** daneyon has quit IRC | 00:49 | |
| *** mriedem has joined #openstack-cinder | 00:50 | |
| *** zhonghua-lee has joined #openstack-cinder | 00:52 | |
| mriedem | jungleboyj: in https://review.openstack.org/#/c/174783/ - where does the test actually mock out the port_status? | 00:58 |
| mriedem | i think i see it now | 01:01 |
| mriedem | def _cmd_lsnode(self, **kwargs): overrides the ssh call, and node2 has an active and inactive port | 01:01 |
| *** dflorea has quit IRC | 01:02 | |
| mriedem | seems unfortunate that the storwize fc ci failed and the change is only for storwize fc :( | 01:03 |
| openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Update deprecated version of novaclient https://review.openstack.org/210737 | 01:03 |
| *** vilobhmm has quit IRC | 01:05 | |
| *** skylerberg has quit IRC | 01:05 | |
| *** xyang1 has quit IRC | 01:08 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Allow CG without snapshot to be deleted https://review.openstack.org/213892 | 01:13 |
| *** mriedem has quit IRC | 01:13 | |
| *** mtanino has joined #openstack-cinder | 01:14 | |
| *** tiny-hands has quit IRC | 01:20 | |
| *** tiny-hands has joined #openstack-cinder | 01:21 | |
| *** zhenguo has joined #openstack-cinder | 01:22 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 01:25 |
| *** andreykurilin__ has quit IRC | 01:25 | |
| *** salv-orlando has joined #openstack-cinder | 01:29 | |
| *** esker has joined #openstack-cinder | 01:30 | |
| *** rick__ has joined #openstack-cinder | 01:33 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 01:33 |
| *** salv-orlando has quit IRC | 01:35 | |
| *** esker has quit IRC | 01:36 | |
| *** bill_az has quit IRC | 01:37 | |
| *** _cjones_ has quit IRC | 01:37 | |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds manage/unmanage methods for HNAS drivers. https://review.openstack.org/197999 | 01:37 |
| *** rick__ has quit IRC | 01:37 | |
| *** rick-chen has joined #openstack-cinder | 01:38 | |
| openstackgerrit | xing-yang proposed openstack/cinder: EMC VMAX Create CG from CG Snapshot https://review.openstack.org/195280 | 01:38 |
| *** lxg has joined #openstack-cinder | 01:38 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add the ability to update type public status https://review.openstack.org/206788 | 01:39 |
| *** takedakn has joined #openstack-cinder | 01:42 | |
| *** takedakn has quit IRC | 01:42 | |
| *** wilson has quit IRC | 01:42 | |
| *** vincent_hou has joined #openstack-cinder | 01:45 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: Fix test failure under python 2.7.9 https://review.openstack.org/213962 | 01:58 |
| openstackgerrit | XinXiaohui proposed openstack/cinder: Calculate virtual free capacity and notify https://review.openstack.org/206923 | 02:01 |
| *** sakaYK has quit IRC | 02:02 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 02:05 |
| *** haomaiwang has joined #openstack-cinder | 02:06 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 02:09 |
| *** mtanino has quit IRC | 02:11 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 02:13 |
| *** p0rtal_ has quit IRC | 02:14 | |
| *** davechen has joined #openstack-cinder | 02:14 | |
| *** vervet has joined #openstack-cinder | 02:20 | |
| *** vervet has quit IRC | 02:21 | |
| *** vincent_hou has quit IRC | 02:22 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 02:23 |
| *** zhonghua-lee has quit IRC | 02:27 | |
| *** smoriya has joined #openstack-cinder | 02:29 | |
| *** dims has quit IRC | 02:30 | |
| *** tiny-hands has quit IRC | 02:30 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Use cinder internal tenant to create the destination volume https://review.openstack.org/210237 | 02:31 |
| *** baojg has joined #openstack-cinder | 02:31 | |
| *** bkopilov has quit IRC | 02:32 | |
| *** vervet has joined #openstack-cinder | 02:39 | |
| *** bswartz has joined #openstack-cinder | 02:42 | |
| *** logan2 has quit IRC | 02:44 | |
| *** akerr has joined #openstack-cinder | 02:50 | |
| *** esker has joined #openstack-cinder | 02:51 | |
| *** akerr_ has joined #openstack-cinder | 02:52 | |
| *** hakimo has joined #openstack-cinder | 02:52 | |
| *** hakimo_ has quit IRC | 02:54 | |
| *** akerr has quit IRC | 02:55 | |
| *** haomaiwang has quit IRC | 03:01 | |
| *** baojg has quit IRC | 03:02 | |
| *** haomaiwa_ has joined #openstack-cinder | 03:02 | |
| *** apoorvad has quit IRC | 03:02 | |
| *** esker has quit IRC | 03:02 | |
| *** esker has joined #openstack-cinder | 03:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/213891 | 03:05 |
| *** diogogmt has joined #openstack-cinder | 03:13 | |
| *** simondodsley has joined #openstack-cinder | 03:15 | |
| *** baojg has joined #openstack-cinder | 03:15 | |
| *** HoloIRCUser has joined #openstack-cinder | 03:18 | |
| *** salv-orlando has joined #openstack-cinder | 03:19 | |
| *** simondodsley has quit IRC | 03:19 | |
| *** HoloIRCUser has quit IRC | 03:23 | |
| *** HoloIRCUser has joined #openstack-cinder | 03:24 | |
| *** diogogmt has quit IRC | 03:26 | |
| *** esker has quit IRC | 03:26 | |
| *** salv-orlando has quit IRC | 03:26 | |
| *** esker has joined #openstack-cinder | 03:26 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 03:28 |
| *** skylerberg has joined #openstack-cinder | 03:30 | |
| *** changbl has quit IRC | 03:30 | |
| openstackgerrit | Dave Chen proposed openstack/python-cinderclient: Clean the unused methods in `volumes.py` https://review.openstack.org/210287 | 03:32 |
| *** garthb has joined #openstack-cinder | 03:32 | |
| *** logan2 has joined #openstack-cinder | 03:34 | |
| *** akerr_ has quit IRC | 03:34 | |
| *** esker has quit IRC | 03:37 | |
| *** esker has joined #openstack-cinder | 03:38 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 03:39 |
| *** sakaYK has joined #openstack-cinder | 03:41 | |
| *** markvoelker has quit IRC | 03:42 | |
| *** sakaYK has quit IRC | 03:42 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Add volume retype support for Huawei driver https://review.openstack.org/201485 | 03:43 |
| *** baojg has quit IRC | 03:49 | |
| *** jgregor has joined #openstack-cinder | 03:50 | |
| *** jgregor has quit IRC | 03:54 | |
| *** bkopilov has joined #openstack-cinder | 03:58 | |
| *** esker has quit IRC | 03:58 | |
| *** esker has joined #openstack-cinder | 04:00 | |
| *** haomaiwa_ has quit IRC | 04:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 04:02 | |
| *** zhenguo has quit IRC | 04:06 | |
| *** mylu has joined #openstack-cinder | 04:09 | |
| *** coolsvap|away is now known as coolsvap | 04:10 | |
| *** esker has quit IRC | 04:13 | |
| *** esker has joined #openstack-cinder | 04:13 | |
| *** esker has quit IRC | 04:14 | |
| *** esker has joined #openstack-cinder | 04:14 | |
| *** baojg has joined #openstack-cinder | 04:16 | |
| *** bill_az has joined #openstack-cinder | 04:18 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 04:19 |
| *** esker has quit IRC | 04:20 | |
| *** mylu has quit IRC | 04:21 | |
| *** lixiaoy1 has quit IRC | 04:22 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 04:32 |
| *** HoloIRCUser has quit IRC | 04:35 | |
| *** deepakcs has joined #openstack-cinder | 04:35 | |
| *** shyama has joined #openstack-cinder | 04:38 | |
| *** markvoelker has joined #openstack-cinder | 04:42 | |
| *** mylu has joined #openstack-cinder | 04:42 | |
| *** vilobhmm has joined #openstack-cinder | 04:43 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 04:44 |
| *** apoorvad has joined #openstack-cinder | 04:46 | |
| *** markvoelker has quit IRC | 04:47 | |
| *** apoorvad has quit IRC | 04:54 | |
| *** Apoorva has joined #openstack-cinder | 04:55 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:58 | |
| *** haomaiwa_ has quit IRC | 05:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 05:02 | |
| *** lixiaoy11 has joined #openstack-cinder | 05:04 | |
| *** nkrinner has joined #openstack-cinder | 05:06 | |
| *** lixiaoy1 has quit IRC | 05:07 | |
| openstackgerrit | yuji hagiwara proposed openstack/cinder: Remove duplicate keys from dictionary https://review.openstack.org/213999 | 05:12 |
| *** dims has joined #openstack-cinder | 05:20 | |
| *** smoriya has quit IRC | 05:21 | |
| *** garthb has quit IRC | 05:22 | |
| *** dims has quit IRC | 05:26 | |
| *** Apoorva has quit IRC | 05:28 | |
| *** p0rtal has joined #openstack-cinder | 05:30 | |
| *** p0rtal has joined #openstack-cinder | 05:31 | |
| *** salv-orlando has joined #openstack-cinder | 05:37 | |
| *** p0rtal has quit IRC | 05:45 | |
| *** p0rtal has joined #openstack-cinder | 05:48 | |
| *** salv-orlando has quit IRC | 05:50 | |
| *** ankit_ag has joined #openstack-cinder | 05:52 | |
| *** haomaiwa_ has quit IRC | 06:01 | |
| *** haomaiwang has joined #openstack-cinder | 06:02 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Return multiple iSCSI portals in VNX Cinder driver https://review.openstack.org/213583 | 06:08 |
| *** skylerberg has quit IRC | 06:14 | |
| *** BharatK has quit IRC | 06:14 | |
| winston-d_zZZ | DuncanT: around? | 06:17 |
| *** winston-d_zZZ is now known as winston-d | 06:17 | |
| *** salv-orlando has joined #openstack-cinder | 06:21 | |
| *** coolsvap is now known as coolsvap|away | 06:22 | |
| *** vlaza has joined #openstack-cinder | 06:24 | |
| *** zhenguo has joined #openstack-cinder | 06:24 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 06:30 |
| *** zhenguo has quit IRC | 06:32 | |
| *** p0rtal has quit IRC | 06:32 | |
| *** mylu has quit IRC | 06:35 | |
| *** mylu has joined #openstack-cinder | 06:36 | |
| *** anshul has joined #openstack-cinder | 06:37 | |
| *** vilobhmm has quit IRC | 06:39 | |
| *** mylu has quit IRC | 06:40 | |
| *** zhonghua-lee has joined #openstack-cinder | 06:40 | |
| *** openstackgerrit_ has joined #openstack-cinder | 06:43 | |
| *** markvoelker has joined #openstack-cinder | 06:43 | |
| *** bill_az has quit IRC | 06:44 | |
| *** markvoelker has quit IRC | 06:48 | |
| *** BharatK has joined #openstack-cinder | 06:50 | |
| *** yrabl has joined #openstack-cinder | 06:52 | |
| *** mylu has joined #openstack-cinder | 06:57 | |
| *** mylu has quit IRC | 06:58 | |
| *** BharatK has quit IRC | 06:58 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:02 | |
| *** rushiagr_away is now known as rushiagr | 07:03 | |
| *** geguileo has joined #openstack-cinder | 07:08 | |
| *** lihit has joined #openstack-cinder | 07:11 | |
| *** lihit has quit IRC | 07:11 | |
| *** BharatK has joined #openstack-cinder | 07:12 | |
| *** markus_z has joined #openstack-cinder | 07:14 | |
| *** alexschm has joined #openstack-cinder | 07:15 | |
| openstackgerrit | Tina Tang proposed openstack/cinder: Fix the virtual port support in VNX driver https://review.openstack.org/214044 | 07:20 |
| *** zhongjun has joined #openstack-cinder | 07:23 | |
| *** vervet has quit IRC | 07:39 | |
| *** vervet has joined #openstack-cinder | 07:39 | |
| *** pschaef has joined #openstack-cinder | 07:40 | |
| *** pschaef has quit IRC | 07:41 | |
| *** IanGovett has joined #openstack-cinder | 07:42 | |
| *** gardenshed has joined #openstack-cinder | 07:42 | |
| *** ronis has joined #openstack-cinder | 07:44 | |
| *** ronis_ has joined #openstack-cinder | 07:46 | |
| *** dims has joined #openstack-cinder | 07:48 | |
| *** sgotliv has joined #openstack-cinder | 07:48 | |
| *** ronis has quit IRC | 07:50 | |
| *** dims has quit IRC | 07:53 | |
| *** agarciam has joined #openstack-cinder | 07:53 | |
| lixiaoy11 | :wanghao here? | 07:55 |
| *** jistr has joined #openstack-cinder | 07:55 | |
| openstackgerrit | yuji hagiwara proposed openstack/cinder: Remove duplicate keys from dictionary https://review.openstack.org/213999 | 07:58 |
| wanghao | lixiaoy11: yes | 07:59 |
| *** haomaiwang has quit IRC | 08:01 | |
| *** haomaiwang has joined #openstack-cinder | 08:02 | |
| lixiaoy11 | :wanghao I am just looking at your patch : https://review.openstack.org/#/c/207696/9 | 08:05 |
| *** jordanP has joined #openstack-cinder | 08:05 | |
| lixiaoy11 | :wanghao one question: image_size = utils.as_int(image_meta['size'], quiet=False) | 08:06 |
| lixiaoy11 | image_size_in_gb = (image_size + GB - 1) / GB | 08:06 |
| lixiaoy11 | You set image size as 1 in unit test, image_size_in_gb = 1. So it is not zero. | 08:06 |
| lixiaoy11 | wanghao: based on your update, it shuold not raise invalidinput exception here | 08:07 |
| wanghao | lixiaoy11: I remembered it was set zero in fake_image_service.... Well I will check the code later. | 08:15 |
| lixiaoy11 | wanghao: ok | 08:16 |
| winston-d | lixiaoy11: Lisa? | 08:17 |
| lixiaoy11 | winston-d: yes | 08:17 |
| winston-d | lixiaoy11: https://review.openstack.org/#/c/213616/, shouldn't we fix Cinder to save encryption metadata to Glance, rather than stopping uploading encrypted volume to image? | 08:18 |
| lixiaoy11 | winston-d: as the liberty is going to freeze. three things need to do: 1. save the encryption metadata into Glance, 2. Create volumes with image, 3. Work with Nova | 08:20 |
| *** zhongjun has quit IRC | 08:20 | |
| lixiaoy11 | winston-d: i am not sure it is worthy to save encryption image in Glance. How do you think | 08:21 |
| *** andreykurilin__ has joined #openstack-cinder | 08:22 | |
| winston-d | lixiaoy11: well, I think it depends on how the volume is encypted. | 08:22 |
| lixiaoy11 | winston-d: what do you mean? | 08:23 |
| winston-d | if the volume is encrypted by the storage backend, which means when the data leaves backend, it should be decrypted first. | 08:24 |
| lixiaoy11 | winston-d: I am afraid that we don't care about this scenario. As both from cinder /nova, it encrypts/unencrypts in backend storage, and they read same data | 08:25 |
| *** gardensh_ has joined #openstack-cinder | 08:26 | |
| *** gardenshed has quit IRC | 08:26 | |
| winston-d | why not? you change will stop user from uploading such volume to glance. | 08:26 |
| *** bluex-pl has joined #openstack-cinder | 08:27 | |
| *** bluex-pl has quit IRC | 08:28 | |
| *** bluex-pl has joined #openstack-cinder | 08:28 | |
| lixiaoy11 | winston-d: no. As currently this just cares about the encryption by dm-crypt. that is a volume_type is encrypted. | 08:28 |
| winston-d | volume_type.is_encrypted != volume is encrypted by dm-crypt | 08:30 |
| lixiaoy11 | winston-d: I mean this encryption uses the third software to do encryption, Cinder needs to record the encryption algorithm, key, etc in db. | 08:32 |
| lixiaoy11 | winston-d: the encryption in backend storage, which is transparent to cinder node/compute node | 08:33 |
| lixiaoy11 | winston-d: if encryption is done by backend storage, no metadata is recorded in cinder, is_encrypted returns false | 08:36 |
| winston-d | lixiaoy11: if 'encryption is transparent to cinder/nova', then i agree, but i am not sure that is true for all backend level encryption. | 08:37 |
| lixiaoy11 | winston-d: let me have a check | 08:39 |
| *** shausy has joined #openstack-cinder | 08:42 | |
| *** gardensh_ has quit IRC | 08:43 | |
| *** markvoelker has joined #openstack-cinder | 08:45 | |
| *** zhongjun has joined #openstack-cinder | 08:45 | |
| *** chenying has joined #openstack-cinder | 08:47 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 08:49 |
| lixiaoy11 | winston-d: from cinder code, no encryption info is trasferred to backend storage. if encryption is done by backend, it should know such things. Any way, wait for others confirmed | 08:49 |
| *** d0ugal has joined #openstack-cinder | 08:50 | |
| *** markvoelker has quit IRC | 08:50 | |
| winston-d | lixiaoy11: sure, thx for confirming. | 08:52 |
| lixiaoy11 | winston-d: my concern is that using different encryption algorithm, so may be create a encrypted image with encryption algorithm 1, and another same image with encryption algorithm 2. is it useful for customers | 08:53 |
| *** Lee1092 has joined #openstack-cinder | 08:55 | |
| *** chenying has quit IRC | 08:58 | |
| lixiaoy11 | winson-d: how about I set work in progress -1 for above patch, and I am trying to save encryption metadata to Glance, if it can be completed in libirty, it is ok, Or else use this patch to prevent | 08:58 |
| *** haomaiwang has quit IRC | 09:01 | |
| *** shikel has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:02 | |
| *** zhongjun has quit IRC | 09:05 | |
| *** zhongjun has joined #openstack-cinder | 09:06 | |
| *** sh1kel has joined #openstack-cinder | 09:06 | |
| *** lixiaoy11 has quit IRC | 09:08 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 09:12 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup list all_tenants=0 issue https://review.openstack.org/214086 | 09:32 |
| *** gardenshed has joined #openstack-cinder | 09:33 | |
| *** rushiagr is now known as rushiagr_away | 09:37 | |
| *** afazkas has joined #openstack-cinder | 09:42 | |
| *** zhongjun has quit IRC | 09:42 | |
| *** davechen has left #openstack-cinder | 09:54 | |
| *** gardenshed has quit IRC | 09:54 | |
| *** tiny-hands has joined #openstack-cinder | 09:55 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Make migration's volume source deletion async https://review.openstack.org/211257 | 10:00 |
| *** tiny-hands has quit IRC | 10:00 | |
| *** haomaiwang has quit IRC | 10:01 | |
| *** haomaiwang has joined #openstack-cinder | 10:02 | |
| *** baojg has quit IRC | 10:06 | |
| *** baojg has joined #openstack-cinder | 10:06 | |
| *** baojg has quit IRC | 10:07 | |
| *** baojg has joined #openstack-cinder | 10:07 | |
| *** baojg has quit IRC | 10:07 | |
| *** baojg has joined #openstack-cinder | 10:07 | |
| *** rick-chen has quit IRC | 10:08 | |
| *** baojg has quit IRC | 10:17 | |
| *** baojg has joined #openstack-cinder | 10:18 | |
| *** baojg has quit IRC | 10:23 | |
| *** sayali has quit IRC | 10:24 | |
| *** sayali has joined #openstack-cinder | 10:25 | |
| *** gardenshed has joined #openstack-cinder | 10:36 | |
| * DuncanT returns from vacation and starts climbing to email mountain | 10:38 | |
| *** gardenshed has quit IRC | 10:44 | |
| *** gardenshed has joined #openstack-cinder | 10:45 | |
| *** markvoelker has joined #openstack-cinder | 10:46 | |
| *** zhonghua-lee has quit IRC | 10:46 | |
| *** gardenshed has quit IRC | 10:47 | |
| *** tiny-hands has joined #openstack-cinder | 10:47 | |
| *** gardenshed has joined #openstack-cinder | 10:47 | |
| *** zhonghua-lee has joined #openstack-cinder | 10:48 | |
| *** sh1kel has quit IRC | 10:48 | |
| *** markvoelker has quit IRC | 10:50 | |
| *** dims has joined #openstack-cinder | 10:51 | |
| *** tiny-hands has quit IRC | 10:52 | |
| BharatK | DuncanT, Please review the patch https://review.openstack.org/#/c/182254/ when you have some time. Thanks. | 10:53 |
| *** haomaiwang has quit IRC | 10:54 | |
| *** salv-orlando has quit IRC | 10:57 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup list all_tenants=0 issue https://review.openstack.org/214086 | 10:58 |
| DuncanT | BharatK: An impressively small patch, I'll review it now | 10:58 |
| BharatK | DuncanT, Thanks :D | 10:59 |
| DuncanT | BharatK: Code looks good, and looking at a couple of the CI failures they don't seem related to the patch | 11:01 |
| BharatK | DuncanT, Yes | 11:02 |
| BharatK | DuncanT, They are failing for all Cinder patches. | 11:03 |
| DuncanT | BharatK: Thanks. I'm approving now | 11:03 |
| BharatK | DuncanT, Thank you very much. | 11:03 |
| DuncanT | BharatK: Thanks for an easy to review patch | 11:04 |
| BharatK | DuncanT, np. :-) | 11:05 |
| *** shausy has quit IRC | 11:09 | |
| *** shausy has joined #openstack-cinder | 11:09 | |
| *** delattec has quit IRC | 11:10 | |
| *** cdelatte has quit IRC | 11:10 | |
| *** gardenshed has quit IRC | 11:14 | |
| *** salv-orlando has joined #openstack-cinder | 11:19 | |
| *** esker has joined #openstack-cinder | 11:20 | |
| *** marcusvrn has joined #openstack-cinder | 11:27 | |
| *** dims_ has joined #openstack-cinder | 11:28 | |
| *** dims has quit IRC | 11:30 | |
| *** zhongjun has joined #openstack-cinder | 11:31 | |
| *** cdelatte has joined #openstack-cinder | 11:32 | |
| *** lpetrut has joined #openstack-cinder | 11:36 | |
| *** DuncanT has quit IRC | 11:38 | |
| *** marcusvrn has quit IRC | 11:38 | |
| *** markvoelker has joined #openstack-cinder | 11:46 | |
| *** esker has quit IRC | 11:50 | |
| *** esker has joined #openstack-cinder | 11:51 | |
| *** markvoelker has quit IRC | 11:51 | |
| *** bkopilov has quit IRC | 11:58 | |
| *** dims has joined #openstack-cinder | 12:00 | |
| *** dims has quit IRC | 12:00 | |
| *** dims_ has quit IRC | 12:00 | |
| *** dims has joined #openstack-cinder | 12:01 | |
| *** esker has quit IRC | 12:01 | |
| *** dims has quit IRC | 12:05 | |
| *** andreykurilin__ has quit IRC | 12:05 | |
| *** markvoelker has joined #openstack-cinder | 12:12 | |
| *** Lee1092 has quit IRC | 12:15 | |
| *** dims has joined #openstack-cinder | 12:15 | |
| *** Lee1092 has joined #openstack-cinder | 12:16 | |
| *** salv-orlando has quit IRC | 12:18 | |
| *** raildo-afk is now known as raildo | 12:18 | |
| *** bswartz has quit IRC | 12:20 | |
| *** annegentle has joined #openstack-cinder | 12:24 | |
| *** zhenguo has joined #openstack-cinder | 12:29 | |
| *** zhongjun has quit IRC | 12:30 | |
| *** nkrinner has quit IRC | 12:32 | |
| *** Adriano__ has quit IRC | 12:32 | |
| *** Adriano__ has joined #openstack-cinder | 12:34 | |
| *** sgotliv_ has joined #openstack-cinder | 12:38 | |
| *** marcusvrn has joined #openstack-cinder | 12:38 | |
| *** diogogmt has joined #openstack-cinder | 12:39 | |
| *** agarciam1 has joined #openstack-cinder | 12:39 | |
| *** agarciam has quit IRC | 12:40 | |
| *** sgotliv has quit IRC | 12:40 | |
| *** dims_ has joined #openstack-cinder | 12:41 | |
| *** dims has quit IRC | 12:41 | |
| *** zhonghua-lee has quit IRC | 12:41 | |
| *** agarciam has joined #openstack-cinder | 12:43 | |
| *** porrua has joined #openstack-cinder | 12:44 | |
| *** agarciam1 has quit IRC | 12:44 | |
| *** lpetrut has quit IRC | 12:45 | |
| *** zhongjun has joined #openstack-cinder | 12:45 | |
| *** akerr has joined #openstack-cinder | 12:46 | |
| *** edmondsw has joined #openstack-cinder | 12:47 | |
| *** marcusvrn has quit IRC | 12:49 | |
| *** chlong has joined #openstack-cinder | 12:52 | |
| *** Adriano__ is now known as adrianofr | 12:55 | |
| *** diogogmt has quit IRC | 12:56 | |
| *** Yogi1 has joined #openstack-cinder | 13:00 | |
| *** lpetrut has joined #openstack-cinder | 13:00 | |
| *** dims has joined #openstack-cinder | 13:01 | |
| *** dims_ has quit IRC | 13:02 | |
| *** bswartz has joined #openstack-cinder | 13:02 | |
| *** haomaiwang has joined #openstack-cinder | 13:03 | |
| *** vervet has quit IRC | 13:03 | |
| *** Yogi1 has quit IRC | 13:04 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Dell SC: Fix error causing missed log message https://review.openstack.org/214178 | 13:11 |
| *** dims_ has joined #openstack-cinder | 13:15 | |
| *** dims has quit IRC | 13:17 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for file I/O volume migration https://review.openstack.org/187270 | 13:19 |
| *** changbl has joined #openstack-cinder | 13:19 | |
| *** dustins has joined #openstack-cinder | 13:20 | |
| *** bill_az has joined #openstack-cinder | 13:21 | |
| *** xyang1 has joined #openstack-cinder | 13:21 | |
| *** jgregor has joined #openstack-cinder | 13:22 | |
| *** mriedem has joined #openstack-cinder | 13:23 | |
| *** cburgess has quit IRC | 13:23 | |
| *** julim has joined #openstack-cinder | 13:24 | |
| *** dustins has quit IRC | 13:26 | |
| *** cburgess has joined #openstack-cinder | 13:26 | |
| *** dustins has joined #openstack-cinder | 13:26 | |
| *** simondodsley has joined #openstack-cinder | 13:27 | |
| *** Yogi11 has joined #openstack-cinder | 13:29 | |
| *** diogogmt has joined #openstack-cinder | 13:30 | |
| *** dims_ has quit IRC | 13:31 | |
| *** dims has joined #openstack-cinder | 13:31 | |
| *** ankit_ag has quit IRC | 13:38 | |
| *** JoseMello has joined #openstack-cinder | 13:40 | |
| *** dims has quit IRC | 13:40 | |
| *** eharney has quit IRC | 13:41 | |
| *** changbl has quit IRC | 13:41 | |
| *** jungleboyj has quit IRC | 13:42 | |
| *** dims has joined #openstack-cinder | 13:43 | |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: force_detach terminate_connection needs connector https://review.openstack.org/213867 | 13:47 |
| *** deepakcs has quit IRC | 13:48 | |
| *** zhongjun has quit IRC | 13:48 | |
| *** bkopilov has joined #openstack-cinder | 13:52 | |
| *** marcusvrn_ has joined #openstack-cinder | 13:53 | |
| *** zhongjun has joined #openstack-cinder | 13:53 | |
| *** thangp has joined #openstack-cinder | 13:56 | |
| *** bluex-pl has quit IRC | 13:57 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 13:59 |
| *** breitz has quit IRC | 13:59 | |
| *** breitz has joined #openstack-cinder | 14:00 | |
| *** tiny-hands has joined #openstack-cinder | 14:00 | |
| *** haomaiwang has quit IRC | 14:01 | |
| *** haomaiwang has joined #openstack-cinder | 14:02 | |
| *** DuncanT has joined #openstack-cinder | 14:03 | |
| *** dims has quit IRC | 14:04 | |
| *** dims has joined #openstack-cinder | 14:05 | |
| *** jecarey has joined #openstack-cinder | 14:11 | |
| *** esker has joined #openstack-cinder | 14:12 | |
| *** martyturner has joined #openstack-cinder | 14:13 | |
| *** IlyaG has joined #openstack-cinder | 14:14 | |
| *** eharney has joined #openstack-cinder | 14:14 | |
| *** diogogmt has quit IRC | 14:14 | |
| *** jungleboyj has joined #openstack-cinder | 14:17 | |
| *** annegentle has quit IRC | 14:20 | |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 14:20 |
| *** lcurtis has joined #openstack-cinder | 14:22 | |
| *** afazkas has quit IRC | 14:27 | |
| *** zul has joined #openstack-cinder | 14:29 | |
| *** salv-orlando has joined #openstack-cinder | 14:30 | |
| *** vlaza has left #openstack-cinder | 14:31 | |
| *** salv-orlando has quit IRC | 14:34 | |
| *** theanalyst has quit IRC | 14:35 | |
| *** dims_ has joined #openstack-cinder | 14:35 | |
| *** jseiler__ has joined #openstack-cinder | 14:36 | |
| *** dims has quit IRC | 14:37 | |
| *** jseiler_ has quit IRC | 14:39 | |
| *** porrua has quit IRC | 14:44 | |
| *** annegentle has joined #openstack-cinder | 14:44 | |
| *** julim_ has joined #openstack-cinder | 14:51 | |
| *** aorourke_ has joined #openstack-cinder | 14:51 | |
| *** hodos has joined #openstack-cinder | 14:52 | |
| hodos | hi guys, can anyone tell if iSCSI cinder driver/brick connector expect it to be just 1 LUN per target? | 14:52 |
| hodos | or it's OK to have many volumes on the same target, 1 LUN per volume? | 14:54 |
| *** erlon has joined #openstack-cinder | 14:54 | |
| *** zul has quit IRC | 14:54 | |
| *** mtanino has joined #openstack-cinder | 14:55 | |
| *** bswartz has quit IRC | 14:57 | |
| smcginnis | hodos: That's the way our storage works. We have multiple LUNs exposed via the same target. | 14:58 |
| *** alexschm has quit IRC | 14:58 | |
| *** martyturner has quit IRC | 14:59 | |
| hodos | smcginnis: what's the driver class? | 14:59 |
| *** lcurtis has quit IRC | 14:59 | |
| *** julim has quit IRC | 14:59 | |
| *** tobasco_ has quit IRC | 14:59 | |
| *** breitz has quit IRC | 14:59 | |
| *** skraynev has quit IRC | 14:59 | |
| *** aorourke has quit IRC | 14:59 | |
| smcginnis | hodos: The DellStorageCenterxxxDrivers. | 14:59 |
| *** tobasco has joined #openstack-cinder | 14:59 | |
| hodos | smcginnis: Thanks. We've tried the same approach, but hit the issue: sometimes /dev/disk/by-path does not have all the available nodes | 15:00 |
| *** haomaiwang has quit IRC | 15:01 | |
| smcginnis | hodos: Hm, don't think I've seen that. | 15:01 |
| hodos | smcginnis: I've tried to walk through what brick does manually, with no luck | 15:01 |
| *** dims has joined #openstack-cinder | 15:01 | |
| smcginnis | hodos: Maybe once Swanson is on he might have some input. He's done much more hands-on testing with our driver. | 15:01 |
| *** haomaiwang has joined #openstack-cinder | 15:02 | |
| hodos | smcginnis: thanks, ok. I'll try to figure this out while | 15:02 |
| *** dims_ has quit IRC | 15:03 | |
| *** julim has joined #openstack-cinder | 15:04 | |
| *** ronis_ has quit IRC | 15:04 | |
| *** martyturner has joined #openstack-cinder | 15:05 | |
| *** breitz has joined #openstack-cinder | 15:05 | |
| *** julim_ has quit IRC | 15:06 | |
| *** lcurtis has joined #openstack-cinder | 15:07 | |
| jgriffith | hodos: as smcginnis pointed out you just modify the lun field in the model info | 15:07 |
| jgriffith | smcginnis: I had a question for you.... | 15:07 |
| jgriffith | smcginnis: but I can't remember what it was :( | 15:07 |
| jgriffith | smcginnis: it was VERY important, the future of Cinder was quite possible determined by it..... | 15:07 |
| jgriffith | :) | 15:08 |
| jordanP | jgriffith, could have a look at https://review.openstack.org/#/c/206935/ It's the backport of "Add deactivate step to extend_lv". I know you co-authored it but as you are not the main author, maybe you can still +A it :) | 15:08 |
| jordanP | (bend the rules... :p) | 15:09 |
| jgriffith | jordanP: looking | 15:09 |
| *** zul has joined #openstack-cinder | 15:09 | |
| jordanP | thanks | 15:10 |
| *** crose has joined #openstack-cinder | 15:10 | |
| jgriffith | jordanP: in the *good old days* I used to submit to master, approve, back port and approve all in the same morning :) | 15:11 |
| jgriffith | jordanP: all my own patches :) | 15:12 |
| *** skraynev has joined #openstack-cinder | 15:12 | |
| jordanP | jgriffith, haha ! I am sure you miss this time.. I would :) | 15:12 |
| jgriffith | jordanP: might explain all the typos :) | 15:12 |
| *** theanalyst has joined #openstack-cinder | 15:12 | |
| smcginnis | jgriffith: Hah! | 15:12 |
| smcginnis | jgriffith: Was you're question around config file vs extra specs? | 15:13 |
| jgriffith | hemnafk: FYI, I switched my CI to pull os-brick from HEAD and that patch fixes the image-copy issue I was seeing as well it appears | 15:13 |
| jgriffith | hemnafk: last 50 runs were all succesful | 15:13 |
| jgriffith | hemnafk: In other words "we need to release that os-brick version" :) | 15:14 |
| jordanP | jgriffith, you used LIBS_FROM_GIT="os-brick" ? | 15:14 |
| *** diogogmt has joined #openstack-cinder | 15:14 | |
| jgriffith | jordanP: correct | 15:15 |
| jordanP | ok | 15:15 |
| jgriffith | jordanP: for some reason every since we split out brick I've been seeing an intermittent disconnect issue on image-copy-to-volume | 15:15 |
| jgriffith | jordanP: The only thing that still bothers me is *why* I never hit it before moving to the lib | 15:16 |
| jgriffith | jordanP: since then hemnafk proposed a patch that removes the rescan on detach | 15:16 |
| jgriffith | jordanP: I'm still slightly nervous.... and TBF I do have to switch back to my other system and make sure the behavior is the same there | 15:17 |
| jgriffith | jordanP: but it's promising thus far | 15:17 |
| *** bswartz has joined #openstack-cinder | 15:17 | |
| *** hodos|2 has joined #openstack-cinder | 15:17 | |
| *** dannywilson has joined #openstack-cinder | 15:18 | |
| *** tsekiyama has joined #openstack-cinder | 15:19 | |
| *** hodos has quit IRC | 15:20 | |
| *** zhongjun has quit IRC | 15:21 | |
| *** ociuhandu has joined #openstack-cinder | 15:23 | |
| *** dannywilson has quit IRC | 15:23 | |
| *** dannywil_ has joined #openstack-cinder | 15:23 | |
| hodos|2 | jgriffith: that was exactly what i've been talking about | 15:24 |
| *** dannywil_ has quit IRC | 15:24 | |
| jgriffith | hodos|2: Oh? That's the same problem you were seeing? | 15:24 |
| jgriffith | hodos|2: sounds like you, me and erlon may have all been in the same boat | 15:24 |
| hodos|2 | jgriffith: i get device not connected on volume from image | 15:25 |
| hodos|2 | jgriffith: or device too small | 15:25 |
| *** dannywilson has joined #openstack-cinder | 15:25 | |
| jgriffith | hodos|2: yup, sounds like the same deal | 15:25 |
| hodos|2 | jgriffith: let me test with lib from git | 15:26 |
| jgriffith | device to small is completely different, don't know what to make of that | 15:26 |
| smcginnis | hodos|2: That one I have seen intermittently in our CI runs. | 15:26 |
| jgriffith | hodos|2: http://54.164.167.86/solidfire-ci-logs/refs-changes-70-187270-17/logs/local.conf.txt | 15:26 |
| *** garthb has joined #openstack-cinder | 15:26 | |
| *** garthb_ has joined #openstack-cinder | 15:26 | |
| jgriffith | hodos|2: just incase you need a reference | 15:26 |
| smcginnis | The best I could diagnose it was it was some kind of network communication issue or something. | 15:26 |
| smcginnis | Only happens on our iSCSI driver, not our FC one. | 15:27 |
| *** garthb has quit IRC | 15:27 | |
| jgriffith | smcginnis: that's interesting... | 15:27 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add support for --interface option in iscsiadm https://review.openstack.org/193451 | 15:27 |
| hodos|2 | smcginnis: smcginnis: thanks; i'll report on the result | 15:27 |
| smcginnis | jgriffith: Yeah, never could nail it down to an exact thing. | 15:27 |
| smcginnis | So it's just my gut feel. | 15:27 |
| smcginnis | Once we fixed up some network settings it seemed to go away, though not completely. | 15:27 |
| jgriffith | smcginnis: do you have the message? I'm trying to find it in the code | 15:28 |
| jgriffith | smcginnis: I had assumed it's the emu-convert but won't to verify | 15:28 |
| smcginnis | jgriffith: No, it's been a while. | 15:28 |
| jgriffith | smcginnis: hodos|2 in other words that one probably doesn't have anything to do with your device | 15:28 |
| smcginnis | jgriffith: Assumed it was some kind of network error that caused a SCSI command to fail, then causing it to seem like an out of space condition or something. | 15:28 |
| jgriffith | smcginnis: hodos|2 you may be running out of space on your Instance to do the conversion | 15:28 |
| smcginnis | jgriffith: Agree | 15:28 |
| *** dims has quit IRC | 15:29 | |
| smcginnis | jgriffith: That's what I initially thought, but I did verify I had plenty of space. | 15:29 |
| smcginnis | Added a lot of debug code to print out info and it had room. | 15:29 |
| jgriffith | smcginnis: hodos|2 I believe that's where I wrote that message... and the dd running out of space gives a clear stack-trace IIRC | 15:29 |
| jgriffith | hmmm | 15:29 |
| jgriffith | weird, well if anybody sees that again feel free to shoot me a log | 15:29 |
| *** dims has joined #openstack-cinder | 15:30 | |
| smcginnis | jgriffith: I'll watch for it. (i.e. Swanson will watch for it and tell me) | 15:30 |
| jgriffith | :) | 15:30 |
| Swanson | um, what? | 15:30 |
| smcginnis | Swanson: Just signing you up for things. :) | 15:30 |
| jgriffith | Swanson: smcginnis just signed you up to rewrite all of the Cinder drivers in golang | 15:30 |
| smcginnis | HAH! :) | 15:31 |
| smcginnis | jgriffith: You've joked about that enough now that I'm starting to think you're not entirely joking. ;) | 15:31 |
| *** p0rtal has joined #openstack-cinder | 15:31 | |
| *** gouthamr has joined #openstack-cinder | 15:31 | |
| jgriffith | smcginnis: haha! Nah... I just have it on the brain because the other project I'm working on is all golang | 15:31 |
| jgriffith | smcginnis: and the swift stuff in go just makes it funny to me | 15:32 |
| jgriffith | I swear I'm not suggesting it :) | 15:32 |
| smcginnis | Cool. Always fun to get a little more experience in another language. | 15:32 |
| jgriffith | smcginnis: indeed... and I have to admit that I really kinda like it as a lang | 15:33 |
| jgriffith | smcginnis: it's good for crusty old C folks like myself :) | 15:33 |
| *** salv-orlando has joined #openstack-cinder | 15:33 | |
| jgriffith | smcginnis: but it does make you appreciate python!!! | 15:33 |
| smcginnis | :) | 15:34 |
| *** DuncanT has quit IRC | 15:36 | |
| *** marcusvrn_ has quit IRC | 15:36 | |
| *** erlon has quit IRC | 15:36 | |
| *** shyama has quit IRC | 15:39 | |
| *** jecarey has quit IRC | 15:43 | |
| *** sayali has quit IRC | 15:45 | |
| *** dims has quit IRC | 15:45 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 15:45 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 15:45 |
| *** zul has quit IRC | 15:46 | |
| *** dims has joined #openstack-cinder | 15:47 | |
| *** lpetrut has quit IRC | 15:47 | |
| gouthamr | Hi, could I have one more core-reviewer review this patch: https://review.openstack.org/#/c/211659/ - thanks for the CR, thingee | 15:49 |
| *** erlon has joined #openstack-cinder | 15:49 | |
| *** shyama has joined #openstack-cinder | 15:50 | |
| *** p0rtal has quit IRC | 15:50 | |
| *** p0rtal has joined #openstack-cinder | 15:50 | |
| *** afazkas has joined #openstack-cinder | 15:51 | |
| *** dflorea has joined #openstack-cinder | 15:53 | |
| *** _cjones_ has joined #openstack-cinder | 15:53 | |
| *** p0rtal has quit IRC | 15:55 | |
| *** hemna has joined #openstack-cinder | 15:56 | |
| *** ccesario_ has quit IRC | 15:57 | |
| *** eharney has quit IRC | 15:58 | |
| *** Apoorva has joined #openstack-cinder | 15:58 | |
| *** marcusvrn_ has joined #openstack-cinder | 15:59 | |
| *** eharney has joined #openstack-cinder | 15:59 | |
| *** Apoorva has quit IRC | 15:59 | |
| *** ccesario_ has joined #openstack-cinder | 16:00 | |
| *** martyturner has quit IRC | 16:00 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** Apoorva has joined #openstack-cinder | 16:01 | |
| *** haomaiwang has joined #openstack-cinder | 16:01 | |
| *** porrua has joined #openstack-cinder | 16:01 | |
| *** Yogi11 has quit IRC | 16:02 | |
| openstackgerrit | Goutham Pacha Ravi proposed openstack/cinder: Configure space reservation on NetApp Data ONTAP https://review.openstack.org/211659 | 16:02 |
| *** sayali has joined #openstack-cinder | 16:03 | |
| *** bswartz has quit IRC | 16:04 | |
| *** haomaiwang has quit IRC | 16:04 | |
| *** bswartz has joined #openstack-cinder | 16:06 | |
| *** jistr has quit IRC | 16:08 | |
| *** BharatK has quit IRC | 16:10 | |
| jordanP | thingee, hi. I've fixed my CI issue in https://review.openstack.org/#/c/200610/ (Rework Scality SOFS driver to use RemoteFS class), if you have a chance to have a look again, that would be great. thanks | 16:12 |
| *** DuncanT has joined #openstack-cinder | 16:12 | |
| *** vilobhmm has joined #openstack-cinder | 16:16 | |
| *** skylerberg has joined #openstack-cinder | 16:22 | |
| eharney | jungleboyj: so about https://review.openstack.org/#/c/214191/ ... i think in the past we've considered it better to backport a harmless patch as a dependency to get a clean backport in two patches rather than manually resolving conflicts in the patch fixing the bug? | 16:22 |
| *** leeantho has joined #openstack-cinder | 16:25 | |
| dannywilson | jgriffith: ping, got a replication question | 16:26 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder: force_detach terminate_connection needs connector https://review.openstack.org/213867 | 16:27 |
| *** Yogi1 has joined #openstack-cinder | 16:28 | |
| *** agarciam has quit IRC | 16:30 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Implement manage/unmanage snapshot in Pure drivers https://review.openstack.org/213855 | 16:31 |
| *** akerr is now known as akerr_away | 16:32 | |
| *** esker has quit IRC | 16:34 | |
| jgriffith | dannywilson: sure, what's up? | 16:35 |
| openstackgerrit | Wilson Liu proposed openstack/cinder: Enhance FC zone support for Huawei driver https://review.openstack.org/201578 | 16:35 |
| *** dims has quit IRC | 16:36 | |
| *** mylu has joined #openstack-cinder | 16:37 | |
| *** IanGovett1 has joined #openstack-cinder | 16:38 | |
| *** dims has joined #openstack-cinder | 16:39 | |
| *** shausy has quit IRC | 16:39 | |
| dannywilson | jgriffith: post replication_failover should we be marking any snapshots that are on original host as not available? | 16:40 |
| *** mylu has quit IRC | 16:40 | |
| dannywilson | jgriffith: snapshots from the failed over volume of course | 16:40 |
| *** IanGovett has quit IRC | 16:40 | |
| *** mylu has joined #openstack-cinder | 16:41 | |
| *** annegentle has quit IRC | 16:41 | |
| *** diogogmt_ has joined #openstack-cinder | 16:41 | |
| *** yrabl has quit IRC | 16:42 | |
| *** afazkas has quit IRC | 16:42 | |
| *** krtaylor has quit IRC | 16:43 | |
| jgriffith | dannywilson: so that's a good question... it depends on whether they're actually available or not | 16:43 |
| *** akerr_away is now known as akerr | 16:43 | |
| *** diogogmt has quit IRC | 16:43 | |
| *** diogogmt_ is now known as diogogmt | 16:43 | |
| jgriffith | dannywilson: in my case I've treated it as "if the backend is actually available and I can get to them... great" | 16:43 |
| *** ronis_ has joined #openstack-cinder | 16:44 | |
| jgriffith | dannywilson: if it's a "true" failure, then they're gone | 16:44 |
| jgriffith | dannywilson: make sense? | 16:44 |
| dannywilson | jgriffith: almost | 16:44 |
| dannywilson | jgriffith: do they get marked as not available at any point? or just fail because you can't reach the array? | 16:44 |
| *** mylu has quit IRC | 16:45 | |
| *** mylu has joined #openstack-cinder | 16:45 | |
| jgriffith | dannywilson: fail because you can't reach the array | 16:45 |
| jgriffith | dannywilson: cinder does it for you | 16:46 |
| jgriffith | dannywilson: because the host is marked as 'down' | 16:46 |
| jgriffith | dannywilson: it's kinda redundant to add something else at this point | 16:46 |
| jgriffith | dannywilson: it's certainly something somebody could enhance in the future though | 16:46 |
| patrickeast | jgriffith: i think one thing we are trying to figure out is what to do with the original volume once failed over, for us we need to keep it around to keep its snapshots alive, but if we keep it around its essentially orphaned | 16:47 |
| *** martyturner has joined #openstack-cinder | 16:47 | |
| jgriffith | patrickeast: I'd certainly keep it around | 16:47 |
| patrickeast | jgriffith: in the case where it isn't a true failure, but like a test failover | 16:47 |
| jgriffith | patrickeast: replication fail over doesn't mean you delete anything IMHO | 16:47 |
| jgriffith | patrickeast: it just provides access, that's all | 16:48 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: Rename free_virtual in capacity filter https://review.openstack.org/214276 | 16:48 |
| patrickeast | jgriffith: true, i guess its up to the admin then to go clean that up later manually | 16:48 |
| jgriffith | patrickeast: in other words... "I can't get to it from path A... no big deal, go to path B" | 16:48 |
| jgriffith | patrickeast: now doing all the directional stuff is your problem :) | 16:48 |
| dannywilson | jgriffith: does this implementation consider failover, enable_replication, failover again to put volume back on original backend? or is that for later? | 16:49 |
| jgriffith | dannywilson: it does, that was a requirement by DuncanT | 16:49 |
| jgriffith | dannywilson: failover can just be called multiple times to toggle the backend | 16:49 |
| patrickeast | dannywilson: hmm so that case we would leave behind a orphan volume on each failover XD | 16:50 |
| jgriffith | patrickeast: so you can't just switch primary/secondary back and forth? | 16:50 |
| patrickeast | jgriffith: pure does snapshot based replication, so the 'replicated' volume is actually just snapshots up until we failover and create a volume from the latest one | 16:50 |
| patrickeast | jgriffith: when we go back we could delete the original and re-make it from the replicated snapshot | 16:51 |
| patrickeast | so maybe not too big of a deal | 16:51 |
| patrickeast | just one potential orphan on each array | 16:51 |
| jgriffith | patrickeast: yeah, this whole replication thing is just kind of a pain :( | 16:51 |
| patrickeast | haha truth | 16:51 |
| patrickeast | i think we can make this work though | 16:52 |
| jgriffith | patrickeast: agreed | 16:52 |
| dannywilson | jgriffith: more than "kind of" a pain, but necessary :) | 16:52 |
| jgriffith | dannywilson: :) | 16:52 |
| jgriffith | dannywilson: patrickeast so honestly I'm not set on the idea of toggling back and forth without some admin intervention | 16:53 |
| jgriffith | dannywilson: patrickeast that's pretty ambitious IMO | 16:53 |
| *** dims has quit IRC | 16:54 | |
| *** krtaylor has joined #openstack-cinder | 16:56 | |
| *** dims has joined #openstack-cinder | 16:57 | |
| *** sgotliv_ has quit IRC | 16:59 | |
| *** martyturner has quit IRC | 17:00 | |
| *** martyturner has joined #openstack-cinder | 17:01 | |
| *** Yogi11 has joined #openstack-cinder | 17:02 | |
| *** jordanP has quit IRC | 17:02 | |
| *** dims_ has joined #openstack-cinder | 17:02 | |
| *** dflorea has quit IRC | 17:02 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 17:03 |
| *** dims has quit IRC | 17:04 | |
| *** Yogi1 has quit IRC | 17:04 | |
| *** zhenguo has quit IRC | 17:05 | |
| *** markus_z has quit IRC | 17:07 | |
| *** p0rtal has joined #openstack-cinder | 17:08 | |
| *** dims has joined #openstack-cinder | 17:08 | |
| *** dims_ has quit IRC | 17:09 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port more unit tests to Python 3 https://review.openstack.org/213817 | 17:09 |
| *** IanGovett1 has quit IRC | 17:10 | |
| *** dims_ has joined #openstack-cinder | 17:11 | |
| mriedem | cinder multi-attach is the same volume attached to multiple instances right? | 17:13 |
| *** dims has quit IRC | 17:14 | |
| *** bswartz has quit IRC | 17:15 | |
| scottda | mriedem: Yes, and also the same volume attached to the same instance >1 time | 17:15 |
| yuriy_n17 | geguileo: Hi! Please review https://review.openstack.org/#/c/207469/ | 17:15 |
| mriedem | scottda: so how does the volume get response look if you have multiple attachments? is instance_uuid a list rather than a single string? | 17:16 |
| mriedem | scottda: i was looking at that here https://review.openstack.org/#/c/184537/9/nova/cmd/manage.py | 17:16 |
| mriedem | if vol['instance_uuid'] != instance.uuid | 17:16 |
| hemna | mriedem: you get multiple attachments in a volume | 17:16 |
| mriedem | sure, but what does the response body look like from the volume GET call? | 17:17 |
| * mriedem checks the v2 view builder | 17:17 | |
| mriedem | ah there is an 'attachments' key | 17:17 |
| hemna | https://github.com/openstack/nova/blob/master/nova/volume/cinder.py#L153-L156 | 17:18 |
| *** bswartz has joined #openstack-cinder | 17:18 | |
| *** ndipanov has joined #openstack-cinder | 17:18 | |
| mriedem | isn't that kind of bogus if you have more than one? | 17:18 |
| mriedem | i guess you're expected to just call the volume API each time until attachments is 0 | 17:19 |
| mriedem | if you're trying to detach all instanes | 17:19 |
| mriedem | *instances | 17:19 |
| *** dims_ has quit IRC | 17:20 | |
| *** nikeshm has joined #openstack-cinder | 17:20 | |
| *** boris-42 has quit IRC | 17:20 | |
| scottda | You mean detach all volumes? | 17:20 |
| *** pv has joined #openstack-cinder | 17:20 | |
| mriedem | yeah | 17:20 |
| *** dims has joined #openstack-cinder | 17:20 | |
| *** yuriy_n17 has quit IRC | 17:20 | |
| hemna | iI don’t think that’s bogus | 17:20 |
| *** dims has quit IRC | 17:20 | |
| scottda | Yes, I'm not sure if there is a function for that in the not-yet-landed nova patch for multi-attach | 17:21 |
| *** dims has joined #openstack-cinder | 17:21 | |
| hemna | so the current model in the cinder volume manager is that you aren’t required to pass in an attachment_id | 17:22 |
| hemna | if you don’t, it checks how many attachments there are | 17:22 |
| hemna | if there is 1, it works | 17:22 |
| hemna | if there is >1, it raises | 17:22 |
| hemna | it was unclear what was the ‘safest’ thing to do | 17:22 |
| mriedem | that seems right | 17:23 |
| mriedem | for backwards compat | 17:23 |
| hemna | so I pushed to make this safe and raise instead of detaching all | 17:23 |
| mriedem | if yo'ure doing multiattach cases then you need to opt into detaching a specific thing | 17:23 |
| hemna | yup | 17:23 |
| mriedem | if the caller wants to do all, they have to iterate that on their side i guess | 17:23 |
| hemna | so if you want to detach all, currently you need to loop over attachments and call detach on each. | 17:23 |
| mriedem | unless you had like a special case attachment_id == 'all' case or something | 17:24 |
| hemna | probably not a bad idea to make a change to detach all | 17:24 |
| hemna | just not sure what the ‘right’ API change is to support that | 17:24 |
| scottda | We talked about an admin action to detach_all for help with cleanup when we don't have a Nova instance or attachement | 17:24 |
| hemna | detach_all ? | 17:24 |
| hemna | or add an additional argument to detach, all=False | 17:24 |
| *** Yogi11 has quit IRC | 17:25 | |
| pv | im getting a volume-attach error in my nova compute logs at File "/usr/local/lib/python2.7/dist-packages/cinderclient/v2/volumes.py", line 448, in initialize_connection {'connector': connector})[1]['connection_info'] | 17:25 |
| pv | that point | 17:25 |
| pv | should i ask on nova irc or here | 17:25 |
| hemna | scottda: that was slightly different though, as it was for force detach, but w/o connectors :( | 17:25 |
| *** ebalduf has joined #openstack-cinder | 17:25 | |
| *** Yogi1 has joined #openstack-cinder | 17:25 | |
| mriedem | pv: you're going to need a paste of the full stacktrace | 17:25 |
| scottda | so, if the Nova instance is gone, but Cinder still thinks it's attached, we'd have something to terminate_connection for the volume in general, without a specific attachement_id | 17:25 |
| scottda | hemna: right | 17:25 |
| mriedem | pv: and level of code you're on, environment, backends used, etc etc | 17:26 |
| mriedem | no dead cats | 17:26 |
| hemna | scottda: we should still think about updating the attachments table to store the connector dict | 17:26 |
| scottda | Yes, indeed | 17:26 |
| hemna | then it would be possible to do a force detach where the nova instance is gone | 17:26 |
| pv | http://pastebin.com/EJw8WpXm | 17:27 |
| scottda | I'll add that to the list of Mitaka stuff | 17:27 |
| pv | im running devstack on an ubuntu server testing my driver's initialize/terminate connection code | 17:27 |
| hemna | scottda: ok. I can definitely help with that | 17:28 |
| pv | its an FC driver and FC enabled machine | 17:28 |
| scottda | swell | 17:28 |
| hemna | pv, that’s the first error? | 17:29 |
| hemna | pv did nova complain about not finding the volume? | 17:29 |
| *** IanGovett has joined #openstack-cinder | 17:29 | |
| hemna | pv: I have a feeling that nova complained before that trace | 17:29 |
| pv | yeah from what im seeing on the logs | 17:30 |
| pv | let me check the overall logfile | 17:30 |
| pv | one sec | 17:30 |
| pv | the first error in the log is 2015-08-18 10:10:12.088 ERROR nova.compute.manager [req-17165518-6dcd-446c-9aab-53bd8fdc7aaa None None] No compute node record for host cli | 17:31 |
| hemna | mriedem: scottda, so this addition to detach seems like a good first change after microversions land | 17:31 |
| pv | then its the failed to attach volume error | 17:31 |
| hemna | pv: check the cinder log around the same time | 17:32 |
| pv | kk | 17:32 |
| hemna | pv looks like something puked in Cinder’s call to initialize_connection | 17:33 |
| mriedem | hemna: scottda: sure, i'm not actually even asking for this, just asking questions about how it works today | 17:33 |
| hemna | mriedem: ok np | 17:33 |
| hemna | the nova multiattach code still hasn’t landed | 17:33 |
| scottda | hemna: Yes Microversions Will Fix All® | 17:33 |
| hemna | so no one is really using multiattach in Cinder | 17:33 |
| hemna | so I think we can make changes as needed in M | 17:33 |
| hemna | scottda: who is going to do it :) | 17:34 |
| mriedem | devananda | 17:34 |
| *** martyturner has quit IRC | 17:34 | |
| hemna | scottda: thingee knows the API better than anyone afaik | 17:34 |
| pv | ERROR cinder.volume.manager [req-0ff37083-9611-483c-b5a6-9bf7aeb33b0a admin] [volume-dc88faa0-b710-4c5a-baaf-d8f2a450efe9] Driver initialize connection failed (error: expected string or buffer). | 17:34 |
| pv | cool theres the issue | 17:34 |
| scottda | I'm going to write the spec, just as soon as I'm done with Nova-Cinder API spec | 17:34 |
| pv | thanks a lot | 17:34 |
| hemna | pv np | 17:34 |
| *** martyturner has joined #openstack-cinder | 17:35 | |
| scottda | Then I'll see if I can s̶t̶e̶a̶l̶ leverage code from Manila or Ironic | 17:35 |
| *** dflorea has joined #openstack-cinder | 17:38 | |
| hemna | scottda: +1 | 17:38 |
| *** p0rtal has quit IRC | 17:40 | |
| *** angela-s has joined #openstack-cinder | 17:40 | |
| erlon | hemna: ping | 17:40 |
| hemna | erlon: hey | 17:41 |
| *** p0rtal has joined #openstack-cinder | 17:42 | |
| *** breitz has quit IRC | 17:42 | |
| *** dflorea has quit IRC | 17:42 | |
| erlon | hemna: hey, so, I'm testing that patch(that removes the scan on disconnect) in a multinode setup, and the devices still are note removed from the hosts | 17:42 |
| *** dflorea has joined #openstack-cinder | 17:42 | |
| skylerberg | I saw on the etherpad that third party CI systems will be expected to run against changes to os-brick. Is there more information on this somewhere? | 17:43 |
| hemna | skylerberg: for folks that have custom connectors in os-brick yes | 17:43 |
| erlon | hemna: do you have any hint why? | 17:43 |
| hemna | erlon: so what kind of use case causes this? | 17:44 |
| skylerberg | hemna: Thanks. I misread the etherpad and thought "all drivers" refered to Cinder drivers. | 17:45 |
| hemna | skylerberg: since os-brick does the volume detection and removal, if you want to make sure that any changes to os-brick doesn’t break your backend, then it’s a good idea to add your CI for your array against os-brick patches | 17:45 |
| erlon | hemna: create a VM, boot from image(creates a new volume), with instance count >= 4 | 17:45 |
| hemna | skylerberg: that’s the upside to having your CI added to os-brick patches. | 17:45 |
| *** changbl has joined #openstack-cinder | 17:46 | |
| hemna | erlon: the volume is a n-cpu lvm volume though no? | 17:46 |
| erlon | hemna: ?? | 17:47 |
| hemna | boot from image isn’t a cinder volume afaik | 17:48 |
| *** JoseMello has quit IRC | 17:49 | |
| erlon | hemna: yes if you select 'boot from image(creates a new volume)' | 17:49 |
| hemna | erlon: afaik that volume is an lvm volume from the nova compute host itself, not cinder | 17:50 |
| *** breitz has joined #openstack-cinder | 17:52 | |
| *** ociuhandu has quit IRC | 17:52 | |
| erlon | hemna: I don't think so, I've been using this to test some features on HNAS, and if the BE is configured as default_volume_type, nova will boot from the volumes on the backend | 17:53 |
| erlon | hemna: I actually not even have LVM configured in my setup | 17:53 |
| jungleboyj | hemna? | 17:53 |
| hemna | jusup | 17:53 |
| hemna | jungleboyj: yes | 17:53 |
| *** simondodsley has quit IRC | 17:53 | |
| hemna | erlon: LVM on the compute host. | 17:53 |
| jungleboyj | hemna: The boot from volume creates a volume in whatever storage you have configured, mounts it on the control node, copies the inmage and then attaches it to the compute node for boot. | 17:54 |
| hemna | boot from volume yes | 17:54 |
| hemna | boot from image ? | 17:54 |
| jungleboyj | Oh, Sorry. | 17:54 |
| hemna | boot from volume, boots from a pre-existing volume on Cinder | 17:54 |
| jungleboyj | hemna: Ok, so in that case, yes the image is put in storage on the Compute node. Where ever images are being held for the hypervisor. | 17:55 |
| erlon | hemna: yes, there's the 3 options in horizon, boot from volume, boot from image and boot from image creating a new volume | 17:55 |
| *** garthb_ has quit IRC | 17:55 | |
| hemna | you can do a boot from snapshot | 17:57 |
| hemna | whicih creates a new cinder volume from the snapshot | 17:57 |
| hemna | but boot from image is a local nova compute lvm volume | 17:58 |
| hemna | unless I’m drunk and have lost my mind | 17:58 |
| *** dflorea has quit IRC | 17:59 | |
| *** dflorea has joined #openstack-cinder | 17:59 | |
| jungleboyj | eharney: Updated my vote on those backports. You are right, doing the two backports makes more sense. | 18:00 |
| erlon | hemna: https://www.dropbox.com/s/ri2bjlme9fsw2eu/horizon.png?dl=0 | 18:01 |
| eharney | jungleboyj: thanks :) | 18:01 |
| hemna | erlon: k that’s a horizon thing. not sure what it does actually | 18:02 |
| hemna | I’m looking at the nova api | 18:02 |
| hemna | could be a wrapper to call cinder first | 18:02 |
| jungleboyj | eharney: Welcome. I was spinning back and forth between those this morning. In the end, you are right. | 18:03 |
| *** mylu has quit IRC | 18:03 | |
| erlon | hemna: I don't know what happens under the hoods in nova, but in cinder it creates a volume (with blank name), attach on volume node, copy the image from glance, and then disconnect the volume | 18:05 |
| erlon | hemna: only after that nova proceeds booting | 18:05 |
| hemna | ok, that must be a wrapper in horizon then to call cinder first to create volume from image | 18:06 |
| *** chlong has quit IRC | 18:06 | |
| hemna | do you get a choice for volume type? | 18:06 |
| hemna | just get a random volume backend ? | 18:06 |
| erlon | hemna: no, it always use the default_volume_type | 18:06 |
| hemna | ick ok | 18:06 |
| *** IanGovett1 has joined #openstack-cinder | 18:07 | |
| hemna | so you boot a vm, then destroy it and you have orphaned volumes on the compute host ? | 18:07 |
| jgriffith | hemna: erlon yes | 18:07 |
| jgriffith | hemna: erlon horizon provides a boot from volume wrapper | 18:08 |
| jgriffith | hemna: erlon it does NOT provide a method to specify type (which is rtes-lame) | 18:08 |
| hemna | yah that’s lame stuff right there | 18:08 |
| jgriffith | hemna: erlon it always uses the configured default type or "no" type | 18:08 |
| jgriffith | hemna: it's been there for 3 or 4 releases :( | 18:08 |
| hemna | :( | 18:09 |
| jgriffith | hemna: erlon honestly I suggest to my customers to NOT use it | 18:09 |
| hemna | erlon: ok I’ll test this out on my backend tomorrow. I’m WAH today and can’t really test it at the moment. | 18:09 |
| *** chlong has joined #openstack-cinder | 18:09 | |
| *** IanGovett has quit IRC | 18:09 | |
| david-lyle | jgriffith: hemna, so would the suggestion be to drop in the create volume workflow there, or another selector for type enough? | 18:10 |
| jgriffith | david-lyle: type selector would solve it | 18:10 |
| erlon | hemna: great, Ill keep trying to understand what happens | 18:10 |
| jgriffith | david-lyle: i tried to hack it in there myself but screwed everything up :) | 18:10 |
| jgriffith | david-lyle: I think you and I talked about this in FTC last summer... not sure | 18:11 |
| *** openstackgerrit_ has quit IRC | 18:11 | |
| david-lyle | most likely, and I probably spaced it, :( | 18:11 |
| jgriffith | david-lyle: just adding the type selector would make the functionality identical | 18:11 |
| jgriffith | david-lyle: me too LOL | 18:11 |
| david-lyle | ok, I can take a look at that | 18:11 |
| jgriffith | david-lyle: there is one other problem though..... | 18:11 |
| jgriffith | david-lyle: the timing in all the wrapped rpc calls | 18:11 |
| jgriffith | david-lyle: I can't remember exactly who's call ends up owning it, but it times out frequently for larger images/volumes | 18:12 |
| *** salv-orlando has quit IRC | 18:12 | |
| hemna | ick yah large volumes can take a while to dd from the image :( | 18:12 |
| jgriffith | david-lyle: because there's a parent call that own the timeout, and it's not long enough for the horribly long download/convert/copy-to-volume/boot process | 18:12 |
| jgriffith | david-lyle: I couldn't quite figure out how to mess with that | 18:13 |
| david-lyle | jgriffith: so even with the type selector it would only work for small images? | 18:13 |
| jgriffith | david-lyle: but I'm happy to help out later this week or next if you like | 18:13 |
| david-lyle | unless we tackle the second part? | 18:13 |
| jgriffith | david-lyle: yeah... there's a time issue for larger issues. At least last I checked | 18:13 |
| jgriffith | david-lyle: in other words "yes, you're statement is correct" | 18:13 |
| david-lyle | I doubt we did anything to fix it | 18:14 |
| david-lyle | :) | 18:14 |
| jgriffith | Never know :) | 18:14 |
| david-lyle | ok, I can start on the selector and then work on timing issues | 18:14 |
| jgriffith | that should be "there's a time issue for larger images" | 18:14 |
| jgriffith | awesome!!! | 18:14 |
| jgriffith | david-lyle: thanks!!! | 18:14 |
| david-lyle | incremental progress at least | 18:14 |
| jgriffith | david-lyle: let me know if I can help | 18:14 |
| jgriffith | david-lyle: yeah, that alone would be awesome | 18:15 |
| *** IanGovett has joined #openstack-cinder | 18:15 | |
| david-lyle | jgriffith: will do. I'll finish up the current bug fix first :) | 18:15 |
| david-lyle | why are there so many? | 18:15 |
| jgriffith | LOL | 18:16 |
| *** dflorea has quit IRC | 18:16 | |
| jgriffith | david-lyle: hemna erlon just to verify we're all on the same page: https://www.dropbox.com/s/shugdv640lj553l/Screenshot%202015-08-18%2012.16.00.png?dl=0 | 18:16 |
| jgriffith | "Boot form image (creates a new volume)" is what I'm referring to | 18:17 |
| david-lyle | yes | 18:17 |
| *** IanGovett1 has quit IRC | 18:17 | |
| jgriffith | awesome! | 18:17 |
| openstackgerrit | Angela Smith proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 18:17 |
| david-lyle | is there a bug already, or should I open one | 18:17 |
| erlon | jgriffith: yes | 18:17 |
| david-lyle | ? | 18:17 |
| jgriffith | david-lyle: I think I logged one at one point... lemme see if I can find it | 18:17 |
| erlon | jgriffith: that's the way I found to simulate the 'very busy' systems situation | 18:18 |
| jgriffith | david-lyle: for me and LP it's easier to just log a new one LOL | 18:18 |
| david-lyle | jgriffith: that's an LP feature | 18:18 |
| jgriffith | david-lyle: haha... they implemented it VERY well | 18:18 |
| jgriffith | david-lyle: doesn't look like I ever logged one | 18:19 |
| jgriffith | david-lyle: you want me to log it? | 18:19 |
| david-lyle | jgriffith: that would help | 18:20 |
| *** dannywilson has quit IRC | 18:20 | |
| david-lyle | jgriffith: and I'll pick it up | 18:20 |
| *** tim_o_ has joined #openstack-cinder | 18:22 | |
| erlon | david-lyle: is it possible to set a timeout based on the size of the image? I mean, not a fixed timeout, but something like, image_mb_timeout, that will leead to an actual 'image.size() * image_mb_timeout' timeout | 18:23 |
| *** IanGovett has quit IRC | 18:24 | |
| david-lyle | erlon: I have no idea, if the APIs support passing it in, the calculation could be done in Horizon | 18:25 |
| *** IanGovett has joined #openstack-cinder | 18:25 | |
| *** dannywilson has joined #openstack-cinder | 18:25 | |
| jgriffith | erlon: that would be super tricky. We just need to make the create volume the parent.. but we can tackle that one later :) | 18:25 |
| erlon | jgriffith: mhm | 18:26 |
| erlon | jgriffith: I don't know may be that would just be a couple lines | 18:27 |
| *** dflorea has joined #openstack-cinder | 18:27 | |
| *** IanGovett1 has joined #openstack-cinder | 18:28 | |
| jgriffith | david-lyle: https://bugs.launchpad.net/horizon/+bug/1486178 | 18:29 |
| openstack | Launchpad bug 1486178 in OpenStack Dashboard (Horizon) "Boot from image (creates a new volume) Doesn't allow specification of volume-type" [Undecided,New] | 18:29 |
| jgriffith | erlon: ok, go for it :) | 18:29 |
| jgriffith | erlon: you might have a great idea that I haven't thought of | 18:29 |
| *** martyturner has quit IRC | 18:29 | |
| david-lyle | jgriffith: on it. Thanks! | 18:29 |
| jgriffith | david-lyle: thank you sir! | 18:29 |
| jgriffith | david-lyle: I'll dig into the timing thing later and file something there if I get more info/details | 18:30 |
| jgriffith | david-lyle: or maybe erlon has a cool proposal to deal with it | 18:30 |
| *** dflorea_ has joined #openstack-cinder | 18:30 | |
| erlon | jgriffith: :) | 18:30 |
| *** IanGovett has quit IRC | 18:30 | |
| david-lyle | jgriffith: let me know what I can do to help on that one. | 18:31 |
| tim_o_ | asselin, i have a couple of follow up questions can we chat? | 18:31 |
| erlon | jgriffith: have you seen what I told to Walter? | 18:31 |
| jgriffith | david-lyle: thanks man! | 18:31 |
| jgriffith | erlon: I'll scroll back.... | 18:31 |
| jgriffith | erlon: you told him lots of things it looks like :) | 18:31 |
| erlon | jgriffith: the fix merged into brick yesterday is not 'fixing' in my multinode setup | 18:32 |
| jgriffith | erlon: oh... which problem are you seeing? Logs? | 18:32 |
| erlon | jgriffith: there are still lost /dev/sdx lost | 18:32 |
| *** martyturner has joined #openstack-cinder | 18:33 | |
| jgriffith | Can somebody please explain to me HOW I can have an Iron Maiden radio channel on Spotify and it plays the firkin Beatles every once in a while!! WTF? | 18:33 |
| tim_o_ | asselin, got pulled into a meeting -- will ping you later | 18:33 |
| jgriffith | I mean... I HATE the Beatles, and they're not even related!!! | 18:33 |
| erlon | no, I create a VM, booting from image + creating the image, when I remove then, at the same time, the block devices are not removed | 18:33 |
| *** dflorea has quit IRC | 18:33 | |
| *** dflorea has joined #openstack-cinder | 18:34 | |
| jgriffith | erlon: not sure I follow exactly; but I think that's very different than what I was worried about | 18:34 |
| jgriffith | erlon: which block devices? nova's? | 18:34 |
| jgriffith | erlon: there's the attach and then the mapped block device on the Nova side AIR | 18:35 |
| erlon | hmm, yes, and the one used on c-volume to copy from the image | 18:35 |
| jgriffith | erlon: I had to filter things on LVM conf for this very problem | 18:35 |
| *** skiafa has joined #openstack-cinder | 18:35 | |
| erlon | jgriffith: filter? | 18:36 |
| hemna | jgriffith: I get worse with Pandora. I fire up a Queens of the Stone Age channel and get Dave Mathews Band. *sigh* | 18:36 |
| pv | spotify radio is pretty on point | 18:36 |
| pv | comparitively | 18:36 |
| hemna | yah it’s pretty good. | 18:36 |
| hemna | Pandora is junk IMHO | 18:36 |
| *** dflorea_ has quit IRC | 18:36 | |
| erlon | jgriffith: try to get back to last.fm :) | 18:37 |
| jgriffith | erlon: jeje | 18:37 |
| *** tim_o_ has quit IRC | 18:37 | |
| skiafa | hi folks, I was wondering if someone is working on the manage/unmanage volume tests... does anybody know? | 18:38 |
| jgriffith | skiafa: not that I know of | 18:38 |
| erlon | skiafa: we have just talked about that today | 18:39 |
| skiafa | tks j. yep... I saw that... rs so this is a tempest project specific or is a cinder specific? | 18:40 |
| erlon | skiafa: I don't know as well, would be nice to have that | 18:40 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota Driver: Get Project Hierarchy https://review.openstack.org/206171 | 18:40 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 18:40 |
| erlon | skiafa: tempest | 18:41 |
| *** shyama has quit IRC | 18:43 | |
| skiafa | erlon: cool... I´ll look into it then... tks | 18:43 |
| *** mylu has joined #openstack-cinder | 18:46 | |
| *** dflorea has quit IRC | 18:50 | |
| *** dflorea has joined #openstack-cinder | 18:53 | |
| *** mylu has quit IRC | 18:53 | |
| *** mylu has joined #openstack-cinder | 18:54 | |
| adrianofr | hemna: Hi... we addressed your comment in https://review.openstack.org/#/c/197999/ | 18:54 |
| adrianofr | hemna: it would be nice if you could take a look on this | 18:54 |
| *** dflorea has quit IRC | 18:55 | |
| *** mylu_ has joined #openstack-cinder | 18:55 | |
| *** mylu has quit IRC | 18:56 | |
| *** ociuhandu has joined #openstack-cinder | 18:56 | |
| hemna | adrianofr: ok | 18:57 |
| adrianofr | hemna: tks :) | 18:57 |
| *** mylu_ has quit IRC | 18:58 | |
| *** mylu has joined #openstack-cinder | 18:59 | |
| *** sgordon has quit IRC | 18:59 | |
| *** sgordon has joined #openstack-cinder | 18:59 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 18:59 |
| *** mylu has quit IRC | 19:01 | |
| *** mylu has joined #openstack-cinder | 19:01 | |
| *** Lee1092 has quit IRC | 19:03 | |
| *** dflorea has joined #openstack-cinder | 19:03 | |
| *** annegentle has joined #openstack-cinder | 19:06 | |
| *** bill_az has quit IRC | 19:06 | |
| *** e0ne has joined #openstack-cinder | 19:07 | |
| *** ndipanov has quit IRC | 19:09 | |
| *** mylu has quit IRC | 19:14 | |
| *** mylu has joined #openstack-cinder | 19:14 | |
| *** esker has joined #openstack-cinder | 19:15 | |
| *** salv-orl_ has joined #openstack-cinder | 19:16 | |
| *** martyturner has quit IRC | 19:16 | |
| *** annegentle has quit IRC | 19:21 | |
| *** skylerberg has quit IRC | 19:22 | |
| *** mylu has quit IRC | 19:23 | |
| *** mylu has joined #openstack-cinder | 19:23 | |
| *** simondodsley has joined #openstack-cinder | 19:24 | |
| *** annegentle has joined #openstack-cinder | 19:26 | |
| *** mylu has quit IRC | 19:27 | |
| *** crose has quit IRC | 19:30 | |
| *** skylerberg has joined #openstack-cinder | 19:32 | |
| *** simondodsley has quit IRC | 19:33 | |
| *** mylu has joined #openstack-cinder | 19:36 | |
| *** belmoreira has joined #openstack-cinder | 19:36 | |
| *** e0ne has quit IRC | 19:44 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Better exception handling in init_conn https://review.openstack.org/214316 | 19:45 |
| openstackgerrit | Adriano Freires Rosso proposed openstack/cinder: Adds the randomic option to cinder retry function https://review.openstack.org/214317 | 19:46 |
| hodos|2 | jgriffith: VolumeDeviceNotFound: Volume device not found at [u'/dev/disk/by-path/ip-172.16.0.74:3260-iscsi-iqn.1986-03.com.sun:02:cinder-172.16.0.74-1-lun-0'] | 19:51 |
| *** belmoreira has quit IRC | 19:51 | |
| hodos|2 | jgriffith: is this the error that os-brick (new one) fixes ? | 19:51 |
| hodos|2 | jgriffith: it's happening during copy image to volume | 19:52 |
| hodos|2 | jgriffith: in the os_brick/initiator/connector.py 413 | 19:52 |
| *** gouthamr has quit IRC | 19:56 | |
| *** annegentle has quit IRC | 19:57 | |
| *** Yogi1 has quit IRC | 19:57 | |
| *** gouthamr has joined #openstack-cinder | 19:58 | |
| *** Yogi1 has joined #openstack-cinder | 19:58 | |
| *** Yogi1 has quit IRC | 19:58 | |
| *** annegentle has joined #openstack-cinder | 20:00 | |
| jgriffith | hodos|2: yes, that looks similar to what I enountered | 20:00 |
| hodos|2 | jgriffith: hmm didn't help me with os-brick from git | 20:01 |
| jgriffith | hodos|2: lemme switch to my other device, maybe it doesn't really fix it for me either | 20:01 |
| hodos|2 | jgriffith: when i do ls -la /dev/disk/by-path/ I see a stale device which is linked to /dev/mapper/.... | 20:02 |
| hodos|2 | jgriffith: when i do ls -la /dev/disk/by-path/ I see a stale device which is linked to /dev/mapper/.... | 20:02 |
| hodos|2 | jgriffith: I've read erlon's troubles looks similar... | 20:03 |
| *** dflorea has quit IRC | 20:04 | |
| jgriffith | hodos|2: there certainly seems to be something that we broke somewhere | 20:05 |
| jgriffith | hodos|2: race condition of some sort or another | 20:05 |
| hemna | hodos|2: /dev/mapper entries are usually created by multipath | 20:05 |
| hemna | I’m on a call at the moment…can’t help much | 20:06 |
| hemna | hodos|2: have you tried the latest os-brick from github? We recently landed a fix to remove iscsi rescan on detach, which was known to cause orphaned entries in /dev/disk/by-path | 20:08 |
| hodos|2 | hemna: yes | 20:10 |
| hemna | hodos|2: did you clean up /dev/disk/by-path prior to doing a test? | 20:10 |
| hemna | just to make sure those weren’t stale before the test | 20:11 |
| hodos|2 | hemna: yes. i can spin up an instance and tell you the version. yes it was new VM | 20:11 |
| hemna | if you know those aren’t valid volume paths | 20:12 |
| hemna | you can nuke them | 20:12 |
| *** ronis_ has quit IRC | 20:12 | |
| hemna | and manually issue an iscsi rescan | 20:12 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Better exception handling in init_conn https://review.openstack.org/214316 | 20:12 |
| hemna | see if they come back | 20:12 |
| hemna | if they do, then it’s because those volumes are still being exported by the backend | 20:12 |
| openstackgerrit | Jay Bryant proposed openstack/cinder: Register the volume_opts config options in remotefs.py https://review.openstack.org/214325 | 20:15 |
| hodos|2 | hemna: ok, so what happens is that when I run multiple 'create volume from image' (which is done also in tempest suite) sometimes i get this exception that "device is not found" while i have these stale mappings in the /dev/disk/by-path | 20:16 |
| hemna | hodos|2: hrmm sounds familair | 20:16 |
| hemna | sounds like something that erlon is seeing related to create volume from image | 20:16 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 20:16 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Conversion to volume object https://review.openstack.org/177054 | 20:16 |
| hodos|2 | hemna: and because it happens only after several create/delete operations I don't really know which volume is supposed to be there | 20:17 |
| hodos|2 | hemna: or not | 20:17 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: WIP FC Stop calling multipath command line https://review.openstack.org/213389 | 20:17 |
| *** mylu has quit IRC | 20:17 | |
| hemna | hodos|2: so is this multiple create volume from image commands directly to cinder | 20:18 |
| hemna | or is it also nova boot from image (created from volume) ? | 20:18 |
| hodos|2 | hemna: yes | 20:18 |
| hemna | which? | 20:18 |
| *** boris-42 has joined #openstack-cinder | 20:19 | |
| hodos|2 | hemna: i think it's cinder | 20:19 |
| hemna | ok only cinder | 20:19 |
| hodos|2 | hemna: I was able to reproduce without booting | 20:19 |
| hemna | I’ll fire up a new vagrant and see if I can reproduce this | 20:20 |
| *** gouthamr has quit IRC | 20:20 | |
| *** bswartz has quit IRC | 20:24 | |
| *** zul has joined #openstack-cinder | 20:28 | |
| *** dflorea has joined #openstack-cinder | 20:40 | |
| *** changbl has quit IRC | 20:40 | |
| *** mylu has joined #openstack-cinder | 20:41 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Revert "mark oslo.vmware as optional dependency" https://review.openstack.org/214340 | 20:46 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Revert "Enhance unit tests for zfssa drivers" https://review.openstack.org/214341 | 20:46 |
| eharney | hrm | 20:46 |
| *** esker has quit IRC | 20:47 | |
| *** mylu has quit IRC | 20:50 | |
| *** IanGovett1 has quit IRC | 20:50 | |
| hemna | hodos|2: just ran a simple test with a cirros image and it worked w/o an orphaned volume | 20:51 |
| hemna | hodos|2: what image are you using? | 20:52 |
| hodos|2 | hemna: cirros | 20:52 |
| hodos|2 | hemna: did you do just a single volume? maybe do a mix of simple volumes and volumes from images | 20:53 |
| hemna | yah a single volume from image | 20:53 |
| hodos|2 | hemna: it works with single all the time | 20:53 |
| erlon | hemna: yesterday I tested the new patch from os-brick and it solved the problem in my onenode devstack setup | 20:54 |
| hemna | erlon: ok great | 20:54 |
| smcginnis | Anyone know how to pull stats on gate tests? Curious as to how many of the sheepdog test runs have actually passed. | 20:54 |
| smcginnis | thingee: ^^ | 20:54 |
| erlon | hemna: today I tried multinode and the problem appeared again | 20:55 |
| erlon | hemna: and 1 volume at time always works ok | 20:55 |
| *** simondodsley has joined #openstack-cinder | 20:56 | |
| hemna | smcginnis: https://github.com/jogo/lastcomment | 20:56 |
| *** cdelatte has quit IRC | 20:57 | |
| smcginnis | hemna: Cool, thanks! | 20:57 |
| *** dustins has quit IRC | 20:57 | |
| hemna | :) np | 20:57 |
| *** dflorea has quit IRC | 20:57 | |
| smcginnis | hemna: Hmm, that's by user though. Sheepdog is done by jenkins. So need to pull out all jenkins comments for openstack/cinder with the results for sheepdog. | 20:58 |
| *** xyang1 has quit IRC | 20:58 | |
| smcginnis | Will have to play around with this and see what it shows. | 20:58 |
| hemna | smcginnis: I think it supports by CI | 20:58 |
| hemna | oh yah I think this is for 3rd party CI | 20:58 |
| boris-42 | jgriffith: hemna hi there | 20:58 |
| hemna | boris-42: hey | 20:59 |
| *** julim has quit IRC | 21:00 | |
| boris-42 | I would like to continue work on osprofiler | 21:00 |
| boris-42 | hemna: ^ | 21:00 |
| boris-42 | I would like to change that way how it is configured via cinder.conf | 21:00 |
| *** zul has quit IRC | 21:00 | |
| boris-42 | hemna: ^ do we need deprecation and other stuff for old conf options | 21:01 |
| openstackgerrit | Scott DAngelo proposed openstack/cinder-specs: WIP draft of Cinder-Nova-API-changes https://review.openstack.org/211781 | 21:01 |
| *** dflorea has joined #openstack-cinder | 21:01 | |
| *** krtaylor has quit IRC | 21:01 | |
| hemna | if you mean to change the options used, then yah, you’ll have to deprecate the existing options and continue to support them at least a release AFAIK | 21:02 |
| smcginnis | hemna, boris-42: Would that policy apply to osprofiler? | 21:03 |
| boris-42 | hemna: ok | 21:03 |
| smcginnis | Seems like we have a little more leeway with that one. | 21:03 |
| boris-42 | smcginnis: ya it's not core functionallity | 21:03 |
| smcginnis | Not like it's functionality most end users are expecting to stay consistent. | 21:03 |
| boris-42 | it can be treat as experimental functionallity | 21:03 |
| hemna | probably still not a good idea to just stop supporting them w/o deprecating in a release no? | 21:03 |
| *** dflorea has quit IRC | 21:03 | |
| smcginnis | I doubt many know about it. | 21:03 |
| smcginnis | hemna: True. | 21:03 |
| boris-42 | hemna: ok I will try to make via deprecation | 21:03 |
| boris-42 | in any case it is broken now =( | 21:04 |
| hemna | if it’s impossible, then we should discuss | 21:04 |
| smcginnis | Hah, then nothing to deprecate! :P | 21:04 |
| hemna | might be a good meeting topic for tomorrow ? | 21:04 |
| *** simondodsley has quit IRC | 21:07 | |
| boris-42 | hemna: I am going to move this in mailing list | 21:07 |
| boris-42 | hemna: it's not only about Cinder | 21:07 |
| hemna | boris-42: ok coolio | 21:07 |
| hodos|2 | hemna: so you just tried 1 volume? | 21:09 |
| hemna | hodos|2: yah | 21:09 |
| *** raildo is now known as raildo-afk | 21:09 | |
| *** krtaylor has joined #openstack-cinder | 21:14 | |
| openstackgerrit | Angela Smith proposed openstack/cinder: Adds VF support with HTTPS southbound connector https://review.openstack.org/184799 | 21:15 |
| eharney | jgriffith: i'd rather keep it, but keeping it requires tox 1.7, thingee says we can't move to tox 1.7 -- i'd also like to be able to actually run tox :) | 21:19 |
| eharney | jgriffith: nobody said previously that we couldn't move to tox 1.7 | 21:20 |
| jgriffith | eharney: haha... what a mess | 21:20 |
| jgriffith | eharney: ok, so I'll approve your revert then | 21:20 |
| eharney | jgriffith: i'm not really sure why we can't, but, regardless, we need something that at least runs :) | 21:20 |
| jgriffith | eharney: I didn't know you were around or would've just pinged you | 21:20 |
| smcginnis | Is there a reason we can't move to 1.7? | 21:20 |
| eharney | i think at least one other project is already on 1.7, so, i don't really think so | 21:20 |
| jgriffith | smcginnis: that's a great question :) | 21:21 |
| jgriffith | eharney: yes, neutron I believe is already there | 21:21 |
| eharney | https://review.openstack.org/#/c/211614/ | 21:21 |
| jgriffith | so it's being used anyway | 21:21 |
| smcginnis | So I wonder if this is just a misconception that it needs to be a cross-project lock step move. | 21:21 |
| *** zul has joined #openstack-cinder | 21:22 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Test whether sample config generation works https://review.openstack.org/214356 | 21:22 |
| eharney | smcginnis: maybe, but either way, currently tests are broken on a significant number of setups, so, we need to land something to fix that if we can't decide about upgrading | 21:22 |
| smcginnis | eharney: I suppose we can always revisit it later. | 21:23 |
| eharney | smcginnis: the optional dependency thing is something we can live without for the moment, as nice as it would be... we've lived without it for a while already | 21:23 |
| *** thangp has quit IRC | 21:23 | |
| smcginnis | Very true | 21:23 |
| eharney | yet other stuff related to tests in that patch ^ | 21:23 |
| smcginnis | eharney: check_uptodate.sh: can not find config file | 21:24 |
| eharney | smcginnis: i figured something would blow up there... so many different envs to check against :) | 21:25 |
| smcginnis | from ~/.../OpenStack/cinder$ ./run_tests.sh -p | 21:25 |
| jgriffith | I honestly don't even know what to think any more... I should probably just give up | 21:25 |
| eharney | smcginnis: err, well, the point is that the current tree is broken | 21:25 |
| eharney | smcginnis: pull Jay's patch too if you want to see it succeed :) | 21:25 |
| eharney | smcginnis: wait... no, i have no idea about that one | 21:26 |
| smcginnis | eharney: Yours is dependent on his, so it should have his changes. | 21:26 |
| smcginnis | Investigating... | 21:27 |
| eharney | smcginnis: i see what i did wrong there... -p doesn't generate the config file, but it asks to check it | 21:27 |
| eharney | smcginnis: will update in a sec | 21:27 |
| smcginnis | eharney: Awesome | 21:29 |
| smcginnis | eharney: Bonus points from me for even remembering to update run_tests.sh :) | 21:29 |
| eharney | smcginnis: i just need to move more of the old code into the "else" section | 21:29 |
| openstackgerrit | Eric Harney proposed openstack/cinder: Test whether sample config generation works https://review.openstack.org/214356 | 21:31 |
| *** garthb has joined #openstack-cinder | 21:34 | |
| *** cbader has quit IRC | 21:35 | |
| *** tiny-hands has quit IRC | 21:36 | |
| smcginnis | eharney: Much better. | 21:36 |
| *** porrua has quit IRC | 21:39 | |
| *** akerr has quit IRC | 21:40 | |
| *** sileht has quit IRC | 21:43 | |
| *** edmondsw has quit IRC | 21:44 | |
| Swanson | If anyone with a +2 hammer could take a look at this https://review.openstack.org/#/c/214316/ it would be great. Minor exception handling change to give me a chance to find a failure. | 21:45 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/python-cinderclient: Add support '--all-tenants' for cinder backup-list https://review.openstack.org/207469 | 21:45 |
| jungleboyj | eharney: Can we add that in for those who still use run_tests.sh as well? | 21:47 |
| eharney | jungleboyj: it is | 21:47 |
| eharney | (i use run_tests.sh actually) | 21:47 |
| smcginnis | jungleboyj: That's what I tested with. Works good now. | 21:47 |
| *** mriedem has quit IRC | 21:48 | |
| jungleboyj | Hmmm, I tried it and it failed for tox but not run_tests. | 21:48 |
| jungleboyj | Let me try again. | 21:48 |
| *** zul has quit IRC | 21:49 | |
| jungleboyj | Ah, I see it now. It was because I had another pep8 fail with run_tests.sh that it didn't continue to run your code. All good. | 21:50 |
| *** jgregor has quit IRC | 21:51 | |
| *** jgregor has joined #openstack-cinder | 21:51 | |
| *** garthb has quit IRC | 21:55 | |
| *** jgregor has quit IRC | 21:55 | |
| *** dannywil_ has joined #openstack-cinder | 21:57 | |
| *** dannywil_ has quit IRC | 21:58 | |
| *** dannywil_ has joined #openstack-cinder | 21:58 | |
| *** martyturner has joined #openstack-cinder | 22:00 | |
| *** dannywilson has quit IRC | 22:01 | |
| *** dannywil_ has quit IRC | 22:02 | |
| *** garthb has joined #openstack-cinder | 22:03 | |
| *** garthb has quit IRC | 22:03 | |
| *** garthb has joined #openstack-cinder | 22:03 | |
| *** jgregor has joined #openstack-cinder | 22:04 | |
| *** jungleboyj has quit IRC | 22:05 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Better exception handling in init_conn https://review.openstack.org/214316 | 22:07 |
| *** andreykurilin__ has joined #openstack-cinder | 22:08 | |
| *** gouthamr has joined #openstack-cinder | 22:14 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Tests: Fix zfssa TestRestClientURL.test_request https://review.openstack.org/214368 | 22:16 |
| *** annegentle has quit IRC | 22:17 | |
| hodos|2 | jgriffith: hemna: you were right guys, os-brick fix helps. just for some reason LIBS_FROM_GIT=os-brick didn't install latest | 22:20 |
| hemna | hodos|2: that’s good to know. :) | 22:21 |
| hemna | hodos|2: did you get /opt/stack/os-brick ? | 22:21 |
| *** jgregor has quit IRC | 22:22 | |
| hodos|2 | hemna: yes it was there | 22:22 |
| *** bswartz has joined #openstack-cinder | 22:22 | |
| hodos|2 | hemna: but when i checked the code the fix wasn't rhe | 22:22 |
| hemna | hrmm | 22:22 |
| hodos|2 | hemna: so i installed it from https://github.com/openstack/os-brick | 22:23 |
| smcginnis | Does os-brick need to be in ENABLED_SERVICES/ | 22:27 |
| Swanson | It isn't a service, is it? | 22:29 |
| hemna | smcginnis: no, it’s in nova and cinder’s requirements.txt | 22:29 |
| *** dflorea has joined #openstack-cinder | 22:29 | |
| *** niknakpaddywak has joined #openstack-cinder | 22:35 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 22:38 |
| openstackgerrit | John Griffith proposed openstack/cinder: Cinder replication V2 https://review.openstack.org/200269 | 22:45 |
| *** hodos|2 has quit IRC | 22:45 | |
| *** cbader has joined #openstack-cinder | 22:47 | |
| *** eharney has quit IRC | 22:47 | |
| *** cbader has quit IRC | 22:53 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port image_utils to Python 3 https://review.openstack.org/213817 | 22:55 |
| *** jungleboyj has joined #openstack-cinder | 22:56 | |
| *** angela-s has quit IRC | 22:57 | |
| *** dflorea has quit IRC | 23:00 | |
| *** dflorea has joined #openstack-cinder | 23:01 | |
| *** cbader has joined #openstack-cinder | 23:02 | |
| hemna | patrickeast: ping | 23:06 |
| *** annegentle has joined #openstack-cinder | 23:08 | |
| *** annegentle has quit IRC | 23:09 | |
| openstackgerrit | John Griffith proposed openstack/cinder: Update devref unit tests doc https://review.openstack.org/214376 | 23:09 |
| *** annegentle has joined #openstack-cinder | 23:10 | |
| *** dannywilson has joined #openstack-cinder | 23:10 | |
| *** dflorea has quit IRC | 23:13 | |
| *** ebalduf has quit IRC | 23:18 | |
| *** dims_ has joined #openstack-cinder | 23:20 | |
| *** dims has quit IRC | 23:22 | |
| patrickeast | hemna: pong | 23:23 |
| hemna | patrickeast: any luck messing aroudn with SR-IOV yet? | 23:23 |
| hemna | I’m hacking on my blade to see if I can enable it | 23:23 |
| hemna | looks like the card supports it, just isn’t enabled | 23:23 |
| patrickeast | hemna: nah i haven't even got an order for an HBA that can support it | 23:24 |
| patrickeast | hemna: ooo sick | 23:24 |
| patrickeast | hemna: ive still got it on my backlog... somewhere... | 23:24 |
| hemna | it’s a bunch of trickery | 23:24 |
| hemna | it’ll be a miracle if it even works | 23:24 |
| patrickeast | lol | 23:24 |
| hemna | I think my card supports 64 VF’s per port | 23:25 |
| patrickeast | nice, that should be a pretty big boost in testing throughput | 23:25 |
| patrickeast | i remember looking through some of the instructions to do it and the commands to configure them were not very straight forward | 23:26 |
| *** lcurtis has quit IRC | 23:26 | |
| *** dannywilson has quit IRC | 23:28 | |
| *** cbader has quit IRC | 23:28 | |
| *** chlong has quit IRC | 23:29 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_nfs to Python 3 https://review.openstack.org/214384 | 23:30 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Adding Consistency Group Support https://review.openstack.org/214386 | 23:31 |
| *** changbl has joined #openstack-cinder | 23:31 | |
| *** vilobhmm has quit IRC | 23:32 | |
| hemna | patrickeast: yah it’s not obvious. basically lspci -vvv | 23:33 |
| hemna | if you don’t do the 3 v’s you don’t get to see it | 23:34 |
| hemna | patrickeast: http://pastebin.com/wDbBsyiv | 23:34 |
| hemna | VFs of 0 | 23:35 |
| hemna | basically means the card is capable, but it’s disabled in BIOS | 23:35 |
| hemna | bleh | 23:35 |
| patrickeast | ahh yea i remember seeing something about that | 23:36 |
| patrickeast | i figured out how to do it for the supermicro initiators i've got | 23:36 |
| patrickeast | just a toggle on kind of deal | 23:36 |
| patrickeast | hemna: i had this page bookmarked, https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/6/html/Virtualization_Host_Configuration_and_Guest_Installation_Guide/sect-Virtualization_Host_Configuration_and_Guest_Installation_Guide-SR_IOV-How_SR_IOV_Libvirt_Works.html | 23:38 |
| hemna | :) | 23:38 |
| patrickeast | i assume its a similar process with a FC device | 23:38 |
| hemna | yah it is | 23:38 |
| hemna | the emulex driver pdf talks about it as well | 23:38 |
| hemna | http://www-dl.emulex.com/support/elx/rt10.6.1/10.6.144.24/Docs/final/linux/linux_manual_elx.pdf | 23:39 |
| hemna | Page 42, setting up SR-IOV | 23:39 |
| hemna | patrickeast: also FWIW, I have a WIP patch in os-brick that you might want to take a look at when you have time. | 23:39 |
| hemna | https://review.openstack.org/#/c/213389/ | 23:39 |
| hemna | basically changes the mechanism for discovering multipath devices for FC (and eventually iSCSI) | 23:40 |
| hemna | your CI passed against it :P | 23:40 |
| patrickeast | oo neat | 23:40 |
| patrickeast | i mean, if the ci passes that means its ready to ship, right? | 23:40 |
| hemna | yah! | 23:41 |
| hemna | ship it! | 23:41 |
| hemna | http://goo.gl/Sbihmd | 23:41 |
| hemna | we have a customer that has hundreds of mpath volumes | 23:41 |
| hemna | on the same box | 23:41 |
| hemna | and running multipath -ll <path> gets REALLY slow | 23:41 |
| patrickeast | ahh yea i bet | 23:41 |
| hemna | like exponentially slower to run on every attach | 23:41 |
| patrickeast | ouch | 23:41 |
| hemna | so my patch avoids even calling multipath -ll at all | 23:42 |
| hemna | I’d like to get some other eyes on it to see if what I’m doing is sane | 23:42 |
| *** gouthamr_ has joined #openstack-cinder | 23:42 | |
| hemna | I’ve done a bunch of digging on how multipath daemon creates it’s devices and it’s multipath id’s | 23:42 |
| patrickeast | at first glance it seems like a pretty solid win, i'll take a closer look and play around with it later today or tomorrow | 23:43 |
| hemna | the multipath id is nothing more than the volume WWN which is at page 0x83 of every volume. | 23:43 |
| patrickeast | awesome, i had wondered about that | 23:43 |
| hemna | so I read that page, to get the WWN, and then look for /dev/mapper/<WWN> to show up. | 23:43 |
| hemna | that path shows up after the multipath daemon creates the mpath device | 23:43 |
| * hemna still waits for HP CI on that patch...... | 23:44 | |
| patrickeast | haha | 23:44 |
| *** gouthamr has quit IRC | 23:44 | |
| patrickeast | need to get that sr-iov stuff working! | 23:44 |
| *** gouthamr has joined #openstack-cinder | 23:45 | |
| hemna | as a side effect of this patch, I’ll be able to get rid of all the multipath_id pooping all over nova | 23:45 |
| hemna | so it will clean up nova a bit, which will also help live migration as well. | 23:45 |
| hemna | so I’m hoping this is the ticket. | 23:45 |
| *** gouthamr_ has quit IRC | 23:47 | |
| *** diogogmt has quit IRC | 23:48 | |
| *** ndipanov has joined #openstack-cinder | 23:50 | |
| patrickeast | hemna: so with that change what happens if they have user_friendly_names set? i thought that changed the /dev/mapper/ entry to something like /dev/mapper/mpath1 | 23:51 |
| patrickeast | would we just say that for openstack you can't enable that? | 23:52 |
| *** ndipanov has quit IRC | 23:52 | |
| *** ndipanov has joined #openstack-cinder | 23:52 | |
| patrickeast | i got burned by this once copying a multipath.conf file that had it enabled and broke some scripts : / | 23:52 |
| openstackgerrit | Kurt Martin proposed openstack/cinder: LeftHand driver is ignoring reserved_percentage https://review.openstack.org/214390 | 23:56 |
| hemna | does that change the dev/mapper path it creates? | 23:58 |
| patrickeast | hemna: check out the first section here https://help.ubuntu.com/lts/serverguide/multipath-devices.html | 23:59 |
| hemna | I’ll have to enable that and see how much stuff I break | 23:59 |
| patrickeast | explains better than i can | 23:59 |
| *** tiny-hands has joined #openstack-cinder | 23:59 | |
| *** ndipanov has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
