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

Add Puppet 6 support #642

Merged
merged 2 commits into from
Nov 18, 2018
Merged

Add Puppet 6 support #642

merged 2 commits into from
Nov 18, 2018

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Oct 18, 2018

@ekohl
Copy link
Member Author

ekohl commented Oct 18, 2018

Note to self: we should drop the debian8 beaker tests in favor of debian9. Going to let this continue though to see what's going to happen.

@ekohl ekohl force-pushed the puppet-6 branch 2 times, most recently from 7f66750 to 500afba Compare November 18, 2018 14:15
The only difference is the 6 CA generation and that we need _core repos
as fixtures because the gem doesn't include the vendored modules.

While we're at it, we also add Puppet 5 acceptance tests but because of
a lack of packages we skip:

* Debian 8 + puppet5 / puppet6
* Debian 9 + pc1
@ekohl ekohl changed the title [WIP] Test with Puppet 6 Add Puppet 6 support Nov 18, 2018
@ekohl
Copy link
Member Author

ekohl commented Nov 18, 2018

This should now be ready to go. I didn't add Ubuntu acceptance tests because they were failing with a timeout and I haven't had time to investigate. That shouldn't hold back this PR though because it was also timing out on Puppet 5.

@mmoll mmoll merged commit 054d079 into theforeman:master Nov 18, 2018
@mmoll
Copy link
Contributor

mmoll commented Nov 18, 2018

merged, bedankt @ekohl!

@ekohl ekohl deleted the puppet-6 branch November 19, 2018 12:50
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