| *** salv-orlando has joined #openstack-cinder | 00:00 | |
| *** salv-orlando has quit IRC | 00:05 | |
| *** mtanino_ has quit IRC | 00:13 | |
| *** mtanino_ has joined #openstack-cinder | 00:15 | |
| *** mtanino_ has quit IRC | 00:20 | |
| *** r-daneel has quit IRC | 00:26 | |
| *** zhaochao has quit IRC | 00:28 | |
| *** wanghao has joined #openstack-cinder | 00:31 | |
| *** Julien-zte has joined #openstack-cinder | 00:44 | |
| *** Shunli has joined #openstack-cinder | 00:48 | |
| *** abhishekk has quit IRC | 00:52 | |
| *** abhishekk has joined #openstack-cinder | 00:52 | |
| *** markstur has quit IRC | 00:55 | |
| *** salv-orlando has joined #openstack-cinder | 01:01 | |
| *** salv-orlando has quit IRC | 01:06 | |
| *** stakeda has joined #openstack-cinder | 01:13 | |
| *** zhengyin has joined #openstack-cinder | 01:15 | |
| *** markstur has joined #openstack-cinder | 01:17 | |
| *** markstur has quit IRC | 01:21 | |
| *** mriedem has quit IRC | 01:22 | |
| *** crushil has joined #openstack-cinder | 01:31 | |
| *** dhinesh has quit IRC | 01:32 | |
| *** zhaochao has joined #openstack-cinder | 01:33 | |
| *** winston-d_ has joined #openstack-cinder | 01:35 | |
| *** Julien-zte has quit IRC | 01:35 | |
| *** Apoorva has joined #openstack-cinder | 01:36 | |
| *** Apoorva_ has quit IRC | 01:40 | |
| *** Apoorva has quit IRC | 01:40 | |
| *** diablo_rojo has quit IRC | 01:46 | |
| *** zhurong has joined #openstack-cinder | 01:48 | |
| *** jamielennox is now known as jamielennox|away | 02:04 | |
| *** bkopilov has quit IRC | 02:12 | |
| *** jamielennox|away is now known as jamielennox | 02:15 | |
| *** diablo_rojo has joined #openstack-cinder | 02:19 | |
| *** hongbin has quit IRC | 02:36 | |
| *** Julien-zte has joined #openstack-cinder | 02:41 | |
| *** crushil_ has joined #openstack-cinder | 02:47 | |
| *** liuqing has quit IRC | 02:47 | |
| *** liuqing has joined #openstack-cinder | 02:47 | |
| *** edmondsw has joined #openstack-cinder | 02:55 | |
| *** Julien-zte has quit IRC | 02:57 | |
| wxy | ping winston-d : Hi about this patch: https://review.openstack.org/#/c/437677/ Would you mind me to update it? Our production has the same problem as well. | 02:58 |
|---|---|---|
| *** edmondsw has quit IRC | 02:59 | |
| *** crushil has quit IRC | 03:05 | |
| *** crushil has joined #openstack-cinder | 03:06 | |
| *** Julien-zte has joined #openstack-cinder | 03:09 | |
| *** Shashi has joined #openstack-cinder | 03:12 | |
| *** dikonoor has joined #openstack-cinder | 03:17 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Support metadata for backup resource-1 https://review.openstack.org/471541 | 03:25 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: [WIP]Support metadata for backup resource-2 https://review.openstack.org/472030 | 03:25 |
| *** lhx__ has joined #openstack-cinder | 03:27 | |
| *** bkopilov has joined #openstack-cinder | 03:32 | |
| *** links has joined #openstack-cinder | 03:34 | |
| *** zsli_ has joined #openstack-cinder | 03:36 | |
| *** Shunli has quit IRC | 03:39 | |
| *** gkadam has joined #openstack-cinder | 03:42 | |
| *** zhurong has quit IRC | 03:45 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Fix 'OS_AUTH_TYPE' doesn't work https://review.openstack.org/472034 | 03:48 |
| *** dikonoor has quit IRC | 03:48 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Fix 'OS_AUTH_TYPE' doesn't work https://review.openstack.org/472034 | 03:49 |
| *** crushil_ has quit IRC | 03:54 | |
| *** crushil_ has joined #openstack-cinder | 03:54 | |
| *** psachin has joined #openstack-cinder | 04:02 | |
| *** itlinux has joined #openstack-cinder | 04:03 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Add Metadata format check https://review.openstack.org/472055 | 04:04 |
| *** stakeda has quit IRC | 04:11 | |
| *** markstur has joined #openstack-cinder | 04:12 | |
| *** stakeda has joined #openstack-cinder | 04:14 | |
| *** dave-mccowan has quit IRC | 04:26 | |
| *** Julien-zte has quit IRC | 04:27 | |
| *** Julien-zte has joined #openstack-cinder | 04:28 | |
| *** pcaruana has joined #openstack-cinder | 04:31 | |
| *** aselius has quit IRC | 04:32 | |
| *** zhurong has joined #openstack-cinder | 04:33 | |
| *** zhengyin has quit IRC | 04:34 | |
| *** zhengyin has joined #openstack-cinder | 04:36 | |
| *** salv-orlando has joined #openstack-cinder | 04:39 | |
| *** crushil_ has quit IRC | 04:41 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 04:43 |
| *** adisky__ has joined #openstack-cinder | 04:55 | |
| *** lhx__ has quit IRC | 04:57 | |
| *** pcaruana has quit IRC | 04:57 | |
| *** lhx__ has joined #openstack-cinder | 04:57 | |
| *** salv-orlando has quit IRC | 05:05 | |
| *** salv-orlando has joined #openstack-cinder | 05:06 | |
| *** lkuchlan has joined #openstack-cinder | 05:10 | |
| *** jamielennox is now known as jamielennox|away | 05:12 | |
| *** Poornima has joined #openstack-cinder | 05:13 | |
| *** ska has quit IRC | 05:15 | |
| *** jamielennox|away is now known as jamielennox | 05:19 | |
| *** itlinux has quit IRC | 05:21 | |
| *** markstur has quit IRC | 05:24 | |
| *** ska has joined #openstack-cinder | 05:29 | |
| *** diablo_rojo has quit IRC | 05:34 | |
| *** zhengyin has quit IRC | 05:34 | |
| *** zhengyin has joined #openstack-cinder | 05:35 | |
| *** kwathore has joined #openstack-cinder | 05:35 | |
| openstackgerrit | Peter Wang proposed openstack/cinder master: Add VNX replication group support https://review.openstack.org/459205 | 05:54 |
| *** jamielennox is now known as jamielennox|away | 05:56 | |
| *** dikonoor has joined #openstack-cinder | 05:58 | |
| *** jamielennox|away is now known as jamielennox | 06:03 | |
| *** ChubYann has quit IRC | 06:05 | |
| *** rcernin has joined #openstack-cinder | 06:15 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Clarify some details related blockbox deployment https://review.openstack.org/472101 | 06:20 |
| *** Julien-zte has quit IRC | 06:26 | |
| openstackgerrit | Béla Vancsics proposed openstack/cinder master: Use more specific asserts in tests https://review.openstack.org/433671 | 06:27 |
| *** zhurong has quit IRC | 06:32 | |
| *** lkuchlan has quit IRC | 06:32 | |
| *** Poornima_K has joined #openstack-cinder | 06:35 | |
| *** Poornima has quit IRC | 06:36 | |
| *** Julien-zte has joined #openstack-cinder | 06:36 | |
| *** Sara__ has joined #openstack-cinder | 06:42 | |
| Sara__ | Hello everyone | 06:46 |
| Sara__ | I have faced an issue with the setup where one node is configured as cinder | 06:48 |
| Sara__ | After upgrading the hardware (which contains our setup) with its latest version, cinder node is not operating properly | 06:52 |
| Sara__ | i.e., not able to delete and create volumes | 06:53 |
| tommylikehu | Sara__: can you paste you error detail with a paste link here? | 06:54 |
| *** pcaruana has joined #openstack-cinder | 06:57 | |
| Sara__ | tommylikehu_: Actually I didn't get any errors | 07:01 |
| Sara__ | commands got stuck when i try to create and delete(existing) volumes | 07:02 |
| Sara__ | in logs i could see the errors: http://paste.openstack.org/show/611805/ | 07:03 |
| Sara__ | but i dont think these are related to this issue | 07:03 |
| *** Poornima_K has quit IRC | 07:04 | |
| Sara__ | one more thing, cinder starts working after rebooting the cinder node | 07:05 |
| *** lkuchlan has joined #openstack-cinder | 07:05 | |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Support metadata for backup resource-1 https://review.openstack.org/471541 | 07:08 |
| *** tesseract has joined #openstack-cinder | 07:12 | |
| Sara__ | tommylikehu__: Can u have any idea regarding this issue? | 07:12 |
| *** e0ne has joined #openstack-cinder | 07:29 | |
| *** gkadam is now known as gkadam-afk | 07:30 | |
| *** mlakat has joined #openstack-cinder | 07:32 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 07:33 |
| *** Julien-zte has quit IRC | 07:34 | |
| openstackgerrit | zhuangzejian proposed openstack/cinder master: Modify a little desc to stisfy the code https://review.openstack.org/472118 | 07:34 |
| tommylikehu | Sara__: something wrong with your network? this is the only thing I can tell from your link | 07:37 |
| *** gcb has joined #openstack-cinder | 07:39 | |
| *** Poornima has joined #openstack-cinder | 07:40 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 07:42 |
| *** nicolasbock has joined #openstack-cinder | 07:45 | |
| *** zhurong has joined #openstack-cinder | 07:46 | |
| *** bkopilov has quit IRC | 07:47 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 07:52 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Support metadata for backup resource-1 https://review.openstack.org/471541 | 07:57 |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-cinder | 08:01 | |
| *** e0ne has quit IRC | 08:04 | |
| *** zhurong has quit IRC | 08:13 | |
| *** salv-orlando has quit IRC | 08:13 | |
| *** salv-orlando has joined #openstack-cinder | 08:14 | |
| *** zhurong has joined #openstack-cinder | 08:15 | |
| *** gkadam-afk is now known as gkadam | 08:16 | |
| *** salv-orlando has quit IRC | 08:17 | |
| *** bjornar has joined #openstack-cinder | 08:20 | |
| *** e0ne has joined #openstack-cinder | 08:25 | |
| *** links has quit IRC | 08:25 | |
| *** Julien-zte has joined #openstack-cinder | 08:28 | |
| *** Julien-zte has quit IRC | 08:30 | |
| *** Julien-zte has joined #openstack-cinder | 08:31 | |
| *** isaacb has joined #openstack-cinder | 08:34 | |
| *** links has joined #openstack-cinder | 08:38 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 08:40 |
| *** salv-orlando has joined #openstack-cinder | 08:45 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs master: Add directory for Queens spec https://review.openstack.org/472156 | 08:48 |
| *** odyssey4me_ is now known as odyssey4me | 08:49 | |
| *** Poornima has quit IRC | 08:56 | |
| *** stakeda has quit IRC | 09:02 | |
| *** mvk has joined #openstack-cinder | 09:02 | |
| openstackgerrit | lihaijing proposed openstack/cinder master: Api-ref: add documentation for 'os-volume_upload_image' action https://review.openstack.org/472060 | 09:04 |
| *** Poornima has joined #openstack-cinder | 09:04 | |
| *** zhengyin has quit IRC | 09:21 | |
| *** zhengyin has joined #openstack-cinder | 09:21 | |
| *** dalgaaf has quit IRC | 09:31 | |
| *** zsli_ has quit IRC | 09:32 | |
| *** zhengyin has quit IRC | 09:33 | |
| *** crushil has quit IRC | 09:33 | |
| *** zhengyin has joined #openstack-cinder | 09:33 | |
| *** crushil has joined #openstack-cinder | 09:33 | |
| *** dalgaaf has joined #openstack-cinder | 09:34 | |
| openstackgerrit | Gaurang Tapase proposed openstack/cinder master: Add CG capability to generic groups in GPFS driver https://review.openstack.org/469825 | 09:39 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: [WIP]Support metadata for backup resource-2 https://review.openstack.org/472030 | 09:45 |
| openstackgerrit | wangxiyuan proposed openstack/cinder master: Support metadata for backup resource-1 https://review.openstack.org/471541 | 09:45 |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs master: Add directory for Queens spec https://review.openstack.org/472156 | 09:45 |
| *** kaisers has quit IRC | 10:00 | |
| *** wanghao has quit IRC | 10:04 | |
| *** sdague has joined #openstack-cinder | 10:11 | |
| *** kaisers has joined #openstack-cinder | 10:16 | |
| *** Sara__ has quit IRC | 10:19 | |
| *** kwathore_ has joined #openstack-cinder | 10:21 | |
| *** kwathore has quit IRC | 10:21 | |
| *** markvoelker has quit IRC | 10:27 | |
| *** markvoelker has joined #openstack-cinder | 10:27 | |
| *** med_ has quit IRC | 10:31 | |
| *** cristicalin has joined #openstack-cinder | 10:31 | |
| *** cristicalin has quit IRC | 10:33 | |
| *** med_ has joined #openstack-cinder | 10:35 | |
| *** med_ is now known as Guest72612 | 10:35 | |
| *** dikonoor has quit IRC | 10:43 | |
| *** salv-orlando has quit IRC | 10:44 | |
| *** salv-orlando has joined #openstack-cinder | 10:44 | |
| *** winston-d_ has quit IRC | 10:55 | |
| *** zhengyin has quit IRC | 11:05 | |
| *** kaisers has quit IRC | 11:06 | |
| openstackgerrit | Merged openstack/cinder-specs master: Add directory for Queens spec https://review.openstack.org/472156 | 11:12 |
| openstackgerrit | Sean McGinnis proposed openstack/os-brick master: Add libssl to bindep https://review.openstack.org/472211 | 11:14 |
| *** Julien-zte has quit IRC | 11:15 | |
| *** mvk has quit IRC | 11:16 | |
| smcginnis | e0ne: If you're around, that one is blocking the os-brick gate. ^^ | 11:17 |
| tommylikehu | hey jgriffith_ , could you take a look at these two patches? https://review.openstack.org/#/c/472109/， https://review.openstack.org/#/c/472117/ thanks in advance :) | 11:22 |
| *** dikonoor has joined #openstack-cinder | 11:23 | |
| *** kaisers has joined #openstack-cinder | 11:23 | |
| *** pgadiya has joined #openstack-cinder | 11:30 | |
| *** pgadiya has quit IRC | 11:30 | |
| *** dikonoor has quit IRC | 11:33 | |
| *** dikonoor has joined #openstack-cinder | 11:36 | |
| *** liuqing has quit IRC | 11:36 | |
| *** liuqing has joined #openstack-cinder | 11:36 | |
| *** edmondsw has joined #openstack-cinder | 11:40 | |
| openstackgerrit | TommyLike proposed openstack/cinder master: Enable backend specific async migration https://review.openstack.org/334923 | 11:41 |
| *** mikal has quit IRC | 11:41 | |
| *** mikal has joined #openstack-cinder | 11:43 | |
| *** dikonoor has quit IRC | 11:45 | |
| *** mvk has joined #openstack-cinder | 11:47 | |
| openstackgerrit | Sean Dague proposed openstack/cinder master: Send global_request_id to nova/glance when calls are made https://review.openstack.org/472224 | 11:49 |
| *** dikonoor has joined #openstack-cinder | 11:49 | |
| *** kwathore_ has quit IRC | 11:51 | |
| *** abishop has joined #openstack-cinder | 11:53 | |
| openstackgerrit | wang yong proposed openstack/cinder master: Inspur Cinder FC/iSCSI driver https://review.openstack.org/439488 | 11:55 |
| openstackgerrit | Michal Dulko proposed openstack/cinder master: Move driver options to "backend" section https://review.openstack.org/453654 | 11:57 |
| openstackgerrit | Michal Dulko proposed openstack/cinder master: Remove usage of CONF from DISCO driver https://review.openstack.org/458167 | 11:57 |
| *** kwathore has joined #openstack-cinder | 11:58 | |
| *** Shashi has quit IRC | 12:05 | |
| *** pkoraca has quit IRC | 12:08 | |
| *** mriedem has joined #openstack-cinder | 12:13 | |
| *** lhx__ has quit IRC | 12:17 | |
| *** jwcroppe has joined #openstack-cinder | 12:26 | |
| *** catintheroof has joined #openstack-cinder | 12:32 | |
| e0ne | smcginns: hi. looking on it | 12:35 |
| *** zhurong has quit IRC | 12:36 | |
| e0ne | smcginnis: I'll +2 on it once CI passed | 12:36 |
| *** liverpooler has joined #openstack-cinder | 12:37 | |
| *** jroll has left #openstack-cinder | 12:47 | |
| *** dustins has joined #openstack-cinder | 12:47 | |
| *** lhx_ has joined #openstack-cinder | 12:49 | |
| *** gkadam has quit IRC | 12:50 | |
| *** crushil has quit IRC | 12:56 | |
| *** mriedem1 has joined #openstack-cinder | 12:59 | |
| *** mriedem has quit IRC | 13:00 | |
| *** mriedem1 is now known as mriedem | 13:02 | |
| *** cristicalin has joined #openstack-cinder | 13:03 | |
| *** dustins_ has joined #openstack-cinder | 13:04 | |
| *** crushil has joined #openstack-cinder | 13:07 | |
| *** jonesn has joined #openstack-cinder | 13:08 | |
| *** dave-mccowan has joined #openstack-cinder | 13:12 | |
| *** crushil has quit IRC | 13:15 | |
| *** links has quit IRC | 13:15 | |
| *** lucasxu has joined #openstack-cinder | 13:16 | |
| *** dustins has quit IRC | 13:17 | |
| *** bjornar has quit IRC | 13:31 | |
| *** lkuchlan has quit IRC | 13:32 | |
| sdague | smcginnis et al - https://review.openstack.org/#/c/472224/ - the global_request_id calling from cinder I think is ready to go | 13:36 |
| sdague | it is hopefully a quick review | 13:36 |
| *** asselin_ has joined #openstack-cinder | 13:38 | |
| *** lucasxu has quit IRC | 13:38 | |
| *** asselin has quit IRC | 13:40 | |
| *** lucasxu has joined #openstack-cinder | 13:40 | |
| *** asselin__ has joined #openstack-cinder | 13:42 | |
| *** asselin_ has quit IRC | 13:45 | |
| *** Jack_Iv has joined #openstack-cinder | 13:46 | |
| Jack_Iv | Hey Folks, I need to use another user as a quota manager, I configured policy.json, but still can not get quotas. What is the problem? cinderclient says "clean_up ShowQuota: Access was denied to this resource.". The same schema works for nova | 13:47 |
| *** Poornima has quit IRC | 13:47 | |
| *** lucasxu has quit IRC | 13:51 | |
| *** chenying has quit IRC | 13:52 | |
| *** chenying has joined #openstack-cinder | 13:53 | |
| *** lucasxu has joined #openstack-cinder | 14:02 | |
| *** chlong has joined #openstack-cinder | 14:03 | |
| *** crushil has joined #openstack-cinder | 14:06 | |
| *** mtanino has joined #openstack-cinder | 14:06 | |
| *** dikonoor has quit IRC | 14:09 | |
| *** cknight has joined #openstack-cinder | 14:13 | |
| *** esker has joined #openstack-cinder | 14:15 | |
| *** eharney has joined #openstack-cinder | 14:16 | |
| *** r-daneel has joined #openstack-cinder | 14:17 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Live Migration, replacing SMI-S with REST https://review.openstack.org/472279 | 14:18 |
| *** felipemonteiro_ has joined #openstack-cinder | 14:18 | |
| *** esker has quit IRC | 14:19 | |
| *** esker has joined #openstack-cinder | 14:20 | |
| *** xyang has joined #openstack-cinder | 14:23 | |
| *** diablo_rojo has joined #openstack-cinder | 14:27 | |
| *** Julien-zte has joined #openstack-cinder | 14:30 | |
| *** felipemonteiro__ has joined #openstack-cinder | 14:30 | |
| *** felipemonteiro_ has quit IRC | 14:34 | |
| danpawlik | hemna: Hi, are you there? | 14:34 |
| openstackgerrit | Michael Dovgal proposed openstack/cinder master: (WIP) Add cinder backup service initialize check https://review.openstack.org/446518 | 14:35 |
| *** bjornar has joined #openstack-cinder | 14:35 | |
| *** salv-orl_ has joined #openstack-cinder | 14:44 | |
| *** aselius has joined #openstack-cinder | 14:46 | |
| *** salv-orlando has quit IRC | 14:47 | |
| e0ne | smcginnis: are you around? | 14:50 |
| *** hongbin has joined #openstack-cinder | 14:50 | |
| isaacb | jungleboyj: GM, can you please go over: https://review.openstack.org/#/c/459649/ | 14:51 |
| *** psachin has quit IRC | 14:57 | |
| *** markstur has joined #openstack-cinder | 14:58 | |
| xyang | @smcginnis: looks like the openssl issue is also in os-brick? http://logs.openstack.org/94/455394/13/check/gate-os-brick-pep8-ubuntu-xenial/9af35e0/console.html | 15:01 |
| *** dustins_ is now known as dustins | 15:02 | |
| e0ne | xyang, smcginnis: here is a patch https://review.openstack.org/#/c/472211/ | 15:03 |
| tommylikehu | hey jgriffith_ about patch here: https://review.openstack.org/#/c/472109/4/ubuntu/Dockerfile, do you mean it's better to keep the ubuntu mirror in this patch but move the pypi mirror into loci repo? | 15:03 |
| xyang | e0ne: thanks! | 15:04 |
| jgriffith_ | tommylikehu Yeah, so the pypi stuff inparticular here: https://github.com/openstack/loci/tree/master/scripts | 15:05 |
| tommylikehu | jgriffith_: Ok, I will update the patch :) | 15:06 |
| jgriffith_ | tommylikehu I'll be curious to see what sam thinks | 15:06 |
| danpawlik | jbernard: Hi. Have you check PS: https://review.openstack.org/#/c/456672/ ? | 15:06 |
| tommylikehu | jgriffith_: when does he usually around? | 15:07 |
| jgriffith_ | tommylikehu always :). But he's in #openstack-loci usually | 15:07 |
| tommylikehu | jgriffith_: thanks ! | 15:08 |
| jgriffith_ | tommylikehu so I'm unclear on the nettools add | 15:08 |
| tommylikehu | jgriffith_: Ok~ | 15:09 |
| jgriffith_ | what's missing in terms of functionality without that? | 15:09 |
| jgriffith_ | asking out of ignorance :) | 15:09 |
| tommylikehu | missing the file /etc/protocol | 15:09 |
| *** mnestratov|2 has joined #openstack-cinder | 15:10 | |
| jgriffith_ | tommylikehu but isn't that *ok* since you're using the hosts or docker's networks? | 15:10 |
| jgriffith_ | tommylikehu so by using the overlay or bridge or whatever you choose, you don't necessarily want the base networking functionality in the container do you? | 15:11 |
| tommylikehu | jgriffith_: yeah. I just wana the service could start up :) | 15:11 |
| jgriffith_ | tommylikehu I'm not objecting to it, I just don't know enough about the problem it solves so I'm asking :) | 15:11 |
| jgriffith_ | tommylikehu well.. that's important :) | 15:12 |
| tommylikehu | jgriffith_: maybe you have easy solutions or you can point it out it's incorrect:) | 15:12 |
| jgriffith_ | tommylikehu so are you saying when you try and run without this the service doesn't start? | 15:13 |
| tommylikehu | jgriffith_: yes :) | 15:13 |
| jgriffith_ | hmm... that's not right | 15:13 |
| tommylikehu | jgriffith_: so I must misunderstand something | 15:14 |
| jgriffith_ | tommylikehu yeah, not sure | 15:14 |
| jgriffith_ | I haven't tried it with ubuntu but it seems to work fine with debian... I can try and change that on my side and test it | 15:15 |
| *** rcernin has quit IRC | 15:16 | |
| jgriffith_ | tommylikehu I'll modify the Makefile to use ubuntu and see if I can kick off the docker-compose should only take a few minutes | 15:17 |
| tommylikehu | jgriffith_: thanks | 15:19 |
| tommylikehu | jgriffith_: one more question related to cinder-volume image with lvm backend | 15:21 |
| jgriffith_ | tommylikehu oh, interesting... is this what you're seeing: cinder-scheduler_1 | _proto_tcp = socket.getprotobyname('tcp') | 15:22 |
| jgriffith_ | cinder-scheduler_1 | socket.error: protocol not found | 15:22 |
| tommylikehu | jgriffith_: based on your consideration, the user who wants to use cinder-lvm with container should make the vgs ready before starting the service? | 15:23 |
| tommylikehu | jgriffith_: that's what I had this morning | 15:23 |
| jgriffith_ | tommylikehu Yes, my opinion is that one should set that up ahead of time | 15:23 |
| jungleboyj | isaacb: I will take a look later. | 15:23 |
| jungleboyj | e0ne: I think sean is traveling today. | 15:24 |
| isaacb | jungleboyj: Thanks | 15:24 |
| tommylikehu | jgriffith_: does it possible to make it done automatically when setting up the service | 15:24 |
| jgriffith_ | tommylikehu maybe but it wasn't exactly high on my priority list | 15:25 |
| jgriffith_ | :) | 15:25 |
| tommylikehu | jgriffith_: just confirm whether I am heading to the possible way :) | 15:25 |
| jgriffith_ | tommylikehu I personally was good with having that be a prerequisite, that way a user could determine if they're using loop-backs, how large of a VG etc etc | 15:27 |
| jgriffith_ | I guess it depends on what direction and purpose we want this tool to have | 15:27 |
| e0ne | jungleboyj: thanks for the information | 15:28 |
| tommylikehu | jgriffith_ My purpose is to make the container easier to use, so either make it automatically or add some notes in the readme is both fine to me:) | 15:30 |
| jgriffith_ | tommylikehu yeah, I think there's some *other things* that need attention; like plugging up keystone as an option if desired, more flexibility in backends, and H/A :) | 15:34 |
| jgriffith_ | scale-out, multiple nodes etc | 15:34 |
| jgriffith_ | but at some point this doesn't *fit* in Cinder any more IMO | 15:35 |
| jgriffith_ | or maybe it does... that's up to the core Cinder folks to decide assuming there's interest and things move forward | 15:35 |
| openstackgerrit | Isaac Beckman proposed openstack/cinder master: IBM Storage- revert vol size capacity optimization https://review.openstack.org/472300 | 15:35 |
| tommylikehu | jgriffith_: thanks for you notes! | 15:36 |
| jgriffith_ | sure! | 15:36 |
| *** jgriffith_ is now known as jgriffith | 15:36 | |
| *** Jack_Iv has quit IRC | 15:41 | |
| *** Jack_Iv has joined #openstack-cinder | 15:41 | |
| *** Jack_Iv has quit IRC | 15:41 | |
| *** Jack_Iv has joined #openstack-cinder | 15:42 | |
| *** Jack_Iv has quit IRC | 15:42 | |
| *** Jack_Iv has joined #openstack-cinder | 15:43 | |
| *** zhaochao has quit IRC | 15:44 | |
| pewp | hemna (^-^*)/ | 15:46 |
| openstackgerrit | Eric Harney proposed openstack/python-cinderclient master: Unicode value support for "--filters" https://review.openstack.org/472303 | 15:47 |
| *** Jack_Iv has quit IRC | 15:47 | |
| tommylikehu | ping eharney | 15:47 |
| eharney | hey | 15:47 |
| *** e0ne has quit IRC | 15:48 | |
| tommylikehu | hey eharney since you reviewed this patch before | 15:48 |
| tommylikehu | https://review.openstack.org/#/c/469428/ | 15:48 |
| tommylikehu | can you share your point on this one? https://review.openstack.org/#/c/472034/ | 15:49 |
| *** Julien-zte has quit IRC | 15:49 | |
| eharney | sure | 15:49 |
| *** Julien-zte has joined #openstack-cinder | 15:49 | |
| *** Julien-zte has quit IRC | 15:50 | |
| *** Julien-zte has joined #openstack-cinder | 15:51 | |
| *** Julien-zte has quit IRC | 15:51 | |
| *** Apoorva has joined #openstack-cinder | 15:52 | |
| *** Apoorva has quit IRC | 15:52 | |
| *** Julien-zte has joined #openstack-cinder | 15:52 | |
| *** esker has quit IRC | 15:53 | |
| *** Julien-zte has quit IRC | 15:53 | |
| *** dikonoor has joined #openstack-cinder | 15:54 | |
| sdague | anyone want to hit the approve on https://review.openstack.org/#/c/472224/, the requirements patch is now approved | 15:54 |
| *** mlakat has quit IRC | 15:54 | |
| *** eharney has quit IRC | 15:57 | |
| *** Swanson has quit IRC | 16:00 | |
| *** Apoorva has joined #openstack-cinder | 16:01 | |
| *** isaacb has quit IRC | 16:07 | |
| *** eharney has joined #openstack-cinder | 16:08 | |
| *** Swanson has joined #openstack-cinder | 16:09 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST https://review.openstack.org/460191 | 16:10 |
| *** gkadam has joined #openstack-cinder | 16:12 | |
| *** crose has joined #openstack-cinder | 16:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder master: Updated from global requirements https://review.openstack.org/472311 | 16:16 |
| *** tesseract has quit IRC | 16:32 | |
| Swimingly | cmcginnis I've addressed your issues with https://review.openstack.org/#/c/464003/ | 16:34 |
| Swimingly | or smcginnis ^^ | 16:35 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Replication, replacing SMI-S with REST https://review.openstack.org/466027 | 16:41 |
| *** itlinux has joined #openstack-cinder | 16:48 | |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Retype, replacing SMI-S with REST https://review.openstack.org/456757 | 16:52 |
| *** lhx_ has quit IRC | 16:56 | |
| *** pcaruana has quit IRC | 17:00 | |
| jgriffith | stvnoyes ping | 17:00 |
| stvnoyes | jgriffith: to debug that os-brick issue I'm seeing, I went back to master nova and logged what the connection info that is sent to os-brick looks like on libvirt.driver.connect_volume. | 17:01 |
| stvnoyes | big differences in what is sent, i'll post it on pastebin - one sec | 17:02 |
| stvnoyes | http://paste.openstack.org/show/612031/ | 17:03 |
| stvnoyes | that's pretty ugly, let me reformat it | 17:04 |
| jgriffith | stvnoyes where are you plucking that out from? | 17:04 |
| stvnoyes | I inserted a log in libvirt.driver.connect_volume | 17:05 |
| stvnoyes | to print out the connection info going to so-brick | 17:05 |
| jgriffith | k | 17:05 |
| *** gkadam has quit IRC | 17:05 | |
| jgriffith | and this is from the live-migration path correct? | 17:06 |
| stvnoyes | no, normal attach | 17:06 |
| jgriffith | oh... but you're looking after just the attachment_create without the actual connection? | 17:07 |
| jgriffith | which makes no sense | 17:07 |
| stvnoyes | live migrate with the new attach looks pretty similar, only diff is serial is in the connect_info on migrate | 17:07 |
| jgriffith | Ok... I'll look at it. | 17:07 |
| stvnoyes | ~line 1158 in driver.py | 17:07 |
| jgriffith | I'm running stack.sh right now; should be done shortly | 17:07 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - QoS, replacing SMI-S with REST https://review.openstack.org/456759 | 17:08 |
| stvnoyes | it looks like os-brick wants that 'data' key in connection_info - you can see in here - https://github.com/openstack/os-brick/blob/c2bad1f4009e518b927b437469961b650e550e13/os_brick/initiator/initiator_connector.py | 17:09 |
| jgriffith | stvnoyes note that omitting the connector is intentional | 17:09 |
| stvnoyes | ok. I don't see os-brick looking for that. | 17:11 |
| jgriffith | stvnoyes at which point the delta isn't quite as significant as it would first seem | 17:11 |
| jgriffith | right | 17:11 |
| jgriffith | it doesn't need it | 17:11 |
| jgriffith | ie already has it | 17:11 |
| stvnoyes | i have to say i'm surprised that somehow the vol is connected even without having that data clause. but i'm wondering if not having it is causing problems during migrate. tho this is just speculation on my part | 17:12 |
| jgriffith | I hacked the code around that, all the info should still be there, just not in sub-keys | 17:13 |
| jgriffith | I also recently added a change to fake putting it back in... but have to look at that again | 17:13 |
| *** lucasxu has quit IRC | 17:14 | |
| jgriffith | stvnoyes https://review.openstack.org/#/c/330285/47/nova/volume/cinder.py | 17:14 |
| jgriffith | L#423 | 17:14 |
| jgriffith | so the "old" works the same that way | 17:14 |
| jgriffith | and everywhere else we should dump the check for that data key | 17:15 |
| ildikov | and connect_volume also took what was under 'data': https://review.openstack.org/#/c/330285/47/nova/virt/libvirt/volume/iscsi.py | 17:16 |
| stvnoyes | but i guess what I don't understand is whether os-brick needs that? | 17:16 |
| *** ericyoung has quit IRC | 17:16 | |
| jgriffith | stvnoyes needs which? device-path? | 17:17 |
| ildikov | jgriffith: the data key | 17:17 |
| jgriffith | ildikov the data key is irrelevant | 17:18 |
| jgriffith | https://review.openstack.org/#/c/330285/47/nova/volume/cinder.py | 17:18 |
| stvnoyes | http://paste.openstack.org/show/612035/ | 17:18 |
| ildikov | jgriffith: I think stvnoyes asked because of this: https://github.com/openstack/os-brick/blob/master/os_brick/initiator/initiator_connector.py#L63 | 17:19 |
| jgriffith | got it.. but that's an abstract interface and the fact is that the implementation in connectors/iscsi.py doesn't actually do anything with it or even check if it exists | 17:22 |
| jgriffith | had I know this was going to be such a hangup I would've just left the sub-dict in there | 17:22 |
| *** bjornar has quit IRC | 17:22 | |
| jgriffith | I'll just put it back and return all the extra info... maybe that would be better at this point | 17:22 |
| stvnoyes | not a hang up as such, i'm just looking at the differences on connection to see what could be going wrong. | 17:23 |
| jgriffith | just because there's some stuff in the docstring doesn't mean the docstring is right :) | 17:23 |
| stvnoyes | where is the actual impl for that? | 17:23 |
| jgriffith | stvnoyes well, you're the 4'th person I think to bring this up :) | 17:23 |
| ildikov | jgriffith: Nova doesn't seem to send it that way anyway | 17:23 |
| jgriffith | so it is a hangup :) | 17:23 |
| ildikov | jgriffith: put it back in Cinder to the original format you mean? | 17:25 |
| jgriffith | ildikov yes | 17:25 |
| jgriffith | it's definitely causing confusion | 17:25 |
| ildikov | jgriffith: but that changing the response format which I'm not sure we should do | 17:25 |
| jgriffith | stvnoyes so I removed it because when I was looking at this I was just like you and thought "why is this here, nobody uses it" | 17:26 |
| openstackgerrit | Helen Walsh proposed openstack/cinder master: VMAX driver - Compression, replacing SMI-S with REST https://review.openstack.org/460191 | 17:26 |
| jgriffith | so I *thought* it would be easier to simplify the structure. But in hindsight.. not such a great idea | 17:26 |
| jgriffith | ildikov we can do whatever we want | 17:26 |
| jgriffith | it's just code | 17:26 |
| jgriffith | it's not like it's the 10 commandments etched in stone :) | 17:27 |
| stvnoyes | :-) | 17:27 |
| stvnoyes | ok, I will dig more into this to see if I can see whee things are going bad | 17:27 |
| ildikov | jgriffith: yeah, until don't have it deployed and messed up with the different service versions... | 17:27 |
| *** ChubYann has joined #openstack-cinder | 17:27 | |
| jgriffith | stvnoyes ok, you're using the latest WIP patch right? | 17:27 |
| stvnoyes | well... no. I'm back a bit | 17:28 |
| jgriffith | stvnoyes let me know... I have this working on my side for the basic attach/detach | 17:28 |
| stvnoyes | basic attach/detach os working ok for me | 17:28 |
| stvnoyes | is working ok | 17:28 |
| jgriffith | stvnoyes oh.. so it's just the migration stuff? | 17:28 |
| stvnoyes | things go south on the migrate. | 17:28 |
| jgriffith | ok | 17:29 |
| jgriffith | that makes more sense | 17:29 |
| jgriffith | I haven't touched the migration stuff at all | 17:29 |
| jgriffith | I'll try and look on my side as well and we can sync back up | 17:29 |
| jgriffith | I'm rerunning devstack to set screen true because debugging with systemd is not something I've figured out yet | 17:29 |
| stvnoyes | btw, do you see errors like this when attaching? this using master (not your patch) - http://paste.openstack.org/show/612036/ | 17:31 |
| openstackgerrit | Merged openstack/cinder master: Send global_request_id to nova/glance when calls are made https://review.openstack.org/472224 | 17:31 |
| stvnoyes | are these 'normal'? | 17:31 |
| stvnoyes | i'm still using screens too. | 17:32 |
| patrickeast | stvnoyes: those are normal, its os-brick looking to see if it needs to start a session | 17:32 |
| stvnoyes | ok, i thought that might be the case | 17:32 |
| stvnoyes | thx | 17:32 |
| ildikov | stvnoyes: I did | 17:32 |
| ildikov | patrickeast: ah, good to know, tnx :) | 17:33 |
| openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Disallow use of assertTrue(str) https://review.openstack.org/471809 | 17:33 |
| jgriffith | we should really be catching the exception and not tracing | 17:33 |
| patrickeast | jgriffith: iirc its garbage that privsep spews out, not from os-brick | 17:34 |
| *** dinesh_b has quit IRC | 17:35 | |
| jgriffith | patrickeast yeah.. you're right | 17:37 |
| jgriffith | I've given up on that whole disaster ever being fixed | 17:37 |
| *** dhinesh has joined #openstack-cinder | 17:37 | |
| *** nicolasbock has quit IRC | 17:37 | |
| jgriffith | I still can't believe there's no way to stop polluting the logs from privsep | 17:37 |
| patrickeast | jgriffith: we probably could, iirc this was a thing a while back that hemna was fighting since it was spamming at like warning or error level before | 17:38 |
| patrickeast | at least its debug level now... progress! | 17:39 |
| hemna | heh | 17:39 |
| jgriffith | +1 | 17:39 |
| hemna | good luck with that | 17:39 |
| jgriffith | baby steps | 17:39 |
| hemna | I tried to fight that fight. | 17:39 |
| jgriffith | hemna yeah, I remember you were embroiled there for a while :( | 17:39 |
| jgriffith | this house of cards we've built sure is tall | 17:40 |
| hemna | yah | 17:40 |
| jgriffith | uhoh.. I feel a sneeze coming on | 17:40 |
| hemna | the privsep dumps are so amazingly annoying | 17:40 |
| jgriffith | they really are! | 17:40 |
| jgriffith | Especially if you're trying to debug a real problem | 17:41 |
| patrickeast | jgriffith: +1 | 17:41 |
| patrickeast | annoying, except when very useful ;) | 17:41 |
| hemna | I tried arguing over and over that we expect certain calls to fail, and we are catching those. don't log the issues, let us, the caller, decide what to log. | 17:41 |
| jgriffith | patrickeast left, except when it's right | 17:41 |
| jgriffith | hemna most would agree that's how a lib should work | 17:41 |
| jgriffith | at least in this particular context | 17:41 |
| *** crose has quit IRC | 17:42 | |
| hemna | it's the same way processutils works. | 17:42 |
| hemna | oh well. back under the rock.... | 17:42 |
| *** itlinux has quit IRC | 17:45 | |
| *** nicolasbock has joined #openstack-cinder | 17:53 | |
| *** jwcroppe has quit IRC | 17:56 | |
| *** jwcroppe has joined #openstack-cinder | 17:57 | |
| *** jwcroppe_ has joined #openstack-cinder | 17:59 | |
| *** lucasxu has joined #openstack-cinder | 18:00 | |
| *** jwcroppe has quit IRC | 18:01 | |
| *** pewp has quit IRC | 18:10 | |
| *** pewp has joined #openstack-cinder | 18:12 | |
| *** _alastor__ has quit IRC | 18:17 | |
| *** mvk has quit IRC | 18:24 | |
| *** salv-orl_ has quit IRC | 18:25 | |
| *** salv-orlando has joined #openstack-cinder | 18:26 | |
| jgriffith | hemna you mean bridge, right? | 18:28 |
| *** _alastor__ has joined #openstack-cinder | 18:30 | |
| *** salv-orlando has quit IRC | 18:30 | |
| *** dustins has quit IRC | 18:33 | |
| *** dustins has joined #openstack-cinder | 18:33 | |
| scottda | ha | 18:35 |
| *** dikonoor has quit IRC | 18:35 | |
| hemna | :) | 18:36 |
| *** itlinux_ has joined #openstack-cinder | 18:38 | |
| *** diablo_rojo has quit IRC | 18:38 | |
| *** nicolasbock has quit IRC | 18:45 | |
| *** nicolasbock has joined #openstack-cinder | 18:48 | |
| *** crushil has quit IRC | 18:50 | |
| *** nicolasbock has quit IRC | 18:50 | |
| *** nicolasbock has joined #openstack-cinder | 18:50 | |
| *** mnestratov|2 has quit IRC | 18:56 | |
| *** jwcroppe_ has quit IRC | 18:58 | |
| *** dave-mccowan has quit IRC | 18:58 | |
| *** eharney has quit IRC | 18:58 | |
| *** cristicalin has quit IRC | 19:00 | |
| *** nicolasbock has quit IRC | 19:00 | |
| *** dave-mccowan has joined #openstack-cinder | 19:02 | |
| *** jwcroppe has joined #openstack-cinder | 19:03 | |
| *** jwcroppe has quit IRC | 19:04 | |
| *** jwcroppe has joined #openstack-cinder | 19:05 | |
| *** nicolasbock has joined #openstack-cinder | 19:05 | |
| *** diablo_rojo has joined #openstack-cinder | 19:05 | |
| *** Jack_Iv has joined #openstack-cinder | 19:09 | |
| Swanson | Something change with multipath in cinder/brick/nova/tempest? | 19:13 |
| *** pewp has quit IRC | 19:13 | |
| *** e0ne has joined #openstack-cinder | 19:14 | |
| *** e0ne has quit IRC | 19:17 | |
| *** pewp has joined #openstack-cinder | 19:17 | |
| *** nicolasbock has quit IRC | 19:18 | |
| *** nicolasbock has joined #openstack-cinder | 19:18 | |
| *** nicolasbock has quit IRC | 19:23 | |
| patrickeast | Swanson: its kinda f'ed at the moment from what i've seen | 19:23 |
| Swanson | patrickeast, Oh, thank god. I thought something broke my driver like a fat man on a trampoline. | 19:25 |
| Swanson | (May have watched too many fail videos last night.) | 19:25 |
| *** itlinux_ has quit IRC | 19:28 | |
| patrickeast | Lol | 19:31 |
| patrickeast | Yea I didn't spend too much time trying to fix it, I tested from source open-iscsi and multipath-tools with gorkas os-brick changes and it works so my plan of action for now is to ignore it | 19:32 |
| patrickeast | But my CI couldn't pass an iscsi job right now to save it's life | 19:33 |
| *** e0ne has joined #openstack-cinder | 19:36 | |
| *** mvk has joined #openstack-cinder | 19:41 | |
| *** itlinux_ has joined #openstack-cinder | 19:44 | |
| Swanson | Same with mine. Thanks for the update. | 19:45 |
| *** asselin__ has quit IRC | 19:46 | |
| *** salv-orlando has joined #openstack-cinder | 19:57 | |
| *** harlowja has quit IRC | 20:08 | |
| *** xyang has quit IRC | 20:08 | |
| *** liverpooler has quit IRC | 20:09 | |
| *** xyang has joined #openstack-cinder | 20:14 | |
| *** xyang has quit IRC | 20:15 | |
| *** xyang has joined #openstack-cinder | 20:16 | |
| *** tomh_ has quit IRC | 20:16 | |
| *** tomh_ has joined #openstack-cinder | 20:17 | |
| *** xyang has quit IRC | 20:17 | |
| *** Jack_Iv has quit IRC | 20:17 | |
| *** Jack_Iv has joined #openstack-cinder | 20:18 | |
| *** Jack_Iv has quit IRC | 20:22 | |
| *** e0ne has quit IRC | 20:30 | |
| *** tomh_ has quit IRC | 20:33 | |
| *** tomh has joined #openstack-cinder | 20:33 | |
| *** laurent\ has quit IRC | 20:33 | |
| *** Apoorva_ has joined #openstack-cinder | 20:33 | |
| *** Apoorva has quit IRC | 20:36 | |
| *** laurent\ has joined #openstack-cinder | 20:37 | |
| *** Apoorva_ has quit IRC | 20:41 | |
| *** Apoorva has joined #openstack-cinder | 20:41 | |
| *** salv-orl_ has joined #openstack-cinder | 20:44 | |
| *** salv-orlando has quit IRC | 20:47 | |
| *** Apoorva_ has joined #openstack-cinder | 20:59 | |
| *** jonesn has quit IRC | 20:59 | |
| *** xyang has joined #openstack-cinder | 21:00 | |
| *** xyang has quit IRC | 21:01 | |
| *** Apoorva has quit IRC | 21:02 | |
| *** xyang has joined #openstack-cinder | 21:04 | |
| *** xyang1 has joined #openstack-cinder | 21:05 | |
| *** lucasxu has quit IRC | 21:11 | |
| *** rods has quit IRC | 21:11 | |
| *** abishop has quit IRC | 21:22 | |
| *** rods has joined #openstack-cinder | 21:30 | |
| *** dustins has quit IRC | 21:32 | |
| *** mriedem is now known as mriedem_afk | 21:48 | |
| *** mriedem_afk has quit IRC | 21:48 | |
| *** catintheroof has quit IRC | 21:50 | |
| Swanson | If one is retypeing an attached volume from one backend to another and the luns change this used to fail right? Was this fixed? | 21:50 |
| *** chlong has quit IRC | 21:53 | |
| *** xyang has quit IRC | 21:55 | |
| *** xyang1 has quit IRC | 21:57 | |
| *** xyang has joined #openstack-cinder | 21:58 | |
| *** xyang1 has joined #openstack-cinder | 21:59 | |
| *** adisky__ has quit IRC | 22:00 | |
| stvnoyes | jgriffith - yt? | 22:00 |
| jgriffith | stvnoyes hey | 22:01 |
| jgriffith | Swanson no, it works | 22:01 |
| jgriffith | sadly I'm doing things with that right now and just ran it | 22:01 |
| *** edmondsw has quit IRC | 22:02 | |
| jgriffith | Swanson but don't forget you need the --migration-policy flag | 22:02 |
| jgriffith | and probably some policy things to go with it | 22:02 |
| stvnoyes | hey, i have a question about attachment_delete. i am finding that when that is executed. i lose connection to the volume during remove_iscsi_target | 22:02 |
| jgriffith | stvnoyes yeah... I would expect that | 22:03 |
| jgriffith | if the target gets nuked you no longer have anything to be connected too | 22:03 |
| stvnoyes | this is on migration. I am using the old attach id, and have the new attachment in place. but attach_delete seems to sever the connection | 22:03 |
| stvnoyes | even though the new connection is there | 22:04 |
| jgriffith | yes, attach delete is "terminate connection" | 22:04 |
| *** lyarwood has quit IRC | 22:04 | |
| jgriffith | stvnoyes hmm... maybe I'm not following here | 22:04 |
| jgriffith | *which* connection? | 22:04 |
| jgriffith | I mean, if you issue attachment_delete it's kinda doing what I would expect no? | 22:04 |
| stvnoyes | this is on post_migration, when I want to drop the connection to volume on the source host | 22:04 |
| jgriffith | "deleting the attachment" | 22:04 |
| *** lyarwood has joined #openstack-cinder | 22:04 | |
| ildikov | it sounds like the multi-attach detach issue | 22:04 |
| jgriffith | stvnoyes oh you mean it kills both connections? | 22:05 |
| stvnoyes | during pre migration I setup the connection to the same volume using a new attachment | 22:05 |
| jgriffith | ie both attachments? | 22:05 |
| stvnoyes | yes, it seems to (although it could be the new connectino is not right) | 22:05 |
| jgriffith | stvnoyes :) | 22:05 |
| *** dave-mccowan has quit IRC | 22:05 | |
| *** xyang has quit IRC | 22:06 | |
| jgriffith | shared_connections | 22:07 |
| jgriffith | can you look in your cinder db and check the connection info there? | 22:07 |
| jgriffith | I suspect we're ending up using the same target instead of a new one | 22:07 |
| jgriffith | and there's no ref counting on cinder on the attachment-delete to deal with that | 22:07 |
| jgriffith | and yes ildikov is right | 22:08 |
| ildikov | even for a different host? | 22:08 |
| stvnoyes | which table is that in? | 22:08 |
| ildikov | volume_attachments maybe? | 22:10 |
| stvnoyes | not sure, I don't see connection info data in there. jgriffith- where exactly is connection info? | 22:11 |
| jgriffith | stvnoyes geesh... just ask the hard questions why don't ya! | 22:12 |
| jgriffith | just a sec | 22:12 |
| jgriffith | :) | 22:12 |
| jgriffith | stvnoyes attachment_specs has the connector.... | 22:12 |
| ildikov | or wasn't this attachment_specs thing the one? | 22:12 |
| ildikov | ok, cool :) | 22:13 |
| jgriffith | volume_attachment has the "other stuff" | 22:13 |
| jgriffith | like everythings | 22:13 |
| stvnoyes | yikes! big table | 22:13 |
| jgriffith | yeah :( | 22:13 |
| stvnoyes | now to find the specifc attachments... | 22:13 |
| ildikov | jgriffith: yep, I remember now | 22:13 |
| jgriffith | stvnoyes so I have a feeling this is what ildikov was mentioning | 22:14 |
| jgriffith | we need to make a change on cinder I suspect to deal with it | 22:14 |
| stvnoyes | ok. | 22:14 |
| jgriffith | stvnoyes it's pretty easy to fix for lvm... | 22:14 |
| stvnoyes | excellent, those are the best ones. | 22:14 |
| patrickeast | anyone happen to know if there is a way to get devstack to *not* install some package listed in https://github.com/openstack-dev/devstack/blob/master/files/debs/cinder | 22:14 |
| jgriffith | we just iterate through and if there are any current attachments we don't nuke it | 22:15 |
| jgriffith | patrickeast only way I know is to edit that file | 22:15 |
| Swanson | jgriffith, Yeah, it is mostly working. OSP-8 is giving me issues and I see some luns changing in the logs. Vaguely remember an issue but have found nothing. | 22:15 |
| patrickeast | jgriffith: dang, i was worried that might be the answer | 22:15 |
| jgriffith | patrickeast but I suspect that doesn't work | 22:15 |
| jgriffith | for you | 22:15 |
| patrickeast | it does, just kinda gross | 22:15 |
| patrickeast | script in some sed command in a pre-test hook... | 22:16 |
| * patrickeast wanders back into ci land | 22:16 | |
| jgriffith | yeah... figured this was ci creating some difficulty for ya | 22:16 |
| patrickeast | :( yep, need to use tip of tree open-iscsi | 22:16 |
| patrickeast | but installing it before or after an apt-get install open-iscsi breaks things badly | 22:17 |
| jgriffith | patrickeast haha :) | 22:17 |
| patrickeast | so i need to stop devstack from being helpful | 22:17 |
| jgriffith | yes, I can imagine | 22:17 |
| jgriffith | hmmm | 22:17 |
| jgriffith | patrickeast what about a files/xxx/pure | 22:17 |
| jgriffith | and post config it to apt-get remove and put tip in it's place? | 22:18 |
| jgriffith | might be cleaner | 22:18 |
| patrickeast | the problem is that devstack slurps in anything that matches the enabled services | 22:18 |
| patrickeast | apt-get remove and purge only works with a reboot :*( | 22:18 |
| jgriffith | awwwww | 22:18 |
| patrickeast | which screws the freshly stacked devstack | 22:18 |
| jgriffith | table-flip | 22:18 |
| patrickeast | lol | 22:18 |
| pewp | hemna (ノ ゜Д゜)ノ ︵ ┻━┻ | 22:19 |
| *** salv-orl_ has quit IRC | 22:19 | |
| ildikov | jgriffith: so to avoid any sort of race things to happen there I guess this is where we were mentioning some locking | 22:20 |
| *** salv-orlando has joined #openstack-cinder | 22:20 | |
| patrickeast | i'll just do a find -exec sed thing, it'l be a beautiful line of bash to compliment all the other duct-tape and toothpicks | 22:20 |
| ildikov | jgriffith: that might be more the case when we have multi-attach, just thought bring up to keep it in mind if we would end up fixing this issue | 22:20 |
| *** Apoorva_ has quit IRC | 22:21 | |
| *** jdurgin has quit IRC | 22:21 | |
| stvnoyes | jgriffith - here's the db info for the attachments - http://paste.openstack.org/show/612061/ | 22:21 |
| *** Apoorva has joined #openstack-cinder | 22:21 | |
| jgriffith | patrickeast :) | 22:21 |
| Swanson | patrickeast, jebus. I was going to just let mine fail until it works for I get a "cease and desist" patch sent to me from smcginnis | 22:21 |
| Swanson | or I get | 22:22 |
| jgriffith | stvnoyes well shit, that's not useful :) | 22:22 |
| jgriffith | the tgt info that's likely in the volume model | 22:22 |
| jgriffith | stvnoyes don't worry though, I think I know what the problem is and how to fix it | 22:23 |
| jgriffith | let me see if I can figure out a quick cinder patch for it | 22:23 |
| stvnoyes | ok cool. time for dinner here. So this is a good stopping point for me. if you want to grab my migrate patch, i posted it here - https://review.openstack.org/#/c/463987/ - I wasn't going to set it for review until I see it work all the way thru | 22:24 |
| jgriffith | stvnoyes thanks! | 22:24 |
| *** salv-orlando has quit IRC | 22:24 | |
| stvnoyes | my pleasure. It's nice to see this getting close... | 22:24 |
| jgriffith | If I somehow manage to get through this swap stuff before midnight I'll take a look | 22:24 |
| jgriffith | stvnoyes it wouldn't be without your help!!! So THANK YOU!! | 22:25 |
| ildikov | jgriffith: +1 | 22:25 |
| *** cknight has quit IRC | 22:25 | |
| *** jdurgin has joined #openstack-cinder | 22:30 | |
| *** xyang1 has quit IRC | 22:39 | |
| *** jwcroppe has quit IRC | 22:39 | |
| openstackgerrit | TommyLike proposed openstack/python-cinderclient master: Enabled like filter support in client https://review.openstack.org/466214 | 22:48 |
| *** diablo_rojo has quit IRC | 22:51 | |
| dhinesh | is a third party CI setup necessary to evaluate and merge code to openstack-cinder? | 22:58 |
| *** itlinux_ has quit IRC | 23:03 | |
| jgriffith | dhinesh it's only required if you are a 3'rd party driver/maintainer | 23:03 |
| jgriffith | dhinesh to merge you're driver it's required to have a 3'rd party CI system that tests it | 23:03 |
| jgriffith | dhinesh but in general contributing to the project, no certainly not | 23:04 |
| *** felipemonteiro__ has quit IRC | 23:05 | |
| *** sdague has quit IRC | 23:08 | |
| *** Jack_Iv has joined #openstack-cinder | 23:18 | |
| *** pewp has quit IRC | 23:21 | |
| *** pewp has joined #openstack-cinder | 23:22 | |
| *** Jack_Iv has quit IRC | 23:22 | |
| *** Apoorva has quit IRC | 23:25 | |
| *** dhinesh has quit IRC | 23:29 | |
| *** dhinesh has joined #openstack-cinder | 23:30 | |
| *** markvoelker has quit IRC | 23:34 | |
| dhinesh | jgriffith: so if i have a new driver implementation like this https://review.openstack.org/#/c/433341/ its required? | 23:36 |
| jgriffith | dhinesh yes | 23:37 |
| jgriffith | https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 23:39 |
| dhinesh | jgriffith: thanks | 23:40 |
| *** Apoorva has joined #openstack-cinder | 23:40 | |
| *** harlowja has joined #openstack-cinder | 23:57 | |
| *** jamielennox is now known as jamielennox|away | 23:57 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!