19. October 2001
Bugzilla Status Update
This is the first in a (hopefully) regular series of status updates about the progress of Bugzilla development. The aim is to let people know about upcoming versions of Bugzilla, so they can better prepare for new features, as well as to raise any concerns about them before they leave CVS and become a stable tarball.
Highly Misleading & Meaningless Statistics
|Date||2.16 Bugs With Patches Waiting For Review||2.16 Bugs Waiting For Patches||2.18 Bugs||Other Bugs|
|2001-10-19||112 bugs||329 bugs||36 bugs||299 bugs|
The 2.14 Release
The 2.14 release went relatively smoothly, after some last minute bu … err hitches were encountered on bugzilla.mozilla.org. This site, for those that don’t know, was the original installation of Bugzilla and is the “shakedown” site for the CVS code when there are no known non-documentation release blockers.
Inevitably release blockers are found when this occurs, and there were blockers for 2.14 just as there were for 2.12. However, the actual release process went relatively smoothly this time around. The process includes updating web pages, preparing release announcements/security advisories, checking in the final documentation, including double checking the release notes, checking out and preparing a tarball, and so on. Generally this process occurs over the space of one frantic day.
The first order of business after 2.14 was to introduce some of the features into CVS that were needed on bugzilla.mozilla.org, but weren’t considered necessary for the 2.14 release because their benefit/risk ratio was not high enough. Remembering that bugzilla.mozilla.org is the bug system that is used for tracking bugs in Bugzilla (as well as Mozilla), we obviously are acutely aware of these issues. These features were checked in and bugzilla.mozilla.org updated to CVS. This point was nicknamed “2.14+”.
The major part of this was the new “attachment manager”. This allows you to edit the attributes of attachments. This includes MIME types, is-patch status, the new is-obsolete status, as well as a feature that allows you to add your own “attachment statuses” to attachments. These can be thought of as to attachments what keywords are to bugs. Examples include “first-review”, “needs-work”, etc.
The checkin of the attachment manager was followed by a flurry of reporting of minor bugs and desired features to be added. These include bugs 97729, 97733, 97739, 97764, 97784, 97825, 97868, 97877, 98201, 98074, 98103, 98110, 98111, 98112, 99215, 99716, 101056, 101770, 103605, 103661 and 104521. Some of these have been since resolved, while others have not.
The other two patches added a summary to “dependent bug has had its status changed” e-mails, as well as fixes to properly shut down a Bugzilla installation when the syncshadowdb script is run. bugzilla.mozilla.org used this script to do backups.
With the introduction of the attachment manager came the start of a process that many Bugzilla administrators will surely cheer - templatisation (bug 86168). With templates, HTML is moved out of the Perl code and into separate files. Dynamically generated data is passed by Bugzilla to the templates and the result is then displayed.
This makes an administrators job easier, firstly, because they don’t need to know any Perl to change the user interface, and secondly, because they don’t need to make changes to their customised templates every time they upgrade.
Previously RedHat’s fork of Bugzilla supported this feature using the “Text::Template” software. However, the attachment manager uses the “Template Toolkit” software for its templates. This is because the Template Toolkit was judged to be technically superior.
Although the templates will be different, and administrators of RH Bugzilla would need to convert their templates to use mainline Bugzilla, this and other recent events indicate we may see a reunification of RH Bugzilla and mainline Bugzilla at some happy point in the future.
The introduction of the Template Toolkit brought up some issues with the software, namely that it failed install tests in certain parts of the world (now fixed) and that the CPAN package did not do proper dependency checking.
The introduction also meant that CVS Bugzilla now requires Perl 5.005 or later, as that is a template toolkit requirement. 2.14 will be the last version of Bugzilla that allows you to use 5.004.
It is hoped that templatisation may be complete by the end of 2.16.
Automatically Preventing Problems
One direction the team is heading is to try and be proactive in preventing problems rather than reacting to them.
During the 2.13 cycle Tinderboxen were introduced to check that Bugzilla compiled. For those not familiar with Tinderbox, it is another mozilla.org “web tool” that regularly checks a piece of software in CVS for problems.
After 2.14, a new “testing suite” has been included in CVS. Currently this checks the code for some bad “code patterns” we have encountered in the past. The testing suite does not do testing in the standard sense - the Bugzilla code currently does not have adequate separation between “back-end” and “front-end” (user interface) code to do this easily.
New checks have been suggested that will be added, but more are always welcome. See bug 97976 for more information.
Current tests are:
- checking everything compiles, Perl and templates
- checking no tabs are present
- checking only the multi-parameter versions of system and exec are used
- checking all files use -w and use strict
The testing suite is now a pre-checkin requirement - all new code must not cause problems it detects. There are currently 3 Tinderbox Clients running this test suite to catch the naughty people who checkin without running these tests.
Another area in which problems can be automatically prevented is by using “taint mode”. Taint mode is about ensuring data from an untrusted source (such as the user) must be properly checked before being used to influence something else (such as the database). Its intention is to make it harder to introduce security holes into the product where data is not properly checked or escaped. It is a goal to get all of Bugzilla to use taint mode.
Currently, only “processmail” runs in taint mode.
The aim is to quickly explain everything a developer needs to know about contributing code to Bugzilla. This includes both the things we require (eg the avoidance of certain features) and prefer (eg style issues).
The document is still in its infancy and it may take some time for all of the conventions and rules of Bugzilla development to be compiled, but it should still be useful in its incomplete state. Suggestions are welcome.
As such it is recommended you consult the online version rather than the one shipped with the 2.14 tarball, as it is out of date._Editor's note: the Hacker's Guide is no longer in the Bugzilla guide, but on the Bugzilla website, renamed as the Developers Guide. The link above will take you to the new Developers Guide instead._
The Bugzilla team felt that having a component in the Webtools product on mozilla.org was too limiting and it would be better to move into a Bugzilla product. Bugzilla has expanded a lot faster than any of the other mozilla.org web tools (such as Bonsai and Tinderbox), both in amount of code and number of developers.
While it may have seemed reasonable to have only a Bugzilla component in the beginning, that no longer is the case. The new product means we can have different Bugzilla components with different default owners, as well as 10 new votes solely for Bugzilla (as opposed to all the web tools), as it should be.
Once the product was created the work began moving the open bugs to the correct component. This has now been completed, and around 800 open bug reports were moved.
It was decided that the resolved bugs would be all moved to the “Bugzilla-General” component, as there was too many to worry about sorting them into their components. At the same time, we took to opportunity to piggy back update the assignee and QA fields on some of the bugs, because emails were already going to be sent out about these bugs.
Because of this, lack of sleep and the long neglected bug 30731 combined to cause an unfortunate incident where 151 resolved bugs got reopened. This resulted in much grumbling, and a fix checked in. This caused special problems for bugs marked DUPLICATE, so related bugs 91808 and 97971 also received attention.
A message was posted to the webtools mailing list/newsgroup regarding the goals for the 2.16 release. This goes into detail about the two main goals which are (slightly simplified):
- no known bugs
- no pending patches
See the message for more details. Other goals that may be dropped include fully working on Win32, all files running in taint mode, full templatisation and fully valid HTML 4.01 output.
The first “review day” was held on the 5th of October. The idea of review day was that developers would not develop but instead help to review some of the patches waiting for review and check in.
The results were:
- 9 positive reviews.
- 3 negative reviews.
- 5 checkins.
- The patch queue size went from approximately 124 to 111.
Another review day was held on the 12th of October, but we forgot to keep track of the stats for it. However, during that preceding week, the patch queue size reduced to around 99. At the time of writing, it was back up. Hopefully further review days can result in bouncing the patch queue size off zero.
There are many ways you can help the Bugzilla team.
- Patches to Fix Bugs/Implement New Features. These are very welcome, especially if they are targetted for the 2.16 milestone! They need to be appropriately generic for all Bugzilla installations and conform to our other requirements (see the hackers’ guide) before they can appear in CVS, but if you don’t wish to do this, anything is better than nothing, and we can use your work as a base.
- New documentation. If you think you can help with the documentation for Bugzilla, please contact Matthew Barnson.
- Testing. Search for bugs in the Bugzilla software, as well as trying out pending patches in the bug system.
- Review. If you have experience with Perl and Bugzilla code, it would be very useful if you look over pending patches in the bug system and see if there are any problems with them. As dealing with all pending patches is a 2.16 goal, review and testing of them is especially important. Generally we expect reviewers to have submitted some patches first so we can evaluate their ability. If you fit into this category, please contact Dave Miller about this.
- Automatic Problem Finding. If you have ideas for automatically detecting problems, please let the team know by filing a bug in the Testing Suite component.
The Bugzilla team mainly communicates through the IRC channel #mozwebtools on irc.mozilla.org. All are welcome on this channel, whether you are an administrator of a Bugzilla installation or wish to contribute. The more the merrier.
Upcoming Major Features
Major new features are being working on that will likely appear in 2.16. These include:
- PostgreSQL support. (Bug 98304)
- Ability to have more than 55 groups, which will also allow a finer grained rights system to be introduced. (Bug 68022)
- Customised resolutions, that allow adding, removing, deactivating and renaming of resolutions. (Bug 94534)
- Expanding the e-mail preferences to allow watching components, keywords, etc. (Bug 73665)
- Request tracker, for managing requests to change things about bugs. (Bug 98801)
Checkins Since 2.14
- Bug 8647 - Added option to sort by last change date directly to the query page (query.cgi).
- Bug 19910 - Added ‘cookiepath’ parameter for when you have multiple installations at one address, to keep the cookies separate.
- Bug 27420 - Added extra space to the Component and Target Milestone fields in the long list display of bugs (long_list.cgi).
- Bug 28736 - Added summary to dependency status change notification mails (processmail).
- Bug 30480 - Added ‘show dependencies as buglist’ links to the dependency tree (dependencytree.cgi).
- Bug 30597 - Made Votes field on the query page more consistent with the rest of the page (query.cgi).
- Bug 30731 - Prevented the status being set to NEW when reassigning non-open bugs on the mass change page (process_bug.cgi).
- Bug 42337 - Added the usual red box for errors when uploading attachments (createattachment.cgi).
- Bug 42810 - Made apostrophes work in product names.
- Bug 46935 - Fixed a typo on the ‘headerhtml’ parameter description that implied it could contain ‘%commandmenu%’ (defparams.pl).
- Bug 51519 - Made links to index on the admin edit users page point to ‘.’ instead of ‘index.html’, to handle customised bases (editusers.cgi).
- Bug 51521 - Fixed bad HTML on the user permissions page (userprefs.cgi).
- Bug 52577 - Added Mac OS X to the default operating systems (checksetup.pl).
- Bug 52782 - Made ‘whineatnews.pl’ email use the ‘sendmailnow’ parameter. This is needed for mail agents other than sendmail.
- Bug 52885 - Made automatic radio selection for the reassignment action not trigger if the new text is same as original owner or empty (show_bug.cgi).
- Bug 53612 - Made the milestone URL appear correctly when deleting a product or a component (editproducts.cgi and editcomponents.cgi).
- Bug 55630 - Added email suffix to the users’ account creation page (createaccount.cgi).
- Bug 57819 - Pruned down large SQL error strings.
- Bug 57821 - Prevented an empty regular expression from causing a software error when searching for users (editusers.cgi).
- Bug 65164 - Made Bugzilla send </html> at the end of its pages.
- Bug 66651 - Included link to attachment when a notification mail is sent for an attachment creation (processmail).
- Bug 69533 - Added HP-UX to the OS auto detection (enter_bug.cgi).
- Bug 69616 - Made the footer not imply admins had the ability to “Edit Sanity Check”. You actually run sanity checks, not edit them.
- Bug 71664 - Fixed sanity check so it doesn’t complain about a bad keyword cache on a bug that does not exist, as the error is detected elsewhere, and it can’t be fixed automatically like other bad keyword cache problems (sanitycheck.cgi).
- Bug 73959 - Fixed some pages not displaying the “Show Votes” link in the footer (buglist.cgi and attachment.cgi).
- Bug 75840 - Made ‘syncshadowdb’ shut down Bugzilla when it runs, and accept a parameter for an alternative temporary directory.
- Bug 76161 - Made the ‘resolve as duplicate’ action not select the radio button when exiting empty an duplicate bug number field (show_bug.cgi).
- Bug 76714 - Fixed the incorrect nesting of FORM and TABLE elements in the footer.
- Bug 76140 - Added more referential sanity checks, plus sanity checking the existence of default milestones (sanitycheck.cgi).
- Bug 82809 - Made comments in bugs no longer use where the linefeeds should be.
- Bug 83058 - Added ability to hide resolved bugs and limit depth to the dependency tree (dependencytree.cgi).
- Bug 83474 - Fixed sanity check so the tables get unlocked properly when rebuilding the keyword cache, if there were no keywords to rebuild (sanitycheck.cgi)
- Bug 84338 - Added attachment tracker (attachment.cgi and editattachstatuses.cgi).
- Bug 87818 - Added support for HTML tag for buglists (buglist.cgi and show_bug.cgi).
- Bug 90333 - Made Bugzilla give an appropriate error message if a user tries to mass change when no bugs are selected to change (process_bug.cgi).
- Bug 91486 - Added a “changed from” option to the boolean charts (query.cgi).
- Bug 91808 - Added sanity checks to make sure the resolution is DUPLICATE if and only if there is an entry on the duplicates table (sanitycheck.cgi).
- Bug 93006 - Made the mass change page auto-select the correct radio button (buglist.cgi).
- Bug 93388 - Made full name be trimmed of whitespace before going into database.
- Bug 95060 - Corrected instructions on how to vote, regarding using checkboxes as opposed to textboxes where only one vote is allowed.
- Bug 96603 - Fixed a grammatical error on bug_status.html.
- Bug 97185 - Made make_select add a newline after each option in the source.
- Bug 97588 - Made “localconfig” mention access.conf for old Apache servers.
- Bug 97657 - Made the sillyness sub be called defparams_pl_sillyness in defparams.pl.
- Bug 97721 - Added check to ensure Perl is at least version 5.005, and Template Toolkit dependencies are fulfilled (checksetup.pl).
- Bug 97764 - Made the attachment tracker not send mail to people about their own changes who didn’t want to receive their own changes.
- Bug 97784 - Made comments in attachment update form get properly word-wrapped (server side).
- Bug 97877 - Made the changed* operators work for attachment statuses on the boolean charts (query.cgi).
- Bug 97971 - Added sanity checks to check that there is a non-open status if and only if there is a resolution, the status is UNCONFIRMED only if everconfirmed is 0, and that bugs that have enough votes to be confirmed have been.
- Bug 97976 - Testing suite. Partial checkins.
- Bug 98074 - Made the attachment tracker properly HTML escape bug titles.
- Bug 98095 - Made importxml.pl pass tests in the testing suite.
- Bug 98146 - Made doeditvotes.cgi give a safer error message if login information is bad.
- Bug 98468 - Made the default “emailregexp” parameter not accept blank emails.
- Bug 99465 - Added FreeBSD to the OS auto detection (enter_bug.cgi).
- Bug 99716 - Made query for “attachment is obsolete” and on attachment status work.
- Bug 101056 - Prevented incorrect error messages when editing an attachment if you needed to log in first.
- Bug 101659 - Added email suffix (where applicable) to the component owner mailto tag.
- Bug 102032 - Fixed test errors and warnings in CVS.
- Bug 103121 - Made editusers.cgi include ‘add user’ links.
- Bug 103554 - Made the HTML generated by the PutHeader and GetCommandMenu subroutines validate as HTML 4.01.
- Bug 103592 - Made email addresses that are longer than 30 characters be truncated in the bug list, to reduce column size.
- Bug 104105 - Made a cosmetic change to the login page to make it obvious that you can create an account.
- Bug 104117 - Fixed the edit keywords (editkeywords.cgi) link on footer broken as a result of the fix for bug 103554.
- Bug 104180 - Made not be used in the URL for a saved query with a space. It should only be used in the display of the query.
- Bug 104247 - Made colours in bug lists work again (buglist.cgi).
- Bug 104516 - Removed all TAB characters from source code.