Noam Slomianko has uploaded a new change for review.

Change subject: engine: add comment field to Cluster
......................................................................

engine: add comment field to Cluster

    As part of the Free Text RFE

    Added to the entities flow in the system
    DB <-> DAO <-> Entity <-> Model <-> UI
    DB <-> DAO <-> Entity <-> REST

Change-Id: Id4b203b757d6225ff1986aee47e8484e2838e795
Bug-Url: https://bugzilla.redhat.com/61050
Signed-off-by: Noam Slomianko <[email protected]>
---
M backend/manager/dbscripts/create_views.sql
R backend/manager/dbscripts/upgrade/03_03_0180_add_comment_storage_pool.sql
A backend/manager/dbscripts/upgrade/03_03_0190_add_comment_vds_groups.sql
M backend/manager/dbscripts/vds_groups_sp.sql
M 
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java
M 
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
M 
backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M 
backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
M 
backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/ClusterConditionFieldAutoCompleter.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
M 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDataCenterView.java
18 files changed, 56 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/14/15414/1

diff --git a/backend/manager/dbscripts/create_views.sql 
b/backend/manager/dbscripts/create_views.sql
index 39ed78db..cd627e2 100644
--- a/backend/manager/dbscripts/create_views.sql
+++ b/backend/manager/dbscripts/create_views.sql
@@ -897,7 +897,7 @@
 CREATE OR REPLACE VIEW vds_groups_storage_domain
 
 AS
-SELECT     vds_groups.vds_group_id, vds_groups.name, vds_groups.description, 
vds_groups.cpu_name, vds_groups._create_date,
+SELECT     vds_groups.vds_group_id, vds_groups.name, vds_groups.description, 
vds_groups.comment, vds_groups.cpu_name, vds_groups._create_date,
                       vds_groups._update_date, vds_groups.selection_algorithm, 
vds_groups.high_utilization, vds_groups.low_utilization,
                       vds_groups.cpu_over_commit_duration_minutes, 
vds_groups.storage_pool_id,
                       vds_groups.max_vds_memory_over_commit, 
vds_groups.count_threads_as_cores, vds_groups.compatibility_version,
diff --git 
a/backend/manager/dbscripts/upgrade/03_03_0150_add_comment_storage_pool.sql 
b/backend/manager/dbscripts/upgrade/03_03_0180_add_comment_storage_pool.sql
similarity index 100%
rename from 
backend/manager/dbscripts/upgrade/03_03_0150_add_comment_storage_pool.sql
rename to 
backend/manager/dbscripts/upgrade/03_03_0180_add_comment_storage_pool.sql
diff --git 
a/backend/manager/dbscripts/upgrade/03_03_0190_add_comment_vds_groups.sql 
b/backend/manager/dbscripts/upgrade/03_03_0190_add_comment_vds_groups.sql
new file mode 100644
index 0000000..87b1973
--- /dev/null
+++ b/backend/manager/dbscripts/upgrade/03_03_0190_add_comment_vds_groups.sql
@@ -0,0 +1 @@
+select fn_db_add_column('vds_groups', 'comment', 'text');
diff --git a/backend/manager/dbscripts/vds_groups_sp.sql 
b/backend/manager/dbscripts/vds_groups_sp.sql
index efc08d4..82d81d4 100644
--- a/backend/manager/dbscripts/vds_groups_sp.sql
+++ b/backend/manager/dbscripts/vds_groups_sp.sql
@@ -10,6 +10,7 @@
 Create or replace FUNCTION InsertVdsGroups(
        v_vds_group_id UUID,
        v_description VARCHAR(4000),
+       v_comment text,
        v_name VARCHAR(40),
        v_cpu_name VARCHAR(255),
        v_selection_algorithm INTEGER,
@@ -28,10 +29,10 @@
 RETURNS VOID
    AS $procedure$
 BEGIN
-      INSERT INTO vds_groups(vds_group_id,description, name, cpu_name, 
selection_algorithm, high_utilization, low_utilization,
+      INSERT INTO vds_groups(vds_group_id,description, comment, name, 
cpu_name, selection_algorithm, high_utilization, low_utilization,
        cpu_over_commit_duration_minutes, storage_pool_id,  
max_vds_memory_over_commit, count_threads_as_cores, compatibility_version,
     transparent_hugepages, migrate_on_error, virt_service, gluster_service, 
tunnel_migration)
-       VALUES(v_vds_group_id,v_description, v_name, v_cpu_name, 
v_selection_algorithm, v_high_utilization, v_low_utilization,
+       VALUES(v_vds_group_id,v_description, v_comment, v_name, v_cpu_name, 
v_selection_algorithm, v_high_utilization, v_low_utilization,
        v_cpu_over_commit_duration_minutes, v_storage_pool_id,  
v_max_vds_memory_over_commit, v_count_threads_as_cores, v_compatibility_version,
     v_transparent_hugepages, v_migrate_on_error, v_virt_service, 
v_gluster_service, v_tunnel_migration);
 END; $procedure$
@@ -42,6 +43,7 @@
 
 
 Create or replace FUNCTION UpdateVdsGroup(v_description VARCHAR(4000) ,
+       v_comment text,
        v_name VARCHAR(40),
        v_vds_group_id UUID,
        v_cpu_name VARCHAR(255) ,
@@ -64,7 +66,7 @@
    AS $procedure$
 BEGIN
       UPDATE vds_groups
-      SET description = v_description,name = v_name,cpu_name = 
v_cpu_name,selection_algorithm = v_selection_algorithm,
+      SET description = v_description, comment = v_comment, name = 
v_name,cpu_name = v_cpu_name,selection_algorithm = v_selection_algorithm,
       high_utilization = v_high_utilization,
       low_utilization = v_low_utilization,cpu_over_commit_duration_minutes = 
v_cpu_over_commit_duration_minutes,
       storage_pool_id = v_storage_pool_id,_update_date = LOCALTIMESTAMP,
diff --git 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java
 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java
index b429f1b..6aa25b5 100644
--- 
a/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java
+++ 
b/backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java
@@ -15,7 +15,7 @@
 import org.ovirt.engine.core.compat.Version;
 
 @ValidVdsGroup(groups = { CreateEntity.class })
-public class VDSGroup extends IVdcQueryable implements Serializable, 
BusinessEntity<Guid>, HasStoragePool<NGuid> {
+public class VDSGroup extends IVdcQueryable implements Serializable, 
BusinessEntity<Guid>, HasStoragePool<NGuid>, Commented {
 
 
     private static final long serialVersionUID = 5659359762655478095L;
@@ -33,6 +33,8 @@
 
     @Size(max = BusinessEntitiesDefinitions.GENERAL_MAX_SIZE)
     private String description;
+
+    private String comment;
 
     @Size(max = BusinessEntitiesDefinitions.CLUSTER_CPU_NAME_SIZE)
     private String cpu_name;
@@ -115,6 +117,14 @@
         description = value;
     }
 
+    public String getComment() {
+        return comment;
+    }
+
+    public void setComment(String value) {
+        comment = value;
+    }
+
     public String getcpu_name() {
         return this.cpu_name;
     }
diff --git 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
index e20fb72..5a51181 100644
--- 
a/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
+++ 
b/backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VdsGroupDAODbFacadeImpl.java
@@ -131,6 +131,7 @@
     private MapSqlParameterSource getVdsGroupParamSource(VDSGroup group) {
         MapSqlParameterSource parameterSource = 
getCustomMapSqlParameterSource()
                 .addValue("description", group.getdescription())
+                .addValue("comment", group.getComment())
                 .addValue("name", group.getname())
                 .addValue("vds_group_id", group.getId())
                 .addValue("cpu_name", group.getcpu_name())
@@ -163,6 +164,7 @@
                 throws SQLException {
             VDSGroup entity = new VDSGroup();
             entity.setdescription(rs.getString("description"));
+            entity.setComment(rs.getString("comment"));
             entity.setname(rs.getString("name"));
             entity.setId(Guid.createGuidFromString(rs
                     .getString("vds_group_id")));
diff --git 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
index a946390..c4cad22 100644
--- 
a/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
+++ 
b/backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
@@ -1095,6 +1095,7 @@
           <xs:element name="threads_as_cores" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>
           <xs:element name="tunnel_migration" type="xs:boolean" minOccurs="0" 
maxOccurs="1"/>
           <!-- Also a rel="networks" link -->
+          <xs:element name="comment" type="xs:string" minOccurs="0" 
maxOccurs="1"/>
         </xs:sequence>
       </xs:extension>
     </xs:complexContent>
diff --git 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
index c9d06e5..e2d820d 100644
--- 
a/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
+++ 
b/backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/ClusterMapper.java
@@ -36,6 +36,9 @@
         if (model.isSetDescription()) {
             entity.setdescription(model.getDescription());
         }
+        if (model.isSetComment()) {
+            entity.setComment(model.getComment());
+        }
         if (model.isSetCpu() && model.getCpu().isSetId()) {
             entity.setcpu_name(model.getCpu().getId());
         }
@@ -78,6 +81,7 @@
         model.setId(entity.getId().toString());
         model.setName(entity.getname());
         model.setDescription(entity.getdescription());
+        model.setComment(entity.getComment());
         if (entity.getcpu_name() != null) {
             CPU cpu = new CPU();
             cpu.setId(entity.getcpu_name());
diff --git 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/ClusterConditionFieldAutoCompleter.java
 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/ClusterConditionFieldAutoCompleter.java
index cf3d180..c61d142 100644
--- 
a/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/ClusterConditionFieldAutoCompleter.java
+++ 
b/backend/manager/modules/searchbackend/src/main/java/org/ovirt/engine/core/searchbackend/ClusterConditionFieldAutoCompleter.java
@@ -6,6 +6,7 @@
         // Building the basic vervs Dict
         mVerbs.add("NAME");
         mVerbs.add("DESCRIPTION");
+        mVerbs.add("COMMENT");
         mVerbs.add("INITIALIZED");
 
         // Building the autoCompletion Dict
@@ -13,11 +14,13 @@
         // Building the types dict
         getTypeDictionary().put("NAME", String.class);
         getTypeDictionary().put("DESCRIPTION", String.class);
+        getTypeDictionary().put("COMMENT", String.class);
         getTypeDictionary().put("INITIALIZED", Boolean.class);
 
         // building the ColumnName Dict
         columnNameDict.put("NAME", "name");
         columnNameDict.put("DESCRIPTION", "description");
+        columnNameDict.put("COMMENT", "comment");
         // mColumnNameDict.put("INITIALIZED", "is_initialized");
 
         // Building the validation dict
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
index 0e24e935..14a3c3c 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
@@ -261,6 +261,7 @@
         obj.setId(instance.getId());
         obj.setname(instance.getname());
         obj.setdescription(instance.getdescription());
+        obj.setComment(instance.getComment());
         obj.setcpu_name(instance.getcpu_name());
 
         obj.setselection_algorithm(instance.getselection_algorithm());
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
index f4aa0f2..13c5c60 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterListModel.java
@@ -601,6 +601,7 @@
 
         cluster.setname((String) model.getName().getEntity());
         cluster.setdescription((String) model.getDescription().getEntity());
+        cluster.setComment((String) model.getComment().getEntity());
         cluster.setStoragePoolId(((StoragePool) 
model.getDataCenter().getSelectedItem()).getId());
         if (model.getCPU().getSelectedItem() != null)
         {
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
index 7d27d1d..2ff9442 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java
@@ -153,6 +153,18 @@
         privateDescription = value;
     }
 
+    private EntityModel privateComment;
+
+    public EntityModel getComment()
+    {
+        return privateComment;
+    }
+
+    public void setComment(EntityModel value)
+    {
+        privateComment = value;
+    }
+
     private ListModel privateDataCenter;
 
     public ListModel getDataCenter()
@@ -588,6 +600,7 @@
         setIsEdit(isEdit);
         setName(new EntityModel());
         setDescription(new EntityModel());
+        setComment(new EntityModel());
         setClusterPolicyModel(new ClusterPolicyModel());
         setAllowClusterWithVirtGlusterEnabled(true);
         AsyncDataProvider.getAllowClusterWithVirtGlusterEnabled(new 
AsyncQuery(this, new INewAsyncCallback() {
@@ -838,6 +851,7 @@
     private void postInit()
     {
         getDescription().setEntity(getEntity().getdescription());
+        getComment().setEntity(getEntity().getComment());
         setMemoryOverCommit(getEntity().getmax_vds_memory_over_commit());
 
         getCountThreadsAsCores().setEntity((boolean) 
getEntity().getCountThreadsAsCores());
diff --git 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
index 2135124..515b576 100644
--- 
a/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
+++ 
b/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterGuideModel.java
@@ -1382,6 +1382,7 @@
 
         cluster.setname((String) model.getName().getEntity());
         cluster.setdescription((String) model.getDescription().getEntity());
+        cluster.setComment((String) model.getComment().getEntity());
         cluster.setStoragePoolId(((StoragePool) 
model.getDataCenter().getSelectedItem()).getId());
         if (model.getCPU().getSelectedItem() != null)
         {
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
index c79f9b1..e43a5b0 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
@@ -948,9 +948,6 @@
     @DefaultStringValue("Description")
     String descriptionDc();
 
-    @DefaultStringValue("Comment")
-    String commentDc();
-
     // Storage DC
     @DefaultStringValue("Domain Status in Data-Center")
     String domainStatusInDcStorageDc();
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
index 5ead769..2e430b7 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java
@@ -98,6 +98,11 @@
     @WithElementId
     EntityModelTextBoxEditor descriptionEditor;
 
+    @UiField
+    @Path(value = "comment.entity")
+    @WithElementId
+    EntityModelTextBoxEditor commentEditor;
+
     @UiField(provided = true)
     @Path(value = "cPU.selectedItem")
     @WithElementId
@@ -332,6 +337,7 @@
         dataCenterEditor.setLabel(constants.clusterPopupDataCenterLabel());
         nameEditor.setLabel(constants.clusterPopupNameLabel());
         descriptionEditor.setLabel(constants.clusterPopupDescriptionLabel());
+        commentEditor.setLabel(constants.commentLabel());
         cPUEditor.setLabel(constants.clusterPopupCPULabel());
         versionEditor.setLabel(constants.clusterPopupVersionLabel());
         
enableOvirtServiceEditor.setLabel(constants.clusterEnableOvirtServiceLabel());
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
index 9e50cab..cf527e8 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml
@@ -141,6 +141,7 @@
                                                                </g:FlowPanel>  
                                                                
<e:EntityModelTextBoxEditor ui:field="nameEditor" />
                                                                
<e:EntityModelTextBoxEditor ui:field="descriptionEditor" />
+                                                               
<e:EntityModelTextBoxEditor ui:field="commentEditor" />
                                                                
<e:ListModelListBoxEditor ui:field="cPUEditor" />
                                                                
<e:ListModelListBoxEditor ui:field="versionEditor" />
                                                                
<g:VerticalPanel ui:field="servicesCheckboxPanel">
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java
index 45ca065..fd671f5 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java
@@ -20,6 +20,7 @@
 import org.ovirt.engine.ui.webadmin.widget.action.WebAdminButtonDefinition;
 import 
org.ovirt.engine.ui.webadmin.widget.action.WebAdminImageButtonDefinition;
 import 
org.ovirt.engine.ui.webadmin.widget.action.WebAdminMenuBarButtonDefinition;
+import org.ovirt.engine.ui.webadmin.widget.table.column.CommentColumn;
 
 import com.google.gwt.core.client.GWT;
 import com.google.inject.Inject;
@@ -84,6 +85,8 @@
             getTable().addColumn(cpuNameColumn, constants.cpuNameCluster(), 
"150px"); //$NON-NLS-1$
         }
 
+        getTable().addColumn(new CommentColumn<VDSGroup>(), 
constants.commentLabel());
+
         getTable().addActionButton(new 
WebAdminButtonDefinition<VDSGroup>(constants.newCluster()) {
             @Override
             protected UICommand resolveCommand() {
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDataCenterView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDataCenterView.java
index 6b9ba47..919ecbd 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDataCenterView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabDataCenterView.java
@@ -88,7 +88,7 @@
         };
         getTable().addColumn(descColumn, constants.descriptionDc(), "300px"); 
//$NON-NLS-1$
 
-        getTable().addColumn(new CommentColumn<StoragePool>(), 
constants.commentDc(), "80px"); //$NON-NLS-1$
+        getTable().addColumn(new CommentColumn<StoragePool>(), 
constants.commentLabel(), "80px"); //$NON-NLS-1$
 
         getTable().addActionButton(new 
WebAdminButtonDefinition<StoragePool>(constants.newDC()) {
             @Override


--
To view, visit http://gerrit.ovirt.org/15414
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id4b203b757d6225ff1986aee47e8484e2838e795
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Noam Slomianko <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to