clean code - remove V1 implementations

.travis.yml

@@ -19,11 +19,6 @@ go_import_path: sigs.k8s.io/kubebuilder
 
 services: docker
 
-# Install must be set to prevent default `go get` to run.
-# The dependencies have already been vendored by `dep` so
-# we don't need to fetch them.
-install: skip
-
 script: ./test.sh
 
 jobs:

CONTRIBUTING.md

@@ -11,7 +11,6 @@ Please see https://git.k8s.io/community/CLA.md for more info.
 ## Prerequisites
 
 - [go](https://golang.org/dl/) version v1.13+.
-- [dep](https://github.com/golang/dep) dep v0.5+
 - [docker](https://docs.docker.com/install/) version 17.03+.
 - [kubectl](https://kubernetes.io/docs/tasks/tools/install-kubectl/) version v1.11.3+.
 - [kustomize](https://sigs.k8s.io/kustomize/docs/INSTALL.md) v3.1.0+

Makefile

@@ -61,11 +61,6 @@ generate: ## Update/generate all mock data. You should run this commands to upda
 .PHONY: generate-testdata
 generate-testdata: ## Update/generate the testdata in $GOPATH/src/sigs.k8s.io/kubebuilder
 	GO111MODULE=on ./generate_testdata.sh
-
-.PHONY: generate-vendor
-generate-vendor: ## (Deprecated) Update/generate the vendor by using the path $GOPATH/src/sigs.k8s.io/kubebuilder-test
-	GO111MODULE=off ./generate_vendor.sh
-
 .PHONY: lint
 lint: ## Run code lint checks
 	./scripts/verify.sh

check_testdata.sh

@@ -23,7 +23,6 @@ export TRACE=1
 export GO111MODULE=on
 
 fetch_tools
-install_go_dep
 build_kb
 
 setup_envs

cmd/edit.go

@@ -76,12 +76,6 @@ func (o *editOptions) bindFlags(cmd *cobra.Command) {
 }
 
 func (o *editOptions) Validate() error {
-	if !o.config.IsV2() && !o.config.IsV3() {
-		if o.config.MultiGroup {
-			return fmt.Errorf("multiple group support can't be enabled for version %s", o.config.Version)
-		}
-	}
-
 	return nil
 }
 

cmd/main.go

@@ -20,19 +20,23 @@ import (
 	"log"
 
 	"sigs.k8s.io/kubebuilder/cmd/version"
+	"sigs.k8s.io/kubebuilder/internal/config"
 	"sigs.k8s.io/kubebuilder/pkg/cli"
-	pluginv1 "sigs.k8s.io/kubebuilder/pkg/plugin/v1"
 	pluginv2 "sigs.k8s.io/kubebuilder/pkg/plugin/v2"
 )
 
 func main() {
+
+	// before create the cmd tree ensure that the project version is supported
+	if _, err := config.IsProjectVersionSupported(); err != nil {
+		log.Fatal(err)
+	}
+
 	c, err := cli.New(
 		cli.WithPlugins(
-			&pluginv1.Plugin{},
 			&pluginv2.Plugin{},
 		),
 		cli.WithDefaultPlugins(
-			&pluginv1.Plugin{},
 			&pluginv2.Plugin{},
 		),
 		cli.WithExtraCommands(

common.sh

@@ -155,33 +155,6 @@ function build_kb {
   GO111MODULE=on go build $opts -o $tmp_root/kubebuilder/bin/kubebuilder ./cmd
 }
 
-# Required to install the go dep in the https://prow.k8s.io/.
-function install_dep_by_git {
-  header_text "Checking for dep"
-  export PATH=$(go env GOPATH)/src/github.com/golang/dep/bin:$PATH
-  if ! is_installed dep ; then
-    header_text "Installing dep"
-    DEP_DIR=$(go env GOPATH)/src/github.com/golang/dep
-    mkdir -p $DEP_DIR
-    pushd $DEP_DIR
-    git clone https://github.com/golang/dep.git .
-    DEP_LATEST=$(git describe --abbrev=0 --tags)
-    git checkout $DEP_LATEST
-    mkdir bin
-    GO111MODULE=off go build -ldflags="-X main.version=$DEP_LATEST" -o bin/dep ./cmd/dep
-    popd
-  fi
-}
-
-function install_go_dep {
-  header_text "Checking for dep"
-  export PATH=$(go env GOPATH)/src/github.com/golang/dep/bin:$PATH
-  if ! is_installed dep ; then
-    header_text "Installing dep"
-    curl https://raw.githubusercontent.com/golang/dep/master/install.sh | sh
-  fi
-}
-
 function install_kind {
   header_text "Checking for kind"
   if ! is_installed kind ; then
@@ -218,27 +191,6 @@ function setup_envs {
   export TEST_DEP=$tmp_root/kubebuilder/init_project
 }
 
-# download_vendor_archive downloads vendor tarball for v1 projects. It skips the
-# download if tarball exists.
-function download_vendor_archive {
-  archive_name="vendor.v1.tgz"
-  archive_download_url="https://storage.googleapis.com/kubebuilder-vendor/$archive_name"
-  archive_path="$tmp_root/$archive_name"
-  header_text "checking the path $archive_path to download the $archive_name"
-  if [ -f $archive_path ]; then
-    header_text "removing file which exists"
-    rm $archive_path
-  fi
-  header_text "downloading vendor archive from $archive_download_url"
-  curl -sL ${archive_download_url} -o "$archive_path"
-}
-
-function restore_go_deps {
-  header_text "restoring Go dependencies"
-  download_vendor_archive
-  tar -zxf $tmp_root/vendor.v1.tgz
-}
-
 function cache_project {
   header_text "caching initialized projects"
   if [ -d "$TEST_DEP" ]; then
@@ -252,6 +204,5 @@ function dump_project {
   header_text "restoring cached project"
   if [ -d "$TEST_DEP" ]; then
     cp -r $TEST_DEP/* .
-    restore_go_deps
   fi
 }

generate_testdata.sh

@@ -37,25 +37,7 @@ scaffold_test_project() {
     cd testdata/$project
     kb=$testdata_dir/../bin/kubebuilder
     oldgopath=$GOPATH
-    if [ $version == "1" ]; then
-        export GO111MODULE=off
-        export GOPATH=$(pwd)/../.. # go ignores vendor under testdata, so fake out a gopath
-        # untar Gopkg.lock and vendor directory for appropriate project version
-        download_vendor_archive
-        tar -zxf $tmp_root/vendor.v$version.tgz
-
-        $kb init --project-version $version --domain testproject.org --license apache2 --owner "The Kubernetes authors" --dep=false
-        $kb create api --group crew --version v1 --kind FirstMate --controller=true --resource=true --make=false
-        $kb alpha webhook --group crew --version v1 --kind FirstMate --type=mutating --operations=create,update --make=false
-        $kb alpha webhook --group crew --version v1 --kind FirstMate --type=mutating --operations=delete --make=false
-        $kb create api --group ship --version v1beta1 --kind Frigate --example=false --controller=true --resource=true --make=false
-        $kb alpha webhook --group ship --version v1beta1 --kind Frigate --type=validating --operations=update --make=false
-        $kb create api --group creatures --version v2alpha1 --kind Kraken --namespaced=false --example=false --controller=true --resource=true --make=false
-        $kb alpha webhook --group creatures --version v2alpha1 --kind Kraken --type=validating --operations=create --make=false
-        $kb create api --group core --version v1 --kind Namespace --example=false --controller=true --resource=false --namespaced=false --make=false
-        $kb alpha webhook --group core --version v1 --kind Namespace --type=mutating --operations=update --make=false
-        $kb create api --group policy --version v1beta1 --kind HealthCheckPolicy --example=false --controller=true --resource=true --namespaced=false --make=false
-    elif [ $version == "2" ] || [ $version == "3-alpha" ]; then
+    if [ $version == "2" ] || [ $version == "3-alpha" ]; then
         header_text "Starting to generate projects with version $version"
         header_text "Generating $project"
 
@@ -96,10 +78,8 @@ scaffold_test_project() {
             $kb create api --group crew --version v1 --kind Admiral --controller=true --resource=true --namespaced=false --make=false --pattern=addon
         fi
     fi
-    make all test # v2 doesn't test by default
-    rm -f Gopkg.lock
+    make all test
     rm -f go.sum
-    rm -rf ./vendor
     rm -rf ./bin
     export GOPATH=$oldgopath
     popd
@@ -108,7 +88,6 @@ scaffold_test_project() {
 set -e
 
 build_kb
-scaffold_test_project gopath/src/project 1
 scaffold_test_project project-v2 2
 scaffold_test_project project-v2-multigroup 2
 scaffold_test_project project-v2-addon 2

internal/config/config.go

@@ -32,6 +32,9 @@ const (
 
 	// DefaultVersion is the version which will be used when the version flag is not provided
 	DefaultVersion = config.Version3Alpha
+
+	// To print error with colored
+	NoticeColor = "\033[1;36m%s\033[0m"
 )
 
 func exists(fs afero.Fs, path string) (bool, error) {
@@ -179,3 +182,23 @@ type saveError struct {
 func (e saveError) Error() string {
 	return fmt.Sprintf("unable to save the configuration: %v", e.err)
 }
+
+// IsProjectVersionSupported returns true if the project is already configured and it is v2
+func IsProjectVersionSupported() (bool, error) {
+	cfg, err := Read()
+	if os.IsNotExist(err) {
+		return true, nil
+	}
+
+	if err != nil {
+		return false, err
+	}
+
+	if cfg.IsV1() {
+		err = fmt.Errorf(NoticeColor, "The v1 projects are no longer supported.\n"+
+			"See how to upgrade your project to v2: https://book.kubebuilder.io/migration/guide.html\n")
+		return false, err
+	}
+
+	return true, nil
+}

pkg/cli/cli.go

@@ -32,7 +32,6 @@ import (
 )
 
 const (
-	noticeColor         = "\033[1;36m%s\033[0m"
 	runInProjectRootMsg = `For project-specific information, run this command in the root directory of a
 project.
 `
@@ -249,7 +248,7 @@ func (c *cli) initialize() error {
 	// Write deprecation notices after all commands have been constructed.
 	for _, p := range c.resolvedPlugins {
 		if d, isDeprecated := p.(plugin.Deprecated); isDeprecated {
-			fmt.Printf(noticeColor, fmt.Sprintf("[Deprecation Notice] %s\n\n",
+			fmt.Printf(internalconfig.NoticeColor, fmt.Sprintf("[Deprecation Notice] %s\n\n",
 				d.DeprecationWarning()))
 		}
 	}
@@ -359,16 +358,17 @@ func validatePlugins(plugins ...plugin.Base) error {
 // buildRootCmd returns a root command with a subcommand tree reflecting the
 // current project's state.
 func (c cli) buildRootCmd() *cobra.Command {
-	configuredAndV1 := c.configured && c.projectVersion == config.Version1
+	// before create the cmd tree ensure that the project version is supported
+	if _, err := internalconfig.IsProjectVersionSupported(); err != nil {
+		log.Fatal(err)
+	}
 
 	rootCmd := c.defaultCommand()
 
 	// kubebuilder alpha
 	alphaCmd := c.newAlphaCmd()
-	// kubebuilder alpha webhook (v1 only)
-	if configuredAndV1 {
-		alphaCmd.AddCommand(c.newCreateWebhookCmd())
-	}
+
+	// Only add alpha group if it has subcommands
 	if alphaCmd.HasSubCommands() {
 		rootCmd.AddCommand(alphaCmd)
 	}
@@ -377,10 +377,7 @@ func (c cli) buildRootCmd() *cobra.Command {
 	createCmd := c.newCreateCmd()
 	// kubebuilder create api
 	createCmd.AddCommand(c.newCreateAPICmd())
-	// kubebuilder create webhook (!v1)
-	if !configuredAndV1 {
-		createCmd.AddCommand(c.newCreateWebhookCmd())
-	}
+	createCmd.AddCommand(c.newCreateWebhookCmd())
 	if createCmd.HasSubCommands() {
 		rootCmd.AddCommand(createCmd)
 	}