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

(SUP-3952) Remove Puppet 6 as a supported platform #78

Merged
merged 3 commits into from
Apr 18, 2023

Conversation

elainemccloskey
Copy link
Contributor

Update to puppet 7 & update OS

@elainemccloskey elainemccloskey requested a review from a team as a code owner April 17, 2023 16:50
metadata.json Outdated
]
}
],
"requirements": [
{
"name": "puppet",
"version_requirement": ">= 6.21.0 < 8.0.0"
"version_requirement": ">= 7.18.0 < 8.0.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why 7.18 and no older version of 7.x?

Copy link
Contributor

Choose a reason for hiding this comment

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

As predominantly PE modules, our CI only tests against the LTS PE and STS PE, this is an indicator of what we test against

Copy link
Contributor

Choose a reason for hiding this comment

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

Perhaps influxdb we could just to 7, since it got an extended uses case

@MartyEwings MartyEwings merged commit 8e9bde9 into puppetlabs:main Apr 18, 2023
@MartyEwings MartyEwings changed the title (SUP-3952) Update metadata (SUP-3952) Remove Puppet 6 as a supported platform Apr 18, 2023
kenyon added a commit to kenyon/puppetlabs-influxdb that referenced this pull request Sep 7, 2023
Debian was maybe mistakenly removed in puppetlabs#78.
kenyon added a commit to kenyon/puppetlabs-influxdb that referenced this pull request Sep 8, 2023
Debian was maybe mistakenly removed in puppetlabs#78.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants