{"action":"created","issue":{"url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/53","repository_url":"https://api.github.com/repos/bloomberg/chef-bcs","labels_url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/53/labels{/name}","comments_url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/53/comments","events_url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/53/events","html_url":"https://github.com/bloomberg/chef-bcs/issues/53","id":143991943,"number":53,"title":"Build fails after merging #52","user":{"login":"kelvk","id":9847174,"avatar_url":"https://avatars.githubusercontent.com/u/9847174?v=3","gravatar_id":"","url":"https://api.github.com/users/kelvk","html_url":"https://github.com/kelvk","followers_url":"https://api.github.com/users/kelvk/followers","following_url":"https://api.github.com/users/kelvk/following{/other_user}","gists_url":"https://api.github.com/users/kelvk/gists{/gist_id}","starred_url":"https://api.github.com/users/kelvk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/kelvk/subscriptions","organizations_url":"https://api.github.com/users/kelvk/orgs","repos_url":"https://api.github.com/users/kelvk/repos","events_url":"https://api.github.com/users/kelvk/events{/privacy}","received_events_url":"https://api.github.com/users/kelvk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2016-03-28T14:38:02Z","updated_at":"2016-03-28T15:15:00Z","closed_at":null,"body":"Rebased and tried building, but hit below error.\r\n```\r\n/var/chef/cache/cookbooks/chef-bcs/libraries/utils.rb:306: warning: found = in conditional, should be ==\r\nRecipe: ceph-chef::default\r\n  * chef_gem[netaddr] action install (up to date)\r\n[2016-03-28T14:33:12+00:00] WARN: Cloning resource attributes for chef_gem[netaddr] from prior resource (CHEF-3694)\r\n[2016-03-28T14:33:12+00:00] WARN: Previous chef_gem[netaddr]: /var/chef/cache/cookbooks/ceph-chef/recipes/default.rb:36:in `from_file'\r\n[2016-03-28T14:33:12+00:00] WARN: Current  chef_gem[netaddr]: /var/chef/cache/cookbooks/ceph-chef/recipes/default.rb:43:in `from_file'\r\n  * chef_gem[netaddr] action install (skipped due to not_if)\r\n  \r\n  ================================================================================\r\n  Recipe Compile Error in /var/chef/cache/cookbooks/ceph-chef/recipes/mon.rb\r\n  ================================================================================\r\n  \r\n  Chef::Exceptions::ImmutableAttributeModification\r\n  ------------------------------------------------\r\n  Node attributes are read-only when you do not specify which precedence level to set. To set an attribute use code like `node.default[\"key\"] = \"value\"'\r\n  \r\n  Cookbook Trace:\r\n  ---------------\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:32:in `block (4 levels) in ceph_chef_build_federated_pool'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:28:in `each'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:28:in `block (3 levels) in ceph_chef_build_federated_pool'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:27:in `each'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:27:in `block (2 levels) in ceph_chef_build_federated_pool'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:26:in `each'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:26:in `block in ceph_chef_build_federated_pool'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:25:in `each'\r\n    /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:25:in `ceph_chef_build_federated_pool'\r\n    /var/chef/cache/cookbooks/ceph-chef/recipes/default.rb:50:in `from_file'\r\n    /var/chef/cache/cookbooks/ceph-chef/recipes/mon.rb:43:in `from_file'\r\n  \r\n  Relevant File Content:\r\n  ----------------------\r\n  /var/chef/cache/cookbooks/ceph-chef/libraries/ceph_chef_helper.rb:\r\n  \r\n   25:    node['ceph']['pools'][pool]['federated_regions'].each do |region|\r\n   26:      node['ceph']['pools'][pool]['federated_zones'].each do |zone|\r\n   27:        node['ceph']['pools'][pool]['federated_instances'].each do |instance|\r\n   28:          node['ceph']['pools'][pool]['names'].each do |name|\r\n   29:            # name should already have '.' as first character so don't add it to formating here\r\n   30:            federated_name = \".#{region}-#{zone}-#{instance}#{name}\"\r\n   31:            if !node['ceph']['pools'][pool]['federated_names'].include? federated_name\r\n   32>>             node['ceph']['pools'][pool]['federated_names'] << federated_name\r\n   33:            end\r\n   34:          end\r\n   35:        end\r\n   36:      end\r\n   37:    end\r\n   38:  end\r\n   39:  \r\n   40:  def ceph_chef_create_pool(pool)\r\n   41:    if !node['ceph']['pools'][pool]['federated_names'].empty?\r\n```"},"comment":{"url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/comments/202432946","html_url":"https://github.com/bloomberg/chef-bcs/issues/53#issuecomment-202432946","issue_url":"https://api.github.com/repos/bloomberg/chef-bcs/issues/53","id":202432946,"user":{"login":"cloudm2","id":548936,"avatar_url":"https://avatars.githubusercontent.com/u/548936?v=3","gravatar_id":"","url":"https://api.github.com/users/cloudm2","html_url":"https://github.com/cloudm2","followers_url":"https://api.github.com/users/cloudm2/followers","following_url":"https://api.github.com/users/cloudm2/following{/other_user}","gists_url":"https://api.github.com/users/cloudm2/gists{/gist_id}","starred_url":"https://api.github.com/users/cloudm2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cloudm2/subscriptions","organizations_url":"https://api.github.com/users/cloudm2/orgs","repos_url":"https://api.github.com/users/cloudm2/repos","events_url":"https://api.github.com/users/cloudm2/events{/privacy}","received_events_url":"https://api.github.com/users/cloudm2/received_events","type":"User","site_admin":false},"created_at":"2016-03-28T15:15:00Z","updated_at":"2016-03-28T15:15:00Z","body":"The following can be added to your version if you like but I will be pushing a new one out later that has it in there. Only one new line (first one) but I included the whole block for reference:\r\n\r\n# Most important cookbook\r\n# Remove it from cache for now since making so many changes to it. Remove this line later once ceph-chef is final.\r\nrm -f $BOOTSTRAP_CACHE_DIR/cookbooks/ceph-chef-*\r\n# If set then it will not download but remove from cache and use the development version that should be set in the chef-bcs/cookbooks directory.\r\nif [[ $CHEF_BCS_DEBUG -eq 0 ]]; then\r\n  download_file cookbooks/ceph-chef-0.9.17.tar.gz http://cookbooks.opscode.com/api/v1/cookbooks/ceph-chef/versions/0.9.17/download\r\nelse\r\n  # Remove it so it's not used.\r\n  rm -f $BOOTSTRAP_CACHE_DIR/cookbooks/ceph-chef-*\r\nfi\r\n"}}