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

overseer: remove connect/disconnect state of the Handle #3777

Closed
drahnr opened this issue Sep 3, 2021 · 0 comments · Fixed by #3868
Closed

overseer: remove connect/disconnect state of the Handle #3777

drahnr opened this issue Sep 3, 2021 · 0 comments · Fixed by #3868

Comments

@drahnr
Copy link
Contributor

drahnr commented Sep 3, 2021

The existance of this state is merely due to a cyclic dependency that currently exists. It does not represent the state of the overseer.

The chain selection is currently built atop of that, but it should really only consider a is_relay_chain: bool which is based on the known chains that indeed are relay chains.

#3775 introduces a way to avoid breaking the cycle for subsystems, but also lays the ground work to break non-subsystem cycles, yet requires #3773 to land before this can be done cleanly.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
1 participant