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

When only listen_ports are defined, the merging in the helper library fails #459

Closed
daften opened this issue Dec 31, 2016 · 4 comments
Closed

Comments

@daften
Copy link

daften commented Dec 31, 2016

Cookbook version

3.2.2

Chef-client version

irrelevant

Platform Details

Ubuntu 14.04 but irrelevant

Scenario:

Still use the cookbook with the old listen_ports (legacy that was in place)

Steps to Reproduce:

Run with only listen_ports defined and not listen_addresses

Expected Result:

It should merge the old attributes and convert to the new scheme

Actual Result:

It throws an error because I don't have listen_addresses. The uniq on line 37 in listen.rb fails node['apache']['listen_addresses'].uniq since node['apache']['listen_addresses'] is a nil class.

@laradji
Copy link

laradji commented Feb 9, 2017

+1

@hrak
Copy link
Contributor

hrak commented Feb 24, 2017

Appears to be fixed in #422 but that change has not made it into a new release yet. @svanzoest could you please cut a new release? (master includes some neat changes & fixes)

@svanzoest
Copy link
Contributor

Released https://github.com/sous-chefs/apache2/releases/tag/v3.3.0

@lock
Copy link

lock bot commented Jul 24, 2018

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 Jul 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants