)]}'
{"id":"openstack%2Fwatcher~949640","triplet_id":"openstack%2Fwatcher~master~Ib0621b0e03451faca803018d6a2f3ad657a25fb5","project":"openstack/watcher","branch":"master","topic":"efficacy_data_migration","attention_set":{},"removed_from_attention_set":{"34452":{"account":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"last_update":"2025-05-16 18:16:23.000000000","reason":"Change was submitted"},"11604":{"account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"last_update":"2025-05-16 10:38:29.000000000","reason":"\u003cGERRIT_ACCOUNT_11604\u003e replied on the change","reason_account":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"}}},"hashtags":[],"change_id":"Ib0621b0e03451faca803018d6a2f3ad657a25fb5","subject":"Migrate value column of efficacy indicator on load","status":"MERGED","created":"2025-05-13 14:35:33.000000000","updated":"2025-05-16 18:17:26.000000000","submitted":"2025-05-16 18:16:23.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"949640-efficacy_data_migration","meta_rev_id":"0dbcd79491ccaa656bfa4c3413ae35803a1ab2b8","_number":949640,"virtual_id_number":949640,"owner":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"value":0,"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"tag":"autogenerated:zuul:gate","value":2,"date":"2025-05-16 18:16:23.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":"","default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"all":[{"value":2,"date":"2025-05-16 17:40:40.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":2,"date":"2025-05-16 10:38:29.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":1,"date":"2025-05-13 17:32:15.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"value":0,"_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":{"approved":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"all":[{"value":1,"date":"2025-05-16 17:40:40.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"value":0,"_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,"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},{"value":0,"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"value":0,"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"value":0,"_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":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-05-13 15:56:51.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"},{"updated":"2025-05-13 17:32:15.000000000","updated_by":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"reviewer":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"state":"REVIEWER"},{"updated":"2025-05-15 12:30:12.000000000","updated_by":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"reviewer":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"state":"REVIEWER"},{"updated":"2025-05-16 17:40:40.000000000","updated_by":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"reviewer":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"state":"REVIEWER"}],"messages":[{"id":"d574dd9b067c7e15e51d51f1f8367167620e6aac","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-05-13 14:35:33.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"8e951d42ee859eee397ad036d49f2e1d6548d3ac","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-05-13 14:38:24.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"a39a8155c448f70a4ac36715a3f18c3cdb4e4230","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-13 15:56:51.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/c08195cc675d4e9084f34f9d277fc310\n\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/baa433d91c6a4208a0a0d40c6636a48b : SUCCESS in 5m 36s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/e45617d55c9c496d981fb13d93ae3a84 : SUCCESS in 4m 02s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/7cfca2daf5a648539ef3d4a4f18f36f7 : SUCCESS in 5m 03s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/f9c8da75663e4411823a1ae8be5b11df : SUCCESS in 4m 48s\n- openstack-tox-py313 https://zuul.opendev.org/t/openstack/build/f5aee04a86984deebce474cda3999a31 : SUCCESS in 6m 09s (non-voting)\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c8ecb66742b54ad09b91d8282934f26a : SUCCESS in 4m 39s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/f8fb13381cf6452ab23e1a4cb46a15be : SUCCESS in 13m 34s\n- watcher-grenade https://zuul.opendev.org/t/openstack/build/f716f112cf9a4ce698f35f9328ba25ef : SUCCESS in 34m 04s\n- watcher-tempest-strategies https://zuul.opendev.org/t/openstack/build/71a51e3dd0aa400ba1d439fc8aeaaf57 : SUCCESS in 47m 07s\n- watcher-tempest-actuator https://zuul.opendev.org/t/openstack/build/1cb6a3308bc14bb88dfc182b358f32c9 : SUCCESS in 48m 17s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/46cbd9b872eb41208d297a79db9b06b2 : SUCCESS in 36m 31s\n- watcher-prometheus-integration https://zuul.opendev.org/t/openstack/build/8d622231aa9d456cab32e749abf804e5 : SUCCESS in 1h 13m 29s","accounts_in_message":[],"_revision_number":2},{"id":"70a1fb2b06c1ea61eddf4f82d2f0401c50c41092","author":{"_account_id":26471,"name":"Software Factory CI","email":"softwarefactory-operations-team@redhat.com","username":"sf-project-io"},"date":"2025-05-13 17:32:15.000000000","message":"Patch Set 2: Code-Review+1\n\nBuild succeeded (check pipeline).\nhttps://softwarefactory-project.io/zuul/t/rdoproject.org/buildset/ce12da271ada4d048bb21a58781a0dcf\n\n- openstack-meta-content-provider-master https://softwarefactory-project.io/zuul/t/rdoproject.org/build/53fa76efc750441aaa6200569a4b0bb0 : SUCCESS in 2h 52m 25s\n- watcher-operator-validation-master https://softwarefactory-project.io/zuul/t/rdoproject.org/build/f6135a31c6e0482b980b9e3fee0cb6e9 : SUCCESS in 1h 55m 34s","accounts_in_message":[],"_revision_number":2},{"id":"d5af93e70066d96d632a7437241c0fba7118fa4f","author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"date":"2025-05-16 10:38:29.000000000","message":"Patch Set 2: Code-Review+2\n\n(2 comments)","accounts_in_message":[],"_revision_number":2},{"id":"dbe0f8cd07a78c9b6b6f81e8e945c367b644657b","author":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"date":"2025-05-16 10:44:01.000000000","message":"Patch Set 2:\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"99605f786a15ef38ffe8d6b7eb6fdcad11ad5a3a","author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"date":"2025-05-16 17:40:40.000000000","message":"Patch Set 2: Code-Review+2 Workflow+1\n\n(1 comment)","accounts_in_message":[],"_revision_number":2},{"id":"7493b5347517da3cbb2077d5c464910b45ffc5b8","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-16 17:41:02.000000000","message":"Patch Set 2: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":2},{"id":"7bec29450dfc8c6b007097e31db5185fb2d6b481","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-16 18:16:23.000000000","message":"Patch Set 2: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/9b9c0b7234b74d7c960bf1236eae310c\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/788f2ac2b2ed4488af0a00343b947fa2 : SUCCESS in 4m 01s\n- openstack-tox-py310 https://zuul.opendev.org/t/openstack/build/45123eecd399421aa12dee3ae2ff8056 : SUCCESS in 4m 24s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/cff96df446184886ae266107eb42abca : SUCCESS in 6m 57s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/a56d31d078594ca993374cfa4210ec70 : SUCCESS in 9m 33s\n- watcher-tempest-functional https://zuul.opendev.org/t/openstack/build/812f585444f24160a28105e178b580bd : SUCCESS in 31m 01s\n- watcher-tempest-functional-ipv6-only https://zuul.opendev.org/t/openstack/build/cb88a9956d674be9b348eae9b7568a40 : SUCCESS in 28m 28s","accounts_in_message":[],"_revision_number":2},{"id":"346343f96b48fb32be643efa406316c05b4b3f7d","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-16 18:16:23.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":2},{"id":"0dbcd79491ccaa656bfa4c3413ae35803a1ab2b8","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-05-16 18:17:26.000000000","message":"Patch Set 2:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/42cf01c0547a4efc822a3837a5fd26f5\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9c8fa90e8199478492e03e9257d8168e : SUCCESS in 55s","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"322c89d9820e2f2ee153eb56d8e2068506f6f309","revisions":{"b15b3ba27e0e48634f1452d52ee67c2f35c277c4":{"kind":"REWORK","_number":1,"created":"2025-05-13 14:35:33.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/40/949640/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/40/949640/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/40/949640/1"}}},"commit":{"parents":[{"commit":"fd3d8b67ffbd7b89b6a84f5f17cb4d5890c1bb29","subject":"Merge \"Set number of decimal digits in efficacy indicator\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/fd3d8b67ffbd7b89b6a84f5f17cb4d5890c1bb29"}]}],"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-13 14:32:07.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","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/b15b3ba27e0e48634f1452d52ee67c2f35c277c4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/b15b3ba27e0e48634f1452d52ee67c2f35c277c4"}]},"branch":"refs/heads/master"},"322c89d9820e2f2ee153eb56d8e2068506f6f309":{"kind":"REWORK","_number":2,"created":"2025-05-13 14:38:24.000000000","uploader":{"_account_id":34452,"name":"Joan Gilabert","display_name":"jgilaber","email":"jgilaber@redhat.com","username":"jgilaber"},"ref":"refs/changes/40/949640/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/watcher","ref":"refs/changes/40/949640/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/watcher refs/changes/40/949640/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/watcher refs/changes/40/949640/2"}}},"commit":{"parents":[{"commit":"fd3d8b67ffbd7b89b6a84f5f17cb4d5890c1bb29","subject":"Merge \"Set number of decimal digits in efficacy indicator\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/fd3d8b67ffbd7b89b6a84f5f17cb4d5890c1bb29"}]}],"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-13 14:36:59.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","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/322c89d9820e2f2ee153eb56d8e2068506f6f309"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/watcher/commit/322c89d9820e2f2ee153eb56d8e2068506f6f309"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[{"rule_name":"gerrit~DefaultSubmitRule","status":"CLOSED","labels":[{"label":"Verified","status":"MAY","applied_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}},{"label":"Code-Review","status":"MAY","applied_by":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"}}]}],"submit_requirements":[{"name":"Verified","description":"Verified in gate by CI","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Verified\u003dMAX AND -label:Verified\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Verified\u003dMAX"],"failing_atoms":["label:Verified\u003dMIN"],"atom_explanations":{}}},{"name":"Code-Review","description":"Code reviewed by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Code-Review\u003dMAX AND -label:Code-Review\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Code-Review\u003dMAX"],"failing_atoms":["label:Code-Review\u003dMIN"],"atom_explanations":{}}},{"name":"Workflow","description":"Approved for gate by core reviewer","status":"SATISFIED","is_legacy":false,"submittability_expression_result":{"expression":"label:Workflow\u003dMAX AND -label:Workflow\u003dMIN","fulfilled":true,"status":"PASS","passing_atoms":["label:Workflow\u003dMAX"],"failing_atoms":["label:Workflow\u003dMIN"],"atom_explanations":{}}}]}
