New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update coordinators admin interface #1494
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes, but otherwise seems fine.
I'm also wondering if there is an immediate use case for this, and if so, if there is any feedback from devs who would use this? Is this mainly for debugging?
Yikes ... anything named HOT swap in Holochain is misleading... many people looking at our open repo might assume the Holo HOT token swap when I'm sure that's not what was intended! |
Ahh good point. I didn't think of that. I was just using the term like this. I'll try and think of a better name. |
@@ -87,11 +87,23 @@ impl DnaFile { | |||
} | |||
|
|||
/// Hot swap coordinator zomes for this dna. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We still have code comments referencing Hot swap. Might want to change these to Update Coordinators as well. I saw some tests as well that have the term Hot swap in there. Looks something like
@@ async fn test_hotswap() {
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good
@@ -198,3 +198,92 @@ async fn test_hotswap() { | |||
|
|||
assert_eq!(expect, dna); | |||
} | |||
|
|||
#[tokio::test(flavor = "multi_thread")] | |||
async fn test_hot_swap_checks_deps() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
async fn test_hot_swap_checks_deps() { | |
async fn test_update_coordinators_checks_deps() { |
Summary
Adds updating of coordinator zomes to the admin interface.
TODO: