Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add HostConfig to nerdctl inspect response #3812

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

coderbirju
Copy link
Contributor

nerdctl inspect currently doesn't report HostConfig details in its response.

Attempting to add this PR as a starting point, aim is to get the dockercompat response closer to the docker inspect response schema

pkg/labels/labels.go Outdated Show resolved Hide resolved
@coderbirju coderbirju force-pushed the add-hostconfig-inspect-response branch from f57dca7 to cfdf044 Compare January 15, 2025 05:15
pkg/labels/labels.go Outdated Show resolved Hide resolved
@coderbirju coderbirju force-pushed the add-hostconfig-inspect-response branch 5 times, most recently from 0b7af18 to 64ad424 Compare January 22, 2025 22:52
@coderbirju coderbirju changed the title [WIP]: Add HostConfig to nerdctl inspect response Add HostConfig to nerdctl inspect response Jan 22, 2025
@coderbirju coderbirju force-pushed the add-hostconfig-inspect-response branch 19 times, most recently from 7052b62 to c5794d5 Compare January 30, 2025 17:41
@coderbirju coderbirju force-pushed the add-hostconfig-inspect-response branch 2 times, most recently from a002197 to f36f024 Compare January 30, 2025 18:54
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
Signed-off-by: Arjun Raja Yogidas <[email protected]>
@coderbirju coderbirju force-pushed the add-hostconfig-inspect-response branch from f36f024 to 41ace87 Compare January 30, 2025 19:27
@coderbirju
Copy link
Contributor Author

@AkihiroSuda PTAL, I've tried to limit the use of labels and use the native inspect as much as possible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants