diff --git a/CHANGELOG.md b/CHANGELOG.md index 530c2b0c8..1fa00a790 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -15,6 +15,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0. - `DEPLOY_ENVIRONMENT` is available to all module deployments as an environment variable - if it does not exist in the current environment, value is derived from branch or directory name - Updated static site CFN template to use node v10 for path rewrite lambda +- embedded stacker will not resolve dependancies for `locked` stacks when they rely on other stacks + - accepted upstream in https://github.com/cloudtools/stacker/pull/746 ## [1.3.3] - 2019-11-26 ### Changed diff --git a/setup.py b/setup.py index 7eaa7e276..8b7558312 100644 --- a/setup.py +++ b/setup.py @@ -40,6 +40,7 @@ # and the following patches applied # https://github.com/cloudtools/stacker/pull/731 (CAPABILITY_AUTO_EXPAND) # https://github.com/cloudtools/stacker/pull/744 (diffs via CFN changesets) + # https://github.com/cloudtools/stacker/pull/746 (locked stack dependencies) # and the following files/folders deleted: # * tests # * blueprints/testutil.py diff --git a/src/runway/embedded/stacker/stack.py b/src/runway/embedded/stacker/stack.py index aa5ab81b4..950fcd548 100644 --- a/src/runway/embedded/stacker/stack.py +++ b/src/runway/embedded/stacker/stack.py @@ -85,12 +85,6 @@ def required_by(self): @property def requires(self): - # By definition, a locked stack has no dependencies, because we won't - # be performing an update operation on the stack. This means, resolving - # outputs from dependencies is unnecessary. - if self.locked and not self.force: - return [] - requires = set(self.definition.requires or []) # Add any dependencies based on output lookups