[PATCH v3] Avoid timezone confusion

Stephen Finucane stephen at that.guru
Tue Feb 27 22:31:22 AEDT 2018


On Thu, 2018-02-22 at 16:24 +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>

This all looks good to me and you've resolved Daniel's once outstanding
comment via the additional reno. As such,

Reviewed-by: Stephen Finucane <stephen at that.guru>

I'll give it a few days to give Daniel the chance to sign off before
applying this.

Stephen


More information about the Patchwork mailing list