karel/postgres-formula
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Handle test=True correctly for tablespace owner

Browse Source 
We weren't handling test=True correctly for when the table already existed, but
the owner was changing.
This commit is contained in:
Brian Jackson
2015-04-23 12:04:20 -05:00
parent 17baaa9f8c
commit e23f4bf8e8
1 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
_states/postgres_tablespace.py
View File

@@ -79,7 +79,11 @@ def present(name,
ret['result'] = False ret['result'] = False
return ret # This isn't changeable, they need to remove/remake return ret # This isn't changeable, they need to remove/remake
if (not owner and not tblspaces[name]['Owner'] == owner): if (owner and not tblspaces[name]['Owner'] == owner):
if __opts__['test']:
ret['result'] = None
ret['comment'] = 'Tablespace {0} owner to be altered'.format(name)
return ret
if __salt__['postgres_ext.tablespace_alter'](name, new_owner=owner): if __salt__['postgres_ext.tablespace_alter'](name, new_owner=owner):
ret['comment'] = 'Tablespace {0} owner changed'.format(name) ret['comment'] = 'Tablespace {0} owner changed'.format(name)
ret['result'] = True ret['result'] = True