Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upUse UTC Timezone #303
Closed
Use UTC Timezone #303
Labels
Comments
|
I'll start working on this. |
|
It's all yours! Let me know if you have any questions. |
ghost
mentioned this issue
|
I think I've correctly added this, should I go about working on adding NTP support, or will that be done from a separate issue? |
|
Let's save NTP for another issue/PR; that one will be more involved because we will need to write custom Salt code for that. (It's also blocked on #350, which will enable us to load custom modules and states). |
|
@charlesvdv is interested in taking this up now: #439 (comment) |
Merged
bors-servo
added a commit
that referenced
this issue
Jan 7, 2017
Use UTC timezone Fix the issue #303 and follow up of #439 PR. I'm not sure about the spot to put the ```sls.common``` tests. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/569) <!-- Reviewable:end -->
|
Looks like this was fixed by #569. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
This should be applied to all machines, so this should go into the
common/init.slsfile.We can use the
timezone.systemstate to configure a UTC timezone (docs).I think this would be a great E-Easy issue.
(A follow-up would be ensuring that NTP is on and configured on all machines, but there's no Salt state for this yet so that wouldn't be E-Easy.)