{"action":"created","issue":{"url":"https://api.github.com/repos/diaspora/diaspora/issues/6530","repository_url":"https://api.github.com/repos/diaspora/diaspora","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":[{"url":"https://api.github.com/repos/diaspora/diaspora/labels/orphan","name":"orphan","color":"dddddd"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/diaspora/diaspora/milestones/14","html_url":"https://github.com/diaspora/diaspora/milestones/0.6.0.0","labels_url":"https://api.github.com/repos/diaspora/diaspora/milestones/14/labels","id":1063323,"number":14,"title":"0.6.0.0","description":"","creator":{"login":"jhass","id":141294,"avatar_url":"https://avatars.githubusercontent.com/u/141294?v=3","gravatar_id":"","url":"https://api.github.com/users/jhass","html_url":"https://github.com/jhass","followers_url":"https://api.github.com/users/jhass/followers","following_url":"https://api.github.com/users/jhass/following{/other_user}","gists_url":"https://api.github.com/users/jhass/gists{/gist_id}","starred_url":"https://api.github.com/users/jhass/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhass/subscriptions","organizations_url":"https://api.github.com/users/jhass/orgs","repos_url":"https://api.github.com/users/jhass/repos","events_url":"https://api.github.com/users/jhass/events{/privacy}","received_events_url":"https://api.github.com/users/jhass/received_events","type":"User","site_admin":false},"open_issues":10,"closed_issues":290,"state":"open","created_at":"2015-04-11T18:44:30Z","updated_at":"2016-06-08T21:42:20Z","due_on":null,"closed_at":null},"comments":23,"created_at":"2015-10-30T23:50:00Z","updated_at":"2016-06-09T01:10:30Z","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/224776511","html_url":"https://github.com/diaspora/diaspora/pull/6530#issuecomment-224776511","issue_url":"https://api.github.com/repos/diaspora/diaspora/issues/6530","id":224776511,"user":{"login":"denschub","id":344777,"avatar_url":"https://avatars.githubusercontent.com/u/344777?v=3","gravatar_id":"","url":"https://api.github.com/users/denschub","html_url":"https://github.com/denschub","followers_url":"https://api.github.com/users/denschub/followers","following_url":"https://api.github.com/users/denschub/following{/other_user}","gists_url":"https://api.github.com/users/denschub/gists{/gist_id}","starred_url":"https://api.github.com/users/denschub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/denschub/subscriptions","organizations_url":"https://api.github.com/users/denschub/orgs","repos_url":"https://api.github.com/users/denschub/repos","events_url":"https://api.github.com/users/denschub/events{/privacy}","received_events_url":"https://api.github.com/users/denschub/received_events","type":"User","site_admin":false},"created_at":"2016-06-09T01:10:30Z","updated_at":"2016-06-09T01:10:30Z","body":"> how will it manage to pick proper workers for the old jobs scheduled in redis?\r\n\r\nYou can specify the queue via the workes cli arguments, `-q [queue name]`. Since queues are only identified by their name without any magic, I see no issues here.\r\n\r\n> And what if we leave the old queues names in the config/sidekiq.yml along with the new ones for a migration period? Will it work? Or it would bloat the memory?\r\n\r\nNo idea. Sidekiq claims it's not designed for \"more than a handful of queues\", but who knows what would happen. I doubt the `sidekiq.yml` will be an issue when specifying the queue manually as mentioned above, so removing them in the config should not cause any issues. Obviously, this has to be tested, i'm replying right off the top of my head here."}}
{"actor":{"display_login":"denschub"}}