)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d8c051b318632929cc859c752d5c73cb46b84d76","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Add a test with mysql as backend to show that the current"},{"line_number":10,"context_line":"EfficacyIndicator model does not store any decimal digit for the value."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I0cdbd7d87cd6869a10b48eda3d59558831c8dd36"},{"line_number":13,"context_line":"(cherry picked from commit 6c5845721bd64df98e040d0678f30db8d433ce3f)"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"923b090d_722be5eb","line":11,"updated":"2025-07-01 16:31:07.000000000","message":"this should have had Related-Bug: #2103458 on master\nthis is the repoducer test","commit_id":"b54ddcd19b56e5f51d2276592d15822c83197f49"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d8c051b318632929cc859c752d5c73cb46b84d76","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"950ca32c_5cc37f51","updated":"2025-07-01 16:31:07.000000000","message":"this is a clean backport, i would like to take a look if they have time at the later patches as the fix actully has a db migratoin which while technically backprotable is very rarely done.\nhttps://review.opendev.org/c/openstack/watcher/+/950190/2\nthe third patch is optional https://review.opendev.org/c/openstack/watcher/+/950283/2\n\nbut if we proceed with the first two we shoudl include it","commit_id":"b54ddcd19b56e5f51d2276592d15822c83197f49"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"40796de50b3a00ab8d5c7213982b343e29ffde2e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c6f04fe3_befae1fd","in_reply_to":"950ca32c_5cc37f51","updated":"2025-07-02 06:21:59.000000000","message":"It\u0027s there any documented policy about what db changes can be backported to stable branches or it\u0027s mostly general rule of backwards compatibility + non API changed involved, etc... ?","commit_id":"b54ddcd19b56e5f51d2276592d15822c83197f49"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"b5a381c63d3929f43a38e37839b3dc1f741a8c9b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a81c254f_2be3d399","in_reply_to":"c6f04fe3_befae1fd","updated":"2025-07-02 06:35:15.000000000","message":"My main concern is about how this may affect the upgrade process. IIUC, if we backport this, users doing minor updates inside epoxy branch should do watcher-db-manage upgrade. While this I think this is a regular step in other openstack services, we are not documenting the process in watcher documentation.\n\nSaid this, i think it is a valid bug to fix in stable branches and per your comments i understand is backportable.","commit_id":"b54ddcd19b56e5f51d2276592d15822c83197f49"}],"test-requirements.txt":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"d8c051b318632929cc859c752d5c73cb46b84d76","unresolved":true,"context_lines":[{"line_number":7,"context_line":"WebTest\u003e\u003d2.0.27 # MIT"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Include drivers for opportunistic testing."},{"line_number":10,"context_line":"oslo.db[mysql]\u003e\u003d6.0.0 # Apache-2.0"}],"source_content_type":"text/plain","patch_set":1,"id":"11a2b70a_478d0748","line":10,"updated":"2025-07-01 16:31:07.000000000","message":"this is required for the opertunistic test fixture.\n\nuc on 2025.1 specifes\noslo.db\u003d\u003d\u003d17.2.1\n\nso this shoudl not have any real impact of testablity.\n\nso i tink im ok with this.\n\nthe mysql targert is just inculding the mysql supprot in sqlachem/pymysql.\n\nin reality those are already runtime deps.","commit_id":"b54ddcd19b56e5f51d2276592d15822c83197f49"}]}