Re: pg_upgrade not detecting version properly

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Chris Ernst <cernst(at)zvelo(dot)com>
Cc: pgsql-general(at)postgresql(dot)org
Subject: Re: pg_upgrade not detecting version properly
Date: 2012-10-10 17:54:49
Message-ID: 20121010175449.GG11892@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-general pgsql-hackers

On Wed, Oct 10, 2012 at 10:35:06AM -0600, Chris Ernst wrote:
> On 10/10/2012 09:56 AM, Bruce Momjian wrote:
> > Can you show me what is in the PG_VERSION file in the old cluster? It
> > should be "9.1".
>
> Hi Bruce,
>
> Thank you for the reply. Indeed it is "9.1":
>
> # cat /postgresql/9.1/main/PG_VERSION
> 9.1
>
> And just for good measure:
>
> cat /postgresql/9.2/main/PG_VERSION
> 9.2
>
> And there are no other PostgreSQL versions on this machine.
>
> Hmm... I was just about to send this when something else occurred to me.
> I had initially tried to run pg_upgrade as root and it said it couldn't
> be run as root. So I've been running it as my own user (which is in the
> postgres group). However, everything in /postgresql/9.1/main is owned
> by postgres with 700 permissions.
>
> I switched to the postgres user and now pg_upgrade is running. Perhaps
> just a more informative error message is in order.
>
> Thank you for the shove in the right direction =)

Oops, that code was returning zero if it couldn't open the file. The
attached, applied patch to head and 9.2 issues a proper error message.

Seems this "zero return" has been in the code since the beginning. :-(

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

Attachment Content-Type Size
open.diff text/x-diff 929 bytes

In response to

Responses

Browse pgsql-general by date

  From Date Subject
Next Message Alban Hertroys 2012-10-10 18:11:38 Re: pymssql Connection to the database failed for an unknown reason
Previous Message Greg Sabino Mullane 2012-10-10 17:45:33 Re: Planner chooses multi-column index in 9.2 when maybe it should not

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2012-10-10 17:57:31 Re: Minor document updates
Previous Message Daniel Frey 2012-10-10 17:32:16 Bug / feature request for floating point to string conversion