-
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
cdk bootstrap prints "hotswap deployment skipped" when there is no diff #25736
cdk bootstrap prints "hotswap deployment skipped" when there is no diff #25736
Comments
Unfortunately I can't reproduce this. Does this issue still exist in the latest CDK version? |
@pahud This issue is still happening in v2.85.0.
|
@pahud This issue is still happening in v2.99.1.
|
#29877) ### Issue # (if applicable) Closes #25736. ### Reason for this change The bootstrap calls `deployStack` without specifying hotswap, and `deployStack` does not correctly default the hotswap mode to `FULL_DEPLOYMENT`. ### Description of changes Make `deployStack` correctly default `hotswapMode` to `FULL_DEPLOYMENT`. ### Description of how you validated changes Manual testing. Before this change: ``` ✨ hotswap deployment skipped - no changes were detected (use --force to override) ✅ Environment aws://123456789012/us-east-1 bootstrapped (no changes). ``` After: ``` ✅ Environment aws://123456789012/us-east-1 bootstrapped (no changes). ``` Unit tests not added because we don't have any unit tests for bootstrap and this is cosmetic. ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
|
Describe the bug
This appears to be the same issue as #24602 purported to fix. I guess it fixed it for normal stack deployments but not the bootstrap.
If we run
cdk bootstrap
and there are no changes, we now get a confusing message:Expected Behavior
This message should not be displayed unless a hotswap was requested. (Is that even a thing for
cdk bootstrap
?)It should simply say the same thing it used to.
Current Behavior
See above.
Reproduction Steps
Run
cdk bootstrap
twice for a given account/region.Possible Solution
No response
Additional Information/Context
No response
CDK CLI Version
2.74.0 (build 70e2a33)
Framework Version
No response
Node.js Version
v18.16.0
OS
Alpine 3.17
Language
Python
Language Version
3.11.3
Other information
No response
The text was updated successfully, but these errors were encountered: