{"action":"created","issue":{"url":"https://api.github.com/repos/learningequality/barbequeue/issues/8","repository_url":"https://api.github.com/repos/learningequality/barbequeue","labels_url":"https://api.github.com/repos/learningequality/barbequeue/issues/8/labels{/name}","comments_url":"https://api.github.com/repos/learningequality/barbequeue/issues/8/comments","events_url":"https://api.github.com/repos/learningequality/barbequeue/issues/8/events","html_url":"https://github.com/learningequality/barbequeue/pull/8","id":230740871,"number":8,"title":"[WIP] More impro","user":{"login":"benjaoming","id":374612,"avatar_url":"https://avatars3.githubusercontent.com/u/374612?v=3","gravatar_id":"","url":"https://api.github.com/users/benjaoming","html_url":"https://github.com/benjaoming","followers_url":"https://api.github.com/users/benjaoming/followers","following_url":"https://api.github.com/users/benjaoming/following{/other_user}","gists_url":"https://api.github.com/users/benjaoming/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaoming/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaoming/subscriptions","organizations_url":"https://api.github.com/users/benjaoming/orgs","repos_url":"https://api.github.com/users/benjaoming/repos","events_url":"https://api.github.com/users/benjaoming/events{/privacy}","received_events_url":"https://api.github.com/users/benjaoming/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"aronasorman","id":191955,"avatar_url":"https://avatars3.githubusercontent.com/u/191955?v=3","gravatar_id":"","url":"https://api.github.com/users/aronasorman","html_url":"https://github.com/aronasorman","followers_url":"https://api.github.com/users/aronasorman/followers","following_url":"https://api.github.com/users/aronasorman/following{/other_user}","gists_url":"https://api.github.com/users/aronasorman/gists{/gist_id}","starred_url":"https://api.github.com/users/aronasorman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aronasorman/subscriptions","organizations_url":"https://api.github.com/users/aronasorman/orgs","repos_url":"https://api.github.com/users/aronasorman/repos","events_url":"https://api.github.com/users/aronasorman/events{/privacy}","received_events_url":"https://api.github.com/users/aronasorman/received_events","type":"User","site_admin":false},"assignees":[{"login":"aronasorman","id":191955,"avatar_url":"https://avatars3.githubusercontent.com/u/191955?v=3","gravatar_id":"","url":"https://api.github.com/users/aronasorman","html_url":"https://github.com/aronasorman","followers_url":"https://api.github.com/users/aronasorman/followers","following_url":"https://api.github.com/users/aronasorman/following{/other_user}","gists_url":"https://api.github.com/users/aronasorman/gists{/gist_id}","starred_url":"https://api.github.com/users/aronasorman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aronasorman/subscriptions","organizations_url":"https://api.github.com/users/aronasorman/orgs","repos_url":"https://api.github.com/users/aronasorman/repos","events_url":"https://api.github.com/users/aronasorman/events{/privacy}","received_events_url":"https://api.github.com/users/aronasorman/received_events","type":"User","site_admin":false}],"milestone":null,"comments":8,"created_at":"2017-05-23T15:04:18Z","updated_at":"2017-05-29T19:16:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/learningequality/barbequeue/pulls/8","html_url":"https://github.com/learningequality/barbequeue/pull/8","diff_url":"https://github.com/learningequality/barbequeue/pull/8.diff","patch_url":"https://github.com/learningequality/barbequeue/pull/8.patch"},"body":"Sorry, didn't think you'd be up this early so I wasn't really clear about the other PR..\r\n\r\nI'm doing some more improvisation...\r\n\r\n * [x] Tox\r\n * [x] Coverage\r\n * [x] Travis\r\n * [ ] Python 2.7, ~3.4~ and pypy broken\r\n * [x] Not understanding why `scheduler` and `storage` don't have `__init__.py` files to make them packages"},"comment":{"url":"https://api.github.com/repos/learningequality/barbequeue/issues/comments/304717823","html_url":"https://github.com/learningequality/barbequeue/pull/8#issuecomment-304717823","issue_url":"https://api.github.com/repos/learningequality/barbequeue/issues/8","id":304717823,"user":{"login":"benjaoming","id":374612,"avatar_url":"https://avatars3.githubusercontent.com/u/374612?v=3","gravatar_id":"","url":"https://api.github.com/users/benjaoming","html_url":"https://github.com/benjaoming","followers_url":"https://api.github.com/users/benjaoming/followers","following_url":"https://api.github.com/users/benjaoming/following{/other_user}","gists_url":"https://api.github.com/users/benjaoming/gists{/gist_id}","starred_url":"https://api.github.com/users/benjaoming/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/benjaoming/subscriptions","organizations_url":"https://api.github.com/users/benjaoming/orgs","repos_url":"https://api.github.com/users/benjaoming/repos","events_url":"https://api.github.com/users/benjaoming/events{/privacy}","received_events_url":"https://api.github.com/users/benjaoming/received_events","type":"User","site_admin":false},"created_at":"2017-05-29T19:16:21Z","updated_at":"2017-05-29T19:16:21Z","body":"@aronasorman sorry I can't pursue the Py2 stuff any further. The PR is growing too much :) I recommend that you use logging outputs in tests and add `-s` for `pytest`, which you can invoke outside of tox, provided that you do `pip install -e .` to get the latest requirements.\r\n\r\nThere are some threads/queues that never join, and then the following error that I think you'd be better at debuggin :)\r\n\r\n```\r\n✗ pytest -s tests/worker/backends/test_inmem.py\r\n================================================================= test session starts =================================================================\r\nplatform linux2 -- Python 2.7.12, pytest-3.0.7, py-1.4.33, pluggy-0.4.0\r\nrootdir: /code/barbequeue, inifile:\r\nplugins: mock-1.6.0, cov-2.5.1\r\ncollected 3 items \r\n\r\ntests/worker/backends/test_inmem.py .WARNING:barbequeue.worker.backends.inmem.WORKER[moon-oscar-spaghetti-spring]:Got exception for job simplejob: id() takes exactly one argument (0 given)\r\nException in thread MONITOR-fanta-stairway-monkey-west:\r\nTraceback (most recent call last):\r\n  File \"/usr/lib/python2.7/threading.py\", line 801, in __bootstrap_inner\r\n    self.run()\r\n  File \"/code/barbequeue/src/barbequeue/common/classes.py\", line 94, in run\r\n    self.main_loop(timeout=self.DEFAULT_TIMEOUT_SECONDS)\r\n  File \"/code/barbequeue/src/barbequeue/worker/backends/inmem.py\", line 84, in main_loop\r\n    self.receive_incoming_messages(timeout)\r\n  File \"/code/barbequeue/src/barbequeue/worker/backends/inmem.py\", line 98, in receive_incoming_messages\r\n    self.handle_incoming_message(msg)\r\n  File \"/code/barbequeue/src/barbequeue/worker/backends/inmem.py\", line 109, in handle_incoming_message\r\n    raise UnknownMessageError()\r\nUnknownMessageError\r\n```"}}
{"actor":{"display_login":"benjaoming"}}