*** salv-orl_ has joined #openstack-cinder | 00:02 | |
*** dims_ has joined #openstack-cinder | 00:03 | |
*** dims has quit IRC | 00:04 | |
*** mylu has quit IRC | 00:05 | |
*** salv-orlando has quit IRC | 00:05 | |
*** yangyapeng has quit IRC | 00:06 | |
*** yangyapeng has joined #openstack-cinder | 00:06 | |
*** salv-orl_ has quit IRC | 00:07 | |
*** baojg has joined #openstack-cinder | 00:07 | |
*** baojg has quit IRC | 00:08 | |
*** yangyapeng has quit IRC | 00:11 | |
*** mylu has joined #openstack-cinder | 00:20 | |
*** garthb has quit IRC | 00:22 | |
*** mylu has quit IRC | 00:25 | |
*** sasukeh has quit IRC | 00:27 | |
*** sasukeh has joined #openstack-cinder | 00:28 | |
*** laughterwym has joined #openstack-cinder | 00:29 | |
*** laughterwym has quit IRC | 00:33 | |
*** mriedem has joined #openstack-cinder | 00:34 | |
*** laughterwym has joined #openstack-cinder | 00:35 | |
*** laughterwym has quit IRC | 00:36 | |
*** laughterwym has joined #openstack-cinder | 00:36 | |
*** sasukeh has quit IRC | 00:40 | |
*** krotscheck has joined #openstack-cinder | 00:41 | |
*** laughterwym has quit IRC | 00:43 | |
*** laughterwym has joined #openstack-cinder | 00:45 | |
*** erlon has quit IRC | 00:47 | |
itlinux | I have made some translation to the Italian lang today for Cinder .. | 00:50 |
---|---|---|
itlinux | should be pushed out tonight | 00:50 |
*** gcb has quit IRC | 00:56 | |
*** cheneydc has joined #openstack-cinder | 01:00 | |
*** daneyon has quit IRC | 01:00 | |
*** jdillaman has quit IRC | 01:05 | |
*** mylu has joined #openstack-cinder | 01:06 | |
*** gcb has joined #openstack-cinder | 01:07 | |
*** yangyapeng has joined #openstack-cinder | 01:09 | |
*** sasukeh has joined #openstack-cinder | 01:12 | |
*** ebalduf has quit IRC | 01:13 | |
*** yangyapeng has quit IRC | 01:14 | |
*** kprabhu has quit IRC | 01:21 | |
*** EinstCrazy has joined #openstack-cinder | 01:21 | |
*** mylu has quit IRC | 01:23 | |
*** Lee1092 has joined #openstack-cinder | 01:23 | |
*** mtanino has joined #openstack-cinder | 01:23 | |
*** arch-nemesis has quit IRC | 01:24 | |
*** yangyapeng has joined #openstack-cinder | 01:28 | |
*** daneyon has joined #openstack-cinder | 01:29 | |
*** mriedem has quit IRC | 01:31 | |
*** mylu has joined #openstack-cinder | 01:33 | |
*** salv-orlando has joined #openstack-cinder | 01:35 | |
*** daneyon has quit IRC | 01:35 | |
*** daneyon has joined #openstack-cinder | 01:36 | |
*** Julien-zte has joined #openstack-cinder | 01:37 | |
*** IlyaG has joined #openstack-cinder | 01:38 | |
*** dims_ has quit IRC | 01:40 | |
*** jwcroppe has joined #openstack-cinder | 01:42 | |
*** salv-orlando has quit IRC | 01:42 | |
*** baojg has joined #openstack-cinder | 01:46 | |
*** gouthamr has joined #openstack-cinder | 01:52 | |
*** mylu has quit IRC | 01:54 | |
*** Fdaisuke has joined #openstack-cinder | 01:54 | |
*** Fdaisuke has quit IRC | 01:54 | |
*** gouthamr_ has quit IRC | 01:55 | |
*** Fdaisuke has joined #openstack-cinder | 01:55 | |
*** mylu has joined #openstack-cinder | 01:57 | |
*** bardia has quit IRC | 01:57 | |
*** tpsilva has quit IRC | 01:57 | |
*** mylu has quit IRC | 01:58 | |
*** mylu has joined #openstack-cinder | 02:00 | |
*** haplo37 has joined #openstack-cinder | 02:03 | |
*** panda has quit IRC | 02:05 | |
*** panda has joined #openstack-cinder | 02:05 | |
*** haplo37 has quit IRC | 02:13 | |
*** houming has joined #openstack-cinder | 02:15 | |
*** mylu has quit IRC | 02:15 | |
*** qeelee has joined #openstack-cinder | 02:16 | |
*** mylu has joined #openstack-cinder | 02:17 | |
*** kprabhu has joined #openstack-cinder | 02:17 | |
*** laughterwym has quit IRC | 02:22 | |
*** laughterwym has joined #openstack-cinder | 02:22 | |
*** IlyaG has quit IRC | 02:24 | |
*** iceyao has joined #openstack-cinder | 02:24 | |
*** sasukeh has quit IRC | 02:25 | |
*** iceyao has quit IRC | 02:25 | |
*** laughter_ has joined #openstack-cinder | 02:25 | |
*** lprice has joined #openstack-cinder | 02:26 | |
*** laughterwym has quit IRC | 02:26 | |
*** apoorvad has quit IRC | 02:26 | |
*** iceyao has joined #openstack-cinder | 02:31 | |
*** sasukeh has joined #openstack-cinder | 02:32 | |
*** akshai_ has quit IRC | 02:32 | |
*** gcb has quit IRC | 02:33 | |
*** knikolla has quit IRC | 02:34 | |
*** mylu has quit IRC | 02:38 | |
*** kprabhu has quit IRC | 02:38 | |
*** mylu has joined #openstack-cinder | 02:39 | |
*** haplo37 has joined #openstack-cinder | 02:39 | |
*** akshai has joined #openstack-cinder | 02:40 | |
*** salv-orlando has joined #openstack-cinder | 02:41 | |
*** laughter_ has quit IRC | 02:42 | |
*** laughterwym has joined #openstack-cinder | 02:43 | |
*** laughterwym has quit IRC | 02:44 | |
*** laughterwym has joined #openstack-cinder | 02:44 | |
*** gcb has joined #openstack-cinder | 02:48 | |
*** baojg has quit IRC | 02:48 | |
*** baojg has joined #openstack-cinder | 02:50 | |
*** laughterwym has quit IRC | 02:50 | |
*** laughterwym has joined #openstack-cinder | 02:51 | |
*** haomaiwa_ has joined #openstack-cinder | 02:51 | |
*** salv-orlando has quit IRC | 02:51 | |
*** wN has quit IRC | 02:53 | |
*** gcb has quit IRC | 02:55 | |
*** laughterwym has quit IRC | 02:55 | |
*** iceyao has quit IRC | 02:55 | |
*** laughterwym has joined #openstack-cinder | 02:58 | |
*** markvoelker has quit IRC | 02:59 | |
*** wN has joined #openstack-cinder | 02:59 | |
*** wN has joined #openstack-cinder | 02:59 | |
*** haomaiwa_ has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
openstackgerrit | Xuepeng Ji proposed openstack/cinder-specs: Support volume migration in Ceph driver https://review.openstack.org/296150 | 03:02 |
*** iceyao has joined #openstack-cinder | 03:07 | |
*** gouthamr has quit IRC | 03:10 | |
*** gcb has joined #openstack-cinder | 03:13 | |
*** laughterwym has quit IRC | 03:15 | |
*** laughterwym has joined #openstack-cinder | 03:15 | |
*** sasukeh has quit IRC | 03:16 | |
*** sasukeh has joined #openstack-cinder | 03:17 | |
*** mylu has quit IRC | 03:17 | |
*** ebalduf has joined #openstack-cinder | 03:19 | |
*** laughterwym has quit IRC | 03:19 | |
*** mylu has joined #openstack-cinder | 03:22 | |
*** IlyaG has joined #openstack-cinder | 03:22 | |
*** links has joined #openstack-cinder | 03:28 | |
*** jwcroppe has quit IRC | 03:33 | |
*** jwcroppe has joined #openstack-cinder | 03:34 | |
*** bharath has joined #openstack-cinder | 03:34 | |
*** jwcroppe has quit IRC | 03:39 | |
*** haplo37 has quit IRC | 03:39 | |
*** gcb has quit IRC | 03:40 | |
*** iceyao_ has joined #openstack-cinder | 03:40 | |
*** laughterwym has joined #openstack-cinder | 03:41 | |
*** IlyaG has quit IRC | 03:42 | |
*** cknight has joined #openstack-cinder | 03:42 | |
*** edtubill has joined #openstack-cinder | 03:43 | |
*** iceyao has quit IRC | 03:43 | |
*** laughterwym has quit IRC | 03:45 | |
*** qeelee has quit IRC | 03:51 | |
*** Poornima has joined #openstack-cinder | 03:55 | |
*** markvoelker has joined #openstack-cinder | 03:59 | |
*** arch-nemesis has joined #openstack-cinder | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** haomaiwang has joined #openstack-cinder | 04:01 | |
*** markvoelker has quit IRC | 04:04 | |
*** kprabhu has joined #openstack-cinder | 04:05 | |
*** arch-nemesis has quit IRC | 04:07 | |
*** salv-orlando has joined #openstack-cinder | 04:10 | |
*** haomaiw__ has joined #openstack-cinder | 04:12 | |
*** haomaiwang has quit IRC | 04:13 | |
*** edtubill has quit IRC | 04:14 | |
*** salv-orlando has quit IRC | 04:14 | |
*** laughterwym has joined #openstack-cinder | 04:17 | |
*** arch-nemesis has joined #openstack-cinder | 04:20 | |
*** dave-mccowan has quit IRC | 04:24 | |
*** cknight has quit IRC | 04:28 | |
*** laughter_ has joined #openstack-cinder | 04:31 | |
*** mylu has quit IRC | 04:32 | |
*** laughterwym has quit IRC | 04:33 | |
*** jwcroppe has joined #openstack-cinder | 04:35 | |
*** mylu has joined #openstack-cinder | 04:36 | |
*** mylu has quit IRC | 04:38 | |
*** IlyaG has joined #openstack-cinder | 04:42 | |
*** jwcroppe has quit IRC | 04:45 | |
*** IlyaG has quit IRC | 04:47 | |
*** mylu has joined #openstack-cinder | 04:50 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Raise if no FC port found https://review.openstack.org/296176 | 04:53 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei: Fix getting admin_metadata https://review.openstack.org/295776 | 04:55 |
*** qeelee has joined #openstack-cinder | 04:58 | |
*** haomaiw__ has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** laughter_ has quit IRC | 05:07 | |
*** laughterwym has joined #openstack-cinder | 05:07 | |
*** akshai has quit IRC | 05:07 | |
*** laughter_ has joined #openstack-cinder | 05:08 | |
*** qeelee has quit IRC | 05:11 | |
*** laughterwym has quit IRC | 05:12 | |
*** laughter_ has quit IRC | 05:12 | |
*** mylu has quit IRC | 05:17 | |
*** laughterwym has joined #openstack-cinder | 05:17 | |
*** qeelee has joined #openstack-cinder | 05:17 | |
*** zerda has joined #openstack-cinder | 05:28 | |
*** kprabhu has quit IRC | 05:29 | |
*** sgotliv has joined #openstack-cinder | 05:34 | |
*** lpetrut has joined #openstack-cinder | 05:37 | |
*** jwcroppe has joined #openstack-cinder | 05:41 | |
*** salv-orlando has joined #openstack-cinder | 05:42 | |
*** salv-orlando has quit IRC | 05:49 | |
*** jwcroppe has quit IRC | 05:49 | |
*** qeelee has quit IRC | 05:54 | |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Support for non-default port https://review.openstack.org/267450 | 05:55 |
*** ankit_ag has joined #openstack-cinder | 05:56 | |
*** qeelee has joined #openstack-cinder | 05:57 | |
*** markvoelker has joined #openstack-cinder | 06:00 | |
*** haomaiwang has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** wanghao has quit IRC | 06:01 | |
*** Dongsheng has quit IRC | 06:02 | |
*** wanghao has joined #openstack-cinder | 06:02 | |
*** markvoelker has quit IRC | 06:04 | |
*** panda has quit IRC | 06:05 | |
*** panda has joined #openstack-cinder | 06:06 | |
openstackgerrit | Abhishek Kekane proposed openstack/cinder: Provide user friendly message for FK failure https://review.openstack.org/291083 | 06:06 |
*** lpetrut has quit IRC | 06:15 | |
*** Dongsheng has joined #openstack-cinder | 06:17 | |
*** nkrinner has joined #openstack-cinder | 06:17 | |
*** akolbasov has joined #openstack-cinder | 06:18 | |
*** qeelee has quit IRC | 06:21 | |
*** IlyaG has joined #openstack-cinder | 06:23 | |
*** IlyaG has quit IRC | 06:28 | |
*** ChubYann has quit IRC | 06:32 | |
*** lpetrut has joined #openstack-cinder | 06:35 | |
*** lpetrut has quit IRC | 06:41 | |
*** sgotliv has quit IRC | 06:42 | |
*** sgotliv has joined #openstack-cinder | 06:42 | |
*** jwcroppe has joined #openstack-cinder | 06:46 | |
*** Julien-zte has quit IRC | 06:49 | |
*** jwcroppe has quit IRC | 06:53 | |
*** salv-orlando has joined #openstack-cinder | 06:58 | |
*** e0ne has joined #openstack-cinder | 06:58 | |
*** laughterwym has quit IRC | 06:58 | |
*** laughterwym has joined #openstack-cinder | 07:00 | |
*** haomaiwang has quit IRC | 07:01 | |
*** haomaiwang has joined #openstack-cinder | 07:01 | |
*** markvoelker has joined #openstack-cinder | 07:01 | |
openstackgerrit | zhangsong proposed openstack/cinder: Sheepdog:make full use of all sheepdog nodes https://review.openstack.org/296220 | 07:03 |
*** anshul has joined #openstack-cinder | 07:06 | |
*** haomaiwang has quit IRC | 07:06 | |
*** markvoelker has quit IRC | 07:07 | |
*** ndipanov has quit IRC | 07:08 | |
*** haomaiwang has joined #openstack-cinder | 07:09 | |
*** salv-orlando has quit IRC | 07:15 | |
*** ebalduf has quit IRC | 07:17 | |
*** cristicalin has joined #openstack-cinder | 07:17 | |
*** akolbasov has quit IRC | 07:20 | |
*** bjornar has quit IRC | 07:20 | |
openstackgerrit | Dinesh Bhor proposed openstack/cinder: Provide user friendly messages for db sync https://review.openstack.org/284156 | 07:21 |
*** qeelee has joined #openstack-cinder | 07:22 | |
*** arecknag has joined #openstack-cinder | 07:27 | |
*** e0ne has quit IRC | 07:28 | |
*** shyama_ has joined #openstack-cinder | 07:30 | |
*** qeelee has quit IRC | 07:32 | |
*** shyama has quit IRC | 07:33 | |
*** shyama_ is now known as shyama | 07:33 | |
*** shyama has quit IRC | 07:33 | |
*** shyama has joined #openstack-cinder | 07:34 | |
*** sasukeh has quit IRC | 07:38 | |
*** sgotliv has quit IRC | 07:40 | |
*** tesseract has joined #openstack-cinder | 07:41 | |
*** tesseract is now known as Guest60842 | 07:41 | |
*** sasukeh has joined #openstack-cinder | 07:41 | |
openstackgerrit | Thomas Bechtold proposed openstack/cinder: Remove PyMySQL and psycopg2 from test-requirements.txt https://review.openstack.org/296235 | 07:42 |
*** sasukeh has quit IRC | 07:43 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: Hitachi: extend volume if larger than snapshot https://review.openstack.org/296083 | 07:48 |
*** jwcroppe has joined #openstack-cinder | 07:50 | |
*** arch-nemesis has quit IRC | 07:50 | |
*** jwcroppe has quit IRC | 08:00 | |
*** haomaiwang has quit IRC | 08:01 | |
*** haomaiwa_ has joined #openstack-cinder | 08:01 | |
*** laughterwym has quit IRC | 08:03 | |
openstackgerrit | Merged openstack/cinder: Updated doc string https://review.openstack.org/295910 | 08:03 |
*** laughterwym has joined #openstack-cinder | 08:04 | |
*** sasukeh has joined #openstack-cinder | 08:05 | |
*** sasukeh has quit IRC | 08:09 | |
*** nkrinner has quit IRC | 08:12 | |
*** gcb has joined #openstack-cinder | 08:12 | |
*** EinstCra_ has joined #openstack-cinder | 08:12 | |
*** EinstCrazy has quit IRC | 08:16 | |
*** nkrinner has joined #openstack-cinder | 08:17 | |
*** bjornar has joined #openstack-cinder | 08:30 | |
*** xek has quit IRC | 08:32 | |
*** xek has joined #openstack-cinder | 08:33 | |
*** laughterwym has quit IRC | 08:39 | |
*** laughterwym has joined #openstack-cinder | 08:39 | |
*** Dongsheng has quit IRC | 08:40 | |
openstackgerrit | xiexs proposed openstack/cinder-specs: Enhance error messages for API user https://review.openstack.org/293306 | 08:42 |
*** laughter_ has joined #openstack-cinder | 08:42 | |
*** permalac has joined #openstack-cinder | 08:43 | |
*** qeelee has joined #openstack-cinder | 08:44 | |
*** laughterwym has quit IRC | 08:44 | |
*** laughter_ has quit IRC | 08:47 | |
openstackgerrit | xiaoqin proposed openstack/cinder: Remove CG db access in Storwize driver https://review.openstack.org/296264 | 08:52 |
*** Dongsheng has joined #openstack-cinder | 08:53 | |
openstackgerrit | zwei proposed openstack/python-cinderclient: volumes number than osapi_max_limit https://review.openstack.org/296266 | 08:54 |
*** jwcroppe has joined #openstack-cinder | 08:56 | |
*** haomaiwa_ has quit IRC | 09:01 | |
*** links has quit IRC | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
*** pradipm has joined #openstack-cinder | 09:01 | |
openstackgerrit | zwei proposed openstack/python-cinderclient: volumes number than osapi_max_limit https://review.openstack.org/296266 | 09:02 |
*** e0ne has joined #openstack-cinder | 09:03 | |
*** markvoelker has joined #openstack-cinder | 09:03 | |
*** openstackgerrit has quit IRC | 09:03 | |
*** openstackgerrit has joined #openstack-cinder | 09:04 | |
*** salv-orlando has joined #openstack-cinder | 09:06 | |
openstackgerrit | zwei proposed openstack/python-cinderclient: volumes number greater than osapi_max_limit https://review.openstack.org/296266 | 09:07 |
*** zioproto_switch has joined #openstack-cinder | 09:07 | |
*** markvoelker has quit IRC | 09:08 | |
*** sgotliv has joined #openstack-cinder | 09:08 | |
zioproto_switch | hello. I have a problem where cinder snapshot-list does not return all the snapshots. How ever snapshots are in the database, and they appear if called with cinder snapshot-show. I am running Kilo. I tried both cinder client and openstack client. am I hitting a known bug ? | 09:09 |
zioproto_switch | I did not mention I am running cinder with ceph rbd backend, so the snapshots are ceph snapshots. Also, the snapshots are correctly created in ceph. The bug is that the snapshot-list is incomplete | 09:10 |
*** Dongsheng has quit IRC | 09:11 | |
*** gcb has quit IRC | 09:11 | |
*** houming has quit IRC | 09:12 | |
*** houming has joined #openstack-cinder | 09:14 | |
openstackgerrit | Cao ShuFeng proposed openstack/cinder: [WIP] ehanche error message https://review.openstack.org/296304 | 09:16 |
*** links has joined #openstack-cinder | 09:18 | |
dulek | zioproto_switch: Are these snapshot deleted/unmanaged? Check out `deleted` column in the DB. | 09:22 |
dulek | zioproto_switch: Also these can be owned by another tenant. Admin can see snapshots from all the tenants using --all-tenants option. | 09:23 |
zioproto_switch | dulek: no they are fine | 09:23 |
zioproto_switch | dulek: no same tenant | 09:23 |
zioproto_switch | I found the bug | 09:23 |
zioproto_switch | because I have a script | 09:23 |
zioproto_switch | that makes snapshot create | 09:23 |
zioproto_switch | and then | 09:23 |
zioproto_switch | to check that the snapshot is there, it will call later snapshot-list to check if the snapshot is the creating state | 09:23 |
*** gcb has joined #openstack-cinder | 09:24 | |
zioproto_switch | so it turned out that sometimes I create a snapshot. The snapshot is created successfully and I also get the ID from the output of snapshot-create, However the next snapshot-list calls will not list the snapshot | 09:24 |
zioproto_switch | please check here. I wrote all the details of the troubleshooting http://lists.openstack.org/pipermail/openstack-operators/2016-March/009929.html | 09:25 |
zioproto_switch | all the entries in the 'snapshots' table in the cinder DB look valid. However not all are listed in the snapshot-list output | 09:27 |
openstackgerrit | Vipin Balachandran proposed openstack/cinder: VMware: Support for non-default port https://review.openstack.org/267450 | 09:27 |
zioproto_switch | dulek: is there any other database table that is updated when creating a snapshot ? | 09:28 |
dulek | zioproto_switch: Can you dump a few records that are shown and few that are not from the DB? | 09:28 |
zioproto_switch | yes sure give me a minute | 09:28 |
dulek | zioproto_switch: Also you can check out --debug option for cinderclient to take a look on the raw REST API output. | 09:29 |
zioproto_switch | dulek: you want a select * from snapshots where id='id'; right ? for two snapshots, one that appears and one that does not appear, right ? | 09:31 |
dulek | zioproto_switch: That would be cool. :) | 09:31 |
zioproto_switch | I checked with --debug and the JSON object returned looks OK. The snapshots aremissing also there | 09:31 |
*** cristicalin has quit IRC | 09:33 | |
*** Dongsheng has joined #openstack-cinder | 09:34 | |
zioproto_switch | dulek: http://pastebin.com/N5q9fSi2 | 09:37 |
zioproto_switch | dulek: I have to be away from keyboard 10 minutes, coming back soon | 09:37 |
openstackgerrit | Bhagyashri Shewale proposed openstack/cinder: NexentaStor: Remove extra format string specifier https://review.openstack.org/296344 | 09:41 |
openstackgerrit | ChangBo Guo(gcb) proposed openstack/os-brick: Don't use oslo-incubator stuff https://review.openstack.org/296346 | 09:45 |
*** Dongsheng has quit IRC | 09:45 | |
zioproto_switch | dulek: I am back | 09:46 |
dulek | zioproto_switch: I've looked through the code and I can't find any place where snap can be dropped. DB query looks fine as well. | 09:46 |
dulek | zioproto_switch: Your grep is faulty maybe? :D | 09:46 |
zioproto_switch | dulek: I tried to less and to find the id in the big list | 09:47 |
zioproto_switch | dulek: anyway the list is like 5 times smaller than the entries in the database | 09:47 |
*** anshul has quit IRC | 09:47 | |
dulek | zioproto_switch: I guess you should place a breakpoint using pdb/ipdb in cinder/volume/api.py (get_all_snapshots) and see what's returned from the DB. | 09:47 |
dulek | zioproto_switch: I don't really have an idea what's happening here… | 09:47 |
zioproto_switch | did you read my email on the openstack-operators mailing list ? | 09:48 |
dulek | zioproto_switch: Yup. | 09:49 |
zioproto_switch | the problem is that this is a production system, with 2TB cinder volumes. When I try to reproduce the bug in a small scale in staging system I cannot repoduce it | 09:49 |
*** qeelee has quit IRC | 09:49 | |
zioproto_switch | it sounds like a race condition when creating the 2TB snapshots. I am not sure if i can insert the breakpoint the production system | 09:49 |
dulek | zioproto_switch: Ah… No, breakpoint in production sounds like a very bad idea. | 09:50 |
zioproto_switch | dulek: do you want I paste the output with --debug ? | 09:50 |
dulek | I don't think this will change anything. | 09:50 |
dulek | Thing is that snapshot-list call is really simple - it just gets stuff from DB and prints it. | 09:51 |
dulek | And your DB entries look perfectly sane. | 09:51 |
dulek | Are you using V1 API maybe? | 09:51 |
zioproto_switch | so I am using CinderV2, how to double check this ? | 09:52 |
sheel | dulek: could it be problem at cinder client end? | 09:52 |
sheel | dulek: may be we can check cURL output once to check | 09:52 |
zioproto_switch | I tried different version of cinder client and also openstack client | 09:52 |
dulek | --debug should give you URLs it's calling. | 09:52 |
zioproto_switch | ok give me a moment to make the pastebin | 09:52 |
dulek | sheel: Well, zioproto_switch says that raw REST outputs from --debug are still missing the snapshots. | 09:52 |
sheel | dulek: hmm | 09:53 |
dulek | zioproto_switch: Yeah, maybe this --debug will be useful, let's look at the URLs generated. | 09:53 |
*** sdague has joined #openstack-cinder | 09:58 | |
zioproto_switch | dulek: the paste is too big for the free version of pastebin, hold on! | 10:00 |
*** lpetrut has joined #openstack-cinder | 10:00 | |
*** dims has joined #openstack-cinder | 10:00 | |
dulek | zioproto_switch: You can use paste.openstack.org | 10:00 |
*** haomaiwang has quit IRC | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
*** Dongsheng has joined #openstack-cinder | 10:01 | |
*** cheneydc has quit IRC | 10:02 | |
zioproto_switch | dulek: http://paste.openstack.org/show/491550/ I just deleted the token from the debug output | 10:04 |
*** panda has quit IRC | 10:05 | |
*** links has quit IRC | 10:05 | |
zioproto_switch | dulek: I just see 17 lines... did it work > | 10:05 |
zioproto_switch | ? | 10:05 |
dulek | zioproto_switch: I think so, the last one is a looong one. | 10:06 |
*** baojg has quit IRC | 10:06 | |
*** panda has joined #openstack-cinder | 10:06 | |
zioproto_switch | dulek: yes here is better http://paste.openstack.org/raw/491550/ | 10:07 |
zioproto_switch | dulek: this pastes last forever ? I cannot set an expiration time like on pastebin.com ? | 10:07 |
zioproto_switch | dulek: I feel this paste is not complete ! | 10:08 |
dulek | zioproto_switch: It looks like the line isn't proper JSON. | 10:08 |
dulek | But well - v1/0dbf138e395a4c3cbf2acadf2c7fdf61/snapshots/detail?all_tenants=1 is a correct URL. | 10:09 |
dulek | That's v1 API however. | 10:09 |
*** ndipanov has joined #openstack-cinder | 10:09 | |
*** LZ has quit IRC | 10:09 | |
dulek | zioproto_switch: OKAY! | 10:10 |
dulek | zioproto_switch: Check out CONF.osapi_max_limit. | 10:10 |
dulek | zioproto_switch: V1 is slicing your request according to that setting. | 10:10 |
*** EinstCra_ has quit IRC | 10:10 | |
* dulek feels victorious. :) | 10:11 | |
*** laughterwym has joined #openstack-cinder | 10:11 | |
zioproto_switch | dulek: in cinder.conf ? | 10:12 |
dulek | zioproto_switch: Yup. 1000 by default. | 10:13 |
*** yangyapeng has quit IRC | 10:13 | |
zioproto_switch | ok, so I have more than 1000 snapshots and are notlisted ? | 10:13 |
*** laughterwym has quit IRC | 10:15 | |
dulek | zioproto_switch: I think so. You can use offset parameter to list next page I think. | 10:16 |
zioproto_switch | great, I will try that ! thanks for support | 10:16 |
*** iceyao_ has quit IRC | 10:16 | |
zioproto_switch | I will write to openstack-operators for an update | 10:16 |
dulek | zioproto_switch: I've noticed that on your DB dump - listed snap was waaaay older. | 10:16 |
dulek | zioproto_switch: You can also try V2 API - I haven't noticed similar limit there (I'm not 100% sure though). | 10:17 |
*** Dongsheng has quit IRC | 10:17 | |
*** links has joined #openstack-cinder | 10:22 | |
*** laughterwym has joined #openstack-cinder | 10:23 | |
sheel | dulek: 👍 | 10:25 |
*** tpsilva has joined #openstack-cinder | 10:26 | |
*** laughterwym has quit IRC | 10:28 | |
*** liverpooler has joined #openstack-cinder | 10:28 | |
*** qeelee has joined #openstack-cinder | 10:32 | |
*** Dongsheng has joined #openstack-cinder | 10:34 | |
*** e0ne has quit IRC | 10:35 | |
*** mvk has joined #openstack-cinder | 10:39 | |
*** baojg has joined #openstack-cinder | 10:42 | |
*** laughterwym has joined #openstack-cinder | 10:44 | |
*** e0ne has joined #openstack-cinder | 10:45 | |
openstackgerrit | zwei proposed openstack/python-cinderclient: volumes number greater than osapi_max_limit https://review.openstack.org/296266 | 10:46 |
*** baojg has quit IRC | 10:47 | |
*** laughterwym has quit IRC | 10:48 | |
*** _degorenko|afk is now known as degorenko | 10:51 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** markvoelker has joined #openstack-cinder | 11:04 | |
*** dims_ has joined #openstack-cinder | 11:05 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot https://review.openstack.org/296371 | 11:05 |
*** dims has quit IRC | 11:06 | |
*** markvoelker has quit IRC | 11:08 | |
*** zioproto_switch has quit IRC | 11:12 | |
*** ociuhandu has quit IRC | 11:29 | |
*** shyama_ has joined #openstack-cinder | 11:31 | |
*** shyama has quit IRC | 11:31 | |
*** shyama_ is now known as shyama | 11:31 | |
*** laughterwym has joined #openstack-cinder | 11:33 | |
*** jordanP has joined #openstack-cinder | 11:36 | |
*** laughterwym has quit IRC | 11:37 | |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: NexentaEdge: extend volume if larger than snapshot https://review.openstack.org/296387 | 11:39 |
*** yangyapeng has joined #openstack-cinder | 11:42 | |
*** haplo37 has joined #openstack-cinder | 11:50 | |
*** Dongsheng has quit IRC | 11:52 | |
*** david-lyle_ has joined #openstack-cinder | 11:58 | |
*** ociuhandu has joined #openstack-cinder | 11:59 | |
*** Poornima has quit IRC | 12:00 | |
*** david-lyle has quit IRC | 12:00 | |
*** haomaiwang has quit IRC | 12:01 | |
*** pradipm has quit IRC | 12:01 | |
*** haomaiwang has joined #openstack-cinder | 12:01 | |
*** yangyapeng has quit IRC | 12:01 | |
*** salv-orl_ has joined #openstack-cinder | 12:02 | |
*** yangyapeng has joined #openstack-cinder | 12:02 | |
*** yangyape_ has joined #openstack-cinder | 12:04 | |
*** haplo37 has quit IRC | 12:04 | |
*** lpetrut has quit IRC | 12:04 | |
*** salv-orlando has quit IRC | 12:04 | |
*** Dongsheng has joined #openstack-cinder | 12:04 | |
*** yangyapeng has quit IRC | 12:07 | |
*** smoriya_ has quit IRC | 12:09 | |
*** haomaiwang has quit IRC | 12:16 | |
*** houming has quit IRC | 12:17 | |
*** lpetrut has joined #openstack-cinder | 12:19 | |
*** markvoelker has joined #openstack-cinder | 12:20 | |
*** laughterwym has joined #openstack-cinder | 12:20 | |
openstackgerrit | Erlon R. Cruz proposed openstack/os-brick: Reduce ISCSIConnector rescan intervals https://review.openstack.org/296401 | 12:21 |
*** zerda has quit IRC | 12:22 | |
*** adisky has joined #openstack-cinder | 12:23 | |
openstackgerrit | Merged openstack/cinder: Datera: resize volume if cloned image is larger https://review.openstack.org/294917 | 12:24 |
*** ankit_ag has quit IRC | 12:25 | |
*** edmondsw has joined #openstack-cinder | 12:28 | |
*** raildo-afk is now known as raildo | 12:30 | |
*** qeelee has quit IRC | 12:31 | |
*** krtaylor has quit IRC | 12:32 | |
*** ociuhandu has quit IRC | 12:34 | |
*** ociuhandu has joined #openstack-cinder | 12:35 | |
*** qeelee has joined #openstack-cinder | 12:39 | |
*** EinstCrazy has joined #openstack-cinder | 12:42 | |
*** lpetrut has quit IRC | 12:46 | |
*** lpetrut has joined #openstack-cinder | 12:46 | |
*** krtaylor has joined #openstack-cinder | 12:46 | |
*** baojg has joined #openstack-cinder | 12:50 | |
*** dave-mccowan has joined #openstack-cinder | 12:52 | |
*** ociuhandu has quit IRC | 12:52 | |
*** ociuhandu has joined #openstack-cinder | 12:53 | |
*** mtanino has quit IRC | 12:53 | |
*** haomaiwang has joined #openstack-cinder | 12:57 | |
*** porrua has joined #openstack-cinder | 12:57 | |
*** markus_z is now known as markus_z_meeting | 12:59 | |
*** haomaiwang has quit IRC | 13:01 | |
*** links has quit IRC | 13:01 | |
*** arecknag has quit IRC | 13:01 | |
*** akerr has joined #openstack-cinder | 13:02 | |
*** lpetrut has quit IRC | 13:03 | |
openstackgerrit | Merged openstack/cinder: typo: add a missing '}' in comment. https://review.openstack.org/291513 | 13:03 |
*** darosale has joined #openstack-cinder | 13:04 | |
*** david-lyle_ is now known as david-lyle | 13:06 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 13:08 |
*** rlrossit has joined #openstack-cinder | 13:08 | |
*** akerr has quit IRC | 13:09 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 13:09 |
dulek | e0ne: +13, -6553 :) | 13:10 |
e0ne | dulek: I think that there are too many code in cinder :) | 13:11 |
dulek | e0ne: You're not alone in that opinion. :D | 13:11 |
e0ne | :) | 13:11 |
geguileo | +1 | 13:11 |
*** jwcroppe has quit IRC | 13:12 | |
e0ne | geguileo: why only +1? | 13:13 |
geguileo | Because I have mixed feelings, as it probably breaks like 10 of my patches that are up for review and another 5 I have downstream for the Active/Active cleanup | 13:14 |
geguileo | rofl | 13:14 |
geguileo | Just kidding, +1 is because I meant as in "I agree with that statement" | 13:15 |
geguileo | Not as in a review vote ;-) | 13:15 |
e0ne | :) | 13:15 |
e0ne | btw, it took about hour to rabase it :( | 13:15 |
e0ne | mostly because of new imports | 13:15 |
dulek | So it looks like besides having too much code we also have too much patches. ;) | 13:16 |
geguileo | e0ne: I can relate, until last week or so I've been rebasing my patches like once or twice a day... | 13:16 |
geguileo | dulek: lol | 13:16 |
e0ne | it's a rebase hell | 13:16 |
geguileo | Yep | 13:17 |
e0ne | no code, no patches, no needs to review, no bugs - I like it | 13:17 |
geguileo | rofl | 13:17 |
dulek | e0ne: Looks like you can add a release note in your patch. | 13:17 |
e0ne | dulek: good point | 13:18 |
e0ne | I'll do | 13:18 |
*** mriedem has joined #openstack-cinder | 13:18 | |
jgriffith | WTF? | 13:20 |
smcginnis | jgriffith: Uh oh, what's that for? | 13:21 |
dulek | Good morning John! :) | 13:22 |
jgriffith | LOL | 13:22 |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Remove XML API https://review.openstack.org/231663 | 13:22 |
e0ne | dulek: done | 13:22 |
e0ne | jgriffith: morning, John :) | 13:22 |
jgriffith | Anybody try setting a volume to an Error status and then deleting it lately? | 13:22 |
jgriffith | Looks like we never send the call down to the driver any more if the volume is in Error status | 13:23 |
* dulek spins up a VM… | 13:23 | |
e0ne | jgriffith: didn't try to do such things | 13:23 |
jgriffith | dulek: hehe... I was just doing the same thing; incase I've got a mess in my current branch | 13:23 |
*** arecknag has joined #openstack-cinder | 13:24 | |
dulek | jgriffith: Current master: http://paste.openstack.org/show/491578/ | 13:25 |
dulek | Seems to work okay. | 13:26 |
jgriffith | dulek: hmm... odd | 13:26 |
jgriffith | dulek: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L369 | 13:27 |
dulek | jgriffith: https://github.com/openstack/cinder/blob/master/cinder/volume/api.py#L359 | 13:27 |
dulek | jgriffith: If it has no host set, then we don't know where to send the request. | 13:28 |
jgriffith | dulek: Ohh.... you misunderstand me | 13:28 |
*** arecknag has quit IRC | 13:28 | |
dulek | jgriffith: So we're just deleting that from the DB. | 13:28 |
jgriffith | dulek: we DO delete the object and db entry | 13:28 |
jgriffith | BUT we never send the call to the driver which is really bad | 13:28 |
*** qeelee has quit IRC | 13:28 | |
jgriffith | dulek: so we get orphaned volumes on the backend | 13:28 |
dulek | jgriffith: In that case we don't know where to send it. | 13:28 |
jgriffith | dulek: well, that's a bug | 13:28 |
jgriffith | we should never have that problem | 13:28 |
dulek | jgriffith: Wait, wait, wait. | 13:29 |
jgriffith | dulek: I'm not even sure that's the path it hit, checking now | 13:29 |
*** arecknag has joined #openstack-cinder | 13:29 | |
*** arecknag has quit IRC | 13:29 | |
dulek | jgriffith: If a volume fails before scheduling, then it was never created on the backend. | 13:29 |
dulek | jgriffith: And the only moment when host=NULL should be when request never got to c-vol. | 13:30 |
jgriffith | dulek: so I've found a case where the volume is created.. but I then modify the create call in my driver to raise an error | 13:30 |
jgriffith | dulek: it's entirely possible/normal to expect that a call can go to the driver, and return an exception BUT that the volume was actually created | 13:30 |
dulek | jgriffith: Isn't volume.host set by cinder-scheduler? | 13:31 |
jgriffith | dulek: ie; a resize failed... replication setup failed etc | 13:31 |
dulek | Let's get one step back. | 13:31 |
*** arecknag has joined #openstack-cinder | 13:31 | |
dulek | The path you're showing happens *only* when volume.host=NULL. | 13:31 |
jgriffith | dulek: no worries... I'll trace through it and figure it out | 13:31 |
jgriffith | dulek: like I said that may not even be the path I hit :) Could be something else, but that's still bad | 13:32 |
jgriffith | we've apparantly introduced some fragility somewhere | 13:32 |
dulek | jgriffith: Okay! I don't think this will be the path you're hitting. | 13:32 |
jgriffith | dulek: haha... it is :) | 13:33 |
dulek | jgriffith: So your volume.host is NULL? | 13:33 |
jgriffith | dulek: indeed | 13:34 |
jgriffith | and I tihnk it's because the manager catches the exception and bales | 13:34 |
jgriffith | dulek: without ever updating the object... looking now | 13:35 |
dulek | jgriffith: But if manager (you mean c-vol) got the request, then volume.host should be set. | 13:35 |
jgriffith | dulek: indeed, it *should* | 13:35 |
dulek | jgriffith: It is set just before executing RPC call to c-vol by c-sch or c-api. | 13:35 |
dulek | jgriffith: Any specifics on how you're creating a volume? | 13:35 |
jgriffith | dulek: so do this... just modify the last line of driver.create; | 13:36 |
jgriffith | dulek: have it raise a driver api exception | 13:36 |
jgriffith | or some random exception | 13:36 |
jgriffith | *after* actually creating the volume on the backend though | 13:36 |
dulek | jgriffith: I've added one as last line in create_volume in driver.lvm, restarting services… | 13:37 |
dulek | jgriffith: uh, oh… | 13:38 |
jgriffith | dulek: yup | 13:39 |
dulek | jgriffith: In my case rescheduling happened. Maybe there's a problem. | 13:39 |
*** knikolla has joined #openstack-cinder | 13:39 | |
jgriffith | dulek: oh... yeah, that's the other thing I've had distaste on for a couple years | 13:39 |
jgriffith | dulek: Tflow will retry on unknown exceptions | 13:40 |
*** arecknag has quit IRC | 13:40 | |
*** haomaiwang has joined #openstack-cinder | 13:40 | |
jgriffith | which causes a whole host of other issues... like creating a bunch of volumes that now Cinder isn't tracking | 13:40 |
dulek | jgriffith: Makes sense. We could've identify cases when rescheduling is needed because of race-condition-prone scheduler and reschedule only then. | 13:41 |
*** iceyao has joined #openstack-cinder | 13:41 | |
jgriffith | dulek: well... I've always felt it would be *better* to reverse it from a blacklist of retries to a whitelist | 13:42 |
jgriffith | dulek: so ONLY retry known retryable things as opposed to what we do with "retry anything not on list" | 13:42 |
jgriffith | although, I thought we did change that | 13:42 |
jgriffith | perhaps we just need to add another item to the list :) | 13:43 |
jgriffith | regardless... one problem at a time :) | 13:43 |
*** timcl has joined #openstack-cinder | 13:43 | |
dulek | jgriffith: I think it's still a blacklist - no_reschedule_types. | 13:43 |
jgriffith | dulek: yup it is :( | 13:43 |
jgriffith | dulek: so where's that host update supposed to happen from the create flow? | 13:44 |
*** mtanino has joined #openstack-cinder | 13:44 | |
dulek | jgriffith: https://github.com/openstack/cinder/blob/master/cinder/scheduler/filter_scheduler.py#L93 | 13:45 |
dulek | jgriffith: In case of rescheduling it should be the same - request gets back to scheduler. | 13:45 |
*** mvk_ has joined #openstack-cinder | 13:45 | |
*** porrua has quit IRC | 13:46 | |
jgriffith | dulek: hmm... but does the object ever get updated? | 13:46 |
jgriffith | dulek: because that's still going direct to db no? | 13:46 |
jgriffith | dulek: never mind | 13:46 |
dulek | jgriffith: Yeah, objects are mostly proyxing requests to DB. | 13:46 |
jgriffith | dulek: db isn't set either... so it's certainly not that | 13:46 |
*** Yogi1 has joined #openstack-cinder | 13:47 | |
*** klkumar has joined #openstack-cinder | 13:48 | |
*** mkrcmari__ has joined #openstack-cinder | 13:48 | |
dulek | jgriffith: ipdb tells me that something's cleans up host on rescheduling. Looking further… | 13:49 |
*** jgregor has joined #openstack-cinder | 13:49 | |
*** mvk has quit IRC | 13:50 | |
jgriffith | dulek: yeah... just saw that. So we do infact set it, but after failure it's kind enough to "clean up" the entry :( | 13:50 |
dulek | jgriffith: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L110-L117 | 13:51 |
dulek | jgriffith: git blame shows me, but I think I've just moved that code, let me look… | 13:51 |
jgriffith | dulek: it's certainly possible this was introduced *before* you moved the code | 13:52 |
*** mvk_ has quit IRC | 13:52 | |
jgriffith | dulek: so there's still something fishy though | 13:52 |
dulek | jgriffith: Yeah, I've just moved the method up to maintain logic order in _pre_rescheduling, rescheduling, _post_rescheduling. | 13:53 |
jgriffith | dulek: first there's the resched problem... | 13:53 |
*** cknight has joined #openstack-cinder | 13:53 | |
*** dustins has joined #openstack-cinder | 13:53 | |
jgriffith | dulek: but aside from that... the pre-schedule should be sequential | 13:53 |
jgriffith | dulek: so the *last* retry shouldn't call this again | 13:53 |
jgriffith | dulek: see what I mean? | 13:53 |
jgriffith | dulek: I'm not being very clear :) | 13:54 |
*** mriedem is now known as mriedem_meeting | 13:55 | |
dulek | jgriffith: I know what you mean. | 13:55 |
jgriffith | dulek: ok, thanks!! | 13:55 |
jgriffith | I'm sitting here trying to type it out :) Easier when you just say "yeah, I get it" :) | 13:55 |
jgriffith | dulek: so we're kinda F'd in two ways here: | 13:55 |
jgriffith | 1. Rescheduling causing us to create multiple orphaned volumes on a backend in certain cases | 13:56 |
*** prazumovsky has joined #openstack-cinder | 13:56 | |
*** baumann has joined #openstack-cinder | 13:56 | |
jgriffith | 2. the host update to None screwing us over | 13:56 |
prazumovsky | Hi! You have 1.5.0 version of your client, but global requirements still has >=1.3.1. When you think to update it? | 13:57 |
*** mvk has joined #openstack-cinder | 13:57 | |
*** haplo37 has joined #openstack-cinder | 13:57 | |
*** mkrcmari__ has quit IRC | 13:58 | |
dulek | jgriffith: I'm tracking that further to see why this host is added here. | 13:58 |
dulek | (it still blames to me BTW…) | 13:59 |
jgriffith | dulek: haha | 13:59 |
*** BigWillie has joined #openstack-cinder | 13:59 | |
*** haomaiwang has quit IRC | 14:01 | |
jgriffith | dulek: that's what you get for submitting code :) | 14:01 |
*** haomaiwang has joined #openstack-cinder | 14:01 | |
*** mvk has quit IRC | 14:01 | |
dulek | jgriffith: Don't worry, you've approved it: https://review.openstack.org/#/c/154920 :D | 14:02 |
jgriffith | dulek: LOL | 14:02 |
jgriffith | dulek: touchet | 14:03 |
dulek | jgriffith: I don't really know why this resetting of volume.host is added there… | 14:04 |
*** dims has joined #openstack-cinder | 14:04 | |
*** dims_ has quit IRC | 14:04 | |
*** panda has quit IRC | 14:04 | |
jgriffith | dulek: yeah, me neither; and I don't see where it "was" prior | 14:05 |
*** panda has joined #openstack-cinder | 14:05 | |
*** akerr has joined #openstack-cinder | 14:05 | |
jgriffith | dulek: so even if we just did an update without clearing it that would be better | 14:05 |
jgriffith | dulek: we still have some issues with the reschedule, but at least we wouldn't have cases where we nuke the host column | 14:07 |
*** salv-orl_ has quit IRC | 14:07 | |
jgriffith | dulek: I'm of the opinion that the auto-retry is mostly useless and more harmful than good anyway | 14:07 |
jgriffith | dulek: 9 9% of the time it picks the same host anyway | 14:08 |
jgriffith | dulek: we're not like Nova with 100 compute nodes to choose from | 14:08 |
*** openstack has joined #openstack-cinder | 14:24 | |
*** ametts has joined #openstack-cinder | 14:24 | |
mmarin | Hello I'm the French translator for Openstack Cinder and I have a question regarding strings like: "MSG_DENIED: RPC_MISMATCH: %r" | 14:24 |
smcginnis | mmarin: Hmm, yeah. :) | 14:24 |
mmarin | Is there anything to translate in here or are those variables that will be replaced by messages | 14:24 |
smcginnis | mmarin: You can probably just skip those? | 14:24 |
mmarin | OK we have to have 100% complete by Friday | 14:25 |
mmarin | so can't skip | 14:25 |
smcginnis | mmarin: Or MSG_REFUSE ? | 14:25 |
smcginnis | mmarin: Or MSG_REFUS ? | 14:25 |
smcginnis | mmarin: But I would probably just leave it as is. That's not really a user friendly string anyway. | 14:26 |
mmarin | I totally agree with you! | 14:26 |
smcginnis | mmarin: Cool. Thanks for translating! | 14:26 |
mmarin | Do you have some time to answer other questions? | 14:26 |
smcginnis | mmarin: Sure, I can try. | 14:26 |
mmarin | What does "child usage" mean in the following string ? Sum of child usage '%(sum)s' is greater than free quota of '%(free)s' for project '%(proj)s' for resource '%(res)s'. Please lower the limit or usage for one or more of the following projects: '%(child_ids)s' | 14:27 |
mmarin | is it the usage by a child device? | 14:27 |
smcginnis | mmarin: That is for the nested quotas. | 14:27 |
DuncanT | mmarin: That's for hierarchical quotas | 14:27 |
smcginnis | mmarin: So there is a "parent" quota that contains one or more "child" quotas. | 14:27 |
mmarin | Ok great | 14:27 |
mmarin | that makes sense now! | 14:28 |
smcginnis | mmarin: Whew, that was an easy one. :) | 14:28 |
mmarin | Not for the translator believe me, without your explanation it's really a nightmare | 14:28 |
smcginnis | mmarin: I can imagine! That is an enormous undertaking. | 14:29 |
mmarin | Should we translate concatenated words or words with underscores or are they keywords like: "consistencygroup assigned" "consistencygroup changed" "glance_metadata changed" | 14:30 |
mmarin | Should we leave them in English? | 14:30 |
mmarin | consistencygroup and glance_metadata or they can be translated with space instead of concatenation? | 14:30 |
*** IlyaG has joined #openstack-cinder | 14:30 | |
*** liverpooler has quit IRC | 14:31 | |
smcginnis | glance_metadata may be referring to a field, so that should probably stay. | 14:31 |
smcginnis | consistencygroup could be broken out to "consistency group" or its translated equivalent. | 14:32 |
mmarin | ok thanks | 14:32 |
*** crose has joined #openstack-cinder | 14:32 | |
mmarin | What does vnr stands for in the following string? DRBDmanage timeout waiting for volume size; volume ID "%(id)s" (res "%(res)s", vnr %(vnr | 14:32 |
smcginnis | Huh, now you got me. Where is that one located. Maybe I can get some context from the source. | 14:33 |
mmarin | well I have no context it's in cinder.po file somewhere | 14:33 |
mmarin | I only have this segment without further context | 14:34 |
smcginnis | Oh, OK. I'll do a quick search. | 14:34 |
mmarin | I have other questions that might be easier, I can leave vnr for the moment | 14:34 |
*** tongli has joined #openstack-cinder | 14:34 | |
*** kprabhu has joined #openstack-cinder | 14:34 | |
smcginnis | OK | 14:34 |
mmarin | "Option map (cls._map) is not defined." What is an Option map ? | 14:35 |
mmarin | a map of options? | 14:35 |
smcginnis | Sounds reasonable. Yeah, I think that is correct. | 14:36 |
smcginnis | We really have some confusing strings, don't we? :) | 14:36 |
geguileo | mmarin: vnr stands for volume number | 14:36 |
geguileo | mmarin: It's something specific to that driver | 14:37 |
mmarin | ok so I will leave it as is | 14:37 |
geguileo | mmarin: Yes, I think it's best | 14:37 |
mmarin | Is portgroup translatable or not in the following string? "Cannot get the portgroup from the masking view." | 14:39 |
*** adisky has quit IRC | 14:40 | |
mmarin | sometime we have portgroup, sometime port group, for translators when words are concatenated they suppose to be keywords and are not translated will that rule applies here? | 14:40 |
*** IlyaG has quit IRC | 14:40 | |
*** andymaier has joined #openstack-cinder | 14:40 | |
smcginnis | I think I would treat that as a keyword. | 14:40 |
mmarin | ok portgroup will stay portgroup | 14:40 |
*** arecknag has joined #openstack-cinder | 14:42 | |
mmarin | "Storage-assisted migration failed during manage volume." What is a storage-assisted migration? | 14:42 |
*** mriedem_meeting is now known as mriedem | 14:43 | |
smcginnis | mmarin: Migration performed with help of the storage backend. Offloaded from the nova host. | 14:43 |
mmarin | waow I could really not have guessed that one! Thank you so much | 14:43 |
smcginnis | ;) | 14:44 |
*** jungleboyj has joined #openstack-cinder | 14:44 | |
mmarin | I have a question for "manage volume" and "unmanage volume", manage volume ok but what does unmanage volume mean to stop managing a volume or are "manage volume" and "unmanage volume" command names? | 14:47 |
*** david-lyle_ is now known as david-lyle | 14:47 | |
smcginnis | mmarin: Correct, it is to stop managing a volume. So the opposite meaning of manage volume. | 14:47 |
*** mdenny has quit IRC | 14:48 | |
mmarin | Ok thanks a lot | 14:48 |
*** akshai has joined #openstack-cinder | 14:48 | |
*** mdenny has joined #openstack-cinder | 14:48 | |
*** zzzeek has quit IRC | 14:50 | |
mmarin | I have a warning when I use apostrophe in several segments, should I double the apostrophe or should I leave my apostrophes as is? | 14:51 |
*** iceyao has quit IRC | 14:52 | |
*** atuvenie has quit IRC | 14:53 | |
smcginnis | mmarin: Is that a zanata issue? I'm not sure how that works. | 14:53 |
*** arch-nemesis has joined #openstack-cinder | 14:53 | |
mmarin | Here is another not user friendly string: "create_volume, volume: %(volume)s, volumename: %(volumename)s, eternus_pool: %(eternus_pool)s, Storage Configuration Service not found." | 14:53 |
mmarin | I am wondering what to translate here except for Storage Configuration Service not found | 14:54 |
mmarin | should I leave the rest as is create_volume, volumename, eternus_pool? | 14:54 |
jgriffith | uh oh.... lights are flickering again | 14:54 |
smcginnis | jgriffith: NetApp forget to pay the electric bill. :P | 14:55 |
smcginnis | mmarin: Yeah, I think that makes sense. | 14:55 |
openstackgerrit | Vincent Hou proposed openstack/cinder: Storwize: do the correct update after failover https://review.openstack.org/292570 | 14:56 |
mmarin | smcginnis: ok thanks a lot for your help! | 14:56 |
jgriffith | smcginnis: LOL... we're having a blizzard here | 14:57 |
*** bjornar has quit IRC | 14:57 | |
jgriffith | seems I've been lucky to have power this morning, the outage map shows pretty much everywhere around my house is "out" | 14:57 |
jgriffith | smcginnis: once again proving that overhead power lines are a bad idea! | 14:58 |
jgriffith | burry all the things!!! | 14:58 |
*** haomaiwang has quit IRC | 15:01 | |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
*** zzzeek has joined #openstack-cinder | 15:02 | |
patrickeast | Meeting time? | 15:02 |
patrickeast | Err wait | 15:02 |
patrickeast | Nvm | 15:03 |
Swanson | DST | 15:03 |
*** baojg has quit IRC | 15:03 | |
*** EinstCrazy has quit IRC | 15:03 | |
patrickeast | Yea I also forgot that I'm one hour off from PST so my calendar thingy is wrong | 15:03 |
diablo_rojo | patrickeast: One more hour :) | 15:03 |
* jgriffith keeps saying we should all just use UTC and be done with it :) | 15:05 | |
Swanson | jgriffith, with underground power lines you are at the mercy of the C.H.U.D.s. I'd rather have them above ground. | 15:05 |
jgriffith | Swanson: I don't know what C.H.U.D.s is? | 15:05 |
Swanson | jgriffith, cannibalistic humanoid underground dweller. | 15:06 |
jgriffith | Swanson: ha! I should've known! | 15:06 |
* smcginnis shakes head | 15:06 | |
smcginnis | jgriffith: We're supposed to potentially get a foot of snow here. | 15:07 |
smcginnis | jgriffith: Time to get out the skis. | 15:07 |
Swanson | smcginnis, wait, what? | 15:07 |
jgriffith | Swanson: but overhead power lines mean man eating electric monkeys | 15:07 |
patrickeast | I should have checked the weather forecast before coming to Denver, thus snow is a pain | 15:07 |
jgriffith | smcginnis: nice!! | 15:07 |
patrickeast | This* | 15:07 |
smcginnis | patrickeast: You in Denver right now? | 15:07 |
Swanson | smcginnis, I took the snow tires off! | 15:07 |
patrickeast | Yea | 15:07 |
smcginnis | Swanson: Of course. And that is why we are getting snow. | 15:08 |
smcginnis | patrickeast: Nice. I'd rather be in Denver with snow than MN with snow. | 15:08 |
jgriffith | smcginnis: the problem here is it started raining at about 2 am when I woke up... then the temp dropped and everything froze. Add a foot of snow and 50 MPH winds... it's VERY PLEASANT | 15:08 |
smcginnis | jgriffith: Sounds darn lovely. | 15:08 |
jgriffith | smcginnis: I'm not interested in going outside again, let alone skiing | 15:08 |
smcginnis | 2am ? | 15:08 |
jgriffith | yeah... couldn't sleep | 15:08 |
smcginnis | Bleh | 15:08 |
jgriffith | indeed | 15:08 |
smcginnis | I get the same thing time to time. I've gone in to work at 3-4am because I finally gave up on any chance of falling asleep again. | 15:09 |
jgriffith | smcginnis: exactly... might as well "use" the time | 15:09 |
jgriffith | smcginnis: beats sitting there being pissed off staring at the ceiling | 15:10 |
smcginnis | I usually pay for it later, but at least I get a few hours of uninterupted productivity. | 15:10 |
*** thrawn01_ has joined #openstack-cinder | 15:10 | |
jgriffith | haha... yeah, I"ll be good til about noon | 15:10 |
smcginnis | If you suddenly drop off from the meeting we'll know either the snow took you out or you fell asleep on your keyboard and the drool shorted it out. :) | 15:11 |
*** ebalduf has joined #openstack-cinder | 15:11 | |
*** openstackstatus has joined #openstack-cinder | 15:11 | |
*** ChanServ sets mode: +v openstackstatus | 15:11 | |
*** garthb has joined #openstack-cinder | 15:12 | |
*** lpetrut has joined #openstack-cinder | 15:12 | |
Swanson | smcginnis, Wait a minute. I've paid for snow removal all season and gotten nothing out of it. I hope 6 feet fall. | 15:15 |
*** knikolla_ has joined #openstack-cinder | 15:15 | |
smcginnis | :) | 15:17 |
*** sgotliv has quit IRC | 15:17 | |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Move test from test_glusterfs and test_quobyte to test_remotefs https://review.openstack.org/261549 | 15:19 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: add ploop volume format support https://review.openstack.org/262246 | 15:19 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: vzstorage: fix creating and deleting snapshots of in-use volumes https://review.openstack.org/276465 | 15:19 |
openstackgerrit | Evgeny Antyshev proposed openstack/cinder: Use versionedobjects in remotefs.py https://review.openstack.org/261558 | 15:19 |
*** knikolla_ has quit IRC | 15:25 | |
*** crose has quit IRC | 15:27 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
*** tongli has quit IRC | 15:36 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: Fix coverage generation https://review.openstack.org/296107 | 15:38 |
Swanson | Thanks for the review jungleboyj. Anyone else want to give the high sign to https://review.openstack.org/#/c/296040/ ? Quick patch to fix volume size in create_volume_from_snapshot. | 15:40 |
*** markus_z_meeting is now known as markus_z | 15:44 | |
*** bjornar has joined #openstack-cinder | 15:49 | |
*** adrianofr_ has joined #openstack-cinder | 15:52 | |
*** bharath has quit IRC | 15:56 | |
mmarin | Hi, anyone can tell me what dGB means for volume size? Is it translatable? "source volume %(src_vol)s size is %(src_size)dGB and doesn't fit in target volume %(tgt_vol)s of size %(tgt_size)dGB." or is the "d" part of the variable here? | 15:56 |
*** tongli has joined #openstack-cinder | 15:56 | |
*** apoorvad has joined #openstack-cinder | 15:58 | |
*** bjornar has quit IRC | 15:58 | |
*** IlyaG has joined #openstack-cinder | 15:59 | |
*** mvk has joined #openstack-cinder | 15:59 | |
*** apoorvad has quit IRC | 15:59 | |
*** laughterwym has quit IRC | 15:59 | |
*** apoorvad has joined #openstack-cinder | 15:59 | |
*** lpetrut has quit IRC | 15:59 | |
*** daneyon has quit IRC | 16:00 | |
*** haomaiwang has quit IRC | 16:01 | |
mc_nair | mmarin: no, it's separate from the GB. The "d" is saying that the "tgt_size" variable is an base 10 integer | 16:01 |
*** tongli has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-cinder | 16:01 | |
mc_nair | as opposed to a string, or things like that | 16:01 |
mc_nair | mmarin: https://docs.python.org/2/library/string.html#format-specification-mini-language | 16:01 |
*** lpetrut has joined #openstack-cinder | 16:03 | |
*** vincent_hou has joined #openstack-cinder | 16:04 | |
mmarin | mc_nair: I will leave it for now since it's attached to a variable and we don't know how it will be resolved | 16:05 |
*** links has joined #openstack-cinder | 16:06 | |
mmarin | mc_nair: %(src_size)dGB usually the variable include a letter after the parenthesis so it's tricky here | 16:06 |
*** wanghao has quit IRC | 16:07 | |
*** wanghao has joined #openstack-cinder | 16:08 | |
*** erlon has joined #openstack-cinder | 16:08 | |
*** tongli has joined #openstack-cinder | 16:17 | |
*** sage has quit IRC | 16:18 | |
mc_nair | mmarin: sorry - I don't think I'm being clear. The '%(src_size)d' will just get replaced with some number. So the end string could be "source volume vol_id_1 size is 5GB and doesn't fit in target volume vol_id_2 of size 3GB." | 16:18 |
mc_nair | mmarin: does that help? | 16:18 |
*** sage has joined #openstack-cinder | 16:19 | |
*** timcl has quit IRC | 16:19 | |
*** Guest60842 has quit IRC | 16:21 | |
mmarin | mc_nair : thank you so much for your answer | 16:21 |
mmarin | that makes sense | 16:21 |
mmarin | Hi another question for strings like "Create hypermetro error. %s." does it mean there is an error creating hypermetro or to create an error for hypermetro? | 16:22 |
*** tongli has quit IRC | 16:22 | |
*** andymaier has quit IRC | 16:22 | |
diablo_rojo | mmarin: I believe that means there was an error creating hypermetro | 16:24 |
*** arecknag has quit IRC | 16:28 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/cinder: Remove local_dev from cinder https://review.openstack.org/260756 | 16:29 |
*** crose has joined #openstack-cinder | 16:31 | |
*** salv-orlando has joined #openstack-cinder | 16:31 | |
*** rajinir has joined #openstack-cinder | 16:32 | |
*** tongli has joined #openstack-cinder | 16:32 | |
*** raunak has joined #openstack-cinder | 16:32 | |
*** bardia has joined #openstack-cinder | 16:33 | |
*** david-lyle has quit IRC | 16:33 | |
*** tongli has quit IRC | 16:33 | |
*** tongli has joined #openstack-cinder | 16:33 | |
*** david-lyle has joined #openstack-cinder | 16:33 | |
*** jwcroppe has joined #openstack-cinder | 16:34 | |
*** salv-orlando has quit IRC | 16:38 | |
openstackgerrit | Walter A. Boring IV (hemna) proposed openstack/os-brick: os-brick refactor get_connector_properties https://review.openstack.org/275943 | 16:39 |
*** salv-orlando has joined #openstack-cinder | 16:42 | |
mmarin | diablo_rojo: thank you I will take the meaning "there was an error creating hypermetro" | 16:44 |
diablo_rojo | mmarin: Yes I looked at the code and I believe that is the correct interpretation of that phrasing. | 16:45 |
*** timcl has joined #openstack-cinder | 16:47 | |
mmarin | diablo_rojo: We have a lot of strings like this, for example "Create replication pair failed.", etc so I guess it's ok | 16:47 |
mmarin | to interpreate it as error while creating xxx | 16:47 |
diablo_rojo | mmarin: Yeah I've seen that a lot throughout the code. | 16:47 |
mmarin | Create replication pair failed. what is a replication pair? | 16:48 |
mmarin | pair is a pair system or two elements for the replication? | 16:48 |
diablo_rojo | mmarin: The pairing between two backends in a replication relationship. | 16:49 |
*** lpetrut has quit IRC | 16:49 | |
*** links has quit IRC | 16:49 | |
mmarin | so it means creating the association of two backends failed? | 16:50 |
mmarin | creating association of two backends for the replication failed, correct? | 16:51 |
openstackgerrit | Danny Al-Gaaf proposed openstack/cinder: HPE LeftHand: extend volume if larger than snapshot https://review.openstack.org/296371 | 16:51 |
smcginnis | mmarin: Yes, I think that is correct. | 16:52 |
diablo_rojo | hemna: I found a selfie stick that doubles as a tripod on amazon that I am going to get for the summit so you don't need to worry about a tripod. | 16:52 |
hemna | oh? | 16:52 |
hemna | url ? | 16:52 |
smcginnis | I have duct tape. | 16:52 |
diablo_rojo | hemna: http://www.amazon.com/gp/product/B00KD74KKQ/ref=crt_ewc_title_dp_2?ie=UTF8&psc=1&smid=ABSHX6ONBFDMO | 16:52 |
diablo_rojo | hemna: Has the regular camera mount too | 16:52 |
mmarin | diablo_rojo: Split replication failed. what does that mean? the action of splitting a replication failed? | 16:52 |
hemna | diablo_rojo, nice | 16:53 |
*** gouthamr has quit IRC | 16:53 | |
diablo_rojo | hemna: Yep. So I will have that :) Just need your camera/sound equipment. | 16:53 |
*** cdelatte has joined #openstack-cinder | 16:53 | |
diablo_rojo | jgriffith: wanna help mmarin out with that translation? | 16:53 |
jungleboyj | mmarin: Where is that in the code? | 16:54 |
jgriffith | diablo_rojo: hmm... sounds like maybe a typo? | 16:54 |
*** salv-orlando has quit IRC | 16:54 | |
smcginnis | Huawei replication error message. | 16:55 |
smcginnis | Maybe something specific to them. | 16:55 |
mmarin | I cannot tell you where it is in the code, in the translation tool I use I just have segments without any context | 16:55 |
smcginnis | jgriffith: What is this call? https://github.com/openstack/cinder/blob/69e8575dc85eee308522623f426eea2638096623/cinder/volume/drivers/huawei/replication.py#L187 | 16:55 |
jgriffith | smcginnis: that's not one of mine | 16:56 |
smcginnis | jgriffith: I wonder if that's left over cruft from some interim repl work. | 16:56 |
mmarin | I wish I were a developer to understand all this code... so can you actually split a replication? | 16:56 |
smcginnis | mmarin: That appears to be a vendor specific thing. | 16:57 |
*** tongli has joined #openstack-cinder | 16:57 | |
smcginnis | mmarin: So if you're going to translate that, think the action of splitting a replication failed. | 16:58 |
smcginnis | My best guess at least. | 16:58 |
jgriffith | smcginnis: mmarin diablo_rojo that's an internal huawei method | 16:58 |
jgriffith | and log statement | 16:58 |
jungleboyj | Looking at the code it looks like they are trying to separate the replication relationship before failing over to the replica. | 16:59 |
jgriffith | it's defined by their internal AbsReplicaOp parent class | 16:59 |
*** vincent_hou has quit IRC | 16:59 | |
*** haomaiwang has quit IRC | 17:01 | |
*** lpetrut has joined #openstack-cinder | 17:01 | |
*** krtaylor has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
*** tongli has quit IRC | 17:01 | |
*** pots has joined #openstack-cinder | 17:02 | |
mmarin | smcginnis: thank you, without further context it's probably the best guess | 17:02 |
*** edtubill has quit IRC | 17:02 | |
*** darosale has quit IRC | 17:04 | |
*** dims_ has quit IRC | 17:05 | |
*** dims has joined #openstack-cinder | 17:06 | |
*** rhagarty has quit IRC | 17:07 | |
*** rhagarty has joined #openstack-cinder | 17:08 | |
*** gouthamr has joined #openstack-cinder | 17:10 | |
*** bjornar has joined #openstack-cinder | 17:12 | |
*** porrua has joined #openstack-cinder | 17:12 | |
*** krtaylor has joined #openstack-cinder | 17:13 | |
*** martyturner has joined #openstack-cinder | 17:13 | |
mmarin | Hi, in the following string: "Set pair secondary access error." pair means two systems? | 17:14 |
jungleboyj | mmarin: I think that would make sense. | 17:15 |
mmarin | ok thanks | 17:16 |
*** diablo_rojo has quit IRC | 17:17 | |
*** krtaylor has quit IRC | 17:17 | |
*** jungleboyj has quit IRC | 17:21 | |
*** mylu has joined #openstack-cinder | 17:22 | |
*** krtaylor has joined #openstack-cinder | 17:24 | |
*** vincent_hou has joined #openstack-cinder | 17:24 | |
*** klkumar has quit IRC | 17:27 | |
*** baumann has quit IRC | 17:29 | |
*** rcernin has quit IRC | 17:29 | |
*** doffm has left #openstack-cinder | 17:34 | |
*** jdurgin has quit IRC | 17:42 | |
*** jdurgin has joined #openstack-cinder | 17:43 | |
*** lpetrut has quit IRC | 17:47 | |
*** lpetrut has joined #openstack-cinder | 17:47 | |
*** salv-orlando has joined #openstack-cinder | 17:48 | |
*** jdurgin has quit IRC | 17:50 | |
*** vincent_hou has quit IRC | 17:50 | |
*** mriedem1 has joined #openstack-cinder | 17:51 | |
*** lpetrut has quit IRC | 17:52 | |
*** mriedem has quit IRC | 17:52 | |
*** tongli has joined #openstack-cinder | 17:54 | |
*** daneyon has joined #openstack-cinder | 17:54 | |
*** tongli has quit IRC | 17:55 | |
*** tongli has joined #openstack-cinder | 17:55 | |
*** jdurgin has joined #openstack-cinder | 17:57 | |
*** mriedem has joined #openstack-cinder | 18:00 | |
*** mriedem1 has quit IRC | 18:00 | |
*** jordanP has quit IRC | 18:00 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwang has joined #openstack-cinder | 18:01 | |
*** salv-orl_ has joined #openstack-cinder | 18:01 | |
*** salv-orlando has quit IRC | 18:04 | |
*** panda has quit IRC | 18:04 | |
*** panda has joined #openstack-cinder | 18:05 | |
*** lixiaoy1 has quit IRC | 18:13 | |
openstackgerrit | Nate Potter proposed openstack/cinder: Added more options while uploading volume as image https://review.openstack.org/230617 | 18:14 |
*** krotscheck has quit IRC | 18:14 | |
*** krotscheck has joined #openstack-cinder | 18:15 | |
*** ericksonsantos has quit IRC | 18:16 | |
*** bardia has quit IRC | 18:20 | |
*** mvk_ has joined #openstack-cinder | 18:22 | |
*** esker has joined #openstack-cinder | 18:23 | |
*** mvk has quit IRC | 18:25 | |
dulek | jgriffith: Can I bother you more about this volume.host problem we were discussing previously. | 18:26 |
jgriffith | dulek: for sure | 18:26 |
*** ericksonsantos has joined #openstack-cinder | 18:26 | |
dulek | jgriffith: I now know why it's important to clean up host field. | 18:26 |
*** krotscheck has quit IRC | 18:26 | |
*** raildo is now known as raildo-afk | 18:26 | |
*** raildo-afk is now known as raildo | 18:26 | |
*** krotscheck has joined #openstack-cinder | 18:26 | |
dulek | jgriffith: It's because of capacity tracking - in case of rescheduling we're not increasing it and we want not to decrease it when deleting. | 18:27 |
jgriffith | dulek: meh... yeah, I can see that | 18:27 |
jgriffith | dulek: but while were' "in" a request I think it's ok to error on the side of caution and assume we've allocated/used the storage | 18:28 |
dulek | jgriffith: I was thinking about that and it seems like a broken model - we cannot do both things accurately. | 18:28 |
jgriffith | dulek: even worse in this case we do in fact allocate it (3 times) | 18:28 |
*** arch-nemesis has quit IRC | 18:28 | |
jgriffith | dulek: not sure I follow what you are thinking specifically there | 18:28 |
dulek | jgriffith: https://bugs.launchpad.net/cinder/+bug/1408763 | 18:29 |
openstack | Launchpad bug 1408763 in Cinder "allocated_capacity tracking is broken in manager" [Medium,Fix released] - Assigned to Michal Dulko (michal-dulko-f) | 18:29 |
jgriffith | dulek: ha! Are you trying to make my head explode :) | 18:30 |
*** xinli has joined #openstack-cinder | 18:30 | |
*** cknight has quit IRC | 18:30 | |
jgriffith | dulek: so yes, you agree that our whole way of doing this is "flawed and broken"? | 18:30 |
openstackgerrit | Scott DAngelo proposed openstack/python-cinderclient: Add docs for running tests https://review.openstack.org/296000 | 18:31 |
dulek | jgriffith: You've wrote that more than a year ago. ;) | 18:31 |
jgriffith | dulek: yes! That's why I said my head might explode | 18:31 |
jgriffith | dulek: I actually still remember clearly the day I was working on that | 18:32 |
jgriffith | dulek: it was VERY frustrating for me to try and make things work correctly in our model | 18:32 |
dulek | I guess you were surprised your head survived that? ;) | 18:32 |
jgriffith | dulek: and as you see I punted and gave up, because my best proposal was to rewrite the whole mess | 18:32 |
jgriffith | dulek: LOL.. yes! | 18:33 |
dulek | jgriffith: So let me sleep with that, I'll try to get all of my thoughts on that in a neat bugreport tomorrow in the morning. | 18:33 |
jgriffith | dulek: fair enough | 18:34 |
dulek | jgriffith: And maybe the problem is worthy of a discussion at the working session in Austin. | 18:34 |
jgriffith | dulek: well... perhaps (been there done that) | 18:34 |
jgriffith | dulek: my concern however is I'll need to push some sort of fix for Mitaka | 18:34 |
dulek | jgriffith: Anyway it doesn't look like fixable for Mitaka to me. :( | 18:34 |
jgriffith | dulek: the orphaned resources is a big problem IMO | 18:34 |
dulek | I cannot disagree. | 18:35 |
*** mmarin has quit IRC | 18:35 | |
jgriffith | dulek: well, I agree... I don't think a "full" fix | 18:35 |
*** arch-nemesis has joined #openstack-cinder | 18:35 | |
jgriffith | dulek: but I might have an idea to band-aid the orphan problem | 18:35 |
jgriffith | dulek: I'll sync up with you in the morning (my morning) :) | 18:35 |
dulek | jgriffith: You can ping me if you come up with a partial fix, but I really need to get off work for today (7:30 PM here). | 18:36 |
dulek | jgriffith: Thanks! :) | 18:36 |
jgriffith | dulek: I understand completely | 18:36 |
jgriffith | dulek: besides, I probably need to go get the tractor fired up and start clearing some snow drifts before I'm trapped inside until summer :) | 18:36 |
jgriffith | dulek: or heck... maybe I'll just hold up until everything melts | 18:37 |
*** cknight has joined #openstack-cinder | 18:37 | |
* dulek haven't seen snow for a month now. ;) | 18:37 | |
*** xinli has quit IRC | 18:38 | |
*** krotscheck has quit IRC | 18:38 | |
*** xinli has joined #openstack-cinder | 18:39 | |
*** krotscheck has joined #openstack-cinder | 18:39 | |
*** permalac has quit IRC | 18:39 | |
*** xinli has quit IRC | 18:39 | |
openstackgerrit | Szymon Wróblewski proposed openstack/cinder: Start/Stop coordinator with Services https://review.openstack.org/263313 | 18:39 |
jgriffith | dulek: well here ya go then https://goo.gl/RLo9J4 | 18:40 |
*** bardia has joined #openstack-cinder | 18:45 | |
*** angela-s has joined #openstack-cinder | 18:45 | |
*** xinli has joined #openstack-cinder | 18:45 | |
*** sgotliv has joined #openstack-cinder | 18:49 | |
*** diablo_rojo has joined #openstack-cinder | 18:49 | |
*** jungleboyj has joined #openstack-cinder | 18:49 | |
*** openstack has joined #openstack-cinder | 19:23 | |
*** zhiyan_ is now known as zhiyan | 19:23 | |
*** serverascode_ is now known as serverascode | 19:23 | |
*** DuncanT_ is now known as DuncanT | 19:24 | |
*** xinli has joined #openstack-cinder | 19:24 | |
*** martyturner has quit IRC | 19:24 | |
*** rlrossit has joined #openstack-cinder | 19:26 | |
*** timcl1 has joined #openstack-cinder | 19:26 | |
*** mylu has quit IRC | 19:27 | |
*** timcl has quit IRC | 19:27 | |
*** alaski_ is now known as alaski | 19:28 | |
*** martyturner has joined #openstack-cinder | 19:30 | |
*** dustins_ has joined #openstack-cinder | 19:34 | |
*** dustins has quit IRC | 19:36 | |
*** mylu has joined #openstack-cinder | 19:37 | |
*** leeantho has quit IRC | 19:39 | |
*** leeantho_ has joined #openstack-cinder | 19:39 | |
*** baumann has joined #openstack-cinder | 19:40 | |
*** simondodsley has joined #openstack-cinder | 19:43 | |
*** dims_ has joined #openstack-cinder | 19:43 | |
*** daneyon has quit IRC | 19:44 | |
*** daneyon has joined #openstack-cinder | 19:45 | |
*** dims has quit IRC | 19:45 | |
*** ChubYann has joined #openstack-cinder | 19:45 | |
*** baumann has quit IRC | 19:46 | |
*** sheel has quit IRC | 19:47 | |
*** daneyon_ has joined #openstack-cinder | 19:47 | |
*** daneyon has quit IRC | 19:50 | |
*** baumann has joined #openstack-cinder | 19:51 | |
*** xiaohui has quit IRC | 19:51 | |
*** kprabhu is now known as karthikp | 19:51 | |
*** karthikp has quit IRC | 19:52 | |
*** kprabhu has joined #openstack-cinder | 19:52 | |
*** dims_ has quit IRC | 19:53 | |
*** karthikp has joined #openstack-cinder | 19:54 | |
*** kprabhu has quit IRC | 19:54 | |
*** Yogi1 has quit IRC | 19:55 | |
*** dims has joined #openstack-cinder | 19:56 | |
*** jgregor has joined #openstack-cinder | 19:56 | |
*** karthikp has quit IRC | 19:59 | |
*** karthikp has joined #openstack-cinder | 19:59 | |
*** haomaiw__ has quit IRC | 20:01 | |
*** haomaiwang has joined #openstack-cinder | 20:01 | |
*** xiaohui has joined #openstack-cinder | 20:04 | |
*** ociuhandu has quit IRC | 20:05 | |
diablo_rojo | What protocol(s) does the Disco driver use? | 20:07 |
Swanson | Duck? | 20:08 |
jungleboyj | diablo_rojo: The 'Ball' protocol /. | 20:08 |
smcginnis | Travolta? | 20:08 |
angela-s | YMCA | 20:09 |
Swanson | jungleboyj, That's much better. | 20:09 |
smcginnis | :) | 20:09 |
jungleboyj | Swanson: I try. | 20:09 |
angela-s | LOL, so glad this team has a sense of humor | 20:10 |
smcginnis | angela-s: ;) | 20:10 |
Swanson | Excellent, Spotify has the Bee Gees! | 20:10 |
diablo_rojo | Swanson: You should have come to hang out with us. We are at the same coffee shop we were at earlier. | 20:11 |
smcginnis | diablo_rojo: I made sure not to tell him which one. | 20:11 |
*** lpetrut has joined #openstack-cinder | 20:12 | |
diablo_rojo | smcginnis: Itd be worse if we were at like starbucks or caribou. There are hundreds of those. There are only so many of these coffee shops. | 20:12 |
*** BigWillie has quit IRC | 20:13 | |
*** timcl1 has quit IRC | 20:14 | |
Swanson | snowed in? | 20:14 |
Swanson | And has anyone heard from jgriffith since he said he need to go out and plow? | 20:15 |
*** rajinir has quit IRC | 20:15 | |
*** xinli has quit IRC | 20:17 | |
*** e0ne has joined #openstack-cinder | 20:17 | |
*** openstack has joined #openstack-cinder | 20:33 | |
*** cknight has joined #openstack-cinder | 20:34 | |
*** gouthamr has quit IRC | 20:35 | |
*** e0ne has quit IRC | 20:37 | |
*** martyturner has joined #openstack-cinder | 20:39 | |
*** haplo37 has quit IRC | 20:41 | |
*** sdague has quit IRC | 20:44 | |
*** mdenny has quit IRC | 20:44 | |
*** mylu_ has quit IRC | 20:44 | |
*** cknight has quit IRC | 20:45 | |
*** mylu has joined #openstack-cinder | 20:46 | |
*** BigWillie has quit IRC | 20:48 | |
*** porrua has quit IRC | 20:56 | |
*** tongli has quit IRC | 20:58 | |
*** david-lyle has quit IRC | 20:59 | |
*** haomaiwang has quit IRC | 21:01 | |
*** haomaiwang has joined #openstack-cinder | 21:01 | |
*** Lee1092 has quit IRC | 21:02 | |
*** david-lyle has joined #openstack-cinder | 21:05 | |
*** mylu has quit IRC | 21:09 | |
*** alkhodos has joined #openstack-cinder | 21:11 | |
*** mylu has joined #openstack-cinder | 21:12 | |
*** mylu has quit IRC | 21:15 | |
*** tpsilva has quit IRC | 21:17 | |
*** david-lyle has quit IRC | 21:21 | |
alkhodos | Hi guys, there was a recent patch for Nexenta driver (unfortunately I didn't notice the bug myself) https://review.openstack.org/#/c/295304/ and yesterday our CI started to fail with this error http://paste.openstack.org/show/491639/. Any idea why volume object does not have 'size' key ? | 21:21 |
*** edmondsw has quit IRC | 21:22 | |
*** edmondsw has joined #openstack-cinder | 21:22 | |
*** edmondsw has quit IRC | 21:22 | |
*** IlyaG has quit IRC | 21:23 | |
smcginnis | alkhodos: That actually looks like a new one. | 21:23 |
smcginnis | alkhodos: There was the issue that cloning a volume could be a larger size. | 21:23 |
*** david-lyle has joined #openstack-cinder | 21:23 | |
smcginnis | alkhodos: That was that patch referred, and your CI appears to be happy with that one. | 21:24 |
smcginnis | alkhodos: There's a second new one that's similar but different. | 21:24 |
alkhodos | smcginnis: yes, I understand the reasoning for the patch, but that's the only place where the driver call volume['size'] | 21:24 |
smcginnis | alkhodos: That's for creating a volume from snapshot. | 21:24 |
smcginnis | The paste appears to be related to that. | 21:24 |
smcginnis | There is a check that volume has size, but not that snapshot has size. | 21:24 |
smcginnis | alkhodos: Is the failing patch on a fix for that one? | 21:25 |
*** ociuhandu has joined #openstack-cinder | 21:25 | |
alkhodos | smcginnis: from some point it fails all the patches, so looks like some merged patch is the reason | 21:26 |
smcginnis | alkhodos: Odd, I didn't think any of those snapshot ones went through yet. | 21:27 |
smcginnis | Looking... | 21:27 |
smcginnis | alkhodos: You are right, that snapshot['size'] check was from the clone size fix. | 21:29 |
alkhodos | smcginnis: I should probably add a 'size' in snapshot check ? | 21:30 |
smcginnis | Not sure why snapshot doesn't have size. | 21:30 |
smcginnis | It's possible it is only passed in if a new size is specified. | 21:30 |
smcginnis | alkhodos: If that's the case, it could be safe to change it from snapshot['size'] to snapshot.get('size', 0) | 21:31 |
smcginnis | In fact, line 326 could probably just be changed to "if volume.get('size', 0) > snapshot.get('size', 0)" | 21:32 |
alkhodos | smcginnis: but in that case if volume has size and snapshot doesn't, we will always call extend | 21:32 |
smcginnis | alkhodos: Oh, right. :] | 21:33 |
alkhodos | smcginnis: doesn't look like correct workflow :) | 21:34 |
*** charlesrose has quit IRC | 21:34 | |
*** IlyaG has joined #openstack-cinder | 21:37 | |
*** jgregor has quit IRC | 21:38 | |
*** bluex has quit IRC | 21:38 | |
*** jgregor has joined #openstack-cinder | 21:41 | |
alkhodos | smcginnis: shouldn't we use snapshot['volume']['size'] there ? | 21:42 |
smcginnis | alkhodos: Quite possible. Honestly, I haven't looked in to that too much other than to notice that the majority of drivers do not comply to new volume sizes on the clone. | 21:47 |
*** adrianofr_ has quit IRC | 21:50 | |
*** rlrossit has quit IRC | 21:51 | |
*** mriedem has quit IRC | 21:53 | |
*** IlyaG has quit IRC | 21:58 | |
*** haomaiwang has quit IRC | 22:01 | |
*** jgregor has quit IRC | 22:01 | |
openstackgerrit | Jay Bryant proposed openstack/cinder: Fix misspelled word in help text https://review.openstack.org/296815 | 22:01 |
*** haomaiwang has joined #openstack-cinder | 22:02 | |
*** knikolla has quit IRC | 22:04 | |
*** panda has quit IRC | 22:04 | |
*** bluex has joined #openstack-cinder | 22:05 | |
*** panda has joined #openstack-cinder | 22:05 | |
jgriffith | Swanson: I live!! | 22:06 |
Swanson | jgriffith, not trapped under your tractor? | 22:08 |
jgriffith | Swanson: nope, not this time! | 22:08 |
jgriffith | Almost rolled it over once.. but that's table stakes | 22:08 |
Swanson | nice! | 22:08 |
*** mylu has joined #openstack-cinder | 22:09 | |
*** alyson_ has quit IRC | 22:12 | |
*** ametts has quit IRC | 22:14 | |
*** diablo_rojo has quit IRC | 22:17 | |
*** martyturner has quit IRC | 22:19 | |
*** gouthamr has joined #openstack-cinder | 22:19 | |
*** jungleboyj has quit IRC | 22:21 | |
*** bluex has quit IRC | 22:24 | |
*** arch-nemesis has quit IRC | 22:26 | |
*** knikolla has joined #openstack-cinder | 22:40 | |
*** liuke has quit IRC | 22:40 | |
*** sheel has joined #openstack-cinder | 22:40 | |
*** garthb has joined #openstack-cinder | 22:43 | |
*** angela-s has quit IRC | 22:50 | |
*** karthikp has quit IRC | 22:53 | |
*** IlyaG has joined #openstack-cinder | 22:59 | |
*** haomaiwang has quit IRC | 23:01 | |
*** mriedem has joined #openstack-cinder | 23:01 | |
*** haomaiwa_ has joined #openstack-cinder | 23:01 | |
*** IlyaG has quit IRC | 23:04 | |
*** alaski has quit IRC | 23:05 | |
*** mylu has quit IRC | 23:06 | |
*** mylu has joined #openstack-cinder | 23:07 | |
*** gouthamr has quit IRC | 23:10 | |
*** alaski has joined #openstack-cinder | 23:11 | |
*** BigWillie has joined #openstack-cinder | 23:12 | |
*** BigWillie has quit IRC | 23:16 | |
*** erlon has quit IRC | 23:17 | |
*** openstack has joined #openstack-cinder | 23:24 | |
*** sgotliv has quit IRC | 23:34 | |
*** yangyapeng has quit IRC | 23:34 | |
*** gouthamr has joined #openstack-cinder | 23:37 | |
*** dustins_ has quit IRC | 23:40 | |
*** smoriya_ has joined #openstack-cinder | 23:52 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!