From 50e8f5e5ec58e685362421e5bf46bddb0f225197 Mon Sep 17 00:00:00 2001
From: Valentin Lab <valentin.lab@kalysto.org>
Date: Tue, 18 Dec 2018 09:24:00 +0100
Subject: [PATCH] new: [odoo-tecnativa] logs are not managed by default

Do not redirect logs to ``/var/log/odoo`` unless logrotate is linked.
---
 odoo-tecnativa/hooks/log_rotate-relation-joined | 3 +++
 odoo-tecnativa/metadata.yml                     | 4 +---
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/odoo-tecnativa/hooks/log_rotate-relation-joined b/odoo-tecnativa/hooks/log_rotate-relation-joined
index 823e4a43..726aeeba 100755
--- a/odoo-tecnativa/hooks/log_rotate-relation-joined
+++ b/odoo-tecnativa/hooks/log_rotate-relation-joined
@@ -47,4 +47,7 @@ services:
   $MASTER_BASE_SERVICE_NAME:
     volumes:
       - $SERVICE_DATASTORE$LOGS:$LOGS:rw
+    command:
+      - \"--logfile=$LOGS/odoo.log\"
+
 "
diff --git a/odoo-tecnativa/metadata.yml b/odoo-tecnativa/metadata.yml
index e330743a..99d0a567 100644
--- a/odoo-tecnativa/metadata.yml
+++ b/odoo-tecnativa/metadata.yml
@@ -1,7 +1,6 @@
 docker-image: git.panda-chi.io:5002/panda-chi/docker-odoo-pandachi:11.0-all-module-test
 data-resources:
   - /var/lib/odoo
-  - /var/log/odoo
 # config-resources:
 #   ## XXXvlab: should this be there or declared dynamically in the relation that
 #   ## create/update this file ?
@@ -11,7 +10,6 @@ docker-compose:
   command:
     - odoo
     - --config=/opt/odoo/auto/odoo.conf
-    - --logfile=/var/log/odoo/odoo.log
   ## YYYvlab: useful ?
   tty: true
   environment:
@@ -33,7 +31,7 @@ uses:
     constraint: recommended
     auto: pair
     solves:
-      disk-leak: "/var/log/odoo"
+      unmanaged-logs: "in docker logs"
     #default-options:
   web-proxy:
     #constraint: required | recommended | optional