Replace enum type for ToolsMode to string

api/proto/teleport/autoupdate/v1/autoupdate.proto

@@ -41,17 +41,7 @@ message AutoUpdateConfigSpec {
 // AutoUpdateConfigSpecTools encodes the parameters for client tools auto updates.
 message AutoUpdateConfigSpecTools {
   // Mode defines state of the client tools auto update.
-  ToolsUpdateMode mode = 1;
-}
-
-// ToolsMode type for client tools enabling state.
-enum ToolsUpdateMode {
-  // TOOLS_UPDATE_MODE_UNSPECIFIED is undefined mode.
-  TOOLS_UPDATE_MODE_UNSPECIFIED = 0;
-  // TOOLS_UPDATE_MODE_ENABLED client tools auto update is enabled.
-  TOOLS_UPDATE_MODE_ENABLED = 1;
-  // TOOLS_UPDATE_MODE_DISABLED client tools auto update is disabled.
-  TOOLS_UPDATE_MODE_DISABLED = 2;
+  string mode = 1;
 }
 
 // AutoUpdateVersion is a resource singleton with version required for

api/types/autoupdate/config.go

@@ -26,6 +26,13 @@ import (
 	"github.com/gravitational/teleport/api/types"
 )
 
+const (
+	// ToolsUpdateModeEnabled enables client tools automatic updates.
+	ToolsUpdateModeEnabled = "enabled"
+	// ToolsUpdateModeDisabled disables client tools automatic updates.
+	ToolsUpdateModeDisabled = "disabled"
+)
+
 // NewAutoUpdateConfig creates a new auto update configuration resource.
 func NewAutoUpdateConfig(spec *autoupdate.AutoUpdateConfigSpec) (*autoupdate.AutoUpdateConfig, error) {
 	config := &autoupdate.AutoUpdateConfig{
@@ -58,18 +65,11 @@ func ValidateAutoUpdateConfig(c *autoupdate.AutoUpdateConfig) error {
 	if c.Spec == nil {
 		return trace.BadParameter("Spec is nil")
 	}
+	if c.Spec.Tools != nil {
+		if c.Spec.Tools.Mode != ToolsUpdateModeDisabled && c.Spec.Tools.Mode != ToolsUpdateModeEnabled {
+			return trace.BadParameter("ToolsMode is not valid")
+		}
+	}
 
 	return nil
 }
-
-// ToolsModeToAPI transforms client tools mode enum to API string representation.
-func ToolsModeToAPI(mode autoupdate.ToolsUpdateMode) string {
-	switch mode {
-	case autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_ENABLED:
-		return "enabled"
-	case autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_DISABLED:
-		return "disabled"
-	default:
-		return ""
-	}
-}

api/types/autoupdate/config_test.go

@@ -42,7 +42,7 @@ func TestNewAutoUpdateConfig(t *testing.T) {
 			name: "success tools autoupdate disabled",
 			spec: &autoupdate.AutoUpdateConfigSpec{
 				Tools: &autoupdate.AutoUpdateConfigSpecTools{
-					Mode: autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_DISABLED,
+					Mode: ToolsUpdateModeDisabled,
 				},
 			},
 			assertErr: func(t *testing.T, err error, a ...any) {
@@ -56,7 +56,7 @@ func TestNewAutoUpdateConfig(t *testing.T) {
 				},
 				Spec: &autoupdate.AutoUpdateConfigSpec{
 					Tools: &autoupdate.AutoUpdateConfigSpecTools{
-						Mode: autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_DISABLED,
+						Mode: ToolsUpdateModeDisabled,
 					},
 				},
 			},
@@ -65,7 +65,7 @@ func TestNewAutoUpdateConfig(t *testing.T) {
 			name: "success tools autoupdate enabled",
 			spec: &autoupdate.AutoUpdateConfigSpec{
 				Tools: &autoupdate.AutoUpdateConfigSpecTools{
-					Mode: autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_ENABLED,
+					Mode: ToolsUpdateModeEnabled,
 				},
 			},
 			assertErr: func(t *testing.T, err error, a ...any) {
@@ -79,7 +79,7 @@ func TestNewAutoUpdateConfig(t *testing.T) {
 				},
 				Spec: &autoupdate.AutoUpdateConfigSpec{
 					Tools: &autoupdate.AutoUpdateConfigSpecTools{
-						Mode: autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_ENABLED,
+						Mode: ToolsUpdateModeEnabled,
 					},
 				},
 			},
@@ -91,6 +91,17 @@ func TestNewAutoUpdateConfig(t *testing.T) {
 				require.ErrorContains(t, err, "Spec is nil")
 			},
 		},
+		{
+			name: "invalid tools mode",
+			spec: &autoupdate.AutoUpdateConfigSpec{
+				Tools: &autoupdate.AutoUpdateConfigSpecTools{
+					Mode: "invalid-mode",
+				},
+			},
+			assertErr: func(t *testing.T, err error, a ...any) {
+				require.ErrorContains(t, err, "ToolsMode is not valid")
+			},
+		},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {

lib/cache/cache_test.go

@@ -3982,7 +3982,7 @@ func newAutoUpdateConfig(t *testing.T) *autoupdate.AutoUpdateConfig {
 
 	r, err := update.NewAutoUpdateConfig(&autoupdate.AutoUpdateConfigSpec{
 		Tools: &autoupdate.AutoUpdateConfigSpecTools{
-			Mode: autoupdate.ToolsUpdateMode_TOOLS_UPDATE_MODE_ENABLED,
+			Mode: update.ToolsUpdateModeEnabled,
 		},
 	})
 	require.NoError(t, err)