Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LOG_DRIVER=journald no longer works in 1.6 #950

Closed
v1ru535 opened this issue Feb 24, 2019 · 4 comments
Closed

LOG_DRIVER=journald no longer works in 1.6 #950

v1ru535 opened this issue Feb 24, 2019 · 4 comments

Comments

@v1ru535
Copy link

v1ru535 commented Feb 24, 2019

Hi all,

I've upgrade from 1.5 to 1.6 one week ago.

LOG_DRIVER with 1.5 was OK (journald), but now with 1.6 I've a problem with it.
I've set LOG_DRIVER=journald in my mailu.env but nothing appear in the logs on my host.
I've test with option syslog but same appear.

The previous issue in 1.5 for this option was #584.

Thanks for the investigation.

journalctl -b CONTAINER_NAME=mailu_front_1
-- No entries --
@UnrealKazu
Copy link

I just migrated from 1.5 to 1.6, and am able to use the journald log driver. Using the same commands I do get a log trace, so it is working for me.

Are you sure that the LOG_DRIVER setting from the mailu.env file is being used in your docker run or docker-compose command?

@tete2soja
Copy link

Hello,

I get the same error. I didn't migrate from any previous version.

@daniellibonati
Copy link

Hi, I've experienced the same behavior and I've found the docker-compose.yml file contains a hard reference to the logging driver:

# Core services
  front:
    image: ${DOCKER_ORG:-mailu}/${DOCKER_PREFIX:-}nginx:${MAILU_VERSION:-1.6}
    restart: always
    env_file: mailu.env
    logging:
      driver: json-file
    ports:
    ...
    volumes:
    ...

Seems the auto-generated docker-compose.yml no longer reads this value from the environment. Replacing json-file with journald or syslog worked fine for me.

@v1ru535
Copy link
Author

v1ru535 commented Jun 27, 2019

Wow, I hadn't seen this parameter, my eyes have a real problem... I think...

So, I added:

    logging:
      driver: journald

On all container and it works like a charm 👍
Now my old jails are OK:
#584 (comment)

Thanks Superman dixit @daniellibonati :]

@v1ru535 v1ru535 closed this as completed Jun 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants