RFR: Filing bug, ProblemListing, Backing out [v2]
Joe Darcy
joe.darcy at oracle.com
Wed Jul 8 02:36:59 UTC 2020
On 7/6/2020 4:26 PM, David Holmes wrote:
> On 7/07/2020 8:56 am, Igor Ignatyev wrote:
>> Hi David,
>>
>>> On Jul 6, 2020, at 3:41 PM, David Holmes <david.holmes at oracle.com
>>> <mailto:david.holmes at oracle.com>> wrote:
>>>
>>> Hi Igor,
>>>
>>> On 7/07/2020 8:36 am, Igor Ignatyev wrote:
>>>> On Mon, 6 Jul 2020 22:25:58 GMT, Jesper Wilhelmsson
>>>> <jwilhelm at openjdk.org <mailto:jwilhelm at openjdk.org>> wrote:
>>>>>> src/next.md line 154:
>>>>>>
>>>>>>> 153: #. Close the original JBS issue **(O)**.
>>>>>>> 154: * "Verify" the issue and choose "Fix Failed".
>>>>>>> 155: #. If the intention is to fix the change and submit it
>>>>>>> again, create a redo-issue **(R)** to track that the work
>>>>>>> still needs to be done.
>>>>>>
>>>>>> I have strong objections to fix failed ever being used and oppose
>>>>>> it being recommended here. Unless the fixer and their
>>>>>> reviewers completely failed at their job what you usually have is
>>>>>> some other problem caused by the fix and the fix
>>>>>> actually succeeded.
>>>>>
>>>>> I guess this is a question for those who normally handle fix
>>>>> verification and may have scripts that look for different
>>>>> verifications.
>>>>> There's only four values to choose from: "None", "Verified", "Not
>>>>> verified", and "Fix failed".
>>>>> "Verified" means that the fix solved the problem and no more
>>>>> action is required, so this is clearly not right. "Not
>>>>> verified" seems wrong since it actually was verified that the fix
>>>>> caused problems - or it wouldn't need to be backed
>>>>> out. "None" could be used in my mind, but I can imagine that there
>>>>> are filters that treats "None" as issues that needs
>>>>> verification. So changing to using this would probably cause
>>>>> problems. That leaves "Fix failed". Maybe Joe knows why
>>>>> this was designed as it is? Anyhow, it is the current process and
>>>>> we need to bring it up with the right people before
>>>>> changing it.
>>>> AFAIK, The verification process isn't part of any OpenJDK process
>>>> and is used/done mainly internally by Oracle. why do
>>>> you think that verification status should be set for all backed out
>>>> issues?
>>>
>>> That was the process that was defined. If a fix has to be backed out
>>> then the fix is considered to have "failed".
>>
>> ok, but this meaning is a bit different from the one in the
>> verification process, where
>> - 'Verified' means that someone checked that the defect described
>> in the issue has been addressed by the fix, and it actually was;
>> - 'Fix failed' means that someone checked that the defect described
>> in the issue has been addressed by the fix, but it actually was not,
>> i.e. the defect still exists after the fix.
>
> I thought "fix failed" was more broadly defined as either not fixing
> the original issue, or introducing additional breakage.
>
> In terms of backports I would expect anything marked as "fix failed"
> to not be backported, and then the backout issue would also not be
> backported. But sometimes it isn't that simple.
>
> But these are just my recollections from the earlier definition of the
> backing out process. And as you note the Verification process is
> really an Oracle specific concept exposed in JBS. I'm not sure how to
> deal with that. There aren't really any guidelines on how to use JBS
> in the OpenJDK process docs.
>
Admittedly it isn't easy to find, but there is a bit written about using
JBS in the OpenJDK wiki:
https://wiki.openjdk.java.net/display/general/JBS+Overview
-Joe
More information about the guide-dev
mailing list