)]}'
{"environments/ssl/enable-memcached-tls.yaml":[{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"e663a51c939d293ecb052dfb2405cb4613917e3b","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  MemcachedPort: 11212"},{"line_number":8,"context_line":"  ExtraConfig:"},{"line_number":9,"context_line":"    memcached_port: 11212"},{"line_number":10,"context_line":"    memcached_authtoken_port: 11211"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"154037d3_6f54b669","side":"PARENT","line":10,"updated":"2021-06-17 12:26:14.000000000","message":"This is still needed, since the keystone authtoken middleware doesn\u0027t support TLS","commit_id":"1150081e83e8513165807167ab9381d274cddbda"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"01b894122b4b63126b18d916dc8aead4ec94ccea","unresolved":true,"context_lines":[{"line_number":7,"context_line":"  MemcachedPort: 11212"},{"line_number":8,"context_line":"  ExtraConfig:"},{"line_number":9,"context_line":"    memcached_port: 11212"},{"line_number":10,"context_line":"    memcached_authtoken_port: 11211"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"8be0f3cb_c09fa23c","side":"PARENT","line":10,"in_reply_to":"154037d3_6f54b669","updated":"2021-06-17 12:39:41.000000000","message":"If I read the current implementation in keystone-container-puppet.yaml, then the same parameters are set automatically when MemcachedTLS: true and MemcachedPort: 11212  .\n\nI don\u0027t see any reason why we should override that implementation by setting ExtraConfig.\n\nMy initial guess was that this was required to propagate these two parameters to all roles but is there any reason why we need this redundant definitions ?","commit_id":"1150081e83e8513165807167ab9381d274cddbda"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"78bcb559144c2253d08308cf45774fe228d8b0e5","unresolved":false,"context_lines":[{"line_number":7,"context_line":"  MemcachedPort: 11212"},{"line_number":8,"context_line":"  ExtraConfig:"},{"line_number":9,"context_line":"    memcached_port: 11212"},{"line_number":10,"context_line":"    memcached_authtoken_port: 11211"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1b0e4b01_eb5b6f88","side":"PARENT","line":10,"in_reply_to":"8be0f3cb_c09fa23c","updated":"2021-06-17 12:53:01.000000000","message":"you\u0027re right, I think I didn\u0027t get enough sleep last night","commit_id":"1150081e83e8513165807167ab9381d274cddbda"}]}