)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a866e4784bed767d9123c755f6a147ead835443f","unresolved":false,"context_lines":[{"line_number":36,"context_line":"  I361af845d6a733618ecd056aa7df973191184ae9 not being present."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* CinderFixtureNewAttachFlow is used by all tests due to"},{"line_number":39,"context_line":"  I6a777b4b7a5729488f939df8c40e49bd40aec3dd not being present."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* _get_vcpu_total is used instead of _get_vcpu_available due to"},{"line_number":42,"context_line":"  I98efdc61fd456fc7f9e1a85238c9ef9bc04a1252 not being present."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff570b3c_062f6988","line":39,"updated":"2020-06-12 11:43:51.000000000","message":"note: also missing this patch from stein: Ie29307d7b4a6d0118a93d2390141403ebe8d5353\nbut this is just a minor thing as it is mentioned in the above patch, so no need to update the commit message :)","commit_id":"d7a1cb57247c4b1faa09b9c81c5cd2fbd4639095"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d28e0c3520b7765681dc8d5413b30b78cebe6fcc","unresolved":false,"context_lines":[{"line_number":30,"context_line":"backports. Future changes should be made to use this class removing some"},{"line_number":31,"context_line":"of the common setUp logic from _LibvirtEvacuateTest."},{"line_number":32,"context_line":""},{"line_number":33,"context_line":"NOTE(lyarwood): The following changes are required for stable/stein:"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"* [libvirt]/rbd_user is now set within LibvirtRbdEvacuateTest due to"},{"line_number":36,"context_line":"  I361af845d6a733618ecd056aa7df973191184ae9 not being present."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"* CinderFixtureNewAttachFlow is used by all tests due to"},{"line_number":39,"context_line":"  I6a777b4b7a5729488f939df8c40e49bd40aec3dd not being present."},{"line_number":40,"context_line":""},{"line_number":41,"context_line":"* _get_vcpu_total is used instead of _get_vcpu_available due to"},{"line_number":42,"context_line":"  I98efdc61fd456fc7f9e1a85238c9ef9bc04a1252 not being present."},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"Co-Authored-By: Lee Yarwood \u003clyarwood@redhat.com\u003e"},{"line_number":45,"context_line":"Related-Bug: #1550919"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"ff570b3c_85ba8807","line":42,"range":{"start_line":33,"start_character":0,"end_line":42,"end_character":62},"updated":"2020-06-09 15:20:08.000000000","message":"Bears out by diffing this vs. a manual backport:\n\n    diff --git nova/tests/functional/libvirt/test_evacuate.py nova/tests/functional/libvirt/test_evacuate.py\n    index 605c0792f8..1ff871da7a 100644\n    --- nova/tests/functional/libvirt/test_evacuate.py\n    +++ nova/tests/functional/libvirt/test_evacuate.py\n    @@ -202,6 +202,7 @@ class _RbdTest(object):\n             super(_RbdTest, self).setUp()\n     \n             self.flags(group\u003d\u0027libvirt\u0027, images_type\u003d\u0027rbd\u0027)\n    +        self.flags(group\u003d\u0027libvirt\u0027, rbd_user\u003d\u0027rbd\u0027)\n             self.flags(group\u003d\u0027libvirt\u0027, rbd_secret_uuid\u003d\u00271234\u0027)\n     \n             self.created \u003d set()\n    @@ -423,7 +424,7 @@ class _LibvirtEvacuateTest(integrated_helpers.InstanceHelperMixin):\n         def setUp(self):\n             super(_LibvirtEvacuateTest, self).setUp()\n     \n    -        self.useFixture(nova_fixtures.CinderFixture(self))\n    +        self.useFixture(nova_fixtures.CinderFixtureNewAttachFlow(self))\n             self.useFixture(nova_fixtures.NeutronFixture(self))\n             self.useFixture(func_fixtures.PlacementFixture())\n             fake_network.set_stub_network_methods(self)\n    @@ -468,8 +469,8 @@ class _LibvirtEvacuateTest(integrated_helpers.InstanceHelperMixin):\n             get_local_gb_info.return_value \u003d {\n                 \u0027total\u0027: lots, \u0027free\u0027: lots, \u0027used\u0027: 1}\n             get_vcpu_available \u003d self.useFixture(fixtures.MockPatch(\n    -            \u0027nova.virt.libvirt.driver.LibvirtDriver._get_vcpu_available\u0027)).mock\n    -        get_vcpu_available.return_value \u003d set(cpu for cpu in range(24))\n    +            \u0027nova.virt.libvirt.driver.LibvirtDriver._get_vcpu_total\u0027)).mock\n    +        get_vcpu_available.return_value \u003d 24\n             get_memory_mb_total \u003d self.useFixture(fixtures.MockPatch(\n                 \u0027nova.virt.libvirt.host.Host.get_memory_mb_total\u0027)).mock\n             get_memory_mb_total.return_value \u003d lots\n\nSimilar to the stable/train change, I\u0027d like a \u0027Changes\u0027 section to list the files actually modified (even though there\u0027s only one file here) but that\u0027s a nice-to-have","commit_id":"d7a1cb57247c4b1faa09b9c81c5cd2fbd4639095"}]}
