Skip to content

Commit db09847

Browse files
authored
Fix: Platform destroy cleans up machineconfigtemplates (#3979)
* Add machineconfigtemplates to platform destroy clean up list * Fix: fail platform destroy on unknown managed resources
1 parent c707caa commit db09847

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

pkg/cli/destroy/destroy.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ var resourceOrder = []string{
6161
"nodeproviders",
6262
"nodetypes",
6363
"osimages",
64+
"machineconfigtemplates",
6465
"sshkeys",
6566

6667
// access
@@ -155,8 +156,7 @@ func destroy(ctxWithoutTimeout context.Context, opts DeleteOptions) error {
155156

156157
unhandledResourceSet := clusterResourceSet.Difference(handledResourceSet)
157158
if unhandledResourceSet.Len() != 0 {
158-
opts.Log.Errorf("some storage.loft.sh resources are unhandled: %v. Try a newer cli version", unhandledResourceSet.UnsortedList())
159-
return err
159+
return fmt.Errorf("some storage.loft.sh resources are unhandled: %v, try a newer cli version", unhandledResourceSet.UnsortedList())
160160
}
161161

162162
for _, resourceName := range resourceOrder {

0 commit comments

Comments
 (0)