Skip to content

daemon: remove Daemon.children(), Daemon.parents() wrappers#49369

Merged
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:daemon_remove_wrappers
Jan 30, 2025
Merged

daemon: remove Daemon.children(), Daemon.parents() wrappers#49369
thaJeztah merged 2 commits intomoby:masterfrom
thaJeztah:daemon_remove_wrappers

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

daemon: remove Daemon.children(), Daemon.parents() wrappers

Remove the wrappers to make it more explicit that these are related to
the legacy links feature.

daemon: Daemon.restore: make legacy-link code conditional

Make it more clear that this loop is for legacy-links, and the timer is
only needed for that purpose.

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Remove the wrappers to make it more explicit that these are related to
the legacy links feature.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
Make it more clear that this loop is for legacy-links, and the timer is
only needed for that purpose.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added status/2-code-review area/daemon Core Engine kind/refactor PR's that refactor, or clean-up code labels Jan 30, 2025
@thaJeztah thaJeztah self-assigned this Jan 30, 2025
@thaJeztah thaJeztah merged commit 8860e98 into moby:master Jan 30, 2025
@thaJeztah thaJeztah deleted the daemon_remove_wrappers branch January 30, 2025 17:04
@thaJeztah thaJeztah added this to the 28.0.0 milestone Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Core Engine kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants