| *** EinstCrazy has joined #openstack-cinder | 00:01 | |
| *** akshai has quit IRC | 00:01 | |
| *** markvoelker has quit IRC | 00:01 | |
| *** xyang1 has quit IRC | 00:03 | |
| *** bapalm has quit IRC | 00:03 | |
| *** bapalm has joined #openstack-cinder | 00:04 | |
| *** ntpttr has joined #openstack-cinder | 00:09 | |
| *** mc_nair has quit IRC | 00:11 | |
| *** EinstCrazy has quit IRC | 00:12 | |
| *** zhangjn has quit IRC | 00:15 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Show qos_specs_id based on policy https://review.openstack.org/241659 | 00:15 |
|---|---|---|
| *** rs007 has joined #openstack-cinder | 00:19 | |
| *** ntpttr has quit IRC | 00:19 | |
| *** davechen1 has left #openstack-cinder | 00:20 | |
| *** edtubill has joined #openstack-cinder | 00:26 | |
| *** setmason_ has joined #openstack-cinder | 00:28 | |
| *** daneyon_ has joined #openstack-cinder | 00:29 | |
| *** setmason has quit IRC | 00:30 | |
| *** daneyon has quit IRC | 00:32 | |
| *** jamielennox|away is now known as jamielennox | 00:32 | |
| *** setmason_ has quit IRC | 00:33 | |
| *** edtubill has quit IRC | 00:36 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error in _find_host_exhaustive() https://review.openstack.org/247546 | 00:38 |
| *** Splitpeas has quit IRC | 00:52 | |
| *** salv-orl_ has joined #openstack-cinder | 00:55 | |
| *** salv-orlando has quit IRC | 00:58 | |
| *** apoorvad has quit IRC | 00:58 | |
| *** rs007 has quit IRC | 00:58 | |
| *** jaypipes has quit IRC | 00:59 | |
| *** EinstCrazy has joined #openstack-cinder | 01:01 | |
| *** leeantho has quit IRC | 01:03 | |
| *** zhangjn has joined #openstack-cinder | 01:05 | |
| *** houming has joined #openstack-cinder | 01:06 | |
| *** zhenguo has joined #openstack-cinder | 01:08 | |
| *** flip214 has quit IRC | 01:09 | |
| *** houming has quit IRC | 01:10 | |
| *** flip214 has joined #openstack-cinder | 01:11 | |
| *** flip214 has quit IRC | 01:11 | |
| *** flip214 has joined #openstack-cinder | 01:11 | |
| *** garthb has quit IRC | 01:16 | |
| *** bswartz has joined #openstack-cinder | 01:17 | |
| *** markvoelker has joined #openstack-cinder | 01:17 | |
| *** markvoelker has quit IRC | 01:22 | |
| *** setmason has joined #openstack-cinder | 01:30 | |
| *** wilson-liu has quit IRC | 01:31 | |
| *** wilson-liu has joined #openstack-cinder | 01:32 | |
| *** Lee1092 has joined #openstack-cinder | 01:32 | |
| *** chhavi has joined #openstack-cinder | 01:37 | |
| *** mriedem has joined #openstack-cinder | 01:41 | |
| *** markvoelker has joined #openstack-cinder | 01:50 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Delete unused codes in rbd.retype https://review.openstack.org/244460 | 01:54 |
| openstackgerrit | Merged openstack/os-brick: Updated from global requirements https://review.openstack.org/246102 | 01:56 |
| *** apoorvad has joined #openstack-cinder | 01:58 | |
| openstackgerrit | Zhang Ni proposed openstack/cinder-specs: Add local tgt to lvm When the instance is in the same host with LVM volume, it's better to use local dev ( eg. /dev/ubuntu50-vg2/volume-9531dcba-16c5-4f1f-8b61-f0043d0b6a15) of lvm volume for high performence. So this blueprint will add a local tgt for LV https://review.openstack.org/247880 | 02:03 |
| *** apoorvad has quit IRC | 02:03 | |
| mriedem | anyone know tomoki sekiyama? | 02:03 |
| jgriffith | mriedem: yes | 02:03 |
| openstackgerrit | Thang Pham proposed openstack/cinder: Update replication to use versionedobjects https://review.openstack.org/238906 | 02:04 |
| jgriffith | mriedem: just taking a poll? :) | 02:04 |
| mriedem | jgriffith: wondering if they are still working on https://review.openstack.org/#/c/196482/ | 02:05 |
| jgriffith | hrmm | 02:05 |
| mriedem | didn't know irc nick | 02:05 |
| jgriffith | He's tsekiyama | 02:06 |
| jgriffith | I usually see him around in the mornings | 02:06 |
| jgriffith | but I must admit I haven't caught up with him here in quite a while | 02:06 |
| jgriffith | mriedem: I'll keep an eye out for him | 02:07 |
| jgriffith | mriedem: else maybe we should reassign that work | 02:07 |
| mriedem | that's fine, i just asked in the change | 02:07 |
| mriedem | i'm going through old reviews and looking for abandons | 02:07 |
| jgriffith | mriedem: good deal | 02:08 |
| openstackgerrit | Zhang Ni proposed openstack/cinder-specs: Add local tgt to lvm https://review.openstack.org/247880 | 02:10 |
| *** bswartz has quit IRC | 02:11 | |
| *** jwcroppe has quit IRC | 02:12 | |
| *** _amrith_ is now known as amrith | 02:14 | |
| *** jerrygb has quit IRC | 02:14 | |
| *** apoorvad has joined #openstack-cinder | 02:16 | |
| *** Yogi1 has joined #openstack-cinder | 02:18 | |
| *** bswartz has joined #openstack-cinder | 02:19 | |
| *** jerrygb has joined #openstack-cinder | 02:24 | |
| *** apoorvad has quit IRC | 02:30 | |
| *** haomaiwang has joined #openstack-cinder | 02:32 | |
| *** dalgaaf has quit IRC | 02:32 | |
| lixiaoy1 | mriedem: ping | 02:35 |
| lixiaoy1 | mriedem: As both Cinder and Nova need nova/volume/encryptors, I am adding this part to os-brick | 02:36 |
| *** haomaiwang has quit IRC | 02:37 | |
| *** Yogi1 has quit IRC | 02:37 | |
| mriedem | lixiaoy1: are you referring to https://review.openstack.org/#/c/196482/ ? | 02:37 |
| lixiaoy1 | please have a look at https://review.openstack.org/#/c/247372/ | 02:37 |
| *** gouthamr has joined #openstack-cinder | 02:38 | |
| mriedem | oh boy | 02:38 |
| mriedem | so that needs a blueprint i think | 02:38 |
| mriedem | jgriffith: ^ | 02:38 |
| mriedem | smcginnis: ^ | 02:38 |
| mriedem | hemna: ^ | 02:38 |
| *** haomaiwang has joined #openstack-cinder | 02:38 | |
| smcginnis | mriedem: Agreed | 02:38 |
| mriedem | specifically b/c before nova is going to consume that, nova has to integrate the privsep stuff | 02:38 |
| mriedem | for rootwrap filters | 02:38 |
| mriedem | that came up in the nova meeting today | 02:39 |
| *** dims_ has quit IRC | 02:39 | |
| smcginnis | mriedem: Are you going to privsep already? | 02:39 |
| lixiaoy1 | I asked Sean, and he thinks this is just moving to os-brick. no need bp | 02:39 |
| smcginnis | Well.. | 02:39 |
| smcginnis | Moving where things are, but a bp might be good to make sure all users (cinder and nove) understand what's going on. | 02:40 |
| mriedem | smcginnis: sdague said in the nova meeting today that nova will have a bp for tracking the privsep work that has apparently landed in oslo but hasn't been released yet | 02:40 |
| mriedem | we have to get going on that earlier than later probably | 02:40 |
| mriedem | otherwise we're going to have to keep duplicate rootwrap filters | 02:41 |
| mriedem | *duplicating | 02:41 |
| smcginnis | mriedem: Ah, cool. I've been taking a wait and see approach. | 02:41 |
| mriedem | mikal was going to follow up with gus on the bp work in nova | 02:41 |
| smcginnis | mriedem: I'm a little nervous picking it up right away before anyone has worked it out. | 02:41 |
| mriedem | early adopters :) | 02:41 |
| smcginnis | mriedem: I'll let Nova do that. :D | 02:41 |
| mriedem | heh | 02:41 |
| mriedem | you bastard! | 02:41 |
| smcginnis | hehe | 02:42 |
| lixiaoy1 | smcginnis: you mean let Nova maintain rootwrap filters | 02:42 |
| lixiaoy1 | ? | 02:42 |
| smcginnis | lixiaoy1: No, apparently the plan is to get rid of rootwrap faster than I thought would happen. | 02:42 |
| smcginnis | mriedem: You'd like lixiaoy1 to submit a bp on the nova side, right? | 02:43 |
| lixiaoy1 | I have a bp in cinder | 02:43 |
| smcginnis | lixiaoy1: Right, but nova changes will be required as well, right? | 02:44 |
| lixiaoy1 | early, I mean no spec is wroten | 02:44 |
| smcginnis | IIRC, the plan is to have both nova and cinder use the encryptors from os-brick rather than copy them between both, right? | 02:44 |
| lixiaoy1 | I will put encryptors in os-brick, and later nova will be changed to call | 02:44 |
| lixiaoy1 | yes | 02:45 |
| smcginnis | lixiaoy1: So I think getting them in os-brick - you are all set. | 02:45 |
| mriedem | nova will need a compainion bp yes | 02:45 |
| smcginnis | lixiaoy1: But in order to update nova to use them, you will need a bp there for that piece of work. | 02:46 |
| lixiaoy1 | this is bp in cinder: https://blueprints.launchpad.net/cinder/+spec/improve-encrypted-volume | 02:46 |
| mriedem | but i think the nova team should have input on the blueprint/spec for this, i.e. why it's needed and what the plan is | 02:46 |
| smcginnis | mriedem: OK, cool. | 02:46 |
| mriedem | also noting that os-brick has historically really been only the pieces of nova that have been in the libvirt driver in nova, | 02:46 |
| mriedem | the volume encryptor stuff is not necessarily libvirt specific | 02:46 |
| smcginnis | lixiaoy1: Can you submit one now, even if things aren't quite ready to make any changes there yet? | 02:46 |
| lixiaoy1 | ok | 02:47 |
| *** elonden has joined #openstack-cinder | 02:47 | |
| mriedem | although i feel like there are assumptions made that they are used together (since no other virt driver supports encrypted volumes from what i can tell) | 02:47 |
| smcginnis | mriedem: Yeah, this may be a slight expansion of scope. | 02:47 |
| mriedem | posting this to the -dev list would be good too as a heads up for nova people, this is the first i've heard of it | 02:47 |
| mriedem | bbiab | 02:47 |
| smcginnis | mriedem: It's kind of another rootwrap/copy the same stuff everywhere kind of situation. | 02:47 |
| smcginnis | mriedem: Good idea re: -dev. | 02:48 |
| smcginnis | lixiaoy1: Can you post something to the mailing list giving an overview of what is being doen. | 02:48 |
| smcginnis | ? | 02:48 |
| lixiaoy1 | ok, I will do it | 02:48 |
| smcginnis | lixiaoy1: Put [Cinder] and [Nova] in the subject line so both teams pay attention to it. | 02:48 |
| smcginnis | lixiaoy1: Thanks! | 02:48 |
| lixiaoy1 | yes | 02:49 |
| lixiaoy1 | welcome | 02:49 |
| smcginnis | lixiaoy1: I think it makes sense, but mriedem is right, we should probably make sure everyone is aware and understands what's being proposed. | 02:49 |
| lixiaoy1 | I will | 02:49 |
| smcginnis | Awesome, thank you. | 02:50 |
| lixiaoy1 | my pleasure | 02:50 |
| *** sghanekar_ has quit IRC | 02:53 | |
| *** chenying has quit IRC | 02:55 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Robustify writing iscsi target persistence file https://review.openstack.org/227943 | 02:55 |
| *** houming has joined #openstack-cinder | 02:56 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Robustify writing iscsi target persistence file https://review.openstack.org/247886 | 02:59 |
| *** gouthamr has quit IRC | 03:00 | |
| *** gouthamr has joined #openstack-cinder | 03:01 | |
| *** rushiagr_away is now known as rushiagr | 03:02 | |
| *** jdurgin1 has quit IRC | 03:03 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: [WIP] Robustify writing iscsi target persistence file https://review.openstack.org/227943 | 03:09 |
| *** bardia has quit IRC | 03:12 | |
| smcginnis | hemna: Question about bug 1487169... | 03:15 |
| openstack | bug 1487169 in os-brick "multipath -l can be very slow " [Medium,In progress] https://launchpad.net/bugs/1487169 - Assigned to Walt Boring (walter-boring) | 03:15 |
| smcginnis | hemna: You got a patch in that said it was a partial fix. | 03:15 |
| smcginnis | hemna: Wondering what else we can do there? | 03:15 |
| *** setmason_ has joined #openstack-cinder | 03:16 | |
| *** setmason has quit IRC | 03:19 | |
| *** setmason_ is now known as setmason | 03:19 | |
| *** edmondsw has quit IRC | 03:19 | |
| *** dims has joined #openstack-cinder | 03:21 | |
| *** amrith is now known as _amrith_ | 03:22 | |
| *** gcb has quit IRC | 03:35 | |
| *** gcb has joined #openstack-cinder | 03:35 | |
| *** setmason has quit IRC | 03:44 | |
| *** chhavi has quit IRC | 03:44 | |
| *** setmason has joined #openstack-cinder | 03:45 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Register RPC and object versions https://review.openstack.org/209701 | 03:50 |
| *** thangp has joined #openstack-cinder | 03:51 | |
| *** thangp has quit IRC | 03:53 | |
| *** Thelo has quit IRC | 03:59 | |
| *** mriedem has quit IRC | 03:59 | |
| *** yusuke has left #openstack-cinder | 04:02 | |
| *** yusuke has joined #openstack-cinder | 04:03 | |
| *** Thelo has joined #openstack-cinder | 04:05 | |
| *** dave-mccowan has quit IRC | 04:07 | |
| *** swebb has quit IRC | 04:07 | |
| *** Thelo has quit IRC | 04:08 | |
| *** EmilienM has quit IRC | 04:08 | |
| *** EmilienM has joined #openstack-cinder | 04:10 | |
| *** Thelo has joined #openstack-cinder | 04:10 | |
| *** zhangjn has quit IRC | 04:14 | |
| *** swebb has joined #openstack-cinder | 04:14 | |
| *** Thelo has quit IRC | 04:16 | |
| *** Thelo has joined #openstack-cinder | 04:16 | |
| *** gouthamr has quit IRC | 04:22 | |
| *** mc_nair has joined #openstack-cinder | 04:23 | |
| *** asselin has quit IRC | 04:25 | |
| *** asselin has joined #openstack-cinder | 04:25 | |
| *** zhangjn has joined #openstack-cinder | 04:29 | |
| *** houming has quit IRC | 04:30 | |
| *** dims has quit IRC | 04:36 | |
| *** rushiagr is now known as rushiagr_away | 04:44 | |
| *** zhangjn has quit IRC | 04:45 | |
| *** zhonghua-lee has quit IRC | 04:56 | |
| *** zhonghua-lee has joined #openstack-cinder | 04:56 | |
| yusuke | Hi all, excuse me. Do you know alternative url or web site of following url? Although I know how to setup Cinder CI for FibreChannel Protocol, this link looks dead.... | 05:10 |
| yusuke | https://git.openstack.org/cgit/stackforge/third-party-ci-tools/tree/provisioning_scripts/fibre_channel | 05:10 |
| yusuke | (from https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers) | 05:10 |
| *** zhangjn has joined #openstack-cinder | 05:11 | |
| *** zhangjn has quit IRC | 05:12 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add a GateDriver https://review.openstack.org/244956 | 05:13 |
| *** houming has joined #openstack-cinder | 05:14 | |
| *** zhangjn has joined #openstack-cinder | 05:18 | |
| *** mc_nair has quit IRC | 05:21 | |
| *** houming has quit IRC | 05:22 | |
| yusuke | s/Although I know how to/Although I want to know how to/ | 05:23 |
| *** jerrygb has quit IRC | 05:29 | |
| *** rushiagr_away is now known as rushiagr | 05:32 | |
| openstackgerrit | Merged openstack/cinder: VNX: Fix metadata get overriden issue https://review.openstack.org/246238 | 05:32 |
| *** stevemar_ has quit IRC | 05:34 | |
| *** stevemar_ has joined #openstack-cinder | 05:35 | |
| *** stevemar_ has quit IRC | 05:38 | |
| *** haomaiwang has quit IRC | 05:46 | |
| openstackgerrit | LisaLi proposed openstack/os-brick: Copy encryptors from Nova to os-brick https://review.openstack.org/247372 | 05:50 |
| openstackgerrit | Cedric Zhuang proposed openstack/cinder: VNX: Get volume type from location throws error https://review.openstack.org/246230 | 05:51 |
| openstackgerrit | LisaLi proposed openstack/os-brick: Copy encryptors from Nova to os-brick https://review.openstack.org/247372 | 05:51 |
| *** CORTEX\krfa has quit IRC | 05:57 | |
| *** haomaiwang has joined #openstack-cinder | 05:59 | |
| *** shausy has joined #openstack-cinder | 06:01 | |
| openstackgerrit | Merged openstack/cinder: Remove .mailmap file https://review.openstack.org/247676 | 06:02 |
| yusuke | I'm sorry. I found alternative one, thanks. | 06:07 |
| *** jkraj has joined #openstack-cinder | 06:08 | |
| *** lpetrut has joined #openstack-cinder | 06:10 | |
| *** smoriya is now known as smoriya_afk | 06:11 | |
| *** haomaiwang has quit IRC | 06:14 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature https://review.openstack.org/243017 | 06:15 |
| *** haomaiwa_ has joined #openstack-cinder | 06:18 | |
| openstackgerrit | wanghao proposed openstack/cinder-specs: Improvement to query consistency group detail https://review.openstack.org/242998 | 06:21 |
| *** rcernin has joined #openstack-cinder | 06:30 | |
| *** jerrygb has joined #openstack-cinder | 06:30 | |
| *** jerrygb has quit IRC | 06:36 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature https://review.openstack.org/243017 | 06:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata https://review.openstack.org/247943 | 06:39 |
| *** elonden has left #openstack-cinder | 06:39 | |
| *** haomaiwa_ has quit IRC | 06:43 | |
| *** rushiagr is now known as rushiagr_away | 06:52 | |
| *** haomaiwang has joined #openstack-cinder | 06:53 | |
| *** salv-orlando has joined #openstack-cinder | 06:54 | |
| *** salv-orl_ has quit IRC | 06:58 | |
| *** haomaiwang has quit IRC | 07:01 | |
| *** haomaiwang has joined #openstack-cinder | 07:02 | |
| openstackgerrit | zhangsong proposed openstack/cinder: Fix the bug of can't get the desired image info https://review.openstack.org/245241 | 07:06 |
| *** bluex has joined #openstack-cinder | 07:07 | |
| openstackgerrit | zhangsong proposed openstack/cinder: Fix the bug of can't get the desired image info https://review.openstack.org/245241 | 07:13 |
| *** nkrinner has joined #openstack-cinder | 07:23 | |
| *** haomaiwang has quit IRC | 07:25 | |
| *** haomaiwang has joined #openstack-cinder | 07:26 | |
| *** anshul has joined #openstack-cinder | 07:28 | |
| *** Dhaval has joined #openstack-cinder | 07:30 | |
| *** ankit_ag has joined #openstack-cinder | 07:37 | |
| *** chenying_ has quit IRC | 07:40 | |
| *** alexschm has joined #openstack-cinder | 07:40 | |
| *** anshul has quit IRC | 07:40 | |
| *** anshul has joined #openstack-cinder | 07:40 | |
| *** zhangjn has quit IRC | 07:47 | |
| *** vgridnev has joined #openstack-cinder | 07:52 | |
| *** haomaiwang has quit IRC | 07:59 | |
| *** haomaiwang has joined #openstack-cinder | 08:02 | |
| Dhaval | I have installed Openstack on single machine using devstack | 08:07 |
| Dhaval | I am trying to develop sample third party driver for cinder for POC purpose | 08:07 |
| Dhaval | But not sure where should I start from | 08:08 |
| Dhaval | Can anyone please guide me about procedure to register sample cinder driver with openstack | 08:09 |
| *** egonzalez has joined #openstack-cinder | 08:11 | |
| dulek | Dhaval: Maybe this will be helpful: https://wiki.openstack.org/wiki/Cinder/how-to-contribute-a-driver | 08:15 |
| Dhaval | dulek : Thanks for sharing link | 08:18 |
| *** zhangjn has joined #openstack-cinder | 08:18 | |
| Dhaval | I am actually trying to implement sample cinder driver which could add/delete volumes just like LVM driver | 08:21 |
| Dhaval | I even modified cinder.conf file as follows and restarted cinder-api, cinder-scheduler and cinder-volume services | 08:22 |
| Dhaval | following contents were added in cinder.conf file: | 08:22 |
| Dhaval | enabled_backends = lvmdriver-1,lvmdriver-2 | 08:22 |
| Dhaval | [lvmdriver-2] volume_group = stack-volumes-lvmdriver-2 volume_driver = cinder.volume.drivers.lvm.LVMVolumeDriver volume_backend_name = lvmdriver-2 26,1 Bot | 08:23 |
| Dhaval | Even after restarting the cinder services; I am unable to see lvmdriver-2 in cinder service-list output | 08:23 |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Don't add project_id to auth_url in SwiftBkpDriver https://review.openstack.org/247964 | 08:24 |
| dulek | Dhaval: Take a look into the logs of c-vol. If driver was unable to be initialized, then c-vol won't report it's presence. | 08:24 |
| Dhaval | okay | 08:24 |
| dulek | Dhaval: LVM is uninitialized when for example volume_group doesn't exist. | 08:24 |
| dulek | Dhaval: You may also let DevStack configure everything for you. | 08:25 |
| dulek | Dhaval: Just add this to local.conf: | 08:25 |
| dulek | Dhaval: CINDER_ENABLED_BACKENDS=${CINDER_ENABLED_BACKENDS:-lvm:lvmdriver-1,lvm:lvmdriver-2} | 08:25 |
| Dhaval | dulek : Okay, let me try it out | 08:26 |
| *** rushiagr_away is now known as rushiagr | 08:26 | |
| *** zhangjn has quit IRC | 08:32 | |
| *** lpetrut has quit IRC | 08:35 | |
| *** houming has joined #openstack-cinder | 08:35 | |
| *** zhangjn has joined #openstack-cinder | 08:36 | |
| *** vgridnev has quit IRC | 08:40 | |
| *** shausy has quit IRC | 08:41 | |
| *** shausy has joined #openstack-cinder | 08:42 | |
| *** gcb has quit IRC | 08:43 | |
| *** gcb has joined #openstack-cinder | 08:44 | |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Adds Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 08:51 |
| *** zhangjn has quit IRC | 08:51 | |
| *** houming has quit IRC | 08:53 | |
| *** setmason has quit IRC | 08:53 | |
| *** markus_z has joined #openstack-cinder | 08:54 | |
| *** nkrinner has quit IRC | 08:58 | |
| *** nkrinner has joined #openstack-cinder | 08:58 | |
| *** haomaiwang has quit IRC | 09:01 | |
| *** smoriya_afk has quit IRC | 09:01 | |
| *** haomaiwang has joined #openstack-cinder | 09:01 | |
| *** houming has joined #openstack-cinder | 09:04 | |
| *** ndipanov has joined #openstack-cinder | 09:06 | |
| *** nkrinner has quit IRC | 09:10 | |
| *** houming has quit IRC | 09:12 | |
| *** nkrinner has joined #openstack-cinder | 09:14 | |
| *** egonzalez is now known as egonzalez90 | 09:17 | |
| *** egonzalez90 is now known as Guest24185 | 09:17 | |
| *** houming has joined #openstack-cinder | 09:18 | |
| *** Guest24185 has quit IRC | 09:18 | |
| *** egonzalez has joined #openstack-cinder | 09:18 | |
| smcginnis | yusuke: Please feel free to edit that wiki page with the correct link. Thanks! | 09:18 |
| *** chhavi has joined #openstack-cinder | 09:19 | |
| yusuke | OK, Thanks! | 09:19 |
| smcginnis | yusuke: Thanks for pointing it out! | 09:20 |
| *** haomaiwang has quit IRC | 09:25 | |
| *** haomaiwang has joined #openstack-cinder | 09:27 | |
| Dhaval | dulek : Thanks a lot for helping out. I am now able to see both lvmdriver-1 and lvmdriver-2 in cinder service-list output | 09:27 |
| Dhaval | dulek : had to create volume group for successful initialization | 09:27 |
| dulek | Dhaval: You're welcomed. | 09:28 |
| *** vgridnev has joined #openstack-cinder | 09:29 | |
| *** markvoelker has quit IRC | 09:30 | |
| *** e0ne has joined #openstack-cinder | 09:31 | |
| *** zhangjn has joined #openstack-cinder | 09:31 | |
| *** sgotliv has joined #openstack-cinder | 09:34 | |
| *** haomaiwang has quit IRC | 09:37 | |
| *** wilson-liu has quit IRC | 09:38 | |
| *** wilson-liu has joined #openstack-cinder | 09:39 | |
| *** haomaiwang has joined #openstack-cinder | 09:42 | |
| *** zhangjn has quit IRC | 09:44 | |
| *** jistr has joined #openstack-cinder | 09:44 | |
| *** jordanP has joined #openstack-cinder | 09:49 | |
| *** EinstCrazy has quit IRC | 09:49 | |
| openstackgerrit | Tobias Urdin proposed openstack/cinder: Correct small misses in debug output for cinder-volume-usage-audit. Fixes variables not being included in output. Changed according to comments. https://review.openstack.org/247344 | 09:53 |
| flip214 | help, please. with the LVM driver the "os-vol-host-attr:host" volume attribute includes the host, service, and backend; with my own driver only the host is listed. | 09:53 |
| flip214 | I believe this is the reason for initialize_connection() (etc.) failing. | 09:54 |
| flip214 | what is my driver missing that the LVM driver does, so that the "os-vol-host-attr:host" attribute gets filled in correctly? | 09:54 |
| flip214 | The "class Volume_host_attribute" in cinder/api/contrib/volume_host_attribute.py should be used for all backends, as I understand it. | 09:55 |
| *** zhenguo has quit IRC | 10:01 | |
| *** lpetrut has joined #openstack-cinder | 10:01 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 10:04 |
| *** aix has joined #openstack-cinder | 10:07 | |
| *** haomaiwang has quit IRC | 10:10 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Add metadata aliases to Volume object https://review.openstack.org/247763 | 10:12 |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Recognize extra fields in CinderObjectDictCompat https://review.openstack.org/247762 | 10:12 |
| *** lpetrut1 has joined #openstack-cinder | 10:13 | |
| *** lpetrut has quit IRC | 10:13 | |
| *** lpetrut1 is now known as lpetrut | 10:13 | |
| *** haomaiwang has joined #openstack-cinder | 10:16 | |
| *** gcb has quit IRC | 10:28 | |
| *** haomaiwang has quit IRC | 10:28 | |
| *** haomaiwang has joined #openstack-cinder | 10:28 | |
| *** gcb has joined #openstack-cinder | 10:28 | |
| *** markvoelker has joined #openstack-cinder | 10:31 | |
| *** markvoelker has quit IRC | 10:36 | |
| *** Rekha has joined #openstack-cinder | 10:37 | |
| Rekha | hi all | 10:37 |
| Rekha | when volume creation from image is getting failed. when i checked in cinder-volume.log i could find this "TRACE cinder.volume.manager ImageCopyFailure: Failed to copy image to volume: Unable to find a Fibre Channel volume device." | 10:40 |
| Rekha | can anybody help me out in resolving the issue... | 10:41 |
| *** Dhaval has quit IRC | 10:41 | |
| Rekha | can anybody point out the reason for failure? | 10:41 |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Adds Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 10:42 |
| *** e0ne has quit IRC | 10:43 | |
| *** ankit_ag has quit IRC | 10:46 | |
| *** rushiagr is now known as rushiagr_away | 10:46 | |
| *** ankit_ag has joined #openstack-cinder | 10:47 | |
| *** lprice1 has joined #openstack-cinder | 10:49 | |
| *** lprice has quit IRC | 10:49 | |
| *** e0ne has joined #openstack-cinder | 10:51 | |
| *** EinstCrazy has joined #openstack-cinder | 10:54 | |
| *** lpetrut has quit IRC | 10:58 | |
| *** lpetrut has joined #openstack-cinder | 10:59 | |
| openstackgerrit | Merged openstack/cinder: Imported Translations from Zanata https://review.openstack.org/247943 | 11:00 |
| *** egonzalez has quit IRC | 11:00 | |
| openstackgerrit | Merged openstack/cinder: Manage existing: fix volume object saving https://review.openstack.org/247769 | 11:03 |
| *** houming has quit IRC | 11:05 | |
| *** haomaiwang has quit IRC | 11:08 | |
| nikeshm | Rekha: what type of setup are yu using? | 11:16 |
| nikeshm | openstack or devstack | 11:16 |
| nikeshm | what volume driver are you using | 11:17 |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: Robustify writing iscsi target persistence file https://review.openstack.org/247886 | 11:20 |
| *** rushiagr_away is now known as rushiagr | 11:23 | |
| *** dims has joined #openstack-cinder | 11:24 | |
| *** salv-orlando has quit IRC | 11:28 | |
| *** salv-orlando has joined #openstack-cinder | 11:29 | |
| *** rushiagr is now known as rushiagr_away | 11:30 | |
| *** ociuhandu has quit IRC | 11:34 | |
| *** yusuke has quit IRC | 11:34 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Eager load columns in volume_get_active_by_window https://review.openstack.org/248035 | 11:41 |
| *** salv-orlando has quit IRC | 11:45 | |
| *** markus_z is now known as markus_z_meeting | 11:58 | |
| *** bluex has quit IRC | 11:58 | |
| *** houming has joined #openstack-cinder | 11:58 | |
| *** bluex has joined #openstack-cinder | 12:00 | |
| *** zhangjn has joined #openstack-cinder | 12:02 | |
| *** zhangjn has quit IRC | 12:02 | |
| *** zhangjn has joined #openstack-cinder | 12:03 | |
| *** haomaiwa_ has joined #openstack-cinder | 12:05 | |
| Rekha | @nikeshm openstack setup and hp3parfcdriver | 12:12 |
| *** _amrith_ is now known as amrith | 12:14 | |
| *** salv-orlando has joined #openstack-cinder | 12:17 | |
| *** e0ne has quit IRC | 12:19 | |
| *** e0ne has joined #openstack-cinder | 12:19 | |
| *** houming has quit IRC | 12:20 | |
| *** houming has joined #openstack-cinder | 12:20 | |
| nikeshm | Rekha : then i think some packages might be missing | 12:23 |
| nikeshm | regarding FC | 12:24 |
| nikeshm | on the node | 12:24 |
| nikeshm | better check hp3parfcdriver guide | 12:24 |
| *** houming has left #openstack-cinder | 12:25 | |
| Rekha | @nikeshm on wic node you are talking about...you men on storage node | 12:25 |
| *** delatte has quit IRC | 12:25 | |
| Rekha | @nikeshm *you mean on storage node | 12:25 |
| nikeshm | Rekha: ya i think so | 12:28 |
| nikeshm | or may be on controller node | 12:29 |
| *** alejandrito has joined #openstack-cinder | 12:30 | |
| nikeshm | Rekha : not sure better wait for hp guys hemna DuncanT kmartin | 12:30 |
| Rekha | @nikeshm thank you | 12:31 |
| nikeshm | Rekha: are you able to create FC volumes and attach FC volumes | 12:31 |
| *** markvoelker has joined #openstack-cinder | 12:32 | |
| Rekha | i able to create empty fc volumes but not able to attach this created volume to instance | 12:33 |
| nikeshm | Rekha: thats the issue | 12:33 |
| Rekha | @nikeshm but on same 3par storage array i able to create ISCSI volume from image and attach to instance also | 12:34 |
| *** houming has joined #openstack-cinder | 12:34 | |
| Rekha | @nikeshm how this issue can be resolved.. | 12:35 |
| nikeshm | Rekha: why the error are you getting when try to attach FC volumes | 12:36 |
| nikeshm | what | 12:36 |
| *** aix has quit IRC | 12:36 | |
| nikeshm | please check nova-compute logs | 12:37 |
| nikeshm | any Traceback | 12:37 |
| *** markvoelker has quit IRC | 12:37 | |
| *** raildo-afk is now known as raildo | 12:39 | |
| nikeshm | Rekha: please paste Traceback in compute logs while attaching FC volume in a pastebin and share to us | 12:40 |
| nikeshm | smcginnis: how is parris, be careful there | 12:41 |
| *** bluex has quit IRC | 12:45 | |
| *** delatte has joined #openstack-cinder | 12:46 | |
| Rekha | @nikeshm I am not getting any information in nova logs | 12:47 |
| nikeshm | Rekha: are you checking on compute node | 12:47 |
| nikeshm | i mean where nova-compute service is running | 12:48 |
| *** delatte has quit IRC | 12:49 | |
| *** cdelatte has joined #openstack-cinder | 12:49 | |
| Rekha | @nikeshm sorry i was checking on the controller node | 12:50 |
| *** salv-orl_ has joined #openstack-cinder | 12:54 | |
| *** jwcroppe has joined #openstack-cinder | 12:55 | |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Adds Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 12:56 |
| *** salv-orlando has quit IRC | 12:57 | |
| Rekha | @nikeshm http://paste.openstack.org/show/479583/ traceback is pasted here | 13:00 |
| *** jaypipes has joined #openstack-cinder | 13:00 | |
| *** delattec has joined #openstack-cinder | 13:02 | |
| nikeshm | Rekha: hmm. is any other? | 13:02 |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Adds Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 13:02 |
| nikeshm | Tracebacks | 13:03 |
| nikeshm | presents | 13:03 |
| *** cdelatte has quit IRC | 13:05 | |
| Rekha | @nikeshm http://paste.openstack.org/show/479584/ | 13:06 |
| *** houming has quit IRC | 13:07 | |
| *** salv-orl_ has quit IRC | 13:09 | |
| *** salv-orlando has joined #openstack-cinder | 13:09 | |
| dulek | Anyone aware on the status of HPE CI? | 13:12 |
| *** gouthamr has joined #openstack-cinder | 13:13 | |
| *** setmason has joined #openstack-cinder | 13:15 | |
| *** salv-orlando has quit IRC | 13:16 | |
| nikeshm | Rekha: hmm better wait for hemna kmartin DuncanT | 13:16 |
| *** salv-orlando has joined #openstack-cinder | 13:16 | |
| *** mkoderer has quit IRC | 13:18 | |
| *** shausy has quit IRC | 13:21 | |
| *** mkoderer has joined #openstack-cinder | 13:22 | |
| *** jkraj has quit IRC | 13:24 | |
| Rekha | @nikeshm okay | 13:25 |
| *** jkraj has joined #openstack-cinder | 13:25 | |
| *** jkraj has quit IRC | 13:26 | |
| *** aix has joined #openstack-cinder | 13:30 | |
| *** dave-mccowan has joined #openstack-cinder | 13:30 | |
| *** eikke has joined #openstack-cinder | 13:33 | |
| eikke | what's the procedure to remove things from rootwrap filters? As in, how can you be sure not to remove things others depend on? | 13:34 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Add atomic conditional updates to objects https://review.openstack.org/205834 | 13:34 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move get_by_id to CinderObject https://review.openstack.org/218012 | 13:34 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Improve metadata update operations https://review.openstack.org/216376 | 13:34 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Remove API races from attach and detach methods https://review.openstack.org/216377 | 13:34 |
| *** markvoelker has joined #openstack-cinder | 13:35 | |
| *** akerr has joined #openstack-cinder | 13:35 | |
| *** ociuhandu has joined #openstack-cinder | 13:38 | |
| *** sborkows has joined #openstack-cinder | 13:40 | |
| *** bswartz has quit IRC | 13:41 | |
| *** amrith is now known as _amrith_ | 13:43 | |
| *** porrua has joined #openstack-cinder | 13:46 | |
| *** sgotliv has quit IRC | 13:47 | |
| *** ankit_ag has quit IRC | 13:50 | |
| *** dustins has joined #openstack-cinder | 13:52 | |
| *** salv-orlando has quit IRC | 13:52 | |
| *** salv-orl_ has joined #openstack-cinder | 13:52 | |
| *** jaypipes has quit IRC | 13:59 | |
| *** [1]Thelo has joined #openstack-cinder | 13:59 | |
| *** Thelo has quit IRC | 14:02 | |
| *** [1]Thelo is now known as Thelo | 14:02 | |
| *** smoriya_ has quit IRC | 14:08 | |
| *** stevemar_ has joined #openstack-cinder | 14:13 | |
| *** bswartz has joined #openstack-cinder | 14:15 | |
| *** eharney has joined #openstack-cinder | 14:15 | |
| *** mriedem has joined #openstack-cinder | 14:16 | |
| *** salv-orl_ has quit IRC | 14:17 | |
| lyarwood | hemna: looks like our `DIE DIE DIE` spam in https://review.openstack.org/#/c/238838/ finally worked :) | 14:17 |
| gouthamr | lyarwood: for a random lurker, what is DIE DIE DIE all about? | 14:20 |
| *** salv-orlando has joined #openstack-cinder | 14:21 | |
| *** jerrygb has joined #openstack-cinder | 14:21 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Add check_uptodate.sh --checkopts to "pep8" https://review.openstack.org/247796 | 14:25 |
| jgriffith | gouthamr: apparantly it's top secret | 14:27 |
| jgriffith | ;) | 14:27 |
| jgriffith | gouthamr: honestly I'm not quite sure either suspect it's CI recheck related but unclear | 14:28 |
| gouthamr | jgriffith: hah. wanted to join the spamming... :P | 14:30 |
| jgriffith | gouthamr: :) | 14:30 |
| *** merooney has joined #openstack-cinder | 14:31 | |
| scottda | gouthamr: People are always asking each other in this channel "How do I initiate a recheck on your CI system?" and someone mentioned that it would be great if you could do this by saying "DIE DIE DIE". It was a joke. | 14:31 |
| scottda | But there is probably some value in having a single command for this, to avoid having to ask each vendor the syntax. | 14:32 |
| jgriffith | scottda: oh, really? | 14:32 |
| scottda | "DIE DIE DIE" is a bit violent for my taste. | 14:32 |
| jgriffith | scottda: umm... what happened to "recheck <ci-name>" | 14:32 |
| scottda | jgriffith: I don't know. It just came up one day and I think it was patrickeast how changed his CI to allow the DIEDIEDIE to work. | 14:33 |
| jgriffith | scottda: and so that means everybody that listens for DDD does a recheck needed or not? | 14:33 |
| jgriffith | huh | 14:33 |
| jgriffith | scottda: well alrighty then :) | 14:33 |
| jgriffith | scottda: thanks for the explanation :) | 14:33 |
| scottda | no, it's still just a joke. | 14:33 |
| gouthamr | jgriffith: too mainstream. We don't want mainstream. We're the new kids on the block. 8) | 14:34 |
| jgriffith | ha | 14:34 |
| scottda | Personally, I think you storage vendors are all crazy. | 14:35 |
| *** akshai has joined #openstack-cinder | 14:35 | |
| *** jgregor has joined #openstack-cinder | 14:35 | |
| gouthamr | scottda: :D okay, I take that as a compliment. :) | 14:36 |
| scottda | :) | 14:36 |
| scottda | geguileo: dulek Hi. Is there an etherpad that gathers all the info, specs, and patches for Active-Active HA? | 14:37 |
| jgriffith | scottda: LOL | 14:37 |
| *** dansmith is now known as superdan | 14:41 | |
| *** baumann has joined #openstack-cinder | 14:41 | |
| *** lprice has joined #openstack-cinder | 14:46 | |
| *** xyang1 has joined #openstack-cinder | 14:46 | |
| *** aix has quit IRC | 14:47 | |
| *** lprice1 has quit IRC | 14:47 | |
| *** aix has joined #openstack-cinder | 14:49 | |
| lyarwood | gouthamr: haha sorry went AFK, I honestly didn't know the background for `DIE DIE DIE`, I just saw hemna using it on my change and asked :) | 14:51 |
| dulek | scottda: Not the etherpad, but you may find related patches here: https://review.openstack.org/#/q/topic:bp/cinder-volume-active-active-support,n,z | 14:51 |
| dulek | scottda, hemna: Speaking of CIs - what's up with HPE one? | 14:52 |
| *** xek has quit IRC | 14:55 | |
| *** nkrinner has quit IRC | 14:55 | |
| *** sgundur has joined #openstack-cinder | 14:56 | |
| scottda | dulek: Thanks for the pointer to patches. | 14:56 |
| scottda | dulek: I don't know about HPE CI. I'm just a Helion distribution guy. HPE Storage is another world..... | 14:57 |
| dulek | scottda: Ah, sorry for mistaking that then. | 14:58 |
| scottda | dulek: I'm wondering if there's a way to help move these HA/AA specs and patches along better? I think it's hard for reviewers to figure out what to look at just from that list of patches. | 14:58 |
| *** markus_z_meeting is now known as markus_z | 14:59 | |
| dulek | There's a main one here: https://review.openstack.org/#/c/232599/ It explains all the subsequent specs. | 14:59 |
| dulek | Maybe an etherpad would work, I don't know. I always felt that etherpads tend to get updated. | 15:00 |
| dulek | s/updated/outdated | 15:00 |
| scottda | Yeah, that kinda works. But still doesn't give any ordering to help reviewers. It can kind of be figured out with dependencies. | 15:00 |
| scottda | I'm just thinking that at this rate, we won't have HA/AA in Mitaka. Spec approval is pretty slow, and then there are the patches... | 15:00 |
| *** boris-42 has joined #openstack-cinder | 15:01 | |
| scottda | That's 8 specs on that list ^^^ ! | 15:02 |
| *** kfarr has joined #openstack-cinder | 15:02 | |
| *** breitz has joined #openstack-cinder | 15:03 | |
| dulek | I'm a little behind on geguileo's updates in specs. I'll try to sync up and maybe come up with something. | 15:03 |
| asselin | dulek, hpe ci seems to be failing lefthand-iscsi-driver-master-client-pip-vsa125http-dsvm, otherwise looks ok to me | 15:03 |
| scottda | Yeah, I'm not sure of the answer. I've teammates working on testing the patches, and they were pretty confused about the dependency chain. | 15:04 |
| *** _amrith_ is now known as amrith | 15:05 | |
| dulek | scottda: It's more like specs reviews work now, not much implementation. I was focusing on upgrades for M-1, I'll try to work more on A/A next week. | 15:06 |
| gouthamr | lyarwood: Nice. Thanks :) | 15:07 |
| scottda | cool. I'm mostly out next week, but I'll attend the Cinder meeting Wed. Maybe one of us can add info to the spec priority page and nag people to review: https://etherpad.openstack.org/p/mitaka-cinder-spec-review-tracking | 15:08 |
| *** jasondotstar is now known as jasondotstar_afk | 15:08 | |
| *** stevemar_ has quit IRC | 15:10 | |
| *** e0ne has quit IRC | 15:11 | |
| *** haomaiwa_ has quit IRC | 15:12 | |
| *** krtaylor has joined #openstack-cinder | 15:16 | |
| *** lpabon has joined #openstack-cinder | 15:21 | |
| openstackgerrit | Edwin Wang proposed openstack/cinder: FlashSystem reports error in _find_host_exhaustive() https://review.openstack.org/247546 | 15:22 |
| *** jerrygb has quit IRC | 15:22 | |
| *** jerrygb has joined #openstack-cinder | 15:23 | |
| *** kfarr has quit IRC | 15:23 | |
| *** setmason has quit IRC | 15:24 | |
| *** vgridnev has quit IRC | 15:26 | |
| *** kfarr has joined #openstack-cinder | 15:26 | |
| *** crose has joined #openstack-cinder | 15:30 | |
| *** mc_nair has joined #openstack-cinder | 15:32 | |
| patrickeast | jgriffith: haha yea so that die die die thing was a joke, but kind a of symptom of folks confused by how to recheck since not all the ci's respond to recheck <ci name> | 15:32 |
| jgriffith | patrickeast: got it :) | 15:32 |
| patrickeast | Plus lots, including Jenkins get triggered by that format | 15:33 |
| jgriffith | patrickeast: a number of folks seem to have been having fun putting it in comments | 15:33 |
| *** alexschm has quit IRC | 15:33 | |
| *** e0ne has joined #openstack-cinder | 15:33 | |
| patrickeast | Lol yea I added it as an extra way to do it, zuul makes it trivial to add more trigger regexs | 15:33 |
| jgriffith | :) | 15:34 |
| jgriffith | I'll just stick with "recheck solidfire" for now | 15:34 |
| patrickeast | Seems to be a hit though... Gets more action than the real recheck syntax now | 15:34 |
| jgriffith | :) | 15:34 |
| *** rcernin has quit IRC | 15:34 | |
| jgriffith | patrickeast: yeah, it's QUITE popular with all the kids these days | 15:34 |
| jgriffith | :) | 15:34 |
| *** thangp has joined #openstack-cinder | 15:35 | |
| tbarron | so I dunno but I've been told that 'recheck <backend>-ci' can trigger more than just the CI for that backend. True? | 15:36 |
| tbarron | patrickeast, jgriffith ^^^: I think that's why we do 'run netapp-ci' | 15:37 |
| jgriffith | tbarron: apparantly so can "DDD" :0 | 15:37 |
| tbarron | jgriffith: yeah :-) | 15:37 |
| jgriffith | tbarron: actually it's "recheck-<ciName>" so Jenkins doesn't catch it | 15:37 |
| jgriffith | tbarron: yes, you're correrct, if there's whitespace after the "recheck" it will grab it | 15:38 |
| jgriffith | tbarron: anywho... DDD seems popular and seems to work so whatevs :) | 15:38 |
| tbarron | jgriffith: kk, I guess we did 'run ...' just to reduce the probability of people putting a space in there. | 15:38 |
| jgriffith | tbarron: yeah, that was another bike-shedding argument one day a while back | 15:39 |
| jgriffith | tbarron: I'd prefer not to re-live :) | 15:39 |
| tbarron | jgriffith: 'nuf said | 15:40 |
| jgriffith | tbarron: :) | 15:40 |
| *** rhagarty_ has quit IRC | 15:40 | |
| *** ntpttr has joined #openstack-cinder | 15:40 | |
| *** rhagarty_ has joined #openstack-cinder | 15:40 | |
| *** wN has quit IRC | 15:44 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Tooz locks to support A/A HA spec https://review.openstack.org/202615 | 15:46 |
| *** davechen has joined #openstack-cinder | 15:48 | |
| geguileo | scottda: dulek In theory this should have them | 15:49 |
| geguileo | But I'll check now that it is up to date and maybe structure it a little bit better | 15:49 |
| *** wN has joined #openstack-cinder | 15:50 | |
| *** wN has joined #openstack-cinder | 15:50 | |
| *** stevemar_ has joined #openstack-cinder | 15:52 | |
| scottda | geguileo: Which should have them? | 15:53 |
| geguileo | Uuups, https://blueprints.launchpad.net/cinder/+spec/cinder-volume-active-active-support | 15:54 |
| geguileo | Updating it now | 15:54 |
| scottda | ok, cool. | 15:54 |
| geguileo | But I'm updating it now | 15:54 |
| *** e0ne_ has joined #openstack-cinder | 15:57 | |
| *** e0ne has quit IRC | 15:58 | |
| *** svasheka has quit IRC | 15:58 | |
| *** stevemar_ has quit IRC | 15:58 | |
| *** stevemar_ has joined #openstack-cinder | 15:59 | |
| *** rcernin has joined #openstack-cinder | 16:00 | |
| *** rcernin is now known as rcernin|dinner | 16:00 | |
| *** sborkows has quit IRC | 16:03 | |
| *** svasheka has joined #openstack-cinder | 16:05 | |
| *** setmason has joined #openstack-cinder | 16:08 | |
| kmartin | nikeshm, are you still have issues? | 16:12 |
| *** daneyon_ has quit IRC | 16:13 | |
| *** daneyon has joined #openstack-cinder | 16:13 | |
| kmartin | nikeshm, could be one of a few problems, either you don't have the required FC packages, http://docs.openstack.org/liberty/config-reference/content/fibrechannel.html on your compute node or your SAN fabric is not zoned correctly. Are you using the fibre channel zone manager? | 16:17 |
| *** stevemar_ has quit IRC | 16:26 | |
| *** baumann has quit IRC | 16:29 | |
| *** jdurgin1 has joined #openstack-cinder | 16:31 | |
| *** eharney has quit IRC | 16:33 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient https://review.openstack.org/248163 | 16:33 |
| *** stevemar_ has joined #openstack-cinder | 16:39 | |
| *** rcernin|dinner is now known as rcernin | 16:42 | |
| *** amrith is now known as _amrith_ | 16:42 | |
| *** wilson has joined #openstack-cinder | 16:42 | |
| *** wilson is now known as Guest3211 | 16:43 | |
| *** stevemar_ has quit IRC | 16:44 | |
| *** dims_ has joined #openstack-cinder | 16:44 | |
| *** dims has quit IRC | 16:44 | |
| *** stevemar_ has joined #openstack-cinder | 16:45 | |
| *** wilson-liu has quit IRC | 16:45 | |
| dulek | e0ne_: Coverage for cinder/image/image_utils.py : 94% | 16:47 |
| e0ne_ | dulek: hi. not bad:) | 16:47 |
| dulek | e0ne_: That's why I haven't noticed, but yes, I was curious. ;) | 16:47 |
| e0ne_ | dulek: thanks for review! I'll update patch with lost tests | 16:48 |
| *** kfarr has quit IRC | 16:55 | |
| *** e0ne_ has quit IRC | 16:58 | |
| *** stevemar_ has quit IRC | 16:59 | |
| *** stevemar_ has joined #openstack-cinder | 17:00 | |
| *** chhavi has quit IRC | 17:00 | |
| dulek | xyang1: Hi! You're rechecking EMC VNX because it failed on volume_metadata aliases patch, or just to trigger it? | 17:00 |
| xyang1 | dulek: I was just triggering it to test your patch | 17:00 |
| xyang1 | dulek: but the other patch is already merged so this won't really test your change | 17:01 |
| dulek | xyang1: Yes, that's why I've asked for testing. But don't bother, through the weekend I'll post a DO-NOT-MERGE change dependent on mine that's removing Peter's fix and trigger VNX CI there. This should make me sure on my patch. :) | 17:03 |
| *** eharney has joined #openstack-cinder | 17:03 | |
| xyang1 | dulek: sure, that is way to do it:) | 17:03 |
| *** stevemar_ has quit IRC | 17:04 | |
| xyang1 | dulek: I have asked Peter to test your patch but he is in. different time zone | 17:04 |
| *** leeantho has joined #openstack-cinder | 17:07 | |
| *** sghanekar_ has joined #openstack-cinder | 17:09 | |
| *** jordanP has quit IRC | 17:09 | |
| *** jordanP has joined #openstack-cinder | 17:10 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Fix debug output for cinder-volume-usage-audit https://review.openstack.org/247344 | 17:13 |
| *** baumann has joined #openstack-cinder | 17:17 | |
| openstackgerrit | Falk Reimann proposed openstack/cinder: Add Auth Version 3 support in Swift Backup Driver https://review.openstack.org/247977 | 17:20 |
| openstackgerrit | Fergal Mc Carthy proposed openstack/cinder: Refactor SRB Unittest LVM command handlers. https://review.openstack.org/248187 | 17:21 |
| *** wilson has joined #openstack-cinder | 17:26 | |
| *** wilson is now known as Guest49370 | 17:26 | |
| *** rhedlind has joined #openstack-cinder | 17:27 | |
| *** baumann has quit IRC | 17:28 | |
| *** wilson-l has joined #openstack-cinder | 17:29 | |
| *** Guest3211 has quit IRC | 17:29 | |
| *** jdurgin1 has quit IRC | 17:29 | |
| openstackgerrit | Nate Potter proposed openstack/cinder: Move quota checks from manager to API https://review.openstack.org/243673 | 17:32 |
| *** Guest49370 has quit IRC | 17:32 | |
| *** edtubill has joined #openstack-cinder | 17:32 | |
| openstackgerrit | Ryan McNair proposed openstack/cinder: Deprecate *_multipath_enabled flag for IBM drivers https://review.openstack.org/244749 | 17:37 |
| *** zhangjn has quit IRC | 17:51 | |
| *** _amrith_ is now known as amrith | 17:52 | |
| *** crose has quit IRC | 17:54 | |
| *** ericksonsantos has quit IRC | 17:58 | |
| *** gouthamr has quit IRC | 18:00 | |
| *** jistr has quit IRC | 18:01 | |
| smcginnis | dulek, scottda: This would be a great place to start capturing more HA links and info: https://etherpad.openstack.org/p/mitaka-cinder-spec-review-tracking | 18:03 |
| smcginnis | Maybe not everything, but at basis for where to start. | 18:03 |
| *** jordanP has quit IRC | 18:03 | |
| hemna | smcginnis, not sure anyone in Cinder is aware of this change in Nova, but FYI: https://review.openstack.org/#/c/135854 | 18:03 |
| hemna | Looks like Nova is trying to move the iSCSI volume support into qemu | 18:04 |
| hemna | which doesn't support multipath at all | 18:04 |
| hemna | :( | 18:04 |
| smcginnis | Huh, that's an interesting approach. :[ | 18:04 |
| *** dims has joined #openstack-cinder | 18:05 | |
| smcginnis | And it's not even supported by the qemu version in ubuntu. | 18:05 |
| hemna | yah | 18:05 |
| hemna | bah | 18:05 |
| hemna | sure would have been nice to have someone from Cinder review this before it merged | 18:06 |
| hemna | :( | 18:06 |
| *** dims_ has quit IRC | 18:06 | |
| *** jordanP has joined #openstack-cinder | 18:06 | |
| smcginnis | Yeah, kind of surprised we're just finding out about it. | 18:06 |
| *** apoorvad has joined #openstack-cinder | 18:06 | |
| hemna | back...in Feb. | 18:06 |
| hemna | d'oh | 18:06 |
| *** jordanP has quit IRC | 18:06 | |
| smcginnis | Where's our nova liason!! :) | 18:06 |
| hemna | dangit | 18:06 |
| smcginnis | But that was quite a while ago. | 18:06 |
| hemna | yah | 18:06 |
| hemna | smcginnis, did you see the mailing list post about the encryptors patch in os-brick ? | 18:07 |
| smcginnis | hemna: Yep, I had asked (well, Matt R did first) to have something posted there. | 18:07 |
| hemna | Daniel seems to want to move Nova into doing all attachments directly w/ qemu | 18:07 |
| smcginnis | hemna: We had discussed in reviews here, but nova should know what we're doing. | 18:07 |
| * smcginnis thinks and vice versa :/ | 18:08 | |
| hemna | I'm not sure qemu can handle all of the h/w initiators that os-brick supports | 18:08 |
| hemna | and there is no FC support | 18:08 |
| hemna | and there is no multipath support | 18:08 |
| hemna | :( | 18:08 |
| smcginnis | hemna: I don't get it. What is the advantage of using qemu? Seems severely limited with no big win from what I see. | 18:08 |
| hemna | https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/qemu-built-in-iscsi-initiator.html | 18:09 |
| hemna | I also found that | 18:09 |
| hemna | kinda hints at some of their 'issues' | 18:09 |
| hemna | but I'm not sure I'm buying it | 18:09 |
| *** aix has quit IRC | 18:10 | |
| smcginnis | Oh! So the volume isn't attached to the nova host. Directly to the guest VM? | 18:10 |
| smcginnis | I'm still confused. | 18:10 |
| tbarron | unless I'm missing something, there are backends that won't work right without multipath support ... | 18:10 |
| smcginnis | tbarron: It definitely could cause problems. | 18:11 |
| *** e0ne has joined #openstack-cinder | 18:11 | |
| hemna | well it's attached to qemu | 18:11 |
| hemna | they create a virsh/qemu/kvm pool | 18:11 |
| smcginnis | Even with one initiator, on ours we can have multiple front end ports configured that would result in multiple paths. | 18:11 |
| hemna | and hand that to the guest. | 18:11 |
| tbarron | we have backends that use multipath for controller failover. And another backend that load-balances over multiple paths. | 18:11 |
| hemna | this goes against the idea of multipath entirely | 18:11 |
| smcginnis | hemna: So to get to qemu, isn't that exposing it to the host? You would still hit a LUN limit. | 18:12 |
| tbarron | image copies etc. have problems unless the 'use_multipath' options are configured in nova libvirt. | 18:12 |
| hemna | yah I think so | 18:12 |
| smcginnis | Well... maybe the problem will fix itself because anyone who tries to use this will have problems. | 18:13 |
| smcginnis | So they won't use it and it will go away. | 18:13 |
| smcginnis | :] | 18:13 |
| hemna | lol | 18:13 |
| tbarron | yeah, it's a nova.conf option, not default. | 18:13 |
| *** mriedem has quit IRC | 18:13 | |
| hemna | qemu has no concept of FC | 18:14 |
| tbarron | and maybe it has its uses | 18:14 |
| smcginnis | tbarron: Well, that's good news at least that it's not default. That could really screw up some deployments. | 18:14 |
| smcginnis | tbarron: Maybe. Limited though. | 18:14 |
| smcginnis | I still don't see the benefit. | 18:14 |
| hemna | yah I'm grasping at it myself | 18:14 |
| hemna | you lose all h/w initiator support | 18:15 |
| *** harlowja_at_home has joined #openstack-cinder | 18:15 | |
| hemna | and force qemu to do all the iSCSI work, which is less than ideal. | 18:15 |
| smcginnis | mriedem isn't online. I wonder if he knows anything about this. He's pretty good on the storage side of nova. | 18:15 |
| hemna | smcginnis, I don't see daniel either. berrange ? | 18:16 |
| smcginnis | hemna: Right, why use a well established and featureful iSCSI initiator. Throw it all on qemu. | 18:16 |
| smcginnis | scottda: Maybe something you can bring up with nova, Mr. Liason? :) | 18:16 |
| hemna | oh maybe it's danpb ? | 18:17 |
| xyang1 | the doc says volume_drivers = iscsi=nova.virt.libvirt.volume.LibvirtNetVolumeDriver | 18:17 |
| hemna | xyang1, yah, and they modified the netvolumedriver to support iscsi | 18:17 |
| hemna | but only for single path | 18:17 |
| xyang1 | :( | 18:17 |
| hemna | and cinder drivers are going to get the initiator information from the host | 18:17 |
| hemna | which has it's iqn and multipath=True/False | 18:18 |
| *** mriedem has joined #openstack-cinder | 18:18 | |
| hemna | so some backends are going to export as if it's multipath enabled | 18:18 |
| hemna | but only get 1 path connected. | 18:18 |
| hemna | that's bad mmmkay | 18:18 |
| smcginnis | Problem Description: The VMs on a compute node will be stopped when an iSCSI volume is attached or detached to a VM and it causes the host OS crash. | 18:19 |
| smcginnis | There are some fundamental misconceptions here, I think. | 18:19 |
| hemna | yah I saw that too and thought...what?! | 18:20 |
| hemna | that simply doesn't happen | 18:20 |
| smcginnis | That whole list under Problem Description has me confused. | 18:20 |
| xyang1 | wonder what backend they tested with to come up with that problem description | 18:21 |
| hemna | lvm ? | 18:21 |
| smcginnis | mriedem: You're here now! | 18:21 |
| smcginnis | mriedem: Don't suppose you've seen any of the scrollback? | 18:21 |
| mriedem | nope | 18:21 |
| mriedem | been at lunch for the last hour | 18:21 |
| smcginnis | mriedem: We're a bit puzzled by https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/qemu-built-in-iscsi-initiator.html | 18:22 |
| smcginnis | mriedem: Wondering if you have any innsight on it. | 18:22 |
| smcginnis | Was about to ask on -nova, but since you're back from lunch... :) | 18:22 |
| *** e0ne has quit IRC | 18:22 | |
| hemna | ooh | 18:23 |
| hemna | the other downside is that extending an attached volume will never work in that scenario | 18:23 |
| mriedem | smcginnis: puzzled how? | 18:23 |
| smcginnis | mriedem: Main points being, it seems to have some fundamental issues and unclear advantage as to why. | 18:23 |
| mriedem | i remember pointing out long after it was in tree that the dependency story sucks | 18:23 |
| smcginnis | mriedem: Everything under Problem Description doesn't appear to be accurate. | 18:23 |
| mriedem | you have to build your own qemu | 18:24 |
| smcginnis | Ya, there's that too. | 18:24 |
| hemna | mriedem, so what brings this up is the whole ML discussion about the encryptors support | 18:24 |
| smcginnis | Or hey, just rewrite qemu and compile it into your kernel while you're at it. | 18:24 |
| smcginnis | Sorry. :) | 18:24 |
| *** markus_z has quit IRC | 18:24 | |
| hemna | it looks like Daniel wants to move all volume attaches to this mechanism eventually | 18:24 |
| hemna | there are some serious downsides to doing this | 18:25 |
| *** bardia has joined #openstack-cinder | 18:25 | |
| hemna | 1) no h/w initiator support. 2) No multipath support 3) no extending attached volumes support 4) no fibre channel support 5) no vendor based transports support | 18:25 |
| smcginnis | Looks like different discussion going on in -nova at the moment. | 18:26 |
| *** edtubill has quit IRC | 18:27 | |
| hemna | I wonder if multiattach would even work as wel | 18:27 |
| mriedem | ok i'm not up to date on the ML thread yet either | 18:27 |
| smcginnis | hemna: Doesn't look like it. | 18:27 |
| hemna | mriedem, what you don't read the ML at 3am? | 18:27 |
| hemna | :P | 18:27 |
| mriedem | what was wrong with the spec again? | 18:27 |
| mriedem | i do, just haven't gotten to that one yet | 18:27 |
| mriedem | lots of plates spinning | 18:27 |
| smcginnis | mriedem: I think hemna summed it up well a few lines back. | 18:28 |
| mriedem | i'm all for ripping crappy stuff out of nova | 18:28 |
| mriedem | btw | 18:28 |
| smcginnis | mriedem: There are definite "shortcomings" with this approach. | 18:28 |
| mriedem | which is "1) no h/w initiator support. 2) No multipath support 3) no extending attached volumes support 4) no fibre channel support 5) no vendor based transports support" | 18:28 |
| smcginnis | mriedem: And questionable motivation for needing it. | 18:28 |
| mriedem | ? | 18:28 |
| smcginnis | mriedem: Yeah, those are some of the issues with it. | 18:28 |
| mriedem | i seem to remember the reason for that spec was to just keep some gorp out of the host to make debugging easier | 18:29 |
| hemna | https://github.com/openstack/nova/blob/master/nova/virt/libvirt/volume/net.py#L56 | 18:29 |
| hemna | so it looks like that code made it into the split out libvirt drivers | 18:29 |
| hemna | so it's still there. | 18:29 |
| mriedem | and possibly some performance thing, but i never reviewed that before it went in | 18:29 |
| hemna | fwiw | 18:29 |
| mriedem | it's in os-brick you mean? | 18:29 |
| hemna | no | 18:29 |
| hemna | the https://review.openstack.org/#/c/135854 | 18:29 |
| hemna | that guy landed back in Feb. | 18:30 |
| hemna | and it's still in the 'new' libvirt/volume/net.py code | 18:30 |
| mriedem | https://review.openstack.org/#/c/135854/ | 18:30 |
| mriedem | yeah, | 18:30 |
| hemna | so I guess there is 2 issues here really | 18:30 |
| hemna | the existing patch https://review.openstack.org/#/c/135854 | 18:30 |
| mriedem | it even says no multipath in the commit message | 18:30 |
| hemna | and 2) the direction that "Nova" wants to go in the future wrt volume attaches | 18:31 |
| mriedem | "3) no extending attached volumes support" wasn't a thing when this was added to nova right? | 18:31 |
| hemna | mriedem, correct | 18:31 |
| mriedem | (4) could be added for fibrechannel i'd think? | 18:31 |
| hemna | I bring it up now, because I'm working on os-brick code to add that ability | 18:31 |
| smcginnis | mriedem: Yeah, not really at the time. | 18:31 |
| hemna | mriedem, https://review.openstack.org/#/c/243730/ | 18:31 |
| smcginnis | mriedem: But it prevents adding support for that. And possibly other betterness in the future. | 18:32 |
| hemna | that's my WIP that I'm testing now to add the ability to resize attached | 18:32 |
| mriedem | fwiw, the if statements in https://review.openstack.org/#/c/135854/19/nova/virt/libvirt/volume.py make me said | 18:32 |
| mriedem | *sad | 18:32 |
| mriedem | if that helps | 18:32 |
| hemna | err https://review.openstack.org/#/c/243776/ | 18:32 |
| hemna | that one...not the spec :P | 18:32 |
| hemna | mriedem, yah I saw that as well. that seems like a painful way to go, long term | 18:33 |
| hemna | eventually everything will be a big if/then/else | 18:33 |
| mriedem | ok, so what i'm hearing is this very niche thing that you have to build your owen qemu for (on trusty at least), and with little gain (makes mgmt / debug easier for ops?), is blocking cinder from doing things it wants to do with volume resize | 18:34 |
| *** harlowja_at_home has quit IRC | 18:34 | |
| mriedem | plus it doesn't support multipath and fibre | 18:34 |
| hemna | correct | 18:34 |
| *** e0ne has joined #openstack-cinder | 18:34 | |
| smcginnis | mriedem: Yeah. And we otherwise generally thing it's a Bad Thing to do for end users. | 18:35 |
| hemna | no idea how you'd be able to debug failed attaches/detaches | 18:35 |
| smcginnis | s/thing/think/ | 18:35 |
| mriedem | bad thing b/c of not knowing how to debug attach/detach failures? | 18:35 |
| mriedem | or the multipath/fibre limitations? | 18:35 |
| mriedem | and/or | 18:36 |
| hemna | yes :) | 18:36 |
| smcginnis | D: All of the abobe | 18:36 |
| * smcginnis can't type tonight. Must be the wine. | 18:36 | |
| mriedem | tonight? | 18:36 |
| hemna | :P | 18:36 |
| mriedem | where are you? | 18:36 |
| smcginnis | mriedem: Paris :) | 18:37 |
| mriedem | oh | 18:37 |
| *** gouthamr has joined #openstack-cinder | 18:37 | |
| smcginnis | Doing code reviews on my anniversary. Good thing my wife is jetlagged. :) | 18:37 |
| mriedem | \o/ | 18:37 |
| mriedem | so you guys have raised this all in the ML and danpb said it's not a problem and you're crazy and i should just go read the ML? | 18:37 |
| openstackgerrit | Nate Potter proposed openstack/cinder: Move quota checks from manager to API https://review.openstack.org/243673 | 18:38 |
| smcginnis | mriedem: Was thinking that's probably our next step. | 18:38 |
| mriedem | oh, well if you haven't raised it in the ML yet, please do | 18:38 |
| smcginnis | mriedem: Was just curious if you had any extra insight, cuz we certainly didn't. | 18:38 |
| mriedem | i'm definitely not an expert on this specific thing | 18:38 |
| hemna | no I haven't made a post on the ML about it yet | 18:38 |
| mriedem | no, like i said, i stumbled on this spec long after it was merged, | 18:38 |
| hemna | just discovered it like 15 minutes ago | 18:38 |
| mriedem | and i was raising some hell about the 'bring your own qemu' limitatoin | 18:38 |
| smcginnis | mriedem: Yeah, thanks for you're input though. | 18:38 |
| smcginnis | mriedem: Just wasn't sure if there was someting we were missing. | 18:39 |
| mriedem | red hat probably doesn't have a problem with that part, i'm assuming OSP provides that | 18:39 |
| smcginnis | Good question. eharney, any idea if this all works on RH? | 18:39 |
| *** edmondsw has joined #openstack-cinder | 18:39 | |
| smcginnis | eharney: https://specs.openstack.org/openstack/nova-specs/specs/kilo/implemented/qemu-built-in-iscsi-initiator.html | 18:40 |
| smcginnis | eharney: If you weren't following along at home. :) | 18:40 |
| hemna | heh, I'll have to find out if it works on Helion. | 18:40 |
| eharney | umm | 18:40 |
| smcginnis | I guess if it doesn't work on anyone's distro, there's probably not much to worry about for now. :) | 18:40 |
| smcginnis | eharney: :) | 18:40 |
| eharney | i'm not really sure if that's something we support | 18:41 |
| hemna | I'm more concerned w/ the direction that danpb wants to move Nova. it seems like a major step backwards IMHO | 18:41 |
| smcginnis | The spec doesn't call out which version of qemu. | 18:41 |
| *** jbernard has quit IRC | 18:44 | |
| openstackgerrit | Merged openstack/cinder: fast8: Skip git rm'd files https://review.openstack.org/247793 | 18:45 |
| scottda | smcginnis: Sorry, I was not involved when that spec merged ... | 18:45 |
| smcginnis | scottda: Weak excuse. | 18:45 |
| scottda | I don't think there was a Nova liason at the time :) | 18:45 |
| smcginnis | Hah! :) | 18:45 |
| eharney | seems like it's not totally uncharted territory, qemu has used rbd and gluster that way for a long time | 18:46 |
| smcginnis | scottda: Yeah, didn't realize at first it was so long ago. | 18:46 |
| smcginnis | eharney: Oh! Didn't realize there was built in support for those. | 18:46 |
| *** stevemar_ has joined #openstack-cinder | 18:46 | |
| scottda | I agree with hemna . This particular feature is probably not being used much, but it's not a good direction to go. | 18:46 |
| hemna | eharney, yah they have, but moving all transports into qemu seems like a major mistake IMHO | 18:47 |
| smcginnis | eharney: Don't suppose you have any idea the advantage of qemu handling that vs the host OS? | 18:47 |
| eharney | for rbd/gluster it's usually about performance scalability, not really sure in the case of iSCSI | 18:48 |
| *** stevemar_ has quit IRC | 18:48 | |
| eharney | and, well, for rbd it's just the easiest way to implement it anyway | 18:48 |
| smcginnis | eharney: It's more performant having qemu handle it? | 18:48 |
| *** stevemar_ has joined #openstack-cinder | 18:48 | |
| eharney | smcginnis: that's what i hear, but i have no idea how that translates to this case | 18:49 |
| hemna | sounds like a potential for a bottleneck to me | 18:49 |
| smcginnis | eharney: Interesting data point at least. Thanks! | 18:49 |
| *** geguileo has quit IRC | 18:52 | |
| scottda | I think I'd rather debug iscsi attach errors at the OS level than at the libvirt level. | 18:53 |
| hemna | scottda, +10000 | 18:53 |
| mriedem | danpb did mention something about being able to tweak performance with qemu native stuff, | 18:53 |
| mriedem | in a spec asking for rbd encryption support in nova | 18:53 |
| mriedem | let me dig that up | 18:53 |
| mriedem | https://review.openstack.org/#/c/239798/3/specs/mitaka/approved/rbd-volume-encryption.rst | 18:54 |
| mriedem | basically dan is -1 on that since it uses dm-crypt (more dm-crypt usage) | 18:54 |
| mriedem | and he wants to wait for native qemu encryption support | 18:54 |
| mriedem | which is like N at the earliest | 18:54 |
| *** salv-orl_ has joined #openstack-cinder | 18:54 | |
| mriedem | if you guys have comments on that spec too, please provide input | 18:54 |
| hemna | man, I can only see the problems we'll have with this | 18:56 |
| hemna | cinder uses 1 mechanism to do attaches/encryption and nova uses another | 18:56 |
| openstackgerrit | Merged openstack/os-brick: Parse FCoE sysfs device paths https://review.openstack.org/238838 | 18:56 |
| smcginnis | What could go wrong? | 18:56 |
| *** e0ne has quit IRC | 18:57 | |
| *** salv-orlando has quit IRC | 18:57 | |
| hemna | so FWIW, I put a -2 on the os-brick encryptors patch for the time being. | 18:58 |
| hemna | https://review.openstack.org/#/c/247372/ | 18:58 |
| hemna | until we hash this out | 18:58 |
| hemna | danpb already -1'd it for the same reasons he posted in the ML. | 18:58 |
| hemna | ugh | 19:00 |
| hemna | smcginnis, https://review.openstack.org/#/c/247372/ | 19:00 |
| hemna | more discussion happening there. | 19:00 |
| hemna | we need to has this out. | 19:00 |
| hemna | sounds like, danpb thinks cinder shouldn't ever do encryption. | 19:01 |
| hemna | which precludes doing copy volume <--> image operations. | 19:01 |
| hemna | or backup | 19:01 |
| *** setmason has quit IRC | 19:02 | |
| scottda | Or Cinder attach without Nova | 19:02 |
| hemna | heh | 19:02 |
| hemna | and that | 19:02 |
| smcginnis | Hmm, securing against cinder being able to do any encryption/decryption. | 19:03 |
| smcginnis | Cuz we'd be the vulnerable point? | 19:03 |
| hemna | evidently, Cinder will compromise the storage, according to him. | 19:03 |
| *** setmason has joined #openstack-cinder | 19:03 | |
| * hemna has a sad | 19:03 | |
| smcginnis | If Cinder is compromised, I think there are much bigger issues. But that might just be me. | 19:04 |
| hemna | smcginnis, +1 | 19:04 |
| hemna | this isn't what I had planned for my Friday | 19:04 |
| hemna | heh | 19:04 |
| smcginnis | :( | 19:05 |
| smcginnis | FWIW, I think your last comment on there is exactly the point. | 19:05 |
| hemna | thanks. | 19:06 |
| hemna | DuncanT, ping | 19:06 |
| hemna | DuncanT, replied to the ML post, but wasn't specific to the direction of attaches, but more direct to the encryptors code. | 19:06 |
| *** lpetrut has quit IRC | 19:06 | |
| hemna | we need to craft a response about the direction and get our heads aligned with Nova on this. | 19:06 |
| hemna | as this is a vast departure from the current situation and has major issues IMHO. | 19:07 |
| scottda | Duncan likely won't be back to work until Sunday, FYI. It's his weekend. | 19:07 |
| hemna | scottda, ah yah good poing | 19:08 |
| hemna | point | 19:08 |
| *** Lee1092 has quit IRC | 19:09 | |
| *** timcl has quit IRC | 19:10 | |
| scottda | so, hemna is right about getting our heads aligned and crafting a response, but the logistics are tricky.... | 19:10 |
| scottda | With some people on weekend, some folks living it up in Paris, and Thanksgiving Holiday week coming up... | 19:11 |
| hemna | yah | 19:11 |
| *** jordanP has joined #openstack-cinder | 19:11 | |
| hemna | I'll be around Monday, Tuesday. | 19:11 |
| *** jordanP has quit IRC | 19:12 | |
| scottda | It seems that some things are clear: Cinder is not going to remove any existing encryption/decrypt support, as it is necessary. | 19:12 |
| smcginnis | Hah! I'll be around Tuesday for sure. | 19:12 |
| smcginnis | scottda: +1 | 19:12 |
| *** dustins has quit IRC | 19:12 | |
| hemna | yah, that assertion about Cinder not doing encryption is unrealistic. | 19:13 |
| *** dustins has joined #openstack-cinder | 19:13 | |
| *** stevemar_ has quit IRC | 19:13 | |
| scottda | So is the real question about what can be moved into Brick, and whether it should? | 19:13 |
| smcginnis | I can see the desire for it. In general it probably is a good thing to limit decrypt points as much as possible. | 19:13 |
| smcginnis | But that's just not the reality. | 19:13 |
| *** mc_nair has quit IRC | 19:13 | |
| hemna | the only way to do it, is to completely change how we do the copy volume <--> image operations entirely | 19:14 |
| hemna | and spin up worker VMs | 19:14 |
| hemna | that then fire up a cinder-data service inside the vm | 19:14 |
| hemna | and attach the volume to the qemu host. | 19:14 |
| hemna | that seems like a nightmare to me | 19:14 |
| *** jordanP has joined #openstack-cinder | 19:14 | |
| hemna | good luck supporting FC | 19:14 |
| scottda | There's still work in-flight to have the ability to attach/detach without nova | 19:14 |
| smcginnis | Oi. :( | 19:15 |
| smcginnis | scottda: Right, this would completely preclude the ability to do non-Nova stuff with encryption. | 19:15 |
| hemna | I haven't looked at the encryptor code in the brick patch yet. | 19:15 |
| *** stevemar_ has joined #openstack-cinder | 19:16 | |
| hemna | it may be fine, or it may be borked. | 19:16 |
| smcginnis | Well, wife finally waking up. Better stop working now. :) | 19:16 |
| hemna | but the idea in general seems ok. | 19:16 |
| hemna | :) | 19:16 |
| *** gouthamr has quit IRC | 19:16 | |
| scottda | smcginnis: Have a good one. | 19:17 |
| scottda | hemna: I need to go eat breakfast. Let's put our heads together later. Maybe others will catch up on the subject in scroll back and weigh in. | 19:18 |
| tbarron | smcginnis: enjoy the weekend! | 19:18 |
| hemna | scottda, ok sounds good | 19:18 |
| tbarron | scottda, hemna: the idea that the storage service (cinder, not various backends) is more likely to be compromised than nova seems to me a bit weird. | 19:20 |
| tbarron | I get an argument that the attack surface should be made as small as possible though - but not smaller than possible :-) and the copy to/from image counter seems pretty decisive. | 19:21 |
| hemna | tbarron, yah, I can't see how a c-vol host is more vulnerable than a n-cpu host | 19:21 |
| tbarron | an n-cpu host with guests whom from a security standpoint should be assumed to be hostile. | 19:22 |
| *** jdurgin has quit IRC | 19:22 | |
| hemna | yup | 19:22 |
| hemna | and cinder only has that volume for a 'short' period of time | 19:22 |
| hemna | it's transient | 19:22 |
| tbarron | now if somene wanted to implement a general data-transfer service and limit the use of the keys to that service ... | 19:23 |
| *** jdurgin has joined #openstack-cinder | 19:25 | |
| hemna | but that's another point of access | 19:25 |
| hemna | which is danpb's argument that it should only ever be n-cpu | 19:25 |
| *** baumann has joined #openstack-cinder | 19:25 | |
| tbarron | yup | 19:27 |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder: add force flag on first vdisk host mapping call https://review.openstack.org/248225 | 19:28 |
| tbarron | I wonder if when he says "a core aim .. is to provide against a compromised storage service" he's conflating cinder code and storage backend service? | 19:28 |
| scottda | c-vol is only accessible via admin API, and any sane deployment has it on a separate control plane network | 19:29 |
| scottda | And storage backend is even further isolated. | 19:29 |
| hemna | tbarron, yah I'm not exactly sure what he's implying | 19:29 |
| hemna | scottda, c-vol is also accessible to an operator | 19:29 |
| tbarron | cloud administrators might not trust backend storage administrators, but I don't see cloud nova admins not trusting cloud storage admins, as it were | 19:29 |
| scottda | true. but not to an end user | 19:29 |
| hemna | scottda, correct. | 19:29 |
| scottda | The operator has access to the nova nodes as well. | 19:30 |
| hemna | it's a good goal to have | 19:30 |
| hemna | but, in this case unrealistic. | 19:30 |
| tbarron | the operator can get at the guest's memory if they really want to. | 19:30 |
| openstackgerrit | Alex O'Rourke proposed openstack/cinder: LeftHand: Remove self.db assignment https://review.openstack.org/248228 | 19:30 |
| scottda | In our public cloud, Cinder team must have access to the Nova nodes. There's no way to debug and deal with incidents without access. | 19:31 |
| scottda | Seems like this is a problem that is better solved with smart key management. | 19:31 |
| tbarron | scottda: there you go, messing up a good argument with real world facts. | 19:31 |
| scottda | I respect security experts. But they are kinda like lawyers...find a possible flaw and shut things down just in case. Not realistic. | 19:32 |
| tbarron | yeah, I've had to debug/support customers who are inside an airgap. Not fun telling them what to grep for and read back over the phone. | 19:34 |
| * tbarron leaves for a meeting | 19:34 | |
| *** gouthamr has joined #openstack-cinder | 19:35 | |
| hemna | OpenStack has key management service already | 19:37 |
| hemna | ok I posted a quick reply on the ML | 19:37 |
| hemna | wrt to the encryptors stuffs. | 19:37 |
| *** jdurgin has quit IRC | 19:41 | |
| scottda | Cool | 19:42 |
| tbarron | hemna: well said. | 19:42 |
| *** daneyon_ has joined #openstack-cinder | 19:44 | |
| *** mc_nair has joined #openstack-cinder | 19:44 | |
| *** timcl has joined #openstack-cinder | 19:45 | |
| *** daneyon has quit IRC | 19:48 | |
| *** ociuhandu has quit IRC | 19:49 | |
| *** eharney has quit IRC | 19:57 | |
| *** Guest37770 has quit IRC | 19:57 | |
| *** lpabon has quit IRC | 20:03 | |
| *** setmason has quit IRC | 20:07 | |
| *** sghanekar__ has joined #openstack-cinder | 20:11 | |
| *** sghanekar_ has quit IRC | 20:15 | |
| *** apoorvad has quit IRC | 20:21 | |
| *** sghanekar__ has quit IRC | 20:22 | |
| *** dims has quit IRC | 20:23 | |
| *** thangp has quit IRC | 20:26 | |
| *** setmason has joined #openstack-cinder | 20:27 | |
| *** daneyon has joined #openstack-cinder | 20:30 | |
| *** sghanekar__ has joined #openstack-cinder | 20:31 | |
| *** wilson has joined #openstack-cinder | 20:32 | |
| *** daneyon_ has quit IRC | 20:33 | |
| *** wilson is now known as Guest57508 | 20:33 | |
| *** mriedem has quit IRC | 20:33 | |
| *** setmason_ has joined #openstack-cinder | 20:35 | |
| *** eharney has joined #openstack-cinder | 20:36 | |
| *** wilson-l has quit IRC | 20:36 | |
| *** setmason has quit IRC | 20:37 | |
| *** setmason_ is now known as setmason | 20:37 | |
| *** porrua has quit IRC | 20:40 | |
| *** akerr has quit IRC | 20:41 | |
| *** merooney has quit IRC | 20:42 | |
| openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: api-microversion support for python-cinderclient https://review.openstack.org/248163 | 20:48 |
| *** timcl has left #openstack-cinder | 20:49 | |
| *** setmason has quit IRC | 20:50 | |
| *** jdurgin has joined #openstack-cinder | 20:52 | |
| *** jgregor has quit IRC | 20:59 | |
| *** openstackstatus has quit IRC | 21:02 | |
| *** openstack has joined #openstack-cinder | 21:04 | |
| nikeshm | kmartin: thanks, actually Rekha was facing issue | 21:08 |
| nikeshm | Rekha: are you online | 21:09 |
| nikeshm | Rekha: you can see chat history for solution of your problem | 21:10 |
| nikeshm | smcginnis : whats up | 21:11 |
| *** daneyon_ has joined #openstack-cinder | 21:12 | |
| *** jgregor has joined #openstack-cinder | 21:13 | |
| Swanson | nikeshm, I'm pretty sure smcginnis is out on the town with his wife for their anniversary. | 21:13 |
| *** marcusvrn_ has quit IRC | 21:15 | |
| nikeshm | Swanson: nice, but i heard he is in Paris, so better be careful due to recent activities in Paris | 21:15 |
| *** daneyon has quit IRC | 21:16 | |
| nikeshm | :) | 21:16 |
| openstackgerrit | Gerald McBrearty proposed openstack/cinder: add config option to control flash copy rate https://review.openstack.org/248250 | 21:16 |
| *** ociuhandu has joined #openstack-cinder | 21:20 | |
| *** apoorvad has joined #openstack-cinder | 21:21 | |
| *** apoorvad has quit IRC | 21:27 | |
| *** jbernard has joined #openstack-cinder | 21:30 | |
| *** stevemar_ has quit IRC | 21:31 | |
| *** dustins has quit IRC | 21:33 | |
| *** sghanekar__ has quit IRC | 21:35 | |
| *** amrith is now known as _amrith_ | 21:39 | |
| *** _amrith_ is now known as amrith | 21:39 | |
| *** crose has joined #openstack-cinder | 21:39 | |
| openstackgerrit | xing-yang proposed openstack/cinder: Add a GateDriver https://review.openstack.org/244956 | 21:40 |
| *** jordanP has quit IRC | 21:41 | |
| *** delattec has quit IRC | 21:44 | |
| hemna | gatedriver! | 21:46 |
| *** mriedem has joined #openstack-cinder | 21:46 | |
| * hemna waits for someone to file a bug against the GateDriver not supporting data consistency....... | 21:48 | |
| xyang1 | hemna: I know it is going to come:) | 21:49 |
| hemna | hehe yah | 21:49 |
| hemna | rename it to the DieDieDieDriver | 21:49 |
| hemna | j/k | 21:50 |
| *** gouthamr has quit IRC | 21:50 | |
| xyang1 | hemna: :) | 21:50 |
| *** rcernin has quit IRC | 21:50 | |
| hemna | :) | 21:50 |
| *** IlyaG has joined #openstack-cinder | 21:53 | |
| *** amrith is now known as _amrith_ | 21:56 | |
| *** dave-mccowan has quit IRC | 22:03 | |
| *** davechen has left #openstack-cinder | 22:06 | |
| *** bswartz has quit IRC | 22:08 | |
| *** sgundur has quit IRC | 22:11 | |
| *** edtubill has joined #openstack-cinder | 22:18 | |
| *** sghanekar__ has joined #openstack-cinder | 22:36 | |
| *** baumann has quit IRC | 22:38 | |
| *** crose has quit IRC | 22:42 | |
| *** jwcroppe has quit IRC | 22:44 | |
| *** jgregor has quit IRC | 22:49 | |
| *** tpeoples has quit IRC | 22:49 | |
| *** simondodsley_ has quit IRC | 22:50 | |
| *** mc_nair has quit IRC | 22:52 | |
| *** ntpttr has left #openstack-cinder | 22:52 | |
| *** Guest57508 has quit IRC | 22:52 | |
| *** Guest57508 has joined #openstack-cinder | 22:53 | |
| *** krtaylor has quit IRC | 22:55 | |
| *** stevemar has joined #openstack-cinder | 23:03 | |
| *** xyang1 has quit IRC | 23:03 | |
| *** stevemar_ has joined #openstack-cinder | 23:04 | |
| *** stevemar_ has quit IRC | 23:04 | |
| *** krtaylor has joined #openstack-cinder | 23:08 | |
| *** wilson has joined #openstack-cinder | 23:12 | |
| *** wilson is now known as Guest81126 | 23:12 | |
| *** Guest57508 has quit IRC | 23:14 | |
| *** jwcroppe has joined #openstack-cinder | 23:14 | |
| *** edtubill has quit IRC | 23:19 | |
| *** changbl has quit IRC | 23:23 | |
| *** jwcroppe has quit IRC | 23:23 | |
| *** changbl has joined #openstack-cinder | 23:27 | |
| *** alejandrito has quit IRC | 23:29 | |
| *** changbl has quit IRC | 23:33 | |
| *** markvoelker has quit IRC | 23:34 | |
| *** adelia has joined #openstack-cinder | 23:38 | |
| *** adelia_ has joined #openstack-cinder | 23:38 | |
| *** jwcroppe has joined #openstack-cinder | 23:39 | |
| *** wilson-l has joined #openstack-cinder | 23:41 | |
| *** adelia has quit IRC | 23:43 | |
| *** adelia_ has quit IRC | 23:43 | |
| *** Guest81126 has quit IRC | 23:44 | |
| *** [1]Thelo has joined #openstack-cinder | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!