Skip to content
This repository has been archived by the owner on Apr 13, 2023. It is now read-only.

Support Multi-tenancy #165

Closed
medhost-bfindeisen opened this issue Dec 7, 2020 · 9 comments
Closed

Support Multi-tenancy #165

medhost-bfindeisen opened this issue Dec 7, 2020 · 9 comments
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@medhost-bfindeisen
Copy link

Add support for multi-tenancy for a more economic Saas offering without affecting the existing non-tenant based approach.

Initial design proposal:

  • API Gateway will be enhanced to pass tenant-id as part of URL - https://fhir.server.com/tenant/{tenant-id}/{resource}

  • API Gateway - extract tenant-id and pass it as parameter

  • Lambda write into tenant based table - resource-db-dev-1-tenant-id

  • DynamoDB - streaming pass this data to ElasticSearch

  • Separate - index per tenant/resource OR - global resource index

  • Meta-data/conformance page need to incorporate tenant-id based approach

  • lack of a tenant id would use the existing non-tenant design

Open design items:

  • Tenant S3 bucket binary storage

  • Finalize ES design decision

@Dunmail
Copy link

Dunmail commented Dec 7, 2020

+1 We use FWoA as a facade on multiple systems. We would benefit from the tenant-id in the URL and as a parameter.

@rsmayda rsmayda added enhancement New feature or request help wanted Extra attention is needed labels Dec 7, 2020
@rsmayda
Copy link
Contributor

rsmayda commented Dec 7, 2020

Thanks for opening this issue -- If folks from our community are interested in this please 'thumbs-up' up the issue.

We would love to get diverse use-cases to ensure FWoA is built to support flexibility

@asrour-vytalize
Copy link

Any updates on this?

@medhost-bfindeisen
Copy link
Author

Any updates on this?

Thanks for the interest - our team is wrapping up work on this and hope to have a PR up in the near future

@asrour-vytalize
Copy link

Can you share the code/branch?
Because I am working on a solution for us.

@rashedoumar
Copy link

hey @medhost-bfindeisen you are doing a great job here , what's the update on the Multi-tenancy ?

@ericfuxealth
Copy link

@@medhost-bfindeisen We are interested to contribute to the multi-tenant solution. Would love to connect.

@rsmayda
Copy link
Contributor

rsmayda commented Aug 17, 2021

We will be merging in this feature by end of this week!

@rsmayda
Copy link
Contributor

rsmayda commented Aug 19, 2021

@rsmayda rsmayda closed this as completed Aug 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

6 participants