Merge pull request kubernetes#84836 from yuxiaobo96/k8s-checks

k8s-ci-robot · web-flow · commit 46472773cbc4 · 2019-11-06T12:21:11.000-08:00
Correct spelling mistakes
diff --git a/build/common.sh b/build/common.sh
@@ -306,7 +306,7 @@ function kube::build::has_ip() {
 # Detect if a specific image exists
 #
 # $1 - image repo name
-# #2 - image tag
+# $2 - image tag
 function kube::build::docker_image_exists() {
   [[ -n $1 && -n $2 ]] || {
     kube::log::error "Internal error. Image not specified in docker_image_exists."
diff --git a/pkg/api/testing/unstructured_test.go b/pkg/api/testing/unstructured_test.go
@@ -52,7 +52,7 @@ func doRoundTrip(t *testing.T, internalVersion schema.GroupVersion, externalVers
 	fuzzer.FuzzerFor(FuzzerFuncs, rand.NewSource(seed), legacyscheme.Codecs).
 		// We are explicitly overwriting custom fuzzing functions, to ensure
 		// that InitContainers and their statuses are not generated. This is
-		// because in thise test we are simply doing json operations, in which
+		// because in this test we are simply doing json operations, in which
 		// those disappear.
 		Funcs(
 			func(s *api.PodSpec, c fuzz.Continue) {
diff --git a/pkg/controller/certificates/authority/policies.go b/pkg/controller/certificates/authority/policies.go
@@ -47,7 +47,7 @@ type PermissiveSigningPolicy struct {
 	// TTL is the certificate TTL. It's used to calculate the NotAfter value of
 	// the certificate.
 	TTL time.Duration
-	// Usages are the allowed usages of a certficate.
+	// Usages are the allowed usages of a certificate.
 	Usages []capi.KeyUsage
 }
 
diff --git a/pkg/controller/daemon/daemon_controller.go b/pkg/controller/daemon/daemon_controller.go
@@ -893,7 +893,7 @@ func (dsc *DaemonSetsController) manage(ds *apps.DaemonSet, nodeList []*v1.Node,
 }
 
 // syncNodes deletes given pods and creates new daemon set pods on the given nodes
-// returns slice with erros if any
+// returns slice with errors if any
 func (dsc *DaemonSetsController) syncNodes(ds *apps.DaemonSet, podsToDelete, nodesNeedingDaemonPods []string, hash string) error {
 	// We need to set expectations before creating/deleting pods to avoid race conditions.
 	dsKey, err := controller.KeyFunc(ds)
diff --git a/pkg/controller/nodeipam/ipam/range_allocator.go b/pkg/controller/nodeipam/ipam/range_allocator.go
@@ -23,7 +23,7 @@ import (
 
 	"k8s.io/klog"
 
-	"k8s.io/api/core/v1"
+	v1 "k8s.io/api/core/v1"
 	apierrors "k8s.io/apimachinery/pkg/api/errors"
 	"k8s.io/apimachinery/pkg/types"
 	utilruntime "k8s.io/apimachinery/pkg/util/runtime"
@@ -61,7 +61,7 @@ type rangeAllocator struct {
 	// This increases a throughput of CIDR assignment by not blocking on long operations.
 	nodeCIDRUpdateChannel chan nodeReservedCIDRs
 	recorder              record.EventRecorder
-	// Keep a set of nodes that are currectly being processed to avoid races in CIDR allocation
+	// Keep a set of nodes that are currently being processed to avoid races in CIDR allocation
 	lock              sync.Mutex
 	nodesInProcessing sets.String
 }
diff --git a/pkg/controller/nodelifecycle/scheduler/taint_manager.go b/pkg/controller/nodelifecycle/scheduler/taint_manager.go
@@ -42,7 +42,7 @@ import (
 
 const (
 	// TODO (k82cn): Figure out a reasonable number of workers/channels and propagate
-	// the number of workers up making it a paramater of Run() function.
+	// the number of workers up making it a parameter of Run() function.
 
 	// NodeUpdateChannelSize defines the size of channel for node update events.
 	NodeUpdateChannelSize = 10
diff --git a/pkg/controller/nodelifecycle/scheduler/taint_manager_test.go b/pkg/controller/nodelifecycle/scheduler/taint_manager_test.go
@@ -222,7 +222,7 @@ func TestCreatePod(t *testing.T) {
 			}
 		}
 		if podDeleted != item.expectDelete {
-			t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
+			t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
 		}
 		close(stopCh)
 	}
@@ -319,7 +319,7 @@ func TestUpdatePod(t *testing.T) {
 			}
 		}
 		if podDeleted != item.expectDelete {
-			t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
+			t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
 		}
 		close(stopCh)
 	}
@@ -375,7 +375,7 @@ func TestCreateNode(t *testing.T) {
 			}
 		}
 		if podDeleted != item.expectDelete {
-			t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
+			t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
 		}
 		close(stopCh)
 	}
@@ -499,7 +499,7 @@ func TestUpdateNode(t *testing.T) {
 			}
 		}
 		if podDeleted != item.expectDelete {
-			t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
+			t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)
 		}
 		close(stopCh)
 	}
diff --git a/pkg/controller/podautoscaler/horizontal_test.go b/pkg/controller/podautoscaler/horizontal_test.go
@@ -348,7 +348,7 @@ func (tc *testCase) prepareTestClient(t *testing.T) (*fake.Clientset, *metricsfa
 			if err := json.Unmarshal([]byte(obj.ObjectMeta.Annotations[autoscaling.HorizontalPodAutoscalerConditionsAnnotation]), &actualConditions); err != nil {
 				return true, nil, err
 			}
-			// TODO: it's ok not to sort these becaues statusOk
+			// TODO: it's ok not to sort these because statusOk
 			// contains all the conditions, so we'll never be appending.
 			// Default to statusOk when missing any specific conditions
 			if tc.expectedConditions == nil {
diff --git a/pkg/controller/replicaset/replica_set_test.go b/pkg/controller/replicaset/replica_set_test.go
@@ -1081,7 +1081,7 @@ func TestDeleteControllerAndExpectations(t *testing.T) {
 	manager.syncReplicaSet(GetKey(rs, t))
 
 	if _, exists, err = manager.expectations.GetExpectations(rsKey); exists {
-		t.Errorf("Found expectaions, expected none since the ReplicaSet has been deleted.")
+		t.Errorf("Found expectations, expected none since the ReplicaSet has been deleted.")
 	}
 
 	// This should have no effect, since we've deleted the ReplicaSet.
diff --git a/pkg/controller/service/controller.go b/pkg/controller/service/controller.go
@@ -81,7 +81,7 @@ const (
 
 	// serviceLoadBalancerFinalizerFeature is the feature gate name that
 	// enables Finalizer Protection for Service LoadBalancers.
-	// orginated from: https://github.com/kubernetes/kubernetes/blob/28e800245e/pkg/features/kube_features.go#L433
+	// originated from: https://github.com/kubernetes/kubernetes/blob/28e800245e/pkg/features/kube_features.go#L433
 	serviceLoadBalancerFinalizerFeature = "ServiceLoadBalancerFinalizer"
 
 	// legacyNodeRoleBehaviro is the feature gate name that enables legacy
diff --git a/pkg/controller/statefulset/stateful_set_utils_test.go b/pkg/controller/statefulset/stateful_set_utils_test.go
@@ -57,7 +57,7 @@ func TestIsMemberOf(t *testing.T) {
 	set2.Name = "foo2"
 	pod := newStatefulSetPod(set, 1)
 	if !isMemberOf(set, pod) {
-		t.Error("isMemberOf retruned false negative")
+		t.Error("isMemberOf returned false negative")
 	}
 	if isMemberOf(set2, pod) {
 		t.Error("isMemberOf returned false positive")
@@ -90,7 +90,7 @@ func TestStorageMatches(t *testing.T) {
 	set := newStatefulSet(3)
 	pod := newStatefulSetPod(set, 1)
 	if !storageMatches(set, pod) {
-		t.Error("Newly created Pod has a invalid stroage")
+		t.Error("Newly created Pod has a invalid storage")
 	}
 	pod.Spec.Volumes = nil
 	if storageMatches(set, pod) {
@@ -144,7 +144,7 @@ func TestUpdateStorage(t *testing.T) {
 	set := newStatefulSet(3)
 	pod := newStatefulSetPod(set, 1)
 	if !storageMatches(set, pod) {
-		t.Error("Newly created Pod has a invalid stroage")
+		t.Error("Newly created Pod has a invalid storage")
 	}
 	pod.Spec.Volumes = nil
 	if storageMatches(set, pod) {
diff --git a/pkg/controller/util/node/controller_utils.go b/pkg/controller/util/node/controller_utils.go
@@ -56,7 +56,7 @@ func DeletePods(kubeClient clientset.Interface, pods []*v1.Pod, recorder record.
 			continue
 		}
 
-		// Pod will be modified, so making copy is requiered.
+		// Pod will be modified, so making copy is required.
 		pod := pods[i].DeepCopy()
 		// Set reason and message in the pod object.
 		if _, err := SetPodTerminationReason(kubeClient, pod, nodeName); err != nil {
@@ -122,7 +122,7 @@ func MarkPodsNotReady(kubeClient clientset.Interface, pods []*v1.Pod, nodeName s
 			continue
 		}
 
-		// Pod will be modified, so making copy is requiered.
+		// Pod will be modified, so making copy is required.
 		pod := pods[i].DeepCopy()
 		for _, cond := range pod.Status.Conditions {
 			if cond.Type == v1.PodReady {
diff --git a/pkg/controller/volume/attachdetach/reconciler/reconciler.go b/pkg/controller/volume/attachdetach/reconciler/reconciler.go
@@ -219,7 +219,7 @@ func (rc *reconciler) reconcile() {
 				continue
 			}
 
-			// Trigger detach volume which requires verifing safe to detach step
+			// Trigger detach volume which requires verifying safe to detach step
 			// If timeout is true, skip verifySafeToDetach check
 			klog.V(5).Infof(attachedVolume.GenerateMsgDetailed("Starting attacherDetacher.DetachVolume", ""))
 			verifySafeToDetach := !timeout
diff --git a/pkg/controller/volume/persistentvolume/framework_test.go b/pkg/controller/volume/persistentvolume/framework_test.go
@@ -486,7 +486,7 @@ var (
 
 // wrapTestWithPluginCalls returns a testCall that:
 // - configures controller with a volume plugin that implements recycler,
-//   deleter and provisioner. The plugin retunrs provided errors when a volume
+//   deleter and provisioner. The plugin returns provided errors when a volume
 //   is deleted, recycled or provisioned.
 // - calls given testCall
 func wrapTestWithPluginCalls(expectedRecycleCalls, expectedDeleteCalls []error, expectedProvisionCalls []provisionCall, toWrap testCall) testCall {
diff --git a/pkg/controller/volume/persistentvolume/index_test.go b/pkg/controller/volume/persistentvolume/index_test.go
@@ -1093,7 +1093,7 @@ func TestVolumeModeCheck(t *testing.T) {
 			pvc:                makeVolumeModePVC("8G", &filesystemMode, nil),
 			enableBlock:        true,
 		},
-		"feature enabled - pvc nil and pv filesytem": {
+		"feature enabled - pvc nil and pv filesystem": {
 			isExpectedMismatch: false,
 			vol:                createVolumeModeFilesystemTestVolume(),
 			pvc:                makeVolumeModePVC("8G", nil, nil),
diff --git a/pkg/controller/volume/persistentvolume/metrics/metrics.go b/pkg/controller/volume/persistentvolume/metrics/metrics.go
@@ -98,7 +98,7 @@ var (
 	volumeOperationErrorsMetric = metrics.NewCounterVec(
 		&metrics.CounterOpts{
 			Name:           "volume_operation_total_errors",
-			Help:           "Total volume operation erros",
+			Help:           "Total volume operation errors",
 			StabilityLevel: metrics.ALPHA,
 		},
 		[]string{"plugin_name", "operation_name"})
diff --git a/pkg/controller/volume/persistentvolume/pv_controller.go b/pkg/controller/volume/persistentvolume/pv_controller.go
@@ -206,7 +206,7 @@ type PersistentVolumeController struct {
 
 	// operationTimestamps caches start timestamp of operations
 	// (currently provision + binding/deletion) for metric recording.
-	// Detailed lifecyle/key for each operation
+	// Detailed lifecycle/key for each operation
 	// 1. provision + binding
 	//     key:        claimKey
 	//     start time: user has NOT provide any volume ref in the claim AND
@@ -1071,7 +1071,7 @@ func (ctrl *PersistentVolumeController) recycleVolumeOperation(volume *v1.Persis
 	}
 
 	// Verify the claim is in cache: if so, then it is a different PVC with the same name
-	// since the volume is known to be released at this moment. Ths new (cached) PVC must use
+	// since the volume is known to be released at this moment. The new (cached) PVC must use
 	// a different PV -- we checked that the PV is unused in isVolumeReleased.
 	// So the old PV is safe to be recycled.
 	claimName := claimrefToClaimKey(volume.Spec.ClaimRef)
diff --git a/pkg/kubelet/certificate/bootstrap/bootstrap.go b/pkg/kubelet/certificate/bootstrap/bootstrap.go
@@ -356,7 +356,7 @@ func requestNodeCertificate(client certificatesv1beta1.CertificateSigningRequest
 }
 
 // This digest should include all the relevant pieces of the CSR we care about.
-// We can't direcly hash the serialized CSR because of random padding that we
+// We can't directly hash the serialized CSR because of random padding that we
 // regenerate every loop and we include usages which are not contained in the
 // CSR. This needs to be kept up to date as we add new fields to the node
 // certificates and with ensureCompatible.
diff --git a/pkg/kubelet/cloudresource/cloud_request_manager_test.go b/pkg/kubelet/cloudresource/cloud_request_manager_test.go
@@ -85,7 +85,7 @@ func TestNodeAddressesUsesLastSuccess(t *testing.T) {
 	cloud := &fake.Cloud{}
 	manager := NewSyncManager(cloud, "defaultNode", 0).(*cloudResourceSyncManager)
 
-	// These tests are stateful and order dependant.
+	// These tests are stateful and order dependent.
 	tests := []struct {
 		name      string
 		addrs     []v1.NodeAddress
diff --git a/pkg/kubelet/cm/container_manager_linux.go b/pkg/kubelet/cm/container_manager_linux.go
@@ -297,7 +297,7 @@ func NewContainerManager(mountUtil mount.Interface, cadvisorInterface cadvisor.I
 			return nil, err
 		}
 
-		klog.Infof("[topologymanager] Initilizing Topology Manager with %s policy", nodeConfig.ExperimentalTopologyManagerPolicy)
+		klog.Infof("[topologymanager] Initializing Topology Manager with %s policy", nodeConfig.ExperimentalTopologyManagerPolicy)
 	} else {
 		cm.topologyManager = topologymanager.NewFakeManager()
 	}
diff --git a/pkg/kubelet/cm/cpumanager/state/state.go b/pkg/kubelet/cm/cpumanager/state/state.go
@@ -20,7 +20,7 @@ import (
 	"k8s.io/kubernetes/pkg/kubelet/cm/cpuset"
 )
 
-// ContainerCPUAssignments type used in cpu manger state
+// ContainerCPUAssignments type used in cpu manager state
 type ContainerCPUAssignments map[string]cpuset.CPUSet
 
 // Clone returns a copy of ContainerCPUAssignments
diff --git a/pkg/kubelet/cm/cpumanager/state/state_file_test.go b/pkg/kubelet/cm/cpumanager/state/state_file_test.go
@@ -465,11 +465,11 @@ func TestClearStateStateFile(t *testing.T) {
 
 			state.ClearState()
 			if !cpuset.NewCPUSet().Equals(state.GetDefaultCPUSet()) {
-				t.Error("cleared state shoudn't has got information about available cpuset")
+				t.Error("cleared state shouldn't has got information about available cpuset")
 			}
 			for containerName := range testCase.containers {
 				if !cpuset.NewCPUSet().Equals(state.GetCPUSetOrDefault(containerName)) {
-					t.Error("cleared state shoudn't has got information about containers")
+					t.Error("cleared state shouldn't has got information about containers")
 				}
 			}
 		})
diff --git a/pkg/kubelet/dockershim/helpers_unsupported.go b/pkg/kubelet/dockershim/helpers_unsupported.go
@@ -54,6 +54,6 @@ func getNetworkNamespace(c *dockertypes.ContainerJSON) (string, error) {
 	return "", fmt.Errorf("unsupported platform")
 }
 
-// applyExperimentalCreateConfig applys experimental configures from sandbox annotations.
+// applyExperimentalCreateConfig applies experimental configures from sandbox annotations.
 func applyExperimentalCreateConfig(createConfig *dockertypes.ContainerCreateConfig, annotations map[string]string) {
 }
diff --git a/pkg/kubelet/eviction/eviction_manager.go b/pkg/kubelet/eviction/eviction_manager.go
@@ -292,7 +292,7 @@ func (m *managerImpl) synchronize(diskInfoProvider DiskInfoProvider, podFunc Act
 
 	// determine the set of thresholds we need to drive eviction behavior (i.e. all grace periods are met)
 	thresholds = thresholdsMetGracePeriod(thresholdsFirstObservedAt, now)
-	debugLogThresholdsWithObservation("thresholds - grace periods satisified", thresholds, observations)
+	debugLogThresholdsWithObservation("thresholds - grace periods satisfied", thresholds, observations)
 
 	// update internal state
 	m.Lock()
diff --git a/pkg/kubelet/eviction/eviction_manager_test.go b/pkg/kubelet/eviction/eviction_manager_test.go
@@ -927,7 +927,7 @@ func TestNodeReclaimFuncs(t *testing.T) {
 
 	// no image gc should have occurred
 	if diskGC.imageGCInvoked || diskGC.containerGCInvoked {
-		t.Errorf("Manager chose to perform image gc when it was not neeed")
+		t.Errorf("Manager chose to perform image gc when it was not needed")
 	}
 
 	// no pod should have been killed
@@ -950,7 +950,7 @@ func TestNodeReclaimFuncs(t *testing.T) {
 
 	// no image gc should have occurred
 	if diskGC.imageGCInvoked || diskGC.containerGCInvoked {
-		t.Errorf("Manager chose to perform image gc when it was not neeed")
+		t.Errorf("Manager chose to perform image gc when it was not needed")
 	}
 
 	// no pod should have been killed
diff --git a/pkg/kubelet/eviction/helpers.go b/pkg/kubelet/eviction/helpers.go
@@ -329,7 +329,7 @@ func memoryUsage(memStats *statsapi.MemoryStats) *resource.Quantity {
 }
 
 // localVolumeNames returns the set of volumes for the pod that are local
-// TODO: sumamry API should report what volumes consume local storage rather than hard-code here.
+// TODO: summary API should report what volumes consume local storage rather than hard-code here.
 func localVolumeNames(pod *v1.Pod) []string {
 	result := []string{}
 	for _, volume := range pod.Spec.Volumes {
diff --git a/pkg/kubelet/eviction/threshold_notifier_linux.go b/pkg/kubelet/eviction/threshold_notifier_linux.go
@@ -133,7 +133,7 @@ func (n *linuxCgroupNotifier) Start(eventCh chan<- struct{}) {
 }
 
 // wait waits up to notifierRefreshInterval for an event on the Epoll FD for the
-// eventfd we are concerned about.  It returns an error if one occurrs, and true
+// eventfd we are concerned about.  It returns an error if one occurs, and true
 // if the consumer should read from the eventfd.
 func wait(epfd, eventfd int, timeout time.Duration) (bool, error) {
 	events := make([]unix.EpollEvent, numFdEvents+1)
diff --git a/pkg/kubelet/kubelet_pods.go b/pkg/kubelet/kubelet_pods.go
@@ -899,7 +899,7 @@ func (kl *Kubelet) IsPodDeleted(uid types.UID) bool {
 // been reclaimed by the kubelet.  Reclaiming resources is a prerequisite to deleting a pod from the API server.
 func (kl *Kubelet) PodResourcesAreReclaimed(pod *v1.Pod, status v1.PodStatus) bool {
 	if !notRunning(status.ContainerStatuses) {
-		// We shouldnt delete pods that still have running containers
+		// We shouldn't delete pods that still have running containers
 		klog.V(3).Infof("Pod %q is terminated, but some containers are still running", format.Pod(pod))
 		return false
 	}
@@ -918,7 +918,7 @@ func (kl *Kubelet) PodResourcesAreReclaimed(pod *v1.Pod, status v1.PodStatus) bo
 		return false
 	}
 	if kl.podVolumesExist(pod.UID) && !kl.keepTerminatedPodVolumes {
-		// We shouldnt delete pods whose volumes have not been cleaned up if we are not keeping terminated pod volumes
+		// We shouldn't delete pods whose volumes have not been cleaned up if we are not keeping terminated pod volumes
 		klog.V(3).Infof("Pod %q is terminated, but some volumes have not been cleaned up", format.Pod(pod))
 		return false
 	}
diff --git a/pkg/kubelet/kuberuntime/kuberuntime_container_test.go b/pkg/kubelet/kuberuntime/kuberuntime_container_test.go
@@ -327,7 +327,7 @@ func TestLifeCycleHook(t *testing.T) {
 		// Now try to create a container, which should in turn invoke PostStart Hook
 		_, err := m.startContainer(fakeSandBox.Id, fakeSandBoxConfig, testContainer, testPod, fakePodStatus, nil, "")
 		if err != nil {
-			t.Errorf("startContainer erro =%v", err)
+			t.Errorf("startContainer error =%v", err)
 		}
 		if fakeRunner.Cmd[0] != cmdPostStart.PostStart.Exec.Command[0] {
 			t.Errorf("CMD PostStart hook was not invoked")
diff --git a/pkg/kubelet/kuberuntime/kuberuntime_manager_test.go b/pkg/kubelet/kuberuntime/kuberuntime_manager_test.go
@@ -719,7 +719,7 @@ func TestComputePodActions(t *testing.T) {
 	_, _, m, err := createTestRuntimeManager()
 	require.NoError(t, err)
 
-	// Createing a pair reference pod and status for the test cases to refer
+	// Creating a pair reference pod and status for the test cases to refer
 	// the specific fields.
 	basePod, baseStatus := makeBasePodAndStatus()
 	noAction := podActions{
diff --git a/pkg/kubelet/logs/container_log_manager.go b/pkg/kubelet/logs/container_log_manager.go
@@ -295,7 +295,7 @@ func isInUse(l string, logs []string) bool {
 	if strings.HasSuffix(l, tmpSuffix) {
 		return false
 	}
-	// All compresed logs are in use.
+	// All compressed logs are in use.
 	if strings.HasSuffix(l, compressSuffix) {
 		return true
 	}
diff --git a/pkg/kubelet/pluginmanager/operationexecutor/operation_executor.go b/pkg/kubelet/pluginmanager/operationexecutor/operation_executor.go
@@ -63,7 +63,7 @@ func NewOperationExecutor(
 }
 
 // ActualStateOfWorldUpdater defines a set of operations updating the actual
-// state of the world cache after successful registeration/deregistration.
+// state of the world cache after successful registration/deregistration.
 type ActualStateOfWorldUpdater interface {
 	// AddPlugin add the given plugin in the cache if no existing plugin
 	// in the cache has the same socket path.
diff --git a/pkg/kubelet/pod/pod_manager.go b/pkg/kubelet/pod/pod_manager.go
@@ -59,7 +59,7 @@ type Manager interface {
 	// whether the pod is found.
 	GetPodByUID(types.UID) (*v1.Pod, bool)
 	// GetPodByMirrorPod returns the static pod for the given mirror pod and
-	// whether it was known to the pod manger.
+	// whether it was known to the pod manager.
 	GetPodByMirrorPod(*v1.Pod) (*v1.Pod, bool)
 	// GetMirrorPodByPod returns the mirror pod for the given static pod and
 	// whether it was known to the pod manager.
diff --git a/pkg/kubelet/stats/cri_stats_provider.go b/pkg/kubelet/stats/cri_stats_provider.go
@@ -672,7 +672,7 @@ func (p *criStatsProvider) getAndUpdateContainerUsageNanoCores(stats *runtimeapi
 	}()
 
 	if err != nil {
-		// This should not happen. Log now to raise visiblity
+		// This should not happen. Log now to raise visibility
 		klog.Errorf("failed updating cpu usage nano core: %v", err)
 	}
 	return usage
diff --git a/pkg/kubelet/status/status_manager_test.go b/pkg/kubelet/status/status_manager_test.go
diff --git a/pkg/kubelet/util/manager/cache_based_manager_test.go b/pkg/kubelet/util/manager/cache_based_manager_test.go
diff --git a/pkg/master/master_test.go b/pkg/master/master_test.go
diff --git a/pkg/proxy/apis/config/types.go b/pkg/proxy/apis/config/types.go
diff --git a/pkg/registry/core/pod/strategy.go b/pkg/registry/core/pod/strategy.go
diff --git a/pkg/registry/rbac/rest/storage_rbac.go b/pkg/registry/rbac/rest/storage_rbac.go
diff --git a/pkg/registry/rbac/validation/rule.go b/pkg/registry/rbac/validation/rule.go
diff --git a/pkg/registry/storage/volumeattachment/strategy_test.go b/pkg/registry/storage/volumeattachment/strategy_test.go
diff --git a/pkg/scheduler/algorithm/predicates/predicates.go b/pkg/scheduler/algorithm/predicates/predicates.go
diff --git a/pkg/scheduler/algorithm/predicates/predicates_test.go b/pkg/scheduler/algorithm/predicates/predicates_test.go
diff --git a/pkg/scheduler/algorithm/priorities/interpod_affinity_test.go b/pkg/scheduler/algorithm/priorities/interpod_affinity_test.go
diff --git a/pkg/scheduler/framework/plugins/interpodaffinity/interpod_affinity_test.go b/pkg/scheduler/framework/plugins/interpodaffinity/interpod_affinity_test.go
diff --git a/pkg/scheduler/framework/plugins/podtopologyspread/pod_topology_spread_test.go b/pkg/scheduler/framework/plugins/podtopologyspread/pod_topology_spread_test.go
diff --git a/pkg/scheduler/internal/cache/cache_test.go b/pkg/scheduler/internal/cache/cache_test.go

Original file line number	Diff line number	Diff line change
`@@ -306,7 +306,7 @@ function kube::build::has_ip() {`
`306`	`306`	`# Detect if a specific image exists`
`307`	`307`	`#`
`308`	`308`	`# $1 - image repo name`
`309`		`-# #2 - image tag`
	`309`	`+# $2 - image tag`
`310`	`310`	`function kube::build::docker_image_exists() {`
`311`	`311`	`[[ -n $1 && -n $2 ]] \|\| {`
`312`	`312`	`kube::log::error "Internal error. Image not specified in docker_image_exists."`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ type PermissiveSigningPolicy struct {`
`47`	`47`	`// TTL is the certificate TTL. It's used to calculate the NotAfter value of`
`48`	`48`	`// the certificate.`
`49`	`49`	`TTL time.Duration`
`50`		`- // Usages are the allowed usages of a certficate.`
	`50`	`+ // Usages are the allowed usages of a certificate.`
`51`	`51`	`Usages []capi.KeyUsage`
`52`	`52`	`}`
`53`	`53`
Original file line number	Diff line number	Diff line change
`@@ -893,7 +893,7 @@ func (dsc DaemonSetsController) manage(ds apps.DaemonSet, nodeList []*v1.Node,`
`893`	`893`	`}`
`894`	`894`
`895`	`895`	`// syncNodes deletes given pods and creates new daemon set pods on the given nodes`
`896`		`-// returns slice with erros if any`
	`896`	`+// returns slice with errors if any`
`897`	`897`	`func (dsc DaemonSetsController) syncNodes(ds apps.DaemonSet, podsToDelete, nodesNeedingDaemonPods []string, hash string) error {`
`898`	`898`	`// We need to set expectations before creating/deleting pods to avoid race conditions.`
`899`	`899`	`dsKey, err := controller.KeyFunc(ds)`
Original file line number	Diff line number	Diff line change
`@@ -222,7 +222,7 @@ func TestCreatePod(t *testing.T) {`
`222`	`222`	`}`
`223`	`223`	`}`
`224`	`224`	`if podDeleted != item.expectDelete {`
`225`		`- t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
	`225`	`+ t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
`226`	`226`	`}`
`227`	`227`	`close(stopCh)`
`228`	`228`	`}`
`@@ -319,7 +319,7 @@ func TestUpdatePod(t *testing.T) {`
`319`	`319`	`}`
`320`	`320`	`}`
`321`	`321`	`if podDeleted != item.expectDelete {`
`322`		`- t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
	`322`	`+ t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
`323`	`323`	`}`
`324`	`324`	`close(stopCh)`
`325`	`325`	`}`
`@@ -375,7 +375,7 @@ func TestCreateNode(t *testing.T) {`
`375`	`375`	`}`
`376`	`376`	`}`
`377`	`377`	`if podDeleted != item.expectDelete {`
`378`		`- t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
	`378`	`+ t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
`379`	`379`	`}`
`380`	`380`	`close(stopCh)`
`381`	`381`	`}`
`@@ -499,7 +499,7 @@ func TestUpdateNode(t *testing.T) {`
`499`	`499`	`}`
`500`	`500`	`}`
`501`	`501`	`if podDeleted != item.expectDelete {`
`502`		`- t.Errorf("%v: Unexepected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
	`502`	`+ t.Errorf("%v: Unexpected test result. Expected delete %v, got %v", item.description, item.expectDelete, podDeleted)`
`503`	`503`	`}`
`504`	`504`	`close(stopCh)`
`505`	`505`	`}`
Original file line number	Diff line number	Diff line change
`@@ -348,7 +348,7 @@ func (tc testCase) prepareTestClient(t testing.T) (fake.Clientset, metricsfa`
`348`	`348`	`if err := json.Unmarshal([]byte(obj.ObjectMeta.Annotations[autoscaling.HorizontalPodAutoscalerConditionsAnnotation]), &actualConditions); err != nil {`
`349`	`349`	`return true, nil, err`
`350`	`350`	`}`
`351`		`- // TODO: it's ok not to sort these becaues statusOk`
	`351`	`+ // TODO: it's ok not to sort these because statusOk`
`352`	`352`	`// contains all the conditions, so we'll never be appending.`
`353`	`353`	`// Default to statusOk when missing any specific conditions`
`354`	`354`	`if tc.expectedConditions == nil {`
Original file line number	Diff line number	Diff line change
`@@ -1081,7 +1081,7 @@ func TestDeleteControllerAndExpectations(t *testing.T) {`
`1081`	`1081`	`manager.syncReplicaSet(GetKey(rs, t))`
`1082`	`1082`
`1083`	`1083`	`if _, exists, err = manager.expectations.GetExpectations(rsKey); exists {`
`1084`		`- t.Errorf("Found expectaions, expected none since the ReplicaSet has been deleted.")`
	`1084`	`+ t.Errorf("Found expectations, expected none since the ReplicaSet has been deleted.")`
`1085`	`1085`	`}`
`1086`	`1086`
`1087`	`1087`	`// This should have no effect, since we've deleted the ReplicaSet.`