)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"ce5d59c26e22612b371cc27475aed702883711cb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c641c1e2_d56b6d22","updated":"2023-03-02 09:11:06.000000000","message":"Another question I have is why, during the router creation, the port created has no qos_policy_id? From the logs reported in the bug: https://e138a887655b8fda005f-ea1d911c7c7db668a9aa6765a743313b.ssl.cf5.rackcdn.com/874133/2/check/neutron-tempest-plugin-openvswitch-enforce-scope-new-defaults/7e5cbf9/controller/logs/screen-q-svc.txt","commit_id":"f33332aeec0e21c1b37f2f12205e8bbe0336e67d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"881576989854fd2a3cbeec8c3349c40ca8a8c383","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"65b33a11_b3a1fde6","updated":"2023-03-03 12:20:14.000000000","message":"recheck\nfullstack test_securitygroup failure is not related (xit code: -6; Cmd: [\u0027iptables-save\u0027, \u0027-t\u0027, \u0027raw\u0027]; Stdin: ; Stdout: ; Stderr: free(): double free detected in tcache 2 )","commit_id":"f33332aeec0e21c1b37f2f12205e8bbe0336e67d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"071f581f744d7500a0324a29aeb3617dcacd3e93","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"40fbf682_13396896","in_reply_to":"c641c1e2_d56b6d22","updated":"2023-03-02 13:09:17.000000000","message":"By traceback the notification is from here:\nhttps://opendev.org/openstack/neutron/src/branch/master/neutron/db/l3_db.py#L236-L240\nAnd there we add router_id to the payload, and that\u0027s why get_object fails to fetch Network.\nWhat I can\u0027t understand how this method which is regeistered for Network Precommit_Create receives this event.\nIf it can happen perhaps a check can also work for the resource type, and if it is not resources.NETWORK just return.","commit_id":"f33332aeec0e21c1b37f2f12205e8bbe0336e67d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"7e49df2d7a0e41d69a62cfca9cc1da572cf87638","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6ef3c5cc_9c277034","updated":"2023-03-17 15:34:04.000000000","message":"recheck\nFullstack failure is not related (and I can\u0027t reproduce it locally)","commit_id":"4d633c088000eaa8aa77532e1865c9acd7f5be37"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"ae918d99ff9d5c5374c04df775dacc7112d644b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"25291209_a1b09335","updated":"2023-03-23 09:39:46.000000000","message":"recheck\nneutron-tempest-plugin-openvswitch (test_remove_sec_grp_from_active_vm ) failure is not related\n(by opensearch the failure happened just once)","commit_id":"4d633c088000eaa8aa77532e1865c9acd7f5be37"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"6be889343980a5a8ded74f4d6c60696729afd5c1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9e3fb10f_ae47dce2","updated":"2023-03-22 14:24:00.000000000","message":"recheck\ntest_unshelve_to_specific_host failed in neutron-ovs-tempest-multinode-full","commit_id":"4d633c088000eaa8aa77532e1865c9acd7f5be37"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"cd2f71564c9b413503de1b22758cef470a308314","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d80c8a4c_d8614387","updated":"2023-03-23 05:26:25.000000000","message":"recheck flaky test_get_all_devices, plan is to move it to run serially","commit_id":"4d633c088000eaa8aa77532e1865c9acd7f5be37"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"31a2850ea1637835af8291595df62cde8e772b5e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2be701f3_0fc8b07f","updated":"2023-06-27 07:51:00.000000000","message":"recheck\nneutron-fullstack-with-uwsgi timeout (tests passed) and rally post_failure","commit_id":"ec4bfb91f0d3ae4aa4cbe8ba05c20cb00515b00d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"77907bd4abf67157b3b8905badc713905dcd6dcc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"7ac859e6_c92d5993","updated":"2023-06-26 07:36:19.000000000","message":"recheck\nneutron-ovn-rally-task failure is irrelevant, and tempest-integrated timeout is not related","commit_id":"ec4bfb91f0d3ae4aa4cbe8ba05c20cb00515b00d"}],"neutron/services/qos/qos_plugin.py":[{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a03f4302985b6ea43cd624b411e1554c10a5c93e","unresolved":true,"context_lines":[{"line_number":610,"context_line":"        network_id \u003d payload.resource_id"},{"line_number":611,"context_line":"        network \u003d network_object.Network.get_object(context, id\u003dnetwork_id)"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"        policy_id \u003d network.qos_policy_id"},{"line_number":614,"context_line":"        if policy_id is None:"},{"line_number":615,"context_line":"            return"},{"line_number":616,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"e35aaed3_64bf560d","side":"PARENT","line":613,"range":{"start_line":613,"start_character":8,"end_line":613,"end_character":41},"updated":"2023-03-02 09:08:41.000000000","message":"nit: I think this patch could be easier changing just this line\n\npolicy_id \u003d getattr(network, \u0027qos_policy_id\u0027, None)","commit_id":"92766c85b642bdece24c629625bf80050882c797"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"071f581f744d7500a0324a29aeb3617dcacd3e93","unresolved":true,"context_lines":[{"line_number":610,"context_line":"        network_id \u003d payload.resource_id"},{"line_number":611,"context_line":"        network \u003d network_object.Network.get_object(context, id\u003dnetwork_id)"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"        policy_id \u003d network.qos_policy_id"},{"line_number":614,"context_line":"        if policy_id is None:"},{"line_number":615,"context_line":"            return"},{"line_number":616,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"d383e959_c231ab78","side":"PARENT","line":613,"range":{"start_line":613,"start_character":8,"end_line":613,"end_character":41},"in_reply_to":"e35aaed3_64bf560d","updated":"2023-03-02 13:09:17.000000000","message":"that was my first idea also but here even the network is empty","commit_id":"92766c85b642bdece24c629625bf80050882c797"},{"author":{"_account_id":7730,"name":"Sahid Orentino Ferdjaoui","email":"sahid.ferdjaoui@industrialdiscipline.com","username":"sahid"},"change_message_id":"a8c911399161b3c78c2b563170d9b8a0a11a0d0f","unresolved":true,"context_lines":[{"line_number":611,"context_line":"        network \u003d network_object.Network.get_object(context, id\u003dnetwork_id)"},{"line_number":612,"context_line":""},{"line_number":613,"context_line":"        if network:"},{"line_number":614,"context_line":"            policy_id \u003d getattr(network, \u0027qos_policy_id\u0027, None)"},{"line_number":615,"context_line":"            if policy_id is None:"},{"line_number":616,"context_line":"                return"},{"line_number":617,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":1,"id":"1bca6abd_04d2333d","line":614,"updated":"2023-03-14 09:08:33.000000000","message":"Yeah I think as Rodolfo the algorithm looks \"difficult\".\n\n  if not network or \"qos_policy_id\" not in network:\n    LOG.info(...)\n    return\n  policy_id \u003d network.qos_policy_id","commit_id":"f33332aeec0e21c1b37f2f12205e8bbe0336e67d"}]}