Andy Nogueira
Andy Nogueira
Just an update here, I am waiting the nightly recurring errors to be fixed before adding this again to avoid introducing potential additional errors on nightly
now that the nightly tests are passing again, will try to merge this back
A lot of tests were failing so not merging this for now. I will close this PR and open an issue to follow up in the future
I think this might be a valid point based on the referenced PRs in the SDK. We need to test and see why not all services are stopped and the...
@alesforz can you please take this one over since yo are already working on related logic?
I've also did a quick test on a local setup with a `max_discovery_time` of `30 seconds` and it switches to `blocksync` ``` I[2024-10-25|19:59:43.213] Starting state sync module=statesync I[2024-10-25|19:59:43.213] Downloading trusted...
> I wonder whether we shouldn't consider to have an additional command to the runner to dump all the data. I am considering here, in particular, when we use the...
> Recalling that the "full" tests means: `setup`, `start`, `load`, `perturb`, `wait`, `test`, `stop`, and `cleanup`. To save data we should call this new method between `stop` and `cleanup`. I...
I'm going to switch this to a draft PR for now until we have some decision and agreement on implementation details and I can tackle them.
Blocked until it is refactored as per comments and suggestions above.