)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"f1a2c1232209720a758a8b2fb88868a6e78733f1","unresolved":true,"context_lines":[{"line_number":22,"context_line":"Conflicts:"},{"line_number":23,"context_line":"      nova/tests/functional/libvirt/test_pci_sriov_servers.py"},{"line_number":24,"context_line":"      nova/tests/unit/virt/libvirt/fakelibvirt.py"},{"line_number":25,"context_line":"      nova/tests/functional/libvirt/base.py"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"To avoid the conflicts and make the new functional test execute,"},{"line_number":28,"context_line":"following changes are performed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"d8acf1cc_f4f2b2c5","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":43},"updated":"2021-01-20 11:54:04.000000000","message":"This was dropped in the previous stable/train change so you don\u0027t need to list it here.","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"fc552717d2a6875415ac005cd245483a2744878b","unresolved":false,"context_lines":[{"line_number":22,"context_line":"Conflicts:"},{"line_number":23,"context_line":"      nova/tests/functional/libvirt/test_pci_sriov_servers.py"},{"line_number":24,"context_line":"      nova/tests/unit/virt/libvirt/fakelibvirt.py"},{"line_number":25,"context_line":"      nova/tests/functional/libvirt/base.py"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"To avoid the conflicts and make the new functional test execute,"},{"line_number":28,"context_line":"following changes are performed"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"9a2f0571_09af7d3f","line":25,"range":{"start_line":25,"start_character":0,"end_line":25,"end_character":43},"in_reply_to":"d8acf1cc_f4f2b2c5","updated":"2021-01-21 04:47:56.000000000","message":"Done","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"},{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"f1a2c1232209720a758a8b2fb88868a6e78733f1","unresolved":true,"context_lines":[{"line_number":24,"context_line":"      nova/tests/unit/virt/libvirt/fakelibvirt.py"},{"line_number":25,"context_line":"      nova/tests/functional/libvirt/base.py"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"To avoid the conflicts and make the new functional test execute,"},{"line_number":28,"context_line":"following changes are performed"},{"line_number":29,"context_line":"- Modified the test case to use flavor extra spec pci_passthrough"},{"line_number":30,"context_line":"  :alias to create a server with sriov port instead of creating a"},{"line_number":31,"context_line":"  sriov port and passing port information during server creation."},{"line_number":32,"context_line":"- Removed changes in nova/tests/functional/libvirt/base.py as they"},{"line_number":33,"context_line":"  are required only if neutron sriov port is created in the test"},{"line_number":34,"context_line":"  case."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Changes:"},{"line_number":37,"context_line":"- Included img_mock for the new functional testcase"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b083ec90_35ffd97b","line":34,"range":{"start_line":27,"start_character":0,"end_line":34,"end_character":7},"updated":"2021-01-20 11:54:04.000000000","message":"I wouldn\u0027t include this as it had already changed in the stable/train change.","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"fc552717d2a6875415ac005cd245483a2744878b","unresolved":false,"context_lines":[{"line_number":24,"context_line":"      nova/tests/unit/virt/libvirt/fakelibvirt.py"},{"line_number":25,"context_line":"      nova/tests/functional/libvirt/base.py"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"To avoid the conflicts and make the new functional test execute,"},{"line_number":28,"context_line":"following changes are performed"},{"line_number":29,"context_line":"- Modified the test case to use flavor extra spec pci_passthrough"},{"line_number":30,"context_line":"  :alias to create a server with sriov port instead of creating a"},{"line_number":31,"context_line":"  sriov port and passing port information during server creation."},{"line_number":32,"context_line":"- Removed changes in nova/tests/functional/libvirt/base.py as they"},{"line_number":33,"context_line":"  are required only if neutron sriov port is created in the test"},{"line_number":34,"context_line":"  case."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"Changes:"},{"line_number":37,"context_line":"- Included img_mock for the new functional testcase"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"79ffa9ca_4798ee58","line":34,"range":{"start_line":27,"start_character":0,"end_line":34,"end_character":7},"in_reply_to":"b083ec90_35ffd97b","updated":"2021-01-21 04:47:56.000000000","message":"Done","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"}],"nova/tests/functional/libvirt/test_pci_sriov_servers.py":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"f1a2c1232209720a758a8b2fb88868a6e78733f1","unresolved":true,"context_lines":[{"line_number":221,"context_line":"        self._run_build_test(flavor_id_vfs)"},{"line_number":222,"context_line":"        self._run_build_test(flavor_id_pfs, end_status\u003d\u0027ERROR\u0027)"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    @mock.patch(\u0027nova.virt.libvirt.LibvirtDriver._create_image\u0027,"},{"line_number":225,"context_line":"                return_value\u003d(False, False))"},{"line_number":226,"context_line":"    def test_create_server_after_change_in_nonsriov_pf_to_sriov_pf("},{"line_number":227,"context_line":"            self, img_mock):"},{"line_number":228,"context_line":"        # Starts a compute with PF not configured with SRIOV capabilities"}],"source_content_type":"text/x-python","patch_set":3,"id":"edee2a5c_45378943","line":225,"range":{"start_line":224,"start_character":0,"end_line":225,"end_character":44},"updated":"2021-01-20 11:54:04.000000000","message":"Would it be better to use the fake image service fixture?\n\nhttps://github.com/openstack/nova/blob/d0cb291eea663beff333c466a439ff392e44adc5/nova/tests/functional/integrated_helpers.py#L94-L95","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"},{"author":{"_account_id":10366,"name":"Hemanth N","email":"hemanth.nakkina@canonical.com","username":"Hemanth"},"change_message_id":"fc552717d2a6875415ac005cd245483a2744878b","unresolved":false,"context_lines":[{"line_number":221,"context_line":"        self._run_build_test(flavor_id_vfs)"},{"line_number":222,"context_line":"        self._run_build_test(flavor_id_pfs, end_status\u003d\u0027ERROR\u0027)"},{"line_number":223,"context_line":""},{"line_number":224,"context_line":"    @mock.patch(\u0027nova.virt.libvirt.LibvirtDriver._create_image\u0027,"},{"line_number":225,"context_line":"                return_value\u003d(False, False))"},{"line_number":226,"context_line":"    def test_create_server_after_change_in_nonsriov_pf_to_sriov_pf("},{"line_number":227,"context_line":"            self, img_mock):"},{"line_number":228,"context_line":"        # Starts a compute with PF not configured with SRIOV capabilities"}],"source_content_type":"text/x-python","patch_set":3,"id":"bf5e1248_56544a02","line":225,"range":{"start_line":224,"start_character":0,"end_line":225,"end_character":44},"in_reply_to":"edee2a5c_45378943","updated":"2021-01-21 04:47:56.000000000","message":"Done","commit_id":"46f73acd6d02be19e42b8e18a5f662d1422cb802"}]}