)]}'
{"id":"openstack%2Fwatcher~950283","triplet_id":"openstack%2Fwatcher~stable%2F2025.1~Ib0621b0e03451faca803018d6a2f3ad657a25fb5","project":"openstack/watcher","branch":"stable/2025.1","topic":"bug/2103458","attention_set":{"34452":{"account":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"last_update":"2025-05-19 09:13:30.000000000","reason":"A robot voted negatively on a label"}},"removed_from_attention_set":{},"hashtags":[],"change_id":"Ib0621b0e03451faca803018d6a2f3ad657a25fb5","subject":"Migrate value column of efficacy indicator on load","status":"NEW","created":"2025-05-19 07:46:17.000000000","updated":"2025-07-01 16:25:29.000000000","submit_type":"MERGE_IF_NECESSARY","mergeable":true,"submittable":false,"total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"9fc807d012d9a1bf33d4e6c39583b21c0b83cf6a","_number":950283,"virtual_id_number":950283,"owner":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"actions":{},"labels":{"Verified":{"disliked":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":-1,"date":"2025-05-19 10:33:25.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":-1,"default_value":0,"optional":true},"Code-Review":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}],"values":{" 0":"Standard Change","+1":"Important Change","+2":"High Priority Change"},"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-05-19 09:13:30.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"}],"messages":[{"id":"cf62bddee14c816519cf18613c726dfe6dc1196c","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-05-19 07:46:17.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"ad2d96c790f66db8aa4c591e83d3740bc477ff58","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-19 09:13:30.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/ea17eac2987d459486c4efc37ed7182c\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/96523ffef2ca47da9b4642e506633bca : FAILURE in 4m 19s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/ecc70753d81243a388be6902c6a210d3 : SUCCESS in 3m 30s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/5631f813fe834911b2da7bd44e6f3460 : FAILURE in 3m 55s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/01f9756366f8477b8bc100c0cb7797da : FAILURE in 4m 36s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/6f0f8899896641559dd55e75d69bfb28 : SUCCESS in 7m 29s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/aaf4682816ce45238b267fc812a699a1 : SUCCESS in 32m 23s\n- watcher-tempest-functional-jammy https://zuul.opendev.org/t/openstack/build/4f75113d1a6f4eaab85503f775983b07 : SUCCESS in 23m 05s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/00453d14a13049bd96c6b2a8556bd8c0 : SUCCESS in 45m 25s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/628af1a8cf8a456a9a4f43a997d95f97 : FAILURE in 46m 40s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/4957efe4ec2c49f48de85ca520f26f7a : SUCCESS in 48m 06s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/8fbd02fc71fa489584c9e24d42433732 : SUCCESS in 35m 05s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/09c7c2b5d52c4e00b612a24d0dcdfcd7 : SUCCESS in 30m 36s\n- watcher-prometheus-integration https://zuul.opendev.org/t/openstack/build/a47ef41e014743de94f527dc52b8e4dc : FAILURE in 1h 21m 14s","accounts_in_message":[],"_revision_number":1},{"id":"dcd30a425efbcef0e3bf74d06330b2843feaa002","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-05-19 09:17:50.000000000","message":"Uploaded patch set 2.\n\nOutdated Votes:\n* Verified-1\n","accounts_in_message":[],"_revision_number":2},{"id":"957c84791020db0a229dddab33b26f94a765967a","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-19 10:33:25.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\nand https://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\n\nhttps://zuul.opendev.org/t/openstack/buildset/90dd52f6d68e4cfaada0af9b128156c8\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/1b21c8cd6e98498cb460fe3c594d449d : SUCCESS in 4m 10s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/05beeda474924e64b0f4d6d9a871d78d : SUCCESS in 4m 19s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/335f36f699704b2da92ef1e3fe36c551 : SUCCESS in 4m 30s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/2f8dd2b5f4c34d39921da5e6ca0da21b : SUCCESS in 4m 49s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/4132f51db73744bda3945eb0c583ddd4 : SUCCESS in 10m 18s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/bf3440ffae484b4d871e00dba5701acd : SUCCESS in 32m 29s\n- watcher-tempest-functional-jammy https://zuul.opendev.org/t/openstack/build/b1f7271fe29042d2801a1c55204d77cc : SUCCESS in 14m 20s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/c2554ea5367d4ba8a5ebb13f696d19dd : SUCCESS in 47m 20s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/86e7b3b73197418a85a942d5853613cc : FAILURE in 51m 25s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/7594b092c0d046b3beac843a88ba196e : SUCCESS in 50m 41s\n- watcherclient-tempest-functional https://zuul.opendev.org/t/openstack/build/868beab5d9c4421ba7c290ae1ca5ebee : SUCCESS in 37m 48s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/2f8253c80b5745a9ba586a8211ff1a1d : SUCCESS in 32m 41s\n- watcher-prometheus-integration https://zuul.opendev.org/t/openstack/build/a69d8b793aa249feb1df05e08a3a84eb : FAILURE in 1h 10m 35s","accounts_in_message":[],"_revision_number":2},{"id":"c978f327b68a15afc7ce77a265d7d3efe642ec4c","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-07-01 16:25:25.000000000","message":"Topic fix_efficacy_indicator_schema-stable/2025.1 removed","accounts_in_message":[],"_revision_number":2},{"id":"9fc807d012d9a1bf33d4e6c39583b21c0b83cf6a","tag":"autogenerated:gerrit:setTopic","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-07-01 16:25:29.000000000","message":"Topic set to bug/2103458","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"7b928b6815b5875cc50fb7152d69c983ca684163","revisions":{"bfc8d4e88475ba945422cea6feeb708cb7526adf":{"kind":"REWORK","_number":1,"created":"2025-05-19 07:46:17.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/83/950283/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/83/950283/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/83/950283/1"}}},"commit":{"parents":[{"commit":"8a99d4c5c14e543ef0c35376dfaccbab5535ef91","subject":"Add support for pyproject.toml and wsgi module paths","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/8a99d4c5c14e543ef0c35376dfaccbab5535ef91"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-13 11:33:10.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-19 07:46:09.000000000","tz":120},"subject":"Migrate value column of efficacy indicator on load","message":"Migrate value column of efficacy indicator on load\n\nIn a recent change [1] we modified the database schema for efficacy\nindicators to use a \u0027data\u0027 column. However, that patch only contained\nthe schema migration and a fallback to be able to read from older\ndatabases, and not any kind of data migration. This change introduces\na migration on load, so whenever an efficacy indicator without a \u0027data\u0027\ncolumn is loaded, the column is populated in the database. The change\nalso modifies the migration test to verify the procedure works well.\n\n[1] https://review.opendev.org/c/openstack/watcher/+/945199\n\nChange-Id: Ib0621b0e03451faca803018d6a2f3ad657a25fb5\n(cherry picked from commit 322c89d9820e2f2ee153eb56d8e2068506f6f309)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/bfc8d4e88475ba945422cea6feeb708cb7526adf"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/bfc8d4e88475ba945422cea6feeb708cb7526adf"}]},"branch":"refs/heads/stable/2025.1"},"7b928b6815b5875cc50fb7152d69c983ca684163":{"kind":"REWORK","_number":2,"created":"2025-05-19 09:17:50.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/83/950283/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/83/950283/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/83/950283/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/83/950283/2"}}},"commit":{"parents":[{"commit":"1e4e0b7dd233348884ed6973541b9748827c3c95","subject":"Set number of decimal digits in efficacy indicator","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/1e4e0b7dd233348884ed6973541b9748827c3c95"}]}],"author":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-13 11:33:10.000000000","tz":120},"committer":{"name":"jgilaber","email":"jgilaber@redhat.com","date":"2025-05-19 09:17:10.000000000","tz":120},"subject":"Migrate value column of efficacy indicator on load","message":"Migrate value column of efficacy indicator on load\n\nIn a recent change [1] we modified the database schema for efficacy\nindicators to use a \u0027data\u0027 column. However, that patch only contained\nthe schema migration and a fallback to be able to read from older\ndatabases, and not any kind of data migration. This change introduces\na migration on load, so whenever an efficacy indicator without a \u0027data\u0027\ncolumn is loaded, the column is populated in the database. The change\nalso modifies the migration test to verify the procedure works well.\n\n[1] https://review.opendev.org/c/openstack/watcher/+/945199\n\nChange-Id: Ib0621b0e03451faca803018d6a2f3ad657a25fb5\n(cherry picked from commit 322c89d9820e2f2ee153eb56d8e2068506f6f309)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/7b928b6815b5875cc50fb7152d69c983ca684163"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/7b928b6815b5875cc50fb7152d69c983ca684163"}]},"branch":"refs/heads/stable/2025.1"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"OK","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY"},{"label":"Workflow","status":"MAY"},{"label":"Review-Priority","status":"MAY"}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Verified\u003dMAX","label:Verified\u003dMIN"],"atom_explanations":{"label:Verified\u003dMAX":"","label:Verified\u003dMIN":""}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Code-Review\u003dMAX","label:Code-Review\u003dMIN"],"atom_explanations":{"label:Code-Review\u003dMAX":"","label:Code-Review\u003dMIN":""}}},{"name":"Review-Priority","description":"Review Priority","status":"NOT_APPLICABLE","is_legacy":false,"applicability_expression_result":{"fulfilled":false,"status":"FAIL"},"submittability_expression_result":{"expression":"is:true","fulfilled":true,"status":"NOT_EVALUATED","passing_atoms":[],"failing_atoms":[],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"UNSATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":false,"status":"FAIL","passing_atoms":[],"failing_atoms":["label:Workflow\u003dMAX","label:Workflow\u003dMIN"],"atom_explanations":{"label:Workflow\u003dMAX":"","label:Workflow\u003dMIN":""}}}]}