Merge pull request #5165 from natasha41575/pin

pin to kyaml and cmd/config released versions

api/internal/builtins/HelmChartInflationGenerator.go

@@ -91,12 +91,7 @@ func (p *HelmChartInflationGeneratorPlugin) validateArgs() (err error) {
 	// be under the loader root (unless root restrictions are
 	// disabled).
 	if p.ValuesFile == "" {
-		// If the version is specified, use the versioned values file.
-		if p.Version != "" {
-			p.ValuesFile = filepath.Join(p.ChartHome, fmt.Sprintf("%s-%s", p.Name, p.Version), p.Name, "values.yaml")
-		} else {
-			p.ValuesFile = filepath.Join(p.ChartHome, p.Name, "values.yaml")
-		}
+		p.ValuesFile = filepath.Join(p.ChartHome, p.Name, "values.yaml")
 	}
 	for i, file := range p.AdditionalValuesFiles {
 		// use Load() to enforce root restrictions
@@ -137,17 +132,10 @@ func (p *HelmChartInflationGeneratorPlugin) errIfIllegalValuesMerge() error {
 }
 
 func (p *HelmChartInflationGeneratorPlugin) absChartHome() string {
-	var chartHome string
 	if filepath.IsAbs(p.ChartHome) {
-		chartHome = p.ChartHome
-	} else {
-		chartHome = filepath.Join(p.h.Loader().Root(), p.ChartHome)
+		return p.ChartHome
 	}
-
-	if p.Version != "" {
-		return filepath.Join(chartHome, fmt.Sprintf("%s-%s", p.Name, p.Version))
-	}
-	return chartHome
+	return filepath.Join(p.h.Loader().Root(), p.ChartHome)
 }
 
 func (p *HelmChartInflationGeneratorPlugin) runHelmCommand(

cmd/pluginator/go.mod

@@ -5,7 +5,7 @@ go 1.19
 require (
 	github.com/spf13/cobra v1.4.0
 	github.com/stretchr/testify v1.8.1
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 )
 

kustomize/go.mod

@@ -8,7 +8,7 @@ require (
 	github.com/spf13/pflag v1.0.5
 	github.com/stretchr/testify v1.8.1
 	golang.org/x/text v0.6.0
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/cmd/config v0.11.2
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0

plugin/builtin/annotationstransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/annotationstransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/configmapgenerator/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/configmapgenerator
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/hashtransformer/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/hashtransformer
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator.go

@@ -97,12 +97,7 @@ func (p *plugin) validateArgs() (err error) {
 	// be under the loader root (unless root restrictions are
 	// disabled).
 	if p.ValuesFile == "" {
-		// If the version is specified, use the versioned values file.
-		if p.Version != "" {
-			p.ValuesFile = filepath.Join(p.ChartHome, fmt.Sprintf("%s-%s", p.Name, p.Version), p.Name, "values.yaml")
-		} else {
-			p.ValuesFile = filepath.Join(p.ChartHome, p.Name, "values.yaml")
-		}
+		p.ValuesFile = filepath.Join(p.ChartHome, p.Name, "values.yaml")
 	}
 	for i, file := range p.AdditionalValuesFiles {
 		// use Load() to enforce root restrictions
@@ -143,17 +138,10 @@ func (p *plugin) errIfIllegalValuesMerge() error {
 }
 
 func (p *plugin) absChartHome() string {
-	var chartHome string
 	if filepath.IsAbs(p.ChartHome) {
-		chartHome = p.ChartHome
-	} else {
-		chartHome = filepath.Join(p.h.Loader().Root(), p.ChartHome)
+		return p.ChartHome
 	}
-
-	if p.Version != "" {
-		return filepath.Join(chartHome, fmt.Sprintf("%s-%s", p.Name, p.Version))
-	}
-	return chartHome
+	return filepath.Join(p.h.Loader().Root(), p.ChartHome)
 }
 
 func (p *plugin) runHelmCommand(

plugin/builtin/helmchartinflationgenerator/HelmChartInflationGenerator_test.go

@@ -9,7 +9,6 @@ import (
 	"path/filepath"
 	"testing"
 
-	"github.com/stretchr/testify/assert"
 	kusttest_test "sigs.k8s.io/kustomize/api/testutils/kusttest"
 )
 
@@ -580,139 +579,3 @@ valuesInline:
 `)
 	th.AssertActualEqualsExpected(rm, "")
 }
-
-func TestHelmChartInflationGeneratorWithSameChartMultipleVersions(t *testing.T) {
-	th := kusttest_test.MakeEnhancedHarnessWithTmpRoot(t).
-		PrepBuiltin("HelmChartInflationGenerator")
-	defer th.Reset()
-	if err := th.ErrIfNoHelm(); err != nil {
-		t.Skip("skipping: " + err.Error())
-	}
-
-	tests := []struct {
-		name        string
-		chartName   string
-		repo        string
-		version     string
-		releaseName string
-	}{
-		{
-			name:        "terraform chart with no version grabs latest",
-			chartName:   "terraform",
-			repo:        "https://helm.releases.hashicorp.com",
-			version:     "",
-			releaseName: "terraform-latest",
-		},
-		{
-			name:        "terraform chart with version 1.1.1",
-			chartName:   "terraform",
-			repo:        "https://helm.releases.hashicorp.com",
-			version:     "1.1.1",
-			releaseName: "terraform-1.1.1",
-		},
-		{
-			name:        "terraform chart with version 1.1.1 again",
-			chartName:   "terraform",
-			repo:        "https://helm.releases.hashicorp.com",
-			version:     "1.1.1",
-			releaseName: "terraform-1.1.1-1",
-		},
-		{
-			name:        "terraform chart with version 1.1.2",
-			chartName:   "terraform",
-			repo:        "https://helm.releases.hashicorp.com",
-			version:     "1.1.2",
-			releaseName: "terraform-1.1.2",
-		},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			config := fmt.Sprintf(`
-apiVersion: builtin
-kind: HelmChartInflationGenerator
-metadata:
- name: %s
-name: %s
-version: %s
-repo: %s
-releaseName: %s
-`, tt.chartName, tt.chartName, tt.version, tt.repo, tt.releaseName)
-
-			rm := th.LoadAndRunGenerator(config)
-			assert.True(t, len(rm.Resources()) > 0)
-
-			var chartDir string
-			if tt.version != "" {
-				chartDir = fmt.Sprintf("charts/%s-%s/%s", tt.chartName, tt.version, tt.chartName)
-			} else {
-				chartDir = fmt.Sprintf("charts/%s", tt.chartName)
-			}
-
-			d, err := th.GetFSys().ReadFile(filepath.Join(th.GetRoot(), chartDir, "Chart.yaml"))
-			if err != nil {
-				t.Fatal(err)
-			}
-
-			assert.Contains(t, string(d), fmt.Sprintf("name: %s", tt.chartName))
-			if tt.version != "" {
-				assert.Contains(t, string(d), fmt.Sprintf("version: %s", tt.version))
-			}
-		})
-	}
-}
-
-// Test that verifies +1 instances of same chart with different versions
-// https://github.com/kubernetes-sigs/kustomize/issues/4813
-func TestHelmChartInflationGeneratorWithMultipleInstancesSameChartDifferentVersions(t *testing.T) {
-	th := kusttest_test.MakeEnhancedHarnessWithTmpRoot(t).
-		PrepBuiltin("HelmChartInflationGenerator")
-	defer th.Reset()
-	if err := th.ErrIfNoHelm(); err != nil {
-		t.Skip("skipping: " + err.Error())
-	}
-
-	podinfo1 := th.LoadAndRunGenerator(`
-apiVersion: builtin
-kind: HelmChartInflationGenerator
-metadata:
- name: podinfo
-name: podinfo
-version: 6.2.1
-repo: https://stefanprodan.github.io/podinfo
-releaseName: podinfo1
-`)
-
-	podinfo2 := th.LoadAndRunGenerator(`
-apiVersion: builtin
-kind: HelmChartInflationGenerator
-metadata:
- name: podinfo
-name: podinfo
-version: 6.1.8
-repo: https://stefanprodan.github.io/podinfo
-releaseName: podinfo2
-`)
-
-	podinfo1Img, err := podinfo1.Resources()[1].GetFieldValue("spec.template.spec.containers.0.image")
-	assert.NoError(t, err)
-	assert.Equal(t, "ghcr.io/stefanprodan/podinfo:6.2.1", podinfo1Img)
-
-	podinfo2Img, err := podinfo2.Resources()[1].GetFieldValue("spec.template.spec.containers.0.image")
-	assert.NoError(t, err)
-	assert.Equal(t, "ghcr.io/stefanprodan/podinfo:6.1.8", podinfo2Img)
-
-	podinfo1ChartsDir := filepath.Join(th.GetRoot(), "charts/podinfo-6.2.1/podinfo")
-	assert.True(t, th.GetFSys().Exists(podinfo1ChartsDir))
-
-	podinfo2ChartsDir := filepath.Join(th.GetRoot(), "charts/podinfo-6.1.8/podinfo")
-	assert.True(t, th.GetFSys().Exists(podinfo2ChartsDir))
-
-	podinfo1ChartContents, err := th.GetFSys().ReadFile(filepath.Join(podinfo1ChartsDir, "Chart.yaml"))
-	assert.NoError(t, err)
-	assert.Contains(t, string(podinfo1ChartContents), "version: 6.2.1")
-
-	podinfo2ChartContents, err := th.GetFSys().ReadFile(filepath.Join(podinfo2ChartsDir, "Chart.yaml"))
-	assert.NoError(t, err)
-	assert.Contains(t, string(podinfo2ChartContents), "version: 6.1.8")
-}

plugin/builtin/helmchartinflationgenerator/go.mod

@@ -4,8 +4,7 @@ go 1.19
 
 require (
 	github.com/imdario/mergo v0.3.6
-	github.com/stretchr/testify v1.8.1
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )
@@ -26,7 +25,6 @@ require (
 	github.com/mailru/easyjson v0.7.7 // indirect
 	github.com/monochromegane/go-gitignore v0.0.0-20200626010858-205db1a8cc00 // indirect
 	github.com/pkg/errors v0.9.1 // indirect
-	github.com/pmezard/go-difflib v1.0.0 // indirect
 	github.com/xlab/treeprint v1.1.0 // indirect
 	go.starlark.net v0.0.0-20200306205701-8dd3e2ee1dd5 // indirect
 	golang.org/x/sys v0.3.0 // indirect

plugin/builtin/iampolicygenerator/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/iampolicygenerator
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/imagetagtransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/imagetagtransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/labeltransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/labeltransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/namespacetransformer/go.mod

@@ -4,7 +4,7 @@ go 1.19
 
 require (
 	github.com/stretchr/testify v1.8.1
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/builtin/patchjson6902transformer/go.mod

@@ -4,7 +4,7 @@ go 1.19
 
 require (
 	github.com/evanphx/json-patch v4.11.0+incompatible
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/builtin/patchstrategicmergetransformer/go.mod

@@ -4,7 +4,7 @@ go 1.19
 
 require (
 	github.com/stretchr/testify v1.8.1
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/patchtransformer/go.mod

@@ -4,7 +4,7 @@ go 1.19
 
 require (
 	github.com/evanphx/json-patch v4.11.0+incompatible
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/builtin/prefixtransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/prefixtransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 )
 

plugin/builtin/replacementtransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/replacementtransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/replicacounttransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/replicacounttransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/builtin/secretgenerator/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/secretgenerator
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/builtin/sortordertransformer/go.mod

@@ -4,7 +4,7 @@ go 1.18
 
 require (
 	github.com/stretchr/testify v1.8.1
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/builtin/suffixtransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/suffixtransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 )
 

plugin/builtin/valueaddtransformer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/builtin/valueaddtransformer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/someteam.example.com/v1/bashedconfigmap/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/bashedconfigmap
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/someteam.example.com/v1/calvinduplicator/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/calvinduplicator
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/someteam.example.com/v1/dateprefixer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/dateprefixer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/someteam.example.com/v1/printpluginenv/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/printpluginenv
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/someteam.example.com/v1/secretsfromdatabase/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/secretsfromdatabase
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/someteam.example.com/v1/sedtransformer/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/sedtransformer
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/someteam.example.com/v1/someservicegenerator/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/someservicegenerator
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/yaml v1.3.0
 )
 

plugin/someteam.example.com/v1/starlarkmixer/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/starlarkmixer
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/someteam.example.com/v1/stringprefixer/go.mod

@@ -3,7 +3,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/stringprefixer
 go 1.19
 
 require (
-	sigs.k8s.io/kustomize/api v0.13.2
+	sigs.k8s.io/kustomize/api v0.13.3
 	sigs.k8s.io/kustomize/kyaml v0.14.2
 	sigs.k8s.io/yaml v1.3.0
 )

plugin/someteam.example.com/v1/validator/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/someteam.example.com/v1/validator
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require (
 	github.com/davecgh/go-spew v1.1.1 // indirect

plugin/untested/v1/gogetter/go.mod

@@ -2,7 +2,7 @@ module sigs.k8s.io/kustomize/plugin/untested/v1/gogetter
 
 go 1.19
 
-require sigs.k8s.io/kustomize/api v0.13.2
+require sigs.k8s.io/kustomize/api v0.13.3
 
 require github.com/josharian/intern v1.0.0 // indirect