Bug 1127599

Summary: When running: engine-config -s "CustomDeviceProperties={type=interface;prop={other-nic-properties;queues=[0-9]*}}", provides an error only after asking to select a version
Product: [Retired] oVirt Reporter: Michael Burman <mburman>
Component: ovirt-engine-configAssignee: Eli Mesika <emesika>
Status: CLOSED NOTABUG QA Contact: Pavel Stehlik <pstehlik>
Severity: low Docs Contact:
Priority: unspecified    
Version: 3.5CC: acathrow, bugs, ecohen, gklein, iheim, oourfali, yeylon
Target Milestone: ---Keywords: Triaged
Target Release: 3.5.0   
Hardware: x86_64   
OS: Linux   
Whiteboard: infra
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-08-11 04:49:50 UTC 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:
Attachments:
Description Flags
Print screen none

Description Michael Burman 2014-08-07 08:38:57 UTC
Created attachment 924788 [details]
Print screen

Description of problem:
When running engine-config -s "CustomDeviceProperties={type=interface;prop={other-nic-properties;queues=[0-9]*}}". An error appears only after providing a vesrion. The error should be displayed before that. right away, if the engine-config command is wrong.

Version-Release number of selected component (if applicable):
ovirt-engine-3.5.0-0.0.master.20140804172041.git23b558e.el6.noarch

How reproducible:
always

Steps to Reproduce:
1.Run command - engine-config -s "CustomDeviceProperties={type=interface;prop={other-nic-properties;queues=[0-9]*}}"
2. 
3.

Actual results:
After running the command, you asked to provide a version, and in this point you thing that the command is right. but after providing the version, you got an error.

Expected results:
If the engine-config -s command is wrong, the error should be right away. 

Additional info:

Comment 1 Eli Mesika 2014-08-10 20:36:19 UTC
I don't think this is a bug.
The reason you are asked for the version is because you didn't gave it in the command line using the --cver option.
With --cver, you got the error immediately

---snip---
engine-config -s "CustomDeviceProperties={type=interface;prop={other-nic-properties;queues=[0-9]*}}" --cver=3.1
Cannot set value {type=interface;prop={other-nic-properties;queues=[0-9]*}} to key CustomDeviceProperties. Device custom properties are not supported in version 3.1
---snip---

You can not do the verification before getting the version since format can be changed between versions so you will not be able to know which format to match unless you got the version

I think this should be closed as NOTABUG 

Oved?

Comment 2 Oved Ourfali 2014-08-11 04:49:50 UTC
I think you're right. 
Closing as notabug.