)]}'
{"ironic/common/utils.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"5cb902693808e4ece9b22599e92187fe34d96c44","unresolved":false,"context_lines":[{"line_number":94,"context_line":"                pkey \u003d paramiko.DSSKey(data\u003dkey_contents)"},{"line_number":95,"context_line":"            else:"},{"line_number":96,"context_line":"                # Can\u0027t include the key contents - secure material."},{"line_number":97,"context_line":"                raise ValueError(\"Invalid private key\")"},{"line_number":98,"context_line":"        else:"},{"line_number":99,"context_line":"            pkey \u003d None"},{"line_number":100,"context_line":"        ssh.connect(connection.get(\u0027host\u0027),"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAAWH%2F%2FUxU%3D","line":97,"updated":"2014-03-18 16:10:57.000000000","message":"i18n","commit_id":"97124ef9b67393ba8c518c39566ef625fa1fa25b"}],"ironic/tests/drivers/test_ssh.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ea995982aeae32c13e36826d5e27e97e75292301","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        self.assertIsNotNone(info.get(\u0027uuid\u0027))"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"    def test__parse_driver_info_bad_file(self):"},{"line_number":89,"context_line":"        # make sure we get back the expected things"},{"line_number":90,"context_line":"        info \u003d db_utils.get_test_ssh_info(\u0027file\u0027)"},{"line_number":91,"context_line":"        node \u003d db_utils.get_test_node(driver\u003d\u0027fake_ssh\u0027, driver_info\u003dinfo)"},{"line_number":92,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FqWI%3D","line":89,"updated":"2014-03-14 16:41:42.000000000","message":"I think this comment doesn\u0027t make much sense for this test","commit_id":"0c037361682c8bc616e08df4fe553bbc472f3c8c"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c0be464d5e2766033297846b27e9c9e3ab3af455","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        self.assertIsNotNone(info.get(\u0027uuid\u0027))"},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"    def test__parse_driver_info_bad_file(self):"},{"line_number":89,"context_line":"        # make sure we get back the expected things"},{"line_number":90,"context_line":"        info \u003d db_utils.get_test_ssh_info(\u0027file\u0027)"},{"line_number":91,"context_line":"        node \u003d db_utils.get_test_node(driver\u003d\u0027fake_ssh\u0027, driver_info\u003dinfo)"},{"line_number":92,"context_line":"        self.assertRaises("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FoiE%3D","line":89,"in_reply_to":"AAAAWH%2F%2FqWI%3D","updated":"2014-03-14 20:53:58.000000000","message":"Done","commit_id":"0c037361682c8bc616e08df4fe553bbc472f3c8c"},{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"ea995982aeae32c13e36826d5e27e97e75292301","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        node \u003d db_utils.get_test_node(driver\u003d\u0027fake_ssh\u0027, driver_info\u003dinfo)"},{"line_number":92,"context_line":"        self.assertRaises("},{"line_number":93,"context_line":"            exception.InvalidParameterValue, ssh._parse_driver_info, node)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def test__parse_driver_info_missing_host(self):"},{"line_number":96,"context_line":"        # make sure error is raised when info is missing"},{"line_number":97,"context_line":"        info \u003d db_utils.get_test_ssh_info()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FqV0%3D","line":94,"updated":"2014-03-14 16:41:42.000000000","message":"It\u0027s missing one test for testing if more than one type of password is being passed to the ssh driver, I see u already did the plumbing for it by adding the auth_type\u003d\u0027too_many\u0027 option for the get_test_ssh_info()\n\n    def test__parse_driver_info_too_many(self):\n        info \u003d db_utils.get_test_ssh_info(\u0027too_many\u0027)\n        node \u003d db_utils.get_test_node(driver\u003d\u0027fake_ssh\u0027, driver_info\u003dinfo)\n        self.assertRaises(\n            exception.InvalidParameterValue, ssh._parse_driver_info, node)","commit_id":"0c037361682c8bc616e08df4fe553bbc472f3c8c"},{"author":{"_account_id":4190,"name":"lifeless","email":"robertc@robertcollins.net","username":"lifeless"},"change_message_id":"c0be464d5e2766033297846b27e9c9e3ab3af455","unresolved":false,"context_lines":[{"line_number":91,"context_line":"        node \u003d db_utils.get_test_node(driver\u003d\u0027fake_ssh\u0027, driver_info\u003dinfo)"},{"line_number":92,"context_line":"        self.assertRaises("},{"line_number":93,"context_line":"            exception.InvalidParameterValue, ssh._parse_driver_info, node)"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"    def test__parse_driver_info_missing_host(self):"},{"line_number":96,"context_line":"        # make sure error is raised when info is missing"},{"line_number":97,"context_line":"        info \u003d db_utils.get_test_ssh_info()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAWH%2F%2FokA%3D","line":94,"in_reply_to":"AAAAWH%2F%2FqV0%3D","updated":"2014-03-14 20:53:58.000000000","message":"thanks, I managed to only half add this!","commit_id":"0c037361682c8bc616e08df4fe553bbc472f3c8c"}]}