Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pin chef server api version to 0, fixes #56 #57

Merged
merged 1 commit into from
Aug 5, 2015
Merged

Conversation

sersut
Copy link

@sersut sersut commented Aug 5, 2015

@sersut
Copy link
Author

sersut commented Aug 5, 2015

Also tested this in the context of #56 and looks like it works.

@tyler-ball tyler-ball added Bug and removed Developing labels Aug 5, 2015
@tyler-ball tyler-ball changed the title Pin chef server api version to 0. Pin chef server api version to 0, fixes #56 Aug 5, 2015
@tyler-ball
Copy link
Contributor

Verified locally! Thanks so much @sersut

tyler-ball added a commit that referenced this pull request Aug 5, 2015
Pin chef server api version to 0, fixes #56
@tyler-ball tyler-ball merged commit 087f683 into master Aug 5, 2015
@tyler-ball tyler-ball deleted the sersut/pin-v0 branch August 5, 2015 22:57
@tyler-ball tyler-ball added this to the 1.3.1 milestone Aug 5, 2015
@aleksey-hariton
Copy link

Guys, this bugfix brokes chef-provisioning.

  • cheffish 1.6.0
  • chef-provisioning 1.5.0
  • chef-server 12.1.2
  • chef-client 12.3
  machine_batch("Queue 6") do
    action [:converge]
    driver "fog:OpenStack"
    chef_server {:chef_server_url=>"https://chefserver//organizations/ORG", :options=>{:client_name=>"haritona", :signing_key_filename=>"client.pem"}}
    declared_type :machine_batch
    machines ["frontend01"]
  end

Raises error:

Chef::Exceptions::ImmutableAttributeModification: 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"'

Stack trace:

...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/node/immutable_collections.rb:165:in `block (2 levels) in <class:ImmutableMash>'
...gems/ruby-2.2.1/gems/cheffish-1.6.0/lib/cheffish.rb:23:in `chef_server_api' <<<ROOT CAUSE
...gems/ruby-2.2.1/gems/cheffish-1.6.0/lib/cheffish/chef_provider_base.rb:8:in `rest'
...gems/ruby-2.2.1/gems/cheffish-1.6.0/lib/chef/provider/chef_node.rb:42:in `load_current_resource'
...gems/ruby-2.2.1/bundler/gems/chef-provisioning-65f644dffe38/lib/chef/provider/machine.rb:179:in `load_current_resource'
...gems/ruby-2.2.1/bundler/gems/chef-provisioning-65f644dffe38/lib/chef/provider/machine_batch.rb:167:in `block in load_current_resource'
...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/chef_fs/parallelizer/parallel_enumerable.rb:267:in `call'
...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/chef_fs/parallelizer/parallel_enumerable.rb:267:in `process_input'
...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/chef_fs/parallelizer/parallel_enumerable.rb:257:in `process_one'
...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/chef_fs/parallelizer.rb:93:in `call'
...gems/ruby-2.2.1/gems/chef-12.3.0/lib/chef/chef_fs/parallelizer.rb:93:in `worker_loop'

If comment this 087f683#diff-a70f0dce71a85be109b4f0cca2d8d14fR22 everything works fine.

@thommay thommay added Type: Bug Does not work as expected. and removed Bug labels Jan 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Bug Does not work as expected.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants