allow dependency versions to be overwritten on package update

Signed-off-by: Gregory Hill <greg.hill@monax.io>

cmd/helm/package.go

@@ -83,13 +83,14 @@ func newPackageCmd(out io.Writer) *cobra.Command {
 
 				if client.DependencyUpdate {
 					downloadManager := &downloader.Manager{
-						Out:              ioutil.Discard,
+						Out:                ioutil.Discard,
-						ChartPath:        path,
+						ChartPath:          path,
-						Keyring:          client.Keyring,
+						Keyring:            client.Keyring,
-						Getters:          p,
+						Getters:            p,
-						Debug:            settings.Debug,
+						Debug:              settings.Debug,
-						RepositoryConfig: settings.RepositoryConfig,
+						DependencyVersions: client.DependencyVersions,
-						RepositoryCache:  settings.RepositoryCache,
+						RepositoryConfig:   settings.RepositoryConfig,
+						RepositoryCache:    settings.RepositoryCache,
 					}
 
 					if err := downloadManager.Update(); err != nil {
@@ -114,6 +115,7 @@ func newPackageCmd(out io.Writer) *cobra.Command {
 	f.StringVar(&client.AppVersion, "app-version", "", "set the appVersion on the chart to this version")
 	f.StringVarP(&client.Destination, "destination", "d", ".", "location to write the chart.")
 	f.BoolVarP(&client.DependencyUpdate, "dependency-update", "u", false, `update dependencies from "Chart.yaml" to dir "charts/" before packaging`)
+	f.StringToStringVar(&client.DependencyVersions, "set-dependency-version", make(map[string]string), "set dependency versions. Used if --dependency-update is true")
 	addValueOptionsFlags(f, valueOpts)
 
 	return cmd

pkg/action/package.go

@@ -36,13 +36,14 @@ import (
 //
 // It provides the implementation of 'helm package'.
 type Package struct {
-	Sign             bool
+	Sign               bool
-	Key              string
+	Key                string
-	Keyring          string
+	Keyring            string
-	Version          string
+	Version            string
-	AppVersion       string
+	AppVersion         string
-	Destination      string
+	Destination        string
-	DependencyUpdate bool
+	DependencyUpdate   bool
+	DependencyVersions map[string]string
 
 	RepositoryConfig string
 	RepositoryCache  string
@@ -77,6 +78,15 @@ func (p *Package) Run(path string, vals map[string]interface{}) (string, error)
 		ch.Metadata.AppVersion = p.AppVersion
 	}
 
+	if p.DependencyUpdate {
+		// only alter these upon successful update
+		for _, d := range ch.Metadata.Dependencies {
+			if v, ok := p.DependencyVersions[d.Name]; ok {
+				d.Version = v
+			}
+		}
+	}
+
 	if reqs := ch.Metadata.Dependencies; reqs != nil {
 		if err := CheckDependencies(ch, reqs); err != nil {
 			return "", err

pkg/downloader/manager.go

@@ -56,9 +56,11 @@ type Manager struct {
 	// SkipUpdate indicates that the repository should not be updated first.
 	SkipUpdate bool
 	// Getter collection for the operation
-	Getters          []getter.Provider
+	Getters []getter.Provider
-	RepositoryConfig string
+	// DependencyVersions allows us to explicitly set versions
-	RepositoryCache  string
+	DependencyVersions map[string]string
+	RepositoryConfig   string
+	RepositoryCache    string
 }
 
 // Build rebuilds a local charts directory from a lockfile.
@@ -123,6 +125,13 @@ func (m *Manager) Update() error {
 		return nil
 	}
 
+	// Allow dependency versions to be overwritten.
+	for _, d := range c.Metadata.Dependencies {
+		if v, ok := m.DependencyVersions[d.Name]; ok {
+			d.Version = v
+		}
+	}
+
 	// Check that all of the repos we're dependent on actually exist and
 	// the repo index names.
 	repoNames, err := m.resolveRepoNames(req)