[PATCH v2] Avoid timezone confusion

Stephen Finucane stephen at that.guru
Thu Jan 18 06:18:15 AEDT 2018


On Tue, 2018-01-16 at 18:58 +0100, vkabatov at redhat.com wrote:
> From: Veronika Kabatova <vkabatov at redhat.com>
> 
> Patchwork saves patches, comments etc with UTC timezone and reports
> this time when opening the patch details. However, internally generated
> processes such as events are reported with the instance's local time.
> There's nothing wrong with that and making PW timezone-aware would add
> useless complexity, but in a world-wide collaboration a lot of confusion
> may arise as the timezone is not reported at all. Instance's local time
> might be very different from the local time of CI integrating with PW,
> which is different from the local time of person dealing with it etc.
> 
> Use UTC everywhere by default instead of UTC for sumbissions and local
> timezone for internally generated events (which is not reported).
> 
> Signed-off-by: Veronika Kabatova <vkabatov at redhat.com>

What effect does this have on existing information in the database?
Does this mean they'll all remain UTC+12 or are they stored in UTC
format already? The Django docs [1] would lead me to suggest the
former, given that we don't have USE_TZ set to True.

I guess you can test that by setting up a deployment, creating
information, then switching this option over. I'd do this myself but
I'm not going to have time this week :(

Stephen

[1] https://docs.djangoproject.com/en/2.0/topics/i18n/timezones/


More information about the Patchwork mailing list