Bug 1809877

Summary: rhv-image-discrepancies sends dump-volume-chains with parameter that is ignored
Product: Red Hat Enterprise Virtualization Manager Reporter: Germano Veit Michel <gveitmic>
Component: rhv-log-collector-analyzerAssignee: Germano Veit Michel <gveitmic>
Status: CLOSED CURRENTRELEASE QA Contact: meital avital <mavital>
Severity: low Docs Contact:
Priority: unspecified    
Version: 4.3.8CC: michal.skrivanek, rdlugyhe
Target Milestone: ovirt-4.4.0Keywords: FieldEngineering
Target Release: ---   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: rhv-log-collector-analyzer-1.0.0 Doc Type: No Doc Update
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2020-04-16 10:36:51 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:

Description Germano Veit Michel 2020-03-04 05:06:16 UTC
Description of problem:

rhv-image-discrepancies uses this on each SPM per DC:
DUMP_VOLUME_CHAINS = "vdsm-tool dump-volume-chains {sd_id} -o json -f {sd_id}"

But json output is always to STDOUT, never to file. So the "-f" parameter is ignored as "-f" is only meaningful if the output is sqlite, see below:

# vdsm-tool dump-volume-chains -h
usage: vdsm-tool [-h] [-u] [-H HOST] [-o {text,json,sqlite}] [-p PORT]
                 [-f SQLITE_FILE]
                 sd_uuid

positional arguments:
  sd_uuid               storage domain UUID

optional arguments:
  -h, --help            show this help message and exit
  -u, --unsecured       use unsecured connection
  -H HOST, --host HOST
  -o {text,json,sqlite}, --output {text,json,sqlite}
                        select output format
  -p PORT, --port PORT
  -f SQLITE_FILE, --sqlite-file SQLITE_FILE                 <------------
                        sqlite3 db output file


Version-Release number of selected component (if applicable):
rhv-log-collector-analyzer-0.2.15-0.el7ev.noarch

This is harmless, but could have a clean-up, for example:
DUMP_VOLUME_CHAINS = "vdsm-tool dump-volume-chains {sd_id} -o json"

Comment 5 Lukas Svaty 2020-04-16 10:36:51 UTC
Closing low severity bugs, based on QE capacity, if you would like to still verify this issue please reopen.