Skip to content

Implement basic resource equivalence #132

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

Closed
otaviomacedo opened this issue Feb 25, 2025 · 0 comments · Fixed by #342
Closed

Implement basic resource equivalence #132

otaviomacedo opened this issue Feb 25, 2025 · 0 comments · Fixed by #342
Assignees

Comments

@otaviomacedo
Copy link
Contributor

To detect which resources should be refactored, we need to indentify which
resources have only changed their location, but have remained "the same", in
some sense. This can be made precise by defining an equivalence relation on
the set of resources.

Before that, let's define a digest function, d:

d(resource) = hash(type + properties + dependencies.map(d))

where hash is a cryptographic hash function. In other words, the hash
is computed from its type, its own properties (that is, excluding properties
that refer to other resources), and the digests of each of its dependencies.

The digest of a resource, defined recursively this way, remains stable even if
one or more of its dependencies gets renamed. Since the resources in a
CloudFormation template form an acyclic graph, this function is well-defined.

The equivalence relation then follows directly: two resources r1 and r2
are equivalent if d(r1) = d(r2).

@otaviomacedo otaviomacedo reopened this Apr 9, 2025
github-merge-queue bot pushed a commit that referenced this issue Apr 11, 2025
Add a `refactor` command. For now, it only works in dry run mode
(`-dry-run` in the command line, or `dryRun: true` to the toolkit). It
computes the mappings based on the difference between the deployed
stacks and the cloud assembly stacks, and shows them in a table.
Example:

```
$ cdk refactor --dry-run --unstable=refactor

The following resources were moved or renamed:
┌───────────────────────┬────────────────────────────────────────┬───────────────────────────────────────┐
│ Resource Type         │ Old Construct Path                     │ New Construct Path                    │
├───────────────────────┼────────────────────────────────────────┼───────────────────────────────────────┤
│ AWS::IAM::Role        │ Consumer/Function/ServiceRole/Resource │ Consumer/NewName/ServiceRole/Resource │
├───────────────────────┼────────────────────────────────────────┼───────────────────────────────────────┤
│ AWS::Lambda::Function │ Consumer/Function/Resource             │ Consumer/NewName/Resource             │
└───────────────────────┴────────────────────────────────────────┴───────────────────────────────────────┘
```

Note that we are launching this feature under unstable mode, which
requires the user to acknowledge that by passing the
`--unstable=refactor` flag.

Closes #132,
#133,
#141,
#134 and
#135.

---
By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache-2.0 license
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant