)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"22b3a29805d595911b979e7c8ebeba9f3df798cb","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Nick Shobe \u003cnickshobe@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2012-11-17 04:26:08 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added --extra-opts to the nova ssh command"},{"line_number":8,"context_line":"This will allow for remote command execution and advanced ssh options like portforwarding and connection sharing. The goal is to support ssh commands like nova ssh fe55adc9-cb1e-44a4-bd36-6a537b238172 --extra-opts\u003d\u0027-NfnMS ~/.ssh/master-fe55adc9-cb1e-44a4-bd36-6a537b238172\u0027 and ssh fe55adc9-cb1e-44a4-bd36-6a537b238172 --extra-opts\u003d\u0027-t \"sudo puppet agent --test\"\u0027. This will alow easier scripting of the cli... It is not intended however to replace using the nova python api. Therefore I kept it simple."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: Icce811caf637fc92c6d6374bfb846ea9525a7e05"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMH%2F%2F3bQ%3D","line":7,"updated":"2012-11-19 17:04:30.000000000","message":"Please set this off from the descriptive text with a newline; otherwise, your entire commit message becomes the summary.","commit_id":"9c4360907e692df4ab9bbe9f17405257ef103d83"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"22b3a29805d595911b979e7c8ebeba9f3df798cb","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2012-11-17 04:26:08 -0600"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Added --extra-opts to the nova ssh command"},{"line_number":8,"context_line":"This will allow for remote command execution and advanced ssh options like portforwarding and connection sharing. The goal is to support ssh commands like nova ssh fe55adc9-cb1e-44a4-bd36-6a537b238172 --extra-opts\u003d\u0027-NfnMS ~/.ssh/master-fe55adc9-cb1e-44a4-bd36-6a537b238172\u0027 and ssh fe55adc9-cb1e-44a4-bd36-6a537b238172 --extra-opts\u003d\u0027-t \"sudo puppet agent --test\"\u0027. This will alow easier scripting of the cli... It is not intended however to replace using the nova python api. Therefore I kept it simple."},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: Icce811caf637fc92c6d6374bfb846ea9525a7e05"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAAMH%2F%2F3bI%3D","line":8,"updated":"2012-11-19 17:04:30.000000000","message":"Please word-wrap the text of your commit message to 72 columns to make it easier to read.","commit_id":"9c4360907e692df4ab9bbe9f17405257ef103d83"}],"novaclient/v1_1/shell.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"22b3a29805d595911b979e7c8ebeba9f3df798cb","unresolved":false,"context_lines":[{"line_number":2124,"context_line":"    dest\u003d\u0027extra\u0027,"},{"line_number":2125,"context_line":"    help\u003d\u0027Extra options to pass to ssh. see: man ssh\u0027,"},{"line_number":2126,"context_line":"    default\u003d\u0027\u0027)"},{"line_number":2127,"context_line":""},{"line_number":2128,"context_line":"def do_ssh(cs, args):"},{"line_number":2129,"context_line":"    \"\"\"SSH into a server.\"\"\""},{"line_number":2130,"context_line":"    addresses \u003d _find_server(cs, args.server).addresses"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMH%2F%2F3bE%3D","line":2127,"updated":"2012-11-19 17:04:30.000000000","message":"Please eliminate this blank line.","commit_id":"9c4360907e692df4ab9bbe9f17405257ef103d83"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"22b3a29805d595911b979e7c8ebeba9f3df798cb","unresolved":false,"context_lines":[{"line_number":2143,"context_line":"            break"},{"line_number":2144,"context_line":""},{"line_number":2145,"context_line":"    identity \u003d \u0027-i %s\u0027 % args.identity if len(args.identity) else \u0027\u0027"},{"line_number":2146,"context_line":"    extra \u003d args.extra if len(args.extra) else \u0027\u0027"},{"line_number":2147,"context_line":""},{"line_number":2148,"context_line":"    #print \"ssh -%d -p%d %s %s@%s %s\" % (version, args.port, identity,"},{"line_number":2149,"context_line":"    #                                         args.login, ip_address, extra)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMH%2F%2F3bA%3D","line":2146,"updated":"2012-11-19 17:04:30.000000000","message":"It should be sufficient to test args.extra, i.e.:\n\n    extra \u003d args.extra or \u0027\u0027","commit_id":"9c4360907e692df4ab9bbe9f17405257ef103d83"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"22b3a29805d595911b979e7c8ebeba9f3df798cb","unresolved":false,"context_lines":[{"line_number":2147,"context_line":""},{"line_number":2148,"context_line":"    #print \"ssh -%d -p%d %s %s@%s %s\" % (version, args.port, identity,"},{"line_number":2149,"context_line":"    #                                         args.login, ip_address, extra)"},{"line_number":2150,"context_line":"    #raise Exception, \u0027tst\u0027"},{"line_number":2151,"context_line":"    if ip_address:"},{"line_number":2152,"context_line":"        os.system(\"ssh -%d -p%d %s %s@%s %s\" % (version, args.port, identity,"},{"line_number":2153,"context_line":"                                             args.login, ip_address, extra))"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMH%2F%2F3a4%3D","line":2150,"updated":"2012-11-19 17:04:30.000000000","message":"Please remove code rather than commenting it out.","commit_id":"9c4360907e692df4ab9bbe9f17405257ef103d83"}]}