)]}'
{"nova/policies/server_usage.py":[{"author":{"_account_id":782,"name":"John Garbutt","email":"john@johngarbutt.com","username":"johngarbutt"},"change_message_id":"9cf6ff32cc449b214d662377a2962316e2e97721","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        \"\"\"Add \u0027OS-SRV-USG:launched_at\u0027 \u0026 \u0027OS-SRV-USG:terminated_at\u0027 attribute"},{"line_number":27,"context_line":"in the server response."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"This check is performed only after the check"},{"line_number":30,"context_line":"\u0027os_compute_api:servers:show\u0027 for GET /servers/{id} and"},{"line_number":31,"context_line":"\u0027os_compute_api:servers:detail\u0027 for GET /servers/detail passes\"\"\","},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"        ["}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_d7e3f3a3","line":31,"range":{"start_line":29,"start_character":0,"end_line":31,"end_character":62},"updated":"2017-05-17 15:35:34.000000000","message":"Part of me wonders if this last bit is implied because its about adding an attribute to an existing response, and so we should not include it here. But now we have the extra detail, lets keep it.","commit_id":"4d272c09f52da39664495dc80ac47a9fac80bc40"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"d8ab53977c34118ab0519170aacd25e759f275fe","unresolved":false,"context_lines":[{"line_number":26,"context_line":"        \"\"\"Add \u0027OS-SRV-USG:launched_at\u0027 \u0026 \u0027OS-SRV-USG:terminated_at\u0027 attribute"},{"line_number":27,"context_line":"in the server response."},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"This check is performed only after the check"},{"line_number":30,"context_line":"\u0027os_compute_api:servers:show\u0027 for GET /servers/{id} and"},{"line_number":31,"context_line":"\u0027os_compute_api:servers:detail\u0027 for GET /servers/detail passes\"\"\","},{"line_number":32,"context_line":""},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"        ["}],"source_content_type":"text/x-python","patch_set":1,"id":"ff0f0b1f_a04d219b","line":31,"range":{"start_line":29,"start_character":0,"end_line":31,"end_character":62},"in_reply_to":"ff0f0b1f_d7e3f3a3","updated":"2017-05-23 22:53:00.000000000","message":"Yeah, the original mechanism itself made confusions. I misunderstood there was os-server-usage resource, but not.\nIt is fine to keep the policy check itself with current way.","commit_id":"4d272c09f52da39664495dc80ac47a9fac80bc40"}]}