Adds ability to add multiple base directories to kustomization

pkg/commands/addbase.go

@@ -1,5 +1,5 @@
 /*
-Copyright 2017 The Kubernetes Authors.
+Copyright 2018 The Kubernetes Authors.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -19,6 +19,7 @@ package commands
 import (
 	"errors"
 	"fmt"
+	"strings"
 
 	"github.com/spf13/cobra"
 
@@ -27,7 +28,7 @@ import (
 )
 
 type addBaseOptions struct {
-	baseDirectoryPath string
+	baseDirectoryPaths string
 }
 
 // newCmdAddBase adds the file path of the kustomize base to the kustomization file.
@@ -36,9 +37,9 @@ func newCmdAddBase(fsys fs.FileSystem) *cobra.Command {
 
 	cmd := &cobra.Command{
 		Use:   "base",
-		Short: "Adds a directory path to a base kustomization to the current directory's kustomization file.",
+		Short: "Adds one or more bases to the kustomization.yaml in current directory",
 		Example: `
-		add base {filepath}`,
+		add base {filepath1},{filepath2}`,
 		RunE: func(cmd *cobra.Command, args []string) error {
 			err := o.Validate(args)
 			if err != nil {
@@ -59,7 +60,7 @@ func (o *addBaseOptions) Validate(args []string) error {
 	if len(args) != 1 {
 		return errors.New("must specify a base directory")
 	}
-	o.baseDirectoryPath = args[0]
+	o.baseDirectoryPaths = args[0]
 	return nil
 }
 
@@ -70,11 +71,6 @@ func (o *addBaseOptions) Complete(cmd *cobra.Command, args []string) error {
 
 // RunAddBase runs addBase command (do real work).
 func (o *addBaseOptions) RunAddBase(fsys fs.FileSystem) error {
-	_, err := fsys.Stat(o.baseDirectoryPath)
-	if err != nil {
-		return err
-	}
-
 	mf, err := newKustomizationFile(constants.KustomizationFileName, fsys)
 	if err != nil {
 		return err
@@ -85,11 +81,19 @@ func (o *addBaseOptions) RunAddBase(fsys fs.FileSystem) error {
 		return err
 	}
 
-	if stringInSlice(o.baseDirectoryPath, m.Bases) {
+	// split directory paths
-		return fmt.Errorf("base %s already in kustomization file", o.baseDirectoryPath)
+	paths := strings.Split(o.baseDirectoryPaths, ",")
+	for _, path := range paths {
+		_, err := fsys.Stat(path)
+		if err != nil {
+			return err
 		}
+		if stringInSlice(path, m.Bases) {
+			return fmt.Errorf("base %s already in kustomization file", path)
+		}
+		m.Bases = append(m.Bases, path)
 
-	m.Bases = append(m.Bases, o.baseDirectoryPath)
+	}
 
 	return mf.write(m)
 }

pkg/commands/addbase_test.go

@@ -1,5 +1,5 @@
 /*
-Copyright 2017 The Kubernetes Authors.
+Copyright 2018 The Kubernetes Authors.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -26,16 +26,19 @@ import (
 )
 
 const (
-	baseDirectoryPath = "my/path/to/wonderful/base"
+	baseDirectoryPaths = "my/path/to/wonderful/base,other/path/to/even/more/wonderful/base"
 )
 
 func TestAddBaseHappyPath(t *testing.T) {
 	fakeFS := fs.MakeFakeFS()
-	fakeFS.Mkdir(baseDirectoryPath, 0777)
+	bases := strings.Split(baseDirectoryPaths, ",")
+	for _, base := range bases {
+		fakeFS.Mkdir(base, 0777)
+	}
 	fakeFS.WriteFile(constants.KustomizationFileName, []byte(kustomizationContent))
 
 	cmd := newCmdAddBase(fakeFS)
-	args := []string{baseDirectoryPath}
+	args := []string{baseDirectoryPaths}
 	err := cmd.RunE(cmd, args)
 	if err != nil {
 		t.Errorf("unexpected cmd error: %v", err)
@@ -44,19 +47,25 @@ func TestAddBaseHappyPath(t *testing.T) {
 	if err != nil {
 		t.Errorf("unexpected read error: %v", err)
 	}
-	if !strings.Contains(string(content), baseDirectoryPath) {
+
-		t.Errorf("expected patch name in kustomization")
+	for _, base := range bases {
+		if !strings.Contains(string(content), base) {
+			t.Errorf("expected base name in kustomization")
+		}
 	}
 }
 
 func TestAddBaseAlreadyThere(t *testing.T) {
 	fakeFS := fs.MakeFakeFS()
-	// Create fake directory
+	// Create fake directories
-	fakeFS.Mkdir(baseDirectoryPath, 0777)
+	bases := strings.Split(baseDirectoryPaths, ",")
+	for _, base := range bases {
+		fakeFS.Mkdir(base, 0777)
+	}
 	fakeFS.WriteFile(constants.KustomizationFileName, []byte(kustomizationContent))
 
 	cmd := newCmdAddBase(fakeFS)
-	args := []string{baseDirectoryPath}
+	args := []string{baseDirectoryPaths}
 	err := cmd.RunE(cmd, args)
 	if err != nil {
 		t.Fatalf("unexpected cmd error: %v", err)
@@ -66,9 +75,15 @@ func TestAddBaseAlreadyThere(t *testing.T) {
 	if err == nil {
 		t.Errorf("expected already there problem")
 	}
-	if err.Error() != "base "+baseDirectoryPath+" already in kustomization file" {
+	var expectedErrors []string
+	for _, base := range bases {
+		error := "base " + base + " already in kustomization file"
+		expectedErrors = append(expectedErrors, error)
+		if !stringInSlice(error, expectedErrors) {
 			t.Errorf("unexpected error %v", err)
 		}
+	}
+
 }
 
 func TestAddBaseNoArgs(t *testing.T) {

pkg/commands/commands.go

@@ -93,8 +93,9 @@ func newCmdAdd(fsys fs.FileSystem) *cobra.Command {
 	# Adds a patch to the kustomization
 	kustomize edit add patch <filepath>
 
-	# Adds a base directory to the kustomization
+	# Adds one or more base directories to the kustomization
 	kustomize edit add base <filepath>
+	kustomize edit add base <filepath1>,<filepath2>,<filepath3>
 `,
 		Args: cobra.MinimumNArgs(1),
 	}