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 goleveldb via dex 0.5.5 update #1934

Merged
merged 1 commit into from
Nov 1, 2022
Merged

Conversation

chappjc
Copy link
Member

@chappjc chappjc commented Nov 1, 2022

With goleveldb at v1.0.1-0.20220614013038-64ee5596c38a:

2022-10-31 17:10:30.794 [INF] SKDB: Successfully loaded 707739 ticket pool diffs
2022-10-31 17:10:31.243 [ERR] SKDB: Error reading from database (missing key 707739 for new tickets).  Reinitializing.
2022-10-31 17:10:31.243 [DBG] SKDB: Initialized new stake db.
2022-10-31 17:10:31.251 [DBG] SKDB: badger: Got compaction priority: {level:0 score:1.73 dropPrefixes:[]}
2022-10-31 17:10:31.253 [ERR] DATD: Unable to create stake DB: bucket already exists
2022-10-31 17:10:31.253 [INF] DATD: Closing connection to dcrd.
2022-10-31 17:10:31.253 [INF] DATD: Bye!
2022-10-31 17:10:31.504 [ERR] DATD: StakeDatabase recovery failed: bucket already exists

This PR resolves the issue by updating to dcrdex v0.5.5, which has the goleveldb require reverted to 2ae1ddf74ef7

Comment on lines -1162 to -1163
github.com/syndtr/goleveldb v1.0.1-0.20220614013038-64ee5596c38a h1:1ur3QoCqvE5fl+nylMaIr9PVV1w343YRDtsy+Rwu7XI=
github.com/syndtr/goleveldb v1.0.1-0.20220614013038-64ee5596c38a/go.mod h1:RRCYJbIwD5jmqPI9XoAFR0OcDxqUctll6zUj/+B4S48=
Copy link
Member Author

@chappjc chappjc Nov 1, 2022

Choose a reason for hiding this comment

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

This little gremlin took me a very long time to smoke out.

It was breaking dcrd's database/v3/ffldb code at stake.LoadBestNode > ticketdb.DbFetchNewTickets, at least.

And it was forcing this revision because of dcrdex -> go-ethereum 1.10.20, which had the bad revision required, thus hoisting the revision used by dcrd.

@chappjc
Copy link
Member Author

chappjc commented Nov 1, 2022

Work as intended on affected host

@chappjc chappjc merged commit eb9df87 into decred:master Nov 1, 2022
@chappjc chappjc deleted the repair-goleveldb branch November 1, 2022 21:38
@chappjc chappjc added this to the 6.1.1 milestone Nov 2, 2022
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.

1 participant