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

When prefetching the IDs also set the Names of the resources #274

Open
xescugc opened this issue Apr 11, 2022 · 0 comments
Open

When prefetching the IDs also set the Names of the resources #274

xescugc opened this issue Apr 11, 2022 · 0 comments
Labels
Feature roadmap feature

Comments

@xescugc
Copy link
Member

xescugc commented Apr 11, 2022

Abstract

When on the Provider logic we fetch all the IDs we should also store a value that represents the name of set resource.

Proposal

As an example on https://github.com/cycloidio/terracognita/blob/master/aws/resources.go#L935 which is for aws_db_instance the id is DBInstanceIdentifier but we can also store the DBName alongside so we have a user friendly representation of the ID which most of the times is not readable.

This will be helpful for logging/debugging and for 3rd party users of TC

Implementation

Open issues (if applicable)

@xescugc xescugc added the Feature roadmap feature label Apr 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature roadmap feature
Projects
None yet
Development

No branches or pull requests

1 participant