)]}'
{"/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":"71bcfde745a522ec714d1c50cfeff4820ccc4635","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"25f012aa_dcdc85d6","updated":"2026-01-14 08:27:19.000000000","message":"recheck\nthe errors seems to be not related","commit_id":"1f36dadf71f7a2adc7bf953df1534bf7ab2ed573"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"81be42d47c6eecc842129fefab33e0beb5ded9c3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"6e70ff79_879f67ae","updated":"2026-01-27 18:12:46.000000000","message":"recheck\ntempest failures with server-delete are not related","commit_id":"4566fbf5cd01ff76bccdc8896808bccf38e79a06"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"f4e9883410f82ed19012ef57a9d15a271854156c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"c6c01249_3abfc045","updated":"2026-02-17 15:43:17.000000000","message":"recheck\nfailure seems to be not related (based on opensearch result the same failure appears on many patches not just on mine)","commit_id":"30bc68497b81911a1faefef5634a453dcc484dfb"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"cc4f59ab6867ac05ab05f9d39da614fc1bd4ade2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"4ad22c91_396b3b8c","updated":"2026-02-18 11:37:38.000000000","message":"recheck\nhttps://review.opendev.org/c/openstack/nova/+/977150 fixed the ceph job","commit_id":"30bc68497b81911a1faefef5634a453dcc484dfb"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"289ad4a44898910da155cf38e7137d89a2e987d8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"7137a90f_c4190c3d","updated":"2026-02-13 08:32:35.000000000","message":"recheck\nnova-ceph-multistore and test_boot_cloned_encrypted_volume failure is not related (based on logs)","commit_id":"30bc68497b81911a1faefef5634a453dcc484dfb"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"58f2cd4a84adb9ab267975f8869009707ed4b02c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"77d0bb13_b205f2c8","updated":"2026-02-16 09:09:45.000000000","message":"recheck\npost_failure is not related","commit_id":"30bc68497b81911a1faefef5634a453dcc484dfb"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"72734d73858efbb1c74125bc95ae030eab2e0b5b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"79b4b595_0893e31f","updated":"2026-03-18 08:42:37.000000000","message":"recheck\ncover job failure is not related","commit_id":"b33768a9411a7a290b2000f2730e1ba9f571490d"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"2211434b91ccfb348d65b4696d309bcf57144124","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":12,"id":"96000d6d_1113cbbe","updated":"2026-03-11 08:33:57.000000000","message":"recheck\ntimeout","commit_id":"b33768a9411a7a290b2000f2730e1ba9f571490d"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7237396bdf6e92852f618b3ceff2a1f3163c62aa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"76cfad9a_2ebd1097","updated":"2026-06-15 20:45:41.000000000","message":"i think before we do any of these sdk conversion we need to move all neutron client usage and indreict useage into nova/netwoks/neutron.py first\n\nthere are not a lot of cases that arnt already there.\n\n```\n - nova/cmd/manage.py\n     - import: nova/cmd/manage.py:71\n     - call: nova/cmd/manage.py:2320\n     - neutron_api.get_client(ctxt, admin\u003dTrue)\n - nova/network/security_group_api.py\n     - import: nova/network/security_group_api.py:35\n     - calls:\n         - :65\n         - :220\n         - :241\n         - :294\n         - :309\n         - :344\n         - :366\n         - :429\n         - :444\n         - :526\n         - :588\n         - :650\n - nova/api/metadata/handler.py\n     - import: nova/api/metadata/handler.py:35\n     - call: nova/api/metadata/handler.py:193\n     - neutronapi.get_client(context, admin\u003dTrue)\n```","commit_id":"99cdbe22e7ef7a3d9f2ee2eff7f3bb9eeb21dad3"},{"author":{"_account_id":1131,"name":"Brian Haley","email":"haleyb.dev@gmail.com","username":"brian-haley"},"change_message_id":"38debd904f7a6625e5759524c4582bc3528f1366","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":17,"id":"7fb9417d_31de03cb","updated":"2026-06-15 16:42:48.000000000","message":"recheck POST_FAILURE","commit_id":"99cdbe22e7ef7a3d9f2ee2eff7f3bb9eeb21dad3"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"722474b8a32b90d33d62c2a6495f00fd280994df","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":17,"id":"0338990f_acefdafa","in_reply_to":"76cfad9a_2ebd1097","updated":"2026-06-15 20:55:47.000000000","message":"nova/network/security_group_api.py is debatabel.\n\nmodules outside fo nova/network shoudl not use the neturon client and isntead uses our internal public networkign API mehtod \n\nhttps://github.com/openstack/nova/blob/3ac36dbfd1f3ac00c17ce8dbb4ae97ab120e153e/nova/network/neutron.py#L280\n\n\nits oke for code to be group into there own file \n\nbut either the security_group_api modeule shoudl also only use fucntionf from teh nutron moduels public api\n\nor we can allwo it to use the raw client but it method shoudl be private  exposed via the public API class for that api to use.\n\nit was intoduced in \n\nhttps://github.com/openstack/nova/commit/110a683486b750488ee147b16a17451750af4c5e\n\nas part of nova netwrok removal but tis not really the right abstraction.\n\nso im torn but before we do a masive rewrite of how nova talks to neturon if feels liek we shoudl take the time to get our house in order first. in a couple of clean up patches that move the neturon interaction back into the file it shoudl be in then we shoudl be abel to juse refacotr that tile and its tests.","commit_id":"99cdbe22e7ef7a3d9f2ee2eff7f3bb9eeb21dad3"}],"nova/api/metadata/handler.py":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7237396bdf6e92852f618b3ceff2a1f3163c62aa","unresolved":true,"context_lines":[{"line_number":206,"context_line":"        #  read the instance_id, tenant_id from that port entry."},{"line_number":207,"context_line":""},{"line_number":208,"context_line":"        # Retrieve networks which are connected to metadata provider"},{"line_number":209,"context_line":"        md_subnets \u003d neutron.list_subnets("},{"line_number":210,"context_line":"            context,"},{"line_number":211,"context_line":"            advanced_service_providers\u003d[provider_id],"},{"line_number":212,"context_line":"            fields\u003d[\u0027network_id\u0027])"},{"line_number":213,"context_line":""},{"line_number":214,"context_line":"        if not md_subnets or not md_subnets.get(\u0027subnets\u0027):"},{"line_number":215,"context_line":"            msg \u003d _(\u0027Could not find any subnets for provider %s\u0027) % provider_id"}],"source_content_type":"text/x-python","patch_set":17,"id":"4c7b971f_fc480412","side":"PARENT","line":212,"range":{"start_line":209,"start_character":7,"end_line":212,"end_character":34},"updated":"2026-06-15 20:45:41.000000000","message":"this is an example of a place we were failign to propelroy isolatie the \nother nova moduel form the neturon client\n\nit shoudl be reciveign a set of nova.network.model.Subnet obejcts\n\nhttps://github.com/openstack/nova/blob/master/nova/network/model.py#L307-L356\n\nnote the raw clietn respocen","commit_id":"b1ea7543f6bfff151f564d6a361be86940d4e803"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"7237396bdf6e92852f618b3ceff2a1f3163c62aa","unresolved":true,"context_lines":[{"line_number":191,"context_line":"        # inter-Edge network port"},{"line_number":192,"context_line":"        context \u003d nova_context.get_admin_context()"},{"line_number":193,"context_line":"        neutron \u003d neutronapi.get_client(context, admin\u003dTrue)"},{"line_number":194,"context_line":"        sdk_neutron \u003d neutronapi.get_network_client(context)"},{"line_number":195,"context_line":""},{"line_number":196,"context_line":"        # Tenant, instance ids are found in the following method:"},{"line_number":197,"context_line":"        #  X-Metadata-Provider contains id of the metadata provider, and since"}],"source_content_type":"text/x-python","patch_set":17,"id":"39ab3d0d_cd30285a","line":194,"updated":"2026-06-15 20:45:41.000000000","message":"so instead of doublign down on the existing bug we proably shoudl refactor this to us the nova.network.modle moduel and have this use subnet model objects\n\nalthogu if you actully look at the code\n\nthe only thing we use the subnet object for\nis to constuct md_networks\n\nand what this fucntion actully returns is  instance_id, tenant_id\n\nso i think this entire fucntion should be in nova.networks.neutron\n\nnot here.\n\nif its in that module its allowed to use the sdk directly but we shoudl not be using the sdk or the neutron client here.","commit_id":"99cdbe22e7ef7a3d9f2ee2eff7f3bb9eeb21dad3"}]}