)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9236,"name":"Jon Bernard","email":"jobernar@redhat.com","username":"jbernard"},"change_message_id":"6506ccbe15ee6d60cec51b6cd79b300e8428206c","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"51e19430_fbaf63d5","updated":"2023-11-14 18:23:47.000000000","message":"Stephen, can you explain the TODO note?  I\u0027m not following the logic for\nthat.","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"7bcfeeb063c0725c5b46b36d7ab08a9f2ae7bf8d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"11b998dd_b8358eae","updated":"2023-11-13 17:04:26.000000000","message":"recheck unrelated failure","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"d9b6106870854a6932e9a3783dfd0ee2e9c61215","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"9d840336_ca37de8c","updated":"2023-11-16 14:38:52.000000000","message":"recheck unrelated failure","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3295e8a317b632aa639cfdb807fb402af17df8eb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"48c8ed83_bc4dc7a5","in_reply_to":"1055abd8_ab902d4a","updated":"2025-10-30 16:49:28.000000000","message":"Done","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"61b0f6105caccbc6c5909da44b5b9725c59ab0fc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1055abd8_ab902d4a","in_reply_to":"3aafa31d_a94287d5","updated":"2025-10-30 16:45:45.000000000","message":"I added a comment here","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"68eba0a737a276628614b8f6acad1884de718caa","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3aafa31d_a94287d5","in_reply_to":"51e19430_fbaf63d5","updated":"2023-11-15 12:22:11.000000000","message":"I\u0027ve been told (by zzzeek) that the whole `check_constraint` thing has been the source of many bugs over the years and offers very little value. SQLAlchemy-Migrate used it but Alembic neither uses it not is it able to detect the constraint (see [github.com/sqlalchemy/alembic#508](https://github.com/sqlalchemy/alembic/issues/508) and the [alembic docs](https://alembic.sqlalchemy.org/en/latest/autogenerate.html#what-does-autogenerate-detect-and-what-does-it-not-detect) for more info). The combo of bugs and mismatches between the models suggests to me we\u0027d be better off being consistent and removing its use everywhere, hence the TODOs. I should probably not that this is in the category of TODOs that I don\u0027t ever expect us to do though...\n\nFeel free to stick that in the commit message assuming it makes sense? I\u0027m trying to avoid touching anything in this stack unnecessarily as it takes a long time to get eyes on","commit_id":"c10285bbab91a8f9d3f0fca5f840e4ea9fa904ec"}]}