Can't install Zabbix server from source when ['zabbix']['server']['configure_options'] is not empty. about zabbix

[andrewGarson]

@AdallomRoy what was the issue you saw?

I can install from source with
node[:zabbix][:server][:configure_options] = ["--with-libcurl", "--with-net-snmp"] without losing those options.

[AdallomRoy]

Hi,

This could be since I'm using chef client 11.4.2 (have you tested it
against this version?)

The error I'm getting is "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"')" ...
This happens due to this line:
configure_options = (node['zabbix']['server']['configure_options'] || Array.
new).delete_if do |option|

If the configure_options are not empty, it will put the node object as the
configure options -> but node objects are immutable, which will cause an
error (either when attempting to delete, or later when attempting to append).

Cheers.

[andrewGarson]

I see. I'm still using chef 10 at this point. So the problem is trying to persist it back to the chef-server. Maybe we don't even need to do that given that we are only removing "--prefix", we can do that each time.

@laradji When server_source runs (agent_source will have this problem too) the "--prefix" option gets removed from the options array and then we try to persist it back to the chef-server. We can either update the code to persist properly for Chef 10 and Chef 11 or just not persist it back. The check has to happen on each run anyway. What do you think?

[AdallomRoy]

It's a bit more problematic than that, because the new code uses configure_options to add other parameters later (e.g. mysql stuff) so even if you remove that delete_if, you will still have trouble..

[andrewGarson]

good point. So we need to change all of the node.set calls (and the delete_if) to set at a particular precedence level.

[laradji]

Hello guys,

maybe we can remove the cleaning of prefix with delete_if. it was put to help migration from older version of the cookbook prior 0.0.18.