)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7e89297e3eb574650934015247e868bade59511c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Fix test_port_resource_request_inherited_policy test"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"In \u0027test_port_resource_request_inherited_policy\u0027 test a QoS policy"},{"line_number":10,"context_line":"is assigned to a network. The network used in this test is created"},{"line_number":11,"context_line":"in resource_setup(), which is run as part of setUpClass() phase."},{"line_number":12,"context_line":"Resources allocated in resource_setup() are shared by all tests"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"11641b44_7eaade45","line":9,"range":{"start_line":9,"start_character":48,"end_line":9,"end_character":61},"updated":"2021-11-16 12:04:58.000000000","message":"nit: we test if a QoS....","commit_id":"24c88a489ea08edc66f3e591e8f4effe0a3cc462"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"e88b6722200d730e1421ad90ca837ef6af44897c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3098930e_5e9137cc","updated":"2021-11-10 12:39:46.000000000","message":"Blocked until new Tempest release.","commit_id":"7e58c27ff44fd6ee406b61a62c91c8a3c246af73"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"1c32f5a0fa45f269f95e4863f5398d8e5ecfc538","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1e9dd2fd_43f35228","updated":"2021-11-24 07:52:36.000000000","message":"recheck","commit_id":"1de6098403a2771607c34ac7e5feb07c43be3e8a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"59b8e1b7027904b469823307183267e9237fcb64","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"e2bfeeef_e9d57f4d","updated":"2022-01-20 10:14:51.000000000","message":"recheck","commit_id":"83e340c9ac0c6f1b8962e2fbb1d3c53983ac4d6a"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f3b548ef14cdda36257e6acf4ba47e0e2eac4d26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"aa168fc4_78259a35","updated":"2022-01-24 15:56:54.000000000","message":"recheck\nxena hit https://bugs.launchpad.net/neutron/+bug/1799790\nthe API tests failure is strange, it failed with listing qos policies with name\u003dtest","commit_id":"591b75826d870b86f97f0d49cbfa9b6626c157c2"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"43a072a3e0a3fc1b4ac51d7c1ae0dbb5e36a5c12","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b0610a24_41111dba","updated":"2022-02-09 13:05:23.000000000","message":"recheck","commit_id":"1f3d6b92783aa20bdfe751ec64c9699e1ac61dfd"}],"neutron_tempest_plugin/api/admin/test_ports.py":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"cab9fbcb2b0208bd1562ef1d340db309689e25d7","unresolved":true,"context_lines":[{"line_number":92,"context_line":"        cls.physnet_name \u003d CONF.neutron_plugin_options.provider_vlans[0]"},{"line_number":93,"context_line":"        base_segm \u003d CONF.neutron_plugin_options.provider_net_base_segm_id"},{"line_number":94,"context_line":"        cls.prov_network \u003d cls.create_provider_network("},{"line_number":95,"context_line":"            physnet_name\u003dcls.physnet_name, start_segmentation_id\u003dbase_segm)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @classmethod"},{"line_number":98,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"39f581a4_814733d9","line":95,"updated":"2021-11-16 13:00:05.000000000","message":"so do we need to create that network here? Maybe we should create it in each test?","commit_id":"24c88a489ea08edc66f3e591e8f4effe0a3cc462"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"a7ca8b2d33815415541d12eaf12d1692da24a796","unresolved":true,"context_lines":[{"line_number":92,"context_line":"        cls.physnet_name \u003d CONF.neutron_plugin_options.provider_vlans[0]"},{"line_number":93,"context_line":"        base_segm \u003d CONF.neutron_plugin_options.provider_net_base_segm_id"},{"line_number":94,"context_line":"        cls.prov_network \u003d cls.create_provider_network("},{"line_number":95,"context_line":"            physnet_name\u003dcls.physnet_name, start_segmentation_id\u003dbase_segm)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @classmethod"},{"line_number":98,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"8fb9e92b_03e28001","line":95,"in_reply_to":"39f581a4_814733d9","updated":"2021-11-19 08:05:28.000000000","message":"It\u0027s a common pattern in Tempest to create resources in resource_setup (setUpClass). Personally, I\u0027d prefer if resource_setup was run as part of setUp, to ensure that resources are deallocated after each test. The question is whether we want to stick to Tempest best practices or not.","commit_id":"24c88a489ea08edc66f3e591e8f4effe0a3cc462"},{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"504aa416ab89f0ebb9ed0a2ff4f56588ed23287f","unresolved":true,"context_lines":[{"line_number":92,"context_line":"        cls.physnet_name \u003d CONF.neutron_plugin_options.provider_vlans[0]"},{"line_number":93,"context_line":"        base_segm \u003d CONF.neutron_plugin_options.provider_net_base_segm_id"},{"line_number":94,"context_line":"        cls.prov_network \u003d cls.create_provider_network("},{"line_number":95,"context_line":"            physnet_name\u003dcls.physnet_name, start_segmentation_id\u003dbase_segm)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    @classmethod"},{"line_number":98,"context_line":"    def setup_clients(cls):"}],"source_content_type":"text/x-python","patch_set":3,"id":"82f8dc76_f7ad6fc1","line":95,"in_reply_to":"8fb9e92b_03e28001","updated":"2021-11-22 19:44:31.000000000","message":"I think that if we don\u0027t need it at all, what\u0027s the point to create it? It may save few seconds in the test execution. But it\u0027s not critical thing for me 😊","commit_id":"24c88a489ea08edc66f3e591e8f4effe0a3cc462"}]}