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

[Documentation] [Low priority] Add to the config.toml template and documentation a section for logging #7284

Closed
uriel-kluk opened this issue May 2, 2024 · 2 comments
Assignees

Comments

@uriel-kluk
Copy link

After running iotedge check I get the following warning

‼ production readiness: logs policy - Warning
    Container engine is not configured to rotate module logs which may cause it run out of disk space.
    Please see https://aka.ms/iotedge-prod-checklist-logs for best practices.
    You can ignore this warning if you are setting log policy per module in the Edge deployment.

The link points to the config.toml documentation, but the section for log-policy doesn't exist.

Suggestion: add the log-policy description to the best practices document.

@yophilav yophilav self-assigned this May 6, 2024
@yophilav
Copy link
Contributor

yophilav commented May 8, 2024

@uriel-kluk
Thank you for reporting the issue. 😄 The document https://aka.ms/iotedge-prod-checklist-logs suggested that we updated the log policy in daemon.json which configured the underlying docker log service. On the other hand, the config.toml is a configuration that we used to configure the iotedged. I believe the document is accurate. Please let me know if I'm missing something. Thanks!

@ryanwinter
Copy link

Closing as no action required. @uriel-kluk please reopen if you think this is still an issue or have more questions.

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

3 participants