Skip to content

WIP: Add support for package install options - #164

Open
yorickps wants to merge 1 commit into
voxpupuli:masterfrom
yorickps:install_options
Open

WIP: Add support for package install options#164
yorickps wants to merge 1 commit into
voxpupuli:masterfrom
yorickps:install_options

Conversation

@yorickps

@yorickps yorickps commented Aug 4, 2021

Copy link
Copy Markdown

Pull Request (PR) description

Add support for package install options.
See puppet package resource for more info:
https://puppet.com/docs/puppet/7/types/package.html#package-attribute-install_options

Comment thread manifests/install.pp Outdated
@yorickps

yorickps commented Aug 4, 2021

Copy link
Copy Markdown
Author

@kenyon thanks for the review, fixed the accidental mistake.

kenyon
kenyon previously requested changes Aug 4, 2021

@kenyon kenyon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you also add a test that uses this new parameter? https://github.com/voxpupuli/puppet-fail2ban/blob/master/spec/classes/init_spec.rb
Thanks!

Comment thread manifests/init.pp Outdated
@bastelfreak bastelfreak added the enhancement New feature or request label Aug 5, 2021
@yorickps
yorickps force-pushed the install_options branch 2 times, most recently from fa27150 to 5b5527f Compare August 5, 2021 16:03
@kenyon
kenyon self-requested a review August 5, 2021 17:59
@yorickps yorickps changed the title Add support for package install options WIP: Add support for package install options Aug 5, 2021
See puppet package resource for more info
<https://puppet.com/docs/puppet/7/types/package.html#package-attribute-install_options>

Better define type

Set to undef

Add spec test for package_install_options
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request needs-tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants