Skip to content

Commit adea970

Browse files
authored
Merge pull request voxpupuli#663 from JvGinkel/dberrors
Catch errors on database creation or destroy
2 parents 60e16ce + 4d706b7 commit adea970

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

lib/puppet/provider/mongodb_database/mongodb.rb

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,15 +27,17 @@ def self.prefetch(resources)
2727

2828
def create
2929
if db_ismaster
30-
mongo_eval('db.dummyData.insert({"created_by_puppet": 1})', @resource[:name])
30+
out = mongo_eval('db.dummyData.insert({"created_by_puppet": 1})', @resource[:name])
31+
raise "Failed to create DB '#{@resource[:name]}'\n#{out}" if %r{writeError} =~ out
3132
else
3233
Puppet.warning 'Database creation is available only from master host'
3334
end
3435
end
3536

3637
def destroy
3738
if db_ismaster
38-
mongo_eval('db.dropDatabase()', @resource[:name])
39+
out = mongo_eval('db.dropDatabase()', @resource[:name])
40+
raise "Failed to destroy DB '#{@resource[:name]}'\n#{out}" if %r{writeError} =~ out
3941
else
4042
Puppet.warning 'Database removal is available only from master host'
4143
end

0 commit comments

Comments
 (0)