Merge pull request #11666 from mountcount/main

chore: fix function names in comment
kubevirt · Apr 9, 2024 · 15be836 · 15be836
2 parents 9db1b13 + 9b5fd5a
commit 15be836
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/pkg/controller/controller_ref_manager.go b/pkg/controller/controller_ref_manager.go
@@ -279,7 +279,7 @@ func (m *VirtualMachineControllerRefManager) ClaimVirtualMachineInstances(vmis [
 	return claimed, utilerrors.NewAggregate(errlist)
 }
 
-// ReleaseDetachVirtualMachines removes ownership of detached VMs.
+// ReleaseDetachedVirtualMachines removes ownership of detached VMs.
 //
 // It will reconcile the following:
 //   - Release owned objects if the selector no longer matches.

diff --git a/pkg/util/util.go b/pkg/util/util.go
@@ -276,7 +276,7 @@ func CalcExpectedMemoryDumpSize(vmi *v1.VirtualMachineInstance) *resource.Quanti
 	return expectedPvcSize
 }
 
-// GenerateRandomString creates a securely generated random string using crypto/rand
+// GenerateSecureRandomString creates a securely generated random string using crypto/rand
 func GenerateSecureRandomString(n int) (string, error) {
 	ret := make([]byte, n)
 	for i := range ret {

diff --git a/pkg/virt-api/webhooks/validating-webhook/admitters/vmclone-admitter.go b/pkg/virt-api/webhooks/validating-webhook/admitters/vmclone-admitter.go
@@ -54,7 +54,7 @@ type VirtualMachineCloneAdmitter struct {
 	Client kubecli.KubevirtClient
 }
 
-// NewMigrationPolicyAdmitter creates a MigrationPolicyAdmitter
+// NewVMCloneAdmitter creates a VM Clone Admitter
 func NewVMCloneAdmitter(config *virtconfig.ClusterConfig, client kubecli.KubevirtClient) *VirtualMachineCloneAdmitter {
 	return &VirtualMachineCloneAdmitter{
 		Config: config,