)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"b54701e4386e2988cb033070a8f3dca82a217f4d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8984df6c_0472fd46","updated":"2023-01-23 08:22:51.000000000","message":"I\u0027m not worried about the probe test failure, but for the others, looks like:\n\n- cryptography gets upgraded at some point, which\n- doesn\u0027t get installed from wheels so it tries to compile, only\n- the toolchain got more complicated over the years, so now it complains\n\n This package requires Rust \u003e\u003d1.56.0.\n\nI feel like I\u0027m outta my depth trying to fix that -- would it be so bad to make those non-voting? Or maybe even just drop the failing jobs for these years-old branches?","commit_id":"00c877fb0691baacc216a14be9beb7553257d9d3"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"4cb3e32576d6618cf468f849039d21668c51c49b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0789a32b_a480be0c","updated":"2023-01-21 03:18:39.000000000","message":"recheck","commit_id":"00c877fb0691baacc216a14be9beb7553257d9d3"},{"author":{"_account_id":6968,"name":"Christian Schwede","email":"cschwede@nvidia.com","username":"cschwede"},"change_message_id":"c47e682bb7ab1362ad0bb4335209877ca71531dd","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"40a8139c_879e55c6","updated":"2023-01-24 08:47:55.000000000","message":"Given the age of the branch, I\u0027m fine with removing these tests to unblock the gate.\n\nThanks Tim and Matt for working on this!","commit_id":"91144596c670741c2f452c864c026eff5583989c"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"2c8d475f6ebc456fa733b96f57f6a1bda0d34399","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"3bbfb62d_8fa3bed8","updated":"2023-01-24 05:27:56.000000000","message":"Hmm if there are issues, I\u0027m probably putting them in the wrong place, when you take a closer look at the logs its when it\u0027s setting tempest up. So maybe the --only-binary would need to be in there.\n\nThis is getting more and more down the path of, maybe we just remove intergrated testing for very old stables.","commit_id":"91144596c670741c2f452c864c026eff5583989c"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"ccf5cc9a23387b7b292c62087b4bf33f2f73caae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"dc55c011_daf132e0","updated":"2023-01-24 00:32:28.000000000","message":"I\u0027ll push up a DNM stable just to test","commit_id":"91144596c670741c2f452c864c026eff5583989c"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"772f31264f955c8c047132a0e98379ef25a51c61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"19eea148_fcc77579","updated":"2023-01-24 23:49:47.000000000","message":"In line with what we did for ussuri in https://review.opendev.org/c/openstack/swift/+/871658","commit_id":"91144596c670741c2f452c864c026eff5583989c"},{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"30cde6b143f6f474dc9215f2afde835cff86e19a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ae43d6ec_bd9a27a4","updated":"2023-01-24 00:30:00.000000000","message":"aparently bcrypt as of version 4.0 is written in rust. But we shoudn\u0027t need the compiler if we just install it via wheels.\n\nI wonder if we could add:\n\n  bcrypt --only-binary\n\nTo our requirements.txt before crypogaphy, and wonder if that\u0027ll be enough to pull the dependency in as a wheel.","commit_id":"91144596c670741c2f452c864c026eff5583989c"},{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"772f31264f955c8c047132a0e98379ef25a51c61","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"41490243_ebc6c0b5","in_reply_to":"40a8139c_879e55c6","updated":"2023-01-24 23:49:47.000000000","message":"I thought I might need to call attention to the dropping of integrated testing on the ML, but https://docs.openstack.org/project-team-guide/stable-branches.html#maintenance-phases is pretty explicit about \"reduced CI commitment\" expectations:\n\n\u003e Without regular comprehensive maintenance, it is quite possible that someone proposing a backport to an EM branch will find that tests have broken since the last successful merge. This means that tests (or test configuration) might need to be fixed, reduced, or reconfigured before the backport itself can be evaluated and merged","commit_id":"91144596c670741c2f452c864c026eff5583989c"}]}