)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"5e33ba49252f080310beb9e5444bf4eecfc18b74","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b74a0f10_d0970b17","updated":"2022-11-17 16:00:58.000000000","message":"Would be nice to see this for OVN deployments.","commit_id":"b6fcf5bfbeaab2005f125b59a0794cd037c6b84e"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"f98964a3b5951b1706dab2c13896a8014a780bec","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cfdf134f_bc9e068c","updated":"2024-01-31 12:06:35.000000000","message":"it\u0027s still W-1 because there\u0027s no way to disable all the features/parameters that are not allowed by the other providers\n\n1. octavia-dashboard was initially created to follow closely the specs of the amphora driver, and for instance, at most 90% of the params/options are not supported by the ovn-provider\n2. right now, if we create a ovn-provider LB + a listener with http protocol, the octavia-dashboard displays \"cannot create load balancer\" and without any explanation, we don\u0027t even know why it is denied\n\nwe probably need to fix https://bugs.launchpad.net/octavia/+bug/2013722 before approving this patch","commit_id":"ade3dd13b86adecd3b0d8d19c77734e1823fd7cf"},{"author":{"_account_id":29268,"name":"Matthew Heler","email":"matthew.heler@hotmail.com","username":"mheler"},"change_message_id":"9d9f53b0e9eb116cc8340215e12eada9d367a92e","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"98b79bdc_8fb4c386","updated":"2023-02-07 09:47:14.000000000","message":"recheck","commit_id":"ade3dd13b86adecd3b0d8d19c77734e1823fd7cf"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"2b8f7bae65b1a9a7a493c876a4a555bbf3c4ff4c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"66440164_b8d54e7e","updated":"2026-04-15 15:05:08.000000000","message":"One issue with a test, although I\u0027ll admit Claude spotted that one as I\u0027m not a Javascript expert 🙂. Otherwise the changes worked so one this is fixed and rebased it should be good to go\n\nReview assisted by Claude Code","commit_id":"879ed574ed16e3acc22c7e23f5fb479253a8650b"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"1babc103fcafc9708759c9052957a9c686c8df02","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"86e3291e_58cd47fa","updated":"2026-04-17 12:22:49.000000000","message":"LGTM now, thanks!","commit_id":"7fb10d05beabc1c963a47656f766aab2eeceaa99"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"43fe76f6da62edd476dce2c50c9c384a09f50d41","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"86d95554_40462d0f","updated":"2026-05-12 08:05:33.000000000","message":"Tested, Looks good","commit_id":"92a39daa96ac91acd52e4f9a01e244f8bbcf480d"}],"octavia_dashboard/static/dashboard/project/lbaasv2/workflow/loadbalancer/loadbalancer.controller.spec.js":[{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"2b8f7bae65b1a9a7a493c876a4a555bbf3c4ff4c","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              is_enabled: true"},{"line_number":114,"context_line":"            }"},{"line_number":115,"context_line":"          },"},{"line_number":116,"context_line":"          providers: {"},{"line_number":117,"context_line":"            name: \u0027amphora\u0027,"},{"line_number":118,"context_line":"            description: \u0027amphora description\u0027"},{"line_number":119,"context_line":"          },"}],"source_content_type":"text/javascript","patch_set":4,"id":"a851d544_8c014136","line":116,"updated":"2026-04-15 15:05:08.000000000","message":"This mock needs to be restructured to match the data model. The provider name should be a key, not \u0027name\u0027 as it currently is\n\n```suggestion\n  providers: {\n    amphora: { name: \u0027amphora\u0027, description: \u0027amphora description\u0027 },\n    octavia: { name: \u0027octavia\u0027, description: \u0027octavia description\u0027 }\n    }\n```","commit_id":"879ed574ed16e3acc22c7e23f5fb479253a8650b"},{"author":{"_account_id":38562,"name":"Richard Cruise","email":"rcruise@redhat.com","username":"rcruise"},"change_message_id":"da7c8731625162e80cd2679cd316d3d2c993b47d","unresolved":false,"context_lines":[{"line_number":113,"context_line":"              is_enabled: true"},{"line_number":114,"context_line":"            }"},{"line_number":115,"context_line":"          },"},{"line_number":116,"context_line":"          providers: {"},{"line_number":117,"context_line":"            name: \u0027amphora\u0027,"},{"line_number":118,"context_line":"            description: \u0027amphora description\u0027"},{"line_number":119,"context_line":"          },"}],"source_content_type":"text/javascript","patch_set":4,"id":"78f2993e_e14891d3","line":116,"in_reply_to":"9ce904c1_7573e726","updated":"2026-04-17 12:23:06.000000000","message":"Done","commit_id":"879ed574ed16e3acc22c7e23f5fb479253a8650b"},{"author":{"_account_id":24882,"name":"Sami","email":"sami.yessou@switch.ch","username":"sami","status":"Cloud engineer"},"change_message_id":"b90dbd2423e3a4690d3081364ad6022e6f9a54dd","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              is_enabled: true"},{"line_number":114,"context_line":"            }"},{"line_number":115,"context_line":"          },"},{"line_number":116,"context_line":"          providers: {"},{"line_number":117,"context_line":"            name: \u0027amphora\u0027,"},{"line_number":118,"context_line":"            description: \u0027amphora description\u0027"},{"line_number":119,"context_line":"          },"}],"source_content_type":"text/javascript","patch_set":4,"id":"ff64d703_1209c0cb","line":116,"in_reply_to":"a851d544_8c014136","updated":"2026-04-17 09:52:53.000000000","message":"That suggestion should be correct IMHO let\u0027s wait for Gregory or we could apply it and mark it as solved?","commit_id":"879ed574ed16e3acc22c7e23f5fb479253a8650b"},{"author":{"_account_id":29244,"name":"Gregory Thiemonge","email":"gthiemon@redhat.com","username":"gthiemonge"},"change_message_id":"deeb15342de4af92fe1e6553475239919e279c47","unresolved":true,"context_lines":[{"line_number":113,"context_line":"              is_enabled: true"},{"line_number":114,"context_line":"            }"},{"line_number":115,"context_line":"          },"},{"line_number":116,"context_line":"          providers: {"},{"line_number":117,"context_line":"            name: \u0027amphora\u0027,"},{"line_number":118,"context_line":"            description: \u0027amphora description\u0027"},{"line_number":119,"context_line":"          },"}],"source_content_type":"text/javascript","patch_set":4,"id":"9ce904c1_7573e726","line":116,"in_reply_to":"ff64d703_1209c0cb","updated":"2026-04-17 11:46:26.000000000","message":"Updated, thanks","commit_id":"879ed574ed16e3acc22c7e23f5fb479253a8650b"}]}