Bug 1478139
| Summary: | Command "ovirt-log-collector --quiet" is verbose | ||
|---|---|---|---|
| Product: | [oVirt] ovirt-log-collector | Reporter: | David Necpal <dnecpal> |
| Component: | Core | Assignee: | Douglas Schilling Landgraf <dougsland> |
| Status: | CLOSED CURRENTRELEASE | QA Contact: | David Necpal <dnecpal> |
| Severity: | medium | Docs Contact: | |
| Priority: | unspecified | ||
| Version: | 4.1.3 | CC: | bugs, dougsland, lsvaty, lveyde, sbonazzo, ylavi |
| Target Milestone: | ovirt-4.2.2 | Keywords: | Reopened |
| Target Release: | 4.2.4 | Flags: | rule-engine:
ovirt-4.2+
ylavi: exception+ |
| Hardware: | All | ||
| OS: | All | ||
| Whiteboard: | |||
| Fixed In Version: | ovirt-log-collector-4.2.4-1 | Doc Type: | If docs needed, set a value |
| Doc Text: | Story Points: | --- | |
| Clone Of: | Environment: | ||
| Last Closed: | 2018-03-29 11:08:37 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: | |||
| Bug Blocks: | 1456888 | ||
|
Description
David Necpal
2017-08-03 17:01:33 UTC
Closing as not a bug. The warning exposed is meant to avoid to generate huge reports which takes hours to complete. (In reply to Sandro Bonazzola from comment #1) > Closing as not a bug. > The warning exposed is meant to avoid to generate huge reports which takes > hours to complete. Which part is notifying admin about fact it can be a huge report lasting hours? """ This command will collect system configuration and diagnostic information from this system. The generated archive may contain data considered sensitive and its content should be reviewed by the originating organization before being passed to any third party. No changes will be made to system configuration. """ Unfortunately, I do see an issue here, if we decided we want some kind of verbosity during --quite parameter: 1. Agree with reporter, all info should be hidden except asking for password 2. Not sure where this verbose mode is written to, but at least it should be written to stderr 3. If we are indeed writing something out to stdout we should consider at least adding name and path to the archive created, that seems like the most relevant info. Can you reconsider? Reopening as per comment #2 (In reply to Douglas Schilling Landgraf from comment #4) > PR: https://github.com/oVirt/ovirt-log-collector/pull/2 Please push to gerrit.ovirt.org; github is backup only. Verified on version ovirt-log-collector-4.2.4-1.el7ev.noarch [root@example]# ovirt-log-collector --quiet Please provide the REST API password for the admin@internal oVirt Engine user (CTRL+D to skip): [root@example]# This bugzilla is included in oVirt 4.2.2 release, published on March 28th 2018. Since the problem described in this bug report should be resolved in oVirt 4.2.2 release, it has been closed with a resolution of CURRENT RELEASE. If the solution does not work for you, please open a new bug report. |