Skip to content
This repository has been archived by the owner. It is now read-only.

add initial run/store interface #1

Closed
wants to merge 1 commit into from

Conversation

ddadlani
Copy link

@ddadlani ddadlani commented Jun 3, 2019

Signed-off-by: Divya Dadlani <ddadlani@pivotal.io>
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

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

Copy link
Author

@ddadlani ddadlani Jun 5, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@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.

@muntac
Copy link

@muntac muntac commented Jan 11, 2021

Closing this. Some of this will be addressed in concourse/concourse#6387.

@muntac muntac closed this Jan 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants