Its impossible use changeset_name parameter in cloudformation module

[gregory-dan]

ISSUE TYPE

• Bug Report

COMPONENT NAME

cloudformation

ANSIBLE VERSION

ansible 2.4.0 (devel 97aaf103e8) last updated 2017/07/31 15:29:18 (GMT +300)
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/admin/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /home/admin/ansible/lib/ansible
  executable location = /home/admin/ansible/bin/ansible
  python version = 2.7.5 (default, Sep 15 2016, 22:37:39) [GCC 4.8.5 20150623 (Red Hat 4.8.5-4)]

CONFIGURATION

OS / ENVIRONMENT

N/A

SUMMARY

Using "changeset_name" parameter results in error:

"msg": "local variable 'changeset_name' referenced before assignment local variable 'changeset_name' referenced before assignment - <type 'exceptions.UnboundLocalError'>"

STEPS TO REPRODUCE

- name: Creating/Updating stack
  cloudformation:
    stack_name:   "{{stack_name}}"
    profile:      "{{profile}}"
    region:       "{{region}}"
    state:        "{{stack_state}}"
    template_url: "test-change-set.yaml"
    changeset_name: test1
    create_changeset: true
  register:     "{{stack_name}}"

EXPECTED RESULTS

change set is created with predefined name

ACTUAL RESULTS

Fails with error above

[gregory-dan]

The fix is simple. I tried to push the fix but I get 403 error.
How can I contribute the fix to project ?

[ansibot]

cc @jsmartin @ryansb @tedder
click here for bot help

[tedder]

Hi- thanks for trying to fix! You'll need to clone the Ansible repo, (optionally but recommended) create a branch, and then create a pull request. Once you push the branch to your repo, Github will show you a button to create the PR. Just reference #27487 in that pull request (by using the # sign and the number 27487). I'll then review the code and we can work through any minor issues.

This is a good tutorial on how to fork/branch/PR: https://yangsu.github.io/pull-request-tutorial/

Feel free to ask here for any help.

[gregory-dan]

Not related to this issue,

Do you have plans to implement execute-change-set as well ?