| *** karthikp_ has joined #openstack-cinder | 00:03 | |
| *** yangyapeng has quit IRC | 00:04 | |
| *** diogogmt has quit IRC | 00:04 | |
| *** sdake has joined #openstack-cinder | 00:08 | |
| *** arch-nemesis has quit IRC | 00:11 | |
| *** Apoorva has quit IRC | 00:12 | |
| *** Apoorva has joined #openstack-cinder | 00:12 | |
| *** leeantho has quit IRC | 00:19 | |
| *** ngraf has quit IRC | 00:20 | |
| *** itzdilip has quit IRC | 00:26 | |
| *** laughterwym has joined #openstack-cinder | 00:30 | |
| *** laughterwym has quit IRC | 00:31 | |
| *** laughterwym has joined #openstack-cinder | 00:31 | |
| *** laughterwym has quit IRC | 00:31 | |
| *** laughterwym has joined #openstack-cinder | 00:31 | |
| *** gouthamr has quit IRC | 00:32 | |
| *** gouthamr has joined #openstack-cinder | 00:32 | |
| *** karthik__ has joined #openstack-cinder | 00:33 | |
| *** karthikp_ has quit IRC | 00:35 | |
| *** ducttape_ has joined #openstack-cinder | 00:41 | |
| *** ducttape_ has quit IRC | 00:41 | |
| *** ducttape_ has joined #openstack-cinder | 00:41 | |
| *** salv-orl_ has quit IRC | 00:42 | |
| *** david-lyle has quit IRC | 00:42 | |
| *** zhonghua-lee has quit IRC | 00:43 | |
| *** zhonghua-lee has joined #openstack-cinder | 00:44 | |
| *** david-lyle__ has joined #openstack-cinder | 00:45 | |
| *** david-lyle__ has quit IRC | 00:45 | |
| *** Julien-zte has joined #openstack-cinder | 00:47 | |
| *** qeelee has joined #openstack-cinder | 00:47 | |
| *** karthik__ has quit IRC | 00:48 | |
| *** karthikp_ has joined #openstack-cinder | 00:48 | |
| *** sdake has quit IRC | 00:49 | |
| *** ducttape_ has quit IRC | 00:51 | |
| *** ducttape_ has joined #openstack-cinder | 00:53 | |
| *** enriquetaso has quit IRC | 00:54 | |
| *** lixiaoy1 has joined #openstack-cinder | 00:56 | |
| *** iceyao has joined #openstack-cinder | 00:56 | |
| *** Apoorva_ has joined #openstack-cinder | 00:58 | |
| *** iceyao has quit IRC | 01:01 | |
| *** Apoorva has quit IRC | 01:02 | |
| *** Apoorva_ has quit IRC | 01:03 | |
| *** karthik__ has joined #openstack-cinder | 01:03 | |
| *** Suyash has joined #openstack-cinder | 01:04 | |
| *** iceyao has joined #openstack-cinder | 01:04 | |
| *** karthikp_ has quit IRC | 01:07 | |
| *** ducttape_ has quit IRC | 01:07 | |
| lixiaoy1 | DuncanT: please help to review the spec https://review.openstack.org/#/c/248593/ | 01:09 |
|---|---|---|
| lixiaoy1 | DuncanT: this is about creating volume from source vol in different backends: https://review.openstack.org/#/c/315316/ | 01:10 |
| *** yangyapeng has joined #openstack-cinder | 01:17 | |
| *** yangyapeng has quit IRC | 01:17 | |
| *** yangyapeng has joined #openstack-cinder | 01:18 | |
| openstackgerrit | xiexs proposed openstack/cinder: Add strict Boolean checking for volume manage https://review.openstack.org/338244 | 01:19 |
| *** jichen has joined #openstack-cinder | 01:20 | |
| *** EinstCrazy has joined #openstack-cinder | 01:22 | |
| openstackgerrit | tanyy proposed openstack/cinder: Delete unused function in emc_vmax_utils https://review.openstack.org/333766 | 01:25 |
| *** ducttape_ has joined #openstack-cinder | 01:28 | |
| *** mtanino has joined #openstack-cinder | 01:33 | |
| *** karthikp_ has joined #openstack-cinder | 01:34 | |
| *** iceyao has quit IRC | 01:36 | |
| *** laughterwym has quit IRC | 01:36 | |
| *** laughterwym has joined #openstack-cinder | 01:37 | |
| *** Lee1092 has joined #openstack-cinder | 01:37 | |
| *** karthik__ has quit IRC | 01:38 | |
| openstackgerrit | Merged openstack/cinder: Snapshot object has no attribute about size https://review.openstack.org/338270 | 01:38 |
| *** sdake has joined #openstack-cinder | 01:50 | |
| *** wxy__ has quit IRC | 01:50 | |
| *** karthikp_ has quit IRC | 01:52 | |
| *** karthikp_ has joined #openstack-cinder | 01:54 | |
| *** bardia has quit IRC | 01:54 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder-specs: Inspection Mechanism For Capacity Limited Host https://review.openstack.org/308869 | 01:56 |
| openstackgerrit | wanghao proposed openstack/cinder: Fix manage existing volume fails after object merged https://review.openstack.org/331989 | 01:59 |
| *** julim has joined #openstack-cinder | 02:05 | |
| *** karthikp_ has quit IRC | 02:07 | |
| *** diogogmt has joined #openstack-cinder | 02:07 | |
| *** cdelatte has quit IRC | 02:14 | |
| *** salv-orlando has joined #openstack-cinder | 02:19 | |
| *** wanghao_ has joined #openstack-cinder | 02:25 | |
| *** salv-orlando has quit IRC | 02:26 | |
| *** gouthamr has quit IRC | 02:29 | |
| *** ducttape_ has quit IRC | 02:34 | |
| *** xiexs has quit IRC | 02:47 | |
| *** laughterwym has quit IRC | 02:52 | |
| *** laughterwym has joined #openstack-cinder | 02:52 | |
| *** laughterwym has quit IRC | 02:53 | |
| *** laughterwym has joined #openstack-cinder | 02:53 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Initialise oslo.privsep early in main https://review.openstack.org/331973 | 02:54 |
| *** salv-orlando has joined #openstack-cinder | 02:55 | |
| *** Nakato has quit IRC | 02:57 | |
| *** salv-orlando has quit IRC | 02:59 | |
| *** Nakato has joined #openstack-cinder | 03:03 | |
| openstackgerrit | ningwei proposed openstack/cinder: Add cinder backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 03:06 |
| openstackgerrit | Bin Zhou proposed openstack/cinder: remove unused CONF.import_opt https://review.openstack.org/338671 | 03:19 |
| *** laughterwym has quit IRC | 03:19 | |
| *** laughterwym has joined #openstack-cinder | 03:19 | |
| *** ducttape_ has joined #openstack-cinder | 03:22 | |
| *** laughterwym has quit IRC | 03:24 | |
| *** jwcroppe has quit IRC | 03:24 | |
| *** jwcroppe has joined #openstack-cinder | 03:24 | |
| *** Thelo has joined #openstack-cinder | 03:28 | |
| *** jwcroppe has quit IRC | 03:29 | |
| *** salv-orlando has joined #openstack-cinder | 03:29 | |
| *** Thelo has quit IRC | 03:32 | |
| *** sgotliv__ has joined #openstack-cinder | 03:32 | |
| *** ducttape_ has quit IRC | 03:35 | |
| *** laughterwym has joined #openstack-cinder | 03:36 | |
| *** salv-orlando has quit IRC | 03:36 | |
| *** sgotliv__ has quit IRC | 03:38 | |
| *** laughterwym has quit IRC | 03:40 | |
| *** jamielennox is now known as jamielennox|away | 03:43 | |
| *** sdake has quit IRC | 03:48 | |
| *** laughterwym has joined #openstack-cinder | 03:48 | |
| *** laughterwym has quit IRC | 03:52 | |
| *** laughterwym has joined #openstack-cinder | 03:54 | |
| *** shyama has joined #openstack-cinder | 03:54 | |
| *** rajinir has quit IRC | 03:54 | |
| *** Srinu has joined #openstack-cinder | 03:55 | |
| Srinu | hi | 03:55 |
| *** laughterwym has quit IRC | 03:55 | |
| *** jamielennox|away is now known as jamielennox | 03:56 | |
| *** mtanino has quit IRC | 03:57 | |
| *** shyama has quit IRC | 04:00 | |
| *** links has joined #openstack-cinder | 04:03 | |
| *** gouthamr has joined #openstack-cinder | 04:04 | |
| *** laughterwym has joined #openstack-cinder | 04:07 | |
| *** sheel has joined #openstack-cinder | 04:09 | |
| *** karthikp_ has joined #openstack-cinder | 04:10 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:12 | |
| *** Suyash has quit IRC | 04:16 | |
| openstackgerrit | Merged openstack/cinder: Huawei: Use versionedObject https://review.openstack.org/335028 | 04:19 |
| openstackgerrit | Merged openstack/cinder: Implemented unit test cases for HPE LeftHand driver https://review.openstack.org/319207 | 04:19 |
| openstackgerrit | xiaoqin proposed openstack/cinder: Check flashcopy mapping before deleting volume https://review.openstack.org/268701 | 04:19 |
| *** shyama has joined #openstack-cinder | 04:22 | |
| *** harlowja_at_home has quit IRC | 04:22 | |
| openstackgerrit | Merged openstack/cinder: RBD: uppercase rbd_opts for consistency https://review.openstack.org/335593 | 04:23 |
| *** jwcroppe has joined #openstack-cinder | 04:26 | |
| *** jwcroppe has quit IRC | 04:28 | |
| *** sdake has joined #openstack-cinder | 04:32 | |
| *** sgotliv__ has joined #openstack-cinder | 04:35 | |
| *** jamielennox is now known as jamielennox|away | 04:35 | |
| *** karthikp_ has quit IRC | 04:36 | |
| *** karthikp_ has joined #openstack-cinder | 04:37 | |
| *** Poornima has joined #openstack-cinder | 04:37 | |
| *** gouthamr has quit IRC | 04:38 | |
| *** salv-orlando has joined #openstack-cinder | 04:39 | |
| *** karthikp_ has quit IRC | 04:41 | |
| *** sgotliv__ has quit IRC | 04:41 | |
| *** jamielennox|away is now known as jamielennox | 04:42 | |
| *** sgotliv__ has joined #openstack-cinder | 04:44 | |
| *** salv-orlando has quit IRC | 04:47 | |
| *** karthikp_ has joined #openstack-cinder | 04:47 | |
| *** salv-orlando has joined #openstack-cinder | 04:49 | |
| *** sdake has quit IRC | 04:51 | |
| *** sgotliv__ has quit IRC | 04:52 | |
| *** sdake has joined #openstack-cinder | 04:55 | |
| *** karthikp_ has quit IRC | 04:56 | |
| *** ducttape_ has joined #openstack-cinder | 05:06 | |
| *** gouthamr has joined #openstack-cinder | 05:06 | |
| *** laughter_ has joined #openstack-cinder | 05:10 | |
| *** lpetrut has joined #openstack-cinder | 05:10 | |
| *** ducttape_ has quit IRC | 05:12 | |
| *** laughterwym has quit IRC | 05:13 | |
| *** wxy has quit IRC | 05:20 | |
| *** savihou has joined #openstack-cinder | 05:21 | |
| *** asselin__ has joined #openstack-cinder | 05:22 | |
| *** bharathk has joined #openstack-cinder | 05:23 | |
| *** yangyapeng has quit IRC | 05:25 | |
| *** yangyapeng has joined #openstack-cinder | 05:25 | |
| *** sdake has quit IRC | 05:26 | |
| *** asselin__ has quit IRC | 05:26 | |
| *** bardia has joined #openstack-cinder | 05:27 | |
| *** M00nr41n has joined #openstack-cinder | 05:37 | |
| *** gcb has joined #openstack-cinder | 05:37 | |
| *** ChubYann has quit IRC | 05:37 | |
| *** karthikp_ has joined #openstack-cinder | 05:39 | |
| DuncanT | Hey, it's bug squash in china, and they only invite me to keep the review flow good, so I'm likely to be pinging here for specific review if people would be kind enough. First one is https://review.openstack.org/#/c/335388/ | 05:45 |
| *** Srinu has quit IRC | 05:50 | |
| *** shyama has quit IRC | 05:51 | |
| *** shyama has joined #openstack-cinder | 05:51 | |
| *** lpetrut has quit IRC | 05:54 | |
| *** shyh has joined #openstack-cinder | 05:54 | |
| shyh | hi | 05:54 |
| shyh | i'm trying to run a manage volume command on HEAD and it seems brocken | 05:55 |
| shyh | anyone else has this issue? | 05:55 |
| *** lpetrut has joined #openstack-cinder | 05:58 | |
| *** laughter_ has quit IRC | 05:58 | |
| *** wanghao_ has quit IRC | 05:58 | |
| *** laughterwym has joined #openstack-cinder | 05:58 | |
| *** mjura has joined #openstack-cinder | 05:59 | |
| *** bardia has quit IRC | 06:00 | |
| DuncanT | shyh: I've not heard anybody else report it. What backend driver are you using? It might be backend specific | 06:01 |
| *** shyama has quit IRC | 06:01 | |
| shyh | xtremio :-) | 06:01 |
| shyh | the error starts at the api service | 06:02 |
| DuncanT | I'm in the middle of some work on my only handy devstack instance at the moment, so I can't easily test. Can you test on devstack? | 06:03 |
| DuncanT | (devstack/lvm) | 06:03 |
| DuncanT | I don't think tempest covers manage/unmanage at all, so it is entirely possible something got broken | 06:03 |
| shyh | TRACE cinder.volume.api Traceback (most recent call last): TRACE cinder.volume.api File "/usr/local/lib/python2.7/dist-packages/taskflow/engines/action_engine/executor.py", line 56, TRACE cinder.volume.api result = task.execute(**arguments) TRACE cinder.volume.api File "/opt/stack/cinder/cinder/volume/flows/api/manage_existing.py", line 72, in execute TRACE cinder.volume.api volume = objects.Volume(context, volum | 06:05 |
| shyh | this from api so no driver involvment | 06:05 |
| shyh | i tried to change this volume = objects.Volume(context, volume_properties) to volume = objects.Volume(context=context, **volume_properties) | 06:06 |
| shyh | it fixed the issue here but got a lot of errors later | 06:06 |
| *** mjura has quit IRC | 06:07 | |
| *** karthikp_ has quit IRC | 06:07 | |
| *** karthikp_ has joined #openstack-cinder | 06:08 | |
| shyh | this is the rest of the trace File "/opt/stack/cinder/cinder/objects/volume.py", line 179, in __init__ super(Volume, self).__init__(*args, **kwargs) TypeError: __init__() takes at most 2 arguments (3 given) | 06:08 |
| *** jwcroppe has joined #openstack-cinder | 06:09 | |
| *** sdake has joined #openstack-cinder | 06:10 | |
| *** lpetrut has quit IRC | 06:10 | |
| DuncanT | shyh: Looking at the create flow, it looks like your fix is correct | 06:11 |
| *** pgadiya has joined #openstack-cinder | 06:12 | |
| shyh | DuncanT: however it looks like the metadata of volume object doesn't get set correctly and i'm getting a lot of errors around it | 06:12 |
| DuncanT | I'm guessing that the commit that changes manage to use oslo versioned objects is buggy. (commit 092a01f2). You might be able to reverse that commit to make some progress, but please raise a bug since it definitely looks broken. Ping that bug id to me here and I'll take a look within the hour - I'm at a bugsquash event for the day, as it happens | 06:17 |
| *** laughterwym has quit IRC | 06:17 | |
| *** laughterwym has joined #openstack-cinder | 06:18 | |
| *** lkuchlan has joined #openstack-cinder | 06:18 | |
| *** rcernin has joined #openstack-cinder | 06:18 | |
| shyh | DuncanT: this is what I get next from api service | 06:19 |
| *** sdake has quit IRC | 06:19 | |
| shyh | cinder.volume.api File "/opt/stack/cinder/cinder/objects/volume.py", line 200, in _reset_metadata_tracking cinder.volume.api if 'metadata' in self else {}) cinder.volume.api TypeError: 'NoneType' object is not iterable | 06:19 |
| *** sdake has joined #openstack-cinder | 06:20 | |
| shyh | Traceback (most recent call last): File "/usr/local/lib/python2.7/dist-packages/taskflow/engines/action_engine/executor.py", line 56, result = task.execute(**arguments) File "/opt/stack/cinder/cinder/volume/flows/api/manage_existing.py", line 72, in execute volume = objects.Volume(context=context, **volume_properties) File "/opt/stack/cinder/cinder/objects/volume.py", line 184, in __init__ self._reset_metad | 06:20 |
| *** mjura has joined #openstack-cinder | 06:20 | |
| *** laughter_ has joined #openstack-cinder | 06:22 | |
| *** laughterwym has quit IRC | 06:23 | |
| DuncanT | Ok, it looks pretty broken. Please raise a bug and I'll dig into it ASAP | 06:24 |
| *** jwcroppe has quit IRC | 06:24 | |
| *** lpetrut has joined #openstack-cinder | 06:24 | |
| openstackgerrit | xiaoqin proposed openstack/cinder: Add multipath enhancement to Storwize iSCSI driver https://review.openstack.org/308882 | 06:26 |
| *** karthikp_ has quit IRC | 06:26 | |
| *** laughter_ has quit IRC | 06:28 | |
| shyh | DuncanT: thanks | 06:28 |
| *** laughterwym has joined #openstack-cinder | 06:28 | |
| *** sdake has quit IRC | 06:29 | |
| *** laughterwym has quit IRC | 06:33 | |
| *** sdake has joined #openstack-cinder | 06:39 | |
| *** liverpooler has joined #openstack-cinder | 06:39 | |
| *** laughterwym has joined #openstack-cinder | 06:39 | |
| *** lkuchlan has quit IRC | 06:42 | |
| *** rdx_matt has joined #openstack-cinder | 06:44 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Modify array LUNs naming rules https://review.openstack.org/305642 | 06:44 |
| *** jichen has quit IRC | 06:46 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Add cinder backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 06:47 |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Reduce logging level of API validation messages. https://review.openstack.org/338715 | 06:47 |
| *** winston-d_ has joined #openstack-cinder | 06:49 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Reduce logging level of API validation messages. https://review.openstack.org/338715 | 06:50 |
| DuncanT | A nice easy one: https://review.openstack.org/#/c/338715/ | 06:50 |
| *** laughterwym has quit IRC | 06:51 | |
| *** laughterwym has joined #openstack-cinder | 06:51 | |
| DuncanT | Ready for review: https://review.openstack.org/#/c/336427/ | 06:53 |
| *** lkuchlan has joined #openstack-cinder | 06:55 | |
| shyh | DuncanT: looks like there is already a fix to the manage volume issue, waiting for workflow https://review.openstack.org/#/c/331989 | 06:55 |
| *** sdake has quit IRC | 06:55 | |
| *** laughterwym has quit IRC | 06:55 | |
| DuncanT | Client bug: https://review.openstack.org/#/c/335327 | 06:56 |
| DuncanT | shyh: Looking now | 06:56 |
| DuncanT | shyh: Can you cherry-pick that onto your system and see if it fixes the issues, please? | 06:57 |
| shyh | DuncanT: sure | 06:57 |
| *** lpetrut has quit IRC | 06:58 | |
| DuncanT | shyh: It looks about right, and it looks like it explains why you were getting problems with None type not being iterable with your fix | 06:58 |
| *** rdx_matt has quit IRC | 06:58 | |
| shyh | DuncanT: yap | 06:59 |
| *** rdx_matt has joined #openstack-cinder | 06:59 | |
| *** lkuchlan has quit IRC | 07:01 | |
| openstackgerrit | liyuanzhen proposed openstack/python-cinderclient: OS_TENANT_NAME is not required when we have OS_PROJECT_NAME https://review.openstack.org/325067 | 07:01 |
| *** tesseract- has joined #openstack-cinder | 07:03 | |
| *** laughterwym has joined #openstack-cinder | 07:03 | |
| openstackgerrit | Weiwei Gu proposed openstack/cinder: LVM+tgt:iscsi target deleted after VM live migrate https://review.openstack.org/338721 | 07:06 |
| *** laughter_ has joined #openstack-cinder | 07:07 | |
| *** laughterwym has quit IRC | 07:08 | |
| *** shyh_ has joined #openstack-cinder | 07:08 | |
| openstackgerrit | jingtao liang proposed openstack/cinder: Replace 10 ** 9 with a constant units.G https://review.openstack.org/338723 | 07:08 |
| *** shyh has quit IRC | 07:10 | |
| *** laughter_ has quit IRC | 07:11 | |
| *** Lickitysplitted has quit IRC | 07:12 | |
| shyh_ | DuncanT: cherry picked the fix and now it works | 07:12 |
| *** Lickitysplitted has joined #openstack-cinder | 07:12 | |
| *** gouthamr has quit IRC | 07:14 | |
| *** rdx_matt has quit IRC | 07:15 | |
| DuncanT | shyh_: That's great, thanks for testing | 07:16 |
| *** avishay has joined #openstack-cinder | 07:17 | |
| *** rdx_matt has joined #openstack-cinder | 07:17 | |
| *** rdx_matt has quit IRC | 07:17 | |
| *** rdx_matt has joined #openstack-cinder | 07:18 | |
| *** laughterwym has joined #openstack-cinder | 07:18 | |
| *** laughterwym has quit IRC | 07:22 | |
| *** laughterwym has joined #openstack-cinder | 07:22 | |
| *** rdx_matt has quit IRC | 07:22 | |
| *** laughterwym has joined #openstack-cinder | 07:22 | |
| weiweigu | DuncanT: Please help to review this patch: https://review.openstack.org/#/c/338721, thanks | 07:22 |
| *** akapil has joined #openstack-cinder | 07:23 | |
| openstackgerrit | LisaLi proposed openstack/os-brick: Fix multipath iSCSI encrypted volume attach failure https://review.openstack.org/338017 | 07:25 |
| *** amoralej|off is now known as amoralej | 07:29 | |
| *** zhangjn has quit IRC | 07:29 | |
| *** gouthamr has joined #openstack-cinder | 07:30 | |
| *** zhangjn has joined #openstack-cinder | 07:32 | |
| *** jwcroppe has joined #openstack-cinder | 07:32 | |
| *** yangyape_ has joined #openstack-cinder | 07:34 | |
| *** yangyapeng has quit IRC | 07:38 | |
| *** rdx_matt has joined #openstack-cinder | 07:43 | |
| *** shyh_ has quit IRC | 07:44 | |
| *** yangyape_ has quit IRC | 07:46 | |
| *** salv-orlando has quit IRC | 07:51 | |
| *** salv-orlando has joined #openstack-cinder | 07:52 | |
| *** gouthamr has quit IRC | 07:54 | |
| *** mvk has joined #openstack-cinder | 07:57 | |
| *** nkrinner_afk is now known as nkrinner | 07:58 | |
| *** zzzeek has quit IRC | 08:00 | |
| openstackgerrit | LisaLi proposed openstack/cinder-specs: Retype encrypted volumes https://review.openstack.org/248593 | 08:00 |
| *** zzzeek has joined #openstack-cinder | 08:00 | |
| *** nkrinner has quit IRC | 08:01 | |
| *** yangyapeng has joined #openstack-cinder | 08:01 | |
| *** jwcroppe has quit IRC | 08:05 | |
| *** savihou has quit IRC | 08:06 | |
| *** nkrinner has joined #openstack-cinder | 08:06 | |
| *** savihou has joined #openstack-cinder | 08:06 | |
| *** tyrola has joined #openstack-cinder | 08:09 | |
| *** shyama has joined #openstack-cinder | 08:09 | |
| *** tyrola has quit IRC | 08:10 | |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 08:10 |
| *** tyrola has joined #openstack-cinder | 08:10 | |
| *** nkrinner has quit IRC | 08:12 | |
| *** ducttape_ has joined #openstack-cinder | 08:12 | |
| openstackgerrit | Duncan Thomas proposed openstack/cinder: Add cinder backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 08:12 |
| *** yangyape_ has joined #openstack-cinder | 08:17 | |
| *** yangyapeng has quit IRC | 08:17 | |
| openstackgerrit | Merged openstack/cinder: check the validity of metadata when update volume https://review.openstack.org/336427 | 08:17 |
| *** ducttape_ has quit IRC | 08:17 | |
| *** yangyapeng has joined #openstack-cinder | 08:18 | |
| *** openstackgerrit has quit IRC | 08:18 | |
| *** yangyape_ has quit IRC | 08:18 | |
| *** openstackgerrit has joined #openstack-cinder | 08:19 | |
| *** lpetrut has joined #openstack-cinder | 08:21 | |
| *** lkuchlan has joined #openstack-cinder | 08:23 | |
| *** nkrinner has joined #openstack-cinder | 08:26 | |
| *** ociuhandu has joined #openstack-cinder | 08:35 | |
| *** jwcroppe has joined #openstack-cinder | 08:35 | |
| *** jwcroppe has quit IRC | 08:37 | |
| DuncanT | lixiaoy1: https://etherpad.openstack.org/p/cinder-nova-api-changes | 08:37 |
| openstackgerrit | ningwei proposed openstack/cinder: Add cinder backend driver for Huawei FusionStorage https://review.openstack.org/328887 | 08:40 |
| *** jwcroppe has joined #openstack-cinder | 08:41 | |
| lixiaoy1 | DuncanT: thanks | 08:41 |
| openstackgerrit | Mattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio https://review.openstack.org/332032 | 08:46 |
| *** laughterwym has quit IRC | 08:47 | |
| *** thurloat has quit IRC | 08:48 | |
| *** laughterwym has joined #openstack-cinder | 08:48 | |
| *** thurloat has joined #openstack-cinder | 08:48 | |
| *** Julien-zte has quit IRC | 08:51 | |
| *** tyrola has quit IRC | 08:51 | |
| *** laughterwym has quit IRC | 08:52 | |
| openstackgerrit | Abhishek Kekane proposed openstack/python-cinderclient: Fix log statement logged twice https://review.openstack.org/309276 | 08:55 |
| *** tyrola has joined #openstack-cinder | 09:00 | |
| *** salv-orlando has quit IRC | 09:03 | |
| *** jwcroppe has quit IRC | 09:04 | |
| *** yangyape_ has joined #openstack-cinder | 09:05 | |
| *** cheneydc has joined #openstack-cinder | 09:08 | |
| *** yangyapeng has quit IRC | 09:09 | |
| *** ducttape_ has joined #openstack-cinder | 09:13 | |
| *** ociuhandu has quit IRC | 09:14 | |
| *** ducttape_ has quit IRC | 09:17 | |
| *** tyrola has quit IRC | 09:18 | |
| *** kaisers1 has joined #openstack-cinder | 09:19 | |
| *** tyrola has joined #openstack-cinder | 09:19 | |
| *** gahan has joined #openstack-cinder | 09:20 | |
| gahan | I created a cinder volume and wonder if it's been allocated on one node or across few nodes? if one can I check which it is? | 09:21 |
| *** savihou has quit IRC | 09:22 | |
| *** savihou has joined #openstack-cinder | 09:23 | |
| lixiaoy1 | gahan: one node. you can use "cinder show volume_id" to look into the host | 09:25 |
| lixiaoy1 | geguileo: Hi Gorka, about your comments " I still believe it would be good to mention the microversion that adds this in the commit message." in my patch https://review.openstack.org/#/c/263179/14//COMMIT_MSG, sorry I don't understand it. | 09:27 |
| geguileo | lixiaoy1: You just add to the commit message something like "This new API endpoint is added in microversion 3.x" or something like that | 09:28 |
| lixiaoy1 | geguileo: oh, I just update the title to add microversion in patch 13. OK, got it, thanks | 09:29 |
| geguileo | lixiaoy1: It's just a nit, no big deal | 09:30 |
| lixiaoy1 | geguileo: :) | 09:30 |
| openstackgerrit | LisaLi proposed openstack/cinder: Add backup update function (microversion) https://review.openstack.org/263179 | 09:31 |
| openstackgerrit | liuke proposed openstack/cinder: Huawei: Backend capabilies report optimization https://review.openstack.org/337061 | 09:34 |
| *** cheneydc has quit IRC | 09:40 | |
| gahan | lixiaoy1: thanks | 09:42 |
| *** locust has joined #openstack-cinder | 09:42 | |
| locust | hello | 09:43 |
| *** bjornar_ has joined #openstack-cinder | 09:46 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Handle API NotFound exceptions at WSGI level https://review.openstack.org/314193 | 09:49 |
| openstackgerrit | Merged openstack/cinder: Fix manage existing volume fails after object merged https://review.openstack.org/331989 | 09:49 |
| *** avishay has quit IRC | 09:52 | |
| *** shyama has quit IRC | 09:55 | |
| gahan | lixiaoy1: unfortunately host_name is not specified, comes back as 'None' | 09:57 |
| *** yangyape_ has quit IRC | 10:04 | |
| *** lixiaoy1 has quit IRC | 10:05 | |
| *** gcb has quit IRC | 10:08 | |
| openstackgerrit | Mattan Shalev proposed openstack/cinder: Added initial backend ISCSI driver for Reduxio https://review.openstack.org/332032 | 10:09 |
| openstackgerrit | Davanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c https://review.openstack.org/303766 | 10:10 |
| *** EinstCrazy has quit IRC | 10:11 | |
| *** vbala has joined #openstack-cinder | 10:11 | |
| *** qeelee has quit IRC | 10:13 | |
| *** ducttape_ has joined #openstack-cinder | 10:13 | |
| *** ducttape_ has quit IRC | 10:18 | |
| *** salv-orlando has joined #openstack-cinder | 10:22 | |
| *** akapil has quit IRC | 10:25 | |
| *** thurloat has quit IRC | 10:25 | |
| *** thurloat has joined #openstack-cinder | 10:26 | |
| *** jwcroppe has joined #openstack-cinder | 10:26 | |
| *** jwcroppe has quit IRC | 10:27 | |
| *** akapil has joined #openstack-cinder | 10:29 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: OVO request_spec in migrate/manage/retype calls https://review.openstack.org/252458 | 10:30 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec in create volume calls to ovo https://review.openstack.org/246600 | 10:30 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Switch request_spec_list in create CG to OVO https://review.openstack.org/253033 | 10:30 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Convert filter_properties to OVO (create, retype) https://review.openstack.org/257305 | 10:30 |
| *** sgotliv__ has joined #openstack-cinder | 10:33 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Replace use of mox with mock in test_base_filter https://review.openstack.org/338859 | 10:34 |
| *** asselin__ has joined #openstack-cinder | 10:35 | |
| *** yuriy_n17 has quit IRC | 10:39 | |
| openstackgerrit | zengjia proposed openstack/cinder: Add MacroSAN cinder driver https://review.openstack.org/337415 | 10:40 |
| *** laughterwym has joined #openstack-cinder | 10:40 | |
| gahan | is it possible to share a cinder volume across instances? | 10:41 |
| openstackgerrit | xiexs proposed openstack/cinder: Add validation for the metadata properties https://review.openstack.org/330913 | 10:42 |
| *** salv-orl_ has joined #openstack-cinder | 10:43 | |
| *** salv-orlando has quit IRC | 10:46 | |
| *** qeelee has joined #openstack-cinder | 10:46 | |
| *** vbala has quit IRC | 10:48 | |
| *** yuriy_n17 has joined #openstack-cinder | 10:48 | |
| erlon | gahan: yes you can, though not all backends support that: https://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/multi-attach-volume.html | 10:51 |
| *** jwcroppe has joined #openstack-cinder | 10:58 | |
| swamireddy | smcginnis: hi | 11:02 |
| *** f10d4 has quit IRC | 11:03 | |
| *** salv-orlando has joined #openstack-cinder | 11:04 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Replace use of mox with mock in test_base_filter https://review.openstack.org/338859 | 11:04 |
| *** salv-orlando has quit IRC | 11:04 | |
| *** ccesario has joined #openstack-cinder | 11:06 | |
| *** salv-orl_ has quit IRC | 11:07 | |
| openstackgerrit | Alyson proposed openstack/cinder: Improve Hitachi HNAS volume drivers log messages https://review.openstack.org/310785 | 11:07 |
| *** laughterwym has quit IRC | 11:09 | |
| openstackgerrit | Alyson proposed openstack/cinder: HNAS: Add support for manage/unmanage snapshots in NFS driver https://review.openstack.org/333992 | 11:13 |
| *** ducttape_ has joined #openstack-cinder | 11:14 | |
| *** f10d4 has joined #openstack-cinder | 11:16 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Modify array LUNs naming rules https://review.openstack.org/305642 | 11:16 |
| *** ducttape_ has quit IRC | 11:19 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Support microversions on inherited Controllers https://review.openstack.org/336030 | 11:20 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve api_version decorator to avoid noqa https://review.openstack.org/338883 | 11:20 |
| *** amoralej is now known as amoralej|lunch | 11:21 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Modify array LUNs naming rules https://review.openstack.org/305642 | 11:22 |
| *** f10d4 has quit IRC | 11:28 | |
| *** jwcroppe has quit IRC | 11:30 | |
| *** GB21 has joined #openstack-cinder | 11:32 | |
| *** laughterwym has joined #openstack-cinder | 11:35 | |
| *** ociuhandu has joined #openstack-cinder | 11:35 | |
| *** yangyapeng has joined #openstack-cinder | 11:36 | |
| *** tpsilva has joined #openstack-cinder | 11:40 | |
| *** EinstCrazy has joined #openstack-cinder | 11:40 | |
| *** laughterwym has quit IRC | 11:42 | |
| *** f10d4 has joined #openstack-cinder | 11:42 | |
| *** laughterwym has joined #openstack-cinder | 11:43 | |
| *** akapil has quit IRC | 11:44 | |
| *** akapil has joined #openstack-cinder | 11:44 | |
| *** avishay has joined #openstack-cinder | 11:46 | |
| *** laughterwym has quit IRC | 11:47 | |
| *** cdelatte has joined #openstack-cinder | 11:48 | |
| *** jwcroppe has joined #openstack-cinder | 11:49 | |
| *** bharathk has quit IRC | 11:51 | |
| *** jwcroppe has quit IRC | 11:54 | |
| *** f10d4 has quit IRC | 11:59 | |
| *** f10d4 has joined #openstack-cinder | 12:00 | |
| *** yangyapeng has quit IRC | 12:01 | |
| *** yangyapeng has joined #openstack-cinder | 12:02 | |
| *** ducttape_ has joined #openstack-cinder | 12:03 | |
| *** ducttape_ has quit IRC | 12:04 | |
| openstackgerrit | Ryan Liang proposed openstack/cinder: VNX: New Cinder driver in Newton https://review.openstack.org/312427 | 12:04 |
| *** ducttape_ has joined #openstack-cinder | 12:05 | |
| *** IanGovett has joined #openstack-cinder | 12:10 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix 'driver does not honor clone/snapshot' bugs https://review.openstack.org/336092 | 12:10 |
| *** mdavidson has quit IRC | 12:21 | |
| *** liverpooler has quit IRC | 12:22 | |
| *** tyrola has quit IRC | 12:23 | |
| *** tyrola has joined #openstack-cinder | 12:24 | |
| *** f10d4 has quit IRC | 12:27 | |
| *** eharney has joined #openstack-cinder | 12:27 | |
| *** ducttape_ has quit IRC | 12:28 | |
| *** f10d4 has joined #openstack-cinder | 12:28 | |
| *** akapil_ has joined #openstack-cinder | 12:30 | |
| *** f10d4 has quit IRC | 12:30 | |
| *** f10d4 has joined #openstack-cinder | 12:30 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Support backup snapshot https://review.openstack.org/329336 | 12:32 |
| *** akapil has quit IRC | 12:32 | |
| *** winston-d_ has quit IRC | 12:33 | |
| *** tyrola has quit IRC | 12:36 | |
| *** zzzeek has quit IRC | 12:37 | |
| *** tyrola has joined #openstack-cinder | 12:37 | |
| *** mdavidson has joined #openstack-cinder | 12:37 | |
| *** IanGovett has quit IRC | 12:38 | |
| *** EinstCrazy has quit IRC | 12:40 | |
| *** wxy has joined #openstack-cinder | 12:41 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Support backup snapshot https://review.openstack.org/329336 | 12:44 |
| *** sandanar has joined #openstack-cinder | 12:50 | |
| *** f10d4 has quit IRC | 12:52 | |
| *** jwcroppe has joined #openstack-cinder | 12:52 | |
| *** jwcroppe has quit IRC | 12:57 | |
| *** rlrossit has joined #openstack-cinder | 12:59 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Modify array LUNs naming rules https://review.openstack.org/305642 | 13:00 |
| *** pgadiya has quit IRC | 13:01 | |
| *** links has quit IRC | 13:02 | |
| *** sdake has joined #openstack-cinder | 13:07 | |
| *** xyang has joined #openstack-cinder | 13:08 | |
| *** xyang1 has joined #openstack-cinder | 13:08 | |
| *** bhavik has joined #openstack-cinder | 13:09 | |
| *** amoralej|lunch is now known as amoralej | 13:10 | |
| *** ducttape_ has joined #openstack-cinder | 13:12 | |
| *** ducttape_ has quit IRC | 13:13 | |
| *** itzdilip has joined #openstack-cinder | 13:14 | |
| *** M00nr41n has quit IRC | 13:15 | |
| openstackgerrit | huanan proposed openstack/cinder: Huawei: Modify array LUNs naming rules https://review.openstack.org/305642 | 13:18 |
| *** GB21 has quit IRC | 13:18 | |
| *** mtanino has joined #openstack-cinder | 13:19 | |
| *** xyang has quit IRC | 13:20 | |
| *** dustins has joined #openstack-cinder | 13:20 | |
| *** diogogmt has quit IRC | 13:21 | |
| *** sdake_ has joined #openstack-cinder | 13:23 | |
| *** diogogmt has joined #openstack-cinder | 13:26 | |
| *** sdake has quit IRC | 13:27 | |
| *** takedakn has joined #openstack-cinder | 13:29 | |
| *** akapil_ has quit IRC | 13:30 | |
| *** adrianofr has joined #openstack-cinder | 13:30 | |
| *** mriedem_afk is now known as mriedem | 13:30 | |
| *** akapil has joined #openstack-cinder | 13:31 | |
| *** akapil has quit IRC | 13:31 | |
| *** shyama has joined #openstack-cinder | 13:31 | |
| *** rlrossit has quit IRC | 13:31 | |
| *** akapil has joined #openstack-cinder | 13:32 | |
| *** akapil has quit IRC | 13:34 | |
| *** rlrossit has joined #openstack-cinder | 13:34 | |
| *** diogogmt has quit IRC | 13:40 | |
| *** mtanino has quit IRC | 13:43 | |
| *** f10d4 has joined #openstack-cinder | 13:43 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 13:44 |
| *** akapil has joined #openstack-cinder | 13:44 | |
| *** zzzeek has joined #openstack-cinder | 13:44 | |
| *** ducttape_ has joined #openstack-cinder | 13:45 | |
| *** akapil has quit IRC | 13:46 | |
| *** akapil has joined #openstack-cinder | 13:46 | |
| *** laughterwym has joined #openstack-cinder | 13:47 | |
| *** vbala has joined #openstack-cinder | 13:58 | |
| dulek | yuriy_n17: Hi! I wanted to ask about my comment - "I can also see you're mocking the new quota_utils method very frequently and always in a positive way. Why not add such mock in cinder.test.TestCase.setUp() and save it into the class? If someone needed negative test case then he would be able to modify return_value of the mock for his test." | 13:58 |
| dulek | yuriy_n17: Can't we go with that approach? This way any new tests will already include the method mocked to return True. | 13:58 |
| *** ericksonsantos has quit IRC | 14:00 | |
| yuriy_n17 | dulek: Hi. Thanks a lot for your comments. I've done it. If I miss it somewhere let me know. | 14:01 |
| *** iceyao_ has joined #openstack-cinder | 14:01 | |
| dulek | yuriy_n17: I've seen the updated patch. I just wonder if mocking the new is_valid_project_id method just once in cinder.test.TestCase.setUp() wouldn't be possible? | 14:02 |
| yuriy_n17 | dulek: Oh. Sure. Thanks. | 14:03 |
| dulek | yuriy_n17: :) This will reduce code duplication and will make writing new tests easier. | 14:04 |
| yuriy_n17 | dulek: Sure. Just a few minutes... | 14:04 |
| *** dustins has quit IRC | 14:05 | |
| *** sheel has quit IRC | 14:05 | |
| openstackgerrit | Alexey Morlang proposed openstack/cinder: Use default provisioning type from config https://review.openstack.org/332030 | 14:11 |
| dulek | geguileo: ping, c-vol A/A. | 14:18 |
| geguileo | dulek: Hi! | 14:18 |
| *** iceyao_ has left #openstack-cinder | 14:18 | |
| dulek | geguileo: So "cluster" setting is in [DEFAULT] section, right…? | 14:18 |
| geguileo | dulek: Yes | 14:19 |
| *** dustins has joined #openstack-cinder | 14:19 | |
| dulek | geguileo: Let's think of having NFS backend as cluster foo, shared between hosts bar and baz. | 14:19 |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO: support of snapshot manage commands https://review.openstack.org/339024 | 14:20 |
| dulek | geguileo: Would I be able to also add totally independent LVM backends on such hosts? | 14:20 |
| geguileo | dulek: You would need to run a different Cinder service with another configuration file | 14:21 |
| dulek | geguileo: I'm reconfiguring my DevStack to test your patches and this struck me. | 14:21 |
| dulek | geguileo: And if cluster setting was on backend level settings? | 14:22 |
| dulek | geguileo: Like it's done for backend_host setting. | 14:22 |
| geguileo | dulek: I don't think I'll be doing that anytime soon | 14:23 |
| geguileo | dulek: Honestly I'm getting tired of working on this... | 14:23 |
| geguileo | dulek: I think that's a weird configuration | 14:24 |
| geguileo | dulek: And there is a workaround, so... | 14:24 |
| *** EinstCra_ has joined #openstack-cinder | 14:24 | |
| *** Poornima has quit IRC | 14:24 | |
| dulek | geguileo: Okay, definitely we can focus to solve simplest case first. | 14:24 |
| geguileo | dulek: And I think it could work if the backend names for those LVM are different | 14:25 |
| geguileo | dulek: In the different nodes of the cluster | 14:25 |
| geguileo | dulek: And that would be a good solution | 14:25 |
| dulek | geguileo: Yeah, then it will probably work, but it would be hard to setup a volume type that will target all LVMs. | 14:25 |
| * dulek gets back into cinder.conf. | 14:25 | |
| *** mtanino has joined #openstack-cinder | 14:28 | |
| *** rajinir has joined #openstack-cinder | 14:29 | |
| *** gcb has joined #openstack-cinder | 14:29 | |
| *** mriedem1 has joined #openstack-cinder | 14:29 | |
| *** xinli has joined #openstack-cinder | 14:31 | |
| *** mriedem has quit IRC | 14:31 | |
| openstackgerrit | Ricardo Noriega proposed openstack/cinder: Fix simple typos https://review.openstack.org/335087 | 14:31 |
| *** diogogmt has joined #openstack-cinder | 14:32 | |
| *** Suyash has joined #openstack-cinder | 14:33 | |
| openstackgerrit | Huanxuan Ao proposed openstack/python-cinderclient: Fix output error for type-show command https://review.openstack.org/325739 | 14:37 |
| *** diablo_rojo has joined #openstack-cinder | 14:39 | |
| erlon | geguileo: Hey Gorka, I'm looking at the tests suggestion you'v added: https://etherpad.openstack.org/p/Cinder-testing | 14:40 |
| erlon | geguileo: the Multinode tests for AA | 14:41 |
| *** jungleboyj has joined #openstack-cinder | 14:41 | |
| geguileo | erlon: What did I add there? r:-?? | 14:42 |
| erlon | geguileo: it's not clear to me how would that tests work, Its been a while since I looked at first to the AA implementation | 14:42 |
| geguileo | erlon: There's no AA implementation yet... | 14:42 |
| geguileo | erlon: But there are updated specs | 14:42 |
| geguileo | erlon: And some updated patches | 14:42 |
| erlon | geguileo: wasn't you? sorry, I always recall to you when I see AA :) | 14:42 |
| geguileo | erlon: I am working on AA | 14:43 |
| geguileo | erlon: I'm just not working on the tests | 14:43 |
| geguileo | erlon: The day has only so many hours | 14:43 |
| erlon | geguileo: hmmm, I saw a few patches about the AA getting merged I think | 14:44 |
| dulek | erlon: scottda_vacation was interested in implementing tests for that I think. | 14:44 |
| erlon | geguileo: I though we had already something to work on on tests | 14:44 |
| geguileo | erlon: Probably a couple related to the API races | 14:44 |
| dulek | erlon: And Tooz locks. | 14:44 |
| xinli | dulek: if you have time, could you please review 251989: https://review.openstack.org/#/c/251989/ | 14:44 |
| *** salv-orlando has joined #openstack-cinder | 14:44 | |
| erlon | dulek: dulek, yes, he is working in the migration tests, but I think he hanven't started the AA | 14:45 |
| *** salv-orl_ has joined #openstack-cinder | 14:45 | |
| erlon | geguileo: do you have an idea on when would you have that in a point we can start to put things together to test? | 14:46 |
| geguileo | erlon: I think next week we'll be able to test things against the patches that I have upstream | 14:47 |
| geguileo | erlon: And in the gate with the Depends-On we could test them in gate as well | 14:48 |
| erlon | geguileo: ok, ill try to get myself updated with the specs and start setting up a local AA env | 14:48 |
| erlon | geguileo: I never done AA with devstack before, how do you set it up? | 14:49 |
| *** salv-orlando has quit IRC | 14:49 | |
| geguileo | erlon: For now, I'm being lazy and running both cinder nodes in the same VM | 14:49 |
| dulek | geguileo: Hey, that's cool idea. :D | 14:49 |
| *** takedakn has quit IRC | 14:50 | |
| dulek | erlon: I'm setting up two c-vols pointing to same NFS share to simulate it. | 14:50 |
| erlon | that is only 1 A :) | 14:50 |
| geguileo | erlon: I just add a different config to each of the services defining the cluster configuration option on top of the common configuration | 14:50 |
| geguileo | erlon: You run 2 c-vol services in the same VM | 14:50 |
| erlon | geguileo: mhm | 14:51 |
| erlon | geguileo: I thought I would need to setup the HAproxy stufs | 14:51 |
| geguileo | erlon: It's good enough to manually test most of the stuff (excluding the DLM side of things) | 14:51 |
| *** ametts has joined #openstack-cinder | 14:51 | |
| geguileo | erlon: For c-vol A-A you don't need HAProxy | 14:51 |
| *** bjornar_ has quit IRC | 14:52 | |
| geguileo | erlon: For a real HA A-A setup things are completely different | 14:52 |
| erlon | geguileo: hmm, HAProxy is only for the API services? | 14:52 |
| geguileo | erlon: Yep | 14:53 |
| geguileo | erlon: In Cinder at least | 14:53 |
| erlon | geguileo: that make things easier | 14:53 |
| *** jwcroppe has joined #openstack-cinder | 14:54 | |
| locust | Hello folks, Have anybody faced warning from puppet "Warning: /Apt_key[Add key: D50582E6 from Apt::Source jenkins]: The id should be a full fingerprint (40 characters), see README." | 14:55 |
| locust | ? | 14:55 |
| *** xyang has joined #openstack-cinder | 14:57 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 14:58 |
| *** jwcroppe has quit IRC | 15:00 | |
| *** bharath has joined #openstack-cinder | 15:02 | |
| *** mjura has quit IRC | 15:02 | |
| *** rhagarty__ has quit IRC | 15:03 | |
| *** sdake_ has quit IRC | 15:13 | |
| *** sdake has joined #openstack-cinder | 15:14 | |
| *** obutenko has quit IRC | 15:17 | |
| dansmith | jbernard: have you looked into the cinder ceph fail yet? | 15:17 |
| xinli | geguileo: Ed adds comment in 338542:https://review.openstack.org/#/c/338542/1/cinder/objects/cgsnapshot.py@43 , do you agree to change method _from_db_object () to @classmethod? | 15:18 |
| *** avishay has quit IRC | 15:19 | |
| *** mriedem has joined #openstack-cinder | 15:19 | |
| *** xyang has quit IRC | 15:20 | |
| *** akshai has joined #openstack-cinder | 15:21 | |
| *** david-lyle__ has joined #openstack-cinder | 15:21 | |
| *** mriedem1 has quit IRC | 15:22 | |
| geguileo | xinli: Yes, I agree | 15:23 |
| *** david-lyle__ is now known as david-lyle | 15:24 | |
| *** mriedem1 has joined #openstack-cinder | 15:25 | |
| xinli | geguileo: I have one concern: http://paste.openstack.org/show/526985/ | 15:25 |
| *** mriedem has quit IRC | 15:26 | |
| *** shyama has quit IRC | 15:28 | |
| *** lkuchlan has quit IRC | 15:30 | |
| xinli | geguileo: After modifying the signature, the unit test case test_cgsnapshot passed. | 15:33 |
| *** mriedem1 is now known as mriedem | 15:33 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Fix quota-usage list information with invalid tenant-id https://review.openstack.org/315673 | 15:36 |
| openstackgerrit | Merged openstack/python-cinderclient: Fix _get_rate_limit when resp is None https://review.openstack.org/338278 | 15:37 |
| *** GB21 has joined #openstack-cinder | 15:42 | |
| *** leeantho has joined #openstack-cinder | 15:43 | |
| *** liverpooler has joined #openstack-cinder | 15:46 | |
| *** sdake has quit IRC | 15:48 | |
| *** sdake has joined #openstack-cinder | 15:51 | |
| openstackgerrit | Xinli Guan proposed openstack/cinder: Moving the OPTIONAL_FIELDS inside the OVO object https://review.openstack.org/338542 | 15:54 |
| *** karthikp_ has joined #openstack-cinder | 15:57 | |
| *** seldenr has joined #openstack-cinder | 16:03 | |
| *** bhavik has quit IRC | 16:03 | |
| geguileo | xinli: Sure, and what's the concern? | 16:05 |
| *** sdake has quit IRC | 16:09 | |
| *** rcernin has quit IRC | 16:10 | |
| *** laughterwym has quit IRC | 16:11 | |
| *** arch-nemesis has joined #openstack-cinder | 16:14 | |
| *** satya__ has joined #openstack-cinder | 16:16 | |
| *** nkrinner is now known as nkrinner_afk | 16:18 | |
| *** sdake has joined #openstack-cinder | 16:20 | |
| *** M00nr41n has joined #openstack-cinder | 16:21 | |
| *** tesseract- has quit IRC | 16:23 | |
| *** bardia has joined #openstack-cinder | 16:24 | |
| *** sdake has quit IRC | 16:25 | |
| *** sdake has joined #openstack-cinder | 16:27 | |
| *** ducttape_ has quit IRC | 16:29 | |
| *** mkoderer has quit IRC | 16:30 | |
| *** haplo37_ has joined #openstack-cinder | 16:31 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR add entry point debug tracing https://review.openstack.org/339116 | 16:31 |
| *** mkoderer has joined #openstack-cinder | 16:31 | |
| *** gcb has quit IRC | 16:32 | |
| *** mriedem is now known as mriedem_fudd | 16:32 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: 3PAR add entry point debug tracing https://review.openstack.org/339116 | 16:36 |
| *** lpetrut has quit IRC | 16:37 | |
| *** julim has quit IRC | 16:38 | |
| *** julim has joined #openstack-cinder | 16:38 | |
| *** jungleboyj has quit IRC | 16:39 | |
| *** julim has quit IRC | 16:42 | |
| *** rdx_matt has quit IRC | 16:46 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: LeftHand add entry point tracing https://review.openstack.org/339128 | 16:46 |
| *** bardia has quit IRC | 16:47 | |
| *** tyrola has quit IRC | 16:47 | |
| *** Apoorva has joined #openstack-cinder | 16:48 | |
| *** Apoorva has quit IRC | 16:49 | |
| swamireddy | smcginnis: hi | 16:50 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: HPE XP add entry point debug tracing https://review.openstack.org/339132 | 16:50 |
| swamireddy | smcginnis: could you please check the https://review.openstack.org/#/c/335067/ ? | 16:51 |
| *** jungleboyj has joined #openstack-cinder | 16:51 | |
| *** xinli has quit IRC | 16:52 | |
| karthikp_ | Hi, I got a question.. Whenever we update quota for a particular backend should be consider to update the volumes as well?? | 16:52 |
| karthikp_ | http://paste.openstack.org/show/527009/ | 16:52 |
| *** Apoorva has joined #openstack-cinder | 16:57 | |
| *** jwcroppe has joined #openstack-cinder | 16:58 | |
| *** cdelatte has quit IRC | 16:58 | |
| openstackgerrit | Weiwei Gu proposed openstack/cinder: Fix the exception message of VolumeIsBusy and SnapshotIsBusy https://review.openstack.org/339138 | 16:59 |
| *** jwcroppe has quit IRC | 17:02 | |
| *** savihou has quit IRC | 17:04 | |
| *** sandanar has quit IRC | 17:10 | |
| *** wxy has quit IRC | 17:11 | |
| *** sdake has quit IRC | 17:11 | |
| *** ducttape_ has joined #openstack-cinder | 17:11 | |
| *** rdx_matt has joined #openstack-cinder | 17:13 | |
| *** ducttape_ has quit IRC | 17:15 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 17:16 |
| *** gouthamr has joined #openstack-cinder | 17:17 | |
| *** ducttape_ has joined #openstack-cinder | 17:18 | |
| *** xinli has joined #openstack-cinder | 17:19 | |
| *** rdx_matt has quit IRC | 17:19 | |
| *** julim has joined #openstack-cinder | 17:20 | |
| *** julim has quit IRC | 17:21 | |
| *** julim has joined #openstack-cinder | 17:23 | |
| *** GB21 has quit IRC | 17:23 | |
| *** shyama has joined #openstack-cinder | 17:27 | |
| *** sdake has joined #openstack-cinder | 17:32 | |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add secondary DSM support https://review.openstack.org/338548 | 17:32 |
| *** sdake_ has joined #openstack-cinder | 17:34 | |
| *** rdx_matt has joined #openstack-cinder | 17:34 | |
| *** daneyon has joined #openstack-cinder | 17:35 | |
| *** sdake has quit IRC | 17:37 | |
| *** salv-orlando has joined #openstack-cinder | 17:38 | |
| *** daneyon_ has joined #openstack-cinder | 17:38 | |
| *** rlrossit has quit IRC | 17:38 | |
| *** rdxnoam_ has joined #openstack-cinder | 17:39 | |
| *** salv-orl_ has quit IRC | 17:39 | |
| *** bardia has joined #openstack-cinder | 17:39 | |
| *** rdx_matt has quit IRC | 17:41 | |
| openstackgerrit | Karthik Prabhu Vinod proposed openstack/cinder: Extra-spec as bool key does not clear host passes https://review.openstack.org/338589 | 17:41 |
| *** lpetrut has joined #openstack-cinder | 17:41 | |
| *** daneyon has quit IRC | 17:41 | |
| *** amoralej is now known as amoralej|off | 17:42 | |
| *** akapil has quit IRC | 17:43 | |
| ntpttr | karthikp_: I don't know, I think that individual volume type quotas should be able to be defined independently of volumes in general | 17:44 |
| ntpttr | karthikp_: if some admin for some reason defines that a tenant can have 100 volumes of type 'gold' but only 10 volumes, then the default one would win out but I don't think that means it should automatically set default volumes to 100 | 17:45 |
| *** M00nr41n has quit IRC | 17:47 | |
| karthikp_ | ntpttr: I agree to soem extent..but I think it is logically wrong to allow this then | 17:48 |
| karthikp_ | ntpttr: maybe it should error out whenever per backend volumes are set tot greater than volumes itslef | 17:49 |
| karthikp_ | sorry about the typos | 17:49 |
| ntpttr | karthikp_: It's up to the admin to set logical quotas, I don't think throwing an error would be the best thing to do | 17:50 |
| *** sgotliv__ has quit IRC | 17:50 | |
| *** lpetrut has quit IRC | 17:50 | |
| *** hemna has quit IRC | 17:51 | |
| *** catintheroof has joined #openstack-cinder | 17:53 | |
| *** ducttape_ has quit IRC | 17:53 | |
| *** M00nr41n has joined #openstack-cinder | 17:54 | |
| *** rlrossit has joined #openstack-cinder | 17:54 | |
| *** mriedem_fudd is now known as mriedem | 17:55 | |
| *** M00nr41n has quit IRC | 17:55 | |
| *** hemna has joined #openstack-cinder | 17:55 | |
| *** salv-orlando has quit IRC | 17:55 | |
| *** sdake_ has quit IRC | 17:57 | |
| *** jwcroppe has joined #openstack-cinder | 17:58 | |
| *** salv-orlando has joined #openstack-cinder | 17:58 | |
| *** lpetrut has joined #openstack-cinder | 18:02 | |
| *** karthikp_ has quit IRC | 18:03 | |
| *** jwcroppe has quit IRC | 18:04 | |
| *** david-lyle has quit IRC | 18:04 | |
| *** bharath has quit IRC | 18:05 | |
| *** ducttape_ has joined #openstack-cinder | 18:05 | |
| *** ducttape_ has quit IRC | 18:05 | |
| *** david-lyle has joined #openstack-cinder | 18:06 | |
| xinli | @smcginnis: could you please help to review this patch? It's OVO one: https://review.openstack.org/#/c/338542/ | 18:06 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor sqlalchemy service methods https://review.openstack.org/286598 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Add workers table https://review.openstack.org/303018 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster job distribution https://review.openstack.org/327687 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: {WIP] Add worker's DB operations https://review.openstack.org/303019 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve cinder-manage arg parsing https://review.openstack.org/335029 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update Versioned Objects with Cluster object https://review.openstack.org/327686 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Add cleanable base object and cleanup request VO https://review.openstack.org/303020 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Refactor create, save, and destroy OVO methods https://review.openstack.org/315541 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Make c-vol use workers table for cleanup https://review.openstack.org/303021 | 18:08 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Suport A/A in attach/detach operations https://review.openstack.org/318573 | 18:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster table and related methods https://review.openstack.org/318572 | 18:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Modify API to include cluster related operations https://review.openstack.org/327689 | 18:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update manage with cluster related commands https://review.openstack.org/327688 | 18:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix CinderPersistentObject.refresh https://review.openstack.org/335138 | 18:09 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent doc generation failure on OVO decorators https://review.openstack.org/335139 | 18:09 |
| *** daneyon_ has quit IRC | 18:09 | |
| *** sheel has joined #openstack-cinder | 18:12 | |
| *** savihou has joined #openstack-cinder | 18:17 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Add workers table https://review.openstack.org/303018 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster job distribution https://review.openstack.org/327687 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: {WIP] Add worker's DB operations https://review.openstack.org/303019 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve cinder-manage arg parsing https://review.openstack.org/335029 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update Versioned Objects with Cluster object https://review.openstack.org/327686 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Add cleanable base object and cleanup request VO https://review.openstack.org/303020 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Make c-vol use workers table for cleanup https://review.openstack.org/303021 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: [WIP] Suport A/A in attach/detach operations https://review.openstack.org/318573 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add cluster table and related methods https://review.openstack.org/318572 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Modify API to include cluster related operations https://review.openstack.org/327689 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Update manage with cluster related commands https://review.openstack.org/327688 | 18:18 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Prevent doc generation failure on OVO decorators https://review.openstack.org/335139 | 18:18 |
| *** crose has joined #openstack-cinder | 18:19 | |
| *** rdxnoam_ has quit IRC | 18:20 | |
| *** rdx_matt has joined #openstack-cinder | 18:23 | |
| *** rdx_matt has quit IRC | 18:24 | |
| *** eharney has quit IRC | 18:29 | |
| hemna | review spam! | 18:31 |
| *** shyama has quit IRC | 18:35 | |
| *** rcernin has joined #openstack-cinder | 18:39 | |
| *** abhi has joined #openstack-cinder | 18:39 | |
| *** crose has quit IRC | 18:39 | |
| *** ChubYann has joined #openstack-cinder | 18:41 | |
| *** karthikp_ has joined #openstack-cinder | 18:41 | |
| openstackgerrit | Parashuram Hallur proposed openstack/cinder: Implementation for CoprHD Cinder Drivers https://review.openstack.org/317643 | 18:43 |
| mtanino | amazing! | 18:51 |
| *** julim has quit IRC | 18:51 | |
| xinli | eharney: could you please help to review the patch: https://review.openstack.org/338542 | 18:55 |
| *** bjornar_ has joined #openstack-cinder | 18:57 | |
| *** ociuhandu has quit IRC | 18:58 | |
| *** karthikp_ has quit IRC | 18:59 | |
| *** claudiub has joined #openstack-cinder | 19:00 | |
| *** f10d4 has quit IRC | 19:00 | |
| *** f10d4 has joined #openstack-cinder | 19:00 | |
| claudiub | hemna: hello. lpetrut has answered your comments on the fibre channel connector. is it an acceptable answer for you? https://review.openstack.org/#/c/323780/ | 19:01 |
| *** EinstCra_ has quit IRC | 19:01 | |
| *** EinstCrazy has joined #openstack-cinder | 19:02 | |
| hemna | I'll check | 19:04 |
| *** lpetrut has quit IRC | 19:04 | |
| *** rdx_matt has joined #openstack-cinder | 19:04 | |
| *** jwcroppe has joined #openstack-cinder | 19:05 | |
| *** jwcroppe has quit IRC | 19:05 | |
| *** jwcroppe has joined #openstack-cinder | 19:05 | |
| *** rdx_matt has quit IRC | 19:10 | |
| *** cknight has joined #openstack-cinder | 19:10 | |
| *** rdx_matt has joined #openstack-cinder | 19:10 | |
| *** rdx_matt has quit IRC | 19:11 | |
| *** seldenr has quit IRC | 19:12 | |
| *** laughterwym has joined #openstack-cinder | 19:12 | |
| *** salv-orlando has quit IRC | 19:12 | |
| Swanson | Anyone want to take a look at https://review.openstack.org/#/c/329531/ | 19:12 |
| *** karthikp_ has joined #openstack-cinder | 19:13 | |
| Swanson | Don't be afraid! 1500 of those lines are just unit tests! | 19:13 |
| *** laughterwym has quit IRC | 19:17 | |
| *** dramakri has quit IRC | 19:18 | |
| hemna | -2, too many lines! | 19:21 |
| hemna | :P | 19:21 |
| Swanson | I knew Gorka's method was better! | 19:22 |
| hemna | :) | 19:22 |
| hemna | 1500 patches, each 1 line. | 19:22 |
| *** cknight has quit IRC | 19:23 | |
| *** ametts has quit IRC | 19:23 | |
| Swanson | That 1500th patch would have quite the dependency list. | 19:23 |
| hemna | rebase! | 19:23 |
| *** cknight has joined #openstack-cinder | 19:24 | |
| *** cdelatte has joined #openstack-cinder | 19:25 | |
| *** cknight has quit IRC | 19:27 | |
| *** karthikp_ has quit IRC | 19:29 | |
| xinli | xyang1: could you please help to review this OVO patch? | 19:33 |
| hemna | patrickeast, ping | 19:34 |
| *** karthikp_ has joined #openstack-cinder | 19:35 | |
| *** laughterwym has joined #openstack-cinder | 19:35 | |
| hemna | patrickeast, looks like pure CI is puking in 8seconds. DNS failure ? | 19:35 |
| hemna | fyi | 19:35 |
| patrickeast | hemna: recently? | 19:35 |
| patrickeast | my apt-cacher server went down last night for a bit | 19:36 |
| patrickeast | so it was failing a bunch of jobs | 19:36 |
| patrickeast | nagios claims things are OK now, i'll double check | 19:36 |
| *** satya__ has quit IRC | 19:38 | |
| xinli | xyang1:https://review.openstack.org/#/c/338542/ | 19:38 |
| *** laughterwym has quit IRC | 19:39 | |
| *** Apoorva has quit IRC | 19:43 | |
| *** ociuhandu has joined #openstack-cinder | 19:44 | |
| openstackgerrit | Merged openstack/cinder: EMC ScaleIO should return ceiling of volume size https://review.openstack.org/335887 | 19:47 |
| *** Apoorva has joined #openstack-cinder | 19:48 | |
| *** lpetrut has joined #openstack-cinder | 19:49 | |
| *** satya__ has joined #openstack-cinder | 19:51 | |
| xinli | patrickeast: if you have time, please help to review the patch: https://review.openstack.org/#/c/338542/ | 19:52 |
| *** gouthamr has quit IRC | 19:52 | |
| *** jwcroppe has quit IRC | 20:03 | |
| *** karthikp_ has quit IRC | 20:08 | |
| hemna | patrickeast, yah I noticed it on a few patches I've pushed up today | 20:10 |
| hemna | patrickeast, https://review.openstack.org/#/c/339116/ | 20:11 |
| patrickeast | hemna: ah yea, 9:37 AM was about 20 min before i got into the office and un-borked the vm | 20:11 |
| hemna | ok coolio | 20:12 |
| patrickeast | hemna: after ~10 or so things are green again | 20:12 |
| *** lpetrut has quit IRC | 20:12 | |
| hemna | https://review.openstack.org/#/c/339147/ | 20:13 |
| hemna | ok it looks happy on that one | 20:13 |
| Swanson | gate-tempest-dsvm-postgress-full seems to be choking for everything today. | 20:16 |
| hemna | yah It keeps puking on my patch for no reason | 20:16 |
| hemna | ssh timeouts, etc. | 20:16 |
| Swanson | I just asked over in infra if it is a known thing. | 20:17 |
| hemna | kewl | 20:17 |
| Swanson | Seems to be broken on just about every patch. | 20:18 |
| Swanson | "un-borked the vm" is a euphemism for something. Not sure what.... | 20:19 |
| *** mdenny has quit IRC | 20:28 | |
| *** liverpooler has quit IRC | 20:32 | |
| *** Lee1092 has quit IRC | 20:33 | |
| *** ducttape_ has joined #openstack-cinder | 20:41 | |
| *** salv-orlando has joined #openstack-cinder | 20:44 | |
| *** hemanthm is now known as hemanthm|afk | 20:45 | |
| *** sheel has quit IRC | 20:45 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Add trace facility https://review.openstack.org/339231 | 20:46 |
| *** salv-orlando has quit IRC | 20:46 | |
| *** salv-orlando has joined #openstack-cinder | 20:46 | |
| *** mdenny has joined #openstack-cinder | 20:47 | |
| *** karthikp_ has joined #openstack-cinder | 20:50 | |
| *** jungleboyj has quit IRC | 20:50 | |
| *** rdx_matt has joined #openstack-cinder | 20:51 | |
| *** karthikp_ has quit IRC | 21:14 | |
| *** mtanino has quit IRC | 21:14 | |
| *** rdx_matt has quit IRC | 21:14 | |
| *** rdx_matt has joined #openstack-cinder | 21:15 | |
| *** rcernin has quit IRC | 21:17 | |
| *** rdx_matt_ has joined #openstack-cinder | 21:17 | |
| *** karthikp_ has joined #openstack-cinder | 21:18 | |
| *** rdx_matt_ has quit IRC | 21:21 | |
| *** rdx_matt has quit IRC | 21:21 | |
| *** david-lyle has quit IRC | 21:21 | |
| *** haplo37_ has quit IRC | 21:22 | |
| *** david-lyle has joined #openstack-cinder | 21:23 | |
| *** lpetrut has joined #openstack-cinder | 21:29 | |
| *** diablo_rojo has quit IRC | 21:32 | |
| *** jungleboyj has joined #openstack-cinder | 21:33 | |
| *** ozialien10 has joined #openstack-cinder | 21:35 | |
| *** Julien-zte has joined #openstack-cinder | 21:37 | |
| *** satya__ has quit IRC | 21:37 | |
| *** Julien-zte has quit IRC | 21:37 | |
| *** Julien-zte has joined #openstack-cinder | 21:38 | |
| *** diogogmt has quit IRC | 21:40 | |
| *** diogogmt has joined #openstack-cinder | 21:47 | |
| openstackgerrit | Shlomi Avihou proposed openstack/cinder: Add backend driver for Zadara Storage VPSA https://review.openstack.org/263026 | 21:48 |
| *** IlyaG has joined #openstack-cinder | 21:48 | |
| *** aleksey3 has joined #openstack-cinder | 21:49 | |
| *** aleksey3 is now known as alekseyruban | 21:49 | |
| *** adrianofr has quit IRC | 21:51 | |
| *** rlrossit has quit IRC | 21:53 | |
| *** xyang1 has quit IRC | 21:54 | |
| openstackgerrit | Merged openstack/os-brick: Fix multipath iSCSI encrypted volume attach failure https://review.openstack.org/338017 | 21:54 |
| *** dustins has quit IRC | 21:57 | |
| *** IlyaG has quit IRC | 21:57 | |
| *** david-lyle has quit IRC | 22:07 | |
| *** xinli has quit IRC | 22:07 | |
| *** akshai_ has joined #openstack-cinder | 22:11 | |
| *** cbader has quit IRC | 22:12 | |
| Swanson | https://bugs.launchpad.net/bugs/1600005 | 22:12 |
| openstack | Launchpad bug 1600005 in OpenStack Compute (nova) "gate-tempest-dsvm-postgres-full ssh timeout failures spiking since 7/4" [High,Confirmed] | 22:12 |
| *** alyson_ has quit IRC | 22:13 | |
| *** akshai has quit IRC | 22:15 | |
| *** ducttape_ has quit IRC | 22:17 | |
| *** mdenny has quit IRC | 22:22 | |
| *** ozialien10 has quit IRC | 22:22 | |
| *** jungleboyj has quit IRC | 22:25 | |
| *** lpetrut has quit IRC | 22:27 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 22:28 |
| *** lpetrut has joined #openstack-cinder | 22:28 | |
| *** sticker has joined #openstack-cinder | 22:28 | |
| *** salv-orlando has quit IRC | 22:30 | |
| hemna | unit tests....are like.....drinking poison. | 22:32 |
| patrickeast | haha | 22:33 |
| patrickeast | they shouldn't be so bad | 22:34 |
| Swanson | unit tests are like creating poison and then drinking it. | 22:34 |
| *** salv-orlando has joined #openstack-cinder | 22:34 | |
| hemna | trying to force feed myself the 3par iscsi unit tests changes required to do the ovo patch | 22:35 |
| hemna | so much pain | 22:35 |
| patrickeast | oh yea | 22:35 |
| hemna | they should write themselves. | 22:35 |
| patrickeast | that stuff sucks, i looked at doing the same thing in mine... i'll stick with dictionary objects | 22:36 |
| hemna | the changes are pretty trivial | 22:36 |
| patrickeast | took the "new tests use ovo, old tests... don't look too closely at them" | 22:36 |
| hemna | it's the EFFING unit tests.... | 22:36 |
| hemna | DIE DIE DIE! | 22:36 |
| *** lpetrut has quit IRC | 22:38 | |
| *** EinstCrazy has quit IRC | 22:38 | |
| *** salv-orlando has quit IRC | 22:39 | |
| *** catintheroof has quit IRC | 22:40 | |
| *** savihou has quit IRC | 22:42 | |
| *** abhi has quit IRC | 22:43 | |
| *** julim has joined #openstack-cinder | 22:44 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: WIP 3PAR use OSLO versioned Object for volume https://review.openstack.org/339147 | 22:44 |
| hemna | phew | 22:44 |
| hemna | iSCSI unit tests working... | 22:44 |
| hemna | now only 6*10^50 for FC and I'm done! | 22:45 |
| patrickeast | could always just delete the unit tests that dont pass :D | 22:46 |
| hemna | ooh | 22:46 |
| hemna | sssh | 22:46 |
| Swanson | While doing that you could also run your CI against LVM. Tests should never be a problem again. | 22:47 |
| patrickeast | its foolproof | 22:47 |
| patrickeast | just add a patch on top of every commit the ci runs to add one of the name mappings for the driver | 22:47 |
| Swanson | Be a bloodbath the day duncan get's ptl but until then... | 22:47 |
| patrickeast | so cinder.conf says its your driver... but its really lvm | 22:48 |
| hemna | :P | 22:48 |
| hemna | https://bugs.launchpad.net/oslo.privsep/+bug/1593791 | 22:49 |
| openstack | Launchpad bug 1593791 in oslo.privsep "Need to suppress expected command failures" [Undecided,New] | 22:49 |
| hemna | that needs to get fixied | 22:49 |
| hemna | n-cpu logs are filled with failures | 22:49 |
| hemna | due to not having FC HBA's | 22:49 |
| hemna | annoying | 22:50 |
| *** karthikp_ has quit IRC | 22:50 | |
| patrickeast | ah yea, both n-cpu and c-vol logs are full of privsep command output warnings | 22:56 |
| hemna | yup | 22:56 |
| patrickeast | i poked around the privsep code a while ago, it was unclear where that was coming from | 22:56 |
| hemna | https://github.com/openstack/oslo.privsep/blob/master/oslo_privsep/daemon.py#L284 | 22:57 |
| patrickeast | might take another look | 22:57 |
| hemna | right there | 22:57 |
| *** karthikp_ has joined #openstack-cinder | 22:57 | |
| hemna | afaik | 22:57 |
| hemna | stderr=_fd_logger() | 22:57 |
| hemna | that poops out all stderr all the time. | 22:57 |
| hemna | stdout and stderr should go to the caller | 22:57 |
| patrickeast | right but i thought it was configurable | 22:57 |
| hemna | and left up to the caller to vomit or not | 22:57 |
| hemna | https://github.com/openstack/oslo.privsep/blob/master/oslo_privsep/daemon.py#L193 | 22:58 |
| hemna | poops every time. | 22:58 |
| patrickeast | oh i see | 22:58 |
| patrickeast | the method allows it to be configurable on level | 22:58 |
| patrickeast | but nothing ever changes it | 22:58 |
| *** karthikp_ has quit IRC | 23:00 | |
| *** salv-orlando has joined #openstack-cinder | 23:01 | |
| hemna | man it's just baked into lots of stuffs | 23:01 |
| patrickeast | i'm kinda surprised the nova folks aren't all over this... ive seen several times in reviews where logging warnings that weren't really warnings was like a blocking issue | 23:03 |
| hemna | yah | 23:03 |
| hemna | it's just puking everywhere :( | 23:03 |
| *** savihou has joined #openstack-cinder | 23:03 | |
| patrickeast | oh speaking of nova and brick | 23:03 |
| hemna | stderr should be captured and returned | 23:03 |
| patrickeast | https://review.openstack.org/#/c/336742/ | 23:03 |
| hemna | oh | 23:04 |
| patrickeast | yea it should behave like a subprocess.check_output kinda thing | 23:04 |
| patrickeast | hemna: i ran into that on my ci, can't run multi-node jobs on os-brick right now :( | 23:04 |
| hemna | ewps | 23:04 |
| patrickeast | well, on latest os-brick | 23:04 |
| patrickeast | pypi os-brick works fine | 23:05 |
| *** garthb has joined #openstack-cinder | 23:05 | |
| hemna | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L151 | 23:07 |
| hemna | I thought about removing that line | 23:07 |
| hemna | in my trace patch | 23:07 |
| hemna | and just decorate the module get_connector_properties() | 23:08 |
| hemna | with the trace | 23:08 |
| patrickeast | ah yea, probably won't need it | 23:08 |
| hemna | it'll poop out the entire connector | 23:08 |
| patrickeast | seems better to me | 23:08 |
| hemna | instead of each individual connector's entry | 23:08 |
| hemna | less noise and the same value basically | 23:08 |
| hemna | https://review.openstack.org/#/c/339231/ | 23:08 |
| hemna | in that patch | 23:09 |
| *** salv-orlando has quit IRC | 23:09 | |
| hemna | heh | 23:09 |
| hemna | first thing in the n-cpu logs is os_brick factory calls | 23:10 |
| hemna | http://openstack-logs.purestorage.com/31/339231/1/check/PureFCDriver-tempest-dsvm-volume-multipath-brick/6033051/logs/screen-n-cpu.txt.gz | 23:10 |
| patrickeast | haha | 23:11 |
| patrickeast | that parts not too bad | 23:11 |
| hemna | not sure why those happen | 23:11 |
| patrickeast | its this view that bugs me: http://openstack-logs.purestorage.com/31/339231/1/check/PureFCDriver-tempest-dsvm-volume-multipath-brick/6033051/logs/screen-n-cpu.txt.gz?level=TRACE | 23:11 |
| hemna | heh yah | 23:12 |
| hemna | privsep | 23:12 |
| *** liuke_ has joined #openstack-cinder | 23:12 | |
| hemna | why are they warnings | 23:12 |
| hemna | I think passing the stderr shit back to the caller would help clean some of that up | 23:13 |
| patrickeast | definitely | 23:13 |
| hemna | but all the "privsep daemon starting" nonsense shouldn't be a warning | 23:13 |
| patrickeast | haha yea | 23:13 |
| hemna | http://openstack-logs.purestorage.com/31/339231/1/check/PureFCDriver-tempest-dsvm-volume-multipath-brick/6033051/logs/screen-n-cpu.txt.gz?level=TRACE#_2016-07-07_22_21_44_614 | 23:14 |
| hemna | trying to make sense of the reply protocol | 23:15 |
| hemna | 4, (<lines of output for stdout ??>) | 23:15 |
| hemna | wtf is 4 ? | 23:15 |
| hemna | maybe 4 = reply | 23:15 |
| hemna | and 3 = request | 23:15 |
| *** savihou has quit IRC | 23:15 | |
| patrickeast | mm yea probably message type or something | 23:16 |
| hemna | ah yep | 23:16 |
| hemna | 3 = CALL | 23:16 |
| hemna | 4 = RET | 23:16 |
| hemna | 5 = ERR | 23:16 |
| hemna | https://github.com/openstack/oslo.privsep/blob/master/oslo_privsep/daemon.py#L91 | 23:16 |
| *** liuke has quit IRC | 23:16 | |
| *** bhagyashri has quit IRC | 23:16 | |
| *** bhagyashri has joined #openstack-cinder | 23:16 | |
| *** liuke_ is now known as liuke | 23:16 | |
| hemna | ah yah, so when the cmdn returns non 0 result code | 23:18 |
| hemna | the response is 5 | 23:19 |
| hemna | and the body | 23:19 |
| hemna | and on systems that don't have an HBA, that will happen 100% of volume attach calls | 23:19 |
| hemna | :( | 23:19 |
| patrickeast | hah, perfect! | 23:19 |
| patrickeast | clearly the fix is to only deploy openstack with FC | 23:19 |
| hemna | heh yah | 23:20 |
| hemna | so _fd_logger( defaults to logging.WARN | 23:20 |
| hemna | smh | 23:20 |
| hemna | that shit should be debug | 23:20 |
| patrickeast | well, thats an easy patch :D | 23:21 |
| hemna | yah | 23:21 |
| hemna | but we'll just get logs of debug vomits instead | 23:21 |
| patrickeast | thats ok though, debug logs should show the command we run | 23:21 |
| patrickeast | thats what used to happen with privsep | 23:21 |
| hemna | ok I'll push that up | 23:22 |
| hemna | it's a small step | 23:22 |
| hemna | I still think the caller should decide if it needs to poop out stderr | 23:23 |
| hemna | or I should be able to suppress logging output on expected command failures | 23:23 |
| patrickeast | yea but that looks like a much larger change :( | 23:24 |
| patrickeast | well maybe | 23:24 |
| *** claudiub has quit IRC | 23:25 | |
| hemna | prior to privsep we didn't log command execution | 23:25 |
| hemna | https://github.com/openstack/os-brick/blob/stable/liberty/os_brick/initiator/linuxfc.py#L44 | 23:25 |
| hemna | we just captured it | 23:26 |
| *** Suyash has quit IRC | 23:27 | |
| patrickeast | i thought our exec still logged the command being run though and its output | 23:27 |
| *** julim has quit IRC | 23:27 | |
| hemna | the default executor was simply putils | 23:28 |
| hemna | which I didn't think logged by default ? | 23:28 |
| hemna | maybe I'm wrong | 23:29 |
| patrickeast | not sure | 23:29 |
| patrickeast | im trying to find an old log somewhere to see | 23:29 |
| patrickeast | i do remember seeing commands we run, but maybe those were once we explicitly logged | 23:30 |
| *** Julien-zte has quit IRC | 23:30 | |
| patrickeast | s/once/ones/ | 23:30 |
| hemna | hrmm | 23:30 |
| hemna | I don't remember now | 23:30 |
| hemna | ah yah ok | 23:31 |
| hemna | https://github.com/openstack/oslo.concurrency/blob/master/oslo_concurrency/processutils.py#L290 | 23:31 |
| hemna | I guess the default os debug | 23:32 |
| hemna | and it shits everything out | 23:32 |
| patrickeast | http://52.8.17.232//26/302926/1/check/PureISCSIDriver-tempest-dsvm-volume-multipath/3bb857f/logs/screen-c-vol.txt.gz? | 23:32 |
| patrickeast | yep | 23:32 |
| hemna | yah | 23:32 |
| patrickeast | err http://52.8.17.232//26/302926/1/check/PureISCSIDriver-tempest-dsvm-volume-multipath/3bb857f/logs/screen-c-vol.txt.gz?#_2016-04-07_17_20_56_838 | 23:32 |
| patrickeast | still has the fc checking | 23:32 |
| *** fmccrthy has quit IRC | 23:32 | |
| patrickeast | but less in the way | 23:32 |
| *** julim has joined #openstack-cinder | 23:32 | |
| hemna | but | 23:33 |
| hemna | it doesn't vomit stderr | 23:33 |
| hemna | http://52.8.17.232//26/302926/1/check/PureISCSIDriver-tempest-dsvm-volume-multipath/3bb857f/logs/screen-c-vol.txt.gz#_2016-04-07_17_21_46_394 | 23:33 |
| patrickeast | ahh yea | 23:33 |
| *** mattymo has quit IRC | 23:34 | |
| *** fmccrthy has joined #openstack-cinder | 23:34 | |
| *** mattymo has joined #openstack-cinder | 23:34 | |
| *** salv-orlando has joined #openstack-cinder | 23:35 | |
| hemna | https://review.openstack.org/339275 | 23:39 |
| hemna | fwiw | 23:39 |
| hemna | FTS o'clock! | 23:41 |
| *** xyang has joined #openstack-cinder | 23:42 | |
| *** claudiub has joined #openstack-cinder | 23:44 | |
| *** ducttape_ has joined #openstack-cinder | 23:46 | |
| *** diogogmt has quit IRC | 23:46 | |
| *** xyang has quit IRC | 23:47 | |
| *** salv-orlando has quit IRC | 23:47 | |
| *** mriedem is now known as mriedem_afk | 23:47 | |
| *** tpsilva has quit IRC | 23:48 | |
| *** itzdilip has quit IRC | 23:49 | |
| *** diogogmt has joined #openstack-cinder | 23:54 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!