)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"54537391dfd24461385ca5ec9d91af67ee1655b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0c34e7e4_0b2f50f0","updated":"2024-07-19 12:21:46.000000000","message":"Also - if we do not have 100% coverage for all services with distro install we still need this to be working?","commit_id":"bab5aaf0535898c3537c3a0ecc4405c5054ac2c8"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"0188de039cec2ddb0c1bf662cffa1950025e3897","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"574eefc7_865d3106","updated":"2024-07-19 11:44:52.000000000","message":"Would it make sense to have `openstack_repo_url` defined for distro instead? As it looks like repo container is still being prepared (for some reason?) for distro setup.\n\nOr, better, run this check only for source install method and just skip as a whole for distro?","commit_id":"bab5aaf0535898c3537c3a0ecc4405c5054ac2c8"},{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"285a3044dfeac0f6ef1970efd3d953e8f420fc9d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9d498670_3b8e4f22","in_reply_to":"0c34e7e4_0b2f50f0","updated":"2024-07-20 05:51:33.000000000","message":"I\u0027d say we should fail explicitly in services where we don\u0027t have distro support, as there might be way more places where distro will fasil if attempt to build wheels at some point.","commit_id":"bab5aaf0535898c3537c3a0ecc4405c5054ac2c8"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"ba1f76bb29ffeb233c4f50c8693a21e264b91ff2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"968edd09_e373732a","in_reply_to":"574eefc7_865d3106","updated":"2024-07-19 12:16:04.000000000","message":"Yes this is all a good question - tbh the patch at the moment just brings the behaviour back to what it was before we moved install_defaults out to the plugins collection. That might still be invalid and the correct thing to do is skip this.\n\nI\u0027ve also seen a fair few failure of this check in CI, were we should instead grab some well known url off the CI provider mirror rather than reach out to opendev.org.","commit_id":"bab5aaf0535898c3537c3a0ecc4405c5054ac2c8"},{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"b13be0301a31caa69c262ba1f4b1746e9ece796f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5712d683_4ad80924","updated":"2024-07-19 18:18:57.000000000","message":"recheck - MODULE_FAILURE","commit_id":"38be9eb43a0d89aacba4172761bbf5adad414fc4"}]}
