-
Notifications
You must be signed in to change notification settings - Fork 51
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
Set default endorser in helm value #1376
Comments
+1 to using the automated endorser on BCovrin test for the default chart |
If running locally with docker then this issue wouldn't be releated as this applies to the Helm charts for deploying to Kubernetes, and which cloud-hosted Endorser would be default for attempting connections. The local docker setup, if running the compose script up, will stand up a local endorser and the Traction (ACA-Py) agent is set to allow connections to that local endorser. If that screenshot there is from running the Tenant UI, then that 404 on the Endorser Connection (if indeed running locally) would be expected (maybe we should not have it console error for this reason...) and schema creation or other ledger write operations would not be able to proceed until the Endorser allows it. On local docker this is all set up to auto allow writes but the Endorser connection and DID registration are required for each tennat before that can go on Locally, you should be able to connect to the Endorser and set the write ledger through the Tenant UI in Profile. The step 4 in this Lab 1 https://aca-py.org/latest/demo/Aries-Workshop/#lab-1-steps-to-follow describes those steps. (There's also the "Make Tenant an Issuer" video here https://github.com/bcgov/traction/blob/main/docs/USE-CASE-ONBOARD.md#make-teanant-an-issuer but I need to re-record that as the UI has changed since then, but the general concept in the Tenant UI is the same) |
Hi @H4LL no prob. I think I see the issue for schemas, if it's just that nothing happens when the button is clicked it's just a frontend validation problem schema versions must be in the form of X.X or X.X.X for Aries, so you'd want to put something like
This feature isn't in the Tenant UI yet, if it's a use case that could be useful then yeah creating an issue in this repo to request it would be great. |
@H4LL for discussion places
We talked about this internally the other day and since Traction is built on top of ACA-Py, and the Traction maintainers are active in the ACA-Py community then questions could be posed on that Discord. Note ACA-Py is (at this exact time) moving from the Linix Foundation Decentralized Trust to the OpenWallet Foundation, so some communication platforms are just getting set up now. The link to the OWF Discord can be found here: https://github.com/openwallet-foundation/acapy?tab=readme-ov-file#troubleshooting |
In
values.yaml
the ledger/endorser default setup isKet75eV5UQvVkW2XBjgDH7 is not a valid endorser any more (was the old one in tools).
Should set this to one that can be used? I think? Is it ok for us to put the DID for our auto-everything sandbox bcovrin-test endorser in here as default config? cc @esune
The text was updated successfully, but these errors were encountered: