From 31afff294400b5a69bdb3ec387ecdf5bad57a038 Mon Sep 17 00:00:00 2001
From: Michael Crosby <crosbymichael@gmail.com>
Date: Tue, 11 Jun 2019 23:44:39 +0000
Subject: [PATCH] Fix backwards compat with v2 containerd configs

Signed-off-by: Michael Crosby <crosbymichael@gmail.com>
---
 .travis.yml                                  | 6 ------
 cmd/containerd/command/config.go             | 5 +++++
 cmd/containerd/command/config_linux.go       | 2 +-
 cmd/containerd/command/config_unsupported.go | 2 +-
 cmd/containerd/command/config_windows.go     | 2 +-
 5 files changed, 8 insertions(+), 9 deletions(-)

diff --git a/.travis.yml b/.travis.yml
index f0711aac5df1..c2698acbe37d 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -74,11 +74,6 @@ script:
   - go build -i .
   - make check
   - if [ "$GOOS" = "linux" ]; then make check-protos check-api-descriptors; fi
-  - |
-    sudo mkdir -p /etc/containerd
-    sudo bash -c "cat > /etc/containerd/config.toml <<EOF
-    version = 1
-    EOF"
   - make build
   - make binaries
   - if [ "$TRAVIS_GOOS" = "linux" ]; then sudo make install ; fi
@@ -91,7 +86,6 @@ script:
     if [ "$TRAVIS_GOOS" = "linux" ]; then
       sudo mkdir -p /etc/containerd
       sudo bash -c "cat > /etc/containerd/config.toml <<EOF
-      version = 1
       [plugins.cri.containerd.default_runtime]
         runtime_type = \"${TEST_RUNTIME}\"
     EOF"
diff --git a/cmd/containerd/command/config.go b/cmd/containerd/command/config.go
index de9790ecfe0e..6eec996cb7b1 100644
--- a/cmd/containerd/command/config.go
+++ b/cmd/containerd/command/config.go
@@ -50,6 +50,11 @@ var configCommand = cli.Command{
 				config := &Config{
 					Config: defaultConfig(),
 				}
+				// for the time being, keep the defaultConfig's version set at 1 so that
+				// when a config without a version is loaded from disk and has no version
+				// set, we assume it's a v1 config.  But when generating new configs via
+				// this command, generate the v2 config
+				config.Config.Version = 2
 				plugins, err := server.LoadPlugins(gocontext.Background(), config.Config)
 				if err != nil {
 					return err
diff --git a/cmd/containerd/command/config_linux.go b/cmd/containerd/command/config_linux.go
index 3bbc0ab0dc8a..9ed7368d741f 100644
--- a/cmd/containerd/command/config_linux.go
+++ b/cmd/containerd/command/config_linux.go
@@ -23,7 +23,7 @@ import (
 
 func defaultConfig() *srvconfig.Config {
 	return &srvconfig.Config{
-		Version: 2,
+		Version: 1,
 		Root:    defaults.DefaultRootDir,
 		State:   defaults.DefaultStateDir,
 		GRPC: srvconfig.GRPCConfig{
diff --git a/cmd/containerd/command/config_unsupported.go b/cmd/containerd/command/config_unsupported.go
index 0408ac23023b..695a7b466a26 100644
--- a/cmd/containerd/command/config_unsupported.go
+++ b/cmd/containerd/command/config_unsupported.go
@@ -25,7 +25,7 @@ import (
 
 func defaultConfig() *srvconfig.Config {
 	return &srvconfig.Config{
-		Version: 2,
+		Version: 1,
 		Root:    defaults.DefaultRootDir,
 		State:   defaults.DefaultStateDir,
 		GRPC: srvconfig.GRPCConfig{
diff --git a/cmd/containerd/command/config_windows.go b/cmd/containerd/command/config_windows.go
index 3bbc0ab0dc8a..9ed7368d741f 100644
--- a/cmd/containerd/command/config_windows.go
+++ b/cmd/containerd/command/config_windows.go
@@ -23,7 +23,7 @@ import (
 
 func defaultConfig() *srvconfig.Config {
 	return &srvconfig.Config{
-		Version: 2,
+		Version: 1,
 		Root:    defaults.DefaultRootDir,
 		State:   defaults.DefaultStateDir,
 		GRPC: srvconfig.GRPCConfig{