Allow repo URL to work on non RedHat systems

Addresses issue #179 #179
sensu · May 30, 2014 · 2b5e3ff · 2b5e3ff
1 parent 1cc4c2a
commit 2b5e3ff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/manifests/repo/yum.pp b/manifests/repo/yum.pp
@@ -13,8 +13,8 @@
       $url = $sensu::repo_source
     } else {
       $url = $sensu::repo ? {
-        'unstable'  => 'http://repos.sensuapp.org/yum-unstable/el/$releasever/$basearch/',
-        default     => 'http://repos.sensuapp.org/yum/el/$releasever/$basearch/'
+        'unstable'  => "http://repos.sensuapp.org/yum-unstable/el/${::operatingsystemmajrelease}/\$basearch/",
+        default     => "http://repos.sensuapp.org/yum/el/${::operatingsystemmajrelease}/\$basearch/"
       }
     }
 

diff --git a/spec/classes/sensu_package_spec.rb b/spec/classes/sensu_package_spec.rb
@@ -80,20 +80,20 @@
       end
 
       context 'redhat' do
-        let(:facts) { { :operatingsystem => 'RedHat' } }
+        let(:facts) { { :operatingsystem => 'RedHat', :operatingsystemmajrelease => '6' } }
 
         context 'default' do
           it { should contain_yumrepo('sensu').with(
             :enabled   => 1,
-            :baseurl   => 'http://repos.sensuapp.org/yum/el/$releasever/$basearch/',
+            :baseurl   => 'http://repos.sensuapp.org/yum/el/6/$basearch/',
             :gpgcheck  => 0,
             :before    => 'Package[sensu]'
           ) }
         end
 
         context 'unstable repo' do
           let(:params) { { :repo => 'unstable' } }
-          it { should contain_yumrepo('sensu').with(:baseurl => 'http://repos.sensuapp.org/yum-unstable/el/$releasever/$basearch/' )}
+          it { should contain_yumrepo('sensu').with(:baseurl => 'http://repos.sensuapp.org/yum-unstable/el/6/$basearch/' )}
         end
 
         context 'override repo url' do