Issue #1449
closed
Users cannot sync v1 content from a registry that implements the v2 API
Status:
CLOSED - CURRENTRELEASE
Target Release - Docker:
2.0.0
Description
The current Pulp sync implementation tries to use the remote registry via the v2 API, and if the registry does not implement v2 it falls back to v1. However, if the registry does implement v2, v1 is never tried. Satellite 6 plans to use pulp_docker's sync function instead of Nodes to move content from a parent to a capsule. Due to the current implementation, any repositories that have v1 and v2 content will only be able to mirror v2 content into the capsule.
When fixing this issue, consider whether it would be advantageous or not to make the fallback behavior a boolean option. Not making it optional has the advantage of being simpler (no additional settings needed). Making it optional has the advantage of giving users control in situations where perhaps they only want to use the v2 API and don't want all the images downloaded twice. You are somebody's kid, do what you think is right.
- Description updated (diff)
- Priority changed from Normal to High
- Status changed from NEW to ASSIGNED
- Assignee set to mhrivnak
- Triaged changed from No to Yes
- Status changed from ASSIGNED to POST
- Target Release - Docker set to 2.0.0
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
- Status changed from MODIFIED to ASSIGNED
This issue also needs to be fixed on the docker_v2_api branch, so I'm sending it back to assigned.
- Assignee changed from mhrivnak to rbarlow
- Status changed from ASSIGNED to POST
Hello Tim!
This pull request allows syncing both v1 and v2, but is not able to give detailed progress reporting in doing so:
https://github.com/pulp/pulp_docker/pull/119
Please test this patch and let me know if it is acceptable for your purposes with Pulp 2.6. The upstream pulp-docker-2.0.0 release will be able to give detailed progress reporting and is implemented a little bit differently than is done in this patch.
- Status changed from POST to 5
- Status changed from 5 to CLOSED - CURRENTRELEASE
- Platform Release set to 2.8.0
Also available in: Atom
PDF
Can sync v1 and v2 APIs together, and optionally disable v1 sync
fixes #1449 https://pulp.plan.io/issues/1449
I combined the root v1 and v2 sync steps into one. The step system requires a single root step, and only one layer of children can report status and progress. This required the refactor, combining each collection of steps onto one parent.