)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d0216c3abd574e0c42f5edc1edee19dd8d416842","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"libvirt: Fix indentation in test__video_model_supported"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This would previously result in"},{"line_number":10,"context_line":"nova.virt.libvirt.host.Host.has_min_version not being correctly mocked"},{"line_number":11,"context_line":"and would instead rely on the versions of libvirt and QEMU being"},{"line_number":12,"context_line":"reported by FakeLibvirt being lower than"},{"line_number":13,"context_line":"MIN_LIBVIRT_VIDEO_MODEL_VERSIONS[\u0027none\u0027] aka libvirt v4.6.0."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I4624ff7dd1e71c8a9656e4ce2118fd04941a4086"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"9f560f44_8d1ff56a","line":13,"range":{"start_line":9,"start_character":0,"end_line":13,"end_character":60},"updated":"2020-08-20 11:30:40.000000000","message":"did you actully  observe that behavior.\n\nthe liftime semantics of with statements and mocks are something i have argued with mridem in the past.\n\nthey dont actully intoduce scopes so you can acess the mock after you dedent out of it but i think it deactivates the mock wehn you do that\n\nin the other cases where i have argured it should be indented we were meally doing assert with the mock not invoking functions that needed to be mocked so the difference did not matter.\n\n\nin this case the difference does matter to the new indentaion is correct","commit_id":"30c30f5a590ccbfe9c1354a0bf9c47bf955b0d6d"}]}