)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":30073,"name":"Brendan Shephard","email":"bshephar@bne-home.net","username":"bshephar"},"change_message_id":"42886a3f6caf2dc282f579e4cca582f249d7ca82","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"868029bd_800437e4","updated":"2022-02-07 04:59:40.000000000","message":"Is this just a temporary change? Generally, I wouldn\u0027t agree with hard coding paths since it will become some technical debt that can bite us later. But if it\u0027s just temporary while we move between versions, I guess it\u0027s ok.","commit_id":"d164e9f2f22ad1266a8862326fbca01e03b0e1d2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"9b75ca6ae4eb0c826ab595f169da01b371c0499a","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ee1a08a2_8fb4b49d","in_reply_to":"6922c448_b4bff8ef","updated":"2022-02-07 05:47:28.000000000","message":"Oh, wait. It seems the current resource agent can deal with both of old path and new path. So this only affects a deployment with standalone redis.","commit_id":"d164e9f2f22ad1266a8862326fbca01e03b0e1d2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"e3047672b382bd3864f648f88efc7cc5d22eaa91","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"6922c448_b4bff8ef","in_reply_to":"868029bd_800437e4","updated":"2022-02-07 05:34:39.000000000","message":"We already hard-code config path in the command to start redis[1] and this is to ensure we use the consistent path.\n\nhttps://github.com/openstack/tripleo-heat-templates/blob/557a392c9a38ad9e139dc9e8d932f587efd400ba/deployment/database/redis-container-puppet.yaml#L134\n\nWe are updating the path in CentOS9 following the change in its package here, and we need this not to be affected by this change.\n\nhttps://github.com/voxpupuli/puppet-redis/pull/434\n\nIdeally we can migrate to the new path but we still support CentOS8 still using the old path, so can\u0027t do this until we drop CentOS8 support.\n\nI\u0027ll add a comment in code to document TODO.","commit_id":"d164e9f2f22ad1266a8862326fbca01e03b0e1d2"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"6e8d194cf974c07c0e5495e05614dc0bc9a170ee","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"b87b3c8b_242a1b69","updated":"2022-02-14 16:16:19.000000000","message":"recheck","commit_id":"e93b454d68dd97e9471e3324c751b5436574e1fc"},{"author":{"_account_id":6681,"name":"Brent Eagles","email":"beagles@redhat.com","username":"beagles"},"change_message_id":"8c343373124dec175ee6b5cacedf3a6c150c5e3a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"fb5c368a_70828296","updated":"2022-02-14 14:39:24.000000000","message":"waiting on CI results I...","commit_id":"e93b454d68dd97e9471e3324c751b5436574e1fc"}]}