Bug 1397248

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/63d14f89432d05a2f369309597338f30874429c6

commit 63d14f89432d05a2f369309597338f30874429c6
Author:     Dan Clarizio <dclarizi>
AuthorDate: Mon Nov 21 10:19:19 2016 -0800
Commit:     Satoe Imaishi <simaishi>
CommitDate: Fri Jan 6 12:55:32 2017 -0500

    Merge pull request #12760 from cben/new-policy-title
    
    Fix policy x_node after create/copy/delete
    (cherry picked from commit 09870b7d62281693e8395dfc812b21a1ffca6f91)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1397248

 app/controllers/miq_policy_controller/policies.rb     | 15 ++++++++++++---
 .../miq_policy_controller/policies_spec.rb            | 19 ++++++++++---------
 2 files changed, 22 insertions(+), 12 deletions(-)

New commit detected on ManageIQ/manageiq/euwe:
https://github.com/ManageIQ/manageiq/commit/5b44b15f3b0fbd0f07c77d8d5dbcb5316ba94c32

commit 5b44b15f3b0fbd0f07c77d8d5dbcb5316ba94c32
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Fri Nov 25 12:56:37 2016 +0100
Commit:     Satoe Imaishi <simaishi>
CommitDate: Fri Jan 6 14:18:47 2017 -0500

    Merge pull request #12802 from cben/no-conditions
    
    Fix entity name in Policies & Conditions lists
    (cherry picked from commit c3aa75376639edb6dd7a542630d4dad4e7610907)
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1397248

 app/views/miq_policy/_condition_list.html.haml | 5 +++--
 app/views/miq_policy/_policy_list.html.haml    | 7 +++++--
 2 files changed, 8 insertions(+), 4 deletions(-)

Beni should this be in POST?

Sorry, yes, verified now that everything fixed on master got backported.

However, there are still known instances of "Containergroup": bug 1404463.
I know how to proceed there but haven't really started, those won't make it to 5.7.1.
I arbitrarily decided to draw a line and keep that a new BZ instead of keeping growing this one.  I think it's already too hard to figure out what to verify in this BZ...
PMs, QE: if do you prefer to grow this BZ, feel free to dup bug 1404463 to this (or rather to the cfme-future bug 1359909).  Then this will have to move back to ASSIGNED.

(In reply to Beni Paskin-Cherniavsky from comment #5)
> PMs, QE: if do you prefer to grow this BZ, feel free to dup bug 1404463 to
> this (or rather to the cfme-future bug 1359909).  Then this will have to
> move back to ASSIGNED.

My preference is to keep that separate and finalize the fix for this one in 5.7.1.

Bad titles still can be found in the UI. Steps to reproduce:

1. Assign an action, event or condition to a container policy.
2. Open details screen of the action, event or condition.
3. Click on policy in "Assigned to Policies" block.

Tested on 5.7.1.0.20170124142149_8a5f4d5.

Yeah, that's why I asked in comment 5 if QE want the (arbitrary) split of bug 1404463 as a separate bug...
In theory we could figure out the multiple scenarios already fixed here, or the scenarios I go are still broken per bug 1404463, and verifying that "this part is fixed".  I totally see how that's harder and wouldn't quite make sense to call this BZ fixed as long as there are bad titles...
"Depends on" makes perfect sense.

Beni work with Dmitry to verify what was fixed here.

This BZ is closed according https://bugzilla.redhat.com/show_bug.cgi?id=1397248#c6

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://rhn.redhat.com/errata/RHSA-2017-0320.html