)]}'
{"nova/console/websocketproxy.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a274d64976c8c511bca1760e5b0a7294175cb654","unresolved":false,"context_lines":[{"line_number":30,"context_line":""},{"line_number":31,"context_line":"LOG \u003d logging.getLogger(__name__)"},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"USE_NEW_API \u003d getattr(websockify, \u0027ProxyRequestHandler\u0027, None) is not None"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class NovaProxyRequestHandlerBase(object):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_e6ffd9ff","line":33,"updated":"2014-05-09 14:16:04.000000000","message":"honestly, I\u0027d much rather have this next to the code in question instead of split off. As it doesn\u0027t seem like anyone else needs this variable.","commit_id":"3e6101e53998e9943cf3c52c290de6fd35b06ab7"},{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"a274d64976c8c511bca1760e5b0a7294175cb654","unresolved":false,"context_lines":[{"line_number":86,"context_line":""},{"line_number":87,"context_line":"# TODO(sross): when the K release rolls around, replace the contents"},{"line_number":88,"context_line":"#              of this class with the contents of NovaWebSocketProxyNew,"},{"line_number":89,"context_line":"#              and remove the legacy code"},{"line_number":90,"context_line":"if USE_NEW_API:"},{"line_number":91,"context_line":"    class NovaProxyRequestHandler(NovaProxyRequestHandlerBase,"},{"line_number":92,"context_line":"                                  websockify.ProxyRequestHandler):"}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_260aa121","line":89,"updated":"2014-05-09 14:16:04.000000000","message":"This really isn\u0027t about K rolling around, as when websockets requirement is bumped.","commit_id":"3e6101e53998e9943cf3c52c290de6fd35b06ab7"},{"author":{"_account_id":5441,"name":"Andrew Laski","email":"andrew@lascii.com","username":"alaski"},"change_message_id":"a5ddd1581234e6ef4427fffb0273fefc81693f12","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                    tsock.recv(len(data))"},{"line_number":71,"context_line":"                    break"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        self.print_traffic(self.traffic_legend)"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        # Start proxying"},{"line_number":76,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_5c7bf4bd","line":73,"updated":"2014-06-17 13:09:55.000000000","message":"This debug print was removed in I392f599e81790376dab11f9323430ee9abcff444 so I question whether it should be added back in.  It seems like this should be left out now.","commit_id":"2b3642243b06cb1bd7c7513b0c63b3aae17d01fa"},{"author":{"_account_id":7677,"name":"Solly Ross","email":"sross@redhat.com","username":"sross"},"change_message_id":"4cf387aa710c079e0176cc9ec49f778b188349de","unresolved":false,"context_lines":[{"line_number":70,"context_line":"                    tsock.recv(len(data))"},{"line_number":71,"context_line":"                    break"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"        self.print_traffic(self.traffic_legend)"},{"line_number":74,"context_line":""},{"line_number":75,"context_line":"        # Start proxying"},{"line_number":76,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":5,"id":"1ae5cdf2_3fa09f8f","line":73,"in_reply_to":"1ae5cdf2_5c7bf4bd","updated":"2014-06-18 13:18:50.000000000","message":"It does only print if self.traffic is true, and it currently is only printed when verbose is enabled.  However, since it seems to have been removed, I\u0027ll just drop it.","commit_id":"2b3642243b06cb1bd7c7513b0c63b3aae17d01fa"}]}