-
Notifications
You must be signed in to change notification settings - Fork 46
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
Refactor Exporter models based on pulpcore changes #366
Conversation
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
9956c03
to
755a3c4
Compare
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
e62011f
to
8e20fde
Compare
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
0cc9d0a
to
17898bc
Compare
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
""" | ||
FileSystemExporters export content from a publication to a path on the file system. | ||
|
||
WARNING: This feature is provided as a tech preview and may change in the future. Backwards | ||
compatibility is not guaranteed. | ||
""" | ||
|
||
endpoint_name = "file" | ||
endpoint_name = "filesystem" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Attached issue: https://pulp.plan.io/issues/6328 |
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
Also adding a new Export model and refactoring existing Exporters. fixes #6328 https://pulp.plan.io/issues/6328 Required PR: pulp/pulp_file#366
b764f38
to
4b5400f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@daviddavis thank you! I left one last optional comment on the 6328.removal
file. Handle as you like I'm lgtm-ing now. 👍
ref #6328
https://pulp.plan.io/issues/6328
Required PR: pulp/pulpcore#585