oo-diagnostics: Fix test_node_mco_log when no log

Make test_node_mco_log fail gracefully when the node mcollective log file
is absent.  Before this commit, the test would hang because it would run
a sed command with no input file provided.

This commit fixes bug 1077664.
oo-diagnostics: Drop unneeded require 'socket'

Drop a "require 'socket'" statement in the test_broker_certificate check.

This require statement was obviated in commit
9b72939f28a6d35354db74cc7464d879bbd3a81e.
oo-diagnostics: Better handle curl error

Catch and report if curl returns an error in the test_broker_certificate
check.

Previously, test_broker_certificate would ignore curl's return code, and so
errors connecting to the broker would turn up as backtraces when
test_broker_certificate would try to parse the output from curl.
oo-diagnostics: Fix comments for test_node_mco_log

Place comments with relevant code, consistently use proper case and
punctuation, and use whitespace for better logical grouping.
corrected section number
Bug 1096591 - Escape filenames for downloadable cartridges
Merge pull request #5410 from lnader/master

Merged by openshift-bot
update oo-exec-ruby for RHSCL-1.1, requires v8 runtime paths also
Merge pull request #5412 from jwhonce/bug/1096591

Merged by openshift-bot
Merge pull request #5413 from maxamillion/admiller/rhscl_1.1-RC2

Merged by openshift-bot
Merge pull request #4983 from Miciah/bug-1077664-oo-diagnostics-test_node_mco_log-fixes

Merged by openshift-bot
Use the local1 facility for Syslog trace logs

Allow trace log segregation by logging trace messages using the local1
facility.

Resolve bug: https://bugzilla.redhat.com/show_bug.cgi?id=1097745
Merge pull request #5414 from ironcladlou/syslog-facility

Merged by openshift-bot
Merge pull request #5324 from Miciah/oo-diagnostics-test_broker_certificate-fixes

Merged by openshift-bot
apache-vhost: fix BZ 1090358 on moving custom certs

Bug 1090358 - Can't move gear of app with alias ssl cert between nodes
with vhost frontend
https://bugzilla.redhat.com/show_bug.cgi?id=1090358

When oo-admin-move copies a gear over from one node to another, it
copies over the frontend config files too. Then it reconstructs the
frontend from JSON anyway. This causes a problem with apache-vhost if a
gear alias has a custom cert.

The frontend construction first creates the aliases. Each gets a simple
config file. Crucially, this is only created if there is not already a
custom cert and vhost for the same alias. In the case of a move, since
the custom cert vhost conf is moved over, it already exists, so alias
conf is not created.

Then it creates the custom cert vhosts. This first checks that the alias
config file already exists and throws an exception if not. In the case
of a move, it doesn't exist and isn't recreated, so the frontend
reconstruction fails and the entire move is rolled back.

The fix simply skips raising that exception in the case that the custom
cert vhost conf file already exists, which should only be the case for
gear moves.
Merge pull request #5417 from sosiouxme/bz1090358

Merged by openshift-bot
Bug 1097959 - Add THROTTLER_CHECK_PERIOD to detune Throttler

* Add THROTTLER_CHECK_PERIOD element to /etc/sysconfig/watchman to
  allow Operator to set period for checking cgroup counters
Bug 1088623 - Removed unused .uservars directory references
Merge pull request #5418 from jwhonce/bug/1097959

Merged by openshift-bot
Merge pull request #5420 from jwhonce/bug/1088623

Merged by openshift-bot