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

cloudformation describe_change_set with IncludePropertyValues=True response lacking StatusReason #4178

Closed
ilons opened this issue Jun 27, 2024 · 3 comments
Assignees
Labels
bug This issue is a confirmed bug. closed-for-staleness cloudformation p2 This is a standard priority issue response-requested Waiting on additional information or feedback. service-api This issue is caused by the service API, not the SDK implementation.

Comments

@ilons
Copy link

ilons commented Jun 27, 2024

Describe the bug

When calling the describe_change_set function on a cloudformation client for a stack with IncludePropertyValues set to True, the response does not include StatusReason if the change set Status is FAILED.

Expected Behavior

The StatusReason should be included in the response both when IncludePropertyValues is set to True.

Current Behavior

The response returned does not include the StatusReason for a change set with a Status of FAILED (at least not when it failed due to no changes).
When performing the same call with IncludePropertyValues set to False, the StatusReason is included in the response.

Reproduction Steps

Create a CloudFormation stack
Call create_change_set using the same template
Call describe_change_set on the created change set with IncludePropertyValues set to True

Possible Solution

No response

Additional Information/Context

This seems to be a bug in CloudFormation itself, as the AWS Console are performing two calls when viewing a change set, one with include property values set to true, and one with it set to false.

SDK version used

1.34.134

Environment details (OS name and version, etc.)

Python 3.10.12, Linux Pop!_OS 22.04 LTS

@ilons ilons added bug This issue is a confirmed bug. needs-triage This issue or PR still needs to be triaged. labels Jun 27, 2024
@tim-finnigan tim-finnigan self-assigned this Jun 27, 2024
@tim-finnigan tim-finnigan added the investigating This issue is being investigated and/or work is in progress to resolve the issue. label Jun 27, 2024
@tim-finnigan
Copy link
Contributor

Thanks @ilons for reaching out. I tried to reproduce this issue but was not able do. When running the describe_change_set command and getting a FAILED status, I got the StatusReason in my response with IncludePropertyValues was either set to True or False.

Could you provide code snippets to help us reproduce the issue? Also if you can share debug logs (with sensitive info redacted) by adding boto3.set_stream_logger('') to your script, that could give us insight into the underlying issue here.

Boto3 commands like describe_change_set involve calls to upstream APIs (in this case DescribeChangeSet) so there may be some edge case issue with the CloudFormation API and/or here that we would need to escalate to that team.

@tim-finnigan tim-finnigan added response-requested Waiting on additional information or feedback. cloudformation service-api This issue is caused by the service API, not the SDK implementation. p2 This is a standard priority issue and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. needs-triage This issue or PR still needs to be triaged. labels Jun 27, 2024
@ilons
Copy link
Author

ilons commented Jun 28, 2024

Could you provide code snippets to help us reproduce the issue? Also if you can share debug logs (with sensitive info redacted) by adding boto3.set_stream_logger('') to your script, that could give us insight into the underlying issue here.

Thank you for the swift response, I'm afraid it would be very hard for me to provide this in the same timely fashion, as I'm leaving for vacation. However, I'll put a note to provide this as soon as I get back.

@github-actions github-actions bot removed the response-requested Waiting on additional information or feedback. label Jun 29, 2024
@tim-finnigan tim-finnigan added the response-requested Waiting on additional information or feedback. label Jul 2, 2024
Copy link

github-actions bot commented Jul 9, 2024

Greetings! It looks like this issue hasn’t been active in longer than five days. We encourage you to check if this is still an issue in the latest release. In the absence of more information, we will be closing this issue soon. If you find that this is still a problem, please feel free to provide a comment or upvote with a reaction on the initial post to prevent automatic closure. If the issue is already closed, please feel free to open a new one.

@github-actions github-actions bot added closing-soon This issue will automatically close in 4 days unless further comments are made. closed-for-staleness and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Jul 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a confirmed bug. closed-for-staleness cloudformation p2 This is a standard priority issue response-requested Waiting on additional information or feedback. service-api This issue is caused by the service API, not the SDK implementation.
2 participants