Bug 239559 - gallery2 update 2.2-0.2.svn20070506.fc5 breaks existing installs
Summary: gallery2 update 2.2-0.2.svn20070506.fc5 breaks existing installs
Alias: None
Product: Fedora
Classification: Fedora
Component: gallery2   
(Show other bugs)
Version: 5
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: John Berninger
QA Contact: Fedora Extras Quality Assurance
: 239619 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2007-05-09 14:18 UTC by Bill McGonigle
Modified: 2007-11-30 22:12 UTC (History)
1 user (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-05-14 18:44:48 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
list of installed gallery2 packages (2.61 KB, text/plain)
2007-05-09 17:15 UTC, Bill McGonigle
no flags Details

Description Bill McGonigle 2007-05-09 14:18:35 UTC
Description of problem:

After the update was installed via yum, e.g.:

  May 09 06:13:08 Updated: gallery2-quotas.noarch 2.2-0.2.svn20070506.fc5

the formerly-working gallery finds itself redirecting to /install/

Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. have a working gallery
2. allow update of version 2.2-0.2.svn20070506.fc5
Actual results:

  back to installer

Expected results:

  gallery continues to work

Additional info:

  I don't have a solution yet - I opened this bug in case others are coming in
this morning and finding the same problem - maybe somebody else will find the
problem first.  

  If not, I'll modify the bug to reflect the root cause.

Comment 1 John Berninger 2007-05-09 16:23:10 UTC
Can you provide a full listing of all gallery2 packages you have installed? 
(rpm -qa | grep gallery2)

Comment 2 Bill McGonigle 2007-05-09 17:15:02 UTC
Created attachment 154410 [details]
list of installed gallery2 packages

Attaching a list of packages.

I haven't yet figured this out - it seems that this symptom can happen if the
config.php file is missing - unfortunately, /srv/ wasn't in my backup routine
so I'm not sure what it looked like before, but now /srv/gallery2/ looks like:

 #ls -la /srv/gallery2/
 total 16
 drwxr-xr-x 2 apache root 4096 May  9 06:14 ./
 drwxr-xr-x 4 root   root 4096 Mar 14 05:15 ../
 -rw-r--r-- 1 root   root   33 Dec  3 23:32 login.txt~

- note the modification date on the directory is the same time yum installed
the update, so I'm guessing the rpm script at least 'touch'-ed the directory -
perhaps deleted something.

Comment 3 John Berninger 2007-05-09 18:31:02 UTC
Where is your g2data directory located, and is it intact?  Your current theory
seems to me to be correct - I believe I didn't handle the config.php file
properly in the upgrade; RPM should have saved the config.php file since it was
marked as a config(noreplace) file but it seems that didn't happen.  What
appears in your /etc/gallery2 directory currently?  If there is a config.php in
there, can you attach it?  I'm attempting to reproduce this by setting up a
working Gallery2 installation using the old package versions, but I'm running
into issues since I no longer have an FC5 machine available.

Comment 4 Bill McGonigle 2007-05-09 19:32:05 UTC
Judging by the date, I'd guess this config.php came from the rpm:

#ls -l /etc/gallery2/config.php 
-rw-r--r-- 1 apache root 170 May  7 11:34 /etc/gallery2/config.php

Not certain what was there previously, but it would have been whatever was there
from the prior package.

#cat /etc/gallery2/config.php
<?php /* This file intentionally empty - it is populated during Gallery's setup
/ configuration process*/ 
$gallery->setConfig('data.gallery.base', '/srv/gallery2/');

I *think* there was a /srv/gallery2/config.php but I'd only bet small dollars on

g2data is at /var/lib/gallery2/g2data/ and appears to be untouched by the update
(by mtime, anyhow).

I could probably setup an fc5 vm for you if it'd help.

Comment 5 Bill McGonigle 2007-05-09 19:40:57 UTC
Oops - forgot to paste this in:

#ls -la /etc/gallery2/
total 24
drwxr-xr-x   2 root   root  4096 May  9 06:14 ./
drwxr-xr-x 110 root   root 12288 May  9 11:23 ../
-rw-r--r--   1 apache root   170 May  7 11:34 config.php
-rw-r--r--   1 root   root     0 May  7 11:34 login.txt

Comment 6 John Berninger 2007-05-10 01:36:05 UTC
*** Bug 239619 has been marked as a duplicate of this bug. ***

Comment 7 Joel 2007-05-10 02:39:47 UTC
My data directory on /media/Drive 4/gallery was untouched by the update 
(FC6).  However my /etc/gallery2/config.php was wiped by the update.  At 
minimum it should have been moved to config.php.rpmsave.  rpm updates can't 
wipe user configurations!  ( I was the creator of bug 239619 on FC6 )

Comment 8 John Berninger 2007-05-10 02:50:00 UTC
The config file issue is one of the items I'm addressing.  Removing config.php
was a result of not checking my specfile carefully enough.  As soon as I've
figured out a recovery procedure, or as close to one as I can, I'll post it here.

Comment 9 John Berninger 2007-05-10 03:42:57 UTC
OK, I'm about to release new packages.  To recover from the failed upgrades will
require a bit more work than just updating to the new packages, unfortunately -
my apologies for that.

First, you'll need to revert to the 2.1 packages.  I've made these available at
the following URLs for convenience:
FC-5: http://www.ncphotography.com/g2dist/FC5/
FC-6: http://www.ncphotography.com/g2dist/FC6/
Please note this is not an official Fedora repository, I've simply made the
gallery2 2.1 packages available here for convenience.  These are exact copies of
the 2.1 packages available on the download.fedora.redhat.com server.

You'll need to go through the initial setup procedure, being sure to specify the
same data store and database information, and the installer will realize you
already have valid data and ask if you want to re-use the existing information.
 Say yes here.  You should then have your original gallery back up and running.

With the new packages I'm about to push, the upgrade should work much better -
the config.php will be saved and you will be prompted to copy it to
/etc/gallery2, then change a symlink in /usr/share/gallery2.  I've tested this
on a local machine starting from a fresh gallery2-*2.1* install and it worked as
expected.  You should also be prompted to step through the upgrade procedure at
http://<hostname>/gallery2/upgrade, which I've also run through, and which
resulted in a functional, upgraded gallery for me.

Most of the Fedora Release Engineering team is at the Summit this week, so I
can't guarantee when the newly built packages will be pushed to live repo's, but
 I have initiated the builds for them.  Look for the 0.3.svn20070506 packages.

Comment 10 Bill McGonigle 2007-05-10 03:52:19 UTC
I'm not sure anybody more sane than me could call this a 'recovery procedure'
but what I've done is to run 'strings' on the block device the file was on,
redirected to a file on another block device, and then used 'less' to search for
gallery->setConfig to find the config.php contents.

ext3 seems to be abysmal for 'undelete' types of operations.

This works - unfortunately 'strings' kills whitespace and /* and */ lines, so
some comment reconstructing was necessary.

After I recovered my config.php file, I was greeted with the Gallery 2.2
Upgrader.  It completed successfully and we're back in business.

There was one warning during the upgrade, and I'm not sure if it's important or not:

  Gallery File Integrity
  [something about modified files]

Even if the config.php problem hadn't happened, it's worth noting that gallery
installs all over the place would have effectively been offline waiting for
admins to enter the password and click, click, click.  

I understand why that is, and wouldn't want to risk excluding gallery2 from yum,
as I'm keen on getting security updates, but I can wish for a world with
backported security patches, right? :)

John, don't feel too bad, this happens on many packages, more often than I care
to admit to clients.  It would be nice if there was a lint for SPEC files that
would throw warnings on easy-to-make mistakes like this, and maybe the build
process could throw on them.

If you can, pester the release folks to get this out now - it's going to save
alot of people headaches, especially knuckleheads like me without properly
functioning backups.

Comment 11 John Berninger 2007-05-10 04:04:35 UTC
I'm talking with some of the release folks now.

This wasn't so much a security fix, though, as correcting some egregiously nasty
hacks I put in place to handle a read-only /usr.  Even had I not updates to 2.2,
the same issue would have cropped up when I shifted config.php into it's new
home-away-from-home.  Basically, I left out a %config(noreplace) on the new
symlink, not thinking about what would happen there.

The file integrity messages are something I need to take up with upstream - they
can be ignored as far as I'm aware, I believe they're due to changing permission
bits after generating the manifest in the build process.  I just haven't had the
bandwidth to check up on that when I've been thinking of it lately.

Comment 12 Bill McGonigle 2007-05-10 04:09:18 UTC
Just for completeness's sake, where is the proper location for config.php at
this point?

Thanks for appreciating the need for a read-only /usr on some installs, and
especially for maintaining gallery2 in Extras so we can have it package-managed
and updated.

Comment 13 John Berninger 2007-05-10 04:21:18 UTC
For the 2.1 packages, it's /usr/share/gallery2/.  For 2.2, /etc/gallery2/ with a
symlink to it in /usr/share/gallery2/.

Comment 14 John Berninger 2007-05-10 21:37:42 UTC
OK, looks like the new packages have made it out to the repos.  I'm going to set
this bug NEEDINFO - feel free to bounce it back to me if you encounter problems,
or close it if your happy.

Comment 15 Jérôme Benoit 2007-05-11 19:38:07 UTC
Maybe related to this bug (maybe not), the lastest FC6 gallery2 update show a
postinstall failed script error. 

Updating  : gallery2                     ######################### [3/6] 
/var/tmp/rpm-tmp.13379: line 3: syntax error near unexpected token `('
/var/tmp/rpm-tmp.13379: line 3: `       echo Your old configuration file
(config.php) has not been replaced.'
error: %post(gallery2-2.2-0.3.svn20070506.fc6.noarch) scriptlet failed, exit
status 2

I have not the time right now to dig this pb further. 

Comment 16 John Berninger 2007-05-13 13:40:19 UTC
Fortunately, the post scriptlet consists entirely of echo statements, and does
not actually do any work.  I've corrceted the syntax and new packages are
building now.

Comment 17 Bill McGonigle 2007-05-14 00:45:11 UTC
I was able to update an existing gallery2 install with the first set of new
packages (my mirror apparently didn't have the latest), but I think there's
still one bug - or I'm misunderstanding.

I had a config.php in /usr/share/gallery2.  The update made a symlink for:

  config.php.rpmnew -> ../../../etc/gallery2/config.php

but left my config.php there.  I expected it would move it to /etc/gallery2/ and
then symlink it.  I did that by hand, and the prompt for upgrade came up fine.

Comment 18 John Berninger 2007-05-14 18:44:48 UTC
That's intended behavior.  I couldn't come up with a reliable way to only move
the config file if you're upgrading from the 2.1 package to the 2.2 versus from
2.2 to a later 2.2.

Note You need to log in before you can comment on or make changes to this bug.