)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32553,"name":"Sven Kieske","email":"sven_oss@posteo.de","username":"skieske"},"change_message_id":"c57f1665d474ecce554aeca15a562aeb1cc04b3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b663b344_5f44430d","updated":"2023-09-21 10:15:05.000000000","message":"I guess we do not need a bug or relnotes, because this is to support brand new functionality.","commit_id":"4871ca4d6b83f6d28d790722eae52de5e56becee"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"e05562a9f8a6cfe9b4ea07a40af5bc5af3057ba8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d31c9de8_592cc883","updated":"2023-09-21 11:59:25.000000000","message":"So can we merge ?","commit_id":"4871ca4d6b83f6d28d790722eae52de5e56becee"}],"docker/macros.j2":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"1c78dc1aff216c2a07b700e2d2d8e668f65d435d","unresolved":true,"context_lines":[{"line_number":61,"context_line":"{% macro install_fluent_plugins(plugins, chain\u003dFalse) -%}"},{"line_number":62,"context_line":"{% if plugins is defined and plugins|length \u003e 0 -%}"},{"line_number":63,"context_line":"    {% if not chain -%} RUN {% endif -%}"},{"line_number":64,"context_line":"    ulimit -n 1024 \u0026\u0026 td-agent-gem install {{ plugins | join(\u0027 \u0027) }}"},{"line_number":65,"context_line":"{%- endif %}"},{"line_number":66,"context_line":"{%- endmacro %}"},{"line_number":67,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"6cd64cbc_13e8b9ad","line":64,"range":{"start_line":64,"start_character":4,"end_line":64,"end_character":22},"updated":"2023-09-21 10:44:46.000000000","message":"If this is the default anyway, can we just drop this statement and work with whatever the default is?\n\nSeems this was introduced a long long time ago with no reason being traceable, so if it now works without it, that would be the simplest approach?","commit_id":"4871ca4d6b83f6d28d790722eae52de5e56becee"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"315d29d180915a827bfbbfcc0769015328e77daa","unresolved":false,"context_lines":[{"line_number":61,"context_line":"{% macro install_fluent_plugins(plugins, chain\u003dFalse) -%}"},{"line_number":62,"context_line":"{% if plugins is defined and plugins|length \u003e 0 -%}"},{"line_number":63,"context_line":"    {% if not chain -%} RUN {% endif -%}"},{"line_number":64,"context_line":"    ulimit -n 1024 \u0026\u0026 td-agent-gem install {{ plugins | join(\u0027 \u0027) }}"},{"line_number":65,"context_line":"{%- endif %}"},{"line_number":66,"context_line":"{%- endmacro %}"},{"line_number":67,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"79a2faae_74864391","line":64,"range":{"start_line":64,"start_character":4,"end_line":64,"end_character":22},"in_reply_to":"408dce61_b6ed43d9","updated":"2023-09-22 08:01:28.000000000","message":"Hmm, that\u0027s not good enough reasoning IMO, but I don\u0027t want to block this, maybe this can be checked again later.","commit_id":"4871ca4d6b83f6d28d790722eae52de5e56becee"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"bab50b1d8d579b1ead1665e120d6b31ef65fa1de","unresolved":true,"context_lines":[{"line_number":61,"context_line":"{% macro install_fluent_plugins(plugins, chain\u003dFalse) -%}"},{"line_number":62,"context_line":"{% if plugins is defined and plugins|length \u003e 0 -%}"},{"line_number":63,"context_line":"    {% if not chain -%} RUN {% endif -%}"},{"line_number":64,"context_line":"    ulimit -n 1024 \u0026\u0026 td-agent-gem install {{ plugins | join(\u0027 \u0027) }}"},{"line_number":65,"context_line":"{%- endif %}"},{"line_number":66,"context_line":"{%- endmacro %}"},{"line_number":67,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"408dce61_b6ed43d9","line":64,"range":{"start_line":64,"start_character":4,"end_line":64,"end_character":22},"in_reply_to":"6cd64cbc_13e8b9ad","updated":"2023-09-21 10:48:16.000000000","message":"It\u0027s default for podman, we were lowering that for Docker surely for a reason (the default there is a lot higher - especially on latest EL9).","commit_id":"4871ca4d6b83f6d28d790722eae52de5e56becee"}]}