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 support for OpenSUSE #281

Merged
merged 1 commit into from
Aug 25, 2020
Merged

Conversation

martijndegouw
Copy link
Contributor

@martijndegouw martijndegouw commented Mar 23, 2020

Add support for OpenSUSE. Tested on OpenSUSE 15

@bastelfreak bastelfreak added enhancement New feature or request needs-work not ready to merge just yet tests-fail labels Mar 25, 2020
@bastelfreak
Copy link
Member

@martijndegouw thanks again for the PR. Please also take a look at the failing tests and add OpenSUSE to the metadata.json

@martijndegouw
Copy link
Contributor Author

martijndegouw commented Mar 25, 2020

I looked into the failing tests, but it looks like the problem is in the test environment/existing code/puppet, not in the newly added code:
Test is running on solaris-11-i86pc and returns unsupported osfamily/arch Solaris/ (params.pp:337)

Looks like ${facts['os']['architecture']} is not set correctly in the correspondig ruby/puppet version

@martijndegouw martijndegouw force-pushed the OpenSUSE branch 2 times, most recently from 1a51616 to f19f78b Compare May 1, 2020 15:00
@vox-pupuli-tasks
Copy link

Dear @martijndegouw, thanks for the PR!

This is Vox Pupuli Tasks, your friendly Vox Pupuli GitHub Bot. I noticed that your pull request contains merge conflict. Can you please rebase?

You can find my sourcecode at voxpupuli/vox-pupuli-tasks

@martijndegouw martijndegouw force-pushed the OpenSUSE branch 4 times, most recently from 899ae50 to 02060ec Compare August 25, 2020 09:32
@bastelfreak
Copy link
Member

thanks for the work!

@bastelfreak bastelfreak merged commit 0fae8fe into voxpupuli:master Aug 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants