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

reverted eth_call shouldn't be marked as error in dd #852

Open
shunjizhan opened this issue Oct 3, 2023 · 1 comment
Open

reverted eth_call shouldn't be marked as error in dd #852

shunjizhan opened this issue Oct 3, 2023 · 1 comment
Labels
infra infra related Lv. 3 need some research to find out root cause or/and solution

Comments

@shunjizhan shunjizhan added infra infra related Lv. 2 relatively clear issue, might need some small digging labels Oct 3, 2023
@shunjizhan shunjizhan changed the title reverted eth_call shouldn't be marked as error when in dd reverted eth_call shouldn't be marked as error in dd Oct 3, 2023
@shunjizhan shunjizhan added Lv. 3 need some research to find out root cause or/and solution and removed Lv. 2 relatively clear issue, might need some small digging labels Jan 24, 2024
@shunjizhan
Copy link
Collaborator Author

shunjizhan commented Nov 8, 2024

also, getFilterChanges should not report filter not found as error

https://app.datadoghq.com/apm/services/eth_rpc/operations/eth_rpc_call/resources?dependencyMap=qson%3A%28data%3A%28telemetrySelection%3Aall_sources%29%2Cversion%3A%210%29&deployments=qson%3A%28data%3A%28search%3A2.9.1%2Chits%3A%28selected%3Aversion_count%29%2Cerrors%3A%28selected%3Aversion_count%29%2Clatency%3A%28selected%3Ap95%29%2CtopN%3A%215%29%2Cversion%3A%210%29&env=bodhi-acala&errors=qson%3A%28data%3A%28issueSort%3AFIRST_SEEN%29%2Cversion%3A%210%29&fromUser=false&graphType=flamegraph&groupMapByOperation=null&infrastructure=qson%3A%28data%3A%28viewType%3Apods%29%2Cversion%3A%210%29&isInferred=false&logs=qson%3A%28data%3A%28indexes%3A%5B%5D%29%2Cversion%3A%210%29&panels=qson%3A%28data%3A%28activePanelKey%3Atrace%2Ctrace%3A%28traceID%3A3898244228048844150%2CspanID%3A7914983786551421423%29%29%2Cversion%3A%210%29&shouldShowLegend=true&sort=time&spanID=7914983786551421423&summary=qson%3A%28data%3A%28visible%3A%21t%2Cchanges%3A%28%29%2Cerrors%3A%28selected%3Acount%29%2Chits%3A%28selected%3Acount%29%2Clatency%3A%28selected%3Alatency%2Cslot%3A%28agg%3A95%29%2Cdistribution%3A%28isLogScale%3A%21f%29%2CshowTraceOutliers%3A%21t%29%2Csublayer%3A%28slot%3A%28layers%3Aservice%29%2Cselected%3Apercentage%29%2ClagMetrics%3A%28selectedMetric%3A%21s%2CselectedGroupBy%3A%21s%29%29%2Cversion%3A%211%29&timeHint=1731038734979&trace=AwAAAZMJ8uqDrklb2QAAABhBWk1KOHZTcEFBQ3hSR3NZRWx2c2tCazIAAAAkMDE5MzA5ZjItZmY4Ni00OGJiLTgxZGItOTk5NmM5YTg5MDExAAAABQ&traceID=3898244228048844150&traces=qson%3A%28data%3A%28%29%2Cversion%3A%210%29&version=2.9.1&start=1731024311606&end=1731038711606&paused=false

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
infra infra related Lv. 3 need some research to find out root cause or/and solution
Projects
None yet
Development

No branches or pull requests

1 participant