Skip to content

daemon/graphdriver: remove deprecated GetDriver(), redundant init(), and leftovers for graphdriver-plugins#50377

Merged
thaJeztah merged 3 commits intomoby:masterfrom
thaJeztah:graphdriver_cleanups
Jul 11, 2025
Merged

daemon/graphdriver: remove deprecated GetDriver(), redundant init(), and leftovers for graphdriver-plugins#50377
thaJeztah merged 3 commits intomoby:masterfrom
thaJeztah:graphdriver_cleanups

Conversation

@thaJeztah
Copy link
Member

relates to:

daemon/graphdriver: remove redundant init()

This was added in a63ff8d, but looks
like the only reason was to just have the var initialized, so let's
do so when we create the var.

daemon/graphdriver: remove deprecated GetDriver()

This was deprecated in 84cabde, which
was part of v28.

daemon/graphdriver: remove error or deprecated graphdriver-plugins

This error was added in 555dac5 to produce
an error for the deprecated graphdriver-plugins.

- Human readable description for the release notes

daemon/graphdriver: remove deprecated `GetDriver()`

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

This was added in a63ff8d, but looks
like the only reason was to just have the var initialized, so let's
do so when we create the var.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This was deprecated in 84cabde, which
was part of v28.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
This error was added in 555dac5 to produce
an error for the deprecated graphdriver-plugins.

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah thaJeztah added this to the 29.0.0 milestone Jul 10, 2025
@thaJeztah thaJeztah changed the title daemon/graphdriver: remove deprecated GetDriver(), redundant init(), and left-overs for graphdriver-plugins daemon/graphdriver: remove deprecated GetDriver(), redundant init(), and leftovers for graphdriver-plugins Jul 11, 2025
@thaJeztah thaJeztah merged commit eb8c54c into moby:master Jul 11, 2025
328 of 334 checks passed
@thaJeztah thaJeztah deleted the graphdriver_cleanups branch July 11, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants