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

Votes for amendments not persisting #4220

Closed
alloynetworks opened this issue Jul 5, 2022 · 3 comments
Closed

Votes for amendments not persisting #4220

alloynetworks opened this issue Jul 5, 2022 · 3 comments
Labels

Comments

@alloynetworks
Copy link
Contributor

Issue Description

Multiple validators have noted that the votes using rippled feature AMENDMENTID accept/reject do not persist after restart of rippled

Steps to Reproduce

Restart rippled and check output of rippled feature. It may not match current votes that operator has registered.

Expected Result

Aside from the first run where rippled migrates from the rippled.cfg file into wallet.db, additional accept/reject should be stored in wallet.db

Actual Result

Votes do not persist in wallet.db after restart

Additional information

This issue has been discussed with @nbougalis and warrants further examination to confirm/deny. If this bug does indeed exist, a fix needs to be merged into the next release, else votes for amendments not yet activated will change either for/against depending on the default.

@rippleitinnz
Copy link

rippleitinnz commented Jul 5, 2022

This happened to me last night after I restarted my server. All 4 current votes to 'accept' amendments, that had previously been cast, failed to persist and returned to the default of 'reject' on restart. The votes have had to be cast again.

@WietseWind
Copy link
Member

I had the same thing. I moved bare metal and moved wallet.db along. All votes had to be put in again. I tried to reproduce by simply restarting the validator and indeed: votes gone again. It seems not to persist the votes.

@jonaagenilsen
Copy link

I had an old timestamp on /var/lib/rippled/db/wallet.db..

I reversed a vote with
/opt/ripple/bin/rippled feature B2A4DB846F0891BF2C76AB2F2ACC8F5B4EC64437135C6E56F3F859DE5FFD5856 reject # 1.9.1 ExpandedSignerList

then re-enabled it again with
/opt/ripple/bin/rippled feature B2A4DB846F0891BF2C76AB2F2ACC8F5B4EC64437135C6E56F3F859DE5FFD5856 accept # 1.9.1 ExpandedSignerList

So from my observation, timestamp (and contents?) seem to update wallet.db atleast. Possible there's an issue with reading /var/lib/rippled/db/wallet.db properly after a daemon-restart?

HowardHinnant added a commit to HowardHinnant/rippled that referenced this issue Jul 5, 2022
* Ensures that if old data exists, it is replaced.
* Fixes: XRPLF#4220
HowardHinnant added a commit to HowardHinnant/rippled that referenced this issue Jul 7, 2022
@XRPLF XRPLF deleted a comment Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants
@WietseWind @jonaagenilsen @rippleitinnz @alloynetworks and others