jgriffith | mucking around with the DB should not be taken lightly :) | 00:01 |
---|---|---|
*** IlyaG has joined #openstack-cinder | 00:02 | |
lifeless | mriedem: no | 00:02 |
lifeless | mriedem: stack_info says 'always provide a call stack' | 00:02 |
lifeless | mriedem: the original code only provides a call stack when an exception is present | 00:02 |
*** pckizer_ is now known as pckizer | 00:03 | |
*** opal has joined #openstack-cinder | 00:04 | |
*** eharney has quit IRC | 00:05 | |
*** lixiaoy1 has joined #openstack-cinder | 00:06 | |
jgriffith | hmm.. dims_ are we going to need to change our approach and combine those patches as well like you did on Nova: https://review.openstack.org/#/c/233772/ | 00:07 |
mriedem | lifeless: ah ok | 00:07 |
dims_ | jgriffith: mriedem was patching something up | 00:09 |
dims_ | i am working another angle now | 00:09 |
mriedem | jgriffith: cinder already has a fix in tree for the webob 1.5.0 issue | 00:09 |
mriedem | so shouldn't be the same as the nova issue | 00:09 |
mriedem | jgriffith: https://github.com/openstack/cinder/commit/867fccf833ffc597aa986cb6ff1b3b5c1101b9ba | 00:10 |
jgriffith | yeah, I saw it just merged | 00:11 |
jgriffith | sorry, my bad | 00:11 |
jgriffith | mriedem: not just either | 00:11 |
*** salv-orl_ has quit IRC | 00:11 | |
jgriffith | quite a while ago.... I was looking at the backports, for which the webob has not merged yet | 00:11 |
jgriffith | mriedem: I vote we abolish stable branches and just have master from now on :) | 00:12 |
jgriffith | is gerrit painfully slow for others today? | 00:13 |
*** EinstCrazy has quit IRC | 00:14 | |
*** lixiaoy11 has joined #openstack-cinder | 00:17 | |
*** opal has left #openstack-cinder | 00:18 | |
*** lixiaoy1 has quit IRC | 00:19 | |
*** stevemar_ has quit IRC | 00:25 | |
*** tpeoples has quit IRC | 00:27 | |
*** DericHorn-HP has joined #openstack-cinder | 00:28 | |
*** stevemar_ has joined #openstack-cinder | 00:29 | |
*** garthb has quit IRC | 00:29 | |
ntpttr | jgriffith: So is there something that I should change with the requirements patch? At least it'll need another recheck, it looks like its hit a failure in python34 with Jenkins... https://jenkins07.openstack.org/job/gate-cinder-python34/1421/console | 00:29 |
*** annasort has quit IRC | 00:42 | |
*** apoorvad has quit IRC | 00:44 | |
jgriffith | ntpttr: nah, I'm just waiting for it to finish and hit recheck again | 00:48 |
jgriffith | ntpttr: the py34 failure is an intermittent one that eharney spotted a while back :( | 00:48 |
jgriffith | ntpttr: just sucks the cycle time is 1.5 hours :( | 00:49 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/232868 | 00:49 |
*** tobe has joined #openstack-cinder | 00:49 | |
jgriffith | haha! | 00:49 |
jgriffith | Go figure | 00:49 |
jgriffith | ^^ that proposal will fix things up if it passes :) | 00:50 |
jgriffith | ntpttr: It's officially a race :) | 00:51 |
ntpttr | jgriffith: hah I'll go get my popcorn | 00:52 |
jgriffith | hehe | 00:52 |
ntpttr | dang bot taking my first patch that was going to get merged to cinder :P | 00:56 |
*** kevincarr1991 has joined #openstack-cinder | 00:56 | |
*** tpeoples has joined #openstack-cinder | 00:56 | |
jgriffith | ntpttr: who would've thought it would take over 8 hours to move something :( | 00:56 |
*** EinstCrazy has joined #openstack-cinder | 00:59 | |
ntpttr | jgriffith: ah well, next time | 00:59 |
*** leeantho has quit IRC | 01:06 | |
*** DericHorn-HP has quit IRC | 01:08 | |
*** ntpttr has left #openstack-cinder | 01:12 | |
thingee | smcginnis: ping | 01:18 |
*** IlyaG has quit IRC | 01:18 | |
*** Lee1092 has joined #openstack-cinder | 01:29 | |
*** zhenguo has joined #openstack-cinder | 01:33 | |
*** mtanino has quit IRC | 01:34 | |
*** DericHorn-HP has joined #openstack-cinder | 01:36 | |
*** sghanekar_ has quit IRC | 01:40 | |
*** p0rtal has quit IRC | 01:41 | |
*** yangyapeng has joined #openstack-cinder | 01:46 | |
*** davechen has joined #openstack-cinder | 01:46 | |
*** stevemar_ has quit IRC | 01:53 | |
*** IlyaG has joined #openstack-cinder | 01:53 | |
*** haomaiwang has joined #openstack-cinder | 01:55 | |
*** haomaiwang has quit IRC | 02:01 | |
*** haomaiwang has joined #openstack-cinder | 02:01 | |
*** kevincarr1991 has quit IRC | 02:01 | |
*** lprice has joined #openstack-cinder | 02:01 | |
*** martyturner has joined #openstack-cinder | 02:02 | |
*** lprice1 has quit IRC | 02:04 | |
*** DericHorn-HP has quit IRC | 02:05 | |
*** lprice1 has joined #openstack-cinder | 02:05 | |
*** martyturner has quit IRC | 02:06 | |
*** lprice has quit IRC | 02:09 | |
*** DericHorn-HP has joined #openstack-cinder | 02:10 | |
*** jdandrea has quit IRC | 02:13 | |
*** ntpttr has joined #openstack-cinder | 02:13 | |
openstackgerrit | wanghao proposed openstack/cinder: CG creation should be schedule on backend level https://review.openstack.org/224405 | 02:20 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei driver handle volume exists error https://review.openstack.org/233333 | 02:26 |
*** chenying has joined #openstack-cinder | 02:27 | |
*** links has joined #openstack-cinder | 02:28 | |
*** dims_ has quit IRC | 02:28 | |
*** DericHorn-HP has quit IRC | 02:34 | |
*** salv-orlando has joined #openstack-cinder | 02:39 | |
*** DericHorn-HP has joined #openstack-cinder | 02:39 | |
*** mriedem has quit IRC | 02:39 | |
*** dims__ has joined #openstack-cinder | 02:41 | |
*** stevemar_ has joined #openstack-cinder | 02:42 | |
*** p0rtal has joined #openstack-cinder | 02:46 | |
*** chenying has quit IRC | 02:46 | |
*** salv-orlando has quit IRC | 02:47 | |
*** bill_az has quit IRC | 02:56 | |
*** yangyapeng has quit IRC | 02:57 | |
*** dims__ has quit IRC | 02:59 | |
*** haomaiwang has quit IRC | 03:01 | |
*** haomaiwang has joined #openstack-cinder | 03:01 | |
*** salv-orlando has joined #openstack-cinder | 03:08 | |
*** ntpttr has left #openstack-cinder | 03:09 | |
*** dave-mccowan has quit IRC | 03:10 | |
*** chenying has joined #openstack-cinder | 03:15 | |
*** salv-orlando has quit IRC | 03:17 | |
openstackgerrit | xing-yang proposed openstack/cinder-specs: Backup snapshots https://review.openstack.org/232305 | 03:20 |
*** mdbooth has quit IRC | 03:22 | |
*** lpabon has joined #openstack-cinder | 03:24 | |
*** mdbooth has joined #openstack-cinder | 03:28 | |
*** zhangjn has quit IRC | 03:32 | |
*** p0rtal_ has joined #openstack-cinder | 03:39 | |
openstackgerrit | WenPing Zhang proposed openstack/cinder: New cinder driver to support Daowoo YeeSan https://review.openstack.org/202440 | 03:40 |
*** haomaiwang has quit IRC | 03:41 | |
*** p0rtal has quit IRC | 03:42 | |
*** lpabon has quit IRC | 03:44 | |
*** haomaiwang has joined #openstack-cinder | 03:45 | |
*** gouthamr_ has quit IRC | 03:46 | |
openstackgerrit | John Griffith proposed openstack/cinder: Change default Exception code to 500 https://review.openstack.org/233923 | 03:46 |
jgriffith | smcginnis: gimmie a shout in the AM, need to check in with you regarding these changes and Liberty | 03:47 |
jgriffith | smcginnis: but for now, I'm going to have a beer :) | 03:47 |
*** jwcroppe has joined #openstack-cinder | 03:49 | |
*** 32NAAJR1K has quit IRC | 03:54 | |
*** links has quit IRC | 03:58 | |
*** dims__ has joined #openstack-cinder | 03:59 | |
*** harlowja_at_home has joined #openstack-cinder | 04:00 | |
*** p0rtal_ has quit IRC | 04:00 | |
*** haomaiwang has quit IRC | 04:01 | |
*** 20WAB9N4Y has joined #openstack-cinder | 04:01 | |
*** zhangjn has joined #openstack-cinder | 04:03 | |
*** dims__ has quit IRC | 04:06 | |
openstackgerrit | John Griffith proposed openstack/cinder: Update config format for replication_devices https://review.openstack.org/233307 | 04:09 |
*** davechen is now known as davechen_afk | 04:09 | |
*** vgridnev has joined #openstack-cinder | 04:13 | |
*** salv-orlando has joined #openstack-cinder | 04:15 | |
*** opal has joined #openstack-cinder | 04:19 | |
*** salv-orlando has quit IRC | 04:19 | |
*** chenying has quit IRC | 04:26 | |
*** lixiaoy11 has quit IRC | 04:28 | |
*** dobson has quit IRC | 04:30 | |
*** opal has left #openstack-cinder | 04:31 | |
*** pradipta has joined #openstack-cinder | 04:33 | |
*** salv-orlando has joined #openstack-cinder | 04:36 | |
*** dobson has joined #openstack-cinder | 04:36 | |
*** jgregor has quit IRC | 04:40 | |
*** jgregor has joined #openstack-cinder | 04:40 | |
*** salv-orlando has quit IRC | 04:40 | |
*** haigang has joined #openstack-cinder | 04:41 | |
*** dosaboy has quit IRC | 04:42 | |
*** skraynev has quit IRC | 04:42 | |
*** haigang has quit IRC | 04:42 | |
*** dosaboy has joined #openstack-cinder | 04:43 | |
*** skraynev has joined #openstack-cinder | 04:43 | |
*** harlowja_at_home has quit IRC | 04:45 | |
*** avishay has joined #openstack-cinder | 04:47 | |
*** jgregor has quit IRC | 04:52 | |
*** 20WAB9N4Y has quit IRC | 05:01 | |
*** haomaiwang has joined #openstack-cinder | 05:01 | |
*** avishay has quit IRC | 05:04 | |
*** avishay_ has joined #openstack-cinder | 05:04 | |
*** links has joined #openstack-cinder | 05:06 | |
*** avishay_ is now known as avishay | 05:12 | |
*** vgridnev has quit IRC | 05:12 | |
*** lixiaoy1 has joined #openstack-cinder | 05:12 | |
*** breitz has quit IRC | 05:13 | |
*** breitz has joined #openstack-cinder | 05:14 | |
*** DericHorn-HP has quit IRC | 05:17 | |
*** lixiaoy11 has joined #openstack-cinder | 05:17 | |
*** lixiaoy1 has quit IRC | 05:20 | |
*** sgotliv_ has joined #openstack-cinder | 05:22 | |
*** davechen_afk is now known as davechen | 05:23 | |
*** sgotliv has quit IRC | 05:24 | |
*** p0rtal has joined #openstack-cinder | 05:27 | |
*** vgridnev has joined #openstack-cinder | 05:27 | |
*** breitz has quit IRC | 05:35 | |
*** breitz has joined #openstack-cinder | 05:36 | |
*** tobe has quit IRC | 05:39 | |
*** daneyon has joined #openstack-cinder | 05:40 | |
*** daneyon has quit IRC | 05:41 | |
openstackgerrit | LisaLi proposed openstack/cinder: Create encrypted volumes from images https://review.openstack.org/216567 | 05:41 |
*** flaper87 has joined #openstack-cinder | 05:44 | |
*** flaper87 has left #openstack-cinder | 05:44 | |
*** nkrinner has joined #openstack-cinder | 05:46 | |
*** haomaiwang has quit IRC | 06:01 | |
*** IlyaG has quit IRC | 06:01 | |
*** haomaiwang has joined #openstack-cinder | 06:01 | |
*** stevemar_ has quit IRC | 06:03 | |
*** dims__ has joined #openstack-cinder | 06:03 | |
*** p0rtal has quit IRC | 06:04 | |
*** dims__ has quit IRC | 06:09 | |
*** haomaiwang has quit IRC | 06:10 | |
*** sgotliv_ has quit IRC | 06:12 | |
*** haomaiwang has joined #openstack-cinder | 06:13 | |
*** vgridnev has quit IRC | 06:15 | |
*** ronis has joined #openstack-cinder | 06:17 | |
*** links has quit IRC | 06:20 | |
*** openstack has joined #openstack-cinder | 06:28 | |
*** vlaza has joined #openstack-cinder | 06:28 | |
*** tsufiev has joined #openstack-cinder | 06:29 | |
*** yrabl has joined #openstack-cinder | 06:30 | |
*** daneyon has joined #openstack-cinder | 06:34 | |
openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Case sensitivity problem in cinder scheduler https://review.openstack.org/225024 | 06:36 |
*** lixiaoy11 has quit IRC | 06:37 | |
*** ankit_ag has joined #openstack-cinder | 06:37 | |
*** lixiaoy1 has joined #openstack-cinder | 06:38 | |
*** daneyon has quit IRC | 06:39 | |
*** jwcroppe has quit IRC | 06:44 | |
*** jwcroppe has joined #openstack-cinder | 06:44 | |
*** wanghao has joined #openstack-cinder | 06:45 | |
*** jwcroppe has quit IRC | 06:45 | |
*** haomaiwang has quit IRC | 06:47 | |
*** 18VAAUGO0 has joined #openstack-cinder | 06:48 | |
*** sgotliv_ has quit IRC | 06:52 | |
openstackgerrit | Alon Marx proposed openstack/cinder: Implement changes in the consistency_group handling in the xiv/ds8k driver. https://review.openstack.org/233280 | 06:54 |
*** vgridnev has quit IRC | 06:55 | |
*** e0ne has joined #openstack-cinder | 06:58 | |
*** 18VAAUGO0 has quit IRC | 06:58 | |
*** e0ne has quit IRC | 06:59 | |
*** chenying has joined #openstack-cinder | 07:02 | |
*** haomaiwang has joined #openstack-cinder | 07:02 | |
openstackgerrit | Yusuke Hayashi proposed openstack/cinder: Case sensitivity problem in cinder scheduler https://review.openstack.org/225024 | 07:03 |
*** dims__ has joined #openstack-cinder | 07:05 | |
*** tpeoples has quit IRC | 07:08 | |
*** vincent_hou has joined #openstack-cinder | 07:09 | |
*** dims__ has quit IRC | 07:10 | |
*** links has joined #openstack-cinder | 07:13 | |
*** ndipanov has quit IRC | 07:13 | |
*** e0ne has joined #openstack-cinder | 07:16 | |
openstackgerrit | dongwenjuan proposed openstack/python-cinderclient: Support forcing service down https://review.openstack.org/234001 | 07:17 |
*** p0rtal has joined #openstack-cinder | 07:18 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Brocade driver add_zone optimization https://review.openstack.org/234004 | 07:20 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API types extra specs to Python 3 https://review.openstack.org/232216 | 07:21 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port WSGI tests to Python 3 https://review.openstack.org/232214 | 07:21 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API to Python 3 https://review.openstack.org/232215 | 07:21 |
openstackgerrit | Victor Stinner proposed openstack/cinder: Port API admin action tests to Python 3 https://review.openstack.org/232242 | 07:21 |
*** haypo has joined #openstack-cinder | 07:23 | |
*** lpetrut has joined #openstack-cinder | 07:27 | |
*** p0rtal has quit IRC | 07:28 | |
*** anshul has joined #openstack-cinder | 07:28 | |
*** daneyon has joined #openstack-cinder | 07:29 | |
*** xek_ is now known as xek | 07:29 | |
vincent_hou | avishay: hi. | 07:29 |
*** anshul has quit IRC | 07:32 | |
*** anshul has joined #openstack-cinder | 07:33 | |
*** daneyon has quit IRC | 07:34 | |
*** vgridnev has joined #openstack-cinder | 07:37 | |
*** ronis has joined #openstack-cinder | 07:41 | |
*** e0ne has quit IRC | 07:44 | |
avishay | vincent_hou: hey, what's up? | 07:47 |
vincent_hou | avishay: How are you? Are you going to Tokyo? | 07:49 |
avishay | vincent_hou: i am well, thanks. yes, i will be at the tokyo summit. | 07:50 |
vincent_hou | avishay: ok . we can meet there. | 07:50 |
*** subscope has joined #openstack-cinder | 07:52 | |
*** salv-orlando has joined #openstack-cinder | 07:59 | |
vincent_hou | geguileo: https://review.openstack.org/#/c/214941/8/doc/source/devref/migration.rst. I have a different opinion about the volume type here. | 07:59 |
vincent_hou | geguileo: When creating a volume, a default volume type will be assigned, if the volume type is not specified. | 07:59 |
geguileo | vincent_hou: Really? r:-?? | 08:00 |
vincent_hou | geguileo: the volume is going to have a volume type anyway, so I think we need to take volume type into account in this doc. | 08:00 |
*** haomaiwang has quit IRC | 08:01 | |
vincent_hou | geguileo: I will point you the code. | 08:01 |
*** haomaiwang has joined #openstack-cinder | 08:01 | |
geguileo | vincent_hou: I mean, I have been creating volumes and migrating them without a type as long as I can remember | 08:01 |
*** markus_z has joined #openstack-cinder | 08:01 | |
*** jordanP has joined #openstack-cinder | 08:02 | |
geguileo | vincent_hou: Which of my comments are we referring to? | 08:02 |
DuncanT | geguileo: What do they end up with in the type field in the DB? | 08:02 |
geguileo | DuncanT: I think they had NULL on the type | 08:03 |
*** salv-orlando has quit IRC | 08:03 | |
*** jistr has joined #openstack-cinder | 08:03 | |
vincent_hou | geguileo: I mean the comment "They don't need to have the same name and we don't need a type to do the migration. It is when you add a volume type to the volume and the volume type has a backend restriction that you need them to have the same backend name." at Line 147. | 08:03 |
vincent_hou | geguileo: Also please check Line 384 def_vol_type = volume_types.get_default_volume_type(), cinder/volume/flows/api/create_volume.py. I think this line gives the volume a type anyway. | 08:04 |
*** dims__ has joined #openstack-cinder | 08:06 | |
vincent_hou | geguileo: DuncanT: I am doubting that we can still have volume with NULL volume type. I need opinions from others. | 08:06 |
DuncanT | vincent_hou: I guess they can still be in the DB from old volume, unless we've at some point fixed it up in a migration | 08:07 |
openstackgerrit | Wilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver https://review.openstack.org/202023 | 08:07 |
DuncanT | vincent_hou: Try not setting a default type in the config | 08:07 |
vincent_hou | DuncanT: OH. That could be an option. | 08:08 |
*** links has quit IRC | 08:08 | |
geguileo | vincent_hou: I'll try testing it again today and come back to you | 08:09 |
vincent_hou | DuncanT: I will verify the case without the default type. Thx. | 08:10 |
vincent_hou | geguileo: Thank you. | 08:10 |
DuncanT | vincent_hou: Might be worth us putting together a db dump and LVM recreation script for volumes made in different eras of cinder, for upgrade testing? | 08:11 |
*** dims__ has quit IRC | 08:11 | |
openstackgerrit | Wilson Liu proposed openstack/cinder: Huawei driver handle volume exists error https://review.openstack.org/233333 | 08:12 |
vincent_hou | DuncanT: Yeah. I think it is good topic to investigate. | 08:13 |
*** sgotliv_ has joined #openstack-cinder | 08:16 | |
*** links has joined #openstack-cinder | 08:18 | |
*** svasheka has joined #openstack-cinder | 08:18 | |
*** ndipanov has joined #openstack-cinder | 08:21 | |
*** daneyon has joined #openstack-cinder | 08:23 | |
*** subscope has quit IRC | 08:23 | |
*** links has quit IRC | 08:24 | |
*** subscope has joined #openstack-cinder | 08:24 | |
*** salv-orlando has joined #openstack-cinder | 08:26 | |
*** daneyon has quit IRC | 08:28 | |
*** links has joined #openstack-cinder | 08:31 | |
*** lixiaoy1 has quit IRC | 08:33 | |
haypo | hi. i had to rebase https://review.openstack.org/#/c/232214/ and it got +2 from 4 different cores. can we stop the unlimited loop by finally accepting this change please? :-p | 08:34 |
haypo | (it's part of a serie of 4 changes) | 08:34 |
*** davechen has quit IRC | 08:34 | |
haypo | i mean it's the patch version 9, i had to rebase it so many times, i don't remember how many :-) | 08:34 |
*** openstack has joined #openstack-cinder | 08:47 | |
vincent_hou | geguileo: I tested volume migration for the volume with NULL volume type and it worked for cinder migration, but not retype. | 08:47 |
vincent_hou | geguileo: I guess I just found another bug. 2015-10-13 16:45:10.332 ERROR cinder.volume.manager [req-1f42a2f3-36d8-4883-85f4-09b11cf5959f None None] Failed to copy volume 84c7eb0 | 08:47 |
vincent_hou | c-c6cd-4d36-b357-c8e07d1be6e2 to 87b45163-a9c8-431e-a653-69c248110012 | 08:47 |
vincent_hou | 2015-10-13 16:45:10.699 ERROR oslo_messaging.rpc.dispatcher [req-1f42a2f3-36d8-4883-85f4-09b11cf5959f None None] Exception during mess | 08:47 |
vincent_hou | age handling: Instance <VolumeTypes at 0x7fdada59cfd0> has a NULL identity key. If this is an auto-generated value, check that the da | 08:47 |
vincent_hou | tabase table allows generation of new primary key values, and that the mapped Column object is configured to expect these generated va | 08:47 |
vincent_hou | lues. Ensure also that this flush() is not occurring at an inappropriate time, such aswithin a load() event. | 08:47 |
vincent_hou | 2015-10-13 16:45:10.699 TRACE oslo_messaging.rpc.dispatcher Traceback (most recent call last): | 08:47 |
vincent_hou | 2015-10-13 16:45:10.699 TRACE oslo_messaging.rpc.dispatcher | 08:47 |
vincent_hou | geguileo: the retype of a volume from null to lvm(for example) fails during IMPL.finish_volume_migration | 08:48 |
geguileo | vincent_hou: I'm on a meeting, give me 10 minutes, please | 08:49 |
vincent_hou | geguileo: Cool. take your time. | 08:50 |
*** mdbooth has joined #openstack-cinder | 08:51 | |
*** BharatK_ has quit IRC | 08:52 | |
*** bluex-pl has quit IRC | 08:54 | |
*** bluex-pl has joined #openstack-cinder | 08:54 | |
*** hideme_ has quit IRC | 08:54 | |
*** hideme_ has joined #openstack-cinder | 08:54 | |
*** links has quit IRC | 08:54 | |
*** links has joined #openstack-cinder | 08:54 | |
*** markus_z has quit IRC | 08:54 | |
*** markus_z has joined #openstack-cinder | 08:54 | |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/222292 | 08:56 |
geguileo | vincent_hou: That is bad :-( | 08:59 |
geguileo | vincent_hou: I'll try to test it as well (although I'm sure I'll get the same results) | 09:00 |
*** wenjuan has joined #openstack-cinder | 09:00 | |
*** wenjuan has quit IRC | 09:00 | |
*** wenjuan has joined #openstack-cinder | 09:00 | |
*** haomaiwang has quit IRC | 09:01 | |
*** wenjuan has quit IRC | 09:01 | |
*** haomaiwang has joined #openstack-cinder | 09:01 | |
*** wenjuan has joined #openstack-cinder | 09:01 | |
wenjuan | https://review.openstack.org/#/c/234001 | 09:02 |
*** wenjuan has quit IRC | 09:02 | |
*** wenjuan has joined #openstack-cinder | 09:02 | |
*** wenjuan has quit IRC | 09:03 | |
*** wenjuan has joined #openstack-cinder | 09:03 | |
*** subscope has quit IRC | 09:03 | |
*** wenjuan has quit IRC | 09:04 | |
*** wenjuan has joined #openstack-cinder | 09:04 | |
*** wenjuan has quit IRC | 09:04 | |
*** wenjuan has joined #openstack-cinder | 09:05 | |
vincent_hou | geguileo: Thank you for your comment about the volume type. It inspired me about this case, otherwise we will let it go right in front of us. | 09:05 |
*** yrabl has quit IRC | 09:05 | |
geguileo | vincent_hou: Thank you for working on that much needed document!! | 09:05 |
*** BharatK_ has joined #openstack-cinder | 09:06 | |
vincent_hou | geguileo: Well, just need your verification of this issue. I logged it here anyway: https://bugs.launchpad.net/cinder/+bug/1505572. | 09:06 |
openstack | Launchpad bug 1505572 in Cinder "Migration: retype of a volume with NULL volume type to a non-null volume type fails during IMPL.finish_volume_migration" [Undecided,New] | 09:06 |
*** wenjuan has quit IRC | 09:07 | |
*** wenjuan has joined #openstack-cinder | 09:07 | |
*** wenjuan has quit IRC | 09:07 | |
*** dims__ has joined #openstack-cinder | 09:08 | |
*** avishay_ has joined #openstack-cinder | 09:08 | |
*** wenjuan has joined #openstack-cinder | 09:08 | |
*** wenjuan has quit IRC | 09:08 | |
*** avishay has quit IRC | 09:09 | |
*** wenjuan has joined #openstack-cinder | 09:09 | |
*** wenjuan has quit IRC | 09:09 | |
*** wenjuan has joined #openstack-cinder | 09:10 | |
*** wenjuan has quit IRC | 09:11 | |
*** wenjuan has joined #openstack-cinder | 09:11 | |
*** wenjuan has quit IRC | 09:11 | |
*** wenjuan has joined #openstack-cinder | 09:12 | |
*** wenjuan has quit IRC | 09:12 | |
*** dims__ has quit IRC | 09:13 | |
*** wenjuan has joined #openstack-cinder | 09:13 | |
*** wenjuan has quit IRC | 09:13 | |
*** wenjuan has joined #openstack-cinder | 09:14 | |
*** wenjuan has quit IRC | 09:14 | |
*** links has quit IRC | 09:15 | |
*** wenjuan has joined #openstack-cinder | 09:15 | |
*** subscope has joined #openstack-cinder | 09:15 | |
*** wenjuan has quit IRC | 09:16 | |
*** wenjuan has joined #openstack-cinder | 09:16 | |
*** wenjuan has quit IRC | 09:17 | |
*** daneyon has joined #openstack-cinder | 09:17 | |
*** wenjuan has joined #openstack-cinder | 09:17 | |
*** wenjuan has quit IRC | 09:17 | |
*** wenjuan has joined #openstack-cinder | 09:18 | |
*** wenjuan has quit IRC | 09:18 | |
*** wenjuan has joined #openstack-cinder | 09:19 | |
*** wenjuan has quit IRC | 09:19 | |
*** wenjuan has joined #openstack-cinder | 09:20 | |
*** wenjuan has quit IRC | 09:21 | |
*** wenjuan has joined #openstack-cinder | 09:21 | |
*** wenjuan has quit IRC | 09:21 | |
*** daneyon has quit IRC | 09:22 | |
*** wenjuan has joined #openstack-cinder | 09:22 | |
*** wenjuan has quit IRC | 09:22 | |
*** wenjuan has joined #openstack-cinder | 09:23 | |
*** wenjuan has quit IRC | 09:24 | |
*** wenjuan has joined #openstack-cinder | 09:24 | |
*** wenjuan has quit IRC | 09:25 | |
*** wenjuan has joined #openstack-cinder | 09:25 | |
vincent_hou | geguileo: When migrating a null volume type to a non-null volume type, the destination volume created has been assigned a volume_type_id in the model, so when we try to set the type to null at Line 1224 setattr(dest_volume_ref, key, value_to_dst), there is a mismatch in the model. This is my understand of this cause of this issue. Well, why cannot we set volume_type_id back to NULL? | 09:25 |
*** wenjuan has quit IRC | 09:26 | |
*** wenjuan has joined #openstack-cinder | 09:26 | |
geguileo | vincent_hou: Don't know, we should be able to r:-?? | 09:27 |
*** wenjuan has joined #openstack-cinder | 09:27 | |
openstackgerrit | dongwenjuan proposed openstack/python-cinderclient: Support forcing service down https://review.openstack.org/234001 | 09:27 |
*** wenjuan has quit IRC | 09:28 | |
geguileo | vincent_hou: Wait... It's volume_type_id the one raising the error or volume_type? | 09:28 |
*** wenjuan has joined #openstack-cinder | 09:28 | |
vincent_hou | geguileo: id, I think. | 09:29 |
vincent_hou | not 100% sure. | 09:29 |
geguileo | vincent_hou: I'll have to look, because like you say, it's not "normal" | 09:29 |
*** wenjuan has quit IRC | 09:30 | |
*** wenjuan has joined #openstack-cinder | 09:30 | |
*** wenjuan has quit IRC | 09:31 | |
vincent_hou | geguileo: https://bugs.launchpad.net/cinder/+bug/1505572 it is full log here. I have to leave now. I can reach out to you after dinner, or you can put any new discoveries in this link. | 09:31 |
openstack | Launchpad bug 1505572 in Cinder "Migration: retype of a volume with NULL volume type to a non-null volume type fails during IMPL.finish_volume_migration" [Undecided,New] | 09:31 |
*** wenjuan has joined #openstack-cinder | 09:31 | |
geguileo | vincent_hou: I'll add my comments to the BP | 09:31 |
*** wenjuan has quit IRC | 09:32 | |
*** vincent_hou has quit IRC | 09:32 | |
*** EinstCrazy has quit IRC | 09:35 | |
*** openstackstatus has joined #openstack-cinder | 09:37 | |
*** ChanServ sets mode: +v openstackstatus | 09:37 | |
-openstackstatus- NOTICE: gerrit is undergoing an emergency restart to investigate load issues | 09:41 | |
*** ChanServ changes topic to "gerrit is undergoing an emergency restart to investigate load issues" | 09:41 | |
openstackgerrit | dongwenjuan proposed openstack/python-cinderclient: Support forcing service down https://review.openstack.org/234001 | 09:46 |
*** dims__ has joined #openstack-cinder | 09:49 | |
*** BharatK_ has quit IRC | 09:52 | |
*** zhangjn has quit IRC | 09:52 | |
*** smoriya has quit IRC | 09:53 | |
*** davechen has left #openstack-cinder | 09:56 | |
*** e0ne has joined #openstack-cinder | 09:58 | |
*** zhenguo has quit IRC | 10:00 | |
*** haomaiwang has quit IRC | 10:01 | |
*** openstackgerrit has quit IRC | 10:01 | |
*** aix has joined #openstack-cinder | 10:01 | |
*** haomaiwang has joined #openstack-cinder | 10:01 | |
*** openstackgerrit has joined #openstack-cinder | 10:02 | |
openstackgerrit | WenPing Zhang proposed openstack/cinder: New cinder driver to support Daowoo YeeSan https://review.openstack.org/202440 | 10:03 |
*** BharatK_ has joined #openstack-cinder | 10:04 | |
openstackgerrit | Ivan Kolodyazhny proposed openstack/cinder: Fix Status-Line in HTTP response https://review.openstack.org/231438 | 10:11 |
*** daneyon has joined #openstack-cinder | 10:11 | |
*** zhongjun22 has joined #openstack-cinder | 10:16 | |
*** vgridnev has quit IRC | 10:16 | |
*** daneyon has quit IRC | 10:16 | |
*** chenying has quit IRC | 10:18 | |
*** dims__ is now known as dims | 10:18 | |
*** haomaiwang has quit IRC | 10:29 | |
*** haomaiwang has joined #openstack-cinder | 10:30 | |
*** aix has quit IRC | 10:30 | |
openstackgerrit | dongwenjuan proposed openstack/python-cinderclient: Support forcing service down https://review.openstack.org/234001 | 10:35 |
*** IanGovett has joined #openstack-cinder | 10:35 | |
*** yrabl has joined #openstack-cinder | 10:38 | |
*** aix has joined #openstack-cinder | 10:42 | |
*** avishay_ has quit IRC | 10:47 | |
*** edmondsw has joined #openstack-cinder | 10:52 | |
*** edmondsw has quit IRC | 10:52 | |
*** subscope has quit IRC | 10:56 | |
*** jaypipes has joined #openstack-cinder | 10:59 | |
yrabl | names | 10:59 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements https://review.openstack.org/234128 | 10:59 |
*** pradipta has quit IRC | 11:00 | |
*** haomaiwang has quit IRC | 11:01 | |
*** haomaiwang has joined #openstack-cinder | 11:01 | |
*** p0rtal has joined #openstack-cinder | 11:02 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements https://review.openstack.org/232889 | 11:04 |
*** daneyon has joined #openstack-cinder | 11:05 | |
*** kevincarr1991 has joined #openstack-cinder | 11:07 | |
*** avishay_ has joined #openstack-cinder | 11:08 | |
*** p0rtal has quit IRC | 11:08 | |
*** daneyon has quit IRC | 11:10 | |
*** ociuhandu has quit IRC | 11:14 | |
*** ChanServ changes topic to "The OpenStack Block Storage Project Cinder | The New Kids On the Block | https://wiki.openstack.org/wiki/Cinder" | 11:16 | |
*** diogogmt has quit IRC | 11:16 | |
-openstackstatus- NOTICE: Gerrit has been restarted and is responding to normal load again. | 11:16 | |
*** avishay_ has quit IRC | 11:17 | |
*** merooney has joined #openstack-cinder | 11:25 | |
*** timcl has joined #openstack-cinder | 11:30 | |
openstackgerrit | Ivan Berezovskiy proposed openstack/cinder: Fix log formatting for rbd driver https://review.openstack.org/234154 | 11:31 |
*** jistr is now known as jistr|biab | 11:32 | |
*** subscope has joined #openstack-cinder | 11:38 | |
*** dims has quit IRC | 11:47 | |
*** dims has joined #openstack-cinder | 11:48 | |
*** haomaiwang has quit IRC | 11:52 | |
*** haomaiwang has joined #openstack-cinder | 11:52 | |
*** ociuhandu has joined #openstack-cinder | 11:56 | |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Fix method volume_types.get_all https://review.openstack.org/234167 | 11:58 |
*** cdelatte has quit IRC | 11:59 | |
*** daneyon has joined #openstack-cinder | 11:59 | |
*** e0ne has quit IRC | 12:00 | |
*** haomaiwang has quit IRC | 12:01 | |
*** haomaiwa_ has joined #openstack-cinder | 12:01 | |
*** dave-mccowan has joined #openstack-cinder | 12:01 | |
*** daneyon has quit IRC | 12:04 | |
*** jgregor has joined #openstack-cinder | 12:06 | |
*** gouthamr has joined #openstack-cinder | 12:08 | |
*** ociuhandu has quit IRC | 12:13 | |
*** salv-orlando has quit IRC | 12:14 | |
kevincarr1991 | I can manually mount a nfs share using the mount command, but the nfsdriver for openstack gets permission denied | 12:16 |
kevincarr1991 | anyone else use nfs share for cinder? | 12:16 |
DuncanT | kevincarr1991: Are you trying the mount as the user cinder is running as? | 12:17 |
*** haomaiwa_ has quit IRC | 12:17 | |
*** haomaiwang has joined #openstack-cinder | 12:17 | |
kevincarr1991 | DuncanT: no I tried as root. I thought root was used for the driver | 12:18 |
*** cdelatte has joined #openstack-cinder | 12:18 | |
*** delattec has joined #openstack-cinder | 12:18 | |
*** martyturner has joined #openstack-cinder | 12:18 | |
DuncanT | kevincarr1991: I'm not sure, it might use rootwrap - there are some secure mode settings that control that maybe? If you're running with debug level logging, you should be able see exactly what it's trying to run | 12:19 |
kevincarr1991 | DuncanT: Sorry I am new to openstack and dont really know how to run it with debug level logging. | 12:20 |
kevincarr1991 | DuncanT: how would i be able to do that? | 12:20 |
DuncanT | kevincarr1991: Set debug=True and verbose=True in cinder.conf then restart | 12:20 |
*** delatte has joined #openstack-cinder | 12:22 | |
kevincarr1991 | DuncanT: I did already have verbose=True. I see that a permission denied statement still shows up but I dont get any more info then that | 12:22 |
DuncanT | kevincarr1991: Can you post the log to paste.openstack.org please? | 12:23 |
kevincarr1991 | DuncanT: can you debug a service when restarting it besides the log files? | 12:23 |
kevincarr1991 | DuncanT: will do | 12:23 |
DuncanT | The log files really are the main source of debug | 12:23 |
*** cdelatte has quit IRC | 12:24 | |
*** raildo-afk is now known as raildo | 12:24 | |
kevincarr1991 | http://paste.openstack.org/show/476120/ | 12:25 |
*** avishay_ has joined #openstack-cinder | 12:26 | |
*** haomaiwang has quit IRC | 12:29 | |
*** ociuhandu has joined #openstack-cinder | 12:29 | |
*** ociuhandu_ has joined #openstack-cinder | 12:31 | |
*** edmondsw has joined #openstack-cinder | 12:31 | |
*** ociuhandu has quit IRC | 12:34 | |
*** ociuhandu_ is now known as ociuhandu | 12:34 | |
*** crose has joined #openstack-cinder | 12:34 | |
kevincarr1991 | DuncanT: http://paste.openstack.org/show/476120/ | 12:34 |
DuncanT | Looking.... | 12:34 |
DuncanT | kevincarr1991: "Exception during mounting Unexpected error while running command. | 12:37 |
DuncanT | Command: mkdir -p /var/lib/cinder/mnt/17e34a943fe7923d403c10240561727f" - I'm guessing there's a perms issue there somewhere in that directory tree | 12:37 |
*** porrua has joined #openstack-cinder | 12:37 | |
*** delattec has quit IRC | 12:38 | |
kevincarr1991 | DuncanT: Yeah I noticed that line and I am trying to mount the nfs as the cinder user. I am not having much luck. I get a mount.nfs: failed to apply fstab options error | 12:38 |
kevincarr1991 | I assume it is because I am specifying version 3 | 12:38 |
*** zhangjn has joined #openstack-cinder | 12:39 | |
*** zhangjn has quit IRC | 12:40 | |
*** zhangjn has joined #openstack-cinder | 12:40 | |
*** EinstCrazy has joined #openstack-cinder | 12:40 | |
*** zhangjn has quit IRC | 12:41 | |
*** zhangjn has joined #openstack-cinder | 12:41 | |
*** zhangjn has quit IRC | 12:41 | |
*** zhangjn has joined #openstack-cinder | 12:42 | |
*** zhangjn has quit IRC | 12:42 | |
*** zhangjn has joined #openstack-cinder | 12:43 | |
*** e0ne has joined #openstack-cinder | 12:46 | |
*** daneyon has joined #openstack-cinder | 12:53 | |
*** vgridnev has joined #openstack-cinder | 12:57 | |
*** Yogi1 has joined #openstack-cinder | 12:57 | |
*** bluex-pl has quit IRC | 12:58 | |
*** bluex-pl has joined #openstack-cinder | 12:58 | |
*** daneyon has quit IRC | 12:58 | |
*** dustins has joined #openstack-cinder | 13:01 | |
*** p0rtal has joined #openstack-cinder | 13:02 | |
*** lpetrut has quit IRC | 13:02 | |
*** haomaiwang has joined #openstack-cinder | 13:03 | |
*** jistr|biab is now known as jistr | 13:03 | |
*** p0rtal has quit IRC | 13:06 | |
*** bluex-pl has quit IRC | 13:07 | |
*** bluex-pl has joined #openstack-cinder | 13:08 | |
*** avishay_ has quit IRC | 13:08 | |
*** alejandrito has joined #openstack-cinder | 13:09 | |
*** salv-orlando has joined #openstack-cinder | 13:10 | |
*** adrianofr has joined #openstack-cinder | 13:10 | |
*** bluex-pl has quit IRC | 13:11 | |
*** bluex-pl has joined #openstack-cinder | 13:12 | |
*** gouthamr has quit IRC | 13:14 | |
*** julim has joined #openstack-cinder | 13:15 | |
e0ne | dulek: hi. just FYI, I've posted comments for https://review.openstack.org/#/c/234154/ | 13:15 |
e0ne | dulek: unit-tests are already implemented but didn't found this issue | 13:15 |
*** bluex-pl has quit IRC | 13:16 | |
*** bluex-pl has joined #openstack-cinder | 13:17 | |
*** gouthamr has joined #openstack-cinder | 13:18 | |
DuncanT | e0ne: Is it worth putting in a mock for logging that tests the format string matches the arguments? | 13:19 |
DuncanT | e0ne: Not necessarily right now, but does that sound useful? | 13:19 |
e0ne | DuncanT: personally, I don't like tests for logging and error messages | 13:20 |
e0ne | DuncanT: such things should be validated with hacking checks | 13:20 |
*** stevemar_ has joined #openstack-cinder | 13:20 | |
*** timcl has quit IRC | 13:21 | |
*** crose has quit IRC | 13:23 | |
smcginnis | thingee: TTL expired | 13:25 |
smcginnis | jgriffith: Around now. Ping me when you want to talk. | 13:25 |
* smcginnis runs for coffee | 13:25 | |
*** julim has quit IRC | 13:25 | |
*** bluex-pl has quit IRC | 13:26 | |
*** julim has joined #openstack-cinder | 13:26 | |
*** mc_nair has joined #openstack-cinder | 13:27 | |
*** lpetrut has joined #openstack-cinder | 13:27 | |
dulek | e0ne: Thanks for clarifying. | 13:28 |
dulek | e0ne: I've changed my vote. | 13:29 |
e0ne | dulek: you're welcome! thanks | 13:29 |
*** baumann has joined #openstack-cinder | 13:29 | |
*** baumann has left #openstack-cinder | 13:29 | |
*** baumann has joined #openstack-cinder | 13:29 | |
*** bluex-pl has joined #openstack-cinder | 13:30 | |
*** dave-mccowan has quit IRC | 13:32 | |
*** eharney has joined #openstack-cinder | 13:36 | |
*** vgridnev has quit IRC | 13:37 | |
openstackgerrit | Sean McGinnis proposed openstack/cinder: Only use LOG.exception in exception handler https://review.openstack.org/233828 | 13:37 |
jordanP | jgriffith, hi. I am going to work on https://bugs.launchpad.net/cinder/+bug/1495560 | 13:38 |
openstack | Launchpad bug 1495560 in Cinder "lvm deactivate race in snapshot extend" [High,Triaged] - Assigned to John Griffith (john-griffith) | 13:39 |
*** martyturner has quit IRC | 13:40 | |
*** martyturner has joined #openstack-cinder | 13:41 | |
*** vgridnev has joined #openstack-cinder | 13:41 | |
*** DericHorn-HP has joined #openstack-cinder | 13:42 | |
*** vgridnev has quit IRC | 13:43 | |
*** vgridnev has joined #openstack-cinder | 13:45 | |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: Fix method VolumeTypeList.get_all https://review.openstack.org/234167 | 13:46 |
*** daneyon has joined #openstack-cinder | 13:48 | |
*** dzamboni has joined #openstack-cinder | 13:48 | |
*** ntpttr has joined #openstack-cinder | 13:48 | |
*** flaper87 has joined #openstack-cinder | 13:48 | |
*** bluex-pl has quit IRC | 13:48 | |
*** ntpttr has quit IRC | 13:52 | |
*** daneyon has quit IRC | 13:53 | |
*** jdandrea has joined #openstack-cinder | 13:53 | |
*** ntpttr has joined #openstack-cinder | 13:54 | |
jdandrea | Regarding https://github.com/openstack/cinder/blob/master/cinder/openstack/common/scheduler/base_handler.py#L45 the comment mentions this being a set, but isn't this a list? | 13:54 |
jdandrea | Trying to determine if we can rely on the calling order of scheduler filter plugins or not. | 13:55 |
*** r-daneel has joined #openstack-cinder | 13:55 | |
*** cfriesen has joined #openstack-cinder | 13:56 | |
*** ntpttr has quit IRC | 13:59 | |
*** DericHorn-HP has quit IRC | 13:59 | |
jgriffith | jordanP: ahh... ok, great | 13:59 |
jgriffith | jordanP: Shall I reassign it to you? | 13:59 |
jordanP | jgriffith, yes please | 14:00 |
*** martyturner has quit IRC | 14:00 | |
jgriffith | jordanP: cool... done | 14:00 |
*** timcl has joined #openstack-cinder | 14:00 | |
*** haomaiwang has quit IRC | 14:01 | |
*** martyturner has joined #openstack-cinder | 14:01 | |
*** haomaiwa_ has joined #openstack-cinder | 14:01 | |
*** DericHorn-HP has joined #openstack-cinder | 14:03 | |
*** BharatK_ has quit IRC | 14:04 | |
*** julim has quit IRC | 14:05 | |
*** jwcroppe has joined #openstack-cinder | 14:07 | |
*** julim has joined #openstack-cinder | 14:07 | |
jordanP | jgriffith, I can reproduce easily, locally on my laptop | 14:08 |
*** bswartz has joined #openstack-cinder | 14:08 | |
jgriffith | jordanP: excellent | 14:09 |
*** jwcroppe has quit IRC | 14:11 | |
*** jwcroppe has joined #openstack-cinder | 14:12 | |
*** dave-mccowan has joined #openstack-cinder | 14:15 | |
e0ne | smcginnis: hi. did you see comment for https://review.openstack.org/#/c/234118/? | 14:16 |
*** baumann1 has joined #openstack-cinder | 14:16 | |
*** salv-orlando has quit IRC | 14:17 | |
e0ne | smcginnis: does it mean that I need to squosh 2 commits into the one insteas of cherry-pick? | 14:17 |
*** jwcroppe has quit IRC | 14:17 | |
*** baumann has quit IRC | 14:19 | |
*** ntpttr has joined #openstack-cinder | 14:19 | |
*** crose has joined #openstack-cinder | 14:20 | |
*** jwcroppe has joined #openstack-cinder | 14:20 | |
*** tpeoples has joined #openstack-cinder | 14:21 | |
*** haomaiwa_ has quit IRC | 14:21 | |
*** haomaiwang has joined #openstack-cinder | 14:21 | |
smcginnis | e0ne: Just talking in infra about this. | 14:23 |
smcginnis | I was asked to squash the two and submit. | 14:23 |
e0ne | smcginnis: thanks. i'm already working on it | 14:23 |
e0ne | smcginnis: will post a review request in a minute | 14:24 |
smcginnis | In meeting, so chances are high I screwed it up, but I just submitted it. | 14:24 |
e0ne | smcginnis: oh.. thanks a lot! you did it faster than me | 14:25 |
smcginnis | e0ne: Although now I can't find it. Commands all worked, but I don't see it out there. | 14:25 |
* smcginnis is confused. | 14:25 | |
e0ne | smcginnis: dims, ttx | 14:26 |
e0ne | ooops, copy-paste issue:( | 14:26 |
smcginnis | e0ne: ;) | 14:28 |
*** baumann1 has quit IRC | 14:29 | |
*** vlaza has quit IRC | 14:31 | |
*** bluex-pl has joined #openstack-cinder | 14:33 | |
*** wilson1 has quit IRC | 14:34 | |
*** diogogmt has joined #openstack-cinder | 14:34 | |
*** salv-orlando has joined #openstack-cinder | 14:34 | |
*** wilson1 has joined #openstack-cinder | 14:36 | |
*** baumann has joined #openstack-cinder | 14:37 | |
*** aorourke_afk is now known as aorourke | 14:38 | |
*** daneyon has joined #openstack-cinder | 14:42 | |
jgriffith | jdurgin: Thanks for jumping on that thread :) | 14:43 |
*** xyang1 has joined #openstack-cinder | 14:43 | |
*** rushil has joined #openstack-cinder | 14:44 | |
jgriffith | patrickeast: couldn't help but notice after those patches landed you were able to rebase :) | 14:44 |
jgriffith | patrickeast: were you able to get all of them? | 14:44 |
jgriffith | patrickeast: or is there still anything missing we need to figure out? | 14:45 |
*** rushil_ has joined #openstack-cinder | 14:45 | |
*** daneyon has quit IRC | 14:47 | |
*** edtubill has joined #openstack-cinder | 14:47 | |
*** rushil has quit IRC | 14:48 | |
*** baumann has quit IRC | 14:52 | |
jordanP | guys, the volume_clear option, if set in the DEFAULT section has no effect. it must be on the per backend section. Devstack sets it in the [DEFAULT] section | 14:56 |
jordanP | who is wrong here ? | 14:56 |
*** baumann has joined #openstack-cinder | 14:56 | |
eharney | jordanP: well... depends on who you ask :) | 14:56 |
jordanP | lol | 14:57 |
jordanP | eharney, what do you think ? | 14:57 |
eharney | well, devstack needs to be setting it in the backend sections currently for it to work as intended | 14:57 |
*** martyturner has quit IRC | 14:57 | |
jordanP | then I'll make a patch to devstack then | 14:57 |
eharney | i find that to be pretty unfortunate from our side, but i don't really know how to fix it (we have some plans to talk about this at summit) | 14:57 |
*** timcl has quit IRC | 14:58 | |
jordanP | maybe if volume_clear is None in the per backend config, we should lookup in the DEFAULT section | 14:58 |
*** mriedem has joined #openstack-cinder | 14:59 | |
*** martyturner has joined #openstack-cinder | 14:59 | |
*** martyturner has quit IRC | 14:59 | |
eharney | this behavior applies to a lot of other options too | 14:59 |
smcginnis | jordanP: There has been some discussion of inheriting default settings in each backend, but it does cause some... concerns. | 14:59 |
*** martyturner has joined #openstack-cinder | 14:59 | |
eharney | smcginnis: as does not inheriting them :) | 15:00 |
jordanP | ok, I am fine with fixing devstack | 15:00 |
smcginnis | eharney: True | 15:00 |
openstackgerrit | John Griffith proposed openstack/cinder: Update config format for replication_devices https://review.openstack.org/233307 | 15:00 |
*** Yogi11 has joined #openstack-cinder | 15:00 | |
*** haomaiwang has quit IRC | 15:01 | |
jordanP | Both ways work with their pros and cons. In the mean time I'll submit a change to devstack | 15:01 |
*** haomaiwang has joined #openstack-cinder | 15:01 | |
jgriffith | jordanP: the problem is that the way config is parsed out I couldn't come up with a way to get those external views | 15:01 |
*** p0rtal has joined #openstack-cinder | 15:02 | |
jgriffith | jordanP: there's no way to "know" if the setting in self.configuration.xxxx is from the default section or from the driver section or it's just the default setting | 15:02 |
eharney | jordanP: i think devstack already handles this for LVM? | 15:02 |
jgriffith | eharney: +1 | 15:02 |
jgriffith | eharney: not think.. it does | 15:02 |
jordanP | eharney, no :( it sets volume_clear in the [default] section | 15:02 |
eharney | but it looks like we have $CINDER_VOLUME_CLEAR for the general case and $CINDER_SECURE_DELETE for LVM | 15:03 |
eharney | maybe you need to set the latter option? | 15:03 |
jordanP | ahh right | 15:03 |
jordanP | I forgot about $CINDER_SECURE_DELETE | 15:03 |
jgriffith | jordanP: that's the one that actually matters :) | 15:03 |
jordanP | yeah but if you use it you get this big warning | 15:04 |
jordanP | "deprecated "Configure secure Cinder volume deletion using CINDER_VOLUME_CLEAR instead of CINDER_SECURE_DELETE."" | 15:04 |
*** Yogi1 has quit IRC | 15:04 | |
jgriffith | Whaaaaa | 15:04 |
eharney | it also appears to only do anything if you set it to False, so it's not really the same thing anyway | 15:04 |
*** ntpttr has quit IRC | 15:05 | |
*** salv-orl_ has joined #openstack-cinder | 15:05 | |
*** leeantho has joined #openstack-cinder | 15:06 | |
*** leeantho_ has joined #openstack-cinder | 15:06 | |
*** salv-orlando has quit IRC | 15:07 | |
*** p0rtal has quit IRC | 15:07 | |
*** ronis has quit IRC | 15:09 | |
*** ntpttr has joined #openstack-cinder | 15:10 | |
*** leeantho has quit IRC | 15:11 | |
*** ntpttr has quit IRC | 15:13 | |
*** timcl has joined #openstack-cinder | 15:14 | |
*** ntpttr has joined #openstack-cinder | 15:15 | |
*** bluex-pl has quit IRC | 15:16 | |
*** bluex-pl has joined #openstack-cinder | 15:19 | |
*** bluex-pl has quit IRC | 15:20 | |
*** bluex-pl has joined #openstack-cinder | 15:20 | |
*** timcl1 has joined #openstack-cinder | 15:20 | |
*** rushil_ has quit IRC | 15:21 | |
*** timcl has quit IRC | 15:22 | |
*** salv-orl_ has quit IRC | 15:23 | |
openstackgerrit | Marian Horban proposed openstack/cinder: Remove wsgi functionality https://review.openstack.org/219671 | 15:25 |
*** salv-orlando has joined #openstack-cinder | 15:28 | |
*** nkrinner has quit IRC | 15:30 | |
*** jdurgin1 has joined #openstack-cinder | 15:31 | |
openstackgerrit | Tom Swanson proposed openstack/cinder: Dell SC: Disable REST verify warnings https://review.openstack.org/234294 | 15:31 |
*** cbader_ has joined #openstack-cinder | 15:35 | |
*** daneyon has joined #openstack-cinder | 15:36 | |
*** salv-orl_ has joined #openstack-cinder | 15:36 | |
openstackgerrit | Alon Marx proposed openstack/cinder: Updates in consistency_group in xiv/ds8k driver https://review.openstack.org/233280 | 15:36 |
*** zhangjn has quit IRC | 15:37 | |
*** salv-orlando has quit IRC | 15:38 | |
*** mudassirlatif has joined #openstack-cinder | 15:38 | |
*** jungleboyj has joined #openstack-cinder | 15:39 | |
*** daneyon has quit IRC | 15:41 | |
*** vgridnev has quit IRC | 15:45 | |
*** daneyon has joined #openstack-cinder | 15:47 | |
*** lpetrut has quit IRC | 15:48 | |
*** anshul has quit IRC | 15:48 | |
hemna | mep | 15:49 |
*** daneyon_ has joined #openstack-cinder | 15:49 | |
*** tpeoples has quit IRC | 15:52 | |
*** vgridnev has joined #openstack-cinder | 15:52 | |
openstackgerrit | Sean Dague proposed openstack/cinder: Change default Exception code to 500 https://review.openstack.org/233923 | 15:52 |
*** daneyon has quit IRC | 15:53 | |
*** harlowja_at_home has joined #openstack-cinder | 15:53 | |
*** vgridnev has quit IRC | 15:54 | |
*** IlyaG has joined #openstack-cinder | 15:56 | |
*** e0ne has quit IRC | 15:56 | |
*** p0rtal has joined #openstack-cinder | 15:57 | |
*** p0rtal has quit IRC | 15:57 | |
*** vgridnev has joined #openstack-cinder | 15:58 | |
*** haomaiwang has quit IRC | 16:01 | |
*** tpeoples has joined #openstack-cinder | 16:01 | |
*** timcl1 has quit IRC | 16:01 | |
*** haomaiwang has joined #openstack-cinder | 16:01 | |
*** martyturner has quit IRC | 16:01 | |
*** martyturner has joined #openstack-cinder | 16:03 | |
*** timcl has joined #openstack-cinder | 16:03 | |
*** Yogi1 has joined #openstack-cinder | 16:05 | |
*** _cjones_ has joined #openstack-cinder | 16:05 | |
*** _cjones_ has quit IRC | 16:05 | |
*** merooney has quit IRC | 16:05 | |
*** _cjones_ has joined #openstack-cinder | 16:05 | |
*** harlowja_at_home has quit IRC | 16:05 | |
*** rhagarty__ has quit IRC | 16:06 | |
*** apoorvad has joined #openstack-cinder | 16:06 | |
*** Yogi11 has quit IRC | 16:06 | |
*** apoorvad has quit IRC | 16:07 | |
*** jdurgin1 has quit IRC | 16:08 | |
*** rhagarty has joined #openstack-cinder | 16:10 | |
*** apoorvad has joined #openstack-cinder | 16:11 | |
hemna | eharney, ping | 16:11 |
eharney | hemna: hey | 16:12 |
hemna | can you take a look at this one for me ?https://review.openstack.org/#/c/199764 | 16:12 |
hemna | I'm not an RBD guy | 16:12 |
hemna | I added some calls in the RBD connector, but they really are shims as I don't know what/if anything, there is to do for those calls for RBD | 16:13 |
eharney | hemna: looking | 16:13 |
hemna | thanks | 16:13 |
*** IlyaG has quit IRC | 16:14 | |
*** ronis has joined #openstack-cinder | 16:16 | |
*** kevincarr1991 has quit IRC | 16:18 | |
*** timcl has quit IRC | 16:18 | |
mriedem | smcginnis: jgriffith: this needs updating it looks like https://review.openstack.org/#/c/233668/ | 16:19 |
mriedem | cinder rc3 is going to need that, but the webob thing has changed on master | 16:19 |
*** vgridnev has quit IRC | 16:20 | |
mriedem | so i think we just take that and squash with https://review.openstack.org/#/c/233923/ | 16:20 |
jgriffith | mriedem: snap! Yes, I pinged smcginnis about that, meant to get ahold of e0ne | 16:20 |
jgriffith | I'll update it now | 16:20 |
jgriffith | mriedem: or...let smcginnis handle it since he's been doing squashes today :) | 16:20 |
mriedem | sounds like ttx would like it asap | 16:20 |
mriedem | i can squash quick if you like | 16:21 |
jgriffith | mriedem: either way... I can do it or if you already have the forumula go for it | 16:21 |
jgriffith | mriedem: TBH last time I did a squash on a gerrit change it didn't go well for me | 16:21 |
mriedem | i'm on it | 16:22 |
jgriffith | mriedem: thanks | 16:22 |
eharney | hemna: added some comments... if i understand this change correctly, i think a lot of this is a no-op for RBD, but i don't have a lot of background | 16:22 |
jgriffith | mriedem: when you have a moment I'd love insight on if the commit message update "just works" when you do the squash or not | 16:22 |
hemna | eharney, ok cool thanks. | 16:23 |
jgriffith | mriedem: that's what I got hung up on the last time I tried it... the commit message was toast; don't know what I did wrong | 16:23 |
mriedem | you have to sweet talk it | 16:23 |
mriedem | the commit message whisperer | 16:23 |
jgriffith | mriedem: LOL... that explains it | 16:23 |
jgriffith | mriedem: I tried the Drill Sergeant approach | 16:24 |
hemna | eharney, ok, so I'm returning [], None for those as it will be safe for a caller to get empty lists and None value for those for RBD. | 16:24 |
jgriffith | mriedem: ahh... I see what I did wrong | 16:24 |
mriedem | man there is already a merge conflict | 16:25 |
mriedem | not sure how that happened | 16:25 |
mriedem | ah the assertEqual nit patch | 16:25 |
mriedem | fun | 16:25 |
jgriffith | mriedem: sigh... who merged that stupid thing | 16:25 |
eharney | hemna: yeah, i think since RBD volumes are attached from within qemu, there isn't really any need to do this validation, since there isn't a connection/device at this level | 16:25 |
hemna | ok that's kinda what I thought | 16:26 |
hemna | thanks for the help | 16:26 |
*** garthb has joined #openstack-cinder | 16:26 | |
mriedem | this kind of seems like a bad idea https://review.openstack.org/#/c/231438/5/cinder/exception.py if webob ever changes that internal stuff | 16:28 |
jgriffith | mriedem: yeah, I saw that and was unsure about it | 16:29 |
hemna | why not push that up into webob then ? | 16:29 |
jgriffith | hemna: ? | 16:30 |
hemna | the comment leads one to believe that it's a strict requirement | 16:30 |
openstackgerrit | Yuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver https://review.openstack.org/222292 | 16:31 |
*** jistr has quit IRC | 16:31 | |
hemna | meh | 16:31 |
mriedem | jgriffith: squashed https://review.openstack.org/#/c/234329/ | 16:31 |
jgriffith | mriedem: thanks | 16:32 |
*** wilson-1 has joined #openstack-cinder | 16:32 | |
jgriffith | mriedem: I'll patiently wait for an education on how you got it to work :) | 16:32 |
mriedem | git review -d 233668 | 16:33 |
mriedem | git fetch ssh://mriedem@review.openstack.org:29418/openstack/cinder refs/changes/23/233923/2 && git cherry-pick -x FETCH_HEAD | 16:33 |
mriedem | git rebase -i HEAD~2 | 16:33 |
mriedem | squash 2nd into 1st | 16:33 |
mriedem | accept the commit message | 16:33 |
jgriffith | mriedem: ahhh | 16:33 |
mriedem | and git review that sucka | 16:33 |
jgriffith | mriedem: ok, I messed up the fetch | 16:33 |
jgriffith | mriedem: awesome!! Thanks! | 16:33 |
mriedem | np | 16:34 |
*** subscope has quit IRC | 16:34 | |
*** wilson1 has quit IRC | 16:34 | |
mriedem | hemna: can you +W this? https://review.openstack.org/#/c/233923/ it was approved before but was missing a unit test fix | 16:41 |
mriedem | and we have it backported to stable/liberty and it's approved there | 16:42 |
*** cbader_ has quit IRC | 16:43 | |
smcginnis | mriedem, jgriffith: Thanks, got pulled away. | 16:44 |
jgriffith | ntpttr: ping | 16:46 |
ntpttr | jgriffith: Hey | 16:48 |
hemna | mriedem, crap sorry. was afk for a bit | 16:48 |
*** jordanP has quit IRC | 16:49 | |
smcginnis | All: Please limit rechecks and such until this lands: https://review.openstack.org/#/c/233923/ | 16:50 |
openstackgerrit | Gorka Eguileor proposed openstack/cinder-specs: Job Distribution to support HA A/A https://review.openstack.org/232595 | 16:50 |
jgriffith | DuncanT: stop approving things for the time being :) | 16:51 |
DuncanT | jgriffith: Sure, sorry, hadn't heard | 16:51 |
jgriffith | DuncanT: no worries | 16:51 |
adrianofr | All: Hi guys. I have a backport to liberty (https://review.openstack.org/#/c/232091/) with a -2 due to "No RC3 window yet". When will RC3 window happening? | 16:52 |
mriedem | adrianofr: right now | 16:52 |
adrianofr | mriedem: So, it's already able to be approved, right? | 16:53 |
mriedem | adrianofr: there is one cinder change on stable/liberty that is needed for rc3 | 16:53 |
mriedem | ^ does not seem critical enough to warrant getting it into rc3 at this point | 16:53 |
mriedem | the importance isn't even set on the bug | 16:53 |
mriedem | it's fine for stable in general, but not a release blocker | 16:54 |
smcginnis | adrianofr: Probably soon, but we need to wait for the final RC3. | 16:54 |
*** lpetrut has joined #openstack-cinder | 16:55 | |
jgriffith | smcginnis: I gave everything that was "close" a -2 to try and prevent anything else until the ec patch lands | 16:55 |
jgriffith | smcginnis: don't let me forget to clear them later :) | 16:55 |
jgriffith | eharney: hemna geguileo jungleboyj just incase you didn't see, don't merge anything for now | 16:56 |
smcginnis | jgriffith: Thanks, good plan. | 16:56 |
adrianofr | mriedem: smcginnis : I see. Thanks for the info :) | 16:56 |
hemna | jgriffith, ok | 16:56 |
smcginnis | adrianofr: np | 16:56 |
geguileo | jgriffith: I didn't see | 16:56 |
jgriffith | geguileo: :) | 16:56 |
geguileo | jgriffith: Was there an email? :-O | 16:56 |
jgriffith | geguileo: no, just conversation here | 16:57 |
hemna | https://review.openstack.org/#/c/233923/ waiting for that guy ? | 16:57 |
geguileo | jgriffith: Yesterday it was a National Holiday here | 16:57 |
jgriffith | hemna: yes | 16:57 |
hemna | coolio | 16:57 |
jgriffith | geguileo: nice | 16:57 |
jgriffith | geguileo: no worries, just came up this morning... not a huge deal | 16:58 |
geguileo | jgriffith: Nice but I missed that XD | 16:58 |
geguileo | jgriffith: Ok, no merging for now, noted | 16:58 |
geguileo | jgriffith: Thanks! | 16:58 |
jgriffith | geguileo: thank you sir | 16:58 |
jungleboyj | jgriffith: Thanks for the warning. Missed the discussion. What is wrong? | 16:59 |
smcginnis | Just have 300+ other patches to make it by, then we're all clear. ;) | 16:59 |
hemna | LO | 16:59 |
jgriffith | lol | 16:59 |
hemna | bah | 16:59 |
smcginnis | jungleboyj: SNAFU. Waiting for one final cleanup to go through. | 16:59 |
jungleboyj | smcginnis: Ok. | 17:00 |
*** DericHorn-HP has quit IRC | 17:00 | |
*** bluex-pl has quit IRC | 17:00 | |
*** haomaiwang has quit IRC | 17:01 | |
*** haomaiwang has joined #openstack-cinder | 17:01 | |
*** sghanekar_ has joined #openstack-cinder | 17:01 | |
*** timcl has joined #openstack-cinder | 17:02 | |
*** Yogi1 has quit IRC | 17:03 | |
*** vgridnev has joined #openstack-cinder | 17:03 | |
apoorvad | sghanekar_: hi | 17:04 |
*** avishay_ has joined #openstack-cinder | 17:04 | |
*** merooney has joined #openstack-cinder | 17:05 | |
*** ntpttr has quit IRC | 17:07 | |
*** ntpttr has joined #openstack-cinder | 17:09 | |
*** kevincarr1991 has joined #openstack-cinder | 17:12 | |
*** Yogi1 has joined #openstack-cinder | 17:15 | |
*** p0rtal has joined #openstack-cinder | 17:15 | |
openstackgerrit | Fergal Mc Carthy proposed openstack/cinder: Use lvm_conf_file directory for LVM_SYSTEM_DIR value. https://review.openstack.org/226562 | 17:18 |
patrickeast | jgriffith: ah yea, i rebased that set of patches to pick up the webob fix, doing a recheck 'fixed' the merge conflict thing it ran into but then they were stuck on that before they could go any further | 17:18 |
patrickeast | jgriffith: although i lost all my +A's in the process :( | 17:18 |
jgriffith | patrickeast: we can fix that :) | 17:20 |
*** ntpttr has quit IRC | 17:20 | |
*** asselin_ has joined #openstack-cinder | 17:20 | |
*** mtanino has joined #openstack-cinder | 17:21 | |
*** aix has quit IRC | 17:22 | |
*** lpetrut has quit IRC | 17:22 | |
*** asselin__ has quit IRC | 17:22 | |
*** markus_z has quit IRC | 17:23 | |
*** salv-orl_ has quit IRC | 17:24 | |
*** salv-orlando has joined #openstack-cinder | 17:25 | |
jgriffith | smcginnis: I'll +2/A it when you get it posted :) | 17:27 |
openstackgerrit | Dmitry Guryanov proposed openstack/cinder: WIP: RemoteFS volume format handlers https://review.openstack.org/234359 | 17:28 |
smcginnis | jgriffith: https://review.openstack.org/234360 | 17:30 |
jgriffith | smcginnis: done | 17:31 |
jgriffith | so it looks like I confused the failure modes here | 17:31 |
jgriffith | master should be ok | 17:31 |
jgriffith | I'll clear my -2's | 17:31 |
jgriffith | It would still probably be ok if we didn't flood the gate with patches while things get caught up | 17:31 |
jgriffith | stable branch fixes should be priority right now | 17:32 |
smcginnis | jgriffith: clark was saying to hold off on master though. | 17:32 |
jgriffith | meh... I'll leave things alone for the time being :) | 17:32 |
jgriffith | smcginnis: oh.. perfect | 17:32 |
smcginnis | jgriffith: I _thought_ we were safe, but.. | 17:32 |
jgriffith | smcginnis: now I won't feel guilty :) | 17:32 |
smcginnis | jgriffith: ;) | 17:32 |
jgriffith | LOL.. like that would happen anyway :) | 17:32 |
*** wilson-1 has quit IRC | 17:34 | |
*** wilson-1 has joined #openstack-cinder | 17:35 | |
*** sghanekar_ has quit IRC | 17:38 | |
*** sghanekar_ has joined #openstack-cinder | 17:40 | |
*** xyang1 has quit IRC | 17:42 | |
*** IlyaG has joined #openstack-cinder | 17:45 | |
*** bill_az has joined #openstack-cinder | 17:46 | |
*** angela-s has joined #openstack-cinder | 17:56 | |
*** asselin_ is now known as asselin__ | 17:58 | |
*** ntpttr has joined #openstack-cinder | 18:00 | |
*** haomaiwang has quit IRC | 18:01 | |
*** haomaiwang has joined #openstack-cinder | 18:01 | |
*** mudassirlatif has quit IRC | 18:04 | |
*** mudassirlatif has joined #openstack-cinder | 18:08 | |
*** crose has quit IRC | 18:13 | |
*** ndipanov has quit IRC | 18:14 | |
*** e0ne has joined #openstack-cinder | 18:17 | |
*** mriedem has quit IRC | 18:18 | |
*** ociuhandu has quit IRC | 18:19 | |
*** mriedem has joined #openstack-cinder | 18:21 | |
*** avishay_ has quit IRC | 18:22 | |
*** vgridnev has quit IRC | 18:22 | |
*** kevincar_ has joined #openstack-cinder | 18:23 | |
*** avishay_ has joined #openstack-cinder | 18:23 | |
*** vgridnev has joined #openstack-cinder | 18:25 | |
openstackgerrit | John Griffith proposed openstack/cinder: Updates in consistency_group in xiv/ds8k driver https://review.openstack.org/233280 | 18:25 |
openstackgerrit | John Griffith proposed openstack/cinder: Fix method VolumeTypeList.get_all https://review.openstack.org/234167 | 18:25 |
openstackgerrit | John Griffith proposed openstack/cinder: Port WSGI tests to Python 3 https://review.openstack.org/232214 | 18:25 |
openstackgerrit | John Griffith proposed openstack/cinder: Cleanup/move code in Storwize Driver https://review.openstack.org/230117 | 18:25 |
openstackgerrit | John Griffith proposed openstack/cinder: Improve performance listing detail for volumes https://review.openstack.org/203530 | 18:26 |
openstackgerrit | John Griffith proposed openstack/cinder: encryption_api_url requires a version https://review.openstack.org/230031 | 18:26 |
openstackgerrit | John Griffith proposed openstack/cinder: Fix updating only volume type is_public https://review.openstack.org/221562 | 18:26 |
smcginnis | Rebases? | 18:26 |
openstackgerrit | John Griffith proposed openstack/cinder: CGSnapshot Object https://review.openstack.org/195027 | 18:26 |
openstackgerrit | John Griffith proposed openstack/cinder: Move CloneableVD to common functions https://review.openstack.org/203593 | 18:26 |
openstackgerrit | John Griffith proposed openstack/cinder-specs: Job Distribution to support HA A/A https://review.openstack.org/232595 | 18:26 |
jgriffith | smcginnis: yes | 18:26 |
*** subscope has joined #openstack-cinder | 18:26 | |
jgriffith | smcginnis: should've warned you :) sorry | 18:26 |
*** kevincarr1991 has quit IRC | 18:27 | |
*** kevincar_ has quit IRC | 18:27 | |
smcginnis | No worries. Didn't think you were that quick with updating patchsets. ;) | 18:27 |
jgriffith | smcginnis: hehe | 18:27 |
jgriffith | I'm doing a Victor/Gorka tick an just queuing them all up | 18:28 |
jgriffith | :) | 18:28 |
*** baumann has left #openstack-cinder | 18:28 | |
*** baumann has joined #openstack-cinder | 18:28 | |
*** e0ne has quit IRC | 18:30 | |
haypo | jgriffith: hey | 18:31 |
haypo | jgriffith: the WSGI patch serie just makes me crazy :-p | 18:31 |
jgriffith | haypo: LOL... which one exactly? | 18:32 |
jgriffith | haypo: You mean all the webob churn? | 18:32 |
haypo | jgriffith: it's a serie of 4 patches | 18:32 |
haypo | jgriffith: the serie starting with https://review.openstack.org/#/c/232214/ | 18:32 |
haypo | jgriffith: it's the version 10 and i have to rebase it multiple times each day !? | 18:33 |
jgriffith | Yeah, I figured that's the one you meant | 18:33 |
haypo | jgriffith: i got a lot of +2 :-p | 18:33 |
jgriffith | haypo: well.. the good news is that we should be in the clear after this morning | 18:33 |
haypo | the WebOb 1.5 issue is behind us, no? | 18:33 |
jgriffith | haypo: sorry about all of that | 18:33 |
jgriffith | haypo: yes, it *should* be behind us at this point | 18:33 |
jgriffith | haypo: there was a couple other issues that you got stuck in as well | 18:33 |
jgriffith | haypo: the good thing is at least you didn't loose votes with the rebase :) | 18:34 |
haypo | jgriffith: my webob 1.5 fix was merged yesterday no? https://review.openstack.org/#/c/233528/ | 18:34 |
jgriffith | haypo: yes, but there was a merge issue with a change to the default code | 18:34 |
jgriffith | haypo: then stable blew up | 18:34 |
jgriffith | haypo: then there was the requirements issue that showed up in messaging | 18:35 |
jgriffith | haypo: so there's been a lot of churn that actually had nothing to do with your patch | 18:35 |
haypo | jgriffith: yeah, i noticed :) i had to recheck multiple times :) | 18:35 |
*** salv-orlando has quit IRC | 18:35 | |
jgriffith | haypo: I'll +2/A your patch later this afternoon when things are all caught up again | 18:35 |
haypo | jgriffith: (note: my WSGI is no more approved, but jenkins didn't check the latest version neither) | 18:35 |
*** avishay_ has quit IRC | 18:36 | |
haypo | jgriffith: thanks & no need to be sorry :) | 18:36 |
jgriffith | haypo: I'll monitor these: https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:bp/cinder-python3,n,z | 18:37 |
jgriffith | and go back to them later today | 18:37 |
jgriffith | if there's something missing in that list let me know | 18:37 |
*** crose has joined #openstack-cinder | 18:46 | |
haypo | jgriffith: hum, usually i have a longer list of patches :-D i'm waiting until the WSGI patches are merged before writing the next patches ;) | 18:46 |
haypo | jgriffith: (you can ignore "WIP: Fix Python 3 issues in EMC vnx client") | 18:47 |
smcginnis | haypo: Yeah, try to keep some of your sanity. ;) | 18:49 |
haypo | haha | 18:49 |
*** haomaiwang has quit IRC | 18:53 | |
smcginnis | RC3 is being cut for Liberty now. I've learned enough that I will refrain from saying it should be the last. | 18:55 |
*** daneyon has joined #openstack-cinder | 18:56 | |
*** mudassirlatif has quit IRC | 18:58 | |
*** daneyon_ has quit IRC | 19:00 | |
*** daneyon has quit IRC | 19:00 | |
*** DericHorn-HP has joined #openstack-cinder | 19:01 | |
*** dzamboni has quit IRC | 19:03 | |
guitarzan | hah | 19:07 |
*** dave-mccowan has quit IRC | 19:15 | |
*** e0ne has joined #openstack-cinder | 19:19 | |
ntpttr | jgriffith: ping | 19:19 |
*** IlyaG has quit IRC | 19:19 | |
*** alejandrito has quit IRC | 19:20 | |
*** alejandrito has joined #openstack-cinder | 19:22 | |
*** mudassirlatif has joined #openstack-cinder | 19:22 | |
jgriffith | ntpttr: hey ya | 19:23 |
jgriffith | ntpttr: so the problem with your suggestion :) | 19:24 |
jgriffith | ntpttr: that limitation isn't actually a result of the setting in the driver you saw | 19:24 |
jgriffith | ntpttr: it's actually a cluster limit | 19:24 |
jgriffith | ntpttr: so creates will fail after 2K | 19:24 |
ntpttr | jgriffith: Ah! Okay that makes sense, I was definitely going in the wrong direction then :) | 19:25 |
jgriffith | ntpttr: yeah... sorry :( | 19:25 |
jgriffith | I wasn't clear about that in the bug | 19:25 |
jgriffith | ntpttr: FWIW, I've grown to hate the fix that I made there :( | 19:25 |
jgriffith | ntpttr: it's kinda clunky | 19:26 |
jgriffith | ntpttr: I need to just refactor all of that code | 19:26 |
ntpttr | jgriffith: Haha no problem, I just thought I'd give it a shot and worst case learn something | 19:26 |
jgriffith | ntpttr: well... you did :) | 19:26 |
*** mudassirlatif has quit IRC | 19:28 | |
*** martyturner has quit IRC | 19:29 | |
*** alejandrito has quit IRC | 19:32 | |
*** delatte has quit IRC | 19:36 | |
*** haomaiwang has joined #openstack-cinder | 19:37 | |
*** xyang1 has joined #openstack-cinder | 19:38 | |
*** e0ne has quit IRC | 19:43 | |
*** haomaiwang has quit IRC | 19:43 | |
DuncanT | Who runs the OpenvStorage CI? It's been failing continuously for a while | 19:44 |
*** martyturner has joined #openstack-cinder | 19:44 | |
*** hodos has joined #openstack-cinder | 19:49 | |
patrickeast | DuncanT: thats this one https://wiki.openstack.org/wiki/ThirdPartySystems/CloudFounders_OpenvStorage_CI right? | 19:54 |
DuncanT | That's the one, yeah. | 19:54 |
DuncanT | Do we have an IRC nick for them? | 19:54 |
DuncanT | If not, I can email them | 19:55 |
openstackgerrit | Nate Potter proposed openstack/cinder: Add qos_specs_id to volume type show https://review.openstack.org/234414 | 19:55 |
patrickeast | DuncanT: I don't see one | 19:56 |
*** dave-mccowan has joined #openstack-cinder | 19:57 | |
DuncanT | Ok, I'll email them, thanks | 19:58 |
*** ociuhandu has joined #openstack-cinder | 20:02 | |
*** martyturner has quit IRC | 20:06 | |
jgriffith | ntpttr: 234414 seems good/reasonable | 20:06 |
*** martyturner has joined #openstack-cinder | 20:06 | |
jgriffith | ntpttr: one thing if you're looking for something else to dive in to :) | 20:06 |
jgriffith | ntpttr: it would appear that we have never implemented a type-show in the client :( | 20:07 |
*** IlyaG has joined #openstack-cinder | 20:07 | |
*** merooney has quit IRC | 20:09 | |
ntpttr | jgriffith: Would that be something that can be done right now? I implemented a client change a bit ago and it got pushed back https://review.openstack.org/#/c/230622/ | 20:10 |
ntpttr | jgriffith: I'd be happy to work on that if it's an option! | 20:10 |
*** haypo has left #openstack-cinder | 20:10 | |
jgriffith | ntpttr: Yeah, I think your timing may have just been off on that other one | 20:10 |
jgriffith | ntpttr: jungleboyj should clear that -2 btw | 20:10 |
*** Yogi1 has quit IRC | 20:12 | |
jgriffith | ntpttr: https://bugs.launchpad.net/python-cinderclient/+bug/1505806 | 20:12 |
openstack | Launchpad bug 1505806 in python-cinderclient "cinderclient has no implementation for type-show" [Undecided,New] | 20:12 |
ntpttr | jgriffith: All right, is there a launchpad report for the type-show implementation? | 20:12 |
ntpttr | jgriffith: that answers that question | 20:12 |
*** Yogi1 has joined #openstack-cinder | 20:12 | |
jgriffith | LOL | 20:12 |
ntpttr | jgriffith: haha well sounds good! Thank you | 20:13 |
*** porrua has quit IRC | 20:14 | |
*** Lee1092 has quit IRC | 20:14 | |
jungleboyj | ntpttr: jgriffith -2 cleared. | 20:15 |
smcginnis | DuncanT: Thanks, been waiting to gather up all the problem CIs and notify them all. | 20:16 |
smcginnis | DuncanT: We have several more than just them unfortunately. | 20:16 |
jungleboyj | ntpttr: That patch still needs a cross project dependency, doesn't it? | 20:16 |
DuncanT | smcginnis: Yeah, I do an occasional round when I notice one has been failing a lot. I don't really check the stats very often though | 20:17 |
ntpttr | jungleboyj: between the cinderclient and cinder you mean? Yeah I can update the commit message | 20:17 |
jgriffith | jungleboyj: thanks! | 20:18 |
jungleboyj | Ok, that would be good. | 20:18 |
jungleboyj | jgriffith: Welcome. Thanks for the pointer. | 20:18 |
jgriffith | DuncanT: you know about this right? http://ci-watch.tintri.com/project?project=cinder&time=7+days | 20:18 |
jgriffith | DuncanT: if not, you might find it useful/handy | 20:18 |
smcginnis | Hmm, those EMC ones are pretty spotty. Maybe I should remove them. | 20:20 |
smcginnis | Joke! | 20:20 |
jgriffith | smcginnis: LOL | 20:20 |
jgriffith | smcginnis: that's a pretty good one | 20:20 |
jungleboyj | He he he. | 20:20 |
* jungleboyj is going to laugh when smcginnis has to drink the EMC kool-aid. | 20:20 | |
jgriffith | smcginnis: you better watch it though... ya never know xyang1 may be your boss at some point :) | 20:20 |
smcginnis | jgriffith: No doubt. :) | 20:20 |
*** ronis has quit IRC | 20:20 | |
xyang1 | jgriffith: what?:) | 20:20 |
jungleboyj | jgriffith: I had the same thought. | 20:21 |
jgriffith | xyang1: LOL... I thought that might get your attention :) | 20:21 |
DuncanT | smcginnis: I had forgotten about that, thanks for the reminder | 20:21 |
jgriffith | xyang1: just joking around because you and smcginnis will be working for the same company | 20:21 |
xyang1 | jgriffith: I guess anything could happen, it could be the other way around too | 20:21 |
jgriffith | xyang1: ya never know | 20:21 |
DuncanT | Wow, none of the CIs look great, TBH | 20:22 |
jgriffith | DuncanT: except mine :) | 20:22 |
jgriffith | DuncanT: mine is awesome LOL | 20:22 |
jungleboyj | DuncanT: Had a meeting about that here this morning. Sending out angry grams today. | 20:22 |
DuncanT | jgriffith: Needs more cowbell | 20:22 |
*** e0ne has joined #openstack-cinder | 20:22 | |
jungleboyj | Again. | 20:22 |
jgriffith | DuncanT: LOL | 20:23 |
jgriffith | DuncanT: the only thing screwing mine up at this point are the dependency tags :( | 20:24 |
jgriffith | DuncanT: http://i.imgur.com/gMRjnK2.jpg | 20:25 |
jgriffith | DuncanT: I really think we need a filter... to show comparison between things that passed jenkins and the 3'rd party CI's | 20:25 |
smcginnis | jgriffith: I prototyped pulling in dependencies. | 20:25 |
smcginnis | It kinda sorta worked. | 20:25 |
DuncanT | jgriffith: Agreed | 20:25 |
smcginnis | Got a little unstable though. | 20:25 |
jgriffith | smcginnis: well, that's more than what I did :) | 20:25 |
jgriffith | DuncanT: smcginnis I think the key metric is the disagreement with Jenkins on passed runs | 20:26 |
smcginnis | jgriffith: On the never ending TODO list to get back to at some point. | 20:26 |
DuncanT | jgriffith: Or even just ignore any patch where more than half the CIs failed? | 20:26 |
smcginnis | jgriffith, DuncanT: Yeah, I'd like to have an easy report that shows just that. | 20:26 |
jgriffith | DuncanT: smcginnis especially after days like today and yesterday where stuff has an extremely high fail rate | 20:26 |
jgriffith | smcginnis: DuncanT I started putting something like that together as an add to Joe Gordons last-comment.... | 20:26 |
jgriffith | smcginnis: DuncanT maybe I'll try and pick that back up again | 20:27 |
DuncanT | jgriffith: Awesome | 20:27 |
smcginnis | jgriffith: That would be cool! | 20:27 |
openstackgerrit | Lukas Bezdicka proposed openstack/os-brick: Remove brackets from path https://review.openstack.org/234425 | 20:29 |
openstackgerrit | Nate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance https://review.openstack.org/230622 | 20:32 |
*** bill_az has quit IRC | 20:33 | |
*** e0ne has quit IRC | 20:35 | |
*** Yogi1 has quit IRC | 20:41 | |
*** qeas has joined #openstack-cinder | 20:41 | |
*** haomaiwang has joined #openstack-cinder | 20:41 | |
qeas | hi everyone, can somebody point where can I see CI results from different vendors? | 20:42 |
*** bswartz has quit IRC | 20:42 | |
*** p0rtal_ has joined #openstack-cinder | 20:42 | |
apoorvad | qeas : http://ci-watch.tintri.com/project?project=cinder | 20:43 |
*** SZhang has joined #openstack-cinder | 20:44 | |
*** ntpttr has quit IRC | 20:44 | |
SZhang | Hi, one question: when a volume is created from an existing volume with -source-volid, do both of them need to be in the same volume type? In other words, can the volume be cloned from other volume-type? | 20:45 |
*** gouthamr has quit IRC | 20:46 | |
*** p0rtal has quit IRC | 20:46 | |
*** gouthamr has joined #openstack-cinder | 20:46 | |
*** gouthamr has quit IRC | 20:47 | |
*** DericHorn-HP has quit IRC | 20:47 | |
*** haomaiwang has quit IRC | 20:47 | |
*** martyturner has quit IRC | 20:48 | |
qeas | apoorvad : thanks! | 20:48 |
*** martyturner has joined #openstack-cinder | 20:49 | |
*** timcl has quit IRC | 20:49 | |
*** vgridnev has quit IRC | 20:57 | |
*** raildo is now known as raildo-afk | 21:00 | |
*** vgridnev has joined #openstack-cinder | 21:01 | |
*** subscope has quit IRC | 21:02 | |
*** salv-orlando has joined #openstack-cinder | 21:02 | |
*** vgridnev has quit IRC | 21:03 | |
*** vgridnev has joined #openstack-cinder | 21:04 | |
smcginnis | all: Last patch needed to clear up issues merged. OK to recheck and approve things again. | 21:05 |
SZhang | Anyone would like to answer my question? Can the volume be cloned from the volume in other volume-type with -source-volid? | 21:12 |
*** crose has quit IRC | 21:13 | |
*** dustins has quit IRC | 21:14 | |
*** ntpttr has joined #openstack-cinder | 21:21 | |
*** bill_az has joined #openstack-cinder | 21:28 | |
*** jgregor has quit IRC | 21:31 | |
*** qeas has quit IRC | 21:32 | |
*** vgridnev has quit IRC | 21:32 | |
*** jgregor has joined #openstack-cinder | 21:33 | |
*** adrianofr has quit IRC | 21:36 | |
*** mc_nair has quit IRC | 21:38 | |
*** julim has quit IRC | 21:39 | |
*** eharney has quit IRC | 21:40 | |
*** baumann has quit IRC | 21:44 | |
*** jungleboyj has quit IRC | 21:45 | |
*** gouthamr has joined #openstack-cinder | 21:46 | |
*** mriedem has quit IRC | 21:48 | |
*** dave-mccowan has quit IRC | 21:56 | |
*** daneyon has joined #openstack-cinder | 21:58 | |
*** ntpttr has quit IRC | 22:01 | |
*** lprice2 has joined #openstack-cinder | 22:06 | |
*** lprice1 has quit IRC | 22:06 | |
*** IlyaG_ has joined #openstack-cinder | 22:06 | |
*** edtubill has quit IRC | 22:07 | |
*** IlyaG has quit IRC | 22:09 | |
*** david-lyle has quit IRC | 22:10 | |
*** david-lyle has joined #openstack-cinder | 22:12 | |
*** martyturner has quit IRC | 22:13 | |
*** jungleboyj has joined #openstack-cinder | 22:16 | |
*** tpeoples has quit IRC | 22:18 | |
*** xyang1 has quit IRC | 22:19 | |
*** lcurtis has quit IRC | 22:23 | |
*** kevincarr1991 has joined #openstack-cinder | 22:24 | |
*** gouthamr_ has joined #openstack-cinder | 22:29 | |
jgriffith | SZhang: you'll need to clone with no type... then retype sadly | 22:30 |
jgriffith | SZhang: it's kind of weird... but the reason behind it is that you don't *know* if the type will be compatible or not... we should probably fix that in M (fail early with incompatible type) | 22:31 |
jgriffith | SZhang: honestly it's been a bit of a long running debate | 22:31 |
*** gouthamr has quit IRC | 22:32 | |
*** edmondsw has quit IRC | 22:33 | |
jgriffith | SZhang: Ohhh... yipee! | 22:33 |
jgriffith | SZhang: that's been changed | 22:33 |
jgriffith | SZhang: you can in fact | 22:33 |
jgriffith | SZhang: although I'm curious why you're asking instead of just trying it :) | 22:34 |
*** IanGovett has quit IRC | 22:34 | |
jgriffith | SZhang: there are caveats, like the types have to be compatible and both specify the same volume-backend | 22:34 |
*** daneyon has quit IRC | 22:41 | |
*** IanGovett has joined #openstack-cinder | 22:42 | |
*** willsama has joined #openstack-cinder | 22:43 | |
*** haomaiwang has joined #openstack-cinder | 22:44 | |
*** edtubill has joined #openstack-cinder | 22:45 | |
Swanson | jgriffith, Is replication_failover a required method? | 22:46 |
jgriffith | Swanson: for replication it will be | 22:47 |
jgriffith | Swanson: I suppose that's debatable in some views | 22:47 |
Swanson | jgriffith, well, I do wonder about the state if leaves you in. If I'm trying to recover a system I might not want the body moved. | 22:48 |
jgriffith | Swanson: well... on the other hand if you're using replication :) | 22:48 |
jgriffith | Swanson: so keep in mind that's supposed to be an admin that does it anyway | 22:48 |
*** haomaiwang has quit IRC | 22:50 | |
*** angela-s has quit IRC | 22:51 | |
Swanson | jgriffith, True. I'd probably break the replication at the device and import to the preferred backend. I kind of wonder if import volume shouldn't be doing the failover. | 22:51 |
jgriffith | Swanson: funny, I was just talking to somebody this morning about that | 22:51 |
jgriffith | Swanson: in essence that's what you'll end up doing | 22:52 |
jgriffith | Swanson: but in that case... you have a "managed" backend | 22:52 |
jgriffith | Swanson: so it's the same for some to just redirect the host column | 22:52 |
jgriffith | Swanson: keep in mind that manage/unmanage doesn't preserve the volume object/id being used though | 22:53 |
jgriffith | Swanson: which might not be so bueano | 22:53 |
Swanson | jgriffith, True. Volume could get lost. | 22:54 |
jgriffith | Swanson: well... the bigger problem is that when you fail-over or whatever it's a "new" volume | 22:54 |
Swanson | jgriffith, In the it has a new cinder id way or in the it has a new scsi id way? Cause the latter is certainly true for us. The former would be true if manage was used vs failover. | 22:55 |
jgriffith | Swanson: cinder ID | 22:56 |
jgriffith | Swanson: scsi ID is a given | 22:56 |
jgriffith | Swanson: but that's transparrant to users anyway so who cares :) | 22:57 |
Swanson | jgriffith, seems like to keep the ID (and we name everything after the GUID) failover is better than manage. | 22:59 |
jgriffith | Swanson: right | 22:59 |
jgriffith | Swanson: I completely agree | 22:59 |
*** chlong has quit IRC | 22:59 | |
Swanson | jgriffith, If I fail to another backend that is supposed to have replication (say to the first backend that just failed) how do we kick off the replication once the world has been fixed? | 23:01 |
jgriffith | Swanson: depends on what your backend/driver can do | 23:01 |
jgriffith | Swanson: either | 23:01 |
jgriffith | A: issue fail-over command and it toggles over | 23:01 |
jgriffith | B. Admin doe ssomething in the background to make things right again | 23:02 |
Swanson | jgriffith, B. For now. | 23:02 |
jgriffith | Swanson: I think that's a perfectly acceptable answer :) | 23:03 |
Swanson | jgriffith, I think the idea is to get a basic replication structure in place. Then people can argue about expanding it out. Seems like recovery is something that individual vendors need to sort out. | 23:03 |
*** kevincarr1991 has quit IRC | 23:05 | |
*** gouthamr_ has quit IRC | 23:06 | |
Swanson | jgriffith, thanks for the help. time for dinner. | 23:06 |
jgriffith | Swanson: no problem.. enjoy! | 23:07 |
*** willsama has quit IRC | 23:08 | |
*** willsama has joined #openstack-cinder | 23:13 | |
*** sgotliv_ has quit IRC | 23:16 | |
*** IlyaG_ has quit IRC | 23:18 | |
*** salv-orlando has quit IRC | 23:19 | |
*** salv-orlando has joined #openstack-cinder | 23:20 | |
*** stevemar_ has quit IRC | 23:23 | |
*** stevemar_ has joined #openstack-cinder | 23:24 | |
*** I has joined #openstack-cinder | 23:25 | |
*** I is now known as Guest90194 | 23:25 | |
openstackgerrit | Nate Potter proposed openstack/cinder: Add qos_specs_id to volume type show https://review.openstack.org/234414 | 23:26 |
*** salv-orlando has quit IRC | 23:27 | |
*** salv-orlando has joined #openstack-cinder | 23:28 | |
*** salv-orlando has quit IRC | 23:31 | |
*** salv-orlando has joined #openstack-cinder | 23:32 | |
*** Guest90194 has quit IRC | 23:33 | |
*** salv-orlando has quit IRC | 23:34 | |
*** cdelatte has joined #openstack-cinder | 23:34 | |
*** delattec has joined #openstack-cinder | 23:34 | |
*** gouthamr has joined #openstack-cinder | 23:38 | |
*** delatte has joined #openstack-cinder | 23:39 | |
*** zhangjn has joined #openstack-cinder | 23:39 | |
*** jwcroppe has quit IRC | 23:39 | |
*** zhangjn has quit IRC | 23:39 | |
*** rhedlind has quit IRC | 23:39 | |
*** zhangjn has joined #openstack-cinder | 23:40 | |
*** dims has quit IRC | 23:41 | |
*** delattec has quit IRC | 23:41 | |
*** cdelatte has quit IRC | 23:42 | |
*** dims has joined #openstack-cinder | 23:42 | |
*** r-daneel has quit IRC | 23:43 | |
*** haomaiwang has joined #openstack-cinder | 23:46 | |
*** delattec has joined #openstack-cinder | 23:46 | |
*** delatte has quit IRC | 23:50 | |
*** haomaiwang has quit IRC | 23:50 | |
*** smoriya has joined #openstack-cinder | 23:52 | |
*** IanGovett has quit IRC | 23:52 | |
*** delattec has quit IRC | 23:55 | |
*** willsama has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!