fix(ci): Ignore lightwalletd hangs for now

[teor2345]

Motivation

Sometimes lightwalletd hangs when it gets near the tip.

We can't really fix that, so we just check that it has synced some blocks instead.

Depends-On: #4669, #4675

Solution

• Allow "adding block to cache" as an alternative tip log, where we previously required "waiting for block"

Review

This is pretty urgent, it's blocking some lightwalletd tests.

Reviewer Checklist

• CI passes

Follow Up Tasks

• ci: re-enable zebra & lightwalletd status checks #4600
• test(grpc): GetMempoolTx and GetMempoolStream test #4537

[codecov]

Codecov Report

Merging #4663 (09ba398) into main (e709420) will increase coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #4663      +/-   ##
==========================================
+ Coverage   78.83%   78.97%   +0.13%     
==========================================
  Files         304      304              
  Lines       37506    37506              
==========================================
+ Hits        29568    29619      +51     
+ Misses       7938     7887      -51     

[gustavovalverde]

Was reviewing to approve, but test is still failing even though it shows a matching regex

{"app":"lightwalletd","level":"info","msg":"Adding block to cache 1020427 000000000049429cfbc02c7d798e23e5db748310eda499977569a5484116ab46","time":"2022-06-21T07:25:34Z"}

But the actual regex is expecting a different output:

Match Regex:
   [
       "finished initial sync to chain tip, using gossiped blocks .*sync_percent.*=.*100\\.",
   ]

FMT is also complaining

Maybe @conradoplg can have a look at this one.

[conradoplg]

@Mergifyio update

[mergify]

update

✅ Branch has been successfully updated

[conradoplg]

Maybe @conradoplg can have a look at this one.

That's the Zebra sync failure which was hopefully solved by #4662 . I've updated this PR to pick up the fix.

[teor2345]

The sync failure in this PR should get better after #4669, and whatever fixes we need to fix the sync speed.

[gustavovalverde]

I'm re-testing #4669. Let's wait for the result, merge it to main and then update this PR to confirm all checks pass

[teor2345]

This is blocking a bunch of work, so I made it "Critical" priority.

[teor2345]

I'm re-testing #4669. Let's wait for the result, merge it to main and then update this PR to confirm all checks pass

#4669 passed, I manually rebased on that branch and the latest main.

So the diff will contain PR #4669 as well, but the CI results will be correct.

[teor2345]

This is still failing, it might need:

• all of Split full sync test into checkpoint and full validation jobs #4661
  • PR fix(ci): Split instance and volume creation out of the test job #4675, and
  • splitting checkpoint sync and full sync into separate jobs based on log output
• PR fix(sync): Slightly increase some syncer defaults #4679
• a successful full sync on main

[upbqdn]

@Mergifyio update

[mergify]

update

✅ Branch has been successfully updated