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

Can't use the cookbook with Chef 11 because of firewall cookbook #392

Closed
barrowkwan opened this issue Jan 9, 2017 · 4 comments · Fixed by #395
Closed

Can't use the cookbook with Chef 11 because of firewall cookbook #392

barrowkwan opened this issue Jan 9, 2017 · 4 comments · Fixed by #395

Comments

@barrowkwan
Copy link

Hi,
I am trying to use the cookbook Chef11 but got error with firewall cookbook because firewall cookbook 2.0.0 and above can only be used with Chef 12. I wonder if consul 2.0 and above has to use firewall cookbook 2.0.0 or can use firewall cookbook with an earlier version?

@legal90
Copy link
Contributor

legal90 commented Jan 10, 2017

IMO, I would remove "firewall" dependency from this cookbook (and the related block from the "default" recipe). That is an important thing, but is really specific and depends on the platform and the concrete use case, so it's better to handle it in a wrapper cookbook.

@johnbellone, what do you think about this?

@johnbellone
Copy link
Contributor

I agree with @legal90 - this should be managed inside of a wrapper such as consul-cluster cookbook. But I want to stress that outside of this, @barrowkwan should note that Chef 11 is not supported by this cookbook. It may work, but that client version is very old.

@johnbellone
Copy link
Contributor

This also constitutes a major version change.

@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants