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

wrapper cookbook - node['rabbitmq']['deb_package_url] has not been overrided during execution phase #270

Closed
chrisduong opened this issue May 25, 2015 · 2 comments

Comments

Projects
None yet
1 participant
@chrisduong
Copy link

commented May 25, 2015

Hi,
In my wrapper cookbook, I define the attribute node['rabbitmq']['version] is '3.5.1' and I included the recipe rabbitmq::default in my cookbook.
However, during execution phase of rabbitmq::default. the node['rabbitmq']['deb_package_url] still see as default - 3.4.4-1

default['rabbitmq']['deb_package_url'] = "https://www.rabbitmq.com/releases/rabbitmq-server/v#{node['rabbitmq']['version']}/"

I don't know the purpose of this, why shouldn't we define deb_package_url is a simple variable, not a node's attribute. I believe this issue won't happen.

The only way to fix for my wrapper cookbook is to override the version in Chef Roles or to force with this code

# Hack to make sure node['rabbitmq']['version'] in recipe loaded inside dependency 'rabbitmq' cookbook

node.set['rabbitmq']['version'] = node['rabbitmq']['version']
node.save

@chrisduong

This comment has been minimized.

Copy link
Author

commented May 26, 2015

Hi,
Sorry for the noise. I haven't wrapped the cookbook correctly.
I should define needed overridden attributes in my wrapper cookbook

@chrisduong chrisduong closed this May 26, 2015

@ghost

This comment has been minimized.

Copy link

commented Feb 28, 2018

This issue was identified again in #457 and resolved in #458. If you're using rabbitmq cookbook 5.3.0 or later, changing the rabbitmq.version attribute will be sufficient to change the version of RabbitMQ that is installed without needing to override additional attributes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.