)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dc82d7ffd43f280d1ad37c03a119a27dedf857df","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1b1eb254_34d4e4f4","updated":"2026-10-16 15:04:39.000000000","message":"The subsequent change includes a few TODO to move some test codes to this. Are you going to update the series for that migration ?","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"d81dfd6b40b0d5de6f9dfd713ab82f971a108da9","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"93043d58_1113faa1","in_reply_to":"1b1eb254_34d4e4f4","updated":"2026-10-17 05:35:37.000000000","message":"Now updated.","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"7b99099da50319cf85122c4c7ee332946f79bf80","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"172e0779_5e21d1bc","in_reply_to":"93043d58_1113faa1","updated":"2026-10-17 05:35:54.000000000","message":"Done","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"ae4584a31ce63e42197796ef5bc2a036aafccfe2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"99928bec_d114739a","updated":"2026-11-05 21:41:02.000000000","message":"recheck unrelated test failure","commit_id":"1f2d4007fd86027c8e18d25752b1b5f94736a13d"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"75e5ff8b4430b201063fcacebd2186d5b0d45d3c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"e61c4789_c97f643f","updated":"2026-11-03 19:03:53.000000000","message":"recheck unrelated test failure","commit_id":"1f2d4007fd86027c8e18d25752b1b5f94736a13d"}],"ceilometer/objectstore/swift.py":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dc82d7ffd43f280d1ad37c03a119a27dedf857df","unresolved":true,"context_lines":[{"line_number":34,"context_line":"               default\u003d\u0027AUTH_\u0027,"},{"line_number":35,"context_line":"               help\u003d\"Swift reseller prefix. Must be on par with \""},{"line_number":36,"context_line":"               \"reseller_prefix in proxy-server.conf.\"),"},{"line_number":37,"context_line":"    cfg.BoolOpt(\u0027swift_drop_zero_account_usage_samples\u0027,"},{"line_number":38,"context_line":"                default\u003dFalse,"},{"line_number":39,"context_line":"                help\u003d(\u0027Drop samples for Swift account usage with a volume of \u0027"},{"line_number":40,"context_line":"                      \u00270. Reduces storage requirements if zero-usage \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"1a07a417_f638371a","line":37,"range":{"start_line":37,"start_character":17,"end_line":37,"end_character":54},"updated":"2026-10-16 15:04:39.000000000","message":"This option is registered to the [swift] section so the `swift_` prefix is redundant.\n\nAlso I think ignore may provides the better expectation because drop can imply that ceilometer actually tries to remove something.\n\nBased on the two points above and with aim to shorten the name ignore_empty_accounts would be my suggestion. (this is aligned with the existing ignore_disabled_projects option)","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"d81dfd6b40b0d5de6f9dfd713ab82f971a108da9","unresolved":false,"context_lines":[{"line_number":34,"context_line":"               default\u003d\u0027AUTH_\u0027,"},{"line_number":35,"context_line":"               help\u003d\"Swift reseller prefix. Must be on par with \""},{"line_number":36,"context_line":"               \"reseller_prefix in proxy-server.conf.\"),"},{"line_number":37,"context_line":"    cfg.BoolOpt(\u0027swift_drop_zero_account_usage_samples\u0027,"},{"line_number":38,"context_line":"                default\u003dFalse,"},{"line_number":39,"context_line":"                help\u003d(\u0027Drop samples for Swift account usage with a volume of \u0027"},{"line_number":40,"context_line":"                      \u00270. Reduces storage requirements if zero-usage \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"76e51b48_d947001e","line":37,"range":{"start_line":37,"start_character":17,"end_line":37,"end_character":54},"in_reply_to":"1a07a417_f638371a","updated":"2026-10-17 05:35:37.000000000","message":"I double checked this and these opts are registered to the `[DEFAULT]` section [here](https://opendev.org/openstack/ceilometer/src/commit/6a90a8c35c51f8e8fa79388e21452b2032645622/ceilometer/opts.py#L69). There is no prefix added when they are added either, so the reseller prefix is just `[DEFAULT]/reseller_prefix`. I\u0027m not sure it\u0027s necessary to move these out into their own Swift section.\n\nI have renamed the opts as requested though, the names you suggested do look better, thanks.","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"dc82d7ffd43f280d1ad37c03a119a27dedf857df","unresolved":true,"context_lines":[{"line_number":41,"context_line":"                      \u0027accounts do not need to be tracked. Applies to the \u0027"},{"line_number":42,"context_line":"                      \u0027following meters: storage.objects, \u0027"},{"line_number":43,"context_line":"                      \u0027storage.objects.size, storage.objects.containers\u0027)),"},{"line_number":44,"context_line":"    cfg.BoolOpt(\u0027swift_drop_zero_container_usage_samples\u0027,"},{"line_number":45,"context_line":"                default\u003dFalse,"},{"line_number":46,"context_line":"                help\u003d(\u0027Drop samples for Swift container usage with a volume \u0027"},{"line_number":47,"context_line":"                      \u0027of 0. Reduces storage requirements if zero-usage \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"5393655e_87821357","line":44,"range":{"start_line":44,"start_character":17,"end_line":44,"end_character":56},"updated":"2026-10-16 15:04:39.000000000","message":"ditto. ignore_empty_containers","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"},{"author":{"_account_id":36393,"name":"Callum Dickinson","email":"callum.dickinson@catalystcloud.nz","username":"Callum027","status":"Catalyst Cloud"},"change_message_id":"d81dfd6b40b0d5de6f9dfd713ab82f971a108da9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"                      \u0027accounts do not need to be tracked. Applies to the \u0027"},{"line_number":42,"context_line":"                      \u0027following meters: storage.objects, \u0027"},{"line_number":43,"context_line":"                      \u0027storage.objects.size, storage.objects.containers\u0027)),"},{"line_number":44,"context_line":"    cfg.BoolOpt(\u0027swift_drop_zero_container_usage_samples\u0027,"},{"line_number":45,"context_line":"                default\u003dFalse,"},{"line_number":46,"context_line":"                help\u003d(\u0027Drop samples for Swift container usage with a volume \u0027"},{"line_number":47,"context_line":"                      \u0027of 0. Reduces storage requirements if zero-usage \u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"2c953539_1718f5d2","line":44,"range":{"start_line":44,"start_character":17,"end_line":44,"end_character":56},"in_reply_to":"5393655e_87821357","updated":"2026-10-17 05:35:37.000000000","message":"Renamed, thanks.","commit_id":"8c0f9d94d142b4415418c3e6815dcf455507596f"}]}
