Bug 1523765 - [RFE] When loading topology file: Display additional information about error and add "dry run" option
Summary: [RFE] When loading topology file: Display additional information about error ...
Keywords:
Status: CLOSED WONTFIX
Alias: None
Product: Red Hat Gluster Storage
Classification: Red Hat Storage
Component: heketi
Version: cns-3.6
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: ---
: ---
Assignee: Michael Adam
QA Contact: Rahul Hinduja
URL:
Whiteboard:
Depends On:
Blocks: 1724792 OCS-3.11.1-devel-triage-done
TreeView+ depends on / blocked
 
Reported: 2017-12-08 16:21 UTC by Thom Carlin
Modified: 2019-06-28 16:04 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2019-01-23 20:23:07 UTC
Embargoed:


Attachments (Terms of Use)

Description Thom Carlin 2017-12-08 16:21:36 UTC
Description of enhancement:

Enhance "heketi-cli topology load" to:
1) display additional information about cause of errors parsing configuration file
2) Add option to allow a "dry run" of the "topology load"

Version-Release number of selected component (if applicable):

cns 3.6

Additional info:

Currently, https://github.com/heketi/heketi/blob/master/client/cli/go/cmds/topology.go line 116 merely displays "Unable to parse config file"

If the "err" variable is also displayed, the end-user would have more information to diagnose the cause.

In my case, jsonlint validated the file.  However, the underlying message was:
json: cannot unmarshal object into Go struct field ConfigFileNode.devices of type string

Having the message along with the dry run option would allow rapid isolation of the cause of the error.

Comment 5 Raghavendra Talur 2019-01-23 20:23:07 UTC
We don't want to support use of topology files beyond the initial configuration. Won't work on this.


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