*** jcru has quit IRC | 00:08 | |
*** mpanetta has quit IRC | 00:24 | |
*** oz_akan_ has joined #openstack-marconi | 00:27 | |
*** amitgandhi has joined #openstack-marconi | 00:28 | |
*** fifieldt_ has joined #openstack-marconi | 00:52 | |
*** fifieldt_ has quit IRC | 00:57 | |
*** nosnos has joined #openstack-marconi | 01:08 | |
*** reed has quit IRC | 01:14 | |
*** fifieldt_ has joined #openstack-marconi | 01:16 | |
*** amitgandhi has quit IRC | 01:26 | |
*** fifieldt_ has quit IRC | 02:35 | |
*** jergerber has quit IRC | 02:43 | |
*** vkmc has quit IRC | 02:58 | |
*** fifieldt has joined #openstack-marconi | 03:51 | |
*** oz_akan_ has quit IRC | 03:52 | |
*** oz_akan_ has joined #openstack-marconi | 04:13 | |
*** oz_akan_ has quit IRC | 04:18 | |
*** oz_akan_ has joined #openstack-marconi | 05:29 | |
*** oz_akan_ has quit IRC | 05:34 | |
*** yassine has joined #openstack-marconi | 06:56 | |
*** cosss has joined #openstack-marconi | 08:01 | |
*** russellb has quit IRC | 08:35 | |
*** russellb has joined #openstack-marconi | 08:36 | |
*** flaper87|afk is now known as flaper87 | 09:32 | |
openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Migrate from nosetest to testr https://review.openstack.org/52443 | 09:40 |
---|---|---|
*** oz_akan_ has joined #openstack-marconi | 10:18 | |
*** flaper87 is now known as flaper87|afk | 10:26 | |
*** oz_akan_ has quit IRC | 10:28 | |
*** flaper87|afk is now known as flaper87 | 11:26 | |
*** nosnos has quit IRC | 11:34 | |
*** nosnos has joined #openstack-marconi | 11:35 | |
*** oz_akan_ has joined #openstack-marconi | 11:39 | |
*** nosnos has quit IRC | 11:40 | |
*** tedross has joined #openstack-marconi | 11:43 | |
*** oz_akan_ has quit IRC | 11:45 | |
*** cosss has quit IRC | 11:47 | |
*** fvollero|gone is now known as fvollero | 11:59 | |
*** alcabrera has joined #openstack-marconi | 12:04 | |
alcabrera | Good morning! | 12:04 |
*** malini_afk is now known as malini | 12:08 | |
*** vkmc has joined #openstack-marconi | 12:10 | |
*** vkmc has quit IRC | 12:10 | |
*** vkmc has joined #openstack-marconi | 12:10 | |
*** wirehead_ has quit IRC | 12:16 | |
*** wirehead_ has joined #openstack-marconi | 12:17 | |
flaper87 | alcabrera: gooood morning! | 12:34 |
flaper87 | :D | 12:34 |
alcabrera | flaper87: Hey! :) | 12:35 |
flaper87 | alcabrera: how are you ? | 12:38 |
flaper87 | alcabrera: btw, were you aware about these failing tests? https://review.openstack.org/#/c/52443/ | 12:38 |
flaper87 | Somehow, nosetest didn't run them | 12:38 |
alcabrera | hmmm... | 12:39 |
alcabrera | I'm doing better today than yesterday. Today's been pretty good, so far. :D | 12:39 |
alcabrera | How're you? | 12:40 |
* alcabrera checks out failing tests | 12:40 | |
alcabrera | oohh, the migration to testr is going to be *awesome*. | 12:40 |
alcabrera | It's one of those little things I've been looking forward to! | 12:41 |
alcabrera | I wasn't aware of any tests failing. :/ | 12:42 |
alcabrera | Err, not being run. | 12:42 |
alcabrera | Though... | 12:42 |
alcabrera | I know nose skips right past tests that have misnamed super()s. | 12:42 |
alcabrera | I ran into that problem several times while adding unit tests for things like ShardDriver, CatalogueTests, etc. | 12:42 |
*** fifieldt has quit IRC | 12:44 | |
flaper87 | alcabrera: mmh, interesting, I didn't know about it silently skipping tests with wrong super() | 12:54 |
alcabrera | Yeah, it'd be something silly like... | 12:54 |
alcabrera | class CatalogueTest: ... super(ClaimsTest, self).setUp() | 12:54 |
flaper87 | alcabrera: It hasn't been *that* straightforward, I'm working on some other improvements to that patch | 12:54 |
alcabrera | that test will just be ignored. | 12:55 |
flaper87 | alcabrera: for example, generate a db name (As for monogdb) for every test | 12:55 |
flaper87 | because they are not as isolated as I expected | 12:55 |
flaper87 | good to know, I'll flag that patch as WIP because I don't feel like it is ready to be merged | 12:56 |
flaper87 | I mean, it looks like it is | 12:56 |
flaper87 | but the mongodb part seems to be failing locally | 12:56 |
flaper87 | plus, it's a bit unstable | 12:56 |
flaper87 | (saw the random py26 / pypy failures ?) | 12:57 |
flaper87 | that's most likely related to tests being run in parallel | 12:57 |
alcabrera | those random failures... something is unstable there, and I don't know what it is. :/ | 12:57 |
alcabrera | python-2.6 and pypy seem to fail periodically. | 12:57 |
alcabrera | It's weird. | 12:57 |
flaper87 | also, we *NEED* a way to test the mongodb code in the gate. I'll ping -infra guys today or next week | 12:58 |
alcabrera | good point on the mongo tests - generating a uuid-based DB name would certainly help with isolation. | 12:58 |
alcabrera | ooohh, that'd be awesome. | 12:58 |
flaper87 | alcabrera: UUID is exaclty what I'm using ;) | 12:58 |
flaper87 | :D | 12:58 |
alcabrera | hehe, uuid fixes *all* the unit test isolation problems. :D | 13:00 |
alcabrera | Well, most of them, anyway. :P | 13:00 |
*** amitgandhi has joined #openstack-marconi | 13:01 | |
*** amitgandhi has quit IRC | 13:13 | |
*** amitgandhi has joined #openstack-marconi | 13:15 | |
*** oz_akan_ has joined #openstack-marconi | 13:16 | |
*** amitgandhi has quit IRC | 13:16 | |
*** amitgandhi has joined #openstack-marconi | 13:25 | |
*** oz_akan_ has quit IRC | 13:38 | |
*** oz_akan_ has joined #openstack-marconi | 13:39 | |
*** malini is now known as malini_afk | 13:50 | |
*** malini_afk is now known as malini | 13:51 | |
*** tedross has quit IRC | 13:52 | |
alcabrera | flaper87: reviwed all the pending client patches - +2. | 13:53 |
alcabrera | s/the/your | 13:53 |
*** jcru has joined #openstack-marconi | 13:54 | |
flaper87 | alcabrera: AWESOME!, thanks a lot! What do you think so far? | 13:55 |
*** etoews1 has joined #openstack-marconi | 13:57 | |
alcabrera | flaper87: it's looking good. I'd love to see the iterator support soon, but that's lower priority than getting an initial, fully-functional client impl. out, IMO. :) | 13:59 |
alcabrera | It also seems like adding a new transport won't be too bad, as it's mostly a matter of filling in the lower-level API, AFAIU. | 14:00 |
alcabrera | flaper87: what do you think of the current client-side wiki page? | 14:01 |
flaper87 | alcabrera: yeah, adding a new transport would be awesome, I'd like to start working on the zmq one asap but we should first brainstorm a bit more on the zmq wiki page. Adding a new transport to the client is just a matter of implementing the `Transport` class for that transport. See https://review.openstack.org/#/c/49788/3/marconiclient/transport/http.py | 14:04 |
flaper87 | that file is all it takes to add support for a new transport | 14:04 |
flaper87 | :) | 14:04 |
flaper87 | it gets a `Request` in and has to send a `Request` out | 14:04 |
flaper87 | erm a `Response` out | 14:05 |
flaper87 | alcabrera: I like that wiki page, I first looked at it before working on the client | 14:05 |
flaper87 | some things changed, though. | 14:06 |
flaper87 | I'll update it as soon as the Queue's code lands | 14:06 |
alcabrera | flaper87: awesome, thanks for helping with that page! | 14:06 |
alcabrera | and if all it takes to get a new transport in is one file, niiiicee. :D | 14:07 |
* alcabrera hasn't kept up with the big picture client-side too well | 14:07 | |
flaper87 | yeah, that's all it takes! | 14:07 |
flaper87 | transport's are completely isolated from the API | 14:07 |
alcabrera | sweet | 14:07 |
flaper87 | actually, I was thinking - I'll perhaps bring this up in our next meeting - that we should do the same server-side | 14:07 |
alcabrera | ahh, I see... | 14:08 |
alcabrera | A lower-level API | 14:08 |
flaper87 | We should have the API spec somewhere and let transport build their routes out of it | 14:08 |
alcabrera | To implement to higher level API | 14:08 |
*** jergerber has joined #openstack-marconi | 14:09 | |
flaper87 | I don't like the fact that our wsgi transport has the routes, methods etc hardcoded | 14:09 |
alcabrera | It seems like the agenda needs an update. | 14:09 |
flaper87 | that makes it necessary to replicate that for every transport | 14:09 |
alcabrera | Which also reminds me that we should decide on that agenda today as soon as kgriffs_afk makes his way in. | 14:09 |
alcabrera | To send out the meeting reminder email on the ML | 14:09 |
flaper87 | alcabrera: nod! | 14:09 |
flaper87 | let me help you with that | 14:10 |
* alcabrera was reading over https://wiki.openstack.org/wiki/Meetings/Marconi#Agenda | 14:10 | |
flaper87 | alcabrera: added some items, anything missing? | 14:11 |
flaper87 | most liekly, that will take the whole hour | 14:12 |
flaper87 | likely* | 14:12 |
alcabrera | looks good to me, flaper87 | 14:12 |
alcabrera | I agree that we'll likely consume to hour with those topics, probably even before we get to versioning. :P | 14:13 |
alcabrera | s/to/the | 14:13 |
*** flaper87 changes topic to "OpenStack Queuing and Notification Service || Smile :D || Meetings every Monday 16:00 UTC || Wiki: https://wiki.openstack.org/wiki/Marconi || Paste: http://paste.openstack.org/ || Smile: :D" | 14:14 | |
flaper87 | alcabrera: I think versioning could be easily covered by a good API spec | 14:14 |
*** jergerber has quit IRC | 14:14 | |
alcabrera | Our topic now has twice as many smiles and a meeting header - nice! | 14:15 |
flaper87 | alcabrera: yeaaaah!! There's no concurrence nor parallelism without smiling | 14:15 |
flaper87 | btw, I was thinking something like: https://review.openstack.org/#/c/49788/3/marconiclient/transport/api.py | 14:15 |
flaper87 | and https://review.openstack.org/#/c/50638/7/marconiclient/queues/v1/api.py | 14:16 |
flaper87 | for the API spec | 14:16 |
flaper87 | the later inherits from the former | 14:16 |
*** amitgandhi has quit IRC | 14:16 | |
flaper87 | basically, jsonschema based API definition | 14:16 |
*** amitgandhi has joined #openstack-marconi | 14:17 | |
flaper87 | readable enough, easy to use for API discovery and easier to update as well | 14:17 |
alcabrera | seems reasonable. There's very little I haven't been able to express given a thorough jsonschema. Have you run into any trouble implementing the client using your current approach? | 14:17 |
* flaper87 still needs to learn how to pronounce 'through' | 14:18 | |
flaper87 | thorough* | 14:18 |
flaper87 | alcabrera: I haven't yet, It allows me to add custom fields - HTTP methods, for example - optional arguments, required ones, etc | 14:19 |
alcabrera | thorough ~-> tho - row | 14:19 |
flaper87 | alcabrera: uuu, that makes sense. Thanks :D | 14:19 |
flaper87 | alcabrera: I guess the missing part there are 'required headers' | 14:19 |
alcabrera | flaper87: yup. If that could be filled in somehow, or represented, that'd be awesome. | 14:20 |
flaper87 | which, I guess, we can workaround with some custom `headers` field in the schema | 14:20 |
alcabrera | The closest thing I have is assert field in (names) | 14:20 |
alcabrera | flaper87: what do you think about merging this in? https://review.openstack.org/#/c/51083/ | 14:26 |
alcabrera | pip install in README.rst vs. python setup.py develop | 14:26 |
* alcabrera is reviewing and assessing | 14:26 | |
* flaper87 looking | 14:30 | |
flaper87 | btw, I wanted to point this out to you guys. https://blueprints.launchpad.net/oslo/+spec/remove-import-side-effects | 14:31 |
alcabrera | flaper87: I added one more item to the meeting agenda. It's coupled with the sharding discussion, "the future of the proxy". | 14:31 |
flaper87 | that's basically why I didn't start working on the remove globals right away | 14:31 |
alcabrera | flaper87: ooohh, I see the word "side effects" and I know it's going to be good. | 14:31 |
flaper87 | http://summit.openstack.org/cfp/details/125 | 14:31 |
flaper87 | :) | 14:31 |
flaper87 | alcabrera: +1 for the future of the proxy | 14:32 |
flaper87 | mmh, I'm a bit unsure about the pip install -e . thing, the last time I tried, it didn't get the version right | 14:33 |
flaper87 | I didn't had the time to look into it | 14:33 |
flaper87 | let me try again | 14:33 |
flaper87 | I do agree that's the best way to do it, though | 14:33 |
alcabrera | I'm giving it a try, as well, using a venv. | 14:34 |
flaper87 | alcabrera: does this happen to you? http://paste.openstack.org/show/48694/ | 14:35 |
alcabrera | flaper87: VersionInfo(marconi:2013.2) # from marconi.version.version_info | 14:35 |
flaper87 | damn, then my venv is borked | 14:35 |
alcabrera | "marconi 2013.2.a466.gd8c26e5 is already the active version in easy-install.pth" | 14:35 |
alcabrera | That's when I try to install again | 14:36 |
flaper87 | trying in a fresh venv | 14:36 |
flaper87 | alcabrera: how did you create your venv ? | 14:36 |
flaper87 | --no-site-packages ? | 14:36 |
alcabrera | hmmm | 14:37 |
alcabrera | virtualenv-2.7 .marconi | 14:37 |
flaper87 | mmh, did you see the Successfully installed UNKNOWN thing? | 14:37 |
alcabrera | not this time, though I've seen it before. | 14:38 |
alcabrera | trying again | 14:38 |
alcabrera | nope, still looks good. | 14:39 |
alcabrera | virtualenv-2,7 --version => 1.10.1 | 14:40 |
alcabrera | pip --version => 1.4.1 | 14:40 |
flaper87 | mmh, cleaned everything and run it again | 14:42 |
flaper87 | it just worked | 14:42 |
flaper87 | :/ not sure what happened | 14:42 |
flaper87 | approved | 14:42 |
alcabrera | sweet | 14:43 |
alcabrera | I'm almost done triaging that review queue - keeping things moving. :) | 14:43 |
alcabrera | oh, btw... https://mail.python.org/pipermail/python-dev/2013-October/129678.html (Tulip, a.k.a, asyncio, a.k.a., PEP 3156 was merged) | 14:43 |
alcabrera | ^^ | 14:43 |
flaper87 | YEAHHHHH!!! | 14:44 |
zyuan | Ran 469 tests in 65.535s | 14:44 |
alcabrera | Python 3.4 is going to be amazing. | 14:44 |
openstackgerrit | A change was merged to openstack/marconi: doc: Recommend installing marconi via pip https://review.openstack.org/51083 | 14:45 |
alcabrera | Time to do some rebasing~ | 14:52 |
*** tedross has joined #openstack-marconi | 15:08 | |
*** tedross has quit IRC | 15:08 | |
*** tedross has joined #openstack-marconi | 15:08 | |
*** whenry has joined #openstack-marconi | 15:08 | |
flaper87 | [OT] MANNNN: http://s.f87.me/19XJTs2 | 15:12 |
*** kgriffs_afk is now known as kgriffs | 15:25 | |
alcabrera | flaper87: I feel like I've listened to some stuff by Kamelot, probably via some custom dance game charts. :P | 15:25 |
alcabrera | kgriffs: o/ | 15:25 |
kgriffs | alcabrera: hey man, good to see you | 15:28 |
alcabrera | things are better today. :) | 15:28 |
kgriffs | no way, tulip will make it in after all?! | 15:29 |
alcabrera | yup | 15:30 |
alcabrera | import asyncio | 15:30 |
alcabrera | expected to drop for python 3.4a04 | 15:30 |
flaper87 | kgriffs: it will :D | 15:30 |
flaper87 | alcabrera: man, I really like their music | 15:30 |
flaper87 | alcabrera: https://play.spotify.com/track/6sxqSN9YxAHzMzJ8u835J5 | 15:31 |
alcabrera | flaper87: you going to make it to that concert? ;) | 15:31 |
flaper87 | alcabrera: probably, it's not near me, I've some friends in that area, I just need to find a coach I can crash into | 15:32 |
alcabrera | kgriffs: I've done a little triaging of the review queue this morning, and started rebasing the admin API feature line wrt the global config patch. | 15:33 |
flaper87 | alcabrera: +1 | 15:33 |
flaper87 | kgriffs: it'd be really cool if we could get some other client patches merged :) | 15:33 |
flaper87 | I mean, reviewed | 15:33 |
flaper87 | ghghghg | 15:33 |
alcabrera | lol | 15:34 |
kgriffs | yes, that would be nice. | 15:34 |
alcabrera | flaper87, kgriffs: I've updated the https://etherpad.openstack.org/p/sharding-merge-strategy with a "Merged" section, as well as a "Other Patches" section. Our review queue is quite busy lately~ | 15:35 |
alcabrera | oh yes! | 15:35 |
alcabrera | Just remembered... | 15:35 |
kgriffs | alcabrera: https://review.openstack.org/#/c/50437/ | 15:35 |
kgriffs | :D | 15:35 |
kgriffs | :D | 15:35 |
alcabrera | kgriffs: https://wiki.openstack.org/wiki/Meetings/Marconi#Agenda (agenda's been updated, with help from flaper87). We should probably send out an email today. | 15:35 |
kgriffs | :D | 15:35 |
alcabrera | lol | 15:35 |
alcabrera | I'll check that out. :D | 15:35 |
alcabrera | That one looks like a *great* candidate for merging. | 15:36 |
alcabrera | There's a very high chance I'll hit that "Approve" button in the next 5 minutes. | 15:36 |
alcabrera | Sanity checking! | 15:36 |
alcabrera | Here it comes... | 15:38 |
alcabrera | kgriffs: approved! | 15:38 |
kgriffs | FANTASTIC | 15:39 |
openstackgerrit | A change was merged to openstack/marconi: feat: Storage sharding foundation https://review.openstack.org/50437 | 15:40 |
alcabrera | kgriffs, flaper87: if possible, let's make it through the queues admin API side soon. We need thecatalogue storage driver from there to continue the sharding core work. | 15:40 |
alcabrera | +, it helps to be able to register shards to actually test this thing. :P | 15:41 |
alcabrera | [OT] flaper87: I ended up listening to that Kamelot list. Good stuff! | 15:41 |
flaper87 | alcabrera: man you're right. I was planning to go through those review this morning but got caught but tons of internal stuff. I'll review it in a bit | 15:42 |
flaper87 | alcabrera: thanks for reminding me! | 15:42 |
alcabrera | np. :) | 15:42 |
flaper87 | alcabrera: I really really like their music. I'm glad you liked it! :) | 15:43 |
alcabrera | we're at 15 patches in the queue at this moment, 7 related to sharding. | 15:43 |
alcabrera | I'm rebasing the first patch in the line, atm. | 15:44 |
alcabrera | admin/public split for queues | 15:44 |
alcabrera | mighty rebasing, it is. | 15:44 |
kgriffs | alcabrera: regarding https://review.openstack.org/#/c/50721/7/marconi/queues/storage/mongodb/driver.py | 15:49 |
kgriffs | I think it may actually be a good idea to split into an admin and a queues driver | 15:49 |
kgriffs | (as you suggested) | 15:50 |
alcabrera | hmmm | 15:50 |
*** reed has joined #openstack-marconi | 15:50 | |
kgriffs | makes more sense in my mind | 15:50 |
alcabrera | alright | 15:50 |
kgriffs | since they will have different users, different DBs | 15:50 |
alcabrera | even different applications launched | 15:51 |
alcabrera | transport.wsgi.admin.app | 15:51 |
kgriffs | yep | 15:51 |
alcabrera | fair enough - for now, I'd like to keep a same-file implementation. | 15:51 |
alcabrera | With intent to figure out how best to structure admin/public storage drivers later. | 15:51 |
alcabrera | the rationale being - the refactoring will get pretty messy considering we have two storage implementations at the moment, and I worry that'll obfuscate the intent of the current patches. | 15:53 |
kgriffs | https://bugs.launchpad.net/marconi/+bug/1241686 | 15:56 |
kgriffs | just so we don't forget to come back to it later | 15:57 |
alcabrera | kgriffs: awesome! | 15:57 |
*** pasquier-s has quit IRC | 16:00 | |
alcabrera | flaper87: Moonlight's my favorite so far. | 16:05 |
flaper87 | alcabrera: I like that one a lot. I also like https://play.spotify.com/track/4oU1fRwM3AYGL5oQCSed5M very very much | 16:12 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: split queues API into public/admin https://review.openstack.org/50701 | 16:25 |
alcabrera | flaper87, kgriffs: queues admin API split now rebased and ready for review. config_split really did a number on things! ;) | 16:26 |
flaper87 | alcabrera: I bet it did :P | 16:29 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: add shard management resource https://review.openstack.org/50702 | 16:30 |
*** kgriffs is now known as kgriffs_afk | 16:36 | |
*** yassine has quit IRC | 16:37 | |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: shards storage controller interface https://review.openstack.org/50721 | 16:44 |
alcabrera | that's three rebased - winning! | 16:44 |
alcabrera | two more to go in the admin API feature line | 16:44 |
alcabrera | I'll brb. I'm going to run some groceries and continue working from home. | 16:44 |
*** alcabrera has quit IRC | 16:46 | |
*** etoews1 has quit IRC | 16:50 | |
*** etoews1 has joined #openstack-marconi | 16:51 | |
openstackgerrit | Dirk Mueller proposed a change to openstack/marconi: Fix Pep8 indentation errors missed by Pep8 1.4.5 https://review.openstack.org/52672 | 17:03 |
*** kgriffs_afk is now known as kgriffs | 17:28 | |
flaper87 | kgriffs: very quick review: https://review.openstack.org/#/c/52672/ | 17:28 |
flaper87 | I need that one to push an update to the testr one | 17:29 |
flaper87 | Heading off now! | 17:29 |
flaper87 | if you guys can go through client patches, it'd be really cool | 17:29 |
*** flaper87 is now known as flaper87|afk | 17:30 | |
*** alcabrera has joined #openstack-marconi | 17:39 | |
openstackgerrit | A change was merged to openstack/marconi: Fix Pep8 indentation errors missed by Pep8 1.4.5 https://review.openstack.org/52672 | 17:52 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: split queues API into public/admin https://review.openstack.org/50701 | 18:27 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: add shard management resource https://review.openstack.org/50702 | 18:28 |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: shards storage controller interface https://review.openstack.org/50721 | 18:28 |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 18:46 |
*** tedross has quit IRC | 18:49 | |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 18:56 |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 19:01 |
zyuan | the jenkins machine with py26 seems to have something different | 19:02 |
alcabrera | I'm out for the day, guys. | 19:07 |
zyuan | see you! | 19:07 |
alcabrera | zyuan: thanks for getting started on the listing patch. :) | 19:07 |
zyuan | just able to use debugger inside .list()... | 19:08 |
alcabrera | kgriffs: let me know if anything stands out to you while reviewing the admin API feature line. :D | 19:08 |
alcabrera | kgriffs: I've rebased about half of it at this point. I'll finish rebasing before I call it a day. I'm still tracking our efforts here: https://etherpad.openstack.org/p/sharding-merge-strategy | 19:09 |
alcabrera | anyway... | 19:09 |
alcabrera | have a good weekend. :D | 19:09 |
*** alcabrera has quit IRC | 19:09 | |
*** reed has quit IRC | 19:30 | |
openstackgerrit | Alejandro Cabrera proposed a change to openstack/marconi: feat: shards mongodb driver + tests https://review.openstack.org/50815 | 19:43 |
etoews1 | when posting message to a queue and you get a partial response back, is it guaranteed that the n ids returned in the response coincide with the first n messages sent in the request? | 19:47 |
zyuan | kgriffs: ^^ i guess so, but i'm not sure | 19:53 |
*** vkmc has quit IRC | 20:02 | |
kgriffs | etoews1: yes, it is | 20:03 |
etoews1 | kgriffs zyuan: thx. | 20:03 |
kgriffs | but, to be honest, you won't ever get partial results | 20:03 |
kgriffs | it used to be possible, but I recently discovered that a change made to fix a different issue actually had the nice side-effect of removing the possibility for partial updates. | 20:04 |
kgriffs | that being said, a 3rd-party driver might make use of it since it is in the v1 API | 20:05 |
kgriffs | We are considering dropping the partial result option in a future API version | 20:05 |
kgriffs | anyway, that's the story. :p | 20:05 |
*** whenry has quit IRC | 20:26 | |
*** whenry has joined #openstack-marconi | 20:29 | |
etoews1 | kgriffs: is the Client-Id header only required for the messages and claims APIs? | 20:39 |
kgriffs | technically, it is only required to be there when posting and listing messages | 20:42 |
kgriffs | it isn't currently used with claims | 20:42 |
kgriffs | but | 20:42 |
kgriffs | it may be some day | 20:42 |
kgriffs | i mean, the spec is admittedly fuzzy there | 20:43 |
kgriffs | we have been telling people to just always send it | 20:44 |
kgriffs | if clients always send client-id then we can compile some interesting stats that could be added to my-queue/stats | 20:45 |
kgriffs | some other uses as well | 20:45 |
kgriffs | Let me add this to my API feedback wiki page. We really should define this better. | 20:46 |
russell_h | not that it makes much difference, but it'd be sort of nice if client-ID didn't have to be a UUID | 20:47 |
russell_h | there's a thing I call the "curl barrier" | 20:47 |
russell_h | basically things that make an API painful to curl, but don't affect real clients | 20:48 |
*** etoews1 has quit IRC | 20:48 | |
russell_h | having to generate a UUID raises the curl barrier | 20:48 |
*** etoews1 has joined #openstack-marconi | 20:52 | |
kgriffs | what would be some alternatives? | 20:53 |
russell_h | just an opaque string | 20:54 |
kgriffs | we used to let people do any arbitrary string but it seemed to cause confusion. | 20:55 |
russell_h | ah | 20:55 |
russell_h | the error message you get if it isn't a UUID could be better too | 20:55 |
russell_h | I forget what it was, but it was confusing | 20:55 |
kgriffs | oh, that could be the case | 20:55 |
kgriffs | I have it on my todo list to scrub all the error messages. there is a lot of improvement to be done there. | 20:55 |
russell_h | kgriffs: did you end up using repose? | 20:56 |
kgriffs | ayway, i am open to changing back in the next API version to allowing opaque string, but I'd like to find out what other things beside a UUID people want to use. | 20:56 |
kgriffs | russell_h: Nope. | 20:56 |
russell_h | good | 20:56 |
kgriffs | they had some blocking bugs | 20:56 |
kgriffs | and performance isn't there (yet?) | 20:57 |
russell_h | heh | 20:57 |
russell_h | the mailgun guys are working on a thing | 20:57 |
kgriffs | so, we are using some WSGI middleware for now | 20:57 |
russell_h | https://github.com/mailgun/vulcan | 20:57 |
russell_h | needs some love | 20:57 |
kgriffs | yeah, I heard someone mention that earlier today. I need to check it out. | 20:57 |
russell_h | but they tried to talk to the repose people about it, and came away.. not too happy | 20:57 |
russell_h | so now they want to turn it into a replacement | 20:58 |
russell_h | https://github.com/russellhaering/txmarconi | 20:58 |
russell_h | this, too, needs some love | 20:58 |
russell_h | but is kind of working | 20:59 |
kgriffs | interesting | 20:59 |
kgriffs | I'll take a look at that as well | 20:59 |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 21:18 |
*** vkmc has joined #openstack-marconi | 21:29 | |
openstackgerrit | A change was merged to openstack/marconi: fix(bootstrap): Logging CLI options don't work https://review.openstack.org/52216 | 21:36 |
*** oz_akan_ has quit IRC | 22:10 | |
*** amitgandhi has quit IRC | 22:10 | |
*** jcru has quit IRC | 22:18 | |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 22:20 |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 22:24 |
*** kgriffs is now known as kgriffs_afk | 22:35 | |
*** etoews1 has quit IRC | 22:40 | |
openstackgerrit | Zhihao Yuan proposed a change to openstack/marconi: WIP: feat(shard): queue listing https://review.openstack.org/52692 | 22:45 |
*** openstackgerrit has quit IRC | 22:46 | |
*** openstackgerrit has joined #openstack-marconi | 22:46 | |
*** fifieldt has joined #openstack-marconi | 23:02 | |
*** ametts has quit IRC | 23:03 | |
*** etoews1 has joined #openstack-marconi | 23:51 | |
*** reed has joined #openstack-marconi | 23:53 | |
*** etoews1 has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!