{"action":"created","issue":{"url":"https://api.github.com/repos/mrworf/plexupdate/issues/137","repository_url":"https://api.github.com/repos/mrworf/plexupdate","labels_url":"https://api.github.com/repos/mrworf/plexupdate/issues/137/labels{/name}","comments_url":"https://api.github.com/repos/mrworf/plexupdate/issues/137/comments","events_url":"https://api.github.com/repos/mrworf/plexupdate/issues/137/events","html_url":"https://github.com/mrworf/plexupdate/issues/137","id":189948931,"number":137,"title":"New logging and cron behavior","user":{"login":"mrworf","id":1848354,"avatar_url":"https://avatars.githubusercontent.com/u/1848354?v=3","gravatar_id":"","url":"https://api.github.com/users/mrworf","html_url":"https://github.com/mrworf","followers_url":"https://api.github.com/users/mrworf/followers","following_url":"https://api.github.com/users/mrworf/following{/other_user}","gists_url":"https://api.github.com/users/mrworf/gists{/gist_id}","starred_url":"https://api.github.com/users/mrworf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrworf/subscriptions","organizations_url":"https://api.github.com/users/mrworf/orgs","repos_url":"https://api.github.com/users/mrworf/repos","events_url":"https://api.github.com/users/mrworf/events{/privacy}","received_events_url":"https://api.github.com/users/mrworf/received_events","type":"User","site_admin":false},"labels":[{"id":91517273,"url":"https://api.github.com/repos/mrworf/plexupdate/labels/enhancement","name":"enhancement","color":"84b6eb","default":true}],"state":"open","locked":false,"assignee":{"login":"mrworf","id":1848354,"avatar_url":"https://avatars.githubusercontent.com/u/1848354?v=3","gravatar_id":"","url":"https://api.github.com/users/mrworf","html_url":"https://github.com/mrworf","followers_url":"https://api.github.com/users/mrworf/followers","following_url":"https://api.github.com/users/mrworf/following{/other_user}","gists_url":"https://api.github.com/users/mrworf/gists{/gist_id}","starred_url":"https://api.github.com/users/mrworf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrworf/subscriptions","organizations_url":"https://api.github.com/users/mrworf/orgs","repos_url":"https://api.github.com/users/mrworf/repos","events_url":"https://api.github.com/users/mrworf/events{/privacy}","received_events_url":"https://api.github.com/users/mrworf/received_events","type":"User","site_admin":false},"assignees":[{"login":"mrworf","id":1848354,"avatar_url":"https://avatars.githubusercontent.com/u/1848354?v=3","gravatar_id":"","url":"https://api.github.com/users/mrworf","html_url":"https://github.com/mrworf","followers_url":"https://api.github.com/users/mrworf/followers","following_url":"https://api.github.com/users/mrworf/following{/other_user}","gists_url":"https://api.github.com/users/mrworf/gists{/gist_id}","starred_url":"https://api.github.com/users/mrworf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrworf/subscriptions","organizations_url":"https://api.github.com/users/mrworf/orgs","repos_url":"https://api.github.com/users/mrworf/repos","events_url":"https://api.github.com/users/mrworf/events{/privacy}","received_events_url":"https://api.github.com/users/mrworf/received_events","type":"User","site_admin":false}],"milestone":null,"comments":27,"created_at":"2016-11-17T06:12:31Z","updated_at":"2016-11-21T03:06:35Z","closed_at":null,"body":"To better track this, here's a new issue for this purpose.\r\n\r\nTo recap, existing logging and cron behavior is not awesome. It started out with the best intentions and good work by @mscreations but it seems to have taken on a life by itself and become unnecessarily kludgy.\r\n\r\nSo... I've kinda done some work now to improve the situation, please checkout the new branch \"reworklog\".\r\n\r\nThe new solution makes away with both quiet and cron options. quiet can be accomplished with >/dev/null and cron is handled by cronwrapper.\r\n\r\nCronwrapper is needed anyway on an ubuntu system since you cannot simply drop plexupdate.sh into any of the cron.* folders. It also adds logging to facility and washing of return codes.\r\n\r\nThis design also leaves room for doing some nicer handling of a busy server by retrying to apply the update if users are online.\r\n\r\nThoughts? It's kinda jarring so I didn't want to drop it on people. Also need to update the wiki with new details on cronwrapper before merging into master.\r\n\r\n@mscreations @demonbane let it rip! 😜\r\n"},"comment":{"url":"https://api.github.com/repos/mrworf/plexupdate/issues/comments/261831915","html_url":"https://github.com/mrworf/plexupdate/issues/137#issuecomment-261831915","issue_url":"https://api.github.com/repos/mrworf/plexupdate/issues/137","id":261831915,"user":{"login":"mrworf","id":1848354,"avatar_url":"https://avatars.githubusercontent.com/u/1848354?v=3","gravatar_id":"","url":"https://api.github.com/users/mrworf","html_url":"https://github.com/mrworf","followers_url":"https://api.github.com/users/mrworf/followers","following_url":"https://api.github.com/users/mrworf/following{/other_user}","gists_url":"https://api.github.com/users/mrworf/gists{/gist_id}","starred_url":"https://api.github.com/users/mrworf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrworf/subscriptions","organizations_url":"https://api.github.com/users/mrworf/orgs","repos_url":"https://api.github.com/users/mrworf/repos","events_url":"https://api.github.com/users/mrworf/events{/privacy}","received_events_url":"https://api.github.com/users/mrworf/received_events","type":"User","site_admin":false},"created_at":"2016-11-21T03:06:35Z","updated_at":"2016-11-21T03:06:35Z","body":"Yes, good idea, very simple and should catch most which miss the README.md\r\n\r\nI'm currently going through the docs and references to .plexupdate so they're gone or explained."}}
{"actor":{"display_login":"mrworf"}}