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

Node/CCQ: Logging changes in watchers #4173

Conversation

bruce-riley
Copy link
Contributor

@bruce-riley bruce-riley commented Nov 27, 2024

Seeing CCQ errors in the log that are not really errors, just the users making bad requests. This PR changes those to infos. Also, for Solana, it adds another case were we will retry to read the block info before failing.

@bruce-riley bruce-riley force-pushed the node/ccq_logging_changes branch 2 times, most recently from 1dcad28 to 2024422 Compare November 27, 2024 17:04
@bruce-riley bruce-riley force-pushed the node/ccq_logging_changes branch from 2024422 to a09db1e Compare November 27, 2024 17:04
@bruce-riley bruce-riley marked this pull request as ready for review November 27, 2024 18:41
@bruce-riley bruce-riley merged commit 5e07882 into wormhole-foundation:main Nov 27, 2024
24 checks passed
@bruce-riley bruce-riley deleted the node/ccq_logging_changes branch November 27, 2024 19:20
kakucodes added a commit to kakucodes/wormhole that referenced this pull request Dec 10, 2024
Remove bruce-riley as CODEOWNER of governor module (wormhole-foundation#4166)

Node: Update libp2p (wormhole-foundation#4162)

* Node: Update libp2p

* Node: p2p address discovery change

Node/CCQ: Logging changes in watchers (wormhole-foundation#4173)

tilt: support latest docker desktop

fix clippy::needless-lifetimes warnings

by removing the binders for inferrable lifetimes

tilt: m1 flag

tilt: fix wormchain deploy

Queries/SDK: Update Solana SDK (wormhole-foundation#4180)

tilt: specify amd64 for cosmwasm containers

tilt: sdk-ci-tests default maxWorkers=50%
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.

4 participants