)]}'
{"searchlight/listener.py":[{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"9540673e7548783fa826e8fd30fcc056c8a46df8","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    def _log_notification(self, handler, ctxt, doc_type, event_type,"},{"line_number":71,"context_line":"                          payload, metadata):"},{"line_number":72,"context_line":"        project \u003d ctxt.get(\u0027project_id\u0027, ctxt.get(\u0027tenant_id\u0027,"},{"line_number":73,"context_line":"                                                  ctxt.get(\u0027tenant\u0027, \u0027-\u0027)))"},{"line_number":74,"context_line":"        log_context \u003d {\u0027event_type\u0027: event_type,"},{"line_number":75,"context_line":"                       \u0027doc_type\u0027: doc_type,"},{"line_number":76,"context_line":"                       \u0027timestamp\u0027: metadata[\u0027timestamp\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_c577d700","line":73,"updated":"2016-07-18 13:37:59.000000000","message":"Found a issue here, when i testing reboot instance action, i got None for project for the last notification. But i can get the right project from payload by payload.get(\u0027tenant_id\u0027).","commit_id":"ecd8b5ab731bac7a5293e3769b10acd63e70b4a1"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"336b065127339c03e4e4889204bb4df4f60e31c0","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    def _log_notification(self, handler, ctxt, doc_type, event_type,"},{"line_number":71,"context_line":"                          payload, metadata):"},{"line_number":72,"context_line":"        project \u003d ctxt.get(\u0027project_id\u0027, ctxt.get(\u0027tenant_id\u0027,"},{"line_number":73,"context_line":"                                                  ctxt.get(\u0027tenant\u0027, \u0027-\u0027)))"},{"line_number":74,"context_line":"        log_context \u003d {\u0027event_type\u0027: event_type,"},{"line_number":75,"context_line":"                       \u0027doc_type\u0027: doc_type,"},{"line_number":76,"context_line":"                       \u0027timestamp\u0027: metadata[\u0027timestamp\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_ee20f2f9","line":73,"in_reply_to":"dada55a8_c577d700","updated":"2016-07-18 15:55:41.000000000","message":"Yeah, i noticed that for some notifications. I was trying to avoid getting the tenant id from the payload but it might be unavoidable.","commit_id":"ecd8b5ab731bac7a5293e3769b10acd63e70b4a1"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"3e5872741c749a9c3777c81907e2452b5eb9861e","unresolved":false,"context_lines":[{"line_number":70,"context_line":"    def _log_notification(self, handler, ctxt, doc_type, event_type,"},{"line_number":71,"context_line":"                          payload, metadata):"},{"line_number":72,"context_line":"        project \u003d ctxt.get(\u0027project_id\u0027, ctxt.get(\u0027tenant_id\u0027,"},{"line_number":73,"context_line":"                                                  ctxt.get(\u0027tenant\u0027, \u0027-\u0027)))"},{"line_number":74,"context_line":"        log_context \u003d {\u0027event_type\u0027: event_type,"},{"line_number":75,"context_line":"                       \u0027doc_type\u0027: doc_type,"},{"line_number":76,"context_line":"                       \u0027timestamp\u0027: metadata[\u0027timestamp\u0027],"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_3381cf4e","line":73,"in_reply_to":"dada55a8_ee20f2f9","updated":"2016-07-22 22:18:18.000000000","message":"I\u0027ve added an attempt to get it from the payload, but resisted the urge to add functionality to the handler classes to decide it for themselves","commit_id":"ecd8b5ab731bac7a5293e3769b10acd63e70b4a1"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"9540673e7548783fa826e8fd30fcc056c8a46df8","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        additional \u003d \" \".join(\"%s:%s\" % (k, v or \u0027-\u0027)"},{"line_number":80,"context_line":"                              for k, v in payload_fields)"},{"line_number":81,"context_line":"        log_context[\u0027additional\u0027] \u003d additional or \u0027\u0027"},{"line_number":82,"context_line":"        LOG.info(_LI(\"%(doc_type)s %(event_type)s \\\"%(timestamp)s\\\" \""},{"line_number":83,"context_line":"                     \"project_id:%(project)s %(additional)s\"), log_context)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    def info(self, ctxt, publisher_id, event_type, payload, metadata):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_25bb5374","line":82,"updated":"2016-07-18 13:37:59.000000000","message":"Maybe add a message prefix something like \u0027Received notification: ...\u0027  for filtering purpose.","commit_id":"ecd8b5ab731bac7a5293e3769b10acd63e70b4a1"},{"author":{"_account_id":10063,"name":"Steve McLellan","email":"steven.j.mclellan@gmail.com","username":"sjmc7"},"change_message_id":"336b065127339c03e4e4889204bb4df4f60e31c0","unresolved":false,"context_lines":[{"line_number":79,"context_line":"        additional \u003d \" \".join(\"%s:%s\" % (k, v or \u0027-\u0027)"},{"line_number":80,"context_line":"                              for k, v in payload_fields)"},{"line_number":81,"context_line":"        log_context[\u0027additional\u0027] \u003d additional or \u0027\u0027"},{"line_number":82,"context_line":"        LOG.info(_LI(\"%(doc_type)s %(event_type)s \\\"%(timestamp)s\\\" \""},{"line_number":83,"context_line":"                     \"project_id:%(project)s %(additional)s\"), log_context)"},{"line_number":84,"context_line":""},{"line_number":85,"context_line":"    def info(self, ctxt, publisher_id, event_type, payload, metadata):"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_4e5f7e72","line":82,"in_reply_to":"dada55a8_25bb5374","updated":"2016-07-18 15:55:41.000000000","message":"OK. At INFO level this is the only message but at debug it\u0027s harder to find.","commit_id":"ecd8b5ab731bac7a5293e3769b10acd63e70b4a1"}]}