{"action":"created","issue":{"url":"https://api.github.com/repos/SSSD/sssd/issues/25","repository_url":"https://api.github.com/repos/SSSD/sssd","labels_url":"https://api.github.com/repos/SSSD/sssd/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/SSSD/sssd/issues/25/comments","events_url":"https://api.github.com/repos/SSSD/sssd/issues/25/events","html_url":"https://github.com/SSSD/sssd/pull/25","id":176865965,"number":25,"title":"TESTS: Add integration tests for the proxy provider of sssd-secrets","user":{"login":"jhrozek","id":715522,"avatar_url":"https://avatars.githubusercontent.com/u/715522?v=3","gravatar_id":"","url":"https://api.github.com/users/jhrozek","html_url":"https://github.com/jhrozek","followers_url":"https://api.github.com/users/jhrozek/followers","following_url":"https://api.github.com/users/jhrozek/following{/other_user}","gists_url":"https://api.github.com/users/jhrozek/gists{/gist_id}","starred_url":"https://api.github.com/users/jhrozek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jhrozek/subscriptions","organizations_url":"https://api.github.com/users/jhrozek/orgs","repos_url":"https://api.github.com/users/jhrozek/repos","events_url":"https://api.github.com/users/jhrozek/events{/privacy}","received_events_url":"https://api.github.com/users/jhrozek/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"celestian","id":1382015,"avatar_url":"https://avatars.githubusercontent.com/u/1382015?v=3","gravatar_id":"","url":"https://api.github.com/users/celestian","html_url":"https://github.com/celestian","followers_url":"https://api.github.com/users/celestian/followers","following_url":"https://api.github.com/users/celestian/following{/other_user}","gists_url":"https://api.github.com/users/celestian/gists{/gist_id}","starred_url":"https://api.github.com/users/celestian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/celestian/subscriptions","organizations_url":"https://api.github.com/users/celestian/orgs","repos_url":"https://api.github.com/users/celestian/repos","events_url":"https://api.github.com/users/celestian/events{/privacy}","received_events_url":"https://api.github.com/users/celestian/received_events","type":"User","site_admin":false},"assignees":[{"login":"celestian","id":1382015,"avatar_url":"https://avatars.githubusercontent.com/u/1382015?v=3","gravatar_id":"","url":"https://api.github.com/users/celestian","html_url":"https://github.com/celestian","followers_url":"https://api.github.com/users/celestian/followers","following_url":"https://api.github.com/users/celestian/following{/other_user}","gists_url":"https://api.github.com/users/celestian/gists{/gist_id}","starred_url":"https://api.github.com/users/celestian/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/celestian/subscriptions","organizations_url":"https://api.github.com/users/celestian/orgs","repos_url":"https://api.github.com/users/celestian/repos","events_url":"https://api.github.com/users/celestian/events{/privacy}","received_events_url":"https://api.github.com/users/celestian/received_events","type":"User","site_admin":false}],"milestone":null,"comments":16,"created_at":"2016-09-14T10:02:18Z","updated_at":"2016-09-20T11:45:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/SSSD/sssd/pulls/25","html_url":"https://github.com/SSSD/sssd/pull/25","diff_url":"https://github.com/SSSD/sssd/pull/25.diff","patch_url":"https://github.com/SSSD/sssd/pull/25.patch"},"body":"Execrcies the basic operations of the sssd-secrets responder and can be\nused as a basis to add more tests."},"comment":{"url":"https://api.github.com/repos/SSSD/sssd/issues/comments/248278221","html_url":"https://github.com/SSSD/sssd/pull/25#issuecomment-248278221","issue_url":"https://api.github.com/repos/SSSD/sssd/issues/25","id":248278221,"user":{"login":"lslebodn","id":8050776,"avatar_url":"https://avatars.githubusercontent.com/u/8050776?v=3","gravatar_id":"","url":"https://api.github.com/users/lslebodn","html_url":"https://github.com/lslebodn","followers_url":"https://api.github.com/users/lslebodn/followers","following_url":"https://api.github.com/users/lslebodn/following{/other_user}","gists_url":"https://api.github.com/users/lslebodn/gists{/gist_id}","starred_url":"https://api.github.com/users/lslebodn/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lslebodn/subscriptions","organizations_url":"https://api.github.com/users/lslebodn/orgs","repos_url":"https://api.github.com/users/lslebodn/repos","events_url":"https://api.github.com/users/lslebodn/events{/privacy}","received_events_url":"https://api.github.com/users/lslebodn/received_events","type":"User","site_admin":false},"created_at":"2016-09-20T11:45:52Z","updated_at":"2016-09-20T11:45:52Z","body":"On (20/09/16 04:09), Jakub Hrozek wrote:\n>I was getting random failures where the socket was not yet ready without\n>the sleep, like this:\n>    ConnectionError: ('Connection aborted.', error(2, 'No such file or\n>    directory'))\n>\n>I think the test was just too fast. I pushed a new version that instead\n>tries to connect to the socket and waits 0.2 sec if it cannot. The test\n>gives up after a second.\n>\nActive checking of socket is much better approach.\nI think you forgot to close socket :-)\n\nI would also appreciate smaller waits (0.1 or even 0.05)\nit's up to you.\n\nLS\n"}}
{"actor":{"display_login":"lslebodn"}}