-
-
Notifications
You must be signed in to change notification settings - Fork 228
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
Support for zabbix::agent on AIX #665
Conversation
try and specify package provider for aix try and better aix fix service owner Define package_provider everywhere To fix failing tests Support for zabbix::agent on AIX This initial pass offers support for installing and managing the Zabbix agent on AIX systems. It has been tested against AIX 7.1 and 7.2. No support is provided for managing a repo; package installation support depends on a `yum` repo being available on the machine.
OK, I think the 2 failing tests are unrelated to work I was doing, so I'm marking this as ready for review. |
Dear @bjschafer, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
1 similar comment
Dear @bjschafer, thanks for the PR! This is pccibot, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase? You can find my sourcecode at voxpupuli/vox-pupuli-tasks |
* foo/master: (30 commits) Drop ruby < 2.3 fakefs gem dependency Change database collation to utf8_bin as it is stated in official docs Install docs on CentOS acceptance tests zabbix::web: Allow httpd to speak to the database fix acceptance test ordering use master branch from rspec-puppet-facts Cleanup acceptance test helper modulesync 2.12.0 CentOS tests: Purge whole yum cache switch to improved rspec-puppet-facts version Add Debian 10 support Improve `zabbix_proxy` type Improve zabbix_proxy acceptance tests Cleanup acceptance test helper Changes suggested in PR review Fix zbx_macros datatype Quotes and commas are unnecessary with %w Refactoring of macro implementation Add $zbx_macros datatype Add support for defining host's macros ...
@bjschafer thanks for the work! can you please add the os to the metadata.json? |
@bastelfreak done -- it looks like in other voxpupuli repos no version is specified with AIX. Is that acceptable here as well? |
Dear @bjschafer, thanks for the PR! This is Vox Pupuli Tasks, your friendly Vox Pupuli Github Bot. I noticed that your pull request has CI failures. Can you please have a look at the failing CI jobs? |
Pull Request (PR) description
This initial pass offers support for installing and managing the Zabbix
agent on AIX systems. It has been tested against AIX 7.1 and 7.2.
No support is provided for managing a repo; package installation support
depends on a
yum
repo being available on the machine.This Pull Request (PR) fixes the following issues
Fixes #504