Bug 1589122

Summary: Changing number of UI Workers errors when using French or Japanese localization
Product: Red Hat CloudForms Management Engine Reporter: Ryan Spagnola <rspagnol>
Component: UI - OPSAssignee: Milan Zázrivec <mzazrivec>
Status: CLOSED CURRENTRELEASE QA Contact: Tasos Papaioannou <tpapaioa>
Severity: high Docs Contact:
Priority: high    
Version: 5.9.0CC: cpelland, hkataria, jcutter, lavenel, mpovolny, obarenbo, simaishi, smallamp, tpapaioa
Target Milestone: GAKeywords: TestOnly, ZStream
Target Release: 5.10.0   
Hardware: All   
OS: All   
Whiteboard:
Fixed In Version: 5.10.0.1 Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of:
: 1592913 (view as bug list) Environment:
Last Closed: 2019-02-11 14:07:13 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: Bug
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: CFME Core Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1592913    
Attachments:
Description Flags
error on 5.9.2.4
none
error on upstream none

Description Ryan Spagnola 2018-06-08 12:57:42 UTC 
Description of problem:
When trying to increase the number of UI workers on UI Appliance from 1 to 2 received this error [1].  The number of workers did increase to 2, however the Workers tab cannot be reloaded to make any additional changes.


[1] 

Version-Release number of selected component (if applicable):
5.9.2.4

How reproducible:
Always

Steps to Reproduce:
1. Under My Settings -> Display Settings, change Locale to Français or Japanese (日本語)and save.
2. Under Appliance Configuration, select the appliance in question, select the Workers Tab, change the Worker IU Nombre from 1 to 2 and save (Enregistrer).
3. Note the errors at the top.
4. Reload UI session and try to browse to the Workers Tab again and note the errors.


Actual results:


Expected results:
UI Workers count changes without error

Additional info:
Tested all available languages these are the only two affected.
Comment 3 Ryan Spagnola 2018-06-08 13:04:36 UTC 
Created attachment 1449127 [details]
error on 5.9.2.4
Comment 4 Ryan Spagnola 2018-06-08 13:04:59 UTC 
Created attachment 1449128 [details]
error on upstream
Comment 6 Milan Zázrivec 2018-06-18 14:36:18 UTC 
https://github.com/ManageIQ/manageiq-ui-classic/pull/4156
Comment 7 CFME Bot 2018-06-19 12:32:18 UTC 
New commit detected on ManageIQ/manageiq-ui-classic/master:

https://github.com/ManageIQ/manageiq-ui-classic/commit/271ab162a8f7d66704b558dcf93925868dff70ac
commit 271ab162a8f7d66704b558dcf93925868dff70ac
Author:     Milan Zazrivec <mzazrivec>
AuthorDate: Mon Jun 18 10:23:04 2018 -0400
Commit:     Milan Zazrivec <mzazrivec>
CommitDate: Mon Jun 18 10:23:04 2018 -0400

    Fix error on localized worker settings screen

    For worker memory threshold, we should be saving numbers, not
    translated strings (e.g. en: "500 MB" -> fr: "500 Mo")

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

 app/controllers/ops_controller/settings/common.rb | 34 +-
 1 file changed, 21 insertions(+), 13 deletions(-)
Comment 9 Tasos Papaioannou 2018-08-01 14:18:28 UTC 
Verified on 5.10.0.6.