)]}'
{"environments/memcached-use-ips.yaml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"5dc6a9798e3e0154eb8ed4fb87aac1f99de6aebb","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"72effe94_6c0d58f7","line":9,"updated":"2021-08-26 13:48:35.000000000","message":"This seems like a massive hack","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"ef5136006d98ec67930f4377622ff6b4781615d8","unresolved":false,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"c4bc3618_b3509045","line":9,"in_reply_to":"178e0076_3b0b9205","updated":"2021-08-27 11:43:39.000000000","message":"Thanks, looks like it worked! The tempest failed [1], but the memcached configuration looks correct[1].\n\n[1] https://logserver.rdoproject.org/61/35061/3/check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035/4567479/logs/undercloud/var/log/tempest/tempest_results.xml.gz\n\n[2] https://logserver.rdoproject.org/61/35061/3/check/tripleo-ci-centos-8-ovb-3ctlr_1comp-featureset035/4567479/logs/overcloud-controller-0/var/lib/config-data/puppet-generated/nova/etc/nova/nova.conf.txt.gz","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"f100fc1f97b6676b9755c618fecee82dc2b73265","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"fad0fe03_5bd9f754","line":9,"in_reply_to":"1d73563e_44f4119c","updated":"2021-08-26 14:48:47.000000000","message":"This is not jinja but hiera/puppet syntax, this is why I couldn\u0027t just assign the memcached_node_ips list, which is a hiera variable.\n\nThe issue itself is with the tempest run being slower when FQDNs are used for memcached servers. It manifested itself with IPv6 without TLS (FS035), when external DNS is used. I think this wouldn\u0027t be an issue with TLS, since this setup would use the DNS included in FreeIPA.\n\n\ntls+ips wouldn\u0027t currently work, because the certificates for Memcached are only created for FQDNs","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"e9093620a92745f3ad16265709b1ac728d05bc20","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"b9a3e2d4_d21c1724","line":9,"in_reply_to":"271816e8_bbc7667d","updated":"2021-08-26 15:53:06.000000000","message":"\u003e Ok. The syntax threw me because I\u0027m only familar with \"%{hiera(...)}\".   Can we just do:\n\u003e \n\u003e   memcache_node_names: \"%{hiera(\u0027memcache_node_ips\u0027)}\" \n\u003e \n\u003e That would prevent issues if we don\u0027t have 3 nodes\n\nI think it would put out a string value like \"[\u00271st ip\u0027, \u00272nd ip\u0027, \u00273rd ip\u0027]\" as the value, so that won\u0027t work.","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"7e10895eeb1d6b6d68091f810875c9d2ed9e9b0d","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"d810728c_73b1854d","line":9,"in_reply_to":"72effe94_6c0d58f7","updated":"2021-08-26 14:28:53.000000000","message":"Hey Alex, I tested it here: https://review.rdoproject.org/r/c/testproject/+/35061 so it works. Do you have an alternative in mind?\n\nThe other way I was thinking of accomplishing this is to have a hiera value like use_memcached_node_ips, which would then have to be checked in all service Memcached configurations (20 files in puppet-tripleo), and then just switch that one value in this environment file.","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"342ba74a657389446717caf3dcfcacd3928011ad","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"178e0076_3b0b9205","line":9,"in_reply_to":"b9a3e2d4_d21c1724","updated":"2021-08-26 18:10:24.000000000","message":"TIL turns out it\u0027s %{alias(\u0027key\u0027)}\n\nhttps://tickets.puppetlabs.com/browse/HI-183\n\nLet\u0027s use that instead.","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"b0839627e90f2dfb336dd8bd88e91a9635e7c4ec","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"1d73563e_44f4119c","line":9,"in_reply_to":"d810728c_73b1854d","updated":"2021-08-26 14:39:04.000000000","message":"It works in this specific configration. I\u0027m concerned about the usage of jinja here and i\u0027m not even sure how it\u0027s working correctly.  This seems to be a blocker with ipv6+tls configurations and I\u0027m not certain that switching to dns name is correct. IIUC the switch to names was for tls for memcache correct?  If a customer has tls everywhere + ipv6, they will likely hit this and switching to IPs is probably not going to fly. Alternatively if tls+ips works, why did we switch to names at all?","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"cde900858f69288944d73834d3f5d32f9a77c80b","unresolved":true,"context_lines":[{"line_number":6,"context_line":"    memcached_node_names:"},{"line_number":7,"context_line":"      - \"{{memcached_node_ips.0}}\""},{"line_number":8,"context_line":"      - \"{{memcached_node_ips.1}}\""},{"line_number":9,"context_line":"      - \"{{memcached_node_ips.2}}\""}],"source_content_type":"text/x-yaml","patch_set":1,"id":"271816e8_bbc7667d","line":9,"in_reply_to":"fad0fe03_5bd9f754","updated":"2021-08-26 15:35:50.000000000","message":"Ok. The syntax threw me because I\u0027m only familar with \"%{hiera(...)}\".   Can we just do:\n\n  memcache_node_names: \"%{hiera(\u0027memcache_node_ips\u0027)}\" \n\nThat would prevent issues if we don\u0027t have 3 nodes","commit_id":"b46342cbd9e8e7856df8b6f01510435adaad071c"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"3e2cacfac8aed76eb1598f2d9df96642bf5c56ad","unresolved":true,"context_lines":[{"line_number":3,"context_line":"# external DNS."},{"line_number":4,"context_line":"parameter_defaults:"},{"line_number":5,"context_line":"  ExtraConfig:"},{"line_number":6,"context_line":"    memcached_node_names: \"%alias(\u0027memcached_node_ips\u0027)}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"a8a8c4f9_76cd13ae","line":6,"range":{"start_line":6,"start_character":27,"end_line":6,"end_character":29},"updated":"2021-08-26 20:20:32.000000000","message":"You dropped a { so %{a","commit_id":"293bf610b9714228b2c71e7ef1ca5978701c0d41"},{"author":{"_account_id":14250,"name":"Grzegorz Grasza","email":"xek@redhat.com","username":"xek"},"change_message_id":"ef5136006d98ec67930f4377622ff6b4781615d8","unresolved":false,"context_lines":[{"line_number":3,"context_line":"# external DNS."},{"line_number":4,"context_line":"parameter_defaults:"},{"line_number":5,"context_line":"  ExtraConfig:"},{"line_number":6,"context_line":"    memcached_node_names: \"%alias(\u0027memcached_node_ips\u0027)}\""}],"source_content_type":"text/x-yaml","patch_set":2,"id":"1ff26bc8_067b8505","line":6,"range":{"start_line":6,"start_character":27,"end_line":6,"end_character":29},"in_reply_to":"a8a8c4f9_76cd13ae","updated":"2021-08-27 11:43:39.000000000","message":"Done","commit_id":"293bf610b9714228b2c71e7ef1ca5978701c0d41"}]}