| *** lakspace has quit IRC | 00:10 | |
| *** lakspace has joined #openstack-marconi | 00:12 | |
| *** anamalagon has joined #openstack-marconi | 00:19 | |
| *** rwsu has quit IRC | 00:27 | |
| *** rwsu has joined #openstack-marconi | 00:30 | |
| *** anamalagon has quit IRC | 00:38 | |
| *** openstackgerrit has quit IRC | 00:44 | |
| *** openstackgerrit has joined #openstack-marconi | 00:44 | |
| *** flwang has quit IRC | 00:56 | |
| *** cpallares has quit IRC | 01:02 | |
| *** rossk has quit IRC | 01:08 | |
| *** rwsu has quit IRC | 01:17 | |
| *** nosnos has joined #openstack-marconi | 01:29 | |
| *** ayoung_afk is now known as ayoung | 01:30 | |
| *** malini_afk is now known as malini | 02:06 | |
| *** flwang has joined #openstack-marconi | 02:47 | |
| *** malini is now known as malini_afk | 02:51 | |
| *** nidhi has joined #openstack-marconi | 03:14 | |
| *** nidhi has joined #openstack-marconi | 03:24 | |
| *** nidhi has quit IRC | 03:32 | |
| *** chandan_kumar has joined #openstack-marconi | 03:37 | |
| openstackgerrit | Angus Salkeld proposed a change to openstack/python-marconiclient: Add support for claims https://review.openstack.org/72614 | 04:08 |
|---|---|---|
| *** chandan_kumar has quit IRC | 04:53 | |
| *** nidhi has joined #openstack-marconi | 05:05 | |
| *** nidhi has quit IRC | 05:35 | |
| *** flwang has quit IRC | 06:58 | |
| *** saju_m has joined #openstack-marconi | 07:12 | |
| *** ayoung is now known as ayoung-ZZzZzzZz_ | 07:12 | |
| *** flwang has joined #openstack-marconi | 07:17 | |
| *** nosnos has quit IRC | 07:31 | |
| *** nosnos_ has joined #openstack-marconi | 07:31 | |
| openstackgerrit | Fengqian Gao proposed a change to openstack/marconi: Sort requirement files in alphabetical order https://review.openstack.org/77078 | 07:44 |
| *** flaper87|afk is now known as flaper87 | 07:58 | |
| flwang | flaper87: morning | 08:02 |
| *** nosnos_ has quit IRC | 08:04 | |
| flaper87 | flwang: yo yo, how are you doing? | 08:04 |
| flwang | flaper87: I guess you're drinking coffee now, haha | 08:04 |
| *** nosnos has joined #openstack-marconi | 08:04 | |
| flwang | flaper87: some questions are coming... | 08:05 |
| flaper87 | flwang: TONS of coffee | 08:06 |
| flaper87 | flwang: shoot | 08:06 |
| flwang | flaper87: i'm working on the /health bp, adding some test cases | 08:08 |
| flwang | flaper87: 1. you know, sqlite is deprecating, and the default test db backend is sqlite, and my test case is trying to get active shards number, so any idea about this? | 08:09 |
| flwang | 2. any easy way to enable the mongodb as unit test backend instead of sqlite? | 08:10 |
| flwang | 3. seems we have not completed the functional test against v1.1, did i miss anything? | 08:11 |
| openstackgerrit | Flavio Percoco proposed a change to openstack/marconi: Deprecate sqlite in favor of sqlalchemy https://review.openstack.org/76484 | 08:16 |
| *** saju_m has quit IRC | 08:26 | |
| flaper87 | flwang: that patch ^ was approed yday but failed to merge | 08:27 |
| flaper87 | I'll ninja approve it now | 08:27 |
| flaper87 | with that you'll be able to switch to sqlalchemy | 08:27 |
| flaper87 | flwang: re mongodb, you can use a config file that enables mongodb | 08:27 |
| flwang | #1 cool | 08:30 |
| flwang | #2 wsgi_mongodb.conf? | 08:32 |
| *** flaper87|afk has joined #openstack-marconi | 08:35 | |
| *** flaper87|afk has quit IRC | 08:50 | |
| *** flwang has quit IRC | 08:54 | |
| *** haomaiwang has quit IRC | 08:57 | |
| *** haomaiwang has joined #openstack-marconi | 09:00 | |
| *** flwang has joined #openstack-marconi | 09:00 | |
| *** whenry has quit IRC | 09:05 | |
| *** sballe has quit IRC | 09:08 | |
| *** sballe has joined #openstack-marconi | 09:08 | |
| openstackgerrit | A change was merged to openstack/marconi: Deprecate sqlite in favor of sqlalchemy https://review.openstack.org/76484 | 09:10 |
| *** malini_afk is now known as malini | 09:11 | |
| flaper87 | flwang: ^^^^^ | 09:12 |
| flwang | coooooooooooooooool | 09:13 |
| *** sballe has quit IRC | 09:17 | |
| *** flwang has quit IRC | 09:24 | |
| seiflotfy | flaper87 how do i run marconi tests | 09:30 |
| seiflotfy | ? | 09:30 |
| flaper87 | seiflotfy: python setup.py testr | 09:32 |
| flaper87 | seiflotfy: the r at the end is not a typo | 09:32 |
| flaper87 | ;) | 09:32 |
| *** ykaplan has joined #openstack-marconi | 09:36 | |
| *** saju_m has joined #openstack-marconi | 09:41 | |
| seiflotfy | flaper87 python setup.py testr | 09:47 |
| seiflotfy | usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...] | 09:47 |
| seiflotfy | or: setup.py --help [cmd1 cmd2 ...] | 09:47 |
| seiflotfy | or: setup.py --help-commands | 09:47 |
| seiflotfy | or: setup.py cmd --help | 09:47 |
| seiflotfy | error: invalid command 'testr' | 09:47 |
| flaper87 | seiflotfy: did you installed test-requirements ? | 09:49 |
| flaper87 | seiflotfy: $ pip install -r test-requirements.txt | 09:49 |
| seiflotfy | yeah just noticed i forgot to do that | 09:49 |
| seiflotfy | im on the wrong computer (on a mac) | 09:49 |
| seiflotfy | ok works | 09:50 |
| seiflotfy | sorry | 09:50 |
| seiflotfy | false alarm | 09:50 |
| seiflotfy | ok fixing time | 09:50 |
| flaper87 | mmh, it should work on mac too | 10:08 |
| flaper87 | that's how cool marconi is | 10:08 |
| seiflotfy | flaper87 MismatchError: 204 != 201 | 10:27 |
| seiflotfy | i keep getting this | 10:27 |
| *** flwang has joined #openstack-marconi | 10:47 | |
| *** jamieh has joined #openstack-marconi | 10:49 | |
| flwang | seiflotfy: ping | 10:49 |
| *** haomaiwang has quit IRC | 11:08 | |
| *** haomaiwang has joined #openstack-marconi | 11:09 | |
| *** haomaiwang has quit IRC | 11:14 | |
| seiflotfy | yo | 11:14 |
| seiflotfy | flwang hacking | 11:15 |
| *** ykaplan has quit IRC | 11:25 | |
| seiflotfy | flwang flaper87 oen of the tests return a 204 instead of a 201 | 11:32 |
| seiflotfy | flaper87 how bad is this then | 11:33 |
| flaper87 | seiflotfy: what test is failing? | 11:46 |
| seiflotfy | ---- | 11:47 |
| seiflotfy | ====================================================================== | 11:47 |
| seiflotfy | FAIL: tests.functional.wsgi.v1.test_queues.TestInsertQueue.test_insert_queue_header_asterisk | 11:47 |
| seiflotfy | tags: worker-1 | 11:47 |
| seiflotfy | ---------------------------------------------------------------------- | 11:47 |
| seiflotfy | Empty attachments: | 11:47 |
| seiflotfy | pythonlogging:'marconi' | 11:47 |
| seiflotfy | stderr | 11:47 |
| seiflotfy | ---- | 11:47 |
| *** balajiiyer has joined #openstack-marconi | 11:50 | |
| seiflotfy | flaper87 generally the test suite seems to be broken | 11:54 |
| seiflotfy | for me | 11:54 |
| flaper87 | is that the only test failing ? Try running it with concurrency 1 | 11:55 |
| flaper87 | seiflotfy: python setup.py testr --slowest --testr-args='--concurrency 1' | 11:55 |
| flaper87 | seiflotfy: and pls, use http://paste.openstack.org/ for logs :) | 11:55 |
| seiflotfy | http://paste.openstack.org/show/72120/ | 11:56 |
| seiflotfy | sorry was on it | 11:56 |
| seiflotfy | senior flaper87 | 11:57 |
| seiflotfy | http://paste.openstack.org/show/72121/ | 11:57 |
| seiflotfy | here is a new one | 11:57 |
| flaper87 | seiflotfy: you're using latest and you cleaned all the .pyc files right ? | 12:01 |
| flaper87 | the test pass for me | 12:01 |
| seiflotfy | flaper87 r u testing using mongodb | 12:04 |
| seiflotfy | ? | 12:05 |
| flaper87 | seiflotfy: nope | 12:05 |
| seiflotfy | what r u using | 12:05 |
| seiflotfy | ? | 12:05 |
| seiflotfy | sqlite? | 12:05 |
| flaper87 | seiflotfy: yes, which is the default | 12:05 |
| seiflotfy | yeah seems to break with mongodb | 12:05 |
| flaper87 | does that error come up when testing with mongo? | 12:06 |
| seiflotfy | yep | 12:06 |
| flaper87 | mmh | 12:06 |
| flaper87 | lemme try | 12:06 |
| seiflotfy | how does the conf look like for sqlite | 12:06 |
| seiflotfy | ? | 12:06 |
| flaper87 | FWIW we're working on gating with mongo | 12:06 |
| flaper87 | no conf needed, it's the default | 12:06 |
| *** ekarlso has quit IRC | 12:09 | |
| seiflotfy | yeah using the default conf results in big issues here when running the tests | 12:13 |
| seiflotfy | seems like it cant connect to 'sqlite://memory' | 12:15 |
| flaper87 | seiflotfy: it's 'sqlite://:memory:' | 12:16 |
| seiflotfy | ah yes | 12:16 |
| seiflotfy | do i need to set that up in the conf | 12:16 |
| seiflotfy | ? | 12:16 |
| flaper87 | you shouldn't need that, if you've the latest code, that should be the default value | 12:17 |
| seiflotfy | from git | 12:18 |
| *** vkmc has joined #openstack-marconi | 12:18 | |
| *** vkmc has quit IRC | 12:18 | |
| *** vkmc has joined #openstack-marconi | 12:18 | |
| seiflotfy | it says | 12:18 |
| seiflotfy | Storage driver module (e.g., mongodb, sqlite) | 12:18 |
| seiflotfy | storage = mongodb | 12:18 |
| *** balajiiyer has quit IRC | 12:21 | |
| seiflotfy | --- | 12:22 |
| seiflotfy | flaper87 http://paste.openstack.org/show/72139/ | 12:23 |
| seiflotfy | here u go | 12:23 |
| *** malini is now known as malini_afk | 12:28 | |
| flaper87 | seiflotfy: ooooooooooooopssssssssssssssssssssssssssssssssssssssssssssss | 12:39 |
| flaper87 | seiflotfy: looks like that didn't get replaced in the last patch | 12:39 |
| seiflotfy | can i do that? | 12:39 |
| seiflotfy | i can fix it i guess | 12:39 |
| flaper87 | seiflotfy: sure you can | 12:39 |
| flaper87 | and thanks a lot | 12:39 |
| flaper87 | seiflotfy: just replace it here: tests/unit/common/storage/test_utils.py | 12:40 |
| flaper87 | I mean, s/memory/:memory:/ | 12:40 |
| *** saju_m has quit IRC | 12:40 | |
| seiflotfy | yep | 12:40 |
| seiflotfy | on it | 12:40 |
| seiflotfy | should i file a bug or just create a gerrit pull request | 12:40 |
| seiflotfy | ? | 12:40 |
| *** mwagner_lap has quit IRC | 12:40 | |
| flaper87 | seiflotfy: you should but not need this time | 12:41 |
| seiflotfy | yeah i am coming from cinder to dedicate my time here | 12:42 |
| seiflotfy | :D | 12:42 |
| seiflotfy | love cinder but this is badass :D | 12:42 |
| flaper87 | seiflotfy: AWESOME! :D | 12:42 |
| seiflotfy | love the specs and everything you guys did | 12:42 |
| flaper87 | I'm glad you like it | 12:42 |
| *** ykaplan has joined #openstack-marconi | 12:42 | |
| *** saju_m has joined #openstack-marconi | 12:51 | |
| seiflotfy | how do i test a specific test case | 12:53 |
| seiflotfy | eg | 12:53 |
| seiflotfy | tests.unit.common.storage.test_utils.TestUtils.test_can_connect_suceeds_if_good_uri_sqlite | 12:53 |
| flaper87 | seiflotfy: you can either use nosetests (you'll need to install nose) or just pass the test name to the testr command | 12:54 |
| flaper87 | seiflotfy: python setup.py testr --slowest --testr-args='--concurrency 1 test.unit.common.....' | 12:54 |
| flaper87 | if you want do use pdb, then nosetests -s is what you need | 12:54 |
| flaper87 | (or whatever testing tool you prefer) | 12:54 |
| seiflotfy | yeah seems like its stil lfailing | 12:57 |
| seiflotfy | this is interesting | 12:57 |
| flaper87 | seiflotfy: git diff ? | 12:57 |
| seiflotfy | http://paste.openstack.org/show/72150/ | 13:00 |
| flaper87 | seiflotfy: and that test still fails ? | 13:00 |
| flaper87 | mmh, | 13:01 |
| flaper87 | that looks correct | 13:01 |
| seiflotfy | File "tests/unit/common/storage/test_utils.py", line 30, in test_can_connect_suceeds_if_good_uri_sqlite | 13:01 |
| seiflotfy | self.assertTrue(utils.can_connect('sqlite://:memory:')) | 13:01 |
| seiflotfy | File "/usr/local/Cellar/python/2.7.6/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/case.py", line 424, in assertTrue | 13:01 |
| seiflotfy | raise self.failureException(msg) | 13:01 |
| flaper87 | don't.... | 13:01 |
| flaper87 | :P | 13:01 |
| seiflotfy | sorry | 13:01 |
| seiflotfy | lol | 13:01 |
| seiflotfy | i need my linux again | 13:01 |
| seiflotfy | i used to pastebin with one command | 13:01 |
| flaper87 | seiflotfy: linux forever :D | 13:01 |
| *** saju_m has quit IRC | 13:04 | |
| seiflotfy | http://paste.openstack.org/show/72151/ flaper87 | 13:07 |
| *** flaper87 has quit IRC | 13:17 | |
| *** flaper87 has joined #openstack-marconi | 13:17 | |
| *** barjavel.freenode.net sets mode: +o flaper87 | 13:17 | |
| *** ykaplan has quit IRC | 13:18 | |
| *** ykaplan has joined #openstack-marconi | 13:18 | |
| *** ekarlso has joined #openstack-marconi | 13:23 | |
| openstackgerrit | Fei Long Wang proposed a change to openstack/marconi: feat(v1.1): Implement new /health endpoint for v1.1 https://review.openstack.org/76555 | 13:24 |
| flwang | flaper87: ping | 13:26 |
| flaper87 | flwang: reviewing | 13:26 |
| flaper87 | :D | 13:26 |
| flwang | flaper87: haha, great, thanks, just want to know if the overall shape is what you want to see :) | 13:26 |
| flaper87 | flwang: yes, it's what I had in mind :D | 13:27 |
| flwang | flaper87: i'm trying to add more test cases | 13:27 |
| flwang | flaper87: any suggestion? | 13:27 |
| flaper87 | flwang: I've some comments, comming | 13:30 |
| flwang | flaper87: cool | 13:30 |
| flwang | flaper87: thanks, guru | 13:30 |
| *** sballe has joined #openstack-marconi | 13:33 | |
| *** tedross has joined #openstack-marconi | 13:44 | |
| flwang | flaper87: ping | 13:48 |
| flwang | flaper87: seems there is no check if the [drivers] storage = sqlite NOT compatiable with [drivers:storage:sqlalchemy], did i miss something? | 13:49 |
| *** nosnos has quit IRC | 13:56 | |
| *** ykaplan has quit IRC | 14:00 | |
| *** balajiiyer has joined #openstack-marconi | 14:02 | |
| *** alcabrera has joined #openstack-marconi | 14:04 | |
| alcabrera | Good morning, all. :) | 14:05 |
| seiflotfy | flwang same thing here | 14:06 |
| flwang | alcabrera: hey, morning | 14:06 |
| seiflotfy | flwang i am trying to deal with that issue too | 14:07 |
| flwang | seiflotfy: what issue? | 14:07 |
| seiflotfy | using the default conf with sqlite as a backend does not work | 14:07 |
| flwang | seiflotfy: really? not my case | 14:09 |
| seiflotfy | ok i cant seem to get it running at all | 14:09 |
| seiflotfy | can you please shoe me your conf | 14:09 |
| alcabrera | hmm | 14:09 |
| seiflotfy | guys please | 14:10 |
| alcabrera | seiflotfy: are you running off of the latest master? sqlite has been removed as a backend. | 14:10 |
| seiflotfy | i am | 14:10 |
| alcabrera | We now have sqlalchemy to provide that | 14:10 |
| alcabrera | :) | 14:10 |
| seiflotfy | so the conf is broken then | 14:10 |
| alcabrera | I suspect that is the case | 14:10 |
| alcabrera | soooo | 14:10 |
| seiflotfy | ok can someone show me how the conf should look like? | 14:10 |
| alcabrera | sure thing | 14:10 |
| flwang | seiflotfy: I think flaper87 has updated the sample conf | 14:10 |
| seiflotfy | nope | 14:10 |
| seiflotfy | i am using it and its not working | 14:11 |
| flwang | seiflotfy: ok, would you mind post yours? because it works for me, and let me share mine | 14:11 |
| alcabrera | lemme help out on that | 14:11 |
| seiflotfy | last update was 15 days ago | 14:11 |
| seiflotfy | sure | 14:11 |
| seiflotfy | its http://paste.openstack.org/show/72180/ | 14:12 |
| flwang | http://paste.openstack.org/show/72182/ | 14:13 |
| alcabrera | storage = sqlite -> storage = sqlalchemy | 14:14 |
| flwang | seiflotfy: oh my gosh | 14:14 |
| alcabrera | that should do it | 14:14 |
| flwang | seiflotfy: you're using mongo and its uri is not correct, IMHO | 14:14 |
| flwang | alcabrera: and seems there is another issue | 14:15 |
| seiflotfy | that is the default | 14:15 |
| seiflotfy | flwang | 14:15 |
| flwang | alcabrera: I asked above, seems there is no check if the [drivers] storage = sqlite NOT compatiable with [drivers:storage:sqlalchemy], did i miss something? | 14:15 |
| flwang | seiflotfy: you should use your correct storage backend | 14:15 |
| alcabrera | I'd expect storage = sqlite to trigger a Stevedore exception, flwang | 14:16 |
| alcabrera | since sqlite was removed from setup.cfg:entry_points | 14:16 |
| flwang | seiflotfy: i'm saying, if you want to use mongodb as the backend, you should make sure it's installed on your env and update the uri in conf with the correct one | 14:16 |
| seiflotfy | no i dont want mongodb | 14:17 |
| seiflotfy | but this is the conf i got | 14:17 |
| flwang | alcabrera: nope, let me reword my concern | 14:17 |
| alcabrera | flwang, seiflotfy: I've got this one confirmed working as of now: http://paste.openstack.org/show/72183/ | 14:18 |
| alcabrera | flwang: yes, thanks! :) | 14:18 |
| * alcabrera listens | 14:18 | |
| flwang | alcabrera: my conf works great, I'm just raising an concern | 14:18 |
| flwang | alcabrera: what will happend if I use [drivers] storage = mongo with [drivers:storage:sqlalchemy] | 14:19 |
| alcabrera | ahh | 14:19 |
| alcabrera | what'll happen there, then | 14:19 |
| flwang | or the opposite | 14:19 |
| *** ykaplan has joined #openstack-marconi | 14:19 | |
| alcabrera | is that the mongo backend will be used | 14:19 |
| alcabrera | the sqlalchemy options wqill be ignored | 14:20 |
| flwang | alcabrera: i don't know, haha, just think alound | 14:20 |
| alcabrera | and if no mongo options are specified, then the defaults are used | 14:20 |
| alcabrera | same for the reverse case | 14:20 |
| alcabrera | 'storage = <backend>' takes precedence | 14:20 |
| alcabrera | and determines the behavior from there on | 14:20 |
| flwang | alcabrera: so is it necessary to add a pre check? | 14:20 |
| flwang | alcabrera: ok | 14:21 |
| alcabrera | I don't think so. :) | 14:21 |
| flwang | alcabrera: I will dig | 14:21 |
| alcabrera | however | 14:21 |
| alcabrera | something I'd like to see | 14:21 |
| alcabrera | is INFO output | 14:21 |
| alcabrera | at start up | 14:21 |
| alcabrera | indicating what storage/transport driver was used | 14:21 |
| alcabrera | thoughts? | 14:21 |
| flwang | alcabrera: yep, it would be better | 14:22 |
| alcabrera | another thought - sqlalchemy operating with the sqlite backend should be the default driver in marconi.conf-sample so that anyone can download marconi and start playing with it immediately | 14:22 |
| flwang | alcabrera: I love this suggestion | 14:23 |
| alcabrera | cool. :) | 14:23 |
| flwang | alcabrera: it will be more friendly for the new comer | 14:23 |
| alcabrera | agreed! | 14:24 |
| flaper87 | kinda back | 14:24 |
| flaper87 | alcabrera: goood morning! | 14:24 |
| alcabrera | flaper87: yo! | 14:24 |
| alcabrera | :) | 14:24 |
| flaper87 | seiflotfy: did you manage to fix that bug ? | 14:25 |
| flwang | alcabrera: I will file a wishlist bug to track that, thoughts? | 14:25 |
| flaper87 | alcabrera: 'sup buddy? | 14:25 |
| * flaper87 announces https://plus.google.com/events/ckjhm8rggnqvrnspftna845kubk | 14:25 | |
| seiflotfy | no | 14:25 |
| seiflotfy | serisouly | 14:25 |
| flwang | flaper87: thanks for the sharing | 14:26 |
| flwang | seiflotfy: did you try alcabrera's conf or mine? | 14:26 |
| seiflotfy | urs | 14:26 |
| seiflotfy | where is alcabrera s? | 14:27 |
| flaper87 | seiflotfy: what's the error? what's the problem your seeing ? | 14:27 |
| flaper87 | and also, why do you need a config file? | 14:27 |
| alcabrera | seiflotfy: http://paste.openstack.org/show/72183/ | 14:27 |
| flaper87 | you shouldn't need one for that test, it's a unittest | 14:27 |
| flaper87 | (i mean, for the one you were fixing) | 14:27 |
| alcabrera | flaper87: nice - re: G+ page | 14:27 |
| flaper87 | alcabrera: that's the link to a Hangout I'll have on the 27th | 14:28 |
| flwang | alcabrera: btw, I think [drivers] storage = sqlite will work fine, because flaper87 has considered the backward compatibility, IIRC based on the code review | 14:29 |
| *** mpanetta has joined #openstack-marconi | 14:29 | |
| alcabrera | flwang: you're right! I verified just a moment ago that sqlite points to sqlalchemy in setup.cfg | 14:29 |
| alcabrera | :) | 14:29 |
| * flaper87 gives himself a pop-tart for that | 14:29 | |
| flwang | flaper87: let's give flaper87 a cookie :) | 14:29 |
| flwang | flaper87: haha | 14:30 |
| alcabrera | heh. :) | 14:30 |
| flwang | flaper87: I assume your comments will be coming soon on my baby patch, is it? :) | 14:33 |
| flaper87 | ah damn, I forgot to click on submit | 14:33 |
| flaper87 | hahahaha | 14:33 |
| flwang | :P | 14:34 |
| flaper87 | that's how easy it is to distract me | 14:34 |
| mpanetta | Morning all | 14:34 |
| flwang | mpanetta: morning | 14:34 |
| flwang | alcabrera: it would be nice if you can take a look at my baby https://review.openstack.org/#/c/76555/ | 14:35 |
| alcabrera | mpanetta: good morning! | 14:35 |
| alcabrera | flwang: certainly. I'll check it out soon. :) | 14:35 |
| flwang | alcabrera: cool, thanks | 14:35 |
| *** ametts has joined #openstack-marconi | 14:36 | |
| flwang | alcabrera: I think we need to discuss the v1.1 feature in today meeting given the FF | 14:37 |
| alcabrera | oh *right*! I completely forgot about today's meeting! | 14:38 |
| alcabrera | thanks for reminding me, flwang! | 14:38 |
| alcabrera | Also, I agree. | 14:38 |
| flaper87 | holy moly, there's a meeting today | 14:38 |
| flaper87 | :D | 14:38 |
| flaper87 | mpanetta: morning | 14:38 |
| alcabrera | lol | 14:38 |
| alcabrera | I'm currently at the dentist with Jess, so depending on when they finish up, I may miss part of the meeting | 14:39 |
| alcabrera | flwang: I added API v1.1 as a discussion item to the meeting agenda | 14:40 |
| flwang | alcabrera: cool, thanks | 14:41 |
| *** amitgandhi has joined #openstack-marconi | 14:43 | |
| alcabrera | balajiiyer: good morning! | 14:43 |
| *** amitgandhi has quit IRC | 14:44 | |
| *** amitgandhi has joined #openstack-marconi | 14:44 | |
| flwang | flaper87: when will you be available to discuss those comments on my /health bp? | 14:45 |
| seiflotfy | works | 14:48 |
| alcabrera | seiflotfy: awesome! glad to hear it | 14:49 |
| flaper87 | flwang: shoot | 14:49 |
| *** haomaiwang has joined #openstack-marconi | 14:49 | |
| flaper87 | seiflotfy: w0000t | 14:49 |
| seiflotfy | sweet | 14:49 |
| seiflotfy | ok pushing a small fix here | 14:49 |
| balajiiyer | alcabrera: Morning. | 14:49 |
| alcabrera | balajiiyer: do you think you'll be up for discussing pecan evaluation at today's meeting? | 14:50 |
| flwang | flaper87: I collected the KPI from those ops guys, and they prefer to return the active shards number. but seems you have some concerns on that, is it? | 14:51 |
| seiflotfy | so | 14:51 |
| seiflotfy | 2014-03-04 15:51:28.977 10172 ERROR marconi.common.pipeline [-] Method does_not_exist not found in any of the registered stages appears | 14:51 |
| seiflotfy | how big of an issue is this | 14:51 |
| seiflotfy | ? | 14:51 |
| balajiiyer | alcabrera: yeah, I can talk about how Im doing the performance numbers. tsung + ab. have to do some due diligence before publishing the numbers. | 14:52 |
| alcabrera | balajiiyer: cool, and of course. :) | 14:52 |
| alcabrera | I ask since pecan evaluation is on the agenda for today's meeting | 14:52 |
| balajiiyer | alcabrera: ok, I will be there | 14:53 |
| alcabrera | w00t, awesome | 14:54 |
| *** malini_afk is now known as malini | 14:56 | |
| openstackgerrit | Seif Lotfy proposed a change to openstack/marconi: Fix assertion for Testutils to check on sqlite://:memory: https://review.openstack.org/77901 | 14:57 |
| *** saikrishna_ has joined #openstack-marconi | 14:58 | |
| flaper87 | seiflotfy: that's not an issue | 14:58 |
| flaper87 | I mean, IIRC, it's expected in those tests | 14:58 |
| flaper87 | flwang: well, the number of shards is useful *from* the shard catalog | 14:59 |
| seiflotfy | yeah i noticed tests worked in both cases | 14:59 |
| flaper87 | it depends on what shards we're talking about | 14:59 |
| seiflotfy | with or without the : | 14:59 |
| *** kgriffs_afk is now known as kgriffs | 14:59 | |
| alcabrera | meeting in 30 seconds | 14:59 |
| alcabrera | kgriffs: you made it! | 14:59 |
| alcabrera | :D | 14:59 |
| flaper87 | if we're talking about mongodb shards, we can't assume mongodb is sharded | 14:59 |
| kgriffs | heh | 14:59 |
| kgriffs | ice storm last night | 14:59 |
| kgriffs | my car was totally encased in ice | 15:00 |
| flaper87 | if we're talking about marconi's then it's necessary to get the shards from the shard catalog | 15:00 |
| kgriffs | fun times | 15:00 |
| kgriffs | meeting time? | 15:00 |
| flaper87 | kgriffs: fuck yeah | 15:00 |
| kgriffs | rock on | 15:00 |
| flwang | flaper87: got it, let's meeting first | 15:00 |
| seiflotfy | flaper87 | 15:00 |
| seiflotfy | [13:40:12] <@flaper87> seiflotfy: just replace it here: tests/unit/common/storage/test_utils.py | 15:00 |
| seiflotfy | [13:40:22] <@flaper87> I mean, s/memory/:memory:/ | 15:00 |
| flaper87 | seiflotfy: what's up with that? | 15:02 |
| flaper87 | :D | 15:02 |
| seiflotfy | this is the commit i did | 15:02 |
| flaper87 | did I say something stupid? | 15:02 |
| flaper87 | ah ok | 15:02 |
| seiflotfy | :D | 15:02 |
| flaper87 | cool | 15:02 |
| flaper87 | :D | 15:02 |
| *** cpallares has joined #openstack-marconi | 15:03 | |
| seiflotfy | flaper87 now fixing some performance issues | 15:09 |
| *** alcabrera has quit IRC | 15:10 | |
| *** haomaiw__ has joined #openstack-marconi | 15:12 | |
| *** jergerber has joined #openstack-marconi | 15:12 | |
| seiflotfy | flaper87 flwang i need help again with the test system | 15:13 |
| flaper87 | seiflotfy: what performance issues? | 15:14 |
| flaper87 | seiflotfy: shoot | 15:14 |
| seiflotfy | the nested functions | 15:14 |
| seiflotfy | but this is nto the issue | 15:14 |
| seiflotfy | my issue is that somehow i change code and run the tests and they pass even if the code i write makes 0 sense | 15:15 |
| *** haomaiwang has quit IRC | 15:16 | |
| seiflotfy | https://review.openstack.org/#/c/77901/ | 15:17 |
| seiflotfy | flaper87 flwang | 15:17 |
| flwang | seiflotfy: yep, will review it, it's on my list now | 15:18 |
| seiflotfy | woops wrong link | 15:18 |
| seiflotfy | http://paste.openstack.org/show/72206/ | 15:18 |
| seiflotfy | flwang flaper87 | 15:18 |
| seiflotfy | this is what i get | 15:18 |
| seiflotfy | whatever stuff i change in my code it still passes | 15:18 |
| seiflotfy | how come | 15:18 |
| seiflotfy | e.g: in marconi/queues/storage/utils.py i wrote | 15:19 |
| seiflotfy | for item in iterable: | 15:19 |
| seiflotfy | yield Look | 15:19 |
| seiflotfy | there is no Look anywhere | 15:19 |
| seiflotfy | so i expect the tests to not pass | 15:19 |
| seiflotfy | they do | 15:19 |
| seiflotfy | flwang ^ | 15:20 |
| flaper87 | seiflotfy: in your latest paste, all tests passed | 15:20 |
| seiflotfy | which shouldnt | 15:20 |
| seiflotfy | ok again | 15:20 |
| seiflotfy | i am fixing a new issue | 15:20 |
| seiflotfy | problem is i dont think the unit test run properly here | 15:21 |
| seiflotfy | so | 15:22 |
| seiflotfy | example | 15:22 |
| seiflotfy | marconi/queues/storage/utils.py line 109 | 15:22 |
| seiflotfy | do a "yield SoFresh" instead of "yield Keyed(item)" and run the tests | 15:23 |
| *** chandan_kumar has joined #openstack-marconi | 15:24 | |
| *** alcabrera has joined #openstack-marconi | 15:27 | |
| *** Kuwagata has joined #openstack-marconi | 15:29 | |
| *** ayoung-ZZzZzzZz_ is now known as ayoung | 15:30 | |
| *** jmckind has joined #openstack-marconi | 15:37 | |
| *** nidhi has joined #openstack-marconi | 15:49 | |
| *** nidhi has quit IRC | 15:53 | |
| flwang | kgriffs: I think we need to figure out what's gap now to land all the v1.1 patches, make sense? | 15:58 |
| kgriffs | ok, sure | 15:58 |
| kgriffs | so, we have /health to finish up - some comments there from flaper87 to address | 15:59 |
| kgriffs | alcabrera: need your eyes on this pls?! https://review.openstack.org/#/c/76093/ | 15:59 |
| kgriffs | flwang: I started work on requiring x-project-id header | 15:59 |
| kgriffs | and removed my name from several of the other v1.1 patches | 15:59 |
| kgriffs | so you or alcabrera can pick those up if you like | 15:59 |
| jay-atl | Hey Guys, so the consensus on the #openstack-dev channel seemed to be that the base url for Marconi in the service catalog should NOT have the version in the URL. But that the client should add the /v1 to the url... Does that sound right? Currently the client is not adding the version to the supplied url. So I'd have to fix the client.. | 16:00 |
| flwang | kgriffs: I would like to land in the /health with less KPI and add more with bug tracking or even in Juno | 16:00 |
| flaper87 | jay-atl: I think there was kind of a bug there | 16:01 |
| flaper87 | I *think* | 16:01 |
| flaper87 | I remember digging into that at some point | 16:01 |
| kgriffs | flwang: +1 | 16:01 |
| kgriffs | let's do the bare minimum | 16:01 |
| flwang | flaper87: any concern? | 16:01 |
| flaper87 | flwang: sounds good | 16:01 |
| flwang | cool | 16:01 |
| flwang | rock on | 16:02 |
| alcabrera | kgriffs: +2/approved bulk delete. the version_changed doc directive is sweet! | 16:02 |
| kgriffs | alcabrera: Can you respond to the [marconi] email on the ML? | 16:02 |
| alcabrera | meeting minutes updated | 16:02 |
| flwang | kgriffs: I will see what I can help on the other v1.1 bp after I submit a new ps for /health | 16:02 |
| * alcabrera missed the marconi email | 16:02 | |
| alcabrera | sure, I can reply. I just need to find it. | 16:02 |
| kgriffs | alcabrera: should be the latest one from openstack-dev | 16:02 |
| alcabrera | ah, I see it | 16:03 |
| kgriffs | "when we compare tools we should be careful. Compare for understanding, not for judgement. " | 16:05 |
| kgriffs | </random> | 16:05 |
| *** saikrishna_ has quit IRC | 16:05 | |
| alcabrera | replied | 16:06 |
| kgriffs | (quote by Henrik Kniberg) | 16:06 |
| alcabrera | flwang: very +1 - I'd love to have you join us as a core-reviewer. :) | 16:06 |
| kgriffs | ok folks, closing the poll | 16:06 |
| flwang | alcabrera: thank you, I will contribute more :) | 16:08 |
| flwang | thanks kgriffs, alcabrera, flaper87 and all you guys voting | 16:08 |
| flaper87 | flwang: you better do | 16:08 |
| flaper87 | :P | 16:08 |
| kgriffs | flwang: you now have +2 super powers | 16:09 |
| cpallares | flwang: yay :) | 16:09 |
| alcabrera | yay! | 16:09 |
| kgriffs | "with great power comes great responsibility" and all that... | 16:09 |
| flwang | flaper87: I will be the top3 reviewers in Juno for Marconi | 16:09 |
| kgriffs | :) | 16:09 |
| malini | I am doing some lobbying on #openstack-qa to get my band-aid approved..Feel free to show your support | 16:09 |
| flwang | kgriffs: yep, I totally understand. but I like the challenge :) | 16:10 |
| flaper87 | kgriffs: that's a nice way to put it | 16:11 |
| flwang | flaper87: so back to the /health, I would suggest to postpone the server status stuff, any concern? | 16:12 |
| flaper87 | flwang: meaning, just doing the storage drivers part ? | 16:12 |
| flwang | flaper87: I would like to know kgriff and alcabrera's comments | 16:13 |
| * alcabrera was going to run off for a bit, but waits to give comments | 16:13 | |
| flwang | except the server cpu util and memory util, any info we'd like to deliver from the storage backend? | 16:14 |
| alcabrera | flwang: you mean the KPIs? | 16:14 |
| flwang | alcabrera: yep | 16:14 |
| alcabrera | I'm fine with those being put off. :) | 16:14 |
| flaper87 | sounds good | 16:14 |
| alcabrera | for me, as long as /ping is a shallow check and /health allows for a deep check, I'm happy | 16:14 |
| *** saikrishna_ has joined #openstack-marconi | 16:14 | |
| flwang | alcabrera: yep, we can add more KPI in the future | 16:15 |
| *** jamieh has quit IRC | 16:15 | |
| alcabrera | for sure | 16:15 |
| alcabrera | alright | 16:15 |
| alcabrera | I'm going to run off for a bit | 16:15 |
| alcabrera | heading into the office. :D | 16:15 |
| alcabrera | see you all in 30-ish | 16:15 |
| alcabrera | o/ | 16:15 |
| flwang | alcabrera: see you later | 16:15 |
| mpanetta | be safe | 16:15 |
| mpanetta | see ya when ya get here | 16:15 |
| *** alcabrera has quit IRC | 16:15 | |
| malini | I am getting ready to be in a mood to rant :-@ | 16:16 |
| *** haomaiw__ has quit IRC | 16:16 | |
| *** haomaiwang has joined #openstack-marconi | 16:16 | |
| malini | kgriffs, flaper87: https://review.openstack.org/#/c/77832 | 16:21 |
| *** haomaiwang has quit IRC | 16:21 | |
| malini | See the last review comment | 16:21 |
| flaper87 | I'm really sick of those comments | 16:22 |
| flaper87 | for gods sake | 16:22 |
| mpanetta | Oh hell... | 16:23 |
| mpanetta | Are we plagued by highschoolers? | 16:23 |
| malini | I thought I was working in a 'community' ! | 16:23 |
| mpanetta | Highschool community? :P | 16:24 |
| malini | Its not as if I am corrupting a pristine code base with a redirect | 16:24 |
| flaper87 | not highschoolers at all and it's not his fault. All those guys are really cool it's just that this stuff about us using falcon is being used as an excuse to block some patches that are critical | 16:25 |
| mpanetta | But why? | 16:25 |
| malini | what has a lo redirect got to do with falcon? | 16:25 |
| mpanetta | That seems so... I donno... Childish? | 16:25 |
| malini | & blocking critical patches coz of the falcon thing, is not a professional response | 16:25 |
| kgriffs | oh boy | 16:25 |
| flaper87 | malini: exactly | 16:25 |
| kgriffs | I am about to write something...severe. | 16:25 |
| flaper87 | kgriffs: I'll let you tackle it this time | 16:26 |
| flaper87 | I'm like seriously frustrated now | 16:26 |
| malini | I worked with two other devstack core reviewers to work on this | 16:26 |
| malini | & nobody has any clue | 16:27 |
| malini | so its not that we didnt try hard enough | 16:27 |
| mpanetta | I still think it is all the odd redirects that are going on... | 16:27 |
| *** vkmc has quit IRC | 16:27 | |
| mpanetta | In the devstack code. | 16:27 |
| malini | mpanetta: but we need to prove its a devstack issue. | 16:28 |
| malini | The thing is we dont where tht is happening | 16:28 |
| mpanetta | That should be easy | 16:28 |
| mpanetta | replace marconi-server with a shell script that does nothing but loops and prints to stdout | 16:28 |
| mpanetta | If *that* crashes then it has to be a devstack issue | 16:29 |
| malini | mpanetta: can you do that? | 16:29 |
| mpanetta | Writing to stdout should not cause a crash | 16:29 |
| kgriffs | um, ok | 16:29 |
| kgriffs | fire me | 16:29 |
| kgriffs | kill me | 16:29 |
| mpanetta | Sure | 16:29 |
| malini | I am too sick & tired to try anything now | 16:29 |
| * kgriffs commented | 16:29 | |
| mpanetta | Let me send you a gist that you can pull down | 16:29 |
| malini | I need to take a break now..I am brain dead to work on anything | 16:30 |
| malini | be back in 30 min | 16:30 |
| mpanetta | Er, crap I have to wait until after this meeting that starts in a minute... | 16:30 |
| *** saikrishna_ has quit IRC | 16:30 | |
| flwang | flaper87: kgriffs: please revisit my baby when you're available https://review.openstack.org/#/c/76555/ | 16:32 |
| flwang | kgriffs: seems I still don't have the superman power, did you add me on the gerrit? | 16:33 |
| kgriffs | flwang: let me check | 16:33 |
| flwang | kgriffs: I'd like to be a superman :) | 16:34 |
| kgriffs | try now | 16:34 |
| flwang | kgriffs: i'm a superman now, guru | 16:34 |
| kgriffs | rock on | 16:34 |
| kgriffs | sorry about that | 16:34 |
| flwang | kgriffs: np | 16:34 |
| kgriffs | flaper87: check me on this logic | 16:34 |
| * flaper87 checks kgriffs circuits | 16:35 | |
| *** whenry has joined #openstack-marconi | 16:35 | |
| kgriffs | stand by ... let me check something | 16:35 |
| flaper87 | malini: is it possible to have a custom localrc in the gate? | 16:35 |
| flaper87 | I don't remember anymore | 16:35 |
| flaper87 | or at least setting custom env variables | 16:36 |
| flaper87 | malini: in other words, I want to set this to sqlalchemy https://github.com/openstack-dev/devstack/blob/master/lib/marconi#L44 :D | 16:36 |
| flaper87 | but *just* in the gate job | 16:36 |
| mpanetta | flaper87: I think she went away to blow off some steam. | 16:37 |
| flaper87 | mpanetta: hahahahahaha | 16:38 |
| mpanetta | flaper87: Are you next? :P | 16:38 |
| * flaper87 is THE saint | 16:38 | |
| mpanetta | haha | 16:38 |
| seiflotfy | flaper87 why are 290 tests on my side being skipped? | 16:40 |
| seiflotfy | PASSED (id=18, skips=290) | 16:41 |
| flwang | seiflotfy: IIRC, it's db specific | 16:42 |
| *** saikrishna_ has joined #openstack-marconi | 16:42 | |
| flaper87 | seiflotfy: because mongodb tests are not enabled by default | 16:45 |
| seiflotfy | hmmmm can i enable them too | 16:45 |
| seiflotfy | or do i need to change my db? | 16:45 |
| flaper87 | seiflotfy: export MARCONI_TEST_MONGODB=1 | 16:46 |
| seiflotfy | the problem i have right now i cant reliably test stuff | 16:46 |
| kgriffs | ok, flaper87: check me on this | 16:48 |
| kgriffs | falcon.Request | 16:48 |
| kgriffs | has self._wsgierrors = env['wsgi.errors'] | 16:48 |
| kgriffs | that is used to implement log_error | 16:48 |
| kgriffs | now, under wsgiref, that probably means writing to the console | 16:49 |
| kgriffs | BUT | 16:49 |
| kgriffs | wait | 16:49 |
| kgriffs | ok, yeah | 16:49 |
| kgriffs | BUT, malini proved by redirecting output to a file that it is zero bytes output | 16:49 |
| kgriffs | and so, log_error could not be the culprit | 16:49 |
| *** alcabrera has joined #openstack-marconi | 16:49 | |
| kgriffs | I will put a breakpoint in there just in case | 16:50 |
| seiflotfy | flaper87 basically anything i change in marconi/queues/storage/utils.py | 16:50 |
| seiflotfy | kgriffs | 16:50 |
| kgriffs | but if that IS the problem, saying this is a "bug" that is Falcon's fault is like saying this is the WSGI spec | 16:50 |
| kgriffs | fault | 16:50 |
| seiflotfy | anything i change in marconi/queues/storage/utils.py even it does not make sense and i yield non existing classes the tests pass | 16:50 |
| kgriffs | seiflotfy: looking | 16:53 |
| flaper87 | kgriffs: mhhh, that looks correct, I'm not sure what wsgiref does, though | 16:53 |
| mpanetta | Ok this is weird | 16:53 |
| seiflotfy | http://paste.openstack.org/show/72246/ | 16:54 |
| mpanetta | I replaced marconi-server with a short C program that prints a line every second and I still see the same output from the devstack scripts, and it kills my fox. | 16:54 |
| seiflotfy | kgriffs http://paste.openstack.org/show/72246/ is me running through the tests | 16:55 |
| seiflotfy | why is it trying keystone | 16:55 |
| alcabrera | devstack - notorious fox slayer | 16:55 |
| mpanetta | yeah I called the program fox... It just prints "A quick brown fox jumps over the moon" every second heh | 16:55 |
| flwang | flaper87: based on my understanding, the active shard number is the row count we stored in Shards table | 16:55 |
| mpanetta | This looks like it isn't a marconi issue to me... | 16:55 |
| mpanetta | Either that or they require that we background and daemonize ourselves... | 16:56 |
| mpanetta | Else they will kill us | 16:56 |
| alcabrera | hmmm, I wonder who the person to talk to about devstack internals is...? | 16:56 |
| flwang | flaper87: so I got the number by accessing db for storge driver, but get it by shard catalog for sharding, pls correct me if there is anything wrong | 16:56 |
| alcabrera | I have to say, the devstack docs are beautiful: http://devstack.org/ | 16:57 |
| flaper87 | flwang: but you shoulnd't be getting the number of shards in the DataDriver | 16:58 |
| flaper87 | that's what my comment was about (assuming you're talking about marconi shards ) | 16:58 |
| flaper87 | flwang: alcabrera can assist you better on the shard part | 16:58 |
| flwang | flaper87: ok, so double confirm, I will postpone the server status of mongo to the future, agree? | 16:59 |
| flwang | for now, let's just do cpu_util, memory_util and active shard number | 16:59 |
| malini | back... | 17:00 |
| mpanetta | alcabrera: Wow they do have extensive docs... | 17:00 |
| malini | Running is a good anger management technique | 17:00 |
| flwang | alcabrera: may I get some help from you about the shard part? | 17:00 |
| alcabrera | flwang: sure thing | 17:00 |
| mpanetta | malini: Devstack killed my fox :( | 17:00 |
| alcabrera | malini: agreed. I danced (DDR'd) this morning to prepare for the day. :D | 17:01 |
| malini | mpanetta: ? | 17:01 |
| flwang | alcabrera: pls take a look before I ask some questions :) | 17:01 |
| *** amitgandhi has quit IRC | 17:01 | |
| mpanetta | https://gist.github.com/anonymous/e980becd169b4bfc8e74 | 17:01 |
| mpanetta | It behaves the same as marconi-server, prints one line and dies | 17:01 |
| *** amitgandhi has joined #openstack-marconi | 17:01 | |
| mpanetta | But my fox program prints forever | 17:02 |
| mpanetta | So devstack must be killing it | 17:02 |
| malini | I found some weird comments y'day abt redirecting stdout | 17:02 |
| mpanetta | I donno what it means but maybe it will help. | 17:02 |
| malini | let me find tht | 17:02 |
| malini | https://github.com/openstack-dev/devstack/blob/master/functions-common#L979 | 17:02 |
| alcabrera | weird, mpanetta. | 17:02 |
| malini | I did chk with dtroyer a few times while hunting this one | 17:02 |
| alcabrera | flwang: where am I taking a look? Is there a patch out already> :) | 17:03 |
| malini | But he didnt mention anything abt the stdout stuff | 17:03 |
| flwang | alcabrera: yep :) https://review.openstack.org/#/c/76555/ | 17:03 |
| mpanetta | Well I wrote my fox in C so it isn't a python thing... | 17:03 |
| * alcabrera reads the review | 17:04 | |
| flwang | alcabrera: and another baby https://www.dropbox.com/s/8xpojqy6ii3b5pc/mmexport1386998377660.jpg?m= | 17:05 |
| mpanetta | alcabrera: Yes quite weird | 17:05 |
| alcabrera | cuuuute, flwang! <3 | 17:06 |
| flwang | alcabrera: 5 months :D | 17:06 |
| alcabrera | +2/Approved instantly on the other baby. :D | 17:06 |
| cpallares | flwang: what a cute baby! | 17:06 |
| mpanetta | Yes, but what does jenkins think? :P | 17:06 |
| flwang | alcabrera: haha, on my plan | 17:06 |
| flwang | mpanetta: good question, hehe | 17:07 |
| kgriffs | malini: does any test succeed in devstack? | 17:07 |
| kgriffs | or does it fail on the first one? | 17:07 |
| malini | kgriffs: all of them fail with the same connection error | 17:10 |
| alcabrera | brb | 17:11 |
| malini | I am going to work with mpanetta to figure out why stdout irritates devstack | 17:11 |
| kgriffs | ok | 17:12 |
| kgriffs | I just noticed that marconi-server -v is logging each request to my console | 17:12 |
| kgriffs | even though I have a file path configured | 17:12 |
| kgriffs | it isn't getting logged though falcon because I commented out Request.log_error | 17:13 |
| kgriffs | so I believe I was correct in saying this bug has NOTHING to do with Falcon vs. Pecan | 17:14 |
| *** rwsu has joined #openstack-marconi | 17:14 | |
| kgriffs | and even if it did, it would be stupid that devstack couldn't handle writing out to the WSGI error stream | 17:14 |
| *** ykaplan has quit IRC | 17:15 | |
| * alcabrera starts reviewing flwang's patch for context | 17:16 | |
| flwang | alcabrera: appreciate it :) | 17:16 |
| flwang | alcabrera: working hard on that | 17:17 |
| alcabrera | thanks! :D | 17:17 |
| * flwang rock on | 17:18 | |
| malini | kgriffs: mpanetta just proved its not a WSGI thing with his fox app | 17:19 |
| malini | Tempest is exactly that :D | 17:23 |
| kgriffs | malini: thanks! | 17:23 |
| malini | I am debugging the devstack script to find the point of failure | 17:24 |
| alcabrera | Everytime I start typing Tempest, I typo is as TempTest | 17:24 |
| alcabrera | Freudian slip, maybe | 17:24 |
| alcabrera | :P | 17:24 |
| malini | this is where it fails https://github.com/openstack-dev/devstack/blob/master/functions-common#L971 | 17:26 |
| * alcabrera continues reviewing flwang's patch - many comments being written! | 17:27 | |
| flwang | alcabrera: waiting from Beijing, China, at 1:28AM :) | 17:28 |
| *** reed has joined #openstack-marconi | 17:30 | |
| seiflotfy | flaper87 any other way to run tests here | 17:31 |
| alcabrera | flwang: hahaha, get some sleep! ;) | 17:32 |
| alcabrera | the review is done now | 17:32 |
| flwang | alcabrera: reading... I want to make it land in ;0 | 17:32 |
| alcabrera | I -1'd, primarily because 'active_shards' shouldn't be present all the time | 17:32 |
| alcabrera | gotcha, flwang. :) | 17:32 |
| alcabrera | going to go fetch a sprite zero. nobody mutate the world while I'm away. ;P | 17:34 |
| *** alcabrera is now known as alcabrera|afk | 17:34 | |
| *** alcabrera|afk is now known as alcabrera | 17:35 | |
| flwang | flaper87: any lucky around? | 17:36 |
| kgriffs | malini: https://gist.github.com/anonymous/29619594145df6db5cf2 | 17:36 |
| kgriffs | I'm trying to figure out where that message is coming from... | 17:36 |
| kgriffs | wsgiref itself? | 17:36 |
| kgriffs | looks like a vanilla web server loglline | 17:37 |
| malini | kgriffs: yes..& It writes to both stdout & the log_file | 17:38 |
| kgriffs | i would say "that's our problem" except when you do the redirect workaround, the result file is 0 bytes! | 17:41 |
| malini | if its ok for devstack to kill an app with stdout, it is OUR problem | 17:45 |
| *** jraim_ has joined #openstack-marconi | 17:45 | |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: feat(api): API v1.1 bulk delete change https://review.openstack.org/76093 | 17:46 |
| kgriffs | malini: sorry, I meant that wsgiref printing out is "the problem" | 17:46 |
| kgriffs | not a marconi vs. devstack thing | 17:46 |
| kgriffs | I found it curious that your workaround wrote 0 bytes | 17:47 |
| alcabrera | wsgiref.silent = True (sure would be nice, heh) | 17:47 |
| kgriffs | if wsgiref is writing out stuff | 17:47 |
| malini | it wud have been good if people can work together to make things better, instead of you don't do stuff like me <sigh/> | 17:48 |
| kgriffs | +1000000 | 17:49 |
| malini | tht is the reason world has wars | 17:49 |
| *** mwagner_lap has joined #openstack-marconi | 17:49 | |
| kgriffs | that is a BIG PROBLEM with a subset of OpenStack community | 17:49 |
| kgriffs | I'm tired of people saying "well, that's just the way OpenStack has always been" | 17:50 |
| malini | I am going to end up a hippie with a CO-EXIST bumper sticker, after this | 17:50 |
| alcabrera | heh, malini. :D | 17:50 |
| alcabrera | get me one, too | 17:50 |
| alcabrera | brb | 17:51 |
| *** alcabrera is now known as alcabrera|afk | 17:51 | |
| *** chandan_kumar has quit IRC | 17:53 | |
| *** jraim has quit IRC | 17:53 | |
| *** jraim_ is now known as jraim | 17:53 | |
| kgriffs | malini: I don't see those request loglines in marconi.log | 17:53 |
| kgriffs | so I suspect it is wsgiref doing it | 17:53 |
| kgriffs | (on my box, at least | 17:53 |
| kgriffs | ) | 17:53 |
| *** balajiiyer has quit IRC | 17:55 | |
| *** alcabrera|afk is now known as alcabrera | 17:55 | |
| flwang | is there any possibility the marconi.conf will be cached? | 17:55 |
| flwang | after changed the storage driver from sqlalchemy to mongodb, seems it's still using the sqlite | 17:56 |
| kgriffs | flwang: after restarting the wsgi server, it should pick up the new one | 17:56 |
| kgriffs | maybe it isn't finding your marconi.conf and defaulting to sqlalchemy? | 17:56 |
| flwang | kgriffs: I assume that as well | 17:57 |
| kgriffs | alcabrera: ping | 17:57 |
| flwang | but I have one under ~/.marconi | 17:57 |
| flwang | kgriffs: when is the next checkpoint for v1.1 today? | 17:57 |
| kgriffs | I guess it would be in a few minutes | 17:58 |
| kgriffs | (checkpoint) | 17:58 |
| kgriffs | so, I was distracted by tempest thing | 17:58 |
| kgriffs | this has been such a time sink | 17:58 |
| kgriffs | I just now started working on v1.1 things | 17:58 |
| alcabrera | kgriffs: pong | 17:59 |
| flwang | i think the overall shape of /health is ok based on the comments from flaper87and alcabrera | 17:59 |
| kgriffs | alcabrera: can your write a quick wsgiref "hello world" app to confirm it is logging and that isn't anything to do with MArconi? | 17:59 |
| kgriffs | i mean, logging to stdout | 17:59 |
| alcabrera | sure thing | 17:59 |
| openstackgerrit | Shawnyu proposed a change to openstack/marconi: Use sqlalchemy as the default storage driver, and update it in sample configuration file https://review.openstack.org/77949 | 17:59 |
| flwang | seems the biggest concerns is the KPI - active shards number, I'm think replace it with a simple one | 17:59 |
| alcabrera | have it in a paste in 5 | 17:59 |
| kgriffs | flwang: ok, let's simplify or remove altogether | 18:00 |
| alcabrera | kgriffs: desired log level? DEBUG> INFO? | 18:00 |
| kgriffs | shouldn't matter | 18:00 |
| kgriffs | wsgiref doesn't respect logging level | 18:01 |
| flwang | kgriffs: i think cpu_util and memory_util are both ok, you know, they are related to the marconi SERVER, and we're trying to add another KPI from the storage backend perspective | 18:01 |
| kgriffs | I just tested running simple_server with simple_server.demo_app and it is writing out the same request lines to console | 18:01 |
| alcabrera | kk | 18:01 |
| alcabrera | kgriffs: http://paste.openstack.org/show/72261/ | 18:02 |
| *** balajiiyer has joined #openstack-marconi | 18:03 | |
| kgriffs | actually, I was thinking to not use logging in there | 18:03 |
| kgriffs | basically a noop | 18:03 |
| flwang | kgriffs: so I'm thinking can we just leverage the is_alive()'s result and return it as a part of the detailed /health, just think alound | 18:03 |
| alcabrera | ah | 18:03 |
| alcabrera | I see | 18:03 |
| alcabrera | I'll simplify it | 18:03 |
| kgriffs | and see if when you run with simple_server.make_server if it still writes to console | 18:03 |
| alcabrera | :P | 18:03 |
| kgriffs | flwang: does is_alive return anything now? | 18:04 |
| alcabrera | kgriffs: http://paste.openstack.org/show/72262/ (barebones) | 18:04 |
| alcabrera | ah | 18:04 |
| alcabrera | yeah, I'll try that and report | 18:04 |
| flwang | true of false to indicate if the backend storage is alive or not | 18:04 |
| flwang | alcabrera: pls check me if i'm wrong | 18:04 |
| flwang | s/of/or | 18:05 |
| kgriffs | flwang: ok, we could just report that as a field "storage_reachable: True/False" or something | 18:05 |
| kgriffs | at this point, I'd just say keep it super simple and we can add more KPIs later | 18:05 |
| flwang | alcabrera: any comments about replacing the active shards with storage_reachable ? | 18:06 |
| kgriffs | we just want to demonstrate pulling one KPI from storage | 18:06 |
| flwang | kgriffs: yep | 18:06 |
| kgriffs | flwang: we could also count total number of messages per shard or something | 18:06 |
| kgriffs | that should be easy to implement | 18:06 |
| kgriffs | (in each driver) | 18:06 |
| kgriffs | but, I think if we can just use the alive call that is good enough for now | 18:07 |
| flwang | kgriffs: yep, I discussed that with alcabrera recently, but seems it's a little bit difficult | 18:07 |
| kgriffs | ok | 18:07 |
| kgriffs | let's KISS | 18:07 |
| flwang | kgriffs: ok, will submit a ps soon | 18:07 |
| alcabrera | kgriffs: it logged nothing on start up, but.... | 18:07 |
| alcabrera | 127.0.0.1 - - [04/Mar/2014 13:07:31] "GET / HTTP/1.1" 200 0 | 18:07 |
| kgriffs | alcabrera: ok | 18:07 |
| kgriffs | then that is wsgiref | 18:07 |
| alcabrera | yup | 18:07 |
| kgriffs | malini: ^^^ this could be the culprit | 18:07 |
| kgriffs | although why other projects don't have the same issue, is beyond me | 18:08 |
| kgriffs | or they have a hack to work around it | 18:08 |
| kgriffs | alcabrera: can you find a way to supress those messages? | 18:08 |
| alcabrera | for ref: http://paste.openstack.org/show/72263/ | 18:08 |
| alcabrera | sure thing | 18:08 |
| kgriffs | then malini can try your solution to see if it helps anything | 18:08 |
| kgriffs | alcabrera: thanks! | 18:08 |
| malini | sure thing.. | 18:08 |
| malini | I hope this fixes it | 18:08 |
| kgriffs | ok, I am going to be back in 5 mins and then dive into v1.1 stuff | 18:08 |
| *** rossk has joined #openstack-marconi | 18:10 | |
| alcabrera | kgriffs, malini: solved - http://paste.openstack.org/show/72269/ | 18:14 |
| *** fvollero is now known as fvollero|afk | 18:14 | |
| alcabrera | I hope. It doesn't log for me. :D | 18:14 |
| alcabrera | http get localhost:8001 # was very quiet | 18:14 |
| kgriffs | ok, malini can you try that? | 18:17 |
| kgriffs | if it works, we could have log_message write to LOG and see if it still works | 18:18 |
| kgriffs | (it should as long as a path is set in marconi.conf for the log file) | 18:18 |
| malini | trying | 18:18 |
| kgriffs | kk, brb | 18:18 |
| kgriffs | I need to cool down | 18:18 |
| openstackgerrit | Fei Long Wang proposed a change to openstack/marconi: feat(v1.1): Implement new /health endpoint for v1.1 https://review.openstack.org/76555 | 18:18 |
| flwang | flaper87: alcabrera: please help review https://review.openstack.org/76555 :) | 18:21 |
| alcabrera | malini: marconi/marconi/queues/transport/wsgi/driver.py:listen | 18:25 |
| malini | listening | 18:25 |
| malini | :D | 18:25 |
| alcabrera | apply the wsgiref fix there | 18:25 |
| alcabrera | :D | 18:25 |
| alcabrera | then we can all | 18:25 |
| alcabrera | CO_EXIST | 18:26 |
| alcabrera | err | 18:26 |
| alcabrera | CO-EXIST | 18:26 |
| alcabrera | yes | 18:26 |
| flwang | alcabrera: flaper87: i'm removing the active shards with storage reachable which can leverage current is_alive to get a basic KPI from the storage backend POV | 18:26 |
| alcabrera | much ☮, such ♥ | 18:26 |
| *** notmyname has left #openstack-marconi | 18:26 | |
| flwang | please let me know if you have any concern/comment about this | 18:26 |
| alcabrera | flwang: I'll look at the health patch again soon. :) | 18:27 |
| flwang | alcabrera: appreciation from China :) | 18:27 |
| *** fvollero|afk is now known as fvollero | 18:28 | |
| *** balajiiyer has quit IRC | 18:33 | |
| mpanetta | Here, to make every ones day better: http://i.imgur.com/jsmh5t0.gif :P | 18:35 |
| flwang | haha, I guess the foot is smelly :) | 18:38 |
| * alcabrera catches up | 18:44 | |
| malini | kgriffs: still no luck :( | 18:45 |
| alcabrera | so far, redirecting to /dev/null feels right to me. :/ | 18:46 |
| alcabrera | I'm going to go read some devstack | 18:46 |
| alcabrera | and cry a little | 18:46 |
| malini | join the club alcabrera ! | 18:46 |
| flwang | i'm going to take some sleep | 18:46 |
| flwang | kgriffs: when is the next checkpoint? | 18:47 |
| openstackgerrit | Caleb Groom proposed a change to openstack/python-marconiclient: Add docstrings to modules to reduce pylint score https://review.openstack.org/77958 | 18:47 |
| flwang | alcabrera: pls feel free drop you comments on my baby patch, I will be back after 5, 6 hours if I can wake up :D | 18:48 |
| alcabrera | flwang: good night! :) | 18:50 |
| flwang | alcabrera: have a nice day, guys | 18:51 |
| openstackgerrit | Fei Long Wang proposed a change to openstack/marconi: feat(v1.1): Implement new /health endpoint for v1.1 https://review.openstack.org/76555 | 18:51 |
| *** balajiiyer has joined #openstack-marconi | 18:53 | |
| malini | alcabrera, kgriffs: looks familiar https://github.com/openstack-dev/devstack/blob/master/lib/trove#L205 ? | 18:55 |
| kgriffs | back | 18:55 |
| kgriffs | flwang: thanks man | 18:55 |
| *** karandash8 has joined #openstack-marconi | 18:55 | |
| *** balajiiyer1 has joined #openstack-marconi | 18:55 | |
| kgriffs | sorry, too many distractions | 18:55 |
| *** cpallares has quit IRC | 18:55 | |
| kgriffs | malini, alcabrera: any luck? | 18:56 |
| malini | kgriffs: not with the WSGI change | 18:56 |
| kgriffs | crap | 18:56 |
| malini | but trove seems to be doing some weird redirections too | 18:56 |
| malini | https://github.com/openstack-dev/devstack/blob/master/lib/trove#L205 | 18:56 |
| mpanetta | The change did not help? :( | 18:57 |
| malini | mpanetta: no | 18:57 |
| malini | wonder if trove folks ran into something similar | 18:57 |
| *** jmckind has quit IRC | 18:57 | |
| mpanetta | This is so very odd | 18:57 |
| *** balajiiyer has quit IRC | 18:58 | |
| malini | I see alot of re-direct to /dev/null in other projects | 18:58 |
| malini | there shud be something going on | 18:58 |
| mpanetta | Then why can't we do that? | 18:58 |
| malini | we need to figure out why they did it | 18:59 |
| mpanetta | Ok | 18:59 |
| mpanetta | Can we ask them? | 18:59 |
| malini | let me try doing a blame to figure out whom to ask | 19:00 |
| *** SnowDust has joined #openstack-marconi | 19:01 | |
| malini | Michael Basnight wrote tht part | 19:01 |
| malini | kgriffs: he is a RAX guy, rt? | 19:02 |
| alcabrera | malini: yes | 19:02 |
| alcabrera | I just checked. :) | 19:02 |
| malini | I hope he is in Austin & sits next to kgriffs ;) | 19:02 |
| SnowDust | malini is who ? | 19:02 |
| malini | Michale Basnight | 19:03 |
| malini | SnowDust: ^ | 19:03 |
| SnowDust | michael basnight ? whozz hub_cap ? | 19:03 |
| malini | hmmm..dont know | 19:03 |
| SnowDust | ROFL .. | 19:04 |
| malini | I think my humor sense is kinda dead today | 19:04 |
| *** amitgandhi has quit IRC | 19:04 | |
| SnowDust | malini is an indian word btw .. | 19:05 |
| malini | not surprising,considering I am from India | 19:05 |
| SnowDust | Michael Basnight from india ? | 19:05 |
| SnowDust | its not shocking anymore .. | 19:06 |
| SnowDust | may be rackspace shifted to lower parel or nehru place .. who knows | 19:06 |
| malini | my brain is already messed up..I think I can do without more help :) | 19:06 |
| *** sballe has quit IRC | 19:07 | |
| *** rossk has quit IRC | 19:08 | |
| * alcabrera is thoroughly confused | 19:08 | |
| mpanetta | Join the club heh | 19:08 |
| * mpanetta creates a Club for the onfused | 19:08 | |
| mpanetta | ... | 19:08 |
| mpanetta | and unable to type, apparently | 19:08 |
| SnowDust | hehe | 19:09 |
| *** amitgandhi has joined #openstack-marconi | 19:09 | |
| kgriffs | ok folks, sorry, but I am getting pulled into the Oslo mailstrom | 19:09 |
| alcabrera | kgriffs: good luck! | 19:10 |
| kgriffs | so, I am going to make the call and push v1.1 to Juno | 19:10 |
| kgriffs | anyone vehemently disagree? | 19:11 |
| openstackgerrit | A change was merged to openstack/marconi: fix(mongodb): Limit kwarg must be an int https://review.openstack.org/76678 | 19:11 |
| alcabrera | I don't disagree, kgriffs | 19:11 |
| alcabrera | I think it's the right call | 19:12 |
| alcabrera | given we need to discuss some of the coming fixes a bit more | 19:12 |
| kgriffs | +1 | 19:12 |
| malini | I might have found something | 19:22 |
| malini | https://github.com/openstack-dev/devstack/blob/master/lib/trove#L205 | 19:22 |
| malini | if we do the same thing as trove, it works too | 19:22 |
| mpanetta | Oh? | 19:22 |
| malini | use '--debug 2>&1' instead of '>/dev/null' | 19:22 |
| mpanetta | So redirecting stderr to stdout worked? | 19:23 |
| mpanetta | Oh you don't need the --debug | 19:23 |
| mpanetta | Just the 2>&1, sorry | 19:23 |
| malini | ok let me try without tht | 19:23 |
| mpanetta | --debug is a flag for whatever they are running | 19:23 |
| malini | but waht does tht mean? | 19:23 |
| malini | I mean 2>&1 ? | 19:23 |
| mpanetta | That means redirect stderr to stdout | 19:24 |
| mpanetta | so all output (error or not) goes to stdout. | 19:24 |
| mpanetta | AKA FD 1 | 19:24 |
| malini | hmm..but tht means our original theory is wrong abt stdout? | 19:24 |
| mpanetta | Apparently | 19:24 |
| alcabrera | weird | 19:25 |
| alcabrera | devstack seems contradictory | 19:25 |
| malini | or it had to be in some format for functions-common to understand? | 19:25 |
| mpanetta | the 2>&1 thing is a bashism | 19:25 |
| alcabrera | I vaguely remember a higher-level command redirecting stderr to /dev/null | 19:25 |
| malini | I hope I am not imagining this to be working :D | 19:25 |
| *** jamieh has joined #openstack-marconi | 19:26 | |
| alcabrera | brb | 19:27 |
| *** alcabrera is now known as alcabrera|afk | 19:27 | |
| malini | mpanetta: it works without the debug too | 19:28 |
| mpanetta | Good, that is what I would expect :) | 19:28 |
| malini | It feels weird going back & saying 'they did it, so we can too' | 19:28 |
| mpanetta | haha | 19:29 |
| mpanetta | Someone somewhere is screwing up the normal file descriptors I think. | 19:29 |
| malini | BUT I am going to go back & ask | 19:30 |
| malini | wish me luck | 19:30 |
| *** alcabrera|afk is now known as alcabrera | 19:31 | |
| alcabrera | malini: good luck! | 19:31 |
| *** alcabrera is now known as alcabrera|afk | 19:33 | |
| *** mpanetta has quit IRC | 19:47 | |
| *** mpanetta_ has joined #openstack-marconi | 19:47 | |
| *** cpallares has joined #openstack-marconi | 19:51 | |
| *** mpanetta_ is now known as mpanetta | 19:51 | |
| *** alcabrera|afk is now known as alcabrera | 19:54 | |
| malini | I need to leave now..I am still waiting to hear back from Sean Dague on if the other alternative is acceptable for https://review.openstack.org/#/c/77832/ | 19:57 |
| malini | If I hear back from, I'll submit a new patchset | 19:57 |
| *** malini is now known as malini_afk | 19:57 | |
| mpanetta | I hope you get something back today | 20:02 |
| openstackgerrit | Shawnyu proposed a change to openstack/marconi: Use sqlalchemy as the default storage driver. https://review.openstack.org/77985 | 20:03 |
| openstackgerrit | Shawnyu proposed a change to openstack/marconi: sqlalchemy: make default storage driver https://review.openstack.org/77986 | 20:06 |
| kgriffs | flaper87: wrt bugs and the feature freeze | 20:07 |
| kgriffs | actually, let me go find that on the wiki | 20:07 |
| alcabrera | raxuanyu: Let's get together tomorrow. You don't need to submit a new patch for each fix. The way to get around that is that each time you make a fix to a given patch, you need to rebase against master. | 20:08 |
| *** amitgandhi has quit IRC | 20:09 | |
| kgriffs | back | 20:10 |
| kgriffs | https://wiki.openstack.org/wiki/Release_Cycle | 20:10 |
| kgriffs | alcabrera, flaper87: the way I read this page is that we should triage remaining bugs and only leave the ones for i-3 that are "milestone-critical" | 20:11 |
| alcabrera | Release Candidates dance - nice | 20:12 |
| openstackgerrit | Shawnyu proposed a change to openstack/marconi: sqlalchemy: make default storage driver https://review.openstack.org/77988 | 20:13 |
| alcabrera | I agree with your interpretation, kgriffs | 20:13 |
| alcabrera | so, we need to triage at some point | 20:13 |
| kgriffs | actually, we need to do it now | 20:14 |
| kgriffs | release mtg in 45 minutes | 20:14 |
| kgriffs | flaper87: ^^^ | 20:15 |
| alcabrera | yikes | 20:15 |
| kgriffs | looks like it is just you and me | 20:27 |
| kgriffs | let's get to it | 20:27 |
| *** SnowDust has quit IRC | 20:29 | |
| alcabrera | kk | 20:32 |
| kgriffs | https://launchpad.net/marconi/+milestone/icehouse-3 | 20:32 |
| kgriffs | Get a set of messages returns wrong hrefs | 20:32 |
| alcabrera | hmmm | 20:33 |
| kgriffs | has anyone reported this bug in the wild? | 20:33 |
| alcabrera | I haven't heard about it | 20:33 |
| kgriffs | this report is for the gate | 20:33 |
| alcabrera | yup | 20:33 |
| kgriffs | ok, sounds like we can move this one | 20:33 |
| alcabrera | yes - non-critical | 20:33 |
| kgriffs | next | 20:34 |
| kgriffs | add functional tests for sharding | 20:34 |
| alcabrera | hmmm | 20:35 |
| alcabrera | non-critical in one sense | 20:36 |
| alcabrera | it doesn't affect i3 features | 20:36 |
| alcabrera | *but* | 20:36 |
| alcabrera | we need soon after | 20:36 |
| alcabrera | so a level below critical | 20:36 |
| alcabrera | kgriffs: thoughts? :) | 20:36 |
| *** saikrishna_ has quit IRC | 20:40 | |
| kgriffs | sorry, someone mentioned pecan/falcon in #openstack-meeting | 20:42 |
| kgriffs | alcabrera: OK, we can keep as high | 20:42 |
| kgriffs | But move to Juno? | 20:42 |
| alcabrera | no worries | 20:42 |
| alcabrera | and yes | 20:42 |
| kgriffs | next: functions defined inline | 20:43 |
| kgriffs | thought I saw someone was working on that? | 20:43 |
| alcabrera | hmmm | 20:45 |
| alcabrera | perf, right? | 20:45 |
| alcabrera | I don't remember anyone mentioning that for several months | 20:46 |
| kgriffs | ah, I think Seif recently picked it up to work on it | 20:47 |
| kgriffs | I don't see a patch for it, do you? | 20:47 |
| *** malini_afk is now known as malini | 20:47 | |
| alcabrera | hmmm | 20:48 |
| alcabrera | none | 20:48 |
| alcabrera | in any case | 20:48 |
| alcabrera | I'm calling it low | 20:48 |
| alcabrera | kgriffs: ^ | 20:48 |
| kgriffs | refresh | 20:51 |
| kgriffs | Let's move this to Juno | 20:51 |
| kgriffs | oops | 20:51 |
| kgriffs | wrong clipboard | 20:51 |
| kgriffs | https://launchpad.net/marconi/+milestone/icehouse-3 | 20:51 |
| alcabrera | migration from ddt to testscenarios - low | 20:51 |
| *** amitgandhi has joined #openstack-marconi | 20:52 | |
| alcabrera | kgriffs: looking at this list, all the low bugs are still low, imho | 20:53 |
| *** amit has joined #openstack-marconi | 20:53 | |
| alcabrera | I'll tackle [bug/tests] ShardTests.test_listing_simple assumes alphabetical sorting order during the Summer, while working with GSoC students | 20:54 |
| *** amit is now known as Guest99336 | 20:54 | |
| kgriffs | ok | 20:55 |
| kgriffs | OK, can you remove icehouse and the milestone from each of the Low's for me? | 20:56 |
| alcabrera | sure thing | 20:56 |
| *** amitgandhi has quit IRC | 20:57 | |
| *** jmckind has joined #openstack-marconi | 20:58 | |
| kgriffs | alcabrera: status on this? https://bugs.launchpad.net/marconi/+bug/1258618 | 21:00 |
| alcabrera | kgriffs: there, that took care of it | 21:00 |
| kgriffs | thanks! | 21:00 |
| alcabrera | status unknown, tbh. :/ | 21:00 |
| alcabrera | I stepped away from that so long ago I'd need to check | 21:00 |
| kgriffs | ok. I thought it was fixed, but I could be wrong | 21:00 |
| kgriffs | go ahead and check | 21:01 |
| alcabrera | k | 21:01 |
| alcabrera | kgriffs: looks fixed: https://github.com/openstack/marconi/blob/master/marconi/queues/storage/sharding.py#L425 | 21:03 |
| *** Guest99336 has quit IRC | 21:03 | |
| *** amitgandhi has joined #openstack-marconi | 21:04 | |
| kgriffs | ok | 21:04 |
| alcabrera | it even got the unset_many update from oslo.cache v1 | 21:04 |
| kgriffs | can you update the bug accordingly? | 21:04 |
| alcabrera | sure thing | 21:04 |
| kgriffs | drop a link to the fix or something in the whiteboard | 21:04 |
| kgriffs | thanks! | 21:05 |
| alcabrera | "The fix to this was not tracked. It's been fixed at some point in the past month." | 21:06 |
| alcabrera | ^^ | 21:06 |
| alcabrera | heh | 21:06 |
| *** jamieh has quit IRC | 21:09 | |
| kgriffs | whatever works, man! | 21:10 |
| kgriffs | ok, so looks like we have one bug pending, the doc patch pending, and the tempest patch pending | 21:10 |
| alcabrera | very close | 21:12 |
| kgriffs | let me check on my bug | 21:12 |
| kgriffs | as far as docs and tempest go... | 21:13 |
| balajiiyer1 | kgriffs: do we have some extra time for doc patch? | 21:13 |
| kgriffs | balajiiyer1: I don't know | 21:13 |
| kgriffs | I think they said it may be OK to let us graduate if the docs are ready within a week or two of i-3 | 21:13 |
| kgriffs | graduating is sort of a strange thing | 21:14 |
| kgriffs | I think that if we show the patch is pending, that should be proof enough for them | 21:15 |
| balajiiyer1 | yeah, we are very close with docs though. I will talk to Catherine and see whether she can submit a patch today. | 21:15 |
| kgriffs | ok | 21:15 |
| kgriffs | that would be great | 21:15 |
| *** rossk has joined #openstack-marconi | 21:15 | |
| *** flaper87 is now known as flaper87|afk | 21:20 | |
| malini | I uploaded a new patchset https://review.openstack.org/#/c/77832/ | 21:24 |
| malini | this replaces the >/dev/null to do something similar to what trove folks did | 21:24 |
| alcabrera | sweet | 21:25 |
| alcabrera | I just saw that | 21:25 |
| alcabrera | thanks, malini! | 21:25 |
| malini | I dont know if it is any better than the prev approach | 21:25 |
| malini | But has a precedent | 21:25 |
| mpanetta | malini: Awesome! They better accept it :P | 21:25 |
| malini | I dont know abt tht :) | 21:25 |
| malini | I need to log off now | 21:26 |
| alcabrera | me, too | 21:26 |
| malini | will chk back later on the patch | 21:26 |
| alcabrera | Good night, all! | 21:26 |
| malini | Good night!! | 21:26 |
| *** malini has left #openstack-marconi | 21:26 | |
| mpanetta | See you guys lay=ter | 21:26 |
| *** alcabrera has quit IRC | 21:27 | |
| *** Kuwagata has left #openstack-marconi | 21:41 | |
| *** AndChat|437241 has joined #openstack-marconi | 21:53 | |
| *** AndChat|437241 has quit IRC | 21:54 | |
| *** rossk has quit IRC | 21:57 | |
| *** rossk has joined #openstack-marconi | 21:58 | |
| openstackgerrit | Catherine Richardson proposed a change to openstack/marconi: adds docs directory with dev guide https://review.openstack.org/72001 | 22:02 |
| *** tedross has quit IRC | 22:02 | |
| balajiiyer1 | kgriffs: Doc patch has landed from Catherine. It lgtm. can you take a look? | 22:18 |
| kgriffs | sure | 22:18 |
| kgriffs | give me just a minute | 22:18 |
| *** balajiiyer1 has quit IRC | 22:25 | |
| openstackgerrit | Kurt Griffiths proposed a change to openstack/marconi: fix(MongoDB): Driver does not retry on AutoReconnect errors https://review.openstack.org/75165 | 22:42 |
| kgriffs | be back later folks. We will need to land the remaining patches by morning (EU time) | 22:50 |
| *** mpanetta has quit IRC | 22:58 | |
| *** kgriffs is now known as kgriffs_afk | 23:00 | |
| *** amitgandhi has quit IRC | 23:16 | |
| *** jergerber has quit IRC | 23:25 | |
| *** cpallares has quit IRC | 23:43 | |
| *** mwagner_lap has quit IRC | 23:44 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
