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

Metadata files upload/download #13918

Merged

Conversation

memsharded
Copy link
Member

@memsharded memsharded commented May 18, 2023

Changelog: Feature: New metadata files associated to recipes and packages that can be uploaded, downloaded and added after the package exists.
Docs: Omit

This proposal implements upload + download flows, but not conan install or conan create installation of metadata. That would add too much extra complexity to the code and uncertainty and fragile developer flows, like metadata not being pulled from server because the package is already in the cache, despite users trying to conan install --metadata=*, thinking that it will pull new metadata from servers.

So probably the best flow to support full-graph download of metadata would be:

  • conan install ... --format=json > graph.json
  • convert graph.json => pkg_list.json
  • conan download --list=pkg_list.json --metadata=logs*

This flow has been already implemented in #13928

Close #13715
Close #4610

@memsharded memsharded added this to the 2.0.6 milestone May 18, 2023
@memsharded memsharded requested a review from AbrilRBS May 18, 2023 09:59
Copy link
Member

@AbrilRBS AbrilRBS left a comment

Choose a reason for hiding this comment

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

Seems like we'll need to implement the package list inputs sooner rather than later then :)

@memsharded memsharded modified the milestones: 2.0.6, 2.0.7 May 24, 2023
@memsharded memsharded merged commit a5d436e into conan-io:release/2.0 Jun 16, 2023
@memsharded memsharded deleted the feature/metadata_cmd_test_package branch June 16, 2023 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[question] Test reports Embed test_package in the recipe
4 participants