[AutoQA] #329: implement RPM checksum verification

[fedora-badges]

#329: implement RPM checksum verification
-------------------------+--------------------------------------------------
 Reporter:  kparal       |       Owner:             
     Type:  enhancement  |      Status:  new        
 Priority:  major        |   Milestone:  Finger Food
Component:  core         |    Keywords:             
-------------------------+--------------------------------------------------
 Currently this method is in autoqa.util module:

 {{{
  261 def valid_rpm(rpm_file):
  262     '''Check that RPM file is valid.
  263        Note: This currently only extracts RPM header and presumes the
 file is
  264        OK if that succeeds. Full RPM hashsum check would be nice to
 have
  265        implemented too (FIXME).
  266        @param rpm_file RPM file path
  267        @return True if file seems OK or False if there is some
 problem (file
  268                can't be read, RPM header is corrupt)
  269     '''
  ...
 }}}

 We would like to have full RPM checksum verification, not just reading the
 header and saying that it's OK. Find out how to do it and change the
 method accordingly. It would be nice to have a method argument to enable
 full checksum verification/fall back to original header verification,
 because it can be performance intensive operation and it may not be
 suitable for all our use cases).

-- 
Ticket URL: <https://fedorahosted.org/autoqa/ticket/329>
AutoQA <http://autoqa.fedorahosted.org>
Automated QA project

[fedora-badges]

#329: implement RPM checksum verification
-------------------------+--------------------------------------------------
 Reporter:  kparal       |        Owner:             
     Type:  enhancement  |       Status:  new        
 Priority:  minor        |    Milestone:  Finger Food
Component:  core         |   Resolution:             
 Keywords:               |  
-------------------------+--------------------------------------------------
Changes (by kparal):

  * priority:  major => minor

-- 
Ticket URL: <https://fedorahosted.org/autoqa/ticket/329#comment:1>
AutoQA <http://autoqa.fedorahosted.org>
Automated QA project

[fedora-badges]

#329: implement RPM checksum verification
-------------------------+--------------------------
 Reporter:  kparal       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  minor        |   Milestone:  Finger Food
Component:  core         |  Resolution:
 Keywords:               |  Blocked By:
 Blocking:               |
-------------------------+--------------------------

Comment (by jsedlak):

 Patch of desired functionality available here:
 https://fedorahosted.org/reviewboard/r/271/
 Please review it.

-- 
Ticket URL: <https://fedorahosted.org/autoqa/ticket/329#comment:2>
AutoQA <http://autoqa.fedorahosted.org>
Automated QA project

[fedora-badges]

#329: implement RPM checksum verification
-------------------------+--------------------------
 Reporter:  kparal       |       Owner:
     Type:  enhancement  |      Status:  closed
 Priority:  minor        |   Milestone:  Finger Food
Component:  core         |  Resolution:  fixed
 Keywords:               |  Blocked By:
 Blocking:               |
-------------------------+--------------------------
Changes (by jsedlak):

 * status:  new => closed
 * resolution:   => fixed


Comment:

 Fixed in master, commit 366914bb5856a962f617f63023844c341a786cc3

-- 
Ticket URL: <https://fedorahosted.org/autoqa/ticket/329#comment:3>
AutoQA <http://autoqa.fedorahosted.org>
Automated QA project