Merge "Allow log directories to be configured"

playbooks/roles/bifrost-ironic-install/README.md

@@ -155,6 +155,15 @@ remote_syslog_server: String value, default undefined. If set, rsyslog is
 remote_syslog_port: String value, default is 514. If set, custom port is
                     configured for remote syslog server.
 
+ironic_log_dir: String value, default undefined. If set, it specifies a
+                a non-default log directory for ironic.
+
+inspector_log_dir: String value, default undefined. If set, it specifies a
+                   non-default log directory for inspector.
+
+nginx_log_dir: String value, default /var/log/nginx. It specifies a log
+               directory for nginx.
+
 ### Hardware Inspection Support
 
 Bifrost also supports the installation of ironic-inspector in standalone

playbooks/roles/bifrost-ironic-install/defaults/main.yml

@@ -133,6 +133,15 @@ inspector_manage_firewall: false
 # Deprecated: ironic_auth_strategy will be removed in Pike.
 ironic_auth_strategy: "noauth"
 
+# Set ironic_log_dir to use a non-default log directory for ironic.
+#ironic_log_dir:
+
+# Set inspector_log_dir to use a non-default log directory for inspector.
+#inspector_log_dir:
+
+# Set nginx_log_dir to use a non-default log directory for nginx.
+nginx_log_dir: /var/log/nginx
+
 inspector_data_dir: "/opt/stack/ironic-inspector/var"
 inspector_store_ramdisk_logs: true
 # Note: inspector_port_addition has three valid values: all, active, pxe

playbooks/roles/bifrost-ironic-install/templates/ironic-inspector.conf.j2

@@ -11,6 +11,10 @@ auth_strategy = {{ inspector_auth | default('noauth') }}
 {% endif %}
 debug = {{ inspector_debug | bool }}
 
+{% if inspector_log_dir is defined %}
+log_dir = {{ inspector_log_dir }}
+{% endif %}
+
 [database]
 connection=mysql+pymysql://{{ ironic_inspector.database.username }}:{{ ironic_inspector.database.password }}@{{ ironic_inspector.database.host }}/{{ ironic_inspector.database.name }}?charset=utf8
 

playbooks/roles/bifrost-ironic-install/templates/ironic.conf.j2

@@ -26,6 +26,10 @@ auth_strategy = keystone
 auth_strategy = noauth
 {% endif %}
 
+{% if ironic_log_dir is defined %}
+log_dir = {{ ironic_log_dir }}
+{% endif %}
+
 [pxe]
 pxe_append_params = systemd.journald.forward_to_console=yes {{ extra_kernel_options | default('') }}
 pxe_config_template = $pybasedir/drivers/modules/ipxe_config.template

playbooks/roles/bifrost-ironic-install/templates/nginx.conf.j2

@@ -30,8 +30,8 @@ http {
 	# Logging Settings
 	##
 
-	access_log /var/log/nginx/access.log;
+	access_log {{ nginx_log_dir }}/access.log;
-	error_log /var/log/nginx/error.log;
+	error_log {{ nginx_log_dir }}/error.log;
 
 	##
 	# Gzip Settings

releasenotes/notes/log-directories-87b86df40464bb2d.yaml

@@ -0,0 +1,11 @@
+---
+features:
+  - |
+    Allows log directories to be configured
+
+    In some cases it is useful to be able to configure the directory into which
+    log files are written by bifrost services.
+
+    It is now possible to configure the ironic, inspector and nginx log
+    directories using the ``ironic_log_dir``, ``inspector_log_dir``, and
+    ``nginx_log_dir`` variables respectively.