)]}'
{"test/sample.conf":[{"author":{"_account_id":12193,"name":"Hisashi Osanai","email":"osanai.hisashi@jp.fujitsu.com","username":"ho"},"change_message_id":"220be1a0919c4bf64663162edfdb8c48df435621","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#auth_prefix \u003d /v3/"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# If you run functional tests with Keystone, please make sure you"},{"line_number":16,"context_line":"# have correct setings in proxy-server.conf and Keystone server."},{"line_number":17,"context_line":"#"},{"line_number":18,"context_line":"# 1) Set `allow_account_management \u003d false` in proxy-server.conf."},{"line_number":19,"context_line":"# 2) First and second test users (tester, tester2) have a role(`admin`"}],"source_content_type":"text/plain","patch_set":2,"id":"fa69d971_2fefff1e","line":16,"range":{"start_line":16,"start_character":15,"end_line":16,"end_character":22},"updated":"2016-01-07 02:17:39.000000000","message":"s/setings/settings/","commit_id":"ef00e4e0602214f2f57d26cf09d85f27e4c150f3"},{"author":{"_account_id":12193,"name":"Hisashi Osanai","email":"osanai.hisashi@jp.fujitsu.com","username":"ho"},"change_message_id":"220be1a0919c4bf64663162edfdb8c48df435621","unresolved":false,"context_lines":[{"line_number":20,"context_line":"#    or `swiftoperator`) defined in operator_roles (keystoneauth middleware)"},{"line_number":21,"context_line":"# 3) Third user (tester3) doesn\u0027t have any role defined in operator_roles or"},{"line_number":22,"context_line":"#    reseller_admin_role."},{"line_number":23,"context_line":"#"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"# Primary functional test account (needs admin access to the account)"},{"line_number":26,"context_line":"account \u003d test"}],"source_content_type":"text/plain","patch_set":2,"id":"fa69d971_ef832701","line":23,"range":{"start_line":23,"start_character":0,"end_line":23,"end_character":1},"updated":"2016-01-07 02:17:39.000000000","message":"Do we need to have 2) and 3)? If L25 and L30 are not enough, I prefer to add the explanation on their places.","commit_id":"ef00e4e0602214f2f57d26cf09d85f27e4c150f3"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"2219162423ecca5a75687fc8d396b3267acacdba","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#auth_prefix \u003d /v3/"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"# If you run functional tests with Keystone, please make sure you"},{"line_number":16,"context_line":"# set `allow_account_management \u003d false` in proxy-server.conf."},{"line_number":17,"context_line":"#"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"# Primary functional test account (needs admin access to the account)"}],"source_content_type":"text/plain","patch_set":3,"id":"1af94dfe_ff96dc92","line":16,"updated":"2016-03-16 11:06:13.000000000","message":"Sorry to be fussy, but allow_account_management is false by default. So by default no action is required for the func tests to pass. This statement implies it *must be set* false, but in fact the requirement is that it should not be set to true.\n\nAlso, this might be more obvious if it was inserted at line 8.","commit_id":"bc4d401ca539f69648132675916e8ab6d32c9517"}]}
