~dave-cheney/juju-core/153-fix-release-tools-script

« back to all changes in this revision

Viewing changes to cmd/juju/destroyenvironment.go

  • Committer: Tarmac
  • Author(s): Andrew Wilkins
  • Date: 2013-08-30 01:48:39 UTC
  • mfrom: (1628.11.22 juju-add-machine)
  • Revision ID: tarmac-20130830014839-infsllif52ywy6yq
[r=axwalk] Update add-machine for manual provisioning

juju add-machine is updated to use a new package,
environs/manual, to manually provision tools and
a machine agent to an existing machine.

When a manually provisioned machine is destroyed
via juju destroy-machine, the machine agent will
detect its termination and remove its upstart
configuration file. There is currently no cleanup
of the data or log directories; this will be done
in a follow-up pending discussion.

When the machine goes to Dead, a provisioner will
remove the machine from state just like any other
machine.

TODO: destroy-environment will currently leak
manually provisioned machines. A follow-up will
address this by requiring users to individually
destroy-machine before destroy-environment will
proceed. Alternatively (or perhaps additionally),
destroy-environment may take a flag to automatically
do this.

https://codereview.appspot.com/12831043/

Show diffs side-by-side

added added

removed removed

Lines of Context:
49
49
                }
50
50
        }
51
51
 
 
52
        // TODO(axw) 2013-08-30 bug 1218688
 
53
        // destroy manually provisioned machines, or otherwise
 
54
        // block destroy-environment until all manually provisioned
 
55
        // machines have been manually "destroyed".
52
56
        return environ.Destroy(nil)
53
57
}
54
58