-
Notifications
You must be signed in to change notification settings - Fork 12
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
Upgrade ACA-Py Version #142
base: main
Are you sure you want to change the base?
Conversation
WadeBarnes
commented
May 24, 2024
- Upgrade to ACA-Py 0.12.1
- Ensure the agent includes the full webhook payload.
- Upgrade to ACA-Py 0.12.1 - Ensure the agent includes the full webhook payload. Signed-off-by: Wade Barnes <[email protected]>
Marked as a draft because I haven't applied these changes yet. I'm waiting to resolve the Sovrin TestNet issues before doing this. |
Deploy this upgrade before bcgov/von-bc-registries-agent-configurations#79. We want to test the compatibility with the older ACA-Py version on the BC Reg side first. |
I've applied the |
Deploying the image requires a bit more coordination, since there are secure storage (wallet) upgrades that will be performed once started, and the OrgBook wallets are rather large. Steps for each environment:
Use the upgrade process in Notes:
|
Related Indy to Askar secure storage migration test results: DEV:
TEST:
PROD:
|
Progress on OrgBook secure storage migration tests - since yesterday:
|
How much of your time is being taken on this @WadeBarnes ? Given that we are looking to revamp how data is fed into OrgBook (likely eliminating the DIDComm and AnonCreds processing), should we abandon this effort? It will likely be necessary to load the data from scratch into a new wallet. |
Currently just monitoring the migration process occasionally so we get metrics. |
Progress on OrgBook secure storage migration tests:
Confirmed the upgrade/migration process does not continue where it left off. Restarting the process on the already partly upgraded and converted wallet encounters the following error:
Not going to restart this test, we know it would take somewhere over 45 hour to complete. |
Progress on OrgBook secure storage migration tests:
So we know we wouldn't be able to upgrade/migrate the production wallet database without further investigation and testing and the process would likely take well over 60 hours to complete. Again, I'm not going to restart this test. |