diff --git a/test/e2e/framework/pods.go b/test/e2e/framework/pods.go
index 760625b7ee1..3e15c59e0ed 100644
--- a/test/e2e/framework/pods.go
+++ b/test/e2e/framework/pods.go
@@ -38,6 +38,7 @@ import (
 	"github.com/onsi/gomega"
 )
 
+// DefaultPodDeletionTimeout is the default timeout for deleting pod
 const DefaultPodDeletionTimeout = 3 * time.Minute
 
 // ImageWhiteList is the images used in the current test suite. It should be initialized in test suite and
diff --git a/test/e2e/framework/provider.go b/test/e2e/framework/provider.go
index ce1b8feccd2..74411557a40 100644
--- a/test/e2e/framework/provider.go
+++ b/test/e2e/framework/provider.go
@@ -129,7 +129,7 @@ func (n NullProvider) GetGroupNodes(group string) ([]string, error) {
 	return nil, fmt.Errorf("provider does not support InstanceGroups")
 }
 
-// DeleteNode is a base implementation which returns group size.
+// GroupSize returns the size of an instance group
 func (n NullProvider) GroupSize(group string) (int, error) {
 	return -1, fmt.Errorf("provider does not support InstanceGroups")
 }
diff --git a/test/e2e/framework/upgrade_util.go b/test/e2e/framework/upgrade_util.go
index 8aa14d76760..6b9a090d096 100644
--- a/test/e2e/framework/upgrade_util.go
+++ b/test/e2e/framework/upgrade_util.go
@@ -57,6 +57,7 @@ func traceRouteToMaster() {
 	}
 }
 
+// CheckMasterVersion validates the master version
 func CheckMasterVersion(c clientset.Interface, want string) error {
 	Logf("Checking master version")
 	var err error
@@ -84,6 +85,7 @@ func CheckMasterVersion(c clientset.Interface, want string) error {
 	return nil
 }
 
+// CheckNodesVersions validates the nodes versions
 func CheckNodesVersions(cs clientset.Interface, want string) error {
 	l := GetReadySchedulableNodesOrDie(cs)
 	for _, n := range l.Items {