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

import-url: --version-aware does not apply to directories #8506

Closed
pmrowla opened this issue Nov 1, 2022 · 0 comments · Fixed by #8528
Closed

import-url: --version-aware does not apply to directories #8506

pmrowla opened this issue Nov 1, 2022 · 0 comments · Fixed by #8528
Assignees
Labels
A: cloud-versioning Related to cloud-versioned remotes A: data-sync Related to dvc get/fetch/import/pull/push bug Did we break something?

Comments

@pmrowla
Copy link
Contributor

pmrowla commented Nov 1, 2022

    One more thing: `--version-aware` doesn't seem to do anything for directories. I would expect it to include the list of all files and the `version_id` for each inside the `.dvc` file.

If I do dvc import-url --version-aware s3://dave-sandbox-versioning/example-cloud-remote/data/, the data.dvc file looks like:

md5: 5e46b5ab89ac35532c82915752d2ddbe
frozen: true
deps:
- md5: 33ab906cdada8d288429ab9884d8e08b.dir
  size: 21
  nfiles: 3
  path: s3://dave-sandbox-versioning/example-cloud-remote/data/
outs:
- md5: 1fff39c58ffb2a046fb72f52eaff02db.dir
  size: 21
  nfiles: 3
  path: data

Originally posted by @dberenbaum in #8249 (comment)

@pmrowla pmrowla self-assigned this Nov 1, 2022
@pmrowla pmrowla added bug Did we break something? A: data-sync Related to dvc get/fetch/import/pull/push A: cloud-versioning Related to cloud-versioned remotes labels Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: cloud-versioning Related to cloud-versioned remotes A: data-sync Related to dvc get/fetch/import/pull/push bug Did we break something?
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant