Merge pull request #101 from yuhanz/master

bug fix in member_config for having _id > 255

libraries/mongodb.rb

@@ -156,7 +156,7 @@ def self.configure_replicaset(node, name, members)
         Chef::Log.error("configuring replicaset returned: #{result.inspect}") unless result.fetch('errmsg', nil).nil?
       else
         # remove removed members from the replicaset and add the new ones
-        max_id = config['members'].map { |member| member['_id'] }.max
+        old_ids = config['members'].map { |member| member['_id'] }
         rs_members.map! { |member| member['host'] }
         config['version'] += 1
         old_members = config['members'].map { |member| member['host'] }
@@ -166,10 +166,11 @@ def self.configure_replicaset(node, name, members)
           host = m['host']
           { '_id' => m['_id'], 'host' => host }.merge(rs_options[host])
         end
+        ids = (0...256).to_a - old_ids
         members_add = rs_members - old_members
         members_add.each do |m|
-          max_id += 1
-          config['members'] << { '_id' => max_id, 'host' => m }.merge(rs_options[m])
+          new_id = ids.shift
+          config['members'] << { '_id' => new_id, 'host' => m }.merge(rs_options[m])
         end
 
         rs_connection = nil