From 966e1d8838b00c8af1205c42275cc8ce3357fa62 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luis=20P=C3=A9rez?= Date: Tue, 17 Sep 2024 15:38:24 -0400 Subject: [PATCH] Update regular filename reference in autotune files (#3961) We've generally decided that we'll be leaving files as eks-*.yaml - and while there's some additional steps that need to be taken, those will need to happen once we've gotten rid of all of the kubernetes-*.yaml files. As we still have some unmigrated operators, that means we still need to wait. However, we can go ahead and update the comment in the autotune files since those have recently caused some confusion (and assume that anyone looking at autotune files is looking at them for non-operator services) --- paasta_tools/config_utils.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/paasta_tools/config_utils.py b/paasta_tools/config_utils.py index 05aa027ed7..926df8d2c6 100644 --- a/paasta_tools/config_utils.py +++ b/paasta_tools/config_utils.py @@ -70,7 +70,14 @@ def write_auto_config_data( if comment: content = ( yaml.round_trip_load( - comment.format(regular_filename=f"{service}/{extra_info}.yaml") + comment.format( + # this is a bit of a hack, but we've decided to not rename files back to kubernetes-* + # files. while we still need to update things to reference the eks files directly, there's + # still a couple of places where we still need kubernetes-* files (e.g., unmigrated operators) + # so for now let's just assume that autotuned things will always actually have their human-managed + # config in eks-* files + regular_filename=f"{service}/{extra_info.replace('kubernetes-', 'eks-')}.yaml", + ) ) if comment else {}