Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix how CloudFormation stack changes are detected #507

Merged
merged 5 commits into from
Oct 6, 2021

Conversation

devt
Copy link
Contributor

@devt devt commented Sep 16, 2021

Most of the times when there are no changes, the change set will have a status FAILED with StatusReason="The submitted information didn't contain changes. Submit different information to create a change set.", but sometimes StatusReason is "No updates are to be performed."

Most of the times when there are no changes, the change set will have a status FAILED with StatusReason="The submitted information didn't contain changes. Submit different information to create a change set.", but sometimes StatusReason is "No updates are to be performed." when
@ansibullbot
Copy link

@ansibullbot ansibullbot added community_review module module needs_triage new_contributor Help guide this first time contributor plugins plugin (any type) small_patch Hopefully easy to review labels Sep 16, 2021
Fix errors from checks at https://dashboard.zuul.ansible.com/t/ansible/build/9e26ca58308747a8829cef60bd96189c
* ERROR: plugins/modules/cloudformation.py:452:133: E271: multiple spaces after keyword
* ERROR: plugins/modules/cloudformation.py:452:161: E501: line too long (193 > 160 characters)
@ansibullbot ansibullbot removed the small_patch Hopefully easy to review label Sep 17, 2021
@alinabuzachis
Copy link
Collaborator

@devt Thank you for taking time to work on this. Could you please include a changelog fragment https://cn-ansibledoc.readthedocs.io/zh_CN/latest/community/development_process.html#creating-a-changelog-fragment?

@alinabuzachis
Copy link
Collaborator

recheck

1 similar comment
@alinabuzachis
Copy link
Collaborator

recheck

@tremble tremble added the gate label Oct 6, 2021
Copy link
Contributor

@ansible-zuul ansible-zuul bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@ansible-zuul ansible-zuul bot merged commit ee22c8c into ansible-collections:main Oct 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community_review module module new_contributor Help guide this first time contributor plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants