Skip to content
This repository has been archived by the owner on Aug 22, 2023. It is now read-only.

decommission the puppet actor #101

Closed
2 tasks done
alanshaw opened this issue Sep 2, 2020 · 3 comments
Closed
2 tasks done

decommission the puppet actor #101

alanshaw opened this issue Sep 2, 2020 · 3 comments
Assignees
@alanshaw
Labels
area/chaos-actor Areas: Chaos Actor dif/easy Difficulty: Easy effort/hours Effort: One or multiple hours. kind/chore Kind: Chore P2 P2: Should be fixed.

Comments

@alanshaw
Copy link
Member

alanshaw commented Sep 2, 2020 •
edited
Loading

Following up from #100 (review):
@alanshaw alanshaw added P2 P2: Should be fixed. kind/chore Kind: Chore area/chaos-actor Areas: Chaos Actor effort/hours Effort: One or multiple hours. dif/easy Difficulty: Easy labels Sep 2, 2020
@alanshaw alanshaw self-assigned this Sep 2, 2020
This was referenced Sep 2, 2020
Merged
Merged
@willscott
Copy link
Contributor

is there a reason we prefer chaos at 98 rather than 97? or just to make it clear that it replaces puppet?

@raulk
Copy link
Member

raulk commented Sep 2, 2020

Yeah, to replace the puppet actor and not leave any gaps in the address space.

@raulk
Copy link
Member

raulk commented Sep 3, 2020

@alanshaw I'll leave the satisfaction to you to close out this issue when filecoin-project/lotus#3521 is merged ;-)

@alanshaw alanshaw closed this as completed Sep 4, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@alanshaw alanshaw

Labels
area/chaos-actor Areas: Chaos Actor dif/easy Difficulty: Easy effort/hours Effort: One or multiple hours. kind/chore Kind: Chore P2 P2: Should be fixed.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@alanshaw @willscott @raulk