Friday, 2014-02-21

*** ykaplan_ has quit IRC00:02
*** amitgandhi has joined #openstack-marconi00:28
*** zyuan has joined #openstack-marconi00:39
*** raxuanyu has joined #openstack-marconi00:39
*** jay-atl has joined #openstack-marconi00:40
*** apmelton1 has joined #openstack-marconi00:49
*** notmyname_ has joined #openstack-marconi00:50
*** lakspace_ has joined #openstack-marconi00:54
*** lakspace has quit IRC00:55
*** notmyname has quit IRC00:55
*** apmelton has quit IRC00:55
*** lakspace_ is now known as lakspace00:55
*** notmyname_ is now known as notmyname00:55
*** flwang has quit IRC01:12
*** amitgandhi has quit IRC01:13
*** amitgandhi has joined #openstack-marconi01:14
*** vkmc has quit IRC01:17
*** amitgandhi has quit IRC01:18
*** openstackgerrit has quit IRC01:18
*** openstackgerrit has joined #openstack-marconi01:19
*** oz_akan_ has joined #openstack-marconi01:21
*** fifieldt has joined #openstack-marconi01:33
*** nosnos has joined #openstack-marconi01:34
*** flwang has joined #openstack-marconi01:40
*** ayoung-dinner is now known as ayoung01:42
*** haomaiwa_ has quit IRC01:55
*** rossk has quit IRC01:59
*** flwang has quit IRC02:01
*** haomaiwang has joined #openstack-marconi02:08
*** flwang has joined #openstack-marconi02:22
*** amitgandhi has joined #openstack-marconi02:24
*** amitgandhi has quit IRC02:29
*** oz_akan_ has quit IRC02:37
*** oz_akan_ has joined #openstack-marconi02:38
*** oz_akan_ has quit IRC02:42
*** amitgandhi has joined #openstack-marconi03:14
*** amitgandhi has quit IRC03:19
openstackgerritA change was merged to openstack/marconi: feat(v1.1): Implement /ping endpoint for v1.1  https://review.openstack.org/7494703:20
*** balajiiyer has joined #openstack-marconi03:22
openstackgerritA change was merged to openstack/marconi: Replace unicode() by six.text_type for python3 compatability  https://review.openstack.org/7509303:24
*** balajiiyer has quit IRC03:24
*** flwang1 has joined #openstack-marconi03:35
*** flwang has quit IRC03:37
*** amitgandhi has joined #openstack-marconi04:15
*** amitgandhi has quit IRC04:19
*** amitgandhi has joined #openstack-marconi05:16
*** amitgandhi has quit IRC05:20
*** amitgandhi has joined #openstack-marconi06:16
*** amitgandhi has quit IRC06:20
*** saju_m has joined #openstack-marconi06:36
*** ykaplan_ has joined #openstack-marconi07:57
*** flaper87 has quit IRC07:57
*** flaper87 has joined #openstack-marconi07:57
*** ChanServ sets mode: +o flaper8707:57
*** jamieh has joined #openstack-marconi07:58
*** jamieh has quit IRC08:10
openstackgerritYeela Kaplan proposed a change to openstack/marconi: Implement Claim controller for sqlalchemy storage backend  https://review.openstack.org/7406808:14
openstackgerritYeela Kaplan proposed a change to openstack/marconi: Implement Queue controller for sqlalchemy storage backend  https://review.openstack.org/7094708:14
openstackgerritYeela Kaplan proposed a change to openstack/marconi: Sqlalchemy Message controller  https://review.openstack.org/7267708:14
*** saju_m has quit IRC08:31
*** ykaplan_ has quit IRC08:54
*** saju_m has joined #openstack-marconi09:01
*** saju_m has quit IRC09:02
*** amitgandhi has joined #openstack-marconi09:18
*** saju_m has joined #openstack-marconi09:21
*** amitgandhi has quit IRC09:22
*** chandan_kumar has joined #openstack-marconi09:25
*** flwang1 has quit IRC09:30
*** chandan_kumar has quit IRC09:51
*** chandan_kumar has joined #openstack-marconi10:05
*** jamieh has joined #openstack-marconi10:27
*** chandan_kumar has quit IRC10:28
*** jamieh has quit IRC10:49
*** jamieh has joined #openstack-marconi10:55
*** nosnos has quit IRC10:58
*** amitgandhi has joined #openstack-marconi11:19
*** amitgandhi has quit IRC11:23
*** jamieh has quit IRC11:32
*** jamieh has joined #openstack-marconi11:36
*** flwang has joined #openstack-marconi11:44
*** jamieh has quit IRC12:51
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat(sqlalchemy): add shards controller  https://review.openstack.org/7133513:03
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat(sql/catalogue): add catalogue controller  https://review.openstack.org/7302713:03
*** jamieh has joined #openstack-marconi13:08
*** mwagner_lap has joined #openstack-marconi13:11
*** oz_akan_ has joined #openstack-marconi13:12
*** oz_akan_ has quit IRC13:13
*** oz_akan_ has joined #openstack-marconi13:13
*** mpanetta has joined #openstack-marconi13:14
*** mpanetta has quit IRC13:19
*** mpanetta has joined #openstack-marconi13:20
*** vkmc has joined #openstack-marconi13:25
*** vkmc has quit IRC13:25
*** vkmc has joined #openstack-marconi13:25
*** malini is now known as malini_afk13:30
*** mwagner_lap has quit IRC13:38
*** sriram has joined #openstack-marconi13:39
*** sriram has quit IRC13:39
*** sriram has joined #openstack-marconi13:40
*** russellb is now known as rustlebee13:44
*** tedross_ has joined #openstack-marconi13:56
*** amitgandhi has joined #openstack-marconi14:02
*** amitgandhi has quit IRC14:04
*** amitgandhi has joined #openstack-marconi14:04
*** saju_m has quit IRC14:10
*** alcabrera has joined #openstack-marconi14:13
alcabreraGoooood morning! :D14:15
*** oz_akan_ has quit IRC14:20
*** oz_akan_ has joined #openstack-marconi14:22
openstackgerritFlavio Percoco proposed a change to openstack/marconi: Add an sqlalchemy storage to Marconi  https://review.openstack.org/7541814:23
flaper87alcabrera: ^14:23
flaper87alcabrera: GOOOOOOOOOOOOOOOOOOD MORNING14:23
flaper87it should be all grean now14:23
flaper87I squashed 3 reviews into 114:23
flaper87our testsuite is not granular enough to implement controllers separately14:23
flaper87I'm pretty sure some of the things there could've been done differently14:24
flaper87but I'm surely not an sqlalchemy expert and I would like us to merge this thing *TODAY*14:24
* alcabrera just noticed that all three controllers were squished into one14:24
alcabrera:)14:24
alcabreraWorks for me.14:24
flaper87alcabrera: btw, I just cough that `sa.exceptions`, did you check if there where more than 1?14:25
alcabreraI haven't looked14:25
alcabrerabut it's a good idea14:25
alcabreraI'll do that for shards and catalogue14:25
*** flwang has quit IRC14:25
alcabreragive it the good 'ol grep14:25
flaper87FWIW, I got rid of the Locked table14:27
flaper87w000t14:27
alcabreranice!14:27
*** flwang has joined #openstack-marconi14:28
flaper87tbh, I don't want to nitpick much this patches, I'd like to start landing the sqlalchemy and fix whatever comes up later14:28
alcabreraso, regarding expertise; I'm totally fine that none of us is an sql master. We'll find improvements over time, just like we did with mongo14:28
flaper87these*14:28
alcabrera+1 about landing these patches14:28
alcabreranits can wait14:29
flaper87alcabrera: btw, if you see something like tables.Messages.c.cid == (None) don't be scare, I'm just raising the middle finger to pep814:29
flaper87otherwise it would raise: "Equality on None should be X is None"14:29
flaper87T_T14:29
flaper87scared*14:30
flwang:-D14:30
alcabreralol14:30
alcabreraflaper87: very effective silencing, there14:30
flaper87:P14:30
flwangalcabrera: flaper87: may I get you guys bless on this tiny baby https://review.openstack.org/#/c/74971/ ?14:32
flwangalcabrera: thanks :)14:33
alcabreraflwang: blessed with love. It will sail away soon! +2/approved14:33
alcabrera:D14:33
flwangflaper87: ping14:34
flaper87flwang: pong ?14:34
flwangflaper87: is there any cool Italian name except Marconi?14:34
flaper87flwang: another baby comming ?14:35
flaper87:D14:35
flwangflaper87: nope, seriously, I'm going to pick a name for my son14:35
flwangbesides his Chinese name :)14:35
flwangif not, I will named him as Marconi :D14:36
flaper87flwang: there's a very very very cool italian name..... Flavio14:36
flaper87:D14:36
flaper87ROFL14:36
flaper87jokes apart14:36
flwangflaper87: lol14:37
*** caleb_ has joined #openstack-marconi14:38
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat(sql/catalogue): add catalogue controller  https://review.openstack.org/7302714:38
alcabreralol14:38
alcabreraI caught one more of those sa.exceptions in ^^14:39
alcabreraIt should be all clean14:39
alcabreranow14:39
alcabrerasqueaky clean14:39
* mpanetta squeaks14:39
alcabrerakgriffs_afk: merge all the things - especially all the sql. ;)14:39
* alcabrera notices mpanetta astounding degree of cleanliness - the squeaks betray him14:40
alcabrera*mpanetta's14:40
mpanettahaha14:40
openstackgerritFlavio Percoco proposed a change to openstack/marconi: Add an sqlalchemy storage to Marconi  https://review.openstack.org/7541814:40
flaper87alcabrera: points to the latest PS ^14:41
*** rektide_ is now known as rektide14:41
flaper87mmh, I feel kinda bad, yeela did most of the work there but it'll look like I did14:41
flaper87:/14:41
alcabrerahmmmm14:42
alcabreracan we have yeela resubmit it somehow?14:42
alcabreraor can we include as a part of the commit message great thanks for putting all that together?14:42
alcabreraflaper87: ^14:43
flaper87I added here to the commit message but I'd like her to submit it14:43
flaper87Her weekend starts today14:43
flaper87so, she's not around14:44
flaper87lets review it +2 it and get her to resubmit it14:44
flaper87and I'll ninja approve on monday morning14:44
alcabreraworks for me. :)14:46
*** balajiiyer has joined #openstack-marconi14:52
*** malini_afk is now known as malini15:00
openstackgerritA change was merged to openstack/marconi: Replace basestring by six for python3 compatability  https://review.openstack.org/7497115:02
*** jamieh has quit IRC15:04
*** jamieh has joined #openstack-marconi15:05
alcabreraw00t15:06
alcabreraI'll be back in a bit.15:06
*** alcabrera is now known as alcabrera|afk15:06
*** balajiiyer has quit IRC15:06
*** balajiiyer has joined #openstack-marconi15:07
balajiiyermalini: I have the marconi running on Pecan hosted here. http://162.242.241.194:7777/v1/queues15:13
malinibalajiiyer: what end-points does it have?15:14
balajiiyerThis now has a fully functional queues controller.15:14
balajiiyerand only that. :(15:14
malinibalajiiyer: hmm..not sure what tht means15:14
malinidoes it mena it has all endpoints related to queues?15:14
malinimean*15:14
balajiiyerthat means, you can add queues, list queues, delete queues and issue a head request15:15
malinicool15:15
balajiiyerprepare your tests against this endpoint and let me know if you face any issues.15:17
malinibalajiiyer: I am working with oneof the devstack guys to figure out my tempest issue15:17
malinican this wait?15:17
balajiiyerdefinitely, yes.15:17
maliniI'll submit my devstack patch & get to this15:18
balajiiyeralso, I was thinking, you could setup the tests and tsung servers and send me a note. I will run it how many ever times I want. :)15:18
malinibalajiiyer: sounds like a plan..I have the steps to create tsung servers etc. documented here https://github.com/rackerlabs/csi-marconi/blob/master/README.rst15:21
malinifor pecan/falcom comparison, we might need just a single tsung server15:22
balajiiyerok15:32
flaper87balajiiyer: AWESOME!15:34
*** flaper87 is now known as flaper87|afk15:42
*** flaper87|afk is now known as flaper8715:46
balajiiyerflaper87: alcabrera|afk kgriffs_afk need some love for this documentation patch https://review.openstack.org/#/c/72001/15:48
*** jcru has joined #openstack-marconi15:53
openstackgerritJay Baugh proposed a change to openstack/python-marconiclient: Added iterable list of queues  https://review.openstack.org/7544015:57
*** oz_akan_ has quit IRC16:01
*** oz_akan_ has joined #openstack-marconi16:02
*** kgriffs_afk is now known as kgriffs16:08
flaper87kgriffs: there you are16:12
flaper87kgriffs: https://review.openstack.org/#/c/75418/16:12
flaper87:D16:12
flaper87kgriffs: and good morning my friend!16:12
kgriffsgood afternoon man!16:14
kgriffsCindy is stopping by Rackspace office today16:14
*** alcabrera|afk is now known as alcabrera16:20
* alcabrera returns16:20
alcabrerawhat'd I miss? :)16:20
kgriffsCindy is the new PTL, I'm moving to China, and flaper87 is quitting Red Hat to mine dogecoin full-time.16:21
alcabrerakgriffs: oh, cool!16:21
alcabreraoh wow16:21
alcabreramuch surprise16:21
kgriffs:D16:21
alcabreraCongrats to Cindy - we all knew she would take over. :)16:21
alcabreraenjoy using librust-dogemine, flaper8716:22
kgriffswas only a matter of time...16:22
flaper87:)16:22
flaper87exactly that16:22
* flaper87 is now mining on alcabrera laptop16:22
* alcabrera shares the profits - 50|5016:23
flaper87kgriffs: btw, please please please please! Take a look at sqla patches16:23
flaper87:)16:23
alcabrerasqla catalogue and shards may be ready to go. They've passed jenkins and flaper87, so far16:23
kgriffsvote! vote! https://www.openstack.org/vote-atlanta/Presentation/deploying-marconi-for-production16:23
flaper87kgriffs: I'll ask ykaplan to resubmit it so she gets the authorship16:24
kgriffsflaper87 has a good one too16:24
kgriffsflaper87: kk16:24
flaper87kgriffs: but I'll like to get +2 from you16:24
kgriffsyes16:24
flaper87so I can ninja approve it16:24
kgriffsso, today I told the Solum guys they aren't going to see much of me16:24
kgriffslots of reviews, lots of code to do16:24
alcabreraw00t16:25
kgriffsvote! vote! https://www.openstack.org/vote-atlanta/Presentation/mentoring-others-and-yourself-an-openstack-community-success-story16:25
alcabrera+3 - flaper8716:25
alcabreralet's see some community advocacy16:26
alcabrera:D16:26
flaper87alcabrera: :D16:27
*** jamieh has quit IRC16:34
kgriffsso, question for you guys16:39
kgriffsI am starting to make v1.1 changes16:39
kgriffsand... testing16:39
alcabreraso, testing changes? :)16:39
kgriffsthe nuclear option is to just copy-paste all our wsgi transport tests16:39
alcabreraoh geez, yeah16:39
alcabreraI had forgotten about that16:39
alcabrerathat would be - slow tests16:39
*** kgriffs is now known as kgriffs_afk16:40
malinibalajiiyer: I am done submitting my devstack patch..do you want me to set up tsung servers/tests ?16:47
flaper87and now he's gone16:50
flaper87T_T16:50
maliniI think I'll just set it up, before my brain goes wandering16:50
maliniI dont trust my attention span :D16:51
balajiiyermalini: yes, thanks. working on some other tasks this morning16:54
malinibalajiiyer: am doing it now16:55
*** jamieh has joined #openstack-marconi16:56
alcabreraflaper87: I'll follow kgriffs_afk around for sql reviews. I suspect lunch/test patches got him for now. ;)17:06
alcabreraflaper87: I'm starting a review of the giga-sql patch. wish me luck. ;)17:08
flaper87alcabrera: oh boy, good luck with that17:09
alcabreraflaper87: do you remember the reason global-reqs is limit sqlalchemy to <= 0.8.99?17:13
alcabrerajust out of curiosity17:13
* alcabrera doesn't even know what sqlalchemy 0.9.x adds17:14
flaper87alcabrera: I think there was some issue with taskflow17:17
*** jamieh has quit IRC17:17
alcabrerathanks!17:19
alcabreraI got super curious and dug a little deeper17:20
alcabrerait appears that many are using sqlalchemy-migrate (https://pypi.python.org/pypi/sqlalchemy-migrate) which only works up to version 0.8.x of sqlalchemy17:20
alcabreramany == openstack projects17:20
alcabrerawhich in turn, I vaguely remember triggered a conversation about using alembic (https://pypi.python.org/pypi/alembic/0.6.3) instead17:21
alcabrerajust some history. :D17:21
* alcabrera 's curiosity is satisfied17:21
alcabreraflaper87: +2 on the sql patch. I saw some commented dead code, but I won't nitpick that. We can clean this up further later.17:24
alcabreraJenkins was happy with, so I feel pretty good about it, too17:24
alcabrera*with it17:24
alcabreraWe've also likely left lots of room for interested contributors to pick up the driver and optimize it17:24
*** alcabrera is now known as alcabrera|afk17:26
malinibalajiiyer: I started a small test against ur pecan server17:28
*** alcabrera|afk is now known as alcabrera17:29
balajiiyermalini: let me check whether it is running17:30
*** alcabrera is now known as alcabrera|afk17:30
maliniit ran - I just need to figure out a few 404s17:31
*** amitgandhi has quit IRC17:35
*** jamieh has joined #openstack-marconi17:48
*** vkmc has quit IRC17:55
*** jamieh has quit IRC17:56
*** caleb_ has quit IRC18:02
*** balajiiyer has quit IRC18:03
*** alcabrera|afk is now known as alcabrera18:11
*** balajiiyer has joined #openstack-marconi18:12
malinibalajiiyer: can you chk the head 404s in marconi logs?18:12
malinibalajiiyer: I was doing a GET & not a HEAD18:13
maliniso maybe thts why..trying again18:13
*** balajiiyer1 has joined #openstack-marconi18:14
*** balajiiyer has quit IRC18:16
*** amitgandhi has joined #openstack-marconi18:17
malinibalajiiyer1: is ur pecan server down ?18:17
balajiiyer1*checking*18:18
*** vkmc has joined #openstack-marconi18:19
malinibalajiiyer1: I sent you the tsung server info18:20
maliniSo you shud be able to run them as needed18:20
maliniLet me know if you run into issues18:20
balajiiyer1sweet, thanks malini18:21
balajiiyer1*just brought the server backup*18:21
maliniDId I bring it down?18:21
balajiiyer1malini: no, you werent the culprit18:28
malinigood :)18:29
balajiiyer1Did you get head reqs to work? I can get a proper response from that endpoint18:29
malinibalajiiyer1: I tried to chk tht & the server went down18:29
malinican you give it a try?18:30
balajiiyer1duh, I will take a look at tsung config file18:30
maliniI did update the xml to use the right verb18:30
jay-atlHey, can you guys +1 my patch for python-marconiclient when you get a chance.. https://review.openstack.org/#/c/75440/18:32
flaper87alcabrera: +2 for yout +218:32
alcabrerajay-atl: I'll give it a review before I head out today18:33
alcabrera:)18:33
alcabreraflaper87: yaaaay18:33
jay-atlThank you!18:33
flaper87alcabrera: we need kgriff's to bless it18:34
alcabrerakgriffs_afk: is the key to everything18:35
alcabreraonly he can save the world18:35
mpanettaAh crap, we are screwed.  He is AFK! :P18:35
jay-atlThe world is now doomed. Have a nice day.18:35
sriramhaha18:36
flaper87alcabrera: i3 is pretty amazing18:44
flaper87:D18:45
* flaper87 used to use awesomewm until 3 hours ago18:45
flaper87:D18:45
alcabreralol18:49
alcabreraI've yet to use a tiling wm18:49
alcabreraIt sounds so cool18:49
alcabreraflaper87: ^18:49
flaper87alcabrera: it is, it's very confortable :D18:51
*** caleb_ has joined #openstack-marconi18:51
maliniflaper87: Can you take a look at this one https://review.openstack.org/#/c/75160/ ?18:52
maliniI am still fighting with the Connection error in tempest tests18:52
malinithis is an attempt towards getting rid of tht18:52
*** ayoung is now known as ayoung-lunch18:55
flaper87malini: NO!18:58
flaper87malini: just kidding18:58
flaper87:D18:58
flaper87malini: looks good. Why do you need $*_PROTOCOL ?18:59
flaper87does devstack uses https ?18:59
flaper87use*18:59
flaper87HOLY CRAP! Linkin park is coming to italy18:59
flaper87O.O18:59
flaper87not that you care or anything19:00
flaper87just sayin'19:00
*** kgriffs_afk is now known as kgriffs19:04
alcabreralol19:04
alcabreraflaper87: lucky! You going to the concert?19:04
* alcabrera used to listen to Linkin' Park way back19:04
maliniflaper87: had to step away for the cupcake..yummm19:04
* alcabrera has a serious case of cupcake moustache19:05
malinialcabrera: now we made everybody hungry19:05
* malini resumes normal broadcast19:05
kgriffso/19:06
maliniflaper87: I believe you can specify ssl19:06
kgriffscpalleres is sitting next to me.19:06
kgriffsflaper87: aren't you jealous?19:06
maliniflaper87: But the main thing is , I was trying to follow exactly what everybody else is doing19:06
*** cpallares has joined #openstack-marconi19:06
malinisince nobody has a clue, why my test cannot connect19:06
alcabreracpallares: heeeey! :D19:07
malinicpallares: Rackspace ATL is even better ;)19:07
maliniWe just got free lunch, cupcakes & we are going to get beer & snacks in another hour19:07
cpallaresheeey alcabrera!19:08
cpallareshi malini :)19:08
cpallareshaha19:08
kgriffscupcakes!19:09
kgriffsWAH19:09
kgriffswe only got the free lunch here.19:09
cpallaresand it was good!19:09
alcabreralol19:10
* alcabrera beams kgriffs and cpallares a cupcake19:10
alcabrerano DRM - promise19:10
cpallaresalcabrera open source cupcake :D19:13
alcabrerayes. :)19:16
alcabrerabrb19:16
*** alcabrera is now known as alcabrera|afk19:16
*** amitgandhi has quit IRC19:16
*** amitgandhi has joined #openstack-marconi19:16
*** alcabrera|afk is now known as alcabrera19:18
flaper87alcabrera: most probably, yes! I found out recently19:22
flaper87:D19:22
flaper87kgriffs: I AM!!!19:22
flaper87cpallares: kick kgriffs for me! Thanks!19:22
flaper87you've my permission19:22
flaper87just, don't break his laptop19:22
flaper87cpallares: btw, does that mean that everything you've pending will be done by EOD ?19:24
flaper87:D19:24
flaper87kgriffs: dude, sqlalchemy patch, pls19:24
flaper87:D19:24
*** jergerber has joined #openstack-marconi19:26
flaper87kgriffs: cpallares you guys ignoring me? NOW I'M JEALOUS!19:28
flaper87:(19:28
alcabrera:P19:36
alcabrerakgriffs, cpallares:  we miss you. Join us again when the fun times in AUS cool down a bit. ;D19:37
*** raxuanyu has left #openstack-marconi19:48
*** zyuan has quit IRC19:55
flaper87kgriffs: man, sqlalchemy19:55
flaper87you ain't going anywhere w/o +2'ng that19:55
flaper87don't approve it, just bless it19:55
alcabreraI'm going to head out a little early today. :)20:03
alcabreraanything you'd like for me to review before I head out, flaper87?20:04
alcabreraanyone? :D20:04
flaper87alcabrera: kgriffs's brain20:04
flaper87it seems to have stopped working20:04
flaper87he doesn't reply back20:04
flaper87:(20:04
flaper87knock there, call him20:04
flaper87do something !!!20:04
flaper87:D20:04
alcabreralol20:04
alcabreraI'd love to.20:04
alcabreraHe's off in another office. :P20:04
jay-atlHe's probably hiding in a closet somewhere... (We've found him coding in some weird locations...)20:04
flaper87jay-atl: LOOOOOL20:05
alcabreraoh20:05
flaper87I won't try to imagine where20:05
alcabrerayes20:05
alcabreraone thing20:05
flaper87:P20:05
jay-atlSeriously, we did find him under a desk once...20:05
alcabreraflaper87: could you abandon or deprecate somehow the other three sql patches?20:05
flaper87alcabrera: ah sure, I will20:05
alcabrerait's confusing, since sql:giga is the real version of that20:05
alcabrerathanks!20:05
flaper87alcabrera: done, -2 ykaplan's and abandoned mine20:07
alcabreracool. :)20:07
alcabreraless confusion20:07
alcabreragreatness20:07
*** malini is now known as malini_afk20:13
alcabreraalright, I'm out. ;)20:14
alcabrerao/20:14
alcabreraall20:14
*** alcabrera has quit IRC20:14
*** jay-atl has quit IRC20:18
*** kgriffs is now known as kgriffs_afk20:36
*** raxuanyu has joined #openstack-marconi20:37
*** cpallares has quit IRC20:38
*** zyuan has joined #openstack-marconi20:39
*** kgriffs_afk is now known as kgriffs20:40
kgriffsflaper87: o/20:40
flaper87kgriffs: 'sup ?20:40
flaper87:D20:40
kgriffssorry, been having some good conversations with Cindy20:41
flaper87>_>20:41
flaper87;P20:41
kgriffslet me do some reviews20:41
flaper87kgriffs: sqlalchemy first, pls :D20:41
kgriffskk20:41
kgriffsflaper87: real quick. do you have some thoughts about organizing tests for v1.1 ?20:41
flaper87I was saying to Alej earlier today, I didn't nitpick much on these patches because I'd like to get them merged. We can fix whatever needs to be fixed later on20:42
kgriffsi could just copy them like with the transport, but that seems a bit silly20:42
kgriffsflaper87: ok20:42
flaper87kgriffs: copy / paste is pretty much what is done in other projects (at least glance)20:43
flaper87I've an idea as to how we could improve that20:43
kgriffsflaper87: shall we file a bug to review sqlalchemy and make it pretty for Juno, or were you thinking the cleanup would happen sooner?20:43
kgriffsflaper87: love to hear your idea20:43
kgriffs(wrt tests)20:43
flaper87kgriffs: I'd like the cleanup to happen right after the patch lands TBH20:43
flaper87but I'd at least be able to say "we have the storage and it works as expected"20:44
flaper87then we can start cleaning it up and improving it20:44
kgriffsah, ok20:44
flaper87s/I20:44
flaper87s/I'd/We'd/20:44
flaper87you got my point20:45
flaper87anyway20:45
flaper87tests20:45
flaper87What if we have a base test class for the API as we have for storages (btw, this will most likely need to be done for storage drivers too) but instead of just having base methods for tests, we can have base methods AND version tags on those methods. Then we can tag the subclass as being the tests for "v1.1" and that will enable and disable some tests depending on the version cap in the class20:47
flaper87not sure how clear is that :/20:47
flaper87The base of this idea is that not all endpoints change from 1 version to the other20:47
*** cpallares has joined #openstack-marconi20:47
flaper87and we want to change / add tests of the endpoints that have changed20:48
flaper87but keep all other tests as they are20:48
flaper87kgriffs: Does that make sense at all?20:48
kgriffsyep, makes sense20:49
flaper87I can mock up the base class if you that helps20:49
*** ekarlso has quit IRC20:50
*** ekarlso has joined #openstack-marconi20:50
kgriffsso, i will need this before I can land my first v1.1 patch. It is done, just needs testing20:50
flaper87kgriffs: we already planned to release v1.1 for Juno, right?20:55
kgriffsthat is our plan. We have the option to slip if we need to, but I am going to be working on this a lot over the next few days20:57
kgriffsso, we'll know soon. :p20:57
*** zyuan has quit IRC20:57
kgriffsi think the implementation isn't bad, mostly testing time is what I am worried about since Malini has been so busy with tempest20:57
*** zyuan has joined #openstack-marconi20:58
kgriffsi mey need to do the functional tests myself20:58
flaper87kgriffs: mmh, well, if it makes it easier to move this forward, I'd be ok to copy / paste tests and then work20:59
flaper87but it seems like a good opportunity to improve the transport testsuite20:59
*** cpallares has quit IRC21:03
*** cpallares has joined #openstack-marconi21:04
*** cpallares has quit IRC21:06
*** cpallares has joined #openstack-marconi21:13
openstackgerritShawnyu proposed a change to openstack/marconi: pep8 fix  https://review.openstack.org/7504821:13
openstackgerritShawnyu proposed a change to openstack/marconi: docstrings revised  https://review.openstack.org/7511321:13
openstackgerritShawnyu proposed a change to openstack/marconi: doc(queues.storage.mongodb.catalogue): Add docstrings  https://review.openstack.org/7480721:13
*** cpallares has quit IRC21:15
*** amitgandhi has quit IRC21:17
*** jcru has quit IRC21:19
openstackgerritA change was merged to openstack/marconi: feat(sqlalchemy): add shards controller  https://review.openstack.org/7133521:23
openstackgerritA change was merged to openstack/marconi: feat(sql/catalogue): add catalogue controller  https://review.openstack.org/7302721:25
*** jay-atl has joined #openstack-marconi21:28
flaper87w0000000000000t first 2 patches merged21:30
flaper87:D21:30
flaper87kgriffs: thanks21:30
*** jay-atl has quit IRC21:30
*** jay-atl has joined #openstack-marconi21:31
*** amitgandhi has joined #openstack-marconi21:42
*** jay-atl has quit IRC21:43
*** amit has joined #openstack-marconi21:43
*** amit is now known as Guest2513421:44
*** jay-atl has joined #openstack-marconi21:46
*** amitgandhi has quit IRC21:47
*** ayoung-lunch is now known as ayoung22:02
*** kgriffs is now known as kgriffs_afk22:03
*** oz_akan_ has quit IRC22:06
*** westmaas has quit IRC22:19
*** westmaas has joined #openstack-marconi22:20
*** kgriffs_afk is now known as kgriffs22:21
*** sriram has quit IRC22:26
*** kgriffs is now known as kgriffs_afk22:31
*** kgriffs_afk is now known as kgriffs22:33
*** malini_afk is now known as malini22:50
*** vkmc has quit IRC22:50
*** Guest25134 has quit IRC22:51
*** caleb_ has quit IRC22:59
*** jergerber has quit IRC23:16
*** tedross_ has quit IRC23:23
kgriffsflaper87: yo, sorry for dissapearing earlier - I had to do a presentation and forgot to AFK my status23:23
kgriffs:p23:23
flaper87kgriffs: you're not forgiven23:23
flaper87:P23:23
kgriffs:(23:23
flaper87ok ok, you are23:23
kgriffs:)23:23
flaper87but don't do it again23:23
kgriffs<phew>23:23
*** malini is now known as malini_afk23:26
kgriffsflaper87: were you going to get Yeela to resubmit this so she gets credit? https://review.openstack.org/#/c/75418/23:28
kgriffsseems like I read something along those lines earlier today23:28
flaper87kgriffs: correct23:28
flaper87if it looks good just +223:28
flaper87and I'll ninja approve it23:28
kgriffsok23:29
kgriffsflaper87: also, wrt testing for 1.1, I can take stab at what you suggested23:29
flaper87kgriffs: awesome! it looks like a great opportunity to finally improve our tests for transports23:30
flaper87that'd help external contributors to build their own stuff23:31
kgriffsok23:31
kgriffsI'll get a POC done and we can refine23:31
kgriffsthanks for the idea!23:31
flaper87cool, I'm pretty sure that idea is not bullet proof but we'll see23:32
kgriffsonly one way to find out...23:32
kgriffs:)23:32
flaper87... let kgriffs do it23:33
flaper87:P23:33
* flaper87 is obviously kidding23:33
kgriffsheh23:33
balajiiyer1Im out guys, have a good weekend23:33
flaper87balajiiyer1: no23:33
flaper87stay23:33
*** mpanetta has quit IRC23:33
flaper87don't go anywhere23:33
balajiiyer1*is frozen*23:34
flaper87it's friday23:34
flaper87is not that you've something to do23:34
flaper87who does things on fridays?23:34
flaper87no one does, right?23:34
flaper87... right?23:34
flaper87right :(23:34
balajiiyer1agreed. I can stay. you know what, Im staying. hell yeah23:34
balajiiyer1so whatcha got?23:35
flaper87balajiiyer1: http://www.rust-lang.org/23:35
*** rustlebee is now known as russellb23:36
balajiiyer1fn main() {23:38
balajiiyer1    let days = ["Monday", "Tuesday","wednesday","thursday","friday"];23:38
balajiiyer1    let weekend = ["friday"]23:38
balajiiyer1    let mut odds = nums.iter().map(|&x| x * 2 - 1);23:38
balajiiyer123:38
balajiiyer1    for weekend in days {23:38
balajiiyer1        do spawn {23:38
balajiiyer1            println!("it's {:s} time to have beer!", weekend);23:38
balajiiyer1        }23:38
balajiiyer1    }23:38
balajiiyer1}23:38
balajiiyer1Rust says time to have beer23:39
flaper87balajiiyer1: mmhh, I don't recall seeing that in the website23:40
balajiiyer1alright, Im out for reals. bye.23:40
flaper87:D23:40
flaper87balajiiyer1: enjoy23:40
flaper87have a good w/e23:40
*** balajiiyer1 is now known as balajiiyer1_afk23:40
*** balajiiyer1_afk has quit IRC23:47

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