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 initial run/store interface #1

Open
wants to merge 1 commit into
base: master
from

Conversation

@ddadlani
Copy link

ddadlani commented Jun 3, 2019

Rendered

Signed-off-by: Divya Dadlani <ddadlani@pivotal.io>
@ddadlani ddadlani force-pushed the ddadlani:master branch from c69d281 to 99ca1a6 Jun 3, 2019
@ddadlani ddadlani requested review from pivotal-jwinters and vito Jun 3, 2019
specified in the task definition as opposed to the pipeline, there exists a use case where a user
may first do a `get` and then run a `task` that was defined in the output `Artifact` of the `get`.

To satisfy these use cases, we need to `Store()` and `Retrieve()` information from within an

This comment has been minimized.

Copy link
@xtreme-sameer-vohra

xtreme-sameer-vohra Jun 5, 2019

Is Retrieve required ? Specially because we're indicating that Runnable would be able to populate inputs from the Storage interface. Unless Runnable would consume a list of artifacts and use Retrieve

This comment has been minimized.

Copy link
@ddadlani

ddadlani Jun 5, 2019

Author

@xtreme-sameer-vohra Retrieve is required for fly execute. If you fly execute a task with outputs, the outputs would need to be downloaded to your local computer. The API would need to call Retrieve to do this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.