Default to DBD::SQLite for new installs for "out of the box" working bugzilla experience

RESOLVED FIXED in Bugzilla 6.0

Status

()

--
enhancement
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: dylan, Assigned: dylan)

Tracking

unspecified
Bugzilla 6.0

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

3 years ago
I would like to be able to checkout bugzilla, run cpanm && checksetup.pl && plackup and have it work. For that another thing that is needed is a working database -- we can do this with SQLite. It's not great for production but drastically lowers the bar to contributing to Bugzilla.

Updated

3 years ago
Assignee: general → installation
Severity: normal → enhancement
Component: Bugzilla-General → Installation & Upgrading
(Assignee)

Comment 1

3 years ago
Created attachment 8716678 [details] [diff] [review]
1244528_1.patch
Assignee: installation → dylan
Status: NEW → ASSIGNED
Attachment #8716678 - Flags: review?(dkl)

Comment 2

3 years ago
Comment on attachment 8716678 [details] [diff] [review]
1244528_1.patch

>     {
>         name    => 'db_name',
>-        default => 'bugs',
>+        default => 'data/bugzilla.sqlite3',
>     },

Why do you edit db_name? By default, the sqlite DB will be installed in data/db/, which is fine. I see no reason to put it in data/ itself. This would also prevent trouble with other DBs (such as MySQL or Pg), because your change makes us think that you have to specify the full path to the DB, which is incorrect. So please revert this change.


>     {
>         name    => 'db_user',
>-        default => 'bugs',
>+        default => '',
>     },

This change is also useless. It's totally fine to have db_user set, and is a good default. Please revert this change too.
(Assignee)

Comment 3

3 years ago
Created attachment 8716803 [details] [diff] [review]
1244528_2.patch

I am very uncomfortable with the level of magic we're doing in our subclass of DBD::SQLite but I'll reserve that battle for another day.
Attachment #8716678 - Attachment is obsolete: true
Attachment #8716678 - Flags: review?(dkl)
Attachment #8716803 - Flags: review?(dkl)
Comment on attachment 8716803 [details] [diff] [review]
1244528_2.patch

Review of attachment 8716803 [details] [diff] [review]:
-----------------------------------------------------------------

r=dkl
Attachment #8716803 - Flags: review?(dkl) → review+
(Assignee)

Comment 5

3 years ago
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
   9faa0dc..6bd668c  master -> master
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED

Updated

3 years ago
Target Milestone: --- → Bugzilla 6.0
You need to log in before you can comment on or make changes to this bug.