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

Re-add support for specifying package version in package_ensure #307

Merged
merged 1 commit into from
Jan 12, 2018

Conversation

antaflos
Copy link

Restricting haproxy::package_ensure to one of 'present', 'absent' or
'latest' prevents the user from letting Puppet install a specific
version of the HAProxy package. This restriction is unwise. It is also
an unnecessary breaking change from version 1.5.0.

This commit replaces the Enum data type for haproxy::package_ensure
and haproxy::instance::package_ensure with the String data type, adds
spec tests for setting a specific package version and updates the
documentation.

Restricting haproxy::package_ensure to one of 'present', 'absent' or
'latest' prevents the user from letting Puppet install a specific
version of the HAProxy package. This restriction is unwise. It is also
an unnecessary breaking change from version 1.5.0.

This commit replaces the Enum data type for haproxy::package_ensure
and haproxy::instance::package_ensure with the String data type, adds
spec tests for setting a specific package version and updates the
documentation.
@pmcmaw
Copy link

pmcmaw commented Jan 8, 2018

Closing and reopening to resolve Travis/bundler issues, this is caused due to https://tickets.puppetlabs.com/browse/MODULES-6339.

@pmcmaw pmcmaw closed this Jan 8, 2018
@pmcmaw pmcmaw reopened this Jan 8, 2018
@hunner hunner merged commit 2adfbd1 into puppetlabs:master Jan 12, 2018
hunner added a commit that referenced this pull request Jan 12, 2018
Re-add support for specifying package version in package_ensure
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants