{"action":"created","comment":{"url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/comments/54312814","id":54312814,"diff_hunk":"@@ -28,12 +28,13 @@\n   service: name=jenkins state=started enabled=yes\n \n - name: Wait for Jenkins to start up before proceeding.\n-  shell: \"curl -D - --silent http://{{ jenkins_hostname }}:{{ jenkins_http_port }}{{ jenkins_url_prefix }}/cli/\"\n+  uri: url='http://{{ jenkins_hostname }}:{{ jenkins_http_port }}{{ jenkins_url_prefix }}/cli/' status_code=200\n   register: result\n-  until: (result.stdout.find(\"403 Forbidden\") != -1) or (result.stdout.find(\"200 OK\") != -1) and (result.stdout.find(\"Please wait while\") == -1)\n+  until: \"(result | success) and (result.status == 200)\"\n   retries: \"{{ jenkins_connection_retries }}\"\n   delay: \"{{ jenkins_connection_delay }}\"\n   changed_when: false\n+  no_log: True","path":"tasks/main.yml","position":12,"original_position":12,"commit_id":"4b02ee55b918deb3d63083cacb13f0051396b364","original_commit_id":"4b02ee55b918deb3d63083cacb13f0051396b364","user":{"login":"geerlingguy","id":481677,"avatar_url":"https://avatars.githubusercontent.com/u/481677?v=3","gravatar_id":"","url":"https://api.github.com/users/geerlingguy","html_url":"https://github.com/geerlingguy","followers_url":"https://api.github.com/users/geerlingguy/followers","following_url":"https://api.github.com/users/geerlingguy/following{/other_user}","gists_url":"https://api.github.com/users/geerlingguy/gists{/gist_id}","starred_url":"https://api.github.com/users/geerlingguy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geerlingguy/subscriptions","organizations_url":"https://api.github.com/users/geerlingguy/orgs","repos_url":"https://api.github.com/users/geerlingguy/repos","events_url":"https://api.github.com/users/geerlingguy/events{/privacy}","received_events_url":"https://api.github.com/users/geerlingguy/received_events","type":"User","site_admin":false},"body":"Huh... TIL; makes the playbook output much more succinct.","created_at":"2016-02-26T22:44:15Z","updated_at":"2016-02-26T22:44:15Z","html_url":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40#discussion_r54312814","pull_request_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40","_links":{"self":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/comments/54312814"},"html":{"href":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40#discussion_r54312814"},"pull_request":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40"}}},"pull_request":{"url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40","id":60769579,"html_url":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40","diff_url":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40.diff","patch_url":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40.patch","issue_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/40","number":40,"state":"open","locked":false,"title":"Refactor to prevent deprecation warnings.","user":{"login":"timstoop","id":729432,"avatar_url":"https://avatars.githubusercontent.com/u/729432?v=3","gravatar_id":"","url":"https://api.github.com/users/timstoop","html_url":"https://github.com/timstoop","followers_url":"https://api.github.com/users/timstoop/followers","following_url":"https://api.github.com/users/timstoop/following{/other_user}","gists_url":"https://api.github.com/users/timstoop/gists{/gist_id}","starred_url":"https://api.github.com/users/timstoop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timstoop/subscriptions","organizations_url":"https://api.github.com/users/timstoop/orgs","repos_url":"https://api.github.com/users/timstoop/repos","events_url":"https://api.github.com/users/timstoop/events{/privacy}","received_events_url":"https://api.github.com/users/timstoop/received_events","type":"User","site_admin":false},"body":"This fixes some deprecation warning and also no longer uses curl to check the status of Jenkins.","created_at":"2016-02-25T23:56:15Z","updated_at":"2016-02-26T22:44:15Z","closed_at":null,"merged_at":null,"merge_commit_sha":"198e34699a8f2d6bd3a94190dd1d22a81cabfc44","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40/commits","review_comments_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40/comments","review_comment_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/comments{/number}","comments_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/40/comments","statuses_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/statuses/4b02ee55b918deb3d63083cacb13f0051396b364","head":{"label":"timstoop:fix_deprecation_warnings","ref":"fix_deprecation_warnings","sha":"4b02ee55b918deb3d63083cacb13f0051396b364","user":{"login":"timstoop","id":729432,"avatar_url":"https://avatars.githubusercontent.com/u/729432?v=3","gravatar_id":"","url":"https://api.github.com/users/timstoop","html_url":"https://github.com/timstoop","followers_url":"https://api.github.com/users/timstoop/followers","following_url":"https://api.github.com/users/timstoop/following{/other_user}","gists_url":"https://api.github.com/users/timstoop/gists{/gist_id}","starred_url":"https://api.github.com/users/timstoop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timstoop/subscriptions","organizations_url":"https://api.github.com/users/timstoop/orgs","repos_url":"https://api.github.com/users/timstoop/repos","events_url":"https://api.github.com/users/timstoop/events{/privacy}","received_events_url":"https://api.github.com/users/timstoop/received_events","type":"User","site_admin":false},"repo":{"id":52561396,"name":"ansible-role-jenkins","full_name":"timstoop/ansible-role-jenkins","owner":{"login":"timstoop","id":729432,"avatar_url":"https://avatars.githubusercontent.com/u/729432?v=3","gravatar_id":"","url":"https://api.github.com/users/timstoop","html_url":"https://github.com/timstoop","followers_url":"https://api.github.com/users/timstoop/followers","following_url":"https://api.github.com/users/timstoop/following{/other_user}","gists_url":"https://api.github.com/users/timstoop/gists{/gist_id}","starred_url":"https://api.github.com/users/timstoop/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/timstoop/subscriptions","organizations_url":"https://api.github.com/users/timstoop/orgs","repos_url":"https://api.github.com/users/timstoop/repos","events_url":"https://api.github.com/users/timstoop/events{/privacy}","received_events_url":"https://api.github.com/users/timstoop/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/timstoop/ansible-role-jenkins","description":"Ansible Role - Jenkins CI","fork":true,"url":"https://api.github.com/repos/timstoop/ansible-role-jenkins","forks_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/forks","keys_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/keys{/key_id}","collaborators_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/teams","hooks_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/hooks","issue_events_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/issues/events{/number}","events_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/events","assignees_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/assignees{/user}","branches_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/branches{/branch}","tags_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/tags","blobs_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/git/refs{/sha}","trees_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/git/trees{/sha}","statuses_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/statuses/{sha}","languages_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/languages","stargazers_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/stargazers","contributors_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/contributors","subscribers_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/subscribers","subscription_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/subscription","commits_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/commits{/sha}","git_commits_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/git/commits{/sha}","comments_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/comments{/number}","issue_comment_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/issues/comments{/number}","contents_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/contents/{+path}","compare_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/compare/{base}...{head}","merges_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/merges","archive_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/downloads","issues_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/issues{/number}","pulls_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/pulls{/number}","milestones_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/milestones{/number}","notifications_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/labels{/name}","releases_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/releases{/id}","deployments_url":"https://api.github.com/repos/timstoop/ansible-role-jenkins/deployments","created_at":"2016-02-25T22:29:10Z","updated_at":"2016-02-25T22:24:06Z","pushed_at":"2016-02-25T23:55:04Z","git_url":"git://github.com/timstoop/ansible-role-jenkins.git","ssh_url":"git@github.com:timstoop/ansible-role-jenkins.git","clone_url":"https://github.com/timstoop/ansible-role-jenkins.git","svn_url":"https://github.com/timstoop/ansible-role-jenkins","homepage":"https://galaxy.ansible.com/geerlingguy/jenkins/","size":35,"stargazers_count":0,"watchers_count":0,"language":null,"has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"geerlingguy:master","ref":"master","sha":"dfb1824c016bcce1523ff733daa1b4dee3580867","user":{"login":"geerlingguy","id":481677,"avatar_url":"https://avatars.githubusercontent.com/u/481677?v=3","gravatar_id":"","url":"https://api.github.com/users/geerlingguy","html_url":"https://github.com/geerlingguy","followers_url":"https://api.github.com/users/geerlingguy/followers","following_url":"https://api.github.com/users/geerlingguy/following{/other_user}","gists_url":"https://api.github.com/users/geerlingguy/gists{/gist_id}","starred_url":"https://api.github.com/users/geerlingguy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geerlingguy/subscriptions","organizations_url":"https://api.github.com/users/geerlingguy/orgs","repos_url":"https://api.github.com/users/geerlingguy/repos","events_url":"https://api.github.com/users/geerlingguy/events{/privacy}","received_events_url":"https://api.github.com/users/geerlingguy/received_events","type":"User","site_admin":false},"repo":{"id":17307266,"name":"ansible-role-jenkins","full_name":"geerlingguy/ansible-role-jenkins","owner":{"login":"geerlingguy","id":481677,"avatar_url":"https://avatars.githubusercontent.com/u/481677?v=3","gravatar_id":"","url":"https://api.github.com/users/geerlingguy","html_url":"https://github.com/geerlingguy","followers_url":"https://api.github.com/users/geerlingguy/followers","following_url":"https://api.github.com/users/geerlingguy/following{/other_user}","gists_url":"https://api.github.com/users/geerlingguy/gists{/gist_id}","starred_url":"https://api.github.com/users/geerlingguy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/geerlingguy/subscriptions","organizations_url":"https://api.github.com/users/geerlingguy/orgs","repos_url":"https://api.github.com/users/geerlingguy/repos","events_url":"https://api.github.com/users/geerlingguy/events{/privacy}","received_events_url":"https://api.github.com/users/geerlingguy/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/geerlingguy/ansible-role-jenkins","description":"Ansible Role - Jenkins CI","fork":false,"url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins","forks_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/forks","keys_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/keys{/key_id}","collaborators_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/teams","hooks_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/hooks","issue_events_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/events{/number}","events_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/events","assignees_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/assignees{/user}","branches_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/branches{/branch}","tags_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/tags","blobs_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/git/refs{/sha}","trees_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/git/trees{/sha}","statuses_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/statuses/{sha}","languages_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/languages","stargazers_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/stargazers","contributors_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/contributors","subscribers_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/subscribers","subscription_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/subscription","commits_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/commits{/sha}","git_commits_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/git/commits{/sha}","comments_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/comments{/number}","issue_comment_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/comments{/number}","contents_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/contents/{+path}","compare_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/compare/{base}...{head}","merges_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/merges","archive_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/downloads","issues_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues{/number}","pulls_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls{/number}","milestones_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/milestones{/number}","notifications_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/labels{/name}","releases_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/releases{/id}","deployments_url":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/deployments","created_at":"2014-03-01T03:58:08Z","updated_at":"2016-02-26T17:33:53Z","pushed_at":"2016-02-25T23:56:17Z","git_url":"git://github.com/geerlingguy/ansible-role-jenkins.git","ssh_url":"git@github.com:geerlingguy/ansible-role-jenkins.git","clone_url":"https://github.com/geerlingguy/ansible-role-jenkins.git","svn_url":"https://github.com/geerlingguy/ansible-role-jenkins","homepage":"https://galaxy.ansible.com/geerlingguy/jenkins/","size":57,"stargazers_count":90,"watchers_count":90,"language":null,"has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":69,"mirror_url":null,"open_issues_count":7,"forks":69,"open_issues":7,"watchers":90,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40"},"html":{"href":"https://github.com/geerlingguy/ansible-role-jenkins/pull/40"},"issue":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/40"},"comments":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/issues/40/comments"},"review_comments":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40/comments"},"review_comment":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/pulls/40/commits"},"statuses":{"href":"https://api.github.com/repos/geerlingguy/ansible-role-jenkins/statuses/4b02ee55b918deb3d63083cacb13f0051396b364"}}}}