Closed shenkeyao closed 1 week ago
I think this looks good, but I noticed we haven't included any tests. Could we add some for the changed tasks?
Sure. I originally planed to add tests after reorganizing consensus2 functions because I found it a bit confusing, but on a second thought, I think it makes sense to add tests now. Will send an update!
Converting this to a draft until I turn the upgrade task tests on for the dependency tasks.
Tests added. Marking this as ready for review.
Closes #3356.
This PR:
Arc<RwLock<...>>
decided upgrade cert as parameter totemp_*
, and adds new functions that takeArc<RwLock<...>>
.temp_*
functions that will be replaced in a separate PR.This PR does not:
3378
3357
Key places to review:
temp_*
functions may be ignored since they only have the name change.)How to test this PR:
RUST_LOG=error,hotshot_task_impls=info just cargo run --example all-push-cdn --features example-upgrade --features dependency-tasks -- --config_file ./crates/orchestrator/run-config.toml
.dependency-tasks
feature.