-
-
Notifications
You must be signed in to change notification settings - Fork 500
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
Drop Puppet 6 support #927
Conversation
@bastelfreak are the non-preexisting CI failures expected along with this? |
failures in #915 look odd. maybe something in the puppet/systemd module changed? I'm not sure what you mean with "non-preexisting CI failures". Our gems don't support testing on ruby 2.5 anymore, which is what we get when testing on puppet 6. also that was blocking puppet 8 adoption. |
@bastelfreak while the arch integration test failure was preexisting (though IMO, needs to get fixed or dropped), this PR also had failures on Puppet / 7 - Debian 9 Puppet / 7 - Ubuntu 16.04 which I don't think were preexisting. So unless they're transient, I think this PR was merged with failing tests? |
It's possible that this is just because the project needs a modulesync? I don't have enough context on that error to know, but since the open modulesync PR is also broken, not sure what the right fix is here? |
ah right. we had a few CI errors in some of the PRs. I had to cleanup ~150 modules and having puppet 6 in metadata.json blocked the whole CI pipeline. We need to drop the EoL debian 9 and Ubuntu 16.04, but I didn't want to do that in the same PR. |
Got it! Ok, makes sense! So there will be a separate one inbound for that, right? |
when I find time I will clean it up in all modules. I also need to do a modulesync. Both require a lot of time. If anybody wants to help please join or IRC channel or slack channel so we can coordinate this. |
I'm not doing much of anything Puppet related these days, and just help out here because no one else is really working on maintaining this module. So one issue is that until #926 and some related stuff is fixed, I don't think this module yet will work on the non deprecated versions of these OSes 😞 I could drop Ubuntu 16.04, but In particular, 9 is the only version of Debian currently marked as supported. |
PR voxpupuli#927 removed Puppet 6 support, so we should be safely able to remove these testing workarounds
PR #927 removed Puppet 6 support, so we should be safely able to remove these testing workarounds
No description provided.