)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"7bd8ca67704d9d949e9ae73122f2ac2a1f736366","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4381cbfc_16f43f80","updated":"2022-11-29 21:54:06.000000000","message":"@kevko: it would be interesting to know why you ran into this issue recently. Cinder updated this change on stable/victoria in May 2021 [1]. This was shipped in cinder 17.2.0 released in September 2021 [2]. Are you using binary images which would have updated their packages only recently?\n\nI would agree with yoctozepto, we can\u0027t add more options to kolla-ansible just to be able to run with old images. I suppose we would advise using kolla images built around the same time as the kolla-ansible code being used to deploy them.\n\n[1] https://review.opendev.org/c/openstack/cinder/+/792128\n[2] https://review.opendev.org/c/openstack/releases/+/809892","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"2a417b895ab40c70923e916835b6971de8a6e95d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4b70f86d_cf71e4ed","updated":"2022-10-18 16:29:27.000000000","message":"It violates two things from our yesterday-discussed Zens - 1) it modifies a default just to get some optimisation by default that might not be applicable to everyone; 2) it does a change that is otherwise very easily achievable with config overrides.","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"da2687583805432e08ed3dfeb694e25c4a5c2f86","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"8f381204_925292eb","in_reply_to":"1890250e_3cbda8ef","updated":"2022-10-18 18:45:31.000000000","message":"Well then it still violates the rule of not owning service\u0027s defaults. Why add a variable if it can be overridden by a user with same efficiency? /me done with this patch.","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"857e80347c0d0c7ad7ff67a50303b2e17fbc3339","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e33f7b00_8d87c3e1","in_reply_to":"4b70f86d_cf71e4ed","updated":"2022-10-18 17:13:29.000000000","message":"Don\u0027t agree -  this default is right one, and of course this is default in upstream. And this is default for whole backend, you have backend snippets in cinder-volume (which are enabled) where you should override via config override.","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"dd6888a6c221cef3234383ed6482ab2090ec5f10","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1890250e_3cbda8ef","in_reply_to":"57c2a215_bbe2ee87","updated":"2022-10-18 17:40:48.000000000","message":"Why kolla sets for example below value : \n\nvolume_name_template \u003d volume-%s  ? \n\nCan we remove it ? it is default from a code ..not needed to have it configuration.","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"41e1d6e7154e9334dbcf65293b998490db7334b9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e63250a7_29b3684b","in_reply_to":"e33f7b00_8d87c3e1","updated":"2022-10-18 17:23:48.000000000","message":"Who is configuring services ? Kolla-ansible or developer in a cinder code ? \nBecause some time ago kolla released cinder images with different code inside.\n\nDD.MM.YYYY - Kolla released image with code where this option was false (unfortunatelly we have it)\nDD.MM.YYYY + several days - Kolla released image with code where this option is true\n\n^^ So that means kolla changed cinder and you didn\u0027t know it.\n\nNot everyone is upgrading working openstack (images) every week. And if this option is true by default and *it is* true even if it is not in configuration. Why to not set up ? Option in your backend snippets will override default one.\n\nThis is not changing anything ..this only adds option to configure if anybody wants.\n\nThis is not same as cluster \u003d XYZ in cinder.conf ..because user need to migrate somehow in that case - because it is not compatible change.\n\nThis option can user change as he wants based on value in /etc/kolla/globals AND can override on another layer - in configuration overrided in /etc/kolla/config/cinder/...*\n\nFrom my opinion kolla-ansible should provide the best default configuration and let user to override in /etc/kolla/config/service.conf\n\nThat\u0027s it ...think about it, if you will not change your opinion ..i will abandon this right after your reply.","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"944880a3fa9439d4c321a69a92b6d6ce95e2a367","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"57c2a215_bbe2ee87","in_reply_to":"e63250a7_29b3684b","updated":"2022-10-18 17:34:22.000000000","message":"You are just saying, nevermind - we are not supporting cinder images before  change https://review.opendev.org/c/openstack/cinder/+/792128 (for victoria).\n\nSo,cinder figured out that default value true is better ..but kolla-ansible no.\n\nOk then","commit_id":"ae29fab49a1f4ac71461be67ea878ad4c8e5d309"}]}