)]}'
{"tools/generate-tempest-plugins-list.py":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"78be946dad3a1c8c103b41b4d10372e51f9f2618","unresolved":false,"context_lines":[{"line_number":74,"context_line":"projects \u003d sorted(filter(is_in_openstack_namespace, json.loads(r.read()[4:])))"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"# Retrieve projects having no deb, ui or spec namespace"},{"line_number":77,"context_line":"projects_without_deb_ui_specs \u003d [i for i in projects"},{"line_number":78,"context_line":"                                 if not (i.startswith(\u0027openstack/deb-\u0027) or"},{"line_number":79,"context_line":"                                         i.endswith(\u0027-ui\u0027) or"},{"line_number":80,"context_line":"                                         i.endswith(\u0027-specs\u0027))]"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f231b9d_65778dd6","line":77,"updated":"2017-06-13 04:46:53.000000000","message":"nit: I think it\u0027s ok that the name is just \u0027projects\u0027 here. Otherwise, if we have more filters, we need to change the variable name.","commit_id":"be5165b41498e482625706a744e0bae9eca7a17c"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"ed890f613f4cbcc30ff0c74b0ca6a4527ccabe87","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"try:"},{"line_number":30,"context_line":"    # For Python 3.0 and later"},{"line_number":31,"context_line":"    from urllib import HTTPError"},{"line_number":32,"context_line":"    from urllib.request import urlopen"},{"line_number":33,"context_line":"except ImportError:"},{"line_number":34,"context_line":"    # Fall back to Python 2\u0027s urllib2"},{"line_number":35,"context_line":"    from urllib2 import HTTPError"}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_4c9f1503","line":32,"range":{"start_line":31,"start_character":0,"end_line":32,"end_character":38},"updated":"2017-06-13 09:45:26.000000000","message":"We should always import modules.","commit_id":"98abd1af0e88bc10c581eed324349b7be5cd422c"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"ed890f613f4cbcc30ff0c74b0ca6a4527ccabe87","unresolved":false,"context_lines":[{"line_number":74,"context_line":"projects \u003d sorted(filter(is_in_openstack_namespace, json.loads(r.read()[4:])))"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"# Retrieve projects having no deb, ui or spec namespace"},{"line_number":77,"context_line":"projects_list \u003d [i for i in projects if not (i.startswith(\u0027openstack/deb-\u0027) or"},{"line_number":78,"context_line":"                                             i.endswith(\u0027-ui\u0027) or"},{"line_number":79,"context_line":"                                             i.endswith(\u0027-specs\u0027))]"},{"line_number":80,"context_line":""},{"line_number":81,"context_line":"found_plugins \u003d filter(has_tempest_plugin, projects_list)"},{"line_number":82,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7f231b9d_cc9205f7","line":79,"range":{"start_line":77,"start_character":44,"end_line":79,"end_character":66},"updated":"2017-06-13 09:45:26.000000000","message":"This is an unrelated change","commit_id":"98abd1af0e88bc10c581eed324349b7be5cd422c"}],"tools/generate-tempest-plugins-list.sh":[{"author":{"_account_id":5689,"name":"Masayuki Igawa","email":"masayuki@igawa.io","username":"igawa"},"change_message_id":"ea9a366e7607ea25bf5bf0463fbc348faa0d3beb","unresolved":false,"context_lines":[{"line_number":48,"context_line":"fi"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Install python requests module"},{"line_number":51,"context_line":"sudo pip install requests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"sorted_plugins\u003d$(python tools/generate-tempest-plugins-list.py)"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7f231b9d_5f736e81","line":51,"updated":"2017-06-12 02:29:49.000000000","message":"The sudo is not permited here. It seems you need to change the infra/project-config side.\n\nhttp://logs.openstack.org/75/473075/1/check/gate-tempest-plugin-sanity-ubuntu-xenial-nv/8984b28/console.html#_2017-06-11_13_21_29_771060","commit_id":"99323e37906513d0b3657ff3112285a6901e430c"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"12fad41b7716eb29e34140826650d59721f3a6ac","unresolved":false,"context_lines":[{"line_number":48,"context_line":"fi"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"# Install python requests module"},{"line_number":51,"context_line":"sudo pip install requests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"sorted_plugins\u003d$(python tools/generate-tempest-plugins-list.py)"},{"line_number":54,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"7f231b9d_b5c06152","line":51,"in_reply_to":"7f231b9d_5f736e81","updated":"2017-06-12 04:38:53.000000000","message":"What about moving the generate-tempest-plugins-list.py script from requests to ulrlib2?","commit_id":"99323e37906513d0b3657ff3112285a6901e430c"}]}