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

Zabbix 6.0 update (#2) #16

Merged
merged 3 commits into from
Nov 18, 2023
Merged

Conversation

kodiakastu
Copy link
Contributor

Updated template to zabbix 6.4, updated readme to reflect changes and now includes modified default template "Generic by SNMP" to resove #15.

* Update readme.md

Corrected a typo and upated compatibility date.

* Update zbx_edgemax_template.xml

Now includes liked template (Generic by SNMP) and the graphs

* Update readme.md

Added comments about now including the modified generic by snmp
@andymarden
Copy link

Can confirm this works on Zabbix 6.4 for Edgerouter 12p target

@kodiakastu
Copy link
Contributor Author

Thanks I will update the readme!

Added er-12p in the confirmed compatible list
@albin-lindstrom
Copy link
Owner

Resolves #15.

@notfixingit3
Copy link

Confirmed working on ER-4 and ER-Pro

@kodiakastu
Copy link
Contributor Author

kodiakastu commented Nov 18, 2023

@albin-lindstrom Why was i closed and not merged ?
@notfixingit3 Will update the readme with this on the next cycle :)

Added more confirmed compatible models
@albin-lindstrom
Copy link
Owner

@albin-lindstrom Why was i closed and not merged ?
@notfixingit3 Will update the readme with this on the next cycle :)

Sorry my bad, my tipsy ass hit the wrong button yesterday.

@albin-lindstrom albin-lindstrom merged commit fc0834d into albin-lindstrom:master Nov 18, 2023
@kodiakastu
Copy link
Contributor Author

@albin-lindstrom Why was i closed and not merged ?
@notfixingit3 Will update the readme with this on the next cycle :)

Sorry my bad, my tipsy ass hit the wrong button yesterday.

Haha no problems! Was just wondering 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants