)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"95920e1bc62a23ee86f8370a95b07faa30b02acd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fa4d4d8d_25ef64a2","updated":"2021-11-03 12:44:37.000000000","message":"One other thing: would it make sense to split this commit into two and add \"remove port/network\" functionality also, i.e. \"Switch add/remove port\" and \"Switch add/remove network\"? If not, what\u0027s the chances of switching the \"remove\" operations over in a follow-up?","commit_id":"ece0c43a4d0799a5491fe581db85402307e59936"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"bfc67ce3f32e8a0a854f0f4611aeeedfc8def48c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8199ae94_3e5fcebb","updated":"2021-11-03 10:51:14.000000000","message":"Works as expected 👌 Just missing a release note and this is good to go, IMO. The -1 is for that","commit_id":"ece0c43a4d0799a5491fe581db85402307e59936"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"dea8f9ef93b1f47db8b9f3e1b7c84cde7088e3f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"24ab2ff4_fdbb15fb","updated":"2021-11-03 12:05:27.000000000","message":"fwiw, here\u0027s how I tested this on a bog-standard devstack deployment:\n\n  # create a server\n  openstack server create --network private --flavor m1.medium --image cirros-0.5.2-x86_64-disk --wait test-server\n\n  # add a network to the server\n  openstack server add network test-server public\n\n  # ensure the network appears\n  openstack server show test-server\n\n  # create a port\n  openstack port create --network private test-port\n\n  # add the port to the server\n  openstack server add port test-server test-port\n\n  # ensure the port appears\n  openstack server show test-server\n\n  # delete the server and port\n  openstack server delete test-server\n  openstack port delete test-port\n\nIt would be relatively easy to package this up as a functional test, if you wanted. That could be a nice follow-up","commit_id":"ece0c43a4d0799a5491fe581db85402307e59936"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"142db956c3d83999a6ae8a56254961d1ea3a4ce1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a2f85029_4a3621b6","updated":"2021-11-15 12:45:15.000000000","message":"Beautiful \u003c3","commit_id":"2183a611475090347863917f6c90f0f38cd80893"}]}
