| *** dannywilson has quit IRC | 00:00 | |
| *** DericHorn-HP has quit IRC | 00:01 | |
| *** dannywil_ has quit IRC | 00:01 | |
| *** Longgeek has quit IRC | 00:03 | |
| *** Rockyg has quit IRC | 00:05 | |
| *** dims has joined #openstack-cinder | 00:12 | |
| *** tsekiyam_ has joined #openstack-cinder | 00:13 | |
| *** mtanino has quit IRC | 00:16 | |
| *** tsekiyama has quit IRC | 00:17 | |
| *** tsekiyam_ has quit IRC | 00:18 | |
| *** boris-42 has joined #openstack-cinder | 00:30 | |
| *** chlong has joined #openstack-cinder | 00:32 | |
| *** leeantho has quit IRC | 00:40 | |
| *** xyang has quit IRC | 00:42 | |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP generic image cache https://review.openstack.org/195795 | 00:46 |
|---|---|---|
| *** heyun has joined #openstack-cinder | 00:46 | |
| openstackgerrit | Patrick East proposed openstack/cinder: WIP generic image cache https://review.openstack.org/195795 | 00:50 |
| *** patrickeast has quit IRC | 00:53 | |
| *** _cjones_ has quit IRC | 00:56 | |
| *** annegentle has joined #openstack-cinder | 01:02 | |
| *** annegentle has quit IRC | 01:06 | |
| *** zhenguo has joined #openstack-cinder | 01:07 | |
| *** MentalRay has quit IRC | 01:12 | |
| *** MentalRay has joined #openstack-cinder | 01:14 | |
| *** lixiaoy1 has joined #openstack-cinder | 01:16 | |
| *** markvoelker has quit IRC | 01:19 | |
| *** markvoelker has joined #openstack-cinder | 01:19 | |
| *** markvoelker has quit IRC | 01:22 | |
| *** davechen has joined #openstack-cinder | 01:23 | |
| *** Yogi1 has joined #openstack-cinder | 01:23 | |
| *** markvoelker has joined #openstack-cinder | 01:23 | |
| *** markvoelker has quit IRC | 01:23 | |
| *** markvoelker has joined #openstack-cinder | 01:24 | |
| *** DericHorn-HP has joined #openstack-cinder | 01:24 | |
| *** MentalRay has quit IRC | 01:27 | |
| *** jasondotstar has quit IRC | 01:30 | |
| *** davechen1 has joined #openstack-cinder | 01:31 | |
| *** davechen has quit IRC | 01:33 | |
| *** laughterwym_ has joined #openstack-cinder | 01:34 | |
| *** laughterwym has quit IRC | 01:34 | |
| *** tobe has joined #openstack-cinder | 01:38 | |
| openstackgerrit | LisaLi proposed openstack/cinder: Add dependency check in RBD delete_snapshot https://review.openstack.org/193003 | 01:39 |
| *** tobe has quit IRC | 01:39 | |
| *** Yogi1 has quit IRC | 01:40 | |
| *** davechen has joined #openstack-cinder | 01:41 | |
| *** Lee1092 has joined #openstack-cinder | 01:44 | |
| *** davechen1 has quit IRC | 01:44 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 01:47 |
| *** Apoorva has quit IRC | 01:50 | |
| *** jasondotstar has joined #openstack-cinder | 01:59 | |
| *** jasondotstar has quit IRC | 02:00 | |
| *** jasondotstar has joined #openstack-cinder | 02:00 | |
| *** jasondotstar has quit IRC | 02:06 | |
| *** jasondotstar has joined #openstack-cinder | 02:06 | |
| *** chlong_ has joined #openstack-cinder | 02:08 | |
| *** chlong has quit IRC | 02:10 | |
| *** chlong__ has joined #openstack-cinder | 02:10 | |
| *** mutoulbj has joined #openstack-cinder | 02:10 | |
| *** chlong_ has quit IRC | 02:13 | |
| *** tobe has joined #openstack-cinder | 02:16 | |
| *** mriedem_away has quit IRC | 02:16 | |
| openstackgerrit | wanghao proposed openstack/cinder: Incremental backup improvements for L https://review.openstack.org/188328 | 02:28 |
| *** aorourke has quit IRC | 02:28 | |
| *** hemna has quit IRC | 02:28 | |
| *** markstur has quit IRC | 02:28 | |
| *** severson_ has quit IRC | 02:28 | |
| *** asselin has quit IRC | 02:28 | |
| *** severson_ has joined #openstack-cinder | 02:28 | |
| *** chlong_ has joined #openstack-cinder | 02:28 | |
| *** markstur has joined #openstack-cinder | 02:28 | |
| *** aorourke has joined #openstack-cinder | 02:29 | |
| *** hemna has joined #openstack-cinder | 02:29 | |
| *** asselin has joined #openstack-cinder | 02:29 | |
| *** chlong__ has quit IRC | 02:31 | |
| *** chlong has joined #openstack-cinder | 02:31 | |
| *** bkopilov has quit IRC | 02:32 | |
| *** chlong_ has quit IRC | 02:34 | |
| *** mriedem has joined #openstack-cinder | 02:45 | |
| *** Longgeek has joined #openstack-cinder | 03:00 | |
| *** davechen has quit IRC | 03:00 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Fix getting wwpn information in infortrend driver for DS4000 https://review.openstack.org/194524 | 03:01 |
| *** laughterwym has joined #openstack-cinder | 03:02 | |
| *** laughterwym_ has quit IRC | 03:02 | |
| *** davechen has joined #openstack-cinder | 03:03 | |
| *** sgotliv has joined #openstack-cinder | 03:04 | |
| *** Longgeek has quit IRC | 03:05 | |
| *** davechen1 has joined #openstack-cinder | 03:07 | |
| *** davechen has quit IRC | 03:09 | |
| *** diogogmt has joined #openstack-cinder | 03:15 | |
| *** davechen has joined #openstack-cinder | 03:18 | |
| *** davechen1 has quit IRC | 03:20 | |
| *** diogogmt_ has joined #openstack-cinder | 03:29 | |
| *** diogogmt has quit IRC | 03:29 | |
| *** diogogmt_ is now known as diogogmt | 03:29 | |
| *** diogogmt has quit IRC | 03:31 | |
| *** markvoelker has quit IRC | 03:31 | |
| *** yamada-h has joined #openstack-cinder | 03:35 | |
| *** bill_az has quit IRC | 03:36 | |
| openstackgerrit | wanghao proposed openstack/cinder: Do not allow to modify access for public volume type https://review.openstack.org/194549 | 03:37 |
| *** Longgeek has joined #openstack-cinder | 03:37 | |
| *** yamada-h has quit IRC | 03:40 | |
| *** laughterwym_ has joined #openstack-cinder | 03:44 | |
| *** laughterwym has quit IRC | 03:44 | |
| *** laughterwym__ has joined #openstack-cinder | 03:44 | |
| *** jasondotstar has quit IRC | 03:47 | |
| *** Apoorva has joined #openstack-cinder | 03:47 | |
| *** dims has quit IRC | 03:48 | |
| *** bkopilov has joined #openstack-cinder | 03:52 | |
| *** mriedem has quit IRC | 03:53 | |
| *** _cjones_ has joined #openstack-cinder | 03:57 | |
| *** _cjones_ has quit IRC | 04:02 | |
| *** Longgeek has quit IRC | 04:05 | |
| *** Longgeek has joined #openstack-cinder | 04:06 | |
| *** coolsvap|away is now known as coolsvap | 04:08 | |
| *** lixiaoy1 has quit IRC | 04:09 | |
| *** lixiaoy1 has joined #openstack-cinder | 04:10 | |
| *** david-lyle has quit IRC | 04:13 | |
| *** david-lyle has joined #openstack-cinder | 04:14 | |
| *** lixiaoy11 has joined #openstack-cinder | 04:15 | |
| *** lixiaoy1 has quit IRC | 04:17 | |
| *** asselin_ has joined #openstack-cinder | 04:17 | |
| *** bkopilov has quit IRC | 04:17 | |
| *** mdenny has quit IRC | 04:19 | |
| *** kmartin_ has joined #openstack-cinder | 04:19 | |
| *** kmartin has quit IRC | 04:19 | |
| *** mdenny has joined #openstack-cinder | 04:19 | |
| *** rakesh_mishra has joined #openstack-cinder | 04:20 | |
| *** rakesh_mishra has quit IRC | 04:20 | |
| *** rakesh_mishra has joined #openstack-cinder | 04:20 | |
| *** bkopilov has joined #openstack-cinder | 04:23 | |
| *** DericHorn-HP has quit IRC | 04:28 | |
| *** tobe has quit IRC | 04:31 | |
| *** shyama has joined #openstack-cinder | 04:31 | |
| *** markvoelker has joined #openstack-cinder | 04:32 | |
| *** tobe has joined #openstack-cinder | 04:32 | |
| *** DericHorn-HP has joined #openstack-cinder | 04:35 | |
| *** yamada-h has joined #openstack-cinder | 04:36 | |
| *** markvoelker has quit IRC | 04:36 | |
| *** laughterwym__ has quit IRC | 04:40 | |
| *** yamada-h has quit IRC | 04:41 | |
| *** avishay has joined #openstack-cinder | 04:45 | |
| *** laughterwym_ has quit IRC | 04:51 | |
| *** DericHorn-HP has quit IRC | 04:58 | |
| *** erlon has quit IRC | 05:00 | |
| *** rakesh_mishra has quit IRC | 05:03 | |
| *** rakesh_mishra has joined #openstack-cinder | 05:03 | |
| *** Apoorva has quit IRC | 05:08 | |
| *** rakesh_mishra has quit IRC | 05:13 | |
| *** zhenguo has quit IRC | 05:23 | |
| *** nihilifer has joined #openstack-cinder | 05:25 | |
| openstackgerrit | chenzongliang proposed openstack/cinder: Refactor Huawei Volume driver https://review.openstack.org/188240 | 05:27 |
| *** rakesh_mishra has joined #openstack-cinder | 05:30 | |
| *** BharatK has joined #openstack-cinder | 05:39 | |
| *** nkrinner has joined #openstack-cinder | 05:44 | |
| *** deepakcs has joined #openstack-cinder | 05:51 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 05:53 |
| *** markvoelker has joined #openstack-cinder | 05:56 | |
| *** Sharath_ has joined #openstack-cinder | 05:58 | |
| Sharath_ | Hi , am getting following error : die 427 'c-api did not start , how to resolve this ? | 05:58 |
| *** ankit_ag has joined #openstack-cinder | 06:00 | |
| *** shyama has quit IRC | 06:01 | |
| *** markvoelker has quit IRC | 06:01 | |
| *** bluex-pl has joined #openstack-cinder | 06:08 | |
| *** liuxg has joined #openstack-cinder | 06:12 | |
| DuncanT | Sharath_: Devstack? | 06:43 |
| Sharath_ | yes | 06:43 |
| *** shyama has joined #openstack-cinder | 06:43 | |
| Sharath_ | am using devstack : stable/kilo | 06:44 |
| DuncanT | Anything in the cinder-api log? | 06:44 |
| Sharath_ | yes | 06:44 |
| Sharath_ | http://www.paste.org/79136 this is the log , (no such option: config_file reason for error) | 06:44 |
| *** liuxg has quit IRC | 06:46 | |
| DuncanT | That's not an error I recognise, but it is your issue. Can you paste up your cinder.conf, please? (Watch out, it migh contain passwords) | 06:46 |
| Sharath_ | ok | 06:48 |
| *** BharatK has quit IRC | 06:49 | |
| Sharath_ | cinder.conf : http://www.paste.org/79140 | 06:50 |
| Sharath_ | yes it contain password : user_domain_id = default | 06:51 |
| Sharath_ | password = password | 06:51 |
| Sharath_ | username = cinder | 06:51 |
| Sharath_ | auth_url = http://192.168.56.101:35357 | 06:51 |
| Sharath_ | auth_plugin = password | 06:51 |
| *** sgotliv has quit IRC | 06:51 | |
| mkoderer | Sharath_: seems there is a syntax error in line 38? | 06:52 |
| Sharath_ | ok | 06:52 |
| *** belmoreira has joined #openstack-cinder | 06:52 | |
| Sharath_ | [database] : line 38 | 06:52 |
| Sharath_ | its correct | 06:52 |
| mkoderer | Sharath_: no 38 is just a "c" | 06:53 |
| mkoderer | at least that what I see in your pasted snipped | 06:53 |
| Sharath_ | oh yeah : connection = mysql://root:password@192.168.56.101/cinder?charset=utf8 | 06:53 |
| Sharath_ | its problem with paste.org | 06:55 |
| mkoderer | Sharath_: I don't see an obvious mistake.. but DuncanT is the expert :) | 06:56 |
| DuncanT | Lots of lines look truncated there, e.g. 23, 28, 29 | 06:58 |
| *** liuxg has joined #openstack-cinder | 07:00 | |
| *** bluex-pl has quit IRC | 07:01 | |
| *** boris-42 has quit IRC | 07:02 | |
| *** BharatK has joined #openstack-cinder | 07:03 | |
| *** anshul has joined #openstack-cinder | 07:05 | |
| openstackgerrit | TaoBai proposed openstack/cinder: Storwize Driver zone removing https://review.openstack.org/196966 | 07:06 |
| *** bluex-pl has joined #openstack-cinder | 07:07 | |
| *** ronis has joined #openstack-cinder | 07:09 | |
| *** shyama has quit IRC | 07:15 | |
| *** markus_z has joined #openstack-cinder | 07:19 | |
| *** kaisers1 has joined #openstack-cinder | 07:19 | |
| *** lpetrut has joined #openstack-cinder | 07:28 | |
| *** kaisers1 has quit IRC | 07:29 | |
| *** kaisers1 has joined #openstack-cinder | 07:29 | |
| *** liuxg has quit IRC | 07:33 | |
| *** e0ne has joined #openstack-cinder | 07:33 | |
| *** chlong has quit IRC | 07:35 | |
| *** Vikash has joined #openstack-cinder | 07:37 | |
| *** jordanP has joined #openstack-cinder | 07:37 | |
| *** e0ne has quit IRC | 07:37 | |
| Vikash | Unable to create volume with sample cinder driver (using devstack) | 07:38 |
| Vikash | detail info posted at https://ask.openstack.org/en/question/69120/failed-to-run-schedulecreatevolumetask-no-weighed-host/ | 07:39 |
| *** rakesh_mishra has quit IRC | 07:40 | |
| Vikash | can someone please help | 07:40 |
| *** ndipanov has quit IRC | 07:42 | |
| *** jistr has joined #openstack-cinder | 07:43 | |
| *** ndipanov has joined #openstack-cinder | 07:44 | |
| *** markvoelker has joined #openstack-cinder | 07:45 | |
| openstackgerrit | Jessy Lee proposed openstack/cinder: Fix getting wwpn information in infortrend driver for DS4000 https://review.openstack.org/194524 | 07:48 |
| *** markvoelker has quit IRC | 07:50 | |
| *** zhenguo has joined #openstack-cinder | 07:50 | |
| *** jistr has quit IRC | 08:03 | |
| *** jistr has joined #openstack-cinder | 08:10 | |
| *** rakesh_mishra has joined #openstack-cinder | 08:13 | |
| *** tobe has quit IRC | 08:17 | |
| winston-d | geguileo: hey | 08:19 |
| geguileo | winston-d: Hi | 08:19 |
| winston-d | geguileo: i'm confused about your comment regarding to https://review.openstack.org/#/c/189261/ | 08:19 |
| winston-d | geguileo: you agreed with me but you +1? | 08:19 |
| geguileo | winston-d: Well, everyone else want the change | 08:19 |
| geguileo | winston-d: And the patch is correct | 08:20 |
| winston-d | Is it? | 08:20 |
| geguileo | winston-d: Correct in the sense that the code is right | 08:20 |
| geguileo | winston-d: Not in the sense that I think we should not enforce reserved space for free virtual | 08:21 |
| winston-d | did you had a chance to take a looke the discussion around patch set 3? https://review.openstack.org/#/c/189261/3/cinder/scheduler/filters/capacity_filter.py | 08:22 |
| *** lpetrut has quit IRC | 08:22 | |
| geguileo | winston-d: Let me look | 08:22 |
| geguileo | winston-d: Mmmm, is it me or the specs specify current functionallity as desired? | 08:24 |
| geguileo | I thought this was not documented | 08:25 |
| winston-d | there is a spec about thin-provisioning | 08:25 |
| geguileo | That's the one I was reading right now | 08:25 |
| *** shyama has joined #openstack-cinder | 08:26 | |
| geguileo | And apparently Capacity filter is explicitly specified there | 08:26 |
| wanghao | Guys, I saw the most cinder gates are "UNSTABLE". What's going on? | 08:28 |
| wanghao | Looks like nova has same problem too.... | 08:29 |
| winston-d | yeah, Capacity filter needed to be change to support thin provision, that's the reason it was mentioned in the spec | 08:29 |
| *** bluex-pl has quit IRC | 08:31 | |
| *** bluex-pl has joined #openstack-cinder | 08:31 | |
| *** Nikolay_St has joined #openstack-cinder | 08:37 | |
| *** rakesh_mishra has quit IRC | 08:38 | |
| *** rakesh_mishra has joined #openstack-cinder | 08:38 | |
| *** yamada-h has joined #openstack-cinder | 08:39 | |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 08:41 |
| *** rakesh_mishra has quit IRC | 08:41 | |
| *** rakesh_mishra has joined #openstack-cinder | 08:42 | |
| openstackgerrit | Peter Wang proposed openstack/cinder: Add white list support for target ports in VNX driver https://review.openstack.org/196979 | 08:43 |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: ConsistencyGroup Object https://review.openstack.org/163641 | 08:43 |
| *** yamada-h has quit IRC | 08:44 | |
| *** rakesh_mishra has quit IRC | 08:46 | |
| -openstackstatus- NOTICE: OpenStack CI is down due to hard drive failures | 08:47 | |
| *** ChanServ changes topic to "OpenStack CI is down due to hard drive failures" | 08:47 | |
| geguileo | winston-d: Now I'm not sure if I completely understanding the problem... | 08:47 |
| geguileo | winston-d: This is what I see | 08:47 |
| geguileo | winston-d: For thin or thick only provisioning there's no difference | 08:48 |
| geguileo | winston-d: It's when mixing thin and thick that there's a different behavior | 08:48 |
| winston-d | well, for thick Capacity filter remains the same. its behavior changed for thin. | 08:49 |
| geguileo | winston-d: Right, for thin changed as well | 08:50 |
| geguileo | winston-d: So before we used "overprovisioned free" to check | 08:51 |
| geguileo | winston-d: I say overprovisioned instead of virtual because one is free * over_subscription_ratio and the other is total * oversubscription_ratio - provisioned | 08:52 |
| geguileo | winston-d: And now we will not check it, right? | 08:53 |
| *** sgotliv has joined #openstack-cinder | 08:54 | |
| Vikash | what is the meaning of this error: cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid host was found. No weighed hosts available | 09:00 |
| winston-d | geguileo: Vipin's change won't check size against overprovisioned free or virtual free | 09:01 |
| winston-d | Vikash: it means your scheduler can't find any volume service that satisfied the volume create requirement. | 09:01 |
| *** e0ne has joined #openstack-cinder | 09:02 | |
| Vikash | winston-d: Thanks, as i am new to openstack. Any clue what and where i went wrong in configuration | 09:05 |
| Vikash | winston-d: detailed information is posted here https://ask.openstack.org/en/question/69120/failed-to-run-schedulecreatevolumetask-no-weighed-host/ | 09:05 |
| geguileo | winston-d: winston-d But if it's checking that we have not exceeded over_provisioned_ratio we know that it hasn't exceeded virtual_free right? | 09:06 |
| *** sgotliv_ has joined #openstack-cinder | 09:06 | |
| winston-d | geguileo: over_provisioned_ratio doesn't take reserved_percentage into account, virtual_free should | 09:07 |
| *** sgotliv has quit IRC | 09:09 | |
| geguileo | winston-d: Oh, right | 09:10 |
| *** lpetrut has joined #openstack-cinder | 09:11 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:13 | |
| *** e0ne is now known as e0ne_ | 09:15 | |
| *** casusbelli has joined #openstack-cinder | 09:17 | |
| winston-d | need to run now. | 09:19 |
| *** winston-d is now known as winston-d_afk | 09:19 | |
| *** rakesh_mishra has quit IRC | 09:24 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:24 | |
| *** rakesh_mishra has quit IRC | 09:25 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:25 | |
| *** e0ne_ has quit IRC | 09:26 | |
| *** e0ne has joined #openstack-cinder | 09:29 | |
| *** breitz has quit IRC | 09:30 | |
| *** breitz has joined #openstack-cinder | 09:30 | |
| *** markvoelker has joined #openstack-cinder | 09:34 | |
| openstackgerrit | Daniel Tadrzak proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 09:35 |
| *** sgotliv_ has quit IRC | 09:39 | |
| *** markvoelker has quit IRC | 09:39 | |
| openstackgerrit | Andrey Pavlov proposed openstack/cinder: Fix doc string definitions https://review.openstack.org/193124 | 09:43 |
| *** aix has quit IRC | 09:46 | |
| shyama | Vikash: does your driver implement the get_volume_stats and publish its stats? the cinder scheduler will not select a backend if it does find enough free capacity | 09:50 |
| shyama | the cinder scheduler logs should have more information | 09:51 |
| *** rakesh_mishra has quit IRC | 09:51 | |
| *** rakesh_mishra has joined #openstack-cinder | 09:51 | |
| *** ociuhandu has quit IRC | 09:52 | |
| *** lixiaoy11 has quit IRC | 09:53 | |
| *** sgotliv_ has joined #openstack-cinder | 09:54 | |
| *** davechen has left #openstack-cinder | 09:55 | |
| Vikash | shyama: As part of learning I added all dummy defination (adding LOG and returning pass) | 09:55 |
| Vikash | to verify whether call is reaching to the new driver | 09:55 |
| openstackgerrit | Teruaki Ishizaki proposed openstack/cinder: sheepdog: Add class for dog command executor https://review.openstack.org/194532 | 09:56 |
| Vikash | shyama: Can you please let me know which all def are mandatory for cinder to atleast claim driver | 09:56 |
| *** Sharath_1 has joined #openstack-cinder | 10:02 | |
| *** Sharath_ has quit IRC | 10:02 | |
| openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder-specs: Cinder API WSGI application under Apache/Nginx https://review.openstack.org/192683 | 10:03 |
| openstackgerrit | Vincent Hou proposed openstack/cinder: Implement the update_migrated_volume for the drivers https://review.openstack.org/180873 | 10:06 |
| shyama | Vikash: what do the scheduler logs have? which filters are failing? | 10:11 |
| Vikash | shyama: logs posted here https://ask.openstack.org/en/question/69120/failed-to-run-schedulecreatevolumetask-no-weighed-host/ | 10:14 |
| *** coolsvap is now known as coolsvap|away | 10:14 | |
| Vikash | shyama: 2015-06-29 08:29:40.466 ERROR cinder.scheduler.flows.create_volume [req-4bf3a53a-6475-420f-8f87-f871c156d406 admin] Failed to run task cinder.scheduler.flows.create_volume.ScheduleCreateVolumeTask;volume:create: No valid host was found. No weighed hosts available | 10:15 |
| *** mutoulbj has quit IRC | 10:17 | |
| *** aix has joined #openstack-cinder | 10:17 | |
| *** rakesh_mishra has quit IRC | 10:18 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 10:18 | |
| shyama | For fake driver implementation check cinder.volume.driver.FakeISCSIDriver | 10:19 |
| *** e0ne is now known as e0ne_ | 10:23 | |
| *** e0ne_ is now known as e0ne | 10:25 | |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Add entry create and cast tasks to manage workflow https://review.openstack.org/139071 | 10:33 |
| openstackgerrit | Anton Arefiev proposed openstack/cinder: Fix lvm manage existing volume https://review.openstack.org/156939 | 10:33 |
| Sharath_1 | cinder failed : http://pastebin.com/ev9vXxay [cinder conf] | 10:34 |
| *** Vikash_ has joined #openstack-cinder | 10:39 | |
| *** rakesh_mishra_ has quit IRC | 10:44 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 10:45 | |
| *** chlong has joined #openstack-cinder | 10:49 | |
| *** ociuhandu has joined #openstack-cinder | 10:50 | |
| *** Vikash_ is now known as Vikash_cz | 10:50 | |
| *** BharatK has quit IRC | 10:51 | |
| *** rakesh_mishra_ has quit IRC | 10:51 | |
| Vikash_cz | shyama: I don't have def get_volume_stats defined in my dummy driver | 10:56 |
| *** BharatK has joined #openstack-cinder | 11:02 | |
| Vikash_cz | shayma: Getting interanal error while accepting DCC | 11:05 |
| *** dims has joined #openstack-cinder | 11:07 | |
| openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Tooz locks https://review.openstack.org/183537 | 11:08 |
| *** anshul has quit IRC | 11:08 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 11:10 | |
| *** rakesh_mishra_ has quit IRC | 11:17 | |
| *** IanGovett has joined #openstack-cinder | 11:18 | |
| *** anshul has joined #openstack-cinder | 11:22 | |
| smcginnis | Wow, gate looks pretty bad this morning. | 11:23 |
| *** sparr has quit IRC | 11:23 | |
| *** markvoelker has joined #openstack-cinder | 11:24 | |
| *** mutoulbj has joined #openstack-cinder | 11:24 | |
| smcginnis | Hah, outage due to drive failures. Darn storage. | 11:24 |
| *** heyun has quit IRC | 11:25 | |
| *** markvoelker has quit IRC | 11:29 | |
| *** sparr has joined #openstack-cinder | 11:38 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/os-brick: remotefs: add virtuozzo storage support https://review.openstack.org/188805 | 11:46 |
| *** anshul has quit IRC | 11:49 | |
| e0ne | smcginnis: thanks for notification! | 11:50 |
| *** Sharath_1 has left #openstack-cinder | 11:58 | |
| *** bill_az has joined #openstack-cinder | 11:58 | |
| *** jistr is now known as jistr|class | 11:59 | |
| *** rakesh_mishra_ has joined #openstack-cinder | 11:59 | |
| *** bkopilov has quit IRC | 12:01 | |
| *** anshul has joined #openstack-cinder | 12:01 | |
| smcginnis | e0ne: ;) | 12:02 |
| *** markvoelker has joined #openstack-cinder | 12:02 | |
| smcginnis | rakesh_mishra_: Hold off on the reverifies for now. CI is having some issues. | 12:02 |
| rakesh_mishra_ | @smcginnis: ok | 12:03 |
| *** marcusvrn has joined #openstack-cinder | 12:05 | |
| *** jistr|class is now known as jistr | 12:07 | |
| dulek | geguileo: Hi, what's happening in lines 94 and 101 here - https://review.openstack.org/#/c/183222/8..9/cinder/objects/backup.py? | 12:08 |
| dulek | self.cinder_obj_get_changes? | 12:09 |
| geguileo | dulek: Yep, remember last weeks conversation with superdan? | 12:10 |
| geguileo | dulek: He recommended not to overwrite obj_get_changes but create our own | 12:10 |
| dulek | Ah, right. Your patch is dependent. Thanks you! :) | 12:10 |
| geguileo | dulek: And use it instead, so it was clear that this was a cinderism | 12:10 |
| dulek | s/Thanks/Thank | 12:10 |
| geguileo | dulek: np | 12:11 |
| *** markvoelker_ has joined #openstack-cinder | 12:11 | |
| *** markvoel_ has joined #openstack-cinder | 12:13 | |
| *** markvoelker has quit IRC | 12:13 | |
| *** belmoreira has quit IRC | 12:15 | |
| *** markvoelker_ has quit IRC | 12:17 | |
| *** kjnelson has joined #openstack-cinder | 12:32 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 12:37 |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix saving tz aware datetimes in Versioned Objects https://review.openstack.org/196071 | 12:37 |
| *** rakesh_mishra_ has quit IRC | 12:39 | |
| *** edmondsw has joined #openstack-cinder | 12:39 | |
| *** zhenguo has quit IRC | 12:43 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port dothill to Python 3 https://review.openstack.org/197049 | 12:46 |
| *** rakesh_mishra_ has joined #openstack-cinder | 12:51 | |
| *** markvoel_ has quit IRC | 12:51 | |
| *** haypo has joined #openstack-cinder | 12:51 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port dothill to Python 3 https://review.openstack.org/197049 | 12:51 |
| *** Guest27041 has joined #openstack-cinder | 12:51 | |
| *** markvoelker_ has joined #openstack-cinder | 12:52 | |
| *** amoturi has quit IRC | 12:52 | |
| haypo | smcginnis: done ;) oh, you should read my comments on the patch set 1 | 12:52 |
| haypo | smcginnis: it explains why/how the code worked before my change :) (by accident) | 12:52 |
| smcginnis | haypo: ;) | 12:53 |
| *** dtynan has quit IRC | 12:53 | |
| haypo | smcginnis: i just discovered that os-brick is not compatible with Python 3 yet | 12:54 |
| *** dtynan has joined #openstack-cinder | 12:54 | |
| haypo | well, not fully | 12:54 |
| smcginnis | haypo: Not too bad I hope. | 12:54 |
| smcginnis | haypo: Are you working with (or aware of) the patch from James Page? | 12:56 |
| *** Vikash_cz_ has joined #openstack-cinder | 12:56 | |
| haypo | smcginnis: which patch? | 12:57 |
| smcginnis | haypo: https://review.openstack.org/#/c/194216 | 12:57 |
| *** smoriya has quit IRC | 12:58 | |
| *** Vikash_cz has quit IRC | 12:58 | |
| haypo | smcginnis: oh, i didn't know this patch | 12:58 |
| *** Vikash_cz_ is now known as Vikash_cz | 12:58 | |
| smcginnis | haypo: In your 5 second evaluation - does that look like it will address the issues? | 13:00 |
| smcginnis | :) | 13:00 |
| haypo | smcginnis: i wrote a very similar patch | 13:01 |
| smcginnis | haypo: Did you push that up? I didn't notice one. | 13:01 |
| *** markvoelker_ has quit IRC | 13:02 | |
| haypo | smcginnis: i worked on cinder last days locally, i'm cleaning my patches before sending them | 13:02 |
| *** markvoelker has joined #openstack-cinder | 13:02 | |
| haypo | dothill was the first patch | 13:02 |
| haypo | i have ~10 other patches in my queue :) | 13:02 |
| *** markvoelker_ has joined #openstack-cinder | 13:03 | |
| smcginnis | haypo: Patches that touch all the code are so much fun. :P | 13:03 |
| haypo | smcginnis: i'm now trying to write very short patches. otherwise, reviewers are too scared | 13:03 |
| haypo | the dothill is "very short" (compared to other of my patches) | 13:04 |
| smcginnis | True! | 13:04 |
| haypo | 12 files changed, 54 insertions(+), 43 deletions(-) | 13:04 |
| haypo | ^^ this patch is probably too big for example | 13:04 |
| *** deepakcs has quit IRC | 13:05 | |
| *** ankit_ag has quit IRC | 13:05 | |
| *** markvoelker_ has quit IRC | 13:05 | |
| smcginnis | haypo: Also depends on what the actual changes are. But I agree, probably easier to get smaller focused patches reviewed than larger ones that are all over the place. | 13:05 |
| *** markvoelker_ has joined #openstack-cinder | 13:06 | |
| haypo | smcginnis: this patch changes many different things to fix various tests | 13:06 |
| *** changbl has quit IRC | 13:06 | |
| haypo | i'm trying to fix tests one by one in cinder/tests/unit/test_*.py | 13:06 |
| haypo | i'm around the letter I :-) | 13:06 |
| scottda | winston-d_afk: e0ne DuncanT I've added some content to CinderNovaAPI etherpad. I'd like to flesh out more of the Design Questions and Use Cases. Goal is to bring some concrete proposals to the mid-cycle next month. | 13:07 |
| smcginnis | haypo: You've got a long road in front of you. :) | 13:07 |
| haypo | smcginnis: yep | 13:07 |
| *** markvoelker has quit IRC | 13:07 | |
| openstackgerrit | Victor Stinner proposed openstack/os-brick: Port os-brick to Python 3 https://review.openstack.org/197062 | 13:15 |
| haypo | smcginnis: i proposed to add a non-voting py34 check job to os-brick: https://review.openstack.org/197064 | 13:18 |
| *** julim has joined #openstack-cinder | 13:19 | |
| *** Yogi1 has joined #openstack-cinder | 13:20 | |
| *** nihilifer has quit IRC | 13:21 | |
| *** Guest27041 is now known as agentle | 13:21 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port hitachi driver to Python 3 https://review.openstack.org/197065 | 13:24 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port huawei driver to Python 3 https://review.openstack.org/197068 | 13:26 |
| scottda | dulek: I see your name in CinderNovaAPI etherpad as well. https://etherpad.openstack.org/p/CinderNovaAPI | 13:26 |
| scottda | and anyone else interested. | 13:26 |
| dulek | scottda: What's up? | 13:27 |
| scottda | I updated the CinderNovaAPI etherpad and wanted to alert those interested. I'd really like to work on Big Picture ® Design Ideas before the mid-cycle | 13:28 |
| *** MentalRay has joined #openstack-cinder | 13:28 | |
| *** MentalRay has quit IRC | 13:28 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port IBM driver to Python 3 https://review.openstack.org/197070 | 13:28 |
| dulek | I'll take a look, thanks! :) | 13:28 |
| *** e0ne is now known as e0ne_ | 13:29 | |
| *** boris-42 has joined #openstack-cinder | 13:31 | |
| *** thangp has joined #openstack-cinder | 13:31 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port test_db_api to Python 3 https://review.openstack.org/197072 | 13:32 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port remotefs driver to Python 3 https://review.openstack.org/197075 | 13:38 |
| *** mriedem has joined #openstack-cinder | 13:39 | |
| *** e0ne_ has quit IRC | 13:39 | |
| *** bkopilov has joined #openstack-cinder | 13:40 | |
| haypo | smcginnis: in cinder/volume/drivers/remotefs.py i found share_info[0].strip().decode('unicode_escape') in the code parsing the configuration file of shares | 13:42 |
| *** BharatK has quit IRC | 13:42 | |
| haypo | smcginnis: do you know if 'unicode_escape' was a deliberate choice? | 13:43 |
| haypo | smcginnis: it's a very special encoding, usually only used in python, internally | 13:43 |
| *** agentle has quit IRC | 13:43 | |
| haypo | smcginnis: it's a question for https://review.openstack.org/197075 | 13:44 |
| smcginnis | haypo: I do not know, but would guess it wasn't a deliberate choice. | 13:44 |
| *** kjnelson has quit IRC | 13:44 | |
| smcginnis | haypo: git blame points to eharney. Maybe once he's around we should see if he remembers. | 13:45 |
| *** Yogi1 has quit IRC | 13:46 | |
| *** e0ne has joined #openstack-cinder | 13:46 | |
| *** kjnelson has joined #openstack-cinder | 13:46 | |
| haypo | smcginnis: a deeper git blame gave me the answer: https://bugs.launchpad.net/cinder/+bug/1180984 | 13:54 |
| openstack | Launchpad bug 1180984 in Cinder "NFS driver breaks if volume name contains spaces" [Undecided,Fix released] - Assigned to Anton Frolov (anton0) | 13:54 |
| smcginnis | haypo: unicode_escape is needed for that? | 13:55 |
| *** BharatK has joined #openstack-cinder | 13:55 | |
| haypo | smcginnis: the purpose of unicode_escape is to support 'x.y.z.w:/foo\040bar' syntax where \040 is an escaped scape | 13:55 |
| *** eharney has joined #openstack-cinder | 13:55 | |
| smcginnis | haypo: Ah, I see. | 13:55 |
| *** rakesh_mishra_ has quit IRC | 13:57 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port remotefs driver to Python 3 https://review.openstack.org/197075 | 13:59 |
| *** jecarey has joined #openstack-cinder | 14:00 | |
| *** dims has quit IRC | 14:01 | |
| *** belmoreira has joined #openstack-cinder | 14:01 | |
| *** dims has joined #openstack-cinder | 14:01 | |
| openstackgerrit | Michal Dulko proposed openstack/cinder: Mock socket.gethostbyaddr in test_v7000_fcp https://review.openstack.org/197091 | 14:02 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port drbdmanagedrv driver to Python 3 https://review.openstack.org/197095 | 14:04 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port image/glance.py to Python 3 https://review.openstack.org/197099 | 14:09 |
| *** ronis has quit IRC | 14:10 | |
| *** mtanino has joined #openstack-cinder | 14:11 | |
| *** annegentle has joined #openstack-cinder | 14:14 | |
| *** r-daneel has joined #openstack-cinder | 14:20 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in targets unit tests https://review.openstack.org/197105 | 14:23 |
| *** jungleboyj has joined #openstack-cinder | 14:26 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Move HDS drivers to Hitachi folder https://review.openstack.org/197108 | 14:27 |
| *** rmesta has joined #openstack-cinder | 14:29 | |
| *** rmesta has left #openstack-cinder | 14:34 | |
| *** BharatK has quit IRC | 14:36 | |
| *** Apoorva has joined #openstack-cinder | 14:37 | |
| *** amoturi has joined #openstack-cinder | 14:40 | |
| *** jyoti-ranjan has joined #openstack-cinder | 14:42 | |
| *** rushiagr_away is now known as rushiagr | 14:45 | |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Move HDS drivers to Hitachi folder https://review.openstack.org/197108 | 14:49 |
| *** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder" | 14:52 | |
| -openstackstatus- NOTICE: The log volume was repaired and brought back online at 14:00 UTC. Log links today from before that time may be missing, and changes should be rechecked if fresh job logs are desired for them. | 14:52 | |
| *** laughterwym has joined #openstack-cinder | 14:52 | |
| *** sgotliv has joined #openstack-cinder | 14:53 | |
| *** jordanP has quit IRC | 14:55 | |
| *** sgotliv_ has quit IRC | 14:55 | |
| *** e0ne is now known as e0ne_ | 14:55 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in cmd https://review.openstack.org/197124 | 14:55 |
| *** Vikash_cz has quit IRC | 14:56 | |
| *** jordanP has joined #openstack-cinder | 14:57 | |
| *** jaypipes has quit IRC | 14:58 | |
| *** garthb__ has joined #openstack-cinder | 14:58 | |
| *** garthb has joined #openstack-cinder | 14:58 | |
| *** e0ne_ is now known as e0ne | 14:58 | |
| *** e0ne is now known as e0ne_ | 14:59 | |
| *** e0ne_ is now known as e0ne | 14:59 | |
| *** Vikash has quit IRC | 14:59 | |
| *** nkrinner has quit IRC | 15:00 | |
| *** jordanP has quit IRC | 15:03 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in HDS tests https://review.openstack.org/197127 | 15:04 |
| *** Yogi11 has joined #openstack-cinder | 15:04 | |
| *** asselin__ has joined #openstack-cinder | 15:08 | |
| *** aix has quit IRC | 15:09 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in Windows tests https://review.openstack.org/197131 | 15:11 |
| haypo | smcginnis: i warned you of the rain of small Python 3 patches :-D | 15:11 |
| smcginnis | :) | 15:11 |
| jungleboyj | Hopefully it doesn't turn to hail. | 15:11 |
| smcginnis | Hah! | 15:11 |
| haypo | oh by the way, i found a bug in eventlet when trying to port cinder/tests/unit/test_emc_vmax.py => https://github.com/eventlet/eventlet/pull/237 | 15:12 |
| *** asselin_ has quit IRC | 15:12 | |
| haypo | jungleboyj: haha | 15:12 |
| smcginnis | haypo: Nice! | 15:12 |
| *** bluex-pl has quit IRC | 15:13 | |
| *** xyang1 has joined #openstack-cinder | 15:13 | |
| *** avishay_ has joined #openstack-cinder | 15:14 | |
| *** avishay has quit IRC | 15:14 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in HDS tests https://review.openstack.org/197127 | 15:15 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port remotefs driver to Python 3 https://review.openstack.org/197075 | 15:15 |
| *** amoturi has quit IRC | 15:16 | |
| *** belmoreira has quit IRC | 15:17 | |
| *** amoturi has joined #openstack-cinder | 15:18 | |
| *** jordanP has joined #openstack-cinder | 15:19 | |
| jordanP | Hi. Can someone point me to a maintainer of the glusterfs cinder driver please ? | 15:19 |
| *** tsekiyama has joined #openstack-cinder | 15:20 | |
| *** alau2 has joined #openstack-cinder | 15:20 | |
| *** belmoreira has joined #openstack-cinder | 15:21 | |
| jungleboyj | jordanP: I would start with eharney . He should be able to get you pointed in the right direction. | 15:22 |
| eharney | jordanP: that would be deepakcs/bharatk but since they aren't here right now, i can maybe help | 15:23 |
| *** aix has joined #openstack-cinder | 15:24 | |
| *** haypo has left #openstack-cinder | 15:24 | |
| jordanP | eharney, hi. So I notice this in the driver: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/glusterfs.py#L331 | 15:25 |
| openstackgerrit | Shay Halsband proposed openstack/cinder: XtremIO support for iscsi discovery auth https://review.openstack.org/189358 | 15:25 |
| jordanP | but in my test, it's possible to extend a volume even though it has some snapshots | 15:25 |
| dulek | xyang1: Hi, I've answered you in https://review.openstack.org/#/c/193167/ . Can you take a look and tell mi if I'm missing something? | 15:25 |
| eharney | jordanP: your test with the glusterfs driver? | 15:25 |
| jordanP | (I am reworking the scality sofs driver to reuse the RemoteFS base code) | 15:26 |
| eharney | ah right | 15:26 |
| jordanP | eharney, my test with a new driver | 15:26 |
| jordanP | that reuse RemoteFS and copy also some parts of the glusterFS driver :p | 15:26 |
| *** laughterwym has quit IRC | 15:27 | |
| eharney | jordanP: how do you do that? just resize the base volume file? | 15:27 |
| jordanP | basically if I comment the line that raise exception.InvalidVolume(msg), everyghint still works fine | 15:27 |
| jordanP | eharney, yes | 15:27 |
| xyang1 | dulek: I am in a meeting for another hour and half, I need to go through your answer later, can you also comment on why you need to keep snapshot_id, source vol id, image id, replica source, etc, how are they used differently | 15:27 |
| eharney | jordanP: seems plausible to me... i think i just put that check there to be safe and make things simpler at the time | 15:28 |
| dulek | xyang1: I will, I should have added that in the first place. Thanks! | 15:28 |
| jordanP | eharney, the thing is, if exception.InvalidVolume is raised, the status of the volume become resize_error and it's not possible to delete it anymore, as a non-admin user | 15:28 |
| eharney | jordanP: well that's pretty unfriendly | 15:29 |
| jordanP | yep | 15:29 |
| *** hodos has joined #openstack-cinder | 15:30 | |
| eharney | jordanP: are you going to submit this change for your driver? | 15:30 |
| jordanP | imo it's safe to extend a volume even if it has dependent snapshot, but I prefere to be safe than sorry... | 15:30 |
| jordanP | eharney, I am. Next week or so | 15:30 |
| jordanP | but it's a big change | 15:30 |
| jordanP | because I rewrote 70% of my driver. But it's worth it because sharing some code is better then duplicating of course | 15:31 |
| *** ronis has joined #openstack-cinder | 15:31 | |
| hodos | thingee: hi Mike, can you please remove block from https://review.openstack.org/#/c/188877/ | 15:31 |
| eharney | jordanP: sounds good | 15:31 |
| *** changbl has joined #openstack-cinder | 15:31 | |
| smcginnis | hodos: Won't make much difference for a few months now. | 15:32 |
| jordanP | so what I'd like for the glusterfs community, if I may ask, is to test volume extend with volume with snapshots, to confirm that 'it works' | 15:32 |
| hodos | smcginnis: interesting | 15:33 |
| hodos | smcginnis: our driver was here | 15:34 |
| eharney | jordanP: i'll write an email connecting you and the guys working on that, and maybe write an enhancement bug to track this too | 15:34 |
| hodos | smcginnis: https://etherpad.openstack.org/p/cinder-liberty-drivers | 15:34 |
| jordanP | eharney, great, thanks ! | 15:34 |
| smcginnis | hodos: Yeah, it was close. The deadline to be merged was the 19th though, so it's not going to be until M opens up. Sorry. | 15:35 |
| hodos | smcginnis: couple of drivers have been merged after 19th | 15:36 |
| *** DericHorn-HP has joined #openstack-cinder | 15:36 | |
| jordanP | eharney, just one last thing, as there been any discussion about removing all the drivers from the cinder-repo ? (like what neutron did more or less ?). If so, what's the status quo on that topic ? | 15:37 |
| eharney | jordanP: it gets brought up from time to time but i don't know that there's anything solid forming around that at this point | 15:38 |
| jordanP | eharney, ok, thanks | 15:39 |
| eharney | jordanP: https://review.openstack.org/#/c/184951/ is the latest discussion | 15:39 |
| jungleboyj | smcginnis: What a man, starting an etherpad for our presentations! | 15:42 |
| smcginnis | jungleboyj: I knew email would be too painful. :) | 15:43 |
| jungleboyj | smcginnis: Good idea. | 15:43 |
| *** kjnelson has quit IRC | 15:45 | |
| *** amoturi has quit IRC | 15:47 | |
| *** aix has quit IRC | 15:48 | |
| *** annashen has joined #openstack-cinder | 15:49 | |
| *** belmoreira has quit IRC | 15:50 | |
| *** _cjones_ has joined #openstack-cinder | 15:52 | |
| *** Yogi11 has quit IRC | 15:52 | |
| hodos | thingee: can you please confirm that nexenta 4 drivers are out of L | 15:54 |
| geguileo | jgriffith: LIO issue on Juno: https://review.openstack.org/#/c/197149/ | 15:55 |
| *** asselin__ has quit IRC | 15:58 | |
| *** markus_z has quit IRC | 15:59 | |
| *** aix has joined #openstack-cinder | 16:00 | |
| *** jistr has quit IRC | 16:01 | |
| *** aix has quit IRC | 16:01 | |
| jgriffith | geguileo: thanks! | 16:03 |
| jgriffith | geguileo: I accidentally hit recheck :( | 16:03 |
| jgriffith | geguileo: scanned to quickly and it was the GPFS job that never works and reports no logs | 16:04 |
| geguileo | jgriffith: No problem | 16:04 |
| *** leeantho has joined #openstack-cinder | 16:05 | |
| *** sgotliv has quit IRC | 16:07 | |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 16:08 |
| *** anshul has quit IRC | 16:09 | |
| openstackgerrit | Dmitry Guryanov proposed openstack/cinder: smbfs: fix invalid check for smbfs_used_ratio correctness https://review.openstack.org/190326 | 16:10 |
| *** dannywilson has joined #openstack-cinder | 16:11 | |
| *** annashen has quit IRC | 16:11 | |
| jyoti-ranjan | I have following specific questions for NetApp driver: (1) will FAS8060 and FAS8020 work with Juno driver (2) Can we use kilo NetApp driver with Juno OpenStack? Or what it will take to make it happen | 16:12 |
| jgriffith | hemna: ping | 16:13 |
| *** dannywilson has quit IRC | 16:13 | |
| hemna | hey | 16:14 |
| jgriffith | hemna: hey there | 16:14 |
| hemna | what's up | 16:14 |
| jgriffith | hemna: do you have any plans to tag an os-brick version any time soon? | 16:14 |
| *** gaurang has joined #openstack-cinder | 16:14 | |
| *** dannywilson has joined #openstack-cinder | 16:14 | |
| jgriffith | hemna: I keep hitting a couple things that I *think* might be fixed | 16:14 |
| hemna | I thought we were going to do releases based on milestones | 16:15 |
| hemna | I can't do it actually. | 16:15 |
| jgriffith | hemna: for brick? | 16:15 |
| hemna | only thingee can. | 16:15 |
| jgriffith | hemna: Oh | 16:15 |
| jgriffith | hemna: well, then never mind I guess :) | 16:15 |
| jgriffith | hemna: just makes things kinda painful if CI's have failures due to something in os-brick | 16:15 |
| openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Filter cgsnapshots data on the DB side https://review.openstack.org/197163 | 16:15 |
| hemna | yah, understandable. | 16:16 |
| hemna | I think we should do milestone based drops, just like cinderclient. | 16:16 |
| jgriffith | hemna: seems reasonable to me | 16:16 |
| jgriffith | hemna: assuming we do them this release :) | 16:16 |
| jgriffith | hemna: we "forgot" to do any during Kilo :( | 16:17 |
| hemna | yah :( that kinda hurt horizon a bit. | 16:17 |
| hemna | live and learn. | 16:17 |
| jgriffith | yup | 16:17 |
| hemna | PTL just has to remember to try and do drops on the milestones....along with the other 3x10^50 things he has to do :) | 16:17 |
| jgriffith | DuncanT: ping | 16:18 |
| *** avishay_ has quit IRC | 16:18 | |
| *** avishay_ has joined #openstack-cinder | 16:19 | |
| *** beekneemech is now known as bnemec | 16:19 | |
| *** e0ne is now known as e0ne_ | 16:23 | |
| *** rmesta has joined #openstack-cinder | 16:25 | |
| *** jaypipes has joined #openstack-cinder | 16:26 | |
| *** kjnelson has joined #openstack-cinder | 16:28 | |
| *** dontalton has joined #openstack-cinder | 16:29 | |
| *** jyoti-ranjan has quit IRC | 16:31 | |
| *** e0ne_ has quit IRC | 16:34 | |
| *** e0ne has joined #openstack-cinder | 16:36 | |
| *** jordanP has quit IRC | 16:37 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: FC Eliminate the need to return devices list https://review.openstack.org/195359 | 16:39 |
| *** anshul has joined #openstack-cinder | 16:42 | |
| mtanino | geguileo: hi, | 16:46 |
| geguileo | mtanino: Hi | 16:46 |
| mtanino | geguileo: I'm looking you stable/juno fix, | 16:47 |
| geguileo | mtanino: The one about the LIO? | 16:47 |
| mtanino | geguileo: yes, LIO ifx | 16:47 |
| mtanino | fix | 16:47 |
| geguileo | mtanino: Seen anything you don't like? :) | 16:47 |
| mtanino | geguileo: so in my understanding, without terminate_connction for LIO driver hits this problem https://review.openstack.org/#/c/130399/4 | 16:48 |
| geguileo | mtanino: Yes, that is correct | 16:48 |
| geguileo | mtanino: But that is a complete different matter :) | 16:48 |
| mtanino | geguileo: oh, really? I'm missing something | 16:49 |
| geguileo | mtanino: This patch tries to make LIO work again as it was expected to work in Juno | 16:49 |
| geguileo | mtanino: Whether that other patch makes it to Juno or not is not related | 16:49 |
| mtanino | geguileo: ok. let me check again. Thanks | 16:50 |
| *** openstackgerrit has quit IRC | 16:50 | |
| *** openstackgerrit has joined #openstack-cinder | 16:51 | |
| *** mildan has joined #openstack-cinder | 16:53 | |
| geguileo | mtanino: What I mean is that I think Juno is on critical and security only fixes now: https://wiki.openstack.org/wiki/StableBranch#Support_phases | 16:54 |
| mtanino | geguileo: OK, I understand your intention. That patch just make LIO work again. | 16:56 |
| geguileo | mtanino: That's right | 16:56 |
| geguileo | mtanino: I agree that it would be interesting to backport the other one and I would have no problem doing that | 16:57 |
| geguileo | mtanino: But I don't think it will meet the criteria to get merged :( | 16:57 |
| mtanino | geguileo: I thought https://review.openstack.org/#/c/130399/4 was landed in Juno but not. | 16:57 |
| mtanino | geguileo: Yes, this is not critical or security. | 16:58 |
| geguileo | mtanino: I thought so too, that's why the other backport broke LIO | 16:58 |
| *** mildan has quit IRC | 16:58 | |
| geguileo | mtanino: Because I didn't realize that that patch wasn't merged :( | 16:58 |
| *** Pv_ has joined #openstack-cinder | 16:58 | |
| mtanino | me too :( | 16:58 |
| geguileo | mtanino: But it's a really easy backport | 16:59 |
| geguileo | mtanino: Even backporting it to Icehouse is quite simple :) | 16:59 |
| mtanino | geguileo: yes, it's correct but I think it's required exception for backporting from cores. | 17:00 |
| Pv_ | hey guys i just was reading the lvm reference/base driver.py file and I wanted to ask about the target mappings and what they're purpose is | 17:00 |
| *** ociuhandu has quit IRC | 17:00 | |
| Pv_ | and i don't see one for FC | 17:00 |
| *** thangp has quit IRC | 17:02 | |
| geguileo | Pv_: That's iSCSI specific. That's why they are configured with iscsi_helper (tgt, lio...) | 17:03 |
| geguileo | Pv_: What you see is correct | 17:03 |
| *** DericHorn-HP has quit IRC | 17:05 | |
| Pv_ | ok, i was looking at the fibre channel base driver and i wanted to know if there's any documentation detailing the implementation of the methods provided in it | 17:05 |
| *** e0ne has quit IRC | 17:05 | |
| geguileo | Pv_: Sorry, I haven't touched any FC drivers, so can't help you | 17:12 |
| *** annashen has joined #openstack-cinder | 17:12 | |
| *** thangp has joined #openstack-cinder | 17:14 | |
| *** lcurtis has joined #openstack-cinder | 17:14 | |
| *** avishay_ has quit IRC | 17:16 | |
| *** annashen has quit IRC | 17:17 | |
| *** agarciam has quit IRC | 17:18 | |
| dulek | geguileo: If you can take a look on https://review.openstack.org/#/c/183222/. I've found a little issue there. | 17:22 |
| * geguileo looks at link | 17:23 | |
| dulek | geguileo: Just a simple leftover file I think. | 17:24 |
| geguileo | dulek: I don't have that file | 17:24 |
| geguileo | dulek: Oh, wait | 17:25 |
| geguileo | dulek: You are totally right XD | 17:26 |
| geguileo | dulek: Really great catch | 17:27 |
| geguileo | dulek: I'll remove it now | 17:27 |
| dulek | geguileo: np :) | 17:27 |
| *** annegentle has quit IRC | 17:28 | |
| *** dannywilson has quit IRC | 17:29 | |
| openstackgerrit | Gorka Eguileor proposed openstack/cinder: Fix backup metadata import missing fields https://review.openstack.org/183222 | 17:33 |
| *** kjnelson has quit IRC | 17:34 | |
| *** dtynan has quit IRC | 17:35 | |
| *** dtynan has joined #openstack-cinder | 17:37 | |
| thingee | hemna, jgriffith: we want to tag something this week? | 17:38 |
| jgriffith | thingee: I'm trying to figure that out :( | 17:38 |
| hemna | thingee, I'd like to get a few of the in flight patches in. | 17:38 |
| jgriffith | thingee: I'm seeing all sorts of brick connection errors in my CI | 17:39 |
| thingee | oh no :( | 17:39 |
| thingee | jgriffith: can you give me your logs? | 17:39 |
| jgriffith | thingee: yeah, just a sec... | 17:39 |
| hemna | jgriffith, it would be helpful to see your CI added to the os-brick patches | 17:39 |
| hemna | that will help a lot. | 17:39 |
| jgriffith | thingee: http://54.164.167.86/solidfire-ci-logs/refs-changes-08-197108-2/logs/screen-c-vol.log.txt | 17:39 |
| hemna | we have a few folks that have done it. | 17:39 |
| *** patrickeast has joined #openstack-cinder | 17:39 | |
| jgriffith | hemna: but it should't matter if the version tested in brick isn't the version in Cinder :) | 17:39 |
| jgriffith | hemna: but yes | 17:40 |
| jgriffith | hemna: thingee also TBC I'm not saying it's somethign wrong with brick | 17:40 |
| hemna | the os-brick patch CI makes sure that the next version of os-brick doesn't break cinder | 17:40 |
| jgriffith | hemna: thingee just noticed everything has been failign this morning | 17:40 |
| hemna | arg | 17:40 |
| hemna | :( | 17:40 |
| thingee | jgriffith: absolutely, was also just seeing if I can see anything to help your ci. | 17:40 |
| jgriffith | hemna: and I'm still getting that no entries error that I pushed the fix for a while back | 17:40 |
| jgriffith | thingee: yeah | 17:40 |
| *** annashen has joined #openstack-cinder | 17:41 | |
| hemna | jgriffith, because we haven't pushed a new drop to pypi ? | 17:41 |
| jgriffith | thingee: hemna it might be something flaked out on my network. Looking into it | 17:41 |
| thingee | jgriffith: ah those login issues? | 17:41 |
| jgriffith | hemna: Yeah, the no entries in DB is *hopefully* fixed by that patch | 17:41 |
| jgriffith | thingee: yeah... the login issues | 17:41 |
| jgriffith | thingee: it's always on the upload image | 17:41 |
| *** annegentle has joined #openstack-cinder | 17:41 | |
| hemna | ok. cool. we would be able to verify it, if we had your CI plugged in to os-brick patches :) | 17:41 |
| jgriffith | hemna: ok, ok, fine... I'll get another system up and running for os-brick | 17:42 |
| hemna | :) | 17:42 |
| thingee | jgriffith: does this also happen consistently if you run just that one upload image test? | 17:43 |
| jgriffith | thingee: anyway... I'm setting up a new run to take a look and see if I can figure out what's going on | 17:43 |
| *** jecarey has quit IRC | 17:43 | |
| hemna | jgriffith, ok if you need any help, let me know | 17:43 |
| jgriffith | thingee: not sure yet | 17:43 |
| jgriffith | hemna: thingee thanks | 17:43 |
| hemna | FWIW, I added a patch on friday to test against py34 | 17:44 |
| hemna | I'd like to get that one in, if we are going to tag a new drop | 17:44 |
| thingee | jgriffith: I would just get to the point of having windows open on the target side, breakpoints to do manual logins from another machine, etc. | 17:44 |
| hemna | https://review.openstack.org/#/c/196284/ | 17:45 |
| jgriffith | thingee: yep, setting that up now :) | 17:45 |
| *** mgarza_ has joined #openstack-cinder | 17:46 | |
| *** kjnelson has joined #openstack-cinder | 17:46 | |
| thingee | jgriffith: ok, let me know if we need to resort to the rubber duck method. | 17:46 |
| jgriffith | LOL | 17:46 |
| thingee | jgriffith: seriously did it last week on an issue I was staring out for days. worked like a charm and I had a duh moment. | 17:47 |
| jungleboyj | thingee: Rubber duck method? | 17:48 |
| thingee | jungleboyj: https://en.wikipedia.org/wiki/Rubber_duck_debugging | 17:49 |
| thingee | I believe it's the reason why enovance was giving rubber ducks as swag at hong kong. | 17:49 |
| EmilienM | lol | 17:50 |
| * jungleboyj laughs | 17:50 | |
| thingee | laugh all you want. I'm a believer! | 17:50 |
| jungleboyj | That does work, I have solved many issues that way, just didn't know I could do it with a rubber duck. | 17:50 |
| jgriffith | thingee: it works... :) | 17:50 |
| jgriffith | thingee: we used to have a stuffed-animal/Gorilla | 17:51 |
| * jungleboyj isn't laughing at the process, just the duck. | 17:51 | |
| * jungleboyj needs to make use of the penguins in my office. | 17:51 | |
| jgriffith | jungleboyj: oooh... Penguins are good at conversation! | 17:52 |
| jgriffith | jungleboyj: they always seem to be listening intently | 17:52 |
| jgriffith | Rubber Ducks just seem to be laughing at you | 17:52 |
| jungleboyj | jgriffith: +2 | 17:52 |
| * thingee wishes a penguin would just tell him not to bother with the "rubbish" of that code path and look here instead | 17:53 | |
| *** Yogi1 has joined #openstack-cinder | 17:53 | |
| smcginnis | hemna: I think there are at least three patches up right now to add py3 testing to os-brick. | 17:54 |
| smcginnis | hemna: Oh, I see you've down voted them now. Good, rather see the efforts focused on just one. | 17:56 |
| *** chlong has quit IRC | 17:56 | |
| *** e0ne has joined #openstack-cinder | 17:58 | |
| *** dannywilson has joined #openstack-cinder | 18:01 | |
| *** e0ne is now known as e0ne_ | 18:04 | |
| smcginnis | hemna: Victor's patch looks like it has some goodness. Maybe merge some of his changes into yours? | 18:05 |
| jungleboyj | thingee: That would be nice. :-) | 18:06 |
| jungleboyj | Though, if the penguin starts talking ... | 18:07 |
| *** e0ne_ is now known as e0ne | 18:07 | |
| smcginnis | jungleboyj: Only in Washington, Oregon, and Colorado. ;) | 18:07 |
| patrickeast | so.. i’m looking into adding some notifications to ceilometer, after poking around for a few mins it looks like this notifier https://github.com/openstack/cinder/blob/master/cinder/rpc.py#L162 and doing .info() calls on it is the way to do that, right? | 18:08 |
| * jungleboyj laughs | 18:08 | |
| patrickeast | or is that a different message bus? | 18:08 |
| e0ne | patrickeast: we've got differet notifiers. just search 'rpc.get_notifier(' across the sources | 18:11 |
| patrickeast | e0ne: gotcha, so if i’m sending them from the volume service i should use one with like rpc.get_notifier(“volume”, host) | 18:12 |
| patrickeast | e0ne: and the .info() publishes to the topic ceilometer is listening on then, right? | 18:12 |
| *** annegentle has quit IRC | 18:13 | |
| *** ericksonsantos has joined #openstack-cinder | 18:14 | |
| *** tellesnobrega_ has joined #openstack-cinder | 18:15 | |
| patrickeast | err wait, service has a bunch of them (backup, replication, volume etc)… not like each actual service service | 18:15 |
| * patrickeast goes to read ceilometer docs | 18:16 | |
| *** annegentle has joined #openstack-cinder | 18:16 | |
| *** e0ne is now known as e0ne_ | 18:18 | |
| *** rushiagr is now known as rushiagr_away | 18:18 | |
| hemna | smcginnis, yah I just noticed those this morning. I had submitted mine last week | 18:18 |
| openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Add support for consistency groups https://review.openstack.org/196818 | 18:18 |
| *** tellesnobrega_ has quit IRC | 18:20 | |
| *** e0ne_ is now known as e0ne | 18:23 | |
| *** avishay_ has joined #openstack-cinder | 18:25 | |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: update os-brick to pass python3 tests https://review.openstack.org/196284 | 18:27 |
| *** tsekiyama has quit IRC | 18:27 | |
| *** lpetrut has quit IRC | 18:28 | |
| *** avishay__ has joined #openstack-cinder | 18:29 | |
| *** avishay_ has quit IRC | 18:29 | |
| *** sgotliv has joined #openstack-cinder | 18:30 | |
| *** ronis has quit IRC | 18:35 | |
| *** avishay__ is now known as avishay | 18:38 | |
| *** tsekiyama has joined #openstack-cinder | 18:38 | |
| *** Yogi1 has quit IRC | 18:39 | |
| *** gaurang has left #openstack-cinder | 18:41 | |
| *** annegentle has quit IRC | 18:43 | |
| e0ne | hemna: hi. just FYI. i filed blueprint https://blueprints.launchpad.net/cinder/+spec/use-cinder-without-nova. i hope, spec will be ready for review in next few days | 18:47 |
| avishay | Hey all, has anyone seen this with tgt? rtslib.utils.RTSLibError: This _Backstore already exists in configFS | 18:47 |
| *** Yogi1 has joined #openstack-cinder | 18:47 | |
| hemna | e0ne, hey cool. I had planned on adding this to the cinderclient. | 18:48 |
| *** dtynan has quit IRC | 18:48 | |
| thingee | avishay: something is wrong there :) ... you either use tgt or rtslib :) | 18:48 |
| e0ne | hemna: we definatly need it :) | 18:48 |
| avishay | thingee: my cinder.conf says "tgtadm" - why would it use rtslib? i don't understand | 18:48 |
| *** dtynan has joined #openstack-cinder | 18:49 | |
| thingee | avishay: lioadm is nowhere in your cinder.conf? | 18:50 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: update os-brick to pass python3 tests https://review.openstack.org/196284 | 18:52 |
| avishay | thingee: it was, but lioadm is broken in juno and has no CI, so we moved to tgtadm...will double check tomorrow to make sure. thanks! | 18:53 |
| *** jwcroppe has quit IRC | 18:53 | |
| Swanson | manage_existing question. This is for both mounted and unmounted volumes? | 18:53 |
| thingee | avishay: ok, that's all I can think of is iscsi_helper=lioadm .. at least how that module gets loaded | 18:54 |
| *** jwcroppe has joined #openstack-cinder | 18:54 | |
| avishay | thingee: ok, a co-worker of mine set up this test env, will need to check it. thanks for the tip! | 18:54 |
| mtanino | avishay: Hi, A patch was proposed to fix juno/stable issue. https://review.openstack.org/#/c/197149/1 | 18:54 |
| *** mutoulbj has quit IRC | 18:56 | |
| avishay | mtanino: thanks, but i won't run LIO as long as it's not tested in the gate | 18:56 |
| mtanino | avishay: I understand. | 18:56 |
| *** Longgeek has quit IRC | 18:57 | |
| *** Longgeek has joined #openstack-cinder | 18:58 | |
| *** jwcroppe has quit IRC | 18:59 | |
| *** anshul has quit IRC | 19:01 | |
| *** Pv_ has quit IRC | 19:03 | |
| mtanino | thingee: Hi, May I ask a quick question about your spec? https://review.openstack.org/#/c/183947/ | 19:04 |
| openstackgerrit | Erickson Filipe Guedes dos Santos proposed openstack/cinder: Nested Quota: Set default values to subproject https://review.openstack.org/194406 | 19:04 |
| thingee | mtanino: sure...ugh yeah need to start that back up again | 19:04 |
| mtanino | thingee: I'm waiting the spec will be landed because my spec refer your spec :) so I'd like to know your plan | 19:05 |
| thingee | mtanino: Yeah I need to take a closer look at what people are asking for with the graffiti stuff | 19:06 |
| thingee | mtanino: do you have thoughts? | 19:06 |
| thingee | personally I don't think this information should be stored | 19:06 |
| thingee | but maybe the cache isn't that big of a deal | 19:06 |
| *** Guest80179 has joined #openstack-cinder | 19:06 | |
| thingee | mtanino: brb | 19:06 |
| mtanino | thingee: I agree the get_capabilities to get capability list from drivers. | 19:07 |
| mtanino | thingee: hmm, should be stored or not.. | 19:08 |
| mtanino | thingee: Since the capabilities are not static information for the back-end storage, I guess we don't need to have these on DB, IMHO | 19:10 |
| *** kjnelson has quit IRC | 19:10 | |
| mtanino | are not / are | 19:10 |
| openstackgerrit | Erlon R. Cruz proposed openstack/cinder: Fix migration on HNAS NFS driver https://review.openstack.org/197233 | 19:11 |
| *** e0ne is now known as e0ne_ | 19:14 | |
| *** Guest80179 has quit IRC | 19:14 | |
| *** avishay has quit IRC | 19:14 | |
| *** e0ne_ is now known as e0ne | 19:15 | |
| *** mgarza_ has quit IRC | 19:18 | |
| *** annashen has quit IRC | 19:24 | |
| *** rmesta has quit IRC | 19:30 | |
| *** rmesta has joined #openstack-cinder | 19:30 | |
| *** rmesta has quit IRC | 19:30 | |
| *** rmesta has joined #openstack-cinder | 19:31 | |
| *** rmesta has left #openstack-cinder | 19:31 | |
| thingee | mtanino: I agree. | 19:33 |
| mtanino | thingee: Nice. | 19:34 |
| thingee | jgriffith: can you do me a favor? Can you post a revision of https://review.openstack.org/#/c/183947/ with the scheduler bit? I think you have a better idea of how that should work than I do. | 19:35 |
| thingee | jgriffith: Or I can have winston-d_afk take a look | 19:35 |
| *** xyang1 has quit IRC | 19:38 | |
| *** xyang1 has joined #openstack-cinder | 19:38 | |
| *** avishay has joined #openstack-cinder | 19:39 | |
| avishay | thingee: just verified - iscsi_helper = tgtadm :-/ | 19:41 |
| thingee | avishay: are you using lvm? | 19:42 |
| *** shyama has quit IRC | 19:42 | |
| eharney | avishay: is this a multi-backend setup? | 19:43 |
| avishay | thingee: volume_driver = cinder.volume.drivers.lvm.LVMISCSIDriver | 19:43 |
| avishay | eharney: it's configured as multi-backend, but there is only one | 19:43 |
| eharney | avishay: you have to set iscsi_helper in the backend conf section and not default | 19:43 |
| eharney | i got bit by this the other day, the behavior of the option and different sections doesn't really make sense | 19:44 |
| avishay | eharney: yes, it's there | 19:44 |
| thingee | avishay: kilo? | 19:44 |
| eharney | humm | 19:44 |
| thingee | oh juno forgot | 19:45 |
| avishay | thingee: juno with latest patches cherry-picked (which broke LIO, hence tgt) | 19:45 |
| avishay | configfs is only for LIO? | 19:46 |
| eharney | yes | 19:46 |
| avishay | strange... | 19:46 |
| thingee | ah there we go | 19:46 |
| * thingee has to get lunch before meetings | 19:47 | |
| eharney | avishay: so it's loading the lioadm helper when you think it shouldn't? | 19:47 |
| geguileo | avishay: It only broke LIO | 19:47 |
| avishay | eharney: not sure...if it was using lioadm then it would be even more broken than it is...looking at the full trace... | 19:48 |
| *** lpetrut has joined #openstack-cinder | 19:48 | |
| geguileo | avishay: Are you using tgt or lio? | 19:48 |
| geguileo | avishay: In your config | 19:48 |
| avishay | geguileo: trying to use tgt :) | 19:48 |
| avishay | geguileo: tgt | 19:48 |
| geguileo | Oh, then bug introduced in Juno shouldn't affect you | 19:49 |
| geguileo | avishay: Because it was a specific call to a nonexistent lioadm target helper method called terminate_connection | 19:50 |
| *** aarefiev_home has joined #openstack-cinder | 19:51 | |
| *** Rockyg has joined #openstack-cinder | 19:51 | |
| eharney | it looks like get_target_helper() in driver.py may be reading the value from the default section instead of the backend section | 19:53 |
| geguileo | eharney: Then it would be a misconfiguration | 19:53 |
| geguileo | default and backends must be in sync | 19:53 |
| *** lpetrut has quit IRC | 19:54 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/196888 | 19:55 |
| avishay | eharney: then it should be tgt anyway for me because i don't define it in the default section and tgt is the default value | 19:56 |
| *** mutoulbj has joined #openstack-cinder | 19:56 | |
| *** mutoulbj has quit IRC | 19:56 | |
| avishay | i'll run it tomorrow with pdb and see what's going on | 19:57 |
| geguileo | avishay: So you don't lioadm defined in any of the configuration files? | 19:57 |
| avishay | geguileo: no, but will triple check tomorrow | 19:58 |
| avishay | thanks everyone for the help, it's 11pm here, time to sign off :) | 19:58 |
| geguileo | avishay: Ok, talk to you tomorrow | 19:59 |
| *** avishay has quit IRC | 20:00 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/os-brick: Updated from global requirements https://review.openstack.org/193996 | 20:01 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/197273 | 20:02 |
| *** patrickeast_ has joined #openstack-cinder | 20:03 | |
| *** Apoorva has quit IRC | 20:03 | |
| *** patrickeast has quit IRC | 20:04 | |
| *** patrickeast_ is now known as patrickeast | 20:04 | |
| *** Apoorva has joined #openstack-cinder | 20:04 | |
| *** aarefiev_home has quit IRC | 20:05 | |
| e0ne | eharney: hi. did you have a time to investigate https://bugs.launchpad.net/cinder/+bug/1401335? | 20:06 |
| openstack | e0ne: Error: Could not gather data from Launchpad for bug #1401335 (https://launchpad.net/bugs/1401335). The error has been logged | 20:06 |
| eharney | e0ne: well... like the bot... i'm also failing to load the page | 20:07 |
| e0ne | eharney: i'm not sure yet, but looks like you're righht and cinder is broke | 20:07 |
| * e0ne doesn't want to speak with bots | 20:07 | |
| e0ne | it's "rbd calls block eventlet threads" issue | 20:07 |
| eharney | e0ne: ah, that one. well.... yeah, I think Cinder is broken | 20:07 |
| e0ne | eharney: after merging patch i found that sometimes cinder 'hangs' connecting to ceph | 20:08 |
| e0ne | eharney: here is strace: http://paste.openstack.org/show/329543/ | 20:08 |
| e0ne | eharney: i'm still not sure what is root cause | 20:09 |
| eharney | that does seem to fit | 20:09 |
| e0ne | :( | 20:09 |
| eharney | do we know of a way to fix the original bug that wouldn't cause this problem? | 20:10 |
| e0ne | not yet | 20:10 |
| *** pv_ has joined #openstack-cinder | 20:12 | |
| e0ne | i've got few ideas. i will test them tomorrow | 20:12 |
| e0ne | eharney: i'm not sure that we need to roll-back fix or not | 20:13 |
| eharney | e0ne: it's mostly whether the behavior is better with the previous bug or the current bug | 20:14 |
| *** changbl has quit IRC | 20:15 | |
| e0ne | eharney: imo, "new" bug is harder to reproduce | 20:15 |
| eharney | i guess we need to see if we can find an alternate approach quickly | 20:16 |
| *** annashen has joined #openstack-cinder | 20:16 | |
| e0ne | eharney: i'm agree with you | 20:16 |
| e0ne | eharney: i've got environment and steps to reproduce old and new bugs. will try to fix it asap | 20:19 |
| eharney | ok | 20:20 |
| *** haypo has joined #openstack-cinder | 20:21 | |
| *** jwcroppe has joined #openstack-cinder | 20:21 | |
| haypo | oh. there are 3 patches to port os-brick to Python 3, 3 patches sent the same week. fun :) | 20:22 |
| haypo | https://review.openstack.org/#/c/197062/ - https://review.openstack.org/#/c/194216 - https://review.openstack.org/#/c/196284/ | 20:22 |
| geguileo | XD | 20:24 |
| e0ne | eharney: one more question | 20:25 |
| eharney | e0ne: sure | 20:26 |
| e0ne | eharney: https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/rbd.py#L544-L545 | 20:26 |
| eharney | e0ne: ok? | 20:27 |
| *** hodos|2 has joined #openstack-cinder | 20:28 | |
| *** jungleboyj has quit IRC | 20:28 | |
| e0ne | eharney: am i right that its using different threads for rados connection and RBD calls after my fix? | 20:28 |
| e0ne | RADOSClient uses _connect_to_rados wich returns tpool.Proxy | 20:29 |
| eharney | e0ne: i think it's using different greenthreads yes | 20:29 |
| eharney | oh right | 20:29 |
| eharney | so yes, i think so | 20:29 |
| e0ne | eharney: i'm afrait its defferent python threads | 20:29 |
| e0ne | not greenthreads | 20:29 |
| eharney | yeah, that's why this is problematic iirc | 20:30 |
| e0ne | i'll try to remove tpool invoke from _connect_to_rados and test | 20:30 |
| eharney | maybe you can do the same thing w/ a greenthread instead of native threads? | 20:30 |
| e0ne | eharney: no, python-rbd invokes C-lib and blocks greenthreads | 20:31 |
| *** hodos has quit IRC | 20:31 | |
| *** mgarza has joined #openstack-cinder | 20:31 | |
| e0ne | original issue was about it | 20:31 |
| eharney | ahh yeah i remember now | 20:32 |
| e0ne | need to get a break to sleep | 20:33 |
| e0ne | bb tomorrow in ~12 hours | 20:33 |
| eharney | good plan :) | 20:34 |
| e0ne | eharney: i'll update you tomorrow with my status on this issue | 20:34 |
| e0ne | :) | 20:34 |
| *** e0ne has quit IRC | 20:35 | |
| *** Yogi1 has quit IRC | 20:35 | |
| *** amit213 has quit IRC | 20:44 | |
| *** amit213 has joined #openstack-cinder | 20:44 | |
| *** amit213 has quit IRC | 20:44 | |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Fix Python 3 issues in HDS tests https://review.openstack.org/197127 | 20:52 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port remotefs driver to Python 3 https://review.openstack.org/197075 | 20:52 |
| *** IlyaG has joined #openstack-cinder | 20:55 | |
| haypo | eharney: in python (python itself), it's common to write an issue number in a comment | 20:55 |
| haypo | eharney: but well, i don't care :) i move the bug number from the comment to the commit message | 20:56 |
| eharney | haypo: well... in my experience it doesn't help much and just leads to mess as the code gets worked on over time | 20:56 |
| eharney | haypo: and some folks always complain about the "i want to be able to understand the code on a plane w/o the bug tracker" case | 20:56 |
| haypo | ah? they don't have access to the bug tracker? or they don't want to read each issue? | 20:58 |
| eharney | both | 20:58 |
| eharney | personally, i think if you have comments, they should mean enough to understand what's going on without requiring you to go read a bug too | 20:59 |
| haypo | hum ok. but the bug number is only an additional information, it doesn't replace the text in the comment | 20:59 |
| eharney | more information != better information :) | 20:59 |
| *** thangp has quit IRC | 20:59 | |
| haypo | eharney: i had to use git blame to understand why unicode_decode was used | 21:00 |
| haypo | which pointed me to the bug tracker | 21:00 |
| haypo | the bug explains that they want to support spaces | 21:00 |
| haypo | it gives more context than a very short sentence in a comment | 21:00 |
| eharney | which is... not a good thing, i think? | 21:00 |
| haypo | i already heard for this trend in openstack to not mention bug number, but i just don't understand it :) | 21:01 |
| haypo | eharney: more context is helpful | 21:01 |
| *** kjnelson has joined #openstack-cinder | 21:02 | |
| haypo | if you don't want to loose time, just skip the bug number | 21:02 |
| haypo | while reading the comment. it's not like the comment is very long :) | 21:02 |
| eharney | the goal is to take the useful info out of the bug and put it in the comment | 21:03 |
| eharney | i'm not sure i still really get what's going on in this particular case | 21:04 |
| eharney | apparently we decided on some weird config format involving \040 which is what should really be described in there | 21:05 |
| openstackgerrit | Anthony Lee proposed openstack/cinder: Improve 3PAR driver VLUN creation and deletion https://review.openstack.org/196164 | 21:06 |
| *** casusbelli has quit IRC | 21:07 | |
| openstackgerrit | Jon Bernard proposed openstack/cinder: Add support for generic volume migration https://review.openstack.org/187270 | 21:08 |
| haypo | eharney: to be honest, i don't understand the bug so it was safer to keep a reference to it :) | 21:08 |
| haypo | eharney: i just read it again. supporting \040 looks unrelated to the bug :) | 21:09 |
| eharney | i didn't either until i chased it back through a chain of two or three bugs/changes | 21:09 |
| eharney | which is kind of the point, after all | 21:09 |
| *** julim has quit IRC | 21:13 | |
| openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 21:14 |
| openstackgerrit | xing-yang proposed openstack/os-brick: Add connector driver for the ScaleIO cinder driver https://review.openstack.org/185835 | 21:16 |
| openstackgerrit | Victor Stinner proposed openstack/cinder: Port image/glance.py to Python 3 https://review.openstack.org/197099 | 21:20 |
| openstackgerrit | Alberto Murillo proposed openstack/cinder: Update expected error message from lvs https://review.openstack.org/197338 | 21:24 |
| *** hodos|2 has quit IRC | 21:25 | |
| *** daneyon has joined #openstack-cinder | 21:27 | |
| *** alau2 has quit IRC | 21:27 | |
| *** IlyaG has quit IRC | 21:28 | |
| *** kjnelson has quit IRC | 21:29 | |
| *** IlyaG has joined #openstack-cinder | 21:30 | |
| *** IlyaG has quit IRC | 21:34 | |
| *** annashen has quit IRC | 21:38 | |
| *** fifieldt has quit IRC | 21:39 | |
| *** IlyaG has joined #openstack-cinder | 21:39 | |
| *** mriedem has quit IRC | 21:41 | |
| *** annashen has joined #openstack-cinder | 21:42 | |
| smcginnis | hemna: ping | 21:45 |
| hemna | yough | 21:45 |
| smcginnis | hemna: Hey! I noticed there's still some cinder/brick stuff. | 21:45 |
| smcginnis | hemna: Looks like mostly local devs. | 21:45 |
| hemna | ? | 21:45 |
| hemna | oh in cinder | 21:46 |
| smcginnis | hemna: Is that sticking around, or does that need to migrate out too? | 21:46 |
| hemna | yah, that's the local dev stuff | 21:46 |
| smcginnis | hemna: Yeah, | 21:46 |
| smcginnis | hemna: That stays there? | 21:46 |
| hemna | we decided that the local dev stuff wasn't going in os-brick at the time | 21:46 |
| hemna | so I just left it there | 21:46 |
| *** edmondsw has quit IRC | 21:46 | |
| hemna | not sure if it needs a new home or what, so I didn't touch it | 21:46 |
| smcginnis | hemna: OK, cool. Just saw a patch changing something there and was surprised. | 21:46 |
| *** ociuhandu has joined #openstack-cinder | 21:46 | |
| smcginnis | I thought the whole folder was going away. | 21:46 |
| hemna | ah yah. | 21:46 |
| smcginnis | hemna: Thanks! | 21:47 |
| hemna | the original idea of brick itself was the local mgmt of lvm | 21:47 |
| hemna | and cinder and nova were going to share that code as the brick lib. | 21:48 |
| hemna | then it got out of hand | 21:48 |
| hemna | heh, so os-brick morphed into the other shareable code. | 21:48 |
| smcginnis | hemna: Oh well. Some shared code is better than none I suppose. :) | 21:48 |
| hemna | I still think there is value in moving the local-dev stuff into brick | 21:48 |
| smcginnis | hemna: Yeah. What's the arguments against. Seems like it would be goodness. | 21:49 |
| hemna | and nova can use it as well. | 21:49 |
| *** skraynev has quit IRC | 21:49 | |
| *** Nikolay_St has quit IRC | 21:49 | |
| hemna | I think the more volume related stuff we take over the better in the long run | 21:49 |
| smcginnis | +1 | 21:49 |
| hemna | I think some folks just weren't happy with the local_dev code itself. | 21:49 |
| smcginnis | hemna: Then they can fix it in one place instead of multiple. ;) | 21:50 |
| hemna | and honestly, I don't know enough about lvm to have an opinion about the code. but the idea is sound I thinks. | 21:50 |
| *** fifieldt has joined #openstack-cinder | 21:52 | |
| *** IlyaG has quit IRC | 21:59 | |
| openstackgerrit | Alberto Murillo proposed openstack/cinder: Update expected error message from lvs https://review.openstack.org/197338 | 22:00 |
| *** IlyaG has joined #openstack-cinder | 22:02 | |
| openstackgerrit | Alberto Murillo proposed openstack/cinder: Update expected error message from lvs https://review.openstack.org/197338 | 22:09 |
| *** Rockyg has quit IRC | 22:12 | |
| openstackgerrit | Eric Harney proposed openstack/cinder: Remove lio_initiator_iqns config option https://review.openstack.org/197354 | 22:12 |
| *** dims_ has joined #openstack-cinder | 22:19 | |
| *** rmesta has joined #openstack-cinder | 22:19 | |
| *** dims has quit IRC | 22:22 | |
| Swanson | so. manage_existing. When getting the size we are returning GB. But since it is an existing volume it might not have been created in even GB. Do I round up or down? | 22:23 |
| smcginnis | Swanson: Up I would think? | 22:25 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: update os-brick to pass python3 tests https://review.openstack.org/196284 | 22:25 |
| patrickeast | smcginnis: Swanson: +1 for up | 22:25 |
| Swanson | Up sez you got a volume bigger than you gots. | 22:25 |
| eharney | Swanson: the LVM driver rounds up | 22:25 |
| Swanson | Madness. | 22:25 |
| Swanson | I can round up. | 22:26 |
| eharney | yeah... that's kinda what i'm trying to figure out right now, this seems odd | 22:26 |
| Swanson | My reasoning for down is that you know you have at least that much space. If it rounds to 0 it is too small for cinder. | 22:27 |
| eharney | wouldn't the LVM driver's idea here be broken if it doesn't resize the LV too? | 22:28 |
| Swanson | Hmmm.. I could resize up on manage. | 22:29 |
| eharney | that's kind of what i would expect, but maybe i'm missing something | 22:29 |
| smcginnis | Swanson: Not sure if we actually want to resize the volume though. | 22:32 |
| smcginnis | Swanson: Down does make sense for the reason you stated. | 22:32 |
| smcginnis | Swanson: Probably better thinking you have less than more. | 22:32 |
| smcginnis | Swanson: We can just tell people "don't do that!" :) | 22:32 |
| *** mriedem has joined #openstack-cinder | 22:32 | |
| Swanson | If we do not want to resize on mange_existing then we need to round down. If resizing is okay then we round up on mange_existing_get_size and actually resize it on the manage_existing. | 22:33 |
| Swanson | manage even. | 22:33 |
| smcginnis | I suppose better to play it safe. | 22:34 |
| eharney | is that playing it safe? then you end up consuming more backend than Cinder thinks you are | 22:34 |
| Swanson | Round down. I doubt it will be a problem very often. | 22:34 |
| eharney | safe would be rejecting things that aren't on a GB boundary | 22:35 |
| smcginnis | eharney: Well, there is that. | 22:35 |
| smcginnis | eharney: That probably would be the safest thing to do. | 22:35 |
| Swanson | I'm thinking that is the safest, too. There is a solution for the user if they want to import it. Resize. | 22:35 |
| smcginnis | Swanson: And for our storage the chance of it not being on a GB boundary is pretty small. | 22:36 |
| openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: update os-brick to pass python3 tests https://review.openstack.org/196284 | 22:36 |
| eharney | Swanson: right | 22:36 |
| *** dontalton has quit IRC | 22:37 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/196888 | 22:37 |
| *** DericHorn-HP has joined #openstack-cinder | 22:38 | |
| *** julim has joined #openstack-cinder | 22:43 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/197273 | 22:45 |
| *** daneyon has quit IRC | 22:52 | |
| *** eharney has quit IRC | 22:53 | |
| *** IlyaG has quit IRC | 22:56 | |
| *** jungleboyj has joined #openstack-cinder | 22:57 | |
| Swanson | eharney: smcginnis thanks! | 23:01 |
| *** dims_ has quit IRC | 23:01 | |
| *** annashen has quit IRC | 23:02 | |
| *** annashen has joined #openstack-cinder | 23:03 | |
| *** markvoelker_ has quit IRC | 23:06 | |
| *** lcurtis has quit IRC | 23:06 | |
| *** annashen has quit IRC | 23:07 | |
| *** annashen has joined #openstack-cinder | 23:07 | |
| *** tsbot_ has quit IRC | 23:07 | |
| *** mgarza has quit IRC | 23:15 | |
| *** mgarza_ has joined #openstack-cinder | 23:15 | |
| *** mgarza_ has quit IRC | 23:16 | |
| *** krtaylor has quit IRC | 23:23 | |
| hemna | ok heading home. USA v Germany should be on now. | 23:24 |
| *** hemna is now known as hemnafk | 23:24 | |
| *** IlyaG has joined #openstack-cinder | 23:25 | |
| *** IlyaG has quit IRC | 23:26 | |
| *** IlyaG has joined #openstack-cinder | 23:27 | |
| *** DericHorn-HP has quit IRC | 23:31 | |
| *** annashen has quit IRC | 23:33 | |
| *** annashen has joined #openstack-cinder | 23:33 | |
| *** dannywilson has quit IRC | 23:35 | |
| *** Lee1092 has quit IRC | 23:35 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Remove unnecessary volume export after generic volume migration https://review.openstack.org/194223 | 23:35 |
| *** annashen has quit IRC | 23:38 | |
| openstackgerrit | Mitsuhiro Tanino proposed openstack/cinder: LVM: Remove unnecessary volume export after generic volume migration https://review.openstack.org/194223 | 23:41 |
| *** krtaylor has joined #openstack-cinder | 23:41 | |
| *** dannywilson has joined #openstack-cinder | 23:43 | |
| *** dims has joined #openstack-cinder | 23:45 | |
| *** dims has quit IRC | 23:46 | |
| *** markvoelker has joined #openstack-cinder | 23:47 | |
| *** rmesta has quit IRC | 23:49 | |
| *** IlyaG_ has joined #openstack-cinder | 23:50 | |
| *** IlyaG has quit IRC | 23:52 | |
| *** dannywilson has quit IRC | 23:54 | |
| *** yamada-h has joined #openstack-cinder | 23:54 | |
| *** dannywilson has joined #openstack-cinder | 23:54 | |
| *** garthb has quit IRC | 23:54 | |
| *** garthb__ has quit IRC | 23:54 | |
| *** yamada-h has quit IRC | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!