-
-
Notifications
You must be signed in to change notification settings - Fork 227
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
Add/rename Linux SNMP templates for Zabbix 6.0+ #855
Add/rename Linux SNMP templates for Zabbix 6.0+ #855
Conversation
But I have no idea how to fix the archlinux tests:
Does someone know anything about this cryptic error? Should we simply disable the archlinux tests suite? |
3b4495c
to
e271d58
Compare
No need to address the archlinux test failure in this PR. That is happening on all of our modules and needs to be fixed in the files that are shared by all of the modules. |
test: snmp renamed only on 6.0 test: linter
e271d58
to
87abb0d
Compare
Co-authored-by: Romain Tartière <[email protected]>
This looks good, I will merge it unless CI detect some kind of breakage. I think it is much more of an enhancement for bringing support for newer versions of Zabbix than a bugfix, so maybe we can change the PR title to something like "Add Linux SNMP templates for Zabbix 6.0+" which will have move value when read in the changelog without context? |
It was a change from upstream in 6.0.11, it was still named |
well... Is zabbix 6 still supported on centos7?
|
Packages seems to exist. Maybe a transient failure? |
Great, we will assume the failure was due to the repo being updated while the test suite was running 🤷 |
Thanks @smortex for all the help to merge my stuff! |
Pull Request (PR) description
As mentionned in #851, the SNMP templates has been renamed upstream, this is an attempt to fix the tests
This Pull Request (PR) fixes the following issues