)]}'
{"web/src/actions/logfile.js":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"f32f1570397db2c09acbd6256d4ccb25f6730a4f","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    m \u003d SYSTEMD_LOGMATCH.exec(line)"},{"line_number":50,"context_line":"    if (m) {"},{"line_number":51,"context_line":"      sev \u003d severityMap[m[7]]"},{"line_number":52,"context_line":"    } else {"},{"line_number":53,"context_line":"      OSLO_LOGMATCH.exec(line)"},{"line_number":54,"context_line":"      if (m) {"}],"source_content_type":"text/javascript","patch_set":1,"id":"fffc6b78_9bc15784","line":51,"updated":"2020-11-18 23:33:25.000000000","message":"Or test m[7] here for validity.","commit_id":"5688c782a0e393dee33920b2262414f8528dc094"},{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"94ebe6b29ce7f45b098a810f83ab59a13d541393","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    m \u003d SYSTEMD_LOGMATCH.exec(line)"},{"line_number":50,"context_line":"    if (m) {"},{"line_number":51,"context_line":"      sev \u003d severityMap[m[7]]"},{"line_number":52,"context_line":"    } else {"},{"line_number":53,"context_line":"      OSLO_LOGMATCH.exec(line)"},{"line_number":54,"context_line":"      if (m) {"}],"source_content_type":"text/javascript","patch_set":1,"id":"fffc6b78_1b3287a8","line":51,"updated":"2020-11-18 23:46:00.000000000","message":"That\u0027s a fair argument for why this is different; I agree.","commit_id":"5688c782a0e393dee33920b2262414f8528dc094"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"8998020c143b504c1443c55db6d2f7eb40151eb2","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    m \u003d SYSTEMD_LOGMATCH.exec(line)"},{"line_number":50,"context_line":"    if (m) {"},{"line_number":51,"context_line":"      sev \u003d severityMap[m[7]]"},{"line_number":52,"context_line":"    } else {"},{"line_number":53,"context_line":"      OSLO_LOGMATCH.exec(line)"},{"line_number":54,"context_line":"      if (m) {"}],"source_content_type":"text/javascript","patch_set":1,"id":"fffc6b78_7b5913fd","line":51,"in_reply_to":"fffc6b78_9bc15784","updated":"2020-11-18 23:41:12.000000000","message":"We tracked this down to \n\nhttps://review.opendev.org/#/c/455831/2/os_loganalyze/filter.py\n\nwhere the comment says \n\n\"We also have to realize that status might not be in a systemd line,\nand that\u0027s ok, but still match it, given that otherwise we run the\nrisk of falling into the syslog match.\"\n\nIn our case, we\u0027re really only interested in marking the line with a severity, whereas the os-loganalyze change was trying to parse out the date, host, etc.\n\nSo I don\u0027t think we want to ever match if there\u0027s no severity?","commit_id":"5688c782a0e393dee33920b2262414f8528dc094"}]}