Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1149468

Summary: Remove Cluster host has text issues and inconsistent usage of trailing period
Product: Red Hat Enterprise Virtualization Manager Reporter: Gilad Lazarovich <glazarov>
Component: ovirt-engineAssignee: jniederm
Status: CLOSED CURRENTRELEASE QA Contact: Karolína Hajná <khajna>
Severity: low Docs Contact:
Priority: low    
Version: 3.5.0CC: acanan, gklein, jniederm, lsurette, michal.skrivanek, pstehlik, rbalakri, Rhev-m-bugs, sherold, srevivo, ykaul
Target Milestone: ovirt-3.6.0-rc3   
Target Release: 3.6.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2016-04-20 01:11:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: Virt RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
Remove Cluster error text none

Description Gilad Lazarovich 2014-10-05 11:31:15 UTC
Created attachment 944014 [details]
Remove Cluster error text

Description of problem:
Remove Cluster host has text issues and inconsistent usage of trailing period

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

How reproducible:
100%

Steps to Reproduce:
1. Attempt to remove a Cluster that has connected hosts, VMs and Templates
2. Note the text on the error dialog

Actual results:
There are several text and trailing period inconsistencies as follows:
- Host Cluster contains one or more Hosts. -> Cluster contains one or more Hosts (with no period)
- One or more Template(s) are still associated with it -> One or more Templates are still associated with the Cluster

Expected results:
Provided as part of the Actual results

Comment 1 Karolína Hajná 2015-09-29 08:36:53 UTC
The "Host Cluster contains one or more Hosts." string was supposed to be changed to "Cluster contains one or more Hosts".
Actual string in 3.6.0-15 is "Host Cluster contains one or more Hosts" (only the period was removed).

Comment 2 jniederm 2015-09-29 14:21:22 UTC
I suggest to keep phrase "Host Cluster" because it is widespread throughout the app:
 * Cannot attach VM to VM-Pool. VM resides on different Host Cluster than VM-Pool.
 * Invalid Host Cluster id.
 * Cannot attach VM to VM-Pool. VM resides on different Host Cluster than VM-Pool.
 * ...

It the global rename is required, please file a separate bug.

Comment 3 Karolína Hajná 2015-09-30 10:00:01 UTC
(In reply to jniederm from comment #2)
> I suggest to keep phrase "Host Cluster" because it is widespread throughout
> the app:
>  * Cannot attach VM to VM-Pool. VM resides on different Host Cluster than
> VM-Pool.
>  * Invalid Host Cluster id.
>  * Cannot attach VM to VM-Pool. VM resides on different Host Cluster than
> VM-Pool.
>  * ...
> 
> It the global rename is required, please file a separate bug.

In that case "One or more VMs are still assigned to the Cluster" and 
"One or more Template(s) are still associated with the Cluster" strings should also contain "Host Cluster" instead of "Cluster". Or is there any reason it's just "Cluster" in these two cases?

Comment 4 jniederm 2015-10-05 12:33:45 UTC
"Host Cluster contains one or more Hosts" changed to "Cluster contains one or more Hosts" in order to make the dialog consistent.

Comment 5 Karolína Hajná 2015-10-15 09:55:24 UTC
Verified on 3.6.0-16 (rhevm-3.6.0.1-0.1.el6.noarch)