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

Use yesterday's files to avoid any time zones troubles #49

Merged
merged 1 commit into from Apr 21, 2015
Merged

Use yesterday's files to avoid any time zones troubles #49

merged 1 commit into from Apr 21, 2015

Conversation

adrianreber
Copy link
Member

No description provided.

@adrianreber
Copy link
Member Author

This should silence the emails from cron.

@pypingou
Copy link
Member

what about using date -u instead?

@adrianreber
Copy link
Member Author

Ah, good idea. Let me respin.

@adrianreber
Copy link
Member Author

No, actually, the problem is not our system which already is on UTC. The problem is the remote system with the Internet2 tables which does not create the new tables because they seem to be on their local time which is multiple hours behind UTC. Looking at the URL now it has the right tables for "today" but the tables were not there a few hours ago. So my current patch still seems to be right fix.

@pypingou
Copy link
Member

ok, mind adding this as a comment in the file before merging then?

Otherwise 👍 for me

adrianreber added a commit that referenced this pull request Apr 21, 2015
Use yesterday's files to avoid any time zones troubles. PR updated with more comments why yesterday's date is the right approach.
@adrianreber adrianreber merged commit 3cae796 into fedora-infra:master Apr 21, 2015
@adrianreber adrianreber deleted the yesterday branch April 27, 2015 12:21
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.

None yet

2 participants