Giter VIP home page Giter VIP logo

Comments (14)

loneil avatar loneil commented on July 30, 2024

That case above was a new cred def actually though. And I can still do it fine on other 0.10.1 branches.

Is it intermittent? Maybe can try stuff again after 0.10.2?

from ditp.

WadeBarnes avatar WadeBarnes commented on July 30, 2024

This could be related to issues being fixed in 0.10.2.

from ditp.

esune avatar esune commented on July 30, 2024

This could be related to issues being fixed in 0.10.2.

Wade beat me to it. @loneil I would track the issue here and assume it will be fixed with 0.10.2 - we could test using the rc/nightly image if we wanted a head start before publishing, but I think it is unrelated to @shaangill025's changes

from ditp.

loneil avatar loneil commented on July 30, 2024

Yeah I don't know either I'm getting really (un)lucky, or it's specific to something in the 681 (PR 790) environment because I can't repro in any other 0.10.1 environment yet.

This is happening each time I try and create a def on 790 so far bcgov/traction#790 (comment)

from ditp.

usingtechnology avatar usingtechnology commented on July 30, 2024

are you using the same tenant? if so, try a new tenant that has not been connected to the endorser.

from ditp.

loneil avatar loneil commented on July 30, 2024

are you using the same tenant? if so, try a new tenant that has not been connected to the endorser.

New tenant, steps to reproduce are

1 Create a new tenant
2 Make issuer (connect/register) for bcovrin-test, endorser Ket75eV5UQvVkW2XBjgDH7
3 Create schema
4 Create revocable cred def
5 Issue it to BC Wallet and accept

On https://pr-790-traction-tenant-ui-dev.apps.silver.devops.gov.bc.ca/ (which is 0.10.1) this errors on step 3 and step 5 (see linked above)

On a different PR also 0.10.1 these steps succeed

And on dev (also 0.10.1), with an existing Tenant it also works.

So I'm doubtful this is specific to 0.10.1 rather than a specific set of changes there, unless I'm missing something? Or did the changes in PR 790 make us susceptible to this ACA-Py bug in a way we are not anywhere else?

from ditp.

usingtechnology avatar usingtechnology commented on July 30, 2024

see if you can find anything in the logs related to didexchange resolution... that's one of the bugs.

see pr 2476 / issue 2475

from ditp.

loneil avatar loneil commented on July 30, 2024

The error is when it's trying to upload Tails

2023-09-15 16:19:44,745 aries_cloudagent.core.event_bus ERROR Error occurred while processing event
Traceback (most recent call last):
File "/home/aries/.venv/lib/python3.9/site-packages/aries_cloudagent/core/event_bus.py", line 121, in notify
await processor()
File "/home/aries/.venv/lib/python3.9/site-packages/aries_cloudagent/revocation/routes.py", line 1581, in on_revocation_registry_endorsed_event
await registry_record.upload_tails_file(profile)
File "/home/aries/.venv/lib/python3.9/site-packages/aries_cloudagent/revocation/models/issuer_rev_reg_record.py", line 446, in upload_tails_file
(upload_success, result) = await tails_server.upload_tails_file(
File "/home/aries/.venv/lib/python3.9/site-packages/aries_cloudagent/tails/indy_tails_server.py", line 50, in upload_tails_file
pool = write_ledger[1].pool
TypeError: 'IndyVdrLedger' object is not subscriptable

does that use DID Exchange?

from ditp.

usingtechnology avatar usingtechnology commented on July 30, 2024

there is a bug @shaangill025 fixed... that write_ledger[1] is the issue... it's just write_ledger.pool.

pr 2482

from ditp.

loneil avatar loneil commented on July 30, 2024

Ah ok, so then yeah we have to wait on 0.10.2 for the multi-ledger PR 790. Thanks.

from ditp.

esune avatar esune commented on July 30, 2024

We experienced this issue again today with the BC Wallet Showcase production instance. As it turns out, part of the problem was also the fact that the tails directory was not being persisted and therefore calls executed by different pods were failing due to missing tails file in the filesystem.

from ditp.

esune avatar esune commented on July 30, 2024

Moving to blocked, pending upgrades in the Traction chart and new ACA-Py image

from ditp.

esune avatar esune commented on July 30, 2024

@WadeBarnes do you think we can close this, given what we discovered and fixed (bug in aca-py, missing tails persistence)?

from ditp.

WadeBarnes avatar WadeBarnes commented on July 30, 2024

Yes

from ditp.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.