{"action":"created","issue":{"url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/128","repository_url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib","labels_url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/128/labels{/name}","comments_url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/128/comments","events_url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/128/events","html_url":"https://github.com/Juniper/ansible-junos-stdlib/pull/128","id":147059412,"number":128,"title":"Add ping module","user":{"login":"dgarros","id":304126,"avatar_url":"https://avatars.githubusercontent.com/u/304126?v=3","gravatar_id":"","url":"https://api.github.com/users/dgarros","html_url":"https://github.com/dgarros","followers_url":"https://api.github.com/users/dgarros/followers","following_url":"https://api.github.com/users/dgarros/following{/other_user}","gists_url":"https://api.github.com/users/dgarros/gists{/gist_id}","starred_url":"https://api.github.com/users/dgarros/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dgarros/subscriptions","organizations_url":"https://api.github.com/users/dgarros/orgs","repos_url":"https://api.github.com/users/dgarros/repos","events_url":"https://api.github.com/users/dgarros/events{/privacy}","received_events_url":"https://api.github.com/users/dgarros/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-04-08T22:53:26Z","updated_at":"2016-04-18T04:45:24Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/pulls/128","html_url":"https://github.com/Juniper/ansible-junos-stdlib/pull/128","diff_url":"https://github.com/Juniper/ansible-junos-stdlib/pull/128.diff","patch_url":"https://github.com/Juniper/ansible-junos-stdlib/pull/128.patch"},"body":"Hi \r\n\r\nI created a new module to execute ping on device\r\nI thought about reusing the rpc module but I thought this one is very specific and there's a value to have it as a standalone module\r\n\r\nIt's very useful to test L3 IP fabric connectivity\r\n\r\nDamien"},"comment":{"url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/comments/211199093","html_url":"https://github.com/Juniper/ansible-junos-stdlib/pull/128#issuecomment-211199093","issue_url":"https://api.github.com/repos/Juniper/ansible-junos-stdlib/issues/128","id":211199093,"user":{"login":"vnitinv","id":767016,"avatar_url":"https://avatars.githubusercontent.com/u/767016?v=3","gravatar_id":"","url":"https://api.github.com/users/vnitinv","html_url":"https://github.com/vnitinv","followers_url":"https://api.github.com/users/vnitinv/followers","following_url":"https://api.github.com/users/vnitinv/following{/other_user}","gists_url":"https://api.github.com/users/vnitinv/gists{/gist_id}","starred_url":"https://api.github.com/users/vnitinv/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vnitinv/subscriptions","organizations_url":"https://api.github.com/users/vnitinv/orgs","repos_url":"https://api.github.com/users/vnitinv/repos","events_url":"https://api.github.com/users/vnitinv/events{/privacy}","received_events_url":"https://api.github.com/users/vnitinv/received_events","type":"User","site_admin":false},"created_at":"2016-04-18T04:45:24Z","updated_at":"2016-04-18T04:45:24Z","body":"@dgarros Looks good to me. Just one feedback. Looks like you missed including rapid description in docstring. Can you please add that"}}