)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":27442,"name":"Quentin GROLLEAU","email":"quentin.grolleau@corp.ovh.com","username":"QG"},"change_message_id":"4da7521028ef20c397022e11aec051622d8bb112","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ab0f22ee_bd42c3c5","updated":"2023-10-19 09:42:28.000000000","message":"So all the functional test that are failing because in noop driver there is no link between the subnet and the network \n\nHere an example :\n```json\nsubnet --\u003e {\u0027id\u0027: \u00278e85740e-4166-477c-9905-abde39625231\u0027, \u0027name\u0027: None, \u0027network_id\u0027: None, \u0027project_id\u0027: None, \u0027gateway_ip\u0027: None, \u0027cidr\u0027: None, \u0027ip_version\u0027: None, \u0027host_routes\u0027: None}\n```\n\n```json\nnetwork --\u003e {\u0027id\u0027: None, \u0027name\u0027: None, \u0027subnets\u0027: [], \u0027project_id\u0027: None, \u0027admin_state_up\u0027: None, \u0027provider_network_type\u0027: None, \u0027provider_physical_network\u0027: None, \u0027provider_segmentation_id\u0027: None, \u0027router_external\u0027: None, \u0027mtu\u0027: None, \u0027port_security_enabled\u0027: True}\n```\n\nand when entering in the \u0027get_network_ip_availability\u0027 function which call the noop_driver it iterate and check if subnet is in the network.subnets : \nhttps://opendev.org/openstack/octavia/src/branch/master/octavia/network/drivers/noop_driver/driver.py#L379\n\nwhich then call :\nhttps://opendev.org/openstack/octavia/src/branch/master/octavia/network/drivers/noop_driver/driver.py#L206\n\nWhich instantiate a new subnet \n\nSo at the end subnet_id didn\u0027t match \n\n\ni try to patch the _validate_subnet to add some code like :\n```python\n        network \u003d validate.network_exists_optionally_contains_subnet(\n            network_id\u003dload_balancer.vip_network_id,\n            subnet_id\u003dload_balancer.vip_subnet_id,\n            context\u003dcontext)\n```\nfor getting the network and it fix most of the tests but i was thinking more about a fix inside the noop_driver","commit_id":"dbb8819d1af8e58e66e048005ef810482d32660e"}]}
