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.
added Servlet 3 dependency

Importing this template application in Eclipse shows error as the snoop.jsp depends on the Servlet API. It will improve user first time experience.
Modified directions so that manually create repo file will be updated by Puppet module when run
nsupdate_plugin: Exception logging now includes error output from nsupdate command

Bug 1089697
Bugzilla link https://bugzilla.redhat.com/show_bug.cgi?id=1089697
When troubleshooting DNS record addition/removal errors, it is helpful to have the error output from the nsupdate command. Specifically in scenarios where record changes only fail <1% of the time.
Follow rc convention and use Rails.logger to avoid large amounts of text in the exception string
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
Automatic commit of package [openshift-origin-cartridge-phpmyadmin] release [1.22.1-1].
Automatic commit of package [openshift-origin-cartridge-jenkins-client] release [1.21.1-1].