Thursday, 2013-09-26

« Wednesday, 2013-09-25 Index Friday, 2013-09-27 »
*** amitgandhi has joined #openstack-marconi00:08
*** amitgandhi has quit IRC00:11
*** malini is now known as malini_afk01:23
*** fifieldt has joined #openstack-marconi01:27
openstackgerritKurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields  https://review.openstack.org/4725902:05
openstackgerritKurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load  https://review.openstack.org/4727202:05
*** amitgandhi has joined #openstack-marconi02:19
openstackgerritKurt Griffiths proposed a change to openstack/marconi: Update oslo to latest and greatest.  https://review.openstack.org/4838002:20
openstackgerritA change was merged to openstack/marconi: proxy: adds oslo.cache, config support, & stevedore  https://review.openstack.org/4747502:25
openstackgerritA change was merged to openstack/marconi: test(proxy): fix proxy-transport tests  https://review.openstack.org/4761202:26
openstackgerritA change was merged to openstack/marconi: Move tests/unit/storage/base under m/tests/storage  https://review.openstack.org/4805702:41
openstackgerritA change was merged to openstack/marconi: Move queues' unittests under tests/unit/queues/  https://review.openstack.org/4806502:42
*** kgriffs_afk is now known as kgriffs02:46
*** kgriffs is now known as kgriffs_afk02:49
*** amitgandhi has quit IRC03:33
*** kgriffs_afk is now known as kgriffs03:36
*** ayoung has quit IRC03:41
*** kgriffs is now known as kgriffs_afk04:07
*** kgriffs_afk is now known as kgriffs04:20
*** reed has quit IRC04:54
*** fifieldt has quit IRC05:21
*** kgriffs is now known as kgriffs_afk07:04
*** kgriffs_afk is now known as kgriffs07:34
*** kgriffs is now known as kgriffs_afk07:43
*** gordonsim has joined #openstack-marconi07:44
*** kgriffs_afk is now known as kgriffs08:04
*** briancline has quit IRC08:10
*** briancline has joined #openstack-marconi08:17
*** kgriffs is now known as kgriffs_afk08:17
*** fifieldt has joined #openstack-marconi08:31
*** fifieldt has quit IRC08:33
*** yassine has joined #openstack-marconi08:57
*** flaper87|afk is now known as flaper8709:07
*** kgriffs_afk is now known as kgriffs09:08
*** kgriffs is now known as kgriffs_afk09:18
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: fix: age presented as negative  https://review.openstack.org/4827709:19
openstackgerritA change was merged to openstack/marconi: fix: age presented as negative  https://review.openstack.org/4827709:40
*** kgriffs_afk is now known as kgriffs10:09
*** kgriffs is now known as kgriffs_afk10:18
*** malini_afk is now known as malini11:03
*** kgriffs_afk is now known as kgriffs11:10
*** kgriffs is now known as kgriffs_afk11:19
*** tedross has joined #openstack-marconi11:34
*** gordonsim1 has joined #openstack-marconi11:34
*** gordonsim has quit IRC11:35
*** acabrera has joined #openstack-marconi11:39
acabreraMorning! :D11:39
acabrerabrb11:43
*** acabrera has quit IRC11:43
*** acabrera has joined #openstack-marconi11:44
*** acabrera is now known as alcabrera11:47
alcabreraflaper87: hey hey! :)11:48
*** malini is now known as malini_afk12:02
*** kgriffs_afk is now known as kgriffs12:10
*** kgriffs is now known as kgriffs_afk12:19
*** oz_akan_ has joined #openstack-marconi12:20
*** ekarlso has quit IRC12:20
*** oz_akan_ has quit IRC12:21
*** oz_akan_ has joined #openstack-marconi12:21
flaper87alcabrera: hey hey!12:27
* flaper87 just got back from lunch12:27
flaper87how are you doing?12:27
alcabreraflaper87: A little groggy, but otherwise pretty good. How about you? :)12:28
flaper87alcabrera: I'm doing well, preparing for the OpenStack meetup12:29
alcabreraflaper87: which one? :o12:30
alcabreraflaper87: oh, ohhhh - meetup. I read that as meeting!12:30
* alcabrera needs to wake up a bit12:31
alcabreracool, cool12:31
flaper87hahahahha12:35
flaper87alcabrera: \_/? take some coffee12:35
* alcabrera drinks it all, with a splash of almond milk12:37
alcabreraflaper87: :D12:37
*** mpanetta has joined #openstack-marconi13:09
*** kgriffs_afk is now known as kgriffs13:11
alcabrerampanetta, kgriffs: o/13:14
flaper87mpanetta: kgriffs \o/13:14
mpanettamornin :)13:14
flaper87mpanetta: I raised both hands, I hope that counts!13:15
flaper87:D13:15
flaper87mpanetta: gooood morning13:15
alcabrera:)13:15
mpanettaHow goes?13:16
mpanettaI've been writing python curses code to help make things easier with testing, etc...13:17
flaper87somhow I read that as: I've been cursing Python's code to make things easier with testing13:17
flaper87somehow*13:17
mpanettahaha13:17
flaper87I haven't had enough coffee13:18
*** cthulhup has joined #openstack-marconi13:20
*** kgriffs is now known as kgriffs_afk13:20
alcabreraflaper87: \_/?13:21
flaper87alcabrera: hahaha, danke sir!13:23
fvolleroflaper87: +113:25
* flaper87 is working in a coworking office today!13:26
* flaper87 is missing his coffee machine13:26
fvolleroflaper87: lol :)13:26
fvolleroflaper87: you can always come in Brno if you want to stay in the office :)13:26
flaper87dude, I just said I'm missing my coffee and I'm like 20kms away from home. What's worng with you?13:27
flaper87wrong*13:27
flaper87:D13:27
fvollerohahahahaha13:27
*** malini_afk is now known as malini13:27
fvolleroflaper87: no need for 'nice' weather and sunny places ? :)13:27
*** ayoung has joined #openstack-marconi13:30
malinigood morning!!13:34
alcabreramalini: morning! :)13:35
flaper87malini: good morning!13:35
*** amitgandhi has joined #openstack-marconi13:40
*** amitgandhi has quit IRC13:45
*** amitgandhi has joined #openstack-marconi13:45
*** jcru has joined #openstack-marconi13:46
*** cthulhup has quit IRC13:46
malinialcabrera: Can you plz update https://wiki.openstack.org/wiki/Marconi/specs/proxy/v1#Register_a_Partition with the allowed values for hosts, weight etc. ? eg. lower, upper limits for weight etc. ?13:56
maliniI am writing tests for proxy & would like to include those13:57
alcabreramalini: hmmm... the current catch is that no upper bounds have been established for weight yet.13:57
alcabreramalini: the lower bound is >= 013:58
alcabreramalini: but sure, I'll add some notes on this!13:58
malinihmm..so I can send in 100000000000000000000000000000000000000000 ?13:58
alcabreramalini: theoretically, yes. :D13:59
alcabreramalini: though I wonder what the limits are on JSON, hmmmmm!13:59
* alcabrera tinkers13:59
maliniI was wondering the same thing :D13:59
malinialcabrera: will proxy requests go thru the same input validations, as the rest of marconi api ?14:00
alcabreramalini: simplejson.dumps({'a': 2 ** 512}) => '{"a": 13407807929942597099574024998205846127479365820592393377723561443721764030073546976801874298166903427690031858186486050853753882811946569946433649006084096}'14:03
alcabreraso... I think weight can be pretty big. :314:03
alcabreraI also tried with plain 'import json'14:03
malinithat is huge :D, so my test needs to be huger(is that a word?) :D14:04
alcabreramalini: in a sense, yes. The administrative API (/v1/partitions, /v1/catalogue) have their own validation paths. /v1/everything_else just forwards to marconi-queues.14:04
alcabreramalini: lol - nah, it just needs to have a test to make sure a HTTP 400 is returned for weight <= 0.14:05
maliniI have tht one already14:05
alcabreramalini: cool14:05
alcabreramalini: weight up to 2**64 is big enough, IMO14:05
alcabrera(though if someone had that many partitions, I'd ask them to implement a smarter proxy)14:06
malini:D14:08
*** kgriffs_afk is now known as kgriffs14:11
*** kgriffs is now known as kgriffs_afk14:21
flaper87alcabrera: https://review.openstack.org/#/c/48223/ :)14:29
flaper87kgriffs_afk: ^14:29
*** kgriffs_afk is now known as kgriffs14:29
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Use latest PBR  https://review.openstack.org/4845914:31
flaper87kgriffs: alcabrera could you please sanity check this review? https://review.openstack.org/#/c/48459/14:31
flaper87client's setup is broken :(14:32
alcabreraflaper87: on it. :)14:33
openstackgerritKurt Griffiths proposed a change to openstack/marconi: perf(mongodb): Combine project and queue message fields  https://review.openstack.org/4725914:33
openstackgerritKurt Griffiths proposed a change to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load  https://review.openstack.org/4727214:33
flaper87alcabrera: thanks, sorry for pushing :(14:34
alcabreraflaper87: no worries. :D14:35
alcabreraflaper87: reviewed - -114:35
alcabrerawhat's with the setup.cfg changes? It seems unrelated. :x14:35
flaper87alcabrera: ah damn it, indeed they are unrelated14:35
flaper87fuck14:35
flaper87let me fix that14:35
flaper87I used those lines to test the setup thing14:36
alcabreraflaper87: woot - code review + 114:37
alcabreraflaper87: marconi-gc looked awfully suspicious in the client. ;)14:37
flaper87ahahahhahahaha14:37
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Use latest PBR  https://review.openstack.org/4845914:37
flaper87indeed14:37
flaper87ooooooooook, there you go14:38
kgriffsmpanetta: can you see flavio's comment here: https://review.openstack.org/#/c/47459/1/marconi/version.py14:40
alcabreraflaper87: +214:40
kgriffsflaper87: good suggestion; I'm not sure how I missed that. :p14:40
flaper87alcabrera: thank you14:40
flaper87kgriffs: :D14:40
alcabrerakgriffs, flaper87: Jenkins likes the marker patch - are we ready to approve that ball of radness? :D14:42
flaper87YES WE ARE!14:42
flaper87done14:43
kgriffsw000t14:43
flaper87kgriffs: great work there, +100000014:43
alcabreraawesome14:43
kgriffsI'll be back in a few minutes - going to head into the office where I have 3 large boxes of pop-tarts to celebrate with14:44
kgriffs:D14:44
alcabrerakgriffs: yeeeeeeeeeesssssss14:44
* alcabrera wants some of those poptarts14:44
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Add a base transport class  https://review.openstack.org/4828714:45
openstackgerritFlavio Percoco proposed a change to openstack/python-marconiclient: Restructure client code  https://review.openstack.org/4822314:45
openstackgerritA change was merged to openstack/marconi: fix(mongodb): Marker generation creates a bottleneck under heavy load  https://review.openstack.org/4727214:45
openstackgerritA change was merged to openstack/marconi: perf(mongodb): Combine project and queue message fields  https://review.openstack.org/4725914:45
flaper87w0000t14:45
* kgriffs cheers14:45
alcabrerahurray for simpler drivers14:46
alcabrerafuture storage drivers are celebrating this moment14:47
openstackgerritA change was merged to openstack/python-marconiclient: Use latest PBR  https://review.openstack.org/4845914:47
alcabrera^^ and let us not forget the small victories14:47
openstackgerritKurt Griffiths proposed a change to openstack/marconi: Update oslo to latest and greatest  https://review.openstack.org/4838014:47
kgriffsoz_akan_: see flaper87's comment here - https://review.openstack.org/#/c/47459/1/marconi/version.py14:48
kgriffswill that work for you? if so I will abandon the patch14:48
oz_akan_kgriffs: hi, let me see14:49
kgriffskk14:49
oz_akan_as long as we have the commit information in version, that is perfect14:50
kgriffsok14:51
oz_akan_I am not sure if we really need 2013.2 section, I'd rather not have it, but would hurt to have it14:51
oz_akan_"would not hurt"14:51
kgriffsi suppose I could strip it for you. iirc when we tag a release it won't have any commit information, btw.14:51
flaper87kgriffs: correct, releases' tag don't have that info14:52
oz_akan_but a release is a specific commit, right?14:52
flaper87just 201X.Y14:52
flaper87oz_akan_: OS releases are branches14:53
* kgriffs wonders what will happen in 7 years14:53
flaper87stable/grizzly etc14:53
flaper87then there are tags for stable releases14:53
flaper87201X.Y14:53
oz_akan_"when we tag a release" we will tag a specific commit and then after each commit we woull see commit number in __version__, would we?14:54
oz_akan_flaper87: ^^14:55
kgriffsI'm not sure that __version__ shows that14:55
flaper87that is true if we're running Marconi from a git repo14:55
kgriffsmarconi.version.version_info for sure14:55
flaper87kgriffs: https://github.com/openstack/marconi/blob/master/marconi/__init__.py#L3014:56
flaper87it does14:56
flaper87mmh, well, it uses version_string14:56
oz_akan_ok14:56
oz_akan_and a marconi installed via pip install, would have tag?14:57
flaper87yeah, well, it'll have whatever we specify in the version field in setup.cfg14:57
kgriffsoz_akan_: can you try out "if teh_hash in marconi.version.version_info.release_string()"14:59
kgriffsif that works for you, I guess I can add a little helper method that does it or something if you want14:59
kgriffsok, pop-tarts are calling my name. be back in a few.14:59
oz_akan_kgriffs: let me try14:59
*** kgriffs is now known as kgriffs_afk15:00
*** cthulhup has joined #openstack-marconi15:00
*** tedross has quit IRC15:02
oz_akan_commit 71c0a7b674509cfb0e2c90cb986c496b48f16c1415:07
oz_akan_2013.2.a384.g71c0a7b15:07
oz_akan_after 2013.2.a384.g it matches15:07
oz_akan_kgriffs_afk: flaper87 ^^15:07
alcabreraoz_akan_: I wonder what the 'a384.g' part means?15:09
oz_akan_i don't yet yet15:09
oz_akan_know yet15:09
flaper87oz_akan_: alcabrera the a\d+ part is the number of commits since the most recent tag15:11
flaper87the g.* part is the commit sha15:11
flaper87oz_akan_: how are you getting the commit sha?15:12
oz_akan_alcabrera: now I know15:12
flaper87try using: git describe --always15:12
flaper87that should give you the commit sha used by pbr15:12
oz_akan_flaper87:  I did git log15:12
alcabreraflaper87: thanks for the info. :)15:13
flaper87oz_akan_: git log --abbrev-commit --pretty=oneline15:13
oz_akan_flaper87: tks15:13
flaper87oz_akan_: np :)15:13
flaper87alcabrera: np :)15:13
flaper87https://twitter.com/flaper87/status/38324838146573926515:15
alcabreraflaper87: sweet! You can already create queues? :)15:17
flaper87erm, nope :( but I'm not far from there15:17
alcabreraflaper87: awesome. getting close!15:19
alcabreraflaper87: I'll have to port my message controller at some point to use your upcoming request/response classes15:19
*** tedross has joined #openstack-marconi15:19
flaper87alcabrera: yeaaah, I'm working on the queue now, as soon as it's ready, you can port yours15:20
alcabreraflaper87: thanks for your work on this. I'm really looking forward to putting the client to use. :D15:21
*** whenry has joined #openstack-marconi15:21
*** kgriffs_afk is now known as kgriffs15:24
fvollerooz_akan_: add this alias in yout gitconfig: lc= rev-list -1 HEAD15:32
oz_akan_what does it do?15:32
oz_akan_shortcut?15:32
fvollerooz_akan_: give you the last commit (HEAD) SHA15:33
flaper87rev-list --abbrev-commit -1 HEAD <- short commit15:33
fvolleroflaper87: yep, that's the other one15:33
*** whenry has quit IRC15:34
flaper87zyuan: ping15:40
fvolleroYour opinion on the error we should return in case of empty project id? I am suggesting 400 Bad Request since the Header is missing from the Client side (Request)15:43
fvolleros/error/HTTP error/15:43
* kgriffs is eating a blueberry pop-tart15:43
* kgriffs is happy15:43
flaper87kgriffs: keep saying that and I'll take those +2s back!15:43
flaper87>.>15:43
flaper87:D15:44
alcabreraflaper87: zyuan us out for the rest of the week.15:44
alcabrera*is15:44
flaper87malini: had chocolate cake yday, you're having pop-tarts15:44
* alcabrera is eating a chocolate fudge poptart15:44
flaper87mmh, not fair, nooooot fair15:44
flaper87you know what? I'm going for a gelato, I'll then have a coffee15:45
maliniflaper87: you shud come join us in atl15:45
malinino..maybe we shud come join you there <:o)15:45
flaper87malini: would love to, Marconi hackton15:45
flaper87well, how to put this. It's cheaper to pay 1 ticket thant 615:46
flaper87^^15:46
flaper87fvollero: 400 sounds right to me15:46
flaper87alcabrera: gtk, I just wanted to ask him to abandone openstack/requirement reviews15:46
fvolleroflaper87: I just need to test it, but seems that get_header have a parameter required if is in True we may have our ass covered15:46
* fvollero is eating a cocosette15:47
* alcabrera looks up cocosette15:49
alcabreraOooohh, do they *do* have coconut in them... :)15:49
fvollero:)15:49
alcabrerafvollero: nice treat!15:50
fvolleroalcabrera: :) You know, i love to drive flaper87 crazy :)15:50
flaper87fvollero: and I love hating you every day more15:51
fvolleroflaper87: <315:51
alcabreralol15:51
flaper87it feels so good, you know. that heat and serial killer feeling warming up15:51
flaper87:D15:51
fvollerohahahahahah15:51
flaper87alcabrera: those are from Venezuela where I'm from and they are DELICIOUUUUSSSSSSS15:52
*** vkmc has joined #openstack-marconi15:52
fvollero^_^15:52
flaper87I used to have like dozens and dozens of those per day15:52
flaper87:D15:52
fvolleroflaper87: :)15:52
alcabreraflaper87, fvollero: maaaan, I'd love to get some. I might be able to around here in the local international markets15:52
fvolleroalcabrera: depend on where you're based...15:53
fvolleroalcabrera: I guess it's tricky to get those15:53
flaper87alcabrera: I bet you can find them.15:53
alcabreraflaper87: speaking of requirements - https://review.openstack.org/#/c/48339/ (:P)15:53
alcabrerafvollero: If all else fails - http://www.amazon.com/Nestle-Cocosette-Galleta-Rellena-grs-each/dp/B005EZVLCC15:53
fvolleroalcabrera: hahahaha :)15:54
* fvollero think to start an exchanging market between Venezuela and Europe, to make people from Venezuela happy 15:54
flaper87WTF? Amazon sells them?15:55
flaper87amazon.it doesn't15:55
flaper87somehow, that doesn't surprises me much15:56
fvolleroof course.15:56
flaper87aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaanyway, at least I already know who to threat when I need some15:56
alcabreralol15:57
fvollerorofl15:57
fvollerook, that's weird... I set up the requirement for the 'X-PROJECT-ID' and send it but i still get 40015:58
flaper87fvollero: git diff ?15:59
fvolleroflaper87: i'm thinking that I can't rely on get_header function for some reason that i'm still investigating16:01
flaper87fvollero: hey, watch it, Marconi's code is 200% reliable16:02
* flaper87 runs away16:02
* flaper87 closes all bugs16:02
fvolleroflaper87: -.-16:02
* flaper87 destroys lp history16:02
fvolleroflaper87: i'm talking about falcon :P16:02
flaper87fvollero: ah ok ok!!16:02
* flaper87 STFU16:02
malinifvollero: maybe you dont have the Accept header ?16:03
fvolleromalini: I have it16:03
malinithere goes my theory :(16:03
flaper87lol16:03
alcabrerafvollero: which endpoint?16:03
flaper87fvollero: git diff16:03
fvolleroflaper87: damme n'attimo16:04
fvolleroalcabrera: /v116:04
* flaper87 STFU, for real!16:04
flaper87fvollero: git diff ?16:04
* flaper87 hides16:04
fvolleroflaper87: http://fpaste.org/42457/38021148/16:04
fvolleroalcabrera: ^^\16:04
alcabrerainteresting, hmmm16:04
flaper87fvollero: mmh, it shouldn't be always required16:05
flaper87what we need to check is that it's not an empty string16:05
flaper87you know, people sending the header but empty16:05
fvolleroflaper87: oh, gotcha, but this cover also this case16:05
fvolleroflaper87: or at least is what the falcon doc say16:06
kgriffsoz_akan_: your script that verifies the commit hash - it will break when we have a release with not hash on the end, nicht?16:06
fvolleroflaper87: http://pydoc.net/Python/falcon/0.0.2-dev/falcon.request/16:06
flaper87mmh, dunno falcon that well, but I read that as "it should always be present"16:06
alcabrerafvollero: weird - that same curl command works for me. Seems like that required=True might be doing something weird. >.>16:07
fvolleroalcabrera: exactly, for that, i said before "I can't rely on get_header" :)16:07
* alcabrera notices that every community has their own pastebin: lpaste (haskell), paste.openstack, now fpaste (fedora)16:07
fvolleroalcabrera: yep :) what a cool idea right?16:08
kgriffsd00d, is that really from version 0.0.2?16:08
flaper87no paste, no community16:08
flaper87fvollero: btw, kgriffs is falcon dad!16:08
fvolleroalcabrera: but, in fedora you can put in pipe the command of fpaste and you've a new fpaste :)16:08
kgriffsfalcon is on 0.1.716:08
*** tedross has quit IRC16:08
fvolleroflaper87: Great!16:08
fvollerokgriffs: you can explain me why does not work required=True ? :D16:08
* kgriffs looking16:10
malinialcabrera: are you planning to retain host in this response https://wiki.openstack.org/wiki/Marconi/specs/proxy/v1#Read_a_Catalogue_Entry ?16:12
maliniimo, we don't need that16:12
alcabreramalini: It's only temporary, until I fix this bug: https://bugs.launchpad.net/marconi/+bug/123039216:14
alcabreramalini: It's a sign of proxy technical debt. D:16:15
malinialcabrera: :D thanks!16:15
alcabreramalini: np. :)16:15
* alcabrera that moment when you accidentally type tox *-r* twice after adding a dependency16:16
alcabrerarecreating the world - hold plz16:16
*** flaper87 is now known as flaper87|afk16:16
kgriffsfvollero: typo in the curl command is causing the header not to be sent16:17
kgriffsReplace16:17
fvollerokgriffs: /me facepalm16:17
kgriffs"="16:17
kgriffswith ": "16:17
kgriffs:D16:17
fvollerokgriffs: \o/ \o/16:18
kgriffsflaper87: I'm considering just using _get_Version_from_git after all so the hash will always be there16:20
kgriffsand renaming "commit" to "commit_hash"16:20
kgriffsfvollero: LOL16:20
fvollerofellas, a quick question, i think it worth to add a test for the project_id empty or not, but where i should put it ? :)16:24
fvolleromostly because base.py have 'if project_id is not None'16:25
*** yassine has quit IRC16:25
malinifvollero: functional test..we have one for missing headers16:25
fvolleromalini: Yahoooo! :D16:25
malinifvollero: or I might have removed :-#16:26
*** tedross has joined #openstack-marconi16:26
* fvollero cry16:26
malinifvollero: we have some header related tests here https://github.com/openstack/marconi/blob/master/tests/functional/wsgi/v1/test_queues.py#L3216:26
malinican you add the missing header 'X-Project-Id' here, plz?16:27
fvolleromalini: i'm looking at it :)16:27
fvolleromalini: sure thing :) i'll add at the bottom16:27
malinithank you fvollero16:27
fvolleromalini: my pleasure16:28
openstackgerritKurt Griffiths proposed a change to openstack/marconi: chore: Expose commit hash in version module  https://review.openstack.org/4745916:30
alcabrerafvollero: have you considered using httpie (https://github.com/jkbr/httpie) for tinkering with marconi? (instead of curl)16:30
alcabreraI find it's syntax much friendlier for passing anything/everything16:30
alcabrera*its16:30
fvolleroalcabrera: not yet :) i'm coming from ruby and sysadmin role, so curl was the default :)16:30
alcabrerafvollero: fair enough. :)16:34
alcabrerafvollero: I'm happy to make your marconi experience more awesome, so I'll suggest things periodically as they come to mind. :D16:35
malinialcabrera: tht sounds like a cruise ship captain :D16:35
malinialcabrera: I have another proxy qn for you..16:36
alcabreramalini: shoot16:36
malinihttps://wiki.openstack.org/wiki/Marconi/specs/proxy/v1#Listing_the_Catalogue I am thinking this one really needs pagination.16:36
malinisince it can go pretty looong with lots of queues16:36
fvolleromalini: alcabrera : i guess the fact that I changed the project-id it generate test faults :)16:36
fvolleroalcabrera: Sounds great!16:36
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap  https://review.openstack.org/4764216:38
alcabrerarebasing time~16:38
* alcabrera goes about fixing merge conflicts16:39
malinialcabrera: are you planning to add pagination soon?16:39
alcabreramalini: no - it's low-priority (esp. for the admin API)16:41
alcabreramalini: very soon, a patchset splitting the proxy into admin/public API will be available16:42
malinialcabrera:that means,  the list will always return a partial list of queues16:43
alcabreramalini: after that, the priorities are: 1) host/weight resources for updating partitions, 2) concurrent requests + merge implementation for LIST queues from proxy, 3) improve proxy unit test suite16:43
alcabrerawith 2) it handles pagination for the forwarded requests (/v1/queues) but not for admin API requests (/v1/partitions, /v1/catalogue)16:44
maliniok16:44
alcabreraas it stands, the proxy should be returning ALL the queues for a given project when LIST queues is issued.16:44
alcabrerarather than just a few. :x16:45
alcabrerahahaha16:45
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: separate config for queues and proxy  https://review.openstack.org/4833316:48
alcabrerarebasing continues - one more patch!16:48
malinialcabrera: hmmm… I am not sure if it'll really return all the queues..Tht JSON could grow really huge16:50
* fvollero blow the test suite!16:50
*** fvollero is now known as fvollero|gone16:51
*** reed has joined #openstack-marconi16:51
alcabreramalini: that's true. If there are too many queues...16:52
alcabreraI'm happy to have 2) near the top of my priority list, then!16:53
malinialcabrera: anyways, I'll make that part of my tests..if it blows up too soon, then we can move it up in priority16:53
alcabrera+1 malini16:54
*** reed has quit IRC17:02
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap  https://review.openstack.org/4764217:02
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: separate config for queues and proxy  https://review.openstack.org/4833317:03
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: split proxy API into admin and public apps  https://review.openstack.org/4849517:07
alcabrerawooooot17:07
* alcabrera completed the rebasing17:08
*** reed has joined #openstack-marconi17:08
*** cthulhup has quit IRC17:27
*** kgriffs is now known as kgriffs_afk17:48
*** kgriffs_afk is now known as kgriffs17:49
openstackgerritKurt Griffiths proposed a change to openstack/marconi: chore: Expose commit hash in version module  https://review.openstack.org/4745918:06
*** jcru has quit IRC18:07
*** jcru has joined #openstack-marconi18:13
*** jcru has quit IRC18:14
*** jcru has joined #openstack-marconi18:14
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap  https://review.openstack.org/4764218:18
*** reed_ has joined #openstack-marconi18:33
*** reed_ has quit IRC18:33
*** reed has quit IRC18:34
*** reed has joined #openstack-marconi18:36
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: separate config for queues and proxy  https://review.openstack.org/4833318:39
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: split proxy API into admin and public apps  https://review.openstack.org/4849518:56
alcabrerakgriffs, malini, oz_akan_: The fixes are in! ^^ (#48495 above)18:56
alcabreraJenkins will soon sing his song.18:56
kgriffsexcellent18:56
malinithanks alcabrera!!18:57
*** jcru has quit IRC19:03
malinialcabrera: httpretty is cool!19:13
*** gordonsim1 has quit IRC19:31
*** ayoung has quit IRC19:43
alcabreramalini: httpretty is awwweeesome. :D20:04
*** malini is now known as malini_afk20:04
alcabreramalini: makes it easy to *actually* write unit tests.20:04
*** ekarlso has joined #openstack-marconi20:13
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: split proxy API into admin and public apps  https://review.openstack.org/4849520:16
*** jcru has joined #openstack-marconi20:24
*** ayoung has joined #openstack-marconi20:31
*** alcabrera has quit IRC20:53
*** kgriffs is now known as kgriffs_afk21:03
*** kgriffs_afk is now known as kgriffs21:04
*** reed_ has joined #openstack-marconi21:06
*** kgriffs is now known as kgriffs_afk21:06
*** reed has quit IRC21:07
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: proxy: mirror structure of marconi queues + bootstrap  https://review.openstack.org/4764221:14
openstackgerritAlejandro Cabrera proposed a change to openstack/marconi: feat: separate config for queues and proxy  https://review.openstack.org/4833321:16
*** kgriffs_afk is now known as kgriffs21:17
*** tedross has quit IRC21:43
*** jcru has quit IRC21:51
*** whenry has joined #openstack-marconi22:01
*** jdprax has joined #openstack-marconi22:05
*** oz_akan_ has quit IRC22:05
*** oz_akan_ has joined #openstack-marconi22:06
*** whenry has quit IRC22:06
*** oz_akan_ has quit IRC22:10
*** mpanetta has quit IRC22:11
*** amitgandhi has quit IRC22:19
*** whenry has joined #openstack-marconi22:24
*** whenry has quit IRC22:55
openstackgerritFrancesco Vollero proposed a change to openstack/marconi: WSGI: added the control for non empty X-PROJECT-ID  https://review.openstack.org/4853722:57
openstackgerritFrancesco Vollero proposed a change to openstack/marconi: WSGI: added the control for non empty X-PROJECT-ID  https://review.openstack.org/4853723:00
*** oz_akan_ has joined #openstack-marconi23:16
*** oz_akan_ has quit IRC23:21
*** fifieldt has joined #openstack-marconi23:31
*** jdprax has quit IRC23:41
*** jdprax has joined #openstack-marconi23:41
« Wednesday, 2013-09-25 Index Friday, 2013-09-27 »

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