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

[Multi-Tenant] Create operator tenant #3149

Closed
2 tasks
Tracked by #2893
mkurapov opened this issue Dec 3, 2024 · 0 comments · Fixed by #3156
Closed
2 tasks
Tracked by #2893

[Multi-Tenant] Create operator tenant #3149

mkurapov opened this issue Dec 3, 2024 · 0 comments · Fixed by #3156
Assignees

Comments

@mkurapov
Copy link
Contributor

mkurapov commented Dec 3, 2024

Context

Given that each tenanted resource requires a tenantId to be set, we need to backfill all of the tenanted resources with the operator tenant id. This requires having a migration to seed the first tenant (the operator).

We can do this by having an OPERATOR_TENANT_ID env var that is the same across auth and backend.
auth and backend migration scripts will create the operator tenant in their DBs respectively.

Note: In the backend tenants DB we will not be able to seed the idpConsentUrl and idpSecret. During the tenant lookups via the backend API, we just reach over to the /tenants API and backfill it into the backend DB.

Todo

  • Add OPERATOR_TENANT_ID to auth and backend
  • Create a migration in both auth and backend to seed the first operator tenant (the primary key will be the OPERATOR_TENANT_ID).
@mkurapov mkurapov mentioned this issue Dec 3, 2024
41 tasks
@github-project-automation github-project-automation bot moved this to Backlog in Rafiki Dec 3, 2024
@mkurapov mkurapov changed the title Prepopulate tenants table with configured environment variable Create operator tenant Dec 3, 2024
@mkurapov mkurapov changed the title Create operator tenant [Multi-Tenant] Create operator tenant Dec 3, 2024
@mkurapov mkurapov self-assigned this Dec 5, 2024
@mkurapov mkurapov moved this from Backlog to In Progress in Rafiki Dec 5, 2024
@mkurapov mkurapov linked a pull request Dec 5, 2024 that will close this issue
6 tasks
@mkurapov mkurapov moved this from In Progress to Ready for Review in Rafiki Dec 9, 2024
@mkurapov mkurapov moved this from Ready for Review to Done in Rafiki Dec 11, 2024
@mkurapov mkurapov closed this as completed by moving to Done in Rafiki Dec 11, 2024
@golobitch golobitch added this to the Multi Tenant v1 milestone Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants