)]}'
{"oslo_policy/generator.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2f87931d630d8094829c10f1240877c4583b0fcd","unresolved":false,"context_lines":[{"line_number":108,"context_line":"                # delineate things"},{"line_number":109,"context_line":"                # TODO(stephenfin): Raise an exception here and stop doing"},{"line_number":110,"context_line":"                # anything else in oslo.policy 2.0"},{"line_number":111,"context_line":"                LOG.warning(\u0027Invalid policy description: literal blocks must \u0027"},{"line_number":112,"context_line":"                            \u0027be preceded by a new line. This will raise an \u0027"},{"line_number":113,"context_line":"                            \u0027exception in a future version of oslo.policy\u0027)"},{"line_number":114,"context_line":"                LOG.debug(description)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f2577a7_1ccb781e","line":111,"updated":"2017-09-13 19:47:48.000000000","message":"This warning should give more info about the policy that has theban description. Can we identify it by name or line number or something?","commit_id":"1e3aeaf6e63bae572b58856dc316476e5f89190f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"2240d3c373a5778bb7cf02c771ac9a6c6860930e","unresolved":false,"context_lines":[{"line_number":108,"context_line":"                # delineate things"},{"line_number":109,"context_line":"                # TODO(stephenfin): Raise an exception here and stop doing"},{"line_number":110,"context_line":"                # anything else in oslo.policy 2.0"},{"line_number":111,"context_line":"                LOG.warning(\u0027Invalid policy description: literal blocks must \u0027"},{"line_number":112,"context_line":"                            \u0027be preceded by a new line. This will raise an \u0027"},{"line_number":113,"context_line":"                            \u0027exception in a future version of oslo.policy\u0027)"},{"line_number":114,"context_line":"                LOG.debug(description)"}],"source_content_type":"text/x-python","patch_set":2,"id":"ff5d4b2f_4adce845","line":111,"in_reply_to":"5f2577a7_1ccb781e","updated":"2017-09-19 12:05:36.000000000","message":"Isn\u0027t that what we\u0027re doing directly below this?","commit_id":"1e3aeaf6e63bae572b58856dc316476e5f89190f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3e61c6e0060e0379cb21fe0e2f5259da59c18b51","unresolved":false,"context_lines":[{"line_number":108,"context_line":"                # delineate things"},{"line_number":109,"context_line":"                # TODO(stephenfin): Raise an exception here and stop doing"},{"line_number":110,"context_line":"                # anything else in oslo.policy 2.0"},{"line_number":111,"context_line":"                LOG.warning(\u0027Invalid policy description: literal blocks must \u0027"},{"line_number":112,"context_line":"                            \u0027be preceded by a new line. This will raise an \u0027"},{"line_number":113,"context_line":"                            \u0027exception in a future version of oslo.policy\u0027)"},{"line_number":114,"context_line":"                LOG.debug(description)"}],"source_content_type":"text/x-python","patch_set":2,"id":"3f4b6375_e3f53a58","line":111,"in_reply_to":"bf57530f_b882854e","updated":"2017-10-26 09:30:43.000000000","message":"Done","commit_id":"1e3aeaf6e63bae572b58856dc316476e5f89190f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"43e46a1b815a3bd5797cdd08955e9c483376ba3b","unresolved":false,"context_lines":[{"line_number":108,"context_line":"                # delineate things"},{"line_number":109,"context_line":"                # TODO(stephenfin): Raise an exception here and stop doing"},{"line_number":110,"context_line":"                # anything else in oslo.policy 2.0"},{"line_number":111,"context_line":"                LOG.warning(\u0027Invalid policy description: literal blocks must \u0027"},{"line_number":112,"context_line":"                            \u0027be preceded by a new line. This will raise an \u0027"},{"line_number":113,"context_line":"                            \u0027exception in a future version of oslo.policy\u0027)"},{"line_number":114,"context_line":"                LOG.debug(description)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bf57530f_b882854e","line":111,"in_reply_to":"ff5d4b2f_4adce845","updated":"2017-09-20 18:55:39.000000000","message":"That is being output at a different log level than the warning, so may not be present in the output.","commit_id":"1e3aeaf6e63bae572b58856dc316476e5f89190f"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"dca4d10bed96cf4c194f8f172f1fdfb5ab552b01","unresolved":false,"context_lines":[{"line_number":112,"context_line":"                # delineate things"},{"line_number":113,"context_line":"                # TODO(stephenfin): Raise an exception here and stop doing"},{"line_number":114,"context_line":"                # anything else in oslo.policy 2.0"},{"line_number":115,"context_line":"                LOG.warning(\u0027Invalid policy description: literal blocks must \u0027"},{"line_number":116,"context_line":"                            \u0027be preceded by a new line. This will raise an \u0027"},{"line_number":117,"context_line":"                            \u0027exception in a future version of oslo.policy\u0027)"},{"line_number":118,"context_line":"                LOG.warning(description)"}],"source_content_type":"text/x-python","patch_set":3,"id":"3f4b6375_ae3c6dbf","line":115,"updated":"2017-10-30 19:57:57.000000000","message":"This is the wrong way to emit this warning. Log messages at warning level imply an operator\u0027s pager is going to go off. (Or cell phone. Whatever.) Operators can\u0027t fix the problem this warning is describing, though, so we shouldn\u0027t wake them up.\n\nInstead of logging the warning, use the warning module to report it. That should make it show up in places like output from unit tests where developers will see it.","commit_id":"ac574753472f1d6a9429a2b9ffab31f63022b253"}]}