Upgrade from 2.14: attachstatusdefs not found

RESOLVED FIXED in Bugzilla 2.18

Status

()

Bugzilla
Installation & Upgrading
RESOLVED FIXED
14 years ago
5 years ago

People

(Reporter: Jouni Heikniemi, Assigned: justdave)

Tracking

2.17.7
Bugzilla 2.18
Bug Flags:
approval +
blocking2.18 +

Details

Attachments

(1 attachment)

(Reporter)

Description

14 years ago
When upgrading from 2.14, I get the following in checksetup:

DBD::mysql::st execute failed: Table 'bugs_214.attachstatusdefs' doesn't exist
at checksetup.pl line 2166.
DBD::mysql::st fetchrow_arrayref failed: fetch() without execute() at
checksetup.pl line 2168.

The migration code doesn't check for existence of attachstatusdefs?
(Reporter)

Updated

14 years ago
Flags: blocking2.18?
Target Milestone: --- → Bugzilla 2.18
Flags: blocking2.18? → blocking2.18+
I reproduced this on landfill using one of the older databases still sitting
around on there.  Tracked it down to the code that converts products from names
to IDs.  At the time that was checked in, attachment statuses were still around,
so it's converting the product field on the attachment status table to product_id.

Patch coming up.
Assignee: zach → justdave
Created attachment 152260 [details] [diff] [review]
Patch v1
Attachment #152260 - Flags: review?(bugreport)

Comment 3

14 years ago
Comment on attachment 152260 [details] [diff] [review]
Patch v1

r=joel
noting that I cannot test it
Attachment #152260 - Flags: review?(bugreport) → review+
Here's how you can test this on landfill:

1) Create a bugzilla install running CVS tip using the site tools and make note
of the name of the database it creates for it.
2) $ mysqldrop $dbname
3) $ mysqlclone bugs_bugzilla2_14_branch $dbname
4) $ ./checksetup.pl
5) Verify that the output doesn't contain any errors.
OK, Joel verified in IRC that he could reproduce this and the patch fixed it.
Checking in...

Checking in checksetup.pl;
/cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v  <--  checksetup.pl
new revision: 1.285; previous revision: 1.284
done
Status: NEW → RESOLVED
Last Resolved: 14 years ago
Flags: approval+
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.