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

All tenants are being created in the same database/schema #1254

Open
esune opened this issue Jul 11, 2024 · 3 comments
Open

All tenants are being created in the same database/schema #1254

esune opened this issue Jul 11, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@esune
Copy link
Member

esune commented Jul 11, 2024

The Traction ACA-Py agent is started with the setting "wallet_scheme":"DatabasePerWallet" in the wallet storage configuration: this is expected to cause each tenant to be created and live in their own database in the wallet. However, it appears that all tenants are actually being created inside the same multitenant_sub_wallet database at this time.

Unsure on whether this issue is in Traction or ACA-Py, we should audit the problem and follow-up in the appropriate repository to find a solution.

@esune esune added the bug Something isn't working label Jul 11, 2024
@esune esune moved this to Assignment Ready in CDT Enterprise Apps Jul 11, 2024
@loneil
Copy link
Contributor

loneil commented Jul 11, 2024

FYI this is what the ACA-Py status/config reads (using x-api-key) when fetched (dev env in this case but same for all)

image

@jamshale
Copy link
Contributor

So. I've confirmed that this is an issue with the multitenant_provider plugin. https://github.com/hyperledger/aries-acapy-plugins/tree/main/multitenant_provider. I don't think there's any configuration or anything that will fix it. It's a bug in the plugin that needs to be fixed.

If I use the plugin in base aca-py and create wallets it starts putting them in multitenant_sub_wallet which I believe is a hardcoded default in aca-py.

Without the plugin everything works as expected.

@jamshale
Copy link
Contributor

Changing the plugin config for the multitenant_provider to use the other multitenant manager class will fix the issue for new tenants. https://github.com/bcgov/traction/blob/main/charts/traction/values.yaml#L226.

This should be changed to BasicMultitokenMultitenantManager

@esune esune moved this from Assignment Ready to Assigned in CDT Enterprise Apps Jul 15, 2024
@esune esune moved this from Assigned to In Progress in CDT Enterprise Apps Jul 15, 2024
@jamshale jamshale moved this from In Progress to Complete in CDT Enterprise Apps Aug 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

No branches or pull requests

3 participants