Skip to content

Comments

deprecate pkg/platform and move internal#48862

Merged
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_pkg_platform
Nov 13, 2024
Merged

deprecate pkg/platform and move internal#48862
thaJeztah merged 1 commit intomoby:masterfrom
thaJeztah:deprecate_pkg_platform

Conversation

@thaJeztah
Copy link
Member

Functions in this package are only used internally in the daemon for the /info endpoint (Architecture), and as part of stats (NumProcs).

I was not able to find external consumers, but deprecating the package first, so that we can remove / dismantle the package in a follow-up.

- What I did

- How I did it

- How to verify it

- Description for the changelog

deprecate pkg/platform: this package is only used internally, and will be removed in the next release.

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

Functions in this package are only used internally in the daemon for
the `/info` endpoint (Architecture), and as part of `stats` (NumProcs).

I was not able to find external consumers, but deprecating the package
first, so that we can remove / dismantle the package in a follow-up.

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 28.0.0 milestone Nov 13, 2024
@thaJeztah thaJeztah mentioned this pull request Oct 28, 2024
79 tasks
@thaJeztah
Copy link
Member Author

Oh! Docker Hub having a bad day?

Error: Error response from daemon: Head "https://registry-1.docker.io/v2/tonistiigi/binfmt/manifests/latest": Get "https://auth.docker.io/token?account=githubactions&scope=repository%3Atonistiigi%2Fbinfmt%3Apull&service=registry.docker.io": net/http: request canceled (Client.Timeout exceeded while awaiting headers)

@thaJeztah
Copy link
Member Author

let me close/reopen to trigger CI

@thaJeztah thaJeztah closed this Nov 13, 2024
@thaJeztah thaJeztah reopened this Nov 13, 2024
@thaJeztah
Copy link
Member Author

Jenkins is green, but didn't report status to GitHub; I'll bring this one in

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

3 participants