)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jakub Libosvar \u003clibosvar@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-04-12 22:02:30 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Log ovsdb commands"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iccac96bb0211a0132afebaff0fcfb5c42d6a3148"},{"line_number":10,"context_line":"Related-Bug: 1567668"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a18252c_1f00e600","line":7,"updated":"2016-04-13 08:10:35.000000000","message":"Please add some more meat here. Why do we need to log the comands ...","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"b899d96daa98c9e8f24b59bddbe10ba06d0c1e87","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Jakub Libosvar \u003clibosvar@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-04-12 22:02:30 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Log ovsdb commands"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Iccac96bb0211a0132afebaff0fcfb5c42d6a3148"},{"line_number":10,"context_line":"Related-Bug: 1567668"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"5a18252c_3bedc825","line":7,"in_reply_to":"5a18252c_1f00e600","updated":"2016-04-13 20:50:16.000000000","message":"Done","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"}],"neutron/agent/ovsdb/impl_idl.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":57,"context_line":"        try:"},{"line_number":58,"context_line":"            result \u003d self.results.get(timeout\u003d5)"},{"line_number":59,"context_line":"        except Queue.Empty as e:"},{"line_number":60,"context_line":"            LOG.error(\"Waiting for results but got exception: %s\", e)"},{"line_number":61,"context_line":"            raise"},{"line_number":62,"context_line":"        if self.check_error:"},{"line_number":63,"context_line":"            if isinstance(result, idlutils.ExceptionResult):"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_ff024a08","line":60,"updated":"2016-04-13 08:10:35.000000000","message":"missing log hint _LE","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"}],"neutron/agent/ovsdb/native/commands.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        self.result \u003d None"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def execute(self, check_error\u003dFalse, log_errors\u003dTrue):"},{"line_number":33,"context_line":"        LOG.debug(\"Running ovsdb command: %s\", self)"},{"line_number":34,"context_line":"        try:"},{"line_number":35,"context_line":"            with self.api.transaction(check_error, log_errors) as txn:"},{"line_number":36,"context_line":"                txn.add(self)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_df1e6e62","line":33,"updated":"2016-04-13 08:10:35.000000000","message":"self?","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"eee93a57a5aebc9694ff9573b55db22c61865f37","unresolved":false,"context_lines":[{"line_number":30,"context_line":"        self.result \u003d None"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"    def execute(self, check_error\u003dFalse, log_errors\u003dTrue):"},{"line_number":33,"context_line":"        LOG.debug(\"Running ovsdb command: %s\", self)"},{"line_number":34,"context_line":"        try:"},{"line_number":35,"context_line":"            with self.api.transaction(check_error, log_errors) as txn:"},{"line_number":36,"context_line":"                txn.add(self)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_df566ed1","line":33,"in_reply_to":"5a18252c_df1e6e62","updated":"2016-04-13 08:13:52.000000000","message":"Right, we want to log the command.","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"}],"neutron/agent/ovsdb/native/connection.py":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    def start(self):"},{"line_number":66,"context_line":"        LOG.debug(\"Starting connection\")"},{"line_number":67,"context_line":"        with self.lock:"},{"line_number":68,"context_line":"            LOG.debug(\"Obtained lock\")"},{"line_number":69,"context_line":"            if self.idl is not None:"},{"line_number":70,"context_line":"                return"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_3f24c2ad","line":68,"updated":"2016-04-13 08:10:35.000000000","message":"what does this log provide? what if there are multiple calls?","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":8655,"name":"Jakub Libosvar","email":"libosvar@redhat.com","username":"jlibosva"},"change_message_id":"eee93a57a5aebc9694ff9573b55db22c61865f37","unresolved":false,"context_lines":[{"line_number":65,"context_line":"    def start(self):"},{"line_number":66,"context_line":"        LOG.debug(\"Starting connection\")"},{"line_number":67,"context_line":"        with self.lock:"},{"line_number":68,"context_line":"            LOG.debug(\"Obtained lock\")"},{"line_number":69,"context_line":"            if self.idl is not None:"},{"line_number":70,"context_line":"                return"},{"line_number":71,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_5f59befd","line":68,"in_reply_to":"5a18252c_3f24c2ad","updated":"2016-04-13 08:13:52.000000000","message":"I suspect that the thread is not started so I\u0027m trying to pinpoint where we still get code executed and where it gets stuck.","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":111,"context_line":"                except Exception as ex:"},{"line_number":112,"context_line":"                    er \u003d idlutils.ExceptionResult(ex\u003dex,"},{"line_number":113,"context_line":"                                                  tb\u003dtraceback.format_exc())"},{"line_number":114,"context_line":"                    LOG.error(\"Got exception commiting transaction %s: %s\","},{"line_number":115,"context_line":"                              txn, ex)"},{"line_number":116,"context_line":"                    LOG.error(\"Results are %s\", er)"},{"line_number":117,"context_line":"                    txn.results.put(er)"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_9f42363d","line":114,"updated":"2016-04-13 08:10:35.000000000","message":"missing hint","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":112,"context_line":"                    er \u003d idlutils.ExceptionResult(ex\u003dex,"},{"line_number":113,"context_line":"                                                  tb\u003dtraceback.format_exc())"},{"line_number":114,"context_line":"                    LOG.error(\"Got exception commiting transaction %s: %s\","},{"line_number":115,"context_line":"                              txn, ex)"},{"line_number":116,"context_line":"                    LOG.error(\"Results are %s\", er)"},{"line_number":117,"context_line":"                    txn.results.put(er)"},{"line_number":118,"context_line":"                self.txns.task_done()"}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_1f9c66bf","line":115,"updated":"2016-04-13 08:10:35.000000000","message":"this should be a dictionary","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"},{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"change_message_id":"8e5824a3ba850874798a446cfb10cb212ed769c5","unresolved":false,"context_lines":[{"line_number":113,"context_line":"                                                  tb\u003dtraceback.format_exc())"},{"line_number":114,"context_line":"                    LOG.error(\"Got exception commiting transaction %s: %s\","},{"line_number":115,"context_line":"                              txn, ex)"},{"line_number":116,"context_line":"                    LOG.error(\"Results are %s\", er)"},{"line_number":117,"context_line":"                    txn.results.put(er)"},{"line_number":118,"context_line":"                self.txns.task_done()"},{"line_number":119,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a18252c_5f6ffec4","line":116,"updated":"2016-04-13 08:10:35.000000000","message":"ditto","commit_id":"90d7bcbbad745bd342f54b6815a1dac0750bd0f8"}]}