{"action":"created","issue":{"url":"https://api.github.com/repos/chef/chef-provisioning/issues/382","labels_url":"https://api.github.com/repos/chef/chef-provisioning/issues/382/labels{/name}","comments_url":"https://api.github.com/repos/chef/chef-provisioning/issues/382/comments","events_url":"https://api.github.com/repos/chef/chef-provisioning/issues/382/events","html_url":"https://github.com/chef/chef-provisioning/pull/382","id":92748647,"number":382,"title":"Create client.rb only if missing. Fixes #318","user":{"login":"dblessing","id":2394772,"avatar_url":"https://avatars.githubusercontent.com/u/2394772?v=3","gravatar_id":"","url":"https://api.github.com/users/dblessing","html_url":"https://github.com/dblessing","followers_url":"https://api.github.com/users/dblessing/followers","following_url":"https://api.github.com/users/dblessing/following{/other_user}","gists_url":"https://api.github.com/users/dblessing/gists{/gist_id}","starred_url":"https://api.github.com/users/dblessing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dblessing/subscriptions","organizations_url":"https://api.github.com/users/dblessing/orgs","repos_url":"https://api.github.com/users/dblessing/repos","events_url":"https://api.github.com/users/dblessing/events{/privacy}","received_events_url":"https://api.github.com/users/dblessing/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chef/chef-provisioning/labels/Developing","name":"Developing","color":"ededed"},{"url":"https://api.github.com/repos/chef/chef-provisioning/labels/Improvement","name":"Improvement","color":"009800"},{"url":"https://api.github.com/repos/chef/chef-provisioning/labels/Signed%20CLA","name":"Signed CLA","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/chef/chef-provisioning/milestones/5","html_url":"https://github.com/chef/chef-provisioning/milestones/1.3.0","labels_url":"https://api.github.com/repos/chef/chef-provisioning/milestones/5/labels","id":1167525,"number":5,"title":"1.3.0","description":"","creator":{"login":"tyler-ball","id":2481463,"avatar_url":"https://avatars.githubusercontent.com/u/2481463?v=3","gravatar_id":"","url":"https://api.github.com/users/tyler-ball","html_url":"https://github.com/tyler-ball","followers_url":"https://api.github.com/users/tyler-ball/followers","following_url":"https://api.github.com/users/tyler-ball/following{/other_user}","gists_url":"https://api.github.com/users/tyler-ball/gists{/gist_id}","starred_url":"https://api.github.com/users/tyler-ball/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tyler-ball/subscriptions","organizations_url":"https://api.github.com/users/tyler-ball/orgs","repos_url":"https://api.github.com/users/tyler-ball/repos","events_url":"https://api.github.com/users/tyler-ball/events{/privacy}","received_events_url":"https://api.github.com/users/tyler-ball/received_events","type":"User","site_admin":false},"open_issues":15,"closed_issues":2,"state":"open","created_at":"2015-06-16T20:54:15Z","updated_at":"2015-07-23T14:49:34Z","due_on":null,"closed_at":null},"comments":6,"created_at":"2015-07-02T22:26:30Z","updated_at":"2015-07-25T15:09:49Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chef/chef-provisioning/pulls/382","html_url":"https://github.com/chef/chef-provisioning/pull/382","diff_url":"https://github.com/chef/chef-provisioning/pull/382.diff","patch_url":"https://github.com/chef/chef-provisioning/pull/382.patch"},"body":"This seems like a straightforward fix for #318. Previously, the `client.rb` file checksum was compared on each `chef-provisioning` run and if it didn't match it was updated. This caused a fight if the node was also running `chef-client` daemon - the daemon might change the client.rb file contents which makes `chef-provisioning` unhappy. I really believe that `chef-provisioning` should only manage the `client.rb` on the first run."},"comment":{"url":"https://api.github.com/repos/chef/chef-provisioning/issues/comments/124852703","html_url":"https://github.com/chef/chef-provisioning/pull/382#issuecomment-124852703","issue_url":"https://api.github.com/repos/chef/chef-provisioning/issues/382","id":124852703,"user":{"login":"dblessing","id":2394772,"avatar_url":"https://avatars.githubusercontent.com/u/2394772?v=3","gravatar_id":"","url":"https://api.github.com/users/dblessing","html_url":"https://github.com/dblessing","followers_url":"https://api.github.com/users/dblessing/followers","following_url":"https://api.github.com/users/dblessing/following{/other_user}","gists_url":"https://api.github.com/users/dblessing/gists{/gist_id}","starred_url":"https://api.github.com/users/dblessing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dblessing/subscriptions","organizations_url":"https://api.github.com/users/dblessing/orgs","repos_url":"https://api.github.com/users/dblessing/repos","events_url":"https://api.github.com/users/dblessing/events{/privacy}","received_events_url":"https://api.github.com/users/dblessing/received_events","type":"User","site_admin":false},"created_at":"2015-07-25T15:09:49Z","updated_at":"2015-07-25T15:09:49Z","body":"We tried the latter method and couldn't quite get it. The first method you mention may work, though. If specifying client.rb using the file attribute will provisioning fight itself because the md5 won't match what it expects?"}}