This feature (and the corresponding tests) were causing intermittent
failures which we were unable to trace.  We will reintroduce this
behavior when we can do so without test fragility.

Reviewed-By: Matt Robinson

This includes various style changes, and assorted fixes to testing.

Paired-With: Matt Robinson

Here is a changeset that adds a new action to the puppet node face.

This application removes all traces of a node on the puppetmaster
(including certs, cached facts and nodes, reports, and storedconfig
entries).

Furthermore it is capable of unexporting exported resources of a
host so that consumers of these resources can remove the exported
resources and we will safely remove the node from our
infrastructure.

Usage:
puppet node clean [--unexport] <host> [<host2> ...]

To achieve this we add different destroy methods to the different
parts of the indirector. So for example for yaml indirections we
already offer read access for the yaml, this changeset adds
the destroy handler which only removes the yaml file for
a request. This can be used to remove cached entries.

This work is based on the initial work of Brice Figureau
<brice-puppet@daysofwonder.com>

By default, the SSL-related faces should all render a strings,
not with `Object#inspect`.

Reviewed-By: Daniel Pittman

The additional quotation marks frustrate certain syntax highlighters,
and are completely unnecessary for their use.

Reviewed-By: Daniel Pittman

The validation for the ca_location option on the certificate
application continued to hang around on the application long
after the face realized its potential to take responsibility
for itself.  This change moves (and adds) validation code as
appropriate into the Face.

Reviewed-By: Matt Robinson

Part of the progress toward getting the `puppet status` invocation working
nicely is that it should default to invoking the `find` operation.  This
implements that, using the new runtime default action facility.

Reviewed-By: Pieter van de Bruggen <pieter@puppetlabs.com>

The Puppet::SSL::CertificateAuthority::Interface class was an early prototype
heading toward building out a system like Faces.  Now that we have done that,
this changeset ports the early code to a new face.

Reviewed-By: Pieter van de Bruggen <pieter@puppetlabs.com>

This commit makes the following changes:

* Changes wrapping on some "returns" blocks to accomodate the way we prepend
  "RETURNS: " to the first line in short help.
* Overrides description for save on faces where save is invalid. (Since save has
  an inherited description, the description was contradicting the summary.)
* Adds notes on dummy arguments to short_description where applicable, so as to
  show up in short help.
* Adds the termini list to the action short help template.
* Removes the authors block from short help for faces. (Save it for the man page.)
* Several trivial wording changes.

Reviewed-By: Jacob Helwig <jacob@puppetlabs.com>

Per UX review of help text, this commit makes several changes over the breadth
of the Faces help:

* Preface API-only action summaries/descriptions with "API only." (issue #7775)
* Provide both CLI and API info in "returns," with the CLI info first. (issue #7778)
* Summaries should be sentences. (Add punctuation.)
* First sentences of descriptions should reiterate summaries. (Summaries and
  descriptions should be displayed far enough apart that this isn't a problem.)
* Standardize on "subcommand" instead of "face" when talking about the entity
  you invoke at the command line. (Use "face" when describing API use.)
* Fix outdated or clunky text in several faces.