)]}'
{"novaclient/tests/v1_1/fakes.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"850cdff59e962770c6438cb3dfeba531bae8935a","unresolved":false,"context_lines":[{"line_number":979,"context_line":"    def get_os_keypairs_test(self, *kw):"},{"line_number":980,"context_line":"        return (200, {}, {\u0027keypair\u0027: self.get_os_keypairs()[2][\u0027keypairs\u0027][0]})"},{"line_number":981,"context_line":""},{"line_number":982,"context_line":"    def get_os_keypairs(self, user_id\u003d\u0027\u0027, *kw):"},{"line_number":983,"context_line":"        return (200, {}, {\"keypairs\": ["},{"line_number":984,"context_line":"            {\u0027fingerprint\u0027: \u0027FAKE_KEYPAIR\u0027, \u0027name\u0027: \u0027test\u0027}"},{"line_number":985,"context_line":"        ]})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F90iI%3D","line":982,"updated":"2014-02-03 16:37:17.000000000","message":"Why add this parameter, if you\u0027re not going to use it?  Is the clear \"*kw\" typo causing it to fail?  If so, I\u0027d rather you fix that typo than add an argument you aren\u0027t going to use.  (It would be convenient if you would fix the similar typo in get_os_keypairs_test() above, at the same time.)","commit_id":"4821c2b99dd23d6a32418505dddd02960423e459"},{"author":{"_account_id":5792,"name":"Dan Radez","email":"dradez@redhat.com","username":"dradez"},"change_message_id":"dcacda812e8771be0e4ac73a6990f9f517fc77fa","unresolved":false,"context_lines":[{"line_number":979,"context_line":"    def get_os_keypairs_test(self, *kw):"},{"line_number":980,"context_line":"        return (200, {}, {\u0027keypair\u0027: self.get_os_keypairs()[2][\u0027keypairs\u0027][0]})"},{"line_number":981,"context_line":""},{"line_number":982,"context_line":"    def get_os_keypairs(self, user_id\u003d\u0027\u0027, *kw):"},{"line_number":983,"context_line":"        return (200, {}, {\"keypairs\": ["},{"line_number":984,"context_line":"            {\u0027fingerprint\u0027: \u0027FAKE_KEYPAIR\u0027, \u0027name\u0027: \u0027test\u0027}"},{"line_number":985,"context_line":"        ]})"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F9zsk%3D","line":982,"in_reply_to":"AAAAUH%2F90iI%3D","updated":"2014-02-03 18:52:14.000000000","message":"Done","commit_id":"4821c2b99dd23d6a32418505dddd02960423e459"}],"novaclient/tests/v1_1/test_keypairs.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"850cdff59e962770c6438cb3dfeba531bae8935a","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    def test_list_keypairs_with_user_id(self):"},{"line_number":44,"context_line":"        kps \u003d self.cs.keypairs.list(\u0027123abc\u0027)"},{"line_number":45,"context_line":"        self.cs.assert_called(\u0027GET\u0027, \u0027/%s?user_id\u003d123abc\u0027 % self.keypair_prefix)"},{"line_number":46,"context_line":"        [self.assertIsInstance(kp, keypairs.Keypair) for kp in kps]"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def test_delete_keypair(self):"},{"line_number":49,"context_line":"        kp \u003d self.cs.keypairs.list()[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F90h4%3D","line":46,"updated":"2014-02-03 16:37:17.000000000","message":"Please do not use list comprehensions for this; use an explicit loop.","commit_id":"4821c2b99dd23d6a32418505dddd02960423e459"},{"author":{"_account_id":5792,"name":"Dan Radez","email":"dradez@redhat.com","username":"dradez"},"change_message_id":"dcacda812e8771be0e4ac73a6990f9f517fc77fa","unresolved":false,"context_lines":[{"line_number":43,"context_line":"    def test_list_keypairs_with_user_id(self):"},{"line_number":44,"context_line":"        kps \u003d self.cs.keypairs.list(\u0027123abc\u0027)"},{"line_number":45,"context_line":"        self.cs.assert_called(\u0027GET\u0027, \u0027/%s?user_id\u003d123abc\u0027 % self.keypair_prefix)"},{"line_number":46,"context_line":"        [self.assertIsInstance(kp, keypairs.Keypair) for kp in kps]"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    def test_delete_keypair(self):"},{"line_number":49,"context_line":"        kp \u003d self.cs.keypairs.list()[0]"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAUH%2F9zsc%3D","line":46,"in_reply_to":"AAAAUH%2F90h4%3D","updated":"2014-02-03 18:52:14.000000000","message":"Done","commit_id":"4821c2b99dd23d6a32418505dddd02960423e459"}],"novaclient/v1_1/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"3aa824527b8354d9d33d40a31adf7394c2d4af4d","unresolved":false,"context_lines":[{"line_number":2399,"context_line":"@utils.arg(\u0027--user\u0027,"},{"line_number":2400,"context_line":"    metavar\u003d\u0027\u003cuser-id\u003e\u0027,"},{"line_number":2401,"context_line":"    default\u003dNone,"},{"line_number":2402,"context_line":"    help\u003d\u0027ID of user to list the keypairs for. (Admin only)\u0027)"},{"line_number":2403,"context_line":"def do_keypair_list(cs, args):"},{"line_number":2404,"context_line":"    \"\"\"Print a list of keypairs for a user\"\"\""},{"line_number":2405,"context_line":"    if args.user:"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAUH%2F9zl0%3D","line":2402,"updated":"2014-02-03 19:08:55.000000000","message":"Make this update to novaclient/v3/shell.py and you should be all set","commit_id":"1bedf971f01cb2ada6ef2db3a4575a0fdda92653"}],"novaclient/v3/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"59816a429de88a210ba18ef23c5b6325475a500c","unresolved":false,"context_lines":[{"line_number":1973,"context_line":"    help\u003d\u0027ID of user to list the keypairs for. (Admin only)\u0027)"},{"line_number":1974,"context_line":"def do_keypair_list(cs, args):"},{"line_number":1975,"context_line":"    \"\"\"Print a list of keypairs for a user\"\"\""},{"line_number":1976,"context_line":"    keypairs \u003d cs.keypairs.list()"},{"line_number":1977,"context_line":"    columns \u003d [\u0027Name\u0027, \u0027Fingerprint\u0027]"},{"line_number":1978,"context_line":"    utils.print_list(keypairs, columns)"},{"line_number":1979,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F9ugY%3D","line":1976,"updated":"2014-02-04 16:24:48.000000000","message":"So, you\u0027ve added the argument, but ignore its value in the v3 shell?  Is this an option not supported by the nova v3 API?","commit_id":"f655b043a1b6a04ee9b2081efab919d672ffa276"},{"author":{"_account_id":5792,"name":"Dan Radez","email":"dradez@redhat.com","username":"dradez"},"change_message_id":"572a5bdfdb8971d485bc1c4bf6588d1d8b71574b","unresolved":false,"context_lines":[{"line_number":1973,"context_line":"    help\u003d\u0027ID of user to list the keypairs for. (Admin only)\u0027)"},{"line_number":1974,"context_line":"def do_keypair_list(cs, args):"},{"line_number":1975,"context_line":"    \"\"\"Print a list of keypairs for a user\"\"\""},{"line_number":1976,"context_line":"    keypairs \u003d cs.keypairs.list()"},{"line_number":1977,"context_line":"    columns \u003d [\u0027Name\u0027, \u0027Fingerprint\u0027]"},{"line_number":1978,"context_line":"    utils.print_list(keypairs, columns)"},{"line_number":1979,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAUH%2F9tfE%3D","line":1976,"in_reply_to":"AAAAUH%2F9ugY%3D","updated":"2014-02-04 19:09:39.000000000","message":"Oh man, that was really half-assed. My apologies for wasting your time with this patch. I\u0027ll update and resubmit it.","commit_id":"f655b043a1b6a04ee9b2081efab919d672ffa276"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"dbd24901aaf253476ab162fcf7ac55cc549f53a2","unresolved":false,"context_lines":[{"line_number":1973,"context_line":"    help\u003d\u0027ID of user to list the keypairs for. (Admin only)\u0027)"},{"line_number":1974,"context_line":"def do_keypair_list(cs, args):"},{"line_number":1975,"context_line":"    \"\"\"Print a list of keypairs for a user\"\"\""},{"line_number":1976,"context_line":"    if args.user:"},{"line_number":1977,"context_line":"        keypairs \u003d cs.keypairs.list(args.user)"},{"line_number":1978,"context_line":"    else:"},{"line_number":1979,"context_line":"        keypairs \u003d cs.keypairs.list()"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAUH%2F9tI4%3D","line":1976,"updated":"2014-02-04 20:03:14.000000000","message":"Better :)","commit_id":"8be5af48d728b623559d4020f37c3344e0bc1008"}]}