Fixed bug in example where updating a user to a group
Fixed bug in create_group and update_group whereby paramaters were not being set correctly.
This commit is contained in:
parent
ccfc9d2f6b
commit
9f3658d901
1 changed files with 4 additions and 5 deletions
|
@ -133,7 +133,7 @@ task:
|
||||||
iam_type: user
|
iam_type: user
|
||||||
name: jdavila
|
name: jdavila
|
||||||
state: update
|
state: update
|
||||||
group: "{{ item.created_group.group_name }}"
|
groups: "{{ item.created_group.group_name }}"
|
||||||
with_items: new_groups.results
|
with_items: new_groups.results
|
||||||
|
|
||||||
'''
|
'''
|
||||||
|
@ -376,7 +376,7 @@ new_name=None):
|
||||||
return (groups, changed)
|
return (groups, changed)
|
||||||
|
|
||||||
|
|
||||||
def create_group(module, iam, name, path):
|
def create_group(module=None, iam=None, name=None, path=None):
|
||||||
changed = False
|
changed = False
|
||||||
try:
|
try:
|
||||||
iam.create_group(
|
iam.create_group(
|
||||||
|
@ -414,8 +414,7 @@ def delete_group(module, iam, name):
|
||||||
changed = True
|
changed = True
|
||||||
return changed, name
|
return changed, name
|
||||||
|
|
||||||
|
def update_group(module=None, iam=None, name=None, new_name=None, new_path=None):
|
||||||
def update_group(module, iam, name, new_name, new_path):
|
|
||||||
changed = False
|
changed = False
|
||||||
try:
|
try:
|
||||||
current_group_path = iam.get_group(
|
current_group_path = iam.get_group(
|
||||||
|
@ -667,7 +666,7 @@ def main():
|
||||||
module.exit_json(changed=changed, group_name=new_group)
|
module.exit_json(changed=changed, group_name=new_group)
|
||||||
elif state in ['present', 'update'] and group_exists:
|
elif state in ['present', 'update'] and group_exists:
|
||||||
changed, updated_name, updated_path, cur_path = update_group(
|
changed, updated_name, updated_path, cur_path = update_group(
|
||||||
iam, name, new_name, new_path)
|
iam=iam, name=name, new_name=new_name, new_path=new_path)
|
||||||
|
|
||||||
if new_path and new_name:
|
if new_path and new_name:
|
||||||
module.exit_json(changed=changed, old_group_name=name,
|
module.exit_json(changed=changed, old_group_name=name,
|
||||||
|
|
Loading…
Reference in a new issue