Bug 452447

Summary: modify the Whines check in the sanitycheck.cgi to work with our cron jobs
Product: [Community] Bugzilla Reporter: Noura El hawary <nelhawar>
Component: Bugzilla GeneralAssignee: Noura El hawary <nelhawar>
Status: CLOSED NEXTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: low    
Version: 3.2   
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-07-02 02:28:13 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 406071    
Attachments:
Description Flags
v1 modifying new whines check in sanitycheck dkl: review+

Description Noura El hawary 2008-06-23 00:56:46 UTC 
Upstream has added new check to sanitycheck.cgi for Whines , sanitycheck.cgi and
sanitycheck.pl needs to be modified to make the new check works fine with our
sanitycheck cron job.

This task will be performed after Tony commits his patch for the flags check. so
i don't mess up his patch.

Noura
Comment 1 David Lawrence 2008-06-23 15:23:51 UTC 
Adding to 3.2 dependencies.
Comment 2 Noura El hawary 2008-06-24 05:04:00 UTC 
Created attachment 310105 [details]
v1 modifying new whines check in sanitycheck 

Patch includes, adding cgi param for whine obsolete target check in
sanitycheck.cgi to enable running it as a single check and calling it from
sanitycheck.pl, also the sanitycheck.pl has been modified accordingly and the
config/sanitycheck.cron also was modified to include this check and the check
for flags in the bugs activity table.

Please review when you can.

Noura
Comment 3 David Lawrence 2008-06-24 16:06:39 UTC 
Comment on attachment 310105 [details]
v1 modifying new whines check in sanitycheck 

Looks good Noura!

Dave
Comment 4 Noura El hawary 2008-06-25 03:31:33 UTC 
Thanks for the review Dave.

Now all committed to cvs
Comment 5 David Lawrence 2008-07-02 02:28:13 UTC 
This change should be on partner-bugzilla.redhat.com now and should be in the
final release.