Bug 1382032 - The Calamari API sometimes incorrectly prints standard output as an error
Summary: The Calamari API sometimes incorrectly prints standard output as an error
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Red Hat Ceph Storage
Classification: Red Hat Storage
Component: Calamari
Version: 2.1
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
: 2.1
Assignee: Andrew Schoen
QA Contact: ceph-qe-bugs
URL:
Whiteboard:
: 1287904 1287905 (view as bug list)
Depends On: 1287904
Blocks: 1287905 1291304 1322504
TreeView+ depends on / blocked
 
Reported: 2016-10-05 14:42 UTC by Andrew Schoen
Modified: 2022-02-21 18:03 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1287904
Environment:
Last Closed: 2016-10-05 20:39:10 UTC
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Ceph Project Bug Tracker 6906 0 None None None 2016-10-05 14:42:51 UTC

Comment 2 Andrew Schoen 2016-10-05 14:44:47 UTC 
*** Bug 1287904 has been marked as a duplicate of this bug. ***
Comment 3 Andrew Schoen 2016-10-05 14:46:39 UTC 
*** Bug 1287905 has been marked as a duplicate of this bug. ***
Comment 4 Andrew Schoen 2016-10-05 20:39:10 UTC 
In looking into this I've found that calamari is doing the correct thing, it's that ceph is writing the output of that command to stderr instead of stdout.

Here's a link to the code in calamari that creates that json output from the description, https://github.com/ceph/calamari/blob/1.4/calamari-common/calamari_common/remote/mon_remote.py#L358-L370

I'm going to close this as NOTABUG because I don't think we can expect uniformity from stderr/stdout and stderr does not always imply an error condition, which is why we have that 'status' field.

I will open an upstream ceph bug though to see if we can get the CLI to standardize on using stdout for CLI output, unless there is an error then stderr will be used.
Note You need to log in before you can comment on or make changes to this bug.