From c61b2a3975abd9d0cbd02530a2cff113c5cb8b41 Mon Sep 17 00:00:00 2001 From: Davanum Srinivas Date: Sun, 3 Mar 2024 18:49:57 -0500 Subject: [PATCH] Check length of instance name before truncating Signed-off-by: Davanum Srinivas --- test/e2e_node/remote/gce/gce_runner.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/e2e_node/remote/gce/gce_runner.go b/test/e2e_node/remote/gce/gce_runner.go index b3264c2da1d..445ddb5de5b 100644 --- a/test/e2e_node/remote/gce/gce_runner.go +++ b/test/e2e_node/remote/gce/gce_runner.go @@ -646,7 +646,10 @@ func (g *GCERunner) imageToInstanceName(imageConfig *internalGCEImage) string { // different machine types with the same image in parallel name := imageConfig.machine + "-" + imageConfig.image + "-" + uuid.New().String()[:8] // Sometimes the image is too long, we need instance names to have a max length of 63 - return name[:63] + if len(name) > 63 { + return name[:63] + } + return name } func (g *GCERunner) registerGceHostIP(host string) error {