Remove redundant major version number from instance name

almalinux and rocky use image names that include the major version
number twice. This commit strips it:

almalinux-8-8.10 → almalinux-8.10
almalinux-9-9.5  → almalinux-9.5
rocky-8-8.10     → rocky-8.10
rocky-9-9.5      → rocky-9.5

Signed-off-by: Jan Dubois <jan.dubois@suse.com>

Author: jandubois

pkg/limatmpl/locator.go

@@ -217,6 +217,17 @@ func InstNameFromImageURL(locator, imageArch string) string {
 	name = regexp.MustCompile(`[-_.]20\d{6}([-_.]\d+)?\b`).ReplaceAllString(name, "")
 	// Normalize archlinux name
 	name = regexp.MustCompile(`^arch\b`).ReplaceAllString(name, "archlinux")
+	// Remove redundant major version, e.g. rocky-8-8.10 becomes rocky-8.10
+	// unfortunately regexp doesn't support back references, so we have to check manually of both numbers are the same
+	re := regexp.MustCompile(`-(\d+)-(\d+)\.`)
+	name = re.ReplaceAllStringFunc(name, func(match string) string {
+		submatch := re.FindStringSubmatch(match)
+		if submatch[1] == submatch[2] {
+			// Replace -X-X. with -X.
+			return "-" + submatch[1] + "."
+		}
+		return match
+	})
 	return name
 }
 

pkg/limatmpl/locator_test.go

@@ -52,4 +52,12 @@ func TestInstNameFromImageURL(t *testing.T) {
 		name := limatmpl.InstNameFromImageURL(image, arch)
 		assert.Equal(t, name, "linux")
 	})
+	t.Run("removes redundant major version", func(t *testing.T) {
+		name := limatmpl.InstNameFromImageURL("rocky-8-8.10.raw", "unknown")
+		assert.Equal(t, name, "rocky-8.10")
+	})
+	t.Run("don't remove non-redundant major version", func(t *testing.T) {
+		name := limatmpl.InstNameFromImageURL("rocky-8-9.10.raw", "unknown")
+		assert.Equal(t, name, "rocky-8-9.10")
+	})
 }