Bug 1552260

Summary: Default Container Image Rate can be deleted
Product: Red Hat CloudForms Management Engine Reporter: Satoe Imaishi <simaishi>
Component: UI - OPSAssignee: Hilda Stastna <hstastna>
Status: CLOSED WONTFIX QA Contact: Tasos Papaioannou <tpapaioa>
Severity: medium Docs Contact:
Priority: medium    
Version: 5.8.0CC: hkataria, hstastna, jhardy, lavenel, mpovolny, obarenbo, simaishi, smallamp, tpapaioa
Target Milestone: GAKeywords: ZStream
Target Release: cfme-future   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: ui:chargeback:flash_msg
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1486658 Environment:
Last Closed: 2018-10-17 17:52:09 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On: 1486658    
Bug Blocks:    

Comment 2 CFME Bot 2018-03-06 21:26:05 UTC
New commits detected on ManageIQ/manageiq/gaprindashvili:

https://github.com/ManageIQ/manageiq/commit/b4ecec83c0bde5903261aef088ea594310522308
commit b4ecec83c0bde5903261aef088ea594310522308
Author:     Gregg Tanzillo <gtanzill>
AuthorDate: Wed Jan 24 09:03:33 2018 -0500
Commit:     Gregg Tanzillo <gtanzill>
CommitDate: Wed Jan 24 09:03:33 2018 -0500

    Merge pull request #16792 from hstastna/Default_Container_Image_Rate_delete_description

    Add condition to fix deletion of Default Container Image Rate
    (cherry picked from commit 164cd082924e89a20acd810cdb9b3f383a2f01a9)

    https://bugzilla.redhat.com/show_bug.cgi?id=1552260

 app/models/chargeback_rate.rb | 2 +-
 spec/models/chargeback_rate_spec.rb | 8 +
 2 files changed, 9 insertions(+), 1 deletion(-)

Comment 3 CFME Bot 2018-03-08 12:47:13 UTC
New commit detected on ManageIQ/manageiq-ui-classic/gaprindashvili:

https://github.com/ManageIQ/manageiq-ui-classic/commit/67174afb2dd94d7443fff3917681998e056b0f4e
commit 67174afb2dd94d7443fff3917681998e056b0f4e
Author:     Harpreet Kataria <hkataria>
AuthorDate: Fri Feb  9 11:17:10 2018 -0500
Commit:     Harpreet Kataria <hkataria>
CommitDate: Fri Feb  9 11:17:10 2018 -0500

    Merge pull request #3215 from hstastna/Default_Container_Image_Rate_delete

    Fix deletion of Default Container Image Rate
    (cherry picked from commit 8806d612ebba610d3319575fd42f4866fcb37808)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1552260

 app/controllers/chargeback_controller.rb | 21 +-
 spec/controllers/chargeback_controller_spec.rb | 53 +
 2 files changed, 62 insertions(+), 12 deletions(-)

Comment 4 Tasos Papaioannou 2018-03-21 19:28:17 UTC
Tested on 5.9.2.0.

The Default Container Image Rate can no longer be deleted, but it can still be edited. To be consistent with the Default Compute and Storage rates, editing should also be disabled.

Comment 8 Hilda Stastna 2018-03-22 17:07:09 UTC
The core part for fixing the bug: https://github.com/ManageIQ/manageiq/pull/17188

Comment 11 CFME Bot 2018-04-06 13:00:45 UTC
New commit detected on ManageIQ/manageiq/master:

https://github.com/ManageIQ/manageiq/commit/f2349afab08e41ecfdd9ccbed01d85c0b073ed88
commit f2349afab08e41ecfdd9ccbed01d85c0b073ed88
Author:     Hilda Stastna <hstastna>
AuthorDate: Thu Mar 22 12:53:33 2018 -0400
Commit:     Hilda Stastna <hstastna>
CommitDate: Thu Mar 22 12:53:33 2018 -0400

    Refactor ensure_nondefault method for chargeback rate

    https://bugzilla.redhat.com/show_bug.cgi?id=1552260

    Refactor ensure_nondefault method so that a
    new default? method is created and improved.

 app/models/chargeback_rate.rb | 6 +-
 1 file changed, 5 insertions(+), 1 deletion(-)

Comment 12 CFME Bot 2018-04-09 11:26:29 UTC
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/267a363436550780fc3d592b64c4a0f9e421f7ce
commit 267a363436550780fc3d592b64c4a0f9e421f7ce
Author:     Hilda Stastna <hstastna>
AuthorDate: Thu Mar 22 13:08:26 2018 -0400
Commit:     Hilda Stastna <hstastna>
CommitDate: Thu Mar 22 13:08:26 2018 -0400

    Refactor disabled? method for chargeback rate remove button

    https://bugzilla.redhat.com/show_bug.cgi?id=1552260

    Refactor the method according to the changes in core repo.

 app/helpers/application_helper/button/chargeback_rate_remove.rb | 8 +-
 1 file changed, 1 insertion(+), 7 deletions(-)