Description: The current master branch of nectar will fail if run against python-requests 2.2.1. The dependency in pulp & nectar should be updated or the threaded downloader should be updated to not use the new python-requests api.
Barnaby, Pulp 2.6 is bundled with python-requests-2.4.3, so this should not be an issue. Please reopen if I am incorrect.
https://github.com/pulp/nectar/pull/26 I see that the problem was not the built dependencies, rather the requires statement in the spec. I think that we should require at least the version that we provide.
fixed in pulp 2.6.0-0.2.beta
Failed QA >> rpm -q pulp-server pulp-server-2.6.0-0.2.beta.fc20.noarch >>rpm -qR python-nectar | grep python-requests python-requests >= 2.2.1
This was fixed in the nectar spec file, but not in the pulp copy of the same spec file. Fixed here: https://github.com/pulp/pulp/pull/1482
I don't think it is necessary for the Pulp package to also require python-requests >= 2.4, unless Pulp is actually using those features. AIUI, only nectar is using the newer features, so it was correct for Pulp to continue to require >= 2.2. That is, unless someone is aware of something Pulp is doing with python-requests that really does need 2.4. I know of no such usage.
Randy and I talked about this, it is appropriate to change this spec file to match that of the nectar repo.
My comment was due to a mistake in my interpretation of the details. Carry on!
This was merged to 2.6-dev (2.6.1) , not 2.6-testing (2.6.0). If you'd like it in 2.6.0, please open a new PR against 2.6-testing and set the bug's target release to 2.6.0.
pulp 2.6.0 beta 5
Do I miss something or it failed QA again? >> rpm -qa pulp-server pulp-server-2.6.0-0.5.beta.el6.noarch >> rpm -qR python-nectar | grep python-requests python-requests >= 2.2.1
The code appears to be correct, but the build date for the nectar dep does not. Cracking open the rpm: Build Date : Thu 21 Aug 2014 02:28:03 PM EDT Chris, I think that the deps were not rebuilt in this build?
Try now:) python-nectar 1.3.1-2 should fix this. The fix was in the src tree before but the newer nectar was not tagged.
>> rpm -qa pulp-server pulp-server-2.6.0-0.5.beta.el6.noarch >> rpm -qa python-nectar python-nectar-1.3.1-2.el6.noarch >> rpm -qR python-nectar | grep python-requests python-requests >= 2.4.3
Moved to https://pulp.plan.io/issues/640