-
Notifications
You must be signed in to change notification settings - Fork 56
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
[zabbix-community/zabbix] wrong value of "StartJavaPollers" in config file #145
Comments
There was a recent change in the docker images of Zabbix as of 7.2: the config files of Zabbix now support using environment variables inside them, natively and the guys maintaining the container images have changed the structure of the config file and the docker-entrypoint.sh accordingly and just recently: now the docker-entrypoint.sh does not try to set all the config variables in the file (/etc/zabbix/zabbix_server.conf) by the according env variables, and the config file of zabbix server is split into several configuration files. Nevertheless, I can not reproduce the error you reported. As in such case it would indeed be a bug in the zabbix docker image, documenting that it is not necessary to supply
...and Zabbix Server starts just nicely, without supplying the above mentioned variable set to "0". May it be there have been a temporary broken image provided by Zabbix? Can you confirm that it's not (anymore?) a problem? |
OK, I have to apologize! Actually, there is a bug in the Helm Chart causing to render the deployment manifest this way:
which causes the error reported by you. I am working on a bigger Pull Request right now and I will fix this issue there. |
Describe the bug
When
zabbixJavaGateway
is set toenabled: false
, the env var- name: ZBX_STARTJAVAPOLLERS
has no value, and creates an errorwrong value of "StartJavaPollers" in config file
Version of Helm and Kubernetes:
helm v3.16.3
k3s v1.30.6+k3s1
Anything else we need to know:
zabbixImageTag: ubuntu-7.2-latest
was set in zabbix-values.yamlAdding for example
in the deployment works around the issue
The text was updated successfully, but these errors were encountered: