{"action":"created","issue":{"url":"https://api.github.com/repos/tas50/chef-nrpe/issues/27","labels_url":"https://api.github.com/repos/tas50/chef-nrpe/issues/27/labels{/name}","comments_url":"https://api.github.com/repos/tas50/chef-nrpe/issues/27/comments","events_url":"https://api.github.com/repos/tas50/chef-nrpe/issues/27/events","html_url":"https://github.com/tas50/chef-nrpe/issues/27","id":62327732,"number":27,"title":"node.set not needed in default recipe","user":{"login":"brugidou","id":167633,"avatar_url":"https://avatars.githubusercontent.com/u/167633?v=3","gravatar_id":"","url":"https://api.github.com/users/brugidou","html_url":"https://github.com/brugidou","followers_url":"https://api.github.com/users/brugidou/followers","following_url":"https://api.github.com/users/brugidou/following{/other_user}","gists_url":"https://api.github.com/users/brugidou/gists{/gist_id}","starred_url":"https://api.github.com/users/brugidou/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brugidou/subscriptions","organizations_url":"https://api.github.com/users/brugidou/orgs","repos_url":"https://api.github.com/users/brugidou/repos","events_url":"https://api.github.com/users/brugidou/events{/privacy}","received_events_url":"https://api.github.com/users/brugidou/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-17T08:31:03Z","updated_at":"2015-05-01T07:44:24Z","closed_at":null,"body":"There is no need to set the nrpe.checks attribute with the set/normal priority. Default priority should be enough and would not mess with the node's persistent data.\r\n\r\nSee https://github.com/tas50/chef-nrpe/blob/af28f203aaf35ac6e94931a14de6b89c534ded81/recipes/default.rb#L83\r\n\r\nJust to get a bit of context: we are managing nodes with git in our organization in a static way (since they are bare metal) and we have a script that check for synchronization b/t git and the chef server. Normal attributes are persistent data that we store in git as opposed to default/overrides which are generated during each chef run by roles/environments/cookbooks. Having normal attributes on a node mean that the cookbook requested to \"store\" some attribute across chef runs."},"comment":{"url":"https://api.github.com/repos/tas50/chef-nrpe/issues/comments/98068504","html_url":"https://github.com/tas50/chef-nrpe/issues/27#issuecomment-98068504","issue_url":"https://api.github.com/repos/tas50/chef-nrpe/issues/27","id":98068504,"user":{"login":"glensc","id":199095,"avatar_url":"https://avatars.githubusercontent.com/u/199095?v=3","gravatar_id":"","url":"https://api.github.com/users/glensc","html_url":"https://github.com/glensc","followers_url":"https://api.github.com/users/glensc/followers","following_url":"https://api.github.com/users/glensc/following{/other_user}","gists_url":"https://api.github.com/users/glensc/gists{/gist_id}","starred_url":"https://api.github.com/users/glensc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/glensc/subscriptions","organizations_url":"https://api.github.com/users/glensc/orgs","repos_url":"https://api.github.com/users/glensc/repos","events_url":"https://api.github.com/users/glensc/events{/privacy}","received_events_url":"https://api.github.com/users/glensc/received_events","type":"User","site_admin":false},"created_at":"2015-05-01T07:44:24Z","updated_at":"2015-05-01T07:44:24Z","body":"so, you propose to use `default`?\r\n\r\n```ruby\r\n    node.default['nrpe']['checks'] = checks\r\n```"}}