)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"36f5f37960331ecc156d2eb2b52d79e41d588ab1","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Stephen Finucane \u003cstephenfin@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2021-11-11 19:03:48 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"test: Restore - don\u0027t reset - warning filters"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are more various warning filters pre-configured in a typical"},{"line_number":10,"context_line":"Python environment, including a few from third-party libraries such as"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"496eb77c_18d37c7c","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":4},"updated":"2021-11-11 19:27:44.000000000","message":"tests\n\ndoh","commit_id":"b1fe37de569458629ad77046247d6ad16adda3f2"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"36f5f37960331ecc156d2eb2b52d79e41d588ab1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are more various warning filters pre-configured in a typical"},{"line_number":10,"context_line":"Python environment, including a few from third-party libraries such as"},{"line_number":11,"context_line":"requests [1][2] and urllib3 [2] as well as stdlib [3]. Our fixture to"},{"line_number":12,"context_line":"configure warnings, \u0027WarningsFixture\u0027, called \u0027warnings.resetwarnings\u0027"},{"line_number":13,"context_line":"which *reset* all the warning filters [5]. This is clearly not something"},{"line_number":14,"context_line":"we want to do, and resulted in tests puking warnings after the initial"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5de7ae23_aca201bd","line":11,"range":{"start_line":11,"start_character":29,"end_line":11,"end_character":30},"updated":"2021-11-11 19:27:44.000000000","message":"3","commit_id":"b1fe37de569458629ad77046247d6ad16adda3f2"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"36f5f37960331ecc156d2eb2b52d79e41d588ab1","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"There are more various warning filters pre-configured in a typical"},{"line_number":10,"context_line":"Python environment, including a few from third-party libraries such as"},{"line_number":11,"context_line":"requests [1][2] and urllib3 [2] as well as stdlib [3]. Our fixture to"},{"line_number":12,"context_line":"configure warnings, \u0027WarningsFixture\u0027, called \u0027warnings.resetwarnings\u0027"},{"line_number":13,"context_line":"which *reset* all the warning filters [5]. This is clearly not something"},{"line_number":14,"context_line":"we want to do, and resulted in tests puking warnings after the initial"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a809078b_858f51ab","line":11,"range":{"start_line":11,"start_character":51,"end_line":11,"end_character":52},"updated":"2021-11-11 19:27:44.000000000","message":"4","commit_id":"b1fe37de569458629ad77046247d6ad16adda3f2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"6898787e3abe508c9e437bdd9cc3e38aec1943e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"edad50a0_0b18a987","updated":"2021-11-11 20:03:56.000000000","message":"Looks sensible to me","commit_id":"e28afc564700a1a35e3bf0269687d5734251b88a"}],"nova/tests/fixtures/nova.py":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"6898787e3abe508c9e437bdd9cc3e38aec1943e1","unresolved":true,"context_lines":[{"line_number":841,"context_line":""},{"line_number":842,"context_line":"        self.addCleanup(self._reset_warning_filters)"},{"line_number":843,"context_line":""},{"line_number":844,"context_line":"    def _reset_warning_filters(self):"},{"line_number":845,"context_line":"        warnings.filters[:] \u003d self._original_warning_filters"},{"line_number":846,"context_line":""},{"line_number":847,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"c78caacb_8f42721e","line":844,"range":{"start_line":844,"start_character":9,"end_line":844,"end_character":14},"updated":"2021-11-11 20:03:56.000000000","message":"I was expecting this to be \"restore\" and not \"reset\" given the commit message 😛","commit_id":"e28afc564700a1a35e3bf0269687d5734251b88a"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"1173a776994b79314da5629bc0690958f3d86b63","unresolved":false,"context_lines":[{"line_number":841,"context_line":""},{"line_number":842,"context_line":"        self.addCleanup(self._reset_warning_filters)"},{"line_number":843,"context_line":""},{"line_number":844,"context_line":"    def _reset_warning_filters(self):"},{"line_number":845,"context_line":"        warnings.filters[:] \u003d self._original_warning_filters"},{"line_number":846,"context_line":""},{"line_number":847,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"239f5593_b6c6c643","line":844,"range":{"start_line":844,"start_character":9,"end_line":844,"end_character":14},"in_reply_to":"c78caacb_8f42721e","updated":"2021-11-12 09:17:07.000000000","message":"Looking back on it, that would have been a better name 🙈","commit_id":"e28afc564700a1a35e3bf0269687d5734251b88a"}]}
