)]}'
{"openstackclient/shell.py":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"95360e3f1a948b9ea8b1840c73c3a9daca5d8f87","unresolved":false,"context_lines":[{"line_number":406,"context_line":"            targs \u003d tparser.parse_args([\u0027-f\u0027, format])"},{"line_number":407,"context_line":"            tcmd.run(targs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def interact(self):"},{"line_number":410,"context_line":"        # NOTE(dtroyer): Maintain the old behaviour for interactive use as"},{"line_number":411,"context_line":"        #                this path does not call prepare_to_run_command()"},{"line_number":412,"context_line":"        self.authenticate_user()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa7fdbe_b9d5f644","side":"PARENT","line":409,"updated":"2014-10-20 23:49:06.000000000","message":"Remove the entire interact() method","commit_id":"2166d7d3afbbdc1659e4cffdb7bcd890cd00ec19"},{"author":{"_account_id":9101,"name":"wanghong","email":"reviewwanghong@126.com","username":"wanghong"},"change_message_id":"1018ee2015776d6429995c7f2e7647f349bcc77c","unresolved":false,"context_lines":[{"line_number":406,"context_line":"            targs \u003d tparser.parse_args([\u0027-f\u0027, format])"},{"line_number":407,"context_line":"            tcmd.run(targs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def interact(self):"},{"line_number":410,"context_line":"        # NOTE(dtroyer): Maintain the old behaviour for interactive use as"},{"line_number":411,"context_line":"        #                this path does not call prepare_to_run_command()"},{"line_number":412,"context_line":"        self.authenticate_user()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa7fdbe_dd4625bf","side":"PARENT","line":409,"in_reply_to":"9aa7fdbe_b9d5f644","updated":"2014-10-21 01:00:38.000000000","message":"I think here can be left. In initialize_clientmanager method we would validate authentication options, and interact method will be called when entering the interactive mode. That is say, we can validating authentication options before we enter the interactive mode. I think this is useful for users.","commit_id":"2166d7d3afbbdc1659e4cffdb7bcd890cd00ec19"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"0293a8b9a86cf8ceb0670e89f78259f3089b09c4","unresolved":false,"context_lines":[{"line_number":406,"context_line":"            targs \u003d tparser.parse_args([\u0027-f\u0027, format])"},{"line_number":407,"context_line":"            tcmd.run(targs)"},{"line_number":408,"context_line":""},{"line_number":409,"context_line":"    def interact(self):"},{"line_number":410,"context_line":"        # NOTE(dtroyer): Maintain the old behaviour for interactive use as"},{"line_number":411,"context_line":"        #                this path does not call prepare_to_run_command()"},{"line_number":412,"context_line":"        self.authenticate_user()"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa7fdbe_60d752c2","side":"PARENT","line":409,"in_reply_to":"9aa7fdbe_dd4625bf","updated":"2014-10-21 02:44:36.000000000","message":"fine, leave it, I\u0027ll rip it out in the next review.  Most of this isn\u0027t going to survive that anyway","commit_id":"2166d7d3afbbdc1659e4cffdb7bcd890cd00ec19"}]}