-
Notifications
You must be signed in to change notification settings - Fork 502
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
Author subwallet setup automation #2791
Conversation
f888084
to
49fbdd4
Compare
Signed-off-by: jamshale <jamiehalebc@gmail.com>
Quality Gate passedIssues Measures |
@@ -18,53 +18,62 @@ | |||
from ...messaging.valid import UUID4_EXAMPLE, JSONWebToken | |||
from ...multitenant.base import BaseMultitenantManager | |||
from ...storage.error import StorageError, StorageNotFoundError | |||
from ...utils.endorsement_setup import attempt_auto_author_with_endorser_setup | |||
from ...wallet.error import WalletSettingsError | |||
from ...wallet.models.wallet_record import WalletRecord, WalletRecordSchema | |||
from ..error import WalletKeyMissingError | |||
|
|||
ACAPY_LIFECYCLE_CONFIG_FLAG_MAP = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just alphabetically sorted these.
wallet_profile = await multitenant_mgr.get_wallet_profile( | ||
context, wallet_record, extra_settings=settings | ||
) | ||
await attempt_auto_author_with_endorser_setup(wallet_profile) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trying to do this inside the manager created a circular dependency.
Signed-off-by: jamshale <jamiehalebc@gmail.com>
Signed-off-by: jamshale <jamiehalebc@gmail.com> Signed-off-by: Golda Velez <gvelez17@gmail.com>
Signed-off-by: jamshale <jamiehalebc@gmail.com> Signed-off-by: Golda Velez <gvelez17@gmail.com>
This allows tenant author setup with an endorser to be automated through the api with the same configs as starting an author agent from the command line. It creates a util with the existing startup code and calls it from the
create_wallet
endpoint. I think this really helps simplify creating author tenants.example:
I had to fix what I believe was a bug with creating the connection for the tenant from the admin context in
aries_cloudagent/protocols/didexchange/v1_0/manager.py
where connection info would be saved for the wrong profile for the responder.