| *** Julien-zte has joined #openstack-cinder | 00:02 | |
| *** akapil has quit IRC | 00:02 | |
| *** merooney_ has joined #openstack-cinder | 00:06 | |
| *** yangyapeng has quit IRC | 00:08 | |
| *** akapil has joined #openstack-cinder | 00:08 | |
| *** merooney has quit IRC | 00:09 | |
| *** alonma has joined #openstack-cinder | 00:12 | |
| *** arch-nemesis has quit IRC | 00:14 | |
| *** alonma has quit IRC | 00:16 | |
| *** mtanino has quit IRC | 00:18 | |
| *** alonma has joined #openstack-cinder | 00:19 | |
| *** Julien-zte has quit IRC | 00:23 | |
| *** alonma has quit IRC | 00:24 | |
| *** Julien-zte has joined #openstack-cinder | 00:25 | |
| *** alonma has joined #openstack-cinder | 00:26 | |
| *** Julien-zte has quit IRC | 00:26 | |
| *** diogogmt has quit IRC | 00:28 | |
| *** alonma has quit IRC | 00:30 | |
| *** GB21 has joined #openstack-cinder | 00:31 | |
| *** mriedem has joined #openstack-cinder | 00:35 | |
| *** thumpba_ has quit IRC | 00:39 | |
| *** thumpba has joined #openstack-cinder | 00:39 | |
| *** akapil_ has joined #openstack-cinder | 00:43 | |
| *** thumpba has quit IRC | 00:44 | |
| *** akapil has quit IRC | 00:46 | |
| *** alonma has joined #openstack-cinder | 00:47 | |
| *** alonma has quit IRC | 00:51 | |
| *** Suyi has quit IRC | 00:55 | |
| *** alonma has joined #openstack-cinder | 00:55 | |
| *** chlong has joined #openstack-cinder | 00:58 | |
| *** alonma has quit IRC | 00:59 | |
| *** cheneydc has joined #openstack-cinder | 00:59 | |
| *** alonma has joined #openstack-cinder | 01:02 | |
| *** raunak has joined #openstack-cinder | 01:05 | |
| raunak | Hi Sean, I tried all the skipped tests and had a successful run. Would upload it as a comment on openstack review until our logs acess are been worked on | 01:05 |
|---|---|---|
| *** alonma has quit IRC | 01:06 | |
| *** alonma has joined #openstack-cinder | 01:09 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 01:11 |
| *** leeantho has quit IRC | 01:12 | |
| *** Apoorva has quit IRC | 01:13 | |
| *** alonma has quit IRC | 01:14 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 01:14 |
| *** alonma has joined #openstack-cinder | 01:16 | |
| openstackgerrit | xing-yang proposed openstack/cinder: WIP: Rename functions names involving groups. https://review.openstack.org/355961 | 01:19 |
| *** cheneydc has quit IRC | 01:19 | |
| *** yangyapeng has joined #openstack-cinder | 01:20 | |
| *** alonma has quit IRC | 01:20 | |
| *** Apoorva has joined #openstack-cinder | 01:21 | |
| *** alonma has joined #openstack-cinder | 01:23 | |
| *** diogogmt has joined #openstack-cinder | 01:25 | |
| *** Asaithambi has joined #openstack-cinder | 01:27 | |
| *** alonma has quit IRC | 01:28 | |
| *** akapil_ has quit IRC | 01:31 | |
| *** jungleboyj has joined #openstack-cinder | 01:32 | |
| *** akapil has joined #openstack-cinder | 01:34 | |
| *** ducttape_ has quit IRC | 01:34 | |
| *** ducttape_ has joined #openstack-cinder | 01:35 | |
| *** GB21 has quit IRC | 01:35 | |
| *** Asaithambi has quit IRC | 01:38 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_volume.py https://review.openstack.org/355815 | 01:42 |
| *** EinstCrazy has joined #openstack-cinder | 01:42 | |
| *** lixiaoy1_ has quit IRC | 01:45 | |
| *** julim has joined #openstack-cinder | 01:46 | |
| *** cknight has joined #openstack-cinder | 01:46 | |
| openstackgerrit | xiexs proposed openstack/cinder: Refactor the using of dict.get() in the test assertion https://review.openstack.org/355323 | 01:50 |
| *** raunak has quit IRC | 01:55 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in test_quota.py https://review.openstack.org/356174 | 01:55 |
| *** bardia_ has quit IRC | 01:55 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_quota.py https://review.openstack.org/356174 | 01:58 |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/test_qos_specs.py https://review.openstack.org/355766 | 01:59 |
| *** EinstCrazy has quit IRC | 02:01 | |
| openstackgerrit | Xu Ao proposed openstack/cinder: Change a typo error in the releasenote https://review.openstack.org/356176 | 02:02 |
| *** EinstCrazy has joined #openstack-cinder | 02:03 | |
| openstackgerrit | Sean McGinnis proposed openstack/cinder: Expand description of test types https://review.openstack.org/356177 | 02:07 |
| *** sdake has quit IRC | 02:13 | |
| *** sdake has joined #openstack-cinder | 02:13 | |
| *** yangyapeng has quit IRC | 02:14 | |
| *** yangyapeng has joined #openstack-cinder | 02:14 | |
| *** EinstCra_ has joined #openstack-cinder | 02:16 | |
| *** GB21 has joined #openstack-cinder | 02:16 | |
| *** julim has quit IRC | 02:17 | |
| *** jungleboyj has quit IRC | 02:18 | |
| openstackgerrit | xiexs proposed openstack/cinder: Refactor the using of dict.get() in the test assertion https://review.openstack.org/355323 | 02:18 |
| *** EinstCrazy has quit IRC | 02:18 | |
| *** Lee1092 has joined #openstack-cinder | 02:19 | |
| *** bkopilov has quit IRC | 02:21 | |
| *** lixiaoy1 has joined #openstack-cinder | 02:25 | |
| openstackgerrit | Dao Cong Tien proposed openstack/cinder: Stop using mox in unit/api/v1/ tests https://review.openstack.org/356185 | 02:32 |
| *** cknight1 has joined #openstack-cinder | 02:33 | |
| *** cknight has quit IRC | 02:33 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Switch service capabilities to ovo https://review.openstack.org/319040 | 02:34 |
| *** zhongjun_ has quit IRC | 02:34 | |
| *** Apoorva_ has joined #openstack-cinder | 02:37 | |
| openstackgerrit | Merged openstack/cinder: Add CI_WIKI_NAME to driver's devref https://review.openstack.org/355862 | 02:37 |
| openstackgerrit | Xu Ao proposed openstack/cinder: Change a typo error in the releasenote https://review.openstack.org/356186 | 02:38 |
| *** Apoorva has quit IRC | 02:40 | |
| *** Apoorva_ has quit IRC | 02:41 | |
| *** haplo37_ has quit IRC | 02:43 | |
| *** akapil has quit IRC | 02:44 | |
| *** GB21 has quit IRC | 02:44 | |
| lixiaoy1 | Hi, I see several tempest tests in tempest.api.image.v1.test_images.CreateRegisterImagesTest.test_register_http_image, http://logs.openstack.org/60/317060/11/check/gate-tempest-dsvm-full-devstack-plugin-ceph/1f34c49/console.html#_2016-08-16_20_44_58_262300 . is it a known issue? Anyone is working on it? | 02:46 |
| *** ducttape_ has quit IRC | 02:49 | |
| *** alonma has joined #openstack-cinder | 02:49 | |
| *** raunak has joined #openstack-cinder | 02:51 | |
| *** akapil has joined #openstack-cinder | 02:51 | |
| *** alonma has quit IRC | 02:54 | |
| *** zhongjun_ has joined #openstack-cinder | 02:55 | |
| *** EinstCrazy has joined #openstack-cinder | 02:56 | |
| *** alonma has joined #openstack-cinder | 02:57 | |
| openstackgerrit | xiexs proposed openstack/cinder: Refactor the case test_storwize_vdisk_copy_ops https://review.openstack.org/356194 | 02:57 |
| *** EinstCra_ has quit IRC | 03:00 | |
| *** alonma has quit IRC | 03:01 | |
| *** alonma has joined #openstack-cinder | 03:09 | |
| *** mriedem has quit IRC | 03:11 | |
| *** ducttape_ has joined #openstack-cinder | 03:13 | |
| *** sticker has quit IRC | 03:13 | |
| *** alonma has quit IRC | 03:14 | |
| *** thumpba has joined #openstack-cinder | 03:19 | |
| openstackgerrit | xiexs proposed openstack/python-cinderclient: Add --is-public option to type-list https://review.openstack.org/335881 | 03:20 |
| *** alonma has joined #openstack-cinder | 03:21 | |
| *** yuelongguang has quit IRC | 03:21 | |
| *** alonma has quit IRC | 03:25 | |
| *** bswartz has joined #openstack-cinder | 03:26 | |
| *** alonma has joined #openstack-cinder | 03:28 | |
| *** diogogmt has quit IRC | 03:28 | |
| *** alonma has quit IRC | 03:32 | |
| *** sdake has quit IRC | 03:34 | |
| *** alonma has joined #openstack-cinder | 03:35 | |
| *** akapil has quit IRC | 03:35 | |
| *** ducttape_ has quit IRC | 03:36 | |
| *** bkopilov has joined #openstack-cinder | 03:37 | |
| *** yangyape_ has joined #openstack-cinder | 03:38 | |
| *** yangyapeng has quit IRC | 03:38 | |
| *** alonma has quit IRC | 03:40 | |
| *** alonma has joined #openstack-cinder | 03:42 | |
| *** akapil has joined #openstack-cinder | 03:43 | |
| *** alonma has quit IRC | 03:47 | |
| *** raunak has quit IRC | 03:52 | |
| *** raunak has joined #openstack-cinder | 03:53 | |
| *** links has joined #openstack-cinder | 03:54 | |
| *** alonma has joined #openstack-cinder | 03:55 | |
| openstackgerrit | xiexs proposed openstack/cinder: Add validation for the metadata properties https://review.openstack.org/330913 | 03:56 |
| openstackgerrit | xiexs proposed openstack/cinder: Convert InvalidVolumeMetadataSize to webob.exc.* https://review.openstack.org/356213 | 03:56 |
| *** alonma has quit IRC | 03:59 | |
| *** Asaithambi has joined #openstack-cinder | 04:00 | |
| *** Asaithambi has quit IRC | 04:02 | |
| *** Asaithambi has joined #openstack-cinder | 04:03 | |
| *** coolsvap has joined #openstack-cinder | 04:04 | |
| *** GB21 has joined #openstack-cinder | 04:06 | |
| *** merooney_ has quit IRC | 04:06 | |
| *** Asaithambi has quit IRC | 04:09 | |
| *** Asaithambi has joined #openstack-cinder | 04:12 | |
| *** bardia has joined #openstack-cinder | 04:14 | |
| openstackgerrit | Anh Tran proposed openstack/cinder: api-ref: Fix parameters attributes https://review.openstack.org/356216 | 04:17 |
| *** bardia has quit IRC | 04:18 | |
| *** GB21 has quit IRC | 04:23 | |
| *** raunak has quit IRC | 04:23 | |
| *** bswartz has quit IRC | 04:27 | |
| *** cknight1 has quit IRC | 04:28 | |
| *** sticker has joined #openstack-cinder | 04:29 | |
| openstackgerrit | Guy Kim proposed openstack/cinder: ITRI DISCO cinder volume driver https://review.openstack.org/347100 | 04:31 |
| *** raunak has joined #openstack-cinder | 04:34 | |
| *** alonma has joined #openstack-cinder | 04:35 | |
| *** ducttape_ has joined #openstack-cinder | 04:36 | |
| *** alonma has quit IRC | 04:40 | |
| *** ducttape_ has quit IRC | 04:41 | |
| *** alonma has joined #openstack-cinder | 04:43 | |
| *** laughterwym has joined #openstack-cinder | 04:45 | |
| *** alonma has quit IRC | 04:47 | |
| openstackgerrit | Soffie Huang proposed openstack/cinder: Refactoring FalconStor FSS storage driver https://review.openstack.org/355836 | 04:48 |
| *** lkuchlan has joined #openstack-cinder | 04:52 | |
| *** GB21 has joined #openstack-cinder | 04:54 | |
| *** ChubYann has quit IRC | 04:54 | |
| *** pbandark has joined #openstack-cinder | 04:55 | |
| *** narayrak has joined #openstack-cinder | 05:00 | |
| *** ChubYann has joined #openstack-cinder | 05:06 | |
| *** dramakri has quit IRC | 05:18 | |
| *** akapil has quit IRC | 05:21 | |
| *** akapil has joined #openstack-cinder | 05:23 | |
| *** alonma has joined #openstack-cinder | 05:34 | |
| *** gcb_ has joined #openstack-cinder | 05:35 | |
| *** sandanar has joined #openstack-cinder | 05:37 | |
| openstackgerrit | xiexs proposed openstack/cinder: Refactor the case test_storwize_vdisk_copy_ops https://review.openstack.org/356194 | 05:38 |
| *** ducttape_ has joined #openstack-cinder | 05:38 | |
| *** alonma has quit IRC | 05:39 | |
| *** liverpooler has joined #openstack-cinder | 05:40 | |
| *** thumpba has quit IRC | 05:40 | |
| *** Poornima has joined #openstack-cinder | 05:41 | |
| *** alonma has joined #openstack-cinder | 05:41 | |
| *** ducttape_ has quit IRC | 05:43 | |
| *** alonma has quit IRC | 05:46 | |
| *** rcernin has joined #openstack-cinder | 05:46 | |
| *** alonma has joined #openstack-cinder | 05:49 | |
| *** thumpba has joined #openstack-cinder | 05:51 | |
| *** yangyape_ has quit IRC | 05:51 | |
| *** yangyapeng has joined #openstack-cinder | 05:52 | |
| *** alonma has quit IRC | 05:53 | |
| *** yangyape_ has joined #openstack-cinder | 05:54 | |
| *** yangyapeng has quit IRC | 05:54 | |
| *** zhangguoqing has joined #openstack-cinder | 05:54 | |
| zhangguoqing | Hi, I get a volumes list by cinderclient, but I can not get the user_id (volume.user_id) of anyone volume. | 05:55 |
| zhangguoqing | how to get the volume's user_id? | 05:56 |
| *** thumpba has quit IRC | 05:56 | |
| *** alonma has joined #openstack-cinder | 05:56 | |
| zhangguoqing | It's importmant for me, who can help me? thanks. :) | 05:57 |
| zhangguoqing | (Pdb) dir(volume) | 06:00 |
| zhangguoqing | ['HUMAN_ID', 'NAME_ATTR', '__class__', '__delattr__', '__dict__', '__doc__', '__eq__', '__format__', '__getattr__', '__getattribute__', '__hash__', '__init__', '__module__', '__new__', '__reduce__', '__reduce_ex__', '__repr__', '__setattr__', '__sizeof__', '__str__', '__subclasshook__', '__weakref__', '_add_details', '_append_request_id', '_info', '_loaded', 'append_request_ids', | 06:00 |
| zhangguoqing | 'attach', 'attachments', 'availability_zone', 'begin_detaching', 'bootable', 'created_at', 'delete', 'delete_image_metadata', 'detach', 'display_description', 'display_name', 'encrypted', 'extend', 'force_delete', 'get', 'get_pools', 'human_id', 'id', 'initialize_connection', 'is_loaded', 'manage', 'manager', 'metadata', 'migrate_volume', 'multiattach', 'os-vol-host-attr:host', | 06:00 |
| zhangguoqing | 'os-vol-mig-status-attr:migstat', 'os-vol-mig-status-attr:name_id', 'os-vol-tenant-attr:tenant_id', 'promote', 'reenable', 'request_ids', 'reserve', 'reset_state', 'retype', 'roll_detaching', 'set_image_metadata', 'set_loaded', 'set_metadata', 'setup', 'show_image_metadata', 'size', 'snapshot_id', 'source_volid', 'status', 'terminate_connection', 'to_dict', 'unmanage', 'unreserve', | 06:00 |
| zhangguoqing | 'update', 'update_all_metadata', 'update_readonly_flag', 'upload_to_image', 'volume_type', 'x_openstack_request_ids'] | 06:00 |
| *** alonma has quit IRC | 06:01 | |
| *** alonma has joined #openstack-cinder | 06:03 | |
| lixiaoy1 | zhangguoqing: seems summary result doesn't include user_id, you can check the detailed info of a volume which includes user_id | 06:06 |
| *** akapil has quit IRC | 06:06 | |
| *** akapil has joined #openstack-cinder | 06:07 | |
| *** alonma has quit IRC | 06:08 | |
| zhangguoqing | yes, I have added the detailed=true. | 06:08 |
| zhangguoqing | and I find a strange that it cant get user_id only in cinderclient in python shell env. | 06:09 |
| lixiaoy1 | even you set detailed=true? | 06:09 |
| zhangguoqing | then I use cinderclient in ceilometer, I can not obtain the volume.user_id | 06:09 |
| zhangguoqing | yes, i'm sure. | 06:10 |
| zhangguoqing | from cinderclient import client | 06:12 |
| zhangguoqing | cinder = client.Client('2', 'admin', 'password', 'admin', 'http://172.16.40.6:5000/v2.0') | 06:12 |
| zhangguoqing | search_opts = {'all_tenants': 1} | 06:12 |
| zhangguoqing | volumes = cinder.volumes.list(detailed=True, search_opts=search_opts) | 06:12 |
| zhangguoqing | volumes[0].user_id | 06:12 |
| lixiaoy1 | I think you can check the url it accessed through this call | 06:14 |
| zhangguoqing | ok, thank you. i will do more tests. :) | 06:15 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/356248 | 06:15 |
| *** alonma has joined #openstack-cinder | 06:16 | |
| *** laughterwym has quit IRC | 06:18 | |
| *** laughterwym has joined #openstack-cinder | 06:20 | |
| *** alonma has quit IRC | 06:21 | |
| *** ebalduf has quit IRC | 06:21 | |
| *** alonma has joined #openstack-cinder | 06:24 | |
| *** laughterwym has quit IRC | 06:25 | |
| *** alonma has quit IRC | 06:28 | |
| *** e0ne has joined #openstack-cinder | 06:30 | |
| *** alonma has joined #openstack-cinder | 06:31 | |
| *** sofie has joined #openstack-cinder | 06:32 | |
| *** sandanar_ has joined #openstack-cinder | 06:32 | |
| *** sandanar has quit IRC | 06:35 | |
| *** alonma has quit IRC | 06:36 | |
| *** akapil_ has joined #openstack-cinder | 06:38 | |
| *** alonma has joined #openstack-cinder | 06:38 | |
| *** arecknag has joined #openstack-cinder | 06:39 | |
| *** ducttape_ has joined #openstack-cinder | 06:39 | |
| *** lkuchlan has quit IRC | 06:40 | |
| *** raunak has quit IRC | 06:41 | |
| *** akapil has quit IRC | 06:41 | |
| *** alonma has quit IRC | 06:43 | |
| *** sofie has quit IRC | 06:44 | |
| *** pgadiya has joined #openstack-cinder | 06:44 | |
| *** ducttape_ has quit IRC | 06:45 | |
| *** alonma has joined #openstack-cinder | 06:46 | |
| *** akapil_ has quit IRC | 06:48 | |
| *** raunak has joined #openstack-cinder | 06:49 | |
| *** alonma has quit IRC | 06:50 | |
| *** akapil has joined #openstack-cinder | 06:52 | |
| *** belmoreira has joined #openstack-cinder | 06:53 | |
| *** raunak has quit IRC | 06:55 | |
| *** alonma has joined #openstack-cinder | 06:56 | |
| *** sandanar__ has joined #openstack-cinder | 06:59 | |
| *** gouthamr has quit IRC | 07:00 | |
| *** markvoelker has quit IRC | 07:00 | |
| *** tesseract- has joined #openstack-cinder | 07:00 | |
| *** alonma has quit IRC | 07:00 | |
| *** sandanar_ has quit IRC | 07:02 | |
| *** raunak has joined #openstack-cinder | 07:03 | |
| *** alonma has joined #openstack-cinder | 07:03 | |
| *** lpetrut has joined #openstack-cinder | 07:04 | |
| *** akapil has quit IRC | 07:05 | |
| *** akapil has joined #openstack-cinder | 07:05 | |
| *** alonma has quit IRC | 07:07 | |
| *** raunak has quit IRC | 07:09 | |
| *** GB21 has quit IRC | 07:10 | |
| *** akapil has quit IRC | 07:10 | |
| *** alonma has joined #openstack-cinder | 07:10 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 07:14 |
| *** alonma has quit IRC | 07:14 | |
| *** alonma has joined #openstack-cinder | 07:17 | |
| *** raunak has joined #openstack-cinder | 07:21 | |
| *** alonma has quit IRC | 07:22 | |
| *** alonma has joined #openstack-cinder | 07:25 | |
| *** andymaier has joined #openstack-cinder | 07:26 | |
| *** lpetrut has quit IRC | 07:29 | |
| *** alonma has quit IRC | 07:29 | |
| *** aohuanxuan has joined #openstack-cinder | 07:29 | |
| *** alonma has joined #openstack-cinder | 07:32 | |
| openstackgerrit | yuyafei proposed openstack/os-brick: Multipath device keeps old size when extending volume https://review.openstack.org/353325 | 07:36 |
| *** alonma has quit IRC | 07:36 | |
| *** alonma has joined #openstack-cinder | 07:39 | |
| *** lkuchlan has joined #openstack-cinder | 07:43 | |
| *** alonma has quit IRC | 07:44 | |
| *** GB21 has joined #openstack-cinder | 07:44 | |
| *** jamielennox is now known as jamielennox|away | 07:45 | |
| *** raunak has quit IRC | 07:45 | |
| *** alonma has joined #openstack-cinder | 07:46 | |
| *** sumit has joined #openstack-cinder | 07:46 | |
| *** alonma has quit IRC | 07:51 | |
| *** e0ne has quit IRC | 07:51 | |
| *** Dinesh_Bhor has joined #openstack-cinder | 07:52 | |
| *** thumpba has joined #openstack-cinder | 07:52 | |
| *** raunak has joined #openstack-cinder | 07:53 | |
| *** thumpba has quit IRC | 07:57 | |
| *** Asaithambi has quit IRC | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** markvoelker has joined #openstack-cinder | 08:00 | |
| *** thumpba has joined #openstack-cinder | 08:01 | |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** alonma has joined #openstack-cinder | 08:01 | |
| *** markvoelker has quit IRC | 08:05 | |
| *** alonma has quit IRC | 08:06 | |
| *** thumpba has quit IRC | 08:06 | |
| *** akapil has joined #openstack-cinder | 08:06 | |
| *** ducttape_ has joined #openstack-cinder | 08:08 | |
| *** alonma has joined #openstack-cinder | 08:08 | |
| *** coolsvap is now known as coolsvap_ | 08:09 | |
| *** raunak has quit IRC | 08:09 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** ducttape_ has quit IRC | 08:12 | |
| *** alonma has quit IRC | 08:13 | |
| *** raunak has joined #openstack-cinder | 08:14 | |
| *** alonma has joined #openstack-cinder | 08:16 | |
| *** belmoreira has quit IRC | 08:18 | |
| *** e0ne has joined #openstack-cinder | 08:19 | |
| *** alonma has quit IRC | 08:20 | |
| *** alonma has joined #openstack-cinder | 08:23 | |
| *** alonma has quit IRC | 08:27 | |
| *** alonma has joined #openstack-cinder | 08:30 | |
| *** alonma has quit IRC | 08:30 | |
| *** alonma has joined #openstack-cinder | 08:30 | |
| *** Asaithambi has joined #openstack-cinder | 08:32 | |
| *** dkehn_ has quit IRC | 08:32 | |
| *** smile-luobin has left #openstack-cinder | 08:33 | |
| *** Jaison has joined #openstack-cinder | 08:35 | |
| *** links has quit IRC | 08:37 | |
| openstackgerrit | Gorka Eguileor proposed openstack/os-brick: Fix cmd execution stderr, stdout unicode errors https://review.openstack.org/341085 | 08:43 |
| *** jamielennox|away is now known as jamielennox | 08:50 | |
| *** Asaithambi has quit IRC | 08:50 | |
| *** Asaithambi has joined #openstack-cinder | 08:50 | |
| *** dkehn_ has joined #openstack-cinder | 08:52 | |
| *** GB21 has quit IRC | 08:52 | |
| *** lixiaoy1 has quit IRC | 08:59 | |
| *** raunak has quit IRC | 08:59 | |
| *** markvoelker has joined #openstack-cinder | 09:01 | |
| *** thumpba has joined #openstack-cinder | 09:02 | |
| *** markvoelker has quit IRC | 09:06 | |
| *** GB21 has joined #openstack-cinder | 09:06 | |
| *** thumpba has quit IRC | 09:08 | |
| *** ducttape_ has joined #openstack-cinder | 09:08 | |
| openstackgerrit | YanLin Ren proposed openstack/cinder: Volume Manage/Unmanage Support for IBM FlashSystem https://review.openstack.org/353856 | 09:09 |
| *** raunak has joined #openstack-cinder | 09:10 | |
| *** ducttape_ has quit IRC | 09:13 | |
| *** raunak has quit IRC | 09:15 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 09:16 |
| xiexs | Is gate-cinder-docs-ubuntu-xenial broken? | 09:19 |
| *** gus has quit IRC | 09:22 | |
| *** andymaier has quit IRC | 09:23 | |
| *** alonma has quit IRC | 09:27 | |
| *** lpetrut has joined #openstack-cinder | 09:27 | |
| *** alonma has joined #openstack-cinder | 09:27 | |
| *** Digvijay has joined #openstack-cinder | 09:28 | |
| *** maestropandy has joined #openstack-cinder | 09:29 | |
| *** raunak has joined #openstack-cinder | 09:30 | |
| *** alonma has quit IRC | 09:32 | |
| *** digu has joined #openstack-cinder | 09:33 | |
| *** alonma has joined #openstack-cinder | 09:34 | |
| *** mvk has quit IRC | 09:35 | |
| *** Digvijay has quit IRC | 09:37 | |
| zhangguoqing | lixiaoy1: I get the volume's user_id after I change the server type of cinder to volumev2. | 09:38 |
| *** alonma has quit IRC | 09:39 | |
| zhangguoqing | but I can not obtain the user_id of volume's snapshot. | 09:39 |
| zhangguoqing | and cinder snapshot-show volume-snapshot also can not see the user_id | 09:40 |
| zhangguoqing | however, I can see the user_id in database table. | 09:41 |
| zhangguoqing | So, how to get the user_id of a volume's snapshot? | 09:42 |
| *** alonma has joined #openstack-cinder | 09:42 | |
| *** jamielennox is now known as jamielennox|away | 09:43 | |
| *** alonma has quit IRC | 09:46 | |
| *** yangyapeng has joined #openstack-cinder | 09:48 | |
| *** alonma has joined #openstack-cinder | 09:49 | |
| *** yangyape_ has quit IRC | 09:50 | |
| *** alonma has quit IRC | 09:53 | |
| *** andymaier has joined #openstack-cinder | 09:53 | |
| *** yangyape_ has joined #openstack-cinder | 09:55 | |
| *** yangyapeng has quit IRC | 09:55 | |
| *** lpetrut1 has joined #openstack-cinder | 09:57 | |
| *** openstack has joined #openstack-cinder | 10:18 | |
| *** sdake_ has quit IRC | 10:18 | |
| *** e0ne has quit IRC | 10:18 | |
| *** sdake_ has joined #openstack-cinder | 10:19 | |
| *** sdake has quit IRC | 10:19 | |
| *** Poornima has quit IRC | 10:19 | |
| openstackgerrit | Tuan proposed openstack/cinder: Stop using mox in cinder/tests/unit https://review.openstack.org/356358 | 10:20 |
| *** yangyape_ has quit IRC | 10:21 | |
| *** dkehn_ has joined #openstack-cinder | 10:22 | |
| *** sdague has joined #openstack-cinder | 10:27 | |
| openstackgerrit | Avishay Traeger proposed openstack/python-cinderclient: List manageable volumes and snapshots https://review.openstack.org/351275 | 10:33 |
| *** EinstCrazy has quit IRC | 10:40 | |
| *** raunak has quit IRC | 10:42 | |
| ArsenC | Hi cinder team, please help review https://review.openstack.org/#/c/341345/ . Thanks~ | 10:42 |
| openstackgerrit | Ellen Leahy proposed openstack/python-cinderclient: Changed backup-restore to accept backup name https://review.openstack.org/352778 | 10:44 |
| *** ociuhandu has quit IRC | 10:50 | |
| openstackgerrit | Hussain Chachuliya proposed openstack/cinder: Use %()d for integer substitution https://review.openstack.org/356380 | 10:50 |
| *** mvk has quit IRC | 10:58 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder: VMAX Driver - Live Migration for VMAX3 https://review.openstack.org/330778 | 10:59 |
| *** vladani has quit IRC | 11:00 | |
| *** andymaier has quit IRC | 11:00 | |
| openstackgerrit | Hussain Chachuliya proposed openstack/cinder: Use min attribute from oslo_config https://review.openstack.org/356388 | 11:04 |
| *** avishay has joined #openstack-cinder | 11:09 | |
| *** ducttape_ has joined #openstack-cinder | 11:10 | |
| *** ducttape_ has quit IRC | 11:15 | |
| *** andymaier has joined #openstack-cinder | 11:19 | |
| *** vbala has joined #openstack-cinder | 11:20 | |
| *** yangyapeng has joined #openstack-cinder | 11:22 | |
| *** sdake_ is now known as sdake | 11:24 | |
| *** ociuhandu has joined #openstack-cinder | 11:28 | |
| *** Digvijay has joined #openstack-cinder | 11:30 | |
| *** alyson_ has joined #openstack-cinder | 11:31 | |
| *** digu has quit IRC | 11:32 | |
| *** bkopilov has quit IRC | 11:39 | |
| *** Digvijay has quit IRC | 11:40 | |
| *** lkuchlan_ has joined #openstack-cinder | 11:43 | |
| *** lkuchlan has quit IRC | 11:44 | |
| swamireddy | vbala: hi | 11:46 |
| vbala | swamireddy: hi | 11:47 |
| swamireddy | vbala: Updated the volume summary patch as per the review comments...can you please review the same...https://review.openstack.org/#/c/335067/ | 11:48 |
| vbala | ok | 11:49 |
| swamireddy | vbala: Thank you | 11:53 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS stats reporting https://review.openstack.org/344477 | 11:53 |
| *** aohuanxuan has quit IRC | 11:56 | |
| *** julim has joined #openstack-cinder | 12:01 | |
| *** sdake_ has joined #openstack-cinder | 12:01 | |
| *** andymaier has quit IRC | 12:02 | |
| *** coolsvap_ is now known as coolsvap | 12:03 | |
| *** gouthamr has joined #openstack-cinder | 12:04 | |
| *** tpsilva has joined #openstack-cinder | 12:04 | |
| *** sdake has quit IRC | 12:05 | |
| *** sdake_ is now known as sdake | 12:05 | |
| *** mtanino has joined #openstack-cinder | 12:07 | |
| *** avishay has quit IRC | 12:08 | |
| *** markvoelker has joined #openstack-cinder | 12:09 | |
| *** ducttape_ has joined #openstack-cinder | 12:10 | |
| *** jamielennox|away is now known as jamielennox | 12:11 | |
| *** andymaier has joined #openstack-cinder | 12:14 | |
| *** huanan_L has quit IRC | 12:15 | |
| *** cdelatte has joined #openstack-cinder | 12:16 | |
| *** huanan has joined #openstack-cinder | 12:16 | |
| *** sandanar__ has quit IRC | 12:17 | |
| *** alonma has quit IRC | 12:23 | |
| *** ducttape_ has quit IRC | 12:23 | |
| *** alonma has joined #openstack-cinder | 12:27 | |
| *** e0ne has joined #openstack-cinder | 12:28 | |
| *** GB21 has quit IRC | 12:28 | |
| *** gcb_ has quit IRC | 12:33 | |
| *** tommylikehu has joined #openstack-cinder | 12:33 | |
| *** nicolasbock has quit IRC | 12:35 | |
| *** vbala has quit IRC | 12:44 | |
| *** gcb_ has joined #openstack-cinder | 12:46 | |
| *** bswartz has joined #openstack-cinder | 12:48 | |
| *** mtanino has quit IRC | 12:49 | |
| openstackgerrit | xiexs proposed openstack/cinder: Add validation for the metadata properties https://review.openstack.org/330913 | 12:55 |
| *** rlrossit has joined #openstack-cinder | 12:59 | |
| *** ducttape_ has joined #openstack-cinder | 13:00 | |
| *** tommylikehu has quit IRC | 13:00 | |
| *** Julien-zte has joined #openstack-cinder | 13:00 | |
| *** jungleboyj has joined #openstack-cinder | 13:02 | |
| *** EinstCrazy has joined #openstack-cinder | 13:03 | |
| *** alonma has quit IRC | 13:04 | |
| *** bkopilov has joined #openstack-cinder | 13:05 | |
| *** alonma has joined #openstack-cinder | 13:05 | |
| *** xyang1 has joined #openstack-cinder | 13:06 | |
| *** mvk has joined #openstack-cinder | 13:06 | |
| *** diablo_rojo has joined #openstack-cinder | 13:08 | |
| *** alonma has quit IRC | 13:09 | |
| *** alonma has joined #openstack-cinder | 13:09 | |
| *** xyang has joined #openstack-cinder | 13:09 | |
| *** coolsvap is now known as _coolsvap_ | 13:11 | |
| nikeshm | smcginnis: hi | 13:13 |
| *** Julien-zte has quit IRC | 13:14 | |
| scottda | What was the status of dynamic reconfiguration? was that diablo_rojo driving that? | 13:15 |
| smcginnis | nikeshm: Hi | 13:15 |
| smcginnis | scottda: Yes, I think so. | 13:16 |
| xyang | e0ne: hi | 13:16 |
| e0ne | xyang: hi | 13:16 |
| smcginnis | Doctor | 13:16 |
| xyang | e0ne: so I used a constant in the migration script now. is that ok | 13:16 |
| e0ne | xyang: it should be | 13:16 |
| scottda | I see Dynamic Config spec merged. I don't see any link to code... | 13:17 |
| nikeshm | smcginnis: we want to add replication failback support in Kaminario driver, our code is ready, do it need bp | 13:17 |
| xyang | e0ne: did you see my other replies to your comments? have they addressed your concerns? | 13:17 |
| e0ne | xyang: I don't remember if we've got final agreement on that bug, but it sounds reasonable for me | 13:17 |
| xyang | e0ne: ok, thanks | 13:17 |
| smcginnis | nikeshm: No, for a driver feature implementation like that it is not really necessary. | 13:18 |
| xyang | e0ne: the link you gave has a patch that was abandoned | 13:18 |
| e0ne | xyang: maybe :) | 13:18 |
| e0ne | xyang: I've got only one consern about your patch now | 13:19 |
| xyang | e0ne: what is it | 13:19 |
| nikeshm | smcginnis: can i change version history because we are adding failback support and do we need releasenote? | 13:19 |
| *** mriedem has joined #openstack-cinder | 13:20 | |
| e0ne | xyang: accodning to tests for https://review.openstack.org/#/c/322459/58/cinder/group/api.py | 13:20 |
| e0ne | xyang: I would like to get more unit tests than cross-module testing | 13:20 |
| *** Julien-zte has joined #openstack-cinder | 13:20 | |
| e0ne | xyang: but community could not agree with me on it | 13:21 |
| xyang | e0ne: oh, so just test this class, not to start from api url? | 13:21 |
| e0ne | xyang: yes | 13:21 |
| *** merooney has joined #openstack-cinder | 13:21 | |
| e0ne | xyang: we can ask about this case to others cores and community | 13:21 |
| xyang | e0ne: I could add some tests there if you want | 13:21 |
| smcginnis | nikeshm: A release note would be good to let users know of the capability. Not sure what you mean by change version history. | 13:21 |
| xyang | e0ne: sure | 13:21 |
| nikeshm | Version history: 1.0 - Initial driver 1.1 - Added manage/unmanage and extra-specs support for nodedup 1.2 - Added replication support 1.3 - Added retype support | 13:21 |
| *** EinstCrazy has quit IRC | 13:22 | |
| xyang | e0ne: I probably just followed what are there for other api unit tests | 13:22 |
| smcginnis | nikeshm: Oh, totally up to you, but I think it's good practice to list stuff like that in the docstring. | 13:22 |
| e0ne | xyang: IMO, if we test more code by one test, it's easier to support it | 13:22 |
| e0ne | xyang: fair enouph :) | 13:22 |
| e0ne | xyang: it's hard to -1 if we've already get such things in the code | 13:23 |
| xyang | e0ne: :) | 13:23 |
| xyang | e0ne: let me take a look and see what tests I can add. | 13:25 |
| *** porrua has joined #openstack-cinder | 13:25 | |
| e0ne | xyang: thanks | 13:25 |
| xyang | e0ne: thank you for reviewing and testing! | 13:25 |
| e0ne | xyang: anyway, I would not -1 on it. maybe, I'll just put +1 | 13:25 |
| e0ne | xyang: np | 13:25 |
| openstackgerrit | xiexs proposed openstack/cinder: Refactor the using of dict.get() in the test assertion https://review.openstack.org/355323 | 13:26 |
| e0ne | xyang: I'm going review and test your snapshot-related patches later today | 13:26 |
| xyang | e0ne: it can't go anywhere anyway now. it can't pass ceph CI:( | 13:26 |
| xyang | e0ne: thanks! | 13:26 |
| *** ducttape_ has quit IRC | 13:26 | |
| *** akerr has joined #openstack-cinder | 13:29 | |
| cFouts | DuncanT: https://review.openstack.org/#/c/346135 would appreciate a review if you have time. fyi, the eseries CI failure doesn't apply to this patch. Thanks! | 13:29 |
| *** Asaithambi has quit IRC | 13:29 | |
| DuncanT | cFouts: Is somebody looking at the eseries failure? Broken CIs make everybody's life difficult | 13:31 |
| cFouts | DuncanT: we are having trouble getting all 8 tests to pass successfully in one pass. We have a number of user stories in the backlog to improve internal and upstream CI once we get to feature freeze :) | 13:32 |
| *** dustins has joined #openstack-cinder | 13:32 | |
| DuncanT | cFouts: Thanks. Good to hear it's being looked at. | 13:33 |
| *** Digvijay has joined #openstack-cinder | 13:36 | |
| *** cknight has joined #openstack-cinder | 13:36 | |
| *** edmondsw has joined #openstack-cinder | 13:37 | |
| *** mtanino has joined #openstack-cinder | 13:37 | |
| *** tongli has joined #openstack-cinder | 13:39 | |
| *** ametts has joined #openstack-cinder | 13:40 | |
| *** Jaison has quit IRC | 13:46 | |
| openstackgerrit | venkata krishna proposed openstack/cinder: Add replication failback in Kaminario K2 drivers https://review.openstack.org/356475 | 13:48 |
| openstackgerrit | Merged openstack/python-brick-cinderclient-ext: Add support of 'local-attach' feature for RBD volumes https://review.openstack.org/355831 | 13:50 |
| *** sdake has quit IRC | 13:50 | |
| *** xyang1 has quit IRC | 13:51 | |
| *** xyang1 has joined #openstack-cinder | 13:51 | |
| scottda | e0ne: I think that's the last python-brick-cinderclient-ext patch that needs to merge ^^^ . Can we remove the 'review reminder' from the meeting agenda? | 13:51 |
| e0ne | scottda: sure. I'll do | 13:52 |
| scottda | e0ne: Cool. | 13:52 |
| e0ne | scottda: and I'll add one more patch... but I don't want to take a meeting time on it | 13:52 |
| e0ne | scottda: thanks | 13:52 |
| *** thumpba has joined #openstack-cinder | 13:52 | |
| scottda | e0ne: Not a big deal to have the patch in the meeting, I just wanted to remove that big, giant URL :) | 13:52 |
| e0ne | scottda: I hope, we'll get more contribution to python-brick-cinderclient-ext in O release | 13:53 |
| scottda | e0ne: It sucks that the wiki won't let us use tinyURL. | 13:53 |
| e0ne | scottda: I hope, we'll get more contribution to python-brick-cinderclient-ext in O release | 13:53 |
| e0ne | scottda: I don't like to ask people for reviews, but python-brick-cinderclient-ext project is not popular now, so it's had to do it | 13:53 |
| scottda | e0ne: Yeah. Do you/we ever talk to Ironic people about their use cases and any wishlist from them? | 13:54 |
| e0ne | scottda: we just merged attache feature last week | 13:54 |
| *** Asaithambi has joined #openstack-cinder | 13:54 | |
| openstackgerrit | zhangguoqing proposed openstack/cinder: Show user_id when cinder snapshot-show https://review.openstack.org/356478 | 13:54 |
| e0ne | scottda: I'll ask for tests and feedback on the next ironic meeting | 13:55 |
| scottda | e0ne: Sounds good. | 13:55 |
| *** ducttape_ has joined #openstack-cinder | 13:55 | |
| e0ne | scottda: maybe, they'll want to wait for a release.. I don't know | 13:55 |
| e0ne | scottda: and CI with Ironic is the next on my TODOs list | 13:55 |
| *** xyang1 has quit IRC | 13:58 | |
| *** sdake_ has joined #openstack-cinder | 14:01 | |
| *** ociuhandu has quit IRC | 14:01 | |
| openstackgerrit | YanLin Ren proposed openstack/cinder: Volume Manage/Unmanage Support for IBM FlashSystem https://review.openstack.org/353856 | 14:01 |
| *** zhangguoqing has quit IRC | 14:04 | |
| sdague | smcginnis / hemna: did we get a plan forward on narrowing the races in os-brick ? | 14:08 |
| openstackgerrit | Merged openstack/cinder: Change a typo error in the releasenote https://review.openstack.org/356186 | 14:10 |
| smcginnis | sdague: In meeting. hemna put up a patch to remove the locks, but it didn't address retries. | 14:11 |
| smcginnis | sdague: Was that retry piece we talked about on the nova side though? | 14:11 |
| *** ociuhandu has joined #openstack-cinder | 14:12 | |
| *** hoobaman has joined #openstack-cinder | 14:14 | |
| hoobaman | hi | 14:14 |
| hoobaman | i got very bad performance while booting from image to a scaleio block volume | 14:15 |
| hoobaman | seems like the validation step qemu-img convert (validate image) is using non-bufffered direct IO | 14:15 |
| hoobaman | max performance i can get is 8MB/s on a Gbit connection :/ | 14:16 |
| hoobaman | take like 8-10 minutes to boot the instance | 14:16 |
| hoobaman | any pointers? :) | 14:16 |
| *** Digvijay has quit IRC | 14:17 | |
| *** tommylikehu has joined #openstack-cinder | 14:19 | |
| diablo_rojo | scottda: Yeah I haven't had much time to work on that, but when I start at the foundation next week I will get going on it. | 14:20 |
| *** haplo37_ has joined #openstack-cinder | 14:22 | |
| hoobaman | Any pointers on how I can increase this performance? | 14:23 |
| *** Asaithambi has quit IRC | 14:24 | |
| smcginnis | xyang: Have you seen this? ^ | 14:24 |
| xyang | @smcginnis: which one? | 14:24 |
| sdague | smcginnis: removing the locks doesn't solve the issue, it makes the races worse | 14:24 |
| smcginnis | xyang: hoobaman was asking about image copy performance with scaleio. | 14:24 |
| xyang | @smcginnis: oh, okay | 14:24 |
| sdague | smcginnis: my understanding was the retry logic would be in os-brick itself | 14:25 |
| smcginnis | sdague: Not sure it can be. The retry was to address not having to have locks. | 14:25 |
| sdague | smcginnis: the retry was to mitigate cinder logging out isciadm while nova was attaching | 14:26 |
| scottda | diablo_rojo: Cool. It just came up at a standup meeting. I'm happy to help if I can, so keep me in the loop. | 14:26 |
| xyang | hoobaman: what's the size of the volume | 14:26 |
| sdague | if you remove the locks all you end up with is now the possibility of nova racing itself | 14:26 |
| smcginnis | sdague: Right. So if we don't lock, that is the risk. So with no locks we could theoretically just retry since a new attach won't have any IO going. | 14:26 |
| sdague | because this is eventlet | 14:26 |
| diablo_rojo | scottda: Awesome. I definitely will. | 14:27 |
| sdague | smcginnis: no, there are multiple levels of locking here | 14:27 |
| smcginnis | The locks keep that from happening (if they worked) by not allowing an attach and detach to overlap. | 14:27 |
| sdague | there is a single process don't screwing itself, still a good idea to happen | 14:27 |
| sdague | and there is global cluster lock, which can not work | 14:27 |
| sdague | but you can mitigate against the lack of global cluster lock with a retry | 14:27 |
| smcginnis | sdague: The root of this was that even with a single node we can't lock across nova-cinder, so effectively it is always a global lock, right? | 14:28 |
| sdague | smcginnis: no | 14:28 |
| sdague | that lock *also* prevents nova-compute from racing itself | 14:29 |
| sdague | in a single process | 14:29 |
| sdague | because it is a single process eventlet system | 14:29 |
| sdague | so every io changes green threads | 14:29 |
| smcginnis | sdague: Guess I don't know enough of the nova side to see how that happens. | 14:29 |
| sdague | so nova-compute can be detaching and attaching at the same time | 14:29 |
| sdague | that's a thing you want to prevent | 14:30 |
| smcginnis | sdague: Seems like that should be locked on the nova side, but with the locks as they are now that should be mitigated. So I don't see with the locks why we would need retries. | 14:30 |
| sdague | smcginnis: the lock is in os-brick | 14:30 |
| sdague | smcginnis: .... | 14:30 |
| sdague | the issue | 14:31 |
| smcginnis | sdague: Right, but this came about because cinders lock dir is different than novas lock dir. | 14:31 |
| smcginnis | At least that was my understanding. | 14:31 |
| sdague | is that c-vol and n-cpu can be on the same host | 14:31 |
| smcginnis | Yes | 14:31 |
| sdague | and doing the same isciadm commands | 14:31 |
| sdague | and there is no workable model to preventing that | 14:31 |
| smcginnis | That's how it was described to me. | 14:31 |
| sdague | *in addition* | 14:31 |
| sdague | the locking constraints is os-brick, also prevent, correctly c-vol and n-cpu from doing pathelogically stupid things | 14:32 |
| sdague | like logging themself out | 14:32 |
| sdague | you have 2 levels of conflict here | 14:32 |
| sdague | can n-cpu stomp itself | 14:32 |
| sdague | can c-vol stomp itself | 14:32 |
| sdague | can n-cpu <=> c-vol stomp each other | 14:33 |
| *** pbandark has quit IRC | 14:33 | |
| sdague | the locks as is prevent the first 1 items | 14:33 |
| sdague | and can't prevent the 3rd | 14:33 |
| smcginnis | That third case was the only one where I was aware there was any kind of issue here. | 14:33 |
| sdague | smcginnis: because of the locks | 14:33 |
| sdague | if you remove the locks | 14:33 |
| sdague | you get 1 & 2 | 14:33 |
| sdague | as well | 14:33 |
| hoobaman | xyang: thx for your reply, the size of the volume is 6GB | 14:34 |
| smcginnis | sdague: I agree. But someone was pushing last time that the locks were useless and we should just do retries if we actually hit an issue. | 14:34 |
| smcginnis | Because of case 3. | 14:34 |
| hoobaman | xyang: however I did some tests with a qcow2 image of 900MB, the behaviour is the same | 14:34 |
| sdague | smcginnis: the lock is useless in case 3 | 14:34 |
| sdague | so we need a retry there | 14:34 |
| hoobaman | xyang: i mean the size of the image is 6GB | 14:34 |
| hoobaman | xyang: volume is 16GB | 14:35 |
| xyang | hoobaman: I'll have to check with scaleio QA and see what they get. do you have a email address I can get back to you | 14:35 |
| sdague | removing the locks otherwise is like saying, "I'm not sure I trust my airbags, so I'm going to cut out my seat belts" | 14:36 |
| smcginnis | sdague: Right. I think we are arguing the same point here basically. | 14:36 |
| smcginnis | I wasn't the one pushing for lock removal. | 14:36 |
| smcginnis | But I thought the plan was to add retries. | 14:36 |
| smcginnis | But I also thought that had to happen above brick. | 14:37 |
| sdague | smcginnis: ok, that's what I was asking. | 14:37 |
| sdague | no, I think that happens in brick | 14:37 |
| sdague | inside the locked area | 14:37 |
| sdague | because c-vol is going to have the same issues where nova can stomp it | 14:37 |
| sdague | anyway, this is why I asked the question | 14:38 |
| *** pgadiya has quit IRC | 14:38 | |
| smcginnis | Well, since it can only happen in the third case, where the locks can become useless, I don't think it has to happen within the locked area. | 14:38 |
| sdague | because I think the answer is "there is no plan" | 14:38 |
| *** jdurgin1 has joined #openstack-cinder | 14:38 | |
| sdague | smcginnis: THE LOCKS ARE NOT USELESS THERE | 14:38 |
| sdague | because they are actively preventing #1/#2 from happening | 14:39 |
| smcginnis | In the third case? Mostly useless. | 14:39 |
| smcginnis | I'm only talking about three. | 14:39 |
| sdague | right, but it's not like while 3 is happening, 1 & 2 aren't also able to happen | 14:39 |
| smcginnis | In 1 and 2 we don't have a problem because we ARE locking and therefore we can't have a logout happen while trying to login. | 14:39 |
| sdague | you could have 5 simultaneous conflicts happening, 3 x 1, 2 x 3 | 14:40 |
| *** xinli has joined #openstack-cinder | 14:40 | |
| sdague | you are defending against the case 1 | 14:40 |
| sdague | and you need to do the retry while those defenses are up | 14:40 |
| smcginnis | OK, semantics, but in those cases I consider them all to be case 3. | 14:40 |
| sdague | smcginnis: but they aren't | 14:40 |
| sdague | they really really aren't | 14:40 |
| smcginnis | It only becomes an issue for 1 & 2 if you also have n-cpu and c-vol on the same host as well as different hosts, so it ends up being case 3. | 14:41 |
| sdague | smcginnis: except, not entirely | 14:41 |
| smcginnis | Well again, semantics. Doesn't change the condition that needs to be protected against, no matter how we want to label them. | 14:42 |
| sdague | because you can either decide that you are handling the concurency of case 3 inside the context where the case 1 concurrency is addressed, and those requests are going to process in some order | 14:42 |
| sdague | or you can give up all handling, let everything race back in again, including the case 1 issues | 14:42 |
| sdague | maybe you can make them the same, but the handling seems a lot more straight forward if you do it in the context of case 1 being a solved problem | 14:43 |
| *** kfarr has joined #openstack-cinder | 14:44 | |
| *** diogogmt has joined #openstack-cinder | 14:47 | |
| *** arecknag has quit IRC | 14:47 | |
| *** erhudy has joined #openstack-cinder | 14:51 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Merge two fake drivers into the one FakeLoggingVolumeDriver https://review.openstack.org/356525 | 14:53 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Merge two fake drivers into the one FakeLoggingVolumeDriver https://review.openstack.org/348428 | 14:55 |
| *** laughterwym has joined #openstack-cinder | 14:59 | |
| scottda | eharney, xyang1, gouthamr, akerr, smcginnis, cFouts, e0ne, geguileo, dulek, flip214, ntpttr patrickeast, _alastor_, DuncanT, erlon | 14:59 |
| scottda | Test-Lovers, this is your summons... | 14:59 |
| cFouts | o/ | 14:59 |
| gouthamr | hey o/ | 14:59 |
| erlon | hey | 14:59 |
| * DuncanT waves | 14:59 | |
| scottda | #startmeeting cinder_testing | 14:59 |
| openstack | Meeting started Wed Aug 17 14:59:47 2016 UTC and is due to finish in 60 minutes. The chair is scottda. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:59 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:59 |
| openstack | The meeting name has been set to 'cinder_testing' | 14:59 |
| patrickeast | hey | 15:00 |
| scottda | #link https://etherpad.openstack.org/p/Cinder-testing | 15:00 |
| ntpttr___ | o/ | 15:00 |
| e0ne | hi | 15:01 |
| *** _coolsvap_ is now known as coolsvap | 15:01 | |
| *** enriquetaso has joined #openstack-cinder | 15:01 | |
| *** rhagarty_ has quit IRC | 15:01 | |
| scottda | From the agenda: | 15:02 |
| scottda | devstack-gate cinder multi-node patch has merged: https://review.openstack.org/#/c/327619/ | 15:02 |
| scottda | Is dulek around? | 15:02 |
| openstackgerrit | Sean McGinnis proposed openstack/os-brick: Add retries to iSCSI connect_volume https://review.openstack.org/356532 | 15:02 |
| scottda | OK | 15:03 |
| scottda | Next on agenda: Add a multibackend list to tempest.conf has merged: https://review.openstack.org/#/c/325895 | 15:03 |
| scottda | I'm waiting on re-approval of next patch in the series.... | 15:03 |
| *** openstackgerrit has quit IRC | 15:03 | |
| scottda | Main thing is that I've updated the status of the patches under review... | 15:04 |
| scottda | I'll go to the QA meeting as well as infra channel and look for help on the ones that need final +2. | 15:04 |
| scottda | Any help with that would also be good. | 15:04 |
| *** openstackgerrit has joined #openstack-cinder | 15:05 | |
| scottda | and some patches have zero reviews, so please have a look. | 15:05 |
| smcginnis | scottda: Is there a common topic we can pull up all of them? | 15:07 |
| *** raunak has joined #openstack-cinder | 15:07 | |
| patrickeast | smcginnis: nope, not yet anyway | 15:07 |
| scottda | smcginnis: maybe...not sure. they are in at least 2 different repos: devstack-gate and devstack. | 15:07 |
| smcginnis | scottda: OK, probably easier just pulling up patches where you're the owner then. | 15:07 |
| *** alonma has quit IRC | 15:07 | |
| *** xyang1 has joined #openstack-cinder | 15:07 | |
| *** alonma has joined #openstack-cinder | 15:07 | |
| scottda | I'll try to keep the status up-to-date on the etherpad. That may be the best way to track 'em | 15:07 |
| smcginnis | scottda: That sounds good. | 15:07 |
| scottda | Has anyone other than me tried geguileo 's manual tests for AA-HA? | 15:08 |
| scottda | #link http://gorka.eguileor.com/manual-validation-of-cinder-aa-patches/ | 15:08 |
| scottda | Works great. Ship it. | 15:08 |
| geguileo | XD | 15:09 |
| smcginnis | geguileo: I haven't been able to run through and test, but very nice write up! | 15:09 |
| *** Digvijay has joined #openstack-cinder | 15:09 | |
| geguileo | I hope it helps understand the flow a little bit better | 15:09 |
| geguileo | Although it doesn't replace a good devref for adding clenable resources/states | 15:10 |
| xyang1 | geguileo: do you want to add that to cinder doc | 15:11 |
| e0ne | I'm going to test them once I finish groups-related patches testing | 15:11 |
| *** ducttape_ has quit IRC | 15:11 | |
| geguileo | xyang1: Not that one, I'll create a doc with an explanation on how to add a new cleanable resource/state | 15:11 |
| *** alonma has quit IRC | 15:11 | |
| xyang1 | geguileo: ok | 15:11 |
| geguileo | xyang1: I don't think adding that info to the cinder doc is that useful | 15:11 |
| *** maestropandy has quit IRC | 15:11 | |
| geguileo | It was more to help people get started in the testing and understand the pieces at play | 15:12 |
| smcginnis | A "how to" sounds useful. | 15:12 |
| e0ne | :) | 15:12 |
| scottda | I'm going to re-test today with the latest set. And I threatened geguileo I would record it an put the webcast on youtube, as I did with api microversions. Some found that helpful... | 15:12 |
| *** lcurtis has joined #openstack-cinder | 15:12 | |
| geguileo | Because sometimes it's hard just reading the specs | 15:12 |
| scottda | Anyone have anything test-wise to bring up? | 15:13 |
| smcginnis | scottda: I finally wrote up test description changes. | 15:13 |
| smcginnis | And switched around what we were saying for "integration" vs "functional", so interested in what folks think of that. | 15:14 |
| e0ne | I'm working on fake drivers refactoring and devstack integration. I hope, all patches will be ready this week. | 15:14 |
| scottda | #link https://review.openstack.org/#/c/356177/ | 15:14 |
| xyang1 | e0ne: I saw you submitted a patch on the fake driver. so we should use that one for functional tests? | 15:14 |
| smcginnis | scottda: Beat me to it! | 15:14 |
| scottda | smcginnis: looks -1'd by you? | 15:14 |
| *** alonma has joined #openstack-cinder | 15:14 | |
| e0ne | and yes, fake driver could be use for functional and integration tests | 15:14 |
| e0ne | xyang1: I hope so | 15:14 |
| smcginnis | scottda: Just to make sure everyone has a chance to take a look and raise any objections before someone decides to just push it through. | 15:14 |
| e0ne | xyang1: I didn't test it with devstack, so it's still in WIP status | 15:15 |
| scottda | smcginnis: Makes sense. Everyone please review. | 15:15 |
| xyang1 | e0ne: ok | 15:15 |
| scottda | #link https://review.openstack.org/356525 | 15:16 |
| e0ne | we need refatcor unit/test_volume.py to get split between unit, funcitonal and integration tests | 15:16 |
| scottda | e0ne: 's fake driver patch ^^ | 15:16 |
| smcginnis | e0ne: +1 | 15:16 |
| e0ne | correct #link https://review.openstack.org/348428 | 15:16 |
| scottda | oops | 15:17 |
| e0ne | scottda: that one is abandoned. I missed correct change-id | 15:17 |
| scottda | k | 15:17 |
| e0ne | #link https://review.openstack.org/348449 - patch to devstack | 15:18 |
| e0ne | xyang1: I'll add FakeGateDriver support to devstack patch ^^ too | 15:18 |
| xyang1 | e0ne: thanks:) | 15:18 |
| *** Julien-zte has quit IRC | 15:18 | |
| *** alonma has quit IRC | 15:19 | |
| scottda | e0ne: Would you like to put those links in the etherpad? | 15:19 |
| e0ne | scottda: I'll do once I test my patch with devstack | 15:20 |
| scottda | thanks | 15:20 |
| e0ne | everybody is welcome to review ot now.. maybe I missed something | 15:20 |
| e0ne | TBH, I removed few asserts from test_volume | 15:20 |
| *** raunak has quit IRC | 15:21 | |
| scottda | So, in general, we're getting good reviews from cinder folks on these test patches. I still don't know how to get more QA/infra people involved, but there's been some progress. | 15:21 |
| *** alonma has joined #openstack-cinder | 15:21 | |
| scottda | ¿Hay mas? Anything else? | 15:23 |
| patrickeast | i had one thing maybe worth discussion | 15:23 |
| scottda | patrickeast: Bring it | 15:23 |
| patrickeast | in the in-tree cg tempest tests https://review.openstack.org/#/c/355271/ i added a new config opt group for tempest.conf | 15:23 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 15:23 |
| patrickeast | instead of re-using/adding to existing groups like "volume" or "volume-features-enabled" (or whatever it is) | 15:24 |
| patrickeast | wanted to make sure no one was strongly opposed to that... it is kinda weird to have so many sections basically for configuring volume/cinder stuff | 15:24 |
| *** kfarr has quit IRC | 15:25 | |
| scottda | patrickeast: Thanks. We'll see what people think in the review... | 15:26 |
| *** akapil has quit IRC | 15:26 | |
| *** alonma has quit IRC | 15:26 | |
| patrickeast | scottda: haha yea, i take silence as a resounding "ship it" | 15:26 |
| *** akapil has joined #openstack-cinder | 15:26 | |
| e0ne | :) | 15:27 |
| scottda | alrighty then....we can finish unless there are other items? | 15:27 |
| scottda | #endmeeting | 15:28 |
| openstack | Meeting ended Wed Aug 17 15:28:13 2016 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:28 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-17-14.59.html | 15:28 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-17-14.59.txt | 15:28 |
| scottda | thanks! | 15:28 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder_testing/2016/cinder_testing.2016-08-17-14.59.log.html | 15:28 |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Added config option to enable SSL https://review.openstack.org/352958 | 15:29 |
| *** alonma has joined #openstack-cinder | 15:29 | |
| *** tommylikehu has quit IRC | 15:29 | |
| *** kfarr has joined #openstack-cinder | 15:32 | |
| *** alonma has quit IRC | 15:34 | |
| karthikp_ | dulek: Hi | 15:34 |
| *** alonma has joined #openstack-cinder | 15:37 | |
| smcginnis | patrickeast: If you happen have time, that large test where we were getting overlapping attach/dettach against this would be good: https://review.openstack.org/#/c/356532/ | 15:37 |
| patrickeast | smcginnis: can do | 15:39 |
| *** ducttape_ has joined #openstack-cinder | 15:40 | |
| *** alonma has quit IRC | 15:41 | |
| patrickeast | smcginnis: unfortunately i have a hunch that its not going to be so easy :(, that should fix the case where the attach command got screwed up by a detach logging it out but the more ugly problem is when two detaches race and both decide not to log out, we get dangling multipath devices and iscsi sessions | 15:41 |
| patrickeast | those ones require manual cleanup on the initiator :*( | 15:42 |
| patrickeast | smcginnis: good news though, its super rare to have c-vol and n-cpu on the same node... so i'm not really concerned about this too much | 15:42 |
| patrickeast | well other than devstack | 15:43 |
| smcginnis | patrickeast: True. | 15:43 |
| smcginnis | patrickeast: I'd rather have a few useless sessions hanging around than inaccessible volumes though. ;) | 15:43 |
| smcginnis | patrickeast: But we should probably think about a better way to handle this. Or have some kind of cleanup processs. | 15:44 |
| *** alonma has joined #openstack-cinder | 15:44 | |
| patrickeast | uh, no, the broken session break all future attachments with the same iqn+lun | 15:44 |
| patrickeast | its way bad | 15:44 |
| smcginnis | patrickeast: Oh, good point. Didn't think of that. | 15:44 |
| *** thumpba has quit IRC | 15:44 | |
| patrickeast | smcginnis: it doesn't affect all backends though | 15:45 |
| patrickeast | smcginnis: things like lvm that have unique iqns and always are lun 0 its no problemo | 15:45 |
| smcginnis | I think just yours and mine and the few others that have shared target ports. | 15:45 |
| patrickeast | yep | 15:45 |
| patrickeast | smcginnis: anyway, i'll give that retry patch a try... seems like we should be doing that anyway, regardless of the locking situation | 15:47 |
| smcginnis | patrickeast: Cool, thanks! | 15:47 |
| openstackgerrit | Merged openstack/os-brick: Wrong param makes exception message throws inaccurate https://review.openstack.org/353892 | 15:48 |
| *** alonma has quit IRC | 15:49 | |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Added config option to enable SSL https://review.openstack.org/352958 | 15:49 |
| *** lprice has quit IRC | 15:49 | |
| *** lprice has joined #openstack-cinder | 15:50 | |
| *** jdurgin1 has quit IRC | 15:50 | |
| *** alonma has joined #openstack-cinder | 15:51 | |
| *** lkuchlan_ has quit IRC | 15:52 | |
| *** leeantho has joined #openstack-cinder | 15:53 | |
| hemna | scottda, ping | 15:55 |
| *** alonma has quit IRC | 15:55 | |
| scottda | hemna: Hi | 15:55 |
| hemna | scottda, do you happen to have links to the nova -> cinder workflows that I created | 15:56 |
| hemna | I can't seem to find them | 15:56 |
| scottda | hemna: They are here https://etherpad.openstack.org/p/cinder-nova-api-changes | 15:56 |
| hemna | thank you! | 15:57 |
| *** ebalduf has joined #openstack-cinder | 15:57 | |
| *** rcernin has quit IRC | 15:58 | |
| *** sdake_ is now known as sdake | 16:00 | |
| *** sdake is now known as sdake_ | 16:00 | |
| *** alonma has joined #openstack-cinder | 16:02 | |
| *** andymaier has quit IRC | 16:03 | |
| *** wewe0901 has joined #openstack-cinder | 16:04 | |
| *** lprice1 has joined #openstack-cinder | 16:04 | |
| *** laughterwym has quit IRC | 16:05 | |
| *** dmellado is now known as dmellado|off | 16:05 | |
| *** lprice has quit IRC | 16:06 | |
| *** alonma has quit IRC | 16:07 | |
| *** Apoorva has joined #openstack-cinder | 16:08 | |
| *** Digvijay has quit IRC | 16:15 | |
| *** netapp has joined #openstack-cinder | 16:16 | |
| *** narayrak has quit IRC | 16:21 | |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers https://review.openstack.org/348060 | 16:25 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP Add new supported driver checks https://review.openstack.org/355608 | 16:29 |
| *** cknight has quit IRC | 16:31 | |
| *** cknight has joined #openstack-cinder | 16:31 | |
| *** bwallis has quit IRC | 16:36 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-cinderclient: deprecate command `cinder endpoints` https://review.openstack.org/349602 | 16:37 |
| *** JoseMello has joined #openstack-cinder | 16:37 | |
| *** Suyi has joined #openstack-cinder | 16:40 | |
| *** raunak has joined #openstack-cinder | 16:46 | |
| *** kfarr has quit IRC | 16:46 | |
| *** xyang has quit IRC | 16:46 | |
| e0ne | jungleboyj: hi! I answered to your comment for https://review.openstack.org/#/c/352924/ | 16:50 |
| *** xyang has joined #openstack-cinder | 16:50 | |
| *** lpetrut has quit IRC | 16:52 | |
| *** venkhari has joined #openstack-cinder | 16:54 | |
| *** sandanar has joined #openstack-cinder | 16:56 | |
| *** bkopilov has quit IRC | 16:59 | |
| *** tesseract- has quit IRC | 17:00 | |
| *** sandanar_ has joined #openstack-cinder | 17:01 | |
| * hemna is off to HR re-education camp.... | 17:02 | |
| * jungleboyj is ignoring the invite to HR re-education camp. | 17:02 | |
| * diablo_rojo sends a tasteful reply to the HR re-education camp invite | 17:03 | |
| *** chris_morrell has joined #openstack-cinder | 17:04 | |
| *** sandanar has quit IRC | 17:04 | |
| *** bwallis has joined #openstack-cinder | 17:05 | |
| *** e0ne has quit IRC | 17:05 | |
| *** jungleboyj has quit IRC | 17:06 | |
| *** liverpooler has quit IRC | 17:07 | |
| *** jdillaman has quit IRC | 17:12 | |
| *** jdillaman has joined #openstack-cinder | 17:13 | |
| *** lpetrut has joined #openstack-cinder | 17:15 | |
| *** bardia has joined #openstack-cinder | 17:16 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Separate create and update rules for volume metadata https://review.openstack.org/356624 | 17:21 |
| *** rcernin has joined #openstack-cinder | 17:26 | |
| *** lpetrut has quit IRC | 17:27 | |
| *** mvk has quit IRC | 17:31 | |
| *** akapil has quit IRC | 17:36 | |
| openstackgerrit | Mike Rooney proposed openstack/cinder: NetApp: Report multiattach as enabled https://review.openstack.org/356635 | 17:40 |
| *** stevemar_ has joined #openstack-cinder | 17:43 | |
| *** stevemar_ has quit IRC | 17:43 | |
| *** bkopilov has joined #openstack-cinder | 17:49 | |
| scottda | Will the cinderclient version after 1.9.0 be 2.0.0 ? or would it be 1.10.0 ? | 17:50 |
| smcginnis | scottda: I think 1.10.0 | 17:53 |
| smcginnis | scottda: We probably don't want to move to 2.0 until there is a major change. | 17:53 |
| smcginnis | Although.. | 17:53 |
| smcginnis | When did we add microversion support? | 17:53 |
| *** alyson_ has quit IRC | 17:53 | |
| scottda | smcginnis: last release...I think that'd be 1.8.0 | 17:55 |
| smcginnis | scottda: OK. That maybe would have warranted bumping to 2.0, but too late for that. So 1.10.0. | 17:56 |
| scottda | cool, thx | 17:56 |
| *** alyson_ has joined #openstack-cinder | 18:00 | |
| *** jordanP has joined #openstack-cinder | 18:04 | |
| *** pauloewerton has joined #openstack-cinder | 18:04 | |
| *** mvk has joined #openstack-cinder | 18:07 | |
| *** dkehn_ has quit IRC | 18:08 | |
| *** alonma has joined #openstack-cinder | 18:10 | |
| bswartz | smcginnis, hemna: who does release management for os-brick? | 18:12 |
| *** mtanino has quit IRC | 18:13 | |
| *** sandanar_ has quit IRC | 18:13 | |
| *** alonma has quit IRC | 18:14 | |
| hemna | bswartz, the cinder team. | 18:15 |
| hemna | it's owned and managed by cinder just like the cinderclient is | 18:15 |
| bswartz | hemna: who has the power to push tags though? | 18:15 |
| bswartz | I'm curious when the next tagged release of os-brick is planned | 18:15 |
| hemna | I think anyone can post a review in gerrit to submit a new release | 18:16 |
| bswartz | hemna: oh | 18:16 |
| hemna | well, our deadline is next week | 18:16 |
| smcginnis | bswartz: It's all managed through the normal release team process. | 18:16 |
| hemna | so I presume we'll do another drop next week | 18:16 |
| bswartz | okay I figured next week | 18:16 |
| smcginnis | I'm thinking probably late Wednesday. | 18:16 |
| bswartz | I forgot about the fact that anyone can push a commit to the releases repo | 18:16 |
| *** alonma has joined #openstack-cinder | 18:20 | |
| *** dkehn_ has joined #openstack-cinder | 18:20 | |
| *** merooney has quit IRC | 18:21 | |
| *** merooney has joined #openstack-cinder | 18:22 | |
| *** merooney has quit IRC | 18:22 | |
| *** rooneym has joined #openstack-cinder | 18:23 | |
| *** Lee1092 has quit IRC | 18:23 | |
| *** ociuhandu has quit IRC | 18:24 | |
| *** jungleboyj has joined #openstack-cinder | 18:24 | |
| *** alonma has quit IRC | 18:25 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 18:25 |
| *** _ducttape_ has joined #openstack-cinder | 18:27 | |
| *** narayrak has joined #openstack-cinder | 18:29 | |
| *** ducttape_ has quit IRC | 18:30 | |
| *** rlrossit_ has joined #openstack-cinder | 18:38 | |
| *** liverpooler has joined #openstack-cinder | 18:38 | |
| *** rlrossit has quit IRC | 18:41 | |
| xyang | @smcginnis: ping | 18:45 |
| *** ametts has quit IRC | 18:45 | |
| xyang | @smcginnis: do you know if anyone is looking at gate-tempest-dsvm-full-devstack-plugin-ceph failure? I'm getting that in several patches and won't go away after rechecks | 18:46 |
| hemna | xyang, DuncanT mentioned that he was proposing making it experimental because it's so unstable | 18:47 |
| xyang | hemna: ok, thanks | 18:47 |
| hemna | not sure about the short term though, it's puking a bunch from what I see | 18:48 |
| xyang | hemna: can you take a look of this patch when you get a chance? https://review.openstack.org/#/c/317060/ You reviewed it before | 18:48 |
| hemna | sure | 18:49 |
| xyang | hemna: thanks | 18:49 |
| openstackgerrit | venkata krishna proposed openstack/cinder: Add replication failback in Kaminario K2 drivers https://review.openstack.org/356475 | 18:52 |
| *** hoonetorg has quit IRC | 18:53 | |
| nikeshm | if some one get time please review https://review.openstack.org/#/c/356475 | 18:54 |
| *** ametts has joined #openstack-cinder | 18:56 | |
| *** venkhari has quit IRC | 18:56 | |
| *** kfarr has joined #openstack-cinder | 18:56 | |
| *** _ducttape_ has quit IRC | 18:56 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 18:56 |
| *** ducttape_ has joined #openstack-cinder | 18:57 | |
| *** lpetrut has joined #openstack-cinder | 18:57 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add generic volume groups https://review.openstack.org/322459 | 18:58 |
| *** thumpba has joined #openstack-cinder | 19:05 | |
| *** jamielennox has quit IRC | 19:06 | |
| *** hoonetorg has joined #openstack-cinder | 19:06 | |
| *** fifieldt has quit IRC | 19:07 | |
| *** xinli has quit IRC | 19:09 | |
| *** jamielennox has joined #openstack-cinder | 19:09 | |
| *** wewe0901 has quit IRC | 19:13 | |
| *** tongli has quit IRC | 19:15 | |
| *** xyang has quit IRC | 19:15 | |
| *** akerr has quit IRC | 19:15 | |
| *** xyang has joined #openstack-cinder | 19:16 | |
| *** fifieldt has joined #openstack-cinder | 19:18 | |
| stevemar | smcginnis: why are you so hard to find in gerrit | 19:24 |
| *** ociuhandu has joined #openstack-cinder | 19:25 | |
| stevemar | ahhh sean_... too many seans | 19:25 |
| stevemar | not like there aren't too many steve's or anything | 19:25 |
| *** edmondsw has quit IRC | 19:28 | |
| *** _ducttape_ has joined #openstack-cinder | 19:28 | |
| smcginnis | stevemar: Hah! | 19:30 |
| stevemar | :) | 19:30 |
| smcginnis | xyang1: I'm not sure if anyone is working on that actually. | 19:31 |
| *** xinli has joined #openstack-cinder | 19:32 | |
| *** ducttape_ has quit IRC | 19:32 | |
| xyang | @smcginnis: oh, thanks | 19:33 |
| mriedem | does this really warrant an ERROR and stacktrace? http://logs.openstack.org/78/346278/5/check/gate-tempest-dsvm-neutron-multinode-full/71882bb/logs/screen-c-api.txt.gz?level=TRACE#_2016-08-16_15_45_02_989 | 19:33 |
| mriedem | it's just a 400 | 19:33 |
| mriedem | ditto http://logs.openstack.org/78/346278/5/check/gate-tempest-dsvm-neutron-multinode-full/71882bb/logs/screen-c-api.txt.gz?level=TRACE#_2016-08-16_15_41_08_291 | 19:34 |
| mriedem | and this is just a 409 http://logs.openstack.org/78/346278/5/check/gate-tempest-dsvm-neutron-multinode-full/71882bb/logs/screen-c-api.txt.gz?level=TRACE#_2016-08-16_15_23_06_315 | 19:34 |
| mriedem | which isn't an ERROR | 19:34 |
| smcginnis | mriedem: Well, it's kind of an error, right? | 19:34 |
| smcginnis | mriedem: Probably better as just a warning though. | 19:35 |
| mriedem | is this something you'd page an operator for? | 19:35 |
| smcginnis | Just for fun. | 19:35 |
| smcginnis | But fair point. I can put up a patch to change it to warning. | 19:35 |
| mriedem | as long as jungleboyj is getting that page | 19:35 |
| mriedem | i'll open a bug | 19:35 |
| mriedem | there are a few of these | 19:35 |
| smcginnis | mriedem: :) | 19:35 |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1614236 | 19:37 |
| openstack | Launchpad bug 1614236 in Cinder "c-api should not log ERROR tracebacks for 400/409 cases" [Undecided,New] | 19:37 |
| smcginnis | mriedem: Hmm, it's that the check for the size raises an exception (which is valid) but the taskflow executor doesn't know the difference so it's at error level. | 19:37 |
| mriedem | rm -rf taskflow | 19:38 |
| smcginnis | Don't start that again. :] | 19:38 |
| mriedem | so, | 19:38 |
| mriedem | i'm guessing taskflow needs some kind of expected_exceptions list or something? | 19:38 |
| mriedem | oslo.messaging has a thing like that | 19:38 |
| smcginnis | mriedem: Probably | 19:38 |
| smcginnis | mriedem: That handling is fairly simple: https://github.com/openstack/taskflow/blob/master/taskflow/engines/action_engine/executor.py#L47 | 19:39 |
| mriedem | where does it log it? | 19:40 |
| smcginnis | Taskflow let's it bubble up. Still looking... | 19:40 |
| *** jdillaman has quit IRC | 19:41 | |
| mriedem | the task must be defined in cinder | 19:41 |
| *** liverpooler has quit IRC | 19:42 | |
| mriedem | looks like it would be a subclass of taskflow.atom.Atom | 19:42 |
| mriedem | oh i see it's the ExtractVolumeRequestTask.execute | 19:44 |
| mriedem | in cinder.flows.api.create_volume | 19:44 |
| -openstackstatus- NOTICE: The volume for logs.openstack.org filled up rather suddenly, causing a number of jobs to fail with a POST_FAILURE result and no logs; we're manually expiring some logs now to buy breathing room, but any changes which hit that in the past few minutes will need to be rechecked and/or approved again | 19:44 | |
| scottda | that explains that | 19:46 |
| smcginnis | mriedem: I can't look into right now. Want to comment that in the bug? Wonder if we should add taskflow to it. | 19:46 |
| mriedem | harlowja: ^ where does taskflow log failed task executions? | 19:47 |
| harlowja | cinder afaik plugged in there own logger | 19:47 |
| harlowja | so the cinder log? | 19:47 |
| harlowja | cinder afaik plugged in one of https://github.com/openstack/taskflow/blob/master/taskflow/listeners/logging.py#L71 | 19:48 |
| harlowja | and provided there own logger | 19:48 |
| harlowja | (i just repeated the same thing) | 19:48 |
| harlowja | ha | 19:48 |
| mriedem | heh | 19:49 |
| mriedem | _NO_TRACE_EXCEPTIONS = (exception.InvalidInput, exception.QuotaError) | 19:49 |
| *** _ducttape_ has quit IRC | 19:49 | |
| karthikp_ | scottda: Hi Scott, got a minute to discuss about our efforts on partial grenade? | 19:49 |
| *** ducttape_ has joined #openstack-cinder | 19:50 | |
| scottda | karthikp_: sure | 19:50 |
| * jungleboyj doesn't want more pages. | 19:50 | |
| harlowja | right mriedem seems like the traceback on failures isn't happening for those exceptions | 19:50 |
| harlowja | (for better or worse) | 19:51 |
| karthikp_ | scottda: we just had a few of our patches merged for multi-node grenade | 19:51 |
| karthikp_ | scottda: but the grenade job keeps failing as it tries to upgrade the cinder c-vol service | 19:52 |
| karthikp_ | the reason is that we have excluded c-vol and c-bkp from features | 19:53 |
| karthikp_ | https://review.openstack.org/#/c/327619/5/features.yaml | 19:53 |
| scottda | looking... | 19:54 |
| *** jungleboyj has quit IRC | 19:55 | |
| scottda | karthikp_: I guess I cannot say for sure the proper way to do this. I wonder if patrickeast could help, since he has a CI running multi-node. Maybe we can figure it out using his system? | 19:57 |
| karthikp_ | scottda: so we got this patch in to check if c-vol is enabled or not https://review.openstack.org/#/c/355678/2 | 19:58 |
| karthikp_ | scottda: i see ..let me ping him now | 19:58 |
| scottda | I think I just did :) | 19:58 |
| *** xyang has quit IRC | 20:04 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Break_replication unhandled NoneType error https://review.openstack.org/356687 | 20:06 |
| Swanson | +2/+A ^^^ | 20:06 |
| scottda | Swanson: What's that, Jedi mind trick? | 20:07 |
| *** xyang has joined #openstack-cinder | 20:07 | |
| hemna | -2! | 20:08 |
| hemna | no unit tests....-2 | 20:08 |
| hemna | fail! | 20:08 |
| Swanson | scottda, I've stopped with the subliminal messaging and am going Superliminal. | 20:08 |
| hemna | green color in the diff on your patch....-2! | 20:08 |
| *** jungleboyj has joined #openstack-cinder | 20:08 | |
| Swanson | hemna and jungleboyj switch names? | 20:09 |
| hemna | ssshhh don't tell jungleboy | 20:09 |
| *** coolsvap has quit IRC | 20:11 | |
| jungleboyj | what? | 20:12 |
| hemna | damn, he found out. | 20:12 |
| Swanson | great. gonna get nailed for that missing apostrophe. | 20:14 |
| jungleboyj | Swanson: WHAT!?! hemna has come to the punctuation side? | 20:15 |
| *** alonma has joined #openstack-cinder | 20:15 | |
| *** porrua has quit IRC | 20:16 | |
| mriedem | jungleboyj: https://bugs.launchpad.net/cinder/+bug/1614249 | 20:18 |
| openstack | Launchpad bug 1614249 in Cinder "Clone doesn't work for NFS driver" [Undecided,New] | 20:18 |
| *** alonma has quit IRC | 20:20 | |
| *** e0ne has joined #openstack-cinder | 20:20 | |
| *** alonma has joined #openstack-cinder | 20:24 | |
| *** cdelatte has quit IRC | 20:25 | |
| *** alonma has quit IRC | 20:29 | |
| *** xyang has quit IRC | 20:32 | |
| openstackgerrit | Matan Sabag proposed openstack/cinder: Changes stats report for ScaleIO driver https://review.openstack.org/356120 | 20:34 |
| jgriffith | mriedem: good lord, really! | 20:36 |
| * jgriffith pretty much gives up at this point | 20:36 | |
| mriedem | jgriffith: true story | 20:36 |
| jgriffith | mriedem: nice thing about Wiki's... anybody can enter anything they want | 20:37 |
| mriedem | we moved that in tree a long time ago | 20:37 |
| mriedem | so the matrix is tied to the actual code when it lands | 20:37 |
| jgriffith | mriedem: yeah, I tried and had a mutiny | 20:37 |
| jgriffith | mriedem: mostly from folks who aren't even actively engaged around here any longer... go figure | 20:37 |
| mriedem | f'em | 20:38 |
| mriedem | push again | 20:38 |
| jgriffith | mriedem: let's keep adding features in Cinder that we don't test and that don't work | 20:38 |
| openstackgerrit | Matan Sabag proposed openstack/cinder: ScaleIO over subscription support https://review.openstack.org/356547 | 20:38 |
| jgriffith | mriedem: wiki's are pure evil with this many people invovled | 20:38 |
| mriedem | fwiw, https://github.com/openstack/nova/blob/master/doc/source/support-matrix.ini | 20:38 |
| mriedem | the nova matrix is based on an ini file | 20:38 |
| bswartz | Yes, wikis are evil, let us use etherpads | 20:39 |
| mriedem | https://github.com/openstack/nova/blob/master/doc/ext/support_matrix.py | 20:39 |
| jgriffith | mriedem: very nice, thanks | 20:39 |
| mriedem | ^ parses the ini and throws it into the rst | 20:39 |
| hemna | that's cool | 20:39 |
| mriedem | gives http://docs.openstack.org/developer/nova/support-matrix.html | 20:39 |
| jgriffith | I'll follow that lead and start some doc refs | 20:39 |
| patrickeast | scottda: karthikp_: oh interesting, that change seems pretty straight-forward... I don't run grenade tests on my ci | 20:40 |
| * bswartz realizes laptop battery is almost zero | 20:40 | |
| mriedem | i'm guessing you'll rage quit because we have like what, 5 drivers? cinder has 400 | 20:40 |
| patrickeast | but can try it out if there is some specific thing we need tested | 20:40 |
| *** rcernin has quit IRC | 20:40 | |
| erlon | mriedem: thats beautiful :) | 20:41 |
| mriedem | all credit goes to danpb | 20:41 |
| patrickeast | mriedem: jgriffith: somethign like that could be useful with our new plan of tagging drivers that aren't tested | 20:41 |
| jgriffith | bswartz: good... no more smart ass comments from you today! | 20:41 |
| jgriffith | patrickeast: exactly!! | 20:41 |
| mriedem | patrickeast: yeah johnthetubaguy has been working on that for awhile now in nova | 20:42 |
| scottda | patrickeast: That'd be great. I think karthikp_ Has a theory, and if you can test it we'd save some time. But I'll let him give the details.... | 20:42 |
| jgriffith | bswartz: unless of course you have a power supply and an outlet... then I'm doomed | 20:42 |
| erlon | jgriffith: isnt possible to pull that information from the drivers interface checks? | 20:42 |
| mriedem | http://docs.openstack.org/developer/nova/feature_classification.html | 20:42 |
| smcginnis | erlon: jgriffith mriedem: I was actually looking at that at one point. | 20:43 |
| jgriffith | erlon: don't get me started... you'll hear my ABC rant and nobody wants that | 20:43 |
| jgriffith | erlon: on a positive note, smcginnis has started a sane approach to doing that.... and yes | 20:43 |
| erlon | jgriffith: haha, I now you love ABC | 20:43 |
| jgriffith | erlon: I'd think that doc should be compiled by code, NOT humans | 20:43 |
| smcginnis | jgriffith: +1 | 20:44 |
| erlon | smcginnis: that would be even better/accurate than maintaining an INI file | 20:44 |
| smcginnis | I actually looked at how Nova is doing it. Not bad, but I think with the interface checking we have now (once we untangle the ABC stuff) I think it would be very easy to generate and publish the matrix. | 20:44 |
| smcginnis | erlon: +1 | 20:44 |
| karthikp_ | scottda: thanks scott. patrickeast; in order to perform a multi-node grenade, we disabled a few services on primary by https://review.openstack.org/#/c/327619/5/features.yaml | 20:45 |
| patrickeast | another cool thing with the ini file, if we put whether or not the driver is tested/validated/whatever... eventually we just put a link to their out of tree git repo and... | 20:45 |
| jgriffith | smcginnis: so I'm thinking use the interface checking to generate the matrix doc | 20:45 |
| jgriffith | smcginnis: erlon I think that's the same thing you both are saying no? | 20:45 |
| smcginnis | jgriffith: Yep | 20:46 |
| karthikp_ | patrickeast:Now, its failing our grenade tests fail as there is no c-vol enabled.... do you think it would be right to patch it like this | 20:46 |
| erlon | jgriffith: mhm | 20:46 |
| karthikp_ | https://review.openstack.org/#/c/355678/ | 20:46 |
| smcginnis | patrickeast: Oh, that would be an advantage of the ini approach. | 20:46 |
| jgriffith | smcginnis: so where are we with getting that interface checking online? Just a matter of consolidating the classes back to something sane again? | 20:46 |
| jgriffith | patrickeast: oh... I like that | 20:47 |
| smcginnis | jgriffith: Yeah. I just have the job to do the basic interface checking. | 20:47 |
| patrickeast | karthikp_: hmm well it seems like it would be ok, i'm not 100% sure what the right way of configuring whether or not we expect c-vol to be there or not | 20:47 |
| jgriffith | smcginnis: maybe next week you and I can sync up on that? | 20:47 |
| *** cknight has quit IRC | 20:47 | |
| smcginnis | jgriffith: But until we stop using the crazy inheritance we have now where pretty much every driver has every interface, we can't do much more. | 20:47 |
| smcginnis | jgriffith: Sounds good. | 20:47 |
| jgriffith | I'd like to somehow do that this release if possible | 20:47 |
| erlon | patrickeast: jgriffith: that would be possible with the interface too, the same way we added the CI WIKIs | 20:47 |
| patrickeast | karthikp_: because sometimes we do expect it to be there, right? like for normal grenade jobs | 20:47 |
| smcginnis | jgriffith: I think it would be a tight schedule, but not impossible at this point. | 20:47 |
| jgriffith | smcginnis: I think I still have a patch out there that removes all of that? It'd need an update/rebase, but it at least exists | 20:48 |
| hemna | the xio driver needs to extend the VolumeDriver | 20:48 |
| hemna | https://review.openstack.org/#/c/355608/7/cinder/volume/drivers/xio.py | 20:48 |
| hemna | I noticed that in my supported driver check patch | 20:48 |
| patrickeast | kind of funny though that apparently you don't actually *need* to | 20:49 |
| hemna | patrickeast, as long as you implement the functions that the manager calls.... | 20:49 |
| smcginnis | Well, really you shouldn't _have_ to inherit. As long as you implement all the right calls and do them right, should be OK. | 20:49 |
| mriedem | jgriffith: https://review.openstack.org/#/c/351918/ would help get the nfs job in a less crazy full 100% failure state | 20:49 |
| hemna | the problem was, my patch was puking because the xio driver didn't have the inherited method to check if it was supported | 20:50 |
| hemna | it should inherit IMHO | 20:50 |
| jgriffith | mriedem: yeah, I'm ok with that I think... but what about the derivitives? | 20:50 |
| hemna | or lets stop doing OOP | 20:50 |
| karthikp_ | patrickeast: yes you are right ..in a normal grenade job.. we do expect it to present..... we tweaked in that way to that we can test cinder in a multi-node setup like c-api c-sch (master) and subnode to run mitaka c-vol and cbak | 20:50 |
| jgriffith | mriedem: they work I believe... ie NetApp NFS, Gluster etc | 20:50 |
| smcginnis | hemna: More like OOPs | 20:50 |
| jgriffith | mriedem: that protocol check would exclude them as well I think... have to look | 20:51 |
| erlon | jgriffith: I *think* the patch will only affect the NFS job | 20:51 |
| mriedem | jgriffith: hmm, good point, not sure about that | 20:51 |
| mriedem | i was basing that on this https://github.com/openstack/devstack-plugin-nfs/blob/master/devstack/override-defaults | 20:52 |
| * erlon checking | 20:52 | |
| jgriffith | erlon: I think a few are inheritting the protocol from the parent class which is the NFS driver | 20:52 |
| patrickeast | karthikp_: looking at the code I think whats there should be ok, on the assumption that the is_service_enabled function looks at like ENABLED_SERVICES or something (like devstack does) | 20:53 |
| jgriffith | mriedem: erlon yeah... netapp for example would get nailed... appears that Gluster would because it pulls protocol from parent | 20:53 |
| jgriffith | likely others as well | 20:53 |
| erlon | mriedem: jgriffith: hmm, yes it will affect, the other NFS drivers report 'nfs' | 20:53 |
| patrickeast | karthikp_: to make sure that we only verify c-vol is alive when we've configured c-vol to be enabled at all | 20:53 |
| mriedem | ok | 20:54 |
| erlon | jgriffith: mriedem: I though you where changing the CI regex when calling tempest | 20:54 |
| mriedem | i think we might be able to handle it in the nfs devstack plugin anyway | 20:54 |
| erlon | mriedem: that would not affect others | 20:54 |
| stevemar | scottda: o/ | 20:54 |
| stevemar | scottda: your comment here: https://review.openstack.org/#/c/349602 | 20:54 |
| jgriffith | I suppose I should make sure everybody isn't just adding a skip on that test in their nfs derivative driver | 20:55 |
| jgriffith | That would pretty much make my head explode | 20:55 |
| stevemar | scottda: i say the "P" release, since that in 2 cycles, or next major release of cinderclient, i don't believe we're allowed to remove things in "feature" releases (bumping from 1.9.0 to 1.10.0), normally when something is removed you bump the "major" number | 20:55 |
| *** xyang has joined #openstack-cinder | 20:56 | |
| jgriffith | hmm... thought I'd poke at the gluster logs... that didn't work out so well | 20:56 |
| jgriffith | http://logs.openstack.org/45/355945/2/check/gate-cinder-tempest-dsvm-glusterfs-nv/b7903cd/ | 20:56 |
| openstackgerrit | xing-yang proposed openstack/cinder: Add group snapshots https://review.openstack.org/328052 | 20:56 |
| erlon | mriedem: where would you need to change to change the tempest regex? In my CI is the jenkins job config, but I know know about the gate job | 20:56 |
| jgriffith | Yeah, Gluster is running the clone tests succesfully | 20:57 |
| scottda | stevemar: You're saying cinderclient would bump the major number to 2.0.0 for removing the 'cinder endpoints' command? I wouldn't think we would do that. But smcginnis might say differently. | 20:58 |
| *** edmondsw has joined #openstack-cinder | 20:58 | |
| stevemar | scottda: i believe that is the case for any python lib that follows semver. if you knowingly make a breaking change you bump the major version | 20:59 |
| stevemar | scottda: first line of http://semver.org/ | 20:59 |
| openstackgerrit | Kaitlin Farr proposed openstack/cinder: Replace key manager with Castellan https://review.openstack.org/280492 | 20:59 |
| scottda | stevemar: Yeah, I get that. | 20:59 |
| stevemar | scottda: *shrugs* number are cheap :) | 21:00 |
| stevemar | numbers | 21:00 |
| *** xyang has quit IRC | 21:00 | |
| stevemar | i don't particularly care either way, it's up to you all | 21:00 |
| stevemar | just wanted to explain my reasoning | 21:00 |
| *** e0ne has quit IRC | 21:00 | |
| scottda | stevemar: That all makes sense. I just don't think we've been that disciplined about our cinderclient versioning. | 21:01 |
| *** bill_az has joined #openstack-cinder | 21:01 | |
| scottda | stevemar: I'll leave it up to smcginnis and others to mull over for a bit. | 21:01 |
| stevemar | scottda: considering you're still at 2, i would think not :D | 21:01 |
| karthikp_ | patrickeast: cool thanks! Do these changes make sense to you ? by this I mean do you know if its being tested in a multi-node like i discussed above | 21:01 |
| stevemar | errr 1.0 | 21:01 |
| scottda | haha | 21:02 |
| stevemar | scottda: i thought all openstack libs were bumped by one when we dropped py26 support... | 21:02 |
| patrickeast | karthikp_: the changes make sense, dunno if we've actually merged a job that runs with that configuration | 21:02 |
| jgriffith | stevemar: NOTHING is cheap | 21:02 |
| stevemar | jgriffith: free lunches are! | 21:02 |
| jgriffith | stevemar: especially things that *seem* like they are | 21:02 |
| patrickeast | scottda: did we turn on the multi-node grenade job? is it in experimental? | 21:02 |
| stevemar | jgriffith: :O | 21:02 |
| jgriffith | :) | 21:02 |
| karthikp_ | patrickeast: we did.... this https://review.openstack.org/#/c/334537/16/jenkins/jobs/devstack-gate.yaml | 21:03 |
| openstackgerrit | Steve Martinelli proposed openstack/python-cinderclient: deprecate command `cinder endpoints` https://review.openstack.org/349602 | 21:03 |
| *** gouthamr has quit IRC | 21:04 | |
| *** xinli has quit IRC | 21:04 | |
| smcginnis | stevemar, scottda: Makes sense on dropping support. | 21:04 |
| smcginnis | So it would stay 1.x for now and marked deprecated. | 21:04 |
| smcginnis | Then when we actually remove it we can bump up to 2.0. | 21:05 |
| karthikp_ | patrickeast: scottda: we got that merged and it is experimental: gate-grenade-dsvm-cinder-multinode | 21:05 |
| *** xyang has joined #openstack-cinder | 21:05 | |
| mriedem | erlon: the job def in project-config | 21:05 |
| scottda | smcginnis: stevemar Yup, it makes sense. We'll just have to make sure we continue to do this properly in the future, as we've never bumped the major version in the past.. | 21:05 |
| mriedem | erlon: for nfs with snapshot and clone, we can just configure tempest rather than blacklist tests with a regex | 21:05 |
| mriedem | s/can/should/ | 21:06 |
| stevemar | smcginnis: coolio | 21:06 |
| patrickeast | karthikp_: gotcha, so yea it seems like that should be ok... but I'm also not an expert at grenade testing so theres a chance Im missing something ;) | 21:06 |
| *** sdague has quit IRC | 21:06 | |
| stevemar | scottda right now "who the hell does this keystone schmuck think he is - coming in here and ruining everything" | 21:06 |
| erlon | mriedem: hmm, and that would be in the devstack plugin | 21:07 |
| mriedem | erlon: yes, i plan on working on it | 21:08 |
| *** narayrak has quit IRC | 21:08 | |
| scottda | stevemar: Your work is welcome. And it's good to force us to think about things like cinderclient versioning, since we obviously missed some stuff. | 21:10 |
| stevemar | :) | 21:11 |
| scottda | stevemar: Hey, what was the deal with https://bugs.launchpad.net/cinder/+bug/1614104 ? Did you previously fix something like that? | 21:11 |
| openstack | Launchpad bug 1614104 in Cinder "Cinder endpoints throws error in Keystone V3" [Undecided,New] - Assigned to Jay Conroy (jayconroy) | 21:11 |
| *** kfarr has quit IRC | 21:13 | |
| *** fifieldt has quit IRC | 21:13 | |
| stevemar | scottda: yep, it was right after the initial migration to keystoneauth | 21:13 |
| stevemar | scottda: i noticed the functional tests were failing | 21:13 |
| stevemar | and fixed it up | 21:13 |
| stevemar | i think 1614104 is using a funky version | 21:13 |
| scottda | stevemar: ok. Can you put something in that bug? i.e. about when that fix was, or a link to it? | 21:14 |
| *** diablo_rojo has quit IRC | 21:17 | |
| *** arch-nemesis has joined #openstack-cinder | 21:18 | |
| openstackgerrit | Steve Martinelli proposed openstack/python-cinderclient: deprecate command `cinder endpoints` https://review.openstack.org/349602 | 21:18 |
| stevemar | scottda: i sure can | 21:18 |
| *** enriquetaso has quit IRC | 21:21 | |
| *** rlrossit_ has quit IRC | 21:22 | |
| karthikp_ | patrickeast: cool :). Thanks. I guess I summon support to get this one merged from infra team | 21:24 |
| *** fifieldt has joined #openstack-cinder | 21:25 | |
| *** jungleboyj has quit IRC | 21:27 | |
| *** alonma has joined #openstack-cinder | 21:37 | |
| *** pauloewerton has quit IRC | 21:39 | |
| *** dustins has quit IRC | 21:41 | |
| *** alonma has quit IRC | 21:41 | |
| *** lpetrut has quit IRC | 21:44 | |
| *** xyang has quit IRC | 21:48 | |
| *** mdenny has quit IRC | 21:50 | |
| patrickeast | jgriffith: hey, looking at https://review.openstack.org/#/c/351275/ did you have any strong objections to merging it as-is? | 21:51 |
| patrickeast | i think all the questions were answered, but kind of with a "nah, i like it this way", so i figured i'd check | 21:51 |
| *** xyang has joined #openstack-cinder | 21:51 | |
| jgriffith | yes, but not strong enough to fight over | 21:51 |
| patrickeast | jgriffith: with the api in general, yea? | 21:52 |
| patrickeast | i'm wondering if we have any leeway to change it up, or if its too late and we might as well ship the client since the api's are locked in forever : / | 21:52 |
| smcginnis | But microversions fix everything! :) | 21:57 |
| scottda | smart Alec | 21:57 |
| patrickeast | smcginnis: well, kinda, we can update the api with a new microversion to make it less bad | 21:57 |
| *** mriedem has quit IRC | 21:58 | |
| openstackgerrit | Merged openstack/cinder: Clean imports in code https://review.openstack.org/355752 | 21:58 |
| patrickeast | smcginnis: and could even just have the client not support the other version of that api, in theory | 21:58 |
| smcginnis | I know, just joking about the panacea it was made out to be. | 21:59 |
| smcginnis | Yep. | 21:59 |
| *** xyang has quit IRC | 21:59 | |
| *** xyang has joined #openstack-cinder | 22:00 | |
| patrickeast | totally unrelated, has anyone tried multi-node FC recently? | 22:02 |
| hemna | not I | 22:02 |
| hemna | what's up | 22:02 |
| openstackgerrit | Merged openstack/cinder: Refactor cinder.utils._copy_volume_with_path https://review.openstack.org/354269 | 22:02 |
| patrickeast | i can't for the life of me get tempest to pass on multi-node jobs for it.... going to spin up a manual env and see if it still has problems | 22:03 |
| patrickeast | hemna: just tests failing, issues with the volume attach/detach kinda thing | 22:03 |
| patrickeast | lemmie dig up a link | 22:03 |
| hemna | logs ? | 22:03 |
| *** alyson_ has quit IRC | 22:03 | |
| *** nikeshm has quit IRC | 22:04 | |
| patrickeast | hemna: both of these failed http://openstack-logs.purestorage.com/PureFCDriver-tempest-dsvm-volume-multi-node-multipath/ | 22:04 |
| patrickeast | although I'm not 100% convinced the env was setup right | 22:04 |
| *** ametts has quit IRC | 22:04 | |
| patrickeast | there are a variety of errors in the logs, but not sure yet who-done-it | 22:04 |
| hemna | http://openstack-logs.purestorage.com/PureFCDriver-tempest-dsvm-volume-multi-node-multipath/2/logs/screen-n-cpu.txt.gz?#_2016-08-16_21_39_31_547 | 22:06 |
| hemna | that's odd | 22:06 |
| patrickeast | thats the new thing we added in for the fc wildcard business | 22:06 |
| *** sdake_ has quit IRC | 22:06 | |
| patrickeast | another "expected" failure : / | 22:06 |
| patrickeast | gg privsep | 22:06 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Break_replication unhandled NoneType error https://review.openstack.org/356687 | 22:07 |
| hemna | fc wildcard business ?! | 22:07 |
| * patrickeast just assumes logging issues with os-brick are privseps fault now | 22:07 | |
| patrickeast | hemna: there was a bug fix just recently...uhh | 22:07 |
| * patrickeast looks for the link | 22:08 | |
| *** jordanP has quit IRC | 22:08 | |
| hemna | https://github.com/openstack/os-brick/commit/28a4d55a0a465ac36ed012a2d634cb64e8f5d599 | 22:08 |
| hemna | that thing ? | 22:08 |
| patrickeast | https://github.com/openstack/os-brick/commit/28a4d55a0a465ac36ed012a2d634cb64e8f5d599 | 22:08 |
| patrickeast | hah | 22:08 |
| patrickeast | yea | 22:08 |
| patrickeast | so it runs grep, and if it fails no biggie, its not that type of hba | 22:09 |
| *** xyang has quit IRC | 22:09 | |
| patrickeast | and it does a wildcard scan | 22:09 |
| patrickeast | well, wildcard + lun | 22:09 |
| patrickeast | but the grep output is logged as that error | 22:10 |
| hemna | ah ok, so yet another example of why privsep is a PITA to use | 22:10 |
| hemna | as it always logs stderr | 22:10 |
| hemna | even though we are try/excepting it | 22:10 |
| hemna | :( | 22:10 |
| patrickeast | actually, now that i look at the change again it shouldn't be logging at error level in brick when we catch the exception | 22:10 |
| patrickeast | we're doing part of that one : / | 22:11 |
| patrickeast | well, if it failed for a real reason we should... | 22:11 |
| patrickeast | bleh | 22:11 |
| hemna | looks like that grep is borked | 22:12 |
| hemna | I dunno | 22:12 |
| *** bardia has quit IRC | 22:12 | |
| patrickeast | oh well, no one uses fc anyway | 22:12 |
| *** bardia has joined #openstack-cinder | 22:13 | |
| hemna | hrmm | 22:14 |
| hemna | come to think of it | 22:14 |
| hemna | I have 3 entries on my server in /sys/class/fc_transport | 22:14 |
| hemna | but no volumes attached | 22:14 |
| hemna | so I'm not sure this is correct to do now | 22:14 |
| patrickeast | i thought those were just the hba pci devices? | 22:15 |
| hemna | I only have 2 HBAs | 22:15 |
| hemna | but there are 3 entries in there | 22:15 |
| patrickeast | weird | 22:16 |
| patrickeast | uhh | 22:17 |
| patrickeast | how many ports? | 22:17 |
| patrickeast | looking at http://linuxmantra.com/2013/10/adding-san-storage-path.html i think you get one per port | 22:18 |
| patrickeast | 3 though is still weird, unless maybe ones in link down state? | 22:18 |
| hemna | yah that's what I don't get | 22:18 |
| hemna | it should be 2 or 4 | 22:18 |
| hemna | 3 ? | 22:18 |
| hemna | wtf | 22:18 |
| patrickeast | or has been gobbled up by libvirt? | 22:18 |
| hemna | I only have 1 vagrant running and it doesn't have any fc | 22:19 |
| * hemna is confused | 22:19 | |
| patrickeast | weird | 22:19 |
| *** JoseMello has quit IRC | 22:20 | |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Added config option to enable SSL https://review.openstack.org/352958 | 22:20 |
| patrickeast | are any of the devices pci_stub'ed or vifo'd? | 22:20 |
| *** sdake has joined #openstack-cinder | 22:21 | |
| hemna | I have a bunch of other vms shut down | 22:21 |
| hemna | maybe one of those have it | 22:21 |
| openstackgerrit | Justin A Wilson proposed openstack/cinder: Added config option to enable SSL https://review.openstack.org/352958 | 22:22 |
| patrickeast | can see with like lspci -vv and look for vfio or pci_stub as the driver | 22:23 |
| hemna | nada | 22:24 |
| *** amrith has joined #openstack-cinder | 22:31 | |
| amrith | smcginnis, hello | 22:32 |
| *** sdake has quit IRC | 22:38 | |
| *** lcurtis has quit IRC | 22:40 | |
| smcginnis | amrith: Hey! | 22:42 |
| amrith | hi smcginnis ... I see you are having good luck with technology today. just imagine what people had to suffer through 60 years ago | 22:42 |
| smcginnis | amrith: I think postal mail would be better at this point. ;) | 22:42 |
| amrith | I'm good anytime from 7am eastern to 5pm eastern | 22:43 |
| amrith | y'all pick | 22:43 |
| smcginnis | amrith: OK, thanks! Hopefully we can get feedback and finalize quickly now. | 22:43 |
| * smcginnis crosses fingers | 22:43 | |
| amrith | i submitted my answers ... | 22:45 |
| amrith | sorry to put you through this | 22:45 |
| amrith | now, you blocked 1h slots but we'll need > 1h :) | 22:45 |
| smcginnis | amrith: Haha, no problem. I guess I'm just too distracted today. It shouldn't have been this difficult. | 22:45 |
| smcginnis | amrith: We can see who is open for multiple slots as well. If we can all only do one hour and then we have to break out into smaller groups for another hour, we can work with that. ;) | 22:46 |
| amrith | true | 22:46 |
| amrith | and surprisingly enough, the answer is NO! | 22:47 |
| amrith | well, ifneedbe is good enough | 22:47 |
| amrith | sounds good | 22:48 |
| amrith | g'night smcginnis | 22:48 |
| smcginnis | amrith: Goodnight. Thanks amrith. | 22:48 |
| *** xyang has joined #openstack-cinder | 22:48 | |
| *** xyang has quit IRC | 22:54 | |
| *** ducttape_ has quit IRC | 22:58 | |
| *** xyang has joined #openstack-cinder | 22:59 | |
| *** xyang has quit IRC | 23:02 | |
| *** erhudy has quit IRC | 23:02 | |
| *** sumit7990 has joined #openstack-cinder | 23:05 | |
| *** sumit has quit IRC | 23:08 | |
| *** xyang1 has quit IRC | 23:10 | |
| *** tpsilva has quit IRC | 23:18 | |
| *** gouthamr has joined #openstack-cinder | 23:28 | |
| *** arch-nemesis has quit IRC | 23:44 | |
| *** diogogmt has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!