)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"24d7bc22c25425cfe2f325b7e8fc38c0f225f5ed","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fb8c3b13_fda255db","updated":"2026-01-21 13:38:54.000000000","message":"Adding oslo.concurrency to mistral.conf namespace list is here: https://review.opendev.org/c/openstack/mistral/+/974057\nSo I removed it from this patch.","commit_id":"2385967f10d3782065b9ae43e71bcf7f525828d9"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"daa7f545f75131bf3710021d2c053ac6498fa521","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ef92c2bb_8bd3489a","updated":"2026-01-21 14:39:39.000000000","message":"my previous concern is still valid.","commit_id":"2385967f10d3782065b9ae43e71bcf7f525828d9"}],"mistral/config.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f1229b14b64e25156315e3237c262e08d6e042e5","unresolved":true,"context_lines":[{"line_number":883,"context_line":"        default_config_files\u003ddefault_config_files,"},{"line_number":884,"context_line":"    )"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"    try:"},{"line_number":887,"context_line":"        lp \u003d CONF.oslo_concurrency.lock_path"},{"line_number":888,"context_line":"    except Exception:"},{"line_number":889,"context_line":"        lp \u003d None"},{"line_number":890,"context_line":""},{"line_number":891,"context_line":"    if lp:"},{"line_number":892,"context_line":"        lockutils.set_defaults(lp)"},{"line_number":893,"context_line":""},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"def set_config_defaults():"}],"source_content_type":"text/x-python","patch_set":1,"id":"a3c0aab0_cdfc6824","line":892,"range":{"start_line":886,"start_character":0,"end_line":892,"end_character":34},"updated":"2025-10-03 13:10:09.000000000","message":"I don\u0027t know if this is really required. What is the problem without this ?","commit_id":"8f2e1dc3fe6956ffcd66d3a56f0ec81c77e18bfe"},{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"41e3458a66a059b02fc445908933846f692913f3","unresolved":true,"context_lines":[{"line_number":883,"context_line":"        default_config_files\u003ddefault_config_files,"},{"line_number":884,"context_line":"    )"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"    try:"},{"line_number":887,"context_line":"        lp \u003d CONF.oslo_concurrency.lock_path"},{"line_number":888,"context_line":"    except Exception:"},{"line_number":889,"context_line":"        lp \u003d None"},{"line_number":890,"context_line":""},{"line_number":891,"context_line":"    if lp:"},{"line_number":892,"context_line":"        lockutils.set_defaults(lp)"},{"line_number":893,"context_line":""},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"def set_config_defaults():"}],"source_content_type":"text/x-python","patch_set":1,"id":"eccee0b8_932ea7e9","line":892,"range":{"start_line":886,"start_character":0,"end_line":892,"end_character":34},"in_reply_to":"048d6ccc_d647a764","updated":"2025-10-03 15:59:17.000000000","message":"I agree with takashi, the fact that you have nova is really weird.","commit_id":"8f2e1dc3fe6956ffcd66d3a56f0ec81c77e18bfe"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"cf543cd3f1cd1c4dccbaf900973dbe3c9380ce98","unresolved":true,"context_lines":[{"line_number":883,"context_line":"        default_config_files\u003ddefault_config_files,"},{"line_number":884,"context_line":"    )"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"    try:"},{"line_number":887,"context_line":"        lp \u003d CONF.oslo_concurrency.lock_path"},{"line_number":888,"context_line":"    except Exception:"},{"line_number":889,"context_line":"        lp \u003d None"},{"line_number":890,"context_line":""},{"line_number":891,"context_line":"    if lp:"},{"line_number":892,"context_line":"        lockutils.set_defaults(lp)"},{"line_number":893,"context_line":""},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"def set_config_defaults():"}],"source_content_type":"text/x-python","patch_set":1,"id":"eab6bc8b_caaae62d","line":892,"range":{"start_line":886,"start_character":0,"end_line":892,"end_character":34},"in_reply_to":"a3c0aab0_cdfc6824","updated":"2025-10-03 13:11:09.000000000","message":"Ignore my previous comment. So I think this is really a problem which should be fixed in oslo.concurrency IMHO...","commit_id":"8f2e1dc3fe6956ffcd66d3a56f0ec81c77e18bfe"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"f233cfcd6cf0cec7976114492b440285868d35f2","unresolved":true,"context_lines":[{"line_number":883,"context_line":"        default_config_files\u003ddefault_config_files,"},{"line_number":884,"context_line":"    )"},{"line_number":885,"context_line":""},{"line_number":886,"context_line":"    try:"},{"line_number":887,"context_line":"        lp \u003d CONF.oslo_concurrency.lock_path"},{"line_number":888,"context_line":"    except Exception:"},{"line_number":889,"context_line":"        lp \u003d None"},{"line_number":890,"context_line":""},{"line_number":891,"context_line":"    if lp:"},{"line_number":892,"context_line":"        lockutils.set_defaults(lp)"},{"line_number":893,"context_line":""},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"def set_config_defaults():"}],"source_content_type":"text/x-python","patch_set":1,"id":"048d6ccc_d647a764","line":892,"range":{"start_line":886,"start_character":0,"end_line":892,"end_character":34},"in_reply_to":"eab6bc8b_caaae62d","updated":"2025-10-03 13:26:55.000000000","message":"Hmmm so I think your problem is that something is injecting lock_path and its default is messed up ? I don\u0027t see any clue in oslo libs which may ends up in selecting /var/lib/nova. We really have to understand what brings this BEFORE we make any code change regarding this part.","commit_id":"8f2e1dc3fe6956ffcd66d3a56f0ec81c77e18bfe"}],"tools/config/config-generator.mistral.conf":[{"author":{"_account_id":11583,"name":"Arnaud Morin","email":"arnaud.morin@gmail.com","username":"arnaudmorin"},"change_message_id":"41e3458a66a059b02fc445908933846f692913f3","unresolved":false,"context_lines":[{"line_number":4,"context_line":"summarize \u003d true"},{"line_number":5,"context_line":"namespace \u003d mistral.config"},{"line_number":6,"context_line":"namespace \u003d cotyledon"},{"line_number":7,"context_line":"namespace \u003d oslo.concurrency"},{"line_number":8,"context_line":"namespace \u003d oslo.db"},{"line_number":9,"context_line":"namespace \u003d oslo.messaging"},{"line_number":10,"context_line":"namespace \u003d oslo.middleware.cors"}],"source_content_type":"text/plain","patch_set":1,"id":"7047b63e_ec01dddd","line":7,"updated":"2025-10-03 15:59:17.000000000","message":"As stated on IRC, we agree with that change, so if you can create a separate for this, we would happily merge it","commit_id":"8f2e1dc3fe6956ffcd66d3a56f0ec81c77e18bfe"}]}
