few bugs have NEEDINFO_REPORTER status in them as reported by tony, this all needs to be cleaned out as part of the NEEDINFO status clean up for the 3.2 bugzilla migration.
Created attachment 311372 [details] v1 added code to clean up NEEDINFO_REPORTER status attached is a patch to the needinfo cleanup script which does clean up of the NEEDINFO_REPORTER status and also now it has code to write the output to a log file so we can keep track and record of the clean up process and keep Meethune happy lol. please review when you can. Noura
just a note when i checked the partner database it had about 35 bugs with NEEDINFO_REPORTER, but currently the live production database has only one bug with that status, i just fixed the script just in case at the time of the migration there are more bugs again with that status. Noura
Comment on attachment 311372 [details] v1 added code to clean up NEEDINFO_REPORTER status Looks good Noura. One little nitpick though: Change MYFILE to LOGFILE or LOG to be more descriptive. Once that is changed, feel free to checkin and then send email for bhowmick to run on partner-bugzilla. Dave
Thanks for the review Dave, changed to LOGFILE, and sent request to run the script on partner-bugzilla. Noura
looking the the result i got from meethune from running the clean up script on partner-bugzilla , 2 things went wrong: 1- the needinfo?($SOMONE) string was appending the reporters from each bug to the other because i forgot to rest the string to its original value inside each loop. 2- new thing came out which is what if the prev status to a bug with NEEDINFO was NEEDINFO_REPORTER or the other way around prev status with NEEDINFO_REPORTER was NEEDINFO this will still change back to the NEEDINFO or NEEDINFO_REPORTER and we haven't done anything. a patch is attached which has the fix for the above 2 cases also i run the script with that patch on bz-db1-test and the output is attached Noura
Created attachment 311443 [details] patch for the rest of NEEDINFO_* cleanup This is the patch that includes fix for the above points, basically i found out that there are some other previous statuses that are not valid and should not be set instead of the NEEDINFO status for example, INVISTIGATE, NEEDINFO_PM ,etc so i put condition for the previous status that it has to be in: removed IN (select value from bug_status where isactive=1 and is_open=1 and value != 'NEEDINFO') please see attached patch. Noura
Comment on attachment 311443 [details] patch for the rest of NEEDINFO_* cleanup Looks good Noura.
Thanks for the review Dave, now committed.
The changes have been pushed to partner-bugzilla.redhat.com and/or have been recorded in the Bugzilla migration pages on the Bugzilla 3 upgrade Wiki. Closing as NEXTRELEASE. Dave