Bug 1359909

Chris we are interested in having this fix for 5.6.1

It's a bit more complicated, let me take a step back.

"Container Group" didn't actually occur in Control Explorer in 5.6.0 because it didn't support pods.  But multiple strings in right sides and some toolbar buttons use non-translatable non-controllable entity names mechanically derived from class names.
It wasn't highly visible in English e.g. "Container Replicator" vs "Replicator" is not too bad, but now that 
https://bugzilla.redhat.com/show_bug.cgi?id=1346057
is landing, it will expose the more inconsistent "Container Group" vs "Pod".
In other locales, it's visibly causing untranslated terms.

Most of these happen both on darga and master,
so I believe this should remain a 5.7.0 BZ and get a Z-stream clone.

Examples (non-exhaustive):
https://cloud.githubusercontent.com/assets/273688/17083392/4b6f2ade-51a8-11e6-8292-b9860a696235.png (*)
https://cloud.githubusercontent.com/assets/6648365/16489785/4e76e76e-3ed7-11e6-9bed-89f7aae043a8.jpg
https://cloud.githubusercontent.com/assets/273688/16265415/e0787e62-3887-11e6-9d69-a8d5d695a022.png (untranslated "Host" in right side title).
https://cloud.githubusercontent.com/assets/3433754/17063911/9222692a-4fee-11e6-96b6-36a51def92d7.png (right title and button dropdown) 

(*) the first one was already made translatable on master by
https://github.com/ManageIQ/manageiq/pull/9543 
and thanks to this BZ already got backported to darga / 5.6.1:

  Darga backport details:

  commit 8b95ad166d37fdacd57d6c88b249bb1809f53cbf
  Author: Martin Povolny <mpovolny>
  Date:   Thu Jun 30 16:35:56 2016 +0200

    Merge pull request #9543 from mzazrivec/gettext_fixes_in_policy_foders_view

    Translatable policy folder names under Control -> Explorer -> Policies
    (cherry picked from commit b517fe5545c84d34c750fb48b229dbe8cf7de38f)

(*) at this moment this particular place turned into emtpy-strings bug,
fixed by https://github.com/ManageIQ/manageiq/pull/10013 (waiting for merge)

Several others can be fixed by
https://github.com/ManageIQ/manageiq/pull/9347

Submitted
https://github.com/ManageIQ/manageiq/pull/9347

It now fixes all issues I'm aware of in the this BZ.
Awaiting review.

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/40d3263a82282c26854eac34791f2678265ce83a

commit 40d3263a82282c26854eac34791f2678265ce83a
Author:     Beni Cherniavsky-Paskin <cben>
AuthorDate: Mon Aug 8 16:19:15 2016 +0300
Commit:     Beni Cherniavsky-Paskin <cben>
CommitDate: Mon Aug 8 16:19:15 2016 +0300

    Fix multiple ui_lookup(:model => nonCamelized)
    
    Hopefully fixed all occurrences in MiqPolicyController.
    Enables some non-working translations, affects even English terminology
    ("Container Image -> Image" etc).
    
    https://bugzilla.redhat.com/show_bug.cgi?id=1359909
    https://bugzilla.redhat.com/show_bug.cgi?id=1360772

 app/controllers/miq_policy_controller.rb | 32 ++++++++++++++++++++------------
 1 file changed, 20 insertions(+), 12 deletions(-)

I can still reproduce the bug in 5.7.0.0.20160906172503_12f65cb. Create any container policy and after that you can see "Containerreplicator", "Containergroup", "Containernode" or "Containerimage" in the right cell.

Created attachment 1199337 [details]
Screenshot of the bug

Confirmed, still happens on master: right after policy creation the title is "Containernode".  If you navigate to same policy later, it's "Node".
Oh, yes, I once opened an issue for this case and forgot to fix it: https://github.com/ManageIQ/manageiq/issues/10500

it is too late to fix for 5.7.0 as this is not a blocker.
Hence moving to cfme-future.

This is minor, Loic please target to the version you want it.

*** Bug 1394797 has been marked as a duplicate of this bug. ***

https://github.com/ManageIQ/manageiq/pull/12760

New commit detected on ManageIQ/manageiq/master:
https://github.com/ManageIQ/manageiq/commit/2dd47b004f620fcdfd18283b9870d58a395fba97

commit 2dd47b004f620fcdfd18283b9870d58a395fba97
Author:     Beni Cherniavsky-Paskin <cben>
AuthorDate: Sun Nov 20 15:44:31 2016 +0200
Commit:     Beni Cherniavsky-Paskin <cben>
CommitDate: Sun Nov 20 15:44:31 2016 +0200

    Fix policy x_node after create/copy/delete
    
    In Control explorer, fixes "Containernode", "Containergroup" etc
    in right side titles to be "Node", "Pod" etc (or their translations)
    immediately after creating, copying or deleting a policy.
    
    Fixes #10500, which was the last known instance of
    https://bugzilla.redhat.com/show_bug.cgi?id=1359909
    
    Modified spec to use 2-word ContainerGroup instead of Host which does
    fail without this fix and works with it.

 app/controllers/miq_policy_controller/policies.rb       |  6 +++---
 spec/controllers/miq_policy_controller/policies_spec.rb | 16 ++++++++--------
 2 files changed, 11 insertions(+), 11 deletions(-)

Steps to verify just the last fix:

- Create a new Pod (control or compliance) policy, verify right side title says "Pod".
- Configure -> Copy Policy, verify right side title says "Pod".
- Configure -> Delete Policy, verify right side title says "Pod".

Bonus points for doing this in non-english locale and verifying the
entity name (Pod in this example) is translated.

And another manifestation: https://github.com/ManageIQ/manageiq/issues/10540
fixing...

(In reply to Barak from comment #12)
> it is too late to fix for 5.7.0 as this is not a blocker.
> Hence moving to cfme-future.
> 
> This is minor, Loic please target to the version you want it.

Yes, we should fix it for Errrata.. not a blocker

Moving to POST - the 2 recent fixes merged on master (not yet euwe):

https://github.com/ManageIQ/manageiq/pull/12760
https://github.com/ManageIQ/manageiq/pull/12802

Chris, Loic: you said "not a blocker" above, the 5.7 clone (bug #1397248) is now blocker+.
Is euwe still open for 5.7.0 blockers?  Can/should I label the PRs blocker to get them backported?

(In reply to Beni Paskin-Cherniavsky from comment #22)
> Moving to POST - the 2 recent fixes merged on master (not yet euwe):
> 
> https://github.com/ManageIQ/manageiq/pull/12760
> https://github.com/ManageIQ/manageiq/pull/12802
> 
> Chris, Loic: you said "not a blocker" above, the 5.7 clone (bug #1397248) is
> now blocker+.

It's a blocker for 5.7.1 as the target release states.

Note: more instances of "Containergroup" have been discovered: bug 1404463.

I arbitrarily decided to draw a line and keep that a new BZ instead of 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.  Then this will have to move back to ASSIGNED.

Verified in 5.8.0.2.20170223184827_489bf7c. Titles of the policies are correct in the right cell.