)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"651018da128104de1ccd3e2ce0b06d2a33ddf8e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b809db4a_6b4d84be","updated":"2026-09-30 09:47:59.000000000","message":"Replacing a dependency by the tiny code change makes clear sense and helps us avoid ignoring the type error.","commit_id":"7abc81892d269367dc5b2c37bb4dbc3f2a294a3f"}],"vif_plug_ovs/tests/functional/ovsdb/test_ovsdb_lib.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"c89f539def6266135f76e90e2eff9e12bf836ca9","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class _TestOVSDBLib(base.VifPlugOvsBaseFunctionalTestCase):"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    interface: str"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def setUp(self):"},{"line_number":41,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"49c2e459_90065e9f","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":18},"updated":"2026-09-30 16:19:52.000000000","message":"i think if you change thei to assign None or \u0027\u0027 it might work\n\n```suggestion\n    interface: str \u003d \u0027\u0027\n```\n\nthis is not a data clasee so while  `interface: str`\nis echogh for mypi to be happy its  not enouch to actully declare a class data member based on the ci failure","commit_id":"7abc81892d269367dc5b2c37bb4dbc3f2a294a3f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"281ef6655d2072e678799cd59fb337f03f254be0","unresolved":true,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class _TestOVSDBLib(base.VifPlugOvsBaseFunctionalTestCase):"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    interface: str"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def setUp(self):"},{"line_number":41,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"695b77a1_86f609a0","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":18},"in_reply_to":"49c2e459_90065e9f","updated":"2026-09-30 16:53:00.000000000","message":"The issue here is that the test is running when it shouldn\u0027t be. Defining attributes like this is fine so long as you don\u0027t attempt to access them at runtime (which we shouldn\u0027t be doing since this shouldn\u0027t be running)","commit_id":"7abc81892d269367dc5b2c37bb4dbc3f2a294a3f"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"27d2597c5e82da32678fe7ac91729be824e3985e","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class _TestOVSDBLib(base.VifPlugOvsBaseFunctionalTestCase):"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    interface: str"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def setUp(self):"},{"line_number":41,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"350de298_a793445d","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":18},"in_reply_to":"53272743_cfa18093","updated":"2026-09-30 17:12:21.000000000","message":"ack +1 for now ill upgrade to +2 once ci reports","commit_id":"7abc81892d269367dc5b2c37bb4dbc3f2a294a3f"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"6373c829180d389f6a85980b2a9f16fac09bfbcd","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"class _TestOVSDBLib(base.VifPlugOvsBaseFunctionalTestCase):"},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"    interface: str"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"    def setUp(self):"},{"line_number":41,"context_line":"        super().setUp()"}],"source_content_type":"text/x-python","patch_set":1,"id":"53272743_cfa18093","line":38,"range":{"start_line":38,"start_character":0,"end_line":38,"end_character":18},"in_reply_to":"695b77a1_86f609a0","updated":"2026-09-30 17:02:23.000000000","message":"Turns out `unittest` (and by extension `stestr`) doesn\u0027t care what the class is called: it only cares that it\u0027s a subclass of `TestCase`. I\u0027ve reworked things to reflect this.","commit_id":"7abc81892d269367dc5b2c37bb4dbc3f2a294a3f"}]}