Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

[cetic/zabbix] Support for zabbix 5 #13

Closed
xeor opened this issue Sep 3, 2020 · 2 comments · Fixed by #14
Closed

[cetic/zabbix] Support for zabbix 5 #13

xeor opened this issue Sep 3, 2020 · 2 comments · Fixed by #14
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@xeor
Copy link

xeor commented Sep 3, 2020

Is your feature request related to a problem? Please describe.
No way to run zabbix 5 on current helm chart

Describe the solution you'd like
Either updated chart to zabbix 5, or a way to configure the chart so it supports zabbix 5 container.

I got zabbix 5 working by changing the container ports (https://github.com/cetic/helm-zabbix/blob/master/templates/Web-deplyment.yaml#L50-L61) to 8080, and updating the images to one with zabbix 5, example :centos-5.0-latest. However, the port they use is 8080 not 80 which is hardcoded in the current chart.

Describe alternatives you've considered
Forking the chart making the small changes needed to run zabbix 5
@banzo banzo added enhancement New feature or request help wanted Extra attention is needed labels Sep 24, 2020
@aeciopires aeciopires mentioned this issue Oct 2, 2020
Merged
3 tasks
@aeciopires
Copy link
Collaborator

Hi @xeor!

This problem is corrected in this Pull Request #14

@alexnuttinck
Copy link
Contributor

The new release is now available. I close this issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Us aecio 3
4 participants
@xeor @banzo @aeciopires @alexnuttinck