-
Notifications
You must be signed in to change notification settings - Fork 4k
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
v1.109.0 CLI cuts off step progress and outputs misleading "Failed resources:" message #15196
Comments
Hello @severn-everett 👋 After investigating this I've found that the CLI output can be a little misleading. The CLI will always output You should be able to see that the stack was successfully deployed via the AWS console. |
What about the cleanup step (4/4)? That output has disappeared in the deployments using 1.109.0 as well. |
I missed that bug, sorry about that. I opened a PR to fix the misleading I am marking this issue as p1 due to the number of users that will be impacted. It has been prioritized as important and should see attention soon. We use +1s to help us prioritize our work. |
Fix for both the Waiting on review and merge 🙂 |
…port all progress steps (#15207) ### Summary This PR changes the CLI output to: - Prevent output of "Failed resources:" message when all resources were deployed successfully. - Fixes an issue where some steps were not being reported. See related issue: #15196 Before: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole Failed resources: ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` After: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 1/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 2/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole 4/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | DeploymentActionRole ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` ### TODO - [x] Prevent output of "Failed resources:" if there are no failures present. - [x] Report all steps before exiting. - [x] Add tests for changes. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
…port all progress steps (aws#15207) ### Summary This PR changes the CLI output to: - Prevent output of "Failed resources:" message when all resources were deployed successfully. - Fixes an issue where some steps were not being reported. See related issue: aws#15196 Before: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole Failed resources: ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` After: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 1/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 2/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole 4/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | DeploymentActionRole ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` ### TODO - [x] Prevent output of "Failed resources:" if there are no failures present. - [x] Report all steps before exiting. - [x] Add tests for changes. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
…port all progress steps (aws#15207) ### Summary This PR changes the CLI output to: - Prevent output of "Failed resources:" message when all resources were deployed successfully. - Fixes an issue where some steps were not being reported. See related issue: aws#15196 Before: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole Failed resources: ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` After: ```shell CDK_NEW_BOOTSTRAP set, using new-style bootstrapping ⏳ Bootstrapping environment aws://##########/eu-west-1... Trusted accounts for deployment: (none) Trusted accounts for lookup: (none) Using default execution policy of 'arn:aws:iam::aws:policy/AdministratorAccess'. Pass '--cloudformation-execution-policies' to customize. AN_AWS_STACK: creating CloudFormation changeset... 0/4 |15:20:04 | UPDATE_IN_PROGRESS | AWS::CloudFormation::Stack | AN_AWS_STACK User Initiated 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | DeploymentActionRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | FilePublishingRole 0/4 |15:20:11 | UPDATE_IN_PROGRESS | AWS::IAM::Role | ImagePublishingRole 1/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 2/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | FilePublishingRole 3/4 |15:20:28 | UPDATE_COMPLETE_CLEA | AWS::IAM::Role | ImagePublishingRole 4/4 |15:20:28 | UPDATE_COMPLETE | AWS::IAM::Role | DeploymentActionRole ✅ Environment aws://##########/eu-west-1 bootstrapped. ``` ### TODO - [x] Prevent output of "Failed resources:" if there are no failures present. - [x] Report all steps before exiting. - [x] Add tests for changes. ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
When deploying an AWS stack using v1.109.0 the CLI...
Reproduction Steps
Upgrade from 1.108.1 to 1.109.0.
What did you expect to happen?
AWS stacks should deploy as they did before.
What actually happened?
The stack deployment fails to clean up, although the stack itself does get deployed.
Notice that step 4/4 did not print.
Environment
Other
This is 🐛 Bug Report
The text was updated successfully, but these errors were encountered: