From 9a702ab29555d0777c7c887d12af4be407a0863f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Gon=C3=A9ri=20Le=20Bouder?= <goneri.lebouder@enovance.com>
Date: Mon, 10 Feb 2014 17:22:33 +0100
Subject: [PATCH] mongod: 10gen packages only on Debian

We should use RedHat packages as much as possible on RHEL system.

closes #183
---
 manifests/database/nosql.pp               | 10 +++++++---
 spec/classes/cloud_database_nosql_spec.rb |  3 ++-
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/manifests/database/nosql.pp b/manifests/database/nosql.pp
index 24195edf..cc68acb8 100644
--- a/manifests/database/nosql.pp
+++ b/manifests/database/nosql.pp
@@ -38,13 +38,17 @@ class cloud::database::nosql(
   # bind_ip should be an array
   $bind_ip_real = any2array($bind_ip)
 
-  # use mongo's own repo instead of the distro's
+  $manage_mongodb_package_repo = $::osfamily ? {
+    'RedHat' => false,
+    default  => true
+  }
+
   class { 'mongodb::globals':
-    manage_package_repo => true
+    manage_package_repo => $manage_mongodb_package_repo
   }->
   class { 'mongodb':
     bind_ip   => $bind_ip_real,
-    nojournal => $nojournal
+    nojournal => $nojournal,
   }
 
 }
diff --git a/spec/classes/cloud_database_nosql_spec.rb b/spec/classes/cloud_database_nosql_spec.rb
index a02bfe74..0e1827a9 100644
--- a/spec/classes/cloud_database_nosql_spec.rb
+++ b/spec/classes/cloud_database_nosql_spec.rb
@@ -28,7 +28,6 @@ describe 'cloud::database::nosql' do
     end
 
     it 'configure mongodb server' do
-      should contain_class('mongodb::globals').with( :manage_package_repo => true)
       should contain_class('mongodb::globals').with_before('Class[Mongodb]')
       should contain_class('mongodb').with(
         :bind_ip   => ['10.0.0.1'],
@@ -43,6 +42,7 @@ describe 'cloud::database::nosql' do
       { :osfamily => 'Debian' }
     end
 
+    it { should contain_class('mongodb::globals').with( :manage_package_repo => true) }
     it_configures 'openstack database nosql'
   end
 
@@ -51,6 +51,7 @@ describe 'cloud::database::nosql' do
       { :osfamily => 'RedHat' }
     end
 
+    it { should contain_class('mongodb::globals').with( :manage_package_repo => false) }
     it_configures 'openstack database nosql'
   end