Bug 868889

Summary: rhevm-config help: Need to make cver optional and note that if version is not "general" it must be given.
Product: Red Hat Enterprise Virtualization Manager Reporter: Ilanit Stein <istein>
Component: ovirt-engine-configAssignee: Ravi Nori <rnori>
Status: CLOSED CURRENTRELEASE QA Contact: Pavel Stehlik <pstehlik>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 3.1.0CC: bazulay, dyasny, iheim, oramraz, Rhev-m-bugs, sgrinber, ykaul, yzaslavs
Target Milestone: ---   
Target Release: 3.2.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: infra
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Infra RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 915537    

Description Ilanit Stein 2012-10-22 11:33:53 UTC
Description of problem:

On rhevm-config (SI20) -help "cver" option description, "The cver version is mandatory for this action.", need to be changed since it is true only for some of the configuration keys. We need to make cver optional and note that if version is not "general" it must be given.

Comment 1 Yair Zaslavsky 2012-10-22 12:17:38 UTC
Altough it doesn't sound too risky/too complex to solve, not sure if in the current timeframe of 3.1, we should fix this. This is not a blocker bug.

Comment 2 Itamar Heim 2012-10-24 05:14:52 UTC
shouldn't user specify 'general' as cver?

Comment 3 Ilanit Stein 2012-10-24 06:39:12 UTC
'general' is the default cver. 
In case --cver option is not used, cver general is chosen. 
This bug asks to be accurate in the help cver option definition. 
cver is mandatory only for the non-general configuration key.

Comment 4 Ravi Nori 2012-10-24 17:34:40 UTC
Link: http://gerrit.ovirt.org/#/c/8787/

Commit Hash: d660b7888dec1fb1867acf6c4df3fa55d1d28cdc

Comment 5 Oded Ramraz 2013-01-24 16:04:15 UTC
From the help : "set the value of the given key for the given version. The cver version is required for this action only when the version is not 'general'."

Verified sf4

Comment 6 Itamar Heim 2013-06-11 08:40:31 UTC
3.2 has been released

Comment 7 Itamar Heim 2013-06-11 08:40:31 UTC
3.2 has been released

Comment 8 Itamar Heim 2013-06-11 08:40:46 UTC
3.2 has been released

Comment 9 Itamar Heim 2013-06-11 08:48:09 UTC
3.2 has been released