RESOLVED FIXED Bug 220589
[build.webkit.org] Use PostgreSQL for new build.webkit.org database 
https://bugs.webkit.org/show_bug.cgi?id=220589
Summary [build.webkit.org] Use PostgreSQL for new build.webkit.org database
Aakash Jain
Reported 2021-01-13 06:15:40 PST
Use PostgreSQL for upcoming build.webkit.org database since it's much better than the default sqlite. The new build.webkit.org would be running latest Buildbot (which uses database heavily).
Attachments
Patch (2.06 KB, patch)
2021-01-13 06:39 PST, Aakash Jain 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Aakash Jain
Comment 1 2021-01-13 06:36:04 PST
Also, it might be better to use passwords.json to store db info (like db url, username etc.) instead of environment variables.
Aakash Jain
Comment 2 2021-01-13 06:39:55 PST
Created attachment 417529 [details] Patch
Radar WebKit Bug Importer
Comment 3 2021-01-13 06:44:15 PST
<rdar://problem/73149418>
lingho@apple.com
Comment 4 2021-01-13 09:51:41 PST
rs+
Jonathan Bedard
Comment 5 2021-01-13 10:55:07 PST
Comment on attachment 417529 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=417529&action=review > Tools/CISupport/build-webkit-org/master_buildbot2.cfg:-2 > -# ex: set syntax=python: Sure we want to remove these lines?
Aakash Jain
Comment 6 2021-01-13 12:04:38 PST
(In reply to Jonathan Bedard from comment #5) > Sure we want to remove these lines? Yeah, I don't see much benefit of keeping them. EWS master.cfg also doesn't have these.
EWS
Comment 7 2021-01-13 12:21:40 PST
Committed r271450: <https://trac.webkit.org/changeset/271450> All reviewed patches have been landed. Closing bug and clearing flags on attachment 417529 [details].
Note You need to log in before you can comment on or make changes to this bug.