Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
1) I take those above and copy them to the already existing but empty severity field. They seem reasonable and we are used to them.
2) Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
3) World trac domination!
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
I'll let that sync for some days before doing anything. Let's not wait anymore and take back our Trac system! Freeeeeeeeeedom!!!!!
Cheers! David
This is a good proposal. Some bugs are higher-priority and affect lots of users, but are really quite trivial and don't affect user security (such as CSS bugs).
I'd add another trac change: 4) hide "milestone" field in query results, replace with "severity"
That way, we can quickly see both severity and priority when looking at large lists of bugs that we might want to patch.
best, Griffin
David Goulet wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
On Wed, Oct 7, 2015 at 11:56 AM, David Goulet dgoulet@ev0ke.net wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
I'll let that sync for some days before doing anything. Let's not wait anymore and take back our Trac system! Freeeeeeeeeedom!!!!!
+1 on this.
Also, I suggest that it might be clever to have it so that only members of GRP_devel can set priorities.
On 8 Oct 2015, at 05:01, Nick Mathewson nickm@alum.mit.edu wrote:
On Wed, Oct 7, 2015 at 11:56 AM, David Goulet <dgoulet@ev0ke.net mailto:dgoulet@ev0ke.net> wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
I'll let that sync for some days before doing anything. Let's not wait anymore and take back our Trac system! Freeeeeeeeeedom!!!!!
+1 on this.
Also, I suggest that it might be clever to have it so that only members of GRP_devel can set priorities.
That would be clever, as long as enough people are in GRP_devel. (And by that, I mean, “pick me!")
Tim
Tim Wilson-Brown (teor)
teor2345 at gmail dot com PGP 968F094B
teor at blah dot im OTR CAD08081 9755866D 89E2A06F E3558B7F B5A9D14F
On Thu, Oct 8, 2015 at 4:37 AM, Tim Wilson-Brown - teor teor2345@gmail.com wrote:
On 8 Oct 2015, at 05:01, Nick Mathewson nickm@alum.mit.edu wrote:
On Wed, Oct 7, 2015 at 11:56 AM, David Goulet dgoulet@ev0ke.net wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
I'll let that sync for some days before doing anything. Let's not wait anymore and take back our Trac system! Freeeeeeeeeedom!!!!!
+1 on this.
Also, I suggest that it might be clever to have it so that only members of GRP_devel can set priorities.
That would be clever, as long as enough people are in GRP_devel. (And by that, I mean, “pick me!")
Yes, of course. GRP_devel is not a very restricted credential; it basically means "This person writes code or something like that, and can probably be trusted not to rage-edit the repository."
(You are now a member. Anybody else who should be a member, please let me know on IRC or something.)
On 07 Oct (11:56:32), David Goulet wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
Ok everyone! We are at "world trac domination" status. We now have a Severity field for which I used the same name from the old Priority field (since we all know semantically what they mean, I kept them).
The priorities have been rename like above.
Happy traccing!
Cheers! David
P.S For now, when you open a ticket, there are no default for both since there is a bug in trac that doesn't allow me to change it. I've pinged a sysadmin so hopefully will be resolved soon.
This could get confusing for some tickets after that but at least we can transition towards using the Severity field from now on.
Anyone object or think it's a bad bad idea?
This is very little work for lots of benefits so I don't want to get into a massive reorg. or bike-shedding the naming convention. I would be open though to drop the "Very [High|Low]" field but that could make some tickets with less semantic.
I'll let that sync for some days before doing anything. Let's not wait anymore and take back our Trac system! Freeeeeeeeeedom!!!!!
Cheers! David
tor-dev mailing list tor-dev@lists.torproject.org https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-dev
David Goulet dgoulet@ev0ke.net writes:
On 07 Oct (11:56:32), David Goulet wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them.
Rename all priorities to something more meaninful:
blocker --> Immediate criticial --> Very High major --> High normal --> Medium minor --> Low trivial --> Very Low
World trac domination!
Ok everyone! We are at "world trac domination" status. We now have a Severity field for which I used the same name from the old Priority field (since we all know semantically what they mean, I kept them).
The priorities have been rename like above.
Happy traccing!
Cheers! David
P.S For now, when you open a ticket, there are no default for both since there is a bug in trac that doesn't allow me to change it. I've pinged a sysadmin so hopefully will be resolved soon.
Thanks for the transitioning!
FWIW, it seems that the current default Priority for new tickets is "Immediate" and the default Severity is "Blocker". This might cause people who don't know how trac works, submitting stressfull trac tickets.
On 14 Oct 2015, at 23:13, George Kadianakis desnacked@riseup.net wrote:
David Goulet <dgoulet@ev0ke.net mailto:dgoulet@ev0ke.net> writes:
On 07 Oct (11:56:32), David Goulet wrote:
Hello tor-dev!
While 028 bug triaging, we realized that we *really* need priorities to not be a banana field deprive of useful meaning. If you are unaware or don't remember, the priority field in a trac ticket can be:
blocker critical major normal minor trivial
Those are "severities" *NOT* priorities. Now I propose the following:
- I take those above and copy them to the already existing but empty
severity field. They seem reasonable and we are used to them. ...
P.S For now, when you open a ticket, there are no default for both since there is a bug in trac that doesn't allow me to change it. I've pinged a sysadmin so hopefully will be resolved soon.
Thanks for the transitioning!
FWIW, it seems that the current default Priority for new tickets is "Immediate" and the default Severity is "Blocker". This might cause people who don't know how trac works, submitting stressfull trac tickets.
And every time I edit an existing ticket, it gets severity “Blocker”, because there’s no severity assigned.
Tim
Tim Wilson-Brown (teor)
teor2345 at gmail dot com PGP 968F094B
teor at blah dot im OTR CAD08081 9755866D 89E2A06F E3558B7F B5A9D14F
Hi David. Personally I find the split severity and priority redundant but that's fine (milestone, tor version, and other fields are only applicable to core tor - it's easy to ignore yet another field). However, in this case you made it mandatory by giving it a default. Mind making priority optional?
Cheers! -Damian