)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"b8595d653996aef5d6393fa1025cf085f978d702","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"532d4501_4d3728df","updated":"2026-04-11 08:35:34.000000000","message":"Two small things I noticed:\n\ntest_get_metadata passes the subnet as a positional argument but the other three tests use subresource\u003dshare_subnet. Should they all match?\n\ntest_delete_metadata and test_update_all_metadata use @ddt.data to test different subnet input types but test_get_metadata and test_set_metadata don\u0027t. Is it worth adding for consistency?","commit_id":"ceb12322cc747b945b8537b38952c839f68f8f6a"},{"author":{"_account_id":38967,"name":"Fiyin Peter","display_name":"Fiyin Peter","email":"insitupeter@gmail.com","username":"kitana"},"change_message_id":"8473b1e5ec0be003c9798887c6ee75f4c628969b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3341e878_63b6e6b9","in_reply_to":"532d4501_4d3728df","updated":"2026-04-11 10:41:19.000000000","message":"Thanks for the review! Fixed both — updated get_metadata signature to match the others and added @ddt.data variants to test_get_metadata and test_set_metadata for consistency.","commit_id":"ceb12322cc747b945b8537b38952c839f68f8f6a"},{"author":{"_account_id":38842,"name":"Silvia Wachira","display_name":"Silvia Wachira","email":"wachirasilvia8@gmail.com","username":"Silvia-Wachira"},"change_message_id":"3f67d6408de6b093134d0eae26491a9d903c8bc7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e01c3ba3_89edb4cf","updated":"2026-04-11 16:06:42.000000000","message":"Thanks for considering my feedback and making the updates","commit_id":"c0c4ba556f1258c5e224a07d9486a9f9789c7946"}]}