)]}'
{"rally/api.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"dca4a92bd62081fcb27bf19ad6e8a992fcd16d50","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                                   system_wide_install\u003dsystem_wide_install)"},{"line_number":294,"context_line":"        if not verifier.is_installed():"},{"line_number":295,"context_line":"            print(\"Tempest is not installed for specified deployment.\")"},{"line_number":296,"context_line":"            print(\"Installing Tempest for deployment %s\" % deployment_uuid)"},{"line_number":297,"context_line":"            verifier.install()"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        return verifier"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_012f50f7","line":296,"updated":"2015-09-18 21:52:32.000000000","message":"I think its better to write here into the LOG. Because user that just want to use api in there own code will saw some trash in the console. Prints better to move to cli method.","commit_id":"b3aa5aa29611eff74400c9124279537e029dff40"},{"author":{"_account_id":7428,"name":"Yaroslav Lobankov","email":"lobankovyv@gmail.com","username":"ylobankov"},"change_message_id":"fdefdfd02a6e77807422d725e1d2effde295e6d3","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                                   system_wide_install\u003dsystem_wide_install)"},{"line_number":294,"context_line":"        if not verifier.is_installed():"},{"line_number":295,"context_line":"            print(\"Tempest is not installed for specified deployment.\")"},{"line_number":296,"context_line":"            print(\"Installing Tempest for deployment %s\" % deployment_uuid)"},{"line_number":297,"context_line":"            verifier.install()"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        return verifier"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_53bd1301","line":296,"in_reply_to":"ba15a1d1_012f50f7","updated":"2015-09-19 13:50:13.000000000","message":"Roman, thanks for review! I will do it in another patch.","commit_id":"b3aa5aa29611eff74400c9124279537e029dff40"},{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"1474d76cd8e02e7834626cb0bed48274eaa133e3","unresolved":false,"context_lines":[{"line_number":293,"context_line":"                                   system_wide_install\u003dsystem_wide_install)"},{"line_number":294,"context_line":"        if not verifier.is_installed():"},{"line_number":295,"context_line":"            print(\"Tempest is not installed for specified deployment.\")"},{"line_number":296,"context_line":"            print(\"Installing Tempest for deployment %s\" % deployment_uuid)"},{"line_number":297,"context_line":"            verifier.install()"},{"line_number":298,"context_line":""},{"line_number":299,"context_line":"        return verifier"}],"source_content_type":"text/x-python","patch_set":5,"id":"ba15a1d1_e1933cff","line":296,"in_reply_to":"ba15a1d1_012f50f7","updated":"2015-09-18 21:54:04.000000000","message":"it is not related this patch IMHO","commit_id":"b3aa5aa29611eff74400c9124279537e029dff40"}]}