Bug 1961391

Summary: String updates
Product: OpenShift Container Platform Reporter: ralpert
Component: Console Storage PluginAssignee: Pranshu Srivastava <prasriva>
Status: CLOSED ERRATA QA Contact: Aman Agrawal <amagrawa>
Severity: low Docs Contact:
Priority: medium    
Version: 4.8CC: afrahman, aos-bugs, ebenahar, nberry, nthomas, yanpzhan
Target Milestone: ---Flags: ralpert: needinfo-
Target Release: 4.10.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Cause: Grammar wasn't correct at some places. Also, there was an instance where the translators were unable to figure out the context. Consequence: Negative impact on readability. Fix: Corrected the grammar in some places and itemized the storage classes for translators. Result: Improved readability.
Story Points: ---
Clone Of: Environment:
Last Closed: 2022-03-12 04:35:20 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2008859, 2015766, 2015990    
Bug Blocks:    

Description ralpert 2021-05-17 19:34:23 UTC
Description of problem:
We looked through some of the files we received back from translation and noticed some errors in the strings:
* There's a string for "and." The translators didn't have context for how it's used and some chose not to translate it. It would be better to provide that context for them.
* What is a BackingStore ? - should not have a space before the question mark
* "Pool management tasks are not supported for default pool and Openshift Container Storage's  external mode." - has an extra space before "external" and "Openshift" should be "OpenShift"
* "Data will be stored as is(no dedupe, compression, encryption) on the namespace stores. Namespace buckets allow for connecting to existing data and serving from them. Best used for existing data or when other applications (and native cloud services) need to access the data from outside the OpenShift Container Storage." - There should be a space before the parenthesis
* "Caching is a policy that creates local copies of the data. It saves the copies locally to improve performance for frequently accessed data. Each cached copy has a TTL and is verified against the hub. Each non-read operation(upload, overwrite, delete) is performed on the hub" - There should be a space before the parenthesis
* "A single namespace-store, defines the read and write target of the namespace bucket" - Shouldn't the comma be a semicolon? It seems like two sentences.
* "Select list of namespace stores, defines the read targets of the namespace bucket" - Shouldn't the comma be a semicolon? It seems like two sentences
* "Select a single namespace store, defines the write target of the namespace bucket" - Shouldn't the comma be a semicolon? It seems like two sentences
* "Select one namespace-store, defines the read and write targets of the namespace bucket" - Shouldn't the comma be a semicolon? It seems like two sentences
* "The number of buckets managed on Multicloud Object Gateway, Object bucket claims and the object bucket created as a result of object bucket claims on both Multicloud Object Gateway and RGW (if deployed)." - Seems like it should be two sentences and have a period instead of a comma.

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

How reproducible:
Always

Steps to Reproduce:
1. Look at ceph-storage-plugin.json

Comment 3 Yanping Zhang 2021-05-27 08:51:31 UTC
Look at ceph-storage-plugin.json, there are many "storage class" in different strings, if they should be "storageclass"? An example string: "Selected nodes are based on the selected storage class."

Comment 4 ralpert 2021-05-27 14:28:22 UTC
We do StorageClass in admin console for what it's worth.

Comment 10 ralpert 2021-06-15 13:34:07 UTC
It looks like a couple were missed:

* "Caching is a policy that creates local copies of the data. It saves the copies locally to improve performance for frequently accessed data. Each cached copy has a TTL and is verified against the hub. Each non-read operation(upload, overwrite, delete) is performed on the hub" - There should be a space before the parenthesis
* "And"/comma separated list - The concern here is that we're not making the full context available. Not all languages use comma-separated lists (or that they may use different styles of commas). The translators have been having trouble with the out-of-context "and" also.

Comment 12 ralpert 2021-09-03 14:14:43 UTC
It looks like a comma separated list is still being used. Other teams are using Intl ListFormat to get around this: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/ListFormat

Comment 44 ralpert 2021-09-20 13:49:09 UTC
Hey Pranshu - I'm not a QE person, so I'm removing the needinfo here. I was still seeing the join function in use that generates the "and" i18n Key fwiw.

Comment 46 Pranshu Srivastava 2021-09-20 20:54:56 UTC
@ralpert Could you link me to where you're seeing that? It seems to be removed here: https://github.com/openshift/console/pull/10008/files#diff-b2817e2b54f0131c313e8820047a349632fffcd1533e01052e43101adb2bc72a (also please ignore the extraneous comment above). Thanks!

Comment 48 ralpert 2021-09-21 13:27:48 UTC
That looks perfect! The last time I looked was more than 2 weeks ago. Thanks for doing that.

Comment 54 errata-xmlrpc 2022-03-12 04:35:20 UTC
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 (Moderate: OpenShift Container Platform 4.10.3 security update), 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-2022:0056