)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"c59990fba8abbb7341ee9c63b66f4a94743d3a21","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jesse Pretorius (odyssey4me) \u003cjesse.pretorius@rackspace.co.uk\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-05 09:21:28 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ndg-httpsclient to global-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible makes use of httplib/urllib for the get_url task. The package"},{"line_number":10,"context_line":"ndg-httpsclient provides the appropriate improvements to support SNI"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bacf61ea_bbf6ad5a","line":7,"updated":"2016-08-05 11:23:30.000000000","message":"you might get a -1\n\ns/global-requirements/requirements","commit_id":"a3b44d00db5db41e767bf1fda000e302f80f3727"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"a771fdda4b2a8c2c8216fad8b9ab542338188c20","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jesse Pretorius (odyssey4me) \u003cjesse.pretorius@rackspace.co.uk\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-05 09:21:28 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ndg-httpsclient to global-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible makes use of httplib/urllib for the get_url task. The package"},{"line_number":10,"context_line":"ndg-httpsclient provides the appropriate improvements to support SNI"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bacf61ea_1dcfb129","line":7,"in_reply_to":"bacf61ea_828ce0d7","updated":"2016-08-08 03:56:39.000000000","message":"Perhaps some sort of differentiation around whether \u0027global-requirements\u0027 covers both g-r and u-c? I don\u0027t know.","commit_id":"a3b44d00db5db41e767bf1fda000e302f80f3727"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"28ab589552e25915ece701110a80c852ce3dfafd","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jesse Pretorius (odyssey4me) \u003cjesse.pretorius@rackspace.co.uk\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-05 09:21:28 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ndg-httpsclient to global-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible makes use of httplib/urllib for the get_url task. The package"},{"line_number":10,"context_line":"ndg-httpsclient provides the appropriate improvements to support SNI"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bacf61ea_8d0d7040","line":7,"in_reply_to":"bacf61ea_bbf6ad5a","updated":"2016-08-05 11:43:10.000000000","message":"Done","commit_id":"a3b44d00db5db41e767bf1fda000e302f80f3727"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"5bf469fb1e151b265490f3226caf812992d3df71","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jesse Pretorius (odyssey4me) \u003cjesse.pretorius@rackspace.co.uk\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-08-05 09:21:28 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add ndg-httpsclient to global-requirements"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Ansible makes use of httplib/urllib for the get_url task. The package"},{"line_number":10,"context_line":"ndg-httpsclient provides the appropriate improvements to support SNI"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bacf61ea_828ce0d7","line":7,"in_reply_to":"bacf61ea_bbf6ad5a","updated":"2016-08-08 03:04:47.000000000","message":"Why?  We\u0027re adding this to global requirements NOT requirements.txt","commit_id":"a3b44d00db5db41e767bf1fda000e302f80f3727"}],"global-requirements.txt":[{"author":{"_account_id":14288,"name":"Matthew Thode","display_name":"prometheanfire","email":"mthode@mthode.org","username":"prometheanfire"},"change_message_id":"4138f41b48abf2a841637f1f80d45098df1bb1e4","unresolved":false,"context_lines":[{"line_number":81,"context_line":"mistral!\u003d2015.1.0,\u003e\u003d2.0.0  # Apache-2.0"},{"line_number":82,"context_line":"monotonic\u003e\u003d0.6                          # Apache-2.0"},{"line_number":83,"context_line":"msgpack-python\u003e\u003d0.4.0  # Apache-2.0"},{"line_number":84,"context_line":"ndg-httpsclient\u003e\u003d0.4.2  # BSD"},{"line_number":85,"context_line":"netaddr\u003e\u003d0.7.12,!\u003d0.7.16  # BSD"},{"line_number":86,"context_line":"netifaces\u003e\u003d0.10.4  # MIT"},{"line_number":87,"context_line":"netmiko!\u003d0.4.3,\u003e\u003d0.3.0 # MIT"}],"source_content_type":"text/plain","patch_set":4,"id":"bacf61ea_efffb531","line":84,"updated":"2016-08-05 15:18:53.000000000","message":"I assume you mean python-2.7.9? if so, then didn\u0027t openstack officially drop support for older pythons (less than 2.7).  I think this would only fix the \u00272.7.0-2.7.9\u0027 gap","commit_id":"ee268dd712db79d540702d35a6281570bd7ae363"},{"author":{"_account_id":14288,"name":"Matthew Thode","display_name":"prometheanfire","email":"mthode@mthode.org","username":"prometheanfire"},"change_message_id":"b62105de09c5cb5da2578fe62cf0f63a6c371365","unresolved":false,"context_lines":[{"line_number":81,"context_line":"mistral!\u003d2015.1.0,\u003e\u003d2.0.0  # Apache-2.0"},{"line_number":82,"context_line":"monotonic\u003e\u003d0.6                          # Apache-2.0"},{"line_number":83,"context_line":"msgpack-python\u003e\u003d0.4.0  # Apache-2.0"},{"line_number":84,"context_line":"ndg-httpsclient\u003e\u003d0.4.2  # BSD"},{"line_number":85,"context_line":"netaddr\u003e\u003d0.7.12,!\u003d0.7.16  # BSD"},{"line_number":86,"context_line":"netifaces\u003e\u003d0.10.4  # MIT"},{"line_number":87,"context_line":"netmiko!\u003d0.4.3,\u003e\u003d0.3.0 # MIT"}],"source_content_type":"text/plain","patch_set":4,"id":"bacf61ea_95bc46dd","line":84,"updated":"2016-08-05 15:38:17.000000000","message":"I\u0027m saying that since openstack only supports python-2.7 and python-3.4 (soon 3.5 I think) we don\u0027t need this since all those versions have sni support (if up to date).  I\u0027m not totally sure about trusty and it\u0027s python, or cent/rh.  But xenial should definitely not need this.","commit_id":"ee268dd712db79d540702d35a6281570bd7ae363"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"0f4256fe023b1e3caafb5b09f74f5c2fa7549e84","unresolved":false,"context_lines":[{"line_number":81,"context_line":"mistral!\u003d2015.1.0,\u003e\u003d2.0.0  # Apache-2.0"},{"line_number":82,"context_line":"monotonic\u003e\u003d0.6                          # Apache-2.0"},{"line_number":83,"context_line":"msgpack-python\u003e\u003d0.4.0  # Apache-2.0"},{"line_number":84,"context_line":"ndg-httpsclient\u003e\u003d0.4.2  # BSD"},{"line_number":85,"context_line":"netaddr\u003e\u003d0.7.12,!\u003d0.7.16  # BSD"},{"line_number":86,"context_line":"netifaces\u003e\u003d0.10.4  # MIT"},{"line_number":87,"context_line":"netmiko!\u003d0.4.3,\u003e\u003d0.3.0 # MIT"}],"source_content_type":"text/plain","patch_set":4,"id":"bacf61ea_cf0d91e9","line":84,"range":{"start_line":84,"start_character":0,"end_line":84,"end_character":22},"updated":"2016-08-05 15:11:41.000000000","message":"Perhaps I should add \";python_version\u003c\u00272.9\u0027\" to this because python 2.9 and above have native SNI support?","commit_id":"ee268dd712db79d540702d35a6281570bd7ae363"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"0ac0411e777a73475ab551f83748c3779090ac6d","unresolved":false,"context_lines":[{"line_number":81,"context_line":"mistral!\u003d2015.1.0,\u003e\u003d2.0.0  # Apache-2.0"},{"line_number":82,"context_line":"monotonic\u003e\u003d0.6                          # Apache-2.0"},{"line_number":83,"context_line":"msgpack-python\u003e\u003d0.4.0  # Apache-2.0"},{"line_number":84,"context_line":"ndg-httpsclient\u003e\u003d0.4.2  # BSD"},{"line_number":85,"context_line":"netaddr\u003e\u003d0.7.12,!\u003d0.7.16  # BSD"},{"line_number":86,"context_line":"netifaces\u003e\u003d0.10.4  # MIT"},{"line_number":87,"context_line":"netmiko!\u003d0.4.3,\u003e\u003d0.3.0 # MIT"}],"source_content_type":"text/plain","patch_set":4,"id":"bacf61ea_55ac8e8b","line":84,"in_reply_to":"bacf61ea_95bc46dd","updated":"2016-08-05 15:40:13.000000000","message":"Trusty \u0026 CentOS7 currently use python \u003c 2.7.9\nXenial uses a higher version.\n\nThis is why I think it may be good to add the version requirement so that more modern distributions do not end up with this as a mistaken requirement.","commit_id":"ee268dd712db79d540702d35a6281570bd7ae363"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"d6bf74b41f0751d35a46d8ea50a47778aa9bf3af","unresolved":false,"context_lines":[{"line_number":81,"context_line":"mistral!\u003d2015.1.0,\u003e\u003d2.0.0  # Apache-2.0"},{"line_number":82,"context_line":"monotonic\u003e\u003d0.6                          # Apache-2.0"},{"line_number":83,"context_line":"msgpack-python\u003e\u003d0.4.0  # Apache-2.0"},{"line_number":84,"context_line":"ndg-httpsclient\u003e\u003d0.4.2  # BSD"},{"line_number":85,"context_line":"netaddr\u003e\u003d0.7.12,!\u003d0.7.16  # BSD"},{"line_number":86,"context_line":"netifaces\u003e\u003d0.10.4  # MIT"},{"line_number":87,"context_line":"netmiko!\u003d0.4.3,\u003e\u003d0.3.0 # MIT"}],"source_content_type":"text/plain","patch_set":4,"id":"bacf61ea_ba36994e","line":84,"in_reply_to":"bacf61ea_efffb531","updated":"2016-08-05 15:30:23.000000000","message":"Yes, thanks for the correction. I don\u0027t see why this requirement needs to care about earlier versions? I\u0027m just trying to ensure that this requirement only applies to environments with the python version that has no native SNI support.","commit_id":"ee268dd712db79d540702d35a6281570bd7ae363"}]}