Fixes #29212 - support el8

theforeman · May 4, 2020 · 3120d7a · 3120d7a
1 parent 2e241f3
commit 3120d7a
Show file tree

Hide file tree

Showing 4 changed files with 29 additions and 4 deletions.
diff --git a/.sync.yml b/.sync.yml
@@ -2,6 +2,7 @@
 .travis.yml:
   beaker_sets:
     - centos7-64
+    - centos8-64
     - debian10-64
     - ubuntu1804-64
   env:

diff --git a/.travis.yml b/.travis.yml
@@ -34,6 +34,28 @@ matrix:
         - echo '{"ipv6":true,"fixed-cidr-v6":"2001:db8:1::/64"}' | sudo tee /etc/docker/daemon.json
         - sudo service docker restart
 
+    - rvm: 2.5.1
+      env:
+        - BEAKER_PUPPET_COLLECTION=puppet5
+        - BEAKER_setfile=centos8-64{hostname=centos8-64.example.com}
+      script: bundle exec rake beaker
+      services: docker
+      bundler_args: --without development
+      before_install:
+        - echo '{"ipv6":true,"fixed-cidr-v6":"2001:db8:1::/64"}' | sudo tee /etc/docker/daemon.json
+        - sudo service docker restart
+
+    - rvm: 2.5.1
+      env:
+        - BEAKER_PUPPET_COLLECTION=puppet6
+        - BEAKER_setfile=centos8-64{hostname=centos8-64.example.com}
+      script: bundle exec rake beaker
+      services: docker
+      bundler_args: --without development
+      before_install:
+        - echo '{"ipv6":true,"fixed-cidr-v6":"2001:db8:1::/64"}' | sudo tee /etc/docker/daemon.json
+        - sudo service docker restart
+
     - rvm: 2.5.1
       env:
         - BEAKER_PUPPET_COLLECTION=puppet5

diff --git a/metadata.json b/metadata.json
@@ -55,13 +55,15 @@
     {
       "operatingsystem": "RedHat",
       "operatingsystemrelease": [
-        "7"
+        "7",
+        "8"
       ]
     },
     {
       "operatingsystem": "CentOS",
       "operatingsystemrelease": [
-        "7"
+        "7",
+        "8"
       ]
     },
     {

diff --git a/spec/acceptance/foreman_cli_plugins_spec.rb b/spec/acceptance/foreman_cli_plugins_spec.rb
@@ -11,7 +11,7 @@ class { 'foreman::cli':
       password    => 'changeme',
     }
 
-    if $facts['osfamily'] == 'RedHat' {
+    if $facts['os']['family'] == 'RedHat' {
       include foreman::cli::ansible
       include foreman::cli::azure
     }
@@ -27,7 +27,7 @@ class { 'foreman::cli':
   it_behaves_like 'hammer'
 
   ['discovery', 'remote_execution', 'tasks', 'templates'].each do |plugin|
-    package_name = case fact('osfamily')
+    package_name = case fact('os.family')
                    when 'RedHat'
                      fact('operatingsystem') == 'Fedora' ? "rubygem-hammer_cli_foreman_#{plugin}" : "tfm-rubygem-hammer_cli_foreman_#{plugin}"
                    when 'Debian'