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

quicksilverd "failed to process message" errors for TMKMS signed votes #798

Closed
gaia opened this issue Oct 25, 2023 · 4 comments
Closed

quicksilverd "failed to process message" errors for TMKMS signed votes #798

gaia opened this issue Oct 25, 2023 · 4 comments

Comments

@gaia
Copy link

gaia commented Oct 25, 2023

tmkms::commands::start: tmkms 0.12.2 starting up...
tmkms::keyring: [keyring:yubihsm] added consensus Ed25519 key: quickvalconspub1
tmkms::connection::tcp: KMS node ID: 9...
tmkms::session: [quicksilver-1@tcp://a.b.c.d:26658] connected to validator successfully
2023-10-13T22:04:41.639895Z  WARN tmkms::session: [quicksilver-1@tcp://a.b.c.d:26658]: unverified validator peer ID! (1...)
tmkms::session: [quicksilver-1@tcp://a.b.c.d:26658] signed PreCommit:<nil> at h/r/s 4135053/0/2 (104 ms)
tmkms::session: [quicksilver-1@tcp://a.b.c.d:26658] signed PreVote:102122D292 at h/r/s 4135054/0/1 (126 ms)
tmkms::session: [quicksilver-1@tcp://a.b.c.d:26658] signed PreCommit:102122D292 at h/r/s 4135054/0/2 (126 ms)

and

quicksilverd tendermint show-validator | jq .key == cat priv_validator_key.json | jq .pub_key.value

from where the only assymetric key present in the YubiHSM was imported.

on the node side,

{"level":"error","module":"consensus","err":"error adding vote","height":4135063,"msg_type":"*consensus.VoteMessage","peer":"","round":0,"time":"2023-10-13T15:05:40-07:00","message":"failed to process message"}
{"level":"error","module":"consensus","err":"error adding vote","height":4135063,"msg_type":"*consensus.VoteMessage","peer":"","round":0,"time":"2023-10-13T15:05:40-07:00","message":"failed to process message"}
{"level":"error","module":"consensus","err":"error adding vote","height":4135064,"msg_type":"*consensus.VoteMessage","peer":"","round":0,"time":"2023-10-13T15:05:46-07:00","message":"failed to process message"}
{"level":"error","module":"consensus","err":"error adding vote","height":4135064,"msg_type":"*consensus.VoteMessage","peer":"","round":0,"time":"2023-10-13T15:05:46-07:00","message":"failed to process message"}

pls ignore that the top snippet is on block 4135054 and the bottom on 4135053+. the errors were consistent.

As requested, new issue opened up.

@tony-iqlusion
Copy link
Member

tony-iqlusion commented Oct 25, 2023

"failed to process message" isn't a whole lot to go on. For whatever reason quicksilverd is rejecting votes signed by TMKMS.

Can you open a corresponding issue on the Quicksilver side?

Also it'd be good if you could upgrade to TMKMS v0.13.0, though I doubt that will correct the issue.

@tony-iqlusion tony-iqlusion changed the title failed to process message quicksilverd "failed to process message" errors for TMKMS signed votes Oct 25, 2023
@tony-iqlusion
Copy link
Member

Is it possible this was related to #835?

Can you try TMKMS v0.13.1?

@tony-iqlusion
Copy link
Member

Please try TMKMS v0.14.0-pre.1

@gaia
Copy link
Author

gaia commented Apr 9, 2024

current version works fine, thanks! opening up another ticket for something else.

@gaia gaia closed this as completed Apr 9, 2024
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

No branches or pull requests

2 participants