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

0.12.1 #2926

Merged
merged 2 commits into from
May 1, 2024
Merged

0.12.1 #2926

merged 2 commits into from
May 1, 2024

Conversation

swcurran
Copy link
Contributor

@swcurran swcurran commented May 1, 2024

Signed-off-by: Stephen Curran [email protected]

swcurran added 2 commits May 1, 2024 10:01
Signed-off-by: Stephen Curran <[email protected]>
Signed-off-by: Stephen Curran <[email protected]>
Copy link

sonarqubecloud bot commented May 1, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@swcurran swcurran requested review from dbluhm and jamshale May 1, 2024 17:03
@swcurran swcurran marked this pull request as ready for review May 1, 2024 17:03
@swcurran
Copy link
Contributor Author

swcurran commented May 1, 2024

@jamshale -- could you please look at the integration test error? Presumably intermittent, but not one I recognize.

Copy link
Contributor

@dbluhm dbluhm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing with example scenarios in minimal example repo all look good: Indicio-tech/acapy-minimal-example#88

Tests against Credo (run locally) also passed using the 0.12.1rc1 image.

LGTM! Assuming the integration test failure is indeed just an intermittent issue.

@jamshale
Copy link
Contributor

jamshale commented May 1, 2024

The integration test got a duplicate record error setting up endorser roles. I've never seen this before. I'd like to have a look at this closer and figure out if it's a blocking issue or not.

@swcurran swcurran merged commit f9d9bad into openwallet-foundation:main May 1, 2024
8 checks passed
@swcurran swcurran deleted the 0.12.1 branch May 1, 2024 19:26
@jamshale
Copy link
Contributor

jamshale commented May 1, 2024

I'm going to look into that error from the integration tests and make a ticket. I'm sure it's intermittent and already existed, but still, it shouldn't be able to happen.

Doesn't affect the release.

@jamshale
Copy link
Contributor

jamshale commented May 2, 2024

FYI on the failed integration test. Seems like, somehow the test created a race condition, by duplicating a /set-endorser-role request, with these lines.

7hAcme.agent | Updating endorser role for connection: d6283a4d-aa71-49b7-9e80-aa58f69cd8f7, TRANSACTION_ENDORSER
7hAcme.agent | Updating endorser role for connection: d6283a4d-aa71-49b7-9e80-aa58f69cd8f7, TRANSACTION_ENDORSER

creating a duplicate record. I tried to recreate the problem but haven't been able to. Because I've never seen this error before and seems very unlikely to happen on this endpoint in practice, I'd say we don't have to worry about it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants