Story #5067
closedAs a user, multiple source/target repositories can be used for recursive copy
0%
Description
Motivation¶
Dependencies for content can be present in multiple repositories.
The problem existed before but became more apparent with the introduction of modularity.
Sometimes repositories are split into 2, modular and non-modular, instead of being a hybrid repo.
For such cases it's essential to have an ability to specify multiple pairs of source/target repositories, otherwise dependency soling can't be performed correctly.
Suggestion¶
Modify the unit association API endpoint[0] to accept a new additional argument. edit: dalley Instead of a new parameter, we pass it through the override config - this prevents other plugins from being impacted by a change to the method signatures. While it's not "ideal", there is precedent for it because that's how the recursive flags work.
POST to /pulp/api/v2/repositories/<destination_repo_id>/actions/associate/ will take an optional parameter called additional_repos. This will be a dictionary where each key is a source repository id and the value is the destination repository id. The additional repositories would then also be used during the recursive copy only. The dispatched task will need to lock on all the destination repositories.
Sample request:¶
{
'source_repo_id' : 'pulp-f17',
'criteria': {
'type_ids' : ['rpm'],
'filters' : {
'unit' : {
'$and': [{'name': {'$regex': 'p.*'}}, {'version': {'$gt': '1.0'}}]
}
}
},
'override_config': {
'recursive': true,
'additional_repos': {'source2': 'destination2', 'source3': 'destination3'},
},
}
The response format will stay the same as it is now:
"result": {
"units_successful": [
{
"unit_key": {
"name": "whale",
"checksum": "3b34234afc8b8931d627f8466f0e4fd352145a2512681ec29db0a051a0c9d893",
"epoch": "0",
"version": "0.2",
"release": "1",
"arch": "noarch",
"checksumtype": "sha256"
},
"type_id": "rpm"
}
]
}
Related issues
Updated by ttereshc over 5 years ago
- Blocked by Story #5108: As a user, a task can reserve multiple resources added
Updated by dalley over 5 years ago
Support added to the depsolver library in this PR: https://github.com/pulp/pulp_rpm/pull/1407
This PR does not add support for copying from multiple source repos as a whole. The work that remains:
- Add the API to pulp_rpm
- Pass the new data from the API into the associate and copy_rpms functions
- Pass the new data into the solver inside the copy_rpms function, which now provides support for passing that information in
- Inside the copy_rpms function, after receiving a dictionary of repos and the units to copy from those repos from the find_dependent_rpms function, copy those units to the correct places by taking into account the repo they came from and the additional_repo data from the API
Added by dalley over 5 years ago
Updated by dalley over 5 years ago
- Status changed from NEW to ASSIGNED
Updated by dalley over 5 years ago
- Priority changed from Normal to High
- Sprint set to Sprint 57
Added by dalley over 5 years ago
Revision e9f8d257 | View on GitHub
Use multi-resource locking when additional_repos override is passed
The RPM plugin needs to accept more than one source and destination repository for copy operations. The "additional_repos" flag is being added to deal with this. Core needs to look for this flag in the override config and then, when present, lock on all the repo ids present within as well as the directly-specified source and destination repo ids.
Added by dalley over 5 years ago
Revision 45373820 | View on GitHub
Only lock destination repos, not source
Locking all source repos too may degrade performance and despite a minimal risk by not doing so, the tradeoff is likely worth it. It is also the current behavior.
Updated by dalley over 5 years ago
- Related to Task #5237: Add CLI support for "additional repos" api added
Updated by dalley over 5 years ago
- Related to Test #5242: Test copy using "additional_repos" to provide multiple source/destination repos via override added
Updated by dalley over 5 years ago
- Status changed from ASSIGNED to POST
Added by dalley about 5 years ago
Revision 1aef2cbc | View on GitHub
Refactor the entire pipeline so that depsolving comes at the very end
Depsolving has to happen after all the direct child units are discovered. It also can't happen inside copy_rpms() because that conflates depsolving and copying, which is a problem. We need these to be separate operations.
Updated by kersom about 5 years ago
- Related to Issue #5449: Multiple source repos copy of errata produces different results added
Updated by dalley about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Add support for multiple input repositories to the solver
re: #5067 https://pulp.plan.io/issues/5067