Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1613371

Summary: Error message when attempting to change the MAC address pool is non descriptive.
Product: Red Hat Enterprise Virtualization Manager Reporter: Dana Lane <dlane>
Component: ovirt-engineAssignee: Ales Musil <amusil>
Status: CLOSED UPSTREAM QA Contact: Michael Burman <mburman>
Severity: medium Docs Contact:
Priority: medium    
Version: 4.1.9CC: amusil, danken, dlane, lsurette, mburman, Rhev-m-bugs, srevivo, tburke
Target Milestone: ovirt-4.2.7Keywords: ZStream
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: ovirt-engine-4.2.7.2 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-10-15 14:38:16 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Network RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Engine and server logs none

Description Dana Lane 2018-08-07 12:44:43 UTC
When attempting to change the mac address pool, if you select a range that is invalid, the only error that is reported is "Internal Error" with no additional description. This also puts the pool in a state where any and all attempts to make any changes, even back to the default values, results in the same "Internal Error"

How reproducible:
Edit the default mac address pool and change the first field to an 01 in both the start and end value fields. 
Select Ok

Actual results:
"Internal Error"

Comment 1 Dan Kenigsberg 2018-08-07 18:42:50 UTC
Please include your engine.log and the complete traceback.

Comment 3 Dana Lane 2018-08-07 19:16:35 UTC
Created attachment 1474081 [details]
Engine and server logs

Engine and server logs, in tar format

Comment 5 RHV bug bot 2018-09-21 11:45:50 UTC
WARN: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops

Comment 6 Michael Burman 2018-10-02 07:15:04 UTC
Ales Hi,

I'm trying to test this report on ovirt-engine-4.2.7.2-0.0.master.20180926072101.git7001334.el7.noarch
same result om master of course.
I'm followed by the reporter's step and i get - 
"Error while executing action UpdateMacPool: Error Initializing MAC Pool"
Is this is the fix? is this message more descriptive than before? i'm not sure..
If yes, i think this error message still not descriptive enough IMHO

Comment 7 Ales Musil 2018-10-02 07:26:38 UTC
Hi Michael, 

from my point of view this is a lot better than just throwing "Internal Error" or "Internal Engine Error" at user. Also the patch didn't changed any messages itself, instead it changed the way that engine propagates errors to users, if there is any. So the message you are getting is correct.

Comment 8 Michael Burman 2018-10-02 07:35:29 UTC
(In reply to Ales Musil from comment #7)
> Hi Michael, 
> 
> from my point of view this is a lot better than just throwing "Internal
> Error" or "Internal Engine Error" at user. Also the patch didn't changed any
> messages itself, instead it changed the way that engine propagates errors to
> users, if there is any. So the message you are getting is correct.

OK, i see, but still, this error message still doesn't say anything meaning full.
We need to indicate the reason for failing, no?
I'm not sure I'm OK with this fix. 

Dana, does "Error while executing action UpdateMacPool: Error Initializing MAC Pool" descriptive enough for you?

Comment 9 Dana Lane 2018-10-02 11:57:52 UTC
Yes, that's plenty descriptive.

Comment 10 Michael Burman 2018-10-03 05:26:50 UTC
OK, if you both fine with this error message than i'm fine as well.
Although personally this is not descriptive enough.

Verified upstream on - 4.2.7.2-0.0.master.20180926072101.git7001334.el7

The new error message for the described scenario will be now - 
"Error while executing action UpdateMacPool: Error Initializing MAC Pool"

Comment 11 RHV bug bot 2018-10-04 12:06:56 UTC
WARN: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops: Bug status wasn't changed from MODIFIED to ON_QA due to the following reason:

[Found non-acked flags: '{'rhevm-4.2.z': '?'}', ]

For more info please contact: rhv-devops