-
Notifications
You must be signed in to change notification settings - Fork 168
ref #1195 - Implement the pulp-streamer. #2065
Conversation
c25b149
to
890a04e
Compare
This is very bare-bones right now since the things left to do depend on the incoming URL format which is not clear to me yet, the catalog, and the pulp task to dispatch upon completion. |
890a04e
to
46a512d
Compare
cb13cd4
to
818a7d7
Compare
General comment: Linux configuration files traditionally don't contain '_'. I know we do this in pulp (which I find annoying and odd) so I guess it's best to follow pulp convention. Just complaining, I guess. |
# redirect_path: /streamer/ | ||
# streamer_port: 8751 | ||
# streamer_interfaces: localhost | ||
# streamer_cache_timeout: 86400 |
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.
Instead of namespacing with streamer_
, consider:
[streamer]
interfaces: localhost
port: 8751
cache_timeout: 86400
mock_repo_controller.get_importer_by_id.assert_called_once_with(mock_catalog.plugin_id) | ||
|
||
# Validate building and invoking the downloader | ||
mock_importer.get_downloader.assert_called_once_with(mock_config, mock_catalog.url, k='v') |
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.
Line length.
Looks good overall but would like to discuss Streamer._download(). |
@@ -94,6 +94,9 @@ | |||
'redirect_host': '', | |||
'redirect_port': '', | |||
'redirect_path': '', | |||
'streamer_port': '8751', | |||
'streamer_interfaces': 'localhost', | |||
'streamer_cache_timeout': '86400', |
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.
Same as other comment. consider separate section.
This commit adds the 'lazy' module inside of pulp.server. It also adds streamer settings to Pulp's server.conf.
ref #1195 - Implement the pulp-streamer.
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
The prefix stripping method used in the oid_validator failed if the prefix was _not_ found in the path. This fixes that particular issue, improves readability, and adds a unit test. fixes pulp#2065
The prefix stripping method used in the oid_validator failed if the prefix was _not_ found in the path. This fixes that particular issue, improves readability, and adds a unit test. fixes pulp#2065
Fixed a bug that caused the wrong repo destination path to be verified on a ostree pull. Look at https://pulp.plan.io/issues/2065 for more info
The prefix stripping method used in the oid_validator failed if the prefix was _not_ found in the path. This fixes that particular issue, improves readability, and adds a unit test. fixes pulp#2065
This commit adds the 'lazy' module inside of pulp.server.
It also adds streamer settings to Pulp's server.conf.