Skip to content

docs: correct path for logs in docs#3640

Merged
georglauterbach merged 1 commit intomasterfrom
correct-docs
Nov 14, 2023
Merged

docs: correct path for logs in docs#3640
georglauterbach merged 1 commit intomasterfrom
correct-docs

Conversation

@georglauterbach
Copy link
Copy Markdown
Member

Description

Self-explanatory.

Type of change

  • This change is a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code

@georglauterbach georglauterbach added area/documentation kind/update Update an existing feature, configuration file or the documentation labels Nov 14, 2023
@georglauterbach georglauterbach added this to the v13.0.0 milestone Nov 14, 2023
@georglauterbach georglauterbach self-assigned this Nov 14, 2023
@georglauterbach georglauterbach enabled auto-merge (squash) November 14, 2023 16:09
Moreover, I removed the path to `mail.log` because this is literally
what you get with `docker logs ...`.
@github-actions
Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: ed46299


If you need more flexibility than what the `docker logs` command offers, then the most useful locations to get relevant DMS logs within the container are:

- `/var/log/mail/mail.log`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This one still exists and doesn't represent a single service AFAIK? Is it worth mentioning?

ln -s /var/log/mail/mail.log /var/log/mail.log

sedfile -i -r '/\/var\/log\/mail\/mail.log/d' /etc/logrotate.d/rsyslog

Copy link
Copy Markdown
Member Author

@georglauterbach georglauterbach Nov 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can probably be removed.

Nevermind: I mixed something up. I think it's not worth mentioning that additonally.

@georglauterbach georglauterbach merged commit d8ebf59 into master Nov 14, 2023
@georglauterbach georglauterbach deleted the correct-docs branch November 14, 2023 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation kind/update Update an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants