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

Ensure there is an easy way to provide container input (and to get output) #1503

Closed
bgrant0607 opened this issue Sep 30, 2014 · 5 comments
Closed
Labels
area/api Indicates an issue on api area. area/app-lifecycle area/downward-api priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery.

Comments

@bgrant0607
Copy link
Member

Simple data-in/data-out restart-until-success (or run-once/restart-never) pods need an easy way to receive input data. Such pods could be used for configuration generation functions, workflow steps, cron actions, etc.

There also needs to be an easy way to retrieve at least simple/small outputs, as discussed in #139, so I'm mostly not going to discuss that here.

Options:

  1. Surface Docker attach to stdin and stdout
  2. Another possibility would be to support an input field in the container spec, which would be fed into stdin. That approach would preclude reuse of a pod template for multiple inputs, though. Using a command-line argument would have that same problem. A URL parameter on the creation POST could potentially work.
  3. Another approach, appropriate for larger inputs, could be image volumes (Image volumes and container volumes #831), though the input image would need to be built and pushed prior to launching the consuming pod.
@jbeda
Copy link
Contributor

jbeda commented Sep 30, 2014

This seems a little overkill -- why not instead just have a bootstrap script that (a) takes a reference from an env variable (b) downloads/reads it (c) streams to the real executable.

This might be some useful sugar but it doesn't seem super core.

@bgrant0607
Copy link
Member Author

This issue is about ensuring we have some mechanism that makes this easy. I wanted to solicit others' thoughts/ideas, so thanks for your suggestion.

A PreStart hook could potentially work, once that's implemented. A disadvantage of this is that it would require adding the fetcher to the image.

@bgrant0607 bgrant0607 added this to the v1.0 milestone Oct 4, 2014
@bgrant0607 bgrant0607 added the priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. label Dec 3, 2014
@bgrant0607 bgrant0607 removed this from the v1.0 milestone Dec 3, 2014
@proppy
Copy link
Contributor

proppy commented Feb 6, 2015

  1. allow the pod create API call to take an arbitrary data payload?
    (I could see that being useful for docker run too, I wonder if that open the door to all sort of crazy issues and abuses).

@bgrant0607 bgrant0607 added the sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery. label Feb 28, 2015
@bgrant0607
Copy link
Member Author

cc @erictune

@bgrant0607
Copy link
Member Author

we support kubectl run -t -i now

mpatlasov pushed a commit to mpatlasov/kubernetes that referenced this issue Mar 20, 2023
…erry-pick-1498-to-release-4.12

[release-4.12] OCPBUGS-8705: Fix mounted volume expansion tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/api Indicates an issue on api area. area/app-lifecycle area/downward-api priority/awaiting-more-evidence Lowest priority. Possibly useful, but not yet enough support to actually get it done. sig/api-machinery Categorizes an issue or PR as relevant to SIG API Machinery.
Projects
None yet
Development

No branches or pull requests

3 participants