)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a216e4b78c07d8cd2dc8bb00f8f7749ad6901457","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Debian: stop tweaking uwsgi/threads"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"411e9ffb_6ee3f95f","line":9,"range":{"start_line":9,"start_character":43,"end_line":9,"end_character":51},"updated":"2021-04-21 12:53:24.000000000","message":"nova-api and nova-metadata-api ?","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"530b434d21f75de0bd785a1df5bb41a9b2af65c5","unresolved":true,"context_lines":[{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"f0b94740_aad5e71a","line":12,"updated":"2021-04-20 15:25:01.000000000","message":"Do we have any bug explaining this issue or limitation ? It\u0027s hard to understand the reason of this change, and it\u0027s not clear whether this change should be implemented in the other components.\n\nAlso, I think it would make more sense to update the default value rather than dropping the thread parameter.","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"173b82bf11cffbdea146560dd7edae7afb989eca","unresolved":true,"context_lines":[{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fd93ff0a_248b3439","line":12,"in_reply_to":"0d9aa8c2_3e1507dd","updated":"2021-04-21 15:14:13.000000000","message":"Done as you suggested. I believe I saw a bug opened for Nova, but couldn\u0027t find it anymore. Anyways, the nova-api logs are very clear in this regard, and there\u0027s a warning, so it should IMO be fine.","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a216e4b78c07d8cd2dc8bb00f8f7749ad6901457","unresolved":true,"context_lines":[{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"0d9aa8c2_3e1507dd","line":12,"in_reply_to":"6d20996c_3e11f075","updated":"2021-04-21 12:53:24.000000000","message":"If we should force threads\u003d1 then I still feel it is valuable to ensure threads\u003d1 .\nMy suggestion here is to just change the default and show a warning message when threads\u003e1.\nThis also allows users to use threads just in case the problem with threading is fixed in a future release.\n\nThis is not mandatory but if you can report a bug for nova and put a link for it then it helps people understand the reason behind this.","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"},{"author":{"_account_id":6476,"name":"Thomas Goirand","email":"thomas@goirand.fr","username":"thomas-goirand"},"change_message_id":"c4e1a3a5e743d7bb7d74b39268783f90e9e1067c","unresolved":false,"context_lines":[{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6d20996c_3e11f075","line":12,"in_reply_to":"f0b94740_aad5e71a","updated":"2021-04-20 16:26:36.000000000","message":"Hi.\nThe issue is simply that by default, the Debian package has threads\u003d1, which is a working value. Unfortunately, changing it to anything else (ie: more than 1) renders nova-api completely broken (the log says: impossible to switch tread).\n\nIt doesn\u0027t make sense to update the default value, because we do not want anyone to even attempt to change the value.\n\nThis shouldn\u0027t be implemented in other components, the problem is only in Nova, others are fine with threads\u003d32 by default.","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"4527b0b02edf9d47985697e59fecd6a40407fc3b","unresolved":true,"context_lines":[{"line_number":9,"context_line":"We should not tweak the number of threads, nova-api is only"},{"line_number":10,"context_line":"mono-threaded."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: Ib109175b2e1a9f9f6b96355d29f9f9a95c303957"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"fba74432_f31e68af","line":12,"in_reply_to":"fd93ff0a_248b3439","updated":"2021-04-23 12:43:14.000000000","message":"Maybe this is related ?\n https://bugs.launchpad.net/nova/+bug/1829062","commit_id":"0148abb6b17f0837637002f8f9979ad11a054322"}]}