Use lxc clone by default

cmd/juju/deploy.go

 // Copyright 2012, 2013 Canonical Ltd.
 // Licensed under the AGPLv3, see LICENCE file for details.
 
 package main
 
 import (
         "errors"
         "fmt"
         "os"
         "strings"
@@ skipping 48 matching lines @@
   local:precise/mysql
 
 <service name>, if omitted, will be derived from <charm name>.
 
 Constraints can be specified when using deploy by specifying the --constraints
 flag.  When used with deploy, service-specific constraints are set so that later
 machines provisioned with add-unit will use the same constraints (unless changed
 by set-constraints).
 
 Charms can be deployed to a specific machine using the --to argument.
+If the destination is an LXC container, as of trusty, the default is
+to use lxc-clone to create the container. A 'template' container is
+created with the name

[menn0, 2014/05/15 00:41:03]

Given that we're moving towards supporting non-Ubuntu workloads, is it a good
idea to be so tied to Ubuntu concepts in the help output?

What about something like:

If the destination is an LXC container the default is to use lxc-clone to create
the container where possible. For Ubuntu deployments, lxc-clone is supported for
the trusty OS series and later. A 'template' container is created with the
name...

[wallyworld, 2014/05/15 10:12:57]

Done.

+  juju-<series>-template
+where <series> is the OS series, for example 'juju-trusty-template'.
+
+You can override the use of clone by changing the provider configuration:

[menn0, 2014/05/15 00:41:03]

s/clone/lxc-clone/

[wallyworld, 2014/05/15 10:12:57]

"clone" here is correct as it is referring to the clone concept itself

+  lxc-clone: false
+
+If you have the main container directory mounted on a btrfs partition,
+then the clone will be using btrfs snapshots to create the containers.
+This means that the clones use up much less disk space.  If you do not have btrfs,

[menn0, 2014/05/15 00:41:03]

s/that the clones/that clones will/

[wallyworld, 2014/05/15 10:12:57]

Done.

+lxc will attempt to use aufs (which is an overlay type filesystem). You can

[menn0, 2014/05/15 00:41:03]

s/which is an overlay type filesystem/an overlay filesystem/

[wallyworld, 2014/05/15 10:12:57]

Done.

+explicitly ask Juju to create full containers and not overlays by specifying
+the following in the provider configuration:
+  lxc-clone-aufs: false
+
+Examples:
+   juju deploy mysql --to 23       (Deploy to machine 23)
+   juju deploy mysql --to 24/lxc/3 (Deploy to lxc container 3 on host machine 24)
+   juju deploy mysql --to lxc:25   (Deploy to a new lxc container on host machine 25)
+···
+   juju deploy mysql -n 5 --constraints mem=8G (deploy 5 instances of mysql with at least 8 GB of RAM each)

[menn0, 2014/05/15 00:41:03]

Capitalisation is inconsistent here: Deploy vs deploy

[wallyworld, 2014/05/15 10:12:57]

Done.

+
+   juju deploy mysql --networks=storage,mynet --exclude-networks=logging
 
 Like constraints, service-specific network requirements can be
 specified with --networks and --exclude-networks arguments, both can
 take a comma-delimited list of provider-specific network names/labels.
 These instruct juju to ensure to add all the networks specified with
 --networks to all new machines deployed to host units of the service
 and to ensure none of the networks in --exclude-networks are added to
 the service's machines. Not supported on all providers.
 
-Examples:
-   juju deploy mysql --to 23       (Deploy to machine 23)
-   juju deploy mysql --to 24/lxc/3 (Deploy to lxc container 3 on host machine 24)
-   juju deploy mysql --to lxc:25   (Deploy to a new lxc container on host machine 25)
-···
-   juju deploy mysql -n 5 --constraints mem=8G (deploy 5 instances of mysql with at least 8 GB of RAM each)
-
-   juju deploy mysql --networks=storage,mynet --exclude-networks=logging
-
 See Also:
    juju help constraints
    juju help set-constraints
    juju help get-constraints
 `
 
 func (c *DeployCommand) Info() *cmd.Info {
         return &cmd.Info{
                 Name:    "deploy",
                 Args:    "<charm name> [<service name>]",
@@ skipping 185 matching lines @@
 func parseNetworks(networksValue string) (networks []string) {
         parts := strings.Split(networksValue, ",")
         for _, part := range parts {
                 network := strings.TrimSpace(part)
                 if network != "" {
                         networks = append(networks, names.NetworkTag(network))
                 }
         }
         return networks
 }