fix(ci): Avoid caching state in lwd-update-sync
job
#9029
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
This test is failing in CI since #9004, it's not clear why it was passing before then as the expected log was missing before #9004 as well.
The test documentation says it's meant to be run with an empty Zebra and lightwalletd test (and it is consistent with the test code). The test should not be producing an updated cache state, it should never be saved to a cached disk for other tests to use.
Depends-On: #9026.
Solution
Sets the
needs_zebra_state
,needs_lwd_state
,save_to_disk
, andforce_save_to_disk
Github action inputs as false for thelightwalletd-update-sync
job.Related changes:
Tests
The test should pass in CI for this PR.
PR Author's Checklist
PR Reviewer's Checklist