Merge lp:~harlowja/cloud-init/cloud-init-file-logging into lp:~cloud-init-dev/cloud-init/trunk

[Joshua Harlow]

It's less problematic to just depend on basic file logging to avoid syslog patterns, syslog being active, syslog being useable as a core dependency of getting debug logs and other from cloudinit.

[Server Team CI bot (server-team-bot)]

FAILED: Continuous integration, rev:1259
https://server-team-jenkins.canonical.com/job/cloud-init-ci/38/
Executed test runs:
    None: https://server-team-jenkins.canonical.com/job/lp-vote-on-merge/12/console

Click here to trigger a rebuild:
https://server-team-jenkins.canonical.com/job/cloud-init-ci/38/rebuild

review: Needs Fixing (continuous-integration)

[Scott Moser (smoser)]

Hello,
Thank you for taking the time to contribute to cloud-init. Cloud-init has moved its revision control system to git. As a result, we are marking all bzr merge proposals as 'rejected'. If you would like to re-submit this proposal for review, please do so by following the current HACKING documentation at http://cloudinit.readthedocs.io/en/latest/topics/hacking.html .

I think this is actually already merged at
https://git.launchpad.net/cloud-init/commit/?id=11b1aba459737

=== modified file 'config/cloud.cfg.d/05_logging.cfg'
--- config/cloud.cfg.d/05_logging.cfg	2014-01-14 18:18:14 +0000
+++ config/cloud.cfg.d/05_logging.cfg	2016-07-21 18:01:22 +0000
@@ -54,13 +54,21 @@
 
 log_cfgs:
 # These will be joined into a string that defines the configuration
- - [ *log_base, *log_syslog ]
+#
+# If your desire syslog, just uncomment the following line.
+# - [ *log_base, *log_syslog ]
+#
 # These will be joined into a string that defines the configuration
+#
+# Otherwise by default we just use the most basic (known to work
+# mechanism that does not depend on a system service to operate; which
+# due to cloud-init execution lifecycle we may not always be able to
+# depend on working correctly).
  - [ *log_base, *log_file ]
-# A file path can also be used
+# A file path can also be used.
 # - /etc/log.conf
 
-# this tells cloud-init to redirect its stdout and stderr to
+# This tells cloud-init to redirect its stdout and stderr to
 # 'tee -a /var/log/cloud-init-output.log' so the user can see output
 # there without needing to look on the console.
 output: {all: '| tee -a /var/log/cloud-init-output.log'}