agora inbox for pgsql-committers@postgresql.org  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: Andrew Dunstan <andrew@dunslane.net>
Cc: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Update time zone data files to tzdata release 2024b.
Date: Wed, 30 Oct 2024 19:07:56 -0400
Message-ID: <871448.1730329676@sss.pgh.pa.us> (raw)
In-Reply-To: <868637.1730328216@sss.pgh.pa.us>
References: <362289.1730241666@sss.pgh.pa.us>
	<55A5E01C-1B0F-40F9-94EA-FEDED1575C92@dunslane.net>
	<595311.1730300440@sss.pgh.pa.us>
	<CAD5tBcLHz6NLCW_65qYka08HRVeT6w15Cd1YFKmHV5Ou43Fu4g@mail.gmail.com>
	<764322.1730320400@sss.pgh.pa.us>
	<CAD5tBcLetMh6sG0SpDB4fPTFDt37=_q_nq3eOExQPK6qHOgnQA@mail.gmail.com>
	<CAD5tBcJC8tuSAzC7b5GdGv0p8O9d1tziHoD01aW8F_WmVj11aw@mail.gmail.com>
	<868637.1730328216@sss.pgh.pa.us>

I wrote:
> ... So I'm fuzzy about the exact details of
> what has happened on crake, but I'm pretty sure that it boils
> down to "the dump timezone changed since 2018".

Oh ... never mind, I was thinking in terms of what would happen
with --use-system-timezone, but of course the buildfarm doesn't
build that way.

So the problem is precisely that *our* interpretation of EST5EDT
changed when we adopted tzdata 2024b, and that is affecting how
we dump these old timestamps.  Or at least, that seems like what
should be happening, but then why is only crake showing a failure?

> I'm inclined to propose that we should modify the pg_upgrade test
> so that it forces the comparison dumps to be taken with PGTZ=UTC,
> thereby taking system-timezone changes out of the picture.

Whatever the details exactly, this still seems like a good
future-proof fix.  I'm not quite sure where we need to make the change
though --- if I edit pg_upgrade/t/002_pg_upgrade.pl, will that affect
how the buildfarm tests these things, or is it different code?

			regards, tom lane





view thread (377+ messages)  latest in thread

Message-ID: <871448.1730329676@sss.pgh.pa.us>
Permalink:  ../871448.1730329676@sss.pgh.pa.us/
Also on:    postgresql.org/message-id/871448.1730329676@sss.pgh.pa.us

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, andrew@dunslane.net, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Update time zone data files to tzdata release 2024b.
  In-Reply-To: <871448.1730329676@sss.pgh.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox