Bug 1425492

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/0ac985ba5e7e8c935c90f9a4fa3546387074b63f

commit 0ac985ba5e7e8c935c90f9a4fa3546387074b63f
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Feb 21 10:15:13 2017 +0100
Commit:     Satoe Imaishi <simaishi>
CommitDate: Mon Mar 13 11:29:30 2017 -0400

    Merge pull request #269 from zeari/chargeback_disable_edit_name
    
    Dont allow to edit the name of Default rate for container images
    (cherry picked from commit 5eb279685d2127ff43d4fe5b694da5669c7e73f6)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1425492

 app/controllers/chargeback_controller.rb | 3 +++
 1 file changed, 3 insertions(+)

Created attachment 1266497 [details]
Default rate for Container images is still editable

Assigning back to Dev. Default rate for Container images is still editable
Screenshot attached.

Assiging back to Dev. Default rate for Container images is still editable. See attached screenshot.

Einat please see this comment from ari[1] 

It seems from your attachment[2] that the description field isn't editable. Can you check if this really causes the report to fail?


[1] https://bugzilla.redhat.com/show_bug.cgi?id=1410012#c10
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1425492#c3

Ari, 

the UI behaviour for the "default rate" is different to the behaviour for the "default container image rate" 
In the default rate, the dropdown "Edit" button is not available to the user. 
In the default container image rate, the dropdown "Edit" is available, the user is able to edit the name of the rate, but, when pressing the "Add" button, the user is presented with an error: 
"Can not change description of 'Default Container Image Rate'"

Obviously this is very confusing to the user. 
So, indeed, the report will not be generated, but,the UI-UX is inconsistent. 
I am returning this BZ back to dev, to resolve the issue and disable the "edit" option for this rate, as is the behaviour of the "default" rate.

Here is the status:

- From functionality point of view this is fully functional as defined by PMs:
  * "Default Container Image Rate" - is editabe (only rates not the name or description)
  * "Default Container Image Rate" - can not be deleted through the UI

The above is exactly the functionality defined by PM for this rate.
If one can not add/edit the amounts in this rate, there will be no default rates for containers. 


- This bug was reopened due to a UI inconsistent behaviour:
  * the fact the one can edit a default rate (the other default rate can not be edited)
  * when editing the "Default Container Image Rate" you can not edit the description (as the charge back engine looks for this defaule rate by name)


So Moving this bug to ON_QA and please verify the above functionality.

Please open a different bu on the UI inconsistencies.

Verified.

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2017:0898