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

Fix issuer index in context of idp update/replace #3748

Open
wants to merge 11 commits into
base: develop
Choose a base branch
from

Conversation

akshaymankar
Copy link
Member

@akshaymankar akshaymankar commented Nov 27, 2023

https://wearezeta.atlassian.net/browse/WPB-5692

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@fisx fisx force-pushed the fisx-spar-refactorings-2023-11-27 branch from 119b241 to 20a4ab4 Compare November 28, 2023 07:38
@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Nov 28, 2023
@fisx fisx marked this pull request as ready for review November 28, 2023 08:23
@fisx fisx changed the title Spar refactorings Make internal (non-user-observable) behavior of Spar easier to understand. Nov 29, 2023
@fisx fisx force-pushed the fisx-spar-refactorings-2023-11-27 branch from e083a2e to 39fd043 Compare November 30, 2023 10:34
@fisx fisx force-pushed the fisx-spar-refactorings-2023-11-27 branch 3 times, most recently from 6133171 to c2788d0 Compare December 12, 2023 08:51
@fisx fisx changed the title Make internal (non-user-observable) behavior of Spar easier to understand. Fix issuer index in context of idp update/replace Jan 8, 2024
@fisx fisx force-pushed the fisx-spar-refactorings-2023-11-27 branch from 14081b5 to 0d17e81 Compare January 18, 2024 13:01
This is a bug that has caused some garbage not being collected in C*.
Non-critical.
This means it's allowed to revert idp updates without
"idp-already-used" error.
The old logic was incoherent and incomprehensible.
- remove threadDelay (temporarily added for ruling out race conditions
during debugging).
- make sure users are removed right after it should happen, not at the
end of the test.
@fisx fisx force-pushed the fisx-spar-refactorings-2023-11-27 branch from 0d17e81 to d982ffa Compare January 24, 2024 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants