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

Amazon Linux v2 should use the new fcontext filetype format #314

Merged
merged 3 commits into from
Jan 2, 2020

Conversation

yitsushi
Copy link
Contributor

Pull Request (PR) description

Amazon Linux v2 is part of the RedHat family, but they started to use their own versioning. Because of that, the originally introduces check for older RHEL distributions thinks Amazon Linux v2 is an older version (operatingsystemmajrelease == 2) and tries to use all files instead of a which is not working.

This Pull Request (PR) fixes the following issues

n/a

@bastelfreak
Copy link
Member

Hi @yitsushi, thanks for the PR. Can you take a look at the failing spec tests? Can you add amazon linux to the metadata.json? And it would be really awesome if you could have a look at https://github.com/camptocamp/facterdb and provide amazon fact sets (not required to get this merged, but it would help us alot).

@yitsushi
Copy link
Contributor Author

yitsushi commented Jan 2, 2020

I will try to check what went wrong with spec-test, but currently on my system:

2100 examples, 0 failures

[I] ❯ ruby --version
ruby 2.4.4p296 (2018-03-28 revision 63013) [x86_64-darwin18]

@bastelfreak
Copy link
Member

this currently fails because a required change in the facterdb isn't released yet. Please apply the following patch:

diff --git a/Gemfile b/Gemfile
index f0381cf..b56cf32 100644
--- a/Gemfile
+++ b/Gemfile
@@ -30,6 +30,9 @@ group :test do
   gem 'coveralls',                                                  :require => false
   gem 'simplecov-console',                                          :require => false
   gem 'parallel_tests',                                             :require => false
+  # we require unreleased Fedora 30 fact sets
+  # https://github.com/camptocamp/facterdb/commit/1bc038fd42aac53344ac1ee9c129cbccf5b5c0aa
+  gem 'facterdb',                                                   :require => false, :git => 'https://github.com/camptocamp/facterdb.git', :branch => 'master'
 end
 
 group :development do

@bastelfreak bastelfreak added the needs-work not ready to merge just yet label Jan 2, 2020
@bastelfreak bastelfreak added enhancement New feature or request and removed needs-tests needs-work not ready to merge just yet tests-fail labels Jan 2, 2020
@bastelfreak
Copy link
Member

Thanks for the PR!

@bastelfreak bastelfreak merged commit 056cf23 into voxpupuli:master Jan 2, 2020
EmRowlands pushed a commit to EmRowlands/puppet-selinux that referenced this pull request Mar 29, 2023
Amazon Linux v2 should use the new fcontext filetype format
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.

3 participants