)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9cb40e61f996d4234a86dcf6f83d41bb71a5f397","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"92177491_696778fe","updated":"2023-09-06 17:36:49.000000000","message":"Do we know the kernel change that triggered this? Would be good to have it here or in the bug if we do.","commit_id":"8cba9a2ee86cb3b65645674ef315c14cfb261143"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a4b22b59e1c47dc4c5c75f2a00d7470aaa0e4cb2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5412cbad_a80007df","in_reply_to":"92177491_696778fe","updated":"2023-09-06 18:54:55.000000000","message":"No, I didn\u0027t find it. In any case, that was used only in this test case (incorrectly). The ingress qdisc cannot replace the default egress qdisc. Now the kernel prevents that, not as before.","commit_id":"8cba9a2ee86cb3b65645674ef315c14cfb261143"}],"neutron/tests/functional/privileged/agent/linux/test_tc_lib.py":[{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"9cb40e61f996d4234a86dcf6f83d41bb71a5f397","unresolved":true,"context_lines":[{"line_number":45,"context_line":"            namespace\u003dself.namespace)"},{"line_number":46,"context_line":"        qdiscs \u003d priv_tc_lib.list_tc_qdiscs(self.device,"},{"line_number":47,"context_line":"                                            namespace\u003dself.namespace)"},{"line_number":48,"context_line":"        self.assertEqual(1, len(qdiscs))"},{"line_number":49,"context_line":"        self.assertEqual(rtnl.TC_H_ROOT, qdiscs[0][\u0027parent\u0027])"},{"line_number":50,"context_line":"        self.assertEqual(0x50000, qdiscs[0][\u0027handle\u0027])"},{"line_number":51,"context_line":"        self.assertEqual(\u0027htb\u0027, linux_utils.get_attr(qdiscs[0], \u0027TCA_KIND\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"1ec1f022_f30853bd","line":48,"updated":"2023-09-06 17:36:49.000000000","message":"I\u0027m stumped why this didn\u0027t have to change link L113 below.","commit_id":"8cba9a2ee86cb3b65645674ef315c14cfb261143"},{"author":{"_account_id":16688,"name":"Rodolfo Alonso","email":"ralonsoh@redhat.com","username":"rodolfo-alonso-hernandez"},"change_message_id":"a4b22b59e1c47dc4c5c75f2a00d7470aaa0e4cb2","unresolved":false,"context_lines":[{"line_number":45,"context_line":"            namespace\u003dself.namespace)"},{"line_number":46,"context_line":"        qdiscs \u003d priv_tc_lib.list_tc_qdiscs(self.device,"},{"line_number":47,"context_line":"                                            namespace\u003dself.namespace)"},{"line_number":48,"context_line":"        self.assertEqual(1, len(qdiscs))"},{"line_number":49,"context_line":"        self.assertEqual(rtnl.TC_H_ROOT, qdiscs[0][\u0027parent\u0027])"},{"line_number":50,"context_line":"        self.assertEqual(0x50000, qdiscs[0][\u0027handle\u0027])"},{"line_number":51,"context_line":"        self.assertEqual(\u0027htb\u0027, linux_utils.get_attr(qdiscs[0], \u0027TCA_KIND\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"73e71011_e55f4054","line":48,"in_reply_to":"1ec1f022_f30853bd","updated":"2023-09-06 18:54:55.000000000","message":"Because this is replacing the egress root default qdisc:\n  qdisc noqueue 0: dev dummy root refcnt 2\n  \nThis replacement does not happen with the ingress qdisc. The ingress qdisc is added along with the egrees default qdisc.","commit_id":"8cba9a2ee86cb3b65645674ef315c14cfb261143"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"5a0ec12eff745a6871faa96d619100d2a0086be2","unresolved":false,"context_lines":[{"line_number":45,"context_line":"            namespace\u003dself.namespace)"},{"line_number":46,"context_line":"        qdiscs \u003d priv_tc_lib.list_tc_qdiscs(self.device,"},{"line_number":47,"context_line":"                                            namespace\u003dself.namespace)"},{"line_number":48,"context_line":"        self.assertEqual(1, len(qdiscs))"},{"line_number":49,"context_line":"        self.assertEqual(rtnl.TC_H_ROOT, qdiscs[0][\u0027parent\u0027])"},{"line_number":50,"context_line":"        self.assertEqual(0x50000, qdiscs[0][\u0027handle\u0027])"},{"line_number":51,"context_line":"        self.assertEqual(\u0027htb\u0027, linux_utils.get_attr(qdiscs[0], \u0027TCA_KIND\u0027))"}],"source_content_type":"text/x-python","patch_set":1,"id":"ee25e3b1_ea0da0e2","line":48,"in_reply_to":"73e71011_e55f4054","updated":"2023-09-06 19:01:38.000000000","message":"Ah, that makes sense now, and since it is UP, when you remove the replacement \u0027noqueue\u0027 is put back.","commit_id":"8cba9a2ee86cb3b65645674ef315c14cfb261143"}]}