)]}'
{"tripleo_common/actions/ansible.py":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"195890b4e09c0447e658b70198104fdd5ba48f22","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"29921386_ac9e95f0","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":32},"updated":"2021-03-08 14:09:02.000000000","message":"In py3 we have access to an actual touch implementation. I think we should use that here.\n\n  from pathlib import Path\n  Path(log_path).touch()","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"084c5cd4c6df5502c5eda9967ce42b7e141f2822","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"23a2e8c4_d8742b3a","line":125,"range":{"start_line":125,"start_character":19,"end_line":125,"end_character":23},"updated":"2021-03-08 14:29:25.000000000","message":"for a proper touch, I think we need to change \u0027w+\u0027 to \u0027a\u0027 so that the file contents isn\u0027t modified if it exists.","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"bce8d2fefd42ec68b5f52dacb3a9d92ae3da1a88","unresolved":false,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"1b60047a_27a27b50","line":125,"range":{"start_line":125,"start_character":19,"end_line":125,"end_character":23},"in_reply_to":"23a2e8c4_d8742b3a","updated":"2021-03-08 14:40:44.000000000","message":"Ack","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"9ed5a044aca5089f19ca31a3b801479197e7e3cc","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"76998ffe_91b87ddb","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":32},"in_reply_to":"29921386_ac9e95f0","updated":"2021-03-08 14:16:00.000000000","message":"since it targets stable/train in the end, we have to get py2.7 support... Using the py3 implementation would prevent a clean backport :/","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"bce8d2fefd42ec68b5f52dacb3a9d92ae3da1a88","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"1ec80839_7357c7b1","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":32},"in_reply_to":"47cb53a6_86db7fe3","updated":"2021-03-08 14:40:44.000000000","message":"so it\u0027s targeting victoria and earlier releases - the code has been removed from master already :).","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"084c5cd4c6df5502c5eda9967ce42b7e141f2822","unresolved":true,"context_lines":[{"line_number":122,"context_line":""},{"line_number":123,"context_line":"    # Create the log file, and set some rights on it in order to prevent"},{"line_number":124,"context_line":"    # unwanted access"},{"line_number":125,"context_line":"    open(log_path, \u0027w+\u0027).close()"},{"line_number":126,"context_line":"    os.chmod(log_path, 0o640)"},{"line_number":127,"context_line":""},{"line_number":128,"context_line":"    config.set(\u0027defaults\u0027, \u0027forks\u0027, str(min("}],"source_content_type":"text/x-python","patch_set":2,"id":"47cb53a6_86db7fe3","line":125,"range":{"start_line":125,"start_character":4,"end_line":125,"end_character":32},"in_reply_to":"76998ffe_91b87ddb","updated":"2021-03-08 14:29:25.000000000","message":"I think we have to develop for master, and adjust to support the earlier versions as needed.","commit_id":"8c7706a4280f8b6ec09bfb372de40cf71e585c52"}]}
