{"action":"created","issue":{"url":"https://api.github.com/repos/dagwieers/mrepo/issues/73","repository_url":"https://api.github.com/repos/dagwieers/mrepo","labels_url":"https://api.github.com/repos/dagwieers/mrepo/issues/73/labels{/name}","comments_url":"https://api.github.com/repos/dagwieers/mrepo/issues/73/comments","events_url":"https://api.github.com/repos/dagwieers/mrepo/issues/73/events","html_url":"https://github.com/dagwieers/mrepo/pull/73","id":37229563,"number":73,"title":"mrepo: Remove temporary reposync config files before raising exceptions","user":{"login":"clayne","id":2467434,"avatar_url":"https://avatars.githubusercontent.com/u/2467434?v=3","gravatar_id":"","url":"https://api.github.com/users/clayne","html_url":"https://github.com/clayne","followers_url":"https://api.github.com/users/clayne/followers","following_url":"https://api.github.com/users/clayne/following{/other_user}","gists_url":"https://api.github.com/users/clayne/gists{/gist_id}","starred_url":"https://api.github.com/users/clayne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clayne/subscriptions","organizations_url":"https://api.github.com/users/clayne/orgs","repos_url":"https://api.github.com/users/clayne/repos","events_url":"https://api.github.com/users/clayne/events{/privacy}","received_events_url":"https://api.github.com/users/clayne/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2014-07-06T22:18:54Z","updated_at":"2016-02-28T03:08:39Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dagwieers/mrepo/pulls/73","html_url":"https://github.com/dagwieers/mrepo/pull/73","diff_url":"https://github.com/dagwieers/mrepo/pull/73.diff","patch_url":"https://github.com/dagwieers/mrepo/pull/73.patch"},"body":"If a single file/fetch fails during a reposync, mrepo will raise an\r\nexception over it (and continue) but will not cleanup the temporary\r\nconfig.\r\n\r\nResults in this:\r\n\r\n```\r\n$ ls -la /tmp/tmp* | tail\r\n-rw------- 1 root root 133 Jun 29 23:15 /tmp/tmpzCRrbK\r\n-rw------- 1 root root 133 Jun 26 22:45 /tmp/tmpzCdtxq\r\n-rw------- 1 root root 133 Jul  6 04:15 /tmp/tmpzJ2rdp\r\n-rw------- 1 root root 133 Jul  2 16:30 /tmp/tmpzKc_s5\r\n-rw------- 1 root root 133 Jun 27 14:15 /tmp/tmpzW4E_h\r\n-rw------- 1 root root 133 Jul  6 15:30 /tmp/tmpzhzBuJ\r\n-rw------- 1 root root 133 Jun 27 07:45 /tmp/tmpziEzGo\r\n-rw------- 1 root root 133 Jun 26 03:30 /tmp/tmpzkp3Ym\r\n-rw------- 1 root root 133 Jun 26 19:30 /tmp/tmpztKB57\r\n-rw------- 1 root root 133 Jul  5 12:15 /tmp/tmpztbK9i\r\n$ ls -la /tmp/tmp* | wc -l\r\n1039\r\n$ sudo cat /tmp/tmpztbK9i\r\n[EnterpriseLinux5-x86_64-percona]\r\nname=EnterpriseLinux5-x86_64-percona\r\nbaseurl=http://repo.percona.com/centos/5/os/x86_64/\r\nenabled=1\r\n```"},"comment":{"url":"https://api.github.com/repos/dagwieers/mrepo/issues/comments/189772544","html_url":"https://github.com/dagwieers/mrepo/pull/73#issuecomment-189772544","issue_url":"https://api.github.com/repos/dagwieers/mrepo/issues/73","id":189772544,"user":{"login":"clayne","id":2467434,"avatar_url":"https://avatars.githubusercontent.com/u/2467434?v=3","gravatar_id":"","url":"https://api.github.com/users/clayne","html_url":"https://github.com/clayne","followers_url":"https://api.github.com/users/clayne/followers","following_url":"https://api.github.com/users/clayne/following{/other_user}","gists_url":"https://api.github.com/users/clayne/gists{/gist_id}","starred_url":"https://api.github.com/users/clayne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clayne/subscriptions","organizations_url":"https://api.github.com/users/clayne/orgs","repos_url":"https://api.github.com/users/clayne/repos","events_url":"https://api.github.com/users/clayne/events{/privacy}","received_events_url":"https://api.github.com/users/clayne/received_events","type":"User","site_admin":false},"created_at":"2016-02-28T03:08:39Z","updated_at":"2016-02-28T03:08:39Z","body":"Any chance this can be merged? It's been sitting here for almost 2 years. We've already implemented the change in our own fork and there are zero issues. This fixes an actual bug in the code."}}