Tuesday, 2015-11-24

*** yhayashi has joined #openstack-cinder00:00
sgundurjgriffith: thanks for your comments00:01
sgundur jgriffith: I need to go through your comments and will get back to you tomoro00:02
*** zhangjn has joined #openstack-cinder00:02
*** apoorvad has quit IRC00:03
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed)  https://review.openstack.org/23120100:06
*** niknakpaddywak has quit IRC00:06
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)  https://review.openstack.org/23124500:06
*** sgundur has quit IRC00:07
*** apoorvad has joined #openstack-cinder00:08
*** akshai has quit IRC00:08
*** EinstCrazy has joined #openstack-cinder00:09
*** gouthamr has quit IRC00:09
*** boris-42 has joined #openstack-cinder00:09
*** davechen has quit IRC00:10
*** mdenny has quit IRC00:11
*** sgotliv_ has joined #openstack-cinder00:11
boris-42thingee: jgriffith_away hi there00:14
*** sgotliv has quit IRC00:14
boris-42thingee: jgriffith_away could you please take a look at OSprofiler spec: https://review.openstack.org/#/c/103825/ I refactored it and make it overall simpler00:14
*** openstackgerrit has quit IRC00:16
*** openstackgerrit has joined #openstack-cinder00:17
boris-42DuncanT: ^00:17
*** zhangjn has quit IRC00:19
*** EinstCrazy has quit IRC00:19
*** diegows has quit IRC00:22
*** sgotliv_ has quit IRC00:22
openstackgerritMerged openstack/cinder: Add guidelines for release notes to devref  https://review.openstack.org/24870100:24
*** niknakpaddywak has joined #openstack-cinder00:24
*** pots has joined #openstack-cinder00:27
*** jordant1 has quit IRC00:28
*** pots2 has quit IRC00:29
bardiaI try to run 'tox -e py34' but get this error: http://paste.openstack.org/show/479806/00:31
bardiaam i missing a lib or something? shouldn't the test-requirements.txt include that?00:32
bardiabase os: Ubuntu 14.0400:32
kmartinbardia, "sudo apt-get install python-dev"00:37
bardiaalready have that. I think it needs python3-dev. Installed that right now and running it. I will update if that was the requirement00:37
kmartinyep, that should fix it00:39
*** rebase has quit IRC00:39
bardiakmartin: thanks00:39
*** jwcroppe has joined #openstack-cinder00:40
*** salv-orlando has quit IRC00:43
openstackgerritHelen Walsh proposed openstack/cinder: Proper error handling for invalid SLO/Workload combination  https://review.openstack.org/24383700:44
*** JoseMello has quit IRC00:44
*** r-daneel has quit IRC00:55
*** xyang1 has quit IRC00:58
*** zhangjn has joined #openstack-cinder01:00
*** zhangjn has quit IRC01:00
*** EinstCrazy has joined #openstack-cinder01:00
*** rushil has joined #openstack-cinder01:01
*** zhangjn has joined #openstack-cinder01:01
*** EinstCra_ has joined #openstack-cinder01:02
*** EinstCrazy has quit IRC01:06
*** jerrygb has quit IRC01:09
*** mc_nair has quit IRC01:09
*** jerrygb has joined #openstack-cinder01:10
*** zhangjn has quit IRC01:12
*** jerrygb has quit IRC01:14
*** rebase has joined #openstack-cinder01:15
*** apoorvad has quit IRC01:17
*** zhangjn has joined #openstack-cinder01:20
*** mc_nair has joined #openstack-cinder01:22
*** zhangjn_ has joined #openstack-cinder01:27
*** zhangjn has quit IRC01:29
*** jgriffith_away is now known as jgriffith01:30
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Extend Volume for VMAX3  https://review.openstack.org/24894101:33
*** Lee1092 has joined #openstack-cinder01:36
*** zhenguo has joined #openstack-cinder01:38
*** leeantho_ has quit IRC01:43
*** julim has joined #openstack-cinder01:43
*** edtubill has joined #openstack-cinder01:49
*** bswartz has joined #openstack-cinder01:52
bardiatox -epy34 -- -n cinder.tests.unit.test_mydriver doesn't work it still runs all the tests. Anyone faced this before?01:52
patrickeastbardia: i vaguely recall seeing something where the py34 ones just look at a text file and don't pass through the additional command line args01:59
patrickeastnot 100% sure though02:00
*** 17SAD4YI2 has joined #openstack-cinder02:00
*** 17SAD4YI2 has quit IRC02:01
patrickeastbardia: so like: https://github.com/openstack/cinder/blob/master/tox.ini#L25 for the py27 vs https://github.com/openstack/cinder/blob/master/tox.ini#L4102:01
*** haomaiwa_ has joined #openstack-cinder02:01
bardiapatrickeast: thanks I will just override it02:01
*** niknakpaddywak has quit IRC02:05
*** gcb has joined #openstack-cinder02:18
*** apoorvad has joined #openstack-cinder02:23
*** gouthamr has joined #openstack-cinder02:34
openstackgerritLisaLi proposed openstack/cinder-specs: Scaling backup service blueprint spec  https://review.openstack.org/24097802:34
*** david-ly_ has quit IRC02:36
*** houming has joined #openstack-cinder02:39
*** dims has joined #openstack-cinder02:40
*** jerrygb has joined #openstack-cinder02:44
*** sghanekar__ has quit IRC02:44
*** salv-orlando has joined #openstack-cinder02:47
*** mc_nair has quit IRC02:49
*** smoriya_afk has joined #openstack-cinder02:55
*** smoriya_afk is now known as smoriya02:55
*** rebase has quit IRC02:56
*** salv-orlando has quit IRC02:59
*** bardia has quit IRC03:00
*** haomaiwa_ has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** smoriya has quit IRC03:03
*** jwcroppe has quit IRC03:05
*** smoriya_afk has joined #openstack-cinder03:05
*** smoriya_afk is now known as smoriya03:06
*** jwcroppe has joined #openstack-cinder03:06
*** jwcroppe has quit IRC03:11
*** jerrygb has quit IRC03:11
*** jerrygb has joined #openstack-cinder03:13
*** diablo_rojo has joined #openstack-cinder03:15
*** dims has quit IRC03:17
*** dims has joined #openstack-cinder03:22
*** diablo_rojo has quit IRC03:24
*** pcaruana has quit IRC03:27
*** apoorvad has quit IRC03:27
*** sghanekar__ has joined #openstack-cinder03:32
*** dims has quit IRC03:35
*** smoriya has quit IRC03:38
*** smoriya_afk has joined #openstack-cinder03:39
*** smoriya_afk is now known as smoriya03:39
*** asselin has joined #openstack-cinder03:51
*** asselin_ has joined #openstack-cinder03:52
*** sgundur has joined #openstack-cinder03:57
*** haomaiwang has quit IRC04:01
*** haomaiwa_ has joined #openstack-cinder04:01
*** zhangjn_ has quit IRC04:02
*** smoriya has quit IRC04:03
*** smoriya_afk has joined #openstack-cinder04:04
*** smoriya_afk is now known as smoriya04:04
*** asselin_ has quit IRC04:05
*** asselin has quit IRC04:05
*** jwcroppe has joined #openstack-cinder04:08
*** david-lyle has joined #openstack-cinder04:08
*** akshai has joined #openstack-cinder04:10
*** akshai_ has joined #openstack-cinder04:11
*** akshai has quit IRC04:15
*** sgundur has quit IRC04:17
*** zhangjn has joined #openstack-cinder04:22
*** edtubill has quit IRC04:22
*** jerrygb has quit IRC04:24
*** edtubill has joined #openstack-cinder04:24
*** jwcroppe has quit IRC04:27
*** edtubill has quit IRC04:29
openstackgerritThang Pham proposed openstack/cinder: Implement refresh() for cinder objects  https://review.openstack.org/24899804:30
*** anish_ is now known as anish04:40
*** zhangjn has quit IRC04:44
*** akshai_ has quit IRC05:00
openstackgerritMerged openstack/cinder: Retry on database deadlock on service_update method  https://review.openstack.org/24529905:00
*** haomaiwa_ has quit IRC05:01
*** haomaiwang has joined #openstack-cinder05:01
*** jwcroppe has joined #openstack-cinder05:04
*** rcernin has joined #openstack-cinder05:16
*** sghanekar__ has quit IRC05:17
*** haomaiwang has quit IRC05:19
*** deepakcs has joined #openstack-cinder05:21
*** zhangjn has joined #openstack-cinder05:23
*** gcb has quit IRC05:24
jgriffithpatrickeast: bardia: FYI, it's because py34 uses a whitelist argument05:24
jgriffithpatrickeast: the tox invocation of ostestr doesn't provide the same options05:24
patrickeastjgriffith: yea, once i looked at the tox.ini file it makes sense05:27
*** haomaiwa_ has joined #openstack-cinder05:27
*** smoriya has quit IRC05:29
jgriffithpatrickeast: :)05:29
*** deepakcs has quit IRC05:32
*** rcernin has quit IRC05:34
lixiaoy1jgriffith: Hi John, may I talk to you about https://review.openstack.org/#/c/244964/?05:35
jgriffithlixiaoy1: sure... BTW I removed my -1 incase you didn't see05:35
lixiaoy1I see. I am just concerned about your comments.05:36
jgriffithlixiaoy1: which ones?  The part about somewhat useless types being created/used?05:36
jgriffithlixiaoy1: your change doesn't really hurt anything, I am curious if maybe there's a special case that I'm not thinking of though05:37
lixiaoy1that I think doing this in a driver that doesn't use extra-specs for anything is kinda wrong. There. if two types, type A has volume_backend_name specified, and type B has nothing, when retyping a volume from type A to type B05:37
*** EinstCra_ has quit IRC05:37
*** EinstCrazy has joined #openstack-cinder05:37
lixiaoy1the scheduler will select the original host, and then retype05:37
lixiaoy1as a result, it leads that retype a volume in a same host05:38
jgriffithlixiaoy1: so step back, and think of it as a system with only  a single volum-service/backend configured05:38
lixiaoy1may the two types have different front-end QoS, volume in available status05:39
jgriffithlixiaoy1: well, my point was in LVM's case any extra-specs (QoS or otherwise) don't do anything at all05:40
jgriffithlixiaoy1: I think I figured out the case you and tbarron mentioned about it kicking off migration though05:40
lixiaoy1ooh, you mean may something lost for extra_specs in lvm, need to do05:41
jgriffithlixiaoy1: but last I looked we don't do migration by default, you have to specify that you want to allow it05:41
*** shausy has joined #openstack-cinder05:41
*** gcb has joined #openstack-cinder05:41
jgriffithlixiaoy1: so what I mean is; if you're setting up extra-specs for an LVM volume (other than encryption I think) it's pointless and does nothing05:41
jgriffithlixiaoy1: so in a way, I'm not sure if that should be valid or not05:42
jgriffithlixiaoy1: in other words I don't know which is right/wrong... reporting success and saying you did something that you really didn't, or since it's a noop just saying "ok, done"05:42
jgriffithlixiaoy1: I was also confused because I was unable to produce a failure case05:43
*** deepakcs has joined #openstack-cinder05:43
jgriffithlixiaoy1: so it's kind of a gray area for me, but your patch is probably the more correct thing to do at this point05:46
lixiaoy1oh, I see. but extra_specs is valid or not, this should be checked when type is created?05:48
*** harlowja_at_home has joined #openstack-cinder05:48
lixiaoy1I mean when extra_specs is set for a type?05:49
jgriffithlixiaoy1: possibly05:49
jgriffithlixiaoy1: honestly it probably adds more confusion than it's worth :(05:49
*** pcaruana has joined #openstack-cinder05:49
lixiaoy1like I create a extra_specs for a type, "pool=test", but without backend specified05:49
jgriffithlixiaoy1: but what I was thinking was for example, if you had type:foo {key1=V1}05:49
jgriffithand you try and retype that to a specific backend05:50
jgriffithif that backend doesn't know what key "key1" is for, or what it does I'm not sure it should say "ok"05:50
jgriffithlixiaoy1: does that make sense?05:50
lixiaoy1oh, no. when driver. retype is called, they are same host05:51
jgriffithlixiaoy1: but my point was that if I say retype and it's same host and it goes to the driver... should the driver be checking what we're feeding it05:51
jgriffithlixiaoy1: or should it just blindly say "ok, pass"05:51
*** wilson-liu has quit IRC05:53
jgriffithlixiaoy1: honestly this is something I don't do in my driver either and have always thought I should go back and fix up05:53
*** deepakcs has quit IRC05:53
*** wilson-liu has joined #openstack-cinder05:54
*** salv-orlando has joined #openstack-cinder05:56
lixiaoy1"checking what we're feeding it.", you mean to check everything about extra_specs? It seems that in kinds of drivers, it just checks it concens.06:00
*** lpetrut has joined #openstack-cinder06:00
*** salv-orlando has quit IRC06:01
*** haomaiwa_ has quit IRC06:01
*** haomaiwa_ has joined #openstack-cinder06:02
jgriffithlixiaoy1: right, my point was that if I send something to the driver that it doesn't understand or know what to do with it... maybe it should report an error06:06
jgriffithlixiaoy1: instead of pretending that everything worked out and is ok06:07
*** gouthamr has quit IRC06:09
lixiaoy1jgriffith: I see. It is not just problem of retype, it is related to extra_spec of a volume type. Like setting extra_specs, check the validation.06:10
*** gouthamr has joined #openstack-cinder06:10
jgriffithlixiaoy1: yeah, kinda06:10
jgriffithlixiaoy1: your patch just reminded me of it :(06:10
jgriffithlixiaoy1: lucky you :)06:10
lixiaoy1jgriffith: seems that we need to split the extra_specs to common and specified for backend06:10
*** sghanekar__ has joined #openstack-cinder06:11
jgriffithlixiaoy1: hehe.. we've tried that06:11
*** smoriya_afk has joined #openstack-cinder06:11
lixiaoy1jgriffith: any bp?06:11
*** smoriya_afk is now known as smoriya06:11
jgriffithlixiaoy1: somewhere...06:11
jgriffithlixiaoy1: but it's a much more convoluted problem06:12
jgriffithlixiaoy1: keep in mind we have things like scoped keys already06:12
lixiaoy1jgriffith: this may call historical problem06:12
jgriffithlixiaoy1: :)06:12
jgriffithlixiaoy1: it is indeed06:12
jgriffithlixiaoy1: which is why we should probably just move forward with the work you've already submitted and let sleeping dogs lie :)06:13
lixiaoy1jgriffith: :)06:13
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (split IO)  https://review.openstack.org/23738706:15
lixiaoy1jgriffith: oh, may I ask you whether we will have cinder meeting this Wednesday? as thanks giving06:15
jgriffithlixiaoy1: ya know, that's a great question06:15
jgriffithlixiaoy1: seems a number of people are off perhaps06:15
* jgriffith checks wiki page06:15
jgriffithlixiaoy1: I suspect there are a lot of folks in the states that will be offline that day06:16
jgriffithlixiaoy1: and there's nothing on the schedule as of yet06:16
lixiaoy1so I asked the questions. In fact, I have a patch about moving volume/encryptors to os-brick. And now is under discusion in Nova and Cinder06:17
jgriffithlixiaoy1: yes, I read the ML posting this morning06:18
jgriffithlixiaoy1: you've been busy :)06:18
*** arnewiebalck__ has joined #openstack-cinder06:18
lixiaoy1no, I just have this thing06:18
lixiaoy1this is fundation of my later work06:18
jgriffithahhh... got ya06:18
lixiaoy1any suggestions?06:19
*** rcernin has joined #openstack-cinder06:19
lixiaoy1Nova people Daniel said Cinder not trusted, :(06:19
*** liverpooler has joined #openstack-cinder06:19
jgriffithlixiaoy1: well... I refrained from inserting my opinion :)06:19
jgriffithlixiaoy1: I kind of agree with his viewpoint06:19
*** sgotliv has joined #openstack-cinder06:19
jgriffithlixiaoy1: I was less concerned about the whole who is/isn't trusted thing06:20
lixiaoy1but if we don't attach, we can't do a lot of things06:20
jgriffithlixiaoy1: but.. IMHO decrypting and backing up, then restoring unencrypted, and copying to glance etc06:20
*** arnewiebalck_ has quit IRC06:20
jgriffithlixiaoy1: sort of defeats the purpose of a user choosing to encrypt their data doesn't it?06:20
lixiaoy1backing up/restore are not related to encryption/decryption06:21
jgriffithlixiaoy1: it sort of seems to me that encrypted volumes are a special case and it is probably "ok" for them to not be able to do all things a regular volume can06:21
lixiaoy1jgriffith: the most important case I am concerned is that I create an encypted volume from image06:21
jgriffithlixiaoy1: well... backup was mentioned by somebody in the posting06:21
lixiaoy1I think that 's not correct06:21
jgriffithlixiaoy1: yes.. that is the single case that I can agree with06:22
lixiaoy1second one is that users want to retype a non-encrypted volume to encrypted06:22
jgriffithlixiaoy1: yeah, that's an interesting one too06:22
jgriffithI hadn't thought of that case..06:22
lixiaoy1these are two items I think we need to add06:22
jgriffithlixiaoy1: I certainly wouldn't argue against your first case06:23
jgriffithBTW.. DuncanT also mentions encryption at rest in the thread which confused me06:23
lixiaoy1you mean that uploading encrypted volumes to image?06:24
*** chirag has joined #openstack-cinder06:25
jgriffithlixiaoy1: here's a list of cases that hemna called out:  http://lists.openstack.org/pipermail/openstack-dev/2015-November/080072.html06:25
jgriffithlixiaoy1: backup is indeed on that list06:26
jgriffithlixiaoy1: as well as copy volume->image06:26
jgriffithlixiaoy1: I don't think either of those cases are valid06:26
jgriffithlixiaoy1: I would agree with you WRT image->volume however06:26
jgriffithlixiaoy1: also the idea of retyping to encrypted06:27
jgriffithlixiaoy1: my recommendation was going to be use nova to do those things06:27
jgriffithlixiaoy1: it's ugly... but it keeps the data secure06:27
jgriffithlixiaoy1: in other words we'd have a dependency on Nova for those special cases06:28
lixiaoy1how to use nova to create a bootable encrypted volume?06:28
jgriffithlixiaoy1: :)06:28
jgriffithlixiaoy1: like we used to do in the old days :)06:28
jgriffithboot an instance, attach a volume and dd06:28
lixiaoy1no, we gave wrong data in the old days06:28
*** pcaruana has quit IRC06:28
jgriffithlixiaoy1: back in nova-volume days?06:28
lixiaoy1Please forget my last words.06:29
chiragHello Everyone, I am facing a problem in cinder backup. Any help would be great. I am using kilo release and have integrated swift for cinder volume backup. During a backup I restarted cinder backup-service and now my volume status is stuck at "backing-up". I am not able to use that volume now. anyone any idea?06:29
lixiaoy1dd? is that too argly?06:29
jgriffithchirag: is there a reset state for backups in Kilo?06:30
lixiaoy1chirag: do you restart cinder backup service in same host?06:30
chirag@jgriffith I am not sure of that06:31
jgriffithchirag: you can try reset state... if that doesn't work you may have to update the column in the db by hand :(06:31
jgriffithchirag: assuming you have admin priveleges06:31
*** rushil has quit IRC06:32
jgriffithchirag: you should be able to reset state to available06:32
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/24902906:32
chirag@lixiaoy1 no swift and cinder are on differnt host06:32
chiragwould try using reset06:32
jgriffithchirag: I think lixiaoy1 means the cinder-backup service06:32
lixiaoy1@chirag: current it will clean the backing backup to error state06:33
chiragsorry .  . Yes I restarted on same host06:33
chiragok would try reset06:33
lixiaoy1jgriffith: I think dd is too trouble. If a lot vm to start from encrypted volumes, do users need to dd one by one? as they need different keys06:35
lixiaoy1jgriffith: key I mean encryption key06:35
chiragthanks it worked :)06:38
jgriffithchirag: cool!06:38
jgriffithlixiaoy1: well actually that's how we do it in cinder anyway :)06:38
jgriffithqemu-convert --> dd > block-device06:38
lixiaoy1jgriffith: but not manual.06:39
*** e0ne has joined #openstack-cinder06:40
lixiaoy1jgriffith: I think this is collide with our future goal, independent SDS06:41
*** chirag has quit IRC06:41
jgriffithlixiaoy1: haha.. you've got me there!06:42
lixiaoy1jgriffith: yeah.06:42
*** deepakcs has joined #openstack-cinder06:43
lixiaoy1jgriffith: thank you John. Glad to talk with you. nice day06:44
jgriffithlixiaoy1: you as well!06:44
jgriffithlixiaoy1: I'll think about that encryption stuff a bit more06:44
jgriffithlixiaoy1: don't know that I'll offer any valuable input or not06:45
lixiaoy1jgriffith: yes06:45
*** sghanekar__ has quit IRC06:45
*** EinstCrazy has quit IRC06:45
lixiaoy1jgriffith: yes is to "think about encryption more"06:46
lixiaoy1jgriffith: of course valuable, at least I know how to answer why not call nova to do encryption06:46
lixiaoy1cinder people is first I have to persuade06:47
*** lpetrut has quit IRC06:47
e0nejgriffith: hi. did you change timezone? :)06:47
jgriffithe0ne: LOL.. nope06:48
jgriffithe0ne: oh dear.. it's late :)06:48
e0nejgriffith: oh.. good. I'll ping you in ~10-12 hours later:)06:48
lixiaoy1sorry John, take you too much time06:48
jgriffithe0ne: haha!!  Ok, sounds good06:48
jgriffithlixiaoy1: nahh.. not at all06:48
*** ankit_ag has joined #openstack-cinder06:48
*** boris-42 has quit IRC06:48
e0nejgriffith: I was a bit surprised to see you here at 9am06:49
*** EinstCrazy has joined #openstack-cinder06:49
jgriffithlixiaoy1: I'm catching up on podcasts and reading :)06:49
e0ne(9am in UTC+3 TZ)06:49
jgriffithe0ne: I'm usually around til about now.. just quiet :)06:49
jgriffithe0ne: incognito :)06:49
e0neoh... so looks like I'm here too early06:49
jgriffithe0ne: you're pretty much always here aren't you?06:50
lixiaoy1eOne: welcome early! you know in06:50
e0nejgriffith: usually, I'm here since 11-12 am im my tz06:50
lixiaoy1eOne: we may have a lot of questions or requirements to talk with you.06:50
e0nelixiaoy1: sure. will you be available here in 3 hours? or at least I need to make my morning coffe06:51
lixiaoy1eOne: OK06:52
e0nelixiaoy1: what timezone are you in?06:52
lixiaoy1eOne: 3 hours is ok06:52
lixiaoy13 hours later06:52
lixiaoy1eOne:utc+8, Iam in China06:52
e0negood. I'll finish my morning busineses and head to office06:53
e0nelixiaoy1: ok, I'll be there next week:)06:53
e0nelixiaoy1: I'll ping you here later today06:53
lixiaoy1eOne: oh, which city are you coming?06:53
*** amrith is now known as _amrith_06:54
e0nelixiaoy1: Nanjing06:54
wanghaojgriffith: hi, John, nice to see you here.06:54
e0nelixiaoy1: hit: my nickname contains 'zero' symbol:06:54
lixiaoy10: like this?06:54
lixiaoy1e0ne: oh, sorry06:55
e0nenp. It's not a big deal. it's just enable or disable notifications06:55
lixiaoy1e0ne: I am in Shanghai, welcome to China06:56
lixiaoy1wanghao: It is too late for John. Seems he has gone06:56
wanghaolixiaoy1: Ops, bad luck...06:56
wanghaolixaoy1: What's John's TZ?06:57
wanghaolixiaoy1: Maybe I need to ping him early.06:58
*** haomaiwa_ has quit IRC07:01
*** haomaiwang has joined #openstack-cinder07:01
wanghaoe0ne: Hi , lucky you're here.  I update the use case about this bp:https://review.openstack.org/#/c/182150/. Would you like to review this again if you have free time.07:03
wanghaolixiaoy1:  Will you attend the next Hackthon in china?07:06
wanghaolixiaoy1: It will be in Chengdu I think.07:06
*** e0ne has quit IRC07:08
*** lpetrut has joined #openstack-cinder07:08
lixiaoy1wanghao: not sure:)07:08
lixiaoy1wanghao: hope I can. you will join, yes?07:08
lixiaoy1wanghao: e0ne will be back in 3 hours07:09
wanghaolixiaoy1: Yes, I think I will go there.07:09
wanghaolixiaoy1: Got it.07:09
wanghaolixiaoy1: It's will great if you can join.07:10
*** nkrinner has joined #openstack-cinder07:10
openstackgerritDeepti Ramakrishna proposed openstack/cinder: CG API should return volume type IDs  https://review.openstack.org/24562407:10
lixiaoy1wanghao: I will try07:10
openstackgerritVincent Hou proposed openstack/cinder: Storwize: Implement v2 replication (global)  https://review.openstack.org/24904207:11
wanghaolixiaoy1: :)07:11
jgriffithnight all07:14
*** jgriffith is now known as jgriffith_away07:15
*** harlowja_at_home has quit IRC07:23
*** lpetrut has quit IRC07:27
*** jerrygb has joined #openstack-cinder07:37
openstackgerritBardia Keyoumarsi proposed openstack/cinder: Volume driver for Coho Data storage solutions  https://review.openstack.org/24669007:41
*** jerrygb has quit IRC07:42
*** smoriya has quit IRC07:44
*** smoriya_afk has joined #openstack-cinder07:46
*** smoriya_afk is now known as smoriya07:46
openstackgerritShay Halsband proposed openstack/cinder: XtremIO: fix iscsi chap discovery bug  https://review.openstack.org/24905107:50
*** anshul has joined #openstack-cinder07:58
*** haomaiwang has quit IRC08:01
*** haomaiwa_ has joined #openstack-cinder08:02
openstackgerritShay Halsband proposed openstack/cinder: XtremIO fix attach readonly bug  https://review.openstack.org/24552708:02
*** [1]Thelo has joined #openstack-cinder08:04
*** Thelo has quit IRC08:06
*** [1]Thelo is now known as Thelo08:06
*** lpetrut has joined #openstack-cinder08:10
*** jerrygb has joined #openstack-cinder08:12
*** salv-orlando has joined #openstack-cinder08:13
*** jerrygb has quit IRC08:17
*** arnewiebalck_ has joined #openstack-cinder08:19
*** salv-orlando has quit IRC08:21
*** arnewiebalck___ has joined #openstack-cinder08:22
*** arnewiebalck__ has quit IRC08:22
*** arnewiebalck_ has quit IRC08:25
*** salv-orlando has joined #openstack-cinder08:29
*** salv-orlando has quit IRC08:33
*** subscope has joined #openstack-cinder08:36
*** arnewiebalck___ has quit IRC08:39
*** arnewiebalck_ has joined #openstack-cinder08:40
*** markus_z has joined #openstack-cinder08:45
*** subscope has quit IRC08:48
*** e0ne has joined #openstack-cinder08:49
*** subscope has joined #openstack-cinder08:52
*** jordanP has joined #openstack-cinder08:59
*** haomaiwa_ has quit IRC09:01
*** haomaiwa_ has joined #openstack-cinder09:01
e0nelixiaoy1: I'm back. dou you still have any questions to me?09:03
openstackgerritwuyuting proposed openstack/cinder: Declare multiattach is True in RBD driver  https://review.openstack.org/24908009:05
*** subscope has quit IRC09:09
*** subscope has joined #openstack-cinder09:10
*** vgridnev has joined #openstack-cinder09:15
*** openstackgerrit has quit IRC09:16
*** openstackgerrit has joined #openstack-cinder09:17
openstackgerritwuyuting proposed openstack/cinder: Declare multiattach is True in RBD driver  https://review.openstack.org/24908009:20
*** pschaef has joined #openstack-cinder09:21
*** ndipanov has quit IRC09:25
*** ndipanov has joined #openstack-cinder09:26
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Implement snapshots-related features for Block Device Driver  https://review.openstack.org/22229209:26
*** mdbooth has joined #openstack-cinder09:27
openstackgerritwanghao proposed openstack/python-cinderclient: support for snapshot management  https://review.openstack.org/24909009:28
*** arnewiebalck__ has joined #openstack-cinder09:33
*** arnewiebalck_ has quit IRC09:33
*** subscope has quit IRC09:37
*** ociuhandu has quit IRC09:42
*** jistr has joined #openstack-cinder09:44
yuriy_n17Duncan: Hi09:48
*** zhangjn has quit IRC09:48
*** EinstCrazy has quit IRC09:48
yuriy_n17DuncanT: Hi09:49
*** arnewiebalck_ has joined #openstack-cinder09:53
*** arnewiebalck__ has quit IRC09:53
*** andymaier has joined #openstack-cinder09:54
*** alexschm has joined #openstack-cinder09:55
*** haomaiwa_ has quit IRC10:01
*** haomaiwa_ has joined #openstack-cinder10:02
*** zhenguo has quit IRC10:02
e0neDuncanT: hi. one proposal about imageutils from oslo-incubator. seems to be better to move it to oslo.utils. it's used by nova and ironic too10:04
*** aix has joined #openstack-cinder10:05
e0nedulek: ^^10:08
*** gcb has quit IRC10:11
*** pschaef has quit IRC10:18
*** houming has quit IRC10:21
*** arnewiebalck_ has quit IRC10:23
*** arnewiebalck_ has joined #openstack-cinder10:23
*** arnewiebalck__ has joined #openstack-cinder10:30
*** arnewiebalck_ has quit IRC10:30
*** subscope has joined #openstack-cinder10:30
*** lixiaoy1 has quit IRC10:30
*** andymaier has left #openstack-cinder10:31
*** andymaier has joined #openstack-cinder10:31
*** lixiaoy1 has joined #openstack-cinder10:31
lixiaoy1DuncanT: e0ne: I hope I can get suggestions about "copy encryptors from nova to os-brick"10:33
*** arnewiebalck_ has joined #openstack-cinder10:33
*** arnewiebalck__ has quit IRC10:35
*** salv-orlando has joined #openstack-cinder10:38
*** subscope has quit IRC10:41
*** salv-orlando has quit IRC10:42
*** markus_z has quit IRC10:47
*** lprice1 has joined #openstack-cinder10:55
*** lprice has quit IRC10:55
*** bswartz has quit IRC10:56
*** lixiaoy1 has left #openstack-cinder10:57
*** lixiaoy1_ has joined #openstack-cinder10:58
geguileoAnybody else getting 503 errors from review.openstack.org?10:59
*** lixiaoy1_ is now known as lixiaoy110:59
*** bswartz has joined #openstack-cinder11:00
*** haomaiwa_ has quit IRC11:01
*** haomaiwang has joined #openstack-cinder11:01
geguileoOk, working again11:02
*** dims has joined #openstack-cinder11:06
*** _amrith_ is now known as amrith11:06
*** houming has joined #openstack-cinder11:08
*** EinstCrazy has joined #openstack-cinder11:08
*** EinstCrazy has quit IRC11:09
*** EinstCrazy has joined #openstack-cinder11:09
*** rcernin is now known as rcernin|lunch11:13
*** aix has quit IRC11:14
*** aix has joined #openstack-cinder11:15
openstackgerritVictor Stinner proposed openstack/cinder: Port IBM storewize_svc driver to Python 3  https://review.openstack.org/24287811:18
lixiaoy1geguileo:Hi Gorka, with your spec "remove cinder api races", can c-api run A-A?11:18
openstackgerritSzymon Wróblewski proposed openstack/cinder: Add some missing fields to Volume object  https://review.openstack.org/24084711:19
*** ociuhandu has joined #openstack-cinder11:21
*** jwcroppe has quit IRC11:29
*** jwcroppe has joined #openstack-cinder11:32
duleklixiaoy1: Nope, there's a lot of other problems. Take a look at https://review.openstack.org/#/c/232599/11:35
*** jwcroppe has quit IRC11:36
duleklixiaoy1: Ah, you've meant c-api. I believe we're having problems with race conditions even with single API service (we have multiple workers).11:36
duleklixiaoy1: So running multiple c-api may make these cases more visible. But I believe people run multiple c-api's.11:37
*** amrith is now known as _amrith_11:37
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Skip unsupported datastore types  https://review.openstack.org/24915011:41
*** markus_z has joined #openstack-cinder11:43
DuncanTlixiaoy1: My only suggestion at this point is to let the discussion run for a while, it will take time for people to come to an agreement11:48
nikeshmDuncanT: any idea about ATC coupons, i have contributed in liberty release so will i get coupon for mitaka ?11:52
DuncanTe0ne: Moving it to oslo.utils seems entirely reasonable11:52
DuncanTnikeshm: I've no idea, sorry. I suggest you keep contributing just to be sure ;-)11:53
*** gcb has joined #openstack-cinder11:53
e0neDuncanT: thanks. I just wanted to confirm it. patch is already proposed: https://review.openstack.org/#/c/249107/11:53
nikeshmDuncanT: thanks11:53
*** Rekha has quit IRC11:54
duleknikeshm: The policy is getting less-inclusive each release. For Liberty summit only contributors to Liberty release were given ATC status (and coupons). I would expect it to be similar for Mitaka.11:55
*** markus_z is now known as markus_z_meeting11:59
*** subscope has joined #openstack-cinder12:00
*** haomaiwang has quit IRC12:01
*** haomaiwa_ has joined #openstack-cinder12:02
*** alexschm has quit IRC12:04
*** alexschm has joined #openstack-cinder12:05
nikeshmdulex: thanks12:09
*** subscope has quit IRC12:13
*** subscope has joined #openstack-cinder12:14
*** salv-orlando has joined #openstack-cinder12:17
*** zhangjn has joined #openstack-cinder12:18
*** zhangjn has quit IRC12:18
*** zhangjn has joined #openstack-cinder12:19
*** zhangjn has quit IRC12:20
*** zhangjn has joined #openstack-cinder12:20
*** zhangjn has quit IRC12:20
*** zhangjn has joined #openstack-cinder12:21
*** zhangjn has quit IRC12:22
*** salv-orlando has quit IRC12:22
*** zhangjn has joined #openstack-cinder12:22
*** zhangjn has quit IRC12:23
*** EinstCrazy has quit IRC12:26
*** EinstCrazy has joined #openstack-cinder12:28
*** zhangjn has joined #openstack-cinder12:29
*** deepakcs has quit IRC12:31
*** jwcroppe has joined #openstack-cinder12:33
*** zhangjn has quit IRC12:33
*** zhangjn has joined #openstack-cinder12:34
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Unit test refactoring  https://review.openstack.org/24917412:34
*** alejandrito has joined #openstack-cinder12:34
*** EinstCra_ has joined #openstack-cinder12:35
*** EinstCrazy has quit IRC12:35
openstackgerritVipin Balachandran proposed openstack/cinder: VMware: Unit test refactoring  https://review.openstack.org/24917512:36
*** rcernin|lunch is now known as rcernin12:36
openstackgerritMichal Dulko proposed openstack/cinder: Block subtractive operations in DB migrations  https://review.openstack.org/24876812:39
openstackgerritRonen Mesonzhnik proposed openstack/cinder-specs: Support backup import on another Storage database  https://review.openstack.org/24917612:39
*** diegows has joined #openstack-cinder12:40
*** wolsen has quit IRC12:41
*** zhipeng has joined #openstack-cinder12:42
*** yhayashi has quit IRC12:43
*** subscope has quit IRC12:44
*** isaacb has joined #openstack-cinder12:45
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Implement snapshots-related features for Block Device Driver  https://review.openstack.org/22229212:45
openstackgerritRonen Mesonzhnik proposed openstack/cinder-specs: Support backup import on another Storage database  https://review.openstack.org/24917612:46
*** subscope has joined #openstack-cinder12:46
*** jwcroppe has quit IRC12:50
*** gcb has quit IRC12:51
*** andymaier has quit IRC12:59
*** haomaiwa_ has quit IRC13:01
*** markus_z_meeting is now known as markus_z13:01
*** haomaiwang has joined #openstack-cinder13:01
*** zhenguo has joined #openstack-cinder13:01
openstackgerritIvan Kolodyazhny proposed openstack/cinder: Implement snapshots-related features for Block Device Driver  https://review.openstack.org/22229213:01
*** gcb has joined #openstack-cinder13:04
*** houming has quit IRC13:04
*** salv-orl_ has joined #openstack-cinder13:05
*** marcusvrn_ has quit IRC13:05
tbarronsmcginnis: We now have a registration link for the hotel for the mid-cycle meetup: http://hiltongardeninn.hilton.com/en/gi/groups/personalized/R/RDUSPGI-NET-20160125/index.jhtml13:06
tbarronsmcginnis: please take a look and make sure it looks sane13:06
tbarronsmcginnis: if so, I'll update the etherpad accordingly and you can send a followup announcement to the list13:07
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Add get_extra_metadata to nfs backup driver  https://review.openstack.org/15396913:11
*** akerr has joined #openstack-cinder13:11
*** haomaiwang has quit IRC13:11
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Add get_extra_metadata to nfs backup driver  https://review.openstack.org/15396913:12
*** bluex has joined #openstack-cinder13:12
*** bluex has quit IRC13:12
*** bluex has joined #openstack-cinder13:13
*** wolsen has joined #openstack-cinder13:14
openstackgerritMerged openstack/cinder: XtremIO fix attach readonly bug  https://review.openstack.org/24552713:16
*** shausy has quit IRC13:23
openstackgerritRonen Mesonzhnik proposed openstack/cinder: Support backup import on another Storage database  https://review.openstack.org/15396913:25
*** markvoelker has quit IRC13:35
*** diablo_rojo has joined #openstack-cinder13:42
*** martyturner has joined #openstack-cinder13:46
openstackgerritYuriy Nesenenko proposed openstack/cinder: Remove stubs and add resource cleanup  https://review.openstack.org/24921813:46
*** haomaiwang has joined #openstack-cinder13:49
*** markvoelker has joined #openstack-cinder13:53
openstackgerritYuriy Nesenenko proposed openstack/cinder: Remove stubs and add resource cleanup  https://review.openstack.org/24921813:57
*** houming has joined #openstack-cinder13:57
*** salv-orl_ has quit IRC14:00
*** zhangjn has quit IRC14:00
*** subscope has quit IRC14:00
*** haomaiwang has quit IRC14:01
smcginnistbarron: That looks good, thanks!14:01
smcginnistbarron: You were able to get them to change the restruction of NetApp paying for any unused rooms, right?14:01
*** haomaiwa_ has joined #openstack-cinder14:01
tbarronsmcginnis: yes14:01
smcginnistbarron: Awesome. Thanks for taking care of that.14:02
tbarronsmcginnis: gotta drive to work, will update the etherpad when I get there14:02
*** gouthamr_ has joined #openstack-cinder14:03
*** rcernin has quit IRC14:03
*** subscope has joined #openstack-cinder14:04
*** zhipeng has quit IRC14:04
smcginnistbarron: Thanks!14:05
openstackgerritPeter Wang proposed openstack/cinder: VNX: Fix failure in SnapCopy feature  https://review.openstack.org/24301714:05
*** dustins has joined #openstack-cinder14:06
*** gouthamr has quit IRC14:06
openstackgerritFelix Ma proposed openstack/os-brick: Raise exception in find_multipath_device  https://review.openstack.org/24923114:08
*** subscope has quit IRC14:10
*** subscope has joined #openstack-cinder14:11
nikeshmsmcginnis: whats up14:11
nikeshmsmcginnis: hope you enjoyed the anniversary in parris14:12
*** ankit_ag has quit IRC14:12
nikeshmsmcginnis: but really attacks in parris was very sad14:13
*** subscope has quit IRC14:13
*** subscope has joined #openstack-cinder14:13
*** merooney has joined #openstack-cinder14:13
*** subscope has quit IRC14:16
*** subscope has joined #openstack-cinder14:17
smcginnisnikeshm: Hey!14:19
smcginnisnikeshm: Yes, it was a very good weekend.14:19
*** eharney has joined #openstack-cinder14:19
smcginnisnikeshm: Very sad, but it was good to see that the city is still going and isn't going to let those events destroy everything.14:20
*** diablo_rojo has quit IRC14:21
*** jgregor has joined #openstack-cinder14:22
nikeshmsmcginnis: congrats for the anniversary, BTW, my marriage is fixed on 29th january 201614:23
*** jerrygb has joined #openstack-cinder14:23
smcginnisnikeshm: Wow, that's coming up quick! Congrats!14:27
*** baumann has joined #openstack-cinder14:28
*** mc_nair has joined #openstack-cinder14:32
*** akshai has joined #openstack-cinder14:32
*** mriedem_away is now known as mriedem14:33
*** baumann has quit IRC14:35
*** ntpttr has joined #openstack-cinder14:35
*** willsama has joined #openstack-cinder14:36
*** mtanino has joined #openstack-cinder14:37
*** arnewiebalck_ has quit IRC14:37
*** arnewiebalck_ has joined #openstack-cinder14:38
*** goodygum_ is now known as noqa_v_qoovnie14:44
*** jwcroppe has joined #openstack-cinder14:47
*** jgriffith_away is now known as jgriffith14:47
dulekDuncanT, e0ne: I'm not sure why this wasn't catched by IRC bot, but: https://review.openstack.org/#/c/249255/14:54
*** jungleboyj has joined #openstack-cinder14:55
*** jerrygb has quit IRC14:56
*** jerrygb has joined #openstack-cinder14:57
*** houming has quit IRC14:58
*** diablo_rojo has joined #openstack-cinder14:58
*** akshai has quit IRC14:59
*** jwcroppe has quit IRC14:59
*** akshai has joined #openstack-cinder15:00
*** davechen has joined #openstack-cinder15:00
*** haomaiwa_ has quit IRC15:01
*** haomaiwa_ has joined #openstack-cinder15:01
*** boris-42 has joined #openstack-cinder15:02
*** akshai has quit IRC15:02
*** salv-orlando has joined #openstack-cinder15:04
*** houming has joined #openstack-cinder15:05
DuncanTDulek: Thanks, I'll take a look15:06
*** breitz has quit IRC15:08
*** breitz has joined #openstack-cinder15:08
*** dims_ has joined #openstack-cinder15:08
*** dims has quit IRC15:09
openstackgerritSzymon Wróblewski proposed openstack/cinder: Tooz locks  https://review.openstack.org/18353715:11
openstackgerritSzymon Wróblewski proposed openstack/cinder: Support locking Versioned Objects  https://review.openstack.org/24635215:11
*** leo___ has joined #openstack-cinder15:14
*** jerrygb has quit IRC15:15
openstackgerritSzymon Wróblewski proposed openstack/cinder: Support locking Versioned Objects  https://review.openstack.org/24635215:15
*** jerrygb has joined #openstack-cinder15:16
*** gcb has quit IRC15:17
*** thangp has joined #openstack-cinder15:18
*** markvoelker has quit IRC15:18
*** markvoelker has joined #openstack-cinder15:18
*** sgundur has joined #openstack-cinder15:18
*** vgridnev has quit IRC15:19
e0neDulek: thanks for the update15:21
*** wN has quit IRC15:23
*** crose has joined #openstack-cinder15:25
tbarronsmcginnis: I updated https://wiki.openstack.org/wiki/Sprints/CinderMitakaSprint and the accompanying etherpad with the lodging reservations link.15:26
*** subscope has quit IRC15:26
*** wN has joined #openstack-cinder15:28
*** wN has joined #openstack-cinder15:28
*** xyang has joined #openstack-cinder15:30
*** jerrygb has quit IRC15:33
*** salv-orlando has quit IRC15:36
*** subscope has joined #openstack-cinder15:36
*** akshai has joined #openstack-cinder15:36
*** asselin_ has joined #openstack-cinder15:36
*** asselin has joined #openstack-cinder15:36
*** subscope has quit IRC15:40
*** salv-orlando has joined #openstack-cinder15:40
*** asselin_ has left #openstack-cinder15:40
*** baumann has joined #openstack-cinder15:41
smcginnisrhedlind: ping15:41
smcginnisrhedlind: Can we get a recheck on this: https://review.openstack.org/#/c/242139/15:41
*** asselin has left #openstack-cinder15:41
*** asselin has quit IRC15:41
*** asselin has joined #openstack-cinder15:42
*** houming has quit IRC15:44
*** houming has joined #openstack-cinder15:45
sgundurjgriffith: thanks for initiating this discussion , at this point I am going to un-assign myself from the bug15:46
sgundurjgriffith: thanks :)15:46
jgriffithsgundur: :(  sorry15:46
jgriffithsgundur: I still say it's broken, but I'm not arguing15:47
e0nejgriffith: morning, John15:49
*** haomaiwa_ has quit IRC15:49
smcginnisjgriffith: Do you sleep? :)15:49
jgriffithe0ne: hola15:49
smcginnise0ne: You too.15:49
*** crose has quit IRC15:49
jgriffithsmcginnis: yes... 15 minutes every 2 hours15:49
smcginnisjgriffith: Hah!15:50
e0nesmcginnis: what are you talking about?15:50
jgriffithsmcginnis: I learned it from Kramer on Seinfeld LOL15:50
smcginnise0ne: Thought I saw you on at all hours too.15:50
smcginnisjgriffith: :)15:50
jgriffithe0ne: yeah... sort of the pot calling the kettle black eh?15:50
diablo_rojojgriffith: So you sleep the uberman schedule?15:51
*** EinstCra_ has quit IRC15:51
e0nejgriffith: em... I failed to translate what you mean:(15:51
*** salv-orlando has quit IRC15:51
jgriffithdiablo_rojo: LOL... that's awesome!15:51
smcginnisdiablo_rojo: You calling jgriffith and uberman? :)15:51
jgriffithsmcginnis: meh.. it happens15:52
diablo_rojojgriffith: Have you never heard of that before?15:52
diablo_rojosmcginnis: Maybe?15:52
jgriffithdiablo_rojo: no, I have not.. but I'm reading about it right now15:52
jgriffithdiablo_rojo: very interesting15:53
jgriffithdiablo_rojo: are you a polyphasic sleeper?15:53
diablo_rojojgriffith: I tried to do some of the other ones. I pretty consistently did the Everyman during college. Sometimes the chase.15:53
e0nejgriffith: will you have a time to discuss cinder w/o nova? I would like to make a desision how will we implement it: cinderclient or not15:53
nikeshmsmcginnis: thanks15:53
jgriffithe0ne: certainly... let's talk about it15:53
jungleboyjdiablo_rojo: Likes to sleep whenever she can.15:54
jgriffithdiablo_rojo: BTW, I was kidding.. I don't really sleep like that.  Although there have been periods in my past when I did15:54
e0nejgriffith: you was against to make it inside cinderclient at the begining..15:54
diablo_rojojgriffith: I figured :) I just cant resist the oppoortunity for a good infographic.15:55
e0nejgriffith: IMO, if our client will deal not only with API, it will be more confusing15:55
jgriffithdiablo_rojo: when I finished my 6 year sleep depravation experiment (Militairy) I like deep sleep :)15:55
smcginnisjungleboyj: ohayo gozaimasu15:55
jgriffithe0ne: so I'm not sure why?15:55
jungleboyjsmcginnis: Ohayo gozimasu .15:55
jgriffithe0ne: so if I say "cinder get-connection-info"15:55
* jungleboyj is fighting the worst jet lag ever.15:56
e0nejgriffith: ok. what about 'attach'?15:56
jgriffithe0ne: and an IQN comes back... that I can do whatever I want with, how is that "confusing"?15:56
jungleboyjsmcginnis: Feel like I should be saying kunbunwa15:56
diablo_rojojgriffith: Ha ha sounds painful. Then again, so was my high school sleep schedule (max of 5.5 hours, average of 4).15:56
jgriffithgood times! :)15:56
smcginnisjungleboyj: ;)15:57
*** leo___ has quit IRC15:57
e0nejgriffith: 'connection info' is pretty simple15:57
*** subscope has joined #openstack-cinder15:57
jgriffithe0ne: so what's the confusing part?15:57
e0nejgriffith: but adding new(second) 'attach' method doesn't sounds good15:57
jgriffithe0ne: why not?15:57
smcginnisjgriffith, e0ne: IIRC, part of the reason for a new client would be there will be similar but differnt calls.15:58
e0nejgriffith: we've already have 'attach'15:58
jgriffithe0ne: "cinder set-status-attached xxxxxxx"15:58
smcginnisThat we don't want to confuse "normal" end users with.15:58
jgriffithe0ne: "cinder mark-as-in-use xxxxx"15:58
smcginnisSo making it a separate client makes it clear and separate.15:58
*** bluex has quit IRC15:59
jgriffithsmcginnis: sure, but there's all sorts of duplication of code there15:59
jgriffithsmcginnis: and some confusion IMO when you throw in things like OSC15:59
*** subscope has quit IRC15:59
e0nejgriffith: we have to move all duplicates code to something like oslo.client15:59
smcginnisjgriffith: The duplication of code is a big concern for me.15:59
jgriffithsmcginnis: e0ne and BTW, there is NO attach command exposed to users currently15:59
sgundurjgriffith:I agree with you that it can be done better, no arguments there :)16:00
sgundurjgriffith:just that I am not sure how we could make all calls to go through scheduler16:00
*** salv-orlando has joined #openstack-cinder16:00
jgriffithsgundur: yeah... I don't even know if that's necessary16:00
smcginnisjgriffith, e0ne: So maybe we should try sticking with one client. Implement the changes and see where the nasty parts really are.16:00
jgriffithsgundur: my argument was (and still is) that most of those calls are pointless, and even the ones that aren't are only there to handle drivers that don't handle errors themeselves16:00
*** isaacb has quit IRC16:01
e0nejgriffith: sure. and we don't need to expoxe 'set-status-attached' to CLI16:01
smcginnisjgriffith, e0ne: And if it ends up not being a big deal - great.16:01
*** zhipeng has joined #openstack-cinder16:01
jgriffithsmcginnis: e0ne so I think we'd be better off... but I could be wrong.  One thing to remember is you can hide the command16:01
sgundurjgriffith: yep , I see your point16:01
smcginnisjgriffith, e0ne: If we find some legitimate (and concrete) examples where it could be an issue, should be as bad to refactor out into a different client.16:01
jgriffithsmcginnis: e0ne so it only shows up if the admin turns it on via policy16:01
jgriffithsmcginnis: sounds like a pragmatic way to approach it16:02
e0nejgriffith, smcginnis: cinderclient doesn't know anything about policies16:02
jgriffithe0ne: no, but the API's on the receiving/Cinder side do :)16:02
e0nejgriffith: it's still unclear for me: how to hide a command on the client-side before connection to API16:03
jgriffithe0ne: and if I'm a user that doesn't have the correct credentials I never even see the command when I do for example a "cinder help"16:03
jgriffithe0ne: the other option/idea is just make it a special extension16:04
smcginnisMaybe we don't need to hide them. Just name them something useful.16:04
jgriffithe0ne: and introduce a mechanism to enable/disable it16:04
*** salv-orlando has quit IRC16:04
jgriffithsmcginnis: yeah... that may work too16:04
e0nejgriffith: I like it. we can make it as a separate binary16:05
jgriffithsmcginnis: e0ne so you know we could do things like set and env var that says "show external device methods"16:05
jgriffithe0ne: or yes, an external binary maybe16:05
e0nejgriffith: got you idea with hiding commands:)16:05
e0nejgriffith: I'll try it16:06
jgriffithe0ne: excellent!16:06
e0nejgriffith: external binary - it could raise a code duplication too:(. Not sure for now, I need to check it16:06
*** ericksonsantos has joined #openstack-cinder16:07
thingeewhew back from the honeymoon :) Hi everyone!16:08
smcginnisthingee: Welcome back!16:08
jgriffithwelcome back thingee16:08
thingeethat vacation was more needed than I thought!16:08
e0nejgriffith: one more concern about "hiding commands": do we to add os-brick as soft-dependency? from the packagers perspective, it's not a big deal16:08
e0nejgriffith: but from the cloud providers - does they want to have potential issues when user find "hidden command"?16:09
e0nethingee: welcome back!16:09
jgriffithe0ne: I honestly don't understand the whole brick relationship here to be honest16:10
e0nejgriffith: I need to call connect_volume/disconnect_volume os-brick's methods during attach/detach procedure16:11
e0nejgriffith: e.g.: https://github.com/e0ne/python-brickclient/blob/master/brickclient/client.py#L8816:11
*** vgridnev has joined #openstack-cinder16:11
*** nkrinner has quit IRC16:12
*** anshul has quit IRC16:15
*** rcernin has joined #openstack-cinder16:16
*** alexschm has quit IRC16:20
*** aviramb has quit IRC16:23
jgriffith"honestly to be honest" ... just call me Abe :)16:23
e0nejgriffith: I'm going to expose not only 'get-connector' info (brick provides it too) but full cycle of attach/detach to any host16:23
jgriffithe0ne: yeah :(16:24
e0nejgriffith: that's why you proposed to move it in a saparate client a while ago16:25
jgriffithe0ne: I proposed moving what to a separate client?16:25
*** _amrith_ is now known as amrith16:26
openstackgerritzhangsong proposed openstack/os-brick: Improve the remove_multipath_device() method for linuxscsi  https://review.openstack.org/24930816:26
jgriffithe0ne: brick?  Not hardly, that is no longer anything remotely like what I initially talked about.  It's good... it's just not what we initially set out on16:26
jgriffithe0ne: it sounds like you have a good plan; you should just run with it16:27
e0nejgriffith: if i remember it right. I could be wrong16:27
jgriffithe0ne: yeah... pretty sure there's a misunderstanding there :)16:27
e0nejgriffith: that's why we are duscissing about it now16:27
jgriffithe0ne: here's my only concern16:28
jgriffithe0ne: well... concerns:16:28
e0neand IMO it's not bad - we as community can find a better solution in such discussions16:28
jgriffith1. We actually get something that works this release16:28
openstackgerritMerged openstack/cinder: Fix the bug of OSError when convert image  https://review.openstack.org/24315216:29
jgriffith2. We work it out iteratively (we don't need to boil the ocean), just come up with a plan and start executing it in phases16:29
*** zhenguo has quit IRC16:29
jgriffith3. I don't think Brick and a new client should be a requirement, but may be a nice to have16:29
*** xiaohui_ has joined #openstack-cinder16:29
*** rebase has joined #openstack-cinder16:30
*** dims_ has quit IRC16:30
jgriffith4. We shouldn't get wrapped around the axle and not have anything useful because we have too many dependencies floating around with external libs, clients etc16:30
jgriffithI think that's it...16:30
jgriffithSo I would propose you start with just the cinder and cinderclient pieces... with the design for other components being kept in mind16:31
e0neI'm 100% agree with you for #1,2,416:31
jgriffithProvide a way for a caller to get tgt-iqn info and a way to set the status of the volume16:31
jgriffiththat's a great start... and then build and refine on top of that16:31
e0negot your point, sounds absolutely reasonable16:32
jgriffithyou can automate the *world* as we go along but at least there's an intermediate point where people can consume the service16:32
jgriffithin any way they choose16:32
*** jdurgin1 has joined #openstack-cinder16:32
e0neone a little coment from me: to get iqn I need brick, so it will be a requirement16:32
*** sgundur has quit IRC16:32
jgriffithe0ne: I realize it's not perfect, and there are gaps/holes (RBD, FC etc), but might be better to fill those as we go and learn more16:33
* jgriffith is not very good at predicting the future 16:33
e0nejgriffith: it's a good plan, John16:33
jgriffithe0ne: cool16:33
jgriffithe0ne: I'm excited about the whole thing16:34
xiaohui_geguileo: are you available now?16:34
e0nejgriffith: now I know in a what way I should to go! it was the main question for me:)16:34
geguileoxiaohui_: On a meeting, sorry16:34
jgriffithe0ne: and you know if there's something you don't have time to work on or would like help... just let me know16:34
*** rebase has quit IRC16:34
*** simondodsley has joined #openstack-cinder16:34
e0nejgriffith: sure, I'll do:)16:34
e0nethanks for the help16:35
e0nenow, it's a good time to take a break and head to home16:35
xiaohui_geguileo: ok, so reply on your comments about your concerns is ok for you then?16:35
geguileoxiaohui_: Yes, please  :-)16:35
xiaohui_geguileo: ok, thanks!16:36
*** sgundur has joined #openstack-cinder16:36
jgriffithmtanino: minor question regarding https://review.openstack.org/#/c/247886/16:38
mtaninojgriffith: sure16:39
jgriffithmtanino: the persistence files are specific for tgt driver16:39
jgriffithmtanino: so shouldn't we move that write_configfile to the tgt class?16:39
mtaninojgriffith: I think tgt and cxt16:39
jgriffithmtanino: I guess it doesn't matter16:40
mtaninojgriffith: ah, let me check a little16:40
jgriffithmtanino: as it's called by the inherited classes....16:40
mtaninoI guestt so16:40
*** aix has quit IRC16:40
*** salv-orlando has joined #openstack-cinder16:41
*** salv-orlando has quit IRC16:41
jgriffithmtanino: it's technically just fine, I'm trying to figure out however if there's a way to keep those driver classes somewhat self-descriptive16:41
jgriffithmtanino: maybe silly....16:41
*** salv-orlando has joined #openstack-cinder16:41
jgriffithmtanino: but even consider just making that write_configfile a part of a utils file and set it as a static method?16:41
jgriffithmtanino: it doesn't appear to rely on any object members16:41
jgriffithmtanino: because you're correct, there are others using it now16:42
mtaninojgriffith: correct, so which one is good place to move it? tgt class or under utils?16:43
jgriffithmtanino: well, tgt class doesn't make as much sense as I thought it did :(16:43
mtaninojgriffith: hmm16:44
jgriffithmtanino: I suppose there's no harm in having it in iscsi.py16:44
jgriffithmtanino: I wonder if it's worth defining it as a static method?16:44
mtaninojgriffith: Ah, ok!16:44
jgriffithmtanino: that makes it clear I think16:45
jgriffithmtanino: and then anybody can use it16:45
jgriffithmtanino: if there's ever another case of somebody wanting to use it then maybe we move it to utils16:45
mtaninojgriffith: ok. make sense.16:46
*** dims has joined #openstack-cinder16:46
jgriffithmtanino: ok, if you're sure that seems right.  I could be completely off my rocker ya know :)16:46
jgriffithmtanino: if you think it's overly picky don't be afraid to say so16:46
mtaninojgriffith: I originally thought we can use this method for iet, lio, etc but currently only cxt and tgt are needed this method.16:46
jgriffithmtanino: yeah... and it's not a terribly big deal since those that need it just call in their respective sub-classes16:47
jgriffithmtanino: but my concern would be somebody sees this at some point in the future and says "oh, I should call this from the manager"16:47
mtaninojgriffith: In that case, we should move this as a more generic method.16:48
jgriffithmtanino: also I just sort of think we have a little too much indirection going on16:48
jgriffithmtanino: that's what I think... make it a true helper method independent of iscsi and targets16:48
*** jdurgin1 has quit IRC16:48
*** zhipeng has quit IRC16:48
jgriffithmtanino: because frankly I think there are some other places where we could benefit from this hardening of writing out to files16:49
openstackgerritMerged openstack/cinder: CG API should return volume type IDs  https://review.openstack.org/24562416:49
ntpttroops wrong window had focus16:49
mtaninojgriffith: ah, If there are some places which we have to cover to atomic file writing, we might be implement this method more generic?16:51
jgriffithmtanino: :)16:51
*** EinstCrazy has joined #openstack-cinder16:51
jgriffithmtanino: exactly16:51
jgriffithmtanino: we don't have to fully genericise it now, but maybe put a TODO pointing out to folks it's there and we may want to use it for such purposes16:52
mtaninojgriffith: OK. let me think again this method for more generic purpose.16:53
jgriffithmtanino: great, thank you as always mtanino !16:53
*** kbyrne has quit IRC16:53
anishoh great, mtanino already gave the explanation :)16:54
mtaninoanish: hi :)16:54
* anish waves16:54
ntpttrIf I have the 'self.client.can_send_version' method called twice in an rpcapi function, how can I mock that in tests? Right now the mock only seems to acknowledge one of the two values I pass into that method, and it makes the test fail16:54
*** jwcroppe has joined #openstack-cinder16:55
jgriffithntpttr: hmm... your mock should cover the object itself and work no matter how many times it's called16:55
*** markstur has quit IRC16:56
jgriffithntpttr: are you using "with mock.object......" ?16:56
*** EinstCrazy has quit IRC16:56
*** gouthamr_ has quit IRC16:57
ntpttryes, here I'll pastebin the test and rpc function16:57
*** e0ne has quit IRC16:58
ntpttrHere is the current not working version I've got http://paste.openstack.org/show/479892/16:59
ntpttrit recognizes being called with version 1.33, but not with version 1.36, it's the 1.36 call that's failing16:59
jgriffithntpttr: ahh... yeah, that won't work that way17:00
*** diegows has quit IRC17:00
jgriffithntpttr: because you use a mock with return value for both17:01
jgriffithand it apparantly just uses the last one (1.33)17:01
ntpttrjgriffith: I figured that would cause an issue, but it didn't work with one mock call either17:01
dulekntpttr: use mock.size_effect = [val1, val2, val3]17:01
jgriffithntpttr: If it were me I'd try using a single mock with a side-effect that calls a method and returns false17:01
ntpttrdulek jgriffith: ah okay, hadn't heard of that, thank you17:02
jgriffithntpttr: also, I personally find using the form "with mock.patch.object xxxxx"{ } easier to deal with17:02
dulekntpttr: I believe you're working on the quotas patch. Right now we're testing only the lowest and newest versions of the calls, no intermediate ones.17:02
ntpttrdulek: Yes, I'll add more tests for intermediate as well17:02
dulekntpttr: Our RPC clients unit tests need solid refactoring, so I wouldn't mind if you won't - it's really painful to work with them.17:03
jgriffithntpttr: take a look at something like this:  https://github.com/openstack/cinder/blob/master/cinder/tests/unit/test_solidfire.py#L28117:03
jgriffithntpttr: or just about any unit test that eharney has submitted over the last year is IMO a great template to follow17:03
*** jwcroppe has quit IRC17:03
ntpttrdulek: do you mean it wouldn't be a big deal if I don't, or that you would prefer that I don't? It wouldn't be too much trouble if it would be helpful17:04
ntpttrjgriffith: thank you, I'll check that out17:04
*** chenying has quit IRC17:05
*** chenying has joined #openstack-cinder17:05
openstackgerritRyan McNair proposed openstack/python-cinderclient: Fix v2 qos-key command  https://review.openstack.org/24894417:08
*** leeantho has joined #openstack-cinder17:09
*** markstur has joined #openstack-cinder17:11
*** jccnd has joined #openstack-cinder17:11
openstackgerritMerged openstack/cinder: Port IBM storewize_svc driver to Python 3  https://review.openstack.org/24287817:14
ntpttrjgriffith: could you clarify one more thing for me? If I change the format of the test would it look like mock.patch.object(oslo_messaging.RPCClient, 'can_send_version', side_effect=?', and then would the assert_called_with statements still be necessary? Sorry, I'm still getting used to the mock framework, I haven't had a lot of experience with it yet17:16
*** jwcroppe has joined #openstack-cinder17:17
*** dustins has quit IRC17:17
jgriffithntpttr: yes, that's how it would look, that's just a suggestion of course, you can use the syntax/format you have but change it to a side-effect17:17
jgriffithntpttr: as far as the assert_called... I think that still works but honestly I don't usually use those :)17:18
*** kfarr has joined #openstack-cinder17:18
jgriffithntpttr: I cheat and do things like add a counter in the method and assert it's incremented to what I expected :)17:18
*** dims has quit IRC17:18
jgriffithntpttr: it's stupid I know17:18
*** dims_ has joined #openstack-cinder17:18
ntpttrjgriffith: Gotcha, I guess my issue was more like since I'm getting rid of the can_send_version parameters in the method itself, how could I even call assert_called_with since those variables don't exist?17:19
*** david-ly_ has joined #openstack-cinder17:19
ntpttrjgriffith: do I put that logic inside of the side effect somehow?17:20
jgriffithntpttr: that's what I do, but it's probably not the best answer :)17:20
jgriffithntpttr: I've never been good with the mock assert called stuff AT ALL17:20
jgriffithYou'll have to play with it or see if somebody else here has better ideas/input for you17:21
jgriffithntpttr: but what I've done in the past is in the test_xxxxx()17:21
jgriffithSet a variable "called_count = 0"17:21
jgriffithThen in my side effect (defined in the method), increment called_count17:21
ntpttrjgriffith: hah it does seem to be a bit to get my head around, I'll try experimenting with it some more17:21
jgriffiththen do a poor mans assert :)17:21
*** lpetrut has quit IRC17:21
ntpttrjgriffith: yeah, that counter makes sense, but then I don't think that would also maintain that the function was called with the specific parameters17:22
jgriffithhas the same result without the mind bending of mock :)17:22
mtanino[Question] If we have multi-hypervisor(KVM, VMware, etc) and multi cinder backends(LVM, VMDK, etc), can we control of target cinder backend when we boot an instance from volume?17:22
jgriffithntpttr: well... you need to define your side-effect with the same params as the method your faking out17:22
jgriffithntpttr: so you can read them and do whatever you want with the arguments inside of it17:22
jgriffithntpttr: so multiple counters, sentinel vars whatever17:22
jgriffithntpttr: you have complete creative control :)17:23
ntpttrjgriffith: oh okay, so the side effect can really be like a multi line bit of code17:23
dulekntpttr: Ah, sorry. If you do that then great, if not - I don't care too much. :)17:23
*** david-l__ has joined #openstack-cinder17:23
*** david-ly_ has quit IRC17:23
jgriffithntpttr: oh.. YES17:23
*** david-lyle has quit IRC17:23
ntpttrjgriffith: I'll go look at some more side effect examples to figure out how they work, thanks for the help!17:23
jgriffithntpttr: so side-effect is a method you define17:23
jgriffithntpttr: as opposed to return_value where you just say "return this"17:24
dulekjgriffith: Not only. ;) side_effect can be an exception - call will throw an exception.17:24
jgriffithdulek: yeah... I'm a big fan of side-effect17:24
dulekjgriffith: It can also be a list - then the mock calls will return next value per call17:24
jgriffithdulek: I can do whatever I want and base that decision on the parameters to the method17:25
ntpttrjgriffith dulek: okay cool, that makes sense. Learn something new every day, thank you guys17:25
jgriffithdulek: Ahh... the list iteration thing is a clever idea!!  :)17:25
jgriffithI like that17:25
jgriffithdulek: just pop things off and roll... nice!17:27
jgriffithdulek: I'll use that instead of saying "if arg == "blah blah blah""  etc17:27
jgriffithdulek: also adds some checking of expected order of calls17:27
dulekjgriffith: Yup, it's nice if you don't have too much logic in mocked method.17:29
*** david-l__ has quit IRC17:30
*** yumapath has joined #openstack-cinder17:34
*** houming has quit IRC17:34
*** xiaohui_ has quit IRC17:37
*** jordanP has quit IRC17:38
Swansonasselin, I think I see a partial problem.  I have no keys registered for the ci account.17:41
asselinSwanson, yeah, you'll need that.....but that would be a failure that occurs have the host key is verified...17:42
Swansonasselin, Warning: /Apt_key[Add key: D50582E6 from Apt::Source jenkins]: The id should be a full fingerprint (40 characters), see README.17:43
asselinSwanson, that one's ok17:43
*** david-lyle has joined #openstack-cinder17:44
*** sghanekar__ has joined #openstack-cinder17:45
Swansonasselin, launchpad id is my gerrit user, right?17:49
asselinSwanson, you should have a separate id for your ci system17:49
Swansonasselin, yeah but I think I have the wrong id in part of this...17:50
asselinSwanson, you should login to gerrit using your ci-account. You can get the name from there17:50
*** aix has joined #openstack-cinder17:57
openstackgerritMerged openstack/cinder: Move oslo-incubator's scheduler module to cinder  https://review.openstack.org/24925518:00
*** e0ne has joined #openstack-cinder18:01
*** yumapath has quit IRC18:02
*** merooney has quit IRC18:03
asselinSwanson, you should have this line if you delete the known host file: Notice: /Stage[main]/Zuul::Known_hosts/File[/home/zuul/.ssh/known_hosts]/ensure: created18:03
*** david-lyle has quit IRC18:03
asselinand rerun puppet18:03
Swansonre-running puppet.18:04
*** [1]Thelo has joined #openstack-cinder18:04
*** jistr has quit IRC18:04
*** pdardeau has joined #openstack-cinder18:06
*** Thelo has quit IRC18:06
*** [1]Thelo is now known as Thelo18:06
*** pdardeau has quit IRC18:06
SwansonThat creates a known_hosts file wildly different than what I get if I just try to clone and have it save the rsa fingerprint then.18:07
SwansonAnd if I try to clone with that known_hosts file I still have to save my rsa fingerprint.18:08
asselinSwanson, yeah, I think that's b/c it's hashed when you do it manually18:08
*** markus_z has quit IRC18:08
asselinSwanson, try adding the one you get manually to your hiera common.yaml...let me look up the var name18:09
*** david-lyle has joined #openstack-cinder18:11
*** martyturner has quit IRC18:12
*** hemna has quit IRC18:12
*** martyturner has joined #openstack-cinder18:14
*** merooney has joined #openstack-cinder18:15
*** mriedem has quit IRC18:17
openstackgerritMerged openstack/cinder: VMware: Skip unsupported datastore types  https://review.openstack.org/24915018:21
*** mriedem has joined #openstack-cinder18:21
*** dustins has joined #openstack-cinder18:24
*** liverpooler has quit IRC18:27
Swansonasselin, If you pick the right key from your list of server keys it works peachy.  At least I can run puppet and then su - zuul and actually clone successfully.18:33
asselinSwanson, what exactly?18:34
*** lprice has joined #openstack-cinder18:34
SwansonI just pulled one of the keys listed under ssh public keys for server host key and put that in the site.pp file.  Re-puppetted and ssh seems to be setup.18:35
*** pots2 has joined #openstack-cinder18:36
asselinSwanson, I'm confused, can you paste it ?18:37
*** lprice1 has quit IRC18:37
SwansonI just grabbed one of the keys under https://review.openstack.org/#/settings/ssh-keys18:38
*** sgundur has quit IRC18:38
*** pots has quit IRC18:39
*** pdardeau has joined #openstack-cinder18:42
*** bardia has joined #openstack-cinder18:43
*** mtanino has quit IRC18:47
*** apoorvad has joined #openstack-cinder18:52
*** pdardeau has quit IRC18:52
*** pdardeau has joined #openstack-cinder18:54
*** pdardeau has quit IRC18:55
*** sghanekar__ has quit IRC18:56
*** merooney has quit IRC18:58
*** martyturner has quit IRC18:58
*** pots has joined #openstack-cinder18:58
*** bardia has quit IRC19:00
*** martyturner has joined #openstack-cinder19:00
*** pots2 has quit IRC19:00
*** jerrygb has joined #openstack-cinder19:01
*** sghanekar__ has joined #openstack-cinder19:02
*** hemna has joined #openstack-cinder19:03
*** pdardeau1 has joined #openstack-cinder19:04
*** pdardeau1 has quit IRC19:04
*** Lee1092 has quit IRC19:09
*** martyturner has quit IRC19:09
openstackgerritMerged openstack/cinder: Fix metadata retrieval in GPFS driver  https://review.openstack.org/24686119:10
*** alejandrito has quit IRC19:10
openstackgerritMerged openstack/cinder: Add check_uptodate.sh --checkopts to "pep8"  https://review.openstack.org/24779619:10
openstackgerritMerged openstack/cinder: Don't build two tox envs for pep8(-constraints)  https://review.openstack.org/24780619:10
*** martyturner has joined #openstack-cinder19:14
angela-shemna: CI is passing now. https://review.openstack.org/#/c/180518/19:15
hemnaangela-s, nice!19:15
angela-shemna: can you review again when you have a chance? thanks!19:16
*** merooney has joined #openstack-cinder19:16
hemnaangela-s, ok I'll add it to my queue today19:16
*** mtanino has joined #openstack-cinder19:18
*** sgundur has joined #openstack-cinder19:20
*** anshul has joined #openstack-cinder19:20
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24938819:26
*** pots has quit IRC19:28
*** IlyaG has joined #openstack-cinder19:31
jgriffithsmcginnis: is the bug here that the user wants more info in the Error message:  https://bugs.launchpad.net/cinder/+bug/1388714 ?19:38
openstackLaunchpad bug 1388714 in Cinder "cinder api don't throw out appropriate exception when deleting volume detached or with snapshot " [Undecided,Triaged]19:38
smcginnisjgriffith: Just had to reread that first sentence 5 times. This might take a little. ;)19:39
jgriffithOk, it's not just me then :)19:39
jgriffiththat's all I needed to know :)19:39
smcginnisjgriffith: Yeah, that's my interpretation too. I think they are saying it should give more of a reason or better message.19:39
smcginnisjgriffith: Actually looks like a patch merged for that one.19:40
smcginnisBut state of bug never got updated.19:41
*** salv-orlando has quit IRC19:41
smcginnisOh, because patch was for oslo.19:41
*** chenying_ has joined #openstack-cinder19:41
openstackgerritKendall Nelson proposed openstack/cinder: Hacking Checks for assertTrue/IsNone()  https://review.openstack.org/24655419:42
*** chenying has quit IRC19:43
*** baumann1 has joined #openstack-cinder19:44
*** baumann has quit IRC19:44
smcginnisjgriffith, hemna: While you guys are here - thinking of getting a 1.5.0 release of python-cinderclient once this lands: https://review.openstack.org/#/c/231708/19:45
smcginnisMake sense?19:45
smcginnisOr is there anything else we should wait for.19:45
smcginnisSeems like a useful thing to get out there now that we have a few drivers implementing it.19:45
hemnaheh yah we need that guy out there if anyone is going to do some work against it19:45
hemnaI haven't been able to test it just yet though19:46
hemnaso I'm not sure how the replication targets list looks19:46
openstackgerritBardia Keyoumarsi proposed openstack/cinder: Volume driver for Coho Data storage solutions  https://review.openstack.org/24669019:47
smcginnishemna: I just figure we have enough time that if we want to improve it before the end of M we can.19:47
jgriffithhemna: this doesn't help you?  https://github.com/openstack/cinder/blob/master/doc/source/devref/replication.rst19:49
hemnawas hoping to see the exact output of the list replication targets command19:52
jgriffithhemna: ok, I'll post it tomorrow19:52
hemnaaorourke, has an example on our internal wiki19:52
jgriffithhemna: with some other examples to try and help19:52
hemnawas hoping it would have been public....19:52
jgriffithOh yeah.. an internal HP wiki19:52
jgriffithsign me up :)19:52
smcginnishemna: Don't suppose you could copy from there and comment in the patch?19:53
hemnasmcginnis, yah, just making sure what he has is up to date/correct in sync w/ his patch19:53
*** dims has joined #openstack-cinder19:53
*** pots has joined #openstack-cinder19:53
*** dims_ has quit IRC19:56
*** martyturner has quit IRC19:56
openstackgerritVictor Stinner proposed openstack/cinder: Port key manager to Python 3  https://review.openstack.org/24939719:59
hemnasmcginnis, jgriffith ok I'm having him rework his example output.20:00
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24938820:00
hemnausing 2 different drivers20:00
*** pots2 has joined #openstack-cinder20:00
*** anshul has quit IRC20:01
jgriffithhemna: smcginnis I posted a few comments in that review20:01
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24938820:01
hemnaaorourke, ping20:02
*** pots has quit IRC20:02
aorourkehemna, hey20:02
hemnajgriffith, cool.  just saw your comments and I agree w/ em.20:03
openstackgerritMerged openstack/cinder: Declare multiattach is True in RBD driver  https://review.openstack.org/24908020:03
*** sgundur has left #openstack-cinder20:03
hemnaaorourke, you have some more feedback on the client review20:03
openstackgerritVictor Stinner proposed openstack/cinder: Port zonemanager to Python 3  https://review.openstack.org/24939820:03
hemnaok lunch.....20:04
aorourkehemna, going over that right now20:04
*** martyturner has joined #openstack-cinder20:04
*** sgundur has joined #openstack-cinder20:06
openstackgerritVictor Stinner proposed openstack/cinder: Test middleware test_faults to Python 3  https://review.openstack.org/24939920:06
aorourkejgriffith, I addressed your comments.20:08
jgriffithaorourke: oh.. well that sounds inviting20:08
aorourkejgriffith, no I agree with them20:08
jgriffithaorourke: :)20:08
aorourkejgriffith, just looking to see how we want to move forward20:08
jgriffithaorourke: so your responses look fine to me20:09
jgriffithaorourke: I would love to do something like "rep-enable"20:10
aorourkejgriffith, great, I will make the changes20:10
jgriffithkeep it easy to type :)20:10
jgriffithbut certainly up to you20:10
jgriffithwait... one thing20:10
aorourkejgriffith, what's up?20:10
jgriffithI seem to recall there's a replication term being used somewhere20:10
jgriffithin OpenStack20:10
jgriffithNM, that's an internal construct20:11
dulekntpttr: I've commented on https://review.openstack.org/#/c/243673/ - pinging you to make sure you'll notice even after abandoning it. :)20:11
*** changbl has quit IRC20:11
aorourkejgriffith, ok. I will make these changes as discussed20:11
jgriffithaorourke: awesome, thanks!20:12
dulekntpttr: BTW - you can add me as reviewer on your series of smaller patches.20:12
*** martyturner has quit IRC20:13
*** ociuhandu has quit IRC20:13
*** amrith is now known as _amrith_20:16
openstackgerritJohn Griffith proposed openstack/cinder: Add replicate by volume-type to SolidFire driver  https://review.openstack.org/23827620:16
openstackgerritJohn Griffith proposed openstack/cinder: Add cluster Pairing setup to SolidFire driver  https://review.openstack.org/23823520:16
*** thangp has quit IRC20:19
*** apoorvad has quit IRC20:21
erlonjgriffith: ping20:23
*** thangp has joined #openstack-cinder20:24
*** IlyaG has quit IRC20:25
*** sghanekar__ has quit IRC20:26
*** lprice1 has joined #openstack-cinder20:28
*** lprice has quit IRC20:30
*** lprice has joined #openstack-cinder20:31
*** changbl has joined #openstack-cinder20:31
*** lprice1 has quit IRC20:33
*** edtubill has joined #openstack-cinder20:34
openstackgerritAlex O'Rourke proposed openstack/python-cinderclient: Adds v2 replication support  https://review.openstack.org/23170820:34
jgriffithmc_nair: BTW... BIG THANK you to the thorough review on this: https://review.openstack.org/#/c/238276/4..5/cinder/volume/drivers/solidfire.py20:34
jgriffithmc_nair: I'm implementing your suggestions now, appreciate the thorough feedback20:34
*** ociuhandu has joined #openstack-cinder20:35
mc_nairjgriffith: anytime :) will look over both those again whenever they're ready20:37
jgriffithmc_nair: cool... should be just a few minutes, trying to figure out my mess with repdata and build_endpoint that you pointed out20:38
*** e0ne has quit IRC20:38
aorourkejgriffith, smcginnis, hemna I added a comment with the link showing what listing replication targets looks like. I have two targets in this example20:38
aorourkehere it is if you dont want to open the patch :)  http://paste.openstack.org/show/479916/20:39
*** lprice has quit IRC20:39
*** lprice has joined #openstack-cinder20:39
smcginnisaorourke: Thanks!20:44
*** baumann1 has left #openstack-cinder20:45
smcginnisaorourke: I'm assuming unmanaged will just have no value in the last column?20:45
aorourkesmcginnis, correct.20:45
aorourkesmcginnis, or None20:45
*** sghanekar__ has joined #openstack-cinder20:48
openstackgerritJohn Griffith proposed openstack/cinder: Add replicate by volume-type to SolidFire driver  https://review.openstack.org/23827620:54
jgriffithaorourke: so I thought we agreed that a device_identifier was required20:55
jgriffithaorourke: regardless of managed/unmanaged20:55
jgriffithaorourke: in which case, I'd suggest we just use that20:55
jgriffithThe Admin should know what they configured, and if they don't "cat /etc/cinder/cinder.conf" :)20:56
*** IlyaG has joined #openstack-cinder20:56
jgriffithAlso this way you're not leaking any abstraction info to the end-user if you ever make "rep-target-list" publicly viewable20:56
smcginnisjgriffith: device_identifier is different than remote_device_id?20:56
jgriffithsmcginnis: same thing :)20:56
jgriffithsmcginnis: I just make up words as I go along :)20:56
smcginnisjgriffith: Just trying to understand what you're saying. That the list should only show that id?20:57
jgriffithsmcginnis: if we wanted to go crazy in the future we could actually assign a UUID and create a DB entry20:57
jgriffithsmcginnis: yes, correct20:57
jgriffithsmcginnis: because that's a required field in the config now20:57
smcginnisNo managed/unmanaged. no managed backend name, just a list of ids, right?20:57
jgriffithso it will be there in all cases20:57
jgriffithsmcginnis: exact-u-mundo20:58
smcginnisjgriffith: Got it. I guess that makes sense. The rest is based on however they've configured it, so I guess that's all they really need.20:58
jgriffithsmcginnis: that's my opinion20:58
smcginnisjgriffith: OK, devil's advocate, what if the client user is not the admin of the host and does not have ready access to the cinder.conf file?20:59
jgriffithsmcginnis: then they're not supposed to know those details anyway :)21:00
smcginnisNot likely, but just sayin'.21:00
*** salv-orlando has joined #openstack-cinder21:00
smcginnisFair enough. :)21:00
jgriffithsmcginnis: even backend-name, host-name etc is all supposed to be hidden from them :)21:00
*** salv-orl_ has joined #openstack-cinder21:01
*** dustins has quit IRC21:01
*** raildo is now known as raildo-afk21:01
smcginnisI do agree it's best to start with a minimal usable feature and make it more complicated only after it's been shown to need it. So in that respect, I agree.21:01
jgriffithsmcginnis: I wonder if we should change that though:  s/device_target_id/target_device_id/21:03
jgriffithsmcginnis: aorourke mc_nair thoughts?21:03
smcginnisjgriffith: I do like the second better.21:03
smcginnisThe first sounds like a device's ID.21:03
jgriffithor even "rep_device_id"21:03
smcginnisThe second sounds more like the ID of the target.21:03
jgriffithsmcginnis: yeah... the first one is confusing I think21:03
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Extend Volume for VMAX3  https://review.openstack.org/24894121:04
smcginnisSubtle but that does sound clearer to me.21:04
jgriffithsmcginnis: ok, last question....21:04
jgriffithDo you prefer "target_device_id" or "replication_device_id"21:04
*** salv-orlando has quit IRC21:05
smcginnisHmmm... target I think.21:05
jgriffithsmcginnis: sold!21:05
*** EinstCrazy has joined #openstack-cinder21:06
aorourkesmcginnis, jgriffith ok, sorry stepped away for a bit. It sounds like we are leaning towards "replication_device_id" and ONLY showing a list of IDs? No managed/unmanaged? No backend name?21:07
aorourkemy only concern is the people who have patches in progress for replication...I have been correcting everyone to use remote_device_id. We can always go back and change all this obviously...21:07
jgriffithaorourke: yes to showing only the ID's21:08
smcginnisaorourke: Yes. Though I was thinking target_device_id makes more sense (at least to me) but really either target or replication work fine.21:08
jgriffithaorourke: jury is out on name... smcginnis is leaning towards target_device_id21:08
jgriffithI'm indiffernt21:08
jgriffithJust need to know so I can finish this patch :)21:08
smcginnisI don't have a strong enough opinion really.21:08
aorourkeok that works for me as well. I will make it "target_device_id"21:08
aorourkeand remove the other keys21:08
aorourkeone sec21:08
smcginnisTimes up.21:08
mc_nairjust caught up but looks like you guys got it all under control.... that was easy :)21:09
openstackgerritBardia Keyoumarsi proposed openstack/cinder: Volume driver for Coho Data storage solutions  https://review.openstack.org/24669021:09
openstackgerritAlex O'Rourke proposed openstack/python-cinderclient: Adds v2 replication support  https://review.openstack.org/23170821:12
*** EinstCrazy has quit IRC21:12
aorourkehope that settles it :)21:16
smcginnisaorourke: I hope so too. ;)21:16
smcginnisaorourke: Thanks for working on it.21:16
*** akshai has quit IRC21:22
*** akshai has joined #openstack-cinder21:23
*** jgregor has quit IRC21:24
*** jgregor has joined #openstack-cinder21:24
*** lpetrut has joined #openstack-cinder21:26
mc_nairaorourke: looking at that v2 patch - is there a reason the replication_list_targets is using a POST?21:29
*** akerr has quit IRC21:29
aorourkemc_nair, I am pretty sure that is how it is designed. I have to have post the volume_id to it. I may be wrong though....was a while since I last looked at these21:31
jgriffithanybody else seeing something like this:  https://gist.github.com/b3652ee25df4aa74aaa521:33
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (managed)  https://review.openstack.org/23120121:34
openstackgerritAlex O'Rourke proposed openstack/cinder: 3PAR: Implement v2 replication (unmanaged)  https://review.openstack.org/23124521:34
smcginnisjgriffith: git status show anything?21:34
jgriffithnope, it just boots my changes back out to uncomitted21:34
jgriffithI've never seen this before21:35
jgriffitheven tried a fresh repo21:35
jgriffitherr... fresh clone21:35
smcginnisgit stash your changes to get them out of the way, pull, then apply the stashed changes?21:35
jgriffithtried that.. just did the same goofy thing :(21:35
smcginnisThat is really odd.21:36
*** aorourke is now known as aorourke-afk21:36
smcginnisjgriffith: You're just trying to rebase on master, right?21:36
jgriffithsmcginnis: well, that's what is weird21:36
jgriffithsmcginnis: I just did "git commit -a" typed in my fancy commit mesg (sans spelling errors) and said "git review"21:37
jgriffithsmcginnis: and it vomitted on me21:37
*** dims has quit IRC21:37
smcginnisjgriffith: Oh, I have seen that.21:38
jgriffithdo tell21:38
smcginnisgit review does a rebase first to make sure you are on the latest and no conflicts before pushing up.21:38
*** dims has joined #openstack-cinder21:38
smcginnisjgriffith: Can you manually: git checkout master; git pull; git checkout yourbranch; git rebase master21:39
jgriffithsmcginnis: hmm... think I just found the problem21:39
smcginnisjgriffith: Yah?21:39
jgriffithsmcginnis: I have an RBD WIP change in my tree21:39
*** merooney has quit IRC21:39
jgriffithalthough... that still seems odd21:39
smcginnisjgriffith: Doh! :)21:39
jgriffithlet me try starting over21:39
openstackgerritKaitlin Farr proposed openstack/cinder-specs: Integrate Castellan for Key Management  https://review.openstack.org/24757721:40
*** jaypipes has quit IRC21:47
openstackgerritJohn Griffith proposed openstack/cinder: Update list_replication_targets  https://review.openstack.org/24943921:47
jgriffithgeesh!  That was painful21:47
mc_nairjgriffith, smcginnis - do you know if / why need to do a POST for os-list_replication_targets to get the targets? This is regarding the v2 replication client patch21:48
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24944121:48
*** boris-42 has quit IRC21:48
jgriffithmc_nair: link to your patch?21:49
*** bardia has joined #openstack-cinder21:49
jgriffithmc_nair: IIRC you don't21:49
mc_nairjgriffith: sorry - this one: https://review.openstack.org/#/c/23170821:50
jgriffithmc_nair: It's ok, I'm just too lazy to look :)21:50
jgriffithmc_nair: ok, sorry... not following wht you mean.  I had assumed you meant a self.post statement21:51
mc_nairjgriffith: me speak bad21:52
mc_nairjgriffith: added a comment to that patch trying to clarifying the question I'm trying to ask - lemme know if I'm just totally missing something / still totally unintelligible21:53
jgriffithmc_nair: I don't know why you would need anything different than what you have21:56
jgriffithmc_nair: self._action is a post21:57
jgriffithI'm rather confused21:57
jgriffithsorry :(21:58
mc_nairjgriffith: think we are still missing each other :) this is aorourke-afk's patch.  It was just odd to me when looking at it that in order to *list* the replication targets that we have to do a POST call.  So I'm trying to figure out if in fact we need a POST call or if we could just be doing a GET (since it is just listing)21:58
jgriffithIIRC we use that on all of the contrib modules21:58
jgriffithmc_nair: OHHHH21:58
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24938821:59
*** akshai has quit IRC21:59
jgriffithmc_nair: honestly, that's a common thing to swap around22:00
jgriffithmc_nair: BUT22:00
jgriffithin the case of admin-actions/ contrib items... we consider this possibly sensitive data22:00
jgriffithin which case the RFC recommends POST22:00
jgriffithand well... since we use it everywhere else, I see no reason to use something different :)22:01
*** akshai has joined #openstack-cinder22:01
jgriffithmc_nair: does that make it about as clear as mud :)22:01
mc_nairjgriffith: ah - gotcha.  Ok that makes sense.  Thanks for the explanation... figured I was missing something22:01
mc_nairthanks for fighting through the missed communications :)22:01
*** sgotliv has quit IRC22:02
jgriffithhaha... I'm the one who should say thanks, it happens with me a lot22:02
jgriffithI'm good at figuring it out in my head... but I sadly tend to assume others know what I'm thinking and have the exact same background thought/process22:02
jgriffithmc_nair: and honestly... well; they should damn it!22:02
*** _amrith_ is now known as amrith22:03
mc_nairjgriffith: but then we don't get to play IRC charades and where's the fun in that?22:03
jgriffithmc_nair: good point!22:04
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24944122:04
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API  https://review.openstack.org/24944122:04
*** lpetrut has quit IRC22:07
*** apoorvad has joined #openstack-cinder22:08
hemnaangela-s, ping22:10
*** xyang has quit IRC22:10
*** edtubill has quit IRC22:13
*** jerrygb has quit IRC22:14
*** jerrygb has joined #openstack-cinder22:15
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Changing PercentSynced to CopyState in isSynched  https://review.openstack.org/24699222:20
*** kfarr has quit IRC22:22
angela-shemna: hey22:22
*** vgridnev has quit IRC22:24
hemnaangela-s, hey, question for you about this22:24
hemnaangela-s, https://review.openstack.org/#/c/180518/21/cinder/zonemanager/utils.py22:24
hemnadoes it ever make sense that an FCZM vendor would want it's own algorithm for generating a friendly name ?22:24
hemnaas certain backends would have different size limitations and/or character limitations on the switch itself, depending on version, firmware, etc. ?22:25
angela-si don't think so.  the only difference i know of is character set22:26
hemnabecause otherwise that method belongs on the FCZM API itself22:27
hemnanot in utils.py22:27
*** diablo_rojo has quit IRC22:28
*** diablo_rojo has joined #openstack-cinder22:29
angela-shemna: ok, i see your point.  i could put it in FCCommon22:30
hemnaangela-s, I think it probably belongs there22:30
hemnaangela-s, at least it gives the vendors a chance to change it, if it doesn't work on their backend in the future, for whatever reason22:31
*** thangp has quit IRC22:31
angela-shemna: ok, ok.  any other nitpicks? :)22:31
hemnaI just started :P22:32
hemnaI wasn't sure if I was just way off base or not22:32
*** diablo_rojo has quit IRC22:36
*** IlyaG has quit IRC22:36
*** davechen has left #openstack-cinder22:37
*** IlyaG has joined #openstack-cinder22:38
*** mriedem has quit IRC22:38
*** IlyaG has quit IRC22:39
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints  https://review.openstack.org/24432822:43
*** mc_nair has quit IRC22:46
*** mc_nair has joined #openstack-cinder22:51
*** mc_nair has quit IRC22:53
*** hemna is now known as hemnafk22:55
*** ntpttr has left #openstack-cinder22:57
*** jungleboyj has quit IRC22:58
*** simondodsley has quit IRC22:58
*** jgregor has quit IRC22:59
*** jgregor has joined #openstack-cinder22:59
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API Part 2  https://review.openstack.org/24944122:59
*** jgregor has quit IRC23:00
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API Part 1  https://review.openstack.org/24938823:00
openstackgerritNate Potter proposed openstack/cinder: Move quota checks from manager to API Part 2  https://review.openstack.org/24944123:01
*** sgundur has quit IRC23:02
Swansonasselin, around?23:04
asselinSwanson, I am23:04
Swansonasselin, my Jenkins jobs are largely the samples.  They refer to a d-p-c node.  Jenkins knows nothing of a d-p-c node.  What step did I miss?23:05
asselindid you build your nodepool image?23:05
asselinand start nodepool?23:05
SwansonI did.23:05
SwansonOn my cloud I have a couple template-dpc-number images.23:06
asselinSwanson, does nodepool show any nodes built?23:06
asselinsudo nodepool list23:06
asselinany vms based off those templates in your cloud?23:06
*** jccnd has quit IRC23:06
SwansonI keep seeing an instance appear and then go away.  And nodepool list shows building.23:07
SwansonSounds like nodepool is choking23:07
Swansonssh fail...23:08
*** boris-42 has joined #openstack-cinder23:10
*** jwcroppe has quit IRC23:11
*** jwcroppe has joined #openstack-cinder23:13
asselinSwanson, why does ssh fail? authentication?23:17
*** jwcroppe has quit IRC23:17
*** jwcroppe has joined #openstack-cinder23:18
Swansonasselin, Not sure yet.  I'm thinking I must have another setup error with my cloud.23:18
asselinok, you know where the nodepool log files are?23:18
Swansonasselin, Yep.  Just needed to be poked in the right direction.  Thanks!23:19
SwansonI hope I remember what I was doing when I'm back in the office next week.23:20
*** jwcroppe has quit IRC23:22
*** apoorvad has quit IRC23:25
asselinyeah :)23:27
*** dims_ has joined #openstack-cinder23:28
*** dims has quit IRC23:29
*** IlyaG has joined #openstack-cinder23:31
*** willsama has quit IRC23:31
openstackgerritHelen Walsh proposed openstack/cinder: EMC VMAX - Replacing depreciated API EMCGetTargetEndpoints  https://review.openstack.org/24432823:38
*** akshai has quit IRC23:39
*** akshai has joined #openstack-cinder23:40
*** rcernin has quit IRC23:42
*** akshai_ has joined #openstack-cinder23:45
*** akshai has quit IRC23:46
*** dustins has joined #openstack-cinder23:46
*** yhayashi has joined #openstack-cinder23:47
Swansonasselin, Dumb question. where do I define the password for the nodepool image I created.  Trying to log into a copy and see if it is functional....23:51
*** apoorvad has joined #openstack-cinder23:51
asselinSwanson, no password. use the ssh private key23:52
asselinwith 'jenkins' user23:52
Swansonasselin, ah.  Was thinking that might be the case.  Not so great from the console.23:53
*** jwcroppe has joined #openstack-cinder23:53
*** EinstCrazy has joined #openstack-cinder23:56
*** angela-s has quit IRC23:59

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