-
Notifications
You must be signed in to change notification settings - Fork 18
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 ephemeral container to a running pod #34
Conversation
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
@@ -17,12 +20,27 @@ func New(client *kubernetes.Clientset, metaOptions metav1.ListOptions, ctx conte | |||
} | |||
} | |||
|
|||
type ContainerOptions struct { |
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.
Please include Go documentation on any public types and methods.
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.
Hi @javaducky I added some comments, but I'm not sure what conventions is the project following in this regard. Could you please point me to some examples I can use as a reference?
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.
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.
Added documentation.
Confirmed create and add with Kube v1.22.7 ( |
Signed-off-by: Pablo Chacin <pablochacin@gmail.com>
fb0c135
to
611489a
Compare
LGTM! Thank you @pablochacin . |
Add function to Pods for creating an ephemeral container in a running pod.
Also added a function for creating a pod to facilitate testing.
Note: tested with K8s cluster 1.23.x. Previous versions may not work or may need enabling ephemeral containers feature gate
closes #30