Bug 878548
Summary: | Proxy information for the yum importer needs to be able to be defaulted in plugin conf file | ||
---|---|---|---|
Product: | [Retired] Pulp | Reporter: | Jay Dobies <jason.dobies> |
Component: | rpm-support | Assignee: | Jay Dobies <jason.dobies> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | Preethi Thomas <pthomas> |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 2.0.6 | CC: | jsherril |
Target Milestone: | --- | ||
Target Release: | Sprint 42 | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2013-01-07 14:12:07 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 854726 |
Description
Jay Dobies
2012-11-20 16:48:23 UTC
Added empty conf files for the plugins in case we need to tell users to edit them in the future. I'd have liked to add comments about possible values, but comments aren't supported in JSON. If the proxy_* properties are in the file, they will be used by the plugin automatically. build: 2.0.6-0.3.beta The conf file is in /usr/lib/pulp/plugins/importers/yum_importer, it's called yum_importer.conf. It's intentionally empty, which is an unfortunate side effect of JSON. Also be careful with the syntax since JSON is kinda fragile. If that's invalid, I'm not sure exactly where in pulp.log it will blow up, but it won't be too pretty. The field names you'll care about are: proxy_url proxy_port proxy_user proxy_pass Realistically, you don't have to worry about an authenticated proxy. If any one of these fields is accessed, they all can be. Here's an example for the contents of that file: { "proxy_url" : "http://localhost/", "proxy_port" : "12345" } A few quick notes: - I forget in JSON if you can omit the quotes on the port to make it an integer. Try both if you're seeing oddities. - Not sure if this will be an issue for you, but in Python you can have a trailing comma on the last item in a dict like this (after the "12345"). This isn't allowed in JSON and always bites me, so I figured I'd mention it. If you really want to go crazy, you can set a value in here and a different value for the repo in the CLI. The repo-level config should override, just for that repo, the values in this file. verified [root@preethi-el6-pulp ~]# rpm -q pulp-server pulp-server-2.0.6-0.9.beta.noarch [root@preethi-el6-pulp ~]# [root@preethi-el6-pulp ~]# cat /usr/lib/pulp/plugins/importers/yum_importer/yum_importer.conf { "proxy_url" : "http://localhost/", "proxy_port" : "12345" } [root@preethi-el6-pulp ~]# pulp-admin rpm repo sync run --repo-id testerrata +----------------------------------------------------------------------+ Synchronizing Repository [testerrata] +----------------------------------------------------------------------+ This command may be exited by pressing ctrl+c without affecting the actual operation on the server. Downloading metadata... [\] ... failed [root@preethi-el6-pulp ~]vi /usr/lib/pulp/plugins/importers/yum_importer/yum_importer.conf [root@preethi-el6-pulp ~]# service httpd restartStopping httpd: [ OK ] Starting httpd: [Tue Nov 27 08:45:00 2012] [warn] NameVirtualHost *:80 has no VirtualHosts [ OK ] [root@preethi-el6-pulp ~]# cat /usr/lib/pulp/plugins/importers/yum_importer/yum_importer.conf { } [root@preethi-el6-pulp ~]# pulp-admin rpm repo sync run --repo-id testerrata+----------------------------------------------------------------------+ Synchronizing Repository [testerrata] +----------------------------------------------------------------------+ This command may be exited by pressing ctrl+c without affecting the actual operation on the server. Downloading metadata... [-] ... completed Downloading repository content... [==================================================] 100% RPMs: 0/0 items Delta RPMs: 0/0 items Tree Files: 0/0 items Files: 0/0 items ... completed Importing errata... [-] ... completed Importing package groups/categories... [-] ... completed Publishing packages... [==================================================] 100% Packages: 0/0 items ... completed Publishing distributions... [==================================================] 100% Distributions: 3/3 items ... completed Generating metadata [\] ... completed Publishing repository over HTTPS [-] ... completed Publishing repository over HTTP [-] ... skipped *** Bug 866621 has been marked as a duplicate of this bug. *** Pulp 2.0 released. |