chore(pdp): add more logging; don't bail early on tx error #549

rvagg · 2025-06-24T06:08:11Z

Parking this as a draft for now, trying to figure out #548

rvagg · 2025-06-24T06:26:40Z

tasks/message/watch_eth.go

+				"error", err,
+				"errorType", fmt.Sprintf("%T", err))
+			// Continue processing other transactions instead of returning
+			continue


I'm suspecting that the change here from a return to a continue might be part of the issue - if we return while we have a tx that gives us something other than ethereum.NotFound early in the list, then the remainder won't be checked

rvagg · 2025-06-24T06:28:51Z

This cleared up the queue on our test server, but I'm not sure if it was the restart that did it or the changes here. But I think this is useful, it gives much more insight in the logs to PDP interactions and tx processing.

pdp/handlers.go

tasks/message/watch_eth.go

* ServiceAuth and NullAuth (#512) * ServiceAuth and NullAuth * fix --------- Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * Fix/public service sql mig (#516) * Add sql migration function to enable public service with dummy key * Fix typo --------- Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * Tigher busy waiting (#521) Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * fix (#523) Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * Look in pdp piecerefs to check for actual readiness (#525) * Look in pdp piecerefs to check for actual readiness * fix --------- Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * YOLO (#526) Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * Remove unpadding transformation from root size (#529) Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> * feat(pdp): add transaction tracking for root additions (synapse branch) (#530) * feat(pdp): add transaction tracking for root additions - Add Location header to root addition response with tx hash - Add GET /proof-sets/{id}/roots/added/{txHash} endpoint for status checks - Implement handleGetProofSetRoot for retrieving root details - Return confirmed root IDs after transaction confirmation * fix(pdp): use lowercase transaction hashes consistently in database operations * fix(pdp): retain root addition records to prevent 404s on status queries Previously, pdp_proofset_root_adds records were deleted after processing, causing clients to receive 404 errors when querying transaction status after successful root additions. This created a poor UX as clients couldn't verify transaction outcomes if they queried too late. This change aligns root addition tracking with proof set creation by: - Adding roots_added column to mark processed records instead of deleting - Updating watcher to UPDATE instead of DELETE when processing completes - Including roots_added status in API response for client visibility The fix ensures transaction status remains queryable indefinitely while maintaining backward compatibility. Existing records will be processed normally with roots_added=FALSE by default. API enhancement: /pdp/proof-sets/{id}/roots/added/{txHash} now includes "rootsAdded" field indicating if roots were fully processed. * fix(pdp): extra checks to make sure the db entries are consistent * fix(pdp): address review feedback * chore(pdp): add more logging; don't bail early on tx error (#549) * chore(pdp): add transaction processing logging * chore(pdp): tweak logging statements * fix(chain): improve chain scheduler robustness and eth tx monitoring (#553) - Add mutex protection and notification timeout handling to CurioChainSched - Implement timeout protection for eth client calls to prevent indefinite blocking - Add comprehensive test coverage for chain scheduler and eth message watcher - Extract interfaces (EthClient, TaskEngine, EthTransactionManager) for testability * chore(logs): log proof data as hex (#551) --------- Co-authored-by: ZenGround0 <5515260+ZenGround0@users.noreply.github.com> Co-authored-by: zenground0 <ZenGround0@users.noreply.github.com> Co-authored-by: Rod Vagg <rod@vagg.org>

chore(pdp): add transaction processing logging

c78da4f

rvagg force-pushed the rvagg/txlogging branch from 70460ab to c78da4f Compare June 24, 2025 06:11