Tri des tickets

Django uses Trac for managing the work on the code base. Trac is a community-tended garden of the bugs people have found and the features people would like to see added. As in any garden, sometimes there are weeds to be pulled and sometimes there are flowers and vegetables that need picking. We need your help to sort out one from the other, and in the end we all benefit together.

Like all gardens, we can aspire to perfection but in reality there’s no such thing. Even in the most pristine garden there are still snails and insects. In a community garden there are also helpful people who – with the best of intentions – fertilize the weeds and poison the roses. It’s the job of the community as a whole to self-manage, keep the problems to a minimum, and educate those coming into the community so that they can become valuable contributing members.

Similarly, while we aim for Trac to be a perfect representation of the state of Django’s progress, we acknowledge that this simply will not happen. By distributing the load of Trac maintenance to the community, we accept that there will be mistakes. Trac is « mostly accurate », and we give allowances for the fact that sometimes it will be wrong. That’s okay. We’re perfectionists with deadlines.

We rely on the community to keep participating, keep tickets as accurate as possible, and raise issues for discussion on our mailing lists when there is confusion or disagreement.

Django is a community project, and every contribution helps. We can’t do this without you!

Triage workflow

Unfortunately, not all bug reports and feature requests in the ticket tracker provide all the required details. A number of tickets have patches, but those patches don’t meet all the requirements of a good patch.

One way to help out is to triage tickets that have been created by other users.

Most of the workflow is based around the concept of a ticket’s triage stages. Each stage describes where in its lifetime a given ticket is at any time. Along with a handful of flags, this attribute easily tells us what and who each ticket is waiting on.

Since a picture is worth a thousand words, let’s start there:

We’ve got two roles in this diagram:

• Committers: people with commit access who are responsible for making the final decision to merge a patch.
• Ticket triagers: anyone in the Django community who chooses to become involved in Django’s development process. Our Trac installation is intentionally left open to the public, and anyone can triage tickets. Django is a community project, and we encourage triage by the community.

En guise d’exemple, voici le cycle de vie d’un ticket moyen :

• Alice crée un ticket et y associe une demande de contribution incomplète (pas de tests, implémentation non correcte).
• Bob reviews the pull request, marks the ticket as « Accepted », « needs tests », and « patch needs improvement », and leaves a comment telling Alice how the patch could be improved.
• Alice updates the pull request, adding tests (but not changing the implementation). She removes the two flags.
• Charlie reviews the pull request and resets the « patch needs improvement » flag with another comment about improving the implementation.
• Alice met à jour la demande de contribution, corrigeant l’implémentation. Elle enlève le drapeau « patch needs improvement » (correctif nécessite amélioration).
• Daisy relit la demande de contribution et marque le ticket comme « Ready for checkin » (prêt pour l’intégration).
• Jacob, un commiteur, révise la demande de contribution et l’intègre dans le code principal (« merge »).

Certains tickets n’ont pas besoin d’autant d’interventions, mais il y a aussi des tickets qui en nécessitent beaucoup plus.

Étapes de tri

Nous décrivons ci-dessous plus en détails les diverses étapes par lesquelles un ticket passe durant son parcours de vie.

Non traité (Unreviewed)

Le ticket n’a pas été relu par quelqu’un qui se sentait qualifié pour juger si ce ticket contient un problème réel, une fonctionnalité raisonnable ou s’il devrait plutôt être fermé pour une ou différentes raisons.

Accepté (Accepted)

La grande zone grise ! La signification absolue de « accepté » est que le problème décrit dans le ticket est valide et se trouve dans un état où on peut travailler sur sa résolution. Au-delà de cela, d’autres aspects sont à prendre en considération :

• Accepted + No Flags

  The ticket is valid, but no one has submitted a patch for it yet. Often this means you could safely start writing a patch for it. This is generally more true for the case of accepted bugs than accepted features. A ticket for a bug that has been accepted means that the issue has been verified by at least one triager as a legitimate bug - and should probably be fixed if possible. An accepted new feature may only mean that one triager thought the feature would be good to have, but this alone does not represent a consensus view or imply with any certainty that a patch will be accepted for that feature. Seek more feedback before writing an extensive patch if you are in doubt.

• Accepted + Has Patch

  The ticket is waiting for people to review the supplied patch. This means downloading the patch and trying it out, verifying that it contains tests and docs, running the test suite with the included patch, and leaving feedback on the ticket.

• Accepted + Has Patch + Needs …

  This means the ticket has been reviewed, and has been found to need further work. « Needs tests » and « Needs documentation » are self-explanatory. « Patch needs improvement » will generally be accompanied by a comment on the ticket explaining what is needed to improve the code.

Prêt pour le commit (Ready For Checkin)

The ticket was reviewed by any member of the community other than the person who supplied the patch and found to meet all the requirements for a commit-ready patch. A committer now needs to give the patch a final review prior to being committed. See the New contributors” FAQ for « My ticket has been in RFC forever! What should I do? »

Un jour peut-être (Someday/Maybe)

Cette étape n’apparaît pas dans le diagramme. Elle est utilisée avec modération pour garder la trace d’idées plus générales ou de demandes de fonctionnalités à plus long terme.

These tickets are uncommon and overall less useful since they don’t describe concrete actionable issues. They are enhancement requests that we might consider adding someday to the framework if an excellent patch is submitted. They are not a high priority.

Autres attributs de tri

Un certain nombre de drapeaux, apparaissant sous forme de cases à cocher dans Trac, peuvent être définis pour un ticket :

Has patch (possède un correctif)

Cela signifie que le ticket possède un correctif associé. Ceux-ci sont analysés pour estimer si le correctif est « bon ».

Les trois champs suivants (Needs documentation, Needs tests, Patch needs improvement) ne s’appliquent que lorsqu’un correctif est disponible.

Needs documentation (a besoin de documentation)

Ce drapeau est utilisé pour les tickets ayant un correctif, mais pas encore de documentation. La documentation complète des fonctionnalités est un prérequis avant de pouvoir faire entrer du code dans Django.

Needs tests (a besoin de tests)

Ce drapeau indique que le correctif a besoin de tests unitaires associés. Encore une fois, il s’agit d’un élément obligatoire pour un correctif valable.

Patch needs improvement (le correctif a besoin d’améliorations)

Ce drapeau signifie que même si le ticket possède un correctif, il n’est pas encore prêt à être commité. Cela peut signifier que le correctif n’est plus applicable, que son implémentation est défaillante ou que le code ne correspond pas aux standards exigés.

Easy pickings (simple à corriger)

Désigne les tickets qui nécessitent de petits correctifs assez simples.

Type

Les billets doivent être classés par type parmi :

• New Feature (nouvelle fonctionnalité)

  Pour ajouter quelque chose de nouveau.

• Bug

  Quand une chose existante est cassé ou n’a pas le fonctionnement attendu.

• Nettoyage/optimisation

  Lorsque rien n’est cassé mais quelque chose pourrait être plus propre, mieux, plus rapide, plus puissant.

Component (composant)

Les tickets doivent être classés en composants indiquant quelle partie du code de Django ils concernent. Cela aide à organiser les tickets et à les rendre plus facilement retrouvables.

Severity (sévérité)

L’attribut severity est utilisé pour identifier les tickets bloquants, c’est-à-dire des problèmes devant être absolument résolus avant de publier la prochaine version de Django. Typiquement, il s’agit de bogues provoquant des régressions en comparaison d’anciennes versions ou qui sont potentiellement la cause de pertes de données importantes. Cet attribut est assez rarement utilisé et la grande majorité des tickets possèdent un niveau de sévérité « normal ».

Version

Il est possible d’utiliser l’attribut version pour indiquer la version dans laquelle le bogue signalé a été identifié.

UI/UX

Ce drapeau est utilisé pour les tickets qui sont liés aux questions d’interface utilisateur et d’expérience d’utilisation. Par exemple, ce drapeau serait approprié pour des fonctionnalités de présentation à l’utilisateur dans les formulaires ou l’interface d’administration.

Cc

Il est possible d’ajouter son nom d’utilisateur ou son adresse électronique dans ce champ pour être averti lorsque de nouvelles contributions sont apportées au ticket concerné.

Keywords (mots-clés)

Avec ce champ, il est possible d’étiqueter un ticket par plusieurs mots-clés. Cela peut par exemple être utile pour grouper plusieurs tickets autour d’un même thème. Les mots-clés peuvent être séparés par des virgules ou des espaces. Les recherches par mot-clé cherchent un mot-clé dans toute l’expression. Par exemple, si on clique sur le mot-clé « form » d’un ticket, les résultats contiendront tous les tickets étiquetés avec des mots tels que « formset », « modelformset » ou « ManagementForm ».

Fermeture de tickets

When a ticket has completed its useful lifecycle, it’s time for it to be closed. Closing a ticket is a big responsibility, though. You have to be sure that the issue is really resolved, and you need to keep in mind that the reporter of the ticket may not be happy to have their ticket closed (unless it’s fixed, of course). If you’re not certain about closing a ticket, just leave a comment with your thoughts instead.

If you do close a ticket, you should always make sure of the following:

• Be certain that the issue is resolved.
• Leave a comment explaining the decision to close the ticket.
• If there is a way they can improve the ticket to reopen it, let them know.
• If the ticket is a duplicate, reference the original ticket. Also cross-reference the closed ticket by leaving a comment in the original one – this allows to access more related information about the reported bug or requested feature.
• Be polite. No one likes having their ticket closed. It can be frustrating or even discouraging. The best way to avoid turning people off from contributing to Django is to be polite and friendly and to offer suggestions for how they could improve this ticket and other tickets in the future.

Un ticket peut être résolu de plusieurs façons :

• fixed (résolu)

  Used once a patch has been rolled into Django and the issue is fixed.

• invalid (non valide)

  Utilisé lorsque le ticket est jugé incorrect. Cela signifie que le problème du ticket est en fait le résultat d’une erreur de l’utilisateur ou que le problème décrit s’applique à un autre composant que Django, ou encore qu’il ne s’agit ni d’un signalement d’erreur, ni d’une demande d’amélioration (par exemple, certains utilisateurs débutants ouvrent des tickets pour des demandes de support).

• wontfix (ne sera pas résolu)

  Utilisé lorsque quelqu’un décide que cette requête n’est pas jugée digne d’être prise en compte dans Django. Il peut arriver qu’un ticket soit fermé comme « wontfix » avec une demande de lancer une discussion sur la liste de diffusion django-developers s’il existe un désaccord avec l’explication fournie par la personne qui a fermé le ticket. D’autres fois, une discussion sur la liste de diffusion précède la fermeture du ticket. Utilisez toujours la liste de diffusion pour obtenir un consensus avant de réouvrir un ticket fermé comme « wontfix ».

• duplicate (doublon)

  Utilisé lorsqu’un autre ticket recouvre le même problème. En fermant les tickets doublons, la discussion reste centralisée à un seul endroit, ce qui aide tout le monde.

• worksforme (marche pour moi)

  Utilisé lorsque le ticket ne contient pas assez de détails pour reproduire l’anomalie originale.

• needsinfo (besoin d’informations)

  Utilisé lorsque le ticket ne contient pas assez d’informations pour reproduire le problème signalé, mais qu’il est potentiellement valable. Le ticket devrait être réouvert après que des informations supplémentaires ont été fournies.

If you believe that the ticket was closed in error – because you’re still having the issue, or it’s popped up somewhere else, or the triagers have made a mistake – please reopen the ticket and provide further information. Again, please do not reopen tickets that have been marked as « wontfix » and bring the issue to django-developers instead.

Commet puis-je aider à trier les tickets ?

Le processus de tri est principalement mené par des membres de la communauté. Vraiment, TOUT LE MONDE peut aider.

Pour vous impliquer, commencez par créer un compte sur Trac. Si vous disposez d’un compte mais que vous avez oublié le mot de passe, vous pouvez le réinitialiser en utilisant la page de réinitialisation de mot de passe.

Puis, vous pouvez aider en :

• Closing « Unreviewed » tickets as « invalid », « worksforme », or « duplicate », or « wontfix ».
• Fermant les nouveaux tickets (« Unreviewed ») comme « needsinfo » lorsque la description est trop vague pour qu’on puisse imaginer une résolution ou lorsqu’il s’agit de demandes de fonctionnalité qui nécessitent une discussion sur la liste django-developers.
• Corrigeant les coches « Needs tests », « Needs documentation » ou « Has patch » pour les tickets où elles ne sont pas correctement définies.
• Activant la coche « Easy pickings » pour les tickets de faible portée et relativement évidents.
• Définissant le type des tickets qui ne sont pas encore catégorisés.
• Vérifiant que les anciens tickets sont toujours valables. SI un ticket n’a pas vu d’activité depuis longtemps, il est possible que le problème ait été résolu dans l’intervalle sans que le ticket ait été fermé.
• Identifiant des tendances et des thèmes dans les tickets. Si de nombreux signalements de bogues concernent une partie particulière de Django, cela peut indiquer que cette partie de code a besoin de réécriture. Si une tendance se dégage, il est suggéré d’amener une discussion (en faisant référence aux tickets concernés) sur la liste django-developers.
• Vérifiant si les correctifs soumis par d’autres utilisateurs sont corrects. S’ils sont corrects et qu’ils contiennent la documentation et les tests nécessaires, vous pouvez faire passer le ticket à l’état « Ready for Checkin ». Dans le cas contraire, écrivez un commentaire expliquant pourquoi et cochez les cases correspondantes (« Patch needs improvement », « Needs tests », etc.).

Note

The Reports page contains links to many useful Trac queries, including several that are useful for triaging tickets and reviewing patches as suggested above.

You can also find more Conseils pour les nouveaux contributeurs.

However, we do ask the following of all general community members working in the ticket database:

• Please don’t promote your own tickets to « Ready for checkin ». You may mark other people’s tickets which you’ve reviewed as « Ready for checkin », but you should get at minimum one other community member to review a patch that you submit.
• Please don’t reverse a decision without posting a message to django-developers to find consensus.
• If you’re unsure if you should be making a change, don’t make the change but instead leave a comment with your concerns on the ticket, or post a message to django-developers. It’s okay to be unsure, but your input is still valuable.

Bisecting a regression

A regression is a bug that’s present in some newer version of Django but not in an older one. An extremely helpful piece of information is the commit that introduced the regression. Knowing the commit that caused the change in behavior helps identify if the change was intentional or if it was an inadvertent side-effect. Here’s how you can determine this.

Begin by writing a regression test for Django’s test suite for the issue. For example, we’ll pretend we’re debugging a regression in migrations. After you’ve written the test and confirmed that it fails on the latest master, put it in a separate file that you can run standalone. For our example, we’ll pretend we created tests/migrations/test_regression.py, which can be run with:

$ ./runtests.py migrations.test_regression

Next, we mark the current point in history as being « bad » since the test fails:

$ git bisect bad
You need to start by "git bisect start"
Do you want me to do it for you [Y/n]? y

Now, we need to find a point in git history before the regression was introduced (i.e. a point where the test passes). Use something like git checkout HEAD~100 to checkout an earlier revision (100 commits earlier, in this case). Check if the test fails. If so, mark that point as « bad » (git bisect bad), then checkout an earlier revision and recheck. Once you find a revision where your test passes, mark it as « good »:

$ git bisect good
Bisecting: X revisions left to test after this (roughly Y steps)
...

Now we’re ready for the fun part: using git bisect run to automate the rest of the process:

$ git bisect run tests/runtests.py migrations.test_regression

You should see git bisect use a binary search to automatically checkout revisions between the good and bad commits until it finds the first « bad » commit where the test fails.

Now, report your results on the Trac ticket, and please include the regression test as an attachment. When someone writes a fix for the bug, they’ll already have your test as a starting point.