{"action":"created","issue":{"url":"https://api.github.com/repos/diaspora/diaspora/issues/6530","labels_url":"https://api.github.com/repos/diaspora/diaspora/issues/6530/labels{/name}","comments_url":"https://api.github.com/repos/diaspora/diaspora/issues/6530/comments","events_url":"https://api.github.com/repos/diaspora/diaspora/issues/6530/events","html_url":"https://github.com/diaspora/diaspora/pull/6530","id":114374184,"number":6530,"title":"Consolidate amount of sidekiq queues from 13 to 5","user":{"login":"jaywink","id":1174866,"avatar_url":"https://avatars.githubusercontent.com/u/1174866?v=3","gravatar_id":"","url":"https://api.github.com/users/jaywink","html_url":"https://github.com/jaywink","followers_url":"https://api.github.com/users/jaywink/followers","following_url":"https://api.github.com/users/jaywink/following{/other_user}","gists_url":"https://api.github.com/users/jaywink/gists{/gist_id}","starred_url":"https://api.github.com/users/jaywink/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jaywink/subscriptions","organizations_url":"https://api.github.com/users/jaywink/orgs","repos_url":"https://api.github.com/users/jaywink/repos","events_url":"https://api.github.com/users/jaywink/events{/privacy}","received_events_url":"https://api.github.com/users/jaywink/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-10-30T23:50:00Z","updated_at":"2015-10-31T01:35:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/diaspora/diaspora/pulls/6530","html_url":"https://github.com/diaspora/diaspora/pull/6530","diff_url":"https://github.com/diaspora/diaspora/pull/6530.diff","patch_url":"https://github.com/diaspora/diaspora/pull/6530.patch"},"body":"Sidekiq documentation says 'Sidekiq is not designed to work well with dozens of queues.'. Having the amount of queues we have at the moment brings no anyway.\r\n\r\nCloses #5571\r\n\r\nIf this is accepted, should put a note in Changelog, in relevant version to be included in, for podmins to run their queues empty on update."},"comment":{"url":"https://api.github.com/repos/diaspora/diaspora/issues/comments/152683959","html_url":"https://github.com/diaspora/diaspora/pull/6530#issuecomment-152683959","issue_url":"https://api.github.com/repos/diaspora/diaspora/issues/6530","id":152683959,"user":{"login":"jaywink","id":1174866,"avatar_url":"https://avatars.githubusercontent.com/u/1174866?v=3","gravatar_id":"","url":"https://api.github.com/users/jaywink","html_url":"https://github.com/jaywink","followers_url":"https://api.github.com/users/jaywink/followers","following_url":"https://api.github.com/users/jaywink/following{/other_user}","gists_url":"https://api.github.com/users/jaywink/gists{/gist_id}","starred_url":"https://api.github.com/users/jaywink/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jaywink/subscriptions","organizations_url":"https://api.github.com/users/jaywink/orgs","repos_url":"https://api.github.com/users/jaywink/repos","events_url":"https://api.github.com/users/jaywink/events{/privacy}","received_events_url":"https://api.github.com/users/jaywink/received_events","type":"User","site_admin":false},"created_at":"2015-10-31T01:35:16Z","updated_at":"2015-10-31T01:35:16Z","body":"I don't think joindiaspora.com configuration is a good way to structure things on, judging by how bad it mostly behaves ;)\r\n\r\nI like the idea of priority queues, but then our default way of running sidekiq should somehow use them. If we have a queue called `urgent` and a queue called `low`, it is just misleading to then treat them the same. Or at least I can't figure out whether we prioritize queues currently, except the way joindiaspora is doing.\r\n\r\nSingle pods can always prioritize things however they want. But if we do it in the core software, our default configuration should behave like the prioritization has some meaning.\r\n\r\nSeparating webfinger jobs back into their own queue wont make them run any faster with the normal configuration?"}}