)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ad3969004b30f817578a2beedd3764a2ccbbdbaf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"9edd1188_00ffb1ce","updated":"2021-11-23 12:47:04.000000000","message":"Thanks for taking it over and finishing it","commit_id":"11d3a2ff0871449bd7c451bdf026c5d0258b3cb4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"c3e8bc6d312fea562e954dc768be1f59ade159d6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"2b9257b9_4691aca5","updated":"2021-11-23 16:31:14.000000000","message":"Thanks","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"a2184d623fad5dc720d5164aab8bc38f56c1e775","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"1953ba51_84a4d06e","updated":"2021-11-24 07:17:56.000000000","message":"recheck","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"79e542f8a74a3c7099b023405def23cb2d24fd70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"a934bb64_049628b5","updated":"2021-11-23 16:31:20.000000000","message":"recheck","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8f41f2ad6e2c32a6bb6cac927f0ee4126ed41b2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e3947699_75ac7cc1","updated":"2021-11-24 10:04:07.000000000","message":"recheck","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"94e0467dea4987248fe9eb964db5382f63badc0f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8f7d7a1d_cd5b56d7","updated":"2021-11-24 10:33:37.000000000","message":"Setting -1 until we can verify that min pps tests were executed by Zuul","commit_id":"0cb7ba4df3ba95424fe26d1b475e65ccb3e53ebd"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"1f7675759ff0c4983a4092785b432f983b394c43","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"914dcc62_00e17314","updated":"2022-01-11 12:33:54.000000000","message":"I think it is safe to merge these (see: https://storage.gra.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_3f4/806689/12/check/DNM-neutron-tempest-plugin-api-openvswitch/3f4b61d/testr_results.html )\nThe 2 tests from neutron_tempest_plugin.api.admin.test_ports.PortTestCasesResourceRequest because those tests are adopted to the new port_resource_request format in https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817357 and that is higher in the relation chain.\nAs I see it is not simple rebasing that solves this (I mean the 2 failing res_request tests), and anyway we execute now API tests on OVN","commit_id":"8565e93ac3d2d8b3d57a51e22d90957b6e88b8b7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"33054ca60b2d1fdded99e9d78cda356e1ee6df76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"5360cd18_f4ef70cd","in_reply_to":"914dcc62_00e17314","updated":"2022-01-11 16:23:30.000000000","message":"the missed conclusion: and when all the patches from the chain merged all will be fine.","commit_id":"8565e93ac3d2d8b3d57a51e22d90957b6e88b8b7"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3c13485cdf173641555f4bf3d4328e46b2bd1b2d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":14,"id":"d6ad3d64_cb3a32a2","updated":"2022-01-12 14:57:45.000000000","message":"let\u0027s see it again with untouched n-lib in requirements.txt as xena failed wit dependency conflict","commit_id":"f2e8efaa509138614c763491fd62f9ac7de03264"},{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"43b7b08d127f91c06f3eb5d23f09d38f413638e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"a708fe77_a783b90f","updated":"2022-01-19 12:58:01.000000000","message":"Can\u0027t find new tests in report: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_9f2/806689/15/check/neutron-tempest-plugin-api/9f2da4d/testr_results.html - is it because OVN does not support it yet?","commit_id":"4a1357b86d9ea50459c0a73f1b59b7aedc39009c"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"3fcb774dc8edd14a0b17ad6189196392bb2f4f21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"bbfcd4ef_7240ccca","updated":"2022-01-18 15:07:12.000000000","message":"LGTM, good patch","commit_id":"4a1357b86d9ea50459c0a73f1b59b7aedc39009c"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f0e5f668d3f90f396c2157534eed2ff1de8dc1c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"86623bcd_df87533e","updated":"2022-01-14 13:44:04.000000000","message":"recheck\nneutron-tempest-plugin-dynamic-routing* jobs failed due to os-ken issue","commit_id":"4a1357b86d9ea50459c0a73f1b59b7aedc39009c"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"6990497d5ecbc6216d109c5679dd8ed4b02c136d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":15,"id":"50fa9522_a145992c","in_reply_to":"a708fe77_a783b90f","updated":"2022-01-19 13:23:08.000000000","message":"exactly, around ps14 I added a DNM job to test it (DNM-neutron-tempest-plugin-api-openvswitch ), see:\nhttps://dbeb82b77042de077d9b-4a9c39ee8b7a6253d43363c4ec56be4d.ssl.cf1.rackcdn.com/806689/14/check/DNM-neutron-tempest-plugin-api-openvswitch/a27b8a5/testr_results.html\n\nIt failed with the test_port_resource_request tests, but both QosMinimumPpsRuleTestJSON \u0026 QosMinimumPpsRuleNegativeTestJSON were executed.\n\nport-resource-requests tests failed because https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/817357 was not merged.","commit_id":"4a1357b86d9ea50459c0a73f1b59b7aedc39009c"}],"neutron_tempest_plugin/api/test_qos.py":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3532955cfb25b0b5d4ab3b2dd40164f00b4e3e7c","unresolved":true,"context_lines":[{"line_number":1337,"context_line":"    RULES_NAME \u003d RULE_NAME + \"s\""},{"line_number":1338,"context_line":"    required_extensions \u003d [qos_apidef.ALIAS]"},{"line_number":1339,"context_line":""},{"line_number":1340,"context_line":"    # Note: This can be enabled/uncommented when ovs for example wil support"},{"line_number":1341,"context_line":"    # minimum_packet_rate_rule, and GET qos/rule-types return with it."},{"line_number":1342,"context_line":"    # @classmethod"},{"line_number":1343,"context_line":"    # @base.require_qos_rule_type(\u0027minimum_packet_rate_rule\u0027)"},{"line_number":1344,"context_line":"    # def resource_setup(cls):"},{"line_number":1345,"context_line":"    #     super(QosMinimumPpsRuleTestJSON, cls).resource_setup()"},{"line_number":1346,"context_line":""},{"line_number":1347,"context_line":"    @decorators.idempotent_id(\u002766a5b9b4-d4f9-4af8-b238-9e1881b78487\u0027)"},{"line_number":1348,"context_line":"    def test_rule_create(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"3301883a_51adc76c","line":1345,"range":{"start_line":1340,"start_character":0,"end_line":1345,"end_character":64},"updated":"2021-08-31 11:16:15.000000000","message":"This patch has to be merged for list qos rule_types contain min pps:\nhttps://review.opendev.org/c/openstack/neutron/+/805391","commit_id":"6b5d06ad5b32fb068cc5665529a51b2271ab1ad8"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ad3969004b30f817578a2beedd3764a2ccbbdbaf","unresolved":true,"context_lines":[{"line_number":1378,"context_line":""},{"line_number":1379,"context_line":"class QosMinimumPpsRuleTestJSON(base.BaseAdminNetworkTest):"},{"line_number":1380,"context_line":"    # TODO(lajoskatona): this should be later in neutron-lib/constants"},{"line_number":1381,"context_line":"    RULE_NAME \u003d \u0027minimum_packet_rate_rule\u0027"},{"line_number":1382,"context_line":"    RULES_NAME \u003d RULE_NAME + \"s\""},{"line_number":1383,"context_line":"    required_extensions \u003d [qos_apidef.ALIAS]"},{"line_number":1384,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"fdad6282_19478f19","line":1381,"updated":"2021-11-23 12:47:04.000000000","message":"nit:\nhttps://opendev.org/openstack/neutron-lib/src/branch/master/neutron_lib/services/qos/constants.py#L19\nRULE_NAME \u003d qos_consts.RULE_TYPE_MINIMUM_PACKET_RATE + \"_rule\"","commit_id":"11d3a2ff0871449bd7c451bdf026c5d0258b3cb4"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"43bd7b2e7d468bd718d61c8b5af363194dc40e05","unresolved":false,"context_lines":[{"line_number":1378,"context_line":""},{"line_number":1379,"context_line":"class QosMinimumPpsRuleTestJSON(base.BaseAdminNetworkTest):"},{"line_number":1380,"context_line":"    # TODO(lajoskatona): this should be later in neutron-lib/constants"},{"line_number":1381,"context_line":"    RULE_NAME \u003d \u0027minimum_packet_rate_rule\u0027"},{"line_number":1382,"context_line":"    RULES_NAME \u003d RULE_NAME + \"s\""},{"line_number":1383,"context_line":"    required_extensions \u003d [qos_apidef.ALIAS]"},{"line_number":1384,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"81c5de1e_bb12d95b","line":1381,"in_reply_to":"fdad6282_19478f19","updated":"2021-11-23 13:25:27.000000000","message":"Done","commit_id":"11d3a2ff0871449bd7c451bdf026c5d0258b3cb4"}],"requirements.txt":[{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"3d9a5610e85cc4b2ac75422935026570ddbda67d","unresolved":true,"context_lines":[{"line_number":11,"context_line":"oslo.serialization!\u003d2.19.1,\u003e\u003d2.18.0 # Apache-2.0"},{"line_number":12,"context_line":"oslo.utils\u003e\u003d3.33.0 # Apache-2.0"},{"line_number":13,"context_line":"paramiko\u003e\u003d2.0.0 # LGPLv2.1+"},{"line_number":14,"context_line":"tempest\u003e\u003d17.1.0 # Apache-2.0"},{"line_number":15,"context_line":"tenacity\u003e\u003d3.2.1 # Apache-2.0"},{"line_number":16,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":17,"context_line":"nose\u003e\u003d1.3.7 # LGPL"}],"source_content_type":"text/plain","patch_set":5,"id":"821ce3dd_f94a398e","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":28},"updated":"2021-11-05 12:56:27.000000000","message":"tempest version needs to be bumped when https://review.opendev.org/c/openstack/tempest/+/806256 is released","commit_id":"1a9896f6147b518dc443096a5237e065bc12b7a8"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"b2e211b1d7d76ce7c6698c34c1dda5d49fb30dcf","unresolved":false,"context_lines":[{"line_number":11,"context_line":"oslo.serialization!\u003d2.19.1,\u003e\u003d2.18.0 # Apache-2.0"},{"line_number":12,"context_line":"oslo.utils\u003e\u003d3.33.0 # Apache-2.0"},{"line_number":13,"context_line":"paramiko\u003e\u003d2.0.0 # LGPLv2.1+"},{"line_number":14,"context_line":"tempest\u003e\u003d17.1.0 # Apache-2.0"},{"line_number":15,"context_line":"tenacity\u003e\u003d3.2.1 # Apache-2.0"},{"line_number":16,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":17,"context_line":"nose\u003e\u003d1.3.7 # LGPL"}],"source_content_type":"text/plain","patch_set":5,"id":"4559fa4d_1e5bbd93","line":14,"range":{"start_line":14,"start_character":0,"end_line":14,"end_character":28},"in_reply_to":"821ce3dd_f94a398e","updated":"2021-11-15 14:47:23.000000000","message":"Done","commit_id":"1a9896f6147b518dc443096a5237e065bc12b7a8"}],"zuul.d/master_jobs.yaml":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"3de455af2b531009b8ec8aeefdf885917e653927","unresolved":true,"context_lines":[{"line_number":54,"context_line":"        - qos"},{"line_number":55,"context_line":"        - qos-bw-minimum-ingress"},{"line_number":56,"context_line":"        - qos-fip"},{"line_number":57,"context_line":"        - quotas"},{"line_number":58,"context_line":"        - quota_details"},{"line_number":59,"context_line":"        - rbac-address-group"},{"line_number":60,"context_line":"        - rbac-address-scope"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"62802e61_9965f311","line":57,"updated":"2021-11-23 12:55:05.000000000","message":"Please add here port-resource-request-groups to make the tests be executed, not these are skipped:\n2021-11-15 15:41:15.421778 | controller | {0} setUpClass (neutron_tempest_plugin.api.test_qos_negative.QosMinimumPpsRuleNegativeTestJSON) ... SKIPPED: Skipped because network extension: port-resource-request-groups is not enabled\n....\n....\n2021-11-15 15:42:03.810850 | controller | {3} setUpClass (neutron_tempest_plugin.api.test_qos.QosMinimumPpsRuleTestJSON) ... SKIPPED: Skipped because network extension: port-resource-request-groups is not enabled","commit_id":"11d3a2ff0871449bd7c451bdf026c5d0258b3cb4"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"43bd7b2e7d468bd718d61c8b5af363194dc40e05","unresolved":false,"context_lines":[{"line_number":54,"context_line":"        - qos"},{"line_number":55,"context_line":"        - qos-bw-minimum-ingress"},{"line_number":56,"context_line":"        - qos-fip"},{"line_number":57,"context_line":"        - quotas"},{"line_number":58,"context_line":"        - quota_details"},{"line_number":59,"context_line":"        - rbac-address-group"},{"line_number":60,"context_line":"        - rbac-address-scope"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"d3c6e365_9296e6aa","line":57,"in_reply_to":"62802e61_9965f311","updated":"2021-11-23 13:25:27.000000000","message":"Done","commit_id":"11d3a2ff0871449bd7c451bdf026c5d0258b3cb4"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"8f41f2ad6e2c32a6bb6cac927f0ee4126ed41b2e","unresolved":true,"context_lines":[{"line_number":98,"context_line":"        # include at least OVN v20.12.0."},{"line_number":99,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":100,"context_line":"        OVN_BRANCH: \"v21.03.0\""},{"line_number":101,"context_line":"        OVS_BRANCH: \"8dc1733eaea866dce033b3c44853e1b09bf59fc7\""},{"line_number":102,"context_line":"      devstack_local_conf:"},{"line_number":103,"context_line":"        post-config:"},{"line_number":104,"context_line":"          # NOTE(slaweq): We can get rid of this hardcoded absolute path when"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"e26a315b_90e4226c","line":101,"updated":"2021-11-24 10:04:07.000000000","message":"perhaps add here:\nNETWORK_API_EXTENSIONS: \"{{ network_api_extensions_common | join(\u0027,\u0027) }}\" \nto have port-resource-request-groups in tempest.conf....","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"},{"author":{"_account_id":33652,"name":"Przemyslaw Szczerbik","email":"przemyslaw.szczerbik@gmail.com","username":"pszczerbik"},"change_message_id":"94e0467dea4987248fe9eb964db5382f63badc0f","unresolved":true,"context_lines":[{"line_number":98,"context_line":"        # include at least OVN v20.12.0."},{"line_number":99,"context_line":"        OVN_BUILD_FROM_SOURCE: True"},{"line_number":100,"context_line":"        OVN_BRANCH: \"v21.03.0\""},{"line_number":101,"context_line":"        OVS_BRANCH: \"8dc1733eaea866dce033b3c44853e1b09bf59fc7\""},{"line_number":102,"context_line":"      devstack_local_conf:"},{"line_number":103,"context_line":"        post-config:"},{"line_number":104,"context_line":"          # NOTE(slaweq): We can get rid of this hardcoded absolute path when"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"13da5d00_c084befe","line":101,"in_reply_to":"e26a315b_90e4226c","updated":"2021-11-24 10:33:37.000000000","message":"Let\u0027s give it a try","commit_id":"ef51c9803f2892967fd4ce6db2829bd7c9516265"}]}
