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

Share time with the host #36

Closed
wants to merge 1 commit into from

Conversation

Random-Liu
Copy link
Member

@Random-Liu Random-Liu commented Oct 27, 2016

Kernel monitor discard old kernel log based on the timestamp.

However, usually kernel log timestamp doesn't contain year and timezone information, e.g. Oct 23 16:59:23. There is no way to apply the right timezone, if the timezone inside the pod is different from the host.

This PR mount the /etc/localtime inside the container to make sure the container in the same zone with the host.

Sometimes /etc/localtime may be a symlink file. I've tested that both docker and kubernetes mount will follow the symlink and mount the real content.

@dchen1107


This change is Reviewable

@dchen1107
Copy link
Member

I am closing this one since #39, which covers this issue, was merged. Please reopen it if I were wrong. Thanks!

@dchen1107 dchen1107 closed this Feb 1, 2017
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

Successfully merging this pull request may close these issues.

2 participants