Document --backup import behaviour #1788
Conversation
jorgeorpinel
left a comment
jorgeorpinel
left a comment
There was a problem hiding this comment.
Thanks a lot for this! Great start. Needs some clarification and expansion still. Oh, and formatting! Please see https://dvc.org/doc/user-guide/contributing/docs#development-environment
Co-authored-by: Jorge Orpinel <jorgeorpinel@users.noreply.github.com>
| ## Checkout failed with `store: false` outputs {#checkout-no-store} | ||
|
|
||
| DVC is unable to checkout any output that is not present in the | ||
| <abbr>cache</abbr>.This can usually be solved by fetching from | ||
| [import storage](/doc/command-reference/remote). However, If `store: false` | ||
| is used in a `.dvc` file, `dvc pull` and `dvc fetch` won't attempt to retrieve the | ||
| data from a DVC remote. |
There was a problem hiding this comment.
We may need to update this title, anchor, and description a little per treeverse/dvc#4581 (review)...
| is used in a `.dvc` file, `dvc pull` and `dvc fetch` won't attempt to retrieve the | ||
| data from a DVC remote. | ||
|
|
||
| To solve this, you must provide the file locally. |
There was a problem hiding this comment.
Is it possible to suggest more elaborate possible solution(s) that will cover most of the cases? Not sure what the options are but off the top of my head: push the data from the original repo; remove store: false and try fetch/pull again; re-import; what else? If you can just write them here I'll copy edit it as needed. Thanks!
jorgeorpinel
left a comment
jorgeorpinel
left a comment
There was a problem hiding this comment.
Will need another round of updates but almost there. Maybe let's wait until the core PR is finalized too? Oh and it needs merge of master branch (solve conflicts) 😅
jorgeorpinel
added
the
⌛ status: wait-core-merge
| - `store`: Whether this file or directory can be | ||
| [pushed](/doc/command-reference/push) and | ||
| [pulled](/doc/command-reference/pull) to/from | ||
| [remote storage](/doc/command-reference/remote) (`true` by default for | ||
| all outputs except for imports, where it is `false` by default). See |
There was a problem hiding this comment.
except for imports
Per #1865 (comment) this isn't the only exception. External outputs are also never stored remotely (since they are already in an external cache).
|
Closing this as stale. We can reopen when/if the core PR gets merged. Thanks |
3 participants
Documenting the changes in treeverse/dvc#4581
REVIEW UPDATE: Jump to #1788 (comment)