Merge pull request kubernetes-sigs#2016 from zhouhao3/fix-ineffassign

fix ineffectual assignment to err
sonal-chauhan · May 18, 2021 · 07a29df · 07a29df
2 parents 8409bd5 + 45d9714
commit 07a29df
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/test/framework/helm/release_manager.go b/test/framework/helm/release_manager.go
@@ -66,6 +66,9 @@ func (m *defaultReleaseManager) InstallRelease(chartRepo string, chartName strin
 	installAction.ReleaseName = releaseName
 
 	cp, err := installAction.ChartPathOptions.LocateChart(chartName, cli.New())
+	if err != nil {
+		return nil, err
+	}
 	chartRequested, err := loader.Load(cp)
 	if err != nil {
 		return nil, err
@@ -85,6 +88,9 @@ func (m *defaultReleaseManager) UpgradeRelease(chartRepo string, chartName strin
 	upgradeAction.Wait = true
 
 	cp, err := upgradeAction.ChartPathOptions.LocateChart(chartName, cli.New())
+	if err != nil {
+		return nil, err
+	}
 	chartRequested, err := loader.Load(cp)
 	if err != nil {
 		return nil, err