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
move zabbixapi gem installation into own class #903
Conversation
'Debian' => ['make', 'g++', 'ruby-dev', 'ruby', 'pkg-config',], | ||
default => [], | ||
} | ||
ensure_packages($compile_packages) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that is deprecated, please use the newer version:
ensure_packages($compile_packages) | |
stdlib::ensure_packages($compile_packages) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can I use this now w/o bumping stdlib to >= 9?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think so
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then I'd prefer not to bump stdlib to >= 9 just yet.
# Determine correct zabbixapi version. | ||
case $zabbix_version { | ||
/^[56]\.[024]/: { | ||
$zabbixapi_version = '5.0.0-alpha1' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note to self: https://rubygems.org/gems/zabbixapi does only list a 5.0.0-alpha1 from 2021.
$zabbix_version = $zabbix::params::zabbix_version, | ||
$puppetgem = $zabbix::params::puppetgem, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this be String[1]
for both?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In theory, yes.
In reality, we do not enforce these anywhere else, so I find it odd to do it here.
(read: we should do it globally, but not in this PR)
Pull Request (PR) description
This Pull Request (PR) fixes the following issues