)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"522e002720342e61b801b9c8ba9db3cc59b07a94","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"a7a55985_fa022f39","updated":"2026-10-02 04:29:49.000000000","message":"I have large concern about this.\n\nThe primary point is that the change likely breaks deployments using gnocchi with swift backend, due to the recursive loop it now enables.\n\nAlso I\u0027m not 100 sure about the statement that \"ceph is better and more popular\" without evidence like user survey, although this may not be as much important as the first one.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"2d4549aff85e920ff6e452bf81a4fb42e7b14ba9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2fdefdf5_20500afe","in_reply_to":"228c3c55_44f411ce","updated":"2026-10-02 05:29:38.000000000","message":"I should also add that Gnocchi\u0027s Swift storage driver didn\u0027t properly support multi-region clouds either until I added support for that too, so combined with lack of storage policy support I don\u0027t think it was inaccurate to say it was unsuitable for production use compared to other storage drivers like Ceph.\n\nOperators should be aware of how Ceilometer operates, and if I was deploying OpenStack for the first time, having it *not* publish metrics for the service project that every OpenStack service is configured to use by default is unintuitive. This is why I think the filtering should be disabled by default.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ad5a19c1bfdede73331809044eed282c8d736ad2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"c3aee6fa_205931a0","in_reply_to":"228c3c55_44f411ce","updated":"2026-10-02 05:45:39.000000000","message":"While I tend to agree that the swift driver lacks several capabilities and scalabilities for large deployments, it has been an convienient solution for small clouds without ceph or nfs, but with swift.\nI\u0027ve personally seen such small clouds using swift backend actually and care about the impact on them.\n\n\u003e Yes, it will cause feedback loops if operators do not take action, but that is the point of adding the upgrade release note, which I hope operators actually look at.\n\nI had hoped so many times, but in reality many operators do not read docs or release notes carefully, and does not notice the problem until their deployment breaks.\n\n\u003e it\u0027s an educated one as when we first evaluated using Gnocchi with Swift it was entirely unsuitable for production use\n\nAgain, I know it\u0027s problematic in large scale frequently but has been a first-easy solution for small clouds.\n\nMaybe the safer path is to log warnings if the enable_filter_project option is not explicitly set and then change the default after 2 cycles with that warning, instead of changing the behavior directly.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"059b0d5a4e70b9c453581a863e75005e2d1d9f89","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"540dfec9_6bc55b79","in_reply_to":"8f6ae171_0b9e1682","updated":"2026-10-02 08:14:44.000000000","message":"New patchset for adding the warning: https://review.opendev.org/c/openstack/ceilometer/+/1008421","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"2eea984b8832bbe65dc29e90f5e5e49b54dddc62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"228c3c55_44f411ce","in_reply_to":"a7a55985_fa022f39","updated":"2026-10-02 05:23:34.000000000","message":"Yes, it will cause feedback loops if operators do not take action, but that is the point of adding the upgrade release note, which I hope operators actually look at. I think operators are more likely to entirely unaware that the Gnocchi service project is being filtered by default, and IMHO is an undesirable behaviour *unless* you use Swift for metric storage, so disabling it by default makes sense to me.\n\n\"Ceph is better and more popular\" is an anecdote, sure, but it\u0027s an educated one as when we first evaluated using Gnocchi with Swift it was entirely unsuitable for production use, with fairly constant API errors, and it didn\u0027t even support setting custom storage policies until I added that feature upstream. Regardless, I have reworded the commit message so that it focuses on what operators should do if they are running Gnocchi using Swift instead.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"46d2ef52d7b3bcf1fdc845dc83de7830445cf8b7","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8f6ae171_0b9e1682","in_reply_to":"ae239a1b_e9abeaa2","updated":"2026-10-02 05:57:46.000000000","message":"Takashi* apologies, autocorrect.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"8e4857487ef817ebc54281cb8c7237011d334c62","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ae239a1b_e9abeaa2","in_reply_to":"c3aee6fa_205931a0","updated":"2026-10-02 05:55:59.000000000","message":"Thank you for your feedback, Takeshi. I now realise that I should have evaluated adding the warning in the original patch, as it was always my intention to propose this change.\n\nI will make a separate patchset adding the deprecation warning, and that should give operators that are using Gnocchi with Swift time to switch their deployments before this change hopefully can be merged in the 2028.1 cycle.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"}],"ceilometer/publisher/gnocchi.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"522e002720342e61b801b9c8ba9db3cc59b07a94","unresolved":true,"context_lines":[{"line_number":217,"context_line":"            # If not provided, check if filter_project or filter_domain"},{"line_number":218,"context_line":"            # have been set, and if they are, enable filtering. Otherwise"},{"line_number":219,"context_line":"            # assume filtering is not required and disable it."},{"line_number":220,"context_line":"            else (\u0027filter_project\u0027 in options or \u0027filter_domain\u0027 in options)"},{"line_number":221,"context_line":"        )"},{"line_number":222,"context_line":"        if self.enable_filter_project:"},{"line_number":223,"context_line":"            self.filter_project \u003d options.get(\u0027filter_project\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"e7240e63_d872eba9","line":220,"range":{"start_line":220,"start_character":19,"end_line":220,"end_character":33},"updated":"2026-10-02 04:29:49.000000000","message":"I\u0027d rather use only filter_project as the trigger. It\u0027s confusing that filter_domain implies filter_project\u003dservice .","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"2eea984b8832bbe65dc29e90f5e5e49b54dddc62","unresolved":false,"context_lines":[{"line_number":217,"context_line":"            # If not provided, check if filter_project or filter_domain"},{"line_number":218,"context_line":"            # have been set, and if they are, enable filtering. Otherwise"},{"line_number":219,"context_line":"            # assume filtering is not required and disable it."},{"line_number":220,"context_line":"            else (\u0027filter_project\u0027 in options or \u0027filter_domain\u0027 in options)"},{"line_number":221,"context_line":"        )"},{"line_number":222,"context_line":"        if self.enable_filter_project:"},{"line_number":223,"context_line":"            self.filter_project \u003d options.get(\u0027filter_project\u0027,"}],"source_content_type":"text/x-python","patch_set":3,"id":"0d1c14f0_5ff422aa","line":220,"range":{"start_line":220,"start_character":19,"end_line":220,"end_character":33},"in_reply_to":"e7240e63_d872eba9","updated":"2026-10-02 05:23:34.000000000","message":"Thanks, changed to only enable base on ``filter_project`` being set.","commit_id":"7d0419d8ede56537693fe109eb7178907dafe740"}]}