adding support for --bundle-dir -b to start, restore, and spec; fixes…

… issue #310

Signed-off-by: Mike Brown <[email protected]>

main.go

@@ -21,14 +21,14 @@ container runtime environment for applications. It can be used with your
 existing process monitoring tools and the container will be spawned as a
 direct child of the process supervisor.
 
-After creating config files for your root filesystem with runc, you can execute a
-container in your shell by running:
+After creating config files for your root filesystem with runc, you can execute 
+a container in your shell by running:
 
     # cd /mycontainer
-    # runc start [ -c spec-config-file ] [ -r runtime-config-file ]
+    # runc start [ -b bundle-dir ] 
 
-If not specified, the default value for the 'spec-config-file' is 'config.json',
-and the default value for the 'runtime-config-file' is 'runtime.json'.`
+If not specified, the default value for the 'bundle-dir' is './'. 'Bundle-dir' 
+is the location where 'config.json' and 'runtime.json' must be located.`
 )
 
 func main() {

restore.go

@@ -50,22 +50,23 @@ var restoreCommand = cli.Command{
 			Usage: "cgroups mode: 'soft' (default), 'full' and 'strict'.",
 		},
 		cli.StringFlag{
-			Name:  "config-file, c",
-			Value: "config.json",
-			Usage: "path to spec file for writing",
-		},
-		cli.StringFlag{
-			Name:  "runtime-file, r",
-			Value: "runtime.json",
-			Usage: "path for runtime file for writing",
+			Name:  "bundle-dir, b",
+			Value: "./",
+			Usage: "path to the root of the bundle directory",
 		},
 	},
 	Action: func(context *cli.Context) {
 		imagePath := context.String("image-path")
 		if imagePath == "" {
 			imagePath = getDefaultImagePath(context)
 		}
-		spec, rspec, err := loadSpec(context.String("config-file"), context.String("runtime-file"))
+		bundleDir := context.String("bundle-dir")
+		if bundleDir != "./" {
+			if err := os.Chdir(bundleDir); err != nil {
+				fatal(err)
+			}
+		}
+		spec, rspec, err := loadSpec("config.json", "runtime.json")
 		if err != nil {
 			fatal(err)
 		}

spec.go

@@ -26,14 +26,9 @@ var specCommand = cli.Command{
 	Usage: "create a new specification file",
 	Flags: []cli.Flag{
 		cli.StringFlag{
-			Name:  "config-file, c",
-			Value: "config.json",
-			Usage: "path to spec config file for writing",
-		},
-		cli.StringFlag{
-			Name:  "runtime-file, r",
-			Value: "runtime.json",
-			Usage: "path to runtime config file for writing",
+			Name:  "bundle-dir, b",
+			Value: "./",
+			Usage: "path to the root of the bundle directory",
 		},
 	},
 	Action: func(context *cli.Context) {
@@ -247,8 +242,14 @@ var specCommand = cli.Command{
 			}
 			return nil
 		}
-		cName := context.String("config-file")
-		rName := context.String("runtime-file")
+		bundleDir := context.String("bundle-dir")
+		if bundleDir != "./" {
+			if err := os.Chdir(bundleDir); err != nil {
+				fatal(err)
+			}
+		}
+		cName := "config.json"
+		rName := "runtime.json"
 		if err := checkNoFile(cName); err != nil {
 			logrus.Fatal(err)
 		}

start.go

@@ -22,18 +22,19 @@ var startCommand = cli.Command{
 	Usage: "create and run a container",
 	Flags: []cli.Flag{
 		cli.StringFlag{
-			Name:  "config-file, c",
-			Value: "config.json",
-			Usage: "path to spec config file",
-		},
-		cli.StringFlag{
-			Name:  "runtime-file, r",
-			Value: "runtime.json",
-			Usage: "path to runtime config file",
+			Name:  "bundle-dir, b",
+			Value: "./",
+			Usage: "path to the root of the bundle directory",
 		},
 	},
 	Action: func(context *cli.Context) {
-		spec, rspec, err := loadSpec(context.String("config-file"), context.String("runtime-file"))
+		bundleDir := context.String("bundle-dir")
+		if bundleDir != "./" {
+			if err := os.Chdir(bundleDir); err != nil {
+				fatal(err)
+			}
+		}
+		spec, rspec, err := loadSpec("config.json", "runtime.json")
 		if err != nil {
 			fatal(err)
 		}