Bug 1859314

Summary: rhv-log-collector-analyzer --json fails with TypeError
Product: Red Hat Enterprise Virtualization Manager Reporter: Guilherme Santos <gdeolive>
Component: rhv-log-collector-analyzerAssignee: Sandro Bonazzola <sbonazzo>
Status: CLOSED ERRATA QA Contact: Guilherme Santos <gdeolive>
Severity: high Docs Contact:
Priority: high    
Version: 4.4.0CC: emarcus, lleistne, michal.skrivanek, sbonazzo
Target Milestone: ovirt-4.4.3Keywords: FieldEngineering, ZStream
Target Release: 4.4.3Flags: gdeolive: testing_plan_complete+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: rhv-log-collector-analyzer-1.0.3-1.el8ev Doc Type: Bug Fix
Doc Text:
Previously, unicode strings were not handled properly by the rhv-log-collector-analyzer after porting to python3. In this release, unicode strings are now handled properly.
Story Points: ---
Clone Of:
: 1967169 (view as bug list) Environment:
Last Closed: 2020-11-24 13:09:25 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Integration RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 1866981, 1885976    
Bug Blocks:    

Description Guilherme Santos 2020-07-21 17:22:42 UTC
Description of problem:
# rhv-log-collector-analyzer --json
Traceback (most recent call last):
  File "/usr/bin/engine-db-query", line 280, in <module>
    sys.exit(main())
  File "/usr/bin/engine-db-query", line 272, in main
    knowledge_base=args.kb_url
  File "/usr/lib/python3.6/site-packages/engine_db_query/__init__.py", line 213, in execute
    knowledge_base=knowledge_base
  File "/usr/lib/python3.6/site-packages/engine_db_query/__init__.py", line 383, in query_return_json
    s=hashlib.sha256(file_name).hexdigest(),
TypeError: Unicode-objects must be encoded before hashing
b''

Version-Release number of selected component (if applicable):
rhv-log-collector-analyzer-1.0.2-1.el8ev.noarch
ovirt-engine-4.4.1.8-0.7.el8ev.noarch

How reproducible:
always

Steps to Reproduce:
1. # rhv-log-collector-analyzer --json
2.
3.

Actual results:
fails

Expected results:
succeed

Additional info:

Comment 2 Michal Skrivanek 2020-08-26 18:45:12 UTC
too late for 4.4.2, isn't it?

Comment 3 Sandro Bonazzola 2020-08-27 05:41:23 UTC
(In reply to Michal Skrivanek from comment #2)
> too late for 4.4.2, isn't it?

right, sad to have to postpone a bug because of technical issues preventing patches being merged.

Comment 7 Guilherme Santos 2020-10-26 13:48:13 UTC
Verified on:
rhv-log-collector-analyzer-1.0.5-1.el8ev.noarch
ovirt-engine-4.4.3-0.1.el8ev.noarch

Steps:
# rhv-log-collector-analyzer --json

Results:
Run went fine and json generated successfully

Comment 8 Eli Marcus 2020-11-22 19:35:19 UTC
Hi Sandro, please review this doc text for the errata and release notes: 

Previously, unicode strings were not handled properly by the rhv-log-collector-analyzer after porting to python3. 
In this release, unicode strings are now handled properly.

Comment 9 Sandro Bonazzola 2020-11-23 08:12:37 UTC
(In reply to Eli Marcus from comment #8)
> Hi Sandro, please review this doc text for the errata and release notes: 
> 
> Previously, unicode strings were not handled properly by the
> rhv-log-collector-analyzer after porting to python3. 
> In this release, unicode strings are now handled properly.

Looks good to me.

Comment 13 errata-xmlrpc 2020-11-24 13:09:25 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Low: Red Hat Virtualization security, bug fix, and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2020:5179