chore(version)!: remove 'hotpatch' versioning (#14510)

Hotpatch versions are unused.

This change removes all references to hotpatch versions from the version
package.

Author: olix0r

pkg/version/channels_test.go

@@ -11,7 +11,6 @@ import (
 )
 
 func TestGetLatestVersions(t *testing.T) {
-	four := int64(4)
 	testCases := []struct {
 		name   string
 		resp   interface{}
@@ -21,18 +20,16 @@ func TestGetLatestVersions(t *testing.T) {
 		{
 			"valid response",
 			map[string]string{
-				"foo":         "foo-1.2.3",
-				"fooHotpatch": "foo-1.2.3-4",
-				"stable":      "stable-2.1.0",
-				"edge":        "edge-2.1.0",
+				"foo":    "foo-1.2.3",
+				"stable": "stable-2.1.0",
+				"edge":   "edge-2.1.0",
 			},
 			nil,
 			Channels{
 				[]channelVersion{
-					{"foo", "1.2.3", nil, "foo-1.2.3"},
-					{"foo", "1.2.3", &four, "foo-1.2.3-4"},
-					{"stable", "2.1.0", nil, "stable-2.1.0"},
-					{"edge", "2.1.0", nil, "edge-2.1.0"},
+					{"foo", "1.2.3", "foo-1.2.3"},
+					{"stable", "2.1.0", "stable-2.1.0"},
+					{"edge", "2.1.0", "edge-2.1.0"},
 				},
 			},
 		},
@@ -102,7 +99,7 @@ func channelsEqual(c1, c2 Channels) bool {
 	for _, cv1 := range c1.array {
 		found := false
 		for _, cv2 := range c2.array {
-			if cv1.channel == cv2.channel && cv1.version == cv2.version && cv1.hotpatchEqual(cv2) {
+			if cv1.channel == cv2.channel && cv1.version == cv2.version {
 				found = true
 				break
 			}
@@ -116,13 +113,12 @@ func channelsEqual(c1, c2 Channels) bool {
 }
 
 func TestChannelsMatch(t *testing.T) {
-	four := int64(4)
 	channels := Channels{
 		[]channelVersion{
-			{"stable", "2.1.0", nil, "stable-2.1.0"},
-			{"foo", "1.2.3", nil, "foo-1.2.3"},
-			{"foo", "1.2.3", &four, "foo-1.2.3-4"},
-			{"version", "3.2.1", nil, "version-3.2.1"},
+			{"stable", "2.1.0", "stable-2.1.0"},
+			{"foo", "1.2.3", "foo-1.2.3"},
+			{"foo", "1.2.3", "foo-1.2.3-4"},
+			{"version", "3.2.1", "version-3.2.1"},
 		},
 	}
 
@@ -141,8 +137,7 @@ func TestChannelsMatch(t *testing.T) {
 			fmt.Errorf("is running version 1.2.2 but the latest foo version is 1.2.3"),
 		},
 		{
-			"foo-1.2.3-3",
-			fmt.Errorf("is running version 1.2.3-3 but the latest foo version is 1.2.3-4"),
+			"foo-1.2.3-3", nil,
 		},
 		{
 			"unsupportedChannel-1.2.3",

pkg/version/channelversion.go

@@ -2,7 +2,6 @@ package version
 
 import (
 	"fmt"
-	"strconv"
 	"strings"
 )
 
@@ -11,53 +10,22 @@ import (
 type channelVersion struct {
 	channel  string
 	version  string
-	hotpatch *int64
 	original string
 }
 
-// hotpatchSuffix is the suffix applied to channel names to indicate that the
-// version string includes a hotpatch number (e.g. dev-0.1.2-3)
-const hotpatchSuffix = "Hotpatch"
-
 func (cv channelVersion) String() string {
 	return cv.original
 }
 
-// updateChannel returns the channel name to check for updates. if there's no
-// hotpatch number set, then it returns the channel name itself. otherwise it
-// returns the channel name suffixed with "Hotpatch" to indicate that a separate
-// update channel should be used.
+// updateChannel returns the channel name to check for updates, returning the
+// channel name.
 func (cv channelVersion) updateChannel() string {
-	if cv.hotpatch != nil {
-		return cv.channel + hotpatchSuffix
-	}
 	return cv.channel
 }
 
-// versionWithHotpatch returns the version string, suffixed with the hotpatch
-// number if it exists.
-func (cv channelVersion) versionWithHotpatch() string {
-	if cv.hotpatch == nil {
-		return cv.version
-	}
-	return fmt.Sprintf("%s-%d", cv.version, *cv.hotpatch)
-}
-
-func (cv channelVersion) hotpatchEqual(other channelVersion) bool {
-	if cv.hotpatch == nil && other.hotpatch == nil {
-		return true
-	}
-	if cv.hotpatch == nil || other.hotpatch == nil {
-		return false
-	}
-	return *cv.hotpatch == *other.hotpatch
-}
-
 // parseChannelVersion parses a build string into a channelVersion struct. it
 // expects the channel and version to be separated by a hyphen (e.g. dev-0.1.2).
-// the version may additionally include a hotpatch number, which is separated
-// from the base version by another hyphen (e.g. dev-0.1.2-3). if the version is
-// suffixed with any other non-numeric build info strings (e.g. dev-0.1.2-foo),
+// if the version is suffixed with any other non-numeric build info strings (e.g. dev-0.1.2-foo),
 // those strings are ignored.
 func parseChannelVersion(cv string) (channelVersion, error) {
 	parts := strings.Split(cv, "-")
@@ -67,16 +35,8 @@ func parseChannelVersion(cv string) (channelVersion, error) {
 
 	channel := parts[0]
 	version := parts[1]
-	var hotpatch *int64
-
-	for _, part := range parts[2:] {
-		if i, err := strconv.ParseInt(part, 10, 64); err == nil {
-			hotpatch = &i
-			break
-		}
-	}
 
-	return channelVersion{channel, version, hotpatch, cv}, nil
+	return channelVersion{channel, version, cv}, nil
 }
 
 // IsReleaseChannel returns true if the channel of the version is "edge" or

pkg/version/version.go

@@ -61,9 +61,9 @@ func match(expectedVersion, actualVersion string) error {
 			actual, expected)
 	}
 
-	if actual.version != expected.version || !actual.hotpatchEqual(expected) {
+	if actual.version != expected.version {
 		return fmt.Errorf("is running version %s but the latest %s version is %s",
-			actual.versionWithHotpatch(), actual.channel, expected.versionWithHotpatch())
+			actual.version, actual.channel, expected.version)
 	}
 
 	return nil

pkg/version/version_test.go

@@ -27,16 +27,6 @@ func TestMatch(t *testing.T) {
 			expected: "dev-0.1.2-bar",
 			actual:   "dev-0.1.2-baz",
 		},
-		{
-			name:     "up-to-date with hotpatch",
-			expected: "dev-0.1.2-3",
-			actual:   "dev-0.1.2-3",
-		},
-		{
-			name:     "up-to-date with hotpatch and different build info",
-			expected: "dev-0.1.2-3-bar",
-			actual:   "dev-0.1.2-3-baz",
-		},
 		{
 			name:     "not up-to-date",
 			expected: "dev-0.1.2",
@@ -49,12 +39,6 @@ func TestMatch(t *testing.T) {
 			actual:   "dev-0.1.1-bar",
 			err:      errors.New("is running version 0.1.1 but the latest dev version is 0.1.2"),
 		},
-		{
-			name:     "not up-to-date with hotpatch",
-			expected: "dev-0.1.2-3",
-			actual:   "dev-0.1.2-2",
-			err:      errors.New("is running version 0.1.2-2 but the latest dev version is 0.1.2-3"),
-		},
 		{
 			name:     "mismatched channels",
 			expected: "dev-0.1.2",