1182053 - Upgrade to modern python-requests in tools

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Rail Aliiev [:rail]]

Passing "config" is not supported in new versions of python-requests. It would be great if we stop using old API so we upgrade to newer and better python-requests.

Comment 1

[Rail Aliiev [:rail]]

Attachment: raise_for_status-tools.diff

If this one sticks (it passes tests), the next step will be replacing requests in the vendor directory and replacing references to it.

Comment 2

[Rail Aliiev [:rail]]

Comment on attachment 8631553 [details] [diff] [review]
raise_for_status-tools.diff

https://hg.mozilla.org/build/tools/rev/1b40fea61b5b

Comment 3

[Rail Aliiev [:rail]]

Attachment: requests-tools.diff

I'd like to land this on Monday.

Comment 4

[[e-mail] (:bhearsum)]

Comment on attachment 8631737 [details] [diff] [review]
requests-tools.diff

Review of attachment 8631737 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me. I'm not sure it's worthwhile checking in the text files and tests, but it doesn't hurt I guess. I'll leave that up to you.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 8631737 [details] [diff] [review]
requests-tools.diff

https://hg.mozilla.org/build/tools/rev/e833ebed151d

Comment 6

[Nick Thomas [:nthomas] (UTC+12)]

FWIW, we're getting this in release-runner logs now:

2015-07-22 14:25:01,100 - DEBUG - Fetching release requests
2015-07-22 14:25:01,100 - DEBUG - Request to https://ship-it.mozilla.org/releases
2015-07-22 14:25:01,100 - DEBUG - Data sent: None
2015-07-22 14:25:01,100 - DEBUG - attempt 1/5
2015-07-22 14:25:01,100 - DEBUG - retry: Calling <function _req at 0x13c8578> with args: (), kwargs: {}, attempt #1
2015-07-22 14:25:01,101 - INFO - Resetting dropped connection: ship-it.mozilla.org
/builds/releaserunner/tools/lib/python/vendor/requests-2.7.0/requests/packages/urllib3/util/ssl_.py:90: InsecurePlatformWarning: A true SSLContext object is not available. This prevents urllib3 from configuring SSL appropriately and may cause certain SSL connections to fail. For more information, see https://urllib3.readthedocs.org/en/latest/security.html#insecureplatformwarning.
  InsecurePlatformWarning
2015-07-22 14:25:01,280 - DEBUG - "GET /releases?ready=1&complete=0 HTTP/1.1" 200 20
2015-07-22 14:25:01,281 - INFO - No new releases: {'releases': []}
2015-07-22 14:25:01,281 - DEBUG - Sleeping for 30 seconds before polling again

We have Python 2.7.3, and need 2.7.9 to avoid that SSL warning (or pyopenssl).