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

"Failed to create CloudFormation change set: No updates are to be performed." generates error code -1 #67

Open
mciureanu opened this issue Mar 13, 2019 · 7 comments
Labels
feature-request A feature should be added or improved. module/cli-ext p2 This is a standard priority issue queued

Comments

@mciureanu
Copy link

Hello,

When calling deploy-serverless with a package created with package-ci command, the second time I call the command with the same package I get this as an error message: "Failed to create CloudFormation change set: No updates are to be performed." and the cli tool returns with exit code -1

This is processed by the most CI tools (for example Teamcity - the one I use) as an error, and the build fails.

I think the normal behavior, at least from the "deploy-serverless" point of view should be not to return exit code -1. I don't know if the aws-sdk-net should return ChangeSetStatus.FAILED or not, probably this is somehow another discussion (that should happen in the aws-sdk-net project, since there is no special status like SKIPPED), but as a user of "deploy-serverless" function I believe this should only seen as a warning, not an error. Even if this means checking the error message I guess.

@klaytaybai klaytaybai added the feature-request A feature should be added or improved. label Mar 13, 2019
@klaytaybai
Copy link

Hi @mciureanu, thank you for your feedback on this. Related commands in other products of ours seem to now either suppress the error automatically and generate a warning or allow a user to use an option to suppress the error. I think it is reasonable that we do the same here.

@pravindra-rathore
Copy link

@mciureanu unable to publish application to AWS as it generates when tried to publish

@Rashmi-278
Copy link

Rashmi-278 commented Oct 12, 2020

I faced this issue while creating a lamda funtion.
Changing the KeyID parameter value to a different one worked for me.
It was successfully deployed

@steveshiny
Copy link

I have a similar desire to see the "no-fail-on-empty-changeset" flag supported by these extensions, for largely the same reason. Our automated deployment finds the non-zero exit code and flags as an error/failure. Is there any update on whether support for this might be added?

@hunanniu hunanniu added the needs-reproduction This issue needs reproduction. label Mar 31, 2022
@ganeshnj
Copy link
Contributor

The status code should be inline with the CF returned response. If CF says no-changeset is success, CLI should return 0.

@ashishdhingra could you verify it?

@ashishdhingra
Copy link
Contributor

ashishdhingra commented Aug 31, 2022

The status code should be inline with the CF returned response. If CF says no-changeset is success, CLI should return 0.

@ashishdhingra could you verify it?

Findings:

This feature request to support command line argument no-fail-on-empty-changeset needs to be reviewed by the team.

@ashishdhingra ashishdhingra removed the needs-reproduction This issue needs reproduction. label Aug 31, 2022
@ashishdhingra ashishdhingra added the p2 This is a standard priority issue label Dec 13, 2022
@ashishdhingra
Copy link
Contributor

Another issue aws/aws-lambda-dotnet#1495 was also opened. Related issue aws/serverless-application-model#71.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request A feature should be added or improved. module/cli-ext p2 This is a standard priority issue queued
Projects
None yet
Development

No branches or pull requests

8 participants