search

nlpsandbox / nlpsandbox-infra

AWS CloudFormation templates for deploying the NLP Sandbox infrastructure
Apache License 2.0
0 stars 1 forks source link

IT-1874: Remove routes to admincentral #41

Closed xschildw closed 2 years ago

xschildw commented 2 years ago

This PR removes the routes to the old VPN on admincentral (10.1.0.0./16). Another PR will remove the peering itself on the other side.

thomasyu888 commented 2 years ago

thanks @xschildw! Is this going to impact how I ssh into my instances?

xschildw commented 2 years ago

@thomasyu888 , you should not be able to ssh into your issues now. Their security group allows access from 10.1.0.0/16 (the old VPN). We need to change that to 10.50.0.0/16 (TGW).

xschildw commented 2 years ago

Security group change will be in nlpsandboxvpc config.

xschildw commented 2 years ago

Actually there's some pre-work to associate this VPC with the TGW. Should be tracked in IT-1909.

thomasyu888 commented 2 years ago

Ahh, right. Thanks @xschildw! I'm happy to validate and merge this - although is there anything I need to do in the console?

thomasyu888 commented 2 years ago

Depends on https://github.com/Sage-Bionetworks-IT/organizations-infra/pull/490

xschildw commented 2 years ago

Forgot I had this one

xschildw commented 2 years ago

This is the same as #42, forgot I had it in already...