Description of problem: Hi, I think that automatic changes done along to user's bug change should be marked explicitly. Reproducer: 1. I am in Devel group and moving kernel bug from ON_QA back to ASSIGNED because of some problem described in comments which is not FailedQA. 2. There is automatically added FailedQA into Verified field under my name which breaks accountability a bit. Expected result: Mark change Verified: FailedQA as "auto" or something like that.
The automatic addition of "FailedQA" to the Verified field was added to Bugzilla in Bug 565827. I presume that this was done to satisfy the requirements of the CDW process used for RHEL releases. The relevant code fragment (from extensions/RedHat/Extension.pm) is: 881 # REDHAT EXTENSION START 565827 826305 882 if ($changes->{'bug_status'} 883 and (grep($_ eq $old_bug->status->name, ('ON_QA', 'VERIFIED', 'RELEASE_PENDING'))) 884 and $bug->status->name eq 'ASSIGNED' 885 and $user->can_set_partner('FailedQA') 886 and not grep($_ eq 'FailedQA', @{$bug->cf_verified}) 887 ){ 888 $dbh->do("INSERT INTO bug_cf_verified (bug_id, value) VALUES (?, ?)", 889 undef, $bug->id, 'FailedQA'); 890 # Log the change 891 LogActivityEntry( 892 $bug->id, 893 'cf_verified', 894 '', 895 'FailedQA', 896 $user->id, 897 $timestamp 898 ); 899 } 900 # REDHAT EXTENSION END 565827 826305 There are a number of other automatic field changes in nearby code that are also logged to the activity history under the identity of the user making the changes that trigger the automatic change. While I have no philosophical objection to changing that behaviour, the mandate for Bugzilla's implementation of the CDW doesn't belong to me. That role is currently filled by Suzanne Yeghiayan (now CC'd on this bug) and you will need to negotiate any change to CDW-related behaviour of Bugzilla with her.
Jason, thank you for the informations! Suzanne, what do you think about this proposed change? It seems to me that it won't break anything, just add more verbosity into state changes descriptions.
AFAIK, the QE Department requested this change. It was not a change requested by Program Management for CDW. I suggest you start by asking Ben Levenson about it.
Ben, are you able to comment on the above?
This bug is specifically about the CDW. It's about changes made by bugzilla (most of which are defined in extension/RedHat/Extension.pm being attributed in the e-mails to the person making the original change, not the bugzilla user. Having said that, this is one of those bugs that once you've spent time looking into it, you can fix it straight away. Targeting for sprint13, need to work on a patch for upstream though. If there are changes required in regards to the CDW, that is a separate bug, and should be filed as such. E-mails with multi user changes will now look like: Red Hat Bugzilla <bugzilla> changed: What |Removed |Added ---------------------------------------------------------------------------- Group| |devel, qa Simon Green <sgreen> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |sgreen
Submitted upstream patch for this. It doesn't go cleanly into our code, so I'll do another patch for us towards the end of the sprint.
We'll inherit this as part of the upstream merge process. There will be a conflict, but that can be fixed on commit. -- simon