Revert "Probes: Fix failing nosetests"

This reverts commit 4f745cc2731f7035f02566ba8bc1a0e9ae1b1a71.

This breaks handling of unicode on python 2.
Probes: Fix unicode probe handling

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
Probes: Disable builtin redefinition check

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
POSIX: Fix unicode file verification

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
POSIX: Fix pylint failure

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
Client: Don't modify running services if ignored

Previously, when a bundle's contents were updated, all services listed
in the bundle were stopped if the status attribute was set to 'ignore'.

Signed-off-by: Sol Jerome <sol.jerome@gmail.com>
Metadata: log now client creation with profile
tools: added git_commit.py script
Metadata: metadata generation improvements

Primarily, this fixes a few potential bugs in initial metadata, which
may result in inconsistent ordering of groups, which can be a problem
when relying on category suppression.

This also deduplicates some category suppression code, and moves some
nested functions to top-level methods, which are surprisingly faster.
Metadata: search groups breadth-first when building metadata

This makes category negation consistent, and also makes it work like
it did in Bcfg2 1.2
Metadata: removed stray blank line
doc: updated pros/cons for builtin yum libraries
Reporting: have bcfg2-report-collector be explicit about detaching

Following the same logic as 360ba2e7, we should be explicit about
the need to detach the bcfg2-report-collector process.

Side note: this bit me because I was starting the bcfg2 server
processes via SSH, and the way python-daemon checks for being
started by inetd is to see if stdin is a socket. (??)