)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"a958367c39a0497a56f25f04d7e2d45e6a218403","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ac8d7a4b_f9686456","updated":"2026-10-23 15:39:28.000000000","message":"Agreed, done: the set_default() is gone, the override_config() right above is enough since it registers the cleanup itself.\n\ntest_start_workflow() in test_cron_trigger.py had the same problem (set_default() without restore) and got the same treatment; note it flips auth_enable twice (True to create the trigger, False to process it), and once the first flip became an override, the second set_default(False) stopped shadowing it (an override wins over a default), so both are override_config() calls now.\n\nWhile verifying this with stestr --dynamic I also hit a related leak in the keystone unit tests added recently: they register the keystoneauth1 conf options without any default, so a later test parsing the config in the same worker fails with RequiredOptError. Fixed separately in https://review.opendev.org/c/openstack/mistral/+/1012002.","commit_id":"b70620454e6a3726529b51224daaee42ddbc4898"}],"mistral/tests/unit/api/v2/test_members.py":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"b8f08a1346389afe12f0ec3095e6f203ccc74d05","unresolved":true,"context_lines":[{"line_number":64,"context_line":"        WORKFLOW_MEMBER_ACCEPTED \u003d copy.deepcopy(WORKFLOW_MEMBER_PENDING)"},{"line_number":65,"context_line":"        WORKFLOW_MEMBER_ACCEPTED[\u0027status\u0027] \u003d \u0027accepted\u0027"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        cfg.CONF.set_default(\u0027auth_enable\u0027, True, group\u003d\u0027pecan\u0027)"},{"line_number":68,"context_line":"        self.addCleanup("},{"line_number":69,"context_line":"            cfg.CONF.set_default,"},{"line_number":70,"context_line":"            \u0027auth_enable\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"3373e925_29b4fd38","line":67,"updated":"2026-10-16 14:27:58.000000000","message":"I believe we can even get rid of this line, since there is an self.override_config right above that does the same.\nNote that self.override_config is also registering the cleanup internally so I believe removing all cfg.CONF.set_default will kill the bug and should be enough","commit_id":"2b00ff0ed309137de6d62693ba3c325f1caa8e57"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"a958367c39a0497a56f25f04d7e2d45e6a218403","unresolved":false,"context_lines":[{"line_number":64,"context_line":"        WORKFLOW_MEMBER_ACCEPTED \u003d copy.deepcopy(WORKFLOW_MEMBER_PENDING)"},{"line_number":65,"context_line":"        WORKFLOW_MEMBER_ACCEPTED[\u0027status\u0027] \u003d \u0027accepted\u0027"},{"line_number":66,"context_line":""},{"line_number":67,"context_line":"        cfg.CONF.set_default(\u0027auth_enable\u0027, True, group\u003d\u0027pecan\u0027)"},{"line_number":68,"context_line":"        self.addCleanup("},{"line_number":69,"context_line":"            cfg.CONF.set_default,"},{"line_number":70,"context_line":"            \u0027auth_enable\u0027,"}],"source_content_type":"text/x-python","patch_set":1,"id":"a2269205_bcdedef0","line":67,"in_reply_to":"3373e925_29b4fd38","updated":"2026-10-23 15:39:28.000000000","message":"Done","commit_id":"2b00ff0ed309137de6d62693ba3c325f1caa8e57"}]}