Add simple page parsing for full syncs#466
Conversation
7096ca2 to
a05db8e
Compare
a05db8e to
88e9dd6
Compare
| Bandersnatch config file. | ||
| * PyPI’s json API at content endpoint ‘/pypi/{package-name}/json’. Allows for basic Pulp-to-Pulp syncing. | ||
| * Fully mirror Python repositories like PyPI. | ||
| * Fully mirror Python repositories like PyPI and Pulp. |
There was a problem hiding this comment.
- Fully mirror Python repositories
likeprovided by PyPI and Pulp itself.
|
This pull request has been marked 'stale' due to lack of recent activity. If there is no further activity, the PR will be closed in another 30 days. Thank you for your contribution! |
|
This pull request is no longer marked for closure. |
| repo3 = self._create_repo_and_sync_with_remote(remote) | ||
| self.assertEqual(get_content_summary(repo3.to_dict()), PYTHON_LG_FIXTURE_SUMMARY) | ||
|
|
||
| def test_full_fixtures_to_pulp_sync(self): |
There was a problem hiding this comment.
I'm a little curious what happens if you sync an on-demand repo from pulp into pulp, in immediate mode. Would be an interesting test to make sure nothing blows up.
Maybe as part of the pulp_file suite?
There was a problem hiding this comment.
Although we might already have that test, I haven't looked.
dralley
left a comment
dralley
left a comment
There was a problem hiding this comment.
Minor question about the error message which could potentially use improvement, but otherwise this all looks good.
|
The pulp-to-pulp sync test is failing however ^ |
Yeah I need to figure out how to properly set the SSL certs in the CI or else the Pulp host won't be trusted. Edit: NVM, it just required a template settings change. |
2 participants
WIP syncing using simple page parsing. Implemented as a fallback for repositories that don't implement the PyPI XMLRPC endpoints (e.g. Pulp). Enables full Pulp to Pulp syncing.