Tuesday, 2015-10-13

jgriffithmucking around with the DB should not be taken lightly :)00:01
*** IlyaG has joined #openstack-cinder00:02
lifelessmriedem: no00:02
lifelessmriedem: stack_info says 'always provide a call stack'00:02
lifelessmriedem: the original code only provides a call stack when an exception is present00:02
*** pckizer_ is now known as pckizer00:03
*** opal has joined #openstack-cinder00:04
*** eharney has quit IRC00:05
*** lixiaoy1 has joined #openstack-cinder00:06
jgriffithhmm.. 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
mriedemlifeless: ah ok00:07
dims_jgriffith: mriedem was patching something up00:09
dims_i am working another angle now00:09
mriedemjgriffith: cinder already has a fix in tree for the webob 1.5.0 issue00:09
mriedemso shouldn't be the same as the nova issue00:09
mriedemjgriffith: https://github.com/openstack/cinder/commit/867fccf833ffc597aa986cb6ff1b3b5c1101b9ba00:10
jgriffithyeah, I saw it just merged00:11
jgriffithsorry, my bad00:11
jgriffithmriedem: not just either00:11
*** salv-orl_ has quit IRC00:11
jgriffithquite a while ago.... I was looking at the backports, for which the webob has not merged yet00:11
jgriffithmriedem: I vote we abolish stable branches and just have master from now on :)00:12
jgriffithis gerrit painfully slow for others today?00:13
*** EinstCrazy has quit IRC00:14
*** lixiaoy11 has joined #openstack-cinder00:17
*** opal has left #openstack-cinder00:18
*** lixiaoy1 has quit IRC00:19
*** stevemar_ has quit IRC00:25
*** tpeoples has quit IRC00:27
*** DericHorn-HP has joined #openstack-cinder00:28
*** stevemar_ has joined #openstack-cinder00:29
*** garthb has quit IRC00:29
ntpttrjgriffith: 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/console00:29
*** annasort has quit IRC00:42
*** apoorvad has quit IRC00:44
jgriffithntpttr: nah, I'm just waiting for it to finish and hit recheck again00:48
jgriffithntpttr: the py34 failure is an intermittent one that eharney spotted a while back :(00:48
jgriffithntpttr: just sucks the cycle time is 1.5 hours :(00:49
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/23286800:49
*** tobe has joined #openstack-cinder00:49
jgriffithhaha!00:49
jgriffithGo figure00:49
jgriffith^^ that proposal will fix things up if it passes :)00:50
jgriffithntpttr: It's officially a race :)00:51
ntpttrjgriffith: hah I'll go get my popcorn00:52
jgriffithhehe00:52
ntpttrdang bot taking my first patch that was going to get merged to cinder :P00:56
*** kevincarr1991 has joined #openstack-cinder00:56
*** tpeoples has joined #openstack-cinder00:56
jgriffithntpttr: who would've thought it would take over 8 hours to move something :(00:56
*** EinstCrazy has joined #openstack-cinder00:59
ntpttrjgriffith: ah well, next time00:59
*** leeantho has quit IRC01:06
*** DericHorn-HP has quit IRC01:08
*** ntpttr has left #openstack-cinder01:12
thingeesmcginnis: ping01:18
*** IlyaG has quit IRC01:18
*** Lee1092 has joined #openstack-cinder01:29
*** zhenguo has joined #openstack-cinder01:33
*** mtanino has quit IRC01:34
*** DericHorn-HP has joined #openstack-cinder01:36
*** sghanekar_ has quit IRC01:40
*** p0rtal has quit IRC01:41
*** yangyapeng has joined #openstack-cinder01:46
*** davechen has joined #openstack-cinder01:46
*** stevemar_ has quit IRC01:53
*** IlyaG has joined #openstack-cinder01:53
*** haomaiwang has joined #openstack-cinder01:55
*** haomaiwang has quit IRC02:01
*** haomaiwang has joined #openstack-cinder02:01
*** kevincarr1991 has quit IRC02:01
*** lprice has joined #openstack-cinder02:01
*** martyturner has joined #openstack-cinder02:02
*** lprice1 has quit IRC02:04
*** DericHorn-HP has quit IRC02:05
*** lprice1 has joined #openstack-cinder02:05
*** martyturner has quit IRC02:06
*** lprice has quit IRC02:09
*** DericHorn-HP has joined #openstack-cinder02:10
*** jdandrea has quit IRC02:13
*** ntpttr has joined #openstack-cinder02:13
openstackgerritwanghao proposed openstack/cinder: CG creation should be schedule on backend level  https://review.openstack.org/22440502:20
openstackgerritWilson Liu proposed openstack/cinder: Huawei driver handle volume exists error  https://review.openstack.org/23333302:26
*** chenying has joined #openstack-cinder02:27
*** links has joined #openstack-cinder02:28
*** dims_ has quit IRC02:28
*** DericHorn-HP has quit IRC02:34
*** salv-orlando has joined #openstack-cinder02:39
*** DericHorn-HP has joined #openstack-cinder02:39
*** mriedem has quit IRC02:39
*** dims__ has joined #openstack-cinder02:41
*** stevemar_ has joined #openstack-cinder02:42
*** p0rtal has joined #openstack-cinder02:46
*** chenying has quit IRC02:46
*** salv-orlando has quit IRC02:47
*** bill_az has quit IRC02:56
*** yangyapeng has quit IRC02:57
*** dims__ has quit IRC02:59
*** haomaiwang has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** salv-orlando has joined #openstack-cinder03:08
*** ntpttr has left #openstack-cinder03:09
*** dave-mccowan has quit IRC03:10
*** chenying has joined #openstack-cinder03:15
*** salv-orlando has quit IRC03:17
openstackgerritxing-yang proposed openstack/cinder-specs: Backup snapshots  https://review.openstack.org/23230503:20
*** mdbooth has quit IRC03:22
*** lpabon has joined #openstack-cinder03:24
*** mdbooth has joined #openstack-cinder03:28
*** zhangjn has quit IRC03:32
*** p0rtal_ has joined #openstack-cinder03:39
openstackgerritWenPing Zhang proposed openstack/cinder: New cinder driver to support Daowoo YeeSan  https://review.openstack.org/20244003:40
*** haomaiwang has quit IRC03:41
*** p0rtal has quit IRC03:42
*** lpabon has quit IRC03:44
*** haomaiwang has joined #openstack-cinder03:45
*** gouthamr_ has quit IRC03:46
openstackgerritJohn Griffith proposed openstack/cinder: Change default Exception code to 500  https://review.openstack.org/23392303:46
jgriffithsmcginnis: gimmie a shout in the AM, need to check in with you regarding these changes and Liberty03:47
jgriffithsmcginnis: but for now, I'm going to have a beer :)03:47
*** jwcroppe has joined #openstack-cinder03:49
*** 32NAAJR1K has quit IRC03:54
*** links has quit IRC03:58
*** dims__ has joined #openstack-cinder03:59
*** harlowja_at_home has joined #openstack-cinder04:00
*** p0rtal_ has quit IRC04:00
*** haomaiwang has quit IRC04:01
*** 20WAB9N4Y has joined #openstack-cinder04:01
*** zhangjn has joined #openstack-cinder04:03
*** dims__ has quit IRC04:06
openstackgerritJohn Griffith proposed openstack/cinder: Update config format for replication_devices  https://review.openstack.org/23330704:09
*** davechen is now known as davechen_afk04:09
*** vgridnev has joined #openstack-cinder04:13
*** salv-orlando has joined #openstack-cinder04:15
*** opal has joined #openstack-cinder04:19
*** salv-orlando has quit IRC04:19
*** chenying has quit IRC04:26
*** lixiaoy11 has quit IRC04:28
*** dobson has quit IRC04:30
*** opal has left #openstack-cinder04:31
*** pradipta has joined #openstack-cinder04:33
*** salv-orlando has joined #openstack-cinder04:36
*** dobson has joined #openstack-cinder04:36
*** jgregor has quit IRC04:40
*** jgregor has joined #openstack-cinder04:40
*** salv-orlando has quit IRC04:40
*** haigang has joined #openstack-cinder04:41
*** dosaboy has quit IRC04:42
*** skraynev has quit IRC04:42
*** haigang has quit IRC04:42
*** dosaboy has joined #openstack-cinder04:43
*** skraynev has joined #openstack-cinder04:43
*** harlowja_at_home has quit IRC04:45
*** avishay has joined #openstack-cinder04:47
*** jgregor has quit IRC04:52
*** 20WAB9N4Y has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** avishay has quit IRC05:04
*** avishay_ has joined #openstack-cinder05:04
*** links has joined #openstack-cinder05:06
*** avishay_ is now known as avishay05:12
*** vgridnev has quit IRC05:12
*** lixiaoy1 has joined #openstack-cinder05:12
*** breitz has quit IRC05:13
*** breitz has joined #openstack-cinder05:14
*** DericHorn-HP has quit IRC05:17
*** lixiaoy11 has joined #openstack-cinder05:17
*** lixiaoy1 has quit IRC05:20
*** sgotliv_ has joined #openstack-cinder05:22
*** davechen_afk is now known as davechen05:23
*** sgotliv has quit IRC05:24
*** p0rtal has joined #openstack-cinder05:27
*** vgridnev has joined #openstack-cinder05:27
*** breitz has quit IRC05:35
*** breitz has joined #openstack-cinder05:36
*** tobe has quit IRC05:39
*** daneyon has joined #openstack-cinder05:40
*** daneyon has quit IRC05:41
openstackgerritLisaLi proposed openstack/cinder: Create encrypted volumes from images  https://review.openstack.org/21656705:41
*** flaper87 has joined #openstack-cinder05:44
*** flaper87 has left #openstack-cinder05:44
*** nkrinner has joined #openstack-cinder05:46
*** haomaiwang has quit IRC06:01
*** IlyaG has quit IRC06:01
*** haomaiwang has joined #openstack-cinder06:01
*** stevemar_ has quit IRC06:03
*** dims__ has joined #openstack-cinder06:03
*** p0rtal has quit IRC06:04
*** dims__ has quit IRC06:09
*** haomaiwang has quit IRC06:10
*** sgotliv_ has quit IRC06:12
*** haomaiwang has joined #openstack-cinder06:13
*** vgridnev has quit IRC06:15
*** ronis has joined #openstack-cinder06:17
*** links has quit IRC06:20
*** openstack has joined #openstack-cinder06:28
*** vlaza has joined #openstack-cinder06:28
*** tsufiev has joined #openstack-cinder06:29
*** yrabl has joined #openstack-cinder06:30
*** daneyon has joined #openstack-cinder06:34
openstackgerritYusuke Hayashi proposed openstack/cinder: Case sensitivity problem in cinder scheduler  https://review.openstack.org/22502406:36
*** lixiaoy11 has quit IRC06:37
*** ankit_ag has joined #openstack-cinder06:37
*** lixiaoy1 has joined #openstack-cinder06:38
*** daneyon has quit IRC06:39
*** jwcroppe has quit IRC06:44
*** jwcroppe has joined #openstack-cinder06:44
*** wanghao has joined #openstack-cinder06:45
*** jwcroppe has quit IRC06:45
*** haomaiwang has quit IRC06:47
*** 18VAAUGO0 has joined #openstack-cinder06:48
*** sgotliv_ has quit IRC06:52
openstackgerritAlon Marx proposed openstack/cinder: Implement changes in the consistency_group handling in the xiv/ds8k driver.  https://review.openstack.org/23328006:54
*** vgridnev has quit IRC06:55
*** e0ne has joined #openstack-cinder06:58
*** 18VAAUGO0 has quit IRC06:58
*** e0ne has quit IRC06:59
*** chenying has joined #openstack-cinder07:02
*** haomaiwang has joined #openstack-cinder07:02
openstackgerritYusuke Hayashi proposed openstack/cinder: Case sensitivity problem in cinder scheduler  https://review.openstack.org/22502407:03
*** dims__ has joined #openstack-cinder07:05
*** tpeoples has quit IRC07:08
*** vincent_hou has joined #openstack-cinder07:09
*** dims__ has quit IRC07:10
*** links has joined #openstack-cinder07:13
*** ndipanov has quit IRC07:13
*** e0ne has joined #openstack-cinder07:16
openstackgerritdongwenjuan proposed openstack/python-cinderclient: Support forcing service down  https://review.openstack.org/23400107:17
*** p0rtal has joined #openstack-cinder07:18
openstackgerritWilson Liu proposed openstack/cinder: Brocade driver add_zone optimization  https://review.openstack.org/23400407:20
openstackgerritVictor Stinner proposed openstack/cinder: Port API types extra specs to Python 3  https://review.openstack.org/23221607:21
openstackgerritVictor Stinner proposed openstack/cinder: Port WSGI tests to Python 3  https://review.openstack.org/23221407:21
openstackgerritVictor Stinner proposed openstack/cinder: Port API to Python 3  https://review.openstack.org/23221507:21
openstackgerritVictor Stinner proposed openstack/cinder: Port API admin action tests to Python 3  https://review.openstack.org/23224207:21
*** haypo has joined #openstack-cinder07:23
*** lpetrut has joined #openstack-cinder07:27
*** p0rtal has quit IRC07:28
*** anshul has joined #openstack-cinder07:28
*** daneyon has joined #openstack-cinder07:29
*** xek_ is now known as xek07:29
vincent_houavishay: hi.07:29
*** anshul has quit IRC07:32
*** anshul has joined #openstack-cinder07:33
*** daneyon has quit IRC07:34
*** vgridnev has joined #openstack-cinder07:37
*** ronis has joined #openstack-cinder07:41
*** e0ne has quit IRC07:44
avishayvincent_hou: hey, what's up?07:47
vincent_houavishay: How are you? Are you going to Tokyo?07:49
avishayvincent_hou: i am well, thanks.  yes, i will be at the tokyo summit.07:50
vincent_houavishay: ok . we can meet there.07:50
*** subscope has joined #openstack-cinder07:52
*** salv-orlando has joined #openstack-cinder07:59
vincent_hougeguileo: 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_hougeguileo: When creating a volume, a default volume type will be assigned, if the volume type is not specified.07:59
geguileovincent_hou: Really? r:-??08:00
vincent_hougeguileo: 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 IRC08:01
vincent_hougeguileo: I will point you the code.08:01
*** haomaiwang has joined #openstack-cinder08:01
geguileovincent_hou: I mean, I have been creating volumes and migrating them without a type as long as I can remember08:01
*** markus_z has joined #openstack-cinder08:01
*** jordanP has joined #openstack-cinder08:02
geguileovincent_hou: Which of my comments are we referring to?08:02
DuncanTgeguileo: What do they end up with in the type field in the DB?08:02
geguileoDuncanT: I think they had NULL on the type08:03
*** salv-orlando has quit IRC08:03
*** jistr has joined #openstack-cinder08: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_hougeguileo: 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-cinder08:06
vincent_hougeguileo: DuncanT: I am doubting that we can still have volume with NULL volume type. I need opinions from others.08:06
DuncanTvincent_hou: I guess they can still be in the DB from old volume, unless we've at some point fixed it up in a migration08:07
openstackgerritWilson Liu proposed openstack/cinder: Add hypermetro support for Huawei driver  https://review.openstack.org/20202308:07
DuncanTvincent_hou: Try not setting a default type in the config08:07
vincent_houDuncanT: OH. That could be an option.08:08
*** links has quit IRC08:08
geguileovincent_hou: I'll try testing it again today and come back to you08:09
vincent_houDuncanT: I will verify the case without the default type. Thx.08:10
vincent_hougeguileo: Thank you.08:10
DuncanTvincent_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 IRC08:11
openstackgerritWilson Liu proposed openstack/cinder: Huawei driver handle volume exists error  https://review.openstack.org/23333308:12
vincent_houDuncanT: Yeah. I think it is good topic to investigate.08:13
*** sgotliv_ has joined #openstack-cinder08:16
*** links has joined #openstack-cinder08:18
*** svasheka has joined #openstack-cinder08:18
*** ndipanov has joined #openstack-cinder08:21
*** daneyon has joined #openstack-cinder08:23
*** subscope has quit IRC08:23
*** links has quit IRC08:24
*** subscope has joined #openstack-cinder08:24
*** salv-orlando has joined #openstack-cinder08:26
*** daneyon has quit IRC08:28
*** links has joined #openstack-cinder08:31
*** lixiaoy1 has quit IRC08:33
haypohi. 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? :-p08:34
haypo(it's part of a serie of 4 changes)08:34
*** davechen has quit IRC08:34
haypoi 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-cinder08:47
vincent_hougeguileo: I tested volume migration for the volume with NULL volume type and it worked for cinder migration, but not retype.08:47
vincent_hougeguileo: 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 84c7eb008:47
vincent_houc-c6cd-4d36-b357-c8e07d1be6e2 to 87b45163-a9c8-431e-a653-69c24811001208:47
vincent_hou2015-10-13 16:45:10.699 ERROR oslo_messaging.rpc.dispatcher [req-1f42a2f3-36d8-4883-85f4-09b11cf5959f None None] Exception during mess08:47
vincent_houage handling: Instance <VolumeTypes at 0x7fdada59cfd0> has a NULL identity key.  If this is an auto-generated value, check that the da08:47
vincent_houtabase table allows generation of new primary key values, and that the mapped Column object is configured to expect these generated va08:47
vincent_houlues.  Ensure also that this flush() is not occurring at an inappropriate time, such aswithin a load() event.08:47
vincent_hou2015-10-13 16:45:10.699 TRACE oslo_messaging.rpc.dispatcher Traceback (most recent call last):08:47
vincent_hou2015-10-13 16:45:10.699 TRACE oslo_messaging.rpc.dispatcher08:47
vincent_hougeguileo: the retype of a volume from null to lvm(for example) fails during IMPL.finish_volume_migration08:48
geguileovincent_hou: I'm on a meeting, give me 10 minutes, please08:49
vincent_hougeguileo: Cool. take your time.08:50
*** mdbooth has joined #openstack-cinder08:51
*** BharatK_ has quit IRC08:52
*** bluex-pl has quit IRC08:54
*** bluex-pl has joined #openstack-cinder08:54
*** hideme_ has quit IRC08:54
*** hideme_ has joined #openstack-cinder08:54
*** links has quit IRC08:54
*** links has joined #openstack-cinder08:54
*** markus_z has quit IRC08:54
*** markus_z has joined #openstack-cinder08:54
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver  https://review.openstack.org/22229208:56
geguileovincent_hou: That is bad :-(08:59
geguileovincent_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-cinder09:00
*** wenjuan has quit IRC09:00
*** wenjuan has joined #openstack-cinder09:00
*** haomaiwang has quit IRC09:01
*** wenjuan has quit IRC09:01
*** haomaiwang has joined #openstack-cinder09:01
*** wenjuan has joined #openstack-cinder09:01
wenjuanhttps://review.openstack.org/#/c/23400109:02
*** wenjuan has quit IRC09:02
*** wenjuan has joined #openstack-cinder09:02
*** wenjuan has quit IRC09:03
*** wenjuan has joined #openstack-cinder09:03
*** subscope has quit IRC09:03
*** wenjuan has quit IRC09:04
*** wenjuan has joined #openstack-cinder09:04
*** wenjuan has quit IRC09:04
*** wenjuan has joined #openstack-cinder09:05
vincent_hougeguileo: 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 IRC09:05
geguileovincent_hou: Thank you for working on that much needed document!!09:05
*** BharatK_ has joined #openstack-cinder09:06
vincent_hougeguileo: Well, just need your verification of this issue. I logged it here anyway: https://bugs.launchpad.net/cinder/+bug/1505572.09:06
openstackLaunchpad 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 IRC09:07
*** wenjuan has joined #openstack-cinder09:07
*** wenjuan has quit IRC09:07
*** dims__ has joined #openstack-cinder09:08
*** avishay_ has joined #openstack-cinder09:08
*** wenjuan has joined #openstack-cinder09:08
*** wenjuan has quit IRC09:08
*** avishay has quit IRC09:09
*** wenjuan has joined #openstack-cinder09:09
*** wenjuan has quit IRC09:09
*** wenjuan has joined #openstack-cinder09:10
*** wenjuan has quit IRC09:11
*** wenjuan has joined #openstack-cinder09:11
*** wenjuan has quit IRC09:11
*** wenjuan has joined #openstack-cinder09:12
*** wenjuan has quit IRC09:12
*** dims__ has quit IRC09:13
*** wenjuan has joined #openstack-cinder09:13
*** wenjuan has quit IRC09:13
*** wenjuan has joined #openstack-cinder09:14
*** wenjuan has quit IRC09:14
*** links has quit IRC09:15
*** wenjuan has joined #openstack-cinder09:15
*** subscope has joined #openstack-cinder09:15
*** wenjuan has quit IRC09:16
*** wenjuan has joined #openstack-cinder09:16
*** wenjuan has quit IRC09:17
*** daneyon has joined #openstack-cinder09:17
*** wenjuan has joined #openstack-cinder09:17
*** wenjuan has quit IRC09:17
*** wenjuan has joined #openstack-cinder09:18
*** wenjuan has quit IRC09:18
*** wenjuan has joined #openstack-cinder09:19
*** wenjuan has quit IRC09:19
*** wenjuan has joined #openstack-cinder09:20
*** wenjuan has quit IRC09:21
*** wenjuan has joined #openstack-cinder09:21
*** wenjuan has quit IRC09:21
*** daneyon has quit IRC09:22
*** wenjuan has joined #openstack-cinder09:22
*** wenjuan has quit IRC09:22
*** wenjuan has joined #openstack-cinder09:23
*** wenjuan has quit IRC09:24
*** wenjuan has joined #openstack-cinder09:24
*** wenjuan has quit IRC09:25
*** wenjuan has joined #openstack-cinder09:25
vincent_hougeguileo: 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 IRC09:26
*** wenjuan has joined #openstack-cinder09:26
geguileovincent_hou: Don't know, we should be able to r:-??09:27
*** wenjuan has joined #openstack-cinder09:27
openstackgerritdongwenjuan proposed openstack/python-cinderclient: Support forcing service down  https://review.openstack.org/23400109:27
*** wenjuan has quit IRC09:28
geguileovincent_hou: Wait... It's volume_type_id the one raising the error or volume_type?09:28
*** wenjuan has joined #openstack-cinder09:28
vincent_hougeguileo: id, I think.09:29
vincent_hounot 100% sure.09:29
geguileovincent_hou: I'll have to look, because like you say, it's not "normal"09:29
*** wenjuan has quit IRC09:30
*** wenjuan has joined #openstack-cinder09:30
*** wenjuan has quit IRC09:31
vincent_hougeguileo: 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
openstackLaunchpad 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-cinder09:31
geguileovincent_hou: I'll add my comments to the BP09:31
*** wenjuan has quit IRC09:32
*** vincent_hou has quit IRC09:32
*** EinstCrazy has quit IRC09:35
*** openstackstatus has joined #openstack-cinder09:37
*** ChanServ sets mode: +v openstackstatus09:37
-openstackstatus- NOTICE: gerrit is undergoing an emergency restart to investigate load issues09:41
*** ChanServ changes topic to "gerrit is undergoing an emergency restart to investigate load issues"09:41
openstackgerritdongwenjuan proposed openstack/python-cinderclient: Support forcing service down  https://review.openstack.org/23400109:46
*** dims__ has joined #openstack-cinder09:49
*** BharatK_ has quit IRC09:52
*** zhangjn has quit IRC09:52
*** smoriya has quit IRC09:53
*** davechen has left #openstack-cinder09:56
*** e0ne has joined #openstack-cinder09:58
*** zhenguo has quit IRC10:00
*** haomaiwang has quit IRC10:01
*** openstackgerrit has quit IRC10:01
*** aix has joined #openstack-cinder10:01
*** haomaiwang has joined #openstack-cinder10:01
*** openstackgerrit has joined #openstack-cinder10:02
openstackgerritWenPing Zhang proposed openstack/cinder: New cinder driver to support Daowoo YeeSan  https://review.openstack.org/20244010:03
*** BharatK_ has joined #openstack-cinder10:04
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Fix Status-Line in HTTP response  https://review.openstack.org/23143810:11
*** daneyon has joined #openstack-cinder10:11
*** zhongjun22 has joined #openstack-cinder10:16
*** vgridnev has quit IRC10:16
*** daneyon has quit IRC10:16
*** chenying has quit IRC10:18
*** dims__ is now known as dims10:18
*** haomaiwang has quit IRC10:29
*** haomaiwang has joined #openstack-cinder10:30
*** aix has quit IRC10:30
openstackgerritdongwenjuan proposed openstack/python-cinderclient: Support forcing service down  https://review.openstack.org/23400110:35
*** IanGovett has joined #openstack-cinder10:35
*** yrabl has joined #openstack-cinder10:38
*** aix has joined #openstack-cinder10:42
*** avishay_ has quit IRC10:47
*** edmondsw has joined #openstack-cinder10:52
*** edmondsw has quit IRC10:52
*** subscope has quit IRC10:56
*** jaypipes has joined #openstack-cinder10:59
yrablnames10:59
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/23412810:59
*** pradipta has quit IRC11:00
*** haomaiwang has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
*** p0rtal has joined #openstack-cinder11:02
openstackgerritOpenStack Proposal Bot proposed openstack/python-cinderclient: Updated from global requirements  https://review.openstack.org/23288911:04
*** daneyon has joined #openstack-cinder11:05
*** kevincarr1991 has joined #openstack-cinder11:07
*** avishay_ has joined #openstack-cinder11:08
*** p0rtal has quit IRC11:08
*** daneyon has quit IRC11:10
*** ociuhandu has quit IRC11: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 IRC11:16
-openstackstatus- NOTICE: Gerrit has been restarted and is responding to normal load again.11:16
*** avishay_ has quit IRC11:17
*** merooney has joined #openstack-cinder11:25
*** timcl has joined #openstack-cinder11:30
openstackgerritIvan Berezovskiy proposed openstack/cinder: Fix log formatting for rbd driver  https://review.openstack.org/23415411:31
*** jistr is now known as jistr|biab11:32
*** subscope has joined #openstack-cinder11:38
*** dims has quit IRC11:47
*** dims has joined #openstack-cinder11:48
*** haomaiwang has quit IRC11:52
*** haomaiwang has joined #openstack-cinder11:52
*** ociuhandu has joined #openstack-cinder11:56
openstackgerritDmitry Guryanov proposed openstack/cinder: Fix method volume_types.get_all  https://review.openstack.org/23416711:58
*** cdelatte has quit IRC11:59
*** daneyon has joined #openstack-cinder11:59
*** e0ne has quit IRC12:00
*** haomaiwang has quit IRC12:01
*** haomaiwa_ has joined #openstack-cinder12:01
*** dave-mccowan has joined #openstack-cinder12:01
*** daneyon has quit IRC12:04
*** jgregor has joined #openstack-cinder12:06
*** gouthamr has joined #openstack-cinder12:08
*** ociuhandu has quit IRC12:13
*** salv-orlando has quit IRC12:14
kevincarr1991I can manually mount a nfs share using the mount command, but the nfsdriver for openstack gets permission denied12:16
kevincarr1991anyone else use nfs share for cinder?12:16
DuncanTkevincarr1991: Are you trying the mount as the user cinder is running as?12:17
*** haomaiwa_ has quit IRC12:17
*** haomaiwang has joined #openstack-cinder12:17
kevincarr1991DuncanT: no I tried as root. I thought root was used for the driver12:18
*** cdelatte has joined #openstack-cinder12:18
*** delattec has joined #openstack-cinder12:18
*** martyturner has joined #openstack-cinder12:18
DuncanTkevincarr1991: 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 run12:19
kevincarr1991DuncanT: Sorry I am new to openstack and dont really know how to run it with debug level logging.12:20
kevincarr1991DuncanT: how would i be able to do that?12:20
DuncanTkevincarr1991: Set debug=True and verbose=True in cinder.conf then restart12:20
*** delatte has joined #openstack-cinder12:22
kevincarr1991DuncanT: I did already have verbose=True. I see that a permission denied statement still shows up but I dont get any more info then that12:22
DuncanTkevincarr1991: Can you post the log to paste.openstack.org please?12:23
kevincarr1991DuncanT: can you debug a service when restarting it besides the log files?12:23
kevincarr1991DuncanT: will do12:23
DuncanTThe log files really are the main source of debug12:23
*** cdelatte has quit IRC12:24
*** raildo-afk is now known as raildo12:24
kevincarr1991http://paste.openstack.org/show/476120/12:25
*** avishay_ has joined #openstack-cinder12:26
*** haomaiwang has quit IRC12:29
*** ociuhandu has joined #openstack-cinder12:29
*** ociuhandu_ has joined #openstack-cinder12:31
*** edmondsw has joined #openstack-cinder12:31
*** ociuhandu has quit IRC12:34
*** ociuhandu_ is now known as ociuhandu12:34
*** crose has joined #openstack-cinder12:34
kevincarr1991DuncanT: http://paste.openstack.org/show/476120/12:34
DuncanTLooking....12:34
DuncanTkevincarr1991: "Exception during mounting Unexpected error while running command.12:37
DuncanTCommand: mkdir -p /var/lib/cinder/mnt/17e34a943fe7923d403c10240561727f" - I'm guessing there's a perms issue there somewhere in that directory tree12:37
*** porrua has joined #openstack-cinder12:37
*** delattec has quit IRC12:38
kevincarr1991DuncanT: 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 error12:38
kevincarr1991I assume it is because I am specifying version 312:38
*** zhangjn has joined #openstack-cinder12:39
*** zhangjn has quit IRC12:40
*** zhangjn has joined #openstack-cinder12:40
*** EinstCrazy has joined #openstack-cinder12:40
*** zhangjn has quit IRC12:41
*** zhangjn has joined #openstack-cinder12:41
*** zhangjn has quit IRC12:41
*** zhangjn has joined #openstack-cinder12:42
*** zhangjn has quit IRC12:42
*** zhangjn has joined #openstack-cinder12:43
*** e0ne has joined #openstack-cinder12:46
*** daneyon has joined #openstack-cinder12:53
*** vgridnev has joined #openstack-cinder12:57
*** Yogi1 has joined #openstack-cinder12:57
*** bluex-pl has quit IRC12:58
*** bluex-pl has joined #openstack-cinder12:58
*** daneyon has quit IRC12:58
*** dustins has joined #openstack-cinder13:01
*** p0rtal has joined #openstack-cinder13:02
*** lpetrut has quit IRC13:02
*** haomaiwang has joined #openstack-cinder13:03
*** jistr|biab is now known as jistr13:03
*** p0rtal has quit IRC13:06
*** bluex-pl has quit IRC13:07
*** bluex-pl has joined #openstack-cinder13:08
*** avishay_ has quit IRC13:08
*** alejandrito has joined #openstack-cinder13:09
*** salv-orlando has joined #openstack-cinder13:10
*** adrianofr has joined #openstack-cinder13:10
*** bluex-pl has quit IRC13:11
*** bluex-pl has joined #openstack-cinder13:12
*** gouthamr has quit IRC13:14
*** julim has joined #openstack-cinder13:15
e0nedulek: hi. just FYI, I've posted comments for https://review.openstack.org/#/c/234154/13:15
e0nedulek: unit-tests are already implemented but didn't found this issue13:15
*** bluex-pl has quit IRC13:16
*** bluex-pl has joined #openstack-cinder13:17
*** gouthamr has joined #openstack-cinder13:18
DuncanTe0ne: Is it worth putting in a mock for logging that tests the format string matches the arguments?13:19
DuncanTe0ne: Not necessarily right now, but does that sound useful?13:19
e0neDuncanT: personally, I don't like tests for logging and error messages13:20
e0neDuncanT: such things should be validated with hacking checks13:20
*** stevemar_ has joined #openstack-cinder13:20
*** timcl has quit IRC13:21
*** crose has quit IRC13:23
smcginnisthingee: TTL expired13:25
smcginnisjgriffith: Around now. Ping me when you want to talk.13:25
* smcginnis runs for coffee13:25
*** julim has quit IRC13:25
*** bluex-pl has quit IRC13:26
*** julim has joined #openstack-cinder13:26
*** mc_nair has joined #openstack-cinder13:27
*** lpetrut has joined #openstack-cinder13:27
duleke0ne: Thanks for clarifying.13:28
duleke0ne: I've changed my vote.13:29
e0nedulek: you're welcome! thanks13:29
*** baumann has joined #openstack-cinder13:29
*** baumann has left #openstack-cinder13:29
*** baumann has joined #openstack-cinder13:29
*** bluex-pl has joined #openstack-cinder13:30
*** dave-mccowan has quit IRC13:32
*** eharney has joined #openstack-cinder13:36
*** vgridnev has quit IRC13:37
openstackgerritSean McGinnis proposed openstack/cinder: Only use LOG.exception in exception handler  https://review.openstack.org/23382813:37
jordanPjgriffith, hi. I am going to work on https://bugs.launchpad.net/cinder/+bug/149556013:38
openstackLaunchpad bug 1495560 in Cinder "lvm deactivate race in snapshot extend" [High,Triaged] - Assigned to John Griffith (john-griffith)13:39
*** martyturner has quit IRC13:40
*** martyturner has joined #openstack-cinder13:41
*** vgridnev has joined #openstack-cinder13:41
*** DericHorn-HP has joined #openstack-cinder13:42
*** vgridnev has quit IRC13:43
*** vgridnev has joined #openstack-cinder13:45
openstackgerritDmitry Guryanov proposed openstack/cinder: Fix method VolumeTypeList.get_all  https://review.openstack.org/23416713:46
*** daneyon has joined #openstack-cinder13:48
*** dzamboni has joined #openstack-cinder13:48
*** ntpttr has joined #openstack-cinder13:48
*** flaper87 has joined #openstack-cinder13:48
*** bluex-pl has quit IRC13:48
*** ntpttr has quit IRC13:52
*** daneyon has quit IRC13:53
*** jdandrea has joined #openstack-cinder13:53
*** ntpttr has joined #openstack-cinder13:54
jdandreaRegarding 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
jdandreaTrying to determine if we can rely on the calling order of scheduler filter plugins or not.13:55
*** r-daneel has joined #openstack-cinder13:55
*** cfriesen has joined #openstack-cinder13:56
*** ntpttr has quit IRC13:59
*** DericHorn-HP has quit IRC13:59
jgriffithjordanP: ahh... ok, great13:59
jgriffithjordanP: Shall I reassign it to you?13:59
jordanPjgriffith, yes please14:00
*** martyturner has quit IRC14:00
jgriffithjordanP: cool... done14:00
*** timcl has joined #openstack-cinder14:00
*** haomaiwang has quit IRC14:01
*** martyturner has joined #openstack-cinder14:01
*** haomaiwa_ has joined #openstack-cinder14:01
*** DericHorn-HP has joined #openstack-cinder14:03
*** BharatK_ has quit IRC14:04
*** julim has quit IRC14:05
*** jwcroppe has joined #openstack-cinder14:07
*** julim has joined #openstack-cinder14:07
jordanPjgriffith, I can reproduce easily, locally on my laptop14:08
*** bswartz has joined #openstack-cinder14:08
jgriffithjordanP: excellent14:09
*** jwcroppe has quit IRC14:11
*** jwcroppe has joined #openstack-cinder14:12
*** dave-mccowan has joined #openstack-cinder14:15
e0nesmcginnis: hi. did you see comment for https://review.openstack.org/#/c/234118/?14:16
*** baumann1 has joined #openstack-cinder14:16
*** salv-orlando has quit IRC14:17
e0nesmcginnis: does it mean that I need to squosh 2 commits into the one insteas of cherry-pick?14:17
*** jwcroppe has quit IRC14:17
*** baumann has quit IRC14:19
*** ntpttr has joined #openstack-cinder14:19
*** crose has joined #openstack-cinder14:20
*** jwcroppe has joined #openstack-cinder14:20
*** tpeoples has joined #openstack-cinder14:21
*** haomaiwa_ has quit IRC14:21
*** haomaiwang has joined #openstack-cinder14:21
smcginnise0ne: Just talking in infra about this.14:23
smcginnisI was asked to squash the two and submit.14:23
e0nesmcginnis: thanks. i'm already working on it14:23
e0nesmcginnis: will post a review request in a minute14:24
smcginnisIn meeting, so chances are high I screwed it up, but I just submitted it.14:24
e0nesmcginnis: oh.. thanks a lot! you did it faster than me14:25
smcginnise0ne: 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
e0nesmcginnis: dims, ttx14:26
e0neooops, copy-paste issue:(14:26
smcginnise0ne: ;)14:28
*** baumann1 has quit IRC14:29
*** vlaza has quit IRC14:31
*** bluex-pl has joined #openstack-cinder14:33
*** wilson1 has quit IRC14:34
*** diogogmt has joined #openstack-cinder14:34
*** salv-orlando has joined #openstack-cinder14:34
*** wilson1 has joined #openstack-cinder14:36
*** baumann has joined #openstack-cinder14:37
*** aorourke_afk is now known as aorourke14:38
*** daneyon has joined #openstack-cinder14:42
jgriffithjdurgin: Thanks for jumping on that thread :)14:43
*** xyang1 has joined #openstack-cinder14:43
*** rushil has joined #openstack-cinder14:44
jgriffithpatrickeast: couldn't help but notice after those patches landed you were able to rebase :)14:44
jgriffithpatrickeast: were you able to get all of them?14:44
jgriffithpatrickeast: or is there still anything missing we need to figure out?14:45
*** rushil_ has joined #openstack-cinder14:45
*** daneyon has quit IRC14:47
*** edtubill has joined #openstack-cinder14:47
*** rushil has quit IRC14:48
*** baumann has quit IRC14:52
jordanPguys, 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] section14:56
jordanPwho is wrong here ?14:56
*** baumann has joined #openstack-cinder14:56
eharneyjordanP: well... depends on who you ask :)14:56
jordanPlol14:57
jordanPeharney, what do you think ?14:57
eharneywell, devstack needs to be setting it in the backend sections currently for it to work as intended14:57
*** martyturner has quit IRC14:57
jordanPthen I'll make a patch to devstack then14:57
eharneyi 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 IRC14:58
jordanPmaybe if volume_clear is None in the per backend config, we should lookup in the DEFAULT section14:58
*** mriedem has joined #openstack-cinder14:59
*** martyturner has joined #openstack-cinder14:59
*** martyturner has quit IRC14:59
eharneythis behavior applies to a lot of other options too14:59
smcginnisjordanP: There has been some discussion of inheriting default settings in each backend, but it does cause some... concerns.14:59
*** martyturner has joined #openstack-cinder14:59
eharneysmcginnis: as does not inheriting them :)15:00
jordanPok, I am fine with fixing devstack15:00
smcginniseharney: True15:00
openstackgerritJohn Griffith proposed openstack/cinder: Update config format for replication_devices  https://review.openstack.org/23330715:00
*** Yogi11 has joined #openstack-cinder15:00
*** haomaiwang has quit IRC15:01
jordanPBoth ways work with their pros and cons. In the mean time I'll submit a change to devstack15:01
*** haomaiwang has joined #openstack-cinder15:01
jgriffithjordanP: the problem is that the way config is parsed out I couldn't come up with a way to get those external views15:01
*** p0rtal has joined #openstack-cinder15:02
jgriffithjordanP: 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 setting15:02
eharneyjordanP: i think devstack already handles this for LVM?15:02
jgriffitheharney: +115:02
jgriffitheharney: not think.. it does15:02
jordanPeharney, no :( it sets volume_clear in the [default] section15:02
eharneybut it looks like we have $CINDER_VOLUME_CLEAR for the general case and $CINDER_SECURE_DELETE for LVM15:03
eharneymaybe you need to set the latter option?15:03
jordanPahh right15:03
jordanPI forgot about $CINDER_SECURE_DELETE15:03
jgriffithjordanP: that's the one that actually matters :)15:03
jordanPyeah but if you use it you get this big warning15:04
jordanP"deprecated "Configure secure Cinder volume deletion using CINDER_VOLUME_CLEAR instead of CINDER_SECURE_DELETE.""15:04
*** Yogi1 has quit IRC15:04
jgriffithWhaaaaa15:04
eharneyit also appears to only do anything if you set it to False, so it's not really the same thing anyway15:04
*** ntpttr has quit IRC15:05
*** salv-orl_ has joined #openstack-cinder15:05
*** leeantho has joined #openstack-cinder15:06
*** leeantho_ has joined #openstack-cinder15:06
*** salv-orlando has quit IRC15:07
*** p0rtal has quit IRC15:07
*** ronis has quit IRC15:09
*** ntpttr has joined #openstack-cinder15:10
*** leeantho has quit IRC15:11
*** ntpttr has quit IRC15:13
*** timcl has joined #openstack-cinder15:14
*** ntpttr has joined #openstack-cinder15:15
*** bluex-pl has quit IRC15:16
*** bluex-pl has joined #openstack-cinder15:19
*** bluex-pl has quit IRC15:20
*** bluex-pl has joined #openstack-cinder15:20
*** timcl1 has joined #openstack-cinder15:20
*** rushil_ has quit IRC15:21
*** timcl has quit IRC15:22
*** salv-orl_ has quit IRC15:23
openstackgerritMarian Horban proposed openstack/cinder: Remove wsgi functionality  https://review.openstack.org/21967115:25
*** salv-orlando has joined #openstack-cinder15:28
*** nkrinner has quit IRC15:30
*** jdurgin1 has joined #openstack-cinder15:31
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Disable REST verify warnings  https://review.openstack.org/23429415:31
*** cbader_ has joined #openstack-cinder15:35
*** daneyon has joined #openstack-cinder15:36
*** salv-orl_ has joined #openstack-cinder15:36
openstackgerritAlon Marx proposed openstack/cinder: Updates in consistency_group in xiv/ds8k driver  https://review.openstack.org/23328015:36
*** zhangjn has quit IRC15:37
*** salv-orlando has quit IRC15:38
*** mudassirlatif has joined #openstack-cinder15:38
*** jungleboyj has joined #openstack-cinder15:39
*** daneyon has quit IRC15:41
*** vgridnev has quit IRC15:45
*** daneyon has joined #openstack-cinder15:47
*** lpetrut has quit IRC15:48
*** anshul has quit IRC15:48
hemnamep15:49
*** daneyon_ has joined #openstack-cinder15:49
*** tpeoples has quit IRC15:52
*** vgridnev has joined #openstack-cinder15:52
openstackgerritSean Dague proposed openstack/cinder: Change default Exception code to 500  https://review.openstack.org/23392315:52
*** daneyon has quit IRC15:53
*** harlowja_at_home has joined #openstack-cinder15:53
*** vgridnev has quit IRC15:54
*** IlyaG has joined #openstack-cinder15:56
*** e0ne has quit IRC15:56
*** p0rtal has joined #openstack-cinder15:57
*** p0rtal has quit IRC15:57
*** vgridnev has joined #openstack-cinder15:58
*** haomaiwang has quit IRC16:01
*** tpeoples has joined #openstack-cinder16:01
*** timcl1 has quit IRC16:01
*** haomaiwang has joined #openstack-cinder16:01
*** martyturner has quit IRC16:01
*** martyturner has joined #openstack-cinder16:03
*** timcl has joined #openstack-cinder16:03
*** Yogi1 has joined #openstack-cinder16:05
*** _cjones_ has joined #openstack-cinder16:05
*** _cjones_ has quit IRC16:05
*** merooney has quit IRC16:05
*** _cjones_ has joined #openstack-cinder16:05
*** harlowja_at_home has quit IRC16:05
*** rhagarty__ has quit IRC16:06
*** apoorvad has joined #openstack-cinder16:06
*** Yogi11 has quit IRC16:06
*** apoorvad has quit IRC16:07
*** jdurgin1 has quit IRC16:08
*** rhagarty has joined #openstack-cinder16:10
*** apoorvad has joined #openstack-cinder16:11
hemnaeharney, ping16:11
eharneyhemna: hey16:12
hemnacan you take a look at this one for me ?https://review.openstack.org/#/c/19976416:12
hemnaI'm not an RBD guy16:12
hemnaI 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 RBD16:13
eharneyhemna: looking16:13
hemnathanks16:13
*** IlyaG has quit IRC16:14
*** ronis has joined #openstack-cinder16:16
*** kevincarr1991 has quit IRC16:18
*** timcl has quit IRC16:18
mriedemsmcginnis: jgriffith: this needs updating it looks like https://review.openstack.org/#/c/233668/16:19
mriedemcinder rc3 is going to need that, but the webob thing has changed on master16:19
*** vgridnev has quit IRC16:20
mriedemso i think we just take that and squash with https://review.openstack.org/#/c/233923/16:20
jgriffithmriedem: snap!  Yes, I pinged smcginnis about that, meant to get ahold of e0ne16:20
jgriffithI'll update it now16:20
jgriffithmriedem: or...let smcginnis handle it since he's been doing squashes today :)16:20
mriedemsounds like ttx would like it asap16:20
mriedemi can squash quick if you like16:21
jgriffithmriedem: either way... I can do it or if you already have the forumula go for it16:21
jgriffithmriedem: TBH last time I did a squash on a gerrit change it didn't go well for me16:21
mriedemi'm on it16:22
jgriffithmriedem: thanks16:22
eharneyhemna: 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 background16:22
jgriffithmriedem: when you have a moment I'd love insight on if the commit message update "just works" when you do the squash or not16:22
hemnaeharney, ok cool thanks.16:23
jgriffithmriedem: 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 wrong16:23
mriedemyou have to sweet talk it16:23
mriedemthe commit message whisperer16:23
jgriffithmriedem: LOL... that explains it16:23
jgriffithmriedem: I tried the Drill Sergeant approach16:24
hemnaeharney, 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
jgriffithmriedem: ahh... I see what I did wrong16:24
mriedemman there is already a merge conflict16:25
mriedemnot sure how that happened16:25
mriedemah the assertEqual nit patch16:25
mriedemfun16:25
jgriffithmriedem: sigh... who merged that stupid thing16:25
eharneyhemna: 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 level16:25
hemnaok that's kinda what I thought16:26
hemnathanks for the help16:26
*** garthb has joined #openstack-cinder16:26
mriedemthis kind of seems like a bad idea https://review.openstack.org/#/c/231438/5/cinder/exception.py if webob ever changes that internal stuff16:28
jgriffithmriedem: yeah, I saw that and was unsure about it16:29
hemnawhy not push that up into webob then ?16:29
jgriffithhemna: ?16:30
hemnathe comment leads one to believe that it's a strict requirement16:30
openstackgerritYuriy Nesenenko proposed openstack/cinder: Implement snapshots-related features for Block Device Driver  https://review.openstack.org/22229216:31
*** jistr has quit IRC16:31
hemnameh16:31
mriedemjgriffith: squashed https://review.openstack.org/#/c/234329/16:31
jgriffithmriedem: thanks16:32
*** wilson-1 has joined #openstack-cinder16:32
jgriffithmriedem: I'll patiently wait for an education on how you got it to work :)16:32
mriedemgit review -d 23366816:33
mriedemgit fetch ssh://mriedem@review.openstack.org:29418/openstack/cinder refs/changes/23/233923/2 && git cherry-pick -x FETCH_HEAD16:33
mriedemgit rebase -i HEAD~216:33
mriedemsquash 2nd into 1st16:33
mriedemaccept the commit message16:33
jgriffithmriedem: ahhh16:33
mriedemand git review that sucka16:33
jgriffithmriedem: ok, I messed up the fetch16:33
jgriffithmriedem: awesome!!  Thanks!16:33
mriedemnp16:34
*** subscope has quit IRC16:34
*** wilson1 has quit IRC16:34
mriedemhemna: can you +W this? https://review.openstack.org/#/c/233923/ it was approved before but was missing a unit test fix16:41
mriedemand we have it backported to stable/liberty and it's approved there16:42
*** cbader_ has quit IRC16:43
smcginnismriedem, jgriffith: Thanks, got pulled away.16:44
jgriffithntpttr: ping16:46
ntpttrjgriffith: Hey16:48
hemnamriedem, crap sorry.  was afk for a bit16:48
*** jordanP has quit IRC16:49
smcginnisAll: Please limit rechecks and such until this lands: https://review.openstack.org/#/c/233923/16:50
openstackgerritGorka Eguileor proposed openstack/cinder-specs: Job Distribution to support HA A/A  https://review.openstack.org/23259516:50
jgriffithDuncanT: stop approving things for the time being :)16:51
DuncanTjgriffith: Sure, sorry, hadn't heard16:51
jgriffithDuncanT: no worries16:51
adrianofrAll: 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
mriedemadrianofr: right now16:52
adrianofrmriedem: So, it's already able to be approved, right?16:53
mriedemadrianofr: there is one cinder change on stable/liberty that is needed for rc316:53
mriedem^ does not seem critical enough to warrant getting it into rc3 at this point16:53
mriedemthe importance isn't even set on the bug16:53
mriedemit's fine for stable in general, but not a release blocker16:54
smcginnisadrianofr: Probably soon, but we need to wait for the final RC3.16:54
*** lpetrut has joined #openstack-cinder16:55
jgriffithsmcginnis: I gave everything that was "close" a -2 to try and prevent anything else until the ec patch lands16:55
jgriffithsmcginnis: don't let me forget to clear them later :)16:55
jgriffitheharney: hemna geguileo jungleboyj just incase you didn't see, don't merge anything for now16:56
smcginnisjgriffith: Thanks, good plan.16:56
adrianofrmriedem: smcginnis : I see. Thanks for the info :)16:56
hemnajgriffith, ok16:56
smcginnisadrianofr: np16:56
geguileojgriffith: I didn't see16:56
jgriffithgeguileo: :)16:56
geguileojgriffith: Was there an email?  :-O16:56
jgriffithgeguileo: no, just conversation here16:57
hemnahttps://review.openstack.org/#/c/233923/ waiting for that guy ?16:57
geguileojgriffith: Yesterday it was a National Holiday here16:57
jgriffithhemna: yes16:57
hemnacoolio16:57
jgriffithgeguileo: nice16:57
jgriffithgeguileo: no worries, just came up this morning... not a huge deal16:58
geguileojgriffith: Nice but I missed that   XD16:58
geguileojgriffith: Ok, no merging for now, noted16:58
geguileojgriffith: Thanks!16:58
jgriffithgeguileo: thank you sir16:58
jungleboyjjgriffith: Thanks for the warning.  Missed the discussion.  What is wrong?16:59
smcginnisJust have 300+ other patches to make it by, then we're all clear. ;)16:59
hemnaLO16:59
jgriffithlol16:59
hemnabah16:59
smcginnisjungleboyj: SNAFU. Waiting for one final cleanup to go through.16:59
jungleboyjsmcginnis: Ok.17:00
*** DericHorn-HP has quit IRC17:00
*** bluex-pl has quit IRC17:00
*** haomaiwang has quit IRC17:01
*** haomaiwang has joined #openstack-cinder17:01
*** sghanekar_ has joined #openstack-cinder17:01
*** timcl has joined #openstack-cinder17:02
*** Yogi1 has quit IRC17:03
*** vgridnev has joined #openstack-cinder17:03
apoorvadsghanekar_: hi17:04
*** avishay_ has joined #openstack-cinder17:04
*** merooney has joined #openstack-cinder17:05
*** ntpttr has quit IRC17:07
*** ntpttr has joined #openstack-cinder17:09
*** kevincarr1991 has joined #openstack-cinder17:12
*** Yogi1 has joined #openstack-cinder17:15
*** p0rtal has joined #openstack-cinder17:15
openstackgerritFergal Mc Carthy proposed openstack/cinder: Use lvm_conf_file directory for LVM_SYSTEM_DIR value.  https://review.openstack.org/22656217:18
patrickeastjgriffith: 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 further17:18
patrickeastjgriffith: although i lost all my +A's in the process :(17:18
jgriffithpatrickeast: we can fix that :)17:20
*** ntpttr has quit IRC17:20
*** asselin_ has joined #openstack-cinder17:20
*** mtanino has joined #openstack-cinder17:21
*** aix has quit IRC17:22
*** lpetrut has quit IRC17:22
*** asselin__ has quit IRC17:22
*** markus_z has quit IRC17:23
*** salv-orl_ has quit IRC17:24
*** salv-orlando has joined #openstack-cinder17:25
jgriffithsmcginnis: I'll +2/A it when you get it posted :)17:27
openstackgerritDmitry Guryanov proposed openstack/cinder: WIP: RemoteFS volume format handlers  https://review.openstack.org/23435917:28
smcginnisjgriffith: https://review.openstack.org/23436017:30
jgriffithsmcginnis: done17:31
jgriffithso it looks like I confused the failure modes here17:31
jgriffithmaster should be ok17:31
jgriffithI'll clear my -2's17:31
jgriffithIt would still probably be ok if we didn't flood the gate with patches while things get caught up17:31
jgriffithstable branch fixes should be priority right now17:32
smcginnisjgriffith: clark was saying to hold off on master though.17:32
jgriffithmeh... I'll leave things alone for the time being :)17:32
jgriffithsmcginnis: oh.. perfect17:32
smcginnisjgriffith: I _thought_ we were safe, but..17:32
jgriffithsmcginnis: now I won't feel guilty :)17:32
smcginnisjgriffith: ;)17:32
jgriffithLOL.. like that would happen anyway :)17:32
*** wilson-1 has quit IRC17:34
*** wilson-1 has joined #openstack-cinder17:35
*** sghanekar_ has quit IRC17:38
*** sghanekar_ has joined #openstack-cinder17:40
*** xyang1 has quit IRC17:42
*** IlyaG has joined #openstack-cinder17:45
*** bill_az has joined #openstack-cinder17:46
*** angela-s has joined #openstack-cinder17:56
*** asselin_ is now known as asselin__17:58
*** ntpttr has joined #openstack-cinder18:00
*** haomaiwang has quit IRC18:01
*** haomaiwang has joined #openstack-cinder18:01
*** mudassirlatif has quit IRC18:04
*** mudassirlatif has joined #openstack-cinder18:08
*** crose has quit IRC18:13
*** ndipanov has quit IRC18:14
*** e0ne has joined #openstack-cinder18:17
*** mriedem has quit IRC18:18
*** ociuhandu has quit IRC18:19
*** mriedem has joined #openstack-cinder18:21
*** avishay_ has quit IRC18:22
*** vgridnev has quit IRC18:22
*** kevincar_ has joined #openstack-cinder18:23
*** avishay_ has joined #openstack-cinder18:23
*** vgridnev has joined #openstack-cinder18:25
openstackgerritJohn Griffith proposed openstack/cinder: Updates in consistency_group in xiv/ds8k driver  https://review.openstack.org/23328018:25
openstackgerritJohn Griffith proposed openstack/cinder: Fix method VolumeTypeList.get_all  https://review.openstack.org/23416718:25
openstackgerritJohn Griffith proposed openstack/cinder: Port WSGI tests to Python 3  https://review.openstack.org/23221418:25
openstackgerritJohn Griffith proposed openstack/cinder: Cleanup/move code in Storwize Driver  https://review.openstack.org/23011718:25
openstackgerritJohn Griffith proposed openstack/cinder: Improve performance listing detail for volumes  https://review.openstack.org/20353018:26
openstackgerritJohn Griffith proposed openstack/cinder: encryption_api_url requires a version  https://review.openstack.org/23003118:26
openstackgerritJohn Griffith proposed openstack/cinder: Fix updating only volume type is_public  https://review.openstack.org/22156218:26
smcginnisRebases?18:26
openstackgerritJohn Griffith proposed openstack/cinder: CGSnapshot Object  https://review.openstack.org/19502718:26
openstackgerritJohn Griffith proposed openstack/cinder: Move CloneableVD to common functions  https://review.openstack.org/20359318:26
openstackgerritJohn Griffith proposed openstack/cinder-specs: Job Distribution to support HA A/A  https://review.openstack.org/23259518:26
jgriffithsmcginnis: yes18:26
*** subscope has joined #openstack-cinder18:26
jgriffithsmcginnis: should've warned you :)  sorry18:26
*** kevincarr1991 has quit IRC18:27
*** kevincar_ has quit IRC18:27
smcginnisNo worries. Didn't think you were that quick with updating patchsets. ;)18:27
jgriffithsmcginnis: hehe18:27
jgriffithI'm doing a Victor/Gorka tick an just queuing them all up18:28
jgriffith:)18:28
*** baumann has left #openstack-cinder18:28
*** baumann has joined #openstack-cinder18:28
*** e0ne has quit IRC18:30
haypojgriffith: hey18:31
haypojgriffith: the WSGI patch serie just makes me crazy :-p18:31
jgriffithhaypo: LOL... which one exactly?18:32
jgriffithhaypo: You mean all the webob churn?18:32
haypojgriffith: it's a serie of 4 patches18:32
haypojgriffith: the serie starting with https://review.openstack.org/#/c/232214/18:32
haypojgriffith: it's the version 10 and i have to rebase it multiple times each day !?18:33
jgriffithYeah, I figured that's the one you meant18:33
haypojgriffith: i got a lot of +2 :-p18:33
jgriffithhaypo: well.. the good news is that we should be in the clear after this morning18:33
haypothe WebOb 1.5 issue is behind us, no?18:33
jgriffithhaypo: sorry about all of that18:33
jgriffithhaypo: yes, it *should* be behind us at this point18:33
jgriffithhaypo: there was a couple other issues that you got stuck in as well18:33
jgriffithhaypo: the good thing is at least you didn't loose votes with the rebase :)18:34
haypojgriffith: my webob 1.5 fix was merged yesterday no? https://review.openstack.org/#/c/233528/18:34
jgriffithhaypo: yes, but there was a merge issue with a change to the default code18:34
jgriffithhaypo: then stable blew up18:34
jgriffithhaypo: then there was the requirements issue that showed up in messaging18:35
jgriffithhaypo: so there's been a lot of churn that actually had nothing to do with your patch18:35
haypojgriffith: yeah, i noticed :) i had to recheck multiple times :)18:35
*** salv-orlando has quit IRC18:35
jgriffithhaypo: I'll +2/A your patch later this afternoon when things are all caught up again18:35
haypojgriffith: (note: my WSGI is no more approved, but jenkins didn't check the latest version neither)18:35
*** avishay_ has quit IRC18:36
haypojgriffith: thanks & no need to be sorry :)18:36
jgriffithhaypo: I'll monitor these: https://review.openstack.org/#/q/status:open+project:openstack/cinder+branch:master+topic:bp/cinder-python3,n,z18:37
jgriffithand go back to them later today18:37
jgriffithif there's something missing in that list let me know18:37
*** crose has joined #openstack-cinder18:46
haypojgriffith: 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
haypojgriffith: (you can ignore "WIP: Fix Python 3 issues in EMC vnx client")18:47
smcginnishaypo: Yeah, try to keep some of your sanity. ;)18:49
haypohaha18:49
*** haomaiwang has quit IRC18:53
smcginnisRC3 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-cinder18:56
*** mudassirlatif has quit IRC18:58
*** daneyon_ has quit IRC19:00
*** daneyon has quit IRC19:00
*** DericHorn-HP has joined #openstack-cinder19:01
*** dzamboni has quit IRC19:03
guitarzanhah19:07
*** dave-mccowan has quit IRC19:15
*** e0ne has joined #openstack-cinder19:19
ntpttrjgriffith: ping19:19
*** IlyaG has quit IRC19:19
*** alejandrito has quit IRC19:20
*** alejandrito has joined #openstack-cinder19:22
*** mudassirlatif has joined #openstack-cinder19:22
jgriffithntpttr: hey ya19:23
jgriffithntpttr: so the problem with your suggestion :)19:24
jgriffithntpttr: that limitation isn't actually a result of the setting in the driver you saw19:24
jgriffithntpttr: it's actually a cluster limit19:24
jgriffithntpttr: so creates will fail after 2K19:24
ntpttrjgriffith: Ah! Okay that makes sense, I was definitely going in the wrong direction then :)19:25
jgriffithntpttr: yeah... sorry :(19:25
jgriffithI wasn't clear about that in the bug19:25
jgriffithntpttr: FWIW, I've grown to hate the fix that I made there :(19:25
jgriffithntpttr: it's kinda clunky19:26
jgriffithntpttr: I need to just refactor all of that code19:26
ntpttrjgriffith: Haha no problem, I just thought I'd give it a shot and worst case learn something19:26
jgriffithntpttr: well... you did :)19:26
*** mudassirlatif has quit IRC19:28
*** martyturner has quit IRC19:29
*** alejandrito has quit IRC19:32
*** delatte has quit IRC19:36
*** haomaiwang has joined #openstack-cinder19:37
*** xyang1 has joined #openstack-cinder19:38
*** e0ne has quit IRC19:43
*** haomaiwang has quit IRC19:43
DuncanTWho runs the OpenvStorage CI? It's been failing continuously for a while19:44
*** martyturner has joined #openstack-cinder19:44
*** hodos has joined #openstack-cinder19:49
patrickeastDuncanT: thats this one https://wiki.openstack.org/wiki/ThirdPartySystems/CloudFounders_OpenvStorage_CI right?19:54
DuncanTThat's the one, yeah.19:54
DuncanTDo we have an IRC nick for them?19:54
DuncanTIf not, I can email them19:55
openstackgerritNate Potter proposed openstack/cinder: Add qos_specs_id to volume type show  https://review.openstack.org/23441419:55
patrickeastDuncanT: I don't see one19:56
*** dave-mccowan has joined #openstack-cinder19:57
DuncanTOk, I'll email them, thanks19:58
*** ociuhandu has joined #openstack-cinder20:02
*** martyturner has quit IRC20:06
jgriffithntpttr: 234414 seems good/reasonable20:06
*** martyturner has joined #openstack-cinder20:06
jgriffithntpttr: one thing if you're looking for something else to dive in to :)20:06
jgriffithntpttr: it would appear that we have never implemented a type-show in the client :(20:07
*** IlyaG has joined #openstack-cinder20:07
*** merooney has quit IRC20:09
ntpttrjgriffith: 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
ntpttrjgriffith: I'd be happy to work on that if it's an option!20:10
*** haypo has left #openstack-cinder20:10
jgriffithntpttr: Yeah, I think your timing may have just been off on that other one20:10
jgriffithntpttr: jungleboyj should clear that -2 btw20:10
*** Yogi1 has quit IRC20:12
jgriffithntpttr: https://bugs.launchpad.net/python-cinderclient/+bug/150580620:12
openstackLaunchpad bug 1505806 in python-cinderclient "cinderclient has no implementation for type-show" [Undecided,New]20:12
ntpttrjgriffith: All right, is there a launchpad report for the type-show implementation?20:12
ntpttrjgriffith: that answers that question20:12
*** Yogi1 has joined #openstack-cinder20:12
jgriffithLOL20:12
ntpttrjgriffith: haha well sounds good! Thank you20:13
*** porrua has quit IRC20:14
*** Lee1092 has quit IRC20:14
jungleboyjntpttr: jgriffith -2 cleared.20:15
smcginnisDuncanT: Thanks, been waiting to gather up all the problem CIs and notify them all.20:16
smcginnisDuncanT: We have several more than just them unfortunately.20:16
jungleboyjntpttr: That patch still needs a cross project dependency, doesn't it?20:16
DuncanTsmcginnis: Yeah, I do an occasional round when I notice one has been failing a lot. I don't really check the stats very often though20:17
ntpttrjungleboyj: between the cinderclient and cinder you mean? Yeah I can update the commit message20:17
jgriffithjungleboyj: thanks!20:18
jungleboyjOk, that would be good.20:18
jungleboyjjgriffith: Welcome.  Thanks for the pointer.20:18
jgriffithDuncanT: you know about this right?  http://ci-watch.tintri.com/project?project=cinder&time=7+days20:18
jgriffithDuncanT: if not, you might find it useful/handy20:18
smcginnisHmm, those EMC ones are pretty spotty. Maybe I should remove them.20:20
smcginnisJoke!20:20
jgriffithsmcginnis: LOL20:20
jgriffithsmcginnis: that's a pretty good one20:20
jungleboyjHe he he.20:20
* jungleboyj is going to laugh when smcginnis has to drink the EMC kool-aid.20:20
jgriffithsmcginnis: you better watch it though... ya never know xyang1 may be your boss at some point :)20:20
smcginnisjgriffith: No doubt. :)20:20
*** ronis has quit IRC20:20
xyang1jgriffith: what?:)20:20
jungleboyjjgriffith: I had the same thought.20:21
jgriffithxyang1: LOL... I thought that might get your attention :)20:21
DuncanTsmcginnis: I had forgotten about that, thanks for the reminder20:21
jgriffithxyang1: just joking around because you and smcginnis will be working for the same company20:21
xyang1jgriffith: I guess anything could happen, it could be the other way around too20:21
jgriffithxyang1: ya never know20:21
DuncanTWow, none of the CIs look great, TBH20:22
jgriffithDuncanT: except mine :)20:22
jgriffithDuncanT: mine is awesome LOL20:22
jungleboyjDuncanT: Had a meeting about that here this morning.  Sending out angry grams today.20:22
DuncanTjgriffith: Needs more cowbell20:22
*** e0ne has joined #openstack-cinder20:22
jungleboyjAgain.20:22
jgriffithDuncanT: LOL20:23
jgriffithDuncanT: the only thing screwing mine up at this point are the dependency tags :(20:24
jgriffithDuncanT: http://i.imgur.com/gMRjnK2.jpg20:25
jgriffithDuncanT: I really think we need a filter... to show comparison between things that passed jenkins and the 3'rd party CI's20:25
smcginnisjgriffith: I prototyped pulling in dependencies.20:25
smcginnisIt kinda sorta worked.20:25
DuncanTjgriffith: Agreed20:25
smcginnisGot a little unstable though.20:25
jgriffithsmcginnis: well, that's more than what I did :)20:25
jgriffithDuncanT: smcginnis I think the key metric is the disagreement with Jenkins on passed runs20:26
smcginnisjgriffith: On the never ending TODO list to get back to at some point.20:26
DuncanTjgriffith: Or even just ignore any patch where more than half the CIs failed?20:26
smcginnisjgriffith, DuncanT: Yeah, I'd like to have an easy report that shows just that.20:26
jgriffithDuncanT: smcginnis especially after days like today and yesterday where stuff has an extremely high fail rate20:26
jgriffithsmcginnis: DuncanT I started putting something like that together as an add to Joe Gordons last-comment....20:26
jgriffithsmcginnis: DuncanT maybe I'll try and pick that back up again20:27
DuncanTjgriffith: Awesome20:27
smcginnisjgriffith: That would be cool!20:27
openstackgerritLukas Bezdicka proposed openstack/os-brick: Remove brackets from path  https://review.openstack.org/23442520:29
openstackgerritNate Potter proposed openstack/python-cinderclient: Add options when uploading images to Glance  https://review.openstack.org/23062220:32
*** bill_az has quit IRC20:33
*** e0ne has quit IRC20:35
*** Yogi1 has quit IRC20:41
*** qeas has joined #openstack-cinder20:41
*** haomaiwang has joined #openstack-cinder20:41
qeashi everyone, can somebody point where can I see CI results from different vendors?20:42
*** bswartz has quit IRC20:42
*** p0rtal_ has joined #openstack-cinder20:42
apoorvadqeas : http://ci-watch.tintri.com/project?project=cinder20:43
*** SZhang has joined #openstack-cinder20:44
*** ntpttr has quit IRC20:44
SZhangHi, 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 IRC20:46
*** p0rtal has quit IRC20:46
*** gouthamr has joined #openstack-cinder20:46
*** gouthamr has quit IRC20:47
*** DericHorn-HP has quit IRC20:47
*** haomaiwang has quit IRC20:47
*** martyturner has quit IRC20:48
qeasapoorvad : thanks!20:48
*** martyturner has joined #openstack-cinder20:49
*** timcl has quit IRC20:49
*** vgridnev has quit IRC20:57
*** raildo is now known as raildo-afk21:00
*** vgridnev has joined #openstack-cinder21:01
*** subscope has quit IRC21:02
*** salv-orlando has joined #openstack-cinder21:02
*** vgridnev has quit IRC21:03
*** vgridnev has joined #openstack-cinder21:04
smcginnisall: Last patch needed to clear up issues merged. OK to recheck and approve things again.21:05
SZhangAnyone 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 IRC21:13
*** dustins has quit IRC21:14
*** ntpttr has joined #openstack-cinder21:21
*** bill_az has joined #openstack-cinder21:28
*** jgregor has quit IRC21:31
*** qeas has quit IRC21:32
*** vgridnev has quit IRC21:32
*** jgregor has joined #openstack-cinder21:33
*** adrianofr has quit IRC21:36
*** mc_nair has quit IRC21:38
*** julim has quit IRC21:39
*** eharney has quit IRC21:40
*** baumann has quit IRC21:44
*** jungleboyj has quit IRC21:45
*** gouthamr has joined #openstack-cinder21:46
*** mriedem has quit IRC21:48
*** dave-mccowan has quit IRC21:56
*** daneyon has joined #openstack-cinder21:58
*** ntpttr has quit IRC22:01
*** lprice2 has joined #openstack-cinder22:06
*** lprice1 has quit IRC22:06
*** IlyaG_ has joined #openstack-cinder22:06
*** edtubill has quit IRC22:07
*** IlyaG has quit IRC22:09
*** david-lyle has quit IRC22:10
*** david-lyle has joined #openstack-cinder22:12
*** martyturner has quit IRC22:13
*** jungleboyj has joined #openstack-cinder22:16
*** tpeoples has quit IRC22:18
*** xyang1 has quit IRC22:19
*** lcurtis has quit IRC22:23
*** kevincarr1991 has joined #openstack-cinder22:24
*** gouthamr_ has joined #openstack-cinder22:29
jgriffithSZhang: you'll need to clone with no type... then retype sadly22:30
jgriffithSZhang: 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
jgriffithSZhang: honestly it's been a bit of a long running debate22:31
*** gouthamr has quit IRC22:32
*** edmondsw has quit IRC22:33
jgriffithSZhang: Ohhh... yipee!22:33
jgriffithSZhang: that's been changed22:33
jgriffithSZhang: you can in fact22:33
jgriffithSZhang: although I'm curious why you're asking instead of just trying it :)22:34
*** IanGovett has quit IRC22:34
jgriffithSZhang: there are caveats, like the types have to be compatible and both specify the same volume-backend22:34
*** daneyon has quit IRC22:41
*** IanGovett has joined #openstack-cinder22:42
*** willsama has joined #openstack-cinder22:43
*** haomaiwang has joined #openstack-cinder22:44
*** edtubill has joined #openstack-cinder22:45
Swansonjgriffith, Is replication_failover a required method?22:46
jgriffithSwanson: for replication it will be22:47
jgriffithSwanson: I suppose that's debatable in some views22:47
Swansonjgriffith, 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
jgriffithSwanson: well...  on the other hand if you're using replication :)22:48
jgriffithSwanson: so keep in mind that's supposed to be an admin that does it anyway22:48
*** haomaiwang has quit IRC22:50
*** angela-s has quit IRC22:51
Swansonjgriffith, 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
jgriffithSwanson: funny, I was just talking to somebody this morning about that22:51
jgriffithSwanson: in essence that's what you'll end up doing22:52
jgriffithSwanson: but in that case... you have a "managed" backend22:52
jgriffithSwanson: so it's the same for some to just redirect the host column22:52
jgriffithSwanson: keep in mind that manage/unmanage doesn't preserve the volume object/id being used though22:53
jgriffithSwanson: which might not be so bueano22:53
Swansonjgriffith, True.  Volume could get lost.22:54
jgriffithSwanson: well... the bigger problem is that when you fail-over or whatever it's a "new" volume22:54
Swansonjgriffith, 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
jgriffithSwanson: cinder ID22:56
jgriffithSwanson: scsi ID is a given22:56
jgriffithSwanson: but that's transparrant to users anyway so who cares :)22:57
Swansonjgriffith, seems like to keep the ID (and we name everything after the GUID) failover is better than manage.22:59
jgriffithSwanson: right22:59
jgriffithSwanson: I completely agree22:59
*** chlong has quit IRC22:59
Swansonjgriffith, 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
jgriffithSwanson: depends on what your backend/driver can do23:01
jgriffithSwanson: either23:01
jgriffithA:  issue fail-over command and it toggles over23:01
jgriffithB. Admin doe ssomething in the background to make things right again23:02
Swansonjgriffith, B.  For now.23:02
jgriffithSwanson: I think that's a perfectly acceptable answer :)23:03
Swansonjgriffith, 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 IRC23:05
*** gouthamr_ has quit IRC23:06
Swansonjgriffith, thanks for the help.  time for dinner.23:06
jgriffithSwanson: no problem.. enjoy!23:07
*** willsama has quit IRC23:08
*** willsama has joined #openstack-cinder23:13
*** sgotliv_ has quit IRC23:16
*** IlyaG_ has quit IRC23:18
*** salv-orlando has quit IRC23:19
*** salv-orlando has joined #openstack-cinder23:20
*** stevemar_ has quit IRC23:23
*** stevemar_ has joined #openstack-cinder23:24
*** I has joined #openstack-cinder23:25
*** I is now known as Guest9019423:25
openstackgerritNate Potter proposed openstack/cinder: Add qos_specs_id to volume type show  https://review.openstack.org/23441423:26
*** salv-orlando has quit IRC23:27
*** salv-orlando has joined #openstack-cinder23:28
*** salv-orlando has quit IRC23:31
*** salv-orlando has joined #openstack-cinder23:32
*** Guest90194 has quit IRC23:33
*** salv-orlando has quit IRC23:34
*** cdelatte has joined #openstack-cinder23:34
*** delattec has joined #openstack-cinder23:34
*** gouthamr has joined #openstack-cinder23:38
*** delatte has joined #openstack-cinder23:39
*** zhangjn has joined #openstack-cinder23:39
*** jwcroppe has quit IRC23:39
*** zhangjn has quit IRC23:39
*** rhedlind has quit IRC23:39
*** zhangjn has joined #openstack-cinder23:40
*** dims has quit IRC23:41
*** delattec has quit IRC23:41
*** cdelatte has quit IRC23:42
*** dims has joined #openstack-cinder23:42
*** r-daneel has quit IRC23:43
*** haomaiwang has joined #openstack-cinder23:46
*** delattec has joined #openstack-cinder23:46
*** delatte has quit IRC23:50
*** haomaiwang has quit IRC23:50
*** smoriya has joined #openstack-cinder23:52
*** IanGovett has quit IRC23:52
*** delattec has quit IRC23:55
*** willsama has quit IRC23:56

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!