Thursday, 2013-12-19

« Wednesday, 2013-12-18 Index Friday, 2013-12-20 »
*** SnowDust has quit IRC00:02
openstackgerritYogesh proposed a change to openstack/trove: fixed the server identifier to be updated into the database  https://review.openstack.org/6302300:03
*** plodronio has joined #openstack-trove00:09
*** greghill has quit IRC00:11
*** krow has quit IRC00:13
*** vipul is now known as vipul-away00:14
openstackgerritYogesh proposed a change to openstack/trove: fixed the server identifier to be updated into the database Closes-Bug: #1262421  https://review.openstack.org/6302300:17
*** krow has joined #openstack-trove00:18
openstackgerritYogesh proposed a change to openstack/trove: server identifier corrected Closes-Bug: #1262421  https://review.openstack.org/6302300:24
*** krast has quit IRC00:51
*** demorris has joined #openstack-trove00:51
*** rwsu has quit IRC01:18
*** rwsu has joined #openstack-trove01:19
*** krow1 has joined #openstack-trove01:31
*** krow has quit IRC01:32
openstackgerritA change was merged to openstack/trove-integration: Removes log generation parameters from EXTRA_OPTS  https://review.openstack.org/6194801:33
*** nosnos has joined #openstack-trove01:38
*** haomaiwang has joined #openstack-trove01:50
*** harlowja has quit IRC01:51
*** demorris has quit IRC01:52
*** haomaiwang has quit IRC01:54
*** jcooley_ has quit IRC01:55
*** laurelm has quit IRC01:57
*** harlowja has joined #openstack-trove02:03
*** haomaiwang has joined #openstack-trove02:09
*** rongze has joined #openstack-trove02:12
*** krow has joined #openstack-trove02:16
*** krow1 has quit IRC02:17
*** jcooley_ has joined #openstack-trove02:27
*** nosnos_ has joined #openstack-trove02:30
*** krast has joined #openstack-trove02:30
*** nosnos has quit IRC02:33
*** amrith has joined #openstack-trove02:34
*** krow has quit IRC02:35
*** jcooley_ has quit IRC02:37
*** erkules_ has joined #openstack-trove02:40
*** erkules has quit IRC02:40
*** krow has joined #openstack-trove02:42
*** yogesh has quit IRC02:43
*** krow has quit IRC03:04
*** dougshelley66 has quit IRC03:22
*** krow has joined #openstack-trove03:22
*** krow has quit IRC03:40
*** yogesh has joined #openstack-trove03:54
*** yogesh has quit IRC03:58
*** vipul-away is now known as vipul04:01
*** vipul has joined #openstack-trove04:01
*** krow has joined #openstack-trove04:14
*** krow has quit IRC04:16
*** jcooley_ has joined #openstack-trove04:39
*** krow has joined #openstack-trove04:41
*** rongze has quit IRC04:44
*** nosnos_ has quit IRC05:06
*** nosnos has joined #openstack-trove05:07
*** Seal has joined #openstack-trove05:10
*** rongze has joined #openstack-trove05:18
openstackgerritSushil Kumar proposed a change to openstack/trove-integration: Fixes .gitignore  https://review.openstack.org/6185305:21
*** rongze has quit IRC05:23
*** harlowja is now known as harlowja_away05:40
*** erkules_ is now known as erkules05:45
*** Seal has quit IRC05:46
*** harlowja_away is now known as harlowja05:59
*** denis_makogon has joined #openstack-trove06:08
*** yogesh has joined #openstack-trove06:09
*** Seal has joined #openstack-trove06:10
*** yogesh_ has joined #openstack-trove06:16
*** yogesh has quit IRC06:16
*** rongze has joined #openstack-trove06:20
*** yogesh_ has quit IRC06:23
*** Seal has quit IRC06:25
*** rongze has quit IRC06:25
*** yogesh has joined #openstack-trove06:27
*** rongze has joined #openstack-trove06:27
*** rongze has quit IRC06:28
*** Seal has joined #openstack-trove06:30
*** dougshelley66 has joined #openstack-trove06:34
*** Seal has quit IRC06:37
*** rongze has joined #openstack-trove06:37
*** yogesh has quit IRC06:44
*** yogesh_ has joined #openstack-trove06:44
*** yogesh_ has quit IRC06:46
*** Seal has joined #openstack-trove06:47
*** SushilKM has joined #openstack-trove06:47
*** krow has quit IRC06:54
*** yogesh has joined #openstack-trove07:01
*** SushilKM has quit IRC07:02
*** Seal has quit IRC07:03
*** yogesh has quit IRC07:21
*** vipul is now known as vipul-away07:34
*** Seal has joined #openstack-trove07:39
*** SushilKM has joined #openstack-trove07:39
*** haomaiwang has quit IRC07:48
*** haomaiwa_ has joined #openstack-trove07:48
*** rongze has quit IRC07:49
*** rongze has joined #openstack-trove07:49
*** yogesh has joined #openstack-trove07:57
*** haomaiwa_ has quit IRC08:00
*** haomaiwang has joined #openstack-trove08:00
*** SushilKM__ has joined #openstack-trove08:03
*** SushilKM has quit IRC08:04
*** harlowja is now known as harlowja_away08:04
*** krow has joined #openstack-trove08:05
*** haomaiwa_ has joined #openstack-trove08:05
*** haomaiwa_ has quit IRC08:07
*** haomaiwa_ has joined #openstack-trove08:08
*** haomaiwang has quit IRC08:09
*** krow has quit IRC08:09
*** krow has joined #openstack-trove08:13
*** amcrn has quit IRC08:15
*** krow has quit IRC08:22
openstackgerritSushil Kumar proposed a change to openstack/trove: server identifier corrected  https://review.openstack.org/6302308:28
*** krow has joined #openstack-trove08:32
*** haomaiwang has joined #openstack-trove08:33
*** haomaiwa_ has quit IRC08:35
*** flaper87|afk is now known as flaper8708:36
SushilKM__hey hub_cap can u review the https://review.openstack.org/63023 commit message has been updated08:37
*** krow has quit IRC08:48
*** jcooley_ has quit IRC08:54
*** krow has joined #openstack-trove08:54
yogeshHi :denis_makogon09:37
denis_makogonhi09:37
yogeshOn heat enabled tests...09:37
yogeshWe'll elaborate them...09:37
denis_makogonwe need unit tests09:38
denis_makogonfor heat support09:38
yogeshyes...09:38
denis_makogonthen int tests09:38
*** jcooley_ has joined #openstack-trove09:38
yogesh+109:38
*** krow has quit IRC09:39
yogeshwe definitely need them...09:39
denis_makogonyogesh, if you are working with heat stuff in trove, please do the tests, because i really don't like idea when someone submits patch which we cannot examine via gates09:39
yogeshwe are working on that as well..09:41
denis_makogonyogesh, when do you plan to submit patch with initial tests ?09:42
yogeshwe started on it today..09:42
yogeshi think we should have something this week and then expand on it as we move09:42
denis_makogonok09:43
denis_makogonwould be nice to see tests in several patchsets09:43
denis_makogon1. fake implementation09:43
denis_makogon2. tests09:43
yogeshok..09:45
yogeshwill discuss more on this...09:45
denis_makogonas i said, would be nice to see heat fake impl with two methods: create, get09:45
denis_makogonplus simple validation of parameters09:46
denis_makogonsame as in heat09:46
denis_makogonbecause i'm seeing that soon we would have network nics for instance, etc.09:47
*** ashestakov has joined #openstack-trove09:49
denis_makogonashestakov, ping09:49
ashestakovdenis_makogon: pong09:49
denis_makogonashestakov, i saw almost completed with network nics, right ?09:50
ashestakovdenis_makogon: just client side09:50
denis_makogonashestakov, ah, nice09:50
denis_makogonashestakov, when you would deal with taskmanager, please do not worget about heat and mysql template09:51
denis_makogon*forget09:51
*** ashestakov has quit IRC10:01
*** SergeyLukjanov has joined #openstack-trove10:03
*** jcooley_ has quit IRC10:09
*** rongze has quit IRC10:11
*** jcooley_ has joined #openstack-trove10:19
*** rongze has joined #openstack-trove10:23
*** yogesh has quit IRC10:24
*** denis_makogon has quit IRC10:40
*** denis_makogon has joined #openstack-trove10:45
*** jcooley_ has quit IRC10:49
*** krow has joined #openstack-trove10:50
*** krast has quit IRC10:51
*** SushilKM__ has quit IRC10:59
*** nosnos has quit IRC11:00
*** Seal has quit IRC11:01
*** haomaiwang has quit IRC11:02
*** rongze has quit IRC11:13
*** jcooley_ has joined #openstack-trove11:32
*** SushilKM__ has joined #openstack-trove11:35
*** yogesh has joined #openstack-trove11:35
*** yogesh has quit IRC11:39
*** krow has quit IRC11:45
*** krow has joined #openstack-trove11:48
*** kanzaros has quit IRC11:49
*** jcooley_ has quit IRC12:04
*** rongze has joined #openstack-trove12:04
*** kanzaros has joined #openstack-trove12:16
*** krow has quit IRC12:16
*** krow has joined #openstack-trove12:18
*** krow has quit IRC12:18
*** rongze_ has joined #openstack-trove12:24
*** laurelm has joined #openstack-trove12:24
*** rongze has quit IRC12:28
*** amrith has quit IRC12:33
openstackgerritSushil Kumar proposed a change to openstack/trove: Cleans and improves heat code path  https://review.openstack.org/6310212:44
*** abramley has quit IRC12:46
*** SushilKM__ has quit IRC12:51
*** jcooley_ has joined #openstack-trove12:57
*** laurelm has quit IRC13:02
*** pdmars has joined #openstack-trove13:03
denis_makogonSushilKM ping13:13
*** pdmars has quit IRC13:18
*** amrith has joined #openstack-trove13:19
*** jcooley_ has quit IRC13:27
*** SushilKM__ has joined #openstack-trove13:36
*** Seal has joined #openstack-trove13:36
*** Seal has quit IRC13:41
*** abramley has joined #openstack-trove13:42
*** pdmars has joined #openstack-trove13:44
*** amrith has quit IRC13:58
*** haomaiwang has joined #openstack-trove14:04
*** demorris has joined #openstack-trove14:09
*** jcooley_ has joined #openstack-trove14:18
*** haomaiwa_ has joined #openstack-trove14:25
*** haomaiwang has quit IRC14:26
*** robertmyers has joined #openstack-trove14:36
*** jcru has joined #openstack-trove14:40
*** openstack has joined #openstack-trove14:49
denis_makogonSushilKM__, ping14:50
openstackgerritSushil Kumar proposed a change to openstack/trove: Cleans and improves heat code path  https://review.openstack.org/6310214:51
SushilKM__yes denis14:51
*** jcooley_ has quit IRC14:52
SushilKM__how r u dear14:52
denis_makogonSushilKM__, i've got question to you14:54
SushilKM__sure14:54
denis_makogonSushilKM__, have you tested your patchset ?14:54
SushilKM__yes, tell me ur question, i would let u know if i have answers from my testing14:55
*** haomaiwa_ has quit IRC14:56
denis_makogonabout verifications, you need to hide everything related to volume provisioning14:56
denis_makogonif volumes = False14:57
denis_makogon1. Parameters in code; 2. Parameter in template14:57
denis_makogon3. Resource defenition14:57
denis_makogonin other way - StackValidationError14:58
SushilKM__if u saying about volumesize, it has not imact in case of trove_volume_support being false14:58
SushilKM__i tested that14:58
SushilKM__did not get any errors for unused parameters in the template14:58
denis_makogonheat 100% would raise UnusedParameterError (or some like that, i don't remember)14:59
SushilKM__actually i never got that14:59
SushilKM__even i tried standalone heat calls14:59
SushilKM__i.e. directly calling heat services with a heat template14:59
SushilKM__the template had more parameters than used one15:00
SushilKM__and i never got any errors15:00
denis_makogonhm, a bit strange15:00
SushilKM__wait i wud share u in a gist what i tested15:01
denis_makogonanyway, pleasem could you make a bit strict conditionals15:01
SushilKM__did not get that15:01
SushilKM__this was my template that i tested for unused parameters https://gist.github.com/sushilkm/280bd132224dd8f13b8115:03
SushilKM__and the call i was making was15:03
denis_makogoni'll take a look at it15:04
SushilKM__heat  --os-auth-url=http://localhost:35357/v2.0 --os-username=radmin --os-password=radmin --os-tenant-name=trove  -v create teststack --template-file=heat.template --parameters="Flavor=m1.small;VolumeSize=2;InstanceId='text';ImageId=ubuntu_mysql;DatastoreManager=mysql"15:05
denis_makogondo you agreed with abit strict conditionals15:05
SushilKM__what is that about15:06
*** grapex has joined #openstack-trove15:06
*** greghill has joined #openstack-trove15:07
*** rnirmal has joined #openstack-trove15:08
denis_makogonSushilKM__, i'm writing comments at review15:09
SushilKM__can we discuss here15:09
SushilKM__regarding server variable u say, we have to create that because need to return that to calling method15:10
SushilKM__so its not xtra baggage15:10
denis_makogonservers id variable used twice15:11
denis_makogonreally, why do we need spent time on creating dict, and perform object pulling ?15:11
SushilKM__twice usage is condition dependent15:14
SushilKM__once we had to pass volume id also, and other is without that15:14
SushilKM__and as far as existence of server variable is concerned we need to return to calling method so, we need to create that15:15
denis_makogoneven once15:15
denis_makogonwe spent time on packing and unpacking, instead of direct assigning15:16
*** amrith has joined #openstack-trove15:16
SushilKM__ok for the cost of unpacking u want me to direct assign15:17
*** kevinconway has joined #openstack-trove15:17
denis_makogonall other methods returns fully packed nova server model15:17
SushilKM__correct but since we are working wid heat, we are not required15:18
SushilKM__i have ran int-tests on this model using heat, and tests work fine  :)15:19
denis_makogoncould you please share the report ?15:19
SushilKM__okies i wud rerun that and send u the report15:19
SushilKM__since old report i did not have15:19
SushilKM__:(15:19
denis_makogonpost gist here15:19
SushilKM__not a problem15:20
denis_makogonok =)15:20
denis_makogonlet's finish with current patch15:20
SushilKM__and for volume, we do not need any condtions15:20
SushilKM__cool15:20
denis_makogonwhy is that /15:21
denis_makogon?15:21
SushilKM__i shared earlier to u the gist15:21
SushilKM__if we keep volumesize in template and used with parameter value it wont create a problem15:21
SushilKM__since we are not using it, any value is good to be thr15:22
denis_makogonok, what about sever variable >15:22
denis_makogon?15:22
denis_makogoni suggest next15:22
SushilKM__server vairbale we need to return to system back, since server id is being used at number of places, and nothing else15:23
denis_makogonreturn {'id': resource.physical_resource_id}15:23
denis_makogonreturn {'id': resource.physical_resource_id}, volume_info15:23
denis_makogonshould be faster15:24
SushilKM__ok15:24
SushilKM__thankx15:24
denis_makogonuntil return statement would be nice to use resource.physical_resource_id15:24
SushilKM__jst a sec15:24
*** jmontemayor has joined #openstack-trove15:31
denis_makogonso, SushilKM__, waiting for new patch15:32
*** datsun180b has joined #openstack-trove15:32
*** rongze has joined #openstack-trove15:42
openstackgerritSushil Kumar proposed a change to openstack/trove: Cleans and improves heat code path  https://review.openstack.org/6310215:43
*** jcooley_ has joined #openstack-trove15:44
greghillvipul-away: when you're back, I wondered if you had any thoughts on the thread I started on the dev list about the datastore stuff15:44
denis_makogonSushilKM__, self.update_db(compute_instance_id=instance_id]15:45
*** rongze_ has quit IRC15:45
SushilKM__oops15:45
denis_makogonSushilKM__, ] berfore coma15:45
denis_makogonSushilKM__, looks like another patch =)))15:45
SushilKM__first time i did not run the tox, and guess what, i commited the MISTAKE15:46
denis_makogonlol15:46
datsun180boh hubris!15:46
datsun180bevery time i skip pep8, sure enough i get dinged15:46
denis_makogonwe need before-review hooks15:46
datsun180bdenis_makogon: yeah that's "run tox you goober"15:46
denis_makogondatsun180b, i heard that heat has them15:47
robertmyersthats what jenkins is for15:47
robertmyersI don't want to wait :)15:48
SushilKM__jst 2 mins15:48
robertmyerstoooo long15:48
openstackgerritSushil Kumar proposed a change to openstack/trove: Cleans and improves heat code path  https://review.openstack.org/6310215:48
SushilKM__get that robertmyers15:49
SushilKM__:)15:49
*** SergeyLukjanov has quit IRC15:56
SushilKM__brb16:00
openstackgerritEugeniya Kudryashova proposed a change to openstack/python-troveclient: Use Rsource() class from common Oslo code  https://review.openstack.org/6316016:04
*** SushilKM__ has quit IRC16:05
*** Barker has joined #openstack-trove16:08
*** rongze has quit IRC16:15
*** jcooley_ has quit IRC16:16
*** rongze has joined #openstack-trove16:16
cp16netdenis_makogon: hey i support the log operation api but i'd like to see the api spec'd out in the wiki16:16
denis_makogoncp16net, what kind of details you are expecting ?16:17
denis_makogonoh, i got you cp16net, i'll do that asap16:18
denis_makogonmaybe tomorrow, ok ?16:18
*** SushilKM__ has joined #openstack-trove16:18
*** Seal has joined #openstack-trove16:18
cp16netthats cools16:20
cp16neti just want to see the request/response and paths16:20
cp16netthat way we can see what you are thinking moving forward16:20
denis_makogoncp16net, of course16:22
*** rongze has quit IRC16:22
denis_makogoncp16net, thanks for reminding me16:22
*** rongze has joined #openstack-trove16:22
*** vipul-away is now known as vipul16:26
greghillthere seems to be a bit of overlap between two blueprints regarding scheduling backups.  There's the generic scheduled tasks stuff outlined here: https://wiki.openstack.org/wiki/Trove/scheduled-tasks and then also one specific to automated backups here: https://blueprints.launchpad.net/trove/+spec/automated-backuping16:30
greghilldoes anyone think we shouldn't go with the more generic approach?  This would enable automated scheduling of other tasks like log dumping16:31
greghillor is the one a dependency of the other and I'm misreading it?16:33
*** jcooley_ has joined #openstack-trove16:33
hub_capgreghill: they should be dependencies16:34
greghillah, cool then16:34
hub_capone of them is the "scheduler" aspect of the code, and the other is the mysql impl for backups16:34
greghillroger roger16:35
hub_capthey can totally be worked on independently (and id hope them to be independent patches)16:35
hub_capor else ill cry horse tears16:37
greghillI'm looking at the blueprint now and I have one suggestion.  there should be an optional field for extra data needed by the scheduled task, maybe just a text field containing a JSON string.  This will provide flexibility for tasks that might require a little extra data from the user16:38
*** SergeyLukjanov has joined #openstack-trove16:38
SushilKM__hey hub_cap plz look into https://review.openstack.org/#/c/63102/16:38
SushilKM__and this one is a small change, take a look on this one too  https://review.openstack.org/#/c/62754/16:39
* kevinconway unsure if greghill is serious16:39
greghillI'm always serious, except when I'm not16:40
hub_capwhich may or may not be the case kevinconway16:40
hub_capdepending on if hes being serious or not16:40
hub_capSushilKM__: that was fast :)16:41
robertmyersgreghill: meta_data16:41
hub_capi think thats a easy approach SushilKM__16:41
SushilKM__thanks hub_cap16:41
hub_capdenis_makogon: what do u think of SushilKM__'s approach for the volume support ?16:41
greghilloh, I totally glossed over that robertmyers /needs_caffeine16:41
SushilKM__had discussions with denis, he has +1d the change after few changes :)16:42
denis_makogonhub_cap, i'm totally ok witht that16:42
greghillI was looking at the response rather than the request16:42
kevinconway#link http://focusfury.files.wordpress.com/2013/07/fry-not-sure-if-trolling_191.jpg?w=53816:42
kevinconwaynot sure fry is not sure16:42
denis_makogonhub_cap, first a was suprised, but after leading it, i'm more than ok with this patch16:43
*** plodronio has quit IRC16:44
SushilKM__soothing words from denis "more than ok"16:44
SushilKM__:)16:44
hub_capBender: Ahh, computer dating. It's like pimping, but you rarely have to use the phrase, 'upside your head'.16:44
*** plodronio has joined #openstack-trove16:44
*** amrith has quit IRC16:45
hub_capSushilKM__: question16:46
hub_capshouldnt this patchset remove some code that talks to cinder ?16:47
SushilKM__its removed from heat code16:47
SushilKM__did u still see that in  _create_server_volume_heat16:48
hub_capSushilKM__: no, im curious, when you do a GET on the instance, you used to be able to see some of the information about the volume16:49
hub_capnow that its not managed by cinder, does that still work?16:49
hub_capid assume you might need to call to heat to get info about the cinder volume, and maybe put that in the database?16:49
cp16netoh bender i have missed you16:49
denis_makogonhub_cap, hey, have you tried to test datastores ?16:50
hub_capdenis_makogon: andrews code?16:51
hub_capi ran thru the integration tests and provisioned a few instances manually during the process, yes16:51
denis_makogonhub_cap, i mean cassandra/mongo/redis16:51
hub_capahhhh16:51
*** jcooley_ has quit IRC16:52
*** kanzaros has quit IRC16:52
hub_capyes im waiting for a patch16:52
hub_capstill.....16:52
SushilKM__yeah, its put in the database the volume id has been taken from heat and stored into database for future refernc16:52
hub_capcweid has updated it though, so it should be ready for +2s. then you should update your -integration code to use it and create a cassandra datastore16:52
hub_caplet me find the dependent review16:52
denis_makogonok16:53
hub_caphttps://review.openstack.org/#/c/57780/16:53
*** jcooley_ has joined #openstack-trove16:55
*** kanzaros has joined #openstack-trove16:55
*** SushilKM__ has quit IRC16:56
*** Seal has quit IRC16:57
*** demorris has quit IRC16:57
*** jcooley_ has quit IRC16:58
*** kanzaros has quit IRC16:58
cp16nethub_cap: your big X on changing the routes? https://review.openstack.org/#/c/54224/16:59
cp16netwhen you call mapper.resource() it creates tons of routes that are not used17:00
cp16netthis explicitly makes only the ones needed17:00
*** glucas_ has joined #openstack-trove17:00
openstackgerritConrad Weidenkeller proposed a change to openstack/trove-integration: Add support for kick-starting new datastores  https://review.openstack.org/5778017:01
cp16netits taking the same approach i saw in keystone which makes the routes pretty clean https://github.com/openstack/keystone/blob/master/keystone/token/routers.py17:01
*** cweid has joined #openstack-trove17:02
denis_makogonhub_cap, what about review requests ? https://review.openstack.org/#/c/51884/17:02
denis_makogonhttps://review.openstack.org/#/c/58845/17:02
denis_makogonhttps://review.openstack.org/#/c/61969/17:02
denis_makogonhttps://review.openstack.org/#/c/62414/17:02
*** SushilKM__ has joined #openstack-trove17:02
*** Seal has joined #openstack-trove17:02
*** glucas has quit IRC17:03
*** glucas_ is now known as glucas17:03
*** SergeyLukjanov is now known as _SergeyLukjanov17:03
*** jmontemayor has quit IRC17:04
*** _SergeyLukjanov has quit IRC17:04
*** jcooley_ has joined #openstack-trove17:04
*** kanzaros has joined #openstack-trove17:04
*** jmontemayor has joined #openstack-trove17:07
*** SergeyLukjanov has joined #openstack-trove17:07
cp16netdenis_makogon: you see my comment on my review as well? about tests17:07
denis_makogoncp16net, which one ?17:07
cp16nethttps://review.openstack.org/#/c/5422417:08
*** Seal has quit IRC17:08
cp16netso no tests would pass if the routes were setup wrong17:08
*** jcooley_ has quit IRC17:09
*** kanzaros has quit IRC17:09
denis_makogoncp16net, yes, i saw that - let me check tests in master, if they cores all stuff, then i'm totally ok with your patch17:09
denis_makogon*covers17:09
cp16netyeah everything would fail in fake/real mode tests17:10
cp16nethttp://routes.readthedocs.org/en/latest/restful.html?highlight=resource17:10
cp16netso map.resource creates a ton of crap17:11
cp16netfor "best practice" rest service17:11
cp16netwhich i dont see why we need all the excess routes17:11
cp16netthats the purpose of this change17:12
denis_makogoni got it17:12
cp16netmaybei  should update the commit msg say that17:12
denis_makogonlast question17:12
denis_makogoncp16net, would be great (about commit msg)17:13
hub_capyea cp16net17:13
hub_capim not sure its necessary to make the change honestly, but im ok w/ it. im sure its a bit faster wrt the routing w/ less routes17:13
hub_capbut likely not noticable17:13
cp16netsure17:13
hub_capand we will move to pecan too eventually ;)17:13
denis_makogoncp16net, yes, tests are coveres each usecase17:14
cp16netwell this led to the special crap i had to do for adding a patch call to instance17:14
cp16netthe resource would not allow me to add it easily17:14
cp16netso i made it all manually17:14
*** jcooley_ has joined #openstack-trove17:15
*** kanzaros has joined #openstack-trove17:15
denis_makogoncp16net, i +1 it =)17:15
cp16nethub_cap: because you know... patch is a common http verb :-P17:15
denis_makogonhub_cap, cp16net 's patch really looks good, explicit routes would be great, they would help us to catch inccorrect http calls easily17:16
hub_capcp16net: patch ftw. dont use ie6 web client to access trove api17:16
hub_capehh denis_makogon it really doesnt change anything17:17
cp16netits easier to read17:17
cp16netbut yeah doesnt change anything17:17
denis_makogonhub_cap, explicit agains generated ?17:17
hub_capits eaiser to read if u print out the routing table ;)17:17
denis_makogonexplicit route pinning is good17:17
cp16nethub_cap: i disagree with that17:17
cp16netlol17:17
cp16netits how i found out all the crap is added17:17
denis_makogonhah17:18
kevinconwayhub_cap: it's scary how many people still use ie6/717:18
hub_capagain im not arguing against it cuz im going to -2 it17:18
hub_capkevinconway: yes i know, ive had to support it before via a website and it sucked17:18
cp16netyou mean you already -2 it17:18
denis_makogonkevinconway, are they live in stone age ?17:18
hub_capcp16net: did i?17:18
cp16netthe x?17:18
hub_capit probably needed a -217:18
kevinconwayour baseline should always be netscape navigator 7 though17:18
cp16netlol17:18
cp16neti hate you hub_cap17:18
hub_caplol17:18
* cp16net bangs hub_cap with a cute little Nintendo Wii17:18
denis_makogonlol17:19
cp16netBender: Hey, I got a busted ass here! I don't see anyone kissing it.17:19
* hub_cap takes the wii and pawns it17:19
cp16netlol17:19
hub_cap:P17:19
* denis_makogon enjoys action movie starring hub_cap and cp16net 17:19
hub_capLOL17:19
kevinconwaydenis_makogon: it's crazy. we used to do user-agent metrics at the last place i worked17:19
kevinconwayso many old IE clients...17:19
hub_capkevinconway: yup. so many css hacks17:20
* cp16net finds an old Atari 2600 and smashes hub_cap's head with it17:20
cp16netyou wouldnt want to pawn that17:20
hub_capso much sadness that like 30% of the world is on windows 9817:20
cp16netrather keep it :-P17:20
* hub_cap takes teh atari and plugs it in17:20
* hub_cap pawns cp16net17:20
* hub_cap gets a dollar17:21
* denis_makogon buys cp16net 17:21
hub_capLOLZ17:21
cp16net /weird17:21
denis_makogondeal17:21
denis_makogon1$17:21
hub_cap$$$cha-ching$$$17:21
denis_makogonSOLD17:21
cp16netglad i am valued17:21
denis_makogonwith a discount =))17:21
cp16netno coupons are allowed17:22
denis_makogoni watched History Channel and saw PawnShop Las Vegas show - that was amazing17:22
hub_capnaw cp16net was on clearance17:22
*** jcooley_ has quit IRC17:22
denis_makogona guy sold hand cannon17:22
cp16net:-/17:22
hub_capdenis_makogon: ya thats a fun show. im pertty sure its all fake though17:23
hub_capjust like all reality shows :)17:23
cp16netyeah like the storage wars17:23
denis_makogonhub_cap, but that was funny17:23
*** yidclare has quit IRC17:23
denis_makogonnow i want hand cannon17:23
denis_makogonLaphet Cannon17:23
hub_capyup. doesnt take away from the fact that cool stuff is shown there. its like antiques roadshow for this generation17:23
cp16netlike the antique roadshow marathon17:24
denis_makogonsome sort17:24
cp16netgod i loved grandmas boy17:24
hub_capdude i love antiques roadshow i must admit17:24
hub_capcp16net: LOL17:24
cp16netnew high score. what does that mean? did i break it?17:25
denis_makogoncp16net, still comes with discount, so, no, not the highest one =)17:25
cp16netthat was a quote from grandmas boy17:26
cp16nethttp://www.imdb.com/title/tt0456554/17:26
denis_makogonyesterday i had amazing photoshoot with Ukrainian Topless DJ/Pornstar17:27
denis_makogonthat was awesome17:27
denis_makogonNSFW - http://cs304411.vk.me/v304411431/7930/uc0sfkUDVYk.jpg17:28
cp16netreally?17:29
denis_makogonskipping topic17:29
hub_capyes, plz :)17:29
robertmyersdenis_makogon: please lets keep this SFW17:29
denis_makogonok17:29
denis_makogonsorry17:29
hub_capsfw - http://www.youtube.com/watch?v=ePfdZV8M_nM17:30
*** Barker has quit IRC17:30
hub_cap^ ^ on my tv right now lol17:30
robertmyersnice17:30
hub_capcompliments of chromecast17:30
denis_makogonwhat about growing to API v2 ?17:30
denis_makogonand only-heat stuff17:31
hub_capi approve that message denis_makogon17:32
denis_makogonso, we could start doing it ?17:32
hub_capnot ripping it out17:32
hub_capthat legacy code has to stay for like N+1 full release17:32
hub_caponce the heat code is usable as the other codepath is today, we can deprecate the old code17:33
openstackgerritConrad Weidenkeller proposed a change to openstack/trove-integration: Add support for kick-starting new datastores  https://review.openstack.org/5778017:33
hub_capin the beginning of a release, of course17:33
denis_makogonso, we could start to implementing thinks like if use_heat: blah.blah17:33
hub_capthen in N+1 release we can remove teh old code17:33
hub_capim not sure what u mean denis_makogon17:33
denis_makogonsecurity groups as an example17:34
*** Barker has joined #openstack-trove17:34
denis_makogonsec.grs  provisioining based on nova client17:34
denis_makogonwe could use stack with only one resource - security group17:34
*** rongze has quit IRC17:39
*** rongze has joined #openstack-trove17:40
*** amytron has quit IRC17:40
*** rongze has quit IRC17:44
*** kevinconway has quit IRC17:56
hub_capok running to get a breakfast, cu guys in ~30 min18:03
*** harlowja_away is now known as harlowja18:04
*** amrith has joined #openstack-trove18:08
*** amcrn has joined #openstack-trove18:10
*** amytron has joined #openstack-trove18:14
openstackgerritCraig Vyvial proposed a change to openstack/trove: changing the routes for the api to be clear  https://review.openstack.org/5422418:18
cp16netthat should be a better commit msg18:18
cp16net:)18:18
*** yidclare has joined #openstack-trove18:20
*** OpenStack1 has joined #openstack-trove18:20
openstackgerritSteve Leon proposed a change to openstack/trove: Using base32 encoding to generate DNS records  https://review.openstack.org/6318618:21
*** yogesh has joined #openstack-trove18:34
hub_capcp16net: would ya look at that, i _did_ -2 it18:35
*** krow has joined #openstack-trove18:35
hub_capya, it did deserve it18:36
*** amrith has quit IRC18:37
*** amrith has joined #openstack-trove18:38
*** rongze has joined #openstack-trove18:40
*** krow has quit IRC18:40
*** krow has joined #openstack-trove18:42
*** jdbarry has quit IRC18:42
*** amrith has quit IRC18:44
*** yogesh has quit IRC18:45
*** rongze has quit IRC18:45
*** yogesh has joined #openstack-trove18:47
denis_makogongrapex, ping18:49
denis_makogonhub_cap, could you help me a bit ?18:52
hub_capsup18:53
*** amrith has joined #openstack-trove18:55
denis_makogoncould you explain me why does Tim gave me "do not merge" for this one https://review.openstack.org/#/c/61169/ ?18:55
denis_makogoni'm a bit cofused18:55
*** Barker has quit IRC18:56
*** yogesh has quit IRC18:58
*** yogesh has joined #openstack-trove18:58
*** Barker has joined #openstack-trove18:59
*** yogesh has quit IRC19:01
hub_capheh /me shrugs19:02
hub_capyou will have to ask grapex about that one19:02
*** rnirmal has quit IRC19:02
*** yogesh_ has joined #openstack-trove19:04
denis_makogonhub_cap, unfortunately, i failed to do that19:05
*** ashestakov has joined #openstack-trove19:07
robertmyersdenis_makogon: there is a difference between testtools.skipIf and probocis.SkipTest that I'm not sure they work the same19:08
denis_makogonrobertmyers, i posted comment at that review19:08
denis_makogonrobertmyers, with timecode and build log19:09
*** demorris has joined #openstack-trove19:10
robertmyersstill, I think we should keep the probocis skip there19:10
robertmyersthere is no need to temp fate19:11
robertmyersthe base class is fine19:11
denis_makogonrobertmyers, we could use always_run flag19:11
*** amrith has quit IRC19:11
denis_makogon@test(always_run=VOLUMES)19:11
*** amrith has joined #openstack-trove19:11
robertmyersor @test(enabled=VOLUMES)19:12
denis_makogonyes19:12
denis_makogonthere're few acceptable options19:12
robertmyersthat would be preferred over mixing testtools and probocis19:12
denis_makogonaccording to bug-report, we need to get rid of direct usage of unittests19:13
*** kevinconway has joined #openstack-trove19:13
robertmyersI'm fine with that19:13
*** radez_g0n3 is now known as radez19:13
denis_makogonthen i'm gonna submit new patch tomorrow19:13
robertmyerspossibly that test itself should be a subclass of object instead19:13
robertmyerslike all the others19:13
robertmyerssince it has a completely different test suite runner19:14
*** yogesh_ has quit IRC19:15
robertmyersIE it is not *really* a unittest19:15
robertmyersit is an integration test19:15
denis_makogonyes, of course19:15
*** yogesh has joined #openstack-trove19:17
*** amrith has quit IRC19:18
*** amrith has joined #openstack-trove19:18
*** Barker has quit IRC19:19
*** demorris_ has joined #openstack-trove19:27
*** demorris has quit IRC19:28
*** demorris_ is now known as demorris19:28
openstackgerritA change was merged to openstack/trove: server identifier corrected  https://review.openstack.org/6302319:34
greghillis there any way to make gerrit not spam you about reviews you +1'd?  Kind of makes me not want to +1 things if I'm gonna get 100 follow up emails I don't care about.  the unsubscribe link doesn't go to any place that actually lets you unsubscribe19:37
*** demorris has quit IRC19:37
*** Barker has joined #openstack-trove19:40
*** demorris has joined #openstack-trove19:41
hub_capgreghill: filters :)19:43
hub_capbut srsly, there is nothing u can do w/ em i think19:43
greghillI figured as much, just wondered if I missed a setting somewhere19:45
*** OpenStack1 has quit IRC19:48
*** krow has quit IRC19:52
openstackgerritA change was merged to openstack/trove: Paginate backup list api  https://review.openstack.org/6035820:00
*** demorris_ has joined #openstack-trove20:01
*** demorris has quit IRC20:02
*** demorris_ is now known as demorris20:02
*** OpenStack1 has joined #openstack-trove20:10
robertmyersdemorris: ^^20:15
demorrisrobertmyers: ?20:17
robertmyersdemorris: pagination is in20:18
denis_makogoncores, could this one been reviewed https://review.openstack.org/#/c/57796/ ?20:18
demorrissweet20:18
openstackgerritSushil Kumar proposed a change to openstack/trove: Cleans and improves heat code path  https://review.openstack.org/6310220:20
*** OpenStack1 has quit IRC20:25
hub_capdenis_makogon: plz put your comments for https://blueprints.launchpad.net/trove/+spec/guestagent-update-upgrade20:28
hub_capinto https://blueprints.launchpad.net/trove/+spec/upgrade-guestagent20:28
hub_capthats the original blueprint20:28
*** robertmyers has quit IRC20:29
*** robertmyers has joined #openstack-trove20:29
SushilKM__needed help to re-run reddwarf for https://review.openstack.org/#/c/62399/20:34
datsun180bSushilKM__: on it20:36
datsun180bkicked20:37
SushilKM__many thanks mate20:37
datsun180bwe aim to please20:37
SushilKM__he he he20:38
*** Barker has quit IRC20:42
*** OpenStack1 has joined #openstack-trove20:55
denis_makogonhub_cap, done20:56
*** Seal has joined #openstack-trove20:56
*** datsun180b has quit IRC20:58
hub_capdone with what denis_makogon?20:59
openstackgerritAndrey Shestakov proposed a change to openstack/trove: Add neutron support  https://review.openstack.org/6321221:01
*** Seal has quit IRC21:01
*** amrith has quit IRC21:02
denis_makogonhub_cap, moving BP description21:03
hub_capoh sweet thx denis_makogon21:03
*** amrith has joined #openstack-trove21:04
denis_makogonhub_cap, profiler said that {} is up to 3 time faster then dict()21:04
openstackgerritConrad Weidenkeller proposed a change to openstack/trove: Update for datastore tests to support multiple datastore types  https://review.openstack.org/6321321:04
*** OpenStack1 has quit IRC21:04
openstackgerritCraig Vyvial proposed a change to openstack/trove: make the bin scripts called with main()  https://review.openstack.org/5422521:06
*** radez is now known as radez_g0n321:08
*** OpenStack1 has joined #openstack-trove21:08
*** ashestakov has quit IRC21:09
hub_capcp16net: i got to thinking21:09
hub_capthe oher projects, iirc, removed their bin scripts21:09
hub_capi wonder if we can do teh same instead of updating em21:10
cp16netoh yea?21:11
cp16netwell i think we still need them for the fake daemon mode stuff21:11
cp16netbut maybe we could do something different21:11
cp16netbtw have you seen gerrit blow up lately? https://gist.github.com/cp16net/804632221:11
cp16netDAMNIT wrong project21:14
cp16netnm21:14
*** vipul is now known as vipul-away21:16
openstackgerritCraig Vyvial proposed a change to openstack/trove-integration: add test.rc file to source and .gitignore addition  https://review.openstack.org/5952721:19
*** vipul-away is now known as vipul21:21
*** OpenStack1 has quit IRC21:32
*** abramley has quit IRC21:32
*** OpenStack1 has joined #openstack-trove21:39
*** SergeyLukjanov has quit IRC21:46
*** SushilKM__ has quit IRC21:52
*** Barker has joined #openstack-trove21:57
*** greghill has quit IRC22:02
*** robertmyers has quit IRC22:03
*** OpenStack1 has quit IRC22:07
*** amrith has quit IRC22:07
*** OpenStack1 has joined #openstack-trove22:07
*** pdmars has quit IRC22:09
*** jcooley_ has joined #openstack-trove22:19
*** OpenStack1 has quit IRC22:23
*** ashestakov has joined #openstack-trove22:24
*** denis_makogon has quit IRC22:37
*** glucas has quit IRC22:41
*** glucas has joined #openstack-trove22:42
*** OpenStack1 has joined #openstack-trove22:43
*** krow has joined #openstack-trove22:46
*** vipul is now known as vipul-away22:56
*** flaper87 is now known as flaper87|afk23:00
*** esp has joined #openstack-trove23:02
*** yogesh has quit IRC23:10
*** OpenStack1 has quit IRC23:11
*** harlowja is now known as harlowja_away23:12
*** laurelm has joined #openstack-trove23:14
*** kevinconway has quit IRC23:14
*** jmontemayor has quit IRC23:22
*** cweid has quit IRC23:29
*** esp has left #openstack-trove23:30
*** OpenStack1 has joined #openstack-trove23:37
*** rongze has joined #openstack-trove23:43
*** rongze has quit IRC23:48
*** Barker has quit IRC23:49
*** demorris has quit IRC23:55
*** OpenStack1 has quit IRC23:57
*** harlowja_away is now known as harlowja23:57
*** ashestakov has quit IRC23:58
« Wednesday, 2013-12-18 Index Friday, 2013-12-20 »

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