Bug 1809877 - rhv-image-discrepancies sends dump-volume-chains with parameter that is ignored
Summary: rhv-image-discrepancies sends dump-volume-chains with parameter that is ignored
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Virtualization Manager
Classification: Red Hat
Component: rhv-log-collector-analyzer
Version: 4.3.8
Hardware: x86_64
OS: Linux
unspecified
low
Target Milestone: ovirt-4.4.0
: ---
Assignee: Germano Veit Michel
QA Contact: meital avital
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2020-03-04 05:06 UTC by Germano Veit Michel
Modified: 2020-07-23 23:57 UTC (History)
2 users (show)

Fixed In Version: rhv-log-collector-analyzer-1.0.0
Doc Type: No Doc Update
Doc Text:
Clone Of:
Environment:
Last Closed: 2020-04-16 10:36:51 UTC
oVirt Team: Integration
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

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.


Note You need to log in before you can comment on or make changes to this bug.