terraform security groups in `ex.Redis` are recreated

[eladcon]

I tried this:

compile and deploy this (tf-aws)

bring ex;
bring cloud;

let api = new cloud.Api() as "api1";
// let ap2 = new cloud.Api() as "api2";

new ex.Redis();

security group id is Redis_KEN25securityGroup_D3232AB7

uncomment the second api creation and recompile and redploy

security group id is Redis_KEN35securityGroup_E4111F48

This happened:

the id of the security group changed without touching the redis resource itself. this causes terraform to fail on deployment since the security group name itself didn't change

I expected this:

No response

Is there a workaround?

No response

Anything else?

No response

Wing Version

No response

Node.js Version

No response

Platform(s)

No response

Community Notes

• Please vote by adding a 👍 reaction to the issue to help us prioritize.
• If you are interested to work on this issue, please leave a comment.

[staycoolcall911]

Have you tried using the Redis winglib? We migrated ex.Redis on tf-aws to the winglib. We plan to also migrate the sim level to the winglib and once that's done we'll deprecate ex.Redis.

[eladcon]

it seems that tf-aws is still not supported?

[staycoolcall911]

You're right, my mistake. sim is supported on the redis winglib, tf-aws still isn't.

[Chriscbr]

I think there's an issue in how the resource IDs are being generated - I noticed in the Terraform output for your example that the resource with a Terraform name of "Redis_KEN31securityGroup_2ECF73BF" is annotated with a construct path of "root/Default/Default/Redis/KEN.31]}securityGroup". When the second API is added, there's a different construct path generated, "root/Default/Default/Redis/KEN.41]}securityGroup"

[Chriscbr]

I believe the plan's to move this code into the redis winglib: https://github.com/winglang/wing/issues/5857

[github-actions[bot]]

Hi,

This issue hasn't seen activity in 90 days. Therefore, we are marking this issue as stale for now. It will be closed after 7 days. Feel free to re-open this issue when there's an update or relevant information to be added. Thanks!

[Chriscbr]

Closing as ex.Redis has been deprecated in favor of the redis winglib: https://github.com/winglang/winglibs/tree/main/redis