)]}'
{"tripleo_common/inventories.py":[{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"76bcedd012d0251ec318e38b6a24380432fb831c","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                inv[\u0027_meta\u0027][\u0027hostvars\u0027]"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        # \u0027plan\u0027 doesn\u0027t make sense when using multiple plans"},{"line_number":81,"context_line":"        self.inventory[\u0027Undercloud\u0027][\u0027vars\u0027][\u0027plan\u0027] \u003d \u0027\u0027"},{"line_number":82,"context_line":"        # sort plans list for consistency"},{"line_number":83,"context_line":"        self.inventory[\u0027Undercloud\u0027][\u0027vars\u0027][\u0027plans\u0027].sort()"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"7faddb67_d6d21034","line":81,"updated":"2019-09-05 13:26:32.000000000","message":"Can\u0027t we drop that entry instead of setting it to empty?","commit_id":"aeaa57176571b4f9613c2495a2fd822bf0661c2f"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"012b9326e3fbcb038e1ed7a8c68edfd6fe1baf7e","unresolved":false,"context_lines":[{"line_number":78,"context_line":"                                inv[\u0027_meta\u0027][\u0027hostvars\u0027]"},{"line_number":79,"context_line":""},{"line_number":80,"context_line":"        # \u0027plan\u0027 doesn\u0027t make sense when using multiple plans"},{"line_number":81,"context_line":"        self.inventory[\u0027Undercloud\u0027][\u0027vars\u0027][\u0027plan\u0027] \u003d \u0027\u0027"},{"line_number":82,"context_line":"        # sort plans list for consistency"},{"line_number":83,"context_line":"        self.inventory[\u0027Undercloud\u0027][\u0027vars\u0027][\u0027plans\u0027].sort()"},{"line_number":84,"context_line":""}],"source_content_type":"text/x-python","patch_set":8,"id":"7faddb67_f10716c8","line":81,"in_reply_to":"7faddb67_d6d21034","updated":"2019-09-05 13:51:40.000000000","message":"It\u0027s likely consumed elsewhere which probably expects it to exist","commit_id":"aeaa57176571b4f9613c2495a2fd822bf0661c2f"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"76bcedd012d0251ec318e38b6a24380432fb831c","unresolved":false,"context_lines":[{"line_number":88,"context_line":"        if not os.path.splitext(inventory_file_path)[1] in allowed_extensions:"},{"line_number":89,"context_line":"            raise ValueError(\"Path %s does not end with one of %s extensions\""},{"line_number":90,"context_line":"                             % (inventory_file_path,"},{"line_number":91,"context_line":"                                \",\".join(allowed_extensions)))"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"        if extra_vars:"},{"line_number":94,"context_line":"            for var, value in extra_vars.items():"}],"source_content_type":"text/x-python","patch_set":8,"id":"7faddb67_5621404c","line":91,"range":{"start_line":91,"start_character":32,"end_line":91,"end_character":35},"updated":"2019-09-05 13:26:32.000000000","message":"would like \", \" for readability :)","commit_id":"aeaa57176571b4f9613c2495a2fd822bf0661c2f"}]}