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

(MODULES-10979) Release prep for 4.5.0 #547

Merged
merged 2 commits into from Mar 23, 2021

Conversation

GabrielNagy
Copy link
Contributor

Update supported platforms in metadata.json, bump version and add changelog.

Update supported platforms in metadata.json, bump version and add
changelog.
PDK 2 has been released which now accepts Facter 4, so we can use it
now.
@GabrielNagy GabrielNagy requested a review from a team March 23, 2021 10:27
@@ -118,7 +116,7 @@
"requirements": [
{
"name": "puppet",
"version_requirement": ">= 4.7.0 < 8.0.0"
"version_requirement": ">= 6.0.0 < 8.0.0"
Copy link
Contributor

@gimmyxd gimmyxd Mar 23, 2021

Choose a reason for hiding this comment

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

We should still let people upgrade their older nodes using this module. So I think we should not update this. Or if we want to drop compatibility, it should be a major release.

I guess this also applies for older platforms, we should verify how people can check what version of puppet-agent module they can use to upgrade and if forge provides the needed information.

Never mind, users would still be able to install it on unsupported platforms/puppet versions but we can't guarantee that it would work since we're not testing on those. I think it is good that metadata.json contains only the supported platforms and puppet versions.

On that note, https://github.com/puppetlabs/puppetlabs-puppet_agent/blob/main/acceptance/tests/test_upgrade_puppet5_to_puppet6.rb (maybe other tests need to be adapted for puppet 6/7) is not relevant anymore, if we no longer state that puppet 5 is supported.

@GabrielNagy GabrielNagy merged commit 79449f0 into puppetlabs:main Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants