Skip to content

Commit

Permalink
Add remove tctl command for AutoUpdateConfig and AutoUpdateVersion (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
vapopov committed Dec 2, 2024
1 parent 9ed545e commit 04533ec
Show file tree
Hide file tree
Showing 2 changed files with 24 additions and 0 deletions.
12 changes: 12 additions & 0 deletions tool/tctl/common/resource_command.go
Original file line number Diff line number Diff line change
Expand Up @@ -1532,6 +1532,8 @@ func (rc *ResourceCommand) Delete(ctx context.Context, client *authclient.Client
types.KindInstaller,
types.KindUIConfig,
types.KindNetworkRestrictions,
types.KindAutoUpdateConfig,
types.KindAutoUpdateVersion,
}
if !slices.Contains(singletonResources, rc.ref.Kind) && (rc.ref.Kind == "" || rc.ref.Name == "") {
return trace.BadParameter("provide a full resource name to delete, for example:\n$ tctl rm cluster/east\n")
Expand Down Expand Up @@ -1926,6 +1928,16 @@ func (rc *ResourceCommand) Delete(ctx context.Context, client *authclient.Client
return trace.Wrap(err)
}
fmt.Printf("static host user %q has been deleted\n", rc.ref.Name)
case types.KindAutoUpdateConfig:
if err := client.DeleteAutoUpdateConfig(ctx); err != nil {
return trace.Wrap(err)
}
fmt.Printf("AutoUpdateConfig has been deleted\n")
case types.KindAutoUpdateVersion:
if err := client.DeleteAutoUpdateVersion(ctx); err != nil {
return trace.Wrap(err)
}
fmt.Printf("AutoUpdateVersion has been deleted\n")
default:
return trace.BadParameter("deleting resources of type %q is not supported", rc.ref.Kind)
}
Expand Down
12 changes: 12 additions & 0 deletions tool/tctl/common/resource_command_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2318,6 +2318,12 @@ version: v1
protocmp.IgnoreFields(&headerv1.Metadata{}, "revision"),
protocmp.Transform(),
))

// Delete the resource
_, err = runResourceCommand(t, clt, []string{"rm", types.KindAutoUpdateConfig})
require.NoError(t, err)
_, err = runResourceCommand(t, clt, []string{"get", types.KindAutoUpdateConfig})
require.ErrorContains(t, err, "autoupdate_config \"autoupdate-config\" doesn't exist")
}

func testCreateAutoUpdateVersion(t *testing.T, clt *authclient.Client) {
Expand Down Expand Up @@ -2353,6 +2359,12 @@ version: v1
protocmp.IgnoreFields(&headerv1.Metadata{}, "revision"),
protocmp.Transform(),
))

// Delete the resource
_, err = runResourceCommand(t, clt, []string{"rm", types.KindAutoUpdateVersion})
require.NoError(t, err)
_, err = runResourceCommand(t, clt, []string{"get", types.KindAutoUpdateVersion})
require.ErrorContains(t, err, "autoupdate_version \"autoupdate-version\" doesn't exist")
}

func TestPluginResourceWrapper(t *testing.T) {
Expand Down

0 comments on commit 04533ec

Please sign in to comment.