| *** vilobhmm has quit IRC | 00:00 | |
| thingee | cool I don't feel as dumb | 00:00 |
|---|---|---|
| *** merooney has quit IRC | 00:02 | |
| *** hodos has quit IRC | 00:03 | |
| *** merooney has joined #openstack-cinder | 00:11 | |
| *** jasondotstar has quit IRC | 00:11 | |
| *** merooney has quit IRC | 00:11 | |
| *** merooney has joined #openstack-cinder | 00:12 | |
| *** _cjones_ has quit IRC | 00:14 | |
| *** merooeny_ has joined #openstack-cinder | 00:14 | |
| thingee | winston-d_zZZ: thinking of something like http://docs.openstack.org/kilo/config-reference/content/section_compute-scheduler.html | 00:15 |
| *** merooney has quit IRC | 00:16 | |
| *** jasondotstar has joined #openstack-cinder | 00:17 | |
| *** merooeny_ has quit IRC | 00:18 | |
| *** xek_ has joined #openstack-cinder | 00:19 | |
| *** xek has quit IRC | 00:20 | |
| *** vokt has quit IRC | 00:20 | |
| *** IlyaG has joined #openstack-cinder | 00:23 | |
| *** esker has quit IRC | 00:23 | |
| *** david-lyle has quit IRC | 00:24 | |
| thingee | scottda: you still around? | 00:28 |
| *** tsekiyam_ has joined #openstack-cinder | 00:32 | |
| *** xyang1 has quit IRC | 00:33 | |
| scottda | Yup | 00:34 |
| *** changbl has joined #openstack-cinder | 00:34 | |
| *** mtanino has quit IRC | 00:34 | |
| *** Yogi1 has quit IRC | 00:34 | |
| *** tsekiyama has quit IRC | 00:35 | |
| *** garthb has quit IRC | 00:35 | |
| thingee | scottda: how hard would it be to have one of the people that brings in coffee to instead bring an empty thermos dispensers? If people are interested, I could make coffee :) | 00:35 |
| *** tsekiyam_ has quit IRC | 00:36 | |
| scottda | I'm not sure...I'll ask tomorrow. | 00:37 |
| scottda | If they cannot, I've one, and could see about scrounging up some others. | 00:37 |
| scottda | It is packaged with pastries/fruit, so I'm not sure if we can just cancel the coffee. | 00:38 |
| jungleboyj | Ooooh, real thingee coffee! :-) | 00:38 |
| thingee | whoa I think my messages didn't make it over cause I switching to vpn... | 00:38 |
| scottda | I'll ask catering, and we can see if people care about the food part at tomorrow's meeting, if that sounds good? | 00:38 |
| thingee | scottda: I'm talking about that hipster coffee stuff. pourover, fresh beans and just grinded. | 00:39 |
| thingee | scottda: all I need is a single outlet for the kettle, dispenser and water. | 00:39 |
| *** albertom has quit IRC | 00:39 | |
| thingee | just so people are happy and productive :) | 00:39 |
| scottda | well, there will certainly be an outlet. | 00:39 |
| thingee | don | 00:39 |
| thingee | t have answer for tea people though | 00:40 |
| scottda | so, you still want one or more thermos to hold the coffee? | 00:40 |
| thingee | scottda: heh just an idea, if it's too much trouble then we can just use the nearby cafe or whatever | 00:41 |
| *** leeantho has quit IRC | 00:41 | |
| *** albertom has joined #openstack-cinder | 00:42 | |
| thingee | scottda: so I usually use a 3liter canister...I can use whatever they have. | 00:42 |
| scottda | Well, I've wrangled kmartin and hemna to arrange for coffee and pastries brought in. We could cancel or change, and/or I could bring a thermos or see if they have an extra for "custom coffee". | 00:42 |
| thingee | scottda: ah that's fine.. I don't want to be a trouble on canceling things | 00:43 |
| scottda | I think they generally use 3L canisters. | 00:43 |
| scottda | I don't think we have arrangements for the afternoon either, so one way or another some good coffee will be welcome. | 00:43 |
| *** Yogi1 has joined #openstack-cinder | 00:43 | |
| scottda | I'll work on the logistics tomorrow, when catering is around to answer questions. | 00:43 |
| thingee | I have a bunch of canisters I use for burning man, but fitting on those in my bag would take up a bit of room, especially since this won't be my first stop. | 00:44 |
| scottda | np, we'll figure it out. | 00:44 |
| openstackgerrit | wanghao proposed openstack/cinder: Fixing notify message of manage_existing flow https://review.openstack.org/205896 | 00:45 |
| *** xek_ has quit IRC | 00:46 | |
| *** xek_ has joined #openstack-cinder | 00:47 | |
| *** zhenguo has joined #openstack-cinder | 00:57 | |
| *** IlyaG has quit IRC | 00:58 | |
| *** Yogi1 has quit IRC | 00:59 | |
| *** zhonghua-lee has joined #openstack-cinder | 00:59 | |
| *** garthb has joined #openstack-cinder | 01:00 | |
| *** garthb_ has joined #openstack-cinder | 01:00 | |
| *** garthb_ has quit IRC | 01:00 | |
| *** laughterwym has joined #openstack-cinder | 01:07 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:12 | |
| *** jasondotstar has quit IRC | 01:12 | |
| *** davechen has joined #openstack-cinder | 01:15 | |
| *** zhonghua-li has joined #openstack-cinder | 01:16 | |
| *** Liu has joined #openstack-cinder | 01:16 | |
| lixiaoy1 | jdurgin: Hi Josh, could like to talk with you ceph backup.OK? | 01:17 |
| *** jasondotstar has joined #openstack-cinder | 01:17 | |
| *** zhonghua-lee has quit IRC | 01:19 | |
| *** LiuXinguo has quit IRC | 01:19 | |
| lixiaoy1 | Or anyone know that why we need to do full restore in ceph if source volume is same as destination volume: https://github.com/openstack/cinder/blob/master/cinder/backup/drivers/ceph.py#L1070 | 01:19 |
| *** r-daneel has quit IRC | 01:19 | |
| *** salv-orlando has joined #openstack-cinder | 01:20 | |
| *** heyun has joined #openstack-cinder | 01:21 | |
| *** jasondotstar has quit IRC | 01:22 | |
| *** breitz has quit IRC | 01:22 | |
| *** lixiaoy1 has quit IRC | 01:23 | |
| *** salv-orlando has quit IRC | 01:25 | |
| *** patrickeast has quit IRC | 01:25 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:26 | |
| davechen | lixiaoy1: seems like this is not a good time to talk :) | 01:26 |
| *** annashen has joined #openstack-cinder | 01:26 | |
| *** mriedem has quit IRC | 01:26 | |
| *** Lee1092 has joined #openstack-cinder | 01:28 | |
| *** annashen has quit IRC | 01:31 | |
| *** lixiaoy11 has joined #openstack-cinder | 01:33 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 01:35 |
| *** lixiaoy1 has quit IRC | 01:35 | |
| *** breitz has joined #openstack-cinder | 01:37 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Validate 'is_public' when creating volume type https://review.openstack.org/206784 | 01:45 |
| *** vincent_hou has joined #openstack-cinder | 01:46 | |
| vincent_hou | tbarron: Hi Tom. | 01:47 |
| *** annashen has joined #openstack-cinder | 01:55 | |
| *** daneyon has quit IRC | 01:57 | |
| *** breitz has quit IRC | 01:57 | |
| tbarron | hi vincent_hou | 01:58 |
| *** Apoorva has quit IRC | 01:58 | |
| *** annashen has quit IRC | 02:00 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add the ability to update volume type public status https://review.openstack.org/206788 | 02:01 |
| *** garthb has quit IRC | 02:10 | |
| *** annashen has joined #openstack-cinder | 02:10 | |
| *** annashen has quit IRC | 02:12 | |
| *** Tross has joined #openstack-cinder | 02:16 | |
| *** vincent_hou has quit IRC | 02:17 | |
| *** david-lyle has joined #openstack-cinder | 02:19 | |
| *** haomaiwang has quit IRC | 02:21 | |
| *** haomaiwa_ has joined #openstack-cinder | 02:22 | |
| *** david-lyle has quit IRC | 02:23 | |
| *** annashen has joined #openstack-cinder | 02:27 | |
| *** jwcroppe has quit IRC | 02:27 | |
| *** jwcroppe has joined #openstack-cinder | 02:28 | |
| *** annashen has quit IRC | 02:32 | |
| *** jwcroppe has quit IRC | 02:32 | |
| *** haomaiwa_ has quit IRC | 02:36 | |
| *** heyun has quit IRC | 02:36 | |
| *** haomaiwang has joined #openstack-cinder | 02:36 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 02:38 |
| *** boris-42 has quit IRC | 02:40 | |
| *** zhonghua-li has quit IRC | 02:41 | |
| *** zhonghua-lee has joined #openstack-cinder | 02:41 | |
| *** angela-s has quit IRC | 02:43 | |
| *** IlyaG has joined #openstack-cinder | 02:50 | |
| *** hakimo has joined #openstack-cinder | 02:51 | |
| *** hakimo_ has quit IRC | 02:53 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 03:02 |
| *** annashen has joined #openstack-cinder | 03:10 | |
| *** mragupat has joined #openstack-cinder | 03:12 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 03:12 |
| *** mragupat_ has joined #openstack-cinder | 03:13 | |
| *** haomaiwang has quit IRC | 03:14 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 03:15 |
| *** srkr has joined #openstack-cinder | 03:15 | |
| *** mragupat has quit IRC | 03:16 | |
| *** coolsvap|away is now known as coolsvap | 03:26 | |
| *** annashen_ has joined #openstack-cinder | 03:32 | |
| *** david-lyle has joined #openstack-cinder | 03:34 | |
| *** annashen has quit IRC | 03:35 | |
| openstackgerrit | Bharat Kumar Kobagana (BharatK) proposed openstack/cinder: GlusterFS: Using 'fallocate' instead of 'dd' https://review.openstack.org/164527 | 03:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/206807 | 03:45 |
| *** ishant has joined #openstack-cinder | 03:48 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/206829 | 03:49 |
| *** BharatK has joined #openstack-cinder | 03:49 | |
| *** davechen is now known as davechen_afk | 03:58 | |
| *** annashen has joined #openstack-cinder | 03:58 | |
| *** zhonghua-lee has quit IRC | 03:59 | |
| *** zhonghua-lee has joined #openstack-cinder | 04:01 | |
| *** Apoorva has joined #openstack-cinder | 04:03 | |
| *** annashen has quit IRC | 04:03 | |
| *** annashen_ has quit IRC | 04:07 | |
| *** mragupat_ has quit IRC | 04:09 | |
| *** jwcroppe has joined #openstack-cinder | 04:14 | |
| *** IlyaG has quit IRC | 04:17 | |
| *** harlowja_at_home has joined #openstack-cinder | 04:17 | |
| *** zhonghua-lee has quit IRC | 04:27 | |
| *** avishay has joined #openstack-cinder | 04:32 | |
| *** zhonghua-lee has joined #openstack-cinder | 04:36 | |
| *** vilobhmm has joined #openstack-cinder | 04:49 | |
| *** annashen has joined #openstack-cinder | 04:49 | |
| *** rushiagr_away is now known as rushiagr | 04:50 | |
| srkr | hi, is there any limitation on the size of a cinder volume that can be used for "boot from volume" provided tenant quota and cinder-backend allows it | 04:51 |
| *** IlyaG has joined #openstack-cinder | 04:51 | |
| *** ub has joined #openstack-cinder | 04:53 | |
| *** LiuXinguo has joined #openstack-cinder | 04:53 | |
| *** simondodsley_ has joined #openstack-cinder | 04:53 | |
| *** Liu has quit IRC | 04:54 | |
| *** ub2 has quit IRC | 04:54 | |
| *** simondodsley has quit IRC | 04:54 | |
| *** annashen_ has joined #openstack-cinder | 04:59 | |
| *** harlowja_still_a has joined #openstack-cinder | 05:02 | |
| *** annashen_ has quit IRC | 05:03 | |
| *** shyama has joined #openstack-cinder | 05:05 | |
| *** harlowja_at_home has quit IRC | 05:05 | |
| *** daneyon has joined #openstack-cinder | 05:05 | |
| *** srkr has quit IRC | 05:06 | |
| *** daneyon_ has joined #openstack-cinder | 05:07 | |
| *** ub has quit IRC | 05:08 | |
| *** BharatK has quit IRC | 05:09 | |
| *** daneyon has quit IRC | 05:10 | |
| *** annashen has quit IRC | 05:10 | |
| *** annashen has joined #openstack-cinder | 05:16 | |
| *** harlowja_still_a has quit IRC | 05:16 | |
| *** Apoorva has quit IRC | 05:16 | |
| *** harlowja_at_home has joined #openstack-cinder | 05:16 | |
| *** Apoorva has joined #openstack-cinder | 05:22 | |
| *** IlyaG has quit IRC | 05:24 | |
| *** jwcroppe has quit IRC | 05:25 | |
| *** jwcroppe has joined #openstack-cinder | 05:25 | |
| *** BharatK has joined #openstack-cinder | 05:25 | |
| *** harlowja_still_a has joined #openstack-cinder | 05:29 | |
| *** jwcroppe has quit IRC | 05:30 | |
| *** deepakcs has joined #openstack-cinder | 05:31 | |
| *** lpetrut has joined #openstack-cinder | 05:31 | |
| *** harlowja_at_home has quit IRC | 05:32 | |
| *** harlowja_still_a has quit IRC | 05:38 | |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 05:41 |
| *** jwcroppe has joined #openstack-cinder | 05:45 | |
| *** shyama has quit IRC | 05:45 | |
| *** heyun has joined #openstack-cinder | 05:49 | |
| *** agarciam has joined #openstack-cinder | 05:55 | |
| *** annashen_ has joined #openstack-cinder | 06:00 | |
| *** annashen has quit IRC | 06:00 | |
| *** lpetrut has quit IRC | 06:00 | |
| *** Apoorva has quit IRC | 06:01 | |
| *** vlaza has joined #openstack-cinder | 06:04 | |
| *** lpetrut has joined #openstack-cinder | 06:04 | |
| *** annashen_ has quit IRC | 06:04 | |
| *** rushiagr is now known as rushiagr_away | 06:07 | |
| *** afazekas has joined #openstack-cinder | 06:09 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/206877 | 06:10 |
| *** rushiagr_away is now known as rushiagr | 06:16 | |
| *** nkrinner has joined #openstack-cinder | 06:18 | |
| *** wanghao has joined #openstack-cinder | 06:30 | |
| *** ub has joined #openstack-cinder | 06:32 | |
| *** zhenguo has quit IRC | 06:32 | |
| *** rushiagr is now known as rushiagr_away | 06:34 | |
| *** abhishekk has joined #openstack-cinder | 06:34 | |
| deepakcs | anyone with DB expertise here ? Whats the advantage of doing index=True in Column('project_id', String(length=255), index=True) | 06:35 |
| *** alonmarx has quit IRC | 06:35 | |
| *** zhenguo has joined #openstack-cinder | 06:40 | |
| dulek | deepakcs: It gives better performance when filtering by project_id, which is done quite often I think. | 06:41 |
| deepakcs | dulek: so does it help in sorting or just plain retrievign ? | 06:42 |
| dulek | deepakcs: It definitely helps with "WHERE project_id='foo'" clauses. Let me think about sorting... | 06:42 |
| deepakcs | dulek: thanks, I am not a DB guy, and i recd some comments from e0ne @ https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/migrate_repo/versions/049_add_snapshot_admin_metadata_table.py | 06:43 |
| deepakcs | dulek: could you help me understand what he means by "I propose to use UUID as primary key to make it more scalable" | 06:43 |
| dulek | deepakcs: http://sqlmag.com/database-performance-tuning/indexing-sort-performance - it seems that index can help with sorting performance. | 06:44 |
| deepakcs | dulek: i did a grep on primary_key in cinder/db/sqlalchemy/migrate_repo/versions/* and none of them use UUID for primary_keys, so not sure how to address that comment of his. Any help appreciated :) | 06:44 |
| *** abehl has joined #openstack-cinder | 06:44 | |
| deepakcs | dulek: good to know. | 06:45 |
| *** lpetrut has quit IRC | 06:45 | |
| dulek | deepakcs: You should have did a grep on cinder/db/sqlalchemy/models | 06:46 |
| dulek | deepakcs: https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/models.py#L113 | 06:46 |
| dulek | deepakcs: Even volume has it's id (which is primary_key) as UUID. | 06:47 |
| deepakcs | dulek: ah, so he meant to use a String type instead of integer type, so that id can hold UUID ? | 06:48 |
| dulek | deepakcs: This is how it's done in current code. | 06:49 |
| deepakcs | dulek: I am adding snapshot_admin_metadata on similar lines of volume_admin_metadata and i just followed https://github.com/openstack/cinder/blob/master/cinder/db/sqlalchemy/migrate_repo/versions/020_add_volume_admin_metadata_table.py#L30 | 06:49 |
| deepakcs | dulek: as u can see there ^^ it uses Integer for 'id' field, which i just followed for my case :) | 06:49 |
| dulek | deepakcs: Then I think you should respond with that and ping e0ne. | 06:51 |
| deepakcs | dulek: ok, but i think agreeing to what he says also makes sense... so in a dilema right now | 06:52 |
| deepakcs | dulek: i just wanted to check with other DB experts around here, if there is any specific reason why 'id'is a Integer for volume_admin_md case ? can u think of any ? | 06:52 |
| dulek | deepakcs: I don't have enough expertise to make decision here, so let's pass it to Ivan. ;) | 06:53 |
| deepakcs | dulek: and i guess Ivan is available in US times ? | 06:53 |
| dulek | deepakcs: No, he is living in eastern Europe, I guess he's eating his breakfast now. ;) | 06:54 |
| * deepakcs really wishes if people can provide more constructive comments for DB files, since not many have DB expertise | 06:54 | |
| lifeless | uhm | 06:54 |
| lifeless | ints are faster to compare than uuids | 06:54 |
| deepakcs | lifeless: e0ne says UUID makes it more scalable, so feel free to debate (see the link i gave above with e0ne's comments) | 06:55 |
| dulek | deepakcs: Well, I find knowledge on indexes rather basic stuff on databases. Anyway the good side of reviews is that you can always learn. | 06:55 |
| deepakcs | dulek: given the TZ difference, its better for people to provide hints that can expedite the work. I am old enuf to justify saying that i don't remember much of DB specifics | 06:56 |
| deepakcs | dulek: esp when adding a new DB table is not the primary focus of my patch series | 06:57 |
| dulek | deepakcs: More descriptive comments are always welcomed, not only in DB-related patches. | 06:57 |
| *** kjelly has joined #openstack-cinder | 06:57 | |
| deepakcs | dulek: any idea how far from now e0ne would be online ? | 06:57 |
| dulek | deepakcs: It's just sometimes hard to realize what level of description is enough. | 06:58 |
| *** davechen_afk is now known as davechen | 06:58 | |
| *** markus_z has joined #openstack-cinder | 06:58 | |
| dulek | I would expect him to be here in an hour or so. If he haven't started his vacation or something. | 06:59 |
| *** vilobhmm has quit IRC | 06:59 | |
| *** zhonghua-lee has quit IRC | 07:00 | |
| lifeless | consistencygroup has a string pk | 07:01 |
| lifeless | as does cgsnapshot and volune | 07:01 |
| *** annashen has joined #openstack-cinder | 07:01 | |
| *** abehl has quit IRC | 07:03 | |
| deepakcs | thanks dulek lifeless | 07:03 |
| *** annashen has quit IRC | 07:05 | |
| openstackgerrit | wanghao proposed openstack/cinder: Implement function to manage/unmanage snapshots https://review.openstack.org/144590 | 07:08 |
| *** rushiagr_away is now known as rushiagr | 07:10 | |
| *** salv-orlando has joined #openstack-cinder | 07:12 | |
| *** abehl has joined #openstack-cinder | 07:16 | |
| *** ronis has joined #openstack-cinder | 07:19 | |
| *** rushiagr is now known as rushiagr_away | 07:20 | |
| openstackgerrit | Sergey Vilgelm proposed openstack/cinder: Switch to the oslo_utils.fileutils https://review.openstack.org/199537 | 07:21 |
| *** jwcroppe has quit IRC | 07:22 | |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 07:25 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 07:28 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Fix use of urllib for py3 compatibility in Huawei driver https://review.openstack.org/203996 | 07:34 |
| *** markus_z has quit IRC | 07:34 | |
| *** lpetrut has joined #openstack-cinder | 07:46 | |
| *** abehl has quit IRC | 07:48 | |
| *** zhonghua-lee has joined #openstack-cinder | 07:48 | |
| *** jwcroppe has joined #openstack-cinder | 07:51 | |
| *** markus_z has joined #openstack-cinder | 07:51 | |
| *** bluex-pl has joined #openstack-cinder | 07:51 | |
| *** bluex-pl has quit IRC | 07:51 | |
| openstackgerrit | XinXiaohui proposed openstack/cinder: capacity-headroom https://review.openstack.org/206923 | 07:59 |
| openstackgerrit | Xi Yang proposed openstack/cinder: Snap copy feature for EMC VNX Cinder driver https://review.openstack.org/184733 | 08:01 |
| *** annashen has joined #openstack-cinder | 08:01 | |
| *** Liu has joined #openstack-cinder | 08:01 | |
| *** abehl has joined #openstack-cinder | 08:01 | |
| *** zhonghua-li has joined #openstack-cinder | 08:01 | |
| *** annashen_ has joined #openstack-cinder | 08:02 | |
| openstackgerrit | Li Yingjun proposed openstack/cinder: Add the ability to update type public status https://review.openstack.org/206788 | 08:02 |
| *** zhonghua-lee has quit IRC | 08:04 | |
| *** LiuXinguo has quit IRC | 08:04 | |
| *** annashen has quit IRC | 08:05 | |
| *** annashen_ has quit IRC | 08:06 | |
| *** jordanP has joined #openstack-cinder | 08:06 | |
| *** daneyon_ has quit IRC | 08:07 | |
| *** jistr has joined #openstack-cinder | 08:08 | |
| *** ndipanov has quit IRC | 08:14 | |
| *** ndipanov has joined #openstack-cinder | 08:14 | |
| *** jasondotstar has joined #openstack-cinder | 08:25 | |
| openstackgerrit | Dave Chen proposed openstack/cinder: Role based properties protection https://review.openstack.org/160090 | 08:26 |
| openstackgerrit | Dave Chen proposed openstack/cinder: Policies based properties protection https://review.openstack.org/160145 | 08:26 |
| *** zhonghua-li has quit IRC | 08:28 | |
| *** lixiaoy11 has quit IRC | 08:28 | |
| *** jwcroppe has quit IRC | 08:35 | |
| *** davechen is now known as davechen_afk | 08:36 | |
| openstackgerrit | wanghao proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 08:36 |
| *** shyama has joined #openstack-cinder | 08:37 | |
| *** e0ne has joined #openstack-cinder | 08:40 | |
| *** markus_z has quit IRC | 08:43 | |
| *** e0ne has quit IRC | 08:43 | |
| openstackgerrit | Jordan Pittier proposed openstack/os-brick: Remotefs: add ScalityFS support https://review.openstack.org/196631 | 08:43 |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA lock spec https://review.openstack.org/202615 | 08:43 |
| *** markus_z has joined #openstack-cinder | 08:43 | |
| dulek | geguileo: ^ | 08:44 |
| dulek | geguileo: I've just updated some nits... I also don't get double spaces after dots, but let's not dig into it. | 08:45 |
| *** zhonghua-li has joined #openstack-cinder | 08:48 | |
| *** davechen_afk is now known as davechen | 08:50 | |
| *** zhonghua-li has quit IRC | 08:56 | |
| *** zhonghua-li has joined #openstack-cinder | 08:57 | |
| *** zhonghua has joined #openstack-cinder | 08:58 | |
| geguileo | dulek: In a meeting, give me 10 minutes | 08:58 |
| *** zhonghua-li has quit IRC | 09:02 | |
| *** jwcroppe has joined #openstack-cinder | 09:02 | |
| *** markus_z has quit IRC | 09:02 | |
| *** annashen has joined #openstack-cinder | 09:02 | |
| *** coolsvap is now known as coolsvap|away | 09:04 | |
| *** BharatK has quit IRC | 09:05 | |
| geguileo | dulek: Ok, I'm back | 09:05 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 09:06 |
| geguileo | dulek: I'll read it now. We can remove double spaces, no problem | 09:06 |
| *** annashen has quit IRC | 09:07 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 09:08 |
| *** coolsvap|away is now known as coolsvap | 09:12 | |
| geguileo | dulek: Look good to me | 09:12 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add volume migration support for Huawei driver https://review.openstack.org/201406 | 09:12 |
| geguileo | dulek: I must have been hungry yesterday, because I ate a bunch of "ed"s ;-) | 09:12 |
| *** davechen has left #openstack-cinder | 09:13 | |
| *** markus_z has joined #openstack-cinder | 09:19 | |
| *** alexpilotti has joined #openstack-cinder | 09:21 | |
| *** ankit_ag has joined #openstack-cinder | 09:26 | |
| -openstackstatus- NOTICE: Currently our CI system is broken, jobs are not getting processed at all. | 09:27 | |
| *** ChanServ changes topic to "Currently our CI system is broken, jobs are not getting processed at all." | 09:27 | |
| *** gaurangt has joined #openstack-cinder | 09:32 | |
| openstackgerrit | wanghao proposed openstack/cinder: Add pagination to backups https://review.openstack.org/204493 | 09:35 |
| *** bluex-pl has joined #openstack-cinder | 09:37 | |
| *** bluex-pl has quit IRC | 09:37 | |
| *** bluex-pl has joined #openstack-cinder | 09:37 | |
| *** heyun has quit IRC | 09:41 | |
| *** nikeshm has joined #openstack-cinder | 09:45 | |
| nikeshm | hi | 09:46 |
| *** markus_z has quit IRC | 09:46 | |
| *** mdbooth has quit IRC | 09:46 | |
| *** mdbooth has joined #openstack-cinder | 09:47 | |
| nikeshm | liuxg: hi | 09:49 |
| nikeshm | Liu: are you Liu Xinguo? | 09:50 |
| *** coolsvap is now known as coolsvap|away | 09:50 | |
| *** e0ne has joined #openstack-cinder | 09:57 | |
| *** jasondotstar has quit IRC | 09:57 | |
| *** shyama has quit IRC | 09:58 | |
| *** coolsvap|away is now known as coolsvap | 09:58 | |
| *** jwcroppe has quit IRC | 10:00 | |
| *** ankit_ag has quit IRC | 10:00 | |
| *** abehl has quit IRC | 10:00 | |
| *** ub has quit IRC | 10:00 | |
| *** rhe00 has quit IRC | 10:00 | |
| *** jwang has quit IRC | 10:00 | |
| *** Swanson has quit IRC | 10:00 | |
| *** mitz has quit IRC | 10:00 | |
| *** mitz has joined #openstack-cinder | 10:00 | |
| *** ankit_ag has joined #openstack-cinder | 10:01 | |
| *** Swanson has joined #openstack-cinder | 10:01 | |
| *** jwang has joined #openstack-cinder | 10:01 | |
| *** abhishekkk has joined #openstack-cinder | 10:01 | |
| *** ub has joined #openstack-cinder | 10:01 | |
| *** rhe00 has joined #openstack-cinder | 10:01 | |
| *** annashen has joined #openstack-cinder | 10:02 | |
| deepakcs | e0ne: need some help wrt your comment @ https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/migrate_repo/versions/049_add_snapshot_admin_metadata_table.py | 10:02 |
| deepakcs | e0ne: i changed the field 'id' from Integer to String(36) | 10:02 |
| *** e0ne_ has joined #openstack-cinder | 10:02 | |
| *** abehl has joined #openstack-cinder | 10:03 | |
| deepakcs | e0ne: I am hitting this error when i try to update an entry in snapshot_admin_metadata : http://paste.fedoraproject.org/249290/64208143 | 10:04 |
| *** annashen_ has joined #openstack-cinder | 10:04 | |
| deepakcs | e0ne: I even tried to adding (if not 'id', id = uuid) my snapshot_admin_metadata_update function. See this: http://paste.fedoraproject.org/249291/14381642 | 10:05 |
| deepakcs | e0ne_: ^^ | 10:05 |
| *** rushiagr_away has quit IRC | 10:05 | |
| *** Lee1092 has quit IRC | 10:05 | |
| *** e0ne has quit IRC | 10:05 | |
| *** abhishekk has quit IRC | 10:05 | |
| *** kaisers has quit IRC | 10:05 | |
| *** timcl1 has quit IRC | 10:05 | |
| *** DTadrzak has quit IRC | 10:05 | |
| deepakcs | e0ne: e0ne_ still hitting the same error (field id doesn't have default value) | 10:05 |
| e0ne_ | deepakcs: is it on aster or some patch? | 10:06 |
| *** e0ne_ is now known as e0ne | 10:06 | |
| *** DTadrzak has joined #openstack-cinder | 10:06 | |
| *** timcl has joined #openstack-cinder | 10:06 | |
| deepakcs | e0ne: looks like u missed my earlier msg, let me repaste it | 10:06 |
| deepakcs | e0ne: need some help wrt your comment @ https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/migrate_repo/versions/049_add_snapshot_admin_metadata_table.py | 10:06 |
| deepakcs | e0ne: i changed the field 'id' from Integer to String(36) | 10:06 |
| deepakcs | e0ne: aster meaning ? | 10:06 |
| *** annashen has quit IRC | 10:06 | |
| *** kaisers has joined #openstack-cinder | 10:06 | |
| e0ne | deepakcs: yes, i missed this message. give me 5 minutes, please | 10:07 |
| deepakcs | e0ne: np | 10:07 |
| *** Lee1092 has joined #openstack-cinder | 10:07 | |
| *** BharatK has joined #openstack-cinder | 10:08 | |
| *** annashen_ has quit IRC | 10:08 | |
| *** jasondotstar has joined #openstack-cinder | 10:09 | |
| *** zhenguo has quit IRC | 10:13 | |
| deepakcs | e0ne: there ? | 10:29 |
| e0ne | deepakcs: looking | 10:30 |
| *** shyama has joined #openstack-cinder | 10:31 | |
| e0ne | deepakcs: sqlalchemy can't generate uuid automacitly for pk | 10:32 |
| deepakcs | e0ne: http://paste.fedoraproject.org/249291/14381642 | 10:32 |
| deepakcs | e0ne: ^^ i did that, still hitting the error as seen here : http://paste.fedoraproject.org/249290/64208143 | 10:32 |
| deepakcs | e0ne: lines 5 & 6 in http://paste.fedoraproject.org/249291/14381642 add the uuid if not present. | 10:33 |
| e0ne | deepakcs: could you please share full traceback? | 10:35 |
| Liu | nikeshm: yeah, I am Liu Xinguo | 10:36 |
| deepakcs | e0ne: http://paste.fedoraproject.org/249290/64208143/ is what i see in my c-api log | 10:36 |
| openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Set virtual disk UUID to volume ID https://review.openstack.org/205494 | 10:36 |
| Liu | nikeshm: or liuxinguo or liuxg :) | 10:36 |
| e0ne | deepakcs: i don't see any cinder code there. only oslo.db traceback | 10:37 |
| deepakcs | e0ne: thats all i am getting. see line 1.. thats the cinder api contrib_actions invoking a snapshot.update followed by save in the code | 10:38 |
| deepakcs | e0ne: after which i see the traceback | 10:38 |
| *** aix has quit IRC | 10:38 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Move import and export backup metadata to object https://review.openstack.org/200927 | 10:39 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 10:39 |
| Liu | nikeshm: I will leave for dinner time, and will be back about two hours latter | 10:39 |
| nikeshm | Liu: ok, thanks, are you thinking to optimize the code for smartx feature, | 10:39 |
| deepakcs | e0ne: can u review my snapshot_admin_metadata _update function in https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/api.py | 10:40 |
| deepakcs | e0ne: and what needs to be done there if i change 'id' field of my table from Integer to String(36) | 10:40 |
| Liu | nikeshm: yes, I have thought about it | 10:40 |
| deepakcs | e0ne: bcos thats what i did, and landed in this issue, so something in my _update function isn't correct | 10:40 |
| *** vlaza has left #openstack-cinder | 10:40 | |
| Liu | nikeshm: but have not yet a good idea about it | 10:41 |
| e0ne | deepakcs: the code looks fine | 10:41 |
| nikeshm | Liu: i have commented some suggestions on the patch, that might help | 10:42 |
| deepakcs | e0ne: to begin with it doesn't initialize the 'id' field (when it was Integer, it was autoincrement, but now its string and i don't set it anywhere) | 10:42 |
| deepakcs | e0ne: there is no create() for metadata tables, they get rows added via snapshot['admin_metadata'] | 10:42 |
| Liu | nikeshm: that's greate! | 10:43 |
| e0ne | deepakcs: it could be an issue | 10:43 |
| e0ne | deepakcs: it affects a lot of code | 10:43 |
| Liu | nikeshm: I will see it and do the optimization if if's OK | 10:43 |
| deepakcs | e0ne: so i did this : http://paste.fedoraproject.org/249291/14381642/ (see line 5 & 6) | 10:44 |
| nikeshm | Liu: donot mind, what should i use he/she for you? | 10:44 |
| e0ne | deepakcs: maybe making pk as uuid was a bad idea | 10:44 |
| Liu | he, thanks | 10:44 |
| deepakcs | e0ne: even after that, sql error happens saying during update it is not finding 'id' field | 10:44 |
| nikeshm | Liu: ok, thanks | 10:44 |
| deepakcs | e0ne: shall i revert back to Integer then ? | 10:44 |
| e0ne | deepakcs: let's do it for this time | 10:44 |
| e0ne | deepakcs: i won't -1 for it | 10:45 |
| deepakcs | e0ne: can u pls put that in https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/migrate_repo/versions/049_add_snapshot_admin_metadata_table.py | 10:45 |
| nikeshm | Liu: which city are you working? | 10:45 |
| e0ne | deepakcs: i need more time to investigate this issue | 10:45 |
| deepakcs | e0ne: ok, but i can proceed by usign Integer, thats fine right ? | 10:45 |
| e0ne | deepakcs: yes | 10:45 |
| deepakcs | e0ne: pls update ur comment in https://review.openstack.org/#/c/203979/1/cinder/db/sqlalchemy/migrate_repo/versions/049_add_snapshot_admin_metadata_table.py | 10:46 |
| deepakcs | :) | 10:46 |
| e0ne | deepakcs: done:) | 10:46 |
| Liu | nikeshm: chengdu, a beaufiful chinese city | 10:46 |
| deepakcs | e0ne: thanks, i was debuggign this for most of my today :( | 10:46 |
| e0ne | deepakcs: i review your patch again after you'll fix other comments | 10:46 |
| deepakcs | e0ne: wasted a lot of time | 10:46 |
| nikeshm | Liu: nice | 10:46 |
| deepakcs | e0ne: yes, willl do | 10:46 |
| e0ne | deepakcs: it was my fail too:( | 10:47 |
| Liu | nikeshm: and what about you | 10:47 |
| deepakcs | e0ne: its ok, it happens :) | 10:47 |
| nikeshm | Liu: i work in hyderabad, INDIA | 10:48 |
| e0ne | deepakcs: in cinder, we need some notes or even guidelines what do we want for PKs: int or uuid | 10:48 |
| Liu | nikeshm: Nice place | 10:48 |
| e0ne | deepakcs: now, it's very confusing to have both of them w/o any algorithm to choose one | 10:48 |
| deepakcs | e0ne: yes, and if its uuid, and if its a lazy loaded (like metadata or admin_metadata) how to handle it, bcos it doesn't work with what i just did | 10:48 |
| nikeshm | Liu: my hometown is near to Varanasi,Uttar Pradesh,India | 10:49 |
| deepakcs | e0ne: it should logically have worke with what i did tho :( so i would be interested to know if u root cause it as to why its not working | 10:49 |
| *** lennyb_ has quit IRC | 10:49 | |
| e0ne | deepakcs: yep. it's very interesting for me to find what gone wrong. i'll try to find time for it | 10:49 |
| deepakcs | e0ne: sure | 10:49 |
| Liu | nikeshm: hah, very nick | 10:49 |
| Liu | nikeshm:s/nick/nice | 10:50 |
| Liu | nikeshm: I must leave for dinner | 10:50 |
| deepakcs | e0ne: just as FYI, i also get this error: "2015-07-29 10:50:48.381 ERROR cinder.api.middleware.fault [req-0e17993b-8e5a-4581-a425-58e82ef65f0d 4702a0f683f14183a55cd0e1d6a6c68f 3d53b60a4b154988a851e8b887231946] Caught error: SQL expression, column, or mapped entity expected - got 'True' | 10:51 |
| deepakcs | " | 10:51 |
| deepakcs | e0ne: when i try to delete the cinder volume | 10:51 |
| deepakcs | e0ne: not sure if it helps in root causing | 10:51 |
| Liu | nikeshm: it's almost evening now in my time zone | 10:51 |
| *** markus_z has joined #openstack-cinder | 10:53 | |
| *** coolsvap is now known as coolsvap|away | 10:55 | |
| ub | hey guys! One week ago i fixed a little bug and it's still not merged. I think one core reviewer (+2) is missing. Did i forgot something? Is it normal? https://review.openstack.org/#/c/204702 | 11:02 |
| openstackgerrit | Michal Dulko proposed openstack/cinder-specs: Add Cinder Volume A/A HA lock spec https://review.openstack.org/202615 | 11:03 |
| dulek | ub: It's completely normal, core reviewers time is a limited resource. Unfortunately you need to be patient on that. | 11:04 |
| *** annashen has joined #openstack-cinder | 11:04 | |
| *** annashen has quit IRC | 11:09 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Restore backup to Vol ID when it doesn't exist https://review.openstack.org/191145 | 11:10 |
| openstackgerrit | Pranali Deore proposed openstack/cinder: Validate string, integer limit for input parameter https://review.openstack.org/194968 | 11:18 |
| *** jasondot_ has joined #openstack-cinder | 11:23 | |
| sgotliv_ | tbarron, good morning! yt? | 11:24 |
| *** e0ne has quit IRC | 11:25 | |
| tbarron | sgotliv_: yes, good afternoon? | 11:25 |
| sgotliv_ | tbarron, yes :-) afternoon. I have a quick question about the NetApp driver | 11:25 |
| *** jasondotstar has quit IRC | 11:25 | |
| tbarron | sgotliv_: sure | 11:25 |
| *** zhonghua has quit IRC | 11:26 | |
| sgotliv_ | tbarron, One of my customers gets this error NaApiError: NetApp api failed. Reason - 17163:Clone operation failed to start: Max Dense limit is reached..\n'] | 11:26 |
| sgotliv_ | tbarron, I only can guess what does it mean | 11:27 |
| tbarron | sgotliv_: I can't defend that message :-) | 11:28 |
| tbarron | sgotliv_: it means that the filers hit a limit on the number of clones they could do in that burst or clone deletes | 11:28 |
| tbarron | I've seen it on clone deletes. | 11:29 |
| tbarron | I think some deferred COW work has to be done and there is a rate limit. | 11:29 |
| * sgotliv_ processing :-) | 11:29 | |
| tbarron | sgotliv_: if you want I can look at logs and see if I can tell you something less speculative. | 11:30 |
| sgotliv_ | tbarron, thanks, let me check if I can send you logs. All these security measures are very restrictive | 11:31 |
| tbarron | sgotliv_: sure, understood. | 11:31 |
| *** DTadrzak has quit IRC | 11:31 | |
| *** belmoreira has joined #openstack-cinder | 11:32 | |
| tbarron | sgotliv_: if they are also netapp customer ... | 11:32 |
| sgotliv_ | tbarron, I already thought about it a sec ago :-) | 11:32 |
| sgotliv_ | i want to find a way to add you to the loop | 11:32 |
| sgotliv_ | but meantime, you are right! They are trying to create a lot of snapshots | 11:33 |
| *** cdelatte has joined #openstack-cinder | 11:33 | |
| *** delattec has joined #openstack-cinder | 11:33 | |
| tbarron | sgotliv_: cinder snapshots we implement as file (or LUN) clones | 11:33 |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 11:34 |
| *** e0ne has joined #openstack-cinder | 11:35 | |
| *** rushiagr_away has joined #openstack-cinder | 11:37 | |
| *** delatte has joined #openstack-cinder | 11:40 | |
| deepakcs | e0ne: ping | 11:40 |
| *** delattec has quit IRC | 11:41 | |
| *** cdelatte has quit IRC | 11:42 | |
| *** shyama has quit IRC | 11:46 | |
| *** bswartz has quit IRC | 11:46 | |
| *** jwcroppe has joined #openstack-cinder | 11:47 | |
| *** delattec has joined #openstack-cinder | 11:48 | |
| *** jwcroppe has quit IRC | 11:48 | |
| *** svilgelm has joined #openstack-cinder | 11:49 | |
| *** delatte has quit IRC | 11:51 | |
| *** coolsvap|away is now known as coolsvap | 11:51 | |
| *** kodoku_ has joined #openstack-cinder | 11:52 | |
| *** svilgelm has left #openstack-cinder | 11:52 | |
| *** svilgelm has joined #openstack-cinder | 11:52 | |
| kodoku_ | Hi, I have issue when I attach volume ==> Unable to fetch connection information from backend: 'NoneType' object has no attribute 'service_content' to caller | 11:58 |
| kodoku_ | I use KILO version. python-cinderclient-1.1.1-1.el7.noarchpython-cinder-2015.1.0-3.el7.noarchopenstack-cinder-2015.1.0-3.el7.noarch | 11:59 |
| *** e0ne has quit IRC | 11:59 | |
| kodoku_ | Somebody have idea ? | 12:00 |
| ccesario | Hello, does someone have idea about this SSL error in cinder-volume service ? http://paste.openstack.org/show/yjMUfLKHourACMphlNVt/ | 12:02 |
| *** annashen has joined #openstack-cinder | 12:02 | |
| *** annashen_ has joined #openstack-cinder | 12:05 | |
| *** aix has joined #openstack-cinder | 12:07 | |
| *** annashen has quit IRC | 12:07 | |
| *** Lee1092 has quit IRC | 12:07 | |
| *** annashen_ has quit IRC | 12:10 | |
| *** svilgelm has quit IRC | 12:10 | |
| *** gaurangt has quit IRC | 12:14 | |
| *** erlon has joined #openstack-cinder | 12:16 | |
| *** ByPasS has joined #openstack-cinder | 12:16 | |
| *** marcusvrn has joined #openstack-cinder | 12:16 | |
| *** coolsvap is now known as coolsvap|away | 12:17 | |
| *** kodoku_ has quit IRC | 12:19 | |
| *** kodoku has joined #openstack-cinder | 12:19 | |
| *** kaisers_ has joined #openstack-cinder | 12:19 | |
| kodoku | ccesario hI | 12:19 |
| kodoku | ccesario cinder connect to vcenter with https ? | 12:20 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 12:21 |
| ccesario | kodoku, hi,.. yes its try.... with nova service I solve it with insecure = True .... but with cinder I dot not know how solve it :/ | 12:21 |
| *** salv-orl_ has joined #openstack-cinder | 12:22 | |
| kodoku | ccesario You can add your CA in ca-trust on your control node | 12:22 |
| *** xek_ is now known as xek | 12:23 | |
| *** akerr has joined #openstack-cinder | 12:24 | |
| *** avishay has quit IRC | 12:24 | |
| *** salv-orlando has quit IRC | 12:25 | |
| *** e0ne has joined #openstack-cinder | 12:25 | |
| *** enikher has joined #openstack-cinder | 12:25 | |
| ccesario | kodoku, insecure mode is not possible with cinder ? | 12:26 |
| *** salv-orlando has joined #openstack-cinder | 12:26 | |
| e0ne | deepakcs: ping | 12:26 |
| *** salv-orl_ has quit IRC | 12:26 | |
| kodoku | ccesario I use cinder with https vcenter, I don't see insecure option :/ But I don't add CA in ca-trust | 12:26 |
| *** edmondsw has joined #openstack-cinder | 12:27 | |
| *** salv-orl_ has joined #openstack-cinder | 12:27 | |
| *** kaisers_ has quit IRC | 12:29 | |
| *** vlaza has joined #openstack-cinder | 12:29 | |
| *** salv-orlando has quit IRC | 12:30 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 12:30 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how deploy API under Apache https://review.openstack.org/207020 | 12:30 |
| *** bluex-pl has quit IRC | 12:30 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how deploy API under Apache https://review.openstack.org/207020 | 12:31 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 12:31 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add instructions on how deploy API under Apache https://review.openstack.org/207020 | 12:33 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add Cinder API WSGI app to run under Apache/Nginx https://review.openstack.org/200551 | 12:33 |
| *** deepakcs has quit IRC | 12:33 | |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 12:34 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: db: Add support for snapshot_admin_metadata https://review.openstack.org/203979 | 12:34 |
| *** salv-orlando has joined #openstack-cinder | 12:35 | |
| *** ganso_ has joined #openstack-cinder | 12:35 | |
| *** ishant has quit IRC | 12:36 | |
| *** salv-orl_ has quit IRC | 12:38 | |
| *** Tross has quit IRC | 12:39 | |
| *** Tross1 has joined #openstack-cinder | 12:39 | |
| *** salv-orl_ has joined #openstack-cinder | 12:44 | |
| ccesario | kodoku, is it possible you show me your cinder.conf ? | 12:44 |
| kodoku | ccesario Yes, one second | 12:46 |
| ccesario | sure | 12:46 |
| *** salv-orlando has quit IRC | 12:47 | |
| *** salv-orl_ has quit IRC | 12:47 | |
| *** coolsvap|away is now known as coolsvap | 12:48 | |
| *** salv-orlando has joined #openstack-cinder | 12:48 | |
| *** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder" | 12:48 | |
| -openstackstatus- NOTICE: zuul's disks were at capacity. Space has been freed up and jobs are being re-queued. | 12:48 | |
| *** bswartz has joined #openstack-cinder | 12:49 | |
| kodoku | ccesario Send by pm | 12:49 |
| *** shyama has joined #openstack-cinder | 12:53 | |
| ccesario | kodoku, thank you.... let me check | 12:55 |
| *** dims has quit IRC | 12:57 | |
| *** salv-orl_ has joined #openstack-cinder | 12:57 | |
| *** annegentle has joined #openstack-cinder | 12:57 | |
| *** dims has joined #openstack-cinder | 12:57 | |
| *** mriedem has joined #openstack-cinder | 12:57 | |
| *** salv-orlando has quit IRC | 13:00 | |
| *** salv-or__ has joined #openstack-cinder | 13:00 | |
| *** zhipeng has joined #openstack-cinder | 13:00 | |
| *** liuxg has joined #openstack-cinder | 13:03 | |
| *** salv-orl_ has quit IRC | 13:04 | |
| ccesario | kodoku, it is similar.... with mime .... | 13:04 |
| *** Yogi1 has joined #openstack-cinder | 13:05 | |
| kodoku | ccesario Hum I don't because I use auto sign for my vcenter | 13:05 |
| *** annashen has joined #openstack-cinder | 13:06 | |
| ccesario | I will try search any wayt to use insecure mode to it | 13:06 |
| ccesario | auto sign!? | 13:07 |
| kodoku | ccesario If I curl on https://myvcenter I have "Use -k because this cert is not sign" | 13:08 |
| kodoku | But cinder works :/ | 13:08 |
| kodoku | What is your version of cinder ? | 13:09 |
| *** salv-or__ has quit IRC | 13:09 | |
| *** annashen has quit IRC | 13:10 | |
| *** salv-orlando has joined #openstack-cinder | 13:11 | |
| *** coolsvap is now known as coolsvap|away | 13:14 | |
| *** salv-orl_ has joined #openstack-cinder | 13:14 | |
| *** salv-orlando has quit IRC | 13:17 | |
| *** kaisers_ has joined #openstack-cinder | 13:17 | |
| *** markstur_ has joined #openstack-cinder | 13:18 | |
| *** salv-orlando has joined #openstack-cinder | 13:18 | |
| ccesario | kodoku, I 'm testing with devstack | 13:19 |
| ccesario | cinder-volume --version | 13:19 |
| ccesario | 7.0.0 | 13:19 |
| *** markstur_ has left #openstack-cinder | 13:19 | |
| ccesario | cinder --version | 13:20 |
| ccesario | 1.3.1 | 13:20 |
| *** salv-orl_ has quit IRC | 13:21 | |
| openstackgerrit | Joel Coffman proposed openstack/cinder-specs: Add volume encryption metadata spec https://review.openstack.org/176050 | 13:22 |
| *** eharney has joined #openstack-cinder | 13:23 | |
| -openstackstatus- NOTICE: zuul jobs after about 07:00 UTC may need a 'recheck' to enter the queue. Look if your change is in http://status.openstack.org/zuul/ and recheck if not. | 13:26 | |
| *** akshai has joined #openstack-cinder | 13:26 | |
| *** BharatK has quit IRC | 13:29 | |
| *** kodoku has quit IRC | 13:30 | |
| *** kaisers_ has quit IRC | 13:34 | |
| *** kaisers_ has joined #openstack-cinder | 13:36 | |
| *** annegentle has quit IRC | 13:36 | |
| *** abhishekkk has quit IRC | 13:39 | |
| *** DTadrzak has joined #openstack-cinder | 13:39 | |
| *** kaisers_ has quit IRC | 13:41 | |
| *** jwcroppe has joined #openstack-cinder | 13:43 | |
| *** BharatK has joined #openstack-cinder | 13:43 | |
| *** markvoelker has joined #openstack-cinder | 13:44 | |
| *** jasondotstar has joined #openstack-cinder | 13:48 | |
| *** jungleboyj has quit IRC | 13:52 | |
| *** salv-orlando has quit IRC | 13:52 | |
| *** jasondot_ has quit IRC | 13:52 | |
| *** timcl has quit IRC | 13:54 | |
| *** mragupat has joined #openstack-cinder | 13:59 | |
| *** mragupat has quit IRC | 13:59 | |
| *** breitz has joined #openstack-cinder | 13:59 | |
| *** thangp has joined #openstack-cinder | 13:59 | |
| *** timcl has joined #openstack-cinder | 14:00 | |
| *** mragupat has joined #openstack-cinder | 14:00 | |
| *** mylu has joined #openstack-cinder | 14:01 | |
| *** annashen has joined #openstack-cinder | 14:03 | |
| *** markvoelker has quit IRC | 14:04 | |
| *** lpetrut has quit IRC | 14:04 | |
| *** EmilienM has quit IRC | 14:05 | |
| *** lpetrut has joined #openstack-cinder | 14:06 | |
| *** annashen_ has joined #openstack-cinder | 14:07 | |
| *** annashen has quit IRC | 14:08 | |
| *** zhipeng has quit IRC | 14:08 | |
| *** EmilienM has joined #openstack-cinder | 14:09 | |
| *** mylu has quit IRC | 14:11 | |
| *** annashen_ has quit IRC | 14:11 | |
| *** mylu has joined #openstack-cinder | 14:12 | |
| *** ankit_ag has quit IRC | 14:13 | |
| *** annegentle has joined #openstack-cinder | 14:14 | |
| *** Miouge has joined #openstack-cinder | 14:18 | |
| *** markvoelker_ has joined #openstack-cinder | 14:21 | |
| *** vlaza has left #openstack-cinder | 14:24 | |
| *** mtanino has joined #openstack-cinder | 14:25 | |
| *** r-daneel has joined #openstack-cinder | 14:26 | |
| *** kodoku has joined #openstack-cinder | 14:31 | |
| *** lpetrut has quit IRC | 14:32 | |
| *** lpetrut has joined #openstack-cinder | 14:36 | |
| *** coolsvap|away is now known as coolsvap | 14:37 | |
| *** markvoelker_ has quit IRC | 14:38 | |
| *** markvoelker has joined #openstack-cinder | 14:39 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS iSCSI 32 targets limitation error https://review.openstack.org/206651 | 14:40 |
| *** kjelly_ has joined #openstack-cinder | 14:45 | |
| *** markvoelker has quit IRC | 14:47 | |
| *** markvoelker has joined #openstack-cinder | 14:47 | |
| *** crose has joined #openstack-cinder | 14:48 | |
| *** Guest100_ has joined #openstack-cinder | 14:51 | |
| *** esker has joined #openstack-cinder | 14:54 | |
| *** mdenny has joined #openstack-cinder | 14:56 | |
| kodoku | Hi, I have issue with cinder (KILO), I can create volume, attache fresh volume, but after detach, I can't re-attach this volume ===> VolumeBackendAPIException: Bad or unexpected response from the storage volume backend API: Unable to fetch connection information from backend: 'NoneType' object has no attribute 'service_content' | 14:56 |
| thingee | kodoku: I'm assuming this is vmware? | 14:57 |
| *** kodoku_ has joined #openstack-cinder | 14:58 | |
| kodoku_ | thingee yes | 14:58 |
| *** jwcroppe has quit IRC | 14:58 | |
| *** belmoreira has quit IRC | 14:59 | |
| *** jasondotstar has quit IRC | 14:59 | |
| *** tsekiyama has joined #openstack-cinder | 14:59 | |
| kodoku_ | thingee Do you know this issue ? | 14:59 |
| thangp | didnt kilo reached EOL? | 14:59 |
| thingee | kodoku_: I'm not associate with vmware, I only assumed it's vmware because they're the only one with service_content in their code. | 14:59 |
| kodoku_ | ok ^^ | 15:00 |
| thingee | kodoku_: I don't know if their driver maintainers hang out here though | 15:00 |
| kodoku_ | I guess it's vipin | 15:00 |
| thingee | vipin balachandran is the maintainer | 15:00 |
| kodoku_ | :p | 15:00 |
| *** jistr has quit IRC | 15:00 | |
| *** kodoku has quit IRC | 15:01 | |
| *** ByPasS has quit IRC | 15:01 | |
| *** haomaiwang has joined #openstack-cinder | 15:01 | |
| *** mragupat has quit IRC | 15:01 | |
| *** e0ne has quit IRC | 15:02 | |
| *** rex_lee has quit IRC | 15:02 | |
| *** sweston has quit IRC | 15:02 | |
| *** Adriano__ has quit IRC | 15:02 | |
| thingee | kodoku_: I looked through their attach related bugs here https://bugs.launchpad.net/cinder/+bugs?field.tag=vmware, and didn't find anything. You mind want to consider reporting something | 15:02 |
| *** jistr has joined #openstack-cinder | 15:02 | |
| *** ByPasS has joined #openstack-cinder | 15:02 | |
| *** haomaiwang has quit IRC | 15:03 | |
| * thingee disappears a bit for meeting | 15:03 | |
| *** edtubill has joined #openstack-cinder | 15:03 | |
| *** haomaiwang has joined #openstack-cinder | 15:03 | |
| kodoku_ | thingee ok I open bug, but on vmware team or cinder ? | 15:04 |
| *** e0ne has joined #openstack-cinder | 15:04 | |
| thingee | kodoku_: you can report the bug and tag it vmware to get their attention | 15:04 |
| thingee | kodoku_: if they have their notifications setup correctly, they'll know | 15:04 |
| thingee | if you want to use other means of letting them know about the bug, that'll be fine too | 15:05 |
| *** nkrinner has quit IRC | 15:05 | |
| *** IlyaG has joined #openstack-cinder | 15:06 | |
| *** annashen has joined #openstack-cinder | 15:07 | |
| *** Lee1092 has joined #openstack-cinder | 15:08 | |
| kodoku_ | thingee https://bugs.launchpad.net/oslo.vmware/+bug/1479402 | 15:09 |
| openstack | Launchpad bug 1479402 in oslo.vmware "Can't re-attach volume : 'NoneType' object has no attribute 'service_content'" [Undecided,New] | 15:09 |
| kodoku_ | :) | 15:09 |
| breitz | koduku_: from the bug……."But I can Re-Attach this volume :" might want to change that "can to a "can't" | 15:12 |
| *** annashen has quit IRC | 15:13 | |
| *** svilgelm has joined #openstack-cinder | 15:14 | |
| kodoku_ | arf | 15:15 |
| kodoku_ | can't >< | 15:15 |
| kodoku_ | edit :) | 15:15 |
| *** mylu has quit IRC | 15:15 | |
| breitz | it is easy to figure out from the title and the error msg - but if it is changed (as you have done) it makes life a little easier. Thanks for doing that. | 15:16 |
| *** mylu has joined #openstack-cinder | 15:17 | |
| kodoku_ | breitz yes of course :) | 15:18 |
| kjelly_ | Hi guys, If there are many ways to solve a bug/issue, where is the best place to discuss the solution? | 15:19 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: Cinder part of volume snapshot improvements https://review.openstack.org/172808 | 15:19 |
| openstackgerrit | Deepak C Shetty (deepakcs) proposed openstack/cinder: db: Add support for snapshot_admin_metadata https://review.openstack.org/203979 | 15:19 |
| openstackgerrit | Ryan Rossiter proposed openstack/os-brick: Changed connector protocols to use constants https://review.openstack.org/207090 | 15:19 |
| *** ronis has quit IRC | 15:20 | |
| scottda | kjelly_: you could put the alternatives in the launchpad bug report. You could also post a patch with your preferred solution and make note in a comment on the review that alternatives exist (in the bug report). | 15:21 |
| scottda | If someone comments on either, you could ping them here. Or you could just post the info here and see who responds. | 15:21 |
| kjelly_ | scottda: Thank you for your suggestion. | 15:22 |
| *** tsekiyama has quit IRC | 15:23 | |
| scottda | sure. It kind of depends on the level of detail. Just ask here if it's simple. | 15:23 |
| Swanson | Are we getting any successful CI runs? | 15:24 |
| Swanson | Wrong window. | 15:24 |
| *** Miouge has quit IRC | 15:25 | |
| *** tsekiyama has joined #openstack-cinder | 15:25 | |
| smcginnis | Swanson: Haha, yes, some. :) | 15:27 |
| smcginnis | Swanson: Seems to be something going on that started yesterday morning. Haven't had time to dig in to it. | 15:27 |
| openstackgerrit | Kendall Nelson proposed openstack/cinder: Enable cinder-manage to remove services https://review.openstack.org/197613 | 15:28 |
| thingee | Swanson: lol | 15:28 |
| kjelly_ | scottda: Ok. I am working on the bug. bug/1477625 . There are many ways to solve the problem. The simple way is https://review.openstack.org/#/c/206450/1 . Another way is change the interface https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L407 . And don't use context.elevated() https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L404 . Any suggestion for the bug ? | 15:28 |
| smcginnis | With split screen irissi I'm just waiting for the day when I accidentally put something private in channel. | 15:28 |
| smcginnis | I've certainly done the reverse plenty of times. | 15:29 |
| scottda | kjelly_: I'll look in a minute, I've a meeting starting ATM.... | 15:29 |
| thingee | smcginnis: can't be as bad when I accidentally announced I was leaving my previous company. | 15:29 |
| kjelly_ | scottda: Ok. Thank you. | 15:29 |
| smcginnis | thingee: Doh! That could be a little awkward. | 15:30 |
| thingee | irssi, pasting a message with a return in it. | 15:30 |
| *** jdurgin1 has joined #openstack-cinder | 15:30 | |
| *** alexpilotti has quit IRC | 15:34 | |
| tbarron | thingee: that's one way to give notice :) | 15:35 |
| guitarzan | kjelly_: it needs to be changed to not use context elevated for the volume_get() | 15:35 |
| guitarzan | kjelly_: and then later use context elevated to fetch the viewable admin metadata in a second db call | 15:35 |
| Swanson | thingee: nice! | 15:35 |
| *** esker has quit IRC | 15:39 | |
| *** diogogmt has joined #openstack-cinder | 15:40 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Remove Cinder API races https://review.openstack.org/207101 | 15:41 |
| *** dannywilson has joined #openstack-cinder | 15:42 | |
| *** lpetrut has quit IRC | 15:43 | |
| *** lpetrut has joined #openstack-cinder | 15:43 | |
| kjelly_ | guitarzan: Thank you for your suggestion. This seems to be a good solution. What do you think about letting add another argument to self.db.volume_get for limiting get volume in the same project ? | 15:43 |
| *** harlowja_at_home has joined #openstack-cinder | 15:44 | |
| guitarzan | kjelly_: I don't quite understand | 15:45 |
| guitarzan | what would the flag do? | 15:45 |
| guitarzan | db.volume_get should always limit getting the volume to the same project or an admin context | 15:45 |
| *** jungleboyj has joined #openstack-cinder | 15:46 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Fix Oracle ZFSSA iSCSI driver to return free space https://review.openstack.org/207106 | 15:47 |
| kjelly_ | guitarzan: but becasue the code "context.elevated()", all user is admin context. | 15:47 |
| guitarzan | kjelly_: it's entirely possible I don't understand what you're saying though :) | 15:47 |
| guitarzan | kjelly_: that is the bug :) | 15:47 |
| guitarzan | it should not be using context.elevated for volume.get() | 15:47 |
| openstackgerrit | Jon Bernard proposed openstack/cinder: RBD: use user-configured value for chunk size https://review.openstack.org/207107 | 15:48 |
| kjelly_ | guitarzan: yes. I agree your solution. I will use your suggestion to solve the bugs. Thank you. | 15:48 |
| *** rhagarty_ has quit IRC | 15:48 | |
| *** rhagarty has quit IRC | 15:48 | |
| guitarzan | kjelly_: looking forward to seeing it | 15:49 |
| *** esker has joined #openstack-cinder | 15:50 | |
| *** esker has quit IRC | 15:50 | |
| *** jseiler has joined #openstack-cinder | 15:53 | |
| *** patrickeast has joined #openstack-cinder | 15:53 | |
| *** esker has joined #openstack-cinder | 15:54 | |
| *** esker has quit IRC | 15:54 | |
| *** vincent_hou has joined #openstack-cinder | 15:56 | |
| *** adurbin_ has joined #openstack-cinder | 15:56 | |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Efficient image transfer for Glance cinder store https://review.openstack.org/201754 | 15:56 |
| openstackgerrit | Tomoki Sekiyama proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 15:57 |
| *** rhagarty has joined #openstack-cinder | 15:58 | |
| *** rhagarty_ has joined #openstack-cinder | 15:58 | |
| *** ub has quit IRC | 15:58 | |
| *** haomaiwang has quit IRC | 16:01 | |
| *** lpetrut has quit IRC | 16:01 | |
| kjelly_ | scottda: It's time for me to sleep. And thank you for your help. If you have any suggesion, plaease comment in https://review.openstack.org/#/c/206450/ or https://bugs.launchpad.net/cinder/+bug/1477625 . Tomorrow, I will try the solution suggested by guitarzan. "it needs to be changed to not use context elevated for the volume_get() and then later use context elevated to fetch the viewable admin metadata in a second db call" | 16:01 |
| openstack | Launchpad bug 1477625 in Cinder "DB access to show volumes may not be properly controlled" [Undecided,In progress] - Assigned to jelly (coding1314) | 16:01 |
| *** lpetrut has joined #openstack-cinder | 16:01 | |
| *** esker has joined #openstack-cinder | 16:01 | |
| *** svilgelm has quit IRC | 16:02 | |
| scottda | kjelly_: I think guitarzan has you pointed in the right direction. | 16:02 |
| *** haomaiwang has joined #openstack-cinder | 16:02 | |
| jungleboyj | thingee: Are we meeting? | 16:02 |
| kjelly_ | scottda: Ok, I will follow your suggesion. Thanks. | 16:03 |
| *** kjelly_ has quit IRC | 16:04 | |
| *** unibeard has joined #openstack-cinder | 16:04 | |
| *** annashen has joined #openstack-cinder | 16:04 | |
| *** Yogi1 has quit IRC | 16:04 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Add Cinder Volume A/A HA lock spec https://review.openstack.org/202615 | 16:05 |
| *** vilobhmm has joined #openstack-cinder | 16:05 | |
| *** angela-s has joined #openstack-cinder | 16:06 | |
| *** garthb has joined #openstack-cinder | 16:06 | |
| *** garthb_ has joined #openstack-cinder | 16:06 | |
| *** garthb_ has quit IRC | 16:07 | |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 16:07 |
| *** jistr has quit IRC | 16:08 | |
| *** annashen_ has joined #openstack-cinder | 16:08 | |
| *** annashen has quit IRC | 16:09 | |
| e0ne | geguileo: hi | 16:09 |
| geguileo | e0ne: Hi | 16:09 |
| e0ne | geguileo: did you see my comments for https://bugs.launchpad.net/cinder/+bug/1476416? | 16:10 |
| openstack | Launchpad bug 1476416 in Cinder "backup.api:import_record sets volume_id to '0000-0000-0000-0000'" [High,In progress] - Assigned to Ivan Kolodyazhny (e0ne) | 16:10 |
| geguileo | e0ne: No I haven't | 16:10 |
| geguileo | e0ne: Today's been crazy for me | 16:10 |
| geguileo | e0ne: Let me have a look now | 16:10 |
| *** unibeard has left #openstack-cinder | 16:10 | |
| *** thangp has quit IRC | 16:11 | |
| *** daneyon has joined #openstack-cinder | 16:11 | |
| geguileo | e0ne: With my patch it gets fixed | 16:11 |
| *** Apoorva has joined #openstack-cinder | 16:11 | |
| geguileo | e0ne: For real | 16:11 |
| e0ne | geguileo: no | 16:11 |
| geguileo | e0ne: Because I restore the original Backup ID | 16:11 |
| e0ne | geguileo: it's true only if we've got it in the db | 16:12 |
| *** jaypipes has joined #openstack-cinder | 16:12 | |
| geguileo | e0ne: What do you mean if we've got it in the DB? | 16:12 |
| *** coolsvap is now known as coolsvap|away | 16:12 | |
| geguileo | I haven't tested the code in a while, so I may be missing something | 16:13 |
| e0ne | geguileo: and after your patch i can't import backup before original is deleted | 16:13 |
| *** scottda has quit IRC | 16:13 | |
| *** ctracey has quit IRC | 16:13 | |
| *** briancurtin has quit IRC | 16:13 | |
| *** zhiyan has quit IRC | 16:13 | |
| *** ameade has quit IRC | 16:13 | |
| *** xyang has quit IRC | 16:13 | |
| *** wanghao has quit IRC | 16:13 | |
| *** thingee has quit IRC | 16:13 | |
| *** rhefner has quit IRC | 16:13 | |
| *** ganso_ has quit IRC | 16:13 | |
| *** serverascode has quit IRC | 16:13 | |
| *** annashen_ has quit IRC | 16:13 | |
| geguileo | e0ne: Of course you can't import something that's already there | 16:13 |
| geguileo | e0ne: And is still alive and kicking | 16:13 |
| e0ne | geguileo: operators could purge raws from db which are marked as deleted | 16:13 |
| geguileo | e0ne: Ok, and when the entry is not there you're saying it doesn't work? | 16:14 |
| e0ne | geguileo: yes | 16:14 |
| geguileo | e0ne: Because it should, unless I've made a mistake in one of the latest patches | 16:14 |
| mriedem | earlephilhower: i forgot to tell you that i got that series rebased last night | 16:14 |
| geguileo | e0ne: Ok, the patch should fix, and before I had to completely redo the patch 3 or 4 times because people didn't like it | 16:14 |
| geguileo | s/fix/work | 16:14 |
| geguileo | e0ne: It used to work | 16:15 |
| mriedem | i see that xing-yang already rebased the scaleio shim on top of it | 16:15 |
| geguileo | e0ne: I'll fix it today | 16:15 |
| geguileo | e0ne: Because it must be something silly with current import mechanism I have | 16:15 |
| *** harlowja_at_home has quit IRC | 16:15 | |
| e0ne | geguileo: why do you think that restoring backup is a good idea? | 16:15 |
| *** thangp has joined #openstack-cinder | 16:16 | |
| geguileo | e0ne: What do you mean restoring backup? | 16:16 |
| e0ne | geguileo: i mean restore backup io | 16:16 |
| geguileo | e0ne: You mean the ID | 16:16 |
| e0ne | s/io/id | 16:16 |
| geguileo | e0ne: Because if you don't not only do you break Ceph backend | 16:16 |
| geguileo | e0ne: But you also break Incremental backup I believe | 16:16 |
| e0ne | geguileo: you're right | 16:16 |
| geguileo | e0ne: Because parent-id will be referring to a non-existen backup id | 16:16 |
| geguileo | e0ne: Otherwise I would have just changed Ceph | 16:17 |
| geguileo | e0ne: But since this affects incremental | 16:17 |
| e0ne | geguileo: but we can remove backup id from the image name | 16:17 |
| geguileo | e0ne: I'd rather leave Ceph as it is and fix the root | 16:17 |
| geguileo | e0ne: Then you break backwards compatibility :( | 16:17 |
| *** leeantho has joined #openstack-cinder | 16:17 | |
| e0ne | geguileo: yes:( | 16:17 |
| *** lpetrut has quit IRC | 16:17 | |
| geguileo | e0ne: And that's a tough sell XD | 16:17 |
| *** xyang1 has joined #openstack-cinder | 16:18 | |
| geguileo | e0ne: I'll work on fixing it after the meeting is over | 16:18 |
| geguileo | e0ne: And ping you once it's resolved | 16:18 |
| *** david-lyle has quit IRC | 16:18 | |
| *** kodoku_ has quit IRC | 16:19 | |
| e0ne | geguileo: thanks | 16:20 |
| e0ne | geguileo: i've just tested you patch again | 16:20 |
| geguileo | e0ne: And it still fails? | 16:20 |
| e0ne | geguileo: restore doesn't work | 16:20 |
| geguileo | e0ne: Ok, I'll fix it | 16:20 |
| geguileo | e0ne: And thank you for testing it and pointing out the problem | 16:21 |
| *** thingee has joined #openstack-cinder | 16:21 | |
| e0ne | geguileo: create volume, backup, export backup, delete original backup, import backup, try to restore imported backup | 16:21 |
| geguileo | e0ne: I should have done it myself, but I got tired after the 3rd time I completely redid the fix | 16:21 |
| *** vokt has joined #openstack-cinder | 16:21 | |
| geguileo | e0ne: Yep, that's the process | 16:21 |
| geguileo | e0ne: Thanks | 16:22 |
| e0ne | geguileo: thanks for fixing it. i'm wating for a new patch | 16:22 |
| *** sree has joined #openstack-cinder | 16:22 | |
| *** belmoreira has joined #openstack-cinder | 16:23 | |
| markus_z | Could a second core have a look at https://review.openstack.org/#/c/203026/ please? | 16:24 |
| openstackgerrit | Patrick East proposed openstack/cinder: Generic image-volume cache https://review.openstack.org/195795 | 16:26 |
| *** afazekas has quit IRC | 16:27 | |
| *** sree has quit IRC | 16:27 | |
| *** agarciam has quit IRC | 16:28 | |
| *** markus_z has quit IRC | 16:29 | |
| *** sweston has joined #openstack-cinder | 16:33 | |
| e0ne | geguileo: i'm going like to ask question about restoring backup is in openstack-dev ml. do you have any concerns? | 16:34 |
| geguileo | e0ne: What is there to ask? r:-?? | 16:35 |
| e0ne | geguileo: i think that id should be one time thing | 16:35 |
| geguileo | e0ne: Sure, everyone says that | 16:36 |
| geguileo | e0ne: But then you need an additional field to store an ID for backups | 16:36 |
| e0ne | geguileo: and you re-use it | 16:36 |
| geguileo | e0ne: And that ID must be unique | 16:36 |
| geguileo | e0ne: And it cannot be repeated when you reimport the backup | 16:36 |
| e0ne | geguileo: two IDs is also bad | 16:37 |
| geguileo | e0ne: That new field really, really sound like thebackup ID | 16:37 |
| *** ctracey has joined #openstack-cinder | 16:37 | |
| e0ne | geguileo: and i'm not fan of my idea to break backward compatibility | 16:37 |
| geguileo | e0ne: Well, you would basically break incremental backups as well... | 16:37 |
| *** jdurgin1 has quit IRC | 16:38 | |
| e0ne | geguileo: i need to test it to be sure what things will be broken or not | 16:38 |
| geguileo | e0ne: If you want to... | 16:38 |
| geguileo | e0ne: But parent-id points to backup-id (if I'm not mistaken) | 16:38 |
| geguileo | e0ne: So it's pretty much a done deal | 16:38 |
| *** coolsvap|away is now known as coolsvap | 16:39 | |
| *** svilgelm has joined #openstack-cinder | 16:39 | |
| *** jordanP has quit IRC | 16:39 | |
| e0ne | geguileo: sounds you're right. i don't remember how it is implemented | 16:40 |
| *** serverascode has joined #openstack-cinder | 16:41 | |
| *** david-lyle has joined #openstack-cinder | 16:42 | |
| geguileo | e0ne: I agree that nobody likes to reuse IDs | 16:42 |
| geguileo | e0ne: But if we don't in this case you basically cannot export/import backups | 16:42 |
| *** enikher has quit IRC | 16:43 | |
| *** Apoorva has quit IRC | 16:43 | |
| *** ameade has joined #openstack-cinder | 16:43 | |
| *** Apoorva has joined #openstack-cinder | 16:43 | |
| *** ub has joined #openstack-cinder | 16:43 | |
| geguileo | e0ne: Anyway, a message to dev list is probably the right idea | 16:45 |
| *** alexpilotti has joined #openstack-cinder | 16:45 | |
| *** zhiyan has joined #openstack-cinder | 16:46 | |
| *** ganso_ has joined #openstack-cinder | 16:48 | |
| *** svilgelm has quit IRC | 16:48 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 16:50 |
| *** crose has quit IRC | 16:50 | |
| *** ronis has joined #openstack-cinder | 16:51 | |
| *** Adriano__ has joined #openstack-cinder | 16:52 | |
| *** laughterwym has quit IRC | 16:53 | |
| *** _cjones_ has joined #openstack-cinder | 16:53 | |
| *** _cjones_ has quit IRC | 16:53 | |
| *** _cjones_ has joined #openstack-cinder | 16:53 | |
| *** briancurtin has joined #openstack-cinder | 16:57 | |
| *** lpetrut has joined #openstack-cinder | 16:58 | |
| *** coolsvap is now known as coolsvap|away | 16:58 | |
| *** coolsvap|away is now known as coolsvap | 16:58 | |
| *** scottda__ has joined #openstack-cinder | 16:59 | |
| openstackgerrit | Matthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements https://review.openstack.org/203237 | 17:00 |
| *** haomaiwang has quit IRC | 17:01 | |
| dannywilson | thingee: bp here for trying to enable discard/unmap/trim for Cinder attached volumes https://blueprints.launchpad.net/cinder/+spec/cinder-backend-report-discard | 17:01 |
| dannywilson | pretty simple implementation and spec too | 17:01 |
| dannywilson | thingee: just wanted to get it on your radar, thank you | 17:01 |
| *** haomaiwang has joined #openstack-cinder | 17:02 | |
| *** e0ne has quit IRC | 17:03 | |
| jgriffith | geguileo: thingee got a minute ? | 17:03 |
| geguileo | jgriffith: Yep | 17:03 |
| thingee | jgriffith: yes | 17:03 |
| thingee | dannywilson: cool, it's on my list to look at | 17:03 |
| jgriffith | geguileo: thingee so I'm trying to better understand the A/A issues that folks are trying to solve | 17:04 |
| thingee | dannywilson: if it's already got DuncanT's approval, that's a good sign | 17:04 |
| hemna | dannywilson, can you explain what you mean by "discard"? | 17:04 |
| geguileo | jgriffith: Just for the record, I have an easier solution to the most complex part of my proposal | 17:04 |
| *** avishay has joined #openstack-cinder | 17:04 | |
| *** thingee_ has joined #openstack-cinder | 17:04 | |
| jgriffith | geguileo: thingee in particular IMO the biggest problem seemed to be the DB no? | 17:04 |
| jgriffith | geguileo: LOL... I think you know where I'm going with this then :) | 17:05 |
| geguileo | jgriffith: Not really | 17:05 |
| geguileo | jgriffith: When you say the DB you mean the atomic changes? | 17:05 |
| geguileo | jgriffith: Because that's easy | 17:05 |
| *** rhefner has joined #openstack-cinder | 17:05 | |
| *** rex_lee has joined #openstack-cinder | 17:05 | |
| geguileo | jgriffith: If we don't change Cinder-Nova interactions | 17:05 |
| jgriffith | geguileo: thingee so I don't have any issues or objections to anything... I just wanted to sort of step back and level set on this a bit | 17:05 |
| patrickeast | hemna: its for scsi trim/unmap/discard | 17:06 |
| patrickeast | hemna: lots of names depending on who is talking about it | 17:06 |
| geguileo | jgriffith: Oh, if you have issues they are welcome!! :-) | 17:06 |
| patrickeast | hemna: in the nova side it got referred to as discard i think because of how libvirt enables it | 17:06 |
| thingee | patrickeast: what does it do exactly? | 17:06 |
| geguileo | But I have to write another posibility that it's easier for the job distribution/unstucking of resources | 17:06 |
| dannywilson | hemna: yes, Nova side uses trim, https://en.wikipedia.org/wiki/Trim_(computing) | 17:06 |
| *** scottda has joined #openstack-cinder | 17:06 | |
| jgriffith | geguileo: thingee I've heard a couple of people going on and on about the HA problem for c-vol over the years, but no good explanation of what they want exactly | 17:06 |
| patrickeast | thingee: if i understand correctly it allows for flash disks to do better GC | 17:07 |
| geguileo | jgriffith: I can't tell you what they want, but I can tell you some of the issues ;-) | 17:07 |
| jgriffith | geguileo: well... that's a great first step :) | 17:07 |
| dannywilson | hemna: sorry Nova uses discard for the terminology, so many similar words | 17:07 |
| geguileo | jgriffith: First of all we have to decide how jobs get distributed to the different cluster Nodes | 17:08 |
| *** angela-s has quit IRC | 17:08 | |
| jgriffith | geguileo: how come? | 17:08 |
| thingee | jgriffith: so some deployers like ebay use different cinder backends to the same storage. One c-vol does everything except delete, the other c-vol does just deletes.... and that's because their choice of storage solution is slow on deletes. | 17:08 |
| thingee | jgriffith: but they do out of tree hacks to accomplish this | 17:08 |
| jgriffith | geguileo: what I mean to say is "is that true, do we have to care" | 17:08 |
| thingee | jgriffith: I know you would agree those solution just don't belong | 17:08 |
| jgriffith | geguileo: as long as the db is replicated it shouldn't matter should it? | 17:09 |
| geguileo | jgriffith: Well, it does matter | 17:09 |
| thingee | patrickeast: oh right, thanks | 17:09 |
| *** annashen has joined #openstack-cinder | 17:09 | |
| geguileo | jgriffith: Do we want to send jobs based on a topic queue that uses the host as reference | 17:09 |
| thingee | dannywilson, patrickeast yea that seems pretty straight forward | 17:09 |
| jgriffith | thingee: I'm sadly not really familiar with that... but that's interesting and I don't really understand it | 17:09 |
| geguileo | And then all nodes need to have the same host configured | 17:09 |
| jgriffith | geguileo: I don't know :) | 17:10 |
| *** wanghao has joined #openstack-cinder | 17:10 | |
| thingee | jgriffith: they end up with a bunch of delete requests, and things slow down for other operations to come in | 17:10 |
| geguileo | Or do we want to configure the nodes with different host configurations | 17:10 |
| thingee | jgriffith: there's not really much more to explain | 17:10 |
| geguileo | and use the backend name as unique identifier | 17:10 |
| *** angela-s has joined #openstack-cinder | 17:10 | |
| thingee | jgriffith: it's a bandaid fix | 17:10 |
| geguileo | to use for the topic queue where we would replace the host with * instead of the host name | 17:10 |
| *** svilgelm has joined #openstack-cinder | 17:10 | |
| jgriffith | thingee: maybe we can circle back and you can help me understand that config better? | 17:11 |
| geguileo | Anyway, that's one of the decissions we need to make | 17:11 |
| guitarzan | thingee: they hacked up the rpc layer to do that? that's pretty wild | 17:11 |
| thingee | jgriffith: I don't understand it myself. I might be able to dig up what DreamHost did...they're doing something similar | 17:11 |
| thingee | also using the same solution, if you can figure that out ;) | 17:11 |
| jgriffith | geguileo: yeah... so what I was wondering was why you couldn't just say "host = [foo, biz, baz....] | 17:11 |
| jgriffith | geguileo: and just issue the roc call to any available from the list | 17:11 |
| geguileo | jgriffith: In the DB you mean? | 17:12 |
| guitarzan | I think ideally all sharing the same host would be preferred | 17:12 |
| earlephilhower | mriedem: Thanks for the update on the rebase. Been in meetings since 7:30AM, will check out at lunch. | 17:12 |
| jgriffith | geguileo: no...we were talking c-vol | 17:12 |
| jgriffith | db is another piece.. that I think is already solved | 17:12 |
| jgriffith | or "solvable" | 17:12 |
| geguileo | jgriffith: And how does the API and the scheduler know the node to send it to? | 17:12 |
| jgriffith | but I could be completely wrong too :) | 17:12 |
| geguileo | jgriffith: Or the list of nodes? | 17:12 |
| jgriffith | geguileo: what I'm saying is why should it care? Why shouldn't | 17:12 |
| jgriffith | geguileo: yes... it has a list, any node will do | 17:13 |
| geguileo | jgriffith: Ok, then how does he maps the list of nodes that he has to the resources? | 17:13 |
| jgriffith | geguileo: this assumes the c-vol node is NOT the storage of course | 17:13 |
| geguileo | jgriffith: He receives a delete, looks in the DB the host field | 17:13 |
| *** annashen has quit IRC | 17:13 | |
| jgriffith | geguileo: same way we do today... we just modify the definition of the "host" column | 17:13 |
| geguileo | jgriffith: I think that doesn't really matter, because if the node is the storage you cannot configure it as A-A | 17:13 |
| geguileo | jgriffith: So the host is a list? | 17:14 |
| jgriffith | geguileo: my point is do we care what path it takes to delete the volume? So long as the command makes it to the backend device? | 17:14 |
| *** salv-orlando has joined #openstack-cinder | 17:14 | |
| geguileo | jgriffith: No, we don't | 17:14 |
| nikeshm | smcginnis: hi | 17:14 |
| geguileo | jgriffith: That's why I propose to use the same host for all nodes in the same cluster | 17:14 |
| *** alexpilotti has quit IRC | 17:14 | |
| geguileo | jgriffith: And any of those nodes can pick the job | 17:15 |
| geguileo | jgriffith: When we queue it we don't care | 17:15 |
| jgriffith | geguileo: ok, so we seem to be on the same page there then | 17:15 |
| geguileo | jgriffith: Good :-) | 17:15 |
| geguileo | jgriffith: But then we have the problem with the init_host of volume manager | 17:15 |
| geguileo | jgriffith: That on start it assumes he's the only one with access to the backend | 17:15 |
| jgriffith | geguileo: although I don't understand your comment "same host for all nodes in same cluster" You mean, they all report same host name and we just treat them as a queue? | 17:16 |
| geguileo | jgriffith: And will break any ongoing operations | 17:16 |
| nikeshm | smcginnis: http://paste.openstack.org/show/406287/ | 17:16 |
| geguileo | jgriffith: Yes, they report the same host | 17:16 |
| *** avishay has quit IRC | 17:16 | |
| *** avishay_ has joined #openstack-cinder | 17:16 | |
| geguileo | jgriffith: So they will access the same topic queue | 17:16 |
| jgriffith | geguileo: ok | 17:16 |
| geguileo | jgriffith: And we don't care how many there are getting jobs from the queue | 17:16 |
| guitarzan | geguileo: the HA discussions have all hinged on having some sort of distributed locking mechanism | 17:16 |
| guitarzan | which should also be the answer to init_host | 17:17 |
| jgriffith | guitarzan: yeah | 17:17 |
| geguileo | guitarzan: The DLM is for resource locking at cinder volume | 17:17 |
| jgriffith | guitarzan: although I'm not sure we need that TBH | 17:17 |
| jgriffith | which is what I'm getting at here | 17:17 |
| geguileo | guitarzan: But now we are talking about API to Volume job distribution | 17:17 |
| geguileo | guitarzan: Which is a step earlier | 17:17 |
| guitarzan | jgriffith: maybe, but init_host does do some goofy things like deleting volumes stuck in 'deleting' | 17:17 |
| jgriffith | guitarzan: hmm... | 17:18 |
| geguileo | guitarzan: Read a few lines before when I begin talking about it | 17:18 |
| jgriffith | guitarzan: I'll have to look at that | 17:18 |
| guitarzan | so they still may need locks | 17:18 |
| geguileo | jgriffith: It's also mentioned in the post | 17:18 |
| jgriffith | guitarzan: honestly we shouldn't do that | 17:18 |
| *** mylu has quit IRC | 17:18 | |
| geguileo | jgriffith: Since now we share the same host we need to fix the init_host | 17:18 |
| guitarzan | jgriffith: I don't think I disagree :) | 17:18 |
| thingee | jgriffith: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L370 | 17:18 |
| *** ub has quit IRC | 17:18 | |
| thingee | jgriffith: why? | 17:19 |
| *** annashen has joined #openstack-cinder | 17:19 | |
| *** xyang has joined #openstack-cinder | 17:19 | |
| *** mylu has joined #openstack-cinder | 17:19 | |
| jgriffith | thingee: guitarzan ahh.. that :) Well, that's not a terribly big deal I don't think... but anyway | 17:19 |
| geguileo | thingee: https://github.com/openstack/cinder/blob/master/cinder/volume/manager.py#L338-L358 | 17:19 |
| geguileo | thingee: That as well | 17:19 |
| guitarzan | no it isn't, it's just a possible point of contention where multiple drivers do the same thing | 17:20 |
| *** ub has joined #openstack-cinder | 17:20 | |
| *** salv-orlando has quit IRC | 17:20 | |
| jgriffith | thingee: I might have misunderstood; so re-issuing a delete on a volume that's "deleting" shouldn't be terrible depending on the backend | 17:20 |
| guitarzan | also, I just saw the snapshot stuck in creating thing... I don't like that one at all :) | 17:20 |
| geguileo | jgriffith: The problem is not so much deleting as the other link I've posted | 17:20 |
| jgriffith | guitarzan: yeah... 'creating' tasks are trickier | 17:21 |
| geguileo | jgriffith: Where any creating operation is set to error | 17:21 |
| jgriffith | geguileo: looking | 17:21 |
| jgriffith | geguileo: yeah | 17:21 |
| jgriffith | geguileo: so TBH though... I think this is sort of wonky | 17:21 |
| geguileo | So we need a way to differentiate between stuck jobs and ongoing jobs | 17:21 |
| geguileo | jgriffith: It made sense when only 1 node could access the backend | 17:22 |
| jgriffith | geguileo: so use a db tag that says "this node owns the task" | 17:22 |
| geguileo | jgriffith: For Active-Passive is ok | 17:22 |
| thingee | I've said it in past discussions and the last summit, we should not attempt to resume creating tasks. Just give up and have the user try again. | 17:22 |
| thingee | it's not worth it | 17:22 |
| jgriffith | geguileo: so when you restart or bring a new c-vol online it knows "what it's allowed to do and what it's not" | 17:22 |
| geguileo | thingee: I agree | 17:22 |
| *** salv-orlando has joined #openstack-cinder | 17:22 | |
| jgriffith | thingee: I fully agree with that | 17:22 |
| geguileo | thingee: But we still need to know when an operation is stuck | 17:22 |
| geguileo | thingee: To set it to error | 17:23 |
| *** patrickeast has quit IRC | 17:23 | |
| jgriffith | thingee: but I see geguileo 's point about a new service coming up currently "doesn't know" | 17:23 |
| geguileo | thingee: Otherwise the user can't tell the difference between a slow operation and a stuck one | 17:23 |
| thingee | jgriffith: oh I see, yeah good point | 17:23 |
| geguileo | And maybe the sevice doesn't come up | 17:23 |
| geguileo | But we still have other 2 nodes working with that backend | 17:23 |
| jgriffith | geguileo: thingee so I'd propose we change our model slightly; and for the volume when it goes into creating, deleting or whatever active op, we mark what node is "doing" it | 17:23 |
| *** alexpilotti has joined #openstack-cinder | 17:23 | |
| geguileo | That should know that resource is in error | 17:23 |
| jgriffith | as part of the status update | 17:24 |
| *** lpetrut has quit IRC | 17:24 | |
| *** mylu has quit IRC | 17:24 | |
| thingee | can't we just the c-vol coming up ask all other c-vols what uuid's they're working with? | 17:24 |
| jgriffith | then when a new service is started (or another one restarted) it knows not to mess with things that aren't under it's current task state | 17:24 |
| thingee | otherwise set to error | 17:24 |
| eharney | jgriffith: but what if that node never comes back up? | 17:24 |
| thingee | working with at this point in time | 17:24 |
| *** ub has quit IRC | 17:25 | |
| geguileo | thingee: I don't think that's reasonable | 17:25 |
| jgriffith | eharney: the it's errored out... | 17:25 |
| jgriffith | eharney: oh... wait | 17:25 |
| jgriffith | eharney: that's a different case... just a minute | 17:25 |
| *** mylu has joined #openstack-cinder | 17:25 | |
| jgriffith | eharney: so I'm separating this into two cases: | 17:25 |
| geguileo | I think the solution would be to use the backend as the message queue topic identifier | 17:25 |
| jgriffith | 1. Everything's "ok" I just started a new service instance | 17:26 |
| jgriffith | 2. The actual "oh, one of my services failed" | 17:26 |
| thingee | geguileo: why is it not reasonable? | 17:26 |
| jgriffith | So that's where the point thingee just brought up comes in to play (situation 2) | 17:26 |
| geguileo | jgriffith: How do you know it's yours if we are using the same host in them all? | 17:26 |
| jgriffith | geguileo: a new column "assigned-node" or whatever | 17:26 |
| geguileo | thingee: Because then you need to keep track in Volume of all ongoing operations | 17:26 |
| jgriffith | each node under the host gets a UUID | 17:27 |
| geguileo | jgriffith: Oh, that, ok | 17:27 |
| jgriffith | that UUID is known by the manager; and when it sets a state on a volume it also sets that it "owns" that task | 17:27 |
| thingee | Sure, c-vol has a list of active uuids. If it gets a request from another c-vol of what it's working with, it sends it | 17:27 |
| guitarzan | this is just a homegrown locking implementation | 17:28 |
| *** lpetrut has joined #openstack-cinder | 17:28 | |
| guitarzan | and I don't think it'd be any cleaner | 17:28 |
| jgriffith | guitarzan: kinda... you might be right | 17:28 |
| geguileo | thingee: Then each node must be aware of other nodes | 17:28 |
| jgriffith | guitarzan: I'm kinda of the opinion that "we" in the OpenStack world like to make our solutions as complex as possible and it's not always needed | 17:29 |
| geguileo | thingee: Or they could use the broadcast queue | 17:29 |
| jgriffith | geguileo: nahh | 17:29 |
| thingee | geguileo: yeah just have c-vol subscribe. they don't need to know about each other | 17:29 |
| jgriffith | thingee: +1 | 17:29 |
| thingee | just that someone is doing something with that uuid | 17:29 |
| jgriffith | thingee: geguileo but guitarzan pointed out that this might not be any simpler/better... so maybe I'm just wasting your time :) | 17:29 |
| thingee | jgriffith: never give up! :P | 17:30 |
| geguileo | thingee: But then you need cross-reference those jobs with the DB and exclude newer jobs | 17:30 |
| jgriffith | thingee: geguileo guitarzan I think it was worth talking through a bit tough | 17:30 |
| thingee | geguileo: have c-api subscribe | 17:30 |
| *** aix has quit IRC | 17:30 | |
| jgriffith | geguileo: so there's only a single case that this matters it seems | 17:30 |
| thingee | "can't do anything with this uuid, some c-vol is active with it." | 17:30 |
| jgriffith | geguileo: and I honestly see a lot of this as being very similar to a problem that's already solved via scheduler | 17:31 |
| thingee | we've already agreed in previous discussions we're not interested in doing parallel operations with resources | 17:31 |
| jgriffith | or I should say "in the" scheduler | 17:31 |
| *** lpetrut has quit IRC | 17:31 | |
| geguileo | jgriffith: I don't think it's all that similar | 17:31 |
| jgriffith | geguileo: thingee anyway... just some additional things to think about | 17:32 |
| eharney | thingee: can you clarify that a little? | 17:32 |
| geguileo | jgriffith: I actually considered scheduler to do the job distribution and I didn't like it | 17:32 |
| *** Yogi1 has joined #openstack-cinder | 17:32 | |
| jgriffith | also, some of this is harder for me to really dig into, because I don't necessarily agree with the philosophy | 17:32 |
| geguileo | jgriffith: What philosophy? | 17:33 |
| jgriffith | philosophy being the whole "A/A HA" thing | 17:33 |
| geguileo | jgriffith: You don't like A-A? | 17:33 |
| jgriffith | geguileo: well, not saying that... | 17:33 |
| hemna | ha ha philosophy | 17:33 |
| jgriffith | geguileo: just saying that there are a couple of folks that I "think" make a bigger issue of that than it really is (as in the real need for it) | 17:34 |
| *** lpetrut has joined #openstack-cinder | 17:34 | |
| jgriffith | geguileo: but that's my own personal view... take it for what it's worth | 17:34 |
| jgriffith | or don't :) | 17:34 |
| thingee | eharney: so each c-vol and c-api are listening for messages of uuids being worked with. each keeping a list in memory and updating as things come in and are done. If c-api gets a request from the user on a particular uuid and sees it has been mentioned by a c-vol as doing something with it, it stops the requests. In c-vol land, they all know which uuids are actively being worked with and will stop operations even on their own layer in addi | 17:34 |
| thingee | I have not figured out the release part if a c-vol goes down | 17:35 |
| eharney | thingee: ahh gotcha | 17:35 |
| hemna | I'd just like to see is get rid of the local file locks in the volume manager. | 17:35 |
| geguileo | jgriffith: I think it's a little weird to say to people that cinder does not support A-A being the block storage of a cloud infrastructure | 17:35 |
| jgriffith | hemna: LOL.. amen to that | 17:35 |
| hemna | geguileo, +1 | 17:35 |
| jgriffith | geguileo: sure, but it's not REALLY going to support it anyway | 17:36 |
| geguileo | jgriffith: Why?? :''-( | 17:36 |
| jgriffith | geguileo: if your c-vol node goes down and you're using LVM you're still toast... if your Ceph cluster barfs, you're toast... etc etc etc | 17:36 |
| hemna | true | 17:36 |
| thingee | so I'm probably crazy, but I think the broadcast idea of resources solves it both in c-api layer and c-vol layer as a fallback. Helping with issues with nova bypassing the api in some places | 17:36 |
| hemna | but folks that typically want A A don't use LVM or ceph, they buy a solidfire/3par/ibm/etc/etc | 17:37 |
| jgriffith | thingee: well.. you are crazy... but not because of that opinion :) | 17:37 |
| jgriffith | hemna: my argument applies to all of them as well... that was the "etc" part :) | 17:37 |
| jgriffith | hemna: it includes every 3'rd party driver | 17:37 |
| hemna | jgriffith, you know you've always wanted to color your hair purple. you are just jealous | 17:37 |
| * thingee is behind on the backlog from all the talking, but it probably restating someone else's idea here | 17:37 | |
| geguileo | jgriffith: Well, if you are using LVM with local storage you CAN'T configure it as A-A | 17:37 |
| jgriffith | hemna: and I don't know if I agree with Ceph being in that first category | 17:37 |
| jgriffith | geguileo: I know.. that's my point :) | 17:38 |
| geguileo | jgriffith: And with Ceph you can have A-A | 17:38 |
| *** patrickeast has joined #openstack-cinder | 17:38 | |
| hemna | jgriffith, but I don't think that precludes us from fixing some "simple" things that would make A-A possible for the rest of us. | 17:38 |
| jgriffith | geguileo: so that leave the case that all of this effort going towards solving of being "my c-vol service node died and I can't get to the backend" | 17:38 |
| jgriffith | geguileo: seems goofy | 17:38 |
| hemna | I don't think it's an either or. | 17:39 |
| geguileo | jgriffith: And also increase workload capacity | 17:39 |
| *** rhagarty_ has quit IRC | 17:39 | |
| thingee | geguileo: thought's? | 17:39 |
| jgriffith | geguileo: so my other crazy thing is docker containers running the services across different nodes, shared storage and just spawn/fail over | 17:39 |
| *** rhagarty has quit IRC | 17:39 | |
| jgriffith | cuz it solves all these problems higher in the stack | 17:39 |
| geguileo | jgriffith: For example if you do backups, they are "expensive" and you may need more nodes | 17:39 |
| jgriffith | and for the most part "just works" | 17:39 |
| *** jaypipes has quit IRC | 17:40 | |
| jgriffith | geguileo: yeah... backups is something that can always be used as a detractor for us | 17:40 |
| jgriffith | geguileo: anyway, I'm certainly not arguing against anything. Just talking through it | 17:41 |
| hemna | putting in 'ing' checks against volumes in the API, would help us get rid of the local file locks, and allow for a limited version of A-A for some backends to work. | 17:41 |
| jgriffith | geguileo: educating myself a bit | 17:41 |
| geguileo | thingee: I think that's quite a complicated solution | 17:41 |
| jgriffith | hemna: we already have them in most places by the way | 17:41 |
| thingee | geguileo: why? | 17:41 |
| hemna | yah that's good | 17:41 |
| geguileo | hemna: Checking ING is not enough | 17:41 |
| thingee | geguileo: I think when people talk about DLM's that's complicated | 17:41 |
| geguileo | hemna: Because reading operations don't have an ING status | 17:41 |
| hemna | geguileo, no it's not, but it has to be a first step | 17:41 |
| thingee | mostly because don't know how they work and throw the problems to them | 17:41 |
| ccesario | hello... does someone have idea about this error on create volume process!? http://paste.openstack.org/show/QJ3BPflkBZC4esEdtdUw/ | 17:41 |
| *** mylu has quit IRC | 17:42 | |
| *** annashen has quit IRC | 17:42 | |
| jgriffith | thingee: that's why I come back to things like etc and letting an orchestration layer above us deal with it :) | 17:42 |
| geguileo | thingee: Because I think it's almost as complicated as the one I proposed. XD | 17:42 |
| *** rhagarty has joined #openstack-cinder | 17:42 | |
| jgriffith | s/etc/etcd/ | 17:42 |
| *** rhagarty_ has joined #openstack-cinder | 17:42 | |
| geguileo | thingee: And there is an easier way to do it that it's similar to how Neutron is doing it | 17:42 |
| geguileo | thingee: I'll try to write something up this weekend | 17:43 |
| thingee | geguileo: I disagree. c-vol and c-apis subscribe. communicate and keep in memory for check on uuid of resources. | 17:43 |
| thingee | that's it | 17:43 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 17:43 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Volume status management during migration https://review.openstack.org/186312 | 17:43 |
| thingee | I can even describe the solution in one sentence | 17:43 |
| geguileo | thingee: Then they need to update before any operation? | 17:43 |
| thingee | the updates are pushed? | 17:43 |
| avishay_ | hemna: gentle reminder :) https://review.openstack.org/#/c/205945/ | 17:44 |
| *** r-daneel has quit IRC | 17:44 | |
| geguileo | thingee: Let me sleep on your solution | 17:44 |
| hemna | avishay_, thanks, I haven't had time to look at reviews just yet this morning...late start | 17:44 |
| geguileo | thingee: And I will compare it with the other solution I'm thinking about | 17:44 |
| avishay_ | hemna: no worries, thanks! | 17:45 |
| *** avishay_ is now known as avishay | 17:45 | |
| *** ByPasS has quit IRC | 17:45 | |
| geguileo | thingee: I'd rather think it well ;-) | 17:45 |
| thingee | geguileo: if anything I learned about working in computer science, is if the solution is simple, it probably doesn't work. That's why I was asking people if I'm crazy on suggesting this. | 17:45 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 17:45 |
| geguileo | thingee: XD XD XD | 17:45 |
| erlon | thingee: Mike, can you explain the deadlines for BP approval? you said in the meeting that liberty-2 is tagged, but feature free is next week, so I'm assuming that will be possible to get changes merged until next friday right? | 17:46 |
| geguileo | thingee: I'll think it more carefully | 17:46 |
| thingee | erlon: code freeze is early september | 17:46 |
| geguileo | erlon: I think it's this Friday | 17:46 |
| avishay | jgriffith: we have a great experience using consul for HA, it's similar to etcd | 17:46 |
| geguileo | erlon: For spec/bp acceptance | 17:46 |
| *** annashen has joined #openstack-cinder | 17:46 | |
| *** dannywilson has quit IRC | 17:46 | |
| jgriffith | avishay: nice, I haven't looked at consul at all | 17:47 |
| thingee | I will be looking over specs some more today to see where we're at. I appreciate people understanding that we just got done with a new milestone, whether people keep track of that stuff or not | 17:47 |
| jgriffith | avishay: so the paxos based stuff like ZK etc are sort of phasing out IMO | 17:47 |
| hemna | avishay, https://www.consul.io/ ? | 17:47 |
| angela-s | thingee, hemna: just wanted to mention that brocade CI is up/running for zoning driver. would be great to get final reviews on the two zoning specs, as CI was the blocker | 17:47 |
| avishay | jgriffith: it gives us a super scalable key-value store for metadata, DNS for all services to route requests to them, cluster membership, etc | 17:47 |
| avishay | hemna: yes | 17:47 |
| avishay | jgriffith: agreed | 17:48 |
| thingee | angela-s: should I start running stats on the ci? | 17:48 |
| thingee | angela-s: I was under the impression this was slow moving | 17:48 |
| hemna | angela-s, nice job, congrats on getting that up. | 17:48 |
| angela-s | thingee: what do you mean by slow moving? | 17:48 |
| *** ByPasS has joined #openstack-cinder | 17:48 | |
| *** edtubill has quit IRC | 17:48 | |
| thingee | angela-s: no one has communicated anything, until now... | 17:49 |
| thingee | angela-s: so if you don't hear about things, you assume slow moving | 17:49 |
| thingee | angela-s: communication is good | 17:49 |
| erlon | geguileo: ok, thingee, can you approve this BP, or is there any criteria for this? | 17:49 |
| erlon | thingee: https://blueprints.launchpad.net/cinder/+spec/hnas-manage-unmanage | 17:49 |
| vilobhmm | jungleboyj : If you can review this patch it would be helpful : https://review.openstack.org/#/c/206170/ | 17:50 |
| angela-s | thingee: i see. sorry about that. it's up now, running on all cinder patchsets | 17:50 |
| thingee | angela-s: I can start running stats on the CI today, but really I wanted to be assured what's in tree today is working | 17:50 |
| *** ub has joined #openstack-cinder | 17:51 | |
| jungleboyj | vilobhmm: Ok, I apologize I haven't been on those. Been a wild week. | 17:51 |
| angela-s | thingee: ok, thank you | 17:51 |
| vilobhmm | jungleboyj : no worries; whenever you are free | 17:51 |
| thingee | angela-s: thank you and great work to hear this moving forward! | 17:52 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Add connector object to create_export https://review.openstack.org/202808 | 17:52 |
| thingee | angela-s: happy brocade has you to help with this | 17:52 |
| thingee | angela-s: hope they're giving you enough recognition for keep things in tree. | 17:53 |
| erlon | thingee: thanks! | 17:53 |
| angela-s | thingee: thanks | 17:53 |
| angela-s | thingee: it's definitely a challenge, that's for sure | 17:53 |
| thingee | angela-s: but just think how happy your customers will be | 17:54 |
| thingee | hemna: do we have someone from cisco in this room? | 17:54 |
| thingee | room channel whatever | 17:54 |
| hemna | thingee, great question. I've not heard a single thing from cisco since Vancouver | 17:54 |
| *** jasondotstar has joined #openstack-cinder | 17:54 | |
| thingee | hemna: I think you started a ML thread | 17:55 |
| thingee | hemna: that I can reference | 17:55 |
| hemna | hrmm | 17:55 |
| patrickeast | thingee: hemna: fyi the pure storage ci FC jobs have been/are currently running the cisco zone manager | 17:56 |
| patrickeast | since all i could get for my system was a cisco swich :( | 17:56 |
| *** mylu has joined #openstack-cinder | 17:56 | |
| *** e0ne has joined #openstack-cinder | 17:56 | |
| angela-s | thingee, hemna: i was contacted by cisco engineer for CI, trying to remember his name | 17:56 |
| patrickeast | so its kind of getting tested | 17:56 |
| thingee | patrickeast: oh | 17:57 |
| thingee | patrickeast, hemna: that makes me feel good | 17:57 |
| hemna | thingee, I have an email thread with a guy from Cisco that included you, but I don't think it was on the ML. | 17:57 |
| thingee | patrickeast: can you speak on the integration work so far? | 17:57 |
| thingee | hemna: alright that works | 17:57 |
| thingee | hemna: thanks | 17:57 |
| * thingee list of things to do is growing :( | 17:57 | |
| patrickeast | thingee: wasn’t really any integration work, just plug and play following the documentation | 17:58 |
| patrickeast | thingee: and it only craps out every once and a while due to parallal request issues or something (its on my list to see if thats our fault or the zone manager) | 17:58 |
| *** xyang1 has quit IRC | 17:58 | |
| thingee | so dumb fc question from your fellow ptl, what exactly is cisco's integration with cinder then? What's the difference with all this work happening from brocade? | 17:58 |
| *** mylu has quit IRC | 17:59 | |
| *** mylu has joined #openstack-cinder | 17:59 | |
| angela-s | thingee: you don't want me to answer that question, i'm biased | 17:59 |
| thingee | haha | 17:59 |
| patrickeast | they have an implementation of the zone manager interface that can do cisco-specific remote api calls to the switch to configure things… | 17:59 |
| patrickeast | i haven’t seen anyone actually working on it before | 17:59 |
| patrickeast | so it looks like maybe a little bit neglected | 17:59 |
| patrickeast | but does seem to mostly work | 18:00 |
| hemna | thingee, I found the thread. want me to forward it to you? It was with Al Lau | 18:00 |
| thingee | hemna: sure | 18:00 |
| thingee | or subject | 18:00 |
| thingee | I can find it in archives | 18:00 |
| *** haomaiwang has quit IRC | 18:01 | |
| *** shyama has quit IRC | 18:02 | |
| hemna | thingee, sent | 18:02 |
| thingee | hemna: I see... "hello" isn't a good subject to search for | 18:02 |
| *** haomaiwang has joined #openstack-cinder | 18:02 | |
| thingee | lol | 18:02 |
| hemna | yah not the best | 18:03 |
| thingee | hemna: cool, I'll send an email to the list about the state of FC zone management in Cinder. | 18:04 |
| hemna | ok great thanks | 18:04 |
| * thingee adds another item to the list :( | 18:04 | |
| thingee | business is booming | 18:04 |
| thingee | hemna: anything we need to connect on with brick? | 18:04 |
| hemna | I think we are doing well right now. | 18:05 |
| *** annashen_ has joined #openstack-cinder | 18:05 | |
| thingee | hemna: I haven't looked at the recent updates with nova integration patch | 18:05 |
| thingee | hemna: I assume mriedem has been of help though | 18:06 |
| hemna | the brick nova patch landed yesterday | 18:06 |
| hemna | so it's in. | 18:06 |
| thingee | lol | 18:06 |
| hemna | yah mriedem was very helpful :) | 18:06 |
| *** e0ne has quit IRC | 18:06 | |
| thingee | hemna: well good work. I don't know if it was recognized in the meeting | 18:07 |
| thingee | hemna: since I got cut off irc | 18:07 |
| thingee | hemna: but I blame smcginnis then since he was chair | 18:07 |
| hemna | thanks. it was a long road. :) | 18:07 |
| hemna | hehe | 18:07 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 18:08 |
| openstackgerrit | Vilobh Meshram proposed openstack/cinder: Nested Quota Driver: Get Project Hierarchy https://review.openstack.org/206171 | 18:09 |
| *** e0ne has joined #openstack-cinder | 18:09 | |
| *** annashen_ has quit IRC | 18:10 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Oracle ZFSSA iSCSI driver to return free space for non thin provisioned storage https://review.openstack.org/207106 | 18:11 |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Oracle ZFSSA iSCSI driver to return free space for non thin provisioned storage https://review.openstack.org/207106 | 18:11 |
| *** jwcroppe has joined #openstack-cinder | 18:11 | |
| *** lpetrut has quit IRC | 18:12 | |
| *** angela-s is now known as angela-s|away | 18:12 | |
| *** ByPasS has quit IRC | 18:13 | |
| *** ub2 has joined #openstack-cinder | 18:14 | |
| mriedem | thingee: hemna: regarding nova and brick, the scaleio change is here https://review.openstack.org/#/c/194454/ - it need some work, but some cinder eyes on the technical connect/disconnect details would be helpful | 18:15 |
| mriedem | keep in mind that nova FF is tomorrow | 18:15 |
| *** mylu has quit IRC | 18:15 | |
| mriedem | and earlephilhower has to get the HGST connector for nova rebased today too https://review.openstack.org/#/c/194412/ | 18:15 |
| johnthetubaguy | mriedem: +1 thank you for helping push on those | 18:15 |
| mriedem | johnthetubaguy: you're supposed to be at dinner | 18:16 |
| johnthetubaguy | mriedem: thats what sally was hinting too, just sending the FF emial :) | 18:16 |
| *** ub has quit IRC | 18:16 | |
| *** dannywilson has joined #openstack-cinder | 18:18 | |
| *** coolsvap is now known as coolsvap|brb | 18:18 | |
| *** lcurtis has joined #openstack-cinder | 18:19 | |
| *** ByPasS has joined #openstack-cinder | 18:19 | |
| *** mylu has joined #openstack-cinder | 18:20 | |
| *** annashen has quit IRC | 18:21 | |
| *** jasondotstar has quit IRC | 18:23 | |
| openstackgerrit | Vincent Hou proposed openstack/python-cinderclient: Change cinderclient according to volume migration improvement https://review.openstack.org/189547 | 18:23 |
| *** annashen has joined #openstack-cinder | 18:24 | |
| *** dims_ has joined #openstack-cinder | 18:24 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 18:24 |
| hemna | mriedem, ok I'll look at it, thanks for the heads up. If you have anything else that needs some Cinder eyes I'm available | 18:25 |
| hemna | xyang, ping | 18:25 |
| *** annegentle has quit IRC | 18:26 | |
| *** annegentle has joined #openstack-cinder | 18:26 | |
| *** svilgelm has quit IRC | 18:27 | |
| *** dims has quit IRC | 18:28 | |
| *** amickus has joined #openstack-cinder | 18:30 | |
| *** openstackgerrit has quit IRC | 18:31 | |
| *** openstackgerrit has joined #openstack-cinder | 18:32 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 18:34 |
| *** svilgelm has joined #openstack-cinder | 18:34 | |
| *** jaypipes has joined #openstack-cinder | 18:34 | |
| *** dannywilson has quit IRC | 18:34 | |
| *** dannywilson has joined #openstack-cinder | 18:34 | |
| *** ByPasS has quit IRC | 18:34 | |
| kaisers | thingee: ping (regarding the Quobyte CI / Driver) | 18:35 |
| *** ByPasS has joined #openstack-cinder | 18:36 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 18:39 |
| *** ByPasS has quit IRC | 18:40 | |
| *** salv-orlando has quit IRC | 18:40 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 18:40 |
| openstackgerrit | Ryan Lucio proposed openstack/cinder: VMEM v7000: support storage pools / volume types https://review.openstack.org/195773 | 18:40 |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add smartX support for Huawei driver https://review.openstack.org/188732 | 18:41 |
| *** annashen has quit IRC | 18:42 | |
| *** ByPasS has joined #openstack-cinder | 18:43 | |
| *** jwcroppe has quit IRC | 18:43 | |
| *** jwcroppe has joined #openstack-cinder | 18:44 | |
| *** jwcroppe has quit IRC | 18:48 | |
| openstackgerrit | Apoorva Deshpande proposed openstack/cinder: Tintri snapshot id https://review.openstack.org/198829 | 18:48 |
| *** edtubill has joined #openstack-cinder | 18:49 | |
| *** svilgelm has quit IRC | 18:52 | |
| openstackgerrit | Thang Pham proposed openstack/cinder: Continue conversion to volume object https://review.openstack.org/201404 | 18:52 |
| openstackgerrit | Matthew Edmonds proposed openstack/cinder: move oslo.vmware to test-requirements https://review.openstack.org/203237 | 18:54 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Adds the migration progress support for migration https://review.openstack.org/204953 | 18:56 |
| *** avishay has quit IRC | 18:56 | |
| *** svilgelm has joined #openstack-cinder | 18:57 | |
| *** Adriano__ has quit IRC | 18:57 | |
| *** svilgelm has quit IRC | 18:59 | |
| *** zhonghua has joined #openstack-cinder | 19:00 | |
| *** haomaiwang has quit IRC | 19:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 19:01 | |
| *** salv-orlando has joined #openstack-cinder | 19:01 | |
| xyang | hemna: hi | 19:02 |
| hemna | xyang, looks like you need to get this one updated. nova FF is tomorrow. https://review.openstack.org/#/c/194454/ | 19:03 |
| xyang | hemna: really? I am in manila meetup today, it will work on that. Thanks | 19:04 |
| hemna | xyang, yup. mriedem pinged me about it earlier. FYI. | 19:04 |
| *** krtaylor has quit IRC | 19:04 | |
| xyang | hemna: thanks for the reminder | 19:05 |
| hemna | xyang, np. | 19:05 |
| *** zhonghua has quit IRC | 19:07 | |
| *** coolsvap|brb is now known as coolsvap|away | 19:10 | |
| *** IanGovett has joined #openstack-cinder | 19:13 | |
| *** krtaylor has joined #openstack-cinder | 19:17 | |
| *** vincent_hou has quit IRC | 19:18 | |
| *** jasondotstar has joined #openstack-cinder | 19:18 | |
| *** lpetrut has joined #openstack-cinder | 19:18 | |
| *** IanGovett has quit IRC | 19:19 | |
| *** dims_ has quit IRC | 19:27 | |
| *** dims has joined #openstack-cinder | 19:27 | |
| thingee | kaisers: just keep the convo on the ML thread. If I haven't gotten to it yet, it's because I have everyone pinging me. :) | 19:31 |
| thingee | kaisers: unfortunately I can't drop everything at the moment. | 19:31 |
| thingee | kaisers: thanks | 19:31 |
| thingee | mriedem: adding this to my list | 19:31 |
| *** sgotliv_ has quit IRC | 19:32 | |
| *** sgotliv has joined #openstack-cinder | 19:32 | |
| *** salv-orlando has quit IRC | 19:35 | |
| *** ByPasS has quit IRC | 19:35 | |
| *** ByPasS has joined #openstack-cinder | 19:35 | |
| *** annashen has joined #openstack-cinder | 19:42 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix HNAS iSCSI 32 targets limitation error https://review.openstack.org/206651 | 19:42 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix concurrent attaches on HNAS iSCSI driver https://review.openstack.org/203780 | 19:42 |
| kaisers | thingee_: ok | 19:44 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota Driver: Get Project Hierarchy https://review.openstack.org/206171 | 19:46 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Cinder Nested Quota Driver https://review.openstack.org/205369 | 19:46 |
| *** krtaylor has quit IRC | 19:46 | |
| *** ByPasS has quit IRC | 19:47 | |
| *** belmoreira has quit IRC | 19:47 | |
| *** HenryG has quit IRC | 19:49 | |
| *** e0ne has quit IRC | 19:51 | |
| *** salv-orlando has joined #openstack-cinder | 19:51 | |
| *** timcl has quit IRC | 19:52 | |
| *** HenryG has joined #openstack-cinder | 19:52 | |
| *** ByPasS has joined #openstack-cinder | 19:52 | |
| *** e0ne has joined #openstack-cinder | 19:55 | |
| *** mylu has quit IRC | 19:56 | |
| *** krtaylor has joined #openstack-cinder | 19:59 | |
| *** haomaiwa_ has quit IRC | 20:01 | |
| *** lpetrut has quit IRC | 20:01 | |
| *** edmondsw has quit IRC | 20:01 | |
| *** haomaiwa_ has joined #openstack-cinder | 20:02 | |
| *** thangp has quit IRC | 20:02 | |
| *** mylu has joined #openstack-cinder | 20:03 | |
| *** e0ne has quit IRC | 20:03 | |
| *** hodos has joined #openstack-cinder | 20:03 | |
| *** annashen_ has joined #openstack-cinder | 20:06 | |
| *** bill_az has joined #openstack-cinder | 20:06 | |
| *** Strony has joined #openstack-cinder | 20:08 | |
| openstackgerrit | liuxinguo proposed openstack/cinder: Add iSCSI multipath support for Huawei driver https://review.openstack.org/188365 | 20:09 |
| *** ByPasS has quit IRC | 20:09 | |
| *** ByPasS has joined #openstack-cinder | 20:09 | |
| *** annashen_ has quit IRC | 20:11 | |
| *** e0ne has joined #openstack-cinder | 20:11 | |
| *** liuxg has quit IRC | 20:13 | |
| *** openstackgerrit has quit IRC | 20:16 | |
| *** openstackgerrit has joined #openstack-cinder | 20:17 | |
| *** annegentle has quit IRC | 20:17 | |
| *** jasondotstar has quit IRC | 20:19 | |
| *** xyang1 has joined #openstack-cinder | 20:26 | |
| xyang | hemna: mriedem , hi, I submitted another patch: https://review.openstack.org/#/c/194454/ | 20:26 |
| xyang | hemna: mriedem, please take a look | 20:27 |
| *** Lee1092 has quit IRC | 20:28 | |
| *** mylu has quit IRC | 20:28 | |
| *** mylu has joined #openstack-cinder | 20:28 | |
| *** xyang1 has quit IRC | 20:29 | |
| hemna | xyang, ok thanks | 20:29 |
| hemna | xyang, does that root_wrap filter work? | 20:30 |
| xyang | hemna: thank you | 20:30 |
| hemna | :) | 20:30 |
| hemna | I'm hacking around trying to add a new /lib/udev/scsi_id CommandFilter and can't get the filter to work at all | 20:30 |
| hemna | I had to add /lib/udev to the paths in rootwrap.conf to make it work | 20:31 |
| xyang | hemna: seems work for us | 20:31 |
| hemna | ok, I'll try hacking some more | 20:32 |
| *** jasondotstar has joined #openstack-cinder | 20:32 | |
| hemna | I'd like to not have to add /lib/udev to the rootwrap.conf paths | 20:32 |
| xyang | hemna: ha | 20:33 |
| xyang | safe | 20:33 |
| xyang | hemna: wrong word | 20:33 |
| xyang | hemna: where is it? It is not in there | 20:34 |
| *** kaisers has quit IRC | 20:34 | |
| hemna | this is on a patch I'm working on locally | 20:34 |
| xyang | hemna: ok | 20:34 |
| hemna | xyang, so the ScaleIOConnector has the command being executed as drv_cfg, but the rootwrap filter lists the entire path. | 20:35 |
| hemna | https://github.com/openstack/os-brick/blob/master/os_brick/initiator/connector.py#L1588 | 20:35 |
| hemna | I'm curious how that works | 20:35 |
| hemna | since the /opt/emc/scaleio/sdc/bin isn't in the env path anywhere | 20:36 |
| *** salv-orlando has quit IRC | 20:36 | |
| xyang | Hemna: there is other lib needs to be installed | 20:37 |
| xyang | hemna: the Scaleio sdc | 20:37 |
| hemna | I just updated my filter to include the full path | 20:38 |
| hemna | but only call scsi_id from the putils.exec and it seems to work as well | 20:38 |
| hemna | oh...I think I still have the paths addition in rootwrap.conf though | 20:38 |
| hemna | that still works | 20:38 |
| hemna | ok | 20:38 |
| hemna | :) | 20:38 |
| xyang | hemna: which one?:) | 20:39 |
| hemna | my patch | 20:39 |
| xyang | Ok | 20:39 |
| hemna | I was calling /lib/udev/scsi_id in my python code's putils.exec | 20:39 |
| hemna | and the filter was puking | 20:39 |
| hemna | had to change it to simply call 'scsi_id' | 20:40 |
| hemna | and the filter found it and worked | 20:40 |
| xyang | hemna: oh, ok | 20:40 |
| jgriffith | thingee: guitarzan geguileo eharney Ok... I've voiced my opinion here: https://review.openstack.org/#/c/202615/ | 20:40 |
| jgriffith | I'll stay out of the way if the consensus is that I'm "crazy" | 20:40 |
| jgriffith | and also fully support and help no matter what direction we go with things | 20:41 |
| *** ronis has quit IRC | 20:41 | |
| guitarzan | jgriffith: does you being crazy have any other impact here? :) | 20:41 |
| geguileo | jgriffith: Thanks for reviewing the spec | 20:41 |
| jgriffith | guitarzan: LOL | 20:41 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/206877 | 20:41 |
| geguileo | jgriffith: I'll read and reply tomorrow, as I was just about to go have dinner (it's 22:42 here) | 20:42 |
| jgriffith | geguileo: Oh, no rush | 20:42 |
| hemna | so, how does the load balancing work when one node has a local file lock operating on a volume, and another request for some other action happens on the same volume getting balanced to another node ? | 20:43 |
| geguileo | jgriffith: Well, Friday is the last day for specs, so there is some hurry ;-) | 20:43 |
| hemna | if we are going to use a higher level package to do this? | 20:43 |
| guitarzan | jgriffith: that's a pretty good post, but I think zookeeper is probably a better example than docker | 20:44 |
| geguileo | hemna: Local file locks are only the firs step | 20:44 |
| geguileo | hemna: They will only be used for A-P configurations | 20:44 |
| hemna | I think for this spec to have value, we have to nuke the local file locks | 20:44 |
| geguileo | hemna: For A-A a DLM will be needed | 20:44 |
| hemna | but I'm sure I'm missing something | 20:44 |
| geguileo | hemna: But then only a configuration change will be needed | 20:44 |
| hemna | or confused, or stupid....or both :) | 20:45 |
| geguileo | hemna: I don't know if I was able to explain it properly | 20:45 |
| jgriffith | guitarzan: blahhh... ZK | 20:46 |
| jgriffith | :) | 20:46 |
| *** salv-orlando has joined #openstack-cinder | 20:46 | |
| guitarzan | I think whatever route is "blessed" you have to deal with multiple c-vols for the same backend, and that'll require some level of cooperation | 20:46 |
| *** Yogi1 has quit IRC | 20:46 | |
| hemna | guitarzan, yup, and the local file locks are a big blocker to that working right now. | 20:47 |
| guitarzan | hemna: no argument there at all | 20:47 |
| geguileo | The idea is to use Tooz with a DLM for Active-Active configurations, but as a first step we just replace current local locks with Tooz local locks, thus leaving everything as it is. And for Active-Passive configuration this will not change, but for Active-Active cinder will be configured to use a DLM (one of those provided by Tooz, like Zookeeper or Redis). | 20:47 |
| jgriffith | guitarzan: yeah, or a backend that's just able to behave in a multi-tenant env. Which is the second/bigger part of the problem here IMO. | 20:47 |
| hemna | and I don't think switching those local file locks to tooz locks is really the right way to go. | 20:47 |
| geguileo | guitarzan: That's what Tooz and the DLM are for | 20:47 |
| jgriffith | geguileo: go eat!!! | 20:47 |
| geguileo | guitarzan: Instead fo local file locks you use a DLM (redis or Zookeeper) | 20:47 |
| jgriffith | geguileo: I didn't mean to stir all this up again late in the day | 20:48 |
| geguileo | jgriffith: I want to, but the conversation here is interesting!! ;-) | 20:48 |
| hemna | hehe | 20:48 |
| guitarzan | geguileo: I think I'm on the same page as you | 20:48 |
| jgriffith | geguileo: FWIW, I think guitarzan was agreeing with you | 20:48 |
| geguileo | XD XD XD | 20:48 |
| hemna | ok, so if this is the first baby step towards moving to atomic volume operations utilizing the API, then I'm ok with it. | 20:48 |
| * jgriffith is the only dissenting voice :) | 20:48 | |
| geguileo | Ok, then I must rest if I'm arguing with those that agree with me XD | 20:48 |
| guitarzan | I think there is something to be said to just using db state transitions as the only real source of locks | 20:49 |
| hemna | heh, we churned on that in Vancouver | 20:49 |
| guitarzan | yes sir | 20:49 |
| geguileo | hemna: Will you be introducing a new Reading state for resources? | 20:49 |
| guitarzan | someone was trying to argue about using a non acid db | 20:49 |
| geguileo | hemna: I mean to be able to return VolumeIsBusy when the resource is being used for reading | 20:49 |
| guitarzan | geguileo: not specifically | 20:49 |
| *** bswartz has quit IRC | 20:49 | |
| hemna | geguileo, that will have to be part of it, but Nova needs to cope as well | 20:50 |
| hemna | which is the bigger hurdle IMO | 20:50 |
| geguileo | hemna: Yes, I agree on the Nova part (pita) | 20:50 |
| geguileo | hemna: I was just wondering on how you intend to remove the locks using just the status | 20:50 |
| *** edmondsw has joined #openstack-cinder | 20:50 | |
| geguileo | hemna: Because when I've been looking at doing the locking with only the DB I've found that it's even more complicated | 20:51 |
| guitarzan | geguileo: why is that? | 20:51 |
| hemna | the first step is to get nova updated to deal w/ VolumeIsBusy first. | 20:51 |
| guitarzan | isn't it easier to just fail any call where the thing isn't in the right state? | 20:51 |
| geguileo | guitarzan: Because we don't change the status of the resource when reading | 20:51 |
| guitarzan | it is much more difficult to figure out if something dies though | 20:51 |
| guitarzan | geguileo: I'm not sure I understand the concept of "reading" | 20:51 |
| geguileo | guitarzan: So in the API we have no idea when a resource is being used for reading (cloning a volume for example) | 20:51 |
| geguileo | guitarzan: Reading is when a resource is being used for another operation, but just for reading | 20:52 |
| geguileo | guitarzan: Like creating a volume with another as the source (cloning) | 20:52 |
| geguileo | guitarzan: Or creaing a volume from a snapshot | 20:52 |
| guitarzan | why does reading matter vs any other operation? | 20:52 |
| hemna | geguileo, if a volume is being cloned via cinder, then the volume's state should be busy | 20:52 |
| geguileo | hemna: But we don't have that right now | 20:52 |
| guitarzan | geguileo: sure, I get it | 20:52 |
| *** dustins has joined #openstack-cinder | 20:52 | |
| guitarzan | so yeah, those are two good examples | 20:53 |
| geguileo | hemna: That's why I was asking if you were going to introduce new statuses | 20:53 |
| hemna | ah I see. yah that's a good point, and another portion of the solution I suppose. | 20:53 |
| hemna | something to talk about next week | 20:54 |
| guitarzan | I think the state only solution only has the problem of no heartbeats | 20:54 |
| geguileo | OK | 20:54 |
| guitarzan | other than that I think it'd work | 20:54 |
| geguileo | guitarzan: I think there's a way to fix that | 20:54 |
| guitarzan | I'm interested in hearing what that would look like | 20:54 |
| geguileo | guitarzan: But I want to give it a good thought together with the idea that thingee proposed earlier | 20:54 |
| guitarzan | sure, you also need to go sleep :) | 20:55 |
| geguileo | guitarzan: First dinner ;-) | 20:55 |
| guitarzan | no rush | 20:55 |
| guitarzan | true, food is good | 20:55 |
| geguileo | See you tomorrow guys | 20:55 |
| *** ByPasS has quit IRC | 20:55 | |
| *** akerr has quit IRC | 20:56 | |
| *** ByPasS has joined #openstack-cinder | 20:56 | |
| *** rmcall has joined #openstack-cinder | 20:56 | |
| *** dustins has left #openstack-cinder | 20:56 | |
| *** Guest100_ has quit IRC | 20:56 | |
| *** dustins has joined #openstack-cinder | 20:56 | |
| *** haomaiwa_ has quit IRC | 21:01 | |
| *** 6JTABPDW5 has joined #openstack-cinder | 21:02 | |
| *** jordanP has joined #openstack-cinder | 21:07 | |
| *** mylu has quit IRC | 21:09 | |
| *** edtubill has left #openstack-cinder | 21:09 | |
| *** dustins has quit IRC | 21:09 | |
| *** mylu has joined #openstack-cinder | 21:10 | |
| *** ByPasS has quit IRC | 21:13 | |
| *** ekarlso has quit IRC | 21:16 | |
| *** ByPasS has joined #openstack-cinder | 21:18 | |
| mriedem | not helpful: http://publiclogs.emc.com/sio_ostack/EMC_SCALEIO/7 | 21:22 |
| *** e0ne has quit IRC | 21:24 | |
| *** mylu has quit IRC | 21:25 | |
| *** mylu has joined #openstack-cinder | 21:25 | |
| *** mylu has quit IRC | 21:25 | |
| jgriffith | mriedem: nope, not really | 21:26 |
| mriedem | https://bugs.launchpad.net/cinder/+bug/1479530 | 21:27 |
| openstack | Launchpad bug 1479530 in Cinder "emc scaleio uses drv_cfg exe which isn't in volume.filters" [Undecided,New] | 21:27 |
| mriedem | xyang: ^ | 21:27 |
| xyang | mriedem: thanks:). I'll fix it | 21:27 |
| *** cbader has quit IRC | 21:28 | |
| xyang | mriedem: our Scaleio CI is in the middle of transition, that is why that link is empty | 21:34 |
| mriedem | xyang: ah | 21:36 |
| mriedem | moving from china to the US? | 21:36 |
| mriedem | i only ask b/c that's a common theme | 21:36 |
| xyang | mriedem: moving from a contractor to internal | 21:37 |
| mriedem | oh that's fun too | 21:37 |
| mriedem | intel recently had some issues with their ci in china being blocked by the firewall | 21:37 |
| mriedem | was down for about a month and no one realized :( | 21:37 |
| xyang | mriedem: let me find an older link, I see that we used our own filter | 21:37 |
| *** akerr has joined #openstack-cinder | 21:38 | |
| *** akerr has quit IRC | 21:38 | |
| xyang | mriedem: the contractor CI is still standing by and running os-brick CI | 21:38 |
| xyang | Until end of August | 21:38 |
| *** annashen has quit IRC | 21:39 | |
| xyang | mriedem: review.openstack.org/#/c/183762/ | 21:40 |
| xyang | mriedem: go to Scaleio cinder CI and go to controller node | 21:40 |
| *** rmcall has quit IRC | 21:41 | |
| mriedem | http://storage.superna.net/openstack/cinder/scaleio/build-260/controller/etc/cinder/rootwrap.d/scaleio.filters.gz | 21:41 |
| mriedem | ah | 21:41 |
| *** jasondotstar has quit IRC | 21:41 | |
| xyang | mriedem: ya, I see that , we need to fix that after it is added to volume filters | 21:42 |
| *** annashen has joined #openstack-cinder | 21:43 | |
| *** ekarlso has joined #openstack-cinder | 21:44 | |
| *** BharatK has quit IRC | 21:48 | |
| hemna | hrmm jungleboyj Storewize CI looks broken | 21:49 |
| hemna | jungleboyj, http://goo.gl/ULkkHJ | 21:49 |
| *** ByPasS has quit IRC | 21:51 | |
| *** bswartz has joined #openstack-cinder | 21:53 | |
| *** ByPasS has joined #openstack-cinder | 21:53 | |
| *** kvidvans has joined #openstack-cinder | 21:58 | |
| kvidvans | thingee: Hi Mike, A BP i filed, https://blueprints.launchpad.net/cinder/+spec/oracle-zfssa-volume-migration needs your approval . Let me know if i need to add any other details to it for approval. Thanks | 22:00 |
| *** 6JTABPDW5 has quit IRC | 22:01 | |
| *** markvoelker has quit IRC | 22:02 | |
| *** 16WAAEHWP has joined #openstack-cinder | 22:02 | |
| *** erlon has quit IRC | 22:03 | |
| *** BharatK has joined #openstack-cinder | 22:04 | |
| *** mriedem has quit IRC | 22:04 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Fix Oracle ZFSSA iSCSI driver to return free space https://review.openstack.org/207106 | 22:05 |
| *** annashen_ has joined #openstack-cinder | 22:07 | |
| openstackgerrit | abhiram moturi proposed openstack/cinder: Oracle ZFSSA iSCSI driver to return free space for non thin provisioned storage https://review.openstack.org/207106 | 22:08 |
| jungleboyj | hemna: Thanks. Was working when I looked yesterday. I will ping our person in a bit. | 22:10 |
| *** daneyon has quit IRC | 22:11 | |
| hemna | jungleboyj, ok thanks man. | 22:11 |
| *** annashen_ has quit IRC | 22:12 | |
| jungleboyj | Gotta drop for a bit. | 22:12 |
| hemna | l8s | 22:13 |
| *** hodos has quit IRC | 22:14 | |
| *** jungleboyj has quit IRC | 22:17 | |
| thingee | kvidvans: done | 22:18 |
| hemna | thingee, https://blueprints.launchpad.net/cinder/+spec/add-create-export-connector | 22:21 |
| hemna | when you have time...please :) | 22:21 |
| hemna | I've had that one up for a while, including spec and code. fwiw. | 22:22 |
| *** jasondotstar has joined #openstack-cinder | 22:28 | |
| thingee | hemna: about that spec, the use case didn't make sense to me. | 22:29 |
| *** jordanP has quit IRC | 22:29 | |
| thingee | hemna: I remember jgriffith mentioning that the way we expect drivers to be doing exports is a bit of a mess. | 22:30 |
| thingee | rather not exports, but just doing attaches because of what some drivers end up doing. jgriffith can you speak on that again? | 22:30 |
| jgriffith | thingee: well... I was just pointing out that some drivers diverged and did "their own thing" | 22:31 |
| thingee | jgriffith: https://review.openstack.org/#/c/202765/3/specs/liberty/create-export-connector.rst,cm | 22:32 |
| jgriffith | thingee: hemna haven't looked at the spec.... I'll look now | 22:32 |
| hemna | thingee, according to jgriffith's blog post about the subject and my research of live migration, we need that change. | 22:32 |
| thingee | sorry which blog post? | 22:33 |
| hemna | http://j-griffith.blogspot.com/2015/07/volume-attach-code-flow-in-cinder.html | 22:33 |
| hemna | the idea of create_export is to do just that, create the target exported on the array. There are many drivers that cannot do that without having that connector object passed in to create_export. | 22:33 |
| hemna | and is most likely why there are quite a bit of drivers that do nothing inside of create_export and do everything inside initialize_connection now. | 22:34 |
| hemna | initialize_connection is supposed to be just collecting data to return. not doing exports. | 22:34 |
| *** akerr has joined #openstack-cinder | 22:35 | |
| *** ByPasS has quit IRC | 22:35 | |
| hemna | I documented the list of drivers that do exporting and other things inside of initialize_connection today, on the CinderNovaAPI etherpad. I think there are about 30 https://etherpad.openstack.org/p/CinderNovaAPI | 22:36 |
| *** ByPasS has joined #openstack-cinder | 22:37 | |
| jgriffith | hemna: seems like that works... only question I would have which is what thingee was mentioning I think is as I mentioned the other week.... | 22:38 |
| jgriffith | hemna: maybe the right answer would be to modify the drivers to all "do" the same thing | 22:38 |
| jgriffith | hemna: in other words, create_export may not really "do" much in your driver, and the initialize_connection is where you would update things and return a model-update etc | 22:39 |
| jgriffith | hemna: I'm not saying that's better/worse | 22:39 |
| jgriffith | hemna: just think that might be the comment I made that's in question. | 22:39 |
| jgriffith | hemna: and the reason I made that comment is not that I'm opposed to passing the connector in on export... | 22:39 |
| hemna | I'd just like the api to be consistent | 22:39 |
| hemna | and right now it's not | 22:40 |
| jgriffith | hemna: but I am sort of unhappy that there are different interpretations of what create_export does | 22:40 |
| jgriffith | hemna: yes... we're saying the same thing I think | 22:40 |
| hemna | because some drivers do stuff here create_export and others in init_conn, | 22:40 |
| jgriffith | hemna: but you're change doesn't fix that | 22:40 |
| Swanson | If everything stays in initialize_connection you just need to make sure you can handle being called more than once without blowing up (I couldn't). Then I wouldn't have to make any create_export tests. | 22:40 |
| jgriffith | hemna: it's still inconsistent no? | 22:40 |
| hemna | my change allows driver developers to fix it | 22:40 |
| hemna | it's impossible otherwise | 22:40 |
| jgriffith | Swanson: coorect | 22:40 |
| jgriffith | hemna: well... I'm not saying i don't like your change. Let's just be clear | 22:41 |
| hemna | sure | 22:41 |
| jgriffith | hemna: but I am saying it's not really the only option | 22:41 |
| hemna | yah I get that totally. | 22:41 |
| Swanson | On the flip side with hemna's change I can actually create the export in create_export. Which I can't do now. | 22:41 |
| hemna | maybe I can add that to the alternatives in the spec? | 22:41 |
| jgriffith | Swanson: right | 22:41 |
| hemna | I just like the idea of folks using the API for what it's meant for | 22:41 |
| jgriffith | hemna: honestly I'm unclear on the need for a spec.. .this is pretty much a bug IMO | 22:41 |
| jgriffith | but that's not up to me | 22:41 |
| hemna | otherwise, why have create_export at all. | 22:42 |
| jgriffith | hemna: well... that's kinda the next question :) | 22:42 |
| hemna | I figured since it was a change to the driver API, it required a spec | 22:42 |
| thingee | yeah, I'm not sure a spec is needed | 22:42 |
| jgriffith | hemna: meh.. it sure doesn't hurt anything | 22:42 |
| hemna | was just trying to do the right thing fwiw | 22:42 |
| *** angela-s|away is now known as angela-s | 22:42 | |
| jgriffith | hemna: so FWIW, I'm ok with your patch; that was pretty much my first thought on how to fix it as well. | 22:43 |
| *** ByPasS has quit IRC | 22:43 | |
| thingee | jgriffith: perhaps this is the first step for us to have drivers to what we expect. | 22:43 |
| Swanson | I'm offended that we have something called "initialize_connection" where we might not be initializing a connection but rather returning information about one. | 22:43 |
| jgriffith | hemna: but it doesn't fix the problem of inconsistent behaviors/expectations which is a bummer | 22:43 |
| hemna | thingee, +1 | 22:43 |
| *** jaypipes has quit IRC | 22:43 | |
| *** ByPasS has joined #openstack-cinder | 22:43 | |
| jgriffith | thingee: well.. it gives them an "either or" which is what lead us to problems to begin with. And IMHO it will lead to more of that and likely cause a problem in the future | 22:44 |
| jgriffith | hemna: thingee again... not saying this isn't the correct answer for now | 22:44 |
| hemna | jgriffith, this is true. I think it's a 2 phase approach. 1) get this in and 2) file bugs against drivers? and/or be more diligent with driver reviews to ensure the consistency. | 22:44 |
| jgriffith | hemna: thingee but I do believe there's deeper follow up work to be done | 22:44 |
| hemna | I have plans to update our 4 drivers after this lands (if it lands) to use create_export as it was intended. | 22:44 |
| jgriffith | hemna: yeah, that's cool | 22:45 |
| Swanson | hemna: +1. I'll move to create_export once it functions such that it can. | 22:45 |
| *** esker has quit IRC | 22:46 | |
| thingee | jgriffith: I'm unsure otherwise. Some target drivers need the connector info for initialize_connection | 22:46 |
| *** dshaw_ has joined #openstack-cinder | 22:46 | |
| *** dshaw_ has quit IRC | 22:46 | |
| thingee | jgriffith: iscsi for multipath check, lio for initiator key | 22:47 |
| jgriffith | thingee: right... so this change is ok because it doesn't break that | 22:47 |
| thingee | jgriffith: well I'm saying if we don't want the either or...it'll be difficult to remove connector from initialize_connection | 22:47 |
| jgriffith | thingee: oh.. yeah, maybe | 22:47 |
| hemna | thingee, we don't want the connector removed from initialize_connection though. | 22:48 |
| jgriffith | thingee: I think it's doable though personally. The other thing is that rather than modify create_export, the other option here was to modify intiialize_connection to receive a model update and update there if present | 22:48 |
| hemna | it contains some info that we need to find the right exported target | 22:48 |
| thingee | hemna: right, so I'm just saying I'm having a hard time of thinking how to enforce things via code instead of us being diligent in reviews. | 22:48 |
| jgriffith | thingee: which wouldn't require changing the signature... just an optional return (tuple) that could be None | 22:48 |
| hemna | we have to be more diligent in reviews either way :) | 22:49 |
| *** samueldmq has quit IRC | 22:49 | |
| hemna | so if we want to go the route of just doing everything in initialize_connection, then I don't see the need to even keep create_export. | 22:50 |
| hemna | it's confusing as heck | 22:50 |
| *** lcurtis has quit IRC | 22:50 | |
| *** ericksonsantos has quit IRC | 22:50 | |
| jgriffith | hemna: well.. more than half of the drivers are still using that :) | 22:50 |
| *** abehl has quit IRC | 22:50 | |
| hemna | damned if we do.....damned if we don't..... | 22:50 |
| patrickeast | fwiw it would probably be beneficial either way to add more documentation into the base driver class explaining what the methods are actually for | 22:50 |
| *** ByPasS has quit IRC | 22:50 | |
| jgriffith | hemna: kinda | 22:50 |
| jgriffith | patrickeast: where in the base driver? | 22:51 |
| jgriffith | patrickeast: there are 4 instances of that method in the base driver now | 22:51 |
| hemna | patrickeast, I added that docs in my cinder patch related to the spec fwiw | 22:51 |
| patrickeast | hah | 22:51 |
| jgriffith | patrickeast: which one gets documented/updated? | 22:51 |
| thingee | patrickeast: you just opened a whole other can of worms. | 22:51 |
| patrickeast | at least one needs it | 22:51 |
| hemna | run! | 22:51 |
| jgriffith | patrickeast: hehe | 22:51 |
| jgriffith | patrickeast: hemna thingee so IMO we just need to add a "driver-development" page to the docs | 22:52 |
| jgriffith | and remember it's there and update it :) | 22:52 |
| jgriffith | or merge my patch that has a single base driver :) | 22:52 |
| *** ByPasS has joined #openstack-cinder | 22:52 | |
| jgriffith | anyway.. thingee hemna I'm fine with adding the connector info to create_export | 22:52 |
| hemna | a driver dev docs page would be nice to have. | 22:52 |
| earlephilhower | hemna: +10000 | 22:53 |
| openstackgerrit | Kurt Martin proposed openstack/cinder: Over subscription for HP LeftHand iSCSI driver https://review.openstack.org/207248 | 22:56 |
| *** markvoelker_ has joined #openstack-cinder | 22:57 | |
| *** ByPasS has quit IRC | 22:59 | |
| *** 16WAAEHWP has quit IRC | 23:01 | |
| kmartin | thingee, if you have a minute I added a driver BP that needs your approval https://blueprints.launchpad.net/cinder/+spec/over-subscription-lefthand | 23:01 |
| patrickeast | ^ do all driver ehancement type features like that need approved bp’s | 23:02 |
| *** haomaiwang has joined #openstack-cinder | 23:02 | |
| patrickeast | ? | 23:02 |
| kmartin | patrickeast, I think so, you added one for this feature | 23:02 |
| thingee | I've done it for tracking purposes. | 23:03 |
| patrickeast | kmartin: haha yea, i have added them in the past and been told it wasn’t really needed | 23:03 |
| * thingee has never changed his answer | 23:03 | |
| patrickeast | thingee: in that case i might have a couple little ones real quick for the pure drivers | 23:03 |
| kmartin | what thingee said, helps for tracking and review of what's going in the release | 23:03 |
| patrickeast | yea that makes sense to me | 23:04 |
| *** ByPasS has joined #openstack-cinder | 23:04 | |
| *** alexpilotti has quit IRC | 23:05 | |
| *** daneyon has joined #openstack-cinder | 23:09 | |
| *** Apoorva has quit IRC | 23:10 | |
| *** daneyon_ has joined #openstack-cinder | 23:10 | |
| patrickeast | thingee: ok, sorry for the last minute request, if you could take a look and approve these http://paste.openstack.org/show/406325/ it would be much appreciated | 23:13 |
| patrickeast | they should be pretty small changes, just pending other features that are slipping into L-3 | 23:13 |
| *** daneyon has quit IRC | 23:13 | |
| kmartin | patrickeast, driver only BPs do not have the strict deadline that general cinder BPs have if I remember correctly. | 23:15 |
| kmartin | thingee, ^ correct? | 23:15 |
| patrickeast | haha yea, i’ve heard that before too | 23:15 |
| kmartin | me too | 23:16 |
| patrickeast | but i’m not having a good day for remembering correct things :( | 23:16 |
| patrickeast | better safe than sending a FFE email | 23:16 |
| patrickeast | :D | 23:16 |
| thingee | yes and the point is to have a cut. | 23:16 |
| thingee | take a look at the l-3 page. plenty to review soon | 23:17 |
| *** diogogmt has quit IRC | 23:17 | |
| kmartin | patrickeast, you have me scared now, going to go add the 3PAR over subscription BP now :) | 23:17 |
| dannywilson | kmartin: hah, did the phrase "FFE email" do it? :) | 23:18 |
| kmartin | dannywilson, lol, we have to have a cut line at some point like Mike said. | 23:19 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Add multipath support to 3PAR iSCSI driver https://review.openstack.org/180779 | 23:25 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Add multipath support to 3PAR iSCSI driver https://review.openstack.org/180779 | 23:26 |
| *** annegentle has joined #openstack-cinder | 23:28 | |
| *** cdelatte has joined #openstack-cinder | 23:28 | |
| *** annashen has quit IRC | 23:28 | |
| *** daneyon_ has quit IRC | 23:29 | |
| *** delattec has quit IRC | 23:30 | |
| leeantho | thingee, when you have a moment can you approve this driver blueprint? https://blueprints.launchpad.net/cinder/+spec/3par-iscsi-multipath-support | 23:30 |
| *** zhenguo has joined #openstack-cinder | 23:31 | |
| *** ByPasS has quit IRC | 23:32 | |
| thingee | tbarron: ping | 23:32 |
| *** jasondotstar has quit IRC | 23:33 | |
| *** ByPasS has joined #openstack-cinder | 23:34 | |
| *** IlyaG has quit IRC | 23:38 | |
| *** jamielennox is now known as jamielennox|away | 23:39 | |
| *** mugithi has joined #openstack-cinder | 23:41 | |
| *** Apoorva has joined #openstack-cinder | 23:45 | |
| *** jasondotstar has joined #openstack-cinder | 23:45 | |
| *** akshai has quit IRC | 23:46 | |
| *** mugithi has quit IRC | 23:47 | |
| *** rmcall has joined #openstack-cinder | 23:47 | |
| *** angela-s has quit IRC | 23:53 | |
| *** daneyon has joined #openstack-cinder | 23:53 | |
| *** amickus has quit IRC | 23:54 | |
| *** akshai has joined #openstack-cinder | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!