)]}'
{"tempest/cmd/run.py":[{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"ed4066e3b6ec293debb195a0291e09e92e416ecb","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        file_contents \u003d init.TESTR_CONF % (top_level_path, discover_path)"},{"line_number":128,"context_line":"        with open(\u0027.testr.conf\u0027, \u0027w+\u0027) as testr_conf_file:"},{"line_number":129,"context_line":"                testr_conf_file.write(file_contents)"},{"line_number":130,"context_line":"        atexit.register(os.remove, \u0027,testr.conf\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def take_action(self, parsed_args):"},{"line_number":133,"context_line":"        returncode \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_49e54d07","line":130,"range":{"start_line":130,"start_character":0,"end_line":130,"end_character":49},"updated":"2016-06-29 23:27:08.000000000","message":"I\u0027m not sure, but I wonder if this may make the local .testrepository difficult to use?","commit_id":"76ca7c6aeea48298f587350182897fc27b6c17bc"},{"author":{"_account_id":17716,"name":"Luz Cazares","email":"luz.cazares@intel.com","username":"luzcazares"},"change_message_id":"9693f006070b79b3b181e0f85454bb587bf1d5f5","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        file_contents \u003d init.TESTR_CONF % (top_level_path, discover_path)"},{"line_number":128,"context_line":"        with open(\u0027.testr.conf\u0027, \u0027w+\u0027) as testr_conf_file:"},{"line_number":129,"context_line":"                testr_conf_file.write(file_contents)"},{"line_number":130,"context_line":"        atexit.register(os.remove, \u0027,testr.conf\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def take_action(self, parsed_args):"},{"line_number":133,"context_line":"        returncode \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_47f9b17f","line":130,"range":{"start_line":130,"start_character":8,"end_line":130,"end_character":49},"updated":"2016-06-30 03:18:47.000000000","message":"file name start with dot instead of comma - s/\u0027,testr.conf\u0027/\u0027.testr.conf\u0027\nAlso if .testr.conf already exists on the path, it will be deleted deleted, not sure if additional check should be added","commit_id":"76ca7c6aeea48298f587350182897fc27b6c17bc"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"178f371665aa95258869a27e12dfea13760924c2","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        file_contents \u003d init.TESTR_CONF % (top_level_path, discover_path)"},{"line_number":128,"context_line":"        with open(\u0027.testr.conf\u0027, \u0027w+\u0027) as testr_conf_file:"},{"line_number":129,"context_line":"                testr_conf_file.write(file_contents)"},{"line_number":130,"context_line":"        atexit.register(os.remove, \u0027,testr.conf\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def take_action(self, parsed_args):"},{"line_number":133,"context_line":"        returncode \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_bee83eb4","line":130,"range":{"start_line":130,"start_character":8,"end_line":130,"end_character":49},"in_reply_to":"3aaa91ec_1eb2324c","updated":"2016-06-30 15:18:06.000000000","message":"Ok, I\u0027ll remove it then and update the docs\n\n@Luz This actually is safe from removing a pre-existing .testr.conf because of the elif at L149. This method only gets run if there isn\u0027t a local .testr.conf already.","commit_id":"76ca7c6aeea48298f587350182897fc27b6c17bc"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"d17a2c5dcd137e577fad6dcb04253a3b83f2c2ab","unresolved":false,"context_lines":[{"line_number":127,"context_line":"        file_contents \u003d init.TESTR_CONF % (top_level_path, discover_path)"},{"line_number":128,"context_line":"        with open(\u0027.testr.conf\u0027, \u0027w+\u0027) as testr_conf_file:"},{"line_number":129,"context_line":"                testr_conf_file.write(file_contents)"},{"line_number":130,"context_line":"        atexit.register(os.remove, \u0027,testr.conf\u0027)"},{"line_number":131,"context_line":""},{"line_number":132,"context_line":"    def take_action(self, parsed_args):"},{"line_number":133,"context_line":"        returncode \u003d 0"}],"source_content_type":"text/x-python","patch_set":5,"id":"3aaa91ec_1eb2324c","line":130,"range":{"start_line":130,"start_character":8,"end_line":130,"end_character":49},"in_reply_to":"3aaa91ec_47f9b17f","updated":"2016-06-30 15:12:25.000000000","message":"Indeed. After checking, removing .testr.conf will break things like \"testr last\" for which we have no proxy on tempest side, so I\u0027d rather leave it around.","commit_id":"76ca7c6aeea48298f587350182897fc27b6c17bc"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"cbd2050bdfebd4654d527f365be0c0cbd592fd31","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    def take_action(self, parsed_args):"},{"line_number":130,"context_line":"        returncode \u003d 0"},{"line_number":131,"context_line":"        if parsed_args.config:"},{"line_number":132,"context_line":"            self._set_env(config)"},{"line_number":133,"context_line":"        else:"},{"line_number":134,"context_line":"            self._set_env()"},{"line_number":135,"context_line":"        # Workspace execution mode"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_21c7ff64","line":132,"range":{"start_line":132,"start_character":26,"end_line":132,"end_character":32},"updated":"2016-06-30 21:46:28.000000000","message":"here should be parsed_args.config, I guess.","commit_id":"2a9e1d85e7b3d96ef99a1d062392faf4f392979c"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1a1f50a1b900dd96b1be6b6620e5afebb3e32819","unresolved":false,"context_lines":[{"line_number":129,"context_line":"    def take_action(self, parsed_args):"},{"line_number":130,"context_line":"        returncode \u003d 0"},{"line_number":131,"context_line":"        if parsed_args.config:"},{"line_number":132,"context_line":"            self._set_env(config)"},{"line_number":133,"context_line":"        else:"},{"line_number":134,"context_line":"            self._set_env()"},{"line_number":135,"context_line":"        # Workspace execution mode"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_777d456c","line":132,"range":{"start_line":132,"start_character":26,"end_line":132,"end_character":32},"in_reply_to":"3aaa91ec_21c7ff64","updated":"2016-06-30 22:28:04.000000000","message":"Done","commit_id":"2a9e1d85e7b3d96ef99a1d062392faf4f392979c"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"cbd2050bdfebd4654d527f365be0c0cbd592fd31","unresolved":false,"context_lines":[{"line_number":183,"context_line":"                            help\u003d\"The path to the workspace file, the default \""},{"line_number":184,"context_line":"                                 \"is ~/.tempest/workspace.yaml\")"},{"line_number":185,"context_line":"        # Configuration flags"},{"line_number":186,"context_line":"        parser.add_argument(\u0027--config-file\u0027, default\u003dNone, dest\u003d\u0027config\u0027,"},{"line_number":187,"context_line":"                            help\u003d\u0027Configuration file to run tempest with\u0027)"},{"line_number":188,"context_line":"        # test selection args"},{"line_number":189,"context_line":"        regex \u003d parser.add_mutually_exclusive_group()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_e1ddf7e0","line":186,"range":{"start_line":186,"start_character":65,"end_line":186,"end_character":71},"updated":"2016-06-30 21:46:28.000000000","message":"it is better to define this as config_file to avoid name conflict with the module \"config\".","commit_id":"2a9e1d85e7b3d96ef99a1d062392faf4f392979c"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"1a1f50a1b900dd96b1be6b6620e5afebb3e32819","unresolved":false,"context_lines":[{"line_number":183,"context_line":"                            help\u003d\"The path to the workspace file, the default \""},{"line_number":184,"context_line":"                                 \"is ~/.tempest/workspace.yaml\")"},{"line_number":185,"context_line":"        # Configuration flags"},{"line_number":186,"context_line":"        parser.add_argument(\u0027--config-file\u0027, default\u003dNone, dest\u003d\u0027config\u0027,"},{"line_number":187,"context_line":"                            help\u003d\u0027Configuration file to run tempest with\u0027)"},{"line_number":188,"context_line":"        # test selection args"},{"line_number":189,"context_line":"        regex \u003d parser.add_mutually_exclusive_group()"}],"source_content_type":"text/x-python","patch_set":6,"id":"3aaa91ec_d79879c2","line":186,"range":{"start_line":186,"start_character":65,"end_line":186,"end_character":71},"in_reply_to":"3aaa91ec_e1ddf7e0","updated":"2016-06-30 22:28:04.000000000","message":"Done","commit_id":"2a9e1d85e7b3d96ef99a1d062392faf4f392979c"}]}