{"action":"created","issue":{"url":"https://api.github.com/repos/mozilla/fjord/issues/447","labels_url":"https://api.github.com/repos/mozilla/fjord/issues/447/labels{/name}","comments_url":"https://api.github.com/repos/mozilla/fjord/issues/447/comments","events_url":"https://api.github.com/repos/mozilla/fjord/issues/447/events","html_url":"https://github.com/mozilla/fjord/pull/447","id":54248495,"number":447,"title":"[bug 1104863] Upgrade to celery v3.1.17","user":{"login":"rlr","id":36629,"avatar_url":"https://avatars.githubusercontent.com/u/36629?v=3","gravatar_id":"","url":"https://api.github.com/users/rlr","html_url":"https://github.com/rlr","followers_url":"https://api.github.com/users/rlr/followers","following_url":"https://api.github.com/users/rlr/following{/other_user}","gists_url":"https://api.github.com/users/rlr/gists{/gist_id}","starred_url":"https://api.github.com/users/rlr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rlr/subscriptions","organizations_url":"https://api.github.com/users/rlr/orgs","repos_url":"https://api.github.com/users/rlr/repos","events_url":"https://api.github.com/users/rlr/events{/privacy}","received_events_url":"https://api.github.com/users/rlr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-01-13T21:08:56Z","updated_at":"2015-01-20T16:33:38Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/mozilla/fjord/pulls/447","html_url":"https://github.com/mozilla/fjord/pull/447","diff_url":"https://github.com/mozilla/fjord/pull/447.diff","patch_url":"https://github.com/mozilla/fjord/pull/447.patch"},"body":"This is the laziest possible upgrade path. I upgraded celery and it's dependencies and added two lines of code. I had to switch to the unreleased django-celery master tip so that `./manage.py celeryd` would work. There are also fixes for django 1.7 in there that weren't released.\r\n\r\nI know we can't do much this week with this other than possibly test it on stage. I tested it locally (`CELERY_ALWAYS_EAGER = False`) and triggered these tasks: celery_health_task, index_chunk_task, index_item_task and unindex_item_task.\r\n\r\nThis sort of kicks the can down the road for updating the code to use the new API for celery. We have a couple options:\r\n\r\n* Ship this as is and keep grinding through the rest of the django 1.7 blockers to get us onto django 1.7 ASAP. Have a followup bug to come back to updating everything to modern celery usage.\r\n* Go through the pains now and pay down the debt. Then continue on django 1.7 work.\r\n\r\nI'm game either way.\r\n\r\nr?"},"comment":{"url":"https://api.github.com/repos/mozilla/fjord/issues/comments/70684643","html_url":"https://github.com/mozilla/fjord/pull/447#issuecomment-70684643","issue_url":"https://api.github.com/repos/mozilla/fjord/issues/447","id":70684643,"user":{"login":"willkg","id":820826,"avatar_url":"https://avatars.githubusercontent.com/u/820826?v=3","gravatar_id":"","url":"https://api.github.com/users/willkg","html_url":"https://github.com/willkg","followers_url":"https://api.github.com/users/willkg/followers","following_url":"https://api.github.com/users/willkg/following{/other_user}","gists_url":"https://api.github.com/users/willkg/gists{/gist_id}","starred_url":"https://api.github.com/users/willkg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/willkg/subscriptions","organizations_url":"https://api.github.com/users/willkg/orgs","repos_url":"https://api.github.com/users/willkg/repos","events_url":"https://api.github.com/users/willkg/events{/privacy}","received_events_url":"https://api.github.com/users/willkg/received_events","type":"User","site_admin":false},"created_at":"2015-01-20T16:33:38Z","updated_at":"2015-01-20T16:33:38Z","body":"This looks great. Can you squash these two commits?"}}