)]}'
{"id":"openstack%2Fmistral~962134","triplet_id":"openstack%2Fmistral~stable%2F2025.1~I51f4128dd023846a4406cd1b414a56d741035483","project":"openstack/mistral","branch":"stable/2025.1","attention_set":{},"removed_from_attention_set":{"11583":{"account":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"last_update":"2025-09-25 06:05:41.000000000","reason":"\u003cGERRIT_ACCOUNT_11583\u003e replied on the change","reason_account":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}}},"hashtags":[],"change_id":"I51f4128dd023846a4406cd1b414a56d741035483","subject":"Do not call setup_db from engine","status":"MERGED","created":"2025-09-23 19:44:32.000000000","updated":"2025-09-25 07:38:12.000000000","submitted":"2025-09-25 07:37:26.000000000","submitter":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"total_comment_count":1,"unresolved_comment_count":0,"has_review_started":true,"submission_id":"962134","cherry_pick_of_change":962032,"cherry_pick_of_patch_set":2,"meta_rev_id":"0732e358000a31bc1d2ad432a18a113725035b0f","_number":962134,"virtual_id_number":962134,"owner":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"actions":{},"labels":{"Verified":{"approved":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:gate","value":2,"date":"2025-09-25 07:37:26.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"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":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2025-09-24 20:43:34.000000000","permitted_voting_range":{"min":2,"max":2},"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"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":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"all":[{"value":0,"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2025-09-24 20:43:34.000000000","permitted_voting_range":{"min":1,"max":1},"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2025-09-23 20:21:27.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":"ecc0505c1caa31d0f3350bf899ca64533aa67922","tag":"autogenerated:gerrit:newPatchSet","author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"date":"2025-09-23 19:44:32.000000000","message":"Patch Set 1: Cherry Picked from branch master.","accounts_in_message":[],"_revision_number":1},{"id":"6a2eb65ffde278bcccfde7cbe1a3e59fa8157b12","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-23 20:21:27.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/dfa6792a5ed9428faf945f50a1d4ceba\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/f22272a3da9a4efd91ea0b8356f8d35c : SUCCESS in 3m 31s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/b9ffaea737f342da9c4887d2b809dd6d : SUCCESS in 9m 13s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/a13f95449b864fd9a8edc42636398da9 : SUCCESS in 10m 26s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/37ee9802d74d4d12ad2ce7b4e923c7ee : SUCCESS in 7m 27s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/9e5f779a85f84415a9c437e7ad6a10ec : SUCCESS in 18m 20s (non-voting)\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/0fbac9d095534efebec18b2aad583e0c : SUCCESS in 25m 15s\n- mistral-devstack-jammy https://zuul.opendev.org/t/openstack/build/e3b1cadc7b5a43d8b50a07e977a2b1fc : SUCCESS in 28m 15s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/68e7de6eb01a4c0cb1fd0e3ef7c4cd59 : SUCCESS in 17m 43s\n- mistral-devstack-non-apache-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/90f6fdca93f54dc7aa85b333e188abfa : SUCCESS in 17m 56s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/13ac0b701805419f831b5c8eda199680 : SUCCESS in 17m 54s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/89269aff5391465cad0b418451c4cea1 : SUCCESS in 33m 18s","accounts_in_message":[],"_revision_number":1},{"id":"1454539a66fe69e820070bdb4ab771fd0c0d601d","author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"date":"2025-09-24 20:43:34.000000000","message":"Patch Set 1: Code-Review+2 Workflow+1","accounts_in_message":[],"_revision_number":1},{"id":"832a5b73f11758ac2506b96f2536f6dc58c4646a","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-24 20:43:59.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"563d380d4c6fc9a6aa9e7f64ecb51a5da9525f97","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-24 21:40:38.000000000","message":"Patch Set 1: Verified-2\n\nBuild failed (gate pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\nhttps://zuul.opendev.org/t/openstack/buildset/6084560970534e6cbef3e7251b58dd95\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/d5c8d39c5e134e4a90950bbc90321423 : SUCCESS in 2m 34s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/7489354907e549d081c33f8524fe04b3 : SUCCESS in 12m 20s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/3229d595af574a30ac17d1ff604c0ef8 : SUCCESS in 9m 27s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/70eebc3002a54d4b8e56c0f6837f80db : SUCCESS in 7m 12s\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/9059661baa54476cba64bba916864a2d : SUCCESS in 16m 55s\n- mistral-devstack-jammy https://zuul.opendev.org/t/openstack/build/fe6f7be9309a4a88840d491c44370b67 : SUCCESS in 31m 24s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/de1f95e8ee0d43fd87592ad3efb4acd8 : FAILURE in 19m 22s\n- mistral-devstack-non-apache-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/bf1310c5fedc4ca5876e6e44a1154adf : SUCCESS in 50m 56s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/e4de506962614af78c42e6d6f69eb093 : SUCCESS in 31m 44s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/d40067f07ec34010aeecd38b55a38370 : FAILURE in 10m 03s","accounts_in_message":[],"_revision_number":1},{"id":"7b66073185183d5205b68d3b07ffef10fcfd8b02","author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"date":"2025-09-25 06:05:41.000000000","message":"Patch Set 1:\n\n(1 comment)","accounts_in_message":[],"_revision_number":1},{"id":"d2918c306fa5e43c2e5818a4a15b46852019b256","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-25 06:51:13.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/f212e48799d146a0815f65e793caf59c\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/6a7eb242ccb2488e867f3c64bde12d8c : SUCCESS in 6m 03s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/cfe68624ce214d10b1197eaa3e231cfa : SUCCESS in 14m 03s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/90e75807f1b34f3fb21def903890a387 : SUCCESS in 9m 31s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/75606c7d2bac4a9a8358231e2fc5dbf0 : SUCCESS in 4m 22s\n- openstack-tox-cover https://zuul.opendev.org/t/openstack/build/b2e5d147ef6a468693545ea5818757a7 : FAILURE in 16m 30s (non-voting)\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/b3623591a6fb40fe98b421b7e519f807 : SUCCESS in 17m 20s\n- mistral-devstack-jammy https://zuul.opendev.org/t/openstack/build/cc31d5ac1abf48c1880afae308046d08 : SUCCESS in 24m 53s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/9eb6164967ad4f8c9cc79854a7a49b25 : SUCCESS in 39m 30s\n- mistral-devstack-non-apache-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/2c4be785c84a46c8a203c31baae2ec24 : SUCCESS in 38m 11s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/5219927ac48c410b9ab0a9e4066a9197 : SUCCESS in 36m 54s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/3a467f6036634f4a9dbd40b627315c3c : SUCCESS in 17m 50s","accounts_in_message":[],"_revision_number":1},{"id":"9263879b389a7b56b0f309cf7cdc52acdc5b4aed","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-25 06:51:34.000000000","message":"Patch Set 1: -Verified\n\nStarting gate jobs.","accounts_in_message":[],"_revision_number":1},{"id":"0223bfb2a5c7687471948126939ba79865231b59","tag":"autogenerated:zuul:gate","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-25 07:37:26.000000000","message":"Patch Set 1: Verified+2\n\nBuild succeeded (gate pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/79eda4c998ba4f7aa718289a9235f63a\n\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/5ceffaeb2e73411a82eb691adf5e365d : SUCCESS in 3m 09s\n- openstack-tox-py39 https://zuul.opendev.org/t/openstack/build/b6cb4f75950d4977bab4bcf49656b397 : SUCCESS in 12m 27s\n- openstack-tox-py312 https://zuul.opendev.org/t/openstack/build/71c1082866434223a47b44a7e6ae22b0 : SUCCESS in 10m 17s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/c493faf2e75b475ea9ca5f0c976f5360 : SUCCESS in 5m 45s\n- mistral-devstack https://zuul.opendev.org/t/openstack/build/18c35caf433740fc914db4362896a3f4 : SUCCESS in 28m 30s\n- mistral-devstack-jammy https://zuul.opendev.org/t/openstack/build/61c16ba5eb534c3c93610d2ef4e6d394 : SUCCESS in 28m 03s\n- mistral-devstack-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/0db89bb88ba74d38bd09356bd77baba7 : SUCCESS in 17m 11s\n- mistral-devstack-non-apache-tempest-ipv6-only https://zuul.opendev.org/t/openstack/build/fe405ec50da84aad8c44008dc21bfe5b : SUCCESS in 17m 40s\n- mistral-devstack-non-apache https://zuul.opendev.org/t/openstack/build/c77cee703179499ebacc068870538ba4 : SUCCESS in 26m 11s\n- mistral-devstack-kombu https://zuul.opendev.org/t/openstack/build/01060728f22442f0ba7c01cb84784fdb : SUCCESS in 41m 14s","accounts_in_message":[],"_revision_number":1},{"id":"caeb6da5cdb71622d3faabe9e140b1daa3368e93","tag":"autogenerated:gerrit:merged","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-25 07:37:26.000000000","message":"Change has been successfully merged","accounts_in_message":[],"_revision_number":1},{"id":"0732e358000a31bc1d2ad432a18a113725035b0f","tag":"autogenerated:zuul:promote","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2025-09-25 07:38:12.000000000","message":"Patch Set 1:\n\nBuild succeeded (promote pipeline).\nhttps://zuul.opendev.org/t/openstack/buildset/7ac383b7c8794a999a84be88f753b6f6\n\n- promote-openstack-tox-docs https://zuul.opendev.org/t/openstack/build/9043bf4133754c3585d308683c2f6014 : SUCCESS in 35s","accounts_in_message":[],"_revision_number":1}],"current_revision_number":1,"current_revision":"c79e903f495e7b85f9713befd5a945fb71efde2b","revisions":{"c79e903f495e7b85f9713befd5a945fb71efde2b":{"kind":"REWORK","_number":1,"created":"2025-09-23 19:44:32.000000000","uploader":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"ref":"refs/changes/34/962134/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/mistral","ref":"refs/changes/34/962134/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/mistral refs/changes/34/962134/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/mistral refs/changes/34/962134/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/mistral refs/changes/34/962134/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/mistral refs/changes/34/962134/1"}}},"commit":{"parents":[{"commit":"6e16fbdad2df2767963b095f3dcec4c0724ad374","subject":"Stop logging/printing user token on failed validation","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/6e16fbdad2df2767963b095f3dcec4c0724ad374"}]}],"author":{"name":"Arnaud M","email":"arnaud.morin@gmail.com","date":"2025-09-22 20:32:09.000000000","tz":120},"committer":{"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","date":"2025-09-23 19:44:32.000000000","tz":0},"subject":"Do not call setup_db from engine","message":"Do not call setup_db from engine\n\nCalling setup_db is creating the db structure if missing (because it\u0027s\ncalling models.Workbook.metadata.create_all after).\n\nThis is not what we want since we want the operator to manage the\ncreation of the db using mistral-db-manage like any other regular\nopenstack service.\n\nI suspect that this call was mostly useful for tests.\nThis is a real issue in production because it create the DB structure\nwithout using the correct alembic migration path (and thus,\nalembic_version table is missing).\n\nChange-Id: I51f4128dd023846a4406cd1b414a56d741035483\nSigned-off-by: Arnaud M \u003carnaud.morin@gmail.com\u003e\n(cherry picked from commit 8be59adc3810e8a68c94b6c63998eef7fb769830)\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/c79e903f495e7b85f9713befd5a945fb71efde2b"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/mistral/commit/c79e903f495e7b85f9713befd5a945fb71efde2b"}]},"branch":"refs/heads/stable/2025.1"}},"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":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}},{"label":"Workflow","status":"MAY","applied_by":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"}}]}],"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":{}}}]}
