-
Notifications
You must be signed in to change notification settings - Fork 565
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
implement add-hosts flag && extra_hosts #328
Conversation
8b8374a
to
058b14c
Compare
cmd/nerdctl/internal_oci_hook.go
Outdated
Name: "add-host", | ||
Usage: "Add a custom host-to-IP mapping (host:ip)", | ||
}, | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not use labels as in other flags?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AkihiroSuda you mean rename add-host
to labels
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, using labels in https://github.com/containerd/nerdctl/blob/master/pkg/labels/labels.go
Other ocihook features use these labels
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@AkihiroSuda fixed
41c20a4
to
116d735
Compare
Signed-off-by: fahed dorgaa <fahed.dorgaa@gmail.com>
116d735
to
aa6f001
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks
Hello
add-host
flagextra_hosts
related ticket #314