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

feat(auth, backend): add migration files for updating Rafiki resources with multiple tenant support #2904

Conversation

sanducb
Copy link
Contributor

@sanducb sanducb commented Aug 27, 2024

Changes proposed in this pull request

Changes consist of creating tenants table for both Rafiki backend and auth services, as well as adding tenantId as a foreign key for Rafiki backend resources.

Context

This PR introduces some of the changes needed for achieving multi-tenancy on Rafiki.

Closes #2894

Checklist

  • Create tenants table in Rafiki auth db
  • Update grants table with tenantId foreign key in Rafiki auth db
  • Create tenants table in Rafiki backend db
  • Update walletAddresses, incomingPayments, outgoingPayments and quotes tables with tenantId foreign key in Rafiki backend db

mkurapov and others added 4 commits August 26, 2024 16:14
* feat(backend): add trace to outgoing payment lifecycle

* feat(backend): encapsulate query lookup in trace

* chore(localenv): remove traces from default localenv

---------

Co-authored-by: Max Kurapov <[email protected]>
@sanducb sanducb linked an issue Aug 27, 2024 that may be closed by this pull request
1 task
@sanducb sanducb self-assigned this Aug 27, 2024
@github-actions github-actions bot added pkg: backend Changes in the backend package. pkg: auth Changes in the GNAP auth package. labels Aug 27, 2024
Copy link

netlify bot commented Aug 27, 2024

Deploy Preview for brilliant-pasca-3e80ec failed.

Name Link
🔨 Latest commit b3096c4
🔍 Latest deploy log https://app.netlify.com/sites/brilliant-pasca-3e80ec/deploys/66cddf097f13c60008aad301

@sanducb sanducb requested review from njlie and golobitch August 27, 2024 13:45
@sanducb sanducb changed the title Add migration files for updating Rafiki resources with multiple tenant support feat: Add migration files for updating Rafiki resources with multiple tenant support Aug 27, 2024
@sanducb sanducb changed the title feat: Add migration files for updating Rafiki resources with multiple tenant support feat(auth,backend): Add migration files for updating Rafiki resources with multiple tenant support Aug 27, 2024
@sanducb sanducb changed the title feat(auth,backend): Add migration files for updating Rafiki resources with multiple tenant support feat(auth, backend): Add migration files for updating Rafiki resources with multiple tenant support Aug 27, 2024
@golobitch golobitch changed the title feat(auth, backend): Add migration files for updating Rafiki resources with multiple tenant support feat(auth, backend): add migration files for updating Rafiki resources with multiple tenant support Aug 27, 2024
@sanducb sanducb changed the base branch from main to 2893-multi-tenant-rafiki August 27, 2024 14:19
@sanducb
Copy link
Contributor Author

sanducb commented Aug 27, 2024

Closing this PR as feature/2900-create-tenant-admin-api already contains the changes proposed here.

@sanducb sanducb closed this Aug 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: auth Changes in the GNAP auth package. pkg: backend Changes in the backend package.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Multi-tenant Rafiki] Update all resources to have a tenantId foreign key
3 participants