RFR: Updates to the JBS section of the OpenJDK Dev Guide [v3]
calnan
duke at openjdk.org
Mon May 8 22:50:29 UTC 2023
On Fri, 5 May 2023 19:36:59 GMT, Jesper Wilhelmsson <jwilhelm at openjdk.org> wrote:
>> src/guide/jbs-jdk-bug-system.md line 192:
>>
>>> 190: 1. Only one fixversion should ever be set, if the issue is to be fixed in additional releases then a separate backport must be created (See Working with backports in JBS). There are exceptions to this rule for: CSRs; and, Release Notes.
>>> 191: 1. make sure the bug has all the required labels – JBS Label Dictionary
>>> 192: 1. bugs that are new in the current release: 'regression'
>>
>> This is tricky. We don't generally use "regression" during active development of the current release, it tends to be used mainly for issues that are found after a release has been released e.g. if we release JDK 19 and then a bug report comes in because something that worked in 18 now doesn't, then it is a regression in 19. But if we are in the process of developing a release and we have added some new thing and then we find a bug in that then we don't mark that as a regression. And a lot of the time if we introduce a bug "by mistake" we don't tend to mark it as a regression.
>
> We probably should be better at marking that last category as regressions though since there are special rules around deferring those.
Have left the text as-is for the moment - let me know if you feel changes are needed
>> src/guide/jbs-jdk-bug-system.md line 198:
>>
>>> 196: 1. project specific issues usually have their own labels as well
>>> 197:
>>> 198: At this point move the issue into the [Open]{.jbs-field} state, bring high priority (P1, P2) bugs to the attention of the team.
>>
>> What "team"?
>
> This section should probably start with a few sentences defining who is expected to perform triage. This is in general done by well defined triage teams who are part of the "team" that will work on the fix, so I would assume that it shouldn't be necessary to bring the bug to anyones attention. P1-P2 bugs are fairly visible anyway :-)
> I would suggest to remove the part after the comma.
Added some text at the start from Jesper on the model of triage teams
-------------
PR Review Comment: https://git.openjdk.org/guide/pull/100#discussion_r1187965035
PR Review Comment: https://git.openjdk.org/guide/pull/100#discussion_r1187964591
More information about the guide-dev
mailing list