chore: fix wrong function name in comment (#58119)

Signed-off-by: promalert <[email protected]>

Author: promalert

tests/integration/helm/util.go

@@ -163,7 +163,7 @@ spec:
 // ManifestsChartPath is path of local Helm charts used for testing.
 var ManifestsChartPath = filepath.Join(env.IstioSrc, "manifests/charts")
 
-// getValuesOverrides returns the values file created to pass into Helm override default values
+// GetValuesOverrides returns the values file created to pass into Helm override default values
 // for the hub and tag.
 //
 // Tag can be the empty string, which means the values file will not have a

tools/bug-report/pkg/cluster/cluster.go

@@ -43,7 +43,7 @@ func ParsePath(path string) (namespace string, deployment, pod string, container
 	return pv[0], pv[1], pv[2], pv[3], nil
 }
 
-// shouldSkip means that current pod should be skip or not based on given --include and --exclude
+// shouldSkipPod means that current pod should be skip or not based on given --include and --exclude
 func shouldSkipPod(pod *corev1.Pod, config *config2.BugReportConfig) bool {
 	for _, eld := range config.Exclude {
 		if len(eld.Namespaces) > 0 {

tools/istio-iptables/pkg/builder/helper.go

@@ -107,7 +107,7 @@ func UndoRules(rules []Rule) []Rule {
 	return output
 }
 
-// BuildCustomCleanup generates a set of iptables commands to clean up unexpected leftover rules and chains.
+// BuildCleanupFromState generates a set of iptables commands to clean up unexpected leftover rules and chains.
 // The function takes the current state of iptables, represented by a map of table names to their associated chains and rules.
 // It first transforms the provided rules into corresponding undo rules.
 // It then appends flush and delete commands for each ISTIO_* chain.

tools/istio-iptables/pkg/builder/iptables_builder_impl.go

@@ -45,7 +45,7 @@ type IptablesRuleBuilder struct {
 	cfg   *config.Config
 }
 
-// NewIptablesBuilders creates a new IptablesRuleBuilder
+// NewIptablesRuleBuilder creates a new IptablesRuleBuilder
 func NewIptablesRuleBuilder(cfg *config.Config) *IptablesRuleBuilder {
 	if cfg == nil {
 		cfg = &config.Config{}
@@ -310,7 +310,7 @@ func (rb *IptablesRuleBuilder) BuildV6Restore() string {
 	return rb.buildRestore(rb.rules.rulesv6)
 }
 
-// getStateFromSave function takes a string in iptables-restore format and returns a map of the tables, chains, and rules.
+// GetStateFromSave function takes a string in iptables-restore format and returns a map of the tables, chains, and rules.
 // Note that if this function is used to parse iptables-save output, the rules may have changed since they were first applied
 // as rules do not necessarily undergo a round-trip through the kernel in the same form.
 // Therefore, these rules should not be used for any critical checks.