[request-tracker-maintainers] Bug#773343: request-tracker4: fails to upgrade from 'wheezy' if rt4-extension-assettracker is installed

[Niko Tyni]

tag 773343 patch
thanks

On Sat, Dec 27, 2014 at 07:27:03PM +0200, Niko Tyni wrote:

> The smallest database change I can think of to fix the problem is
> something like
>  UPDATE Groups set Instance=1 WHERE Domain='RTx::AssetTracker::System-Role' AND Description IS Null;
> which could presumably be dropped into the request-tracker4 dbconfig-common
> upgrade directories. I'd expect it to complete as a no-op on systems
> without RTx::AssetTracker installed.

The attached patch implements this. It's also pushed in the
ntyni/bug-773343 branch in the Alioth repository (see the Vcs-Git header.)

> Disclaimer: I don't really know what the 'Instance' field is used for,
> and the above change could well break RTx::AssetTracker or even RT
> itself. I've only tested that it allows the request-tracker4 upgrade to
> complete successfully with the (default) SQLite backend.  Testing with
> Pg/mysql would be good too, as the 'Description IS Null' part feels a
> bit fragile to me.

It turns out that Pg and mysql backends don't exhibit the issue at all,
so I'm only touching SQlite databases. I've now tested the patch as
far as successfully upgrading dbconfig-common managed RT installations
from wheezy to jessie+the patch on SQLite, Pg, and mysql, both with and
without rt4-extension-assettracker installed.

However, I have not actually tested to use RT after the patched
upgrade. It would be nice if somebody could do that.

> On a related note, does the wheezy RTx::AssetTracker even work with the
> jessie RT? If it doesn't, a Breaks: entry would probably be warranted.

I've added the Breaks as well in the git branch.
-- 
Niko Tyni   ntyni at debian.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: assettracker-sysgroups.diff
Type: text/x-diff
Size: 1085 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-request-tracker-maintainers/attachments/20141228/29b3b50e/attachment.diff>