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

Update Terraform scripts to support multi-tenancy #188

Closed
jordanpadams opened this issue Apr 13, 2023 · 2 comments
Closed

Update Terraform scripts to support multi-tenancy #188

jordanpadams opened this issue Apr 13, 2023 · 2 comments

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Apr 13, 2023

💡 Description

TBD description of what needs to be updated. May be superseded by #186

@jimmie
Copy link
Contributor

jimmie commented Apr 13, 2023

There is really nothing specific in the deployment architecture that's required to support multi-tenancy - just a new rev of the task image to include the code that enables it.

Some logistics of course need to be performed if the move to multi-tenancy is performed in the current JPL AWS environment - namely the removal of remotes from the EN Opensearch domain (which activates the use of CCS in the registry service). I don't see a reason to remove the implementation supporting CCS since there could be cases where it could be usefully leveraged - as long as it is deactivated as above, there is no adverse effect to performance or stability.

@jimmie
Copy link
Contributor

jimmie commented Apr 13, 2023

As correctly indicated in the description, this is superseded by registry-185 so I am going to close this one. Feel free to re-open if any of my comments/assumptions made in the previous comment are invalid or questionable.

@jimmie jimmie closed this as completed Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants